Prop updates are not correctly handled with Picturefill

According to the Picturefill website

If you dynamically change the srcset, sizes attributes, or modify source elements, please use this reevaluate option and provide an array or array-like object of your mutated img elements. As an alternative the mutation plugin can be used.

picturefill({
  reevaluate: true,
  elements: [ document.getElementById( "myImg" ) ]
});

I don't believe this is being done at the moment, and I believe it's causing a bug I'm facing where I'm doing exactly that

Hey @EmilEriksen are you able to provide a bit more context about the bug you're experiencing?
Is that happening in all browsers or older ones?
We're currently just using the simplest call to picturefill so that's definitely not supported at the moment but I'm just trying to understand and recreate that bug you've mentioned.

Yes, of course. I've created an example here. If you open it in a modern browser it'll play a slideshow of images but in IE 11 nothing happens because Picturefill is not being told that the srcset has been updated. If you look in the DOM explorer you can see that the srcset attribute is updated but src is not. This should happen in all older browsers that require Picturefill to work but I've only tested it in IE 11.


Whitelist Url https://web.tutar.app for Embedding in Excalidraw

Hi, we are unable to embed the website https://web.tutar.app/ in Excalidraw. Kindly whitelist this url for embedding.

Hi there! Is this issue still open? I would like to contribute to it. Please assign me this issue under Hacktoberfest2024.
@ad1992

Issue is still not solved @ad1992

I can work on this, but I would need more information about the web tutar app. I can't find any docs about how to embed it. Meanwhile, if someone else wants to try, embeddable URLs are in embeddable.ts.

@ritk20 TutAR will show like if you don't have subscription.


install-sql-server-using-a-configuration (version 2022)

The documentation describes ConfigurationFile.ini , but in version 2022 it does not exists any more.
In version 2022 is generated Settings.xml

@ZAJDAN Can you please provide a screenshot demonstrating this? When I installed SQL Server 2022 recently, I was able to use an INI file.

I have just attempted a fresh install using SQL Server 2022, and confirm that the file used is still ConfigurationFile.ini.

If you have further information, please feel free to comment below. However, I am closing this issue.

@rwestMSFT
I forgot to say I install SQL 2022 EXPRESS
There is not Section Ready to install and anyway it creates just Settings.xml

@ZAJDAN Can you demonstrate that this is different to earlier versions of SQL Server Express edition?


What about custom agent that support tunelling throught proxy?

This is awesome and lightweight lib for http, I begun use it after request.js
Is it possible to add feature for tunelling throught proxy via CONNECT request?
Or how use it if exists. I didn't find something in the code.
Thanks.

The issue in #177 shows an example using a tunnelling proxy. Is that what you need?
(Sorry for the delay, by the way)

This doesn't seem like an issue so I'm closing it. If there's something that's not working as expected feel free to open up a new one.


Avoid inits

Changes

Closes #685
[Worker] cfg.Init() --> cfg.SetConfig() called in cmd/worker.go
[TUM-Live] config.Init() --> config.LoadConfig() called in cmd/tumlive.go
[TUM-Live] api.Init() --> api.ServeWorkerGRPC() called in cmd/tumlive.go

Just had a look here out interest. The changes make sense to me, but I saw there are some more inits() not addressed by this PR. Are these OK? I think they could be also potentially removed.

This looks a bit problematic to me: https://github.com/joschahenningsen/TUM-Live/blob/095ca48c197349fcbcc06f8213d32912186d242e/tools/branding.go#L28
This one could be fine: https://github.com/joschahenningsen/TUM-Live/blob/815492a680744efc568197e6cb626268313d0a69/tools/realtime/channel_store.go#L13

Ah these must have been added after the merge of some later pull requests. At least the branding one is new.
I will remove those two too. 👍🏻

Just had a look here out interest. The changes make sense to me, but I saw there are some more inits() not addressed by this PR. Are these OK? I think they could be also potentially removed.

This looks a bit problematic to me: https://github.com/joschahenningsen/TUM-Live/blob/095ca48c197349fcbcc06f8213d32912186d242e/tools/branding.go#L28
This one could be fine: https://github.com/joschahenningsen/TUM-Live/blob/815492a680744efc568197e6cb626268313d0a69/tools/realtime/channel_store.go#L13

Removed the branding init function. The second one doesn't is fine, yes.
I've also added a list of initializers and the function initAll in cmd/tumlive.go. initAll basically iterates the global variable initializers calls the function at the beginning of execution.


TorbJörn: Turret Damage

Created by: DpsDoomMain
• Currently 11 damage per shot.
• Previously 14 damage per shot.
( tell me how you want me to write them so it’s preferable)

Created by: MaxwellJung
You're doing well (as long as you're not lying about the issue)

moved from ow1_emulator_backup#646


country list objective c

bug**
Trying to present the country list with objective c? is that supported? if Yes please advice
I can't use the setup method to load the countries.
Thanks

Did you find any solution ?

checkout PR #157


create, use, and test fips-compliant go images

Right now Aegis is based on standard go, which is not FIPS compliant.
One option to attain compliance is to base the code on boring crytpo fork instead:
# builder image
FROM my-golang-boringcrypto:latest as builder

RUN mkdir /build
COPY app /build/app
COPY core /build/core
COPY vendor /build/vendor
COPY go.mod /build/go.mod
WORKDIR /build
RUN CGO_ENABLED=0 GOOS=linux go build -mod vendor -a -o aegis-safe ./app/safe/cmd/main.go

# generate clean, final image for end users
FROM gcr.io/distroless/static-debian11

LABEL "maintainers"="Volkan Özçelik<EMAIL_ADDRESS>LABEL "version"="0.17.2"
LABEL "website"="https://aegis.ist/"
LABEL "repo"="https://github.com/shieldworks/aegis-safe"
LABEL "documentation"="https://aegis.ist/docs/"
LABEL "contact"="https://aegis.ist/contact/"
LABEL "community"="https://aegis.ist/contact/#community"
LABEL "changelog"="https://aegis.ist/changelog"

COPY --from=builder /build/aegis-safe .

# executable
ENTRYPOINT [ "./aegis-safe" ]
CMD [ "" ]

In this Dockerfile, my-golang-boringcrypto:latest should be replaced with your actual BoringCrypto Go image. If you don't have this image, you will need to create one. Here's an example of how you can do it:
Clone the BoringCrypto Go repository: git clone https://go.googlesource.com/go && cd go
Check out the dev.boringcrypto branch: git checkout dev.boringcrypto
Build Go: cd src && ./all.bash
Once Go is built, you can use it to create a Docker image.
# Use the official Golang image as a base.
FROM golang:1.20.1-alpine3.17

# Install Git.
RUN apk update && apk upgrade && \
    apk add --no-cache bash git openssh

# Clone and build BoringCrypto Go.

ENV PATH="/go-boringcrypto/bin:${PATH}"

# Set the current working directory inside the container.
WORKDIR /go/src

This Dockerfile will create a Docker image with the BoringCrypto build of Go, which you can use as the builder image in your application's Dockerfile. Remember to replace my-golang-boringcrypto:latest in the application Dockerfile with the name and tag of the BoringCrypto Go Docker image you created.
Please note that this information is based on the state of Go, Docker, and BoringCrypto as of my last update in September 2021, and things might have changed

Looks like boringcrypto is in the mainline now:
RUN GOEXPERIMENT=boringcrypto go build . && \
    go tool nm fips-echo-server > tags.txt && \
    grep '_Cfunc__goboringcrypto_' tags.txt 1> /dev/null

ref:

https://kupczynski.info/posts/fips-golang/
https://developers.redhat.com/articles/2022/05/31/your-go-application-fips-compliant
You might need CGO_ENABLED=1 too » https://projectcontour.io/guides/fips/
more: https://github.com/sunjayBhatia/contour/blob/d4591c0eed06c19695939aa4004850c5ec0349c6/site/content/guides/fips.md?q="GOEXPERIMENT%3Dboringcrypto"#L50-L59

Also update the documentation that FIPS mode incurs performance overhead; if you can use Age encryption, better stick with it.
some ref: https://www.druva.com/blog/overhead-boringssl-fips-mode-go

Done.


Some icons are not showing

Some icons are not showing for me in Chrome 51.0.2704.103 (64-bit) and Safari 9.1.1.
This is how it looks:

What's the name of that icon?


Incorporate knowledge generation context selection improvements

This incorporates fixes from
https://github.com/aakankshaduggal/sdg/pull/21 to the way we select the expanded contexts during the knowledge data generation, cleaning up the logic around how we select which other documents to include in the expanded context during edge cases where we have low numbers of unique documents

I reviewed it the overall logic here in selector distractor/golden documents (as described in the RAFT paper linked in the code comment in this change) in these edges cases with the team in-person yesterday, and believe it to be correct. And, the generated results look better now which should lead to better training results when we have lower numbers of unique documents to pick from. Thanks for the quick work!


Pass eventBus object to javascript function

Example of using EventBus subscriptions with unsubscribe.
The problem:
To prevent memory leaks, subscriptions must be canceled before create new ones or when component is unmounted.
The solution:
Returned function will be called on component update/unmount
const subscription = eventBus.subscribe('hover', () => {
  console.log('hello')
})

return () => {
  subscription.unsubscribe();
}


Test updates

Heroku CI test output needs to be TAP only, so (a) don't call set -x and (b) remove apex-test-format s output must be TAP.
Clean-up test orgs (w/ option to retain)

https://github.com/wadewegner/salesforce-dx-buildpack/tree/cwall/no-set-x-in-test

Good stuff!


AutoGemm only enabled for ColumnMajor matrices?

Per https://github.com/clMathLibraries/clBLAS/wiki/AutoGemm#autogemm-parameters , it looks like autogemm is only run for the orders specified in orders parameter of AutoGemmParameters.py.  However, looking at AutoGemmParameters.py, orders is specified as, https://github.com/clMathLibraries/clBLAS/blob/master/src/library/blas/AutoGemm/AutoGemmParameters.py#L74 :
orders = [ "clblasColumnMajor" ]

Does this mean that AutoGemm is only activated for ColumnMajor matrices?

Yes, that's right. AutoGemm is capable of generating row major kernels. However the gemm implementation can, for example, from row-major NT to column-major TN. So, AutoGemm only needs to generate column-major kernels for clBLAS.

Ah, ok, I see.  The implementation has seemed to be using AutoGemm for row major kernels that I've passed to it, as far as I can see, which makes more sense now :-)


Docker error: Invalid argument, expected lens object

