permissions issue when compiling assets

I have a GitHub Action configured using PHP 8.2 container
on:
  push:
    branches:
      - prod
      - dev
  pull_request:
    branches:
      - dev
      - prod
    types:
      - opened

name: CI
jobs:
  phpunit:
    runs-on: ubuntu-latest
    container:
      image: kirschbaumdevelopment/laravel-test-runner:8.2

    services:
      mysql:
        image: mysql:8.0
        env:
          MYSQL_ROOT_PASSWORD: password
          MYSQL_DATABASE: wahlocal_test
        ports:
          - 33306:3306
        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3

    steps:
      - name: Checkout code
        uses: actions/checkout@v2

      - name: Cache dependencies
        uses: actions/cache@v2
        with:
          path: ~/.composer/cache/files
          key: dependencies-laravel-${{ matrix.laravel }}-php-${{ matrix.php }}-${{ matrix.dependency-version }}-${{ matrix.os }}-${{ hashFiles('composer.json') }}

      - name: Install composer dependencies
        run: |
          composer install --no-scripts
        env:
          COMPOSER_AUTH: '{ "github-oauth": { "github.com": "${{ secrets.PAT }}" }, "http-basic": { "wire-elements-pro.composer.sh": { "username": "xxx", "password": "${{ secrets.WIRE_ELEMENTS_PRO_KEY }}" }, "nova.laravel.com": { "username": "xxx", "password": "${{ secrets.NOVA_KEY }}"}}}'

      - name: Prepare Laravel Application
        run: |
          cp .env.ci .env
          php artisan key:generate

      - name: Install front-end dependencies
        run: |
          npm install
          npm run build

      - name: Run Testsuite
        run: vendor/bin/pest tests/

and I've been running into this error when building the assets in Vite
> build
> vite build

