[SPARK-22092] Reallocation in OffHeapColumnVector.reserveInternal corrupts array data

What changes were proposed in this pull request?
OffHeapColumnVector.reserveInternal() will only copy already inserted values during reallocation if data != null. In vectors containing arrays this is incorrect, since there field data is not used at all. We need to check lengthData or offsetData instead.
How was this patch tested?
Adds a new test to ColumnVectorSuite that reproduces the error.

@hvanhovell

Test build #82038 has started for PR 19308 at commit 8ce2a68.

Test build #82038 has finished for PR 19308 at commit 8ce2a68.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82038/
Test PASSed.

Merged build finished. Test PASSed.

Test build #82077 has started for PR 19308 at commit 0ea5793.

@hvanhovell How about this?

Test build #82077 has finished for PR 19308 at commit 0ea5793.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/82077/
Test PASSed.

LGTM - merging to master. Thanks!

@ala Can you backport this one to 2.2?


Fixup Regression in Return Statement Parsing

When we introduced the at(any:) primitive, this condition got inverted so we were parsing
{
  return x
}

as
{
  return
  x
}

@swift-ci test


Client is in control of the stored file name

The client can use the FSName option to control the name of the file where the uploaded data is saved by the server. This can be used to... well, evil stuff 😄
Meteor Files Version: 1.7.13

Workaround Use an onBeforeUpload hook to check the path property
of the uploaded file.
Collection.onBeforeUpload = function (fileData) {
    if (/\.\.\//.test(fileData.path)) return "Thou shalt not pass through!" ;
    return true;
}

The client code generating the property is around line 1700:
        ...
        @FSName       = if @collection.namingFunction then @collection.namingFunction(@fileData) else @fileId
        ...

and the server side uses it without checking around line 1065 in FilesCollection::_prepareUpload
    ...
    result.path      = "#{@storagePath(result)}#{nodePath.sep}#{opts.FSName}#{extensionWithDot}"
    ...

namingFunction looks like a feature that should be server-side only. The client can already "set a file name" using the fileName property at insert time and should not be granted access to how the file is actually stored as.
BTW, the calls to the naming function made by the server do not supply any parameter. That's odd and doesn't look very useful.

Hello @diegonc ,
Thank you for contribution, could you take a look on https://github.com/VeliovGroup/Meteor-Files/commit/46be3fdc24546ca98c465acc1c17dbb4b2c55244
What do you think?

Yup, looks good.

@diegonc published as v1.7.14, thank you for contribution. Let me know how it work for you.
Please, support this project by:

Star on GitHub
Star on Atmosphere


Fixed spacing issue between messages on iOS, adjusted "Send" button i…

Adjusted the image size of the "Send" button for consistent UI across platforms
Added "fetch" to UIBackgroundModes in the iOS sample's Info.plist for background task support (Sample Project)
Fixed the spacing between messages in CollectionView on iOS, but new issue.

I found a few more issues.
-There is an issue with automatic scrolling to the bottom of the screen. It works when sending a message, but when replying to Echo, the second scroll is either canceled or triggered before the item is added.
-There is an issue with the height of the CollectionView after adding padding when the first message is loaded. (This is noticeable in the difference in screenshots where the oldest message is cut off after changes). The problem disappears, for example, when the keyboard is pulled up or when you tap on the editor.
If you like it, don't merge it for now. I'll take a look at these issues tomorrow or the day after.
Before

After

At first, let me thank you for such a quick response and your will to contribute to the project:) I feel super excited about it!
Regarding issues you mentioned:

Collection view: I see that not only spacing but also vertical text alignment looks different than on Android - it should be centered, but on iOS it seems to be aligned to the top.
Automatic screen scrolling: there is already a workaround made for it (see: Chat.xaml.cs -> ScrollDownChatMessages). The original behavior of collection view scrolling after adding new item (message) was cutting either the last and the first messages in the collection view. Adding workaround fixed issue with visibility of the most recent messages (bottom of the collection), however I couldn't figure out how to fix hiding part of top of the collection when keyboard is shown. I think that  at first you should check how scrolling works without my workaround - maybe it works differently than on Android and no workaround is needed on iOS. The second thing to check is to increase time in Task.Delay in the ScrollDownChatMessages (to be honest I have no idea why it doesn't work without that delay at all).

https://github.com/ArturWyszomirski/Plugin.Maui.Chat/issues/17.


Document the nested pages

I actually lost after #385 PR to figure out how nested pages works.
After I figured out how it works it makes sense, but I don't think it does for newcomers.
I think if it is added to the "Features" section in the README and a simple topic about it explaining how to create it and where it will be and that it will not be found in the posts, that would be good.
@welpo What do you think?

Nested pages aren't a tabi-specific feature. All themes should be able to render them (unless they had a bug like tabi had hehe).
I prefer to keep the tabi docs tabi-centered. A general explanation of nested pages belongs to the Zola docs, in my opinion.
That said, I couldn't find much official information from the Zola docs. Either the (Zola) docs need to be improved, or it's assumed it's obvious.
What do you think?

I was thinking about document it, but it's look likes it mentioned here but not directly. I think it just takes focus to know that it works.

Agreed. It was confusing for me too.


Device Needs Attention

I just wrote T120flash.img to the USB drive as directed.  The printer now starts back up, but has a Device Needs Attention Status.  The file I have is from latest commit. The only log I see in the printer is Event 69666 with a description of Firmware Error.

Hi Jarred. If the printer is starting, then it could be that this is a different error. from this link it seems a solutuion would be to update the printer firmware, https://h30434.www3.hp.com/t5/Printer-Setup-Software-Drivers/Error-69666-What-that-means-and-how-to-solve-it/td-p/6392746,
you could try that, as you would always be able to go back to the initial firmware image. However I have found firmware updates break the printer again...

I dont think that github would have a problem with the file, but this is the sha256sum anyway:
$ sha256sum t120flash.img
5c10ed16417c4fc4fce029e2da4d3a64bfa6e8375866475aeccaf73e72d1654c  t120flash.img


Fixes JedisCluster.psubscribe() to call the correct psubscribe method.

I'll go ahead and merge as it's a simple change

Applied to 2.8 and 2.9 respectively!.


Nuxt fatal error

Describe the bug
npm run build fails
To Reproduce
Steps to reproduce the behavior:

Download the release

wget https://github.com/liyasthomas/postwoman/archive/v1.9.7.tar.gz

Unzip

tar -xvf v1.9.7.tar.gz

cd into the created directory and run:

npm i

Run build:

npm run build

After running these commands, the installation was supposed to complete successfully but it failed with the following error:

Here is the full error log:
WARNING in ./node_modules/yargs-parser/index.js 586:21-48
Critical dependency: the request of a dependency is an expression
 @ ./assets/js/curlparser.js
 @ ./node_modules/babel-loader/lib??ref--2-0!./node_modules/vue-loader/lib??vue-loader-options!./pages/index.vue?vue&type=script&lang=js&
 @ ./pages/index.vue?vue&type=script&lang=js&
 @ ./pages/index.vue
 @ ./.nuxt/router.js
 @ ./.nuxt/index.js
 @ ./.nuxt/client.js
 @ multi ./.nuxt/client.js

ERROR in ./components/layout/footer.vue?vue&type=script&lang=js& (./node_modules/babel-loader/lib??ref--2-0!./node_modules/vue-loader/lib??vue-loader-options!./components/layout/footer.vue?vue&type=script&lang=js&)
Module not found: Error: Can't resolve '../../.postwoman/version.json' in '/home/angad/Repos/postwoman-1.9.7/components/layout'
 @ ./components/layout/footer.vue?vue&type=script&lang=js& (./node_modules/babel-loader/lib??ref--2-0!./node_modules/vue-loader/lib??vue-loader-options!./components/layout/footer.vue?vue&type=script&lang=js&) 77:0-57 86:19-34
 @ ./components/layout/footer.vue?vue&type=script&lang=js&
 @ ./components/layout/footer.vue
 @ ./node_modules/babel-loader/lib??ref--2-0!./node_modules/vue-loader/lib??vue-loader-options!./layouts/default.vue?vue&type=script&lang=js&
 @ ./layouts/default.vue?vue&type=script&lang=js&
 @ ./layouts/default.vue
 @ ./.nuxt/App.js
 @ ./.nuxt/index.js
 @ ./.nuxt/client.js
 @ multi ./.nuxt/client.js

 FATAL  Nuxt build error

  at WebpackBundler.webpackCompile (node_modules/@nuxt/webpack/dist/webpack.js:5351:21)
  at processTicksAndRejections (internal/process/task_queues.js:97:5)
  at async WebpackBundler.build (node_modules/@nuxt/webpack/dist/webpack.js:5300:5)
  at async Builder.build (node_modules/@nuxt/builder/dist/builder.js:5602:5)
  at async Generator.initiate (node_modules/@nuxt/generator/dist/generator.js:70:7)
  at async Generator.generate (node_modules/@nuxt/generator/dist/generator.js:40:5)
  at async Object.run (node_modules/@nuxt/cli/dist/cli-build.js:100:7)
  at async NuxtCommand.run (node_modules/@nuxt/cli/dist/cli-index.js:2759:7)

   ╭─────────────────────────────╮
   │                             │
   │   ✖ Nuxt Fatal Error        │
   │                             │
   │   Error: Nuxt build error   │
   │                             │
   ╰─────────────────────────────╯

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: `nuxt build`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Pushing a permanent fix now. Kindly download latest zip, then unzip and run:
npm i
npm run build

@L04DB4L4NC3R, issue have been identified. Pushing a permanent fix now. Kindly download latest zip, then unzip and run:
npm i
npm run build

It works. Thanks for the quick reply.


[BUG] Typescript types are not detected in nodeNext resolution environment

Description
When using nodeNext module resolution in typescript, react-chakra-select types are not detected. Because when "exports" is present, "types" at root level supposed to be ignored
Solution is to add

"types": "dist/types/index.d.ts"

to package.json "exports" object after "default"
chakra-react-select Version
4.2.2
Link to Reproduction
No response
TypeScript?

[X] Yes I use TypeScript

Steps to reproduce

Create tsconfig with "moduleResolution": "NodeNext",
Include "chaka-react-select"
Try to typecheck

Operating System

[X] macOS
[ ] Windows
[ ] Linux
[ ] iOS/iPadOS
[ ] Android

Additional Information
No response

So just to get this clear, you just want me to change the exports key in the package.json to the following?
"exports": {
  ".": {
    "require": "./dist/cjs/index.js",
    "default": "./dist/esm/index.js",
    "types": "dist/types/index.d.ts"
  }
},