Docker throws this error: Invalid argument, expected lens object
System Information:

OS: Windows 10
Snap Camera Version: 1.21
Signature Patch applied: yes

Log:
Log.txt
Lens share link:** https://www.snapchat.com/unlock/?type=SNAPCODE&uuid=bba25a0521164a52bdfc70137d6e18da&metadata=01

The lens helped me to discover a bug in snap lens tool.
But the real problem here is that this lens is using some assets with an  "unsuported blend mode".
Which will result in log.txt error messages like:
Core: Unknown blend mode

"Lens not loaded exception: Unknown blend mode"

I managed to fix the blend mode manually.
This will be done automatically in the next major release of snap camera server.
And it will be an upcoming feature of snap lens tool.
No Filter Active

Filter Active with Blend Mode (Normal)

Here is the fixed lens for download, you would have to manually upload it to server.
PAEyesBlendFixed.zip
If you are the creator of the lens, you should be able to switch to another blend mode in lens studio.
And then re-publish the lens.
Here is a list of supported blend modes:

BlendMode
Value
Expression

Normal
0
SrcAlpha, OneMinusSrcAlpha

MultiplyLegacy [DEPRECATED]
1
DstColor, OneMinusSrcAlpha

AddLegacy [DEPRECATED]
2
One, One

Screen
3
One, OneMinusSrcColor

PremultipliedAlpha
4
One, OneMinusSrcAlpha

AlphaToCoverage
5
Blend Disabled

Disabled
6
Blend Disabled

Add
7
SrcAlpha, One

AlphaTest
8
Blend Disabled

ColoredGlass
9
Blend Disabled

Multiply
10
DstColor, Zero

Min
11
One, One

Max
12
One, One


Colleagues list not available when using Firefox

Version
click-to-dial-2.0.12 August 2017
Browser Version
Mozilla Nightly 57.0a1 (2017-08-14) (64-bit)
Mozilla Firefox 55.0.1 (64-bit)
Description
Whenever a user logs into the plugin it is impossible to access the Colleagues list. Clicking on the drop down menu for Colleagues does nothing.  Please refer to the screenshot for further information.
File / Feature
Contact List
Expected behaviour
When clicking on Colleagues the colleagues list should appear.
Actual behaviour
When clicking on Colleagues nothing happens.
Stacktrace / Error message
TypeError: parentDocShell.getDocShellEnumerator is not a function
Steps to reproduce

Log in to the plugin
Wait for the "refresh wheel" to stop (you can also click right away)
Notice how the drop down menu does not appear showing the list of Colleagues

Hmm. I can't reproduce this. Will test some more.

This had to do with the sendMessage callback being part of the data payload, which should have been last parameter only.


Create scorecards.yaml

Pull Request
Pull request checklist
Please check if your PR fulfills the following requirements:

[ ]  Tests for the changes have been added (for bug fixes / features)
[ ]  Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ]  Build (cli.py) was run locally and any changes were pushed
[ ]  Tox tests (tox) has passed locally and any fixes were made for failures
[ ]  My code follows the code style of this project.
[ ]  All new and existing tests passed.

Pull request type

Please check the type of change your PR introduces:

[ ]  Bugfix
[ ]  Feature
[ ]  Code style update (formatting, renaming)
[ ]  Refactoring (no functional changes, no api changes)
[x]  Build related changes
[ ]  Documentation content changes
[ ]  Other (please describe):

What is the current behavior?

OSSF scorecrd is not scanning the repo
Related Issue

GitHub Issue Number
What is the new behavior?

OSSF Scorecard will scan the repo for code security issues
Motivation and Context

How Has This Been Tested?

Does this introduce a breaking change?

[ ]  Yes
[x]  No

Other information

Screenshots (if appropriate)

Codecov Report

Merging #365 (e1195cd) into main (703f0bc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #365   +/-   ##
=======================================
  Coverage   90.92%   90.92%
=======================================
  Files          33       33
  Lines        1323     1323
  Branches       48       48
=======================================
  Hits         1203     1203
  Misses        116      116
  Partials        4        4


Avoid numerical issues with low or zero flows

I have friction factor value, Nut in epanet if one has to use Darcy-Weisbatch formula, need to consider roughness. Please help how to find out the value of roughness for the same. Thanks in advance.


Remove spring test modules

The new features are supported in the 4.0 feature, and many modules have been refactored,
so we plan to remove the azure-spring-boot-test-xxx modules to make the spring project structure cleaner,
more unit tests should be included, some unit tests have been added in the previous
PR https://github.com/Azure/azure-sdk-for-java/pull/25820, others will keep or remove.
Changelog:

Remove azure-spring-boot-test-application
Remove azure-spring-boot-test-core
Remove azure-spring-boot-test-cosmos
Remove azure-spring-boot-test-keyvault
Remove azure-spring-boot-test-parent
Remove azure-spring-boot-test-servicebus-jms
Remove azure-spring-boot-test-storage
Remove spring-cloud-azure-stream-binder-test
Remove spring-cloud-azure-test-parent
Remove azure-spring-boot-test-keyvault-certificate

/azp run java - spring - tests

/azp run java - appconfiguration - tests

/azp run java - spring - tests

@saragluna , please help review, thanks!


remove _get_indep_borns, and import elaborate_borns_and_epsilon

After upgrading phonopy v.12.2 to higher versions (e.g. v1.12.4 and v1.12.6), I foud the module '_get_indep_borns' had been removed, and the born data parser caused errors.
I fixed this error through adding a module 'elaborate_borns_and_epsilo' to toe raw_parsers.py.
I have tested the  thermal conductivity calculation of silicon using this patch.

Thanks, this was likely to happen eventually. I will incorporate these changes.


Add tracing/monitoring/metrics

This should be added before we go to release.
Right now we are in the dark about what is happening apart from some very basic cloudwatch logs from the lambda.

Updated the description with specs
please write WDYT the info should be
currently iMO status and the requested provider

@eranelbaz Raised a great comment about what we would need here:

Get notification for each 5xx and 4xx

The notification should contain info

Add the option to mute some notifications (let's say we don't want to get for 404)
Shouldn't block the regular flow

Sure I'm creating relevant issues

Unassigning myself from this issue as I've split it to #106
@Yantrio is there anything left here, or can we resolve this one?

We still don't have consistent tracing when it comes to parallel fetches of versions. I'll keep this open but I don't think we need this before release now as our logging improved greatly with the introduction of slog

I highly recommend using OTEL SDKs that way alternative targets can be supported and users who are self hosting can send telemetry to their observability stack themself
https://github.com/open-telemetry/opentelemetry-go

I was discussing moving to the OTEL SDKs with @cube2222 last week. I think its a good way to go if they work.
Maybe I'll write up an issue and experiment a bit first to make sure it fits the requirements.
This also lead to a conversation around moving the lambdas to containers then we can use the lambda layers easily.


Pen can't read file when file name is Korean

When file name is Korean, Pen just print "Not found"

Will release a new version in a while.

Released as 1.1.1


How replace query without replacing url outside component

