fix: fix bug where search cannot be done without fully clearing input

Overview

Bug discovered where user cannot perform a search without clearing the text box

Fixes https://github.com/maplibre/maplibre-gl-geocoder/issues/73

[x] briefly describe the changes in this PR

[x] write tests for all new functionality

[x] run npm run docs and commit changes to API.md

[x] update CHANGELOG.md with changes under main heading before merging

@wipfli yes! my bad forgot that the issue was filed in the ui repo - https://github.com/maplibre/maplibre-gl-geocoder/issues/73


Doesn't supports Barcodes scanning, only scans QR Codes

It seems like this library doesn't support barcodes as I wasn't able to get them scanned through this. However, was able to get QRCodes scanned. Is this true, or I haven't configured this correctly?

duplicated  of https://github.com/rodgomesc/vision-camera-qrcode-scanner/issues/1


Add system level test class

[X] This contribution adheres to CONTRIBUTING.md.

TODO: Check the above box with an 'x' indicating you've read and followed CONTRIBUTING.md.
What does this Pull Request accomplish?
Adds the System Level Class infrastructure.
Why should this Pull Request be merged?
To allow other developers to add system level tests.
What testing has been done?
The class has been used with VI tester to validate the functionality.

Rename "TCP_Listener" to "TCP Listener", to keep in accordance with LV guidelines on file naming.

"TCP_Listener.vi" has been renamed to "TCP Listener.vi"


Support for 'square' grid cells?

Thanks for submitting an issue to RGL!
Please mark the type of this issue:

[x] Feature Request

Hi. I am using RGL to create a dashboard with square grids, and it seems that setting a rowHeight the same as colWidth requires some pretty complex code to be written for anyone who needs it. For reference: https://github.com/STRML/react-grid-layout/issues/399.
I'd like to propose something like a rowHeightAspectRatio prop which would let users define the height as a ratio of the column width. So, for a 'square' grid, they'll set the ratio as 1.
@STRML Would this be something you'd like to be included in the library? I can submit a PR if you find this interesting.

Considering this can be done outside the library, I'd rather not include it
inside and bloat the options.
On Oct 12, 2017 2:49 AM, "Kumar Harsh"<EMAIL_ADDRESS>wrote:

Thanks for submitting an issue to RGL!
Please mark the type of this issue:

Feature Request

Hi. I am using RGL to create a dashboard with square grids, and it seems
that setting a rowHeight the same as colWidth requires some pretty complex
code to be written for anyone who needs it. For reference: #399
https://github.com/STRML/react-grid-layout/issues/399.
I'd like to propose something like a rowHeightAspectRatio prop which
would let users define the height as a ratio of the column width. So, for a
'square' grid, they'll set the ratio as 1.
@strml Would this be something you'd like to be included in the library? I
can submit a PR if you find this interesting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/STRML/react-grid-layout/issues/644, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABJFP1u7QTFkxjHC97YlsJsrFwawwEl4ks5srcSYgaJpZM4P2mcD
.

You can calculate what your cell width will be by dividing viewportWidth / numOfCols and provide rowHeight accordingly

Yes, that is what I'm doing currently. I was asking if it'd be worth it to include support in core where such a change would be far simpler than everyone maintaining a separate copy of similar code to create square grids.

I would love to see this feature included in RGL.


fix: Helm fails to render the chart

I tried to migrate my local installation to the upstream version today and ran into this issue. This happens when the PrometheusRule feature is enabled:
Error: malformed chart or values:
        templates/: template: promitor-agent-scraper/templates/prometheusrule.yaml:5:20: executing "promitor-agent-scraper/templates/prometheusrule.yaml" at <{{template "promitor-agent-scraper" .}}>: template "promitor-agent-scraper" not defined

Unfortunately I overlooked this little change.
Sorry for the inconvenience 😢
Fixes #

No problem, I've hit approve so it's me to blame ;)
I'll ship it asap.


⚠ Bofrost.at has degraded performance

In a2cb7f5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1332 ms

Resolved: Bofrost.at performance has improved in 7cd89b4.


Preparing release 0.4.5

Note that now we have changelog!
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #213 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #213   +/-   ##
=======================================
  Coverage   89.32%   89.32%
=======================================
  Files         109      109
  Lines        3026     3026
  Branches      308      308
=======================================
  Hits         2703     2703
  Misses        265      265
  Partials       58       58

Impacted Files
Coverage Δ

emukit/__version__.py
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e267da...e7f3b9f. Read the comment docs.


Updated neli.

This PR updates neli to the latest upstream version, along with all the builder types.

Depending on a git version of neli is not planned.

I plan on keeping this PR open until those changes in neli are released to crates.io, when that's the case, I'm going to commit a change to make this rely on the release. I just needed these changes for a personal project and just opened a PR, to avoid having to do it twice.

Note: I don't think using Box<dyn Error> everywhere is a good idea.

I was thinking the same, however I don't know of another option, since there are always two or more different error types.

You can create an enum with these error types as variants. To make life a bit simpler, you can use thiserror crate to derive some impls (like Display and From for the variants).


Example response body has duplicated key

The given example of response body has duplicated "productName" key

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

ID: 24518ab0-bd10-1a04-f3a0-1ba754409841
Version Independent ID: ad632005-a43b-8d81-c4ef-8730d59b4363
Content: Create a new product - Windows drivers
Content Source: windows-driver-docs-pr/dashboard/create-a-new-product.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @balapv
Microsoft Alias: balapv

Looks like this one was fixed.  Thanks for the report @rosamariaramirez and apologies for the delay.


NOISSUE - Remove ARM multi-arch images

Signed-off-by: Drasko DRASKOVIC<EMAIL_ADDRESS>Pull request title should be MF-XXX - description or NOISSUE - description where XXX is ID of issue that this PR relate to.
Please review the CONTRIBUTING.md file for detailed contributing guidelines.
What does this do?
Removes multi-arch image push to DockerHub.
Only amd64 images are produced and pushed.
It should be noted that manifest currently was hard-coded to ARMv7, and besides that it is suitable for RPi, it does not have too much benefit, nor it is generic. It should be left to developers to produce Docker images for their particular architectures.
Also removes CI build of ARM images.
Which issue(s) does this PR fix/relate to?
Put here Resolves #XXX to auto-close the issue that your PR fixes (if such)
List any changes that modify/break current functionality
Have you included tests for your changes?
Did you document any new/modified functionality?
Notes

Codecov Report

Merging #929 into master will decrease coverage by 0.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #929      +/-   ##
==========================================
- Coverage   83.85%   82.89%   -0.96%
==========================================
  Files          75       75
  Lines        5667     5385     -282
==========================================
- Hits         4752     4464     -288
+ Misses        655      654       -1
- Partials      260      267       +7

Impacted Files
Coverage Δ

things/postgres/things.go
79.82% <0%> (-7.11%)
:arrow_down:

things/service.go
84.93% <0%> (-7.07%)
:arrow_down:

things/postgres/channels.go
81.78% <0%> (-3.62%)
:arrow_down:

things/redis/streams.go
92.4% <0%> (-0.77%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 28a176a...c133049. Read the comment docs.


Random Advice Generator

Description 📜
The idea behind the project
Like in javascript we have many projects such as random quote generators and password generators and many more so I took the basic idea from there only.
How does it works
You just need to load the file in the browser you will have an interface as a container containing a message of Click the refresh and below three is refresh button by doing so, as it API project then it will request data from the server after fetching it you will have random advice in your container.
Fixes #1095

Type of change 📝
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] Add a new project
[x] Modified or enhanced the existing projects
[x] Improve the readme of any project

Domain of Contribution 📊
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] Project using HTML, CSS & JS

Checklist ✅
Please delete options that are not relevant. And in order to tick the check box just put x inside them for example [x] like this

[x] I have performed a self-review of my own code or work.
[x] I have commented on my code, particularly in hard-to-understand areas.
[x] My changes generate no new warnings.
[x] I'm GSSoC'22 contributor

Screenshots / Gif (Recommended) 📸

https://user-images.githubusercontent.com/73521123/166118421-a9557a43-ce24-49d5-aedb-12832e2b5eb9.mp4

plz, ignore refresh* meaning I have updated


Fix travis by requiring 1.22.0

Fixes https://github.com/dart-lang/observable/issues/34

@matanlurey should we publish this?

Potentially. This is still versions behind internally so it hasn't been a priority.


Feature request: Change gap in between numbers

Is there a way (or could you implement a way) to decrease the padding in between the numbers?
Both vertically and horizontally.
Thanks!

Hey @fredrikburmester!
This is possible by adjusting the width of the pickerItemContainer in the styles property. For example:
styles={{                         pickerItemContainer: {                             width: 100                         },                     }}
You will also likely need to adjust the styles of pickerLabelContainer to re-align the labels. They're absolutely positioned so you'll need to adjust top/bottom/left/right. There is a minor limitation with the current release that stops those labels overflowing their container; I'll release a patch today that addresses that.
Here's a full example of a TimerPicker with a narrower width and the labels centred and adjusted.
<TimerPicker
    padWithNItems={2}
    hourLabel=":"
    minuteLabel=":"
    secondLabel=""
    LinearGradient={LinearGradient}
    styles={{
        theme: "light",
        pickerItem: {
            fontSize: 34,
        },
        pickerLabel: {
            fontSize: 32,
            marginTop: 0,
        },
        pickerContainer: {
            marginRight: 6,
        },
        pickerItemContainer: {
            width: 100,
        },
        pickerLabelContainer: {
            right: -20,
            top: 0,
            bottom: 6,
            width: 40,
            alignItems: "center",
        },
    }}
/>;

No there isn't, but there should be! I'll put out another release this weekend that adds those props and the ability to add styling for the picker when in the disabled state.

Hey @fredrikburmester, I've added that feature. There's three props (one for each picker):

hoursPickerIsDisabled
minutesPickerIsDisabled
secondsPickerIsDisabled

You can adjust the styling of a disabled picker with the style prop disabledPickerContainer. The default is {opacity: 0.4}.
Let me know if that fits the bill!


FIRMessaging.requestPermissions undefined

Hi,
I'm getting some errors when I included your sample code. What am I missing here?

