Invalid type when uploading a file using the local API

Bug Report
When uploading a file, typescript says sizes is required, but the docs say it is autogenerated.
Steps to Reproduce
const Media: CollectionConfig = {
  ...
  upload: {
    imageSizes: [
      {
        name: 'card',
        width: 640,
        height: 480,
      },
      {
        name: 'feature',
        width: 1024,
        height: 576,
      },
    ],
  },

In a change hook:
    const example = await payload.create({
      collection: 'media',
      file: {
        data: pdfBuffer,
        mimetype: 'application/pdf',
        name: 'example.pdf',
        size: pdfBuffer.byteLength,
      },
      data: {
        alt: 'My PDF file',
      },
    })

Error:
Type '{ alt: string; }' is not assignable to type 'MarkOptional<Media, "id" | "createdAt" | "updatedAt">'.
  Property 'sizes' is missing in type '{ alt: string; }' but required in type 'Omit<Media, "id" | "createdAt" | "updatedAt">'.ts(2322)

Other Details
Payload v1.6.10

Resolved with #2066


Ensure links in the docs are correct

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
There are several broken links in the datafusion crate docs. For example

Describe the solution you'd like
I would like them all links to be valid and ideally have a CI check to ensure they remain that way
Additional context
See similar ticket in arrow-rs: https://github.com/apache/arrow-rs/issues/1267

I con work on it if no one else has been doing!

https://github.com/apache/arrow-datafusion/blob/master/datafusion/src/physical_plan/sorts/sort_preserving_merge.rs#L260
I got
the struct `SortPreservingMergeStream` has no field or associated item named `receivers`

when run cargo doc.
I am not sure what this link should be.

Duplicate of #6042


Add Procfile and update app.json

These changes are to support auto deploying of Heroku Review apps when
a PR for Smart Answers is opened.
As per https://docs.publishing.service.gov.uk/manual/review-apps.html

Closing as Heroku auto-deployed this PR anyway - appears that the app.json that we already have in master is sufficient. 😅

Hey @chao-xian.
I'm slightly surprised that this branch was auto-deployed as a review app in Heroku. I think we were previously using the startup_heroku.sh script to do this manually (as per the instructions in https://github.com/alphagov/smart-answers/blob/3e784b05ad09d91af200b4dacb475f56777956a0/doc/factcheck.md).
If review apps now Just Work then I think we should take this opportunity to update some docs in this repo to explain them, and to link to this app from https://docs.publishing.service.gov.uk/manual/review-apps.html.

Hi @chrisroos I hadn't updated the docs yet as I'd not seen it being used by the rest of the team for passing on for Fact Check so far. But your reminder to do all these things was timely.
@ikennaokpala @pmanrubia are you happy for me to get rid of the heroku script and update the docs? Can you let me know Ikenna if you've passed on one of the auto-deployed previews for Fact Check please and are happy with it, or just willing to use these from now on?

@chao-xian removing the heroku script seems natural.. I would ideally like the manual option retained as a backup.


CompileTimeConstantExpressionMatcher matches but ASTHelpers.constValue returns null

For the following expression (in a static initializer block):
"Debug mode " + (DEBUG ? "enabled." : "disabled.")

where DEBUG is a static boolean class field.
This started failing in errorprone-slf4j when switching to use error-prone version 2.14.0 (on java 11).
See also https://github.com/KengoTODA/errorprone-slf4j/issues/193
Is this a recently introduced bug or was there never a guarantee that this order of helper calls would work?

Is this a recently introduced bug or was there never a guarantee that this order of helper calls would work?

Correct - CompileTimeConstantExpressionMatcher matches more expressions than ASTHelpers.constValue(t) != null does (namely, in your case, ternary expressions where both halves of the ternary are themselves constants). This matcher's primary purpose is to serve the https://errorprone.info/bugpattern/CompileTimeConstant checker, ensuring that callers of a method whose parameter is annotated with @CompileTimeConstant pass in a value entirely under the control of the programmer (think about things like avoiding SQL injection).
One thing to think about: At compile-time, what do you think the "value" of the expression "Debug mode " + (DEBUG ? "enabled." : "disabled.") should be?


documentation: added baseline for xamarin.forms sample application

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Documentation
What is the current behavior? (You can also link to an open issue here)
No samples
What is the new behavior (if this is a feature change)?
Baseline for Xamarin.Forms
What might this PR break?
Nothing
Please check if the PR fulfills these requirements

[ ] The commit follows our guidelines: https://github.com/reactiveui/reactiveui#contribute
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

Other information:

@ghuntley,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

Coverage remained the same at 70.424% when pulling 28ba1a8a2ff4fc6b746d6fe4c24d54aeaed09b51 on samples into 9c996aaa72c452e0439a8a69b60722093017324f on develop.

Coverage increased (+0.03%) to 70.458% when pulling 28ba1a8a2ff4fc6b746d6fe4c24d54aeaed09b51 on samples into 9c996aaa72c452e0439a8a69b60722093017324f on develop.


Update to clap 4.0

This PR updates our cli parser clap from 2.34 to 4.0.
The end-user experience should stay similar, but has a fresh UI, many internal improvements, and better help messages when a user gets the arguments wrong.
On the left you can see the new UI, and on the right the old.

Also it is not in alphabetic order anymore, so one need a few moments of looking up and down...

I was thinking about that as well. The reasoning of the authors of clap is that they want to give the cli authors (us) control over the order. We can just solve this by alphabetically ordering the args.

bors r=Dajamante

why do we need Cargo.lock?

Afaik it is convention to commit the lock file for applications, but not for libraries.

can we put <HELPER_CMD> between arrows, or is it against conventions?

That would be possible.


Add a multi-threading dispatcher

Blocker: #85
The example & test is alike. Do you have some advice? /cc @George-Miao @bdbai

Maybe we can add a builder for Dispatcher? For # of threads/thread name/stack size etc., and by default use available_parallelism as the # of threads.

@George-Miao ping


#425

added #[derive(clone)] for DatabaseConnection

As explained in the other thread, we need a better solution


Drawcalls are not batched in WebGL Version

Hola!
It shows 4 drawcalls in editor, but actually it draws over 4 times in website.
It seems not batched in WebGL Version?
I'm using the v1.52b, and check the drawcalls by spector.js.
Gracias.

A model is not batched in WebGL version at least.
If sprites only exists, it is a bug of WebGL version.
What effect are you showing?

Something like this, a portal effect with sprites, particles, and strips.
I thought there's no model.

https://github.com/effekseer/EffekseerForWebGL/blob/master/src/js/effekseer.src.js#L537
Please specify a large value in this parameters.
If draw call is not decreased, it is a bug. I try to reproduce this bug.

I will try it. Thanks :)

Good day!
I increase the value of instanceMaxCount & squareMaxCount, and the number of drawcall is not changed.
We use the sprite, ribbon and track types. So is it a bug?

No. InstanceMaxCount and  squareMaxCount means the maximum number of polygons by a draw call.

Oh, I got it. The value is to make adjustment to the vertex buffer size.

Hi~I modify the demo of WebGL version, and put some effects in it.
Can you check this problem? Thanks. <3
EffekseerForWebGL152f.zip

I saw this demo.
What is wrong?
It seems that good effects are shown.

It looks good, but the draw calls are not batched.
Ribbons or tracks are drawn many times,  especially. :(

OK. I will check... But I'm a bit busy in May. Checking is slow...

Great, thanks for your patience.

I finished my main work. I will tackle this issue.

@yoki0805 Excuse me. Please give me an efkproj file to check a behavior?

Sure! Here you are. :)
Please, open it with editor 152f version.
DarkDoor_all.zip

I fixed a bug. Batching is now working.
https://github.com/effekseer/EffekseerForWebGL/releases/tag/152g

It works great!
The number of drawcalls is decrease now.
Thanks for your helping & hard work. :D


Relative Component Path

Hi,
Thanks for the really useful extension. I'm having a few issues caused by the path being hard-coded in:
https://github.com/zucska/extract-component/blob/master/extension.js#L185
Would it be possible to make it relative to the current file that I'm editing?
Thanks,
Chris

Update to the latest version


evaluat bats testing framework

See https://github.com/bats-core/bats-core
check if we can use it instead of own-written test framework

closing here in favor of https://github.com/nafets227/tools/issues/172


Broken link https://open-horizon.github.io/docs/admin/backup_recovery.md

Looks like backup_recovery.md file is missing.

Can I work on this issue @Rene-Ch1 @joewxboy

Go ahead, @Akshit42-hue
Post a comment or message in the chat room if you have any questions.  Don't forget to:

Sign off on the commit (-s or --sign-off)
Name the PR beginning with Issue-119:

I think all the links are accessible @joewxboy


Write better documentation

All of the documentation will be living here. As of today, it is incomplete.
Below is a list of PRs that are responsible for documenting each aspect of the codebase

[ ] Mangadex::Api
[ ] Mangadex::Internal
[ ] Mangadex::Artist
[x] Mangadex::Auth
[ ] Mangadex::Author
[ ] Mangadex::Chapter
[ ] Mangadex::ContentRating
[ ] Mangadex::CoverArt
[ ] Mangadex::CustomList
[ ] Mangadex::Manga
[ ] Mangadex::Relationship
[ ] Mangadex::ReportReason
[ ] Mangadex::ScanlationGroup
[ ] Mangadex::Tag
[ ] Mangadex::Upload
[ ] Mangadex::User
[ ] Mangadex::MangadexObject
[ ] Mangadex::Types
[ ] Mangadex::Version

You rule dude, will be looking into this for my project and helping add PRs whenever bugs are found.

Hey @royalgiant thank you! Let me know if you need any help with using the gem. I haven't updated it in a while since the Mangadex API doesn't change much, but I'm planning on going back soon to iron out certain things.

Hey there @royalgiant, I'm also having the same issue on mine :/ (https://dexify.herokuapp.com/mangas/browse). I think using a proxy service works well. I never personally bothered with writing one from scratch (though it would be pretty easy with a lightweight framework like Sinatra or NodeJS runtime).
I have a project where I came up with a solution to serve images through a route (on Rails), download the image and serve it. Not performant but a good workaround for that project in particular (see it in action here). It works well here because I'm lazy loading images, but may not work in the case of showing up a bunch of covers