Yes sir. It's a safe change, as everything that need the JS will stop at default. But it will allow typescript to pick up the types. There's a little bit of uncertainty if it should be index.d.ts or index.js. Both works. However it seems  that index.js is more backward-compatible with older versions, but don't quote me on that. I'd add `index.d.ts.
"exports": {
  ".": {
    "require": "./dist/cjs/index.js",
    "default": "./dist/esm/index.js",
    "types": "./dist/types/index.d.ts"
  }
},

Little correction - ./dist

ok cool, yes I can add that!  Thanks for bringing this to my attention

Should be published now in v4.2.3, let me know if you're still running into any issues!

Seems to be smooth. many thanks for quick turnaround

Hello, this update broke my Next.js project (no Typescript). After upgrading to 4.2.3 I am getting
Module not found: Default condition should be last one.
> 1 | import { Select } from "chakra-react-select";

If I remove from node_modules (package.json) the following line, then everything works fine
"types": "./dist/types/index.d.ts"
I have tried to remove completely node_modules and yarn.lock and install again and the problem remains...
Thanks in advance

@itsam I am facing the same issue.

Oh my god, I'm really sorry it was my fault. I thought it's a good idea to do that. Which version of typescript is that? What's throwing the error?
@csandman Sir I'm super sorry, looks like we need to rearrange it to make non-last. My bad.

Oh my god, I'm really sorry it was my fault. I thought it's a good idea to do that way. I did not get anything of that sort on my build. I thought it's a safe bet.
@itsam Which version of typescript is that? What's throwing the error? Is it yarn?
@csandman Sir I'm super sorry, looks like we need to rearrange it to make non-last. My bad.

Well, actually this is a general comment. I am not using TS in my project at all. Everything worked normally until I 've upgraded to 4.2.3. Quick search led me to this issue, and I tried to remove the line I mentioned above and everything worked again. So currently, version 4.2.3 is broken for non-TS next.js projects (cannot tell about TS)
Thank you for your great work and time :)

CRA throws Module not found: Error: Default condition should be last one

@Nithur-M Downgrade to previous version. Sorry again.

Thanks for the quick feedback @itsam @Nithur-M @luastoned I pushed a fix for this in v4.2.4.

@Inviz
Oh my god, I'm really sorry it was my fault. I thought it's a good idea to do that way. I did not get anything of that sort on my build. I thought it's a safe bet.

Honestly, it's my fault for not doing much testing, didn't expect this to completely break the package for others.

CRA throws Module not found: Error: Default condition should be last one
Here is the fix:
"exports": {
  ".": {
    "import": "./dist/esm/index.js",
    "require": "./dist/cjs/index.js",
    "types": "./dist/types/index.d.ts",
    "default": "./dist/esm/index.js"
  }
},

I explicitly added import and default should be last either way.

Thanks for the fix recommendation @luastoned, helped me get a new version out quick!

If anyone is still having issues, let me know ASAP!

I confirm replacing exports with
"exports": {
  ".": {
    "import": "./dist/esm/index.js",
    "require": "./dist/cjs/index.js",
    "types": "./dist/types/index.d.ts",
    "default": "./dist/esm/index.js"
  }
},

fixes the bug :)
So, I would say it's OK to release the new npm (4.2.4)
Thank you everyone for the prompt reply.
Good job !!!

Oh, it's already online. Cross checked and works fine.

Oh, it's already online. Cross checked and works fine.

Haha yep, I tested it myself in a test app first and it seemed to do the trick so I published quick.  Didn't want to leave a bug which brakes the whole package around for longer than I had to 😅
Thanks for confirming for me though!

Also I finally added the CodeSandbox CI GitHub app to more easily check if a PR works.  Hopefully that will help avoid silly mistakes like this in the future!
(you can see an example of it here: https://github.com/csandman/chakra-react-select/pull/193)


let user to add a new entry

Hello,
Is it possible with this plugin to let users add new value when they dont find result from api? I mean they just write and press enter to add what they have typed to box

You can try set noResultClean option to false
$('#selectpage').selectPage({
  ...
  noResultClean: false
})


IPython does not work with brewed or virtualenv Python(s) in MacOSX

I am using a brewed 2.7 Python in Yosemite. I installed the new Spyder 2.3.3 version from the github dmg. I cannot use IPython in Spyder when I point to the executable of the brewed Python in Preferences>Console>Advanced Settings>Python executable. The following is the path I am using:
/usr/local/bin/python
Pointing to an activated virtualenv Python executable does not work either. The IPython console does not start. In both cases I receive the following error message instead:
'An error ocurred while starting the kernel'
Could you please help me out to set Spyder to work in my Mac?

The solution is?

If you're using Spyder 3.0, you need to install ipykernel in your virtualenv.


Add support for adding an unformatted raw parameter

This allows passing i.e. configuration data directly from network or flash without needing to parse or transform it, such as a pin number which is more a 'series of digits' than an integer.

I don't have problems with this. Would you like me to release it?
Also, that new unused feature detection comes in handy again!

Done, released as 0.5.5
Apparently this crate has a yearly release schedule now :P
Third time in a row there's one release in september lol
Interesting! What part of Embassy is gonna use it? The nRF91 net?

@diondokter Originally for this PR https://github.com/embassy-rs/embassy/pull/3376  - turns out I misread the documentation, the pin is quoted after all according to https://docs.nordicsemi.com/bundle/ref_at_commands/page/REF/at_commands/security/cpin_set.html  :facepalm:  I think I looked at some unofficial code that didn't quote it or something.

Oh hahaha, well the function is a fine addition to the crate!


[KOGITO-129] - Docs for contributions guidelines

See:
https://issues.jboss.org/browse/KOGITO-129
This PR adds some documentation to the repository to help others contributing with the Operator. This is a first, raw and living document that we intent to enrich as soon as we mature the project.

@lazarotti @errantepiphany @bmozaffa @tchughesiv @mswiderski @sutaakar @mramendi please could you guys take a look? I'd be glad to hear from you about this doc.
Please keep in mind that this is a very first, raw version of this document. I plan to update/add new content to this doc with time, during the component mature phase.

Thanks @camilamacedo86 and @mramendi  for your wonderful review. Could you please take a final look before I merge this?
Thanks! Love for you all.

@ricardozanini I know I'm late to the party on this, but it looks good to me as well. Great job; I'm sure it will be helpful for people.

Unfortunately I could get to another look at this PR only now, after it was merged. I found a few grammatical nits.  But then I'd need a JIRA?

Hi @mramendi please create a PR for your fix. :)
I believe that are only small fixes, won't need a JIRA.

@ricardozanini I know I'm late to the party on this, but it looks good to me as well. Great job; I'm sure it will be helpful for people.

Thanks man! I'm very happy for having you guys supporting me on this. :)

Unfortunately I could get to another look at this PR only now, after it was merged. I found a few grammatical nits.  But then I'd need a JIRA?

Hi @mramendi please create a PR for your fix. :)
I believe that are only small fixes, won't need a JIRA.

Did so. https://github.com/kiegroup/kogito-cloud-operator/pull/30


chore: remove one boxing

Kinda interesting that this worked before. Apparently, Box<Fn*> implements Fn*:
https://doc.rust-lang.org/std/boxed/struct.Box.html#impl-FnOnce<Args>

Yay thanks! Sorry I missed that one.


[opentitanlib] set DFT straps before POR

This configures the DFT straps to disable DFT mode on PORs. This is to solve a JTAG reliability issue with test unlocked parts.

Closing this in favor of an alternative approach that will be more reliable. Specifically, the alternative approach is to set the hyp RX (OT TX) to pull down, so that it will automatically be held low while DFT strap sampling happens and OT can drive it high when the UART is configured.
I will imlement the above in a separate PR and close this.

Sounds great, thanks @timothytrippel.


[Comments Requested] Update Datasheet

With @capossele's updates to the datasheet in https://github.com/risc0/risc0/pull/897 (:tada:!), we should update the website to link to these new figures rather than the older datasheet. I see a few ways to do it, and am interested in feedback on which approach to take:

Directly link to the datasheet at https://risc0.github.io/ghpages/datasheet/index.html as it currently exists. The easiest thing to do, this shows everyone the very latest numbers.
Make a long term copy of the datasheet upon each version release, and link these "released datasheet"s on our website. Shows our users consistent numbers coming from a stable version, requires a bit of engineering in https://github.com/risc0/ghpages
Manually copy the datasheet numbers into a "pretty" datasheet format (e.g. like what we have in https://dev.risczero.com/datasheet.pdf) at each version release. Gives easier flexibility and adaptability around formatting, requires manual work (and potentially introduces manual error) on each release.

That's in my preference order (i.e. I like just directly linking the most) but I definitely see arguments for each of these. Anyone else have their own feedback or preferences?

I'm going to get the datasheet to be updated, but I still think we should have a separate release process for the datasheet for public consumption.


HSEARCH-3582 Make the build compatible with Eclipse

https://hibernate.atlassian.net//browse/HSEARCH-3582
I took this opportunity to improve the Jenkinsfile a little bit, allowing to run the build in an arbitrary environment using a new "ENVIRONMENT_FILTER" parameter. I used it to run only the part of the build checking that the eclipse compiler works fine, see here: http://ci.hibernate.org/job/hibernate-search-yoann/job/HSEARCH-3582/25/

Pull Request Test Coverage Report for Build 1

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0006%) to 88.936%

Totals

Change from base Build 249:
0.0006%

Covered Lines:
17612

Relevant Lines:
19803

💛  - Coveralls

Checked on Eclipse, it works. I'll merge and adjust if @gsmet runs into any problem while developing on Eclipse.


Upgrade CoreOS GUI.app to v0.9.2

Thank you for the contribution. It was merged directly as 4cf15ea3253f954f7e4bbe65c5512f779cce35e7 to keep commit history cleaner. Your contribution is still credited to you.


feat: coverage for failed value computation

This simple use-case did not work
odelay.Delay(duration) { throw new TimeoutException() }.future

(if there were tests already in the repo I would have added one.)

Thanks - I think the tests are here? https://github.com/softwaremill/odelay/blob/master/odelay-testing/src/test/scalajvm/TimerSpec.scala

There are some test failures

There are some test failures

Sorry Adam, I fiddled with the tests a bit but didn't manage to fix it in 1hr. I have been hacking Scala for 7 years now but have never went through the pains of working on ScalaJS (like Future not having transformWith or not being about to match on types (e.g. case _: Assertion =>). I'm afraid learning it is not that useful either. I'll leave this to anyone who's interested to fix.

@nimatrueway I think it's simply time to drop Scala 2.11 support :)


Not compatible with latest ecto version

mongodb_ecto requires ecto version 2.1.0 while currently, we are working with ecto-3.1.3, which is causing the error. Please upgrade it to support the latest ecto

@utkarshvaish9 I currently have a branch that works and am trying to contact one of the maintainers to switch over ownership.
https://github.com/commoncurriculum/mongodb_ecto/pull/2#issuecomment-877674728

We are now compatible with the latest ecto, albeit in early beta form.  Closing this issue.


Somebody broke the CSV file

Please review the format of the CSV file. It has been broken for a couple weeks.
When I run this, which is what I have used for the past few years:
$M365Products = convertFrom-CSV (Invoke-WebRequest -Uri $MicrosoftFileLocation).tostring()
and then:
$M365Products |gm
The output is:
TypeName: System.Management.Automation.PSCustomObject
Name                                  MemberType   Definition

Equals                                Method       bool Equals(System.Object obj)
GetHashCode                           Method       int GetHashCode()
GetType                               Method       type GetType()
ToString                              Method       string ToString()
???Product_Display_Name               NoteProperty string ???Product_Display_Name=10-Year Audit Log Retention Add On
GUID                                  NoteProperty string GUID=c2e41e49-e2a2-4c55-832a-cf13ffba1d6a
Service_Plans_Included_Friendly_Names NoteProperty string Service_Plans_Included_Friendly_Names=Auditing 10Year Retention Add On
Service_Plan_Id                       NoteProperty string Service_Plan_Id=7d16094b-4db8-41ff-a182-372a90a85407
Service_Plan_Name                     NoteProperty string Service_Plan_Name=Auditing_10Year_ Retention_ Add_On
String_Id                             NoteProperty string String_Id=10_ALR_ADDON
What are those 3 question marks "???" in front of Product_Display_Name
Thank you.

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

ID: 7c245452-161d-89a1-c026-99a0721cf071
Version Independent ID: 16dec59a-eb83-1837-4efd-4645e7aeaed2
Content: Product names and service plan identifiers for licensing - Microsoft Entra ID
Content Source: docs/identity/users/licensing-service-plan-reference.md
Service: entra-id
Sub-service: users
GitHub Login: @Nicholak-MS
Microsoft Alias: nicholak

Hi @mprentice22153 not sure what is causing that to encode incorrectly. I republished another another version. Can you see if you are still hitting the issue? If so I will have to investigate further.
Thanks
Nick

@Nicholak-MS Still getting the same problem. I'm not seeing the issue in the downloaded CSV when I get it direct from the website. Interesting that it is the first field in the CSV.
Apologies for saying it has been happening for a couple weeks. I thought it was happening before I started my vacation in April but I can't find evidence. I've definitely been seeing it this week.
Not sure where the issue is currently

@mprentice22153
Thanks for your feedback! We will investigate and update as appropriate.

I have not rebooted my system for almost a month but will soon and can test, just in case something is whacky.
When I download the CSV via a browser and save it, this works fine:
$temp = Import-Csv 'C:\Downloads\Product names and service plan identifiers for licensing.csv'
$temp | gm

   TypeName: System.Management.Automation.PSCustomObject

Name                                  MemberType   Definition
----                                  ----------   ----------
Equals                                Method       bool Equals(System.Object obj)
GetHashCode                           Method       int GetHashCode()
GetType                               Method       type GetType()
ToString                              Method       string ToString()
GUID                                  NoteProperty string GUID=c2e41e49-e2a2-4c55-832a-cf13ffba1d6a
Product_Display_Name                  NoteProperty string Product_Display_Name=10-Year Audit Log Retention Add On
Service_Plans_Included_Friendly_Names NoteProperty string Service_Plans_Included_Friendly_Names=Auditing 10Year Retention Add On
Service_Plan_Id                       NoteProperty string Service_Plan_Id=7d16094b-4db8-41ff-a182-372a90a85407
Service_Plan_Name                     NoteProperty string Service_Plan_Name=Auditing_10Year_ Retention_ Add_On
String_Id                             NoteProperty string String_Id=10_ALR_ADDON

Thanks @mprentice22153 for the testing assistance here from your side. I am able to confirm downloading from the site works as well. I haven't tried via PS. But curious to hear if rebooting helps your side. If not I will do a little reconfig on my side to see if we can get it working again.
Thanks
Nick

@Nicholak-MS
Thanks for your response.
@mprentice22153
We are going to close this thread, if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.
#Please-close
#Please-close

This ticket should not have been closed.  I have rebooted my system and confirmed I'm still seeing the 3 question marks. I've built a workaround into my function but I really shouldn't need to.
`Function Get-M365ProductList {
$MicrosoftFileLocation = 'https://download.microsoft.com/download/e/3/e/e3e9faf2-f28b-490a-9ada-c6089a1fc5b0/Product%20names%20and%20service%20plan%20identifiers%20for%20licensing.csv'

write-output 'Downloading current product list from Microsoft website'
try {
    $M365Products = convertFrom-CSV (Invoke-WebRequest -Uri $MicrosoftFileLocation -ErrorAction stop).tostring()
} catch {
    $M365Products = $null
}

# Corrections to the file
If ($M365Products) {
    # The CSV currently has/had "String_ Id" with a space!!!!

$M365Products | add-member -MemberType AliasProperty -Name 'String_Id' -Value 'String_ Id' -erroraction silentlycontinue
    $M365Products | add-member -MemberType AliasProperty -Name 'Product_Display_Name' -Value '???Product_Display_Name' -erroraction silentlycontinue
}

Return $M365Products

}
$prods = Get-M365ProductList
$prods | gm
TypeName: System.Management.Automation.PSCustomObject
Name                                  MemberType    Definition