vite v4.4.9 building for production...
transforming...
✓ 1[7](https://github.com/1stevengrant/site.com/actions/runs/8829879107/job/24241847106?pr=199#step:7:8)92 modules transformed.
rendering chunks...
EACCES: permission denied, mkdir '/__w/site.com/site.com/public/build'
✓ built in 11.[9](https://github.com/1stevengrant/site.com/actions/runs/8829879107/job/24241847106?pr=199#step:7:10)2s
error during build:
Error: EACCES: permission denied, mkdir '/__w/site.com/site.com/public/build'
Error: Process completed with exit code 1.

What's interesting is that it doesn't happen with the 8.1 test runner container and I can't see any real differences between the 2 that could be causing the issue.
Any ideas?

The main difference is the Node.js version. 8.1 has 14.x, while the 8.2 container has Node 16.x. I've tried to come up with some interesting ways to use different node versions, but didn't get very far. I would either rebuild your dependency lock file using Node 16 (should likely work), or install NVM manually and configure the 14.x version.


[modify][weather xml] automatically remove imported urls logs

Coverage decreased (-8.4%) to 39.117% when pulling 861b0d4e48e4da602be9ac7bdd0315173cbc340d on feature/weather-xml-cleaner into c3df3309ac01573707f0443d3ea04818e3920eec on master.


Have a plan to upload it to official addon store？

Dreams it possible
We have official addon store now.
Let's go!

Never gonna happen atleast not the official cats anyway.
https://github.com/absolute-quantum/cats-blender-plugin/issues/670


BindKit - Two-way data binding framework for iOS. Only one API to learn.

Project URL
https://github.com/electricbolt/bindkit
Category
Reactive Programming
Description
Adds the following:

BindKit - Two-way data binding framework for iOS. Only one API to learn.

Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 8 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

254
301
https://github.com/alfa-laboratory/YARCH-Examples redirects tohttps://github.com/alfa-laboratory/YARCH

761
301
https://github.com/ReactiveKit/Bond redirects tohttps://github.com/DeclarativeHub/Bond

788
301
https://github.com/AbsurdAbstractions/Layoutless redirects tohttps://github.com/DeclarativeHub/Layoutless

870
301
https://github.com/aleph7/Upsurge redirects tohttps://github.com/alejandro-isaza/Upsurge

917
301
https://github.com/bahlo/SwiftGif redirects tohttps://github.com/swiftgif/SwiftGif

1335
301
https://github.com/ReactiveKit/ReactiveKit redirects tohttps://github.com/DeclarativeHub/ReactiveKit

2727
301
https://github.com/JetpackSwift/FrameworkTemplate redirects tohttps://github.com/RahulKatariya/FrameworkTemplate

2928
302
https://indieiosfocus.curated.co redirects tohttp://indieiosfocus.com/

  Generated by :no_entry_sign: Danger


Keyboard navigation

XZoom would be awesome if we could ad a keyboard navigation with -> and <-
@payalord XZoom is a really great library that I will use for an e-commerce website.
Feel free publishing (or sending me<EMAIL_ADDRESS>your bitcoin address for a donation.

Hi, thank you. Here is my bitcoin address: 13PqcnMSY5k7LJhyaMa4gS1Jj11SLGaq9V
Good idea to create some public donation button for bitcoin. Will make it later.
With xZoom API and some little jQuery code you can try next approach to implement navigation with -> and <- arrow keys:
// Getting an instance
var xzoom = $('.xzoom, .xzoom-gallery').xzoom();

$("body").keydown(function(e) {
    var index = xzoom.gallery().index;
    var max = xzoom.gallery().ogallery.length - 1;

    if(e.keyCode == 37) {
        // left pressed
        index--;
        if (index < 0) index = max;
        $('.xzoom-gallery').eq(index).trigger('click');
    } else if(e.keyCode == 39) {
        // right pressed
        index++;
        if (index > max) index = 0;
        $('.xzoom-gallery').eq(index).trigger('click');
    }
});

Thank you!
I can't use xZoom for now because de jQuery version seems to be not compliante with some other library.
Have you schedule to upgrade soon?

Which jQuery version you need xZoom to work with? I will test and make sure that xZoom supports all jQuery versions from 1.2.6+

Works fine! Thanks!
I use JQuery 3.2.1.
I made the integration and got a very small zoom. Next steps by e-mail.

I have got 3 small issues (5px offset, moving slow, position of the thumb, ...).
I tried many improvements, without success (CSS modification of 5px, more JS libraries, ...)


Feature Request: Automatic refresh

Is your feature request related to a problem? Please describe.
I often have gh-dash open in a terminal on a secondary monitor to keep track of incoming PRs. This doesn't work very well currently, since I have to manually switch over to the window and press r to refresh.
Describe the solution you'd like
I'd love a setting in the config file that makes the currently active tab auto refresh every x seconds:
defaults:
  autoRefreshInterval: 60

Describe alternatives you've considered
Might also consider automatically refreshing on tab change if it's been longer than auto_refresh_interval since you last visited it.

Yeah I like this a lot!

This is done!
Check out the refetchIntervalMinutes config option.


AC power coming back during shutdown countdown

using firmware v. 4
when AC power comes back on during the shutdown countdown, I realized that the 5V supply for the raspberry pi is not switched off at the end of the countdown. so, after shutdown of the operating system, the raspberry pi will never restart automatically.
IMHO the following would be better: once a shutdown countdown starts, and we have the Back-To-AC Auto Power UP flag set, then:

the shutdown sequence should proceed to its end,
switch off the 5V-power for a certain time (a couple of seconds maybe), regardless of AC Power.
switch the 5V back on when AC power is restored.

otherwise, I am very happy with the product.
one more thing: would be nice to have some sort of alert for new firmware releases and release notes for these.

We are making changes to the firmware for this feature, but we still need some internal testing before release, which is expected to be done within a week.

The test is being conducted by @yoyojacky .

using firmware v. 4
when AC power comes back on during the shutdown countdown, I realized that the 5V supply for the raspberry pi is not switched off at the end of the countdown. so, after shutdown of the operating system, the raspberry pi will never restart automatically.
IMHO the following would be better: once a shutdown countdown starts, and we have the Back-To-AC Auto Power UP flag set, then:

the shutdown sequence should proceed to its end,
switch off the 5V-power for a certain time (a couple of seconds maybe), regardless of AC Power.
switch the 5V back on when AC power is restored.

otherwise, I am very happy with the product.
one more thing: would be nice to have some sort of alert for new firmware releases and release notes for these.

Thank you for your approval. We are updating the firmware. Now the firmware version is v5. You can pull the latest python script file by typing git pull in the warehouse directory, and then please follow the instructions on the wiki to perform the OTA upgrade. After the upgrade, you can download The power will be cut off after the countdown is over. please pay attention to the countdown time default is 240 seconds, you can modify it by Editing Full-featured-demo-code.py.
or you can just do it as following step:
Step1.  Plug the power supply. make sure it charging your batteries.
Step2. Update firmware via OTA  by typing following command in terminal:
i2cset -y 1 0x17  50 127  b
sudo sync && sudo init 0

Step3. Remove the power and batteries and reinsert batteries back to UPS and plug the power supply again.
Step4. Enter into upsplus folder where you cloned the repository and typing following command :
python3 OTA_firmware_upgrade.py

it will shutdown your Pi automatically.
Step5. Remove the batteries and power and reinsert it back to your UPS, plug the power supply again.
Step6.
Step 7.  Setting the function of back-to-ac:
i2cset -y 1 0x17  25 1

Step 8.  Setting shutdown Countdown function:
i2cset -y  1 0x17  24  20

PS: 20 means Counting down in 20 seconds.
Step 9.  Watch and wait
That's all, thank you~

I've been experiencing "no countdown" after setting the UPS to shutdown after 20 seconds (tried 10 and 15 also), but the UPS was not counting down.
So, I upgraded to the last firmware following @yoyojacky instructions, and I got stuck in OTA Firmware Upgrade Mode (0x18).
May you help, please?

I've been experiencing "no countdown" after setting the UPS to shutdown after 20 seconds (tried 10 and 15 also), but the UPS was not counting down.
So, I upgraded to the last firmware following @yoyojacky instructions, and I got stuck in OTA Firmware Upgrade Mode (0x18).
May you help, please?

0x17 Operation mode is back!
I removed the option i2c_arm_baudrate=400000 from the /boot/config.txt.
It may be useful to someone else.

I've been experiencing "no countdown" after setting the UPS to shutdown after 20 seconds (tried 10 and 15 also), but the UPS was not counting down.
So, I upgraded to the last firmware following @yoyojacky instructions, and I got stuck in OTA Firmware Upgrade Mode (0x18).
May you help, please?

0x17 Operation mode is back, and shutdown countdown also! I removed the option i2c_arm_baudrate=400000 from the /boot/config.txt. It may be useful to someone else.. but the way, which is the suggested baudrate for UPS Plus?

default is good, no need to change the value of the i2c_baudrate.


[MRG]using mini batch for gd

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines or the commits will be squashed into one.
[x] Tests for the changes have been added.
[x] Docs have been added / updated.

What kind of change does this PR introduce?
Implements mini batch gradient descent

Does this PR introduce a breaking change?
No

Pull Request Test Coverage Report for Build 422

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 96.37%

Totals

Change from base Build 408:
-0.4%

Covered Lines:
770

Relevant Lines:
799

💛  - Coveralls


direct edit button for non-iframe shared page

When the shared page is not in an iframe, show a "Edit this game" button instead of "show code".
It is currently VERY painful to remix a game: show case (wait editor to load), click on edit on upper right (wait again for editor)

What is non-iframed scenario?

OneNote

From: Abhijith Chatra<EMAIL_ADDRESS>Sent: Friday, August 2, 2019 11:21:35 AM
To: microsoft/pxt-arcade<EMAIL_ADDRESS>Cc: Peli de Halleux<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/pxt-arcade] direct edit button for non-iframe shared page (#1181)
What is non-iframed scenario?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fpxt-arcade%2Fpull%2F1181%3Femail_source%3Dnotifications%26email_token%3DAA73QKMLEPBG44RXNIKYXXDQCR3K7A5CNFSM4II77PBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3OP3MI%23issuecomment-517799345&data=02|01|jhalleux%40microsoft.com|798d9ce302694499693208d71776409e|72f988bf86f141af91ab2d7cd011db47|1|0|637003668964794569&sdata=BK5J1jNN0At6AGRbgJsG%2FW3wDSaU7GpKK6UFBopsS08%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKOT3J4JJWJOUPIR5STQCR3K7ANCNFSM4II77PBA&data=02|01|jhalleux%40microsoft.com|798d9ce302694499693208d71776409e|72f988bf86f141af91ab2d7cd011db47|1|0|637003668964804564&sdata=ya7H%2FZM71udhSdniFTsG98cQBs8uXw4aXP91sX49D%2B4%3D&reserved=0.

OneNote is iframed? Happy to simplify and always go to the editor.

This change needs to be discussed with @Jaqster .

There are teacher\student scenarios where you want to just see the readonly code (like assignment submission).

Even if it's iframe wouldn't it just reload with the full editor?

Also why not have 2 buttons "Show code" & "Remix"? Keeps the existing behavior of readOnly behaviour and allows remixing as well.

We can have both but it gets crowded on mobile. I think we can hide “show code” on mobile - you won’t see anything really.

Maybe hide remix on mobile

straight to editor seems overkill on mobile.

Remove the button all together then.

Phone experience should be optimized for gaming for sure. No editing. Just play, play play

Can you add a screenshot in phone view & desktop view?

It is hidden in mobile and visible as seen on the PR description on landscape (desktop)

Ok cool

Looks good to me.


NotoSansTamilUI: Transformations Filter applies twice to composite glyphs

While working on https://github.com/googlei18n/fontmake/issues/275, I noticed a problem with the way we use Glyphs.app's "Filter" custom parameter in NotoSansTamilUI.
The filter in question is a "Transformations" that is meant to shift all glyphs vertically by 51 units:
Transformations;OffsetY:51;Origin:4;

Filter custom parameters by default apply to all glyphs (unless one specifies "include" or "exclude" values), and are triggered, as the Handbook says, "after decomposition of compound glyphs".
Now, this works as intended when exporting OTF (CFF) instances, where all compound glyphs are always decomposed on export.
However there is a problem when we export TTF instances, because here compound glyphs are not decomposed by default on export, as glyf table can handle these; this means that the "Transformations" filter, being applied to all glyphs (including compounds) will end up being applied multiple times for the composite glyphs, because the transformation will occur both in each of the base glyphs used as components, and again for the composite glyph as a whole...
Compare the highlighted subscript two glyph (a composite glyph of the superscript two) in the OTF:

And in the TTF:

The latter appears to be shifted twice by a doubled amount.
So this means the same generic filter value can't work for producing both OTF and TTF.
To produce the same effect for both output formats, we can do two things:

decompose all glyphs also for the generated TTF (but this will increase its file size); Glyphs has a "Decompose glyphs" custom parameter which takes a list of compound glyphs to be decomposed at export time;

Or turn the Filter into a PreFilter, which is another kind of custom parameter that, unlike Filter, is applied before decomposition of compound glyphs; but we need to apply this pre-filter only to non-composite glyphs (i.e. containing actual paths, not references to other glyphs). This way, the composite glyphs will also be indirectly transformed, because their respective base glyphs were; but they themselves won't be transformed as such, because we exclude them from the filter. To do that, we can append to the "Transformations..." string above, either an ‘include:’ list of  space- or comma-separated names of non-compound glyphs, or which is the same an ‘exclude:’ list with the names of the all the composite glyphs.

The (2) approach is preferable, however we need to check if there are glyphs containing mixed contours and components, which should be decomposed before applying the filter, or else their components will have a wrong transform. Adding only these to the "Decompose Glyphs" custom parameter should work.

What's the resolution here?

What's the resolution here?

The resolution here is that we should use a "PreFilter" instead of a "Filter" filter, and only include non-composite glyphs in it.
But there's a few things we need before we can do this.

glyphLib needs to know about PreFilter custom parameters; currently it only reads Filter (https://github.com/googlei18n/glyphsLib/issues/214). It should be relatively simple to extend -- only that glyphsLib master branch is blocked by the classtree rewrite...

ufo2ft needs to handle both Pre- and regular Filters, however the problem here is that the decomposition of composite glyphs (as well as overlap removal and quadratic conversion) currently happens inside fontmake before and independently of ufo2ft; so by the time the fonts reach ufo2ft, it's too late to apply a PreFilter (i.e one that occurs before decomposition, and thus logically also before overlap removal and quadratic conversion).

I was already thinking for a while of moving the composite glyphs' decomposition and the overlap removal steps from fontmake to ufo2ft, with the latter as the compiler API and the former as its command line front end. As for the simple (non-interpolatable) cubic-to-quadratic conversion, this is already supported by ufo2ft. Having the decomposition and overlap removal steps performed by ufo2ft would allow it to apply the filters in the right order (pre- or not). In this scenario, the interpolatable/variable output would still be done by fontmake rather than ufo2ft's, as the latter only does one single ufo at a time (but that could also be changed).
Another option, which is tempting, is that we let glyphsLib apply the filters' transformations on the generated UFO instances themselves, instead of leaving it to the ufo2ft compiler to apply them. We would call glyph.move((x, y)) and we'd be basically done. I don't know... It would mean we would only support filters with *.glyphs input sources, and not with UFO sources; but it's true that this idea of an export-time filter come from Glyphs.app originally.

We had a use case for a prefilter in UFOs.
Designers wanted to have the outlines in a certain position to make consistent design easier (in our case for a connected script font), only the have them offset at build time.

I just tried to re-export TTFs for NotoSansTamil UI instances using the latest Glyphs.app beta 1093.
The UI instances contain the following custom parameter
(
        {
        Filter = "Transformations;OffsetY:51;Origin:4;";
    }
)

With older Glyphs.app versions, the composite glyphs would be shifted up twice -- as I explained in the original post above.
With the current beta, it seems that they are not being shifted twice...
I implemented the Transformations filter in ufo2ft following the old Glyphs.app behavior... 😞
I was planning to add PreFilter with only the contour-glyphs to fix the above issue, but seems like Glyphs.app behavior has now changed.
@schriftgestalt can you please explain how composite glyphs are supposed to be affected by the Transformations export filter?
What about a filter has include statement which contains some composite glyphs? Do you ignore an explicit request from the user?
And how are mixed contour/component glyphs treated in this regard?

I made some tests, and this is what I can see:

when a glyph contains components and it is included in the transformation, if the base glyph for that component is also included in the list of glyphs to be transformed, then the base glyph is transformed only once, and its component reference in the composite glyphs is not transformed

if the component's base glyph is not also included, then the component itself is transformed, and the base glyph is left untouched.

@schriftgestalt can you confirm this is how it works?

Your observations are spot on.

not sure what you mean by that "compensation of the movement of the base glyphs".
I am simply not transforming a component if its base glyph (which can be another composite glyph) is also included in the transformation.

Try a component that is 180 degrees rotated.

I will try, but still I don’t see why the kind of transformation should make any difference.

Oh... I see! E.g. if user says “shift up” but the component is flipped upside down, you have to shift it down, and things like that.

@schriftgestalt I tried to apply a "Filter = Transformations;OffsetY:200" to a composite glyph flipped upside down (ScaleY -100), as well as to one rotated 180 degrees.
When I export the instance TTF from Glyphs.app, I see that the two composite glyphs have been decomposed, and therefore correctly shifted up by 200 units.
If instead of Filter, I use a PreFilter, the result is that two composite glyphs are still decomposed, but they are now shifted down by 200 units.
Besides the decomposition of transformed glyphs -- there must be some custom parameter to turn this off (I forgot which one) --, I don't see any special "compensation of the movement of the base glyph" that you mentioned above...

this is the test glyphs file I used:
test_transform.zip

ok, I had to turn on "Keep Transformed Components". Now I see what you mean.
that's another issue: Glyphs.app defaults to decomposing transformed components, while fontmake keeps them by default for TTFs https://github.com/googlei18n/fontmake/issues/253

I checked NotoSansTamil which is the only font that uses the Transformations filter, and it seems like no component has any scale or slant or flip, just plain offsets.
So I'm leaning towards releasing this first version of the ufo2ft filter, which doesn't take into account components with complex transformations, and add support for that later when needed.

also, I believe that to be complete, this Tranformations filter should also nullify (invert) the transformation on components that had their base glyph transformed but whose composite glyphs they belong was not included in the set to be transformed.
Glyphs.app doesn't seem to do that.
wow, this is getting complicated...

ok I think I've found an algorithm: when about to transform a component, check if the base glyph was already transformed; if so, then check if the component's current 6-float tuple transformation starts with (1, 0, 0, 1), then don't transform the component again as the base already took care of it; otherwise, if the component's current transformation is not a simple offset, then multiply the component's transformation matrix with the inverse of the filter's transformation matrix (i.e. the "compensation" step Georg was talking about), and then proceed to apply the filter's matrix on top that.


Model for photos

Is there a photo model usable with version of Waifu2x?

Try replacing with these, https://github.com/nagadomi/waifu2x/tree/master/models/vgg_7/photo
I forgot if we made further changes to how we handle models, but I do not think so, those could work.


Second passphrase

Fixing some styles in creating second passphrase dialog. Also nowhere was showing how much it costs!

I created this PR https://github.com/ArkEcosystem/ark-desktop/pull/272. Seems like a better approach to making sure users know what they are doing. Also, fixed some bugs with the form.

@luciorubeens we can close this since it has been resolved right?

Fixed in #231


Android TV player UI focus issue

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
run on Android TV simulator. When I seek the video once, it stays focused on the progress bar.
apple tvOS is work perfect.
Expected Behavior
When the UI is hidden, the focus is reset. mabe better?
Steps To Reproduce
No response
What platform(s) are you experiencing the issue on?

[ ] Android
[X] Android TV / Fire TV
[ ] iOS
[ ] tvOS

Player React Native SDK version
0.7.2
Device / Environment
System:
OS: macOS 13.4.1
CPU: (8) arm64 Apple M1
Memory: 81.16 MB / 8.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
"bitmovin-player-react-native": "^0.7.2",
"expo": "^49.0.3",
"react": "18.2.0",
"react-native"<EMAIL_ADDRESS>Stream URL (Optional)
No response
Additional information / Code snippets / Screenshots
https://github.com/bitmovin/bitmovin-player-react-native/assets/13132789/c25ecf6f-5548-4951-afdf-5a20147c2bb1

Hi @xayoung,
Thank you for your feedback & great to see your interest in using Bitmovin on React Native.
We’ll be monitoring further interest on this issue as well as other channels and will let you know here when there is any movement.
Chris

Hi @xayoung
Thank you for suggesting this, We have logged it with the product team to take into consideration as they continue to contribute to this project. As an open-source project we also welcome contributions from anyone should you wish to make a contribution please see https://github.com/bitmovin/bitmovin-player-react-native#contributing-to-this-project
For any other suggestions please do raise them in our community forum https://github.com/bitmovin/bitmovin-player-react-native#raising-a-feature-suggestion where others can also add their vote towards them.
Many thanks
James


Use string for price in response instead of float32

Looks like twillio changes its API and the price is now a string. Got the error while calling GetSMS
failed to get message:
json: cannot unmarshal string into Go struct field SmsResponse.price of type float32

FWIW my library handles this correctly -
https://godoc.org/github.com/kevinburke/twilio-go#Message
On Sun, Sep 3, 2017 at 09:39 Aleksandr<EMAIL_ADDRESS>wrote:

Looks like twillio changes its API and the price is now a string. Got the
error while calling GetSMS
{"level":"error","logger":"main","msg":"failed to get message: json: cannot unmarshal string into Go struct field SmsResponse.price of type float32","time":"2017-09-03T17:33:46+01:00"}

You can view, comment on, or merge this pull request online at:
https://github.com/sfreiberg/gotwilio/pull/18
Commit Summary

Use string for price in response instead of float32

File Changes

M sms.go
https://github.com/sfreiberg/gotwilio/pull/18/files#diff-0 (28)

Patch Links:

https://github.com/sfreiberg/gotwilio/pull/18.patch
https://github.com/sfreiberg/gotwilio/pull/18.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sfreiberg/gotwilio/pull/18, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOSI1H73Ju1EXDdBDlGQCOFFrlEKJGvks5setY0gaJpZM4PLO9a
.

--
--
Kevin Burke
925.271.7005 | kev.inburke.com

@kevinburke did not pick yours for my project as the last commit was in 2016 :)

Not sure what you are talking about, I've added several API's this year
alone, in addition I'd stand by it even if there hadn't been new commits
On Sun, Sep 3, 2017 at 14:03 Aleksandr<EMAIL_ADDRESS>wrote:

@kevinburke https://github.com/kevinburke did not pick yours for my
project as the last commit was in 2016 :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sfreiberg/gotwilio/pull/18#issuecomment-326831084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOSI2QLBaLlWsZ7ELK59Hxt1lD8cv3zks5sexQugaJpZM4PLO9a
.

--
--
Kevin Burke
925.271.7005 | kev.inburke.com

@kevinburke sorry, I was talking about the library you actually forked. It looked a little bit stale :)

"The library I actually forked" is the same library, I was working on it as part of a consulting engagement. When the engagement ended, I continued contributing under my own name. I'm the only person that's ever made significant contributions to it.

@kevinburke well ... ok. Thank you for the explanation.

Apologies for the late response on this. I didn't see the pull request come in.


Setting the Window Icon?

Is there a way to set the window icon on Windows yet? i.e. the top left corner of the Webview window?
Using Go 1.18.3 on Windows 11. But will need it for MacOS and Linux too.
Thanks

Take a look at the App Distribution section of the readme.

Sorry, I failed to mention that you would typically do this when compiling C/C++ programs, and on a low level that's what would need to happen somehow even if you were to use Go. I think you need to look into how to either combine that with Go or see if there is a Go-specific way to do it.

Here is an example for Go on Windows using a similar method. This is just the first way I tried to make it work and I can't say it's a good approach.
mkdir build libs "libs/webview"

mkdir libs\webview2
curl -sSL "https://www.nuget.org/api/v2/package/Microsoft.Web.WebView2" | tar -xf - -C libs\webview2
copy /Y libs\webview2\build\native\x64\WebView2Loader.dll build

go mod init example.com/basic
go get github.com/webview/webview

set CGO_CXXFLAGS="-I%cd%\libs\webview2\build\native\include"
set CGO_LDFLAGS="-L%cd%\libs\webview2\build\native\x64" build/basic.rc.o

curl -sSLo basic.go "https://raw.githubusercontent.com/webview/webview/master/examples/basic.go"

Put your own icon file:
icons/
└─ main.ico

basic.rc:
32512 ICON "icons\\main.ico"

Build:
windres -i basic.rc -o build/basic.rc.o
go build -ldflags="-H windowsgui" -o build/basic.exe basic.go

You get a bunch of messages saying .rsrc merge failure: duplicate leaf but it works.

goversioninfo does not seem to support specifying the expected resource ID for the icon, and I cannot immediately see that it has documented what the generated ID is,
The alternative method described in the tool's documentation suggests to use windres and following that example you get the same messages as in the example I came up with. So for that seems to be the only way for Go on Windows right now.

As for macOS, please let me know if creating an app bundle is not enough for your use case.
As for Linux I don't think there is any standard way to embed icons or even display them in the top-left corner of the window. I think you'll just have to create a *.desktop file and specify your icon there.

In my particular use case, I am writing a LaunchDaemon. So, an app bundle is not applicable to me. Its an installer script.

Could you please explain and if possIBle show me how you imagine it should behave on macOS?

Now that you mention it... MacOS and Linux do not have a window icon so I suppose this is just a Windows issue. I've got Windows on the brain this week...

I think some MinGW-w64 distributions link them statically by default while others do not. MSYS2 for example does not. Please if @vitalyster's response in #779 helps.
As for this issue, do you agree that we can narrow down the scope to only Windows and consider it solved? I think the easiest "fix" on our end would be to expand the Windows Apps section of the readme and perhaps implement it for the example apps.
In order to introduce any new API for this I think it would be preferable for it to be implemented in a consistent manner on all supported platforms, but first I think we need to have a clear idea about how it would be implemented on non-Windows systems.

#782 adds some more useful information to the readme for Windows but not for Go. The important thing for the window icon is that the resource script file (*.rc) has an entry for the icon with an ID of 32512 (IDI_APPLICATION) since that's what the library loads and sets for the window. That has to happen one way or another when using Go as well, and it is not clear to me at this time how to do it the best way possible.

Given you have shown that the Windows icon can be set in C++, I think it is safe to say it is solved. More documentation around this stuff is always welcome. Understanding how to do it is as important as knowing the tool can do it.
I think a lot of my problem is understanding the interaction between the tools since a lot of these tools were not targeted at Go specifically. e.g. Just because it compiles on the command line doesn't mean you can get vscode to debug it. Or, how to pass the appropriate compiler switches through the vscode/cgo interface to the compiler, etc.
In this regard, the old way of precompiling the webview.dll was a lot easier. You just had to precompile a fixed example, then include the DLL in the bundle. Integrating that into my own app has proven to be a lot more work. My 2 cents.
Go is a fantastic mix of simplicity and robustness. But it really needs a cross-platform GUI framework that is well integrated.
Another example that would be really nice to have is an HTML form submittal using the js bindings that does not require running an http session on the host. I believe from the docs that this is possible. A simple example would be very helpful. e.g. the app exposes a variable to the HTML and then accepts it back on submit.
Thanks again!
Peter

Again, I appreciate your help on this. And yes, a lot of it is beyond the scope of your library specifically. We all appreciate your generosity with what is a truly useful package. And any help you have given me will live forever in Google and no doubt help many others.
Thanks again.

I will just add a bit more... The other reasonable option for the Windows icon would simply have a way to exclude it altogether. I'm sure this is also part of the external resource management. I will check it out... Thanks

I was just thinking about it some more. And I compared it to some of the built-in apps in Windows 11. e.g. The Windows 11 Photos app has no icon at the top left of the screen. Nor do many other modern apps. So one could view that icon as a legacy item from older versions of windows.
Perhaps a better cross-platform default behaviour would be to suppress the icon on Windows to match macOS and Linux. That way all the platforms are consistent and it becomes a non-issue for 99% of your user base. And probably a lot easier than trying to add icons to the other platforms.
I realize this is probably still beyond the scope of the webview library internally. But, it would still be a good default from a consistency point of view...
My 2 cents.
Thanks again!

I don't mind the stuff in the main folder as I am creating it each time and deleting it after I'm done.
IMHO, Go is not really the issue here. The problem is with CGO and all the various undocumented bits in between. You're not using Go's linker anymore and you're pretty much on your own...  If you're a C++ guy, maybe it's somewhat obvious. But not for me.
I have the icons working now. But somehow, if I add the 32512 icon, it somehow ends up being used as the taskbar icon too. And it seems to scale down to x16 and then back up again. Looks terrible. I've tried specifying a bunch of different ico sizes and even a multi-image ico file and it always comes up the same. And, if I eliminate GoVersionInfo entirely, it still seems to link the two this way.
I've not been able to find a good doc on what resources are used where. Or is that more up to how you coded the library?
And... I've not figured out to add the manifest with windres so I still need goversioninfo for now. I'd prefer to use windres as goversioninfo is another partially implemented solution for me.
Do you have any suggestions for understanding the resource issue?
Thanks

I am not sure I can tell why you are seeing that behavior.
1     RT_MANIFEST "basic.exe.manifest"
100   ICON "icons\\application.ico"
32512 ICON "icons\\window.ico"

In my example project application.ico is used as the exe icon and taskbar icon while window.ico shows in the corner of the window.
The first line shows how you could add a manifest. 24 means RT_MANIFEST.
Example manifest:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0" xmlns:asmv3="urn:schemas-microsoft-com:asm.v3">
  <asmv3:application>
    <asmv3:windowsSettings>
      <dpiAware xmlns="http://schemas.microsoft.com/SMI/2005/WindowsSettings">false</dpiAware>
      <dpiAwareness xmlns="http://schemas.microsoft.com/SMI/2016/WindowsSettings">unaware</dpiAwareness>
    </asmv3:windowsSettings>
  </asmv3:application>
</assembly>

This would enforce DPI unawareness and make the app look blurry when using DPI scaling.

One thing I came to realize is that Windows (10) seems to have an icon cache problem. I had icon cache problems many years ago so seeing it again was surprising. You might want to check if that applies to you.

I'm still having problems with the fuzzy taskbar icons on all of my windows systems. It's just the icon I am adding as a resource. And it does not happen unless I try to set the IDI_APPLICATION icon. And it only affects the icon of my app.
I'm not implying it is the fault of the library. It's most likely a gap in my knowledge. But you could kindly enlighten me on that gap, I'm sure I am not the only person on the Internet who will benefit.
Using your C++ code as listed above and a freshly minted ico file... The icon is a BootstrapIcons icon converted with ImageMagick.
If I load the resulting exe into ResourceHacker, it seems the icons are embedded correctly. They show as 128x128 which is what they are. But when rendered on the taskbar and window corner, they are poorly scaled and blurry.
Are there some extra parameters required to adjust? It looks worse with a color icon.
Any suggestions are more appreciated. Thanks again.

I think part of the problem in that particular case could be that downscaling that particular icon makes the lines too crisp and thin, but I cannot rule out that it could be something else too. I will give it a try on Windows 11 and report back.
Is that the real size of your screenshot or is it upscaled by 20%? Everything in it looks blurry to me so I am just assuming that it is upscaled.

If you click on the image, it's 100%. It's a screen capture [alt-prtsc] on a Win11 VM (VMware). But copying the exe to a laptop does the same thing. It looks like a bad scaling problem to me.

I can see that it is 100% but in my browser (Firefox) it shows with +20% scaling on Windows.. Maybe an issue with that particular browser on Windows.
My first observations on Windows 11 were as follows.

Using a 128px icon made the window icon blurry because of DPI awareness settings. The problem with the library is that it does not enable DPI awarenes early enough, but I believe that you can fix it simply by enabling it with the manifest file until fixed in the library.
In my example window.ico is used as the window and taskbar icon instead of application.ico.

Now I see why I saw application.ico as the taskbar window before on Windows 10. If you compile the program as a console app, which I apparently did, then the icon shown on top in the taskbar is the console window's icon which is application.ico.
With proper DPI settings and a single 128px icon, the window icon looks crisp to me but the taskbar icon is blurry.

I found the problem. The library always loads a small icon.
      HICON icon = (HICON)LoadImage(
          hInstance, IDI_APPLICATION, IMAGE_ICON, GetSystemMetrics(SM_CXSMICON),
          GetSystemMetrics(SM_CYSMICON), LR_DEFAULTCOLOR);
[...]
      wc.hIcon = icon;
      wc.hIconSm = icon;

I have submitted PR with fixes for the icon:

#790

With the mentioned fix, these are my observations on Windows 10 with different scaling settings and icon sizes.

Scaling
Window
App

100%
16px
32px

125%
24px
48px

175%
32px
48px

250%
48px
48px

350%
48px
256px

For this test the multi-image icon had the following sizes: 16px, 24px, 32px, 48px, 256px
At 125% scaling I noticed that the 24px icon was not displayed pixel-perfectly but maybe this is the best we can do. In any case the result is better than before so I think this will be the conclusion for now.

I've tried a bunch of permutations and I'm now seeing a clear window icon. But I still get a blurry taskbar icon. Looking with ResourceHacker, they seem to be embedded properly. It seems even to take a multi-resolution icon file correctly.  Though it has the same behaviour with a single icon of the correct resolution.
Would you mind having a look at my config attached/zipped? Forgive me for the one folder. For the test, I am keeping it simple... Not sure if the dpiawareness is actually doing anything.
windres -i basic.rc -o basic.rc.o
windres -i basic.versioninfo.rc -o basic.versioninfo.rc.o
g++ basic.cc basic.rc.o basic.versioninfo.rc.o ^
    -std=c++17 -mwindows ^
    -I.. -I../libs/webview -I../libs/webview2/build/native/include ^
    -L../libs/webview2/build/native/x64 -lWebView2Loader.dll ^
    -lole32 -lshell32 -lshlwapi -luser32 ^
    -o basic.exe

1     RT_MANIFEST "basic.manifest"
100   ICON "assets\\2ho-32.ico"
32512 ICON "assets\\2ho-16.ico"

This version of the manifest I patterned after notepad.exe...
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
<assemblyIdentity
    name="example.com.Basic"
    processorArchitecture="*"
    version="<IP_ADDRESS>"
    type="win32"
  />
  <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
    <security>
      <requestedPrivileges>
        <requestedExecutionLevel
          level="asInvoker"
          uiAccess="false"
        />
      </requestedPrivileges>
    </security>
  </trustInfo>
  <application xmlns="urn:schemas-microsoft-com:asm.v3">
    <windowsSettings xmlns:ws2="http://schemas.microsoft.com/SMI/2016/WindowsSettings">
        <ws2:dpiAwareness>PerMonitorV2</ws2:dpiAwareness>
        <ws2:longPathAware>true</ws2:longPathAware>
    </windowsSettings>
  </application>
</assembly>

basiccc.zip

Merge is pending until another maintainer gets the chance to review and approve it, but you could take the changed file and replace yours if you would like to try it.

You can keep using the manifest file but without the fix in the library then the library will likely be a hinder.

#790 has now been merged.

Thank you for the feedback! 😄

As for Linux I don't think there is any standard way to embed icons or even display them in the top-left corner of the window. I think you'll just have to create a *.desktop file and specify your icon there.

Actually one can embed icons in Linux.  Just not through the resource file as that tends to be a Windows paradigm.  Raylib (as far as C/C++) it just has a SetIcon() (as far as what is exposed to the user of the library);  Nimview, which is the Nim binding version does the same thing and is able to set the icon.
As far as Go goes, Fyne has the same function as well and it works that way in Linux.
If one isn't seeing window icons on the titlebars (there is a generic X11 icon that will show up for the webview on Linux if it isn't set, if using something other than Gnome (or if you manually turned it off in a Plasma DE)), I would be willing to bet that they are using a Gnome/GTK based DE as that is what it does out of the box.

@wwderw Thanks for the info!
GTK has gtk_window_set_icon but results will vary:

This icon is used when the window is minimized (also known as iconified). Some window managers or desktop environments may also place it in the window frame, or display it in other contexts. On others, the icon is not used at all, so your mileage may vary.

For example, my Linux distro (Ubuntu) does not show an icon in the title bar but that function can change the icon shown in the dock. I can however already set an icon there using a *.desktop file.


Error during ".\vcpkg install openssl:x64-windows"

Package: openssl[core]:x64-windows -> 3.0.7#2
Host Environment

Host: x64-windows
Compiler: MSVC 19.34.31937.0
vcpkg-tool version: 2023-01-24-8a88d63f241d391772fbde69af9cab96c3c64c75
vcpkg-scripts version: d558a1b4a 2023-01-24 (6 hours ago)

To Reproduce
vcpkg install openssl:x64-windows
Failure logs
-- Downloading https://github.com/openssl/openssl/archive/openssl-3.0.7.tar.gz -> openssl-openssl-openssl-3.0.7.tar.gz...
-- Extracting source D:/Dev/Git/vcpkg/downloads/openssl-openssl-openssl-3.0.7.tar.gz
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
    Command failed: D:/Dev/Git/vcpkg/downloads/tools/cmake-3.25.1-windows/cmake-3.25.1-windows-i386/bin/cmake.exe -E tar xjf D:/Dev/Git/vcpkg/downloads/openssl-openssl-openssl-3.0.7.tar.gz
    Working Directory: D:/Dev/Git/vcpkg/buildtrees/openssl/src/nssl-3.0.7-96d825e305.clean.tmp
    Error code: 1
    See logs for more information:
      D:\Dev\Git\vcpkg\buildtrees\openssl\extract-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_extract_source_archive.cmake:126 (vcpkg_execute_required_process)
  scripts/cmake/vcpkg_extract_source_archive_ex.cmake:8 (vcpkg_extract_source_archive)
  scripts/cmake/vcpkg_from_github.cmake:113 (vcpkg_extract_source_archive_ex)
  ports/openssl/portfile.cmake:15 (vcpkg_from_github)
  scripts/ports.cmake:147 (include)

D:\Dev\Git\vcpkg\buildtrees\openssl\extract-err.log
CMake Error: Problem with archive_write_header(): Can't create '\\?\D:\Dev\Git\vcpkg\buildtrees\openssl\src\nssl-3.0.7-96d825e305.clean.tmp\openssl-openssl-3.0.7\doc\man3\CMS_EncryptedData_decrypt.pod'
CMake Error: Current file: openssl-openssl-3.0.7/doc/man3/CMS_EncryptedData_decrypt.pod
CMake Error: Problem extracting tar: D:/Dev/Git/vcpkg/downloads/openssl-openssl-openssl-3.0.7.tar.gz

Additional context
The system is a windows 10 with a Microsoft Visual Studio Enterprise 2022 (64-bit) - Current Version 17.4.4

@ohainzaw Thanks for posting this issue.
CMake Error: Problem with archive_write_header(): Can't create '\\?\D:\Dev\Git\vcpkg\buildtrees\openssl\src\nssl-3.0.7-96d825e305.clean.tmp\openssl-openssl-3.0.7\doc\man3\CMS_EncryptedData_decrypt.pod'
CMake Error: Current file: openssl-openssl-3.0.7/doc/man3/CMS_EncryptedData_decrypt.pod
CMake Error: Problem extracting tar: D:/Dev/Git/vcpkg/downloads/openssl-openssl-openssl-3.0.7.tar.gz

Could you please try to install vcpkg to another driver?
Please see the details here https://github.com/microsoft/vcpkg/issues/14300#issuecomment-719396447

@Adela0814 Thanks a lot for the quick response.
I am not sure, what you mean with "Could you please try to install vcpkg to another driver?"
I tried to moved from the Disk "D:..." to the disk "C:". I also checked that the "C:" drive has the NTFS format.
I did the following thing on my machine - but it still fails:

I created the folder C:\Dev\Git
I cloned the repository https://github.com/microsoft/vcpkg.git to C:\Dev\Git\vcpkg
I run the command C:\Dev\Git\vcpkg>.\bootstrap-vcpkg.bat
After this called the command vcpkg.exe install openssl:x64-windows (see the output below)
Then I checked that the C:\ drive has the NTFS format (see also below).

What else can I try to fix my issue. Please advise me.
Output of vcpkg.exe install openssl:x64-windows:
Computing installation plan...
A suitable version of cmake was not found (required v3.25.1) Downloading portable cmake 3.25.1...
Downloading cmake...
https://github.com/Kitware/CMake/releases/download/v3.25.1/cmake-3.25.1-windows-i386.zip->C:\Dev\Git\vcpkg\downloads\cmake-3.25.1-windows-i386.zip
Downloading https://github.com/Kitware/CMake/releases/download/v3.25.1/cmake-3.25.1-windows-i386.zip
Extracting cmake...
The following packages will be built and installed:
    openssl[core]:x64-windows -> 3.0.7#2
  * vcpkg-cmake[core]:x64-windows -> 2022-12-22
  * vcpkg-cmake-config[core]:x64-windows -> 2022-02-06#1
  * vcpkg-cmake-get-vars[core]:x64-windows -> 2022-12-16
Additional packages (*) will be modified to complete this operation.
Detecting compiler hash for triplet x64-windows...
A suitable version of powershell-core was not found (required v7.2.8) Downloading portable powershell-core 7.2.8...
Downloading powershell-core...
https://github.com/PowerShell/PowerShell/releases/download/v7.2.8/PowerShell-7.2.8-win-x86.zip->C:\Dev\Git\vcpkg\downloads\PowerShell-7.2.8-win-x86.zip
Downloading https://github.com/PowerShell/PowerShell/releases/download/v7.2.8/PowerShell-7.2.8-win-x86.zip
Extracting powershell-core...
A suitable version of 7zip was not found (required v21.7.0) Downloading portable 7zip 21.7.0...
Downloading 7zip...
https://www.7-zip.org/a/7z2107-extra.7z->C:\Dev\Git\vcpkg\downloads\7z2107-extra.7z
Downloading https://www.7-zip.org/a/7z2107-extra.7z
Extracting 7zip...
Restored 3 package(s) from C:\Users\ohainz\AppData\Local\vcpkg\archives in 924.5 ms. Use --debug to see more details.
Installing 1/4 vcpkg-cmake:x64-windows...
Elapsed time to handle vcpkg-cmake:x64-windows: 9.674 ms
Installing 2/4 vcpkg-cmake-config:x64-windows...
Elapsed time to handle vcpkg-cmake-config:x64-windows: 11.85 ms
Installing 3/4 vcpkg-cmake-get-vars:x64-windows...
Elapsed time to handle vcpkg-cmake-get-vars:x64-windows: 11.41 ms
Installing 4/4 openssl:x64-windows...
Building openssl[core]:x64-windows...
-- Downloading https://github.com/openssl/openssl/archive/openssl-3.0.7.tar.gz -> openssl-openssl-openssl-3.0.7.tar.gz...
-- Extracting source C:/Dev/Git/vcpkg/downloads/openssl-openssl-openssl-3.0.7.tar.gz
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
    Command failed: C:/Dev/Git/vcpkg/downloads/tools/cmake-3.25.1-windows/cmake-3.25.1-windows-i386/bin/cmake.exe -E tar xjf C:/Dev/Git/vcpkg/downloads/openssl-openssl-openssl-3.0.7.tar.gz
    Working Directory: C:/Dev/Git/vcpkg/buildtrees/openssl/src/nssl-3.0.7-96d825e305.clean.tmp
    Error code: 1
    See logs for more information:
      C:\Dev\Git\vcpkg\buildtrees\openssl\extract-err.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_extract_source_archive.cmake:126 (vcpkg_execute_required_process)
  scripts/cmake/vcpkg_extract_source_archive_ex.cmake:8 (vcpkg_extract_source_archive)
  scripts/cmake/vcpkg_from_github.cmake:113 (vcpkg_extract_source_archive_ex)
  ports/openssl/portfile.cmake:15 (vcpkg_from_github)
  scripts/ports.cmake:147 (include)

error: building openssl:x64-windows failed with: BUILD_FAILED
error: Please ensure you're using the latest port files with `git pull` and `vcpkg update`.
Then check for known issues at:
    https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+openssl
You can submit a new issue at:
    https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[openssl]+Build+error
Include '[openssl] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
    vcpkg-tool version: 2023-01-24-8a88d63f241d391772fbde69af9cab96c3c64c75
    vcpkg-scripts version: 91dd61bd4 2023-01-27 (3 days ago)

Please use the prefilled template from C:\Dev\Git\vcpkg\installed\vcpkg\issue_body.md when reporting your issue.

Output of diskpart - list volume:
Microsoft DiskPart-Version 10.0.19041.964

Copyright (C) Microsoft Corporation.
Auf Computer: SW-201692

DISKPART> list volume

  Volume ###  Bst  Bezeichnung  DS     Typ         Größe    Status     Info
  ----------  ---  -----------  -----  ----------  -------  ---------  --------
  Volume 0     D   SSD-Data     NTFS   Partition    838 GB  Fehlerfre
  Volume 1     C   OS           NTFS   Partition    474 GB  Fehlerfre  Startpar
  Volume 2         ESP          FAT32  Partition    190 MB  Fehlerfre  System
  Volume 3         WINRETOOLS   NTFS   Partition    990 MB  Fehlerfre  Versteck
  Volume 4         DELLSUPPORT  NTFS   Partition   1536 MB  Fehlerfre  Versteck

Maybe a cmake issue.
https://gitlab.kitware.com/cmake/cmake/-/issues/19102
https://gitlab.kitware.com/search?group_id=415&project_id=541&repository_ref=master&scope=issues&search=Problem+with+archive_write_header

And all these files seems to be manuals ..\doc\man3.. and I think they are not needed to build the the package.

Ah, I remember another cmake issue with this but I don't remember the issue id.
It's hard to fix until the upstream removed / fixed the file format or something.

@JackBoosY Thank you for your response.
Now I found a Windows Server 2016. There the build works fine.

So close this issue?


Cannot read property 'find' of undefinedTypeError: Cannot read property 'find' of undefined

$npm run test-discovery
For the [DEBUG] logs I see these:
Vera Request: authenticate, getSession, getControllers, getSession, GetControllerInfo, getSession, getDataRequest, UserData
(all those appear to process)
All of those appear to work, and UserData shows 94 devices found.
I then receive back:
[INTERNAL ERROR] Cannot read property 'find' of undefinedTypeError: Cannot read property 'find' of undefined
at Object.sceneHasForbiddenDevices
at createSceneEndpoint
at udata.scenes.forEach
at Array.forEach
at processScenes
at results.forEach
at Array.forEach
at collectEntpoings
at vera.getControllers.then.then
No clue where to begin to troubleshoot this.

Additional info - the error shows in the payload section as shown:
[DEBUG] Response: {"event":{"header":{"namespace":"Alexa","name":"ErrorResponse","messageId":"771ef76e-c30f-4620-bb18-59ffd6890c78","payloadVersion":"3"},"payload":{"type":"INTERNAL_ERROR","message":"Cannot read property 'find' of undefined"}}}
info: End - Message
info: ------
info: {
"event": {
"header": {
"namespace": "Alexa",
"name": "ErrorResponse",
"messageId": "771ef76e-c30f-4620-bb18-59ffd6890c78",
"payloadVersion": "3"
},
"payload": {
"type": "INTERNAL_ERROR",
"message": "Cannot read property 'find' of undefined"
}
}
}
info: ------
info: Lambda successfully executed in 10892ms.

Hi @xanderphillips it has been several years since I was last tinkering with this (2017) and unfortunately I don't think I documented which Vera firmware version this was working with back then. This project is unsupported, but feel free to fork.


fix cdc use wrong version binary

fix cdc integration test triggered by upstream pr not use tidb latest branch binary

test ci : https://ci.pingcap.net/blue/organizations/jenkins/cdc_ghpr_integration_test/detail/cdc_ghpr_integration_test/2575/pipeline/116/


SPARC-36: Clinicians should have access to a simplified Dashboard

Add support for UI extensions to configure what actions should be allowed

Adds support for disableDeletion, disableMoving and disableCreation to form and subject clinician UIs where applicable
Adds support for disableActions and disableExpansion to subject and form dashboard views

Add support for seperate admin extensions for the clinician UIs

This allows seperate action configurations for admin and non-admin views

Within built in CARDS runmodes, these configurations are unused. This means that no UI changes should be detected when running CARDS in any built in runmode.
To Test:
As these configurations are unused, all UIs within this branch should remain untouched. Instructions on what should be tested to ensure this can be found in the SPARC pull request. That PR also contains instructions and configurations for testing when functionality has been changed.

Replaced by https://github.com/data-team-uhn/cards/pull/1853


Discuss translation of conditional language with group

Option 1:
% if order_type == 'modify':
Reasons you want to modify your order
% else:
Reasons you want to terminate your order
% endif

Option 2:
---
template: translate_word_modify
content: |
  modify
---
template: translate_word_terminate
content: |
  terminate
---
${ translate_word_modify if order_type == "modify" else translate_word_terminate }

Conclusion: Option 1


Interview lonewolf and 45+45 mods

We should go through the weekly activities that these mods do, and make a list of "use cases" that we need to support for them and ensure the django admin supports them.

One thing in particular to talk about: Alternate management.

We've scheduled a meeting for Saturday 07/30 at 20:00 GMT.

The transcript from this meeting is pinned in our Slack channel.


chromium: Add optional vaapi hw video decoding support

Add 'VAAPISupport' parameter to chromium expression. If set to true, a
patch enabling hw video decoding through vaapi is applied.
The patch originates from Arch Linux
https://git.archlinux.org/svntogit/packages.git/plain/trunk/chromium-vaapi.patch?h=packages/chromium
Tested on 72.xx
Motivation for this change
Using HW video decoding should improve performances and "smoothness" while reducing battery consumption.
Seems to be a requested feature per issue #21481
Things done
Import a patch from Arch and enable its application behind the boolean VAAPISupport parameter for the chromium function (default = false).
Naming of the parameter is debatable as there seems to be 2 conventions: xxxSupport and enableXxx.
Also should the value be conditioned by the presence of the libva package and/or the presence of libva in opengl libs?
Should an assertion be added to ensure that the version of chromium is equal to 72.xx ?

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Are you sure this is working? (Easiest way is probably to observe a YouTube 4K video which is served as VP9 and will be very noticeable if done on the CPU.)
There are a few patches missing that Arch Linux uses that are only in Chromium upstream with version 73: https://git.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/chromium&id=53de8f9a9b72a1cf7b0438a109240104d1303eb6#n96
Since we missed the cutoff date for 19.03 (I had higher priorities unfortunately), I suggest we wait until 73 is released (estimated for the 12th) so it's cleaner.
I would also suggest following the example of all the other distros and enabling the patch by default if it works, since we really don't want people to have to build their own version of Chromium.

As a small note, in 73 and up, this is as simple as enabling use_vaapi=true as a GN flag: https://github.com/chromium/chromium/commit/31225b9c5f3f685d65f742dc129241c30c32157c
(in the GH UI, you can see that that commit is present in 73+ builds).
Also, if I had to guess, as a result, the strategy in this PR might not work for beta/dev flavors of the chromium package.

Not just the GN flag, also the blacklist and the flags need to be changed in 73 - just as described in the commit you linked.

This new one add the missing patches from Arch relaxing libva version check while enforcing they are only applied on chromium 72.
It also actually set use_vaapi to true, an oversight in the first patch.
Sadly I must report that it still doesn't work as expected, h264 test videos are still decoded through software ffmpeg.
Can someone take a look at it ? Chromium takes ages to build on my current dev machine so someone with a better rig might have a quicker iterating cycle.
Thanks.

Chromium takes ages to build on my current dev machine

This is also the reason I haven't taken this on yet, I've yet to set up remote builds on a c5.18xlarge instance. I will make this patch work if you haven't done so by then, but it is going to take me some time as I have another thing to finish first.

I have a build that is built similarly, this is the error I get when I run: GDK_BACKEND=x11 /nix/store/lxvj286d3iwh975ijczkcqnb9cbzn8gg-chromium-72.0.3626.121/bin/chromium --v=3 --log-level=0 --enable-logging=stderr &> foo.txt and then search foo.txt for libva/vaapi:
[1077:1077:0305/090408.109028:VERBOSE1:va_stubs.cc(637)] dlopen(libva.so.2) failed, dlerror() says:
libva.so.2: cannot open shared object file: No such file or directory

libva is in buildInputs. Maybe Chromium is doing something in va_stubs.cc that we need to patch for use on Nix?

@colemickens thanks for the input. I actually get the same log.
Looks like libva is linked at runtime through dlopen and not the usual linker. So the lib file aint in the standard LIBDIR and can't be resolved.
Reading from the manual, adding it to ''runtimeDependencies'' should make it visible at runtime through some LD_LIBRARY_PATH or linking magic.
I have a build going on overnight and will report back ASAP.

@amazari I've done just that in my branch and I am progressing further (I'd had to do that already to enable Ozone). I have just been reluctant to post updates as I don't want to spam too much. Also, I fear that some other complications around libva (https://github.com/NixOS/nixpkgs/issues/55975) are related (I can cause similar errors as below with mpv+x11 but not with mpv+wayland.)
Attempt 1 Naive:
[cole@xeep:~]$ unset LIBVA_DRIVER_NAME # it's not normally set
[cole@xeep:~]$ GDK_BACKEND=x11 /nix/store/65n4m3rkn9q2bfcrmz2xpp61823v7cwm-chromium-72.0.3626.121/bin/chromium --log-level=9 --enable-logging=stderr --v=9 &> log.txt
[cole@xeep:~]$ cat log.txt | grep libva
libva error: va_getDriverName() failed with unknown libva error,driver_name=(null)