Dude,
https://youranime.moe/ is so darn cool! You have some awesome side hustles
going on, I'm super impressed! I think your "view more" button is broken
though. Got the usual heroku error. I asked their discord and they said
anything that's not a MD or localhost domain (because their whitelisted)
will get that temp cover to go read at MD. The advice they gave me is to
run a proxy service, which it looks like I'll have to do.
I'll probably be running it on my Node App that I have to connect my Rails
project up to Web3.
Here's some current inspirations I'm looking at:
https://github.com/hoangvu12/requests-proxy
https://github.com/Rob--W/cors-anywhere
https://gist.github.com/jimmywarting/ac1be6ea0297c16c477e17f8fbe51347
Best regards,
Donald Lee
Contact Number: +1<PHONE_NUMBER>
Connect with me on Linkedin: https://ca.linkedin.com/in/donaldlee50
Check me out at Github: https://github.com/royalgiant
Learn Personal Finance on my Youtube Channel:
https://youtube.com/coursehackclub
On Fri, Nov 25, 2022 at 10:25 PM Akinyele Cafe-Febrissy <
@.***> wrote:

Hey there @royalgiant https://github.com/royalgiant, I'm also having
the same issue on mine :/ (https://dexify.herokuapp.com/mangas/browse). I
think using a proxy service works well. I never personally bothered with
writing one from scratch (though it would be pretty easy with a lightweight
framework like Sinatra or NodeJS runtime).
I have a project where I came up with a solution to serve images through a
route (on Rails), download the image and serve it. Not performant but a
good workaround for that project in particular (see it in action here
https://dexify.herokuapp.com/mangas/browse). It works well here because
I'm lazy loading images, but may not work in the case of showing up a bunch
of covers
—
Reply to this email directly, view it on GitHub
https://github.com/thedrummeraki/mangadex/issues/5#issuecomment-1327970709,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGOOOX5OZSSN56LEJLXX3LWKF7IZANCNFSM5ETYPJEA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Update MD file on new branch 2017-12-10_13-41-47

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Correction for raw affiliation Centre International de Recherche en Infectiologie; INSERM U1111; University of Lyon 1; Lyon France; Hospices Civils de Lyon; Lyon France

Correction needed for raw affiliation Centre International de Recherche en Infectiologie; INSERM U1111; University of Lyon 1; Lyon France; Hospices Civils de Lyon; Lyon France
raw_affiliation_name: Centre International de Recherche en Infectiologie; INSERM U1111; University of Lyon 1; Lyon France; Hospices Civils de Lyon; Lyon France
new_rors: 01502ca60;02vjkv261;029brtt94;059sz6q14
previous_rors: 01502ca60;02vjkv261;029brtt94
works_examples: W1920365423
contact: a7db69efed649073c45638681429d345:1065b9a02cdf67ed30afc3ee8e4a @ ens-lyon.fr

This issue was accepted and ingested by the OpenAlex team on 2024-11-01. The new affiliations should be visible within the next 7 days.


New action request: alex-page/github-project-automation-plus

Action name
uses: alex-page/github-project-automation-plus
Reasons
alex-page/github-project-automation-plus

Intended usage
alex-page/github-project-automation-plus

I have checked the internal marketplace for similar actions and couldn't find one that works for us.

[X] I have checked the internal marketplace first

:robot: Found these results:

Check
Results
Links

CodeQL on the forked repo
:white_check_mark:
CodeQL run

Security scan:

Check
Result

No Dependabot configuration found
⛔

No CodeQL Init found
⛔

No CodeQL Analyze found
⛔

:robot: Found these results:

Check
Results
Links

CodeQL on the forked repo
:white_check_mark:
CodeQL run

Security scan:

Check
Result

No Dependabot configuration found
⛔

No CodeQL Init found
⛔

No CodeQL Analyze found
⛔

:robot: Check completed with result: ⚠
Found 13 Dependabot security vulnerabilities:
9 where high and 4 where moderate.
Use this link to verify the alerts.


Minor grammatical updates

Category

[X] Content fix
[ ] New article
Related issues: None

What's in this Pull Request?
Minor grammatical updates

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/community/community.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

thx Chris, appreciated.


Move NDT5 ServerMetadata to avoid panic

This change is

Pull Request Test Coverage Report for Build 1553

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.09%) to 81.398%

Files with Coverage Reduction
New Missed Lines
%

ndt7/measurer/measurer.go
2
89.47%

Totals

Change from base Build 1550:
-0.09%

Covered Lines:
1759

Relevant Lines:
2161

💛  - Coveralls

``


Delete group logic

Groups can be tied to record ownership, zone admin, and acl rules. Currently, the only validation done before deleting a group is that it is not set to be a zone admin group for any zones. It should be further discussed how we want to handle cases in which the group is set as a record owner, and when the group is used in an ACL rule

@pauljamescleary our get group and get groups in the dynamo repo already filter out deleted groups so think ACL rules should be fine since the logic would be the same. For record owner we have cards in progress/ merged that do not allow deleting a group if it is a record owner

Fixed in #548


Optimize creation of UniformSnapshot

Applies #972 to master, and adds an unsafe protected constructor for use by trusted code.

@bentatham Thoughts on the change in b77971a?

Ok with me.  I would have said never copy in the constructor, but at least
this makes it explicit.
On Tue, Jul 5, 2016, 10:28 PM Ryan Tenney<EMAIL_ADDRESS>wrote:

@bentatham https://github.com/bentatham Thoughts on the change in
b77971a
https://github.com/dropwizard/metrics/commit/b77971a26169b176331fd5f75828849fc365cc73
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/metrics/pull/978#issuecomment-230656192,
or mute the thread
https://github.com/notifications/unsubscribe/AAj7JI_2FJQNIET4P2swfxA1aKz5sOeYks5qSxLWgaJpZM4JFsz1
.

Something weird happened here and this PR was not marked as merged. It was merged though.


Issue #13: F-Spot does not compile under Mageia 4 #15

Resolved the "Mono.Cairo <IP_ADDRESS>" issues by making an explicit Cairo <IP_ADDRESS> version reference in the csproj that use it (rather than unversioned which was causing Mono.Cairo <IP_ADDRESS> to be automatically imported), and updated the Hyena patch so that it also includes the same changes.
This fixes Issue #13

So, I tested again. the "DLL not found" error for libgtk-win32 appears again on the latest version of the code. I think it is caused by the fact we actually package gtk-sharp-beans.dll without a gtk-sharp-beans.dll.config in f-spot/bin (and that ends up under $PREFIX/lib/f-spot after make install). Removing gtk-sharp-beans.dll fixed the issue. One of my commits made changes to several csproj files in order to prevent the dll from being copied over, and instead rely on the gtk-sharp-beans.dll from mono (under the gac, with its .config and .mdb alongside it).
Some other people on the mailing list faced the same issue.
What would be the proper course of acion here? Reopen the pull request? Create a new one?


Translucent blend mode and texture are ignored if baseColorFactor >= 0.996

This has been the case since the original implementation in #898, but I don't think there's any good reason for it.

I've encountered a problem in our project.
We have a translucent mesh where the opacity is controlled by the alpha channel of a texture. The texture's alpha will be multiplied by the alpha value of the baseColorFactor , so the alpha value of the baseColorFactor  is set to 1 in our 3DTile data.
However, due to the current condition in the code, this material is being set as Masked, instead of the desired translucent mode.
I believe this condition should not be used for determining the material's blend mode.
I find the codes in  CesiumGltfComponent.cpp.
UMaterialInterface* pBaseMaterial =
      (is_in_blend_mode(loadResult) && pbr.baseColorFactor.size() > 3 &&
       pbr.baseColorFactor[3] < 0.996) // 1. - 1. / 256.
          ? pGltf->BaseMaterialWithTranslucency
          : pGltf->BaseMaterial;

I don't think the condition pbr.baseColorFactor[3] < 0.996  is reasonable.

Yeah, I agree, that's why I wrote this issue. We welcome to pull request to fix it!

Do you have any suggestions for modifications? If not, I will submit a simple pull request to remove pbr.baseColorFactor[3] < 0.996 .


CP-342 Release wGulp 0.7.7

PRs contained in this release:
https://github.com/Workiva/wGulp/pull/126
https://github.com/Workiva/wGulp/pull/135
Diff Between Last Tag and Proposed Release: https://github.com/Workiva/wGulp/compare/0.7.6...Release_0.7.7

	When this pull is merged I will use the following information:
	Version: wGulp 0.7.7
	Release Ticket: CP-342

	This pull is considered the release of wGulp 0.7.7.
	The options defined for this repo will be carried out

Last updated on Tuesday, March 03 04:29 PM CST

+1

QA +1
[x] - Dev +1 only required for release.
Ready for merge and tag.


Refactor models

Pulled refactor out to a separate PR as suggested here: #505
In this PR:

Refactored models package to no longer depend on lockfile
Moved functions that required lockfile types into separate internal package under utility/vulns
Copied over OSV url from osv package to avoid dependency on osv from output/reporter

I went through and graphed out the dependencies,

models no longer depends on any internal dependencies
lockfile doesn't depend on any internal dependencies apart from cacheexp
osv brings in both lockfiles and models. This is probably the most problematic one. Ideally osv should only be referencing models, but it'll be a breaking change to change this API. This is one of the main things we should change for V2 (related to #442) (TODO: Make separate issue for this
reporter depends on the internal output, which depends on osv. This dependency causes lockfile to be pulled into everywhere that needs to print out anything. This can be decoupled without too much trouble by copying in the OSV url. I did so in this PR.
config package references reporter,  with the above change it also no longer depends on lockfile.

idk about enforcing, but you could probably use something like https://github.com/kisielk/godepgraph to post a comment on the PR with a dependency graph - though that'd probably be pretty noisy so might be better as an item on the "before release" checklist, to do a final review of the dependency tree in case you want to make any changes that would be breaking if not done otherwise.


Added Google Login.

Added Login with google in UI.
Added a DB field to check whether user is allowed or not. IF it is authencated with google allowed is set to true.
Modified build.sh the sed command had some issues so it wasnot able to update the Dockerfile. Also the arm64 architecutre was missing added it too.
Gogle configurations can be passed as Environment Variables. Here we can update secrets , cliendID from google developer console and allowed domains checked which email domains are allowed. Required ENV variables for google login to work.

     GOOGLE_CLIENT_ID
     GOOGLE_CLIENT_SECRET
     GOOGLE_REDIRECT_URL
     ALLOWED_DOMAINS

Two new modules were added,
1.  golang.org/x/oauth2 v0.21.0
2 . google.golang.org/api v0.185.0
Updated login.html to show button for login with google. By default the google logged in Users don't have Admin access.

Thank you! Let me test it with different archs and we good to go with merging to main.

@janamkhatiwada Is it possible to enhance this feature for a general implementation of OAuth? Described in #75 this feature is requested by some people (and myself). This would establish openvpn-ui to a mentionable competitor against enterprise vpn solutions with a user portal and would be interesting for companies.
I would support this but i can't write GO code.

I merge it, but updated build process, so it will support arm64 in addition to aarm64.

@janamkhatiwada Is it possible to enhance this feature for a general implementation of OAuth? Described in #75 this feature is requested by some people (and myself). This would establish openvpn-ui to a mentionable competitor against enterprise vpn solutions with a user portal and would be interesting for companies.
I would support this but i can't write GO code.

I would be glad to add it. but have new interesting project on my main job, which soaks all my time.


The Left Action Alignment configuration is not taken into account

Target Platforms
Other
SDK Version
2.7.3
Application Name
Web application
Problem Description
When the ActionAlignment is set to Left, the rendered card doesn't align the actions to left.
When the ActionAlignment is set to Center, the rendered card centers well the actions.
This is potentially due to the fact that the ActionAlignment property is decorated with [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)] but the default value is not 0 but 3 (AdaptiveHorizontalAlignment.Stretch).
        **[JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]**
        public AdaptiveHorizontalAlignment ActionAlignment { get; set; } = **AdaptiveHorizontalAlignment.Stretch**;