Product_Display_Name                  AliasProperty Product_Display_Name = ???Product_Display_Name
Equals                                Method        bool Equals(System.Object obj)
GetHashCode                           Method        int GetHashCode()
GetType                               Method        type GetType()
ToString                              Method        string ToString()
???Product_Display_Name               NoteProperty  string ???Product_Display_Name=10-Year Audit Log Retention Add On
GUID                                  NoteProperty  string GUID=c2e41e49-e2a2-4c55-832a-cf13ffba1d6a
Service_Plans_Included_Friendly_Names NoteProperty  string Service_Plans_Included_Friendly_Names=Auditing 10Year Retention Add On
Service_Plan_Id                       NoteProperty  string Service_Plan_Id=7d16094b-4db8-41ff-a182-372a90a85407
Service_Plan_Name                     NoteProperty  string Service_Plan_Name=Auditing_10Year_ Retention_ Add_On
String_Id                             NoteProperty  string String_Id=10_ALR_ADDON
`

Hi @mprentice22153 I have taken down the CSV and rebuilt from an entirely new csv file and made it available to download. Curious if this solves the encoding issue you are running in to. Can you please run a test?
Thanks,
Nick

@Nicholak-MS
So weird. I just tested this morning and I'm getting the same results.  Wondering now if this is a PowerShell version/module issue.
My workaround works and my scripts are working again but odd that this is happening. I guess you'll just have to turn it into an API request and forego the CSV. ;-)
$psversiontable

Name                           Value
----                           -----
PSVersion                      5.1.19041.4291
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.19041.4291
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

Hi @mprentice22153 , can you help me to test a theory please? I think I traced the timing of the start of this issue back to a new row that was added that starts with digits, it is the only row of its kind... So I removed the row temporarily to see if that is the root of the issue. Could you try your original PS code and see if that removes the "???". Thanks!

Have the same problem when using Invoke-WebRequest. Issue not observed when file is downloaded and imported via Import-Csv
First time problem was noticed May 6 2024. Previous code execution in April 29, May 1 were fine.

Code and replication:
$licenseCsvURL = "https://download.microsoft.com/download/e/3/e/e3e9faf2-f28b-490a-9ada-c6089a1fc5b0/Product names and service plan identifiers for licensing.csv"
$licenseCsvFile = (Invoke-WebRequest -Uri $licenseCsvURL).ToString() | ConvertFrom-Csv
$licenseCsvFile[0]
Workaround:
$licenseCsvURL = "https://download.microsoft.com/download/e/3/e/e3e9faf2-f28b-490a-9ada-c6089a1fc5b0/Product names and service plan identifiers for licensing.csv"
[void](Invoke-WebRequest -Uri $licenseCsvURL -PassThru -OutFile .\LicenseFile.csv)
$licenseCsvFile = Import-Csv -Path .\LicenseFile.csv
$licenseCsvFile[0]

Tagging @fruitninja3 who is helping to debug this issue internally with me.

Tagging @fruitninja3 who is helping to debug this issue internally with me.

Maybe this helps. Not an expert in content types but this seems weird to me.
$licenseCsvURL = "https://download.microsoft.com/download/e/3/e/e3e9faf2-f28b-490a-9ada-c6089a1fc5b0/Product%20names%20and%20service%20plan%20identifiers%20for%20licensing.csv"
$licenseCsvFile = Invoke-WebRequest -Uri $licenseCsvURL
$licenseCsvFile.Headers

Response is
Key                 Value
---                 -----
X-Ms-ApiVersion     Distribute 1.2
X-Ms-Region         prod-neu-z1
Content-Disposition attachment; filename="Product names and service plan identifiers for licensing.csv"; filename*=UTF-8''Product%20names%20and%20service%20plan%20identifiers%20for%20licensing.csv
Connection          keep-alive
Accept-Ranges       bytes
Content-Length      962173
**Content-Type        application/octet-stream**
Date                Tue, 18 Jun 2024 12:20:56 GMT
ETag                "0x8DC849DC76F4FE7"
Last-Modified       Tue, 04 Jun 2024 13:53:57 GMT
Server              Kestrel

Hi @fruitninja3 - Did you have a chance to look at this issue? Please note that we're also trying to close down all GitHub issues as we've moved to Unified User Feedback so following up on this. Thank you.

@Nicholak-MS and @fruitninja3 - Please move the work item to ADO for tracking as we've switched to a user feedback mechanism. Thank you.


Add proportional font support

Since crossfont was previously used solely by Alacritty, there was no
need for anything other than monospaced fonts. This patch adds minimal
proportional font support to the FreeType backend.
The most relevant part for proportional fonts is using the glyph's
individual advance rather than the font metric's advance. This is
exposed as a new advance field on RasterizedGlyph.
Besides advance, it's also possible for two rasterized glyphs to be
moved closer together by the font in a process called kerning. Crossfont
mimics the FreeType API here and provides the kerning method which
returns the relative offset of the second of two glyphs. Kerning for
OpenType fonts implemented in a GPOS table is not supported.

@chrisduerr beep.

@kchibisov No worries, I haven't lost track of this yet. I've just been sick over the last week so things have gotten a bit delayed.

@kchibisov No worries, I haven't lost track of this yet. I've just been sick over the last week so things have gotten a bit delayed.

Take your time. I've thought that you've changed stuff, but forgot to push or something, since you went through review.

Yeah thanks for the reminder, better safe than sorry.

Emojis work properly now:


Fix bug when remove invoker

If deploy mutilple invoker instances on one physical node, and  execute below remove remaining actions, exist problems
TASK [invoker : remove remaining actions] **************************************
Wednesday 15 November 2017  14:58:47 +0900 (0:00:00.618)       0:00:08.693 ****
changed: [invoker0]

TASK [invoker : remove invoker log directory] **********************************
Wednesday 15 November 2017  15:08:45 +0900 (0:09:57.545)       0:10:06.239 ****
changed: [invoker0]

remove remaining actions task will costs much time to execute(about cost 10 mins), becuase when remove invoker0, the task will remove action container wsk0_, wsk1_, and so on, on the other hand, wsk1 is removed by invoker1's stop hook default, so that's the reason why it will cost much time.
it may remove other invoker's action container, this is wrong logic, righ logic is inovker0 remove wsk0_xx, invoker1 remove wsk1_xx, and so on.

So it is necessary to add exact filter to wsk action container when remove invoker.

@markusthoemmes can you go ahead and merge this


gosec with gometalinter reports same error multiple times

gometalinter
~~~ SNIP ~~~
repository/git/manager.go:241::warning: Potential file inclusion via variable,MEDIUM,HIGH (gosec)
repository/git/manager.go:241::warning: Potential file inclusion via variable,MEDIUM,HIGH (gosec)
repository/git/manager.go:241::warning: Potential file inclusion via variable,MEDIUM,HIGH (gosec)
repository/git/manager.go:241::warning: Potential file inclusion via variable,MEDIUM,HIGH (gosec)
repository/git/manager.go:241::warning: Potential file inclusion via variable,MEDIUM,HIGH (gosec)
~~~ SNIP ~~~
DEBUG: [Jul 27 12:32:33.233] total elapsed time 37.680799079s

gosec by hand
jjacobs@dev04:~/$ gosec repository/git/manager.go
[gosec] 2018/07/27 14:33:38 including rules: default
[gosec] 2018/07/27 14:33:38 excluding rules: default
[gosec] 2018/07/27 14:33:38 Searching directory: repository/git
[gosec] 2018/07/27 14:33:40 Checking package: git
[gosec] 2018/07/27 14:33:40 Checking file: repository/git/bucket.go
[gosec] 2018/07/27 14:33:40 Checking file: repository/git/git.go
[gosec] 2018/07/27 14:33:40 Checking file: repository/git/manager.go
[gosec] 2018/07/27 14:33:40 Checking file: repository/git/repository.go
[gosec] 2018/07/27 14:33:40 Checking file: repository/git/template.go
Results:

[repository/git/manager.go:241] - G304: Potential file inclusion via variable (Confidence: HIGH, Severity: MEDIUM)
  > os.Open(src)

Summary:
   Files: 5
   Lines: 600
   Nosec: 0
  Issues: 1

It also appears to ignore // nolint: directives:
pkg/syncthing/fetch.go:141::warning: Errors unhandled.,LOW,HIGH (gosec, gosec, gosec, gosec, gosec, gosec, gosec)
pkg/ksync/syncthing.go:226::warning: Errors unhandled.,LOW,HIGH (gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec, gosec)

pkg/ksync/syncthing.go:226
221     go func() {
 222         for { // nolint: megacheck
 223             select {
 224             case <-SignalLoss:
 225                 log.WithFields(s.Fields()).Info("signal loss dectected. shutting down")
 226                 s.Stop() // nolint: errcheck, gosec
 227                 os.Exit(1)
 228                 return
 229             }
 230         }
 231     }()

I can open a separate issue if needed.

@xor-gate @alecthomas Thanks for raising this issue. I'll have a look to it.

+1

@ccojocar already found some time? ✌

@xor-gate This week, I am quite busy. I hope to find next week some time to look into these issues.

@ccojocar I made one-line fix for this issue. Can you review and merge?

@serjvanilla Thanks! LGTM, gosec operates now on package not anymore on file.

Thanks guys, highly appreciated!

@alecthomas @timfallmk I verified the // nolint: gosec directive, and it seems to work now.


Add support for retrieving activities by foreign id or uuid

We add activities in a serverless manner, i.e some async operation adds the activities,
thus the activity id created by getstream is not useful in terms of retreiving activity
We use foreign_id concept to link our db with getstream activities.
I didnt found any way to get activities by foreign_id.
Suggestion: In Getstream chat, user can specify message.id when creating a message and later get by message id. Can same be  done for feeds as well.
Am i missing something here
gz#12616

Agent comment from Seetha in Zendesk ticket #12616:Hi
Thanks for reaching out to us.
The team will look into this as soon as possible.
In the meantime, you can make use of this -
await client.getActivities({ids: [activity_id1, activity_id2]})
Cheers
Seetha
°°°

hey @fearless23, you can use the combination of the foreign id and time to get/update activities.

Hi @mahboubii the docs you linked to only mentions updating an activity by foreign_id & time. How would one use a foreign_id & time combination to get a single activity?

You can getActivities like this:
client.getActivities({foreignIDTimes: [{foreign_id: ..., time: ...}, ...]})

Thanks! I just found out about the client.getActivities method, you kinda need to dig into the docs to find it. Maybe an idea to also mention it here. Fetching activities by foreign_id is quite a common use-case I think, and it is not immediately apparent in the docs how to do it.
Awesome product btw! I'm having a blast integrating it into my app :)


add beta descriptor for ci aida

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Log tab truncates lines when it encounters certain characters.

I did this: log("shelfDrawerDiff < 0");
This is what was printed: "shelfDrawerDiff "
Console log prints the full line as expected: "shelfDrawerDiff < 0"
Operating system is linux.

It should print fine in your console / debug output (you might need the -d flag for Grabber, I don't remember precisely), but I can see why this would happen in the Log tab, since it's being shown as HTML.
The output should be escaped before being shown I guess, in the logToHtml function:
https://github.com/Bionus/imgbrd-grabber/blob/321954b522ef62aee90f158fb4bfd5aa5044ddae/src/lib/src/utils/logging.cpp#L43
I guess replacing < and > by < and >  should be enough to do the trick.
I'll have a look.

It also trims leading whitespace, so "    example" would print as "example". That whitespace might be useful for indentation.
Apparently Github also squeezes multiple spaces into one...


JS Data Structures and Algorithms Telephone Number Validator Project, passes weak answers.

There is a problem with the test for the telephone validator. They pass validators which do not check for proper spacing or dashing. (i. e. my validator will return true for 55 55-55-555-5).
This is the my code which passed the test, but really shouldn't have:
 let firstReg=new RegExp("^\-");
  if(firstReg.test(str)){return false;}
  let regEx= new RegExp("-","g");
  console.log(firstReg);
  let newStr=str.replace(regEx,"");
  let newRegEx= new RegExp(" ","g")
  newStr=newStr.replace( newRegEx,"");
  console.log(newStr);
  let regExArr=[/^\d\d\d\d\d\d\d\d\d\d$/, /^[(]\d\d\d[)]\d\d\d\d\d\d\d$/, /^1[(]\d\d\d[)]\d\d\d\d\d\d\d$/, /^1\d\d\d\d\d\d\d\d\d\d$/];
  for(let i=0; i<regExArr.length; i++){
    if(regExArr[i].test(newStr)){
      console.log(i);
      return regExArr[i].test(newStr);
    }
  }
  return false;
}

I'd recommend adding some extra tests to the exercise.
Here is a link to the exercise:
Tell us about your browser and operating system:

Browser Name: Chrome
Browser Version: Version 87.0.4280.88
Operating System: Windows 10

If possible, add a screenshot here (you can drag and drop, png, jpg, gif, etc. in this box):

Link to code
Link to project @ freeCodeCamp

I guess a simple fix would be to add this test.
telephoneCheck("55 55-55-555-5") should be false
assert(telephoneCheck("55 55-55-555-5") === false);


[Bug]: Unit and Integration test check prevent PR from merging when changes are on ignored list

Is there an existing issue for the same bug?

[X] I have checked the troubleshooting document at https://opendevin.github.io/OpenDevin/modules/usage/troubleshooting
[X] I have checked the existing issues.

Describe the bug
PRs like this https://github.com/OpenDevin/OpenDevin/pull/1995 won't merge properly because all the edited files are in the ignored list, so the checks will NOT run integration/unit tests, BUT they are still waiting for "All test passed" (and it never passed which blocks the PR from merging).
https://github.com/OpenDevin/OpenDevin/blob/main/.github/workflows/run-integration-tests.yml

Current Version
N/A

Installation and Configuration
N/A

Model and Agent
No response
Reproduction Steps
No response
Logs, Errors, Screenshots, and Additional Context
No response

The current CI setting has a bug:
If a test fails, this "All integration tests passed" check will skip, and then the PR can be merged! In fact, I had to create https://github.com/OpenDevin/OpenDevin/pull/2024 because a PR was merged even if the tests were failing.

Removing this "all tests passed" gate would solve this issue. I'll create a PR soon.


spin does not allow login to http-only registry

env:
spin 0.9.0 (e2f4fac 2023-02-21)
Apple arm

started a registry on my laptop

docker run --rm -p 5000:5000 -e REGISTRY_HTTP_SECRET=secret --name registry arm64v8/registry:2

Login using Docker succeeds:

docker login -u rjindal -p secret http://<IP_ADDRESS>:5000/
WARNING! Using --password via the CLI is insecure. Use --password-stdin.
Login Succeeded

Login using Spin fails:

spin registry login --username rjindal --password secret http://<IP_ADDRESS>:5000/
Error: cannot log in to the registry

Caused by:
    cannot authenticate as rjindal to registry <IP_ADDRESS>: http transport error: error sending request for url (https://<IP_ADDRESS>/v2/): error trying to connect: tcp connect error: Connection refused (os error 61)

on local registry (for testing) we are able to proceed without auth and for real registries, it is recommended not to run with http only registry. so i am ok with this ticket getting closed as won't do

You can trace that back to the line below.
I think you should be able to conditionally make that insecure based on whether it's HTTP or HTTPS.
https://github.com/fermyon/spin/blob/492e50921f44086e2c9e2a0efdb074fce991eccf/crates/oci/src/client.rs#L294


Added a Login page

Hello @codervivek5 ,
Added a login page using HTML, CSS & JavaScript.
Screenshot -

Video -
Screencast from 26-05-23 01:49:45 AM IST.webm
It is ready to Merge

Ok @codervivek5 will raise a PR you can assign me this


Enable running operator ITs in CI

The operator uses the JOSDK's LocallyRunOperatorExtension, which requires an accessible Kube cluster. This is easily done locally by starting minikube. But we don't currently have a Kube cluster available in CI. This means that

We're reliant on developers remembering to run the tests for build-breakage avoidance.
Sonar sees less test coverage than we actually have. Some of the 'missing' coverage is difficult to make up for with unit testing, because we end up having to mock out a bunch of JOSDK internals. That's difficult to do correctly without deep knowledge of JOSDK.

In JOSDK itself they:

Have E2E tests which can run against a "local" or "remote" cluster. Example
use a maven profile to enable/disable running those E2E tests:
Have a github workflow which starts minikube and applies that profile.

We don't have to copy with JOSDK does, but it's a starting point.

keen to pick this up.


Fresh row ids for generic generators should be unique on every invocation

Currently the function bayesdb_generator_fresh_row_id synthesizes a fresh row for an unobserved member of the statistical population by always returning one greater than the number of rows in the generator table.
While this behavior is appropriate for crosscat since it never remembers fake rowids, it is inappropriate for a general GPM. In particular, a GPM may keep track of every rowid it has ever seen, even during queries (i.e invoking SIMULATE with a particular rowid = x will cause a permanent change to the internal state of the GPM).
One simple example is a very simple GPM for a random variable x in venture:
(assume x (mem (lambda (rowid) (normal 0 1))))
One query with rowid=max(rowid)+1 followed by another query with rowd=max(rowid)+1 will cause a failure mode. One correct behavior is to synthesize rowid ~ U[0,1] to ensure the integrity of uniqueness.

That sounds like a broken metamodel to me.  Simulate should not have side effects other than on the random state.

Agreed with @riastradh-probcomp. Venture's sample should have the appropriate semantics here.
(It does have a drawback in that if you intend to simulate and then insert a bunch of rows, and you do simulate and insert as separate operations, then the inserted row will not necessarily have the same per-row latents that it was simulated with. But this is a separate issue. I don't know if Bayeslite has an answer to this; maybe it should be another ticket?)

@axch and I had discussion in past whether simulate obeyed sample or predict semantics. Keeping this ticket open until we explicitly express the difference in the design document (ie a GPM which does predict on simulate and changes its internal staet is violating the contract). Currently we do not enforce that constraints per se.

The implementation of VsCGpm goes to great pains to ensure this invariant is maintained: https://github.com/probcomp/cgpm/blob/master/src/venturescript/vscgpm.py#L104-L106


Target new version of SDK and add Run-time Permission Request

The project has targetSdk = 22, which is not the newest version.
The project does not implement Run-time Permission model of Android 6.0

This sample has been deprecated/archived (check README for more information on newer samples related to this technology).
As recommended by GitHub, we are closing all issues and pull requests.


[puhutv] Add new extractor

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[ ] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[ ] Bug fix
[ ] Improvement
[x] New extractor
[ ] New feature

Description of your pull request and other information
New extractor added due to issue#16010

I have finished all commits, Could you review again? @dstftw

@dstftw I think optional metadata fields does not break extractration if they are not exists. If still there is problem, could you review? You can review what time you want.

Is there any problem in this code, could you have a look to codes? @dstftw


Having problems with false lightning stikes

Hi,
Can any one tell me how to set up the driver to reduce false possitives, unfortuntaly my knowledge of python3 is a little limited, I would be greatful for any help.
Michael

Hi,
Please have a look at the API documentation on Read The Docs. Particularly the indoor property, noise_floor_limit property and the spike_threshold property. Experiment with different combinations and see if they solve your problem.
Cheers,
Martin

Thank you for repling.
Unfortunatly my knowledge of programming is limited.
Looking at :
@property
def noise_floor_limit(self):
"""int: Get or set the noise floor limit threshold in the range 0 - 7 (default is 2). When
this threshold is exceeded, an interrupt is issued. Higher values allow operation with
higher background noise but decrease sensitivity to lightning strikes.
"""
return self._get_register(self._NF_LEV)
How would I set the set the noise floor limit threshold?
Would you be kind enough to show me an example?
Michael

Assuming that you have created a sensor object as in the example code and called it sensor then setting properties works like this:
>>> sensor.indoor = True
>>> sensor.noise_floor_limit = 4
>>> sensor.spike_threshold = 6

Hope that helps.
Cheers,
Martin

Thanks for your help.
Using your as3935_simpletest.py, and adding your examples like below:
sensor = biffobear_as3935.AS3935_I2C(i2c, interrupt_pin=interrupt_pin)
sensor.indoor = True
sensor.noise_floor_limit = 4
sensor.spike_threshold = 6
I hope I've got it right
Michael

Hi, that looks fine. I suggest running it with various combinations of values until you find the best solution for you.
Cheers,
Martin


chore: update Go version to 1.23.0 and golangci-lint to 1.60.1

📑 Description
This PR bumps the Go version to 1.23 across the entire project. golangci-lint is updated to 1.60.1 because older versions are not compatible with Go 1.23.
Additionally, this PR contains some changes that fix lint errors that were uncovered due to additional checks in this new version of golangci-lint.
ℹ Additional Information
Release notes:

Go: https://go.dev/doc/go1.23
golangci-lint: https://github.com/golangci/golangci-lint/releases/tag/v1.60.1

you have to delete bin/golangci-lint


Change laziness in TCT

We don't need complete nodes to lazily compute their hashes, and making them strict simplifies the serialization logic and removes a "gotcha" from parallelizing work, because now you won't be able to accidentally move hash evaluation work to another thread just by failing to call root.

This was resolved by #925.


Guildkick Proposal Failing With Not Enough Gas

Feature
Proposals
App

Admin

Expected Behavior
Would expect the proposal to execute
Actual Behavior
The toast comes up saying Success but the transaction is failing
Steps to Reproduce

Create Guild Kick Proposal
Vote yes to pass
Wait through Grace
Try to execute

Browser
No response
Screenshots and Context
Transaction

@santteegt and @earth2travis  can we retest this once now that the gas was bumped up in that that other pr?
i see this has now been executed:
https://admin.daohaus.fun/#/molochV3/0x5/0xf862337cdc9c6144bc2629a2fc3b9c3aa7dc3a46/proposals/3

@skuhlmann Guildkick proposal seemed to work fine: https://admin.daohaus.fun/#/molochV3/0x5/0x942cd237b22e14e331a6827f578f9868745ce9f4/proposals/5
https://goerli.etherscan.io/tx/0xa74cff13c94f7c37b8d7f1889d9c34fcce4e576010d76d1ee297f1ef68c18034

@earth2travis I was able to test one of these as well and it's looking good.
I think we can close it.

Fixed by https://github.com/HausDAO/daohaus-monorepo/pull/954


Firewall requirements too permissive

[Enter feedback here]
What reason would you allow access to every *.blob.core.windows.net
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: fab35792-13ac-5fc9-c434-49d3a98e77c0
Version Independent ID: 75dc39cc-d528-088c-3f8e-2454a5db56bb
Content: Log Analytics agent overview - Azure Monitor
Content Source: articles/azure-monitor/agents/log-analytics-agent.md
Service: azure-monitor
Sub-service: agents
GitHub Login: @bwren
Microsoft Alias: bwren

@McMatty Thank you for reaching out with your query , we will look into it and get back to you at the earliest.

It was the advice on the page to whitelist every single blob instance in Azure for traffic.
So what you are doing is reducing security because you cannot provide specific information on the actual endpoints that need whitelisting.
Quick example I push stolen data to an Azure blob from an internal network - because the advice provide was to allowlist every instance.


Modular Agent

Create an agent that refactors sub-processes into more modular functions and pieces to enable future flexibility.

We should move toward approving @DannyWeitekamp's branch so we can start to move forward on other things. It might make sense to just trigger a pull request and go through debates on the code there?

I think this sounds reasonable.

@cmaclell @DannyWeitekamp  Is this issue resolved by the recent pull request?

Yes, and more so with the next one I will send today


Mortal kombat

Please describe your documentation issue or suggested improvement in detail here and provide links to any pre-existing/relevant documentation and screenshots if necessary:

Hi @Jaydyzzle813
When opening an issue, please try to provide all of the information indicated in the template.
Closing this issue, since no information has been provided.
Thank you.


Firefox for Windows does not work

Which version? Seems to be working with 56.0.2 (32 bit) on Win 7.

It was the presentation computer, I hope to be able to check it today.
Susanna
2017-10-28 20:18 GMT+00:00 Ari<EMAIL_ADDRESS>
Which version? Seems to be working with 56.0.2 (32 bit) on Win 7.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/artturimatias/GLAMpipe/issues/27#issuecomment-340217168,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEgaqIr-EsodpjgrffIKrV9D_-6R1jhEks5sw4wWgaJpZM4QKCcX
.


Move class name capture for private state after declaration evaluates

Our private field transform inadvertently attempted to capture the class declaration during its evaluation when we encountered a computed property name, resulting in a ReferenceError at runtime due to the binding not being initialized.
Fixes #44113
/cc @dragomirtitian, @mkubilayk

@typescript-bot cherry-pick this to release-4.3


fix mfa mapping

Problem
valid MFA configuration values are back and forth across implementations

the CloudFormation UserPool resource accepts "OFF" | "OPTIONAL" | "ON" https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-cognito-userpool.html#cfn-cognito-userpool-mfaconfiguration
the L1 CfnUserPool construct sets this property to a string https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_cognito.CfnUserPool.html#mfaconfiguration-1
the L2 UserPool construct accepts "OFF" | "OPTIONAL" | "REQUIRED"https://docs.aws.amazon.com/cdk/api/v2/docs/aws-cdk-lib.aws_cognito.Mfa.html#members
the Cognito console alludes to "no mfa" | "optional" | "required"

and our unified schema is looking for "NONE" | "OPTIONAL" | "REQUIRED"
https://github.com/aws-amplify/amplify-backend/blob/main/packages/client-config/src/client-config-schema/schema_v1.json#L178
This PR aims to narrow the output types to their expected values, update client config generator to map CFN values to the new values, and update legacy convertors to map back down to the CFN values
The JS library is not looking for NONE in its parsing, but does account for OPTIONAL and REQUIRED https://github.com/aws-amplify/amplify-js/blob/de6b06db80b02687e597aceaa2c33f65171a6389/packages/core/src/parseAmplifyOutputs.ts#L328-L335

Issue number, if available:
Changes

Corresponding docs PR, if applicable:
Validation

Checklist

[ ] If this PR includes a functional change to the runtime behavior of the code, I have added or updated automated test coverage for this change.
[ ] If this PR requires a change to the Project Architecture README, I have included that update in this PR.
[ ] If this PR requires a docs update, I have linked to that docs PR above.
[ ] If this PR modifies E2E tests, makes changes to resource provisioning, or makes SDK calls, I have run the PR checks with the run-e2e label set.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Before we do this we need to complete #1633 to ensure that this is not introducing a breaking change

@edwardfoyle I was chatting with @josefaidt to trim this PR to just introduce correct mapping in client-config which should make concern about "producing" outputs away for now.
Does this help pushing the fix forward?

@sobolk @edwardfoyle I've removed the type narrowing from the construct's outputs


Adding check-branch-name hook.

Check that branch-name matches a given regex (passed as argument).

Note to reviewers: this is not meant to replace a "break the CI if branch name does not comply" feature, it will just provide an advance warning to developers if they have created a branch that is not named "the right way".

@mgu comments accounted for!


Status 400 when pushing updates

After generating a pass and saving it in my wallet I want to send push notifications to the Pass. I managed to get the pushToken via webservice, but template.pushUpdates(myToken) returns a 400 status code
[Object: null prototype] {   ':status': 400,   'apns-id': 'FD4215FA-6D82-F2F1-952A-CE516A5146FA',   [Symbol(nodejs.http2.sensitiveHeaders)]: [] }

what webservice implementation are u using?

@axjowo How you registered for updates?


"host" config should take precedence over the "socket" config if present

For initializing db connections, if both socket and host configs are passed, trilogy gives precedence to the socket config which results in error messages like below:
Trilogy::SyscallError::ENOENT: No such file or directory - trilogy_connect - unable to connect to /tmp/mysql.sock (Trilogy::SyscallError::ENOENT)

These can be frustrating to debug since the same config works fine with the mysql2 adapter but throws an error with trilogy.
So if both socket and host are passed, precedence should be given to the host config, and this maintains parity with mysql2 as well.

@jjatinggoyal thanks for opening the issue. Other than parity, can you explain why you think host should take precedence over socket?

This is what I'd personally prefer.

@bensheldon I stumbled upon this error through a new rails application I created with trilogy. The default database.yml in rails looks something like this if you are running MySQL locally.

This works perfectly in the development setup, but when I finally decide to deploy the app to prod, I just add the host config as I usually do when using mysql2. But I surprisingly ran into the aforementioned error.
So long story short, the confusion is mainly due to the default rails config, my familiarity with mysql2 and the assumption that trilogy is kinda a drop-in replacement for mysql2.

Submitted a PR in rails to change the default config for trilogy. Closing this issue.


Functional Registration/Aliasing, Tracking, and Listing

This PR provides the following features, along with the necessary fixes:

Sidebar mode switch between "Browse" (Supposed to allow to jump on the right URL for the comic/source) and "Register" (allows creating new entries + adding sources to existing entries)
A new "Register" Button in the sidebar used to validate the creation of a new entry
Functional registration of a Comic into the web-extension's storage
Functional Aliasing of new sources to an existing Comic from the web-extension's storage
Functional Listing of the saved Comics & Sources
Visual Notification (Green fading to default color for success, Red fading to default color for failure) through the toggle button, to notify the user of the latest operation's status. Includes:

Registration
Aliasing
Auto-Tracking

Here is the registration process in screenshots:

Go to your Comic's page (on a supported reader)

Click on the round + button to start the registration process

Use the searchbox to search for your Comic

Finalize Registration
4.1. You found it in your tracking list ! Click on the name of the Manga you with to add a source to.
[No Screenshot]
4.2. You could not find it ! Create a new tracking entry by clicking on the "+" button on the side of the searchbox (Hovering over it highlights it)

Tracking visual notification:

Browsing the registered entries in the sidebar (BROWSE mode)

Note that these following screenshots show a few things that remain to do:
Cleanups:

Register round Button is still showing after Registration is completed (memoization of the ComicID by the BmcEngine object prevents an update). We need to force this update.

User Experience:

The two + buttons for registration feel like an unecessary double. We could remove the Infobar completely, and only use the new Registration button instead.
Searchbox isn't filled by default when attempting to Register. It could be a cool addition
Searchbox content isn't emptied once the SideBar is hidden
Searchbox content (if not emptied) is not taken into account for filtering of the Comics when reopening the Sidebar

I've also identified, while testing this, some odd behaviors:

All readers do not always trigger a page load when switching from one page to another. I sometimes had mangaeden simply switching URL/picture without loading the page again
=> This might mess-up the auto-tracking if not handled !

And finally, the next steps:

Simplify UI where possible
Improve UX in general
Allows the browse mode to redirect to the Comic/Source's page to go on reading

I tried your registration process and I don't really like it: it's way too complicated.

Here's the look I had in mind (which is in my commit):

Having the "+" button directly integrated into the sidebar seems like a must have. And like I said, I really think we should think about the process of adding a manga because currently it's very hard to follow. :)

I was more thinking of having the "+" visible even without the sidebar open, replacing completely the current inforbar's register button.
Now, integrating it in the sidebar is an equivalent in my opinion, while the sidebar is open (and in "register" mode).

I was more thinking of having the "+" from the sidebar visible even without the sidebar open, replacing completely the current infobar's register button.
Now, integrating it in the sidebar the way you did is an equivalent in my opinion to keeping it on the side. This must only be visible when the sidebar is in "register" mode, though.
Additionally, I really want to keep a register button visible when the sidebar is closed to make the UI more intuitive for lambda users: The symbolic and operational path are clearer that way.

Btw, I need to understand clearly what's too complicated in that registration process, because I see the following improvements possible:

pre-filling the searchbox with a name extracted from the page
allowing to use "Enter" to validate the registration - which is desirable - (and not only a click)
Automagically do stuff which would not allow for user customization during registration:
a. define the manga name
b. Detect duplicates, and create new sources for existing manga on behalf of the user

Personally, I'm against 3, because I think it makes stuff like customizing the name less intuitive for lambda users, and it requires quite a bit of work on our part for an incomplete and imperfect result.


Fix the font size for schema input descriptions

Description
The markdown component used to render property descriptions defaults to font-md but label descriptions are supposed to be font-sm. Updating the input component to specific the font size fixes the descriptions so they're the correct size.
Before

After

Nice catch, thanks @pleek91


Fix misspelling of rapids_cpm_init

Description
rapids_cpm_init is misspelled in the README.md file
Checklist

[x] I am familiar with the Contributing Guidelines.
[ ] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[ ] The cmake-format.json is up to date with these changes.
[ ] I have added new files under rapids-cmake/

[ ] I have added include guards (include_guard(GLOBAL))
[ ] I have added the associated docs/ rst file and update the api.rst

/merge

/ok to test

Thanks @dagardner-nv! 🎉


Problems to import in Powerapps

Hi,
I have tried to do an import in Powerapps. But that is not workning.

Hi @AndersDahlNorconsult - Could you provide more information, please? Perhaps the error message you get or a screenshot?
Also, some things to check:

You can't import the component into your default environment - you will need a Production, Sandbox or Developer environment.
You must have administrator rights on the environment you are importing into.
3

Closing due to inactivity - please re-open if you have more information on this. Thanks 🙌


There is still a transparency issue with setting ALPHA=1.0 in the shader script

Tested versions
4.0.4, 4.2, 3.5
System information
macOS 14.4 (23E214) Godot 4.2, 4.0.4
Issue description
Setting ALPHA=1.0 in the shader script still has a transparent effect, just like setting the transparency of StandardMaterial3D to alpha or setting no depth test.
But if the rendering mode of the shadow script adds depth_draw-always, setting ALPHA=1.0 can display normally, but it cannot produce shadow effects.
Steps to reproduce
In the project I uploaded:
In the scene of node_3d_desk, the shader script has set ALPHA=1.0 and I do not want to see the tripod of the table.
Desk_normal has added StandardMaterial3D to the surfaceMaterialOverlay of Box024 without any changes.

Desk_alpha added StandardMaterial3D to the SurfaceMaterialOverrides of Box024 and changed transparency to alpha.

Desk_no_depth_test has been added to StandardMaterial3D and transparency has been changed to no_depth_test in the surface MaterialOverrides of Box 024.

Desk_shaded_alpha added ShaderMaterial in the surfaceMaterialOverride of Box024, with only ALPHA=1.0 set in the script.

In the scene of node_3d_sofa, there is also a similar problem. This problem occurs whenever the mesh has an irregular shape.

How can I set ALPHA=1.0 in the shader without expecting transparency or no_depth_test effects, and still display shadows normally.
When I add render_mode depth_draw_always in the shader script, it can display normally, but it cannot display shadows properly. If the following image is displayed:
Shadow display in normal mode:

Display shadows after setting ALPHA=1.0 and adding render_mode depth_draw_always:

Minimal reproduction project (MRP)
Test0520.zip

This is expected, as setting ALPHA anywhere in the shader will force the material to go through the transparent pipeline. This is the case even if you only use ALPHA = 1.0.
I don't think we should have a special case for ALPHA = 1.0. Could you describe why you want to write to ALPHA in the first place? Perhaps you can achieve the desired effect another way.

I had a similar problem, when using depth_draw_always, ALPHA=0.9 resulted in depth not being written, but ALPHA=0.99 did

@Khasehemwy what you are seeing is a different issue from the OP.
In your case you are seeing a few different things:

Because depth_prepass_alpha is being used, the object is rendered in the depth prepass. However, the depth prepass will only write out the depth for objects that have an alpha higher than 0.99. Accordingly, nothing is written in the depth prepass.
When you read from the depth texture, you are actually reading a copy of the depth texture that was copied right after doing the depth prepass. Therefore, only objects caught by the depth prepass are included
Because depth_draw_always is enabled, the depth is written during the transparent pass. You don't see the results in the depth texture, because the depth texture was copied before the transparent pass

@clayjohn Does that mean there is no way to get the depth of a transparent object in depth texture?

@clayjohn Thank you! I found the changes needed to make the depth pre pass work for transparent objects. But does that mean there is no way to get the depth of a transparent object by depth texture in official version?

When using hint_depth_texture that is correct. You can however get the depth texture after rendering transparent objects using a CompositorEffect in 4.3

This is expected, as setting ALPHA anywhere in the shader will force the material to go through the transparent pipeline. This is the case even if you only use ALPHA = 1.0.
I don't think we should have a special case for ALPHA = 1.0 not making the material go through the transparent pipeline. Could you describe why you want to write to ALPHA in the first place? Perhaps you can achieve the desired effect another way.

Hi,
I have the same issue, I'm developing a multiplayer golf game where each player has a ball.
No collision is possible between balls.
So when the player's ball is too near another ball, I would like to fade the other ball according to the distance to the player ball. It is very similiar to the way trackmania deals with display of others cars (eg: https://www.reddit.com/r/TrackMania/comments/1ejmn1y/just_started_playing_did_i_do_good_and_how_can_i/)
Acoording to the way Alpha is working on Godot shaders, my solution could be to duplicate meshes, one with alpha one without. Any other solution?


Cannot enable/disable DHCP for Wifi in ESP32C6

Answers checklist

[X] I have read the documentation ESP-AT Programming Guide and the issue is not addressed there.
[X] I have used the latest released firmware or have updated my ESP-AT branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

AT+GMR
AT version:<IP_ADDRESS>-dev(0f91d98 - ESP32C6 - Jul 19 2024 02:57:35)
SDK version:v5.1.2-dirty
compile time(368f620f):Aug  9 2024 14:25:28
Bin version:v<IP_ADDRESS>-dev(ESP32C6-4MB)
OK
ESP-AT Firmware Source
Local build from master Branch, also tested with prebuilt Binary for v<IP_ADDRESS>/release for ESP32C6
Hardware Information
ESP32C6-DevKitM-1
Power Supply used
USB
What is the expected behavior?
The DCHP client for Wifi interface (as Station) shall be activated or deactivated with the AT command AT+CWDHCP=<operate>,<mode>. DHCP is active by default, but in some cases one needs to assign an IP address statically, so DHCP first needs to be disabled.
What is the actual behavior?
The DHCP cannot be changed (at least not directly) with the AT command. Either before Wifi Station initialization, or after the initialization, or even after the connection to an Access Point. It simply does not allow to change the DHCP.
Strangely, one can indirectly change the DHCP with the command AT+CIPSTA=<"ip">,<"gateway">,<"netmask">, due to the condition documented:

The Set Command correlates with the commands that set DHCP, such as AT+CWDHCP.

If static IPv4 address is enabled, DHCP will be disabled;
If DHCP is enabled, static IPv4 address will be disabled;
The last configuration overwrites the previous configuration.

So, if we set a fixed IP address, we then see that the DHCP has been deactivated for Wifi Station. This would be OK, but then the DHCP is deactivated persistently, and the only way to activate it is to run a factory reset.
Probability of recurrence
Always
AT+SYSRAM?
AT+SYSRAM?
+SYSRAM:256860,249408
OK
Steps to reproduce
Either before Wifi Station initialization:
AT
    AT
    OK
ATE0
    ATE0
    OK
AT+GMR
    AT version:<IP_ADDRESS>-dev(0f91d98 - ESP32C6 - Jul 19 2024 02:57:35)
    SDK version:v5.1.2-dirty
    compile time(368f620f):Aug  9 2024 14:25:28
    Bin version:v<IP_ADDRESS>-dev(ESP32C6-4MB)
    OK
AT+CWDHCP?
    +CWDHCP:3
    OK
AT+CWDHCP=0,0
    ERROR

Or after the initialization:
AT
    AT
    OK
ATE0
    ATE0
    OK
AT+GMR
    AT version:<IP_ADDRESS>-dev(0f91d98 - ESP32C6 - Jul 19 2024 02:57:35)
    SDK version:v5.1.2-dirty
    compile time(368f620f):Aug  9 2024 14:25:28
    Bin version:v<IP_ADDRESS>-dev(ESP32C6-4MB)
    OK
AT+CWMODE=1,0
    OK
AT+CWDHCP?
    +CWDHCP:3
    OK
AT+CWDHCP=0,0
    ERROR

Or after Wifi connection to external AP:
AT
    AT
    OK
ATE0
    ATE0
    OK
AT+GMR
    AT version:<IP_ADDRESS>-dev(0f91d98 - ESP32C6 - Jul 19 2024 02:57:35)
    SDK version:v5.1.2-dirty
    compile time(368f620f):Aug  9 2024 14:25:28
    Bin version:v<IP_ADDRESS>-dev(ESP32C6-4MB)
    OK
AT+CWMODE=1,0
    OK
AT+CWDHCP?
    +CWDHCP:3
    OK
AT+CWJAP="<redacted>","<redacted>"
    WIFI CONNECTED
    WIFI GOT IP
    OK
AT+CWDHCP?
    +CWDHCP:3
    OK
AT+CWDHCP=0,0
    ERROR

But indirectly (setting fixed IP when DHCP is on) works (but cannot be enabled again, only with a factory reset):
AT+CWDHCP?
    +CWDHCP:3
    OK
AT+CIPSTA="<IP_ADDRESS>","<IP_ADDRESS>","<IP_ADDRESS>"
    WIFI GOT IP
    OK
AT+CWDHCP?
    +CWDHCP:2
    OK
AT+CIPSTA?
    +CIPSTA:ip:"<IP_ADDRESS>"
    +CIPSTA:gateway:"<IP_ADDRESS>"
    +CIPSTA:netmask:"<IP_ADDRESS>"
    OK
AT+CWDHCP=1,0
    ERROR

AT command port output
Already detailed before

AT log port output
[2024-08-09 15:14:00.886] ESP-ROM:esp32c6-20220919
[2024-08-09 15:14:00.886] Build:Sep 19 2022
[2024-08-09 15:14:00.887] rst:0xc (SW_CPU),boot:0xc (SPI_FAST_FLASH_BOOT)
[2024-08-09 15:14:00.887] Saved PC:0x4001975a
[2024-08-09 15:14:00.887] SPIWP:0xee
[2024-08-09 15:14:00.887] mode:DIO, clock div:1
[2024-08-09 15:14:00.888] load:0x4086c410,len:0xd2c
[2024-08-09 15:14:00.888] load:0x4086e610,len:0x2df0
[2024-08-09 15:14:00.889] load:0x40875728,len:0x17d8
[2024-08-09 15:14:00.889] SHA-256 comparison failed:
[2024-08-09 15:14:00.889] Calculated: 63045dfae877bfe6aba67856cf9cd56b6defbda508cb8a0274419830fbca4e24
[2024-08-09 15:14:00.890] Expected: c741a6f3476df9f7051649747915014991fb1693a63114f9c8774552522107fd
[2024-08-09 15:14:00.891] Attempting to boot anyway...
[2024-08-09 15:14:00.891] entry 0x4086c410
[2024-08-09 15:14:00.891] I (45) boot: ESP-IDF v5.1.2-dirty 2nd stage bootloader
[2024-08-09 15:14:00.892] I (46) boot: compile time Aug  9 2024 11:05:27
[2024-08-09 15:14:00.892] I (47) boot: chip revision: v0.0
[2024-08-09 15:14:00.892] I (49) boot.esp32c6: SPI Speed      : 80MHz
[2024-08-09 15:14:00.892] I (53) boot.esp32c6: SPI Mode       : DIO
[2024-08-09 15:14:00.892] I (58) boot.esp32c6: SPI Flash Size : 4MB
[2024-08-09 15:14:00.893] I (63) boot: Enabling RNG early entropy source...
[2024-08-09 15:14:00.893] I (68) boot: Partition Table:
[2024-08-09 15:14:00.894] I (72) boot: ## Label            Usage          Type ST Offset   Length
[2024-08-09 15:14:00.895] I (79) boot:  0 otadata          OTA data         01 00 0000d000 00002000
[2024-08-09 15:14:00.895] I (87) boot:  1 phy_init         RF data          01 01 0000f000 00001000
[2024-08-09 15:14:00.895] I (94) boot:  2 nvs              WiFi data        01 02 00010000 0000e000
[2024-08-09 15:14:00.896] I (101) boot:  3 at_customize     unknown          40 00 0001e000 00042000
[2024-08-09 15:14:00.896] I (109) boot:  4 ota_0            OTA app          00 10 00060000 001d0000
[2024-08-09 15:14:00.896] I (117) boot:  5 ota_1            OTA app          00 11 00230000 001d0000
[2024-08-09 15:14:00.896] I (124) boot: End of partition table
[2024-08-09 15:14:00.896] I (129) esp_image: segment 0: paddr=00060020 vaddr=42150020 size=30670h (198256) map
[2024-08-09 15:14:00.897] I (218) esp_image: segment 1: paddr=00090698 vaddr=40800000 size=0f980h ( 63872) load
[2024-08-09 15:14:00.897] I (249) esp_image: segment 2: paddr=000a0020 vaddr=42000020 size=1488d4h (1345748) map
[2024-08-09 15:14:00.897] I (797) esp_image: segment 3: paddr=001e88fc vaddr=4080f980 size=0a0e4h ( 41188) load
[2024-08-09 15:14:00.897] I (818) esp_image: segment 4: paddr=001f29e8 vaddr=40819a70 size=03bc4h ( 15300) load
[2024-08-09 15:14:00.898] I (826) esp_image: segment 5: paddr=001f65b4 vaddr=50000000 size=00068h (   104) load
[2024-08-09 15:14:00.898] I (839) boot: Loaded app from partition at offset 0x60000
[2024-08-09 15:14:00.899] I (840) boot: Disabling RNG early entropy source...
[2024-08-09 15:14:00.899] no external 32k oscillator, disable it now.
[2024-08-09 15:14:00.899] at param mode: 1
[2024-08-09 15:14:00.899] AT cmd port:uart1 tx:7 rx:6 cts:5 rts:4 baudrate:115200
[2024-08-09 15:14:00.900] module_name: ESP32C6-4MB
[2024-08-09 15:14:00.900] max tx power=78, ret=0
[2024-08-09 15:14:00.900] v<IP_ADDRESS>-dev
[2024-08-09 15:14:00.901] BLUFI BLE is not connected yet
[2024-08-09 15:14:00.901] negotiated phy mode: 3
[2024-08-09 15:14:00.901] <IP_ADDRESS>
[2024-08-09 15:14:00.902] BLUFI BLE is not connected yet
[2024-08-09 15:14:00.902] negotiated phy mode: 3
[2024-08-09 15:14:00.902] wifi disconnected, rc:8
[2024-08-09 15:14:00.902] ESP-ROM:esp32c6-20220919
[2024-08-09 15:14:00.903] Build:Sep 19 2022
[2024-08-09 15:14:00.903] rst:0xc (SW_CPU),boot:0xc (SPI_FAST_FLASH_BOOT)
[2024-08-09 15:14:00.903] Saved PC:0x4001975a
[2024-08-09 15:14:00.903] SPIWP:0xee
[2024-08-09 15:14:00.904] mode:DIO, clock div:1
[2024-08-09 15:14:00.904] load:0x4086c410,len:0xd2c
[2024-08-09 15:14:00.904] load:0x4086e610,len:0x2df0
[2024-08-09 15:14:00.904] load:0x40875728,len:0x17d8
[2024-08-09 15:14:00.905] SHA-256 comparison failed:
[2024-08-09 15:14:00.905] Calculated: 63045dfae877bfe6aba67856cf9cd56b6defbda508cb8a0274419830fbca4e24
[2024-08-09 15:14:00.905] Expected: c741a6f3476df9f7051649747915014991fb1693a63114f9c8774552522107fd
[2024-08-09 15:14:00.906] Attempting to boot anyway...
[2024-08-09 15:14:00.906] entry 0x4086c410
[2024-08-09 15:14:00.906] I (45) boot: ESP-IDF v5.1.2-dirty 2nd stage bootloader
[2024-08-09 15:14:00.907] I (46) boot: compile time Aug  9 2024 11:05:27
[2024-08-09 15:14:00.908] I (47) boot: chip revision: v0.0
[2024-08-09 15:14:00.908] I (49) boot.esp32c6: SPI Speed      : 80MHz
[2024-08-09 15:14:00.909] I (53) boot.esp32c6: SPI Mode       : DIO
[2024-08-09 15:14:00.909] I (58) boot.esp32c6: SPI Flash Size : 4MB
[2024-08-09 15:14:00.909] I (63) boot: Enabling RNG early entropy source...
[2024-08-09 15:14:00.910] I (68) boot: Partition Table:
[2024-08-09 15:14:00.910] I (72) boot: ## Label            Usage          Type ST Offset   Length
[2024-08-09 15:14:00.910] I (79) boot:  0 otadata          OTA data         01 00 0000d000 00002000
[2024-08-09 15:14:00.910] I (87) boot:  1 phy_init         RF data          01 01 0000f000 00001000
[2024-08-09 15:14:00.911] I (94) boot:  2 nvs              WiFi data        01 02 00010000 0000e000
[2024-08-09 15:14:00.911] I (101) boot:  3 at_customize     unknown          40 00 0001e000 00042000
[2024-08-09 15:14:00.911] I (109) boot:  4 ota_0            OTA app          00 10 00060000 001d0000
[2024-08-09 15:14:00.911] I (117) boot:  5 ota_1            OTA app          00 11 00230000 001d0000
[2024-08-09 15:14:00.911] I (124) boot: End of partition table
[2024-08-09 15:14:00.911] I (129) esp_image: segment 0: paddr=00060020 vaddr=42150020 size=30670h (198256) map
[2024-08-09 15:14:00.912] I (218) esp_image: segment 1: paddr=00090698 vaddr=40800000 size=0f980h ( 63872) load
[2024-08-09 15:14:00.912] I (249) esp_image: segment 2: paddr=000a0020 vaddr=42000020 size=1488d4h (1345748) map
[2024-08-09 15:14:00.912] I (797) esp_image: segment 3: paddr=001e88fc vaddr=4080f980 size=0a0e4h ( 41188) load
[2024-08-09 15:14:00.912] I (818) esp_image: segment 4: paddr=001f29e8 vaddr=40819a70 size=03bc4h ( 15300) load
[2024-08-09 15:14:00.912] I (826) esp_image: segment 5: paddr=001f65b4 vaddr=50000000 size=00068h (   104) load
[2024-08-09 15:14:00.912] I (839) boot: Loaded app from partition at offset 0x60000
[2024-08-09 15:14:00.912] I (840) boot: Disabling RNG early entropy source...
[2024-08-09 15:14:00.913] no external 32k oscillator, disable it now.
[2024-08-09 15:14:00.913] at param mode: 1
[2024-08-09 15:14:00.913] AT cmd port:uart1 tx:7 rx:6 cts:5 rts:4 baudrate:115200
[2024-08-09 15:14:00.913] module_name: ESP32C6-4MB
[2024-08-09 15:14:00.913] max tx power=78, ret=0
[2024-08-09 15:14:00.913] v<IP_ADDRESS>-dev
[2024-08-09 15:14:00.913] BLUFI BLE is not connected yet
[2024-08-09 15:14:00.914] negotiated phy mode: 3
[2024-08-09 15:14:00.914] wifi disconnected, rc:8
[2024-08-09 15:14:00.914]

More Information.
No response

Hi,
It is recommended not to set the  parameter of the AT+CWDHCP command to 0, as this value is not supported. If you want to disable DHCP in station mode, please use AT+CWDHCP=0,1. To disable DHCP in softAP mode, use AT+CWDHCP=0,2. If you need to disable DHCP in both station and softAP modes, use AT+CWDHCP=0,3. If you want to enable DHCP in station mode, set it to AT+CWDHCP=1,1. I hope this information is helpful to you.

@Luxin812 Thank you for the recommendation. I have tried as you suggested it and works now.
As feedback for maintainers of the repository, official documentation https://docs.espressif.com/projects/esp-at/en/latest/esp32c6/AT_Command_Set/Wi-Fi_AT_Commands.html#at-cwdhcp-enable-disable-dhcp should be updated, as the way it is shown, the parameters for the Set Command are:
Set Command

Function:
Enable/disable DHCP.

Command:
AT+CWDHCP=<operate>,<mode>

Parameters
    <operate>:
        0: disable
        1: enable
    <mode>:
        Bit0: Station DHCP
        Bit1: SoftAP DHCP

But after your feedback, it is clear that the second parameter mode of the Set Command works as documented for the Query Command:
Query Command

Command:
AT+CWDHCP?

Response:
+CWDHCP:<state>
OK

Parameters
    <state>: the status of DHCP
        Bit0:
            0: Station DHCP is disabled.
            1: Station DHCP is enabled.
        Bit1:
            0: SoftAP DHCP is disabled.
            1: SoftAP DHCP is enabled.
        Bit2:
            0: Ethernet DHCP is disabled.
            1: Ethernet DHCP is enabled.


Improve the editor feature profiles UX

Add an help message when no profile is selected.

This replaces the class/property trees which are now hidden when no profile is selected.

Display (none) as the current profile when no profile is currently active.
Make the newly created/imported profile the current if it's the first profile to be added to the list.
Make more strings localizable.

These changes should hopefully clear up some confusion related to the editor feature profiles UI :slightly_smiling_face:
Preview
No profile has been created yet (freshly opened dialog).

This closes #39022.

Thanks!

why shouldn't godot provide a profile by default instead, wouldn't that make more sense and isn't that the expected behaviour? i bet that most people are going to ignore that message

@RaTcHeT302 The default feature profile implicitly enables all editor features.


Beccani wip

Just wanted to give @beccani a chance to work out a PR that has requested changes. @beccani, all you need to do is update your code and commit again, which will automatically update the pull request and give us the chance to review again.

Thank you for the feedback! Git and pull requests is something I only have limited experience with so this is super helpful!
I believe I have committed the corrected code.


Fixing cannot fetch diet restriction and improve diet filter

What I have done.

I have discovered that the diet restriction filters are broken.
Resolve the issue
Can now filter more diet restrictions.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

webpage/modules/builder.py
0.00%
3 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Files with missing lines
Coverage Δ

webpage/migrations/0012_custom_diet_creation.py
100.00% <100.00%> (ø)

webpage/modules/builder.py
44.30% <0.00%> (ø)

I have move the main logic of adding the diet from SpoonacularRecipeBuilder to NormalRecipeBuilder


PlantUML Salt compatibility with Asciidoctor

This is an example of including a salt diagram in AsciiDocFX:
[uml,file="abc.png"]
----
@startsalt
{
{T
 + index.adoc
 ++ America
 +++ Canada
 +++ USA
 ++++ New York
 ++++ Boston
 +++ Mexico
 ++ Europe
 +++ Italy
 +++ Germany
 ++++ Berlin
 ++ Africa
}
}
@endsalt
----

In Asciidoctor, a different attribute list is required:
[salt, "abc" png]

This gives issues when using both tools.

Hello, you can even use the following; Isn't it same with Asciidoctor's syntax ?
[uml,abc,png]
----
@startsalt
{
{T
 + index.adoc
 ++ America
 +++ Canada
 +++ USA
 ++++ New York
 ++++ Boston
 +++ Mexico
 ++ Europe
 +++ Italy
 +++ Germany
 ++++ Berlin
 ++ Africa
}
}
@endsalt
----

I didn't know the salt block, it can be included in AsciidocFX

[uml,abc,png]

The above syntax in Asciidoctor gives this message:
WARNING: tree.adoc: line 19: invalid style for listing block: uml

Hmm, what is the correct syntax for Asciidoctor ?

There seem to be 2 variations. See:

http://discuss.asciidoctor.org/Using-asciidoc-diagramm-with-plantuml-salt-syntax-td2850.html
https://github.com/asciidoctor/asciidoctor-diagram/issues/66

I think the preferred syntax is this:
[plantuml, basefilename, svg]
----
@startsalt
{
{T
+ A
++ a
+ B
++ b
}
}
@endsalt
----

You can use this in AsciidocFx

Correct. Further syntax differences tracked in #295.

There seem to be 2 variations. See:

http://discuss.asciidoctor.org/Using-asciidoc-diagramm-with-plantuml-salt-syntax-td2850.html
Enable "salt" sketches in plantuml diagrams  asciidoctor/asciidoctor-diagram#66

I think the preferred syntax is this:
[plantuml, basefilename, svg]
----
@startsalt
{
{T
+ A
++ a
+ B
++ b
}
}
@endsalt
----

this is helpfull, still work in 2023


why not use the real API?

Just a heads up, there is a proper API that you can use instead of this. https://www.dokuwiki.org/devel:remote_api

I looked around on GitHub to try and find a python wrapper for any of the official or even nonofficial API to doku.
I found dokuwiki and python-dokuwiki but neither seemed to expose the users (which I needed for my current project)
I also looked into the RPC docs and honestly don't know a ton about it. Decided to just reverse engineer the existing HTTP calls. Seems I could still just use those calls. But looking at the docks now that I've been called out I see these calls which don't seem to include get all current users, bulk submit a user import,etc. I have this implemented to some degree now.
Seeing as I created a PyPi package for this I'm sure you're at least a little annoyed. It certainly crossed my mind that the second you change one of those undocumented endpoints it would break what I've done and that could lead to potential users blaming your side. The entire reason they aren't documented is so you can do that.
(I thought about pairing the tags to the version of doku this works with though)
All this to say, I am using this and just thought to share! The end-user wouldn't know the difference on how the calls are made since it's a wrapper.


[BUG] vga devices adding additional field (type=serial0 vs serial0)

Hi Guys,
I have found an issue in your module which stops you using cloud-init properly.  The issue is when you add a vga definition and assign it to the type you want.  The system seems to be adding type= before the value.
tf config snippet
  vga {
    memory = 0
    type   = "serial0"
  }

You can seen in the screenshot below in black is what your module added in with the snippet above.

Serial terminal 0 (type=serial0)

This black text should read what is shown in red (changed in UI before server restart manually after tf deploy)
Serial terminal 0 (serial0)

I have also tested if you set it to std it even adds type=std and it shouldn't be.  Even if you leave out vga its adding in std and not using the templates config.
I have looked though the code but I am not a go expert or even novice and I can't locate the code/bug which is doing this.
A simple fix would be to just remove this type= because none of the settings set by vga will work in its current state.
Doing a dump of the configs with qm config ID shows the problem.  Below you will see a cloned manually from same template which tf uses and one which your tf module used which doesn't work properly.
tf created config
root@pve-n01:~# qm config 2761
agent: 1
bios: seabios
boot: c
bootdisk: virtio0
cores: 4
cpu: host
description: Kubernetes Nodes
hotplug: network,disk,usb
ide2: lab-vm:vm-2761-cloudinit,media=cdrom,size=4M
ipconfig0: ip=<IP_ADDRESS>/24,gw=<IP_ADDRESS>
memory: 8192
name: kube-n01
net0: virtio=7E:82:49:4D:0E:5E,bridge=vmbr0
numa: 0
onboot: 1
scsihw: virtio-scsi-pci
serial0: socket
smbios1: uuid=cacde703-b7a1-41a7-9290-ac7a35ff8461
sockets: 4
sshkeys: ********
vga: type=serial0
virtio0: lab-vm:vm-2761-disk-0,format=raw,iothread=1,media=disk,size=123084M
vmgenid: 73f731c5-99a0-40b0-8652-02fc7962125a

Manual created machine via clone in UI
agent: 1,fstrim_cloned_disks=1
balloon: 256
boot: c
bootdisk: virtio0
cores: 2
cpu: host
ide2: prod-vm:vm-100-cloudinit,media=cdrom
ipconfig0: ip=dhcp
memory: 512
name: test-node01
net0: virtio=4A:7C:15:C0:A5:DF,bridge=vmbr0
numa: 0
scsihw: virtio-scsi-pci
serial0: socket
smbios1: uuid=7ef75015-616f-4e71-aec8-840582eda083
sockets: 2
sshkeys: ***************
vga: serial0
virtio0: prod-vm:vm-100-disk-0,size=2252M
vmgenid: 1e1acf36-fef2-48c7-949d-c060c82e7e74

I wonder if someone knows or can point me in the right direction in getting this bug squished.
Thank you.
Kind Regards

This was actually a bug with Proxmox itself which was tracked down by KM.
Info found in the following bug report, there for this one is being closed.
https://bugzilla.proxmox.com/show_bug.cgi?id=2787


Use the issuer's name instead of the issuer's issuer when searching for certificate

Use the issuer's name instead of the issuer's issuer when searching for certificate
Fixes #19982
When searching for a valid issuer certificate and the certificate contains the issuer name in the authorized key identifier it was comparing with the issuer's issuer instead of the issuer's name.
Also fix the certificate creation code that also was using the issuer's issuer's name.
Add a test for this specific problem and update the test certificates affected by this change.

Actually this is wrong and the original report is not a real issue. I am sorry for mis-triaging it.


Better control of the animation play state

From the source I noticed that the _timer and _watch fields are not made publicly available. I think it would be nice if the play state could be altered externally. Events and functions for starting, stopping, pausing on a certain frame or time could help manage animations.

Not planned.


bigtable: allow arbitrarily complex gcRules in cbt

Issue #581 had a request to be able to create arbitrarily complex cbt rules.  The canonical example is:
maxage=30d || (maxage=3d && maxversions=100)

I have raised PR for this feature request. Link

@jba, is this something you can take on?


Updater dialog dismisses after finishing the download without any further action.

updater dialog dismissed after finishing the download with console log " UpdateStatus.DialogDismissed " ,
the app is not being updated  and no errors shown.

@EngMohamedNor  Please use the issue template and provide detailed information to fix the issue.

Any update for this issue?


[FR] Consider allowing multiple rows of image tags in the sample grid

The old behavior in the App was to let image tags stack up into multiple rows. More recently, the design was changed to always squash labels onto one row, which, as the example below illustrates, can make them rather unreadable for more densely annotated datasets.
There's at least two separate decision axes here:

How to handle long individual tags?
One row or multiple rows?

On balance, I find that the current design abbreviates tags too aggressively such that I can't fully read them. A recent example I had was an image with two tags: [Parasitized] [Parasitized], but it was abbreviated as something like [Pa...] [Pa...] in a Jupyter notebook, which made it tough to parse.
On balance, I'm inclined to say that allowing multiple rows may be better. With that said, baking some level of abbreviation to handle very long tags when they appear would be appropriate. Perhaps the ideal would be some intelligent logic that would abbreviate more aggressively when there are more tags?
A related thought is that implementing a grid zoom option per https://github.com/voxel51/fiftyone/issues/749 would help to provide more real estate for individual samples in an environment with a fixed width App with dense annotations.

I agree, the abbreviation should exist for longer tags but needs to be much less aggressive, and that they should be expanded to multiple rows as needed. My main concern is coving images completely with tags. Maybe making the tag backgrounds slightly transparent could help.
More aggressive abbreviation for longer labels makes sense but I would still not go too far with it. You should retain at least 3 or 4 characters per label.


RadioButton`s defaultChecked prop is not supported anymore.