Attempt 1 Explicit: i965
[cole@xeep:~]$ ls /run/opengl-driver/lib/dri | grep drv_video | grep i965
i965_drv_video.la
i965_drv_video.so
[cole@xeep:~]$ export LIBVA_DRIVER_NAME=i965
[cole@xeep:~]$ GDK_BACKEND=x11 /nix/store/65n4m3rkn9q2bfcrmz2xpp61823v7cwm-chromium-72.0.3626.121/bin/chromium --log-level=9 --enable-logging=stderr --v=9 &> log.txt
[cole@xeep:~]$ cat log.txt | grep libva
libva error: /run/opengl-driver/lib/dri/i965_drv_video.so init failed
[17809:17809:0305/163001.156500:ERROR:vaapi_wrapper.cc(324)] vaInitialize failed: unknown libva error

I'm wondering if this is related to my local workaround for #55975.
Attempt 2 Explicit: iHD
If you have intel-media-driver installed in nixos's opengl drivers:
[cole@xeep:~]$ ls /run/opengl-driver/lib/dri | grep drv_video | grep iHD
iHD_drv_video.so
[cole@xeep:~]$ export LIBVA_DRIVER_NAME=iHD
[cole@xeep:~]$ GDK_BACKEND=x11 /nix/store/65n4m3rkn9q2bfcrmz2xpp61823v7cwm-chromium-72.0.3626.121/bin/chromium --log-level=9 --enable-logging=stderr --v=9 &> log.txt
[cole@xeep:~]$ cat log.txt | grep libva
[cole@xeep:~]$ cat log.txt | grep --text vaapi
[19095:19095:0305/163315.855345:VERBOSE2:vaapi_video_encode_accelerator.cc(202)] VaapiVideoEncodeAccelerator():
[19095:19095:0305/163315.855375:VERBOSE2:vaapi_video_encode_accelerator.cc(206)] ~VaapiVideoEncodeAccelerator():
[19095:19095:0305/163315.855389:VERBOSE2:vaapi_jpeg_decode_accelerator.cc(306)] ~VaapiJpegDecodeAccelerator(): Destroying VaapiJpegDecodeAccelerator
[19095:19095:0305/163317.384730:VERBOSE2:vaapi_video_decode_accelerator.cc(198)] Initialize(): Initializing VAVDA, profile: h264 main
[19095:19312:0305/163317.385901:VERBOSE2:vaapi_video_decode_accelerator.cc(446)] DecodeTask(): Decoder requesting a new set of surfaces
[19095:19095:0305/163317.385941:VERBOSE2:vaapi_video_decode_accelerator.cc(499)] InitiateSurfaceSetChange(): Initiating surface set change
[19095:19095:0305/163317.385964:VERBOSE2:vaapi_video_decode_accelerator.cc(543)] TryFinishSurfaceSetChange(): Requesting 11 pictures of size: 1280x720
[19316:19316:0305/163317.489346:VERBOSE2:vaapi_video_encode_accelerator.cc(202)] VaapiVideoEncodeAccelerator():
[19316:19316:0305/163317.489375:VERBOSE2:vaapi_video_encode_accelerator.cc(206)] ~VaapiVideoEncodeAccelerator():
[19316:19316:0305/163317.489392:VERBOSE2:vaapi_jpeg_decode_accelerator.cc(306)] ~VaapiJpegDecodeAccelerator(): Destroying VaapiJpegDecodeAccelerator