So when the Context serializes the hostConfig the Left value which is 0 is not serialized  and when it deserialize it the value is set to the default one AdaptiveHorizontalAlignment.Stretch.
      public AdaptiveRenderContext(AdaptiveHostConfig hostConfig, AdaptiveElementRenderers<HtmlTag, AdaptiveRenderContext> elementRenderers)
       {
           // clone it
           Config = **JsonConvert.DeserializeObject<AdaptiveHostConfig>(JsonConvert.SerializeObject(hostConfig))**;
           ElementRenderers = elementRenderers;
           RenderArgs = new AdaptiveRenderArgs { ForegroundColors = Config.ContainerStyles.Default.ForegroundColors };
       }

Extract from https://www.newtonsoft.com/json/help/html/t_newtonsoft_json_defaultvaluehandling.htm
Ignore members where the member value is the same as the member's default value when serializing objects so that it is not written to JSON. This option will ignore all default values (e.g. null for objects and nullable types; 0 for integers, decimals and floating point numbers; and false for booleans).
Possible correction :
        [JsonProperty(DefaultValueHandling = DefaultValueHandling.Ignore)]
        [DefaultValue(AdaptiveHorizontalAlignment.Stretch)]
        public AdaptiveHorizontalAlignment ActionAlignment { get; set; } = AdaptiveHorizontalAlignment.Stretch;

Screenshots

Card JSON
{
  "type": "AdaptiveCard",
  "version": "1.3",
  "body": [
    {
      "type": "Container",
      "$data": "${recommendations}",
      "backgroundImage": {
        "url": "${backgroundImage}",
        "$when": "${backgroundImage != null}"
      },
      "items": [
        {
          "type": "TextBlock",
          "text": "${text}",
          "color": "${color}",
          "weight": "${weight}",
          "wrap": true
        }
      ]
    },
    {
      "type": "Container",
      "items": [
        {
          "type": "TextBlock",
          "text": "${question}",
          "wrap": true,
          "color": "Accent",
          "weight": "Bolder",
          "$when": "${$root.question != null}"
        }
      ]
    },
    {
      "type": "ActionSet"
    }
  ],
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json"
}

Sample Code Language
c#
Sample Code
AdaptiveCardRenderer renderer = new AdaptiveCardRenderer();
renderer.HostConfig.Actions.ActionAlignment = AdaptiveHorizontalAlignment.Left;
RenderedAdaptiveCard renderedCard = renderer.RenderCard(card);
HtmlTag html = renderedCard.Html;

Result is
<button class="ac-pushButton ac-action-submit" type="button" role="button" aria-label="QUESTION SUIVANTE" tabindex="0" data-ac-submitdata=""QUESTION SUIVANTE"" style="overflow: hidden;white-space: nowrap;text-overflow: ellipsis;flex: 0 1 100%;display: flex;align-items: center;justify-content: center;">
QUESTION SUIVANTE
</button>
Expected
<button class="ac-pushButton ac-action-submit" type="button" role="button" aria-label="QUESTION SUIVANTE" tabindex="0" data-ac-submitdata=""QUESTION SUIVANTE"" style="overflow: hidden;white-space: nowrap;text-overflow: ellipsis;flex: 0 1 auto;display: flex;align-items: center;justify-content: center;">
QUESTION SUIVANTE
</button>

Hi - looks like you're using the .NET HTML renderer; this renderer is deprecated and no longer supported. Sorry about that!


Rainstorm/Doppler

Python and R implementations exist. Need to consider if we should make Snakemake workflows for both or just move forward with the most useful/efficient.

The way I've been running it is Python-Rainstorm, R-Doppler (and I think this is the way Prasath ran it too)

I can't assign myself here, but @BrunoGrandePhD you can assign me to this

This was addressed in PR #183 implementing both Rainstorm and Doppler in one snakefile


#174 & #194 Support UTF-8 Encoding in String Method API

This PR aims to support the details stated in #174 and #194.
For future string API optimizations, it deserves another PR. So it is very welcome to help enhance or refactor String API methods!
For now, Goby lang support multilingual by using plain rune and string conversion.

I'm not very familiar with encoding, so I just took a look at test cases and this LGTM. @hachi8833 you can merge it when you think it's ready.

As far as I see, <=> line 225 should also be conscious of rune.
LGTM for the rest. 😄

I noticed that I can merge and then Maxwell can fix <=> after that.
Merging...


feat: impl ToHeader for all typed headers

This adds a to_header(self) -> Result<(HeaderName, HeaderValue)> for every Typed header, as well as anything that can TryInto a (HeaderName, HeaderValue) pair.
This should make typed headers nice to use with builds such as Surf's RequestBuilder, where .header(name, value) could be changed to .header(ToHeader) and also still work with a raw tuple.

I'm not 100% happy with this though - notably to have atrait like this either we need it to be consuming (i.e. consume a typed header) or it has to clone if a tuple is passed.
An alternative would be to have every header impl Into<(HeaderName, HeaderValue)>...

This also uncovers some unpleasantness with the current typed headers - not all of the implement the same methods, one mutates in .value(), some return String or Result<String> instead of HeaderValue, one didn't implement name() either.
I think we should probably consider rolling that all into a trait.

I haven't kept up with the typed header unification trait, but is it ever the case that a typed header would need to add multiple headers?

None of the existing ones do, it seems, although some parse multiple headers if there are multiple, I think.

I believe we talked about this before, but I think a Header type as outlined in https://github.com/http-rs/http-types/issues/286#issuecomment-750890828 might be preferable. This allows us to concretely talk about a Header which contains a HeaderValue and HeaderName, rather than it remaining an un-named concept (ToHeader converts to a tuple, which isn't a header per se, but serves a same role. I propose we actually create a concrete "header" concept).


[Bug]: Test case failing with node js 16.17.0

Version
26.2.2
Steps to reproduce
Here is the following test case which i am testing ( it might be difficult to reproduce but this is the best i can share )
import { HttpChatClientDriver } from '@externals/drivers/http/chat-client'
import { SecretStorageFetchingDriver } from '@externals/drivers/secret-storage/fetcher'
import { ChatSenderUseCase } from '@use-cases/operation/chat-sender'
import { ChatSenderController } from '@controllers/operation/chat-sender'
import { PubSubMessageFixture } from '@test/fixtures/pub-sub/message'
import { Chance } from 'chance'

const chance = new Chance.Chance()

describe('ChatSenderController', () => {
  const useCase = new ChatSenderUseCase({
    httpChatClientRepository: new HttpChatClientDriver(),
    secretStorageFetchingRepository: new SecretStorageFetchingDriver(),
  })
  const controller = new ChatSenderController({ useCase })

  describe('handle', () => {
    describe('When there are multiple records', () => {
      test('The method should complete expectedly', () => {
        const fixture = new PubSubMessageFixture({
          messageCount: 3,
        })
        jest.spyOn(useCase, 'execute').mockResolvedValue()

        expect(controller.handle(fixture)).resolves.toBeUndefined()
      })
    })

    describe('When there are multiple records, some of which cause errors', () => {
      test('The method should complete expectedly', () => {
        const fixture = new PubSubMessageFixture({
          messageCount: 3,
        })
        jest
          .spyOn(useCase, 'execute')
          .mockRejectedValueOnce(new Error(chance.string()))

        expect(controller.handle(fixture)).resolves.toBeUndefined()
      })
    })
  })
})

followed by npm run test -> jest --runInBand
Expected behavior
I expect to see this test case being passed
Actual behavior
The promise is being rejected
error message
/Users/jatinmehrotra/Desktop/classmethod/Clients/xxxxxx/dts-s-api/node_modules/jest-jasmine2/node_modules/expect/build/index.js:134
  const err = new JestAssertionError();
              ^

JestAssertionError: expect(received).resolves.toBeUndefined()

Received promise rejected instead of resolved
Rejected to value: [Error: pxBwE]

Additional context
The following  test case works well node js 14 version
I have tried to post it as stackvoverflow question too https://stackoverflow.com/questions/73621327/updating-nodejs-version-from-14-x-to-16-x-results-in-jest-test-cases-failure
Environment
System :- Os Mac monterey 12.5.1,  node js version 16.17.0, npm version 8.15.0
jest: 26.2.2, types/jest: 26.0.8, jest-extended: 0.11.5, ts-jest: 26.1.4

You need to put together a reproduction I can pull down and run  - a bunch of code in isolation isn't actionable.

That said, you need to await or return the .resolves assertions (e.g. await expect(controller.handle(fixture)).resolves.toBeUndefined()), see https://jestjs.io/docs/asynchronous


Format paths

We often want to split paths, so this seems like an important thing to tackle

easy-ish

Nasty test case: https://github.com/erickt/rust-aster/blob/master/tests/test_item.rs#L56-L63


downloadGNSS.py and raiderDownloadGNSS.py entrypoints are broken

Both downloadGNSS.py and raiderDownloadGNSS.py entry points raise a NameError when run with the --help option:
$ downloadGNSS.py -h
Traceback (most recent call last):
  File "$HOME/mambaforge/envs/RAiDER/bin/downloadGNSS.py", line 8, in <module>
    sys.exit(downloadGNSS())
  File "$HOME/Documents/Code/contributing/RAiDER/tools/RAiDER/cli/raider.py", line 411, in downloadGNSS
    add_out(misc)
NameError: name 'add_out' is not defined

and
$ raiderDownloadGNSS.py -h
Traceback (most recent call last):
  File "$HOME/mambaforge/envs/RAiDER/bin/raiderDownloadGNSS.py", line 8, in <module>
    sys.exit(downloadGNSS())
  File "$HOME/Documents/Code/contributing/RAiDER/tools/RAiDER/cli/raider.py", line 411, in downloadGNSS
    add_out(misc)
NameError: name 'add_out' is not defined

Also, these entry points both point to the same function. Any reason they are duplicated?

Thanks for the catch… they’re duplicated deliberately to maintain backward compatability (i think for SWOT users)