Here http://material-ui.com/#/components/switches
I tried defaultSwitched but somehow that also not working for me. It would be great if docs can be updated with proper usage.

Fixed in recent docs overhaul.


Implement account migration for Unified account

Describe the task
Migrate unified account feature

BA doc: https://www.notion.so/subwallet/Master-acccount-d99f9e17956343e9a11cae981a2eadfa?pvs=4#74b7ff5ce07e46978226db507bbb6844
Flow migrate: https://www.figma.com/board/ULes5pp2IUV04S9kGct58c/SubWallet-Flow?node-id=19-2&t=uqYbyFxK9ZTXB2Zt-4
Design các màn hình Migrate:
https://www.figma.com/design/aEnkI859zkZaIpyzPidYwh/SubWallet-Apps?node-id=25402-294281&t=4GDvlUr1EeQ3NV6F-4

Please check issues:

[ ] Still show popup "Understand unified account" when click on "Migrate" button
Steps:

Upgrade version
Unlock wallet
Click on "Migrate" button on popup "Understand unified account"
Observe the screen displayed
Re-open the popup

Actual:

Step 5: Still show popup
Step 6: Do not show popup Migrate on popup

Expect: Allow next step to migrate account and show migrate screen on both popup and expand view


Edited shaders don't always update

Godot 3.0 master
Windows 10 64 bits nVidia 940M
When editing shaders, I noticed that very often, when I hit Ctrl+S after modifying a shader, nothing happens as if I didn't change it. In these cases I almost always try to write an extra ";", then delete it and re-save with Ctrl+S... and tadaaa, the shader updates.
Did anyone experienced this?
PS: similarly, I see the script editor sometimes gets stuck showing an error with a red highlight even though I corrected the error. Same thing, when I write a character then delete it and re-save to kinda "kick the machine", it updates and the lingering highlight goes away. Would it be the same issue? If not I could open another (https://github.com/godotengine/godot/issues/11616)

I noticed this as well. At certain parts of the script I am forced to retype something that is the same or similar to make the shader do something.

I have seen this recently when testing particles shaders, I don't remember being this way (kind of laggy) a couple of months ago.

This logic shouldn't be too difficult. Relevant code is in:
https://github.com/godotengine/godot/blob/master/editor/plugins/shader_editor_plugin.cpp
Basically after a few seconds of not inputting text, the shader is recompiled, tested and updated. For some reason this may not be happening, so checking these functions should be easy.

I wasn't able to replicate this. I'm on 045bf2320
From what I can tell, Ctrl+S doesn't do anything in the shader editor, it just saves the scene. The shader will update by itself in a couple of seconds.