looks like the native module is not linked properly. Did you do rnpm link? Can you see if native module package is called?

do you mean npm link? is "r" a typo?
If so, I've run it, yes. How can I check if the package is getting called? Do I need to add the package to the package list here?
@Override
protected List<ReactPackage> getPackages() {
    return Arrays.<ReactPackage>asList(
        new MainReactPackage(),
        new RNSimpleAlertDialogPackage(),
        new ReactNativeMapboxGLPackage()
    );

it is not a typo. https://www.npmjs.com/package/rnpm
install globally and run rnpm link react-native-fcm. It will link native code into your android/ios solutions

Got it to work + you teached me that there is such a cool thing like rnpm! thanks a lot! :)

👍

'requestPermissions' of undefined I m getting same error , i already link react-native-fcm


Replace scripts/build.sh with build.js.

This makes the build script cross-platform, closing #1361.
I also added the new build.js to the list of files to be linted.

Fixing to work on node < 4...

For some reason node=1.10 gets into an (infinite?) loop when npm run babel is run from inside build.js. I've reproduced the issue but am having trouble diagnosing it.

@wolfgang42 Ah... yep thats probably shelljs which works a bit different on node 0.10. I had problem with it outputting some really verbose messages. I'm not sure if it is the best choice after all if we need to support node 0.10... I had to make it silent with separate flag:
    shelljs.exec('mkdir -p ' + tempPath, {silent: true});
    shelljs.exec('rm -f ' + path.join(tempPath, '*.deleteme'), {silent: true});
    shelljs.exec('rm -f ' + path.join(tempPath, '*.timestamp'), {silent: true});
    shelljs.exec('rm -f ' + path.join(tempPath, '*.m4'), {silent: true});
    shelljs.exec('rm -f ' + path.join(tempPath, '*.conf'), {silent: true});

But I'm not sure how those direct rm calls will work on windows :| Maybe you could try to set global silent flag on:
var shell = require('shelljs');
shell.config.silent = true;

This issue doesn't reproduce on windows with either v0.10.45 32bit or 64bit.

I think this may be realted shelljs/shelljs#51

@elhigu Yep, shelljs/shelljs#427:

Every command I send to the shell via the shelljs is executed multiple times (random number, usually around 5-10).
...
Unfortunately, we recently dropped support for v0.10 for future development. Since we're getting ready to release v0.7, we probably won't be able to focus on this issue. If you can figure out a fix, please feel free to submit a PR.

The execSync library works on 0.10, but breaks in node 5...

@vellotis I can reproduce it on Ubuntu with 0.10.45; I haven't tried it with 0.10+Windows.

@wolfgang42 Can you try to implement it as async as mentioned in shelljs/shelljs#51 ?

This seems to have been long since obsoleted by #1616 (removed build.sh) and #1708 (added build.js).


Unable to authorize

Hi, I'm trying to set this up but whenever I try to run node src/index.js and authorize via the given URL, I'm being redirected to non working page where the code should be displayed.
I noticed that the URL contains http://localhost from the google-credentials.json file but even if I try to change it to a different address or specify any port, the page never loads. I do not have any service running on port 80 so that should not be an issue.
Google credentials are setup as desktop app.
I tried to authorize on Windows (also in Windows Sandbox) and Ubuntu but I got the same result.
Is there some step which is not mentioned in the readme?
Node version: v16.15.1

That is strange. The URL copy-pasted from the terminal is immediately redirecting to localhost? Would you try a different browser or a private tab? There were not any additional steps last time I set this up, but I know google likes to changes things for the fun of it.

No, it's not. This is the page after I authorize via Google. After I click Continue (by giving access bbyen access to my account) I'm being redirected on this page above.
I tried to open it in fresh Windows Sandbox environment, Firefox in Ubuntu but I ended up with the same result.
Recently I setup rclone which uses similiar authorization but I was redirected correctly as rclone stated Success.
This seems like there is a missing server from bbyen which is required in order to get the code.

Ok, I just tried it with a fresh installation and fresh Google cloud project. When I pasted the link and authorized and everything, I was indeed redirected to http://localhost. I think the code query parameter is what you need to paste in the terminal. Would you please try copy everything after (not including) localhost/?code= up until the first &, which for me is &scope=.... and paste that in the terminal?

Thanks! That works as a workaround.
Any plans to update the readme or the actual authorization process?

Can confirm. Authorization works when using the code from the URL as you described.

Any plans to update the readme or the actual authorization process?

Yes, I might try to do it the "proper" way and have the local server answer that request and save the code. I just have to figure out why it's http://localhost and not http://localhost:3030/some/path. I might change the readme until then.
Glad it's working for you both

I did some more research. It seems like the old google-credentials.json had the redirect_uris to be urn:ietf:wg:oauth:2.0:oob or localhost. The former only showed the code, and the latter sends the request to localhost. I tried forcing the former, and I get a 400 invalid request response. I think Google discontinued this way. I am not sure why they did this. I see no security advantage. Maybe they just want to discourage the clunky "copy paste this code into the terminal".

The new proper and automatic method is in 55a04e1b. It automatically opens the browser tab, and opens a server to listen for the response code. Would either of you be willing to test the new method? If you use the same folder, you will have to delete .google-auth-token.json. You also need to specify a port in config.json. I used 3030.

@NightMean sorry for the delay. Thanks for trying it!

As it no longer waits for code, I can't add it using the workaround. Not sure how this would work if the machine is headless?

That is a good point. I updated it so that both methods are supported. If the automatic method does not work, you can still paste into the terminal.

The strange thing is that after authorization I received this error message however bbyen continued to check the subscriptions.
Not sure if it may cause some issues later on.

I got this once too, but I was not able to reproduce it after that. I suspect what happened is that two requests were made to localhost, and one failed (did not have any code or anything). Maybe the browser was trying to fetch /favicon.ico or something. In my latest commits, I add checks and ignore any requests other than GET to /authorization_code. That should stop such things from happening in the future.
I updated the documentation too to include this new authentication method.
I would like to hear what you think of these updates.