On Dec 20, 2022, at 10:06, Joseph H Kennedy @.***> wrote:
Both downloadGNSS.py and raiderDownloadGNSS.py entry points raise a NameError when run with the --help option:
$ downloadGNSS.py -h
Traceback (most recent call last):
File "$HOME/mambaforge/envs/RAiDER/bin/downloadGNSS.py", line 8, in
sys.exit(downloadGNSS())
File "$HOME/Documents/Code/contributing/RAiDER/tools/RAiDER/cli/raider.py", line 411, in downloadGNSS
add_out(misc)
NameError: name 'add_out' is not defined
and
$ raiderDownloadGNSS.py -h
Traceback (most recent call last):
File "$HOME/mambaforge/envs/RAiDER/bin/raiderDownloadGNSS.py", line 8, in
sys.exit(downloadGNSS())
File "$HOME/Documents/Code/contributing/RAiDER/tools/RAiDER/cli/raider.py", line 411, in downloadGNSS
add_out(misc)
NameError: name 'add_out' is not defined
Also, these entry points both point to the same function https://github.com/dbekaert/RAiDER/blob/dev/pyproject.toml#L47-L48. Any reason they are duplicated?
—
Reply to this email directly, view it on GitHub https://github.com/dbekaert/RAiDER/issues/453, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADWNVR6TZRFNZMT245FQGD3WOHYRNANCNFSM6AAAAAATEY2KVU.
You are receiving this because you are subscribed to this thread.


Make extensions available as CMake components

Currently the extensions to the API are built completely separately. It would be helpful to support building them as components of the library. This would also facilitate transitioning them out of an experimental state.

I think this is closable now after https://github.com/apache/arrow-nanoarrow/pull/517

I think these are already being installed?
https://github.com/apache/arrow-nanoarrow/blob/2d5769bbc30c5ea3814ffcd9e098993ea83a8ec0/CMakeLists.txt#L229
https://github.com/apache/arrow-nanoarrow/blob/2d5769bbc30c5ea3814ffcd9e098993ea83a8ec0/CMakeLists.txt#L298


Adds straight_road_with_crosswalk_integration.xodr map.

🎉 New feature
Related to #46
Summary

Adds straight_road_with_crosswalk_integration.xodr map.

Checklist

[x] Signed all commits for DCO
[ ] Added tests
[ ] Added example and/or tutorial
[ ] Updated documentation (as needed)
[ ] Updated migration guide (as needed)
[ ] Consider updating Python bindings (if it affects the public API)

In our downstream use cases, we confirmed that after changing elementId="11" to elementId="10", the map passed our tests.

@liangfok Can we go ahead and merge this then?


Programmatically set the icon color in a textfield

Hello!
I would like to set the icon property of a textfield in C#. But it is invisible, I think its color has to be set.
How can I set the color of the icon?
Or if it's not the color then what is it?
Thank you!

Works fine. Thank you for your fast help :)


New Term Request: Cinnamycin

Can I request cinnamycin to be added to ChEBI please?
Here is a link for this chemical from Sigma: https://www.sigmaaldrich.com/catalog/product/sigma/c5241?lang=en&region=GB. There are peer-reviewed papers linked in this page.
In addition, here is another paper characterizing cinnamycin biosynthetic gene cluster: http://www.pnas.org/content/100/7/4316
Thanks & best wishes,
Achchuthan

Hi,
An entry for cinnamycin has now been created (CHEBI: 141991).
Best Regards,
Adnan


Stringify int yaml types

Since #86 promotes any integers encoded as strings to integers, this change allows integers to convert back to strings in YAML.
This change also prevents panics as in #108. Fixes #113.
Signed-off-by: Lachlan Donald<EMAIL_ADDRESS>

@lox can you rebase?

@ibuildthecloud rebased.

@lox compilation error
Stderr: # github.com/docker/libcompose/project
project/types_yaml.go:117: undefined: err

This PR probably isn't relevant anymore since #86 was reverted


Fix Makefile: oc login required for docker login

Without oc login, $(shell oc whoami -t) returns an error in _login_to_docker

@slaskawi @mgencur Closing the PR, the problem was just that I was using the wrong command combo after the changes :see_no_evil:.
make stop-openshift start-openshift-with-catalog login-to-openshift prepare-openshift-project build-image push-image-to-local-openshift install-templates test-caching-service-manually

Maybe it would be good to add a single target that would call all the targets you just mentioned?

Yes, maybe something like run-caching-service and run-shared-memory-service. I'll issue a PR later

@ryanemerson @mgencur That sounds good to me? Matbe run-*-locally?

Yeah. sounds good


How to create and run a C# unit testing project

duplicate of #4188

Hello @zolomohan I have created a new PR.
Duplicate of #4188

@zolomohan lets wait until you get your new setup before we take on any new reviews. 👍

@hectorkambow I can take this. Working with a temporary setup, not an issue 😄

Hello, @zolomohan @hectorkambow I wish to enquire about the progress of my PR. Thank you.

Hello, @ahmadmardeni1 @hectorkambow @zolomohan . What's the progress of my PR. Thank you.

Hello @zolomohan I have done the changes you requested. Thank you.

Jumping on this one.

👍 overall nice work - very detailed and straight forward tutorial
Please review and suggestions/changes made by your peer reviewer and/or myself during a final review.
cc @zolomohan please be sure to go through the author index file as well to check for any mistakes and please do not forget to add the new meta image line 👍


Ref Arrays

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: JasonWeathersby:white_check_mark: ryanRfox:x: Jason WeathersbyJason Weathersby seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


README bridgedb-1.1.0.zip inconsistent with jars (and jars names outdated anyway)

The README writes:
bridgedb.jar - always needed.
This includes file, rdb, biomart and BridgeRest drivers.
But the driver is in a separate jar:
$ jar tvf org.bridgedb.jar | grep IDMapperRdb
$ jar tvf org.bridgedb.rdb.jar | grep IDMapperRdb
218 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$1.class
2887 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverClient.class
1170 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverJdbc.class
1201 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverPgdb.class
1788 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb.class

Oh, never mind... already fixed in this commit:
http://svn.bigcat.unimaas.nl/bridgedb/trunk@564

Hi Egon,
Could it be that you're looking at an old README? I updated it a few
weeks ago.
--
Martijn
On 15/02/12 14:38, Egon Willighagen wrote:

The README writes:
bridgedb.jar - always needed.
This includes file, rdb, biomart and BridgeRest drivers.
But the driver is in a separate jar:
$ jar tvf org.bridgedb.jar | grep IDMapperRdb
$ jar tvf org.bridgedb.rdb.jar | grep IDMapperRdb
218 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$1.class
2887 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverClient.class
1170 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverJdbc.class
1201 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverPgdb.class
1788 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb.class

Reply to this email directly or view it on GitHub:
https://github.com/amarillion/BridgeDb/issues/14

Never mind, if you're looking at the README packaged in
bridgedb-1.1.0.zip, it's definitely outdated. But this is fixed in trunk.
--
Martijn
On 15/02/12 14:38, Egon Willighagen wrote:

The README writes:
bridgedb.jar - always needed.
This includes file, rdb, biomart and BridgeRest drivers.
But the driver is in a separate jar:
$ jar tvf org.bridgedb.jar | grep IDMapperRdb
$ jar tvf org.bridgedb.rdb.jar | grep IDMapperRdb
218 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$1.class
2887 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverClient.class
1170 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverJdbc.class
1201 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb$DriverPgdb.class
1788 Thu Feb 24 18:37:06 CET 2011 org/bridgedb/rdb/IDMapperRdb.class

Reply to this email directly or view it on GitHub:
https://github.com/amarillion/BridgeDb/issues/14

Yes, see my comment minutes after the bug report...


Notifications with ACME-Lego (v 4.9.0) not proxyfied through powerdns-admin

Hello all,
After testing last release of ACME-Lego release (which introduces asking to server to send notifys to secondaries servers, I ended up on the following behaviour :  {"msg":"Bad Request"}
reproduced the issue with curl:
With powerdns-admin proxifying  api request to pdns :
curl -k --location --request PUT 'https://myserverip/api/v1/servers/localhost/zones/mytestzone/notify' --header 'Content-Type: application/json' --header 'X-API-Key: mysecretapikey'
Response:
{"msg":"Bad Request"}
ACME-Lego directly to pdns API :
curl -k --location --request PUT 'https://myserverip/api/v1/servers/localhost/zones/mytestzone/notify' --header 'Content-Type: application/json' --header 'X-API-Key: mysecretapikey'
Response:
{"result": "Notification queued"}
After looking at code : https://github.com/go-acme/lego/pull/1679/files
It seems the request is sent with empty body (as also described in  pdns documentation
Workaround : stick to previous ACME-LEGO version, or  add  '{ }' as body (which is also supported by pdns  by the way...)
I'm not able to propose a PR to fix this.. and also wanted to discuss the issue with you before attempting anything...
Best regards,

Hello,
All requests sent to api/v1/servers/localhost are forwarded to PowerDNS directly. You should check on PowerDNS API which format is required (and hence do your test directly on PowerDNS API ~ ie not through PDA).
If you would like that PDA silently handles the empty body for LEGO, you can open a PR to add a decorator that fills the body content when sending to notify.

Hello,   found the issue, and how to solve it, PR incoming:
For the details : The Bad Request is performed by this code :
request.get_json(force=True)  in helper.py
Following  this post https://stackoverflow.com/questions/39451002/ensure-the-post-data-is-valid-json  (second answer) and the Flask documentation,
adding silent=True solves the issue.
I can now have the same response on the notify request than the native pdns API one.
Best regards,

@WhatshallIbreaktoday are you able to test and verify that your provided enhancement is functioning in the default branch now?

Hello,
It completely works (I use the fix on our production).
Best regards,

@WhatshallIbreaktoday excellent, thank you.


Cant set Cross-Origin Cookies

When I tried this plugin along with flutter_webview_plugin it can't able to read or write cookies from the Cross-Origin site. But this works fine in webview_flutter and flutter_inappwebview plugins.

As I know flutter_webview_plugin will be depricated soon or already. I would like to suggest to use webview_flutter.

Yaa I used this plugin with webview_flutter but can't able to get cross-origin cookies. But I am able to get cross-origin cookies when I used flutter_inappwebview for my app.

Could you please share code how you are trying to get cross-origin cookies with flutter_inappwebview?

I added no extra code for using flutter_inappwebview, the plugin itself manages cross-origin cookies. More generally, Bloodline is my app which is built on flutter_inappwebview which manages cross-origin cookies which what I said earlier.

I really don't know how I can help here

Have you ever tried to get cross-origin cookies using this plugin..?!

@varunkumarmedam If you are talking about 'sameSite' value in the cookie, then you are right. For now, this value has not supported yet.

Yaa thanks for the verification.

Any updates?
I'm looking for a way to set SameSite=Strict cookie.
Because Cookie class doesn't have sameSite field, is it still impossible?
Does anyone have any workaround?


Conv1D throws CUDNN_STATUS_EXECUTION_FAILED

Setup:
$ pip install mxnet-cu90==1.1.0

$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Sep__1_21:08:03_CDT_2017
Cuda compilation tools, release 9.0, V9.0.176

$ ls /usr/local/cuda/lib64/libcudnn.so.7.0.3
/usr/local/cuda/lib64/libcudnn.so.7.0.3

Run the following script debug.py:
import mxnet as mx
W_REQ = 'add'
shape = (1, 65536, 1)
ctx = mx.gpu()
kwargs = {'no_bias': True, 'kernel': (1,), 'num_filter': 1}
x = mx.sym.var('x')
w = mx.sym.var('w')
x_grad = mx.nd.zeros(shape, ctx=ctx)
w_grad = mx.nd.zeros(shape, ctx=ctx)
args_grad = {'x': x_grad, 'w': w_grad}
sym = mx.sym.Convolution(x, w, **kwargs)
executor = sym.bind(ctx, grad_req={'x': 'null', 'w': W_REQ},
                    args={'x': mx.nd.ones(shape, ctx=ctx), 'w': mx.nd.ones(shape, ctx=ctx)},
                    args_grad=args_grad)
executor.forward()
executor.backward([mx.nd.ones((1,1,1), ctx=ctx)])
mx.nd.waitall()

Gives the following error:
[06:31:41] src/operator/nn/./cudnn/./cudnn_algoreg-inl.h:107: Running performance tests to find the best convolution algorithm, this can take a while... (setting env variable MXNET_CUDNN_AUTOTUNE_DEFAULT to 0 to disable)
terminate called after throwing an instance of 'dmlc::Error'
  what():  [06:31:41] src/engine/./threaded_engine.h:359: [06:31:41] src/operator/nn/./cudnn/cudnn_convolution-inl.h:242: Check failed: e == CUDNN_STATUS_SUCCESS (8 vs. 0) cuDNN: CUDNN_STATUS_EXECUTION_FAILED

Stack trace returned 10 entries:
[bt] (0) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2a9e78) [0x7f705d9a3e78]
[bt] (1) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2aa288) [0x7f705d9a4288]
[bt] (2) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2a920d1) [0x7f706018c0d1]
[bt] (3) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x262f5e7) [0x7f705fd295e7]
[bt] (4) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x24570bb) [0x7f705fb510bb]
[bt] (5) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x245d7d4) [0x7f705fb577d4]
[bt] (6) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x243e2ed) [0x7f705fb382ed]
[bt] (7) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2442bdb) [0x7f705fb3cbdb]
[bt] (8) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2442db6) [0x7f705fb3cdb6]
[bt] (9) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x243f68b) [0x7f705fb3968b]

A fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.

Stack trace returned 9 entries:
[bt] (0) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2a9e78) [0x7f705d9a3e78]
[bt] (1) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2aa288) [0x7f705d9a4288]
[bt] (2) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x243e594) [0x7f705fb38594]
[bt] (3) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2442bdb) [0x7f705fb3cbdb]
[bt] (4) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x2442db6) [0x7f705fb3cdb6]
[bt] (5) /usr/local/lib/python2.7/dist-packages/mxnet/libmxnet.so(+0x243f68b) [0x7f705fb3968b]
[bt] (6) /usr/lib/x86_64-linux-gnu/libstdc++.so.6(+0xb8c80) [0x7f70d400bc80]
[bt] (7) /lib/x86_64-linux-gnu/libpthread.so.0(+0x76ba) [0x7f70d52a66ba]
[bt] (8) /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7f70d4fdc3dd]

Note that there's no error if W_REQ is changed to 'write'.
Can also be reproduced if I build mxnet from source at commit 5b99b25e5f6ab3a20c7bcf4821a6af0a1a95f823 where Conv1D CUDNN was initially introduced.

Can be also reproduced by the following code:
import mxnet as mx
from mxnet import nd, sym, autograd
from mxnet.gluon import nn, HybridBlock, Block

if __name__ == '__main__':
    ctx = mx.gpu()
    x = mx.nd.ones((1L, 65536L, 1560L), ctx=ctx)
    net = nn.Conv1D(channels=256, kernel_size=2, layout='NCW', use_bias=False)
    net.initialize(ctx=ctx)

    for p in net.collect_params().values():
        p.grad_req = 'add'

    with autograd.record():
        y = net(x)
    y.backward()
    print(net.weight.grad())

with pip install mxnet-cu90 --pre

Tesla V100.
git checkout 5b99b25e5f6ab3a20c7bcf4821a6af0a1a95f823
git submodule update --init --recursive
cp make/config.mk .
echo "USE_BLAS=openblas" >>config.mk
echo "ADD_CFLAGS += -I/usr/include/openblas" >>config.mk
echo "USE_CUDA=1" >>config.mk
echo "USE_CUDA_PATH=/usr/local/cuda" >>config.mk
echo "USE_CUDNN=1" >>config.mk
make -j32

Run python debug.py

Update: CUDNN team is notified for the issue that cudnnFind() is returning algos that will fail.


1.43 run failed with duplicate register on /debug/requests

Sponsors SeaweedFS via Patreon https://www.patreon.com/seaweedfs
Describe the bug
run new downloaded weed with weed -h, got following messages
panic: /debug/requests is already registered. You may have two independent copies of golang.org/x/net/trace in your binary, trying to maintain separate state. This may involve a vendored copy of golang.org/x/net/trace.

goroutine 1 [running]:
go.etcd.io/etcd/vendor/golang.org/x/net/trace.init.0()
	/home/travis/gopath/src/go.etcd.io/etcd/vendor/golang.org/x/net/trace/trace.go:123 +0x1cf

System Setup

OS version
unbuntu 16.04
output of weed version
1.43

Expected behavior
give help messages

Thanks for reporting! fixed now.


conda-forge feedstock

process starts with submitting a PR against https://github.com/conda-forge/staged-recipes with the recipe with LICENSE and meta.yaml.
The process is quite simple, see any of the recent PRs there or the one we had for dandi: https://github.com/conda-forge/staged-recipes/pull/11289/files
After PR gets merged a dedicated repo would get generated for the feedstock, e.g. like https://github.com/conda-forge/dandi-feedstock and bots will submit PRs (to finalize or just merge) upon new releases appearing on pypi (apparently I have missed the one for dandi 0.10.0 heh: https://github.com/conda-forge/dandi-feedstock/pull/15 and it had some tests fail and now logs are gone :-/)

PR: https://github.com/conda-forge/staged-recipes/pull/14015 (Please comment there confirming your status as a maintainer)

done, thank you!

fscacher is now on conda-forge.


GetMany function seems do something  unnecessary

Thanks for your fast JSON parser.
func GetMany(json string, path ...string) []Result {
	res := make([]Result, len(path))
	for i, path := range path {
		res[i] = Get(json, path)
	}
	return res
}

GetMany function just for loop calls Get function. It seems to parse the json string for very loop, and it is not  efficient because of the first loop has got the information about the json string.  Maybe I misunderstand something? Thanks again.

I have the same doubts as you do


Base container with binary only, or url to latest binary to download.

I'd like to build some containers customized to individual languages.  It would make this build process much simpler if there was either a single stage base build consisting of the first stage in the dockerfile now, and running via a CI/CD type action to rebuild when master is updated. Something like codercom/code-server-{arch}:latest.  We could then use it as the first FROM statement and copy the binary from it (but not have to build it each time).
OR
A repeatable way to download the latest code-server binary for linux/OSX as part of an ADD statement in a Dockerfile.
I think either would work, or perhaps there is other options, but really, i just want a consistent way to get the latest code-server binary in an automated fashion.
Bonus points if it's something I can point my own CI/CD to to kick off a rebuild of my container when the binary changes (and I could do that with a base image in docker hub I believe, not sure about just a url to download the binary)

Perhaps hosting the latest binary at something like codesrv.cdr.sh/latest-linux would be good?
Pulling from codercom/code-server:latest should always provide you with the latest image. As for a trigger, I'm not sure the best way to handle that. Any ideas @nhooyr ?

You can just copy the binary from codercom/code-server:1.621. See how the current Dockerfile does it.