In the react 0.13 it it worked something like
        React.render(<Root/>, document.getElementById('react-root'));
    });```

```AppRouter.replaceWith(_.last(AppRouter.getCurrentRoutes()).name, AppRouter.getCurrentParams(), { 'open-discussion': undefined });```

But it doesn't work in the router 1.0.0. I'm create global wariable routerHistory

window.routerHistory = require('history').createHashHistory({
queryKey: false
});

routerHistory.replaceState(null, null, {'open-discussion': discussionId});

Dupe https://github.com/rackt/react-router/issues/2172


How to understand Formula 13 and 14 in your paper?

Describe the error A clear and concise description of what your question is.
Thank you for your work in long-tailed recognition problem. Your work is excellent, and i want to use RIDE in my research file. However, i was confusion  in understanding formula 13 and 14. For example, what do γ means? and How to compute it? as well as what do α means? and How to compute it?

Please have a look at our loss function implementation. You don't need to re-implement it yourself if you just want to use it: https://github.com/frank-xwang/RIDE-LongTailRecognition/blob/main/model/loss.py#L124

Dear Tony,
Thank you very much for your help!
Sincerely your,
Sun
At 2022-06-18 12:21:03, "Long(Tony) Lian" @.***> wrote:
Please have a look at our loss function implementation. You don't need to re-implement it yourself if you just want to use it: https://github.com/frank-xwang/RIDE-LongTailRecognition/blob/main/model/loss.py#L124
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Clippy recommends updating to version 2.4-SNAPSHOT

As of this evening, clippy started insisting on updating to a snapshot version:
[error] New version of clippy plugin available: 0.2.4-SNAPSHOT. Please update!
[error] one error found

I'm assuming this isn't intentional?
In addition, these messages manage to break my build. Probably due to my build settings, but perhaps using an info message instead of an error would be less disruptive?

@easel Thanks for reporting this, I fixed it. Please rm ~/.clippy/clippy.json.gz and now the current version should be 0.2.3.
By default this message is a warning, having "-Xfatal-warnings" in your scalacOptions makes it an error.

Any chance to downgrade the warning to an info? Thanks for the fix!


CLAP plugin, retrieve Track Name

Hello,
Is there any way to retrieve the name of the track the plugin is placed on?
It seems it is supported by the Clap API, see https://github.com/free-audio/clap/blob/main/include/clap/ext/track-info.h#L40
Thanks for the help

It seems it is implemented in clap-sys 0.4.0 https://github.com/glowcoil/clap-sys/blob/master/src/ext/draft/track_info.rs


Fsm implementation

1st commit: project settings. @ngbinh please help reviewing
2nd commit: FSM implementation. Any suggestions are welcome @ngthanhtrung & @nhap96

looks good to me!

Addressed all comments from @ngthanhtrung


Dapp stopped after test BTC deposit

Description:
Dapp send TDT after initiating deposit but stopped after receiving test BTC
Steps to reproduce the behavior:

Go to https://dapp.test.tbtc.network and hit deposit
Connect mestamask, choose 0.5btc
Successful deposit transaction

Then confirm tx
Send btc on generated address

After btc transaction is confirmed dapp still waiting in step: monitoring Bitcoin for transaction to address tb1qn69hhzahrjuc6flv77u57h4rktrp5wznupup40...

After reloading page https://dapp.test.tbtc.network/deposit/0xD361A2D7761dd9C320B6Cb3918e9FaA2aEeed33e/pay situation remains the same

Environment details:
OS: Windows 10 Pro, Version 2004, build 19041.330, 64 bit OS
Web Browser: Mozilla Firefox, Version 78.0.2 (Official Build) (64-bit)
Wallet: Metamask (Chrome extension), Version 8.0.5
Logs
tbtc-log-export-2020-7-24_23-57-31.txt
tbtc-log-2020-7-24_23-28-16.txt

I'm seeing this issue today also. Additional logs for supporting information.
tbtc-dapp-v0.18.0-rc-log-1597177508135.txt
TestBTC Transaction: https://sochain.com/address/BTCTEST/TB1QC27KUEA09ZHCR8VJFDHR9PSJMRZL7DA33L3LXA


Build a privacy policy for mybinder.org

Related to #66
We should have a strong privacy policy that doesn't burden us too much, while still offering users a reasonable amount of privacy.
I consider https://wikimediafoundation.org/wiki/Privacy_policy/FAQ and https://wikimediafoundation.org/wiki/Privacy_policy to be somewhat of gold standards in this case. However, they're all probably too burdensome for someone like us, without the resources of the Wikimedia Foundation. As one example, we'll probably have to use Google Analytics rather than roll our own. We can still try to protect user's privacy tho - by opting out of GA's data sharing, respecting DNT headers, not storing IP info past a few days, etc.

It could just be a simple set of principles to start with, and i think it'll be fairly basic for mybinder. Need not be an iron clad legal document for now.

Perhaps it would be good to combine #66 and #39 with this issue and treat them as one issue since they are all privacy/tracking/anti-abuse related.

I'm +1 to that - if we can get a first start with each of these issues then that'd be a great first step. Having all in one place may make it easier to discuss. We could also open up another issue for a more long-term discussion on this once initial additions are merged.


Procedure for updating GeoLite database

Hi, does telize have a built-in method for updating the geolocation database? Alternatively, what is the procedure for doing so?
GeoLite2's page notes that the database is updated monthly: https://dev.maxmind.com/geoip/geoip2/geolite2/

You need to make a script to be executed by a cronjob or something like that.
What I'm currently doing is updating it manually, but I know it is possible using a script and a cronjob.

Please have a look at geoipupdate: https://dev.maxmind.com/geoip/geoipupdate/


World History button causes error

Sorry to be a pain but clicking World History generates an error in the terminal:
Exception in Tkinter callback
Traceback (most recent call last):
File "/Applications/MAMP/Library/lib/python2.7/lib-tk/Tkinter.py", line 1542, in call
return self.func(*args)
File "history_generator/simulation.py", line 294, in open_world_history_window
'Attack', 'Revolt'])
File "history_generator/internal/event_analysis.py", line 70, in init
self.event_log = Analyser().search('name', '|'.join(self.event_types))
File "history_generator/internal/event_analysis.py", line 42, in init
self.event_list = self.load_events(fname)
File "history_generator/internal/event_analysis.py", line 56, in load_events
with open(fname) as f:
IOError: [Errno 2] No such file or directory: 'event_log.txt'
If I create a blank event_log.txt file I then get:
Exception in Tkinter callback
Traceback (most recent call last):
File "/Applications/MAMP/Library/lib/python2.7/lib-tk/Tkinter.py", line 1542, in call
return self.func(*args)
File "history_generator/simulation.py", line 294, in open_world_history_window
'Attack', 'Revolt'])
File "history_generator/internal/event_analysis.py", line 70, in init
self.event_log = Analyser().search('name', '|'.join(self.event_types))
File "history_generator/internal/event_analysis.py", line 42, in init
self.event_list = self.load_events(fname)
File "history_generator/internal/event_analysis.py", line 58, in load_events
event_list = map(events.Event.from_str, lines)
AttributeError: class Event has no attribute 'from_str'

No worries. I'd like to fix as many bugs as possible and eventually return to this project, but I've been pretty busy with other stuff for a long time now...
At any rate, I believe I fixed this bug now. The event_log.txt file isn't even used anymore, as I've started migrating stuff over to a database-based approach. Unfortunately, I never finished that, but at any rate, all the event log stuff is hopefully working again.


Add options to specify period-check-offset

This is a way to specify "normal" period offsets, for example if an daily dataset appears at the end of the day, we should only start checking it at the end of the day (the offset is 1 period (in this case day))
This fixes #19

Codecov Report

Merging #23 into master will increase coverage by 0.11%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #23      +/-   ##
==========================================
+ Coverage   94.67%   94.79%   +0.11%
==========================================
  Files          17       17
  Lines         263      269       +6
  Branches       12        9       -3
==========================================
+ Hits          249      255       +6
  Misses         14       14

Impacted Files
Coverage Δ

src/main/scala/models/Job.scala
100.00% <ø> (ø)

src/main/scala/AppConfig.scala
97.36% <100.00%> (+0.49%)
: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 377c742...f9017c7. Read the comment docs.


[os-nextcloud-backup] [Feature Request] Add a backup count (keep last n) as in gdrive options

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/plugins/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues, open and closed, and I'm convinced that mine is new.
[x] When the request is meant for an existing plugin, I've added its name to the title.

Describe the solution you'd like
Add a field where put a maximum number of backups to keep

Duplicate of #2893


feat(auth): Implement Passwordless Authentication

Feature Checklist:

[x] Create API for creating/saving a temporary token
[x] Add email template and sending logic
[x] Add validations and timeouts
[x] Update the sign in view
[x] Prevent users without verified email from login
[x] Create API for creating/saving an access token Using loopbacks defaults
[x] Add routes for requesting access token
[x] Invalidate used/expired tokens
[x] Remove password reset logic
[x] Handle all errors gracefully
[x] Remove all passwords from database (#14559 issue created for DevOps)
[x] Add debounce/disable to magic link button
[x] Add some loading indicator to magic link button ajax request
[ ] (berks): Investigate memory consumption issue

Closes #8165
Closes #9027
Closes #15254

@atjonathan @raisedadead @BerkeleyTrue I have created a new issue related to this here: https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10407

@raisedadead Please continue to work on this. #10407 does not change the user stories of this PR.

Reopening this because the underlying passwordless functionality is still needed for updating the workflow.

@raisedadead I know you're recovering from the flu. I just wanted to check and see how this is coming along, and let you know that going passwordless is still a priority for us 😄

Hi @quincylarson, just got back. I plan to work on this from tomorrow. Sorry about the delay.

@raisedadead no need to apologize - thanks for helping us implement this!

Just a rebase of the mess that I created. Will be ready with the feature in the next push.

👍

Basic functionality is there out now… just some more validation touches and code optimisations and we are good (soon).

@raisedadead awesome - thank you for keeping us posted 😄

This is mostly ready, but am at my wits end for getting the request to complete when the link in the email is completed.
Basically I need to get the middleware invalidateAuthToken function to execute correctly.
Would appriciate a review and guidance from anyone.

 I figured out that I was just more sleepy than I should have been last night.
This is ready for QA.

@BerkeleyTrue @QuincyLarson This is ready for a QA.
Thanks a ton to Berkeley for taking the time to help me through this.
I have discovered a few issues with the updateEmail API while doing a load test, and will fix that in a separate PR, because its unrelated and we have open issues for them I checked.
Nonetheless this is ready for a review and deployment on beta for a testing. Berkeley I could not reproduce the issue about bad tokens, maybe beta is a good candidate for a try out!
Also I have created a DevOps Issue for deleting the passwords, which Berkeley can take up when this reaches production.
Awaiting feedback! Cheers.

@raisedadead In accordance with #10407, I think the sign-in/sign-up page should illustrate to campers that signing up for a freeCodeCamp account with GitHub is not allowed.
https://www.freecodecamp.com/signin currently does this best as it does not force returning campers to click an extra time like https://beta.freecodecamp.com/en/signup does and it separates supported sign-in methods from supported sign-up methods. What do you think?

@augmt That is correct, and I plan to make a separate PR as soon as we complete testing this one. We are limiting the change in this PR to email auth and updates.

At the moment, links sent out by beta in the email do not add the subdomain beta. This is not the fault of this PR but we will need to add some sort of feature to include this ability before we can test this on Beta. We can either do it here or as a separate PR.

@BerkeleyTrue I'll add it in this PR itself as soon as I am back tonight. Thanks for the review.

Hey @BerkeleyTrue I have created a new utility for future use to compute the URLs including things like HOST, PORT etc.
Its a overkill as of now, but I am sure would help in the long run.
Finally I have made a minor change in a previous commit to this PR ensuring that at least the message for a success case reaches the client while updating email.
Please check and let me know how this looks.

Done!

Through this up on beta and there are still a couple of issues:

link sent in email is still pointing to freecodecamp and not beta
If I press the 'get magic link` button several times in a row, it will send me several emails back to back

(client side) to prevent this we need to disable the button while there is a request in flight
(server side) We need to figure out some way to lock the user object to prevent the user from constantly sending out emails (unchecked this could provide a vulnerability that could crash servers)

link sent in email is still pointing to freecodecamp and not beta

I tested by setting the below, could NOT reproduce:
NODE_ENV=production
BETA=true

The link is correctly generated, for me.

If I press the 'get magic link` button several times in a row, it will send me several emails back to back

Yup, we need a strong mechanism for this.
Additionally, I am seeing a completely different issue, where createAccessToken from loopback is not correctly associating the correct userId to the accessToken that it generates.
This is a weird one.
Can you hit me up in chat when you can?

@BerkeleyTrue
Regarding the second concern for multiple request, I have two proposals:

Disable the submit, button while the AJAX is in progress.
Migrate the Sign up page to react and use RxJS debounce.

I'll let you suggest which one would be a better alternative?

The problem lies mainly server side. The question is how do you lock a req path by the same user? Like transactions with databases only for expressjs routes.

Maybe we can implement some sort of queuing system to send the emails?

Looks like the beta flag issue is solved.

@BerkeleyTrue:

Added the disable and loading indication to the button when the AJAX is in flight.
For additional debouncing, I think we can take that at the server side rather.

@raisedadead I think we can merge this as soon as the merge conflicts are done. I'll run through it one more time and then we will through it up on beta

This is just an initial rebase of the conflicts, which I basically eye-balled. I am yet to check the changes that I would need to make to get this working as it was earlier.  :rotating_light: :hand:

@raisedadead Somehow all the refactoring I did in user.js has been undone in this PR

@BerkeleyTrue yes, that is correct. I might have messed up the rebase a bit. And have been away a while on an outing. Please give a day or so to clean this up correctly. I'll let you know as soon as it ready.

@raisedadead ping

How is work

@piotrek1543 @frutsy007 refrain from unrequested PR review approvals and unrelated comments.