Thanks for the update!
I did not have issues running it on my Windows machine, the auth code was automatically accepted (I used port 9020 again).
However on my Linux headless VM I was unable to authorize. I tried to paste the whole URL (as per your commit https://github.com/MarcelRobitaille/bbyen/commit/8d1b290b834f729e6b456788d923b14630a58cf7) and then only the code as previously but I received the same error message in both cases.
Enter the code from that page: 4/0AdQt8qgVUUzQR81knYpCeD2zT1I_haFus6QeNJgtF78nVgAeqKMUxDV6pBeog1Nf4W7C7A
[error] [main]: Cannot use 'in' operator to search for '&' in 4/0AdQt8qgVUUzQR81knYpCeD2zT1I_haFus6QeNJgtF78nVgAeqKMUxDV6pBeog1Nf4W7C7A
^C

On another note, I received error invalid_grant on my second Google project, will reopen issue https://github.com/MarcelRobitaille/bbyen/issues/7
Currently, I'm not able to test emails as I have to wait until Google actually removes my previous projects.

May I ask why you're running this twice? Is it for two different YouTube accounts?

I've been doing too much Python. That error should be fixed. Not sure how I didn't catch it.

Thanks for the quick fix! Pasting the URL still throws an error message, but different this time.
Pasting just the code from works like a charm now!
[error] [main]: url.parse is not a function

May I ask why you're running this twice? Is it for two different YouTube accounts?

No it's not for 2 Youtube accounts. I guess I'm just the type of a person who likes to have it working on all environments haha. (Somebody has to catch all the bugs, right?)
I run it primarily on my Linux VM but I tried to run it on my Windows just for test.

[error] [main]: url.parse is not a function

Sorry about that.  Should be fixed now.

Works like a charm now! Links and code is now accepted as expected!
Perhaps a small improvement would be to change the text that is displayed after the auth URL. Enter the code from that page: to something like Paste whole URL after authorization or something like that which is understandable.
Other than that I only found 1 major issue.
I did not find any way to donate! May I request a fix for this?

Wow. That's never come up before. Thank you very much. It's very kind. I am willing to set up donations. What would you prefer? Paypal, GitHub sponsors, bitcoin? I would also be happy if you donated to another open source project as well.
I will think about changing that string.

I changed it to Paste the entire URL of the page you are redirected to (even if you receive 404): . What do you think?

Wow. That's never come up before. Thank you very much. It's very kind. I am willing to set up donations. What would you prefer? Paypal, GitHub sponsors, bitcoin? I would also be happy if you donated to another open source project as well.
I will think about changing that string.

I prefer PayPal if that's okay.
I already donated to couple 😁

I changed it to Paste the entire URL of the page you are redirected to (even if you receive 404): . What do you think?

Yup, sounds better to me! Thanks!

I added my paypal link to the readme. Thank you very much @NightMean. You don't have to donate anything.
Do you think we can close this?

Thanks to you I can keep my Youtube obsession for longer haha!
Sure, we can close it.
Thanks for the quick fixes! 👍


chore(clippy): Cleanup linting

What this PR does 📖

Cleanup linting/clippy warnings

Which issue(s) this PR fixes 🔨

N/A

Special notes for reviewers 🗒
Additional comments 🎤

pr #378 conflicts with this pr. it fixes the clippy warnings too and makes textarea return is_valid in the onreturn callback

pr #378 conflicts with this pr. it fixes the clippy warnings too and makes textarea return is_valid in the onreturn callback

Okay. I will go on and close this since the clippy warnings are fixed there.


made web page responsive

@khushi-joshi-05 pls review the pr

@khushi-joshi-05 Pls review my pr

Solve conflicts and preview changes.


vue.esm.js:610 [Vue warn]: Failed to mount component: template or render function not defined.

I'm getting this error ...
vue.esm.js:610 [Vue warn]: Failed to mount component: template or render function not defined.

found in

---> <ChromePicker>
       <ColorChooser> at components/ptemplate/ColorChooser.vue
         <ColorRow> at components/ptemplate/ColorRow.vue
           <VTabItem>
             <VTabsItems>
               <ThemeProvider>
                 <VTabs>
                   <Recipe> at components/ptemplate/Ptemplate.vue
                     <Pages/ptemplate/Id.vue> at pages/ptemplate/_id.vue
                       <Nuxt>
                         <VContent>
                           <VApp>
                             <Default> at layouts/default.vue
                               <Root>

Using this template ...
<template>
  <v-layout row wrap>
    <v-flex xs7>
      <v-text-field v-model="d_color" :hint="cblabel" persistent-hint prefix="#" @input="save()"/>
    </v-flex>
    <v-flex xs4>
      <div class="color-box" :style="{ backgroundColor: `#${d_color}` }" @click="showPicker=true"/>
    </v-flex>

      <chrome-picker :value="colors" @input="updateValue"></chrome-picker>

  </v-layout>
</template>

Initializing like this ...


Mixed absolute and relative paths in the output

Bug Report

TSLint version: 5.0.0
TypeScript version: 2.2.2
Running TSLint via: (pick one) CLI / VSCode

Command:
tslint -c tslint.json -p tsconfig.json --type-check

Part of the output:
ERROR: C:/Projects/clime/src/castable/array.ts[46, 9]: Operands of '+' operation must either be both strings or both numbers
ERROR: C:/Projects/clime/src/castable/fs.ts[25, 14]: require() style import is forbidden
ERROR: C:/Projects/clime/src/validation/number.ts[8, 9]: Operands of '+' operation must either be both strings or both numbers
ERROR: src/examples/multi-level/cli.ts[9, 1]: Promises must be handled appropriately
ERROR: src/examples/multi-root/cli.ts[15, 1]: Promises must be handled appropriately
ERROR: src/examples/single-level/cli.ts[7, 1]: Promises must be handled appropriately

Possibly related to #1794.

Still an issue on 5.1.0, can you guys take a look? @andy-hanson

can you provide a repro? I don't see this on my machine

@nchen63 Please try:
git clone https://github.com/vilic/vio.git
npm install
git checkout 96d3cbbf3376b177a12b584877fa30365cd38555
npm run lint


Digest auth usage

Hello Collaborators,
From the docs I see there is a support to use DigestAuth, but I couldn't find how exactly am I supposed to pass the credentials for using Digest Auth, I see by default gowebdav uses basic auth. How can I explicitly ask it to use digest auth? Could you provide an example on how to use DigestAuth with gowebdav client?
Software

OS: Mac
Golang:

go version
go version go1.17.1 darwin/amd64

Version:

github.com/studio-b12/gowebdav v0.0.0-20230203202212-3282f94193f2

Additional context
I'm trying to upload my HLS content to AWS Elemental MediaPackager

The master branch has the Basic and Digest Auth burned into the c := NewClient(*root, *user, *password).
After merging #71 you'll have much more control over the setup.
We're close to bring this into master.
Thanks for your patience.

Now you can go with DigestAuth only by
d:= &DigestAuth{user: "user", pw: "password"}
a:= NewPreemptiveAuth(d)
c:= NewAuthClient("<ROOT>", a)
c.Connect()


No-Multiplier Optimized Batch Norm

Hi all,
I've been considering this idea since we heard the feedback that batch norms are essentially "unnecessary" in FPGA...
The PR for the Optimized Batch norm (https://github.com/hls-fpga-machine-learning/hls4ml/pull/142) restructures the batch norm operation to be scale * data + bias...
It might be feasible to replace the "scale" operation by a multiplier-less version. Either:

A power-of-2 scale (would remove multipliers during optimization)
Recasting the fixed-point data type, with or without changing the underlying bits (some kind of reinterpret_cast? Not sure what the mechanics are to implement this with ap_fixed, though. Maybe using the ".range()" operation to extract the underlying uint bits?)

A big question here is whether using the "closest power of 2" scale factor is sufficient for the batch norm? Or, if not, what other scale factors could be achievable without using multipliers?
Could be an easy way to get back a few multipliers and/or clock cycles for bigger designs. Thoughts?

For a Batch norm layer immediately following a dense layer, wouldn't it be possible to fold the Batch norm operations into the dense layer?
The 'scale * data' would involve using 'scale * weight' for every weight of the dense (so do the product at compile time), and the Batch norm 'bias' could just be added to the real dense layer bias.
It wouldn't work generically for a Batch Norm following the activation, but could work the same way for ReLU, I think.

The merged Dense + BatchNormalization layer is now in PR #179


[EngSys] - Verify samples have been regenerated if necessary

currently anyone on the new samples flow (any package containing a samples-dev folder likely) contains a camera-ready snapshot of samples under samples and regenerates them from the source code under samples-dev using dev-tool - see more information here: https://github.com/Azure/azure-sdk-for-js/wiki/Samples-v2-Migration-Guide
The issue: there is nothing preventing us from making changes to samples-dev and forgetting to regenerate the samples so they can get out of sync. Ideally CI would catch this.
Proposal: add a verify samples step in CI which confirms that samples are up to date. One approach would be how we handle api-review.md checks. We can regenerate the samples in CI and verify no git changes.

@witemple-msft given that it's valid to hold off on generating the samples do you think it makes sense to instead do one of:

Release automation around generating samples - similar to how sdk-bot bumps versions
Release time validation around drift - similar to how we validate API review approval before GA
Close this issue and do nothing

What do you think?

Closing as per team discussion - we want sample generation to be separate from making changes to samples-dev


rust metrics improvements

We would like to have the ability to produce a binary without metrics overheads. Ideally, metrics could also be disabled at runtime with low overhead.
It's possible that using macros - similar to the log!() macro - would help address this when combined with conditional compilation.
See #313 where this is reflected in PR feedback.

Relates to and will likely address #272

Relates to and will likely address #272


Hidden or Change route notification when tap

Hello, first of all, thanks for creating and maintaining this great utility.
I'm using the "background-geolocation" and "background-mode" plugins. When the application goes into background mode, two notifications appear, but I only want to keep one active. I can easily hide the "background-mode" notification and keep the "background-geolocation" notification active. However, the "background-geolocation" notification only displays information about the application instead of opening it. I need it to open the application instead of just displaying the information.
I attach two images of what happens
The notification that active "background-geolocation"

When I tap the notification, open the app info

Any help thanks

I had this issue as well, I thin it has to do with the icon name or some thing, see here:
https://github.com/IsraelHikingMap/Site/issues/1613

Jaja OMG, the icon was the solution.
In my case I solved it by adding the next:
BackgroundGeolocation.configure({
notificationIconLarge: '@mipmap/ic_launcher',
notificationIconSmall: '@mipmap/ic_launcher',
})
Thank you @HarelM ✊🏻


Ensure package doesn't trigger a re-compilation after run

I need to add non-regression tests for those changes…

In the last commit, I should rather move them to PostBuildOptions I think 🤔


Tasks completed in Obsidian aren't always completing in Todoist

Describe the bug
Tasks completed in Obsidian aren't always completing in Todoist, and so are autopopulated back into the plugin as incomplete.
To Reproduce
Steps to reproduce the behavior:

Complete a task in Obsidian via Todoist plugin embed.
Check back later.
Task reappears in Todoist plugin embed.
Find task has not completed in Todoist.
Despair

Expected behavior
Tasks checked off in plugin embeds will also check them off in Todoist.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Latest plugin version, latest Obsidian version.

Additional context
This is a somewhat rare bug - I maybe encounter it once a day, and have for a few weeks now.
Not sure how to represent it by screenshots, and not sure how to demonstrate how to reproduce it, since the plugin is mostly working fine for me; it's just once every few hours something I know I checked off in the plugin will reappear as incomplete.

Hey @anvimea, are you still experiencing these issues?
Todoist were having some general performance issues that were recently resolved (https://twitter.com/amix3k/status/1597941319510679552) that might have been the culprit.

I haven't noticed it as much in awhile, so perhaps this was the problem! Gonna close this issue.


envsubst

RFC
include https://github.com/a8m/envsubst
use case:
[root@tugboat] ~/app/controlplane/backend (master) ⚡
❯ kubectl kustomize | grep '$NAMESPACE' | wc                                                                                ⏎
     18      36     436

[root@tugboat] ~/app/controlplane/backend (master) ⚡
❯ kubectl kustomize . | env -i NAMESPACE=backend PATH="$PATH" envsubst -no-{unset,empty} | kubectl apply -f - --dry-run=server
serviceaccount/backend unchanged (server dry run)
role.rbac.authorization.k8s.io/backend unchanged (server dry run)
rolebinding.rbac.authorization.k8s.io/backend unchanged (server dry run)
configmap/config configured (server dry run)
secret/tls-secret unchanged (server dry run)
service/haproxy unchanged (server dry run)
deployment.apps/haproxy unchanged (server dry run)

❯

/msg: template

/add label: invalid

Great job guys closing an issue after 10 months with "invalid" wo further information. This is how you build a failed state.

Hey @hholst80, I am sorry if you feel this way, but the issue isn't closed yet!
Can you give us some guidance, where and how should we update our bot's invalid template comment?
I would be much appreciated!

Derek added the label because the issue template wasn't filled in. Sorry that you get this way, but it's not personal.
Go ahead and fill out the template properly and we can remove the label and consider your request.
If you don't want to follow the contribution process, that's ok. Just let us know and we will get this closed for you.
Thanks for your interest in arkade


node_modules/react-native-live-audio-fft/index.d.ts is not a module.ts

Hello - this sounds exactly like what I am looking for! When I try to follow the example in the Readme, the following line gives an error "File '/Users/myusername/apps/myappname/node_modules/react-native-live-audio-fft/index.d.ts' is not a module.ts(2306)"
import LiveAudioStream, {
PowerLevel,
NativeRecordReceivePCM,
FrequencyHistogramView,
} from 'react-native-live-audio-fft';
I am pretty new to RN, and very new to TS, so I am not sure if it is something I am doing wrong, or a incompatibility with the latest version of RN? (0.72.7)
Thank you!

How about try my https://github.com/flyskywhy/GCanvasRNExamples which is not using ts first?


publish<EMAIL_ADDRESS>

Requested by: @timfish
Merge target: (default)
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Leave a comment containing #retract under this issue to retract the release (original issuer only).
Targets

[ ] npm
[ ] registry
[ ] github

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

I looks like for one specific macOS test, the Electron binary fails to download and all tests then fail. I might have to come back to this next week!

#retract


docs: Add link to Docker build instructions further down in the README to the How To Run

This is a very minor addition to the README.
Because I was (initially) too dumb to see this, and without a Ruby background, did not immediately know how to build this, in order to contribute.

I'm not sure we need this. The point where you've added this is based on the fact you've installed the gem which is detailed right at the beginning of the file.

I'm not sure we need this. The point where you've added this is based on the fact you've installed the gem which is detailed right at the beginning of the file.

All right, no worries; I'm happy to close this. (I'll update my [unrelated] other #7163 some time over the coming days or so.)


Feedback for new SQL Docs hubpage

https://aka.ms/sqldocs
For the month of June we're testing out a new hub page for the SQL Server technical documentation library. It's using a new design with updated links to commonly used or recommended content. Share your feedback by commenting on this issue.
New hub page can be seen here: https://aka.ms/sqldocs
Twitter reference: https://twitter.com/MashaMSFT/status/1115396660228526080

SSMS 18.0 freezes frequently when doing things via the object explorer. ( Win 10 64-bit )
After waiting for to long, the only option is to kill the process using task manager. :-(

Please keep a download link to version 17 on the page!
Version 18 still have so maaaaaany bugs!   Fare from ready to production environment!

And for everybody else who need a working SSMS version, you can find them all here, incl. 17.9.1
https://docs.microsoft.com/en-us/sql/ssms/release-notes-ssms?view=sql-server-2017#download-ssms-1791

Please stop releasing broken software.   Please wait until you have at least fixed the MAJOR BUGS in your software.  If you are unable to fix the query window redraw problems, then why even bother making a release?  That problem renders the entire product useless.  https://feedback.azure.com/forums/908035-sql-server/suggestions/37474042-ssms-18-0-redraw-issues-when-switching-between-mul

Worst ssms ever!!!!!!!!!!!!!!
I have already lost work hours because I simply cracked during the execution of a long-lasting command.

Well, the SSMS developer team (and the rest of MS Co) have never understood KISS.
How much is the installation package...800MB? Should be 10MB
And the setup program,  OMG!.
10 million files, 10 millions registry changes, 10 mill dependencies......no wonder the path of SSMS has always ALWAYS been full of pain and missiry. A system/program with so many dependencies will always be unstable and subject for failing.

Hello! A number of comments were left about SQL Server Management Studio (SSMS). These were all forwarded over to the SSMS team, and then deleted, as this forum is intended for feedback on the hub page :) Thank you all for your participation!

helloo!! please help me to install it

Hi @isaacbouche , thanks for contacting Microsoft! You can find assistance with installing SQL Server here: https://docs.microsoft.com/en-us/sql/database-engine/install-windows/install-sql-server?view=sql-server-2017. Thanks!

@MashaMSFT -- please close this issue 2249 when the hub page preview is over.

Test comment for @MashaMSFT.

Thanks for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We’ll contact you if we need further information, and you’ll receive an email message each time we “@mention” you in a comment.
If you're suggesting a simple change to the documentation, it might be faster to submit the suggestion yourself by creating a GitHub pull request (PR). Here’s how:

On the page you have feedback about, select the Edit link at the top right.
On the next page, select the Pencil icon ( ) at the top right.
On the next page, in the Edit file text window, make your edits directly to the text you want to change.
If you need help with formatting the new or changed text, see our Markdown Cheatsheet.
After you’ve made your edits, under Commit changes:
a. In the first text box, enter a brief description of the change you’ve made.
b. In the Add an optional extended description box, paste the link to your GitHub issue.
Select Propose file change.
On the Comparing changes page, select Create pull request.
On the Open a pull request page, select Create pull request.
Allow a day or more for the article’s author to review and approve your change or offer an alternative solution.
By opening a pull request, you’re helping the author add your approved change directly to the documentation. After the updated article is published, GitHub lists you as an article contributor.
If your issue is more complex than a simple change in the documentation, that’s okay.  We'll continue to research your suggestion and then get back to you with our response.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

Thanks for contacting Microsoft with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. Weâ€™ll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
If you're suggesting a simple change to the documentation, it might be faster to submit the suggestion yourself by creating a GitHub pull request (PR). Here's how:

On the page you have feedback about, select the Edit link at the top right.
On the next page, select the Pencil icon at the top right.
On the next page, in the Edit file text window, make your edits directly to the text you want to change.
If you need help with formatting the new or changed text, see our Markdown Cheatsheet.
After you've made your edits, under Commit changes:
a. In the first text box, enter a brief description of the change you've made.
b. In the Add an optional extended description box, paste the link to your GitHub issue.
Select Propose file change.
On the Comparing changes page, select Create pull request.
On the Open a pull request page, select Create pull request.

Allow a day or more for the article's author to review and approve your change or offer an alternative solution.
By opening a pull request, you're helping the author add your approved change directly to the documentation. After the updated article is published, GitHub lists you as an article contributor.
If your issue is more complex than a simple change in the documentation, that's okay.  We'll continue to research your suggestion and then get back to you with our response.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

Add more images for informative view


some fix

Docs Build status updates of commit c235180:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/activity-feed.md
:warning:Warning

Details

msteams-platform/publishing/apps-localization.md
:white_check_mark:Succeeded

msteams-platform/concepts/activity-feed.md

Line 81: [Warning] Invalid file link:(~/concepts/deep-links).
Line 88: [Warning] Invalid file link:(~/concepts/bots/bots-notification-only).

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

Docs Build status updates of commit 6c2bb9d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/activity-feed.md
:warning:Warning

Details

msteams-platform/publishing/apps-localization.md
:white_check_mark:Succeeded

msteams-platform/concepts/activity-feed.md

Line 79: [Warning] Invalid file link:(~/concepts/deep-links).
Line 86: [Warning] Invalid file link:(~/concepts/bots/bots-notification-only).

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

Docs Build status updates of commit 4a10d93:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/concepts/activity-feed.md
:white_check_mark:Succeeded

msteams-platform/publishing/apps-localization.md
:white_check_mark:Succeeded

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


Poor styling in project_contributors page

Body size inadequate, scrolls horizontally. Bold blue underline on project links looks a bit messy!

Good find. The page can take some responsiveness fixes for small displays as well!
Headings and image sizes can be reduced to not look huge.

The page can take some responsiveness fixes for small displays as well!
Headings and image sizes can be reduced to not look huge.

Body size inadequate, scrolls horizontally.

Should be quick fix, Can you do it?

Bold blue underline on project links looks a bit messy!

Feel free to make PR with an alternate look :)

@abhigyank, Tag this with urgent. Really doesn't look good.

Yes, its an easy fix too. Can one of you quickly fix this?

@kpbot claim


spinner loading forever

hi,
I using device oppo A83 and library
"react-native": "^0.55.4",
"react-native-loading-spinner-overlay": "^0.5.2".
When i use loading spinner  with demo code, it ok with setInterval, but when i use setTimeout, it not working. it not turn off.
It loading forever. I reload app it still there, it still show loading, I can not active on my screen. affter, i set visible : true, and reload app, it load twice loading spinner and override together. then I delete tag spinner, it still loading (like cache). When i shutdown app and start again, it just turn off

@ducvu91 : how to fix for issue ?


Looking for command to rollback specific table and migrate that table again

Discussed in https://github.com/nWidart/laravel-modules/discussions/1422

Originally posted by JaberWiki July 27, 2022
How to rollback migration with step  or --path like default Laravel rollback commands on Laravel modules?
I've tried this command
php artisan migrate:rollback --path=/Modules/ModuleName/Database/Migrations/table_name.php
or
php artisan module:migrate-rollback --path=/Modules/ModuleName/Database/Migrations/table_name.php Blog
But it's not working.
Is there any command to migrate or rollback specific table?

Well, the path option is only to indicate the directory, not the file

It should be like that
php artisan module:migrate-rollback Product

While Product is the module name you wish to rollback current migrations

It should be like that
php artisan module:migrate-rollback Product

While Product is the module name you wish to rollback current migrations

Yeah, It’s also described in documentation. But I want to rollback an specific migration.

I've looked into the inner workings into migrator there's no support for this as far as I can see.


Feature Request: Custom font support

I tried using a custom font on my website, and then I encountered issues. I'd be awesome to have something like var robotsConfiguration = Robots() but for fonts, and then a Font() object.
I was able to get it working by CSS, adding a .style to the Theme() and then another .style on the text I want to change the font, but it's not an ideal solution, clearly a workaround.

Great idea! One helpful thing you could do would be to give us an example of what kind of API you'd like to see here? Just fantasyland stuff – if the API were ideal for you, what would it look like? I'm not saying we can make it happen, but it would give us a good starting point 🙂

Hey, thanks for getting back!
I'm imagining an implementation similar to Robots() but I would make a few changes. I'm thinking of something like this:
import Foundation
import Ignite

struct CustomFont {
    var familyName: String
    var weight: Font.Weight? = .regular
    var typeface: String? = "sans-serif"
    var location: String? // required for custom locations of the fonts
}

enum CustomFontElement {
    case .spaceMonoBold = CustomFont(familyName: "Space Mono", weight: .bold, location: "https://fonts.googleapis.com/css2?family=Space+Mono:ital,wght@0,400;0,700;1,400;1,700&display=swap")
    // ...
}

struct MyCustomFonts: CustomFontsConfiguration {
    var fonts: [CustomFontElement]
}

In Site.swift, we add:
var customFontsConfiguration = MyCustomFonts()
And finally, when we want to use one of our custom fonts, we do:
Text("The quick brown fox jumps over the lazy dog")
    .font(MyCustomFonts.spaceMonoBold)

... or ...
Text("The quick brown fox jumps over the lazy dog")
    .font(CustomFont(familyName: "SF Pro Display", weight: .regular, location: "/resources/sf_pro_display_regular.ttf"))

By doing this, we not only centralise all the custom fonts, but we can also detect when a font is needed or not and decide whether to load it.
I imagine this is tough. There is a lot to take into consideration: convenience of the developer, where the fonts are stored, what if the fonts are renamed, what happens when a font with a specific style or weight is overwritten by .fontWeight, and many others to take into consideration. Now that I found a workaround it's not mission-critical but it would be convenient.
P.S. Please don't judge the code too hard, it's actually one of my first contributions to such a project! I don't have much experience but I'm happy to give a hand!

Zero judgement! Thank you for taking the time to spec things out nicely! I don't know where the final implementation will end up or how long it will take, but at least now we have a starting point 🙌

@Terabyte1385 can you show how you implemented this as a workaround in your existing site?

@Terabyte1385 can you show how you implemented this as a workaround in your existing site?

Using CSS via .style:
Text("This text is in Space Mono")
    .font(.title1)
    .style("font-family: 'SpaceMono-Bold', sans-serif;")

In the Theme you want to do this you must also add:
.style("""
@font-face {
      font-family: 'SpaceMono-Bold';
      src: url('/fonts/SpaceMono-Bold.ttf') format('truetype');
}
""")

Make sure to change the path of the font to wherever you are putting your font file in your project.

As far as current workarounds, I found I can change the default font-family by applying 2 style elements to Body. For some reason, attempting to do both @font-face and font-family: My Custom Font;' in the same style` block doesn't work.
Working:
            .style("""
            @font-face {
                font-family: 'EB Garamond';
                src: url('https://fonts.googleapis.com/css2?family=EB+Garamond:ital,wght@0,400..800;1,400..800&display=swap')
            }
        """)
            .style("font-family: EB Garamond;")

I'm also stumped as to why I can't apply the font-family conditionally on tags such as h1. I.e. .style("h1 { font-family: foo;" } doesn't apply the style to h1 elements. Nor does overwriting --bs-body-font-family.

As far as current workarounds, I found I can change the default font-family by applying 2 style elements to Body. For some reason, attempting to do both @font-face and font-family: My Custom Font;' in the same style block doesn't work.
Working:
            .style("""
            @font-face {
                font-family: 'EB Garamond';
                src: url('https://fonts.googleapis.com/css2?family=EB+Garamond:ital,wght@0,400..800;1,400..800&display=swap')
            }
        """)
            .style("font-family: EB Garamond;")

I'm also stumped as to why I can't apply the font-family conditionally on tags such as h1. I.e. .style("h1 { font-family: foo;" } doesn't apply the style to h1 elements. Nor does overwriting --bs-body-font-family.

Custom font support has been added in the recent changes to Ignite! If you'd like to see further improvements, please open a new issue or submit a PR with your desired feature. 🙌🏼


Add Name and github link

/
Hey there,
Thanks for your PR & contribution :wave:!
Don't forget to join follow me, join our organization, & follow our organization to stay updated with the latest developments 💻
Thank you, and have a good day ahead 😊


Issues with new layout in v2.67

Feature Description
Your v2.67 update changelog stated that you hoped it is well received, or at least few complaints, well unfortunately I must be one of the complaints.
I loved the UI layout of v2.66, it was clean, detailed, and effective. However after updating to v2.67 I not only believe it looks worse from a design standpoint, but it reduces functionality on my screen layout.
I've attached screenshots below showing the differences. In the old one, you can see that the box art doesn't take up unnecessary space, the buttons such as play, edit, HLTB, play history, etc are all on their own line, and that the description pane is 2/3 wide whereas the details pane is 1/3 size, as it should be.
However in the new layout, the box art unnecessarily takes up space, which pushes the details pane further down. Also having everything in only two panes forces it to split 50/50, which reduces the amount of visibility in the description (right) pane, which makes it uglier and harder to read. Also you can see that on the buttons line, the play history gets wrapped to below the play button, which looks ugly.
I propose some changes to this layout if you will.

Move the play buttons line up to its own line again, as it was on v2.66.
Remove the dead space above the play buttons and below the features icons (as you'll see in the screenshot, that dead space is a perfect location for the buttons line)
I would rather the box art be up top again, but if you want to keep that design then at least consider the following:
Move the Video/Steam screenshots over to the right side in a section of their own (like it is in the new layout) but above the description tabs. Then resize the two panes to be 1/3 and 2/3 sizes like before. This way the description and screenshots can use more space as they deserve, whereas the box art and details pane require less size.

I do want to mention that I appreciate that you're trying new things and want to keep the theme alive. But for now I am going to revert to the v2.66 theme. It is frustrating to have to keep mental track of this and to avoid using "Update All" in my Playnite updates pane. Anyway I hope you'll understand the points I've laid out in this post, and I'm open to discussion, either here or in the discord.
Screenshots
v2.66 (old)

v2.67 (new)

I agree with the proposals. So much wasted screen real estate.
But for me I'd say the worst offender is the centered game title. Completely ruined the identity of the theme. I would have used a different one if I wanted that style.
I had to revert to 2.66 as well, thankfully I haven't encountered any bugs from the downgrade.
Please remember that we've had YEARS to make our libraries look the way we wanted by tweaking Stardust, even small changes can alter the experience, let alone such drastic changes.

My proposal to darklinkpower is that he create a child theme of the major update, so that if he wishes to carry on with the new design, that those of us who wish to remain on the previous version can do so without needing to avoid updating accidentally.
I prevented my theme from updating by editing the theme.yaml file and changing the version line to be 99.00 which does prevent play ite from detecting the newest update. However going the child theme route would keep it clean in the themes repo, and give more people a chance to have a choice rather than being forced into the developers vision.
Again I want to reiterate my appreciation of DLP for the theme and his hard work. But I agree with yoshi above that some of us have years (since 2016 for me, so 8 years) fine tuning exactly how we like our UI to appear; We should be allowed to keep it this way if we wish.

I also appreciate the creator's help but I really don't like this update
There are enough designs for other themes, but it's too bad to change the stardust
I think the advantage of stardust is that it is a design that stands out at a glance, so please consider it

Thank you for acknowledging us and providing a solution. A bit of a bummer that we won't be in the main release channel anymore. If Playnite ever breaks the theme fork (I can't imagine why, but anything is possible), I hope that you'll consider updating it to keep it functional.

Definitely the best course of action!
Thank you for the new branch DLP!


uftrace: 0.9.2 -> 0.9.3

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/uftrace/versions.
meta.description for uftrace is: '"Function (graph) tracer for user-space"'.
meta.homepage for uftrace is: '"https://github.com/namhyung/uftrace"
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
/bin/sh passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.9.3 with grep in /nix/store/gkpmv1nisw3rrchlb2qy4j9a59fc1q1p-uftrace-0.9.3
directory tree listing: https://gist.github.com/23d9f4aa325f11a2d0a1e4bc5a80c20b
du listing: https://gist.github.com/620bec9b2d8cdb2d4cabf59583f7df28

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)
1 package rebuild(s)
1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
uftrace

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gkpmv1nisw3rrchlb2qy4j9a59fc1q1p-uftrace-0.9.3 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A uftrace https://github.com/r-ryantm/nixpkgs/archive/2535b73809c6ee982d8a2dec6389f507eb6a4561.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gkpmv1nisw3rrchlb2qy4j9a59fc1q1p-uftrace-0.9.3
ls -la /nix/store/gkpmv1nisw3rrchlb2qy4j9a59fc1q1p-uftrace-0.9.3/bin

cc @nthorne for testing.

News: https://github.com/namhyung/uftrace/blob/v0.9.3/NEWS


Design System > Components > Webhook Subscription >Default > Required message and alignment issue.

Describe the bug
Required message and alignment issue.
Steps to reproduce bug

Go to 'https://stage-react.raaghu.ai/'
Navigate to 'Design System > Components > Webhook Subscription >Default'

Actual result-
Required message and alignment issue.
Screenshots

Expected result-
Required message and alignment should be proper.
Desktop (please complete the following information):
-OS: [Windows 10 Enterprise]
-Browser [ edge]
-Version [ 125.0.2535.92]

@rameshwarkamandar Already fixed this issue in another PR.
PR link for reference: https://github.com/Wai-Technologies/raaghu-react/pull/1299


What is the show function?

In chapter 2 of the textbook the show function is used to display the bikeshare state. Where is this function defined? I am trying (fairly successfully) to run the textbook in my local python environment instead of a workbook.

It's in modsim.py https://github.com/AllenDowney/ModSimPy/blob/master/modsim.py#L700
So you should get it when you from modsim import *

Thanks Allen. Unfortunately this does not work for me. If I have a directory with the following requirements.txt:
pandas
sympy
beautifulsoup4
lxml
html5lib
pint
modsimpy
matplotlib
numpy
scipy
seaborn
tornado
IPython

and the following test.py file:
from modsim import *
print(show)

and then I run the following:
python -m venv .
source ./bin/activate
pip install -r requirements.txt
python test.py

then I get the following error:
Traceback (most recent call last):
  File "test.py", line 2, in <module>
    print(show)
          ^^^^
NameError: name 'show' is not defined

modsim.py is not a package you can install. It is just a file (a Python module) you can download. The notebook should contain code that downloads the file to the current directory, and then you should be able to import it.
If that doesn't work, I strongly encourage you run the notebooks on Colab. The book is really about Modeling and Simulation -- if you spend your time and effort debugging your software development environment, you will never get to the fun part!
In general, if there's a problem running the notebook on Colab, I will fix it. But I can't debug problems with local installations of Python and Jupyter -- sorry, but it's just too much!


[Troubleshooting] Could not read from input file: Is a directory

Having an issue with the latest Docker install.
Steps to reproduce:

Copy the docker-compose.yml from master
docker-compose up -d

The error is listed at the end of this log:
db_1                      | server started
db_1                      | CREATE DATABASE
db_1                      |
db_1                      |
db_1                      | /usr/local/bin/docker-entrypoint.sh: running /docker-entrypoint-initdb.d/schema.postgresql.sql
db_1                      | psql:/docker-entrypoint-initdb.d/schema.postgresql.sql: error: could not read from input file: Is a directory

Any idea why this is happening? I've tried removing the Docker volume, removing the containers, deleting the bind folder before reloading. No dice.

Hello, I had this issue as well, you have to manually create a folder named sql in the path of the docker-compose.yml file and copy the postgresql file from the sql repository folder

I added the postgresql file into the sql folder and now I'm getting this error
`Starting umami_db_1 ... error
ERROR: for umami_db_1  Cannot start service db: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: rootfs_linux.go:76: mounting "/root/umami/sql/schema.postgresql.sql" to rootfs at "/docker-entrypoint-initdb.d/schema.postgresql.sql" caused: mount through procfd: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: for db  Cannot start service db: OCI runtime create failed: container_linux.go:380: starting container process caused: process_linux.go:545: container init caused: rootfs_linux.go:76: mounting "/root/umami/sql/schema.postgresql.sql" to rootfs at "/docker-entrypoint-initdb.d/schema.postgresql.sql" caused: mount through procfd: not a directory: unknown: Are you trying to mount a directory onto a file (or vice-versa)? Check if the specified host path exists and is the expected type
ERROR: Encountered errors while bringing up the project.`

what solution did you arrive at?


Move rescript-apollo-client off the reasonml-community org

Hey @jeddeloh!
As discussed in this forum post, we'd like to move active ReScript projects to their respective maintainer's github org to free up the reasonml-community org.
Would it be viable to move the repo to your personal / company github org? That way it's easier to tell if a project is actively maintained, especially if it is backed by a company.
You can then explicitly add contributors later on to keep collaboration going!
Let me know what you think!
cheers

@ryyppy Thanks for following up on this! It is probably best to transfer back to my personal github for the moment.

Sounds good! I guess i need to grant you admin rights for this org to move the repo? Let me know if you don't have enough credentials.

All done. Thanks!


Translation Tuesday

What kind of changes does this PR include?

Something else!

Description

Closes #1057
What does this PR change? Implements "Translation Tuesday". A weekly ping in the discord alerting people to new/updated content in need of translation.

Note
It currently uses the Docsbot webhook environment secret. It might be worth adding a separate one to ping in #docs-i18n instead of #docs.

I love this idea!

This is so cool, @TheOtterlord! I agree that it's maybe better placed in the i18n channel, but I have no problems with it starting out in the main docs channel, especially to start where it might get more visibility!
I'll let @delucis and @Yan-Thomas approve the implementation, but I'm super happy with any initial version of this we get:

ping just the i18n-gang main role for simplicity's sake?
ping each individual language role on each line??
no ping at all to start?

I would say, do NOT ping @team-docs with these, but I'm happy with whatever else y'all choose to use for the ping.

Glad you like it!
The current version pings the @i18n-gang role, but that can be remove. I don't think I'd recommend pinging each language role, just because that eats into the 2000 character limit in a Discord webhook.

Hey, thanks a lot for tackling this! The screenshot of the resulting message looks great!
I'm wondering if you can reuse existing functionality from my translation-status.ts script to avoid having two places where checks for translation freshness, i18nignore keywords etc. are which could get out of sync.
If I see it correctly, it should already be enough to add an export statement in front of the class TranslationStatusBuilder and then you can import, instantiate and use it in your script as well.
Instead of calling its run method, you would then call the more granular individual methods to get the status information you need (you can see what run does for reference). Everything should already be there.
An alternative approach I would see is to refactor the TranslationStatusBuilder so that the shared logic used by your new script and my old one are extracted to a separate lib file that both scripts can use.

@hippotastic thanks! Using it via an export seemed to work fine. Also separated out the regex into a const so I can still use it against pull request info

Great progress, thank you! :)
I wonder if it's possible to also replace the block where you build an index of all pages and then get the last commit per page and translation with calls to builder.createPageIndex and builder.getTranslationStatusPerPage, because then the scripts would also share the same logic there. It feels to me like the existing methods in the builder should return a structure that contains everything you need at this point in your code.