As for the trigger, you can already do that. You can run docker pull periodically (like https://askubuntu.com/a/1074874), curl the latest release and see if the version number changed, I believe you can also subscribe to releases via the github API but I might be wrong.

yeah building from codercom/code-server:latest as a base works to grab the binary.  All good, thanks.


Feature/study form

In anticipation of our 11/12/19 meeting, I am hoping to get some new enhancements onto staging for the group to test:
This pull request includes a functioning Study Add/Edit/Delete form. There are buttons on the study detail pages to get to the edit, and a button on the main studies page to get to the add. While the functions in a basic way right now, there are a few missing pieces to be added: Investigators, Image upload, and the Topic multi-select tree.
I have also included a little addition of date to User Admin to aid in sorting new users. I have not fleshed out the user date tracking that we've outlined, just surfaced the data that we already had to aid the team while they are waiting for more features.

Sometimes when adding a new study it will save two copies of itself. I can't consistently reproduce it and I thought I'd fixed it, but it happened with the first save on staging... So I'll need to look at that more closely before we merge this.

Also discovered during the demo that the edit function is misbehaving.


Use junction as symlink type on Windows

The default folder symlink requires elevated permissions to work on
Windows. By using junctions we can remove this requirement and work
functionally the same way.
This should resolve the issue in #35

Thanks!

@alexanderGugel no probs! Thanks for merging

This PR causes a problem for me as following
[INFO] ><EMAIL_ADDRESS>build-layout-engine E:\w\araqne\unstable\webfx\araqne-webconsole\src\main
[INFO] > node node_modules/.bin/webpack --config ./resources/WEB-INF/script/layoutEngine/webpack.config.js
[INFO]
[ERROR] module.js:341
[ERROR]     throw err;
[ERROR]     ^
[ERROR]
[ERROR] Error: Cannot find module 'E:\w\araqne\unstable\webfx\araqne-webconsole\src\main\node_modules\.bin\webpack'
[ERROR]     at Function.Module._resolveFilename (module.js:339:15)
[ERROR]     at Function.Module._load (module.js:290:25)
[ERROR]     at Function.Module.runMain (module.js:447:10)
[ERROR]     at startup (node.js:146:18)
[ERROR]     at node.js:404:3
[ERROR]

ied is creating junctions also for symlinks in node_modules/.bin, but junctions can only be created for directories. So the junction points to invalid target - the target is a file.

but junctions can only be created for directories.

See https://technet.microsoft.com/en-us/library/cc753194.aspx

Bummer, so there is no way to get around the administrator requirement then, as we'll always need regular symlinks for the .bin files?

I read somewhere that PowerShell has a su command (also runas is somthing I used to use) for elevated privileges... There's probably a .NET API for this as well. I've only been skimming/lurking this project so I'm not entirely sure how all of this works at this point in time.
Refs:

http://superuser.com/questions/42537/is-there-any-sudo-command-for-windows (perhaps some other possibilities)

Bummer, so there is no way to get around the administrator requirement then, as we'll always need regular symlinks for the .bin files?

Don't throw out the baby with the bathwater, I think junctions for the directories was the right choice. There's no reason .bin files must use the same technology as the module version links.
I think npm's behavior for .bin scripts is the best here, and would like very much if ied adopted it:

create .cmd file for Windows native tools (cmd, powershell, etc)
create shell script (without extension) for Cygwin and MSYS


When sending, have a max button to send the whole balance (sweep wallet)

In the send screen, it would be useful to have a MAX button that autofill the sum amount of all the wallet coins.

Yes. It's something we've discussed. It's a bit intricate to make work as you change the feerate though.

Yes. It's something we've discussed. It's a bit intricate to make work as you change the feerate though.

yes, i think if max has been selected (and until send value not changed by the user) the output amount might be implicitly (total - fee)

This was brought up today again. A path forward could be to have the recovery command logic be moved to a "sweep" helper and add another command, sweep, that takes a single address and sweeps the whole balance there through the primary path.
The integration in the GUI would be more involved. If sweep was used, it should not let the user add more destinations. Nor select coins actually. @edouardparis wdyt?

Adding the daemon label as ideally i think the sweep logic should be in the daemon and tested using the functional tests.

Recovery panel could be renamed sweep and the primary path added to the list of recovery paths.
Wording can be:
2 paths are accessible next block, please select one

The primary path

Signatures from Ed and Antoine can move 11 coins totalling 1.2334 BTC

Recovery path:

Signatures from Ed can move 4 coins totalling 0.6244 BTC

That seems reasonable and pretty straightforward.
------- Original Message -------
On Thursday, July 13th, 2023 at 2:48 PM, Édouard @.***> wrote:

Recovery panel could be renamed sweep and the primary path added to the list of recovery paths.
Wording can be:
2 paths are accessible next block, please select one

The primary path

Signatures from Ed and Antoine can move 11 coins totalling 1.2334 BTC

Recovery path:

Signatures from Ed can move 4 coins totalling 0.6244 BTC

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

sweep, that takes a single address and sweeps the whole balance there through the primary path.

Do you mean to move in batch one to one or consolidate many to one?

See also https://github.com/wizardsardine/liana/pull/821.

Relevant: https://github.com/wizardsardine/liana/pull/863#discussion_r1423661296

I'm not sure. Any opinion?

I think we could do both. Having the max button on the Send page is intuitive I think and users may expect to see that option there.
We could have the max button even for multiple recipients: for a given recipient, clicking the max button would return the max available for that recipient after taking into account the other recipients.

Sounds good. Then let's do the max button first. We can track changing the recovery screen for a sweep screen into another issue.
I think you have access to the beta of Liana Lite (/Online)? There is a max amount there if you want to play with it before starting with the implementation.

We can track changing the recovery screen for a sweep screen into another issue.

This is now in issue https://github.com/wizardsardine/liana/issues/994.


Auto driver

Follows a linear path between waypoints with a trapazoidal velocity profile.
It will follow a trapazoidal velocity profile to stop before the next waypoint if it is holding a ball otherwise will not stop
PR stack:

#24
#25 :point_left: this PR

this now just follows a path of waypoints, no stopping, we will build on top of this adding more states later


⚠ SceneTime has degraded performance

In 8f14663, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1560 ms

Resolved: SceneTime performance has improved in 64c780d after 10 minutes.


Add SLAS setup command

When you're first setting up SLAS for use with PWA Kit (or another headless storefront) it might be nice if there was a fast path "setup" (or similar) command that asked you for the minimal amount of information needed to get started.
It might work something like this:
$ sfcc-ci slas:setup

> What instance will you use? (zzrf_sbx, zzrf_dev, zzrf_stg, zzrf_prd) zzrf_001
> What sites will you use? (RefArch, RefArchGlobal) RefArch
> Will you use it with PWA Kit? (Y/n) Y
> Will you use it with SFRA? (Y/n) Y
> Do you want the default shopper scopes? (Y/n)

Done. You're Client ID is aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa.

Answering this way would create a tenant like this:
{
    "instance": "zzrf_001",
    "merchantName": "blank",
    "description": null,
    "contact": null,
    "emailAddress"<EMAIL_ADDRESS>    "phoneNo": null
}

And a client like this:
{
    "clientId": "aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa",
    "name": "Retail React App",
    "scopes": [
      "sfcc.shopper-myaccount.baskets",
      "sfcc.shopper-baskets-orders.rw",
      "sfcc.shopper-baskets-orders",
      "sfcc.shopper-categories",
      "sfcc.shopper-customers.login",
      "sfcc.shopper-customers.register",
      "sfcc.shopper-discovery-search",
      "sfcc.shopper-gift-certificates",
      "sfcc.shopper-myaccount.addresses.rw",
      "sfcc.shopper-myaccount.addresses",
      "sfcc.shopper-myaccount.orders",
      "sfcc.shopper-myaccount.paymentinstruments.rw",
      "sfcc.shopper-myaccount.paymentinstruments",
      "sfcc.shopper-myaccount.productlists.rw",
      "sfcc.shopper-myaccount.productlists",
      "sfcc.shopper-myaccount.rw",
      "sfcc.shopper-myaccount",
      "sfcc.shopper-product-search",
      "sfcc.shopper-productlists",
      "sfcc.shopper-products",
      "sfcc.shopper-promotions"
    ],
    "redirectUri": [
      "http://<IP_ADDRESS>:3000/callback",
      "http://localhost:3000/callback",
      "https://*.mobify-storefront.com/callback",
    ],
    "channels": ["RefArch"],
    "isPrivateClient": false
  }

Where clientId is a generated UUID.

This is a great idea

Thanks for suggesting @johnboxall ! I like the approach for adding meaningful interactive commands like this one. Do you happen to have a PR for this at that time? 😄


share all workspace reports?

Can I share all reports or all dashboards in my workspace with external users instead of sharing a single report or dashboard?  Or how can I share published APP to a user outside our origination?

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

ID: 21f47742-00cf-eac0-7880-18eb5e988003
Version Independent ID: 0010dce8-b1e0-b1b4-9840-ff2c3e32b53e
Content: Share Power BI dashboards and reports with coworkers and others - Power BI
Content Source: powerbi-docs/service-share-dashboards.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @maggiesMSFT
Microsoft Alias: maggies

Hi, @Ccccche -- Check out template apps. They may be just what you're looking for. https://docs.microsoft.com/en-us/power-bi/service-template-apps-overview


Alternative implementation of ThreadPoolWait.

ThreadPoolWait implement functionality of ThreadPoolWait .RegisterWaitForSingleObject, with three capability:

To remove a particular wait handle and cancel its operation
To show all wait handles queued in the system,
and 3. To show long continuation (not normal).

The old implementation uses a customized threadpool. The new implementation uses ThreadPoolWait.RegisterWaitForSingleObject.
One potential performance issue is that:
ThreadPoolWait.MaxSpinMs = 200 ms.
This is the time between the callback is called, and the registeredWaitHandle is put into the system. I found that if I set
ThreadPoolWait.MaxSpinMs = 50 ms.
Sometime, the system will show it is unable to grab the registeredWaitHandle in 50ms. I can't reason why it takes so long for the
ThreadPoolWait.RegisterWaitForSingleObject to get back. In slower system, we may need to use a even larger wait.

Great stuff! I have some comments in the code, please check them out.

In the test run https://ci.appveyor.com/project/zhuweirong/prajna-rbpft/build/<IP_ADDRESS>9, though it's successful, it seems the test time has significantly increased (compared to latest merged pull request: https://ci.appveyor.com/project/zhuweirong/prajna-rbpft/build/<IP_ADDRESS>). (I'm running it again to see if it's just a glitch).

The 2nd run failed: https://ci.appveyor.com/project/zhuweirong/prajna-rbpft/build/<IP_ADDRESS>0, I'm re-running it again.


Improvement/171/panic with type names in stable rust

Closes #171.

Hm, since this moved from slide-rs, I'm not sure who should be reviewing / approving / merging: (https://github.com/amethyst/shred/issues/173).

bors try

bors r+


update links from master to main

What this PR does / why we need it: this PR updates the links so they point form master to main
Which issue(s) this PR fixes:

What type of PR is this?: /kind documentation

Special notes for your reviewer: NONE
Does this PR introduce a user-facing change? Then add your Release Note here: NONE

Documentation: NONE

Signed-off-by:  steled<EMAIL_ADDRESS>

closed this one, opened a new one with working dco: https://github.com/kubermatic/mla/pull/156


Unsound in presence of other derive/attribute macros

Because Ouroboros blindly forwards user-provided attributes to the inner type, those attributes can cause the type's invariants to be broken. For example:
#[self_referencing]
#[derive(MaliciousMacro)]
struct Foo { /* ... */ }

Ouroboros' output will place the #[derive(MaliciousMacro)] on the actual type inside the private module, giving it access to the internals and potentially causing unsound output.

Actually, I just realized this doesn't even need a malicious macro to occur. Take this code, which contains a use-after-free:
use ouroboros::self_referencing;
use std::fmt::{self, Formatter, Debug};
use std::cell::Cell;

struct S(Cell<Box<i32>>);
impl Debug for S {
    fn fmt(&self, _f: &mut Formatter<'_>) -> fmt::Result {
        self.0.set(Box::new(0));
        Ok(())
    }
}

#[self_referencing]
#[derive(Debug)]
pub struct X {
    x: S,
    #[borrows(mut x)]
    y: &'this mut i32,
}

fn main() {
    let x = X::new(S(Cell::new(Box::new(5))), |r| r.0.get_mut());
    println!("{:?}", x);
    println!("{}", x.borrow_y());
}

Thanks for catching this, this is pretty serious. It looks like the only safe thing to do would be to mimic the behavior of certain derive macros in a safe way inside the ouroboros macro, and error out when an unsupported derive macro is found. I'll start working on this when I get some time.

Finally got around to fixing this with 0.11.0, although only Debug, PartialEq, and Eq are implemented at the moment.


checker.py crashes if the ServiceProviderConfig endpoint returns a 401 status code

Hello :wave:
Context
I ran the following command:
$ poetry run python ./scim2_tester/checker.py http://MY_SERVER/scim

I got the following error:
Traceback (most recent call last):
  File "/home/florentpro/projects/scim2-tester/./scim2_tester/checker.py", line 104, in <module>
    results = check_server(scim)
              ^^^^^^^^^^^^^^^^^^
  File "/home/florentpro/projects/scim2-tester/./scim2_tester/checker.py", line 82, in check_server
    for resource_type in resource_types:
TypeError: 'Response' object is not iterable

Expected behavior
I would have expected a proper error displaying what is wrong with my server, which was that the /ServiceProviderConfig returned a 401 status code with no authentication.
I wonder if the server is expected to return the result when not authenticated or can reject the request.
If I have time, I can take a look and propose a fix, but currently I am a bit in a rush

Thank you for your report. Indeed, the project is still young and most error case are not covered at the moment. It should indeed correctly handle unexpected error statuses.

I just released 0.1.4 which fixes this. Can you confirm me it behaves the way you expect it?

It indeed solves the issue, thanks!
I have other surprising problems, unrelated to the issue here, and was present before the update. You can close this issue, I may bother you in another one :)


Fix http datastore fqdn address

In case the http datastore address has a port number, the datastore address should contain both the hostname and the port.

LGTM, thanks @shjala ! Could you please add comment from your commit to the change, so it won't be gone if we rewrite commit? I think it's a useful comment for people :)

I'm not sure what you mean, this is the commit message, should I add more to it?
Fix http datastore fqdn address
In case the http datastore address has a port number, the datastore address
should contain both the hostname and the port.

Signed-off-by: Shahriyar Jalayeri<EMAIL_ADDRESS>

@uncleDecart got it, added.


Revamp LIFF with FE framework

We may consider solutions like svelte:
https://g0v-slack-archive.g0v.ronny.tw/index/channel/C2PPMRQGP/2019-04#ts-1555954059.013500
This will allow us to put constants in build env so that we can remove the current search-replace based deploy script, which is pretty awkward.

We have discussed about server-rendered LIFF here:
It makes more sense and is easier to implement than FE framework + API. It should be faster as well, since we don't need 2 RTTs over mobile network between user's cellphone and LINE bot server.
Previous discussions:

202020108 https://g0v.hackmd.io/@mrorz/HJt2rymxL#server-served-LIFF

Design doc here:
https://g0v.hackmd.io/860RnxUGTi6z2Kca6ojAbg


Positive Prompt (Metadata) Incorrectly Read

it seems there is something going on. sometimes the prompt is read incorrectly / not complete other times it is empty:

give me the metada with www.metadata2go.com so I can test

give me the metada with www.metadata2go.com so I can test

here we go: https://www.metadata2go.com/result#j=a85cf75a-5635-4e87-a5a5-63048320f526

seems to be fixed.


Add anyobject pseudo type

This is needed for fts::search. It mimics implementation of anytype.
Personally I don't think this is good language design, though. We are basically implementing generics with constraints and now adding new pseudo types for different constraints we need.
With this PR, we have two such constraints: anytuple and anyobject (anytype is just plain generic type argument with no constraints).
Proper generics would look like:
create function fts::search<T: schema::ObjectType>(
     object: T,
     ...
) -> Tuple<
    object: T,
    ...
>

polymorphic functions aren't user-definable

Oh, that is good news - it means we can change it without breaking user code, right? No one would depend on the reflection of our functions?

No one would depend on the reflection of our functions?

Query builders (ours own TypeScript one) do. But that should be fine.

Proper generics would look like:

Yeah, we know. The problem is that postgres polymorphism is very limited and we're basically building around its limitations. We might be able to do better eventually, deprecating all of the any* pseudotypes, but that's far into the future.

The problem is that postgres polymorphism is very limited

Ah, I see. Our functions mostly compile to Postgres functions, so we are inheriting those limitations.


Battery compartment improvement

The original PIKON design calls for a very specific battery pack, the NanoWave 3 5000mAh USB-C & A Power Bank.
That battery pack seems to only be available from British resellers, as I haven't found it on US Amazon or any other US-based seller.
And using a battery pack like that also requires an awkward USB-C to USB-C cable that sticks out the top of the camera.
I would like to see if there's an easier way to non-destructively tie an 18650 battery cell into a charge/5V power circuit, and then inject power through that circuit into the Pi's 5V rail.
This circuit would ideally fit in the grip alongside the battery (with the battery and PCB fitting into a channel that is printed into the case itself). I haven't looked too much into 18650 circuits for the Pi, but I know many exist as HATs—that wouldn't work in this build, it would need to be separate and built into the grip area.

Instead of using a powerbank or any other unreliable solution. The integeration of Open Upcell which is a USB Type C power delivery cum BMS capable of powering Raspberry pi that is slightly larger than 18650 cell will be a great choice. The cherry on the top is that it's OPEN SOURCE :)
We are currently running a Crowd supply campaign for the same. Please do check it out and make powering up your raspberry pi projects much more easier than ever before.
Crowd Supply page: https://www.crowdsupply.com/sikra/open-upcell
Hackaday page:  https://hackaday.io/project/188459-open-upcell

@padmalaya-rawal This seems pretty cool to try out. Given I'm mostly a userland software guy, I hope you consider publishing (detailed?) instructions on how to make the stock Raspberry Pi OS show the battery level in GUI, and how to get it from the remote command line as well -- i.e. how to hook up the relevant connections to a Pi4 and a CM4 dev board. Otherwise this looks like what I was looking for.

@ivucica For sure. I will be releasing detailed tutorials on https://electroboffin.com/ on how to use it or various applications. Also we will update on crowdsupply & hackaday page as well. Please consider subscribing us on Crowdsupply so that you won't miss any update :)


Typo in documentation

The value in the code is different.

Pull Request Test Coverage Report for Build 208

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.789%

Totals

Change from base Build 207:
0.0%

Covered Lines:
601

Relevant Lines:
849

💛  - Coveralls

Thanks!


Latest 0.1.8 YI Home Camera 1080P AI+ Bricked?

I think my camera is bricked?! I updated 0.1.8 thru Web Interface.. All tested good played with it for a few minutes.. Then i used Yi App to shutdown the camera then unplug the power.. few hours later i went to set it up for its final location then the camera wont boot back on.. it stayed yellow light.. although i set the status light off on Yi App.  And my network cant detect it.  I tried removing the microsd. booting without microsd.. same thing.. stayed yellow light.. i tried the unhack procedure.. same thing.. yellow light.. i think the last thing i didnt try is unbricking using serial port.

LOL.. i found the issue.. i changed the cable with a longer one it didn't work.. i change it back to its original cable it worked again.. i used the longer cable on my Yi Dome U Pro it worked. I don't know why it wont work on YI Home Camera 1080P AI+


merge from template-arlac77-github

package.json

chore(deps): add ^5.3.0 remove ^5.2.0 (devDependencies.ava)

:tada: This PR is included in version 1.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update version: EtternaProject.EtternaGame version 0.72.3

Pull request has been created with WinGet Automation 🪟📦🤖
Microsoft Reviewers: Open in CodeFlow

@wingetbot waivers Add Validation-Executable-Error

https://github.com/microsoft/winget-pkgs/pull/131556


Update AMP_for_E-Commerce_Getting_Started.html

Suggestion for consistent text indention, due there have been little inconsistency

@dev7ch Thanks for your pull request, however it seems to me that the indentation is consinstent with other files like https://github.com/ampproject/amp-by-example/blob/master/src/10_Introduction/_Hello_World.html

@kul3r4 do you want me to close this PR? ( i see, AMP is intended with 2 spaces every where)

@dev7ch if we are using 2 spaces everywhere I don't think we should change one file to 4, so I am in favour of closing it.

sure, i ll close it, but you could may have a look screenshot, this show you what i meant and what is inconsistent.
btw. why do you use 2 Spaces in AMP?


run irctest in travis

Let's see if this works!

Looks like the Python version in the testing image is too old.
Anyway, there were unanswered questions about how to coordinate upgrades to irctest with upgrades to Oragono --- let's forget about this for now.

Error for posterity. This is Ubuntu Xenial, which ships Python 3.5. F-strings were introduced in 3.6.
./test.py irctest.controllers.oragono
Traceback (most recent call last):
  File "./test.py", line 8, in <module>
    from irctest.cases import _IrcTestCase
  File "/home/travis/gopath/src/github.com/oragono/oragono/irctest/irctest/cases.py", line 13, in <module>
    from .irc_utils.sasl import sasl_plain_blob
  File "/home/travis/gopath/src/github.com/oragono/oragono/irctest/irctest/irc_utils/sasl.py", line 6
    return f'AUTHENTICATE {blobstr}'
                                   ^
SyntaxError: invalid syntax
Makefile:9: recipe for target 'integration' failed


README: remove references to non RBAC configurations

Fixes #341
Heptio's policy is that all k8s clusters should be running RBAC. In
support of that policy remove all the non RBAC deployment examples and
documentation.
Signed-off-by: Dave Cheney<EMAIL_ADDRESS>

@Bradamant3 thanks for your comments

"RBAC must be enabled on your cluster."

Done

And would it be clearer at this point if the remaining YAML files were renamed to remove the "-rbac" string?

yes, but i can't do that at the moment because joe is on leave and i cannot update the short link in the quickstart guide :(


Update Qulice to 0.18.19

Outdated Qulice prevents migration to Junit 5, since it gives a lot of false positive PMD.DefaultScope.

@paulodamaso Can you put it in scope?

@0crat assign @andreoss

Done in #1086

Done in #1086

@sereshqua/z please review this job completed by @andreoss/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@sereshqua/z please review this job completed by @andreoss/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

@andreoss Problem reporter and problem solver are the same person

@andreoss Problem reporter and problem solver are the same person

@0crat quality acceptable

@0crat quality acceptable


TCP checksum error. New code, based on JSocket Wrench code.