Hey @BerkeleyTrue so sorry about the delay, I have been a swamped a bit lately for the past couple weeks. I plan to work on this tonight.

@raisedadead no worries. Thanks for the update. I'm excited to see this merged in!

woot!!

@BerkeleyTrue this is ready for QA. Please check when you can. Thanks.

Looks good. I'll merge when I get to a computer

merged!!!!!!!!!!!!!!!!!!!!
https://github.com/freeCodeCamp/freeCodeCamp/commit/61d740b8a0fb144e77a6fa963d0ec7fe16cfead4
✨ 🎊 🚀
I owe you a 🍺
Happy Coding!

@BerkeleyTrue awesome, thanks a lot for the QA. Do let me know how it goes when deployed?
Of course, let's grab some 🍺 when we do meet in person in future!  😄


remove CPU/Memory resources settings

This PR fully removes the CPU/Memory limits as it is generally recommended not to enforce them.
goal is to align with other provider like CAPV/CAPA/CAPG

/ok-to-test

/lgtm
/approve

/test e2e-capx-controller-upgrade


[Relay][TOPI] Resize 1D

This PR introduces a Resize1D op and refactors Resize2D and Resize3D so that all three follow the same API and implementation strategy. It also unifies the reference kernels for testing and expands the capabilities of the reference kernels.
cc @jwfromm @masahi

@masahi @jwfromm I think this is finally ready for review. Sorry for dumping such a large PR on you on a Friday :( Totally fine if you don't get to it for a while.

Thanks @mbrookhart


Add support for Microsoft Edge

Right now testcontainers is very nice and we'd like to use it on the XWiki project (we have a working POC :)). We find it more generic that https://github.com/bonigarcia/selenium-jupiter. However it only supports 2 browsers (chrome and FF), while Selenium Jupiter supports a whole set of browsers (ChromeDriver, FirefoxDriver, EdgeDriver, OperaDriver, SafariDriver, HtmlUnitDriver, PhantomJSDriver, InternetExplorerDriver, RemoteWebDriver, AppiumDriver and even local browsers) through the use of https://github.com/bonigarcia/webdrivermanager.
On the XWiki project, we need to run our tests on all the XWiki-supported browsers (https://dev.xwiki.org/xwiki/bin/view/Community/BrowserSupportStrategy), i.e. Chrome, FF, Edge, IE11, and Safari.
Any plan to support those browsers and specially MS Edge, and any idea how hard it would be to support them?
Thanks a lot!

Hi @vmassol,
cool to hear, that you would like to use Testcontainers for testing XWiki 🙂
Our implementation of BrowserWebDriverContainer (and Testcontainers in general) is heavily based on Docker containers and images (more specifically Linux containers and images). This means we can easily support Browsers which run well inside a Linux container. I doubt we will see Linux support for Edge in the near future (though you never know with nowadays Microsoft), which makes it kind of impossible to use it with Testcontainers for the time being,
There might be some way to support Windows containers in the future, which might be able to run Windows specific browsers, but our roadmap regarding Windows containers (WCOW) support isn't that clear at the moment.
If you have any ideas or suggestions, we'd be happy tp consider them, since I'm sure this is a valid use case for many users.

Thanks for your answer. Indeed running a windows docker image on linux is probably not even supported (since a docker container is not a VM and relies on the host OS). However, I thought that maybe if you had a windows host, you could run docker for windows and have a docker image that has some edge browser version in it.
From what you're saying you're focusing only on linux containers FTM (I can understand that ;) - Just found https://www.testcontainers.org/usage/windows_support.html that I had missed) and thus there's currently no solution for testing the XWiki webapp on Edge. Except by using for example the EdgeDriver from https://bonigarcia.github.io/selenium-jupiter/#edge with a locally installed Edge browser on a windows host.
Same issue for Safari since it doesn't run on linux (it runs on windows though but that won't help ;)).
Ok so seems we'll probably need to continue doing manual testing on IE11/Edge/Safari for the time being for XWiki (or have some :)
Thanks

AFAIK there is no support for running Edge in a Windows container as well (funnily Edge is already running in a specialized sandbox like container in recent versions). So even if we would introduce Windows container support, this wouldn't solve the problem ATM.
There was an open issue regarding this topic over at Microsoft:
https://github.com/MicrosoftDocs/Virtualization-Documentation/issues/214
Maybe one could image of running a software virtualized Windows VM inside a Linux container, and I think I even heard people doing such things for extreme use cases, but I doubt we can this usable for the general public.

Hi,
there is an offical edge image from MS on docker hub by now.
https://hub.docker.com/_/microsoft-msedge-msedgedriver
As well as linux support for edge:
https://www.omgubuntu.co.uk/2021/01/how-to-install-edge-on-ubuntu-linux
can this feature be reconsidered again?
best regards
poorSock

@poorSock yes, it can be considered if someone wants to look into this. We are using the Selenium images, they seemingly don't provide an Edge variant yet. Good course of action could be to contribute it to the Selenium images upstream, than we can directly consume it.

@kiview (I don't know if I have to tag you everytime because I don't know much about github notification policies, sorry if this is wrong)
Looks like selenium has added the image: https://hub.docker.com/r/selenium/standalone-edge

You are right, that looks very usable. Would you be interested into contributing this change to BrowserWebDriverContainer?


feat(entity store): allow prepend option when adding

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/datorama/akita/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
I can't find your doc. I can make an update if you allow me.

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
store.add(new Todo({ id: 1 })); cannot prepend
What is the new behavior?
store.add(new Todo({ id: 0 }),{prepend:true}); can prepend to the list
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

I can't find the docs to update guide

I resolved some format code error, and change my test. Sorry for my fault.

:tada: This PR is included in version 1.14.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Support order property

@Benjamin-Dobell is this change okay for you?

Sorry for taking so long to merge this!


Should header definitions be allowed to be overridden at the resource level?

For example, if I have a token that is required for PUT operations but not GET operations

Gut feeling is this seems weird. Headers feel like something I'd want to setup up front for how to interact with a service. Everything else for a particular resource would then be part of the request.
What's your use case?

It goes back to something paul had suggested on hipchat- using a custom header to supplement our delete operation.  We actually decided not to go this route, but I was just curious how it would be implemented, as it is a header we'd only want for this particular operations and we wouldn't need it for any of the other resources on the client.
The swagger spec does allow specification of headers as parameters for each resource, although I'm not sure how this is implemented.

yeah - i think what we would have to do is to expose a "headers" parameter on each method call. That parameter might conflict with user defined parameters - so it might need to be called "_headers" or some other weird variation (or we use headers unless the name is already taken).
That might actually be a nice change to the clients - I can see us standardizing these 'request' parameters then accepting them when you create the client and also on all of the individual methods.

Even in the headers current form, I can't figure out how the client specifies values for them using the generated play 2.3 client.  Here's my apidoc: http://www.apidoc.me/codalytic/codalytic/latest


Feedback on Tutorial: Developing and deploying a basic Java application on SAP HANA Cloud Platform

Hello,
I got each time an error, that hanatrial.ondemand.com could not be reached. It happens on step 17. I've filled all fields as shown in your tutorial.
In the screen of your step 18, there is an error message:
"Cannot reach host [https://hanatrial.ondemand.com]. Check host name for typing errors and make sure your proxy and network settings are correct"
What is here wrong?
Thanks in advance and best regards,
N.

Hi,
Have you tried to browse hanatrial.ondemand.com in your browser? Is it successful? And if yes, another question is do you have a developer trial account on hanatrial.ondemand.com?

This has happened in several environments, usually with corporate computers (like laptops).
There are two possible issues you may be facing.  First, your certificate store in Java may be out of date, or may have been modified.  This can result in an SSL error, where the certificate is not accepted by Java (and Eclipse runs in Java).
Second, it may also be a proxy problem at your organization.  The Eclipse system may not be using the proxy server correctly.  In that case, the proxy server is responding to the request, and since it is not the same host, Eclipse throws an error.
There are several ways to debug the problem.  First, try using a separate Java installation for Eclipse.  You can download the Java VM from Oracle (or use our internal SAP Java VM).  Then point Eclipse to this Java VM directly (using the Eclipse.ini file).  This isolated environment usually solves any Java related issues, including configuration or Certificate mismatches.
If that doesn't solve your problem, then the issue is likely a proxy or firewall issue.  Eclipse uses port 443 to communicate with the HCP server.  Try setting the proxy settings in the Java Control panel, and see if that fixes your issue.  More information on that here
Let us know how that works out.

Hi there.  I haven't heard anything back, so I'm closing this.  If you are still having a problem, please reopen it.


Stop sending openended_status and installments

What's this PR do?
Removes the openended_status and installments form values from front and back end.
Why are we doing this? How does it help us?
We don't need them now that Circle, BusMem and regular donations have discreet pages.
How should this be manually tested?
make, make restart, then make one time, monthly and yearly donations on /circle, /business and /circle. Confirm the proper amount and frequency show up in #stripe.
How should this change be communicated to end users?
NA.
Are there any smells or added technical debt to note?
NA.
What are the relevant tickets?
https://3.basecamp.com/3098728/buckets/10357222/todos/1787329290
Have you done the following, if applicable:

[ ] Added automated tests?
[ ] Tested manually on mobile?
[ ] Checked for performance implications?
[ ] Checked for security implications?
[ ] Updated the documentation/wiki?

@x110dc There were some Python errors when I made test donations on /circle and /business. I have the traces saved; let me know if you'd like to see them.

Ok, I think I've got those fixed.

Yep, Circle and BusMem are both working swell for me now. Feel good about me merging this?

Yep!
On Mon, May 20, 2019 at 4:02 PM Andrew Gibson<EMAIL_ADDRESS>wrote:

Yep, Circle and BusMem are both working swell for me now. Feel good about
me merging this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/texastribune/donations/pull/199?email_source=notifications&email_token=AAIKSGDSXKRSZCJCOLAHKTLPWMGXRA5CNFSM4HODJRH2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODV2CDJY#issuecomment-494150055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIKSGHQ7UGUZCHGOFIWWRLPWMGXRANCNFSM4HODJRHQ
.

--
Daniel Craigmile
Chief Technology and Security Officer
919 Congress Ave., Sixth Floor
Austin, TX 78701
www.texastribune.org
http://texastribune.org/
M<PHONE_NUMBER>
@x110dc


Port argument read as string and not integer

When using the --port argument, the number is read in as a string which in turn causes the startup to fail. The --port argument content needs to be treated as a number.
±  |master U:1 ?:1 ✗| → docker run -p 9158:9158 nexus_exporter --port 9158
INFO:nexus-exporter:starting nexus exporter on port 9158
ERROR:nexus-exporter:unable to fetch data from metric endpoint: <urlopen error [Errno 99] Address not available>
Traceback (most recent call last):
File "/nexus_exporter.py", line 267, in
start_http_server(args.port)
File "/usr/local/lib/python3.7/site-packages/prometheus_client/exposition.py", line 193, in start_http_server
httpd = _ThreadingSimpleServer((addr, port), CustomMetricsHandler)
File "/usr/local/lib/python3.7/socketserver.py", line 452, in init
self.server_bind()
File "/usr/local/lib/python3.7/http/server.py", line 137, in server_bind
socketserver.TCPServer.server_bind(self)
File "/usr/local/lib/python3.7/socketserver.py", line 466, in server_bind
self.socket.bind(self.server_address)
TypeError: an integer is required (got type str)

wow thanks, have created an MR


How to deal with AES256JNCryptorInputStream?

Are there any docs how to deal with AES256JNCryptorInputStream?
My task is to decrypt a whole file on the filesystem and i thought, this stream could maybe help here ...
Any hints are appreciated. Thanks!

What have you tried so far?

For too slow decryption and encryption here is the perfect solution.
https://github.com/TGIO/RNCryptorNative?utm_source=android-arsenal.com&utm_medium=referral&utm_campaign=3643)
found this library while i was searching for alternatives of JNCryptor. This lib derived from native and it does pretty well job.
I had tried many solution but none of that was working and finally this library did.
You can check the compare graph below.

I'm closing this. Please re-open if you can better formulate your question.


Allow RFC2965-style attributes in Cookie header

This patch allows the Mock Server to gracefully handle HTTP requests that include the extra attributes that are defined in IETF's RFC2965 https://tools.ietf.org/html/rfc2965. All existing functionality is preserved, and the code now relies on Netty's implementation for the parsing of the Cookie header instead of splitting the string explicitly.
I would like for this to be reviewed, and hopefully merged and released so that I can revert to using a production version of the library in my project instead of patching the library to be compatible with existing code).
These RFC2965 attributes start with a dollar sign "$". Here's an example Cookie header:
Cookie = $Version=1; cookieName=CookieValue ; $Path=/resource
Before this patch, MockServer would throw an exception while trying to parse this header. The root cause is that it is using Netty's ClientCookieDecoder instead of the ServerCookieDecoder (and these attributes beginning with a dollar-sign are passed from client to server, rather than from server to client)
This is important to me because I'm using the Jersey library on a project with a REST service that uses a cookie, and Jersey follows this style of adding the $Version to all cookies.
To show the issue - this simple JUnit test with some example Jersey client code will fail without the patch, and will now pass after the patch.
package org.mockserver.codec;

import static org.mockserver.integration.ClientAndServer.startClientAndServer;
import static org.mockserver.model.Cookie.cookie;
import static org.mockserver.model.HttpRequest.request;
import static org.mockserver.model.HttpResponse.response;

import javax.ws.rs.client.Client;
import javax.ws.rs.client.ClientBuilder;
import javax.ws.rs.client.WebTarget;
import javax.ws.rs.core.Response;

import org.junit.Test;
import org.mockserver.integration.ClientAndServer;
import org.mockserver.socket.PortFactory;
/*
 *
 * Demonstrate the feature required and why it's important...
 *
 <dependency>
    <groupId>org.glassfish.jersey.core</groupId>
    <artifactId>jersey-client</artifactId>
    <version>2.12</version>
</dependency>
 */
public class JerseyTest {

    @Test
    public void shouldWorkWithJersey() {
    	int SERVER_HTTP_PORT = PortFactory.findFreePort();
    	ClientAndServer mockServerClient = startClientAndServer(SERVER_HTTP_PORT);

		// given
		mockServerClient.when(request("/resource").withMethod("GET"))
		.respond(response().withBody("Hiya!"));;

    	Client client = ClientBuilder.newClient();
    	WebTarget target = client.target("http://localhost:"+ SERVER_HTTP_PORT).path("resource");

    	target.request().cookie("k","v").get();

    	mockServerClient.verify(request().withCookie(cookie("k", "v")));

    	// stop mock server and client
        if (mockServerClient instanceof ClientAndServer) {
            mockServerClient.stop();
        }
    }

}

Brilliant. Thanks James!


how can i use variables as katex functions?

hi. just one question.
how can i use in
katex.render("a + b = c", symbole_40d);

some variables like
var a = 2
var b = 4
var c = 0
katex.render("{$a} + {$b} = {$c}", symbole_40d);
console.log('c=' + c) //Output: c=6

in that kind of meaning. i know it would not works, but for that try to represent that what i need i think its ok as symbolic code :D. big thx

Here are a few ways:

using template literals:katex.render(`${a} + ${b} = ${c}`, symbole_40d);

Using + operator on strings, like you did in the console.log call:katex.render(a + ' + ' + b + ' = ' + c, symbole_40d);

Using KaTeX macros:katex.render("\\a + \\b = \\c", symbole_40d, {
  macros: {
    "\\a": a,
    "\\b": b,
    "\\c": c,
  }
});

(Note that this overrides the default meaning of \c (a type of accent). It might be better to call the macro something else.)

big thx!


[TASK] Full protocol-solidity contract deployment on local EVM chain

After a second thought, looks like that not all crates of alloy got published yet, will try to get the latest version of anvil to work with the current ethers.


2024-06-25: Project Lead sync

Attendees

Project Lead: @tholewis
Project Lead: @tmcclell
Project Manager: @russelltrow

Agenda

[ ] https://github.com/Green-Software-Foundation/higher-education-engagement/issues/4

[ ] Scheduling with https://github.com/Green-Software-Foundation/higher-education-engagement/issues/11
[ ] https://github.com/Green-Software-Foundation/higher-education-engagement/issues/15
[ ] Opening up the synthesis to others?

Meeting Notes

Aim to get Salento university interview scheduled ASAP
Agreed to chase those who haven't responded after 1 week
We will create a simple script for the interviews so we have some consistency
Once we have the interviews in progress figure out how to involve others from across the GSF in analysing the results and determined the roadmap

Actions

[ ] @russelltrow share booking link with Salento University as they have express interested
[ ] @russelltrow send reminder email to those that haven't responded by next week


videos won't play

I just installed the dev build with Greasemonkey and after about 5 minutes YT vids will stop working. I press play and just get an endless spinning circle with the vid never starting to load, and have to restart FF to get them to work for another 5 minutes. I never had this behaviour with the addon before it was pulled. Anyone else had this problem?

If your Firefox version is 43.0.2 then update it to 43.0.3, there was a problem with video decoding in that version.

Thanks, I'm already on that version though


[zh] localize blog: 2023-04-20-read-write-once-pod-access-mode-beta.md

This is a Feature Request
I want to translate this blog.
docs/en/blog/_posts/2023-04-20-read-write-once-pod-access-mode-beta.md
/language zh
/kind feature
/assign

PR #40709 has been merged, so the issue can be closed.
/close


Feature Request: Filter by Collection

The ability to filter so that only cards that are in the user's collection are visible would be handy for building decks out of only cards that a user owns.

You can just increase the number of cards filter for that :)

Set it to 1 and you get all cards you have at least once


word2vec with letterboxd data

Description
movie-recommender word2vec API deployment on cortex library using user letterboxd ratings, watchlist, and watched
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Change Status

[x] Ready for Review/deployment

How Has This Been Tested?

[x] Deployed with Cortex and the endpoint was succesfully returning recommendation id.

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] My code has been reviewed by at least one peer
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] There are no merge conflicts

@rileythejones you want to make sure you fill out the check boxes in the PR template to reflect what the PR is for.

Merging after successful deployment to Cortex

Merging after successful deployment


Add linked list features

This a testing of my git knowledge. I'm using gnl 42 project to make 2 versions: with arrays and with linked list


matrix-synapse: 1.5.0 -> 1.5.1

Motivation for this change
https://github.com/matrix-org/synapse/releases/tag/v1.5.1
Things done

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

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

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

Notify maintainers
cc @

@ofborg test matrix-synapse

@Ekleog do you also want to backport this? %)

This might qualify as a security update as it's supposed to prevent a DoS attack.

Sounds like it'd make sense to me, especially as IIRC releases are already on 1.5.0 :)

Still needs a backport, right?

Ouch forgot about that. Do you want to do this? :)


ca.total_inertia_ > 1.0

Hi,
I was surprised to get a ca.total_inertia_ equal to 1.45.
If I np.sum(ca.explained_inertia_), I then get something very close to 1.
I though that 1 was supposed to be the correct answer, if we ask for a lot of components.
Am I misunderstanding something?

Hey,
The total_inertia is simply the sum of the variances assigned to each component. The explained_inertia is a list of the proportion of variance assigned to each component. In other it is a percentage. explained_inertia is obtained by dividing each eigenvalue by total_inertia.
I hope this makes things clearer.

Hi,
Thanks a lot for clarifying this!


Can't get access token with non-gmail

Hi, i'm playing with a google account that ends with<EMAIL_ADDRESS>i used this on ingress and started to use it here on PKMN GO as well, but even though the phone app accepts it, something doesn't like it here, either the program or api, so i get:
Access token not generated
Session not created successfully
Do you think there's anything you or i can do to fix this problem?
Thanks in advance

Are you using 2FA?
Did you follow everything in the Readme?

Ok, apparently it was a temporary problem on my side, it seems to be working perfectly now! :)


Update overview.md

A resource is created or updated in a scope with a policy assignment.

Removed "deleted".
Policy evaluation is only enforced on PUT operations, that doesn't include DELETE operations.

@ronarieke : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Event popup update

I fixed the bug that made the 'Add Event' popup open after a user clicks 'Close' on the edit event popup. It's ready to be reviewed and merged to main.

@KajaGrabowska Thank you again!


Process does not take into consideration SSRP

An employee that is scoped into SSPR can simply reset the password using standard recovery methods for a forgotten password.  The current MFA setup needs to be removed to prevent recovery after the password is changed.

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

ID: 4e366b9c-0d35-83c6-f509-1774dd351504
Version Independent ID: 702f2dc1-46f8-508a-68bc-b5d033c56a9a
Content: Remove a former employee - Overview - Microsoft 365 admin
Content Source: microsoft-365/admin/add-users/remove-former-employee.md
Service: microsoft-365-business
GitHub Login: @kwekuako
Microsoft Alias: kwekua