screenshot of media-internals with iHD
Does this mean it is working?

Hey @colemickens.

Attempt 1 Naive:
[cole@xeep:~]$ unset LIBVA_DRIVER_NAME # it's not normally set
[cole@xeep:~]$ GDK_BACKEND=x11 /nix/store/65n4m3rkn9q2bfcrmz2xpp61823v7cwm-chromium-72.0.3626.121/bin/chromium --log-level=9 --enable-logging=stderr --v=9 &> log.txt
[cole@xeep:~]$ cat log.txt | grep libva
libva error: va_getDriverName() failed with unknown libva error,driver_name=(null)

Attempt 1 Explicit: i965
[cole@xeep:~]$ ls /run/opengl-driver/lib/dri | grep drv_video | grep i965
i965_drv_video.la
i965_drv_video.so
[cole@xeep:~]$ export LIBVA_DRIVER_NAME=i965
[cole@xeep:~]$ GDK_BACKEND=x11 /nix/store/65n4m3rkn9q2bfcrmz2xpp61823v7cwm-chromium-72.0.3626.121/bin/chromium --log-level=9 --enable-logging=stderr --v=9 &> log.txt
[cole@xeep:~]$ cat log.txt | grep libva
libva error: /run/opengl-driver/lib/dri/i965_drv_video.so init failed
[17809:17809:0305/163001.156500:ERROR:vaapi_wrapper.cc(324)] vaInitialize failed: unknown libva error