Sorry it took so long. I had to rework the order in which things were done a little, but everything works now. I've also added a runtime check to the translation-status method because it would occasionally run that script too (I eventually gave up on finding the reason it only ran sometimes, and so added the check).

Thank you for continuing to work on this!
The reason why the translation status script got executed as well is because it contains top-level code that gets executed on import (the code you added the check to). I'm sorry about that, didn't think of this detail before!
You could solve this by moving the TranslationStatusBuilder class declaration to its own file below the lib folder and importing that class both in the translation status script and your new translation tuesday script. That way, no code gets executed upon import and you wouldn't need the check you added.
One more question regarding your code: Could you elaborate on why you are fetching pull request information? If the goal of the script is to show which translations of a page still need to be updated, the PRs should not be required.
Please note that the call to getTranslationStatusByPage returns an array of the following type:
export type PageTranslationStatus = {
	subpath: string;
	sourcePage: PageData;
	githubUrl: string;
	translations: {
		[language: string]: {
			page: PageData;
			isMissing: boolean;
			isOutdated: boolean;
			githubUrl: string;
			sourceHistoryUrl: string;
		};
	};
};

As you can see, each source page already has an accompanying map of matching translations per language, including per-language flags like isOutdated or isMissing. You should be able to build the entire Discord message just from this array without requiring any further GitHub API calls.

You could solve this by moving the TranslationStatusBuilder class declaration to its own file in the lib/translation-status folder and importing that class both in the translation status script and your new translation Tuesday script. That way, no code gets executed upon import and you wouldn't need the check you added.

Sounds good, I'll move that to a file.

One more question regarding your code: Could you elaborate on why you are fetching pull request information? If the goal of the script is to show which translations of a page still need to be updated, the PRs should not be required.

I'm still using getTranslationStatusByPage, but because sometimes the translation teams are so quick, sometimes a page gets PRed pretty quickly, but might not be merged by the time of the script running on Tuesdays. The solution in place at the moment combines both of these to get a more complete result for what translations are still needed (essentially it eliminates any it's found to already be translated in a PR).
This isn't perfect, e.g. if someone translates a page that has had a major update, but doesn't take that update into account at the time of the last major commit (which I'd assume is really rare because then the translation is outdated). It also might not be worth the effort of fetching each open PR and comparing them against the main branch, because it does make it more complex to debug etc.

I'm still using getTranslationStatusByPage, but because sometimes the translation teams are so quick, sometimes a page gets PRed pretty quickly, but might not be merged by the time of the script running on Tuesdays. The solution in place at the moment combines both of these to get a more complete result for what translations are still needed (essentially it eliminates any it's found to already be translated in a PR).
This isn't perfect, e.g. if someone translates a page that has had a major update, but doesn't take that update into account at the time of the last major commit (which I'd assume is really rare because then the translation is outdated). It also might not be worth the effort of fetching each open PR and comparing them against the main branch, because it does make it more complex to debug etc.

Gotcha! Thank you for the explanation. We also discussed this topic in the past (partially in the related issue), including the "race condition" when open PRs may have become outdated by a source page update that happened after the last commit to the PR, but decided to keep PRs out of scope for now in order not to overcomplicate this topic.
Therefore I'd also recommend keeping the PR check out of this script for now, which should enable you to remove all the extra API calls and use the existing array as described to build the Discord message. This will also ensure that both scripts are perfectly in sync at the time of posting the Discord message and share the same logic.
However, the discussion brought me to the idea that it may be useful to make the link to https://i18n.docs.astro.build a bit more prominent. Thanks to your great work adding the "Translations that need reviews" section, this page always has all the up to date information on PRs, too.
So rather than just adding And [more](https://i18n.docs.astro.build)... to the end of the Discord message, you could change this to: See our [Translation Status page](https://i18n.docs.astro.build) for more, including open PRs.

Migrated the class to a lib file & removed the PR part (it runs so much quicker without all those requests xD)

Changed made (and files restored :sweat_smile:)

Late to the party but I have to say this is so cool! Amazing work @TheOtterlord


[Suggestion] Custom working fluid recipes for boilers

As the title says, it'd be interesting if pack developers could make recipes that use fluids other than water in the boilers to produce whatever fluid, so for example mercury or CO2 or magical fuels (these would basically be gas turbines at that point but those are just examples)

This would go quite well with a JEI tab for how much energy is required to turn water to steam, which I have been planning on doing. It might be a while till I have time to implement it though.

Also a good excuse to do #17 which I have been putting off


Using peak bam files without fastq processing

Hi,
Is it possible to input peak bam files directly and reduce the analysis time than using fastq? I have bam and narrow peak files from peak calling.
Thanks!

Hi,
Thank you for your interest for Cactus.
This feature is currently not supported.
It could be implemented if enough users needs it.
Thanks!
Jérôme


fix: validate atoms modulo leading and trailing whitespace

This PR improves the validation of new syntactic tokens.
Closes #6011

The failing test defines this operator:
/-- `f '' s` denotes the image of `s : Set α` under the function `f : α → β`. -/
infixl:80 " '' " => image

Should this be allowed? I would think not, because deleting the leading space makes current versions of Lean reject it, and I would expect that pretty printing instructions were orthogonal to token rules. On that assumption, I'll update the test.

This notation is also used in Mathlib.

Should this be allowed? I would think not, because deleting the leading space makes current versions of Lean reject it, and I would expect that pretty printing instructions were orthogonal to token rules. On that assumption, I'll update the test.

Why wouldn't this be allowed?

If it should be allowed, then shouldn't it be allowed whether or not the pretty-printer is instructed to insert whitespace before it?
Today, "''" is rejected as an atom, but " '' " is accepted. I think that either both or neither should be accepted here. Does that make sense?

I agree. They should both be accepted.

The exception for empty character literals was added in #1931 . I think we need a similar exception at Lean.Elab.Term.toParserDescr.isValidAtom.

The exception for empty character literals was added in #1931 . I think we need a similar exception at Lean.Elab.Term.toParserDescr.isValidAtom.

agreed

Great, '' is now allowed with or without whitespace as of b2a0735.
While I'm at it, we presently allow atoms with internal spaces. Should we?
This works, and I'm not sure it should:
infix:70 " <I like apples, they are delicious> " => Nat.add
#eval 3 <I like apples, they are delicious> 4

Just to see if it breaks Mathlib, I gave it a go

It seems that open private is a single atom.

https://github.com/leanprover-community/batteries/pull/1050 makes batteries build for me with this. I think we're OK.

While I think batteries#1050 is reasonable, I don't see a particular reason to ban tokens including whitespace?

This was a balance of footgun vs flexibility. A consequence of the way open private was specified in Batteries was that open  private or open /- Extracting the secret treasures! -/ private was banned, which could be surprising. This is also an easy mistake to make - it seems to work well in most cases, and then suddenly there's weird error messages for a few downstream users.
If someone really needs precisely that level of flexiblity for their DSL, it's still possible to drop down and write the corresponding ParserFn and parser info. But if your DSL has token rules that are very different from Lean's, then you're probably better off working mostly at that level anyway (like I do in Verso).

This was a balance of footgun vs flexibility. A consequence of the way open private was specified in Batteries was that open private or open /- Extracting the secret treasures! -/ private was banned, which could be surprising. This is also an easy mistake to make - it seems to work well in most cases, and then suddenly there's weird error messages for a few downstream users.

I agree on this, it's not the first time this mistake has been made and I don't think there are any commands in batteries or mathlib that would want to do this. But that's justification for a linter at best.

@david-christiansen, I'm seeing
/-- `f ''ᵁ U` is notation for the image (as an open set) of `U` under an open immersion `f`. -/
scoped[AlgebraicGeometry] notation3:90 f:91 " ''ᵁ " U:90 => (Scheme.Hom.opensFunctor f).obj U

erroring with "invalid atom" now. Can we add some further flexibility to allow this?

I agree on this, it's not the first time this mistake has been made and I don't think there are any commands in batteries or mathlib that would want to do this. But that's justification for a warning at best.

If a real-world use case arises, then we can revisit this. Right now, it doesn't seem like a good use of development resources - "no internal whitespace" is easy to implement and solves the problem, while making it a warning would require more time and increase the complexity of the code overall.

I had a use case for this come up:
axiom SProp.ip : UInt64 → SProp
prefix:80 "RIP ↦ " => SProp.ip

from some separation logic project I'm working on. As I understand it this will not work in the next version of lean, and the prefix command does not allow adding multiple string literals in that position. Could we make the prefix command either split the input on whitespace (preserving formatting), or allow multiple string tokens in that position?

I'm not totally convinced by this use case for spaces in the tokens. It's not that it doesn't seem like what's being asked for is useful, it's just that it seems to me that the usefulness really comes from new features rather than from undoing this new validation step.
Keeping the old behavior would have resulted in a keyword with a single space in it. RIP ↦ would be valid syntax, while RIP  ↦ or RIP /- This does the thing! -/ ↦ would not. This is precisely the footgun to be avoided, and I don't think that it's a compelling case for whitespace characters in tokens.
It doesn't seem bonkers to allow multi-token operators, though I'd like to see why a notation doesn't do what you want here. I think it's OK if operators make easy things easy, but don't cover a wide range of syntax, and from what I can see, you'd get all the benefits of a two-token prefix operator from a notation. RIP would indeed be a keyword, of course, but this is another one of those balancing acts between flexibility and consistency that doesn't strike me as a huge problem, though I of course don't have a ton of insight into the specifics of your project. What am I missing?

It's not that it doesn't seem like what's being asked for is useful, it's just that it seems to me that the usefulness really comes from new features rather than from undoing this new validation step.

No disagreement from me on this! I just wanted to give you a reasonable use case to center your thinking on this one.


Cherry-pick package updates to release-0.11 branch

What this PR does / why we need it
Updates to packages on the main branch need to be cherry-picked into the release branch.

closing since we don't want a squash commit, will do individual PRs


radioEnabled variable convert to convar

... That's not what that variable is used for. If you want radios disabled look at the already available convars as it already has a way to disable it


Update

Hi @leosuncin,
Thanks for this repo is a useful resource that I use in the documentation.
clean-redux have been renamed redux-clean-architecture, I did the update.
Best regards,

Thanks, I guess I need to improve the code: add some tests and CI; I need to add a README

@leosuncin Thank you for the quick merge.

add some tests and CI; I need to add a README

It would be fantastic, anyway, I gave you credit for it in the readme.
Just so you know. I will share the project on social media next monday at 3pm.
Thanks again for you work, much appreciated!

Just so you know.
I added a screenshot of you cool toodos app and pointed the documentation directly to your repo.
Have a nice day :)


Warning message when clicking downloading on Video

When a user logs in to the IO platform and select to download the video he gets an error message:
The videos that you have selected are larger than 1GB. Do you want to download these videos.
The Video file size:
Approaching lack of funding is - 226MB

Digital information is - 252.5MB

Steps.
Pre-requests:
The app is installed.
The user are enrolled in the course

Login with valid credentials.
Select the course 'Approaching lack of funding' or 'Digital transformation'
Click on powerful insights
Click on the arrow next to 'How to approach a lack of project funding

Results
The user gets the warning message:

Expected Results
The user is able to download the file.

@jagonzalr I will close this ticket, I don't get the warning message anymore.
I think the solution was the 2nd link we add to the video unit that allows for the downloading of the video.


Laravel-Excel Execution Time Out / WSOD

Hello,
I'm trying to export Blade page to Excel,
The page contain a lot of loop within loop. (5 level)
Example :

{{--/$totalbobot=0; /--}}{{--/$totalnilai=0; /--}}{{--/$totalcapaian=0; /--}}

@foreach($aspects as $aspect)

    < tr >
   < th > $aspect->no< /th >
    < th > $aspect->name< /th >
  < /tr >
  @foreach($aspects ->indicators() as $indicator)
    < tr >
   < th > $indicator->no< /th >
    < th > $indicator->name< /th >
  < /tr >
      @foreach($indicator->parameters() as $parameter)
    < tr >
   < th > $parameter->no< /th >
    < th > $parameter->name< /th >
  < /tr >
  @endforeach
  @endforeach
 @endforeach
On the server i got Execution Time Out 30 Second.
After that, i'm adding ini_set('max_execution_time', 600) but i got the WSOD.
If Anyone have a suggest how to export? or i have to change the algorithm?
Thank You..

have u turn on debug? just wondering how much data on $aspects var ?

No, I haven't turn on debug, just standard dev environment.
Actually,
level 1 -> $aspect  = 6
level 2 -> indicator = 43
level 3 -> parameter = 153
level 4 -> factor = 560
level 5 -> subfactor = 356
this is an example on my view,

Because it's displayed well on my view i just create another blade view for exporting it to excel.
Is it to much data to handle? Or
Should i change my method for displaying inherit data like that?
Thank you very much..

Try adding more execution time and memory limit


Optional chaining in Label

Server currently crashes if using the ?. operator in the html tags:

{name}

    {#await budgetP then budget}

{budget?.label}

    {/await}

This issue seems related, but offered solution does not work.

Also present for city.departement

Now handled by more recent version of Svelte


Issues with RenderTree after trying to update from preview8 to preview9

From @justindude521 on Thursday, September 5, 2019 7:39:11 PM
Edit: Am using server-sided blazor if that matters

Severity	Code	Description	Project	File	Line	Suppression State	Suppression State
Error	CS1503	Argument 1: cannot convert from 'Microsoft.AspNetCore.Components.Rendering.RenderTreeBuilder' to 'Microsoft.AspNetCore.Components.RenderTree.RenderTreeBuilder'

Also getting

Severity	Code	Description	Project	File	Line	Suppression State	Suppression State
Error	CS0234	The type or namespace name 'RenderTreeBuilder' does not exist in the namespace 'Microsoft.AspNetCore.Components.RenderTree' (are you missing an assembly reference?)

Using Statements on .razor pages
@using System.Threading
@using Microsoft.EntityFrameworkCore
@using BlazorApp1.Data
@using BlazorApp1.Data.Models
@using Microsoft.AspNetCore.Components.Web;
@using Microsoft.AspNetCore.Components.Authorization;
@inject AuthenticationStateProvider AuthenticationStateProvider

I've tried following the instructions for migration at https://devblogs.microsoft.com/aspnet/asp-net-core-and-blazor-updates-in-net-core-3-0-preview-9/ to no success
Root __Imports.razor
@using System.Net.Http
@using Microsoft.AspNetCore.Authorization
@using Microsoft.AspNetCore.Components.Forms
@using Microsoft.AspNetCore.Components.Routing
@using Microsoft.AspNetCore.Components.Web
@using Microsoft.AspNetCore.Components.Authorization
@using Microsoft.JSInterop
@using BlazorApp1
@using BlazorApp1.Shared

.csproj
<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp3.0</TargetFramework>
    <LangVersion>7.3</LangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Hangfire.AspNetCore" Version="1.7.6" />
    <PackageReference Include="HangFire.Core" Version="1.7.6" />
    <PackageReference Include="HangFire.SqlServer" Version="1.7.6" />
    <PackageReference Include="Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.AspNetCore.Identity.EntityFrameworkCore" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.AspNetCore.Identity.UI" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.Azure.SignalR" Version="1.1.0-preview1-10442" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.SqlServer" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Design" Version="3.0.0-preview9.19424.4" />
    <PackageReference Include="Microsoft.WebSockets" Version="<IP_ADDRESS>" />
  </ItemGroup>

</Project>

Copied from original issue: dotnet/core#3315

@justindude521 can you confirm that you updated to the latest preview of Visual Studio? Can you indicate exactly which version you're running?

@Eilon Seems like I was still on Visual Studio 2019 Preview 2. I must have accidentally cancelled the update when doing so.
Updating to Visual Studio 2019 16.3 Preview 3 fixed the issue!

@justindude521 can you confirm that you updated to the latest preview of Visual Studio? Can you indicate exactly which version you're running?

@Eilon Seems like I was still on Visual Studio 2019 Preview 2. I must have accidentally cancelled the update when doing so.
Updating to Visual Studio 2019 16.3 Preview 3 fixed the issue!


Fix Unit Tests

Since the refactor that happened about a month ago, many of the unit tests are broken due to new paths of things. In particular, the deployment unit tests don't work.

I'm having this same issue, after installing OMK-posm production server.  the two example deployments don't work on OMK mobile app, the app crashes when trying to download either. Is there a manual fix for this?

Ah, this is because the example deployments dont have any MBTiles or OSM XML in them. That aspect is under development and is done in POSM Admin
However, you can just manually put MBTiles and OSM XML in a given deployment and it will download. If there's nothing there, it crashes. I'll file an issue about that.
Maybe I'll put a small OSM and MBTile in the example deployment so you can see how it's supposed to look. However, I haven't put anything in there, because I don't want to bloat the repo.
The following is an example of files I have in the deployment directory. You can put in any number of OSM XML and MBTiles. You do not need to specify them in the manifest. The manifest is mainly to provide a name and description of the deployment.

Take a look at
https://github.com/AmericanRedCross/OpenMapKitServer/blob/master/docs/manual-deployment.md
for more info

Got it, thanks!
From: Nicholas Hallahan<EMAIL_ADDRESS>Sent: Thursday, March 31, 2016 4:56 PM
To: AmericanRedCross/OpenMapKitServer
Cc: David Reinhart
Subject: Re: [AmericanRedCross/OpenMapKitServer] Fix Unit Tests (#48)
Take a look at
https://github.com/AmericanRedCross/OpenMapKitServer/blob/master/docs/manual-deployment.md
for more info
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHubhttps://github.com/AmericanRedCross/OpenMapKitServer/issues/48#issuecomment-204163529

One more question.  Are all spatial features collected automatically broadcast to OSM for public viewing?   If so, is it possible to disable this feature so that spatial data is only saved to the server?
From: Nicholas Hallahan<EMAIL_ADDRESS>Sent: Thursday, March 31, 2016 4:56 PM
To: AmericanRedCross/OpenMapKitServer
Cc: David Reinhart
Subject: Re: [AmericanRedCross/OpenMapKitServer] Fix Unit Tests (#48)
Take a look at
https://github.com/AmericanRedCross/OpenMapKitServer/blob/master/docs/manual-deployment.md
for more info
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHubhttps://github.com/AmericanRedCross/OpenMapKitServer/issues/48#issuecomment-204163529

Nothing is sent to OSM automatically. You have to go into iD or JOSM to submit to OSM.


Spike: PF3 theme for PF4

Evaluate if and how we can provide a PF3 theme for PF4. This spike includes a test page that includes PF3 themed PF4 components with PF3 components.

I believe this is in a good place to move forward with. Please see this document which outlines this approach.


fix: Replace aria-hidden attribute with inert attribute in  component

Replace the aria-hidden="true" attribute with the inert attribute in the button part of the radio button in the Shadow DOM of the  component.
fix #5699

Mentioned in the PR, but we still need a touch target that isn't inert. I think we can remove the input entirely and replace it with a touch target. Can you update this PR for that?
Replace the <input> here with:

Replace the <input> touch target styles here with:
    .touch-target {
      height: 48px;
      position: absolute;
      width: 48px;
    }

    :host([touch-target='none']) .touch-target {
      width: 100%;
      height: 100%;
    }

And keep aria-hidden="true" on the

@asyncLiz
Thank you for your prompt feedback.
We will make adjustments based on the information you provided.
This is my first pull request, so please excuse my rudimentary question.
I looked at the package.json file to set up a development server, but there are no scripts such as dev or start set up, so I couldn't figure out how to start it.
It seems that npm run build will build the server, but will I have to load this script in a separate index.html?
We apologize for the inconvenience, but we would appreciate it if you could let us know.

We don't have a great dev server setup for external contributors yet. The only option right now is to run the package's catalog server and navigate to /components/radio/stories/.

@asyncLiz
Thank you very much for your kind attention.
I was able to successfully set up the development server.
We will contact you again as soon as the adjustments are completed.

@asyncLiz
Please excuse me for repeating myself.
Is <md-radio> in radio/demo/stories.ts provided by unpkg?
I need to edit radio/internal/radio.ts this time, but I am having trouble getting the changes to take effect.
On the other hand, the elements in radio/demo/stories.ts reflect the changes (e.g. changing “Cats” to “Cat”).
Your wisdom would be greatly appreciated.
Thank you in advance.

Is <md-radio> in radio/demo/stories.ts provided by unpkg?

It may be, that sounds familiar. I haven't tried running it locally for development :(
I'm not sure I have a good suggestion. I think @VandeurenGlenn mentioned they had a script in their fork of the repo for development.

@asyncLiz
Thank you for your response.
I understand the content.
So for now, I will only send the changes you proposed and he will be responsible for confirming the operation status, is that correct?

You can make the changes and I'll verify it

@asyncLiz
Thank you very much for the series of guidance and reviews.


Wrongly generate _Session when login by Facebook

Here is how to reproduce:

My database is clean.
My browser is clean (by localStorage.clear())
I logged in with my Facebook account
=> Parse Server create new user with objectId: PRGzlgxcoE
=> Parse Server create _Session with user column point to PRGzlgxcoE, which is correct
I logged out (by Parse.User.logOut())
I logged in again with same Facebook account
=> This time, Parse Server modified the last created session, as I know, what it did is refreshing the token, BUT IT ALSO CHANGE THE user column to an unknown object: L1wysagBDK (which should be PRGzlgxcoE).
And of course after that, I became an invalid user because my sessionToken is not pointed to me. Which made any cloud function I call is not from authenticated user (request.user is null).

Bonus step:

Now I don't log out by Parse.User.logOut(), I use: localStorage.clear(), which will also clear the Installation of Parse and the current logged in user.
I logged in again with same Facebook account
=> Parse Server create new _Session with user column point to another unknown object: C40NzjGJen

Every time I try to sign out or clear local storage, Parse Server will create(modify) a session with invalid user column.
But if I change to api.parse.com endpoint, this does not happen at all, all session is generated with correct user column.
Note that, the api.parse.com and my Parse Server point to the same mongo database, with same AppID and MasterKey.
I've tested this on Parse Server 2.1.6 and 2.1.2.

Fixed by 857, will be in next release. :+1:


[AppiumDriver@d16b]     at next (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:280:10)
[AppiumDriver@d16b]     at Layer.handle [as handle_request] (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/layer.js:91:12)
[AppiumDriver@d16b]     at trim_prefix (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:328:13)
[AppiumDriver@d16b]     at /opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:286:9
[AppiumDriver@d16b]     at Function.process_params (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:346:12)
[AppiumDriver@d16b]     at next (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:280:10)
[AppiumDriver@d16b]     at methodOverride (/opt/homebrew/lib/node_modules/appium/node_modules/method-override/index.js:65:14)
[AppiumDriver@d16b]     at Layer.handle [as handle_request] (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/layer.js:95:5)
[AppiumDriver@d16b]     at trim_prefix (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:328:13)
[AppiumDriver@d16b]     at /opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:286:9
[AppiumDriver@d16b]     at Function.process_params (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:346:12)
[AppiumDriver@d16b]     at next (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:280:10)
[AppiumDriver@d16b]     at urlencodedParser (/opt/homebrew/lib/node_modules/appium/node_modules/body-parser/lib/types/urlencoded.js:91:7)
[AppiumDriver@d16b]     at Layer.handle [as handle_request] (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/layer.js:95:5)
[AppiumDriver@d16b]     at trim_prefix (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:328:13)
[AppiumDriver@d16b]     at /opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:286:9
[AppiumDriver@d16b]     at Function.process_params (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:346:12)
[AppiumDriver@d16b]     at next (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:280:10)
[AppiumDriver@d16b]     at defaultToJSONContentType (/opt/homebrew/lib/node_modules/appium/node_modules/@appium/base-driver/lib/express/middleware.js:82:3)
[AppiumDriver@d16b]     at Layer.handle [as handle_request] (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/layer.js:95:5)
[AppiumDriver@d16b]     at trim_prefix (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:328:13)
[AppiumDriver@d16b]     at /opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:286:9
[AppiumDriver@d16b]     at Function.process_params (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:346:12)
[AppiumDriver@d16b]     at next (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:280:10)
[AppiumDriver@d16b]     at /opt/homebrew/lib/node_modules/appium/node_modules/@appium/base-driver/lib/express/middleware.js:67:5
[AppiumDriver@d16b]     at Layer.handle [as handle_request] (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/layer.js:95:5)
[AppiumDriver@d16b]     at trim_prefix (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:328:13)
[AppiumDriver@d16b]     at /opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:286:9
[AppiumDriver@d16b]     at Function.process_params (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:346:12)
[AppiumDriver@d16b]     at next (/opt/homebrew/lib/node_modules/appium/node_modules/express/lib/router/index.js:280:10)
[HTTP] <-- GET /session/4451f462-6095-4061-a32b-d64aeef238b4/source 404 4 ms - 5913
[HTTP]
[DevCon Factory] Releasing connections for 1C4DE705-A5E3-4C63-9D79-17DE65D17A60 device on any port number
[DevCon Factory] Found cached connections to release: ["1C4DE705-A5E3-4C63-9D79-17DE65D17A60:8100"]
[DevCon Factory] Cached connections count: 0
[XCUITestDriver@7e33 (df104001)] Not clearing log files. Use clearSystemFiles capability to turn on.
[HTTP] Waiting until the server is closed
[HTTP] Received server close event

The attached one still did not have get context endpoint call, unfortunately.
Get context returns https://github.com/appium/appium-flutter-driver/blob/3b03a439c9027c8082161bf5910ee3413536d4ab/driver/lib/commands/context.ts#L27-L30 , so what I could guess is the iOS session start failed? so non-flutter context was an empty

This was my mistake. I was calling getContext() without the "s". That was the reason all of them were not being returned. It is working as expected. Sorry for the trouble. Have a great day!


Disable google analytics ad tracking

I'm finding this difficult to test but this does seem to stop requests
to ga-audience and doubleclick.
Discovered when checking some waterfall logs, seen in external tools such as
SpeedCurve, WebPageTest.
To test I tried this locally, it seems these endpoints are only called on initial visit
which makes it hard to reproduce.
Fixes: https://github.com/alphagov/static/issues/991
Will need sign off from the Performance Analysis community
It'd be good to get someone else to try and reproduce this locally too.

🚢


add timestamp for the music being played

it would also be better to add a timestamp for the music being played, to let the user know of the current time and time left in the song

Good Issue.
Thank you for your interest.❤
Happy Coding.