Fix a small bug

This change fixes a problem such that if namespace has other annotations, the TenantAdminNamespaceAnnotation will not be added.

/assign @rjbez17


DOC: Add pyprep [skip travis] [skip azp]

Closes #7049

@drammock feel free to merge if you're happy with this
https://16809-1301584-gh.circle-artifacts.com/0/dev/install/pre_install.html


chore(deps): bump deno

Codecov Report

Merging #67 (6cc887a) into master (bc09d0d) will decrease coverage by 0.05%.
The diff coverage is 96.66%.

:exclamation: Current head 6cc887a differs from pull request most recent head 8891f73. Consider uploading reports for the commit 8891f73 to get more accurate results

@@            Coverage Diff             @@
##           master      #67      +/-   ##
==========================================
- Coverage   91.12%   91.06%   -0.06%
==========================================
  Files          56       56
  Lines        2963     2966       +3
  Branches       33       33
==========================================
+ Hits         2700     2701       +1
- Misses        263      265       +2

Impacted Files
Coverage Δ

crates/deno/src/runtime.rs
90.86% <83.33%> (-0.85%)
:arrow_down:

crates/deno/script/deps/error.ts
95.45% <100.00%> (-0.07%)
:arrow_down:

crates/deno/script/deps/resource.ts
100.00% <100.00%> (ø)

crates/deno/script/deps/rust_chapter_task.ts
100.00% <100.00%> (ø)

crates/deno/script/deps/rust_http.ts
95.74% <100.00%> (ø)

crates/deno/script/deps/rust_module.ts
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Various errors

Hi, any advice on how I can fix the below errors. I am running Kali Linux as my OS. I have already set REDDIT_CLIENT_ID and REDDIT_CLIENT_SECRET using the export command in the terminal and confirming with echo.
┌──(kali㉿kali)-[~/Desktop/auddit]
└─$ python3 src/main.py
Current Task: get_hottest_post
Current Task: tts
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
Current Task: generate_video
Traceback (most recent call last):
  File "/home/kali/.local/lib/python3.9/site-packages/moviepy/video/VideoClip.py", line 1137, in __init__
    subprocess_call(cmd, logger=None)
  File "/home/kali/.local/lib/python3.9/site-packages/moviepy/tools.py", line 54, in subprocess_call
    raise IOError(err.decode('utf8'))
OSError: convert-im6.q16: attempt to perform an operation not allowed by the security policy<EMAIL_ADDRESS>@ error/property.c/InterpretImageProperties/3706.
convert-im6.q16: no images defined `PNG32:/tmp/tmp8rngpnz4.png' @ error/convert.c/ConvertImageCommand/3229.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kali/Desktop/auddit/src/main.py", line 28, in <module>
    pipeline.execute(subreddit='askreddit', nsfw=False, comment_limit=20)
  File "/home/kali/Desktop/auddit/src/main.py", line 24, in execute
    task(self.context)
  File "/home/kali/Desktop/auddit/src/tasks/generate_video/task.py", line 62, in generate_video
    clips.append(generate_title(post.title, post.title_audio))
  File "/home/kali/Desktop/auddit/src/tasks/generate_video/task.py", line 20, in generate_title
    txt_clip = TextClip(wrapped_text,fontsize=font_size, font=FONT, color=TITLE_FONT_COLOR, align="west")
  File "/home/kali/.local/lib/python3.9/site-packages/moviepy/video/VideoClip.py", line 1146, in __init__
    raise IOError(error)
OSError: MoviePy Error: creation of None failed because of the following error:

convert-im6.q16: attempt to perform an operation not allowed by the security policy<EMAIL_ADDRESS>@ error/property.c/InterpretImageProperties/3706.
convert-im6.q16: no images defined `PNG32:/tmp/tmp8rngpnz4.png' @ error/convert.c/ConvertImageCommand/3229.
.

.This error can be due to the fact that ImageMagick is not installed on your computer, or (for Windows users) that you didn't specify the path to the ImageMagick binary in file conf.py, or that the path you specified is incorrect

┌──(kali㉿kali)-[~/Desktop/auddit]
└─$

(I have no idea why it keeps saying ImageMagick is not installed as it definitely is!)
Any help is greatly appreciated.

----------------------------------------------------------------------
Ran 1 test in 4.751s

OK

It seems to be imagemagick configurations in certain environments that prevent the conversion, check the response to this to see if it helps https://stackoverflow.com/questions/52998331/imagemagick-security-policy-pdf-blocking-conversion

Hi, thanks for your prompt reply. I'm not too familiar with imagemagick or even linux so I couldn't figure out what bit to change based on that thread. So I ended up just deleting policy.xml and replacing it with a blank file.
The good news is that it ran main.py file and generated a video (5mb file). When I play the video the audio seems to be stitched together fine but the screen does not change beyond the opening title. I'm guessing this is still an issue with imagemagick or is there something else I could try?
Thanks

┌──(kali㉿kali)-[~/Desktop/auddit]
└─$ python3 src/main.py
Current Task: get_hottest_post
Current Task: tts
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
TTS Rate limit reached - Fallback on Google text-to-speech
Current Task: generate_video
Moviepy - Building video data/video/f1e6e51f-6ed6-4ec9-85fd-89061906068e.mp4.
MoviePy - Writing audio in f1e6e51f-6ed6-4ec9-85fd-89061906068eTEMP_MPY_wvf_snd.mp3
MoviePy - Done.
Moviepy - Writing video data/video/f1e6e51f-6ed6-4ec9-85fd-89061906068e.mp4

Hmm, i assume imagemagick didn't like the config being deleted, you can always generating a video from the documentation to see if that works, if it doesn't it probably confirms that imagemagick is the problem: https://zulko.github.io/moviepy/examples/moving_letters.html

Is there any chance you could share I copy of your policy.xml file so I could compare? Prob located in /etc/ImageMagick-6/
Many thanks

Maybe it's possible for comments to have no authors, not sure how that would be possible. I am using ImageMagick-7 with following policy:
<!DOCTYPE policymap [
  <!ELEMENT policymap (policy)*>
  <!ATTLIST policymap xmlns CDATA #FIXED ''>
  <!ELEMENT policy EMPTY>
  <!ATTLIST policy xmlns CDATA #FIXED '' domain NMTOKEN #REQUIRED
    name NMTOKEN #IMPLIED pattern CDATA #IMPLIED rights NMTOKEN #IMPLIED
    stealth NMTOKEN #IMPLIED value CDATA #IMPLIED>
]>
<!--
  Configure ImageMagick policies.

  Domains include system, delegate, coder, filter, path, or resource.

  Rights include none, read, write, execute and all.  Use | to combine them,
  for example: "read | write" to permit read from, or write to, a path.

  Use a glob expression as a pattern.

  Suppose we do not want users to process MPEG video images:

    <policy domain="delegate" rights="none" pattern="mpeg:decode" />

  Here we do not want users reading images from HTTP:

    <policy domain="coder" rights="none" pattern="HTTP" />

  The /repository file system is restricted to read only.  We use a glob
  expression to match all paths that start with /repository:

    <policy domain="path" rights="read" pattern="/repository/*" />

  Lets prevent users from executing any image filters:

    <policy domain="filter" rights="none" pattern="*" />

  Any large image is cached to disk rather than memory:

    <policy domain="resource" name="area" value="1GP"/>

  Use the default system font unless overwridden by the application:

    <policy domain="system" name="font" value="/usr/share/fonts/favorite.ttf"/>

  Define arguments for the memory, map, area, width, height and disk resources
  with SI prefixes (.e.g 100MB).  In addition, resource policies are maximums
  for each instance of ImageMagick (e.g. policy memory limit 1GB, -limit 2GB
  exceeds policy maximum so memory limit is 1GB).

  Rules are processed in order.  Here we want to restrict ImageMagick to only
  read or write a small subset of proven web-safe image types:

    <policy domain="delegate" rights="none" pattern="*" />
    <policy domain="filter" rights="none" pattern="*" />
    <policy domain="coder" rights="none" pattern="*" />
    <policy domain="coder" rights="read|write" pattern="{GIF,JPEG,PNG,WEBP}" />
-->
<policymap>
  <!-- <policy domain="resource" name="temporary-path" value="/tmp"/> -->
  <!-- <policy domain="resource" name="memory" value="2GiB"/> -->
  <!-- <policy domain="resource" name="map" value="4GiB"/> -->
  <!-- <policy domain="resource" name="width" value="10KP"/> -->
  <!-- <policy domain="resource" name="height" value="10KP"/> -->
  <!-- <policy domain="resource" name="list-length" value="128"/> -->
  <!-- <policy domain="resource" name="area" value="100MP"/> -->
  <!-- <policy domain="resource" name="disk" value="16EiB"/> -->
  <!-- <policy domain="resource" name="file" value="768"/> -->
  <!-- <policy domain="resource" name="thread" value="4"/> -->
  <!-- <policy domain="resource" name="throttle" value="0"/> -->
  <!-- <policy domain="resource" name="time" value="3600"/> -->
  <!-- <policy domain="coder" rights="none" pattern="MVG" /> -->
  <!-- <policy domain="module" rights="none" pattern="{PS,PDF,XPS}" /> -->
  <!-- <policy domain="delegate" rights="none" pattern="HTTPS" /> -->
  <!-- <policy domain="path" rights="none" pattern="@*" /> -->
  <!-- <policy domain="cache" name="memory-map" value="anonymous"/> -->
  <!-- <policy domain="cache" name="synchronize" value="True"/> -->
  <!-- <policy domain="cache" name="shared-secret" value="passphrase" stealth="true"/> -->
  <!-- <policy domain="system" name="max-memory-request" value="256MiB"/> -->
  <!-- <policy domain="system" name="shred" value="2"/> -->
  <!-- <policy domain="system" name="precision" value="6"/> -->
  <!-- <policy domain="system" name="font" value="/path/to/unicode-font.ttf"/> -->
  <policy domain="delegate" rights="none" pattern="gs" />
</policymap>

Thanks for sharing your file. I got it to work - yay! I removed the auto-upload and clean bit in the script because I wanted to manually deal with those parts. Someone may have already asked this but is it possible to direct the script to a particular post rather than automatically finding the most popular? Many thanks and awesome work!