I'm wondering if this is related to my local workaround for #55975.

That's weird, at least libva is resolved and in the right library search path, which is better the current state of my tree.
I added
runtimeDependencies = optional VAAPISupport [ libva ];
but the dlopen call is still failing. Would you mind sharing your solution?

Attempt 2 Explicit: iHD
If you have intel-media-driver installed in nixos's opengl drivers:
[cole@xeep:~]$ ls /run/opengl-driver/lib/dri | grep drv_video | grep iHD
iHD_drv_video.so
[cole@xeep:~]$ export LIBVA_DRIVER_NAME=iHD
[cole@xeep:~]$ GDK_BACKEND=x11 /nix/store/65n4m3rkn9q2bfcrmz2xpp61823v7cwm-chromium-72.0.3626.121/bin/chromium --log-level=9 --enable-logging=stderr --v=9 &> log.txt
[cole@xeep:~]$ cat log.txt | grep libva
[cole@xeep:~]$ cat log.txt | grep --text vaapi
[19095:19095:0305/163315.855345:VERBOSE2:vaapi_video_encode_accelerator.cc(202)] VaapiVideoEncodeAccelerator():
[19095:19095:0305/163315.855375:VERBOSE2:vaapi_video_encode_accelerator.cc(206)] ~VaapiVideoEncodeAccelerator():
[19095:19095:0305/163315.855389:VERBOSE2:vaapi_jpeg_decode_accelerator.cc(306)] ~VaapiJpegDecodeAccelerator(): Destroying VaapiJpegDecodeAccelerator
[19095:19095:0305/163317.384730:VERBOSE2:vaapi_video_decode_accelerator.cc(198)] Initialize(): Initializing VAVDA, profile: h264 main
[19095:19312:0305/163317.385901:VERBOSE2:vaapi_video_decode_accelerator.cc(446)] DecodeTask(): Decoder requesting a new set of surfaces
[19095:19095:0305/163317.385941:VERBOSE2:vaapi_video_decode_accelerator.cc(499)] InitiateSurfaceSetChange(): Initiating surface set change
[19095:19095:0305/163317.385964:VERBOSE2:vaapi_video_decode_accelerator.cc(543)] TryFinishSurfaceSetChange(): Requesting 11 pictures of size: 1280x720
[19316:19316:0305/163317.489346:VERBOSE2:vaapi_video_encode_accelerator.cc(202)] VaapiVideoEncodeAccelerator():
[19316:19316:0305/163317.489375:VERBOSE2:vaapi_video_encode_accelerator.cc(206)] ~VaapiVideoEncodeAccelerator():
[19316:19316:0305/163317.489392:VERBOSE2:vaapi_jpeg_decode_accelerator.cc(306)] ~VaapiJpegDecodeAccelerator(): Destroying VaapiJpegDecodeAccelerator

screenshot of media-internals with iHD
Does this mean it is working?

It does look like libva intialization is successful but the screenshot contradicts that as it looks like FFmpegVideoDecoder is actually used. AFAIK, HW-accelerated decoding goes through MojoVideoDecoder or GpuVideoDecoder.
Did you patch the enable-mojo patch in ?
Thanks tor your time, dude

That's weird, at least libva is resolved and in the right library search path, which is better the current state of my tree.
I added
runtimeDependencies = optional VAAPISupport [ libva ];
but the dlopen call is still failing. Would you mind sharing your solution?

Ok, so I had a look at your patch and you dont use the runtimeDependencies mechanism which doesn't work in my case but an ad-hoc tweaking of the executable wrapper building. I am gonna try that.
It might be interesting to generalize and abstract that away in the long term to make it reusable.

Ok, so I obviously messed up with git. I am closing this PR and create a new one. Sorry about that.


Support fields of type uuid

Closes #34

See https://github.com/tarantool/ddl/pull/26
Changelog/Readme should be updated. And I assume that tests could be extended in the same way as it was done it MR above.

https://github.com/tarantool/ddl/blob/93d779e1950d67e964eb1bcf8a29d57be96d1e3b/ddl/check.lua#L283-L293
I think it should be added to the scalar types too.
(And tested appropriately)

UUID is not a scalar type, according to the sources:

UUID is not a scalar type, according to the sources:

And tests https://github.com/tarantool/tarantool/commit/b238def8065d20070dcdc50b54c2536f1de4c7c7#diff-ac3bcfa197d7f09e185b4f93a4d8e869R1392


Data Connector for Armorblox Solution

Looks like the KqlValidations are failing even when run for the master branch of the Sentinel repo. Not sure if these failures have got anything to do with this PR.

Please provide sample data in JSON format.

Please provide sample data in JSON format.
We have used .csv format so no need to update JSON format. Thank You.

@NikTripathi Thanks for the review! All review comments have been addressed.

During the deployment function app is loading. But function is not loading in portal. I think it is a problem because of the structure of zip file. Please have a look on the zip file structure of CiscoUmbrella. Let me know if something needs to done by my end.

@NikTripathi I'm facing the same issue, and I'm trying to figure it out. It worked for the first time, but subsequent deployments have this issue.
I faced the same issue even when I just deployed the function app for testing. Changing the name of the function app fixed the issue for me. Any ideas on this?

@NikTripathi I'm facing the same issue, and I'm trying to figure it out. It worked for the first time, but subsequent deployments have this issue.
I faced the same issue even when I just deployed the function app for testing. Changing the name of the function app fixed the issue for me. Any ideas on this?

I changed the function name and redeployed it. Still facing the same issue. Function deploys successfully but function is not loading.

@NikTripathi We tried the steps under Manual Deployment of Azure Functions section and that seems to work correctly - we are able to see the function app files load in the portal after deploying via VSCode. Only when we try the ARM template route using the Deploy To Azure button, it seems to show the behaviour that you are seeing with the deployment being successful, but function not loading. We are unable to spot any material differences between the Cisco one and ours. We also cross-checked against a couple of others like SentinelOne for e.g.

@NikTripathi It looks like there was an extra top-level folder in the zip file. I was able to test it through template-based deployment, and it worked properly. Let me know if you find anything else.

@ujayant - Function gets deployed successfully from template. Invocation logs show error that 'requests' module is missing. I am checking whether there is version issue at my end. Is it running good at your end without error? I will give you update about the version issue at my end by EOD. Please let us know about function invocation result  at your end. Thanks.

Provided zip package doesn't have .python-package folder. I added the python package and tested it. Can you please update the old zip package with the attached zip package.
ArmorbloxAzureFunction.zip

@NikTripathi I've updated and tested this end-to-end. Everything is working fine.


can i use this with angular 2

can i use this with angular 2 environment?

scratch that just found https://github.com/auth0/angular2-jwt


When I send a "get" request, why is there a "=" before the request parameter?

When I send a "get" request, why is there a "=" before the request parameter?

AFN version: 3.x.

For general help, please ask on StackOverflow.


Replace forum links

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[ ] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #39138

This pull request should have replaced all instances of "www.freecodecamp.org/forum" and "freecodecamp.org/forum" with "forum.freecodecamp.org", per the linked issue.

Ah! My apologies - I had thought it best to do it all at once, since it seemed to be rather time-sensitive. I will work on splitting them up now.

@nhcarrigan is this just a straight replacement of the two strings you mentioned?  I'm looking at other links (in the nav and so on) now, and don't want to clash.

We would prefer to merge the English and the Chinese and platform files (if needed) first.

That is correct. We would like to fix everything that is deployed to be merged in first.

I think if you dropped the changes to client/src/templates/Challenges/components/HelpModal.js in this PR and moved it to a different PR, we could merge this one as is after the fact. The redirect handling would take care of it in the interim.
The modal is more critical, because it sends direct traffic to the forum.

@raisedadead I revered the change to the HelpModal.js file, as requested.
@ojeytonwilliams Correct - this is a straight replacement of the link strings.


Memory Bank 🗃

memory bank, specifically for locations for now. we can expand in the future

Looks good. I added an example to the profile since it was not working for me at first unless I manually did it.

oh yeah I think I must have tested with gpt 4 and it got it without examples


 does not support changing store on the fly.

Hey, first of all thanks for a great library!
I have a question:
When using with react-redux and hot reloading, this warning pops up in the console:
<Provider> does not support changing `store` on the fly. It is most likely that you see this error because you updated to Redux 2.x and React Redux 2.x which no longer hot reload reducers automatically. See https://github.com/reactjs/react-redux/releases/tag/v2.0.0 for the migration instructions.

I'm not sure if this has a practical implication, or it's just the warning, but in the link they suggest you do
 if (module.hot) {
    // Enable Webpack hot module replacement for reducers
    module.hot.accept('../reducers', () => {
      const nextRootReducer = require('../reducers/index');
      store.replaceReducer(nextRootReducer);
    });
  }

which of course is not possible in repatch.
Is it anything to worry about, or should I just disable the warning?
(And if you know, how can I disable the warning?)
Cheers!

Hey,
you guess right, it does not make sense. In Repatch the actions are also the reducers on their own, so the store remains unchanged during the application hot-reloading phases, therefore you can ignore this warning.
Does it drop this warning even in production?

No, only when using hot-reloading so it won't show up in production, and also I think react-redux doesn't log much if NODE_ENV is production

What does your entry file look like? I use repatch with the HotModuleReplacementPlugin and do not get such a warning.
I have this at the end of my App.js, so maybe these are the magic lines.

if (module.hot) {
module.hot.accept();
}

This is my entry point:
const store = createStore(initialState);

ReactDOM.render(
  <Provider store={store}>
    <App />
  </Provider>,
  document.getElementById("root")
);

// Hot Module Replacement
if (module.hot) {
  module.hot.accept();
}

I'm using parcel with the built-in HMR. Maybe there's a difference

I have the Provider in a separate ProviderWrapper that's where your Provider is. That ProviderWrapper looks like
<Provider store={createStore(initialState)}>
  <MoreProviders>
    {props.children}
...

No idea why this would change anything, though, just guessing what it could be.


Laravel 5.5 vendor:publish fails

In Laravel 5.4 Route::controllers do not exist anymore. So

vendor:publish

fails with
[InvalidArgumentException] Attribute [controllers] does not exists.

Anyone??

Same error here

Unfortunately, I have not had a need in quite some time for SimpleSAML integration in a laravel project. If you feel you can submit a pull request that would fix this, please feel free. This project would most likely require a fairly large rewrite....and really should have tests in it to be compatible with L5.5.
The issue you are seeing is resulting from large Laravel changes from the version this package was originally intended for


Template request - jobs

Request: a template - possibly a pop up when a job is being performed so that players understand who/why dudes can be in a posse for specific jobs.

I am not sure about this. First, it needs more information how you think it should look like, but mostly I am not sure we need a popup. I will convert this to discussion and see how it turns out.


Part 1 of #98 - Updates to coron1pipeline

Part 1 of #98.
Added a number of updates to the default coron1pipeline, most importantly the exclusion of dark subtraction for subarrays. This is implemented through a new function run_single_file which sets a number of best-practice defaults, such as skipping darks, IPC, and persistence steps. It also sets some defaults for jump thresholds, saturation flagging parameters, and pseudo-ref pixel rows / columns. run_obs now calls run_single_file.
All of the changes here should not change the outcome of existing notebooks and scripts. The only thing I've done is include better defaults, added more options for simplicity, debugging, and and fixed some known bugs within the pipeline itself.
New pipeline options:

Can now save intermediate step files via pipe.save_intermediates=True. Similar to the already existing pipe.save_save_calibrated_ramp but for all steps.
The saturation step now flags RC pixels. Not sure why this isn't included in the default pipeline, but these pixels are persistently hot pixels and should be flagged automatically.
The pipe.rate_int_outliers utilizes a new cube_outlier_detection function that flags outliers by comparing ramp integrations within an exposure with at least 5 integrations.
The pipeline can now return the rateint datamodel instead of the default rate datamodel by setting pipe.return_rateints=True. Great for pipeline processing of a single file where you don't actually want to immediately inspect the results without have to save FITS files.

Changes to existing pipeline steps:

The pseudo refpix correction had some bugs within the indexing logic. I have cleaned it up to make the indexing more consistent and explicit.
The saturation step now uses a new generic expand_mask function that will grow a binary mask using scipy's binary_dilation function, which is more efficient than array shifting via numpy.

Changes to runobs:

Calls run_single_file so inherits those defaults described above.
Added various verbosity options for those who dislike the deluge of log outputs

verbose keyword set to True shows everything at INFO level or above from the pipeline. Setting this to False will show various SpaceKLIP log messages, but suppress pipeline message below ERROR.
quiet suppresses all messages below ERROR and employs progress bars to keep track of data reduction progress.
runobs Skip pipeline processing if data results already exist in output directory rather than re-running unnecessary and overwriting. Can be over-riden using the overwrite keyword.

I realized there were going to be a number of merge conflicts between this PR and 1/f noise correction PR, so I have preemptively merged #102 into here.
Correct 1/f noise striping in stage1:

Create fnoise_clean.py file that houses code for fitting a 1/f noise model to data and iteratively masks outlier pixels.
Add cube_fit function, which utilizes polynomial fitting routines from webbpsf_ext to fit linearized unsaturated and unflagged pixel data to quickly generate a kTC bias and slope images.
Utilize cube_fit to create kTC bias to subtract from linearized ramp data to create clean group-level data
The slope images from cube_fit are then averaged together and used to produce a high-SNR ramp such that group signal levels match those within a given integration.
Use 1/f noise filtering algorithm on group level data to model the spatial 1/f striping and subtract it.

1/f cleaning routine adds about 100 msec for a single 320x320 frame. Expect time to increase proportional to frame size. See issue #97 for more information on operations.


Feat/create get journal attachments function

In this pull request we created a function _get_ledger_attachments that find a real attachments that is linked to the ledger entry.
CashCtrl doesn`t provide us with direct way to retrieve path of file that is linked to the ledger entry.
This function will come in hand for completing the cashctrl_ledger#7 issue.
@lasuk Please review the changes and provide feedback or approval for merging.

Thanks.

Added a commit to enhance code readability. Pleas shout if you disagree.
Added a suggestion for a simpler test setup as a comment.

Back to you, @AlexTheWizardL.

This commit covers following changes:

add_ledger_entry() now returns an Id of created ledger entry. updated returning type and docstring
implemented the test logic that you described in the comment above

Thanks.

Added one minor change.
Ready to merge.


JSDoc info not showing in VSCode on exports.default

I have a very strange experience with VSCode and JSDocs.
JSDocs information is shown for named exports, but not for default exports with the commonJS export method.
Problem: No info popup on exports.default
File: index.js
/**
 * Some explanation
 *
 * @param {object} obj The obj has to be~~~ !
 * @returns {void}
 */
function fn (obj) {
  // ...
}

exports.fn = fn;
exports.default = fn;

Then in another file:
Case 1: import import
    import fn from './index'
    fn // → no JSDoc info...

    import { fn } from './index'
    fn // → JSDoc info works

So it works on named import, but not on default.
Case 2: require() import
    let fn = require('./index')
    fn // → no JSDoc info...

    let fn = require('./index').fn
    fn // → JSDoc info works

So it works on named import, but not on default.
Gif to show exactly what happens

Related my question on StackOverflow: https://stackoverflow.com/questions/52511753/jsdoc-broken-on-exports-default-in-vscode

You shojld post that to VScode.
https://github.com/Microsoft/vscode
This repo here is a tool to create API documentation from JSDoc comments and has nothing to do with whatever VSCode does or doesn't do.

Discussion of VSCode


Rate limits for messages/host requests/friend requests

To avoid spam accounts, we should implement rate limits for

messages
host requests
friend requests

These should be generous to not disturb regular users, but prevent accounts to just spam via scripts.
Open question - what should happen when a rate limit is reached?

Just show notification and deny action
Issue a warning and maybe automatically block repeat offender, as it is quite likely to be a malicious actor

IMO, just showing notification and denying action is good enough, while logging the occurrence in Grafana.


PDF prediction / conversion does not work on Intel CPU Mac

This command just emits an empty mmd file on a Intel Mac running nougat on CPU:
PYTORCH_MPS_HIGH_WATERMARK_RATIO=0.0 PYTORCH_ENABLE_MPS_FALLBACK=1 nougat -m 0.1.0-small -o out --recompute Word2Vec.pdf
[MISSING_PAGE_EMPTY:1]

[MISSING_PAGE_EMPTY:2]

[MISSING_PAGE_EMPTY:3]

[MISSING_PAGE_EMPTY:4]

[MISSING_PAGE_EMPTY:5]

[MISSING_PAGE_EMPTY:6]

[MISSING_PAGE_EMPTY:7]

[MISSING_PAGE_EMPTY:8]

[MISSING_PAGE_EMPTY:9]

[MISSING_PAGE_EMPTY:10]

[MISSING_PAGE_EMPTY:11]

[MISSING_PAGE_EMPTY:12]

[MISSING_PAGE_EMPTY:13]

Model Name:	MacBook Pro
Model Identifier:	MacBookPro16,1
Processor Name:	8-Core Intel Core i9
Processor Speed:	2.3 GHz
Number of Processors:	1
Total Number of Cores:	8
L2 Cache (per Core):	256 KB
L3 Cache:	16 MB
Hyper-Threading Technology:	Enabled
Memory:	16 GB
System Firmware Version:	19<IP_ADDRESS>.0 (iBridge: 20.16.6072.0.0,0)
OS Loader Version:	540.120.3~22
MacOS Monterey

Does it work with adding the --no-skipping flag?

No change. I get the same results.

same problem, mine is Apple M1 Pro


user_setting: Add "Show unread count for" display setting.

Added the new feature of letting the user choose whether he/she wants to see unread messages count on the left sidebar for streams.
This setting has three options,
"All Streams" - This will show the unread messages count for all streams.
"Unmuted Stream" - This will be the default option and it will only show the unread messages count for unmuted streams. "No Stream" - This option will not show the unread messages count on any stream.

Fixes: #24149
CZO

Screenshots and screen captures:

Light Theme

Light Theme on hover

Light Theme with Mention

Dark Theme

Dark Theme on hover

Video Capture

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Hi @alya , This PR is ready now for your review please let me know your suggestions, Thanks! 🙂

@Lalit3716 As noted in the PR template, please include still screenshots, in addition to any videos. I'd like to see screenshots from the left sidebar, both in light and dark theme, with and without hover over the relevant stream. Thanks!

@alya Sure I uploaded still images in the PR description, please check them out 🙂 !

Thanks! For the hover behavior, we don't want a tooltip. We just want the unread count to show up like it does in other streams when you hover anywhere on the stream row.
Looks good otherwise.

Oh Okay! I thought we meant to show a tooltip 😅.
I will fix this right away!

@alya Fixed it now, also attached relevant videos in the PR description please have a look, Thanks!

Thanks! The functionality is inconsistent when I test it. E.g., when I pin a muted stream, the counter reappears, though later I saw it disappear again:

I'm also not always getting the counter on hover:

@alya Sorry my fault, Fixed it, now everything works as expected when pinning/muting/searching streams with the setting on, please test it now, thanks!
But when working on this I found another bug on the main branch related to unread counts and unmuting streams, I have explained the bug in this
CZO thread please check it out also! Thanks.

Thanks for working on this @Lalit3716! I posted what I hope is a pretty complete review of details you can rework to get this into a nice state. I would expect a pure CSS implementation to also eliminate most risk of future "inconsistent hover behavior" bugs.

@Lalit3716 Please post a comment when you've addressed @timabbott 's feedback above and have manually tested your work, and I'll do another testing pass then. Thanks!

Sure I will work on the changes right away.

@timabbott I have fixed all those changes but I am now getting no 100% test coverage on node test need some help there other than that every thing looks good to me now except the bug I have reported here CZO.

but I am now getting no 100% test coverage on node test need some help there

Have you asked for help on  CZO?

Asked it now here!

@timabbott I have addressed your review and fixed them please take a look again!
@alya This is ready for another manual testing now can you please check it once more, thanks!

Should I squash the commits @timabbott here ?

Yes, you should squash these commits; they aren't atomic in a useful way. It's usually not helpful to have commits that just fix the bugs in a previous commit in that same PR -- they just make it harder to read and understand the series of changes we'd be making to main. (To be clear, it may well be worth having preparatory refactoring commits in a PR like this that adjust the structure of the code without functional changes in order to make the PR reviewable, but these aren't that).
Check out the Zulip commit guidelines for more details.

@timabbott Cleaned up the commit history and fixed the migrations numbering, please have a look!

The unread count should be shown on hovering anywhere over the stream row. Currently, it's not shown when the mouse is to the right of the count.

Also, the menu dropdown needs to be styled like other dropdowns;

Otherwise looks good!

@alya Fixed the issues you mentioned, thanks for reviewing it!

I'll retest when conflicts have been resolved. Thanks!

@alya Resolved the conflicts!

Thanks! Some notes:

There shouldn't be any changes to the horizontal alignment of unread counts. Things are misaligned:

Unread counts for topics are getting hidden, which is not correct. As noted in the issue:

Unread counts for topics within streams will still be shown in the left sidebar and Recent conversations (no changes)

Please update the PR and do a round of careful manual testing. See https://zulip.readthedocs.io/en/latest/contributing/code-reviewing.html#manual-testing.

Thanks for the update! Playing with the UX some more, let's just always show the count for the active stream (i.e., once you are viewing topics in the stream). The same should apply to the "more topics" view as well.

Hi @alya I agree that we should always show the count when a stream is active but it's something that seems a little tricky to implement with respect to my current implementation of the setting, so I would like some suggestions/advice on how to approach this maybe by @timabbott and hence I have opened a CZO thread for it, Thanks!

Thanks for the update! Playing with the UX some more, let's just always show the count for the active stream (i.e., once you are viewing topics in the stream). The same should apply to the "more topics" view as well.

Hi, @alya regarding this I have figured out a solution that will show unread counts irrespective of the display setting if a stream is active, but currently it respects the setting if we are in a topic view it is due to the fact that we don't have any reliable class/id in CSS which tells us that the stream is active despite being in a topic view because currently we removes active-filter class when in a topic view.

Is there a question for me here? I really can't comment on the technical approach.

Oh okay, no worries I will try to come up with something then.

@alya This is ready for another manual review. I have tested it on my end and everything works as expected now 🙂.

@timabbott this is ready for a code review.

Made a prep commit for a new CSS class.
Cleaned up some work.

@alya Updated the screenshots and videos and also included "When a stream is active" gif!

Nice, looks good to me in manual testing now! @amanagr would you be up for reviewing this one?

There isn't necessarily something to do right now, but just noting here that we should be mindful of how this interacts with the following point from #24243.

If there are unread messages in an unmuted topic within a muted stream, replace the unread counter for the stream with a counter that just adds up all the unread messages in unmuted topics in the stream. The counter should be shown in regular color (not faded). When there are no unread messages in unmuted topics, we should show the stream counter as we currently do.

I think in this situation, it's probably best to show the unread counter even if the user turned off unread counters for muted streams.

Updates:

Rebased the PR with better commit history.
Reworked the logic for live update of the setting and now it's pure css-based!
Handled the case when there are unread messages in an unmuted topic within a muted stream, in this case, we will just display the unread count for unmuted topics regardless of setting.

Video

Hey @Lalit3716, I noticed you made some updates to the existing changes. Is there anything specific we should know before the integration review?

@timabbott Changed the settings field name to be web_stream_unreads_count_display_policy and addressed all other comments, this is ready for another review now.

I renumbered the migrations and applied this change:
diff --git a/web/styles/left_sidebar.css b/web/styles/left_sidebar.css
index 3c3fc439a2..95dccc6b49 100644
--- a/web/styles/left_sidebar.css
+++ b/web/styles/left_sidebar.css
@@ -153,8 +153,11 @@ li.show-more-topics {
         }

         & .masked_unread_count {
-            /* We add +4px margin for masked unread count span to make it center aligned with unread_count span which has a padding of 4px horizontally in addition to 15px margin. */
-            margin-right: 19px;
+            /* TODO: In theory this should be 19, to make it center aligned with
+               unread_count span which has a padding of 4px
+               horizontally in addition to 15px margin. But 18px looks
+               better centered. */
+            margin-right: 18px;
         }

         &.stream-with-count {

With this changes, this seems to work well and I've marked it to merge once CI passes, thanks for all the work on this @Lalit3716!
Depending on how it goes on chat.zulip.org, we may end up disabling it for further fiddling.
@karlstolley I suspect the CSS part of this (basically just the last commit) may be something you'll want to rewrite as part of general left sidebar work, but I think it's broadly within the theme of the existing implementation, so I didn't think it worth blocking on.
@drrosa can you work on a /help/ PR for this new feature?


Remove "Trying METAL rendering..." from every single new ComposeWindow launch

Every single new ComposeWindow launch is accompanied with "Trying METAL rendering..." in the console output

Fixed in Skiko https://github.com/JetBrains/skiko/commit/a1246846ca60605389fe0109cabc83c3fd644d72 - will be merged with next Skiko update.

Fixed in 0.5.0-build245

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


update readme - install in usermode

The current installation method requires elevated rights as the binary is installed system wide. This change adds the --user flag so the binary will be installed in the user context without the need for root (like the old install.py)

I think I would prefer to go the other way: keep the instructions as-is, but point out the --user flag. Pretty much every python project I have see does it this way, so I'm just aiming to be consistent.

Going to merge anyway. Don't feel strongly enough to make a case against you :) Thanks!

cool thanks! :D


not work  Service External Links

Useful
Info

Versions
Rancher v1.2.0-pre4-rc15 Cattle: v0.173.6 UI: v1.2.31

Access
github admin

Orchestration
Cattle

Route
service.containers

Update
Rancher v1.1.4 and v1.2.0-pre3 work Service External Links

I'm experiencing the same issue with Rancher 1.2.0.

not work 1.2.1

Are your images alpine:3.4 based?

not work v1.2.2-rc2

mccricardo

Are your images alpine:3.4 based?
all services

@andruwa13 Could you open a shell to the phpMyAdmin container and run ping www.google.com? Please paste the output here.

@andruwa13 we had the same issue. There are known issues with alpine 3.4. Try including:
dns_search:
  - rancher.internal

in your docker-compose.yml

I am closing this as a pre-duplicate of https://github.com/rancher/rancher/issues/7026 where the cause of this issue has been identified as relating to using Alpine base images.

docker-compose.yml
version: '2' services:   phpmyadmin:     mem_limit: 268435456     image: phpmyadmin/phpmyadmin:latest     environment:       PMA_ARBITRARY: '1'     stdin_open: true     external_links:     - bankoved/db:db     ports:     - 4444:80/tcp     memswap_limit: 536870912     cpu_shares: 200     labels:       io.rancher.container.pull_image: always
not work

@mccricardo thank you, work

@mccricardo thank you, work


Fix the property expression selection evaluation, add relative vs absolute match

The property paths of the property expressions should be from ROOT to avoid cyclic matches

Property(...) from the root
Property(..., MatchType.Relative) anywhere

Requirement changes, the API would be misleading, the property expressions will be only applied from the root


[MINOR] Fix typo char in log4j conf template

What changes were proposed in this pull request?
Fix typo.
Why are the changes needed?
Fix typo.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests.

Can one of the admins verify this patch?


webdeploy improvements

Separate project, or just part of mffer?
subcommands:

clean: remove .clasp.json files
restore: connect to given existing deployment
undeploy: remove existing deployment/files/projects/scripts

Connect (within mffer or as separate project) to google to allow doing these through api +/- clasp?

deploy final (non-debug) version as an option