Running my experiments I found out that floodlight's checksum algorithm was returning a wrong value in some cases. After a lot of testing I decided to use another algorithm to compare results and did just that after adapting the JS Wrench code to the TCP class in Floodlight.
Please take a lot a the code and the Test class also, there are three example packets there that always resulted in wrong checksum value for me. There are also other example of values and byte[] describing several other packets that I had the same problem here ( https://www.dropbox.com/sh/jmix6ew76sh35ig/AAD9tgk5z5PdN9HBfnuW6GYta?dl=0 ).
The topic at floodlight-dev list in which we are discussing this possible bug is: https://groups.google.com/a/openflowhub.org/forum/#!topic/floodlight-dev/_u1pc_Rbr0A

@moisesber Thanks!


Cleaned area sensor glitches to zero and back

Describe the bug
I often notice that the sensor.NAME_cleaned_area glitches to zero and back, usually during the night.
To Reproduce
Set up the integration, wait for the error to occur, can take a day or two.
Expected behavior
The area cleaned sensor should stay at the same value until another cleaning session has started.
Screenshots

Additional Information (please complete the following information)

Model Name [e.g. dreame.vacuum.p2028]: l10s pro
Firmware Version [e.g. 1156]: 4.3.3_2303
Home Assistant Version: 2023.10.3, plugin version 2.0.6b
Configuration Type [With or without map support]: With map support
Errors or warnings shown in the HA logs (if applicable):  Have not checked, just restarted HA and log was removed. If needed, I can upload tomorrow.

When device is installed to the Dreame cloud, integration relies on a mqtt server for getting the updated values unlike the Mi Home cloud which requires periodically sending requests for getting the updates.

So the Dreame cloud integration would work better you expect? Is it also in the same state as Mi Home in terms of reliability? If so, will complete reconfiguration be needed upon switching from Mi Home to Dreame?
Is this something that can be fixed on the integration's end, or must this be done on the Vacuum's end (Dreame/Mi Home)?
Thanks for the quick response by the way!


"Works For You" not loading works

Logged in as admin. The title of the page and the loader bar show but no works ever load. Robert had mentioned when they did load last week he only got one column of works instead of two.

this is a martsy view @craigspaeth


Sql Alchemy compatibility

I did not find any issue within this repository covering this, which makes me wonder how one could successfully run the docker images after build.
Airflow 1.10.9 does not support SQLAlchemy higher than 1.3.15!
here is the corresponding issue: [/apache/airflow/issues/8211]
Airflow 1.10.10 does support higher SQLAlchemy version!
Within the master branch Dockerfile Airflow 1.10.9 is applied and SQLAlchemy> 1.3.15 this lead to non working webserver!
FIX as stated in the closed issue:

Keep Airflow 1.10.9 and put && pip install SQLAlchemy=='1.3.15' \ into the Dockerfile  OR
Build Airflow 1.10.10 for compatibility with newer SQLAlchemy versions

This also addresses #535.


using regular form with post method throws 404 Error: Page not found: /api/v1/oauth on Cloudflare pags/workers bug

Environment

Operating System: Darwin
Node Version:     v20.14.0
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, modules, app, runtimeConfig, build, nitro
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://github.com/samk-dev/nuxt-appwrite-node
Describe the bug
I am working on a starter that uses Appwrite as backend and Nuxt/Nitro
I have a form in pages/login.vue that has a normal form that uses oauth. When submitting the form only in Cloudflare pages/workers throws 404 Error: Page not found: /api/v1/oauth on Cloudflare pags/workers bug. Everything works fine locally and in a docker deploy on my home server and also works great.
I have tried with the domain and just /api/v1/oauth
<form :action="`${config.public.appDomain}/api/v1/oauth`" method="post">
   <input type="hidden" name="provider" value="spotify" />
  <UButton type="submit" variant="soft">Sign up with Spotify</UButton>
</form>

Additional context
Cloudflare logs doesn't show any error. All requests has 200 status
Logs
No response

Would you see if you can reproduce this in pure Nitro (reproduction sandbox), and if so, raise there? 🙏

Hey Daniel, sure. I'll make a reproduction with pure nitro and see what happens. Thanks 🙏

Hey @danielroe, the problem disappeared after updating to 3.12.2 can I close the issue?

Great!! 🙌


Kar paper scraping

-TO ADD COMMENT-

parking this until "Scraper tests" PR is done.

I don't like these custom db queries in the tests, please write a method in the papers model with its own test and then use that method to test your scraper in the future please, otherwise lgtm.

My view is that testing should be independent from other modules, to avoid circular dependency cycles and weird conversations later on (e.g. why does this change in papers-model break the scraper tests?). It would induce a bit of test-code coupling, which for the scraper tests, which are already limited, i think is avoidable and looks readable


refactor(LoginCard): redesign page

Description of the changes

Move login to auth/login
Add icons
Add dark support in LangPicker
Add global scss(style/app.scss)
Redesign login page

Same my project design from: https://vadmin.heroui.net/auth/login

The design looks great! I haven't looked at code yet. Can you also fix the CI failure and request my review after the code is ready? Thanks!

Can you also fix the CI failure.

The CI check failed is not caused by this PR, but I still try to fix it, and more testing may be needed, because I don’t know whether the repaired lint is available.

The log shows that it is prettier who is not happy (it is in incremental mode so ....) you can fix it with
yarn run prettier -w .

I just updated your branch to keep to date with master in Github UI as well.

It’s probably a wrong auto fix then
On Sat, Mar 13, 2021 at 20:25 Yan Lee @.***> wrote:

@AGDholo commented on this pull request.
In src/common.ts
https://github.com/chafan-dev/chafan-frontend/pull/80#discussion_r593839812
:

@@ -1,2 +1,2 @@
-export const URLRegex = /https?://(www.)?[-a-zA-Z0-9@:%.+~#=]{1,256}(.[a-zA-Z0-9()]{1,6})?\b([-a-zA-Z0-9()@:,%+.~#?&//=]*)/gi;

IDE auto fix, I don't know
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/chafan-dev/chafan-frontend/pull/80#discussion_r593839812,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWWDRUINCDDW6A4RBQV7TDTDQ3FFANCNFSM4ZD755OQ
.

Should I rollback all code or fix some review block?

Should I rollback all code or fix some review block?

I typically will just add a new commit addressing the issues. No need to roll back. Commit history is not so important right now.

The log shows that it is prettier who is not happy (it is in incremental mode so ....) you can fix it with
yarn run prettier -w .

No useful, yarn check:eslint have some problems too

The log shows that it is prettier who is not happy (it is in incremental mode so ....) you can fix it with
yarn run prettier -w .

No useful, yarn check:eslint have some problems too

I see. Then just let me add some commit later to this branch to address the CI issue.

Thanks!


Does COOP work with GCC?

I see MSVC project files all the way so I'm a bit afraid. Does COOP MSVC only? If it's not then how to use COOP with GCC? Thanks.

Hi altPOCA, the cross compiling of the library is not part of the open-source yet. I hope to extend it in the future.

Hi altPOCA, the cross compiling of the library is not part of the open-source yet. I hope to extend it in the future.

No. I mean MinGW GCC on Windows. There is no cross compiling.

Should I send you an email directly? You seems not very active here. I think I shouldn't because I think the email address you provided is for business and I'm only a hobbyist.
I just confirmed your project is MSVC only by trying to compile it with MinGW GCC. I failed to compiler under MinGW GCC. If you really don't know what MinGW GCC is then you could download it here, of course for Windows:
https://jmeubank.github.io/tdm-gcc/download/

Thanks for letting me know,
I plan (hopefully) to continue developing this project in about half a year from now. till then - unfortunately, I can't give too much support. So I'll keep this post open, up until I get there and fix it. :)


fix a wrong description in function page.screenshot

When options.clip.height===0,  ('options' is Page.screenshot's params).  The error description thrown by Function assert is 'Expected options.clip.width not to be 0.' It's misleading.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here with @googlebot I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.


Add Rubocop configuration file and process automated cops.

For some reason CI didn't run, can you please commit --ammend and push -f or something to try to make it run

Sounds better :)


typo

Hi @davidbetz, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@davidbetz : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

sign_off

Terribly sorry @davidbetz, but I used our automation tool wrong and it looks like there is a merge conflict now. Could you potentially update your PR? I'll sign off properly and quickly after you've sent it. :)
#hold-off


CodeQL Security Warning

The GitHub CodeQL security scanning tool is reporting a security issue in this lib for a repo where I include it as a dependency.

Incomplete string escaping or encoding
A string transformer that does not replace or escape all occurrences of a meta-character may be ineffective.

It seems to be referencing the code here:
https://github.com/c4spar/deno-cliffy/blob/99ccc2a38d319d06cd0c8e80a22fdc3f9dde8a1d/command/completions/_zsh_completions_generator.ts#L257
A screenshot of the warning:

.

@grempe sry for late reply!
Thx for the report. I'm aware that this code should be improved and is not working correctly in all cases.
But i think it's not a crytical security issue because this code is only used when the completions script from the completions command is used and the values passed to the script are comming from the cli author anyway.
It could be critical if the values are passed from a third party library to the command.
But i agree we should fix it.


Adding padding not in line with values setting to zero

This causes unwanted padding on using the ol, ul for other use cases than lists.

That's pretty much what .list-unstyled is for. You can also use padding utilities (in that case .pl-0) te remove padding.
I don't think default padding for list should be removed.

That's pretty much what .list-unstyled is for. You can also use padding utilities (in that case .pl-0) te remove padding.
I don't think default padding for list should be removed.

Thanks for your feedback. It just feels wrong that it is placed in the reboot file. I would suggest that these settings are removed. Or in any case, move to "type" settings or somewhere else so when a lists is within a  it gets the padding, or by a class to give lists some "default" styling.

FYI, we're going the opposite way in v5: _type.scss only concerns added classes. What's in _reboot.scss are styles applied to HTML elements.
As a matter of fact, this rule currently only normalizes the value, but removing it won't change anything to your issue since browsers apply an equivalent default styling. You may want to check on browserdefaultstyles.com.
We may use AndyBell's reset:
ul[class],
ol[class] {
  padding: 0;
}

but it would probably lead to regressions and unpredictable behaviour, since it'd remove padding if we use a utility class to only change, say, the list's color.
So I'd recommend you either override this or you simply use the provided utilities.


Slider width occasionally renders too wide in Safari/Webkit

Apologies in advance if I've done something wrong, but I'm wondering if I've found a bug.  Everything works beautifully on Chrome, but on every 3rd or 4th page load in Safari/Webkit, the slider doesn't seem to respect the width option (currently set to 100% of its container) and renders way too wide.
I've tried adding in a .refresh() on mounted, but that doesn't seem to work.  Attached are two screenshots showing the rendering issues (correct and incorrect).  Any thoughts or pointers would be appreciated!

You can call the refresh method after switching pages.
this.switchPage()

Thanks!  Sorry, I should be a bit more clear -- I'm not actually switching pages, this happens on initial page load (but only occasionally, i.e. every 3rd or 4th time the page is loaded initially).
I was guessing a refresh was needed - cool.  What's the optimal time to call after initial load to assure no issues?  I've tried adding it under a mounted hook, but it still seems too soon.  Is there another event that fires after mounted? (something like the equivalent of jQuery's .ready())?

You should make sure to call the refresh method after the container is fully loaded.

Ok cool - will make some changes and report back shortly (and close the issue!)

@noahlh im having your exact same issue, did you get around it?

oh crap I realized I did the bad-contributor thing and never followed-up here.  I don't remember if this solved the issue -- I have a feeling we isolated it to being a Safari bug.

I got aorund it calling $refs.mySlider.refresh() inside a setTimeout()

It is no longer necessary to call refresh in the latest version.