@msbemba Thank you very much for the contribution and sharing this explanation. @troyradtke Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


CON-148 added id maps

@george-moberly do you know if I should be adding an id_map to coreo_aws_advisor_alert "no-global-trails" on line 37? It's a js-runner situation


Tk bug send errors to global error handler

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Loom Video
https://www.loom.com/share/057e4a13481a4d54aa8a453e70ed53f7
Type of change
Please delete options that are not relevant.

[x ] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have removed unnecessary comments/console logs from my code
[x] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

Doesn't seem to be any corrections that need to be made to what you've changed. looks good


Feedback for “Configure GraphQL Code Generator”

I'm having a hard time generating TypedDocumentNodes in mesh. I have had lots of success with the graphql-codegen and the typed DocNodes are generated properly. I can see the document files being ingested at build time, but I dont know where they're going or what mesh is doing with them.
A string search reveals that my client query name is within a getter called rawSDL() but I have no idea how to use it.

SDK -> https://the-guild.dev/graphql/mesh/docs/guides/mesh-sdk#generating-fully-type-safe-sdk
Persisted Operations -> https://the-guild.dev/graphql/mesh/docs/guides/persisted-operations
These are two places Mesh uses documents provided by the user.

In the SDK example, my only option is to execute. This doesn't help me generate TypedDocumentNodes to feed to an Apollo Client query hook in React.
I'm trying to replace/replicate (or even trigger) the client-preset we've been using in codegen (https://the-guild.dev/graphql/codegen/plugins/presets/preset-client)
Is this possible or do I need to manually trigger it in some custom extension?


Update Windows deployment scripts

@fifthposition Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@fifthposition Thank you for signing the Contributor License Agreement!


STATS: Jupyter examples

Looking for the following notebooks (under RAiDER-docs github repo):
Example 1 (Priority)
Get GPS information and visualize station distribution for fixed period of time
this is inline with #11
Example 2

Get GPS information and visualize delays for a fixed period in time, calculate basic statistics
Use the GPS station information to trigger RAiDER and compute delays
use the RaiDER and GPS delays for basic statistical analysis.

@sssangha in reference also to #44 the tropo branch of my RAiDER fork can check whether files exist for a given year and produces a .csv file with a list of years and station names and data availability. You could use this as a start to address this issue.

first version added under https://github.com/dbekaert/RAiDER-docs/pull/1


fuse.E* numerous missing constants

As you said you would prefer one lump report instead of isolated issues, here is a (non-exhaustive) list.  Implementation requires more than just defining constants, hence the different issues pertaining to each.  The rationale behind each is also completely different.

The following system error codes are missing as constants in the fuse package

EACCESS
EBUSY - for network filesystems
EAGAIN - for network filesystems
ENODATA
EISDIR - for OpenFile on directories
EPERM
ERANGE - for directory listings (see: #53 for details)
EEXDEV
EBADF - for close(2) and friends (see #55 for details)
EROFS - for read-only filesystems (see #56 for details)

I don’t understand your overall concern here. Are these constants not
available in your syscall package?

There is a large inconsistency with some error codes being defined in the fuse package, where each code listed above is not defined here.  This would also lead one to believe they are not being correctly utilized within the package (giving no need to define them, previously.)
I had noticed EINVAL used in many incorrect places, which would be resolved by correctly implementing each of the above error codes where they should be expected, as described in the kernel man pages.
Some codes that ARE defined include:

fuse.EIO
fuse.ENOSYS
fuse.EEXIST
fuse.ENOENT
fuse.ENOATTR
fuse.ENOTDIR
fuse.ENOTEMPTY

these constants are there for your use

Here are just three (of many) examples of where we might need these constants to correctly implement a filesystem:

Responding to SetXattr on bad names (missing or incorrect prefix)

The Linux man pages state under ERRORS for setxattr(2) that one must reply with ENOTSUP (which is not semantically the same as fuse.ENOSYS) when attempting to set an attribute with a prefix not defined in the specification.  You can test this on existing filesystems (to see how they respond) with the setfattr tool found in the attr system package.

Responding to ReleaseFileHandle on already closed or stale file handle

The Linux man pages state under ERRORS for close(2) and close(3) that one must reply with EBADF when the file handle passed in " isn't a valid file descriptor. "

Responding to OpenFile on a directory

The Linux man pages state under ERRORS for open(2) that one must reply with EISDIR in one of the multiple conditions.  Here is a quote of the relevant passage from the pages included in Ubuntu 18.04 for reference:
       EISDIR pathname refers to a directory and the access requested
              involved writing (that is, O_WRONLY or O_RDWR is set).

       EISDIR pathname refers to an existing directory, O_TMPFILE and one of
              O_WRONLY or O_RDWR were specified in flags, but this kernel
              version does not provide the O_TMPFILE functionality.

Ideally it would be consistent and define all or none of them, but I don't see how that's possible given that the set of error codes may differ across platforms.

As per conditional compilation
# +build linux darwin
// ... platform specific constants


Update isIssuedBy and isSignedBy to use util::StatusOr

And corresponding change to isSelfSigned + update affected tests.

All reviewed, mostly nits, really.

I think I've fixed everything. Also rebased and fixed up conflicts with other refactorings. It's starting to pay off a bit e.g look at CertChain::IsValidSignatureChain.

LGTM


Generate E064 for fixture E060_E064_root_inventory_digest_mismatch

See https://github.com/OCFL/fixtures/pull/75

root and latest/v1 inventories do not match:
fixtures> diff 1.0/bad-objects/E060_E064_root_inventory_digest_mismatch/inventory.json 1.0/bad-objects/E060_E064_root_inventory_digest_mismatch/v1/inventory.json
20c20
<       "message": "Changed!",
---
>       "message": "Simple example",

For the particular test in question #61 results in:
../ocfl-validate.py 1.0/bad-objects/E060_E064_root_inventory_digest_mismatch
[E060] Mismatch between actual and recorded inventory digests for inventory.json (calculated 5bf08b6519f6692cc83f3d275de1f02414a41972d069ac167c5cf34468fad82ae621c67e1ff58a8ef15d5f58a193aa1f037f588372bdfc33ae6c38a2b349d846 but read cb7a451c595050e0e50d979b79bce86e28728b8557a3cf4ea430114278b5411c7bad6a7ecc1f4d0250e94f9d8add3b648194d75a74c0cb14c4439f427829569e from inventory.json.sha512) (see https://ocfl.io/1.0/spec/#E060)
[E064] Object root inventory and copy in last version MUST be identical but are not (inventory.json and v1/inventory.json) (see https://ocfl.io/1.0/spec/#E064)
INFO:ocfl.object:OCFL object at 1.0/bad-objects/E060_E064_root_inventory_digest_mismatch is INVALID

which is what I want. This work ended up with a number of changes to improve testing of paths in the manifest and to avoid short circuiting the inventory tests too soon.

Fixed in 1.2.2


descriptor: add From impl for key enums

Adds From<u8> impls for KeyboardUsage, MediaKey, and SystemControlKey.
Adds Reserved variants to handle reserved and invalid values.

Please fix the clippy errors, otherwise looks good.


Avoid creating slices that exceed isize::MAX bytes.

I think adding an assertion to MmapOptions::len is sufficient to catch all cases leading to erroneously large slice lengths due to the check in MmapOptions::get_len. I also think an assertion is appropriate for the setter and its indirect call via MmapMut::map_anon so that this is backwards-compatible.
 And memmap2-rs doesn't have any right now, afaik. So I would prefer to check len on creation.

 And memmap2-rs doesn't have any right now, afaik. So I would prefer to check len on creation.

How do you feel about making the setter itself fallible? Or even adding a separate fallible try_len setter?

I moved the check into MmapOptions::map_anon which should still catch all paths as MmapOptions::get_len covers the other calls.

Hm... I honestly not familiar with memmap2-rs API, since I do not use anything beyond read-only file memory mappings.
Now I understand your point. Yes, len should panic and we should have try_len. This way we would be similar to std API.
I'm also confused about map_anon now, because the docs state that length must not be 0, but it would be silently ignored because we use .unwrap_or(0). Shouldn't we use unwrap/expect here instead? Or do we get an error from the OS anyway?

The last commit is good. I think we can leave at as is.

I'm also confused about map_anon now, because the docs state that length must not be 0, but it would be silently ignored because we use .unwrap_or(0). Shouldn't we use unwrap/expect here instead? Or do we get an error from the OS anyway?

I think the comment is outdated. The system call would fail, but special support for zero-length mappings (necessary to gracefully handle empty files) was added in 118344980efaaff161c68921a604fee110a84919. Will update the comment...

Great! I've lost track of all of those changes.
Thanks again for the patch!


DS-680 Select icon placement bug

Jira
https://pegadigitalit.atlassian.net/browse/DS-680
Summary
The chevrons in the select filters within the Best of Content pages were fixed.
Details
On Best of Content pages, select chevron icons were overlaping the text :

BoC - PegaWorld Replays
BoC - Customers

Additionally, on the Calculator - Results I found accidentally a bug related to color theming on those two colorful rectangles. I made a quick fix.
How to test
Pull the branch. Check if the select fields on the BoC pages display correctly, without overlapping the text. Confirm there're no more broken selects with chevron icons overlapping the text.

@colbytcook went over it again today to fix the issue mentioned in ticket DS-680

This is a component bug, we shouldn't just add utility classes to the mockups but fix the component instead.

@mikemai2awesome if we cannot use utility classes like for example u-bolt-visuallyhidden and others, I added new display type for a label which uses visually hidden mixin and updated the padding in a form.scss to prevent overlapping text over the chevron

@mikemai2awesome updated 24px to 1.5rem and used fixing styles only for the select element.


4.2. Implement confirmation dialogs in the frontend to prompt the teacher for confirmation before deleting a quiz.

Task 4.2 has been completed

Task 4.1 has been completed


Fix remaining nearlib rename quirks

Cherry-picked some stuff from https://github.com/near/near-api-js/pull/287 and changed some other (e.g. using nearApi instead of nearAPIJs).
Local storage prefix cannot be changed this easily (we would effectively nuke existing accounts in wallet).

I'd like to eventually see the keystore prefixes renamed

@mikedotexe IMO it's not really worth effort by itself (needs data migration and tests for it). IMO makes sense to do when data migration is needed for other reason.


Padding before the title if there is no leading

I was going to use the App Bar without a leader, but in this case there is a void in front of the title. Even if I set left padding to 0, the situation won't change much. If there is no leading, then the padding on the left and right is different.

 AppBar(
            padding: EdgeInsets.only(left: 0, top: MediaQuery.of(context).padding.top + 8, right: 0, bottom: 8),
            title: const Text('Counter App'),
            trailing: [
              GhostButton(
                onPressed: () {},
                density: ButtonDensity.icon,
                child: const Icon(Icons.search),
              ),
              GhostButton(
                onPressed: () {},
                density: ButtonDensity.icon,
                child: const Icon(Icons.add),
              ),
            ]
)

I don't think this is the case anymore in the latest commit, can you check?


error: package solana-program v2.0.0 cannot be built because it

Problem
use solana-program = "2.0.0" to compile a very simple custom program failed with package solana-program v2.0.0 cannot be built because it requires rustc 1.75.0 or newer, while the currently active rustc version is 1.72.0-dev
the output of cargo-build-sbf --version is solana-cargo-build-sbf 1.18.5  platform-tools v1.39  rustc 1.72.0
I mean where i can find the toolchain supporting rustc 1.75.0 or above for now?

I encountered the same problem, it was a mess

try rustup update


[Doc]update the emr getting started doc for emr-6130 release[skip ci]

update the emr getting started doc for emr-6130 release
https://docs.aws.amazon.com/emr/latest/ReleaseGuide/emr-6130-release.html
Signed-off-by: liyuan<EMAIL_ADDRESS>

verified

build


vgg.py: floating point -> floor division in get_img_output_length

Exchanged floating point for floor division to yield integer valued image dimensions.
This had caused a freeze in train_frcnn.py for me through line 181 next(data_gen_train) coming through get_anchor_gt(...) and ultimately calc_rpn(...).

@JonasHarnau  By freezing you mean no output after it displays ?

Started training
epoch: 1/10000

@Abhijit-2592 yes exactly! The issue is that the float division (/) yields a float valued pixel value which unfortunately doesn't yield an error but simply freezes everything. Floor division (//)  fixes the issue. The Resnet implementation uses floor division and thus does not run into the problem.

@JonasHarnau  Thanks man it is working! And the num 16 in
return input_length//16 is the rpn stride right?

Good to hear :)!
From my understanding the get_img_output_length maps the input image size to the output size of the last shared convolutional layer. Here, that is block5_conv3. In VGG16 the size reductions come solely from the Max-Pooling layers with stride 2 of which there are four. Thus the output of the shared layers is 1/24= 1/16 the input size.

@JonasHarnau  Thanks ! That makes perfect sense

@Abhijit-2592 yes, the 16 is random in as far as the network architecture is arbitrary. Keeping the architecture fixed, the total stride of the last shared conv. layer given the input image size is always 16.  My understanding from the quoted paragraph is that, since they resize the shorter side of the input image to 600, this translates to an 'effective' stride of 10 on the average PASCAL image (375/600 * 16 = 10). However, this would differ for each individual input image: if you use a smaller picture as input the effective stride would decrease, e.g. for a 200x150 image the 'effective' stride on the original image is 150/600 * 16 = 4. On the other hand, for a larger image the 'effective' stride goes up.

@JonasHarnau  Thanks for your quick responce :) . Thanks for confirming it (was pretty vague until now :P). And I guess this code already deals with finding the effective stride (10 for pascal VOC) depending on the specified rpn_stride (default 16) and smaller side of input image before resizing (~375 for Pascal VOC).
Thus we can easily add any arbitrary CNN trained on keras for image classification as a fixed feature extractor by the following the steps:

strip the layers of the custom network until we get the required convolution layer.
And define get_img_output_length()  accordingly.

I am kinda trying to make this code into a small API (Inspired from the recent tensorflow's object detection API ) with this feature. So that almost any classification model trained on keras can be extended to an object detection network with minimal effort. Interested in joining?

@Abhijit-2592 apologies for the delay. That sounds cool, but I don't think I have the time for it right now. It would be great to have a look at it once you're done!

Thanks for the PR - I made a change for python2/3 compatibility and I'm merging it


[HOWTO] Disable +default--newline-indent-and-continue-comments-a in Coq mode

What I want to achieve
+default--newline-indent-and-continue-comments-a is infuriating in Coq mode. Coq comments are not line comments, and I really doubt anyone writes comments like those:
(** Start of a sentence *)
(** that continues on *)
(** other lines. *)

Instead, this is enough:
(** Start of a sentence
    that continues on
    other lines. *)

and allows itself better to things like g q...
I don't know how to disable this feature, either globally, or simply in Coq mode.
Alternatively, is there a way to force a newline character? Sadly, the Ctrl V trick inserts ^M.

The correct fix for this would be PR a better comment-line-break-function for coq-mode upstream, but for the time being, this should work around the issue:
(setq-hook! 'coq-mode-hook comment-line-break-function nil)

Sounds good. Thank you for preserving my sanity! <3


Add retry in case 'aws eks wait cluster-active' returns error due to rate limiting

…rate limiting
Issue #, if available:
Description of changes:
Adds a check if the command 'aws eks wait cluster-active' fails it should retry.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing as stale. If retries need to be used for any of the AWS CLI calls, we should use the built-int mechanisms. https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-retries.html


Current version Az Powershell RM

How much longer will the current Az Powershell RM be supported in production environment?   When will it be completely stopped the service?  Some of our customers are planning to keep using their current version and concerning if/when it is completely stopped using or any fee will be charged to switch to the NEW PowerShell Az module...

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

ID: 65abff75-73f1-3033-d816-e03e0d7fe4ff
Version Independent ID: 524598ca-76aa-c8f5-cde1-d6bd626b757b
Content: Introducing the Azure PowerShell Az module
Content Source: docs-conceptual/azurermps-6.13.0/new-azureps-module-az.md
Service: azure-powershell
GitHub Login: @sptramer
Microsoft Alias: sttramer

@markcowl Please respond with some detailed information about AzureRM lifecycle support.

The biggest complaint I have is that the login method is the same as the CLI which is just horrendous having to copy a login URL and then an access code every time you want to start a session. PowerShell AzureRM had a much better login method because you can just easily type your credentials.

Login shall be fixed on 1/15/2019 per article https://docs.microsoft.com/en-us/powershell/azure/migrate-az-1.0.0?view=azps-1.0.0#temporary-default-device-code-login-instead-of-web-browser-prompt

The new module does not incur any additional fees. The new module was released only to offer better support, more consistent names, and work with PowerShell 6. AzureRM will continue to be supported with bug fixes, but receive no new features. @markcowl can provide information on the support window, and that information will be added to the migration docs.

Keeping the login UX the same as the AzureRm module should have been a required feature for a 1.0.0 release.


Custom types are not handled properly when fields are qualified with a table alias

Describe the bug
Consider a QueryBuilder join such as this:
        qb.joinAndSelect('Foo', 'x', {
            e1.someField: 'abc'
        }, 'leftJoin');

QueryHelper tries to find the property using the full key, which in this case would be e1.someField.
const prop = meta?.properties[key];
That results in prop being null and processCustomType() is not called.
Stack trace
None
To Reproduce
As described above
Expected behavior
I would expect QueryHelper to strip off the table alias before trying to lookup the propery, and the custom type behavior would work.
Additional context
Versions

Dependency
Version

node
4.18.3

typescript
4.6.4

mikro-orm
5.1.5

your-driver
mysql

Correction:  Stripping off the table alias wouldn't be enough.  It would have to lookup the alias to find the proper metadata as well.

QueryHelper does not know anything about aliases, it does not know QB class even exists, so this will need to be fixed elsewhere most probably.

I was just looking at that.  Considered adding aliasMap to ProcessWhereOptions and sending over a PR.  Is that palatable?

Yeah I guess we can do that :] Its not perfect but it will be probably the fastest way without much code duplication (we still need to duplicate this method).

That looks quite good, made some perf improvements so we dont need regexes on such hot path, but other than that all tests are passing. Thanks!


Adds SKIP_HEAD_INSTALLS as an environment variable for installation

When set to a non-empty string, will skip adding packages that install the head version from github, which can overwrite an existing installation of that package. In this case, jax and fiddle will be skipped.
Assuming this change is acceptable, the same change would need to be made in paxml as well since it also installs jax at head
@aireenmei

Thanks!

Closing since it's in main now: https://github.com/google/praxis/commit/ca3035f61a786c0cbb4bcc4cd80009e31facb540


Work of graph indexer service

In my indexer-service have an error "Error: listen EADDRINUSE: address already in use :::7300"
in nano /etc/systemd/system/graph-indexer-service.service i chache metrics-port to 7302 (or other) (in second screen)

systemctl daemon-reload
systemctl restart graph-indexer-service
this commands after chanche i do but the error doesn't go away
help

reopen here https://github.com/graphprotocol/mission-control-indexer/issues/227


Improvements to search results

Since you are already doing this, could you please add this too 😛 :
Change the following code in search.service.ts:
      if (!(term === null || term === '')) {
        const termregex = new RegExp(escape(term).replace(/\s+/g, ' '), 'i');
        return (termregex.test(elem.i) || termregex.test(elem.u) || termregex.test(elem.n.replace(/\s+/g, ' ')));
      }

Instead of the above do:
      if (!(term === null || term === '')) {
        return (fuzzy.test(term, elem.i) || fuzzy.test(term, elem.u) || fuzzy.test(term, elem.n));
      }

Here fuzzy is a node module you'll need to add and import * as fuzzy from 'fuzzy'.

Still not an informative name, but a start. General intention is to convey
the change with some minor specifics so someone can investigate further if
need be. Would not matter here, but you'd find this habit saving you
countless review comments once you join a company.
On Thu, 9 Nov 2017, 23:26 Yash Srivastav<EMAIL_ADDRESS>wrote:

@yashsriv approved this pull request.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pclubiitk/student-search/pull/22#pullrequestreview-75529637,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJ75pDRURe9ZDvYIIaqCxJZU8zF-15vjks5s0zzEgaJpZM4QYRBD
.


New "dependency" optimisation feature breaks "tfenv"

The new dependency optimisation introduced in terragrunt version 0.23.35 breaks tfenv (at least on a Mac) when it's used with a .terraform-version file in the terragrunt root.
I think this is caused by the introduction of a temporary folder here
terragrunt changes its working directory to the temporary folder, and when it runs a terraform command from there tfenv cannot find the .terraform-version file that is in the project root, so it will use the system-wide version, which will often be either unset or a different version
This causes issues like this (tfenv system-wide version unset):
terragrunt plan
tg plan
[terragrunt] [</terragrunt/path>] 2020/09/14 11:35:11 Running command: terraform --version
[terragrunt] 2020/09/14 11:35:13 Terraform version: 0.12.29

...

[terragrunt] [</some/terragrunt/dependency/path] 2020/09/14 11:35:19 Running command: terraform output -json
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
Version could not be resolved (set by /usr/local/Cellar/tfenv/2.0.0/version or tfenv use <version>)
[terragrunt] 2020/09/14 11:35:22 exit status 1

or this (tfenv system-wide version different and incompatible):
terragrunt plan
tg plan
[terragrunt] [</terragrunt/path>] 2020/09/14 11:35:11 Running command: terraform --version
[terragrunt] 2020/09/14 11:35:13 Terraform version: 0.12.29

...

[terragrunt] [</some/terragrunt/dependency/path] 2020/09/14 11:35:19 Running command: terraform output -json
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
Failed to load state: state snapshot was created by Terraform v0.12.29, which is newer than current v0.12.26; upgrade to Terraform v0.12.29 or greater to work with this state
[terragrunt] 2020/09/14 11:38:02 exit status 1

Please note how terragrunt outputs the terraform version at the beginning (the correct version), when the working directory is still the same directory in which I'm running terragrunt plan, but everything changes when it's working on the dependencies, and tfenv suddenly doesn't know the terraform version, or it knows a different one

P.S.:
This can be worked-around by setting the system-wide terraform version for tfenv with:
tfenv use <terraform_version>
or by setting the new terragrunt variable disable_dependency_optimization (introduced in terragrunt version 0.23.37) to true in the remote_state block.
Both the above work-arounds are not acceptable as solutions.
The first one is not the best way to use tfenv and it makes having a .terraform-version file useless.
The second one disables the dependency optimisation, which was indeed a great and much appreciated optimisation.

Maybe changing this line to one of the below could be a solution, @yorinasub17 (tagging you since you seem to be the author of the change)?
tempWorkDir, err := ioutil.TempDir("./", ".*")

or
tempWorkDir, err := ioutil.TempDir("./.terragrunt-cache", "")

or anything similar to create the temporary working directory still inside the terragrunt project

Yup that makes sense, although using ./.terragrunt-cache directly is not correct if the terragrunt cache is configured to a different directory. Will try to work on this this week.

Just opened a PR with a potential fix. Can you see if that works for you? I released this as https://github.com/gruntwork-io/terragrunt/releases/tag/v0.24.2-alpha.1 to make it easy to get the binary.

Just opened a PR with a potential fix. Can you see if that works for you? I released this as https://github.com/gruntwork-io/terragrunt/releases/tag/v0.24.2-alpha.1 to make it easy to get the binary.

It seems to be working perfectly.
I've tested with:

the system-wide terraform version for tfenv unset
the system-wide terraform version for tfenv set to a version that's incompatible with the current version of terragrunt (0.12.26)

Before each test I made sure to remove all .terragrunt-cache folders with:
find . -type d -name ".terragrunt-cache" -prune -exec rm -rf {} \;

Both tests were successful.
I then re-run both tests without removing the .terragrunt-cache folders, just in case, to test both cases.
Both tests were successful again.
The temporary directories, containing only the backend.tf file, get created (one each) in the .terragrunt-cache folder of each dependency for which terragrunt runs terraform output -json and it almost immediately gets removed, leaving behind an empty .terragrunt-cache folder.
LGTM

And sorry for not saying this earlier, but WOW, thank you very much for picking this up so quickly!
Massive-👍 !

Thanks for testing! This is released as  https://github.com/gruntwork-io/terragrunt/releases/tag/v0.24.3 (binaries should show up shortly, ~10 minutes).


[Pager] Horizontal Pager with list of images doesn't work as intended

Description
Currently, I am using the HorizontalPager to load a list of images. In the latest version of accompanist (0.24.12-rc), I read that the page is considered to be the next page when the swipe distance is equal to or greater than half of the screen width . I am using  AsyncImage() from the Coil library to load the image. When the user swipes the images quickly, everything works fine but the issue occurs when the swipe is slow. During a slow swipe, it can be seen that the image is getting changed halfway which appears janky as shown in the attached video.
Steps to reproduce
Just load the list of images in the HorizontalPager using the AsyncImage() from the Coil library. I am attaching the code that I am using to load the images.
HorizontalPager(
                count = args.images.size,
                itemSpacing = 16.dp
            ) {
                AsyncImage(
                    modifier = modifier,
                    model = images[this.currentPage].uri
                )
            }

Expected behavior
The swipe should feel natural without abruptly loading the next image.
https://user-images.githubusercontent.com/13131390/176727207-1536e7c0-bf8f-4834-8d83-074576068876.mp4

Pager can compose multiple pages at one, there is an int parameter in the HorizontalPager lambda which is a page number. You need to use it instead of the same current page on all composed pages

I am sorry but I can't find it. I also checked the documentation and the sample app of Accompanist and can't find its usage. It would be very helpful if you can share the code snippet or the link to the page if available.

Instead of
            HorizontalPager(                 count = ...             ) {                 AsyncImage(                     model = images[this.currentPage].uri                 )             }
do
            HorizontalPager(                 count = ...             ) { page ->                 AsyncImage(                     model = images[page].uri                 )             }

Thank you. Works like charm.


Feature Request: Possibility of a standard app?

Ok, pressing control+shift+g does also the job, but being able to set GPMDP as a standard app for google links would be nice (if its possible at all). According to a friend its possible with the spotify desktop app.

So you're saying we should register a link handler for https://play.google.com/music/listen ?

yeah, but i dont know if (atleast in windows) it triggers uac (probably?)

oh well, about the spotify thing:
IIUC if you set in the settings that you want the desktop app to open and you open a normal http link, it will redirect to spotify://, and spotify:// links will open the app automatically, so they made their own uri scheme which makes it possible to set a standard app.
so yeah, doubt that google will implement that one.
One thing that would be possible: when you press the share button that it shows to links: one for the browser and one for the web. (http and gpmdp maybe). but then there will be a problem if you try to open the gpmdp link in the desktop (i guess).
if we had access to the google url we could then setup a webserver that redirects it to https if the app isnt installed. and that again wont work.
so yeah, control shift g will be the go-to option

I don't think this is possible currently with our current installation method.
https://github.com/electron/electron/issues/2795
We would need to register a protocol handler in the windows registry, which is probably a little bit of security concern for most.
Also it looks like spotify registers a URI: spotify: which google doesn't have anywhere in the web interface so we'd have to have browser extension to convert links on all pages over to the gpmdp: URI. All in all this would be a nice feature to have be would be a lot of trouble to get working.

Ah, and you beat me to the punchline.

@jostrander Making a protocol handler = easy and not much of a security concern.  Electron actually has an API for it 👍
But you are correct in saying it would require a companion chrome extension of some kind, which is beyond the scope of this app 👍

As I understood that issue I posted, it seemed like the protocol handler was only internal to the app, not system wide.

@jostrander
Electron has an internal protocol module and a few protocolClient API's on the app module.
See https://github.com/electron/electron/blob/master/docs/api/app.md#appsetasdefaultprotocolclientprotocol-path-args-macos-windows


XMLException when parsing svg's on Android with DTD definition

I host my resources in a separate project <XXX.Resources>. My resources consists of svg images with build action EmbeddedResource. If I build my app without linking all images show as expected. If I switch to SdkOnly none of my SVG's show in the app. Reading through the log it seems like it's related to my SVG's having a DTD attribute. What I don't understand is why this only happens when LinkSDKOnly is enabled.
I've attached the exception log:
Image loading failed: resource://XXX.Resources.calendar-text.svg?assembly=XXX.Resources,%20Version=1.0.6464.17689,%20Culture=neutral,%20PublicKeyToken=null;(size=40x40,dip=True);70x0;TintTransformation,R=21,G=105,B=127,A=255,HexColor=#FF15697F,EnableSolidColor=True
[Mono] [0x7fcba920] worker parking
[Mono] [0x7e07f920] worker unparking, timeout? no interrupted? no
System.Xml.XmlException: For security reasons DTD is prohibited in this XML document. To enable DTD processing set the DtdProcessing property on XmlReaderSettings to Parse and pass the settings into XmlReader.Create method.
at System.Xml.XmlTextReaderImpl.Throw (System.Exception e) [0x00027] in :0
[Mono] [0x7e07f920] worker parking
at System.Xml.XmlTextReaderImpl.ThrowWithoutLineInfo (System.String res) [0x00017] in :0
at System.Xml.XmlTextReaderImpl.ParseDoctypeDecl () [0x00008] in :0
at System.Xml.XmlTextReaderImpl.ParseDocumentContent () [0x0017a] in :0
at System.Xml.XmlTextReaderImpl.Read () [0x0008c] in :0
at System.Xml.Linq.XDeclaration..ctor (System.Xml.XmlReader r) [0x00039] in <9c5880fa89944acdbefbabdf783002b0>:0
at System.Xml.Linq.XDocument.Load (System.Xml.XmlReader reader, System.Xml.Linq.LoadOptions options) [0x00074] in <9c5880fa89944acdbefbabdf783002b0>:0
at System.Xml.Linq.XDocument.Load (System.Xml.XmlReader reader) [0x00000] in <9c5880fa89944acdbefbabdf783002b0>:0
at FFImageLoading.Svg.Platform.SKSvg.Load (System.Xml.XmlReader reader) [0x00000] in C:\projects\ffimageloading\source\FFImageLoading.Svg.Shared\SkSvg.cs:95
at FFImageLoading.Svg.Platform.SKSvg.Load (System.IO.Stream stream) [0x0000d] in C:\projects\ffimageloading\source\FFImageLoading.Svg.Shared\SkSvg.cs:89
at FFImageLoading.Svg.Platform.SvgDataResolver+d__19.MoveNext () [0x001b1] in C:\projects\ffimageloading\source\FFImageLoading.Svg.Shared\SvgDataResolver.cs:88

Hi @modplug,
I have no clue why it tries to load the DTD, IMO this could be avoided in SkiaSharp, but that's rather a SkiaSharp issue.
I think it would be better to post the bug in SkiaSharp github, then you reference this one.


Can I choose the name of the artifact to upload the flatpak as?

I don't think there is an option to set the name of the artifact to upload.

see the bundle input https://github.com/flatpak/flatpak-github-actions?tab=readme-ov-file#inputs

I should have been more specific. If you are talking about bundle: then it's not quite right. It appends the arch like x86_64 to the name given there. The user doesn't have full control of the name.