Yup, check out the docs for PRAW (reddit api client) https://praw.readthedocs.io/en/stable/tutorials/comments.html


WARNING: Could not parse n transform function.  Please report this issue with the "1728610743569-base.js" file on https://github.com/distubejs/ytdl-core/issues.

Describe the bug

1728610743569-base.zip
Environment

@distube/ytdl-core version: 4.14.4
Node.js version: 21.7.3
Operating system: Linux

Fixed
Click here

Same for me

Same for me

my friend has fixed it, look here
https://github.com/XM4ZE/ytdl-core

Thanks worked for me!!!

Thanks worked for me!!!

okeyy

same for me. what is the fix?

same for me. what is the fix?

this
https://github.com/XM4ZE/ytdl-core/commit/83072a83aa6377313668900a89abe25d93b73d4b

but this is just a warning. for me the dld doesnt work any more. without any error. just times out. is this happening for you too ?

mine fixed by using
"dependencies": {   "@distube/ytdl-core": "github:XM4ZE/ytdl-core" }

this works for me too. let's hope it will be fixed in the main repo also

If you find the same thing, then try doing this
change package.json from "@distube/ytdl-core": "^4.14.4" into "@distube/ytdl-core": "github:XM4ZE/ytdl-core"

this is not a fix, - it's a workaround, sort of. you're actually removing this library in favor of your fork.

yea, the important thing is that it can work rather than not at all

But we should keep the issue open. It is still a bug in this repo

Hmm this seems a bit sketchy but it does work... I will probably change it back to distube's fork once it gets updated

The fix worked for me as well

I thought the last update of this repo would fix the issue, but it didn't for me. And the fix XM4ZE is no longer available!
Is there a way we can get it back? Cheers! @ryukenshiroo

Cheers
he just deleted it without giving any reason :)

Try this.
https://github.com/ybd-project-ver1/ytdl-core


feat: create-facebook-oauth

Facebook login
Endpoint

URL: /api/v1/auth/facebook
Method: POST
Content-Type: application/json

Description
The Facebook login feature to enhance user engagement and onboarding experience. This feature should include user registration, account creation, and authentication processes using facebook social oauth service for facebook account owners.
* Using Postman
i. Open Postman.
ii. Create a new POST request.
iii. Enter the endpoint URL: http://localhost:8080/api/v1/auth/facebook.
iv. Set the request body type to raw and select JSON from the dropdown.
v. Paste the following JSON into the body:
{
  "access_token": "string",
  "expires_in": "integer",
  "refresh_token": "string",
  "scope": "string",
  "token_type": "string",
  "id_token": "string",
  "expires_at": "long",
  "provider": "string",
  "type": "string",
  "provider_account_id": "string",
  "verified_user_id": "string"
}

vi. Click on the Send button.
Responses
Success Response

Status Code: 200 OK
Content-Type: application/json
Response Body:

{
  "status_code": 200,
  "message": "Registration Successful!",
  "data": {
    "token": "some-token",
    "user": {
      "id": "some-user-id",
      "first_name": "John",
      "last_name": "Doe",
      "email"<EMAIL_ADDRESS>      "role": "USER",
      "img_url": "http://example.com/picture",
      "created_at": "2024-07-19T20:28:34.399829"
    }
  }
}

Ensure your server is running on localhost:8080 before making the request.
Ensure you have registered on Facebook Developers Console account gotten your credentials to make use for testing the endpoint on postman
Check your database to verify the user has been created successfully after a successful registration.

@Fifetoyi  please review my code

@Amodu @lurldgbodex please check this out

@Fifetoyi , conflict is resolved


No support for appium MobileBy locator types. Cannot find mobile elements by accessibility id.

SeLion Version
2.0.0-SNAPSHOT
Component
SeLion & SeLion-appium-provider
Expected Behavior
SeLion platform WebElements to be located via appium MobileBy types.
Actual Behavior
SeLion platform WebElements cannot be located by appium MobileBy types.
Steps to Reproduce
The bridge drivers allows findElementBy(By by) override, however limitation still exists on the set of By subclasses that can be passed in. HtmlElementUtils.resolveByType(String locator) cannot be overridden.
https://github.com/paypal/SeLion/blob/develop/client/src/main/java/com/paypal/selion/platform/mobile/ios/UIAElement.java#L67
https://github.com/paypal/SeLion/blob/develop/client/src/main/java/com/paypal/selion/platform/mobile/android/UiObject.java#L50
Proposal:

Add By resolveByType(String locator); method definition in SeLionAndroidBridgeDriver and SeLionIOSBridgeDriver

Appium ios/android providers can resolve MobileBy types in addition to using existing HtmlElementUtils.

Other providers can continue leveraging HtmlElementUtils.

@elhuang Thanks for filing this. I love it when an issue is submitted with a proposal -- even more when it is followed up with a PR. :)


Proposal to squash commits on merge

Description
This is what the commit/merge graph of the project currently looks like

To keep a good/neat commit history, I propose that commits are squashed when merging pull requests. By bundling commits in a pull request into one, it makes it easy to reference in future even.
This is what a graph with squashed commits looks like,

You can read up more about squashing here
Screenshots
No response

yes it looks cleaner


Use ember-cli-rx instead of rx-ember

Bower is painful to deal with. The only reason we are using bower right now is because of an old issue with previous versions of Ember-CLI not allowing addons to use addons. That's fixed, so we should just be using ember-cli-rx.

:+1:


TypeError: (void 0) when trying to run example.

I've been trying to evaluate workers for an upcoming project, and I'm struggling. I used this example, replaced all the config variables, yet can't seem to get around the following error. Can't quite figure out how to get "prettified" JS to work in preview console, or to build it with non-minified. Was hoping the example would work out of the box, but wrangler generate, add the secrets (confirmed they are correct and available) leads to this error.
Uncaught (in promise)
TypeError: (void 0) is not a function
at worker.js:76:207404
at Array.reduce ()
at a (worker.js:76:207352)
at Object.resolve (worker.js:76:209116)
at t.resolveMiddleware (worker.js:76:251451)
at t.e.send (worker.js:76:209572)
at worker.js:31:35587
at worker.js:31:35653
at worker.js:31:35759
Uncaught (in response)
TypeError: (void 0) is not a function

Worth noting: focused on testing the Dynamo DB example. All calls to client.send() fail with this error. I've tried copying/working with AWS documentation examples that also fail using their code.

@JakeLingwall I ran into this issue as well...and then it went away. I am not 100% sure what fixed, but try deleting your node_modules folder and the running npm install again. Sorry I can't be more specific/helpful.


Non-nullables assign null in SelectorExpression

Thanks for the project -- this is turning out to be quite a time-saver! An issue, though: please check the nullability of an assignment in the SelectorExpression. I'm getting things like
OwnerId = p.Owner != null ? p.Owner.Id : null
when OwnerId is a straight int, not an int?. To simplify, the ternary could output
OwnerId = p.Owner != null ? p.Owner.Id : default(TYPE)
where TYPE is the target type (in this case, default(int)) and all would be excellent.
Thanks again.

Glad to hear you like the plugin!
I'll fix this issue in a day or two.

@msawczyn new version is online, please try it out.


Socially 2 app reloads when an exception happens - obscuring the exception

steps to repro:

follow http://angular-meteor.com/tutorials/angular2/adding-removing-objects-and-angular-event-handling
make a typo (e.g. instead of this.partiesForm.controls.name.updateValue(''); type this.partyForm.controls.name.updateValue('');
open the app and add a new item

expected result: the app breaks and exception is visible in the console (or better yet a compile time error is displayed in the meteor console)
actual result: after submitting the new item via the form the app reloads just after posting the data to the server, which means that after the reload the app now contains the new item.
This makes it look like the app works (albeit with a reload) and makes it hard to find the cause for this reload.

Meteor-Angular2 needs an improved typescript compiler.
Currently, sclausen:angular2-typescript is a better fit, but only runs on the client and throws errors for any server side .ts files. I've posted this question to the Typescript community to try to address this problem.

Fixed thanks to the great work of @barbatus who created a new Typescript package for Meteor , updated the Angular2.0-Meteor package and the tutorial.


Add auto-publish functionality for Function resources

Fixes https://github.com/aws-controllers-k8s/community/issues/2041
This patch introduces the ability for CF Functions to be automatically
published after a successful create or update operation. It adds an
annotation cloudfront.services.k8s.aws/auto-publish to the Function
resource, which when set to true, will trigger the automatic
publishing of the Function after a create or update.
Signed-off-by: Amine Hilaly<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest


Overlap estimator bug

This line sometimes throws a key error:
https://github.com/QunaSys/quri-parts/blob/407ae825f995c7db9afd0c2983c1932407ea4f29/packages/core/quri_parts/core/estimator/sampling/overlap_estimator.py#L37
This one too
https://github.com/QunaSys/quri-parts/blob/407ae825f995c7db9afd0c2983c1932407ea4f29/packages/core/quri_parts/core/estimator/sampling/overlap_estimator.py#L42
It seems kind of obvious in hindsight, but the self._sampling_counts.keys() should be checked before requesting a specific count.

Replacing [0] with .get(0, 0) may be the easiest way to fix.


Can't use GraphiQL when SSL is enabled with a configurable variable for port

Description:
$title.
Steps to reproduce:

when I give the configurable variable directly and pass the secureScoket argument directly, I get a compile-time error.

configurable int port = 9000;

listener graphql:Listener ln = new (port,
        secureSocket = {
            key: {
                certFile: "../resources/certs/public.crt",
                keyFile: "../resources/certs/private.key"
            }
        });

ERROR [service.bal:(41:9,46:10)] http:Listener and graphql:ListenerConfiguration are mutually exclusive

when I use an http:Listener to avoid this, GraphiQL doesn't work (errors like NetworkError when attempting to fetch resource., Failed to fetch)

listener graphql:Listener ln = new (check new http:Listener(port,
        httpVersion = http:HTTP_1_1,
        secureSocket = {
            key: {
                certFile: "../resources/certs/public.crt",
                keyFile: "../resources/certs/private.key"
            }
        }));

Note:

works when using a Ballerina GraphQL client (e.g., tests work)
the log that is printed for GraphiQL is wrong - http instead of https - GraphiQL client ready at http://localhost:9000/graphiql

Affected Versions:
2201.9.0-rc2

This seems like an issue when validating the port in the compiler plugin. It checks whether the first argument is a numeric literal. If that's not the case, the compiler plugin throws an error assuming it is an HTTP listener. But in this case, the port is passed as a variable.
We need to update this check to correctly validate the first argument.


[INLONG-9541][ASF] Remove branch protection strategy for the master branch

Prepare a Pull Request
(Change the title refer to the following example)

Title Example: [INLONG-XYZ][Component] Title of the pull request

(The following XYZ should be replaced by the actual GitHub Issue number)

Fixes #9541

Motivation
To fix some wrong commits.
Modifications
Describe the modifications you've done.
Verifying this change
(Please pick either of the following options)

[ ] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a follow-up issue for adding the documentation

The protection of the master branch is important, and should not be removed.


Unable to run script

Hey! I tried running this script, but I got the following errors:

TypeError: Failed to set the 'innerHTML' property on 'Element': This document requires 'TrustedHTML' assignment.

Uncaught TypeError: window.fixAlbum is not a function
I was following the steps as listed. Am I missing something? I'm getting the same errors both on chrome and edge.
I'm guessing that the "window.fixAlbum = " part is causing the error, but I don't know how it could be fixed.

hey, i do not think you missed something. project is not maintained since 5+ years and looks like chrome got some restrictions in the meantime on accessing elements causing the reported error.
sadly, google photos as a platform while well integrated with android devices, feels unfinished, inconsistent and buggy in so many places that is not a reliable place to keep photos if someone takes it seriously


Windows: Error: failed to create file caches from configuration: file exists

See https://discourse.gohugo.io/t/error-failed-to-create-file-caches-from-configuration-file-exists/15635/6

Most likely this issue:
https://github.com/spf13/afero/issues/189
I will also create an issue in Go's issue tracker, as I think the Windows implementation of os.IsExist should potentially also consider syscall.EEXIST even if Windows never uses that error code.

https://github.com/golang/go/issues/29295


Add Timestamps to Masternode Outputs command

This is what I have right now, obviously the info is from testnet, and I have a multisig watch only in my list cause I'm testing other things, but does this look okay to you?
masternode outputs timestamp
{
  "1522849356": "5dec5799cf01549d83154f2ceca1f24f01a5cfcf72a3416d7939ad1e942454d8 1 mtM18w5CpyfhyNaah9u2BLyU9Sy1UKoojQ",
  "1523546205": "3d493ef79aae36e2a9048a5ee5111482d907efebc5f717b1e3a4c876da1b0512 1 2N1sazFuCyAmhg6C7jPdYBZhGWUBCRNSRi1",
  "1533584714": "b36593b510d2f2935b8997ef729241edef4fc34eb4717a9eae25c2cb4f60c46e 1 mxPm5Qpmb5uXdfe2THQ9ojeMQi7Af2WcVY"
}

Not my issue but this looks fine to me. If the outputs were just ordered by timestamp, without even displaying the timestamp, that would work for me too.

so don't display the timestamp or address? keep it the same just sorted?  What will tell you that the list is sorted and different then before? or does that not even matter?

For me, having the dates and addresses would be nice but not essential. Just ordering the list by timestamp (and letting people know the list is sorted) would make it easy to know which is the newest output and that's likely the only one most people are interested in. Wouldn't matter if the ordering is ascending or descending but I'd prefer ascending. That way the newest is always at the end of the list and the closest line on the screen to where I'm going to paste in the address I just copied.

okay I'm sure I could do that, maybe I'll sort both lists, and remove the timestamp form the full list but keep the address.
Thanks for the input, I'll try and get this into <IP_ADDRESS>.

Got it done, it'll be released with <IP_ADDRESS>
masternode outputs is no in ascending order of the txid timestamp,  Options full well also add the address associated with the txid
I'll close this issue when <IP_ADDRESS> is released

This is now in <IP_ADDRESS>

Cool. Thank you.


structory_towers:ocean pillar spawned outside of the water/on land.

The screenshot shows part of the beach as Lukewarm Ocean. The rest of it is Grassy Beach.

https://clips.twitch.tv/AssiduousNiceSnakeKappaWealth-a4yy4VfefjHEZ2DR

imagine that actually moves and says "I'll allow it" lol

I liked to think that the Ocean was once there, but sea levels are declining due to climate change!


don't use repeatCount on layer animations in chains

when chaining animations - EADelayedAnimation takes care of repeating so don't use repeatCount

done


Kernel panic when exiting REPL

When exitting from one of the REPL (for example on nodeos-barebones layer) since they are executed as PID 1 the Linux kernel throw a Kernel panic. In that case, a better alternative if possible is to halt the (virtual) machine. This would maybe be done by listening to the exit event of the REPL or stdin or process.atexit and dispatch instead the halt.

Fixed in master due to nodeos-init, clossing.


After update flutter version in_app_purchase does not return price, only return "Free" word only for android below version 12. But ios working fine and return price properly.

After update flutter version in_app_purchase does not return price, only return "Free" word only for android below version 12. But ios working fine and return price properly.
In Android version 11, 10 and below android version only return "Free" when i am using data of ProductDetails and using price.
Price must be displayed here not free.
I am using
in_app_purchase: 3.0.8 (same issue happened)
When update to - in_app_purchase: 3.1.7 (same issue happened)
Till i am not getting any solution. It happens only in Android not ios device. Please anybody help.

When i am using "rawPrice" instead "price" displayed 0.0 price , price not displayed properly in below Android version 12.
I mean in Android 11, 10, 9 and below Android version. But Android version 12 and upper in_app_purchase is working fine. Price properly displayed. Please help me by reply. And help me to provide solutions.

@gitcoder-23
When you say that the package is returning only free data for Android version below 12, I don't see any changes related to the package itself that'll differentiate between Android 12 and below, so you may need to check from Android release notes perspective to see if there's any changes made from OS version level that could be affecting the behavior you are reporting.
Also, since this repo is used to track bugs and feature requests, asking for help with providing screenshots won't help much without providing details such as code sample, so provide us complete minimal reproducible code sample that shows the reported behavior.


Update GPU vs CPU benchmark

This PR updates the "GPU vs CPU benchmark" with latest changes in WGSL code sample.
@Kangz PTAL

@petele Please merge and push live.


linux: merge from master

To get copy right check and other updates.

Hi @jianchun, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Jianchun Xu). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@jianchun Copyright Check looks good now. :)

Copyright Check looks good now. :)

@dilijev  Thanks for help!


Compiler: raise on new on abstract class

Fixes #3835
This one is a weird one. Given:
abstract class Foo
end

class Bar < Foo
end

class Baz < Foo
end

If you write Bar.new || Baz.new, the compiler will unify the type to "Foo or any of its subclasses" (Foo+). The same happens when you do Bar || Baz, it will be Foo.class+.
You can do:
(Bar || Baz).new

and everything's fine. But you could also do:
(Foo || Baz).new

and that would somehow create a Foo instance, even though Foo is abstract! The compiler has no way to know that when you end up with Foo.class+, the real types might be abstract types.
To solve this, we allow to invoke new but when you try to do it with an abstract class it will raise at runtime. This is not type-safe, but if we disallow this then we couldn't implement, for example, a lookup table:
lookup = {"Bar" => Bar, "Baz" => Baz} # gets the type Hash(String, Foo.class+)
lookup["Bar"].new

I don't like having a compiler-generated runtime error. But I suppose there is no other way to solve this unless we had a way to differentiate Foo+.
Do we have any other incident where the compiler injects a raise call? This is one of these examples where the compiler is tightly coupled with the stdlib and expects certain methods to be available at runtime.

Yes, as does raise at the codegen level.

I agree that this is a bit rushed, plus it doesn't work.


[PATCH] Omit sync package while mocking empty interface

when just an empty interface is has beeing mocked, the import of the sync package caused an error.
I have the terrible feeling, that this solution is not nice, but I don't know what I should change. Especially, because this bug might not occur. But I wanted to provide at least a test case and notify you, that this might happen.

Ou, the test failes, because of https://github.com/matryer/moq/pull/24

retest this please

Sorry, I committed with the -a option and forgot to add the new testpackage files.


[Migrated] binary_support logic in handler.py (0.51.0) broke compressed text response

Originally from: https://github.com/Miserlou/Zappa/issues/2080 by martinv13

Context
The binary_support setting used to allow compressing response at application level (for instance using flask_compress) in version 0.50.0. As of 0.51.0 it no longer works.
Expected Behavior

Compressed response using flask_compress should be possible.
Actual Behavior

In handler.py, response is forced through response.get_data(as_text=True), which fails for compressed payload, thus throwing an error. This is due to the modifications in #2029 which fixed a bug (previously all responses where base64 encoded), but introduced this one.
Possible Fix

A possibility would be to partially revert to the previous version and just change in handler.py the "or" for a "and" in the following condition: not response.mimetype.startswith("text/") or response.mimetype != "application/json"). I can propose a simple PR for this.
Steps to Reproduce
Configure Flask with flask_compress; any text or json response will fail with the following error: 'utf-8' codec can't decode byte 0x8b in position 1: invalid start byte
Your Environment

Zappa version used: 0.51.0
Operating System and Python version: Python 3.7 on Lambda
Your zappa_settings.json: relevant option: binary_support: true

marking as a duplicate of the following and closing.
https://github.com/zappa/Zappa/issues/908


Host header not rewritten on HTTPRoute

Bug description
Trying to set the host header in v1.10 using a header request fails to create the virtualservice.
Related issues:

kubeflow/kfserving#1643
#31844

Example:
  http:
  - match:
    - authority:
        regex: ^torchserve-transformer\.default(\.svc(\.cluster\.local)?)?(?::\d{1,5})?$
      gateways:
      - knative-local-gateway.knative-serving
    - authority:
        regex: ^torchserve-transformer\.default\.example\.com(?::\d{1,5})?$
      gateways:
      - knative-ingress-gateway.knative-serving
    route:
    - destination:
        host: knative-local-gateway.istio-system.svc.cluster.local
        port:
          number: 80
      headers:
        request:
          set:
            Host: torchserve-transformer-transformer-default.default.svc.cluster.local

returns:
2021-06-02T10:52:54.536044Z	info	validationServer	configuration is invalid: invalid header "Host": cannot set Host header

[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
The virtualservice should be created. In Istio 1.9 it works. We are not necessary injecting the proxy in kfserving.
Steps to reproduce the bug
Versions

istio control plane version: 1.10.0
istio data plane version: 1.10.0 (1 proxies)
kubernetes version: v1.19.11

How was Istio installed?
via istio-operator
Environment where the bug was observed (cloud vendor, OS, etc)
using kind
/cc @howardjohn

tracking future in https://github.com/istio/istio/issues/34181


Add typings repo url to typings info

When end user use the typings and discover an error, there is no easy way to discover there is the typings repo so that he can work on a PR.
Currently, he needs to:

Look at typings.json to see it is registry:dt/* or npm:* or registry:npm/* etc to determine is the typings come from: source, DT, or registry
If registry, then need to go to https://github.com/typings/registry to find the json file, and figure out the repo for the version (and there is no way to discover the version today yet) he is using.

Relies on https://github.com/typings/typings/issues/95, this is a feature I'd add when that exists.

Would love to see this feature!

So typings info is remote information, but there could be typings about pretty easily that just reads the typings.json cache file directly and prints it back out.

It would be good to copy npm as much as possible here. I really like that I
can type npm info <package> to get all data, and filter the JSON by
specifying an extra value, e.g. npm info lodash version.
On Sat, 13 Aug 2016 at 01:30 Blake Embrey<EMAIL_ADDRESS>wrote:

So typings info is remote information, but there could be typings about
pretty easily that just reads the typings.json cache file directly and
prints it back out.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/typings/typings/issues/391#issuecomment-239589776,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA4QCavwkcktWoN4BTd3ptaeJGj_Fp-wks5qfRAlgaJpZM4H-Wjo
.

I also really like npm repo which opens the repo URL in your browser.

Wow, I never know that!


[HADOOP-15261] Upgrade commons-io from 2.4 to 2.5

move commons-io up to 2.5 which introduced by kerb-simplekdc.

Related issue:  https://issues.apache.org/jira/browse/HADOOP-15261

Hi @PandaMonkey would you close this PR?


Remove deprecated watson package

The watson package has being replace by the three new packages

/whisk.system/watson-translator
/whisk.system/watson-textToSpeech
/whisk.system/watson-speechToText

Action Items:

[ ] Update installWatson.sh

[ ] Do not create package and actions
[ ] Check if package watson exist if it does then remove

[ ] Remove actions directory watson

Refer to #154 on why the new packages were created to replace the old one.

I suggest not to put the package check and removal inside installWatson.sh.
The install script should be focus on the installation, regardless of what is already installed or now.
Watson is the first package deprecated in openwhisk-catalog, but it is definitely not the last one.
We can have another script called uninstallDeprecation.sh to remove the deprecated actions and packages. The script should be able to run separately. @csantanapr What do you think?

Sounds better Vincent

Vincent let's use this new script to delete the "util" package if present. Now it should be the new "utils"

@csantanapr OK

Any other deprecated actions that need to be deleted? i.e "samples/echo" now new "utils/echo"

"util/cat" "util/date" "util/head" "util/sort" "util/split" "samples/echo"
These are all I know.

@houshengbo didn't you add a similar script before to openwhisk?
i like the idea.

@csantanapr That one only adds an annotation to the deprecated actions, does not delete them.

this issue is closed, since the pr is merged.


FlyBase:FBrf ?xrefs? should not be copied over as synonyms into SOLR

"short_form":"FBbt_00003748",
 "synonym":["FlyBase:FBrf0224194",
          "ME",
          "FlyBase:FBrf0212704",
          "medulla",
          "Med",
          "m",
          "FlyBase:FBrf0212889",
          "optic medulla"],

Don't think xrefs on synonyms belong here.
Maybe this is the relevant code?
https://github.com/VirtualFlyBrain/vfb-pipeline-dumps/blob/master/scripts/obographs-solr.py#L158

Or perhaps upstream in dumps from dump of input json to this script from triple store SPARQL queries?

So if you don't want xrefs of synonyms show up at all, is there a reason not to just remove it from the code as you  indicate?
if 'xrefs' in syn:
    se["synonym"].extend(syn['xrefs'])
    se["synonym_autosuggest"].extend(syn['xrefs'])

Don't see a use case for xrefs showing up in autocomplete, so yes, please remove.

@matentzn FYI this is a lower priority issue.
@dosumis do we not want xrefs for the SkIds, bodyIds, etc.?

@matentzn FYI this is a lower priority issue.

Isn't it important to ensure we're not indexing terms by the xrefs on their synonyms for autocomplete?   Happy to just do the edit myself.

@dosumis do we not want xrefs for the SkIds, bodyIds, etc.?

That should be a separate job - to pull IDs from xref edges to Site nodes.  @matentzn how would I specify that?  Add some sparql to the query that generated JSON input & then edit this dump script to add?

@dosumis lets make a new ticket for that; I am not sure top of my head; but yes, if you want to give it a shot; go to http://ts.p2.virtualflybrain.org/rdf4j-workbench/repositories/vfb/query play with a SELECT query until you find the required information, then add a construct query to the sparql directory as you previously did. I have generalised (and documented) the SPARQL construct pipeline so you will see what to do when you open the Makefile! Let me know if you need help!

This issue is done and checked by @Robbie1977 - can be closed.


[FEATURE] Show understandable errors

Is your feature request related to a problem? Please describe.
I'm working with Tron main net, and USDT (TRC-20). Pretty sure that this will apply to other cryptos as well.
Sometimes I need to do manual payouts.
Sometimes payouts "fail" - to know what went wrong, I need to go to the log, and see the error.
Would be really nice if the error would be shown right next to the payout, together with some alert appearing on the screen.
In some cases, when you specify the max fee amount and click send, there's no error in the logs, no errors on the screen, and the status doesn't change (also applies when there's not enough funds).
You are only supposed to guess that it didn't work.
Would be really good to add some error/warning information for those as well.
Describe the solution you'd like
Show errors in UI.

In case max fee setting is in place, it will indeed just halt and not proceed, without changing the status, that's how you know the fee is larger than max fee specified
As for the fail, well we do not store payout logs, instead it's just logged. You can also batch payouts, or select many of them, it might take some time to process and server logs is always there


Fix Babe revert when last finalized block is a leaf

This PR fix an error condition triggered when the last finalized block is equal to the best block.
Fail scenario can be easily reproduced by issuing a double revert from the CLI:

1st revert => to set the best block equal to last finalized
2nd revert => removes the weight of the last finalized block <=== BAD

This PR prevents removal of weight data of last finalized block

Some variables renaming for better maintenance...
The fix is actually only these lines here:
https://github.com/paritytech/substrate/blob/4b209fbed856e35b149ed452d57e9d3aff4253c4/client/consensus/babe/src/lib.rs#L1911-L1924

bot merge


AttributeError: 'Tensor' object has no attribute 'numpy'

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock
example script provided in TensorFlow):  yes
OS Platform and Distribution (e.g.,
Linux Ubuntu 16.04): Linux Fedora 31
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if
the issue happens on mobile device: no
TensorFlow installed from (source or
binary): binary (anaconda)
TensorFlow version (use command below): 2.1
Python version: 3.7
Bazel version (if compiling from source): no
GCC/Compiler version (if compiling from source): no
CUDA/cuDNN version: 10.1
GPU model and memory: nvidia 1050 ti

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
I can't call numpy() of Tensor arguments passed to the mapping function given to tf.data.Dataset even with tf.executing_eagerly() returning True:
AttributeError: 'Tensor' object has no attribute 'numpy'

Describe the expected behavior
I'd like to be able to access the numpy() property of arguments passed to a mapping function passed to tf.data.Dataset.map
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
import tensorflow as tf

def transformer(x):
    x.numpy()
    return x

dataset = tf.data.Dataset.from_tensors([0,1,2])
dataset.map(transformer)

Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@nscotto , You can convert tensor into numpy array using tensor.numpy(), But you can't do the same in case of MapDataset. In the example code you have given,  the x in the transformer function is MapDataset and not Tensor.
I have seen similar issue #30035 and also you can refer to this function which I think will solve your problem.
Also, if you have any doubts regarding from_tensors and from_tensor_slices, then please review the official docs here. Also, take a look at these example which I recently Added through my PR #36989.

@nscotto
Any update on this issue please. Thanks!

Yes, ashutosh1919 is right, it's because Dataset.map passes Tensor instead of EagerTensor, thus I need to wrap my code inside a tf.py_function if I need to convert to numpy (and obviously reducing the performances).
As a suggestion I think the documentation could mention this in order to make Tensorflow more accessible for newcomers.

@ashutosh1919  @ravikyram @nscotto
I am trying to address the problem by making a numpy() function in the Tensor class. But I am facing a problem. Can anyone tell me from which attribute of Tensor class I can find the data.

@ashutosh1919 I went through all the codes. From the codes it seems that internally the map() function iterates through all the tensors(for each function call). So what x sees from inside the function in map() is the single tensor passed to it at that very time. So using .numpy() inside the function called through map() is pointless.

@nscotto , @ravikyram and @ghosalsattam , I have raised PR #37853 to add an example on tf.numpy_function and how it is different from tf.py_function. Now, 2nd and 3rd point here about py_function and numpy_function clearly explains the difference.
Please give your suggestions if anything else is required.


Add instructions for using napari hub shield to FAQ

Add instructions for embedding shield into markdown (linking back to the hub) or just accessing the SVG.
Not sure if this is the best place to put this, but it made sense to me to put it under "Customizing your plugin's listing" - @neuromusic thoughts and/or concerns?
I'll also open a PR on the cookiecutter reop to add it to our README there, and once we release next week, advertise in the community meeting and on Zulip.

@neuromusic made those changes now (and fixed the tests)


[firebase_analytics]: Error: The pod required by the plugin requires a higher minimum macOS deployment version

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Analytics
Which platforms are affected?
macOS
Description
I cannot build my Flutter project for Macos after installing the firebase_analytics package.
Fails with

Error: The pod "Firebase/Analytics" required by the plugin "firebase_analytics" requires a higher minimum macOS deployment version than the plugin's reported minimum version.

Reproducing the issue
I reproduced the issue from a completely blank project in only 2 steps. Uploaded the project here
1. Flutter create project
Create a new project fba_demo
Flutter 3.24.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision dec2ee5c1f (11 days ago) • 2024-11-13 11:13:06 -0800
Engine • revision a18df97ca5
Tools • Dart 3.5.4 • DevTools 2.37.3
2. Flutter add firebase_analytics and firebase_core
Add the following to pubspec.yaml
dependencies:
  firebase_analytics: ^11.3.5
  firebase_core: ^3.8.0

3. Flutter run macos
Firebase Core version
3.8.0
Flutter Version
3.24.5
Relevant Log Output
$ flutter run -d macOS

Launching lib/main.dart on macOS in debug mode...
Running pod install...
CocoaPods' output:
↳
Preparing

    Analyzing dependencies

    Inspecting targets to integrate
      Using `ARCHS` setting to build architectures of target `Pods-Runner`: (``)
      Using `ARCHS` setting to build architectures of target `Pods-RunnerTests`: (``)

    Fetching external sources
    -> Fetching podspec for `FlutterMacOS` from `Flutter/ephemeral`
    -> Fetching podspec for `firebase_analytics` from `Flutter/ephemeral/.symlinks/plugins/firebase_analytics/macos`
    firebase_analytics: Using Firebase SDK version '11.4.0' defined in 'firebase_core'
    -> Fetching podspec for `firebase_core` from `Flutter/ephemeral/.symlinks/plugins/firebase_core/macos`
    firebase_core: Using Firebase SDK version '11.4.0' defined in 'firebase_core'

    Resolving dependencies of `Podfile`
      CDN: trunk Relative path: CocoaPods-version.yml exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: all_pods_versions_0_3_5.txt exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: Specs/0/3/5/Firebase/11.5.0/Firebase.podspec.json exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: Specs/0/3/5/Firebase/11.4.0/Firebase.podspec.json exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: Specs/0/3/5/Firebase/11.4.0/Firebase.podspec.json exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: Specs/0/3/5/Firebase/11.4.1/Firebase.podspec.json exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: Specs/0/3/5/Firebase/11.4.2/Firebase.podspec.json exists! Returning local because checking is only performed in repo update
      CDN: trunk Relative path: Specs/0/3/5/Firebase/11.4.0/Firebase.podspec.json exists! Returning local because checking is only performed in repo update
    [!] CocoaPods could not find compatible versions for pod "Firebase/Analytics":
      In Podfile:
        firebase_analytics (from `Flutter/ephemeral/.symlinks/plugins/firebase_analytics/macos`) was resolved to 11.3.5, which depends on
          Firebase/Analytics (= 11.4.0)

    Specs satisfying the `Firebase/Analytics (= 11.4.0)` dependency were found, but they required a higher minimum deployment target.

    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolution.rb:317:in `raise_error_unless_state'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolution.rb:299:in `block in unwind_for_conflict'
    <internal:kernel>:90:in `tap'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolution.rb:297:in `unwind_for_conflict'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolution.rb:682:in `attempt_to_activate'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolution.rb:254:in `process_topmost_state'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolution.rb:182:in `resolve'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/molinillo-0.8.0/lib/molinillo/resolver.rb:43:in `resolve'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/resolver.rb:94:in `resolve'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer/analyzer.rb:1082:in `block in resolve_dependencies'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/user_interface.rb:64:in `section'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer/analyzer.rb:1080:in `resolve_dependencies'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer/analyzer.rb:125:in `analyze'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer.rb:422:in `analyze'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer.rb:244:in `block in resolve_dependencies'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/user_interface.rb:64:in `section'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer.rb:243:in `resolve_dependencies'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/installer.rb:162:in `install!'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/command/install.rb:52:in `run'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/claide-1.1.0/lib/claide/command.rb:334:in `run'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/lib/cocoapods/command.rb:52:in `run'
    /Users/kaye375751/.rbenv/versions/3.2.2/lib/ruby/gems/3.2.0/gems/cocoapods-1.16.2/bin/pod:55:in `<top (required)>'
    /Users/kaye375751/.rbenv/versions/3.2.2/bin/pod:25:in `load'
    /Users/kaye375751/.rbenv/versions/3.2.2/bin/pod:25:in `<main>'

Error: The pod "Firebase/Analytics" required by the plugin "firebase_analytics" requires a higher minimum macOS deployment version than the plugin's reported minimum version.
To build, remove the plugin "firebase_analytics", or contact the plugin's developers for assistance.
Error: Error running pod install

Flutter dependencies

Expand Flutter dependencies snippet

Dart SDK 3.5.4
Flutter SDK 3.24.5
fba_demo 1.0.0+1

dependencies:
- cupertino_icons 1.0.8
- firebase_analytics 11.3.5 [firebase_analytics_platform_interface firebase_analytics_web firebase_core firebase_core_platform_interface flutter]
- firebase_core 3.8.0 [firebase_core_platform_interface firebase_core_web flutter meta]
- flutter 0.0.0 [characters collection material_color_utilities meta vector_math sky_engine]

dev dependencies:
- flutter_lints 4.0.0 [lints]
- flutter_test 0.0.0 [flutter test_api matcher path fake_async clock stack_trace vector_math leak_tracker_flutter_testing async boolean_selector characters collection leak_tracker leak_tracker_testing material_color_utilities meta source_span stream_channel string_scanner term_glyph vm_service]

transitive dependencies:
- _flutterfire_internals 1.3.46 [collection firebase_core firebase_core_platform_interface flutter meta]
- async 2.11.0 [collection meta]
- boolean_selector 2.1.1 [source_span string_scanner]
- characters 1.3.0
- clock 1.1.1
- collection 1.18.0
- fake_async 1.3.1 [clock collection]
- firebase_analytics_platform_interface 4.2.7 [_flutterfire_internals firebase_core flutter meta plugin_platform_interface]
- firebase_analytics_web 0.5.10+4 [_flutterfire_internals firebase_analytics_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins]
- firebase_core_platform_interface 5.3.0 [collection flutter flutter_test meta plugin_platform_interface]
- firebase_core_web 2.18.1 [firebase_core_platform_interface flutter flutter_web_plugins meta web]
- flutter_web_plugins 0.0.0 [flutter characters collection material_color_utilities meta vector_math]
- leak_tracker 10.0.5 [clock collection meta path vm_service]
- leak_tracker_flutter_testing 3.0.5 [flutter leak_tracker leak_tracker_testing matcher meta]
- leak_tracker_testing 3.0.1 [leak_tracker matcher meta]
- lints 4.0.0
- matcher 0.12.16+1 [async meta stack_trace term_glyph test_api]
- material_color_utilities 0.11.1 [collection]
- meta 1.15.0
- path 1.9.0
- plugin_platform_interface 2.1.8 [meta]
- sky_engine 0.0.99
- source_span 1.10.0 [collection path term_glyph]
- stack_trace 1.11.1 [path]
- stream_channel 2.1.2 [async]
- string_scanner 1.2.0 [source_span]
- term_glyph 1.2.1
- test_api 0.7.2 [async boolean_selector collection meta source_span stack_trace stream_channel string_scanner term_glyph]
- vector_math 2.1.4
- vm_service 14.2.5
- web 1.1.0

Unfortunately, the issue is still here.

user@user macos % pod install --repo-update
Updating local specs repositories
Analyzing dependencies
firebase_analytics: Using Firebase SDK version '11.4.0' defined in 'firebase_core'
firebase_core: Using Firebase SDK version '11.4.0' defined in 'firebase_core'
Run script to upload symbols already exists.
firebase_crashlytics: Using Firebase SDK version '11.4.0' defined in 'firebase_core'
firebase_messaging: Using Firebase SDK version '11.4.0' defined in 'firebase_core'
[!] CocoaPods could not find compatible versions for pod "Firebase/Analytics":
  In Podfile:
    firebase_analytics (from `Flutter/ephemeral/.symlinks/plugins/firebase_analytics/macos`) was resolved to 11.3.5, which depends on
      Firebase/Analytics (= 11.4.0)

Specs satisfying the `Firebase/Analytics (= 11.4.0)` dependency were found, but they required a higher minimum deployment target.

In your macOS directory, delete the Pods folder and Podfile.lock file. Then, add platform :osx, '10.15' at the top of your Podfile and try running the app again.

This helped: platform :osx, '10.15'.
Its works now. Thanks!

Confirmed, resolved by setting platform :osx, '10.15' in my macos Podfile


Settlement AI Building

Settlement's never get built even if computer player has enough resources to build one.

Settlements are now properly built when the user/AI has enough resources. Logic is correct within smart AI, should also be for dumb AI.


tdjson

What is the acquisition frequency and message sending frequency of tdjson？

It is up to server and unrelated to tdjson.

Thank you for your help.
How many milliseconds can I send or receive a new message?
0ms or 100ms? = =
@levlam

This is up to the server.

ok，Thank you for your help.


Decimal multiply kernel should not cause precision loss

Which issue does this PR close?

Closes #5674.
Rationale for this change

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

There is more broader issue on type coercion. I'm working on this and will update.

rust-decimal only supports precision <= 28...I need to update sqllogictest-rs.

More changes than I expected, but finally making DataFusion all tests happy...

I'll try and find time to review this PR tomorrow

cc @liukun4515

the coercion rule changes make sense to me. However, I don't undertand the need for Expr::PromotePrecision or the new data_type field on Expr. They don't seem to be related to improving the coercion for decimals

The coercion rule that modifies sides of arithmetic op is not idempotent. Multiple runs of the rule will change it to incorrect result. So we need something to prevent the rule on coerced sides. PromotePrecision is such a thing, it's just a wrapper for the purpose.
For the new data_type on BinaryExpr. The coerced type of decimal arithmetic op is not the same as the result type of it as you can see. So we cannot simply take coerced type of left/right sides and use it as result type. We cannot compute the result type on-the-fly in physical BinaryExec because it depends on original datatypes of sides of the op, but we only have coerced at the moment. So we need to record the result type so we can get it when computing the decimal arithmetic result.

The coercion rule that modifies sides of arithmetic op is not idempotent. Multiple runs of the rule will change it to incorrect result. So we need something to prevent the rule on coerced sides. PromotePrecision is such a thing, it's just a wrapper for the purpose.

For the new data_type on BinaryExpr. The coerced type of decimal arithmetic op is not the same as the result type of it as you can see. So we cannot simply take coerced type of left/right sides and use it as result type. We cannot compute the result type on-the-fly in physical BinaryExec because it depends on original datatypes of sides of the op, but we only have coerced at the moment. So we need to record the result type so we can get it when computing the decimal arithmetic result.

This makes sense  -- thank you
My biggest concern with the approach in this PR is that it adds some special case for decimal (which seems reasonable in itself) but that special case bleeds out over the rest of the code / plans, which is probably why it got so big.
I wonder would it be possible to use PromotePrecision only and avoid modifying Expr::Binary to keep the behavior more localized to coercion.
Perhaps something like the following.
/// The target decimal type the expression should be output to
struct PromotePrecision {
  expr: Box<Expr>,
  precision: u8,
  scale: u8,
}

One of the downsides of this approach is that PromotePrecision is now clearly special purpose for Decimal. I am not sure if it would have other uses in the future 🤔

Keeping precision/scale in PromotePrecision seems okay. I don't think PromotePrecision will be used for other uses. It is a specified wrapper (based on its name) for decimal arithmetic type coercion only.

I think then it would be best to keep the data type in Expr::PromotePrecision rather than adding it to BinaryExpr

cc @liukun4515

Thanks for your mention
I will take a look it carefully tomorrow.
@jackwener has moved the type coercion to the analysis phase.
Maybe we have some same comments about the rule of type coercion.

Yea, seems I need to rebase this.

Does this new  PromotePrecision expr only apply to Decimal type?
I remember in the SparkSQL there was a similar PromotePrecision, not sure whether it is for the same purpose.
And looks like in the latest Spark, it is removed or renamed.

Does this new PromotePrecision expr only apply to Decimal type?
I remember in the SparkSQL there was a similar PromotePrecision, not sure whether it is for the same purpose.
And looks like in the latest Spark, it is removed or renamed.

It is for Decimal type only. Yea, the idea is from Spark.
It was removed in latest Spark as the precision promotion is moved to where the computation happens. I was not sure if moving it into decimal kernels is a good idea for DataFusion. Let me re-think about it and maybe refactor this.

marking as draft while it gets worked on


Feature: add number of open prs to non-backfilled insight repos creation flow

Type of feature
🍕 Feature
Current behavior
In the insight page creation process, trying to add a repo we have not backfilled will return the information from GitHub REST API
As reported by @getaheaddev

Suggested solution
Switch from GitHub REST to GraphQL and attach open pull requests information to the repos
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

This is being handled in the API by an initial backfill on demand


test failure on Arch Linux: paint_servers_radialGradient_focal_point_correction

IDK how to make this reproducible.
I build resvg like this, and this one test fails: https://aur.archlinux.org/cgit/aur.git/tree/PKGBUILD?h=resvg
thread 'render::paint_servers_radialGradient_focal_point_correction' panicked at 'assertion failed: `(left == right)`
  left: `23404`,
 right: `0`', crates/resvg/tests/integration/render.rs:564:68
...
test result: FAILED. 1627 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 17.22s

Very strange.
Can you render this SVG manually and sent it to me?

They sure look identical …

mine
reference

What CPU do you have? Looks like a rounding bug.

AMD Ryzen 7 3700X, microcode 0x8701013

Very strange. I had the same CPU and it was fine. Not sure about microcode version. But I was testing on Windows.
Either way, you can ignore this test for now. Will see what can be done here.

I just encountered the same problem. My laptop was able to run the tests fine (i5-8250U). However, on my desktop (Ryzen 3900X) this test fails.
I have installed this version of the amd microcode:
local/amd-ucode 20230804.7be2766d-2
    Microcode update image for AMD CPUs


Add support for multipart OpenEXR images

Currently, tlRender supports only multiview images.  We need to add support for multipart images which load much faster.

Feature Added.


Export to PDF

http://pdfkit.org/

This may need to be a slightly separate script which reads files from the spec, and then processes that in a PDF.

http://www.marknagelberg.com/creating-pdf-reports-with-python-pdfkit-and-jinja2-templates/

Or export to LaTeX?

https://pypi.org/project/pdflatex/

Existing tools are too difficult to feasibly use, as they require installing a bunch of other tools and it's non-trivial to get to a working environment. I'll do a LaTeX export, then you can stick it in Overleaf or whatever.


CodeQL task in CI failing

Run<EMAIL_ADDRESS>/bin/tar xzC /home/runner/work/_temp/temp_1722150946 -f /home/runner/work/_temp/6f294039-24bb-4818-921b-320c0badfce9
Error: Unable to process command '##[set-env name=JAVA_HOME;]/opt/hostedtoolcache/jdk/11.0.9/x64' successfully.
Error: The `set-env` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
Error: Unable to process command '##[set-env name=JAVA_HOME_11.0.9_x64;]/opt/hostedtoolcache/jdk/11.0.9/x64' successfully.
Error: The `set-env` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
Error: Unable to process command '##[add-path]/opt/hostedtoolcache/jdk/11.0.9/x64/bin' successfully.
Error: The `add-path` command is disabled. Please upgrade to using Environment Files or opt into unsecure command execution by setting the `ACTIONS_ALLOW_UNSECURE_COMMANDS` environment variable to `true`. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/
creating settings.xml with server-id: github; environment variables: username=$GITHUB_ACTOR and ***

No benefit until kotlin is supported https://github.com/square/okhttp/pull/6304


how heavy is this edge detector

Hi @MengyangPu and everyone.
Can someone tell me how heavy is this model? (number of parameters)
Cheers,

Input shape: (3, 320, 320)
The params of Stage I: 359.03M
The params of Stage II: 109.81M


Pause/Start All Option

Hey @mayswind
I would like to see the pause all/ start all button which could pause resume all downloads at once.
I found it's need when the download server is using network bandwidth and I need to do some quick web surfing.
You could use following functions:
https://aria2.github.io/manual/en/html/aria2c.html?highlight=pause#aria2.pauseAll
https://aria2.github.io/manual/en/html/aria2c.html?highlight=pause#aria2.unpauseAll

You can choose select all and then choose start or pause.

It doesn't work the way I want if some files are currently downloading.


[Feature Request]: More Guidance for Placeholders

🚀 The feature
https://github.com/speechbrain/speechbrain/blob/develop/recipes/LibriSpeech/self-supervised-learning/wav2vec2/hparams/wav2vec2_base.yaml
isn't this model training based on only audio? I am not sure why is there a csv file request specification. Can you clarify, please?
I am confused about the placeholders - the speechbrain wrapper requirements are not obvious.
Solution outline
I would like to pretrain wav2vec with librispeech audio only.
May you provide more guidance on what to put on placeholders after line 70?
https://github.com/speechbrain/speechbrain/blob/develop/recipes/LibriSpeech/self-supervised-learning/wav2vec2/hparams/wav2vec2_base.yaml
for example, they do the same with wav2vec and librispeech and they give example values: https://github.com/huggingface/transformers/blob/main/examples/pytorch/speech-pretraining/README.md
or
https://github.com/facebookresearch/fairseq/blob/main/examples/wav2vec/config/pretraining/wav2vec2_base_librispeech.yaml
The possible range of values would be appreciated.
Additional context
No response

Please have a look at https://colab.research.google.com/drive/1aFgzrUv3udM_gNJNUoLaHIm78QHtxdIz?usp=sharing
"The goal of data preparation is to create the data manifest files. These files tell SpeechBrain where to find the audio data and their corresponding transcriptions. They are text files written in the popular CSV and JSON formats." this is the main purpose of the csv files. They are generated automatically when running the recipe. You should only have to specify the data_folder path.


Refreshed auth is ignored in SSR'd queries

Describe the bug
Hey folks,
I reported this behavior awhile back as a discussion thread, seeing I didn't have a reproduction at the time, but now I do, so I thought I'd open this as an official issue 😅.  I've been seeing this behavior for quite some time in my SvelteKit app.
I'm trying to preload some of my queries via SSR before my page loads. However, my auth is cookie-based and deals with refresh tokens, so I'd expect that any backend queries (that use .asPromise()) would not resolve until the auth has failed, been refreshed, and then succeeded.
Yet I'm noticing that while my access token cookie is refreshed, the query still produces an auth error and logs me out.
I've set up a reproduction in my own repo, as it's unfortunately a little involved:

Clone this repo and this branch.
In both the root directory and backend, run pnpm install (or whichever package manager you use).
In one terminal, inside backend, run pnpm start.
In another, in the root directory, run pnpm dev.
Open up http://localhost:5173 (should be the default).
For both the CSR and SSR login buttons, click them to log in, wait 15 seconds, then refresh the page.

For the CSR, you'll see that after 15 seconds, the auth expires, it makes a refresh request, then briefly shows undefined before showing the name john again as it refetches the data.
For the SSR, you'll see that after 15 seconds, it bounces you back to the login page. However, the server terminal will still log that the token has been refreshed, and the page's cookie will have updated. Yet URQL still throws the error that redirects you.

I've been agonizing over this for weeks 😭 I've dug through the source code, and can't seem to uncover anything that would be a direct cause. Sometimes, depending on the cache policy, the error behavior can vary. Any help or guidance here would be appreciated.
For now, I'm just making all my URQL queries client-side only.
Reproduction
https://github.com/Nickersoft/sveltekit-bugs/tree/nickersoft/urql-ssr
Urql version
@urql/core: "^3.0.5",
@urql/exchange-auth: "^1.0.0",
@urql/svelte: "^3.0.1",
Validations

[X] I can confirm that this is a bug report, and not a feature request, RFC, question, or discussion, for which GitHub Discussions should be used
[X] Read the docs.
[X] Follow our Code of Conduct

For simplicity's sake, I won't convert this back to a discussion ✌ But, to keep it short, this is a bug in your implementation.
Specifically, there are two things going on, both related to your cookie authentication.
When you SSR and the refresh is performed, the cookie isn't used in the subsequent request.
This is happening here: https://github.com/Nickersoft/sveltekit-bugs/blob/d4fee0f271c236be209bd6a91b73b334d60d0b02/src/hooks.server.ts#L15
You're parsing the SetCookie header from your backend's response, but the token isn't used in the subsequent API request that your SSR server is performing. So, a natural and quick fix would be:
if (request.url.includes('graphql')) {
-  request.headers.set('cookie', event.request.headers.get('cookie') ?? '');
+  const accessToken = event.cookies.get('access_token');
+  request.headers.set(
+    'cookie',
+    accessToken ? `access_token=${accessToken}` : '',
+  );
}

Hence, using event.cookies that you've updated in the refresh request. This cookies register seems to preserve between requests in one SSR-session, so this works fine on the server-side.
The second issue then happens on the client-side. The issue is that your implementation relies on the SSR-session "impersonating" the user. In other words, when a server-side request is made, the server "takes over" the authorisation token and uses it like the client would. It then refreshes the token and — with the above fix — the client-side then takes over and...
The SSR data isn't correctly passed on (I believe, you're using window.__URQL__ but haven't implemented a handover process which passes the SSR side's ssrExchange data to the client-side. So, it attempts to make the request again and doesn't do so with the new access token.
On the client-side, I see REFRESHED: [...], which means that the code further doesn't pass the refreshed access token from the SSR process to the client. I believe, while you refresh SvelteKit actually takes over the caching of the SSR results.
This is basically where SvelteKit's caching of fetch responses conflicts with the ssrExchange approach. However, I believe, the cached first request is overriding the second one, which has the updated result. This might be because the state doesn't match up, which could be based on the cookie mismatch. I don't know enough about SvelteKit's rehydration process to tell you more about why this happens.
I can however see that SvelteKit correctly sends a SetCookie header on the HTML respone of SSR responses. This means that your event.cookies update is correctly translated into a cookie update on the SSR response.
This is good if you don't want to use the ssrExchange and bypass it. However, you'll basically have to figure out why SvelteKit's fetch rehydration isn't working, if you want to use SvelteKit's rehydration. Alternatively, using the ssrExchange correctly would work, but would likely just duplicate SvelteKit's rehydration process.
Hence, let's remove the ssrExchange for a second, since you're not doing anything with it:
// layout.ts
			cacheExchange,
-			ssrExchange({
-				initialState: browser ? window.__URQL__ : undefined,
-				isClient: browser
-			}),
			errorExchange({

To recap again, I don't see any fetch requests on the client-side when I refresh the SSR page, but I can confirm that at this point the server-side sees the correct result. In other words, I can see a correct result here on the server-side but not the client-side: https://github.com/Nickersoft/sveltekit-bugs/blob/d4fee0f271c236be209bd6a91b73b334d60d0b02/src/routes/ssr/%2Bpage.ts#L19
To further explain what's going on here, we now basically see that the client-side is repeating the entire path that the server has done, but I also see that it's trying to do an auth refresh and then seeing an error in the subsequent GraphQL API request. All without actually sending the request, implying that these are cached requests, including the /refresh request.
At this point, we'll basically have to look at why the client-side is making this cached /refresh call and get rid of it, so to make that change, let's ensure it definitely uses the access token that the server-side gives it and never let authExchange make another request.
This is an important note. If you're using refresh+access token cookies, there's never any need to make a refresh call, ever. The API should always be able to refresh the token independently, without a call to /refresh, so you may want to implement this mechanism, if you haven't already.
So, the last change is — in the absence of the above — that the client-side should never make a refresh call and assume that SSR has already made this call: https://github.com/Nickersoft/sveltekit-bugs/blob/d4fee0f271c236be209bd6a91b73b334d60d0b02/src/routes/%2Blayout.ts#L31-L33
// layout.ts
				async getAuth({ authState }) {
-					if (!authState) {
-						return data?.accessToken ?? null;
-					}
+					if (browser && !authState && data?.accessToken) {
+						return data?.accessToken ?? null;
+					}

					const result = await fetch('http://localhost:3000/refresh', {
						method: 'POST',
						credentials: 'include'
					});

This is basically, I believe, a small flaw in how your getAuth is supposed to work. In short, it's called when authentication has failed and on initialisation. However, we only want authState to be initialised to data?.accessToken a) on the client-side, where the token is indeed available, and when data?.accessToken is available.
This is because, again, you're using a refresh auth mechanism with cached fetch requests on the client-side, and without authState actually mattering, since you're using cookies. So, while you may have to make refresh calls on the server-side, the client-side only has to assume that after SSR the access token is correct. In fact, you could even just tell it that, when CSP isn't run, the token is correct.
I hope this helps, but in short, what I'd do, if you want to continue to use cookies, is to refactor the backend to perform refresh-auth opaquely. In other words, don't require a refresh call and ensure that you only error when the refresh has failed, and otherwise let the server-side take care of all the details of when to refresh the tokens, and send new ones using SetCookie (which presumably you're currently only doing on /refresh)


pgpiod build and service autostart.

The new rotary encoder driver is built upon pigpio, which requires pigpiod daemon running in background. The deployment process needs to build the daemon during installation process and start the daemon before the backend. This issue is related to PR #184, and issue #176.

Would it work fine to install pigpiod using sudo apt-get install pigpio rather than downloading and building it ourselves?

I haven't tested it. I'll test it tomorrow and update it in this thread.

I guess sudo apt-get install pigpio is the right way to go. On pigpio's repo they've mentioned that pigpio is currently experimental for RPi 4 , so there is a huge chance of pigpios latest release breaking our deployment process. Furthermore, I've discovered that pigpio is pre-installed on the RPi OS version that we are using, minimizing the need to reinstall it during deployment. However, adding sudo apt-get install pigpio to deployment process will ensure that the daemon is present in the system irrespective of the OS.


feat: improve body helpers in request struct

Description
This PR fixes #

Not needed anymore. Stale.


Add Support for Cascading LogLevel Configuration

JSLogger should cascade the LogLevel configuration for named loggers based on their namespace, so for example:
var loggerA = Logger.get("parentNS.clientA");
// Configure the LogLevel of the parent namespace
Logger.get("parentNS").setLevel(Logger.WARN);
loggerA.info("Info Message");   // No output
loggerA.warn("Warning Message"):        // Logs "Warning Message"
This feature would provide developers a way to express log levels for an entire group of clients without having to manually list them all out.

You could have a look at how debug does this.
Debug has no global logger, only named loggers. And it is basically just a list, there is no hierarchy.
But it can simulate hierarchy through naming conventions. They recommend using colon : to separate levels:
var debug = require('debug')
var log = require('my-app:my-module')
log('Hello!')

By default, this would not produce any output, but you can enable loggers using a pattern:
debug.enable('my-app:my-module') // enables only the one module
debug.enable('my-app:*') // enables every submodule of my-app

They translate the pattern and then use simple regex pattern matching on the name to determine if the logger is enabled or not. It works surprisingly well.


[FEATURE] Students Guide updates

Re: Universities Monday 2021/06/14 / 2021-06-17
4 Misho
4.1 On the home panel - change "Promoted" title to "Student Guide Highlights"
4.2 I love the locked top and scroll screen ("snack") but, for me the Titles (like "Socializing", "Excersize", etc) are clipped. I am using largest font in accessibility on Android.
4.3 Can we sort the entries on the 1st panel by "category" then "section" please so that "Academic support" would be 1st, followed by "Career" etc and then within "Academic support", "Books" would be 1st and "Writing help" would be last?
4.4 JSON file - in "id":"safer.illinois_07" the "title" and "description" are in Latin ("Lorem..."). This item should use the content in the attached file (lorem_fix.xlsx)
lorem_fix.xlsx

Closing this as it was created in a wrong repository.


[feature request] - Add a warning if G6 is used, but readings are only pulled from Dexcom Share, indicating a wrong transmitter ID

Is your feature request related to a problem? Please describe.
If Dexcom Share is configured in the G6 CGM setup, and the transmitter ID is wrong or not updated upon change of transmitted, the user will become dependent on internet connection (and the Dexcom Share server being operational). This is a rather common scenario, and the recommendation for Loop in LoopDocs is actually to not enter the Share credentials to avoid this trap.
I propose we rather make the required changes so that internet dependence will not go unnoticed by the user. This will allow users to get the benefit of backfilled readings when returning from out of bluetooth range, without the risks of unnoticed internet dependency after transmitter changes.
From LoopDocs:

About Dexcom Share credentials
You do NOT need your Share account info listed in Loop settings if you are using a G4, G5, or G6 system. The transmitter ID is sufficient. In fact, the recommendation is that you leave your Share account empty so that you don't accidentally become internet-dependent for CGM data when you forget to update your transmitter ID when you start a new transmitter. Just leave the Share credentials blank.
...
If you don't update your transmitter ID when you change active transmitters, your Loop will not get CGM data from the Dexcom app.
If you did add Share credential, Loop will get data from your Dexcom Share server and will not work without cell or wifi connection.

Loop does display a cloud icon that is intended to make users aware, but this has proven to be too inconspicuous, and does not prevent people falling into the internet dependency trap:

When Loop is using data from Dexcom Share servers, a small cloud will appear above the glucose reading in Loop and should tip you off that maybe you forgot to update your transmitter ID.

Describe the solution you'd like
Add a warning if G6 is used, but readings are only pulled from Dexcom Share, indicating a wrong transmitter ID.
The warning should only be issued after some time, as to not trigger upon backfills. I propose a proper warning to pop up, but a "cloud icon" indicator on or near the GCM value could also be useful too.
Describe alternatives you've considered
No alternatives comes to mind.
User Impact
Judging the reports in Looped FB and the LoopDocs recommendations, this has a big impact.

I don't use Dexcom Share, so how do you get to this? It automatically falls back to Dexcom Share? Or is this the same situation as when you remove the G6 and it automagically falls back to Nightscout instead. (Shows it in dropdown, but no warnings on glucose source there either.)

it automagically falls back to Nightscout instead. (Shows it in dropdown, but no warnings on glucose source there either.)
that should actually not happen anymore with Trio 👀

@dnzxy I'll try again on next sensor change :)

I don't use Dexcom Share, so how do you get to this? It automatically falls back to Dexcom Share? Or is this the same situation as when you remove the G6 and it automagically falls back to Nightscout instead. (Shows it in dropdown, but no warnings on glucose source there either.)

You can optionally enter your Dexcom Share credentials in the G6 CGM configuration. This allows readings to be backfilled after the phone has been out of range of the transmitter. The problem is that it can also cover up the mistake of not changing the transmitter ID in Trio (remove G6 as CGM sources and add it back in with the new ID) when changing the transmitter. This allows internet dependency to creep up on users.


Branch filters and internal links [DOT 2.x develop branch]

On the samples here:
http://www.oxygenxml.com/forum/files/branchFiltering2.zip
The "index.dita" has an xref like:
   <xref href="someOtherTopic.dita#topic_rl4_rc2_ys"/>

When the second branch is created, there is an "index-1.html" which has a reference to "someOtherTopic.html" instead of "someOtherTopic-1.html" so that the references would be self-contained on one branch.
Related links seems to work fine though.

Others may disagree, but this isn't a bug since DITA 1.3 doesn't mandate that links in branch filtering branches need to rewritten to point to duplicated targets in that same branch. To me, this is similar to a feature that links should point to the "closest" occurrence of a duplicated topic.
This could be left as an improvement feature for future.

Ok, I do not have a precise need for this right now. It's good to know the limitations though.
What I was trying to do was to take our user's manual's main DITA Map and refer it in a "main.ditamap" with two or more ditavalref's in order to create a table of contents which contains in it merged both the Oxygen XML Editor and Oxygen XML Author user's manuals. I'm not sure if end users will have similar cases in which they plan to use branch filtering.

You should not be using direct URI references for any links. There is no way for a processor to know, for an arbitrary URI reference, what the appropriate rewriting would be. With keys the map owner has control, but direct URI references are uncontrollable. So Jarno is correct--not a bug.
Having said that, a processor could be smart enough to take a stab at rewriting direct URIs so they "make sense" in the face of the effective copy-tos produced by branch filtering. But anything it did would only be the correct answer for some, but not all, users.

After talking with the oXygen folk at the LavaCon conference, it appears that I am being challenged by this issue.
We are trying to create a library of manuals, so that we can link between our manuals. For these links, I was planning to only allow the use of reltables.
WIthin this library of manuals, two use many of the same topics. These topics are using conditional processing to include or exclude content. So while they both use "topicA.dita", the two output versions of what would become "topicA.html" differ in content. At this point, I understand that I need two differentiated file names.
All the links that I provide (either within the topic, which I understand is not pure DITA, or via reltables) end up redirecting to the first instance of "topicA.html". For the second manual, this link ends up being wrong - it takes it to the topic where the conditional content is no longer valid.
I have tried using the "copy-to" but without successful results. The links then appear as broken, as the copy-to creates a valid HTML file name, but none of the links know to re-name themselves to point to this file.
You imply that the reltables can be a solution here? Can you instruct me - how can I specify that the reltable point to the name of a file that will eventually be created after the transform?
It is critical to me that I be able to link from one manual to another (from our GUI-interface manual to our deeply-technical specifications manual), and I thought that creating a library that comprised of my manuals would be the solution. But as we have topic re-use, I now have too many output files being created with number extensions, thus breaking any and all of my linking strategies.

Hi Scott,
From what I understood on this thread, once you start using ditavalref and you want links between topics to properly point either to a topic in the same publication or to a topic in another publication, you will also be bound to use key references and key scopes in order to properly differentiate targets.
I'm attaching a sample project I made to exemplify this:
ditavalRefWithLinking.zip
In the "topic2.dita" there are two xrefs, one points to the "topic1.dita" from the first publication and it does this by using the "novice" keyscope which I defined on the first reference to the "publication.ditamap" followed by the "firstTopic" key name which I have set on the topic reference to the "topic1.dita". The second xref points to the "topic1.dita" from the current publication, as it does not use the keyscope in the keyref value.
I also used the "dvrResourceSuffix" in the ditavalref to take a little bit of control over how each topic will be named in the published output.
My sample project seems to work with DITA OT 3.1.3, the links text and targets seem to be properly computed.
Regards,
Radu

Thanks for this example!
It helped me to solve the issues I was having with the “copy-to” attribute; using the “dvrResourceSuffix” element provided the proper file re-naming without breaking my branch filtering.
It also helped  me to solve my linking issues when two manuals shared the same map in a branch filtering, through the use of keys and keyrefs.
For my last step, I need to figure out how to reference these files in my reltables.
For example, in the ZIP file included, I want to use reltables to:

LINK between Topic A and Topic C within a manual
LINK from Topic B (either PROD or JPROD manual) to RandoA

As you can see in the attached sample, the reltable entries I have are completely ignored, given that the DITA files mentioned do not exist.
I tried using keys:

But to no avail.
Scott
From: Radu Coravu<EMAIL_ADDRESS>Sent: Thursday, November 1, 2018 12:44 AM
To: dita-ot/dita-ot<EMAIL_ADDRESS>Cc: Scott Barney<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [dita-ot/dita-ot] Branch filters and internal links [DOT 2.x develop branch] (#1999)
Hi Scott,
From what I understood on this thread, once you start using ditavalref and you want links between topics to properly point either to a topic in the same publication or to a topic in another publication, you will also be bound to use key references and key scopes in order to properly differentiate targets.
I'm attaching a sample project I made to exemplify this:
ditavalRefWithLinking.zip https://github.com/dita-ot/dita-ot/files/2537621/ditavalRefWithLinking.zip
In the "topic2.dita" there are two xrefs, one points to the "topic1.dita" from the first publication and it does this by using the "novice" keyscope which I defined on the first reference to the "publication.ditamap" followed by the "firstTopic" key name which I have set on the topic reference to the "topic1.dita". The second xref points to the "topic1.dita" from the current publication, as it does not use the keyscope in the keyref value.
I also used the "dvrResourceSuffix" in the ditavalref to take a little bit of control over how each topic will be named in the published output.
My sample project seems to work with DITA OT 3.1.3, the links text and targets seem to be properly computed.
Regards,
Radu
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dita-ot/dita-ot/issues/1999#issuecomment-434959599 , or mute the thread https://github.com/notifications/unsubscribe-auth/AOWwGkwZf8S7vlyDHwi8sXZMbDOR4nfDks5uqqY8gaJpZM4FoeoC .  https://github.com/notifications/beacon/AOWwGjiPfuKedCivUrrRdJrbAB74gPWQks5uqqY8gaJpZM4FoeoC.gif

@fjssbarney I agree, reltables with keyscopes and branch filtering do not work at all, from what it seems the related links are added in the original topic file names and not in the topic files which have been generated by keyscoping and branch filtering.
I added at some point in the past a separate issue for this: https://github.com/dita-ot/dita-ot/issues/2147

I agree that when it comes to using branch filtering in order for links to properly resolve you need to use keyrefs and key scopes.


Fix README on GitHub and (presumably) PyPI

Follow-on from #15.

PyPI looks good!
https://pypi.python.org/pypi/filesystem_tree/1.1.0


permissions list is missing value: pages

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/actions/learn-github-actions/workflow-syntax-for-github-actions#permissions
What changes are you suggesting?
add
pages: read|write|none
to the list
Additional information
This PR https://github.com/SchemaStore/schemastore/pull/1960 add pages to the permission. But the GitHub doc is missing this info.
Also tested with Yaml file. It is accepted by the build server.
permissions:
  pages: read

I do this


Click on a bar to update the chart with less datasets raises "Cannot read property '_meta' of undefined"

Expected Behavior
I use a bar chart with the onClick options.
When we click on a bar, the datasets of the chart are removed and replaced with others.
The number of datasets is variable, we don't know, when we click, how many datasets will be shown.
Current Behavior
When the number of datasets after the click is equal or smaller than before the click, everything is good.
When it's higher, the error "Cannot read property '_meta' of undefined" raises.
Possible Solution
In the handleEvent: function(e) {}, the click event is treated before some code :
*// Remove styling for last active (even if it may still be active)*
*// Built in hover styling*

If we move this code
    if (e.type === 'mouseup' || e.type === 'click') {
        if (options.onClick) {
            // Use e.native here for backwards compatibility
            options.onClick.call(me, e.native, me.active);
        }
    }

at the end of the function, before the return changed, the bug is fixed.
Steps to Reproduce (for bugs)
See the fiddle https://jsfiddle.net/bcmupn6L/1/
Context
I wanted an interactive chart and with this bug i couldn't use chartjs as i wanted, i lost a day reading my code to find the error before searching in the library code.
Environment

Chart.js version: 2.9.3
Browser name and version: Google Chrome Version 80.0.3987.100

There are multiple workarounds.
You can clear the "active", so it does not try to set hover properties on a removed element: https://jsfiddle.net/b7rnhc1p/
You can update after clearing the ´datasets`:
https://jsfiddle.net/0vso871m/
You could disable hover (but then need to find the elements in the event hanled):
https://jsfiddle.net/zgcx9k2n/
You could use a plugin and afterEvent hook:
https://jsfiddle.net/xtdw9sc2/

No longer appears to be an issue in v3.0.0-beta.13 https://jsfiddle.net/bvhe6f8q/


fuse realized reduceops

I think we can generalize reduceop elementwise fusion to any LazyBuffer graph.
This diff is a starting point towards E2_ kernels, fusing realized reducepos:

The main challenge is fusing diamond assigns: test_fuse_local_assign

CI should work if you rebase on master

Is there a reason this was changed to draft? In general, about code reviews
What I can do with code reviews:

Look for obvious ways things are wrong / easier ways to do things / failing smell test
Read tests and see if they cover what I expect them to cover
Do a quick run on my computer and confirm it works (n/a here)

What I can't do:

Confirm complex logic is correct.
Read changes across many files (n/a here)
Read PRs where style and functional changes are mixed (n/a here)

Makes sense.
I think the existing tests (specially external_test_opt!) where great in helping reason about the fusion approach.
This PR includes tests covering all the possible cases for the new functionality (fuse/nonfuse).

when adding many tests at once like this, it's also good to first merge the tests in (with the master behavior), so that it's very clear what behavior have changed due to code changes

This will be merged after fusing all self-contained subgraphs (we'll get it for free)
For now, there are no major speedup benefits from fusing realized reduces.


btanks: fix building with lua 5.2

Motivation for this change
Make it build with lua 5.2
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p 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)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Cc @mmahut

Thank you for reworking this.
@GrahamcOfBorg build btanks