element-desktop: 1.11.87 -> 1.11.89, element-{web,desktop}: move to pkgs/by-name, format

https://github.com/element-hq/element-desktop/releases/tag/v1.11.88
https://github.com/element-hq/element-desktop/releases/tag/v1.11.89

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

There seem to be some problems with this version, marking WIP for now.

Sorry, also need to remove it from inputs in those files.

Looks like something just needs to be formatted, real quick.

nixpkgs-review result
Generated using nixpkgs-review.


Bug in vendors

[x] missing options in sort by - email, unused credits
[x] sort by under listing screen and detailed screen-Created time,last modified time,Company name(Not functioning properly)

[ ] sort by under listing screen and detailed screen-last modified time,Company name(Not functioning properly)

issue fixed!


Validator Handler should support different base url

e.g. in the OpenAPI spec
servers:
- url: https://${host}/{path1}/{path2}/v1

the default baseUrl in OpenAPIHelper is "/{path1}/{path2}/v1", then whenever a request comes, it will match "/{path1}/{path2}/v1", then rewrite it to "/"
but if there is already a rewrite happening in between (e.g. rewrite by ingress), this will case the normalization fail

update
It is hard for the OpenAPIHelper to know which paths should be rewrite, thus, we should support relative server url

 Validator Handler should support different base url light-rest-4j master


⚠ API (Staging) has degraded performance

In 276f615, API (Staging) (https://api-dev.nest.land/imports/x/denoland/std/0.75.0) experienced degraded performance:

HTTP code: 200
Response time: 2842 ms

Resolved: API (Staging) performance has improved in ffeee16.


Check if state is present in request.

Coverage remained the same at 89.495% when pulling 3afccf32d2ef5024cb4f428313909471e5d229b5 on akatsoulas:oidc-errors into e6793e9516e939a38056bd66850f4e694f5ae928 on mozilla:master.


Update libmetal & openamp v2021.04

update of the west.yml to  integrate to the new OpenAMP release

@arnopo please update west.yml whenever you get a chance now that the module repos PRs are merged.


OIP-2 Internal dripper and protocol reserves

Smooths all yield out and adds a protocol reserve.
By smoothing we improve user experience, and provide hack resistance (since any false yield will be spread out over days, and unavailable in the same block)
Big Picture

The drip rate from the funds reserve is recalculated after every rebase. The rebase rate is based on dividing out the total held in the reserve by the drip duration.
Distribution amount

Fork Simulation

This implements issue #985
And since this is our first Vault Core change since finding this bug, we are adding in #1247.

@DanielVF should I review this PR? Are we still thinking of deploying this one?


Show CTA for paywalled content

This PR adds a CTA in place of paywalled content to encourage businesses to sign up for a paid subscription. The wells link to the pricing page which describes that a business subscription unlocks hidden content.
This PR builds off of @jacobdaddario's #193, merged in earlier today.

Pull request checklist

[x] Your code contains tests relevant for code you modified
[x] You have linted and tested the project with bin/check

Looks good! I like the way that this came out.


CMD调用第三方程序， 也是醉了

CMD调用第三方程序， 也是醉了

Thank you for registration of this issue.
However, I cannot unfortunately understand what you mean...
If you would like to let me know your trouble, please add new issue in English.
Thank you.


Implement Support for Sendgrid on a System and User Level

Fixes #108  Implement Support for Sendgrid on a System and User Level
Short description of what this resolves:
Implement Support for Sendgrid on a System and User Level
Changes proposed in this pull request:

Implement Support for Sendgrid on a System and User Level

Checklist

[x]  I have read the Contribution & Best practices Guide.
[x]  My branch is up-to-date with the Upstream development branch.
[ ]  The acceptance, integration, unit tests and linter pass locally with my changes
[ ]  I have added tests that prove my fix is effective or that my feature works
[ ]  I have added necessary documentation (if appropriate)

Is this latest commit maybe to the wrong branch as it is about the Stripe plugin?

Hi @mariobehling , it just a minor change relate to stripe template, should I create a new pull request?

Please also change:

☑ Use Custom Email
All mail related to your event will be sent over your specified email gateway.

In global settings there should be the info what is the current configuration. This info should be read from the config file.
In the tested system the config was using SMTP, but the SMTP info does not show up in the settings view.

Comments from the video chat today:

default settings should be read from the pretix.cfg which is mounted RO into the pretix docker container
changes/saved values should be saved to a different config file OR the database

Please resolve conflicts.

A few issues and questions:

does this work with read-only configuration file pretix.cfg?
please fix conflict
does this support setting the email per organizer/event, or only in the global config file?

Please resolve conflict.

Hi @mariobehling, conflict is solved, could you help me to check it?

Looks like there are a number of changes that result in conflicts again.

Hi @mariobehling, conflict is resolved.

@lcduong @odkhang Please answer the questions of @norbusan

A few issues and questions:

does this work with read-only configuration file pretix.cfg?
does this support setting the email per organizer/event, or only in the global config file?

Feedback during meeting

does this work with read-only configuration file pretix.cfg?

Yes, it should be working.

does this support setting the email per organizer/event, or only in the global config file?

Yes, it is implemented in that way, but needs further testing in real-world conditions.

After merging this PR there is an internal server error, e.g. check here when you invite a team member: https://tickets-dev.eventyay.com/control/organizer/conferenceorganiser/team/1/


Add "progress: hide" to the wiki page "Updating Dialog with new content"

Describe the bug
The feature to hide a progress bar on a running dialog is not documented in the wiki.
Additional context
Add any other context about the problem here.
@drtaru on Slack posted a proof of concept that this feature exists: https://macadmins.slack.com/archives/C01U5MXNGG6/p1706026053006109?thread_ts=1706021017.622559&cid=C01U5MXNGG6

added a line on this page https://github.com/swiftDialog/swiftDialog/wiki/Updating-Dialog-with-new-content


tf.Variable shape is restricted with copy.deepcopy

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): Ubuntu 18.04
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.4.1
Python version: 3.8
CUDA/cuDNN version: ???
GPU model and memory: GTX 1070

Describe the current behavior
tf.Variable shape is restricted when using copy.deepcopy
>>> x = tf.Variable([[1.2, 3.4]], shape=[None, 2])
>>> x.shape
TensorShape([None, 2])
>>> import copy
>>> copy.deepcopy(x).shape
TensorShape([5, 2])

Describe the expected behavior
copy has the same shape as original.

I was able to reproduce the issue on tf-nightly 2.10.0-dev20220714. Kindly find the gist of it here. Thank you!

@joelberkeley,
I tried to execute the mentioned code on the latest version with the alternative approach and the code was executed without any issues.
Here instead of using copy, you can try to use tf.identity which returns a Tensor with the same shape and contents as input.
https://www.tensorflow.org/api_docs/python/tf/identity
Kindly find the gist of it here.
Thank you!

@tilakrayal I've forgotten the details of this and no longer need it. Do with the ticket what you wish

@joelberkeley,
Could you please feel free to move this issue to the closed status? Thank you!

Closed as no longer interested


[Workspace Feeds] [External] Create new Edit Limit Type page

Part of the Workspace feeds project.
Implement the following part of the design doc.
@koko57 @VickyStash @allgandalf @DylanDylann

Later phase of implementation

@VickyStash will work on this one

Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue.

Assigned you! :)

@MariaHCD @mountiny I have a couple of questions here:

In docs it says If a card limit is changed so that the remaining limit will go from >$0 to $0, we’ll throw a warning that lets the admin know this will decline transactions. Could you confirm how the limit type change affects the remaining limit  so I can calculate it right.
In docs it says If the card is changed to a fixed limit type and the total spend on the card exceeds the “fixed” limit (i.e. total spend over 2 months is $1,800, and admin is changing the limit type with a limit of $1,000), we won’t show Fixed as an option to switch to, because this would just terminate the card. Where can I find the total spend value?

Ah, good point. I would imagine there is no point in the SmartLimit option if approvals are not configured.

@MariaHCD does it mean it's possible this screen will have only one option - Monthly (if Fixed and Smart are hidden)

Yes, that's a possibility!

Updates:
WIP, needs to resolve a couple of things mentioned above.
Updates can be found in the Draft PR

I think we might need some additional thought around how to implement this. Left some initial thoughts here but I will start a fresh discussion in the channel.

@MariaHCD @mountiny Maybe I can open a PR without specific checks for now, and then we can specify them when linking the page with the backend?

@VickyStash Yes, that sounds good to me. I'm aiming to start a discussion in the channel today.

Started a discussion here: https://expensify.slack.com/archives/C036QM0SLJK/p1721311425532449

Updates:

Implemented warning modal display (used TODOs for checks)
Need to polish the code and add screenshots to the PR

Hey @dubielzyk-expensify, I guess you were working on these mockups. Could you please confirm the buttons text

I guess it should be Save instead of Next.
And maybe Change limit type instead of Change limit.
WDYT?
cc @shawnborton

That makes sense to me. When you are just editing a single push-row and you are not in "step-by-step flow mode", we use "Save" as the button text. And your suggestion to use "Change limit type" makes sense too.
But let's definitely get a check from @dubielzyk-expensify too before moving forward.

Yep agree that those labels make more sense 👍

Thanks for raising these questions!
I agree with the latest copy changes, makes sense!

The PR has been opened for the review 👀

Not overdue, the backend has been merged

I have created a fresh issue for this one as I forgot we still had this one open. Nevertheless, I put all the details of the api command into the new issue https://github.com/Expensify/App/issues/46959 and I think its fine to close this one and handle the integration over there @waterim @VickyStash


Add Lynker attribution to Map byline

This...

...should probably include attribution for the use of material from the Lynker repository, as noted here:
https://lynker-spatial.s3.amazonaws.com/copyright.html

well overdue
just fixed it
https://github.com/CIROH-UA/NGIAB_data_preprocess/commit/b2047bc659d02a8b2c5a4d18103de93248b5e80b


Arduino Mega2560 and MCP2515 not working

Hi, I try with all examples and can't pass form the "CAN.begin(500E3)", always get false.
I have a Toyota highlander 2010 and test with 250E3 and also changed the CAN.setClockFrequency(8E6).
Please someone point me to the right direction.
Thank you

Does this fix the issue? https://github.com/sandeepmistry/arduino-CAN/pull/45


Build on JDK11 by default

As discussed and voted by SIG JVM, the minimal supported JDK version for distributed artifacts has moved from Java8 to Java11. Java17 build is also supported by the activation of the jdk17 Maven profile.

Should we add a module-info.java at the same time? I think it would be pretty short, just adding a dependency on jdk.unsupported for Unsafe.

@Craigacp , so I've checked at the different options on how to support unit testing in a classic Maven project setup and none are very appealing, so I suggest that we continue to simply disable the module path in the surefire plugin config like I currently do, and let it use the regular classpath instead.

@Craigacp , so I've checked at the different options on how to support unit testing with Java modules in a classic Maven project setup and none are very appealing, so I suggest that we continue to simply disable the module path in the surefire plugin config like I currently do, and let unit tests use the regular classpath instead.

Sure. Module testing is a pain and if it's going to cause a lot of problems we can leave it till later.


Fix ci-sonarqube-ts input format

fix input format by using ${{ }} everywhere https://github.com/smartcontractkit/.github/pull/472#discussion_r1662637659
I hadn't noticed this in my previous PR that test artifacts generated weren't downloaded properly because they were in a different workflow. Added support for downloading artifacts from different workflows.
Tested in ci-sonarqube action

ooh I forgot - could you add a changeset?

ooh I forgot - could you add a changeset?

I forget it everytime, my bad. I've added the changeset now, PTAL


raftstore: mark peer when trying to do async removal

When a peer consider itself stale, its state machine should be "turned off".
Tests will be added after we are able to create a test-only store easily.

CI failed @BusyJay

Fail cases don't seem to be related to the changes.

LGTM
PTAL @hhkbp2 @javaforfun

LGTM

LGTM


Separate method open() and constructor

Bug description
Any implementation of retrying algorithm makes code complicated
Steps to reproduce
i.e. tenacity package
In device implementation method open() makes it impossible to implement retrying policy if device is not connected since instance is created after method is called. Don't you think that logic of creating object and connecting to device must be separated?
In this case there will be no recursion errors and easy implementation than self-made logic
def discover(self):
        while not self.device:
            try:
                self.device = Usb(idVendor=int(os.environ.get('PRINTER_VENDOR_ID'),16),
                                    idProduct=int(os.environ.get('PRINTER_PRODUCT_ID'),16),
                                    in_ep=int(os.environ.get('PRINTER_IN_EP'),16),
                                    out_ep=int(os.environ.get('PRINTER_OUT_EP'),16),
                                    timeout=int(os.environ.get('PRINTER_TIMEOUT')))
                return self.device
            except Exception as e:
                logger.error({'message':repr(e)})
                time.sleep(0.5)
                continue

Expected
class MyUsbPrinter(Usb):
    def __init__(self):
       super().__init__(idVendor=int(os.environ.get('PRINTER_VENDOR_ID'),16),
                                    idProduct=int(os.environ.get('PRINTER_PRODUCT_ID'),16),
                                    in_ep=int(os.environ.get('PRINTER_IN_EP'),16),
                                    out_ep=int(os.environ.get('PRINTER_OUT_EP'),16),
                                    timeout=int(os.environ.get('PRINTER_TIMEOUT')))
    def connect(self):
        r = retry(**kwargs)
        usb_args = {}
        usb_args['idVendor'] = self.idVendor
        usb_args['idProduct'] = self.idProduct
        r(self.open, usb_args={})(self)

my_printer = MyUsbPrinter()
my_printer.connect()

And also can be used as callback for read() and write() operations if connection is down to prevent application break
def write(self):
    r = retry(**kwargs, retry_error_callback=self.reconnect(self), retry=retry_if_exception_type(USBError))
    # write to device
   r(self._raw, data, timeout)

Discuss if it fits  logic, please. Another way is to implement Usb device without inheritance.

I completely agree. Object creation and possibility to actually connect are two completely different things. printer.Serial has this approach already: if you don't specify the devfile, it doesn't attempt to open anything, so you end up with a valid object, only that it won't be able to connect until you specify the devfile later, and try to open it.
printer.Usb on the other hand was implemented in a different manner and if you can't connect heads up, you don't even have a valid object to work on or retry anymore.
Of course you can work around this issue, but it would be better if the classes had more consistency between each other regarding this aspect.
One possible workaround is basically composing, by embedding an Usb instance instead of inhreiting from it, as rsaleev suggested.
Regards.

I'm currently working on this feature and I think that I will draft a PR in a few days, so @patkan you can assign this issue to me. ;-)


blind-control: increment invalid if start greater zero?

the blind-control shows an error on the increment !
if the opening-pos value leaves zero (error starts, if we change from zero to any number) .
if the opening-value is  0, everything is fine on the increment-input.
i cannot understand why a 1  should be an error on the increment, if my opening value is more than zero. must be a bug!? can you confirm?

it there is anything to test, i will do...

I must check this. It is a time ago, when I had test this.

Will be published with next version.


[Task] Increase the coverage of OC organization service to >50%

What product(s) is this feature for?
OpenChallenges
Description
No response
Anything else?
No response
Code of Conduct

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

02-27-2024 Update
PR #2536 addresses model/entity and has increased overall Jacoco coverage from 48% to 65%.
Initial model/entity coverage:

Current Jacoco coverage after writing unit tests for Challenge Contribution Entity:

All tests executing as planned, but the CI pr test, which failed after syncing the upstream changes to my repo.
To Do:

Write additional unit tests targeted at improving coverage for Organization Category Entity.
Confirm the level of improvement as a result then move to writing additional unit tests until organization service Sonar coverage is > 50%.

Blockers:
CI pr test failing. I will work w/ @tschaffter to resolve this issue.

Added to Backlog


Using static ip address for HA Helm chart

I'm on GCP.  I'd like to use a static IP address for my HA Helm chart setup.

@maguro Are you looking for this: https://cloud.google.com/kubernetes-engine/docs/tutorials/configuring-domain-name-static-ip

https://github.com/kubernetes/charts/pull/4373

@maguro - Once the pull request is merged, we'll also update the sources here.

@maguro - stuck due to errors in automated tests. We are working on it.


split env detection/setup out of cli entrypoint

The main entry point (driver.js) should not grab stuff out of the CLI tool (cli/ares.js).  Instead, move whatever's needed by both into a new module.

this is invalid; lib/cli is not a cli tool; it's just a lib which interfaces with the various LG SDK executables.


Frontend debugging

Fixed favicon console error
Passed login state via props to home page to fix login issue
Added secret phrase for creating higher perm accounts
Added initial profile page progress for each lab with new endpoint call
Added profileLab to accompany new call
Get_profile_progress fixes

Looks good


Hidden icons flash when opening widgets

Describe the bug
When clicking the date to view the widgets in MacOS, the section of the menubar that has the hidden menubar items under it flashes (see video)
Steps to reproduce

Add some menu bar icons to the hidden section
Click the date to open widgets
(does not happen when opening widgets in another way, for example 2 finger swipe)

Screenshots
https://user-images.githubusercontent.com/8353666/181229889-e20cb673-4dd0-456d-8982-a531a771ab20.mov
Environment
You may write here the specifications like the version of the project, operating system, or hardware if applicable.
Desktop (please complete the following information):

MacBook Air M1
OS: MacOS 12.4
Hidden Bar Version 1.9 (13)

I met exactly the same issue, my platform is M1 pro with osx 12.4, hidden bar version 1.8(12)

I also met exactly the same issue,  M1 air with MacOS 12.4, hidden bar version 1.9(13)

Having the exact same issue...

Hey @c-ehrlich , I had the same issue. I went to menubar settings in system preferences and enabed "always show date with time" and the issue is resolved. Previously it was "only when space is available".
I think this hidden caused menubar to confuse about "space"

@Datta0  thanks for sharing the tip!

Thanks ! @Datta0
That's a great workaround for the issue, and it's worked for me as well!
Using an Intel-based MBP. macOS 12.5.1


extending imported abstract classes fails at execution time

I'm attaching a very simple example you can execute it both with ts-node src/index.ts or with tsc && cd dist && node src/index.js
tsissue12.zip
Expected behavior:execute successful
Actual behavior: Error : TypeError: Class extends value undefined is not a constructor or null
I'm using typescript@next and strict: true
I think this is an issue because, if instead of extending from an imported abstract class I extend without importing it it won't fail.

OK this is not an issue, sorry, I'm closing it. My error was using commonsjs for module and moduleResolution and importing the class at the top of the file so I didn't give time to the dependency to run the declaration. Using other moduleResolution like AMD or moving the import to the bottom solve it. Sorry for the noise

Similar error for me. I splitted one the problematic file into two and it worked. Indeed retlated to import order.


bug: filter helm releases by name

We were not filtering the returning helm releases by their name so we were able to install only one helm chart.

This is not needed anymore.


Fixes build issue on master

Description (required)
Deletes res/values-en-gb
Fixes # #3154  Master build issues
What changes did you make and why?

Codecov Report

Merging #3153 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #3153   +/-   ##
======================================
  Coverage    4.66%   4.66%
======================================
  Files         264     264
  Lines       12400   12400
  Branches     1057    1057
======================================
  Hits          579     579
  Misses      11777   11777
  Partials       44      44

Continue to review full report at Codecov.

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


add missing ESM-comment-* block

hopefully this will fix broken monaco-editor (version 18.0.0), see https://github.com/microsoft/monaco-editor/issues/1574

@alexandrudima Could you also please take a look at this little annoying warning abound critical dependency?

Thank you, but I decided to tweak the file in a different way to align with marked.js...


Line endings are converted to \r\n

Ive uploaded a file (raw file here) to https://bpa.st/.
It's a Bash script and I even choose Bash from the programming language selector.
Since I've made it on Linux with nano it has LF as line ending.
But, when I do wget https://bpa.st/raw/6V5Q it is saved with CRLF as line endings like if it was made on Windows, which breaks the script entirely.
Example:
vitor@EliteDesk:~/Testing$ file userip.sh
userip.sh: Bourne-Again shell script, ASCII text executable
vitor@EliteDesk:~/Testing$ wget https://bpa.st/raw/6V5Q
--2024-03-08 16:59:16--  https://bpa.st/raw/6V5Q
Resolving bpa.st (bpa.st)... 2a01:4f9:5a:5159::2, <IP_ADDRESS>
Connecting to bpa.st (bpa.st)|2a01:4f9:5a:5159::2|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 490 [text/plain]
Saving to: ‘6V5Q’

6V5Q                                      100%[=====================================================================================>]     490  --.-KB/s    in 0s

2024-03-08 16:59:16 (75,4 MB/s) - ‘6V5Q’ saved [490/490]

vitor@EliteDesk:~/Testing$ file 6V5Q
6V5Q: ASCII text, with CRLF line terminators

If I try to run the script that I just wget'd:
vitor@EliteDesk:~/Testing$ bash 6V5Q
6V5Q: line 1: !/usr/bin/env: No such file or directory

However, I've uploaded it to Debian pastebin (raw file here) and I use wget https://paste.debian.net/plain/1309920 it is saved with LF line endings.
Example:
vitor@EliteDesk:~/Testing$ wget https://paste.debian.net/plain/1309920
--2024-03-08 17:09:50--  https://paste.debian.net/plain/1309920
Resolving paste.debian.net (paste.debian.net)... 2a01:4f8:201:3437::2, <IP_ADDRESS>
Connecting to paste.debian.net (paste.debian.net)|2a01:4f8:201:3437::2|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [text/plain]
Saving to: ‘1309920’

1309920                                       [ <=>                                                                                ]     453  --.-KB/s    in 0s

2024-03-08 17:09:50 (53,7 MB/s) - ‘1309920’ saved [453]

vitor@EliteDesk:~/Testing$ file 1309920
1309920: Bourne-Again shell script, ASCII text executable

I assume that the issue comes from pinnwand itself?

I would assume the same. It seems the data is already stored with CRLF line endings in the database as requesting the hex view shows the \0d\0a bytes directly.
As a quick sidenote, your initial paste will expire so I'll include it here (by the way, your shebang is missing a #):
€ curl https://bpa.st/raw/6V5Q
!/usr/bin/env bash
read -r -p 'User Name: ' theUser
printf "Adding IPv6 address to netplan"
theIp=$(printf "%s$(echo -n "$theUser" | sha1sum | head -c 16 | sed 's/..../:&/g')\n" "200a:200b:200c:200d")
echo "        # $theUser" >> /etc/netplan/01-netcfg.yaml &&
echo "        - $theIp/64" >> /etc/netplan/01-netcfg.yaml &&
printf "Username: $theUser\nIP: $theIp\n" &&
cat /etc/netplan/01-netcfg.yaml | grep -A1 "$theUser" &&
/usr/sbin/netplan apply &&
sleep 2 &&
ping6 -c4 "$theIp"%
€ curl https://bpa.st/hex/6V5Q
212f7573722f62696e2f656e7620626173680d0a72656164202d72202d70202755736572204e616d653a202720746865557365720d0a7072696e74662022416464696e672049507636206164647265737320746f206e6574706c616e220d0a74686549703d24287072696e74662022257324286563686f202d6e2022247468655573657222207c207368613173756d207c2068656164202d63203136207c207365642027732f2e2e2e2e2f3a262f6727295c6e222022323030613a323030623a323030633a3230306422290d0a6563686f202220202020202020202320247468655573657222203e3e202f6574632f6e6574706c616e2f30312d6e65746366672e79616d6c2026260d0a6563686f202220202020202020202d202474686549702f363422203e3e202f6574632f6e6574706c616e2f30312d6e65746366672e79616d6c2026260d0a7072696e74662022557365726e616d653a2024746865557365725c6e49503a202474686549705c6e222026260d0a636174202f6574632f6e6574706c616e2f30312d6e65746366672e79616d6c207c2067726570202d413120222474686555736572222026260d0a2f7573722f7362696e2f6e6574706c616e206170706c792026260d0a736c65657020322026260d0a70696e6736202d6334202224746865497022%

As a quick sidenote, your initial paste will expire so I'll include all relevant URLs here (by the way, your shebang is missing a # on the bpa.st paste)

Yeah, I've probably mispasted it


Support for terraform apply parallelism option

Could you enable support for specifying -parallelism n option in terraform apply command. AzureRM seems to be a bit picky when running with default value (10) on jenkins. I manually added -parallelism 1 to apply_command.rb options and AzureRM resources are now being created without problems.
https://www.terraform.io/docs/commands/apply.html#_parallelism_n
Thanks!
-Sami

Hi Sami:
Thanks for your interest in the project! Adding a parallelism option to the configuration file should be should be simple. We'll try to get that in to the next release.


webpack 2.6.0 can't find variable: Promise on require async

Do you want to request a feature or report a bug?

What is the current behavior?
Having ReferenceError: Can't find variable: Promise when using require(['dep'], callback)
If the current behavior is a bug, please provide the steps to reproduce.
So, I'm using karma to run tests on phantomjs with webpack, and phantomjs doesn't have native promises. So, this is why I have this in my webpack.config (also, for older browsers):
...
new webpack.ProvidePlugin({
  Promise: 'es6-promise-promise'
}),
...

Then, on my code, I could do something like this, and everything worked fine until<EMAIL_ADDRESS>(and under, in fact, since 1.x this worked fine):
console.log('promise', Promise);
require(['left-pad'], function (leftpad) {
  // ...
});

But then, simply by upgrading to<EMAIL_ADDRESS>I get the ReferenceError: Can't find variable: Promise error now.
I believe this is because webpack uses promises to load the chunks, and somehow stopped replacing variables from ProvidePlugin on its own boilerplate.

What is the expected behavior?
Promises should work when defined on ProvidePlugin, even if it is webpack generated code, right?
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
macOS, but also happened on ubuntu. Node 6.9. Webpack 2.6.0

Experiencing same issue here.
In our case the error it's coming from manifest js: ReferenceError: Can't find variable: Promise at http://<IP_ADDRESS>:42069/unit-manifest-6eed6c926e1a80d7a48c.js:40.

Hm... facing a similar issue (in IE 11) with possibly the same cause, even without any asynchronous loading.
Seems that the generated webpackBootstrap (defines webpackJsonp) code includes a line like this:
var resolvedPromise = new Promise(function(resolve) { resolve(); });

This line was / is not present using webpack 2.5.x.

We just updated from 2.5.1 from 2.6.0 and have the same issue with PhantomJS (local on macOS and remotely with an Ubuntu-based docker container on the CI server).

Same problem. My Promise polyfill stopped working with 2.6.0. It worked again when I downgraded to 2.5.1.
new webpack.ProvidePlugin({
    fetch: 'imports-loader?this=>global!exports-loader?global.fetch!whatwg-fetch',
    Promise: 'core-js/es6/promise',
}),

Same problem here.

Yup, exact same issue here

Hi,
same issue here (actually we don't have the same config than OP, but we see the Promise is undefined error).
Here is a diff between a broken and working chunk (left/right), if that helps:

I think those changes only affect <=IE11 if new Promise() is executed before the polyfill kicks in.

Hit this regression as well in IE <= 11.

+1,  i use babel-polyfill, and it was fine yesterday.

Looks to be introduced in 22e7d254ee27e68569ea9718632accc870f52002.

Same issue

Also the same issue - ts-loader build now failing due to this 😭

@sokra Will there be a bugfix release? It seems to have broken quite a lot of builds.

thanks!

Still have the problem on webpack 2.6.1. Someone else?

Still erroring on:
var resolvedPromise = new Promise(function(resolve) { resolve(); });

all IE <= 11 issue not present with 2.5.1

The 2.6.1 version seems to fix it. All calls to Promise are now done inside a function which gets called at a later time. The first thing I have on my entry file is import 'core-js'; which provides the Promise polyfill.

Is it normal to be forced to add the Promise polyfill ourselves to make webpack work on IE 11?

It was a mistake in 2.6.0 where a Promise was used in the bootstrap code that loads your app. So this was being called before any code (your app) that has babel or other polyfills applied to it. It was fixed with 2.6.1

I am currently using webpack 2.6.1.
I still need a promise polyfill because System.import still leads to ES6 Promise creation.

Then yes, you do need a polyfill.  See
https://webpack.js.org/guides/code-splitting-async/#promise-polyfill
On Thu, Jun 22, 2017 at 6:25 PM Réda Housni Alaoui<EMAIL_ADDRESS>wrote:

I am currently using webpack 2.6.1.
I still need a promise polyfill because System.import still leads to ES6
Promise creation.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/webpack/webpack/issues/4916#issuecomment-310431599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANWcnGig7rx4mEjgPaNk5RKd70A0yKKks5sGpWGgaJpZM4NjETV
.

Actually what I do is have this in my html in the head:
window.Promise||document.write('

I don't want to bother my original Webpack config, so here is a workaround:
<script>
window.Promise || document.write(
  '<script src="//cdn.jsdelivr.net/es6-promise/4.1.0/es6-promise.auto.min.js"><\/script>'
)
</script>
<script type="text/javascript" src="js/manifest.15a34c.js"></script><script type="text/javascript" src="js/vendor.ab046a.js"></script><script type="text/javascript" src="js/app.6fa5fc.js"></script>


keep a record of all changes that includes who made the change, and the timestamp

this is critical for the evaluation chapter. use this to be able to assess when and how often players interact with each others' patch (and need to design it so that this is detectable)
if not done through automerge, do it yourself, and dump it to csv

for now, do it within Patch.js, not automerge, store to csv. then create a script which collates all records of patch changes from each instance.

[ ] build this within Patch.js
[ ] records all user edits as an object, pushed to an array, then written as a line in a csv
[ ] field: timestamp
[ ] field: username
[ ] field: edit action
[ ] field: item owner (if edit action is on an item spawned by another user)


chore: share analysis_options between all projects and fix warnings

Description
This removes the duplicate analysis_options.yaml files and fix the remaining warnings (as not all projects used the updated lints)
Related Issues
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (flutter analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[ ] No, this is not a breaking change.

Closing. I'll first apply the lints on a per-folder basis for the packages that don't use an analysis_options yet, then merge the analysis_options in one


Fix svg warnings

I saw warnings in the Expo web view that looked like:
Invalid DOM property "stroke-width". Did you mean "strokeWidth"?
To test, I copied over the build after running yarn prepare to my local Expo build and the warnings went away.

@fangherk thanks for doing this!

@nandorojo just wanted to bump this PR in case it got lost in the shuffle. I've been getting these errors for a while now and would love to clear up the console. Thanks so much for all your hard work!

Hey I'll publish this tomorrow

Live in 0.12.2

Thanks @nandorojo!


Double dependency reporting for HTTP Requests

It looks like the remote dependencies are getting double logged for HTTP requests. I have two apps that are communicating with each other, so I can see the request getting logged once on the receiving end.

It also seems like the C# library could be double logging dependencies as well.
Could this be related to: https://github.com/Microsoft/ApplicationInsights-node.js/issues/323?
Here is how I have AppInsights configured:
var appInsights = require('applicationinsights');
var winston = require('winston');
var expressWinston = require('express-winston');
require('le_node');

appInsights.setup(config.appInsightsKey)
      .setAutoCollectConsole(false)
      .start();
  //register common properties to global client (used for auto-collection)
  appInsights.defaultClient.commonProperties = log.commonAppInsightsProperties;
  appInsights.defaultClient.addTelemetryProcessor(telemetryProcessor);

var log = new winston.Logger({
  transports:[
    new winston.transports.Console({
      handleExceptions: true,
      prettyPrint: true,
      level: config.logLevel.console
    }),
    new winston.transports.Logentries({
      name: 'logentriesLogger',
      token: config.logentriesKey,
      handleExceptions: true,
      minLevel: config.logLevel.logentries,
      withStack: true
    }),
    new winston.transports.CustomLogger({
      level: config.logLevel.appInsights,
      handleExceptions: true
    })
  ],
  exitOnError: false
});

This could be related to #323. How are you making your http requests? With the built in http library or something else?

With request.

In this case, this is definitely one of the problems described in #323, that also found the same to be happening with request. My current guess is that we're double collecting because of how request works internally.
Thanks for reporting this! Until this is fixed, you may be able to workaround with a telemetry processor that keeps track of what dependency calls it has seen in the current correlation context, or disable automatic dependency collection and do manual trackDependency calls

The double reporting is due to 300-level redirects. The request module follows these, and subsequently makes a second request. In 1.0.0 status codes were not reported correctly to showcase this difference, and that was fixed in the newly-released SDK v1.0.1.
If you don't want this redirect telemetry sent, you can now use a telemetry processor to filter out dependency telemetry for status codes indicating redirect.
Please feel free to re-open if you're experiencing something different than this in 1.0.1!


[App Config] Initial setup for record and playback

Please read this through and make sure that you agree.
This PR only adds record and playback to the test files auth.spec.ts and synctokenpolicy.spec.ts. The changes to the other test files should look similar to the changes to auth.spec.ts. Please take over this PR as soon as you find time.
Some tests of synctokenpolicy.spec.ts have been skipped only for test modes record and playback. These tests use the nock package, just like the recorder tool. The recorder tool probably doesn't clean nock properly to have something else using it.
Some of the custom replacements should not be necessary. I filed this bug to fix the underlying issue: https://github.com/Azure/azure-sdk-for-js/issues/6539

Replacing myself with @jonathandturner in the list of reviewers


10.49.1 failing to connect to Atlas Device Sync

Description
I'm using MongoDB Atlas with Device Sync.
The bug appear after upgrading after upgrading from Realm v10.49.0 to v10.49.1.
After launching the app, the sync error RLMSyncErrorClientSessionError is passed into the closure at app.syncManager.errorHandler. The error does not contain any additional details.
The error is triggered by a RLM_ERR_CONNECTION_CLOSED that's originating here:
https://github.com/realm/realm-core/blob/4eef991270a5e2161e37e8c227404327ac61f618/src/realm/sync/network/websocket.cpp#L1021
The SDK keeps trying to connect to Atlas Device Sync every few seconds and all attempts keep failing.
No log entries are shown in Atlas Device Sync for the device in question.
The error is reproducible on both Macs I've tested (Mac Studio and MacBook Pro, both with Apple Silicon) and on an iPad. One Mac is using wired Ethernet and the other devices are on Wi-Fi.
Can you reproduce the bug?
95% of the time. It first only appeared on the Macs, but after a few hours, the iPad started exhibiting the same behavior.
If the app and all associated data is deleted from the device and then reinstalled, it will work fine the first time it is launched.
Version
10.49.1
What Atlas Services are you using?
Atlas Device Sync
Are you using encryption?
Yes
Platform OS and version(s)
macOS Sonoma 14.4.1 (23E224)
iPadOS 17.4.1
Build environment
Xcode 15.4
Swift Package Manager

The problem seems to be that the SDK is trying to connect to a wrong host.
In SyncSession::create_sync_session, session_config.server_address is...

ws.realm.mongodb.com when the connection fails
ws.europe-west1.gcp.realm.mongodb.com when the connection succeeds

This bug may have been introduced in https://github.com/realm/realm-core/commit/255cb33d838122e23fae4685dc253d708aac435e.

I think I duplicated this issue and was seeing the same thing.
However, right in the middle of typing this, it seemed to resolve itself.
The issue appeared to revolve around running the app with an already authenticated user - it seemed to be trying to connect to an incorrect endpoint. I had to log out and log back in and then it work and connected to the correct endpoint.
Can you try your code again and see if it's an ongoing issue?

Well, let me retract my above comment as it's doing it again.
After updating to 10.49.1 the app does not connect and the only error shown (in the app) was
Info: Connection[1]: Closing the websocket with error code=WebSocket: Internal Server Error, message='error', was_clean=true
Upon checking the Logs in the Realm Console, there was no error.
I then re-ran the app (with logged the user out), logged in and it connected and worked correctly. No further issues.
However, if I quit the app while still authenticated, and then re-run the app, it attempts to connect but then fails with the same above error. I have to log out and log back in to connect.
I can verify the endpoint it attempts to connect to when starting the app with an already-authenticated user is different than the connection if I log out and log back in - maybe that's by design?
Here's what I see if logging in, quitting and re-running the app
Info: Connected to endpoint '<IP_ADDRESS>:443' (from 'my ip address')
Here's what I see if I log out, quit the app, re-run it and log back in
Info: Connected to endpoint '<IP_ADDRESS>:443' (from 'my ip address')
There are no errors in the console at all - but that being said, the endpoint doesn't seem to the correct one if the user is already authenticated.
My endpoint looks like
Info: Connection[1]: Connecting to 'wss://ws.us-east-1.aws.realm.mongodb.com:443/api/client/v2.0/app/my_cool_app-xyzzy/realm-sync'

We are facing exactly the same issue, please help

After migrating version to 10.49.1, we also faced the same error.
Error :  Info: Connection[1]: Closing the websocket with error code=WebSocket: Internal Server Error, message='error', was_clean=true

Hi @andreasley,
This looks like it is related to the latest base URL changes added recently - the websocket will try to connect to the default websocket address determined by the provided App::Config::base_url value, or based on the default (https://realm.mongodb.com, soon to be https://services.cloud.mongodb.com) if none is provided. If this fails, it will try to query the location endpoint to try to retrieve the correct localized websocket URL for the app and use that when reconnecting.
When you log in before starting the sync session with the cloud, the location info was already queried/updated per the log in request to the server. The websocket connection is a bit different, since it uses a different code path/underlying driver and updating the location takes a bit more coordination.
I am working on a couple of fixes around this area, but a temporary workaround, for now, would be to specify the localized base_url value (e.g. https://us-east-1.aws.realm.mongodb.com) when creating the App object in your client app.

Thanks for looking into this, @michael-wb.
RealmSwift.App doesn't have an initializer that allows specifying the URL, but I can deploy with v10.49.0 in the meantime.

You can configure the baseUrl in swift using AppConfiguration:
https://www.mongodb.com/docs/atlas/device-sdks/sdk/swift/app-services/connect-to-app-services-backend/#configuration

Thank you @michael-wb
For those of us not familiar with directly working with App Services, since the workaround is to create a custom AppConfiguration, what values do the other parameters need to be set to? From the docs
let configuration = AppConfiguration(
   baseURL: "https://services.cloud.mongodb.com", // point this to https://us-east-1.aws.realm.mongodb.com
   transport: nil, // Custom RLMNetworkTransportProtocol
   localAppName: "My App",
   localAppVersion: "3.14.159",
   defaultRequestTimeoutMS: 30000
)

let app = App(id: "my-app-services-app-id", configuration: configuration)

can they stay the same as in the docs per above? If not what should the values be?

Hi @Jaycyn
If you don't need to configure those values, then you can just not define them and they will use default values. e.g.:
let configuration = AppConfiguration(
   baseURL: "https://us-east-1.aws.realm.mongodb.com/"
)

let app = App(id: "my-app-services-app-id", configuration: configuration)

API reference: https://www.mongodb.com/docs/realm-sdks/swift/latest/Extensions/AppConfiguration.html

Original code which logs in and out correctly
let gTodoApp = App(id: Constants.REALM_APP_ID)
new code
let configuration = AppConfiguration(
   baseURL: "https://us-east-1.aws.realm.mongodb.com/"
)

let gTodoApp = App(id: Constants.REALM_APP_ID, configuration: configuration)

results in

Error: App: request location failed (MalformedJson): [json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - invalid literal; last read: '<'
Error: App: log_in_with_credentials failed: 0 message: [json.exception.parse_error.101] parse error at line 1, column 1: syntax error while parsing value - invalid literal; last read: '<'

I am sure I am overlooking something but it's not clear where I should begin looking since only one like of code was changed.

Oh, sorry - try removing the / from the end of baseUrl and see if that fixes it.

That corrected the issue and it appears to be working. Thank you for the super fast responses.

That corrected the issue and it appears to be working. Thank you for the super fast responses.

But there was some issue related to meta data version in earlier version.

You can configure the baseUrl in swift using AppConfiguration: https://www.mongodb.com/docs/atlas/device-sdks/sdk/swift/app-services/connect-to-app-services-backend/#configuration

Unfortunately, my app is using User/Password authentication and I have to use user.configuration() to get an AppConfiguration. There doesn't seem to be a way to set the baseUrl later (perhaps an oversight in the API?).

@andreasley Not sure the issue being described is clear.
We also use User/Password Authentication and generally, getting an AppConfiguration is unrelated to the login process.
This code is working for us
let configuration = AppConfiguration(
   baseURL: "https://us-east-1.aws.realm.mongodb.com"
)

let gTodoApp = App(id: Constants.REALM_APP_ID, configuration: configuration)

and then the login is something like this
@MainActor
func login() async throws -> User {
    let creds = Credentials.emailPassword(email: "some email", password: "some password)
    let user = try await gTodoApp.login(credentials: creds)
    print("successfully logged in user: \(user)")
    return user
}

@andreasley / @Jaycyn / @vishaldeshai / @anton-plebanovich / @KudeusVince
FYI - there was a server fix to address this failure that you were seeing.
Under certain circumstances, (e.g. starting a sync websocket session with a cached user) the client will connect to the "global" MongoDB realm endpoint "https://realm.mongodb.com", (or "https://services.cloud.mongodb.com" after the domain update), which would be translated to a localized server by the DNS based on your area. If the cloud app happens to reside on the localized server you connected to, everything was good. If not, then the server was responding with the WebSocket: Internal Server Error error and an arbitrary "error" message.
The server fix was to tell the client to connect to the correct localized server via a redirect response from the server instead of returning the error you were seeing.
As a result, your client app should no longer be required to specify the localized server base URL and it should "just work" without crashing.

@michael-wb I have tried with the latest 10.50.0 version and while I confirm it works the solution still seems to be a workaround. Are there plans to improve it in the future? It works fine on the first install but on the second run (probably due to a cached user?) I see the issue happening in the logs again and the first connection even retried it once before the second connection succeeded (third run).
First run
05.05 08:59:16.629 | R | [Info] Connection[1]: Session[1]: Binding ***
05.05 08:59:16.629 | R | [Info] Connection[1]: Session[1]: client_reset_config = false, Realm exists = true
05.05 08:59:16.630 | R | [Info] Connection[1]: Connecting to 'wss://eu-west-1.aws.ws.services.cloud.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 08:59:17.517 | R | [Detail] Connection[1]: Negotiated protocol version: 12
05.05 08:59:17.517 | R | [Info] Connection[1]: Connected to app services with request id: "6636f605aa360504a1c2d301"

Seconds run
05.05 09:01:01.654 | R | [Info] Connection[1]: Session[1]: Binding ***
05.05 09:01:01.654 | R | [Info] Connection[1]: Session[1]: client_reset_config = false, Realm exists = true
05.05 09:01:01.655 | R | [Info] Connection[1]: Connecting to 'wss://ws.services.cloud.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 09:01:02.434 | R | [Detail] Connection[1]: Negotiated protocol version: 12
05.05 09:01:02.434 | R | [Info] Connection[1]: Connected to app services with request id: "6636f66e7eac53ca6f1a93bc"
05.05 09:01:02.435 | R | [Info] Connection[1]: Closing the websocket with error code=WebSocket: Moved Permanently, message='moved permanently', was_clean=true
05.05 09:01:02.436 | R | [Detail] Connection[1]: Allowing reconnection in 803 milliseconds
05.05 09:01:03.176 | R | [Detail] Connection[1]: Canceling reconnect delay
05.05 09:01:03.176 | R | [Info] Connection[2]: Session[2]: Binding ***
05.05 09:01:03.176 | R | [Info] Connection[2]: Session[2]: client_reset_config = false, Realm exists = true
05.05 09:01:03.177 | R | [Info] Connection[2]: Connecting to 'wss://eu-west-1.aws.ws.services.cloud.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 09:01:04.074 | R | [Detail] Connection[2]: Negotiated protocol version: 12
05.05 09:01:04.075 | R | [Info] Connection[2]: Connected to app services with request id: "6636f670c7cd8566b84eb775"

Third run
05.05 09:01:40.776 | N | VPN connection required: false
05.05 09:01:41.019 | R | [Info] Connection[1]: Session[1]: Binding ***
05.05 09:01:41.020 | R | [Info] Connection[1]: Session[1]: client_reset_config = false, Realm exists = true
05.05 09:01:41.021 | R | [Info] Connection[1]: Connecting to 'wss://ws.services.cloud.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 09:01:41.522 | R | [Detail] Connection[1]: Negotiated protocol version: 12
05.05 09:01:41.522 | R | [Info] Connection[1]: Connected to app services with request id: "6636f6957eac53ca6f1c4c71"
05.05 09:01:41.523 | R | [Info] Connection[1]: Closing the websocket with error code=WebSocket: Moved Permanently, message='moved permanently', was_clean=true
05.05 09:01:41.524 | R | [Detail] Connection[1]: Allowing reconnection in 798 milliseconds
05.05 09:01:42.323 | R | [Info] Connection[1]: Connecting to 'wss://ws.services.cloud.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 09:01:42.331 | R | [Info] Connection[2]: Session[2]: Binding ***
05.05 09:01:42.331 | R | [Info] Connection[2]: Session[2]: client_reset_config = false, Realm exists = true
05.05 09:01:42.332 | R | [Info] Connection[2]: Connecting to 'wss://eu-west-1.aws.ws.services.cloud.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 09:01:42.812 | R | [Detail] Connection[1]: Negotiated protocol version: 12
05.05 09:01:42.812 | R | [Info] Connection[1]: Closing the websocket with error code=WebSocket: Moved Permanently, message='moved permanently', was_clean=true
05.05 09:01:42.940 | R | [Detail] Connection[2]: Negotiated protocol version: 12
05.05 09:01:42.940 | R | [Info] Connection[2]: Connected to app services with request id: "6636f697aa360504a1c448e3"

This is the intended behavior. The websocket connection is established against the default url, after which the server responds with the local url that should be used. Why do you believe this is a workaround that needs to be fixed?

@nirinchev Longer overall connection duration: 2-3 seconds for subsequent runs versus 1 second for the initial run in my case. As an optimization, it may preserve the returned URL instead of always trying the default URL and failing. Currently, the logic I have uses cached user to speed up application startup but with the new behavior it looks like login every time is actually faster startup flow which is counterintuitive and killing optimizations I have.
This is the old framework behavior which takes only 1 second to finish the connection without errors:
05.05 09:50:09.540 | R | [Info] Connection[1]: Session[1]: Binding ***
05.05 09:50:09.543 | R | [Info] Connection[1]: Session[1]: client_reset_config = false, Realm exists = true
05.05 09:50:09.543 | R | [Info] Connection[1]: Connecting to 'wss://ws.eu-west-1.aws.realm.mongodb.com:443/api/client/v2.0/app/dtsandboxanton-nmfsd/realm-sync'
05.05 09:50:10.480 | R | [Detail] Connection[1]: Negotiated protocol version: 11
05.05 09:50:10.480 | R | [Info] Connection[1]: Connected to app services with request id: "663701f2c7cd8566b872cc77"

Updating to the new framework version causes longer overall connection time and so is regression and update blocker from my point of view.

@anton-plebanovich - understood. If the sync time from app start is a priority/concern, the best method is to specify the localized server, since you will connect directly to the appropriate server on the first attempt.
We have some other work going on around the app services and sync components of the client which may help at some point in the future to reduce the sync connect on app start time, but it still won't be as fast as specifying the localized server.
FYI-If you ever decide to change the deployment models (server) in the future, any clients that haven't updated and still try to connect to the original server will be redirected to the appropriate one.

Thank you @michael-wb I verified the solution and it works for me 👍
Knowing it will work in a deployment model change case is also helpful. Thank you for the info 🤝


fullsync gives distributed not initialized error

🐛 Describe the bug
Hi, here we are training in webdataset format with torchdata. Everything else works fine without fullsync, just the training hangs at the last iteration. So I tried to apply fullsync to my datapipe, then it would give the following error:
RuntimeError: Caught RuntimeError in DataLoader worker process 0.
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/datapipes/_hook_iterator.py", line 173, in wrap_generator
    response = gen.send(None)
  File "/root/miniconda3/lib/python3.8/site-packages/torchdata/datapipes/iter/util/prefetch.py", line 173, in __iter__
    raise RuntimeError("Torch Distributed is required to be initialized")
RuntimeError: Torch Distributed is required to be initialized
This exception is thrown by __iter__ of FullSyncIterDataPipe()

My training scirpt:
def build_torchdata_loader(rootdir):
    import torch.distributed as ddist
    # both true
    print("dist: ", ddist.is_available(), ddist.is_initialized())
    assert ddist.is_initialized()

    tar_files = list(glob.glob('{}/*.tar'.format(rootdir)))

    train_dp = dp.iter.FileLister(tar_files).shuffle().sharding_filter()

    # the last iter hangs
    # train_dp = dp.iter.FileOpener(train_dp, mode="rb").load_from_tar().webdataset()
    # dist not init error
    train_dp = dp.iter.FileOpener(train_dp, mode="rb").load_from_tar().webdataset().fullsync()

    train_dp = train_dp.shuffle(buffer_size=128)
    train_dp = train_dp.map(postprocess_func)

    train_loader = DataLoader(train_dp, batch_size=128, num_workers=8, pin_memory=True)
    return train_loader

Also, I'm not sure if this is the correct way to use torchdata with dataLoader and ddp. I only managed to find the colab tutorial with dataloader2 ddp. I'm wondering if it is possible to also provide one for the original dataloader.
Many thanks in advance!
Versions
Versions of relevant libraries:
[pip3] numpy==1.23.5
[pip3] torch==1.13.1
[pip3] torchdata==0.5.1
[pip3] torchvision==0.14.1

It seems like there is an issue with import. If you run this, do you see an error?
from torchdata.datapipes.iter import IterableWrapper
dp = IterableWrapper(list(range(5)))
dp2 = dp.fullsync()

Hi @NivekT , the above scipt works fine. And I did a little digging.
This works and gives the correct results with
python -m torch.distributed.launch --nproc_per_node=2 --master_port 29100 fullsync.py

import torch
import torch.distributed as dist
import torch.multiprocessing as mp
from torchdata.datapipes.iter import IterableWrapper
from torch.utils.data import DataLoader

if __name__ == '__main__':
    dist.init_process_group(backend="nccl", init_method="env://")
    rank, world_size = dist.get_rank(), dist.get_world_size()

    dp = IterableWrapper(list(range(5))).sharding_filter()
    torch.utils.data.graph_settings.apply_sharding(dp, world_size, rank)
    dp = dp.fullsync()

    for _, x in enumerate(dp):
        print("rank={}, world_size={}, x={}".format(rank, world_size, x))

But this doesn't and gives the following error.
import torch
import torch.distributed as dist
import torch.multiprocessing as mp
from torchdata.datapipes.iter import IterableWrapper
from torch.utils.data import DataLoader

if __name__ == '__main__':
    dist.init_process_group(backend="nccl", init_method="env://")
    rank, world_size = dist.get_rank(), dist.get_world_size()

    dp = IterableWrapper(list(range(5))).sharding_filter()
    torch.utils.data.graph_settings.apply_sharding(dp, world_size, rank)
    dp = dp.fullsync()

    dl = DataLoader(dp, batch_size=2, num_workers=2, pin_memory=True)
    for _, x in enumerate(dl):
        print("rank={}, world_size={}, x={}".format(rank, world_size, x))

WARNING:torch.distributed.run:
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
Traceback (most recent call last):
  File "fullsync.py", line 20, in <module>
    for _, x in enumerate(dl):
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 628, in __next__
    data = self._next_data()
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1333, in _next_data
    return self._process_data(data)
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/dataloader.py", line 1359, in _process_data
    data.reraise()
  File "/root/miniconda3/lib/python3.8/site-packages/torch/_utils.py", line 543, in reraise
    raise exception
MemoryError: Caught MemoryError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 302, in _worker_loop
    data = fetcher.fetch(index)
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 34, in fetch
    data.append(next(self.dataset_iter))
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/datapipes/_hook_iterator.py", line 173, in wrap_generator
    response = gen.send(None)
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/datapipes/datapipe.py", line 351, in __iter__
    yield from self._datapipe
  File "/root/miniconda3/lib/python3.8/site-packages/torch/utils/data/datapipes/_hook_iterator.py", line 173, in wrap_generator
    response = gen.send(None)
  File "/root/miniconda3/lib/python3.8/site-packages/torchdata/datapipes/iter/util/prefetch.py", line 174, in __iter__
    self._process_group = dist.new_group(backend="gloo")
  File "/root/miniconda3/lib/python3.8/site-packages/torch/distributed/distributed_c10d.py", line 3335, in new_group
    pg = _new_process_group_helper(
  File "/root/miniconda3/lib/python3.8/site-packages/torch/distributed/distributed_c10d.py", line 862, in _new_process_group_helper
    pg = ProcessGroupGloo(prefix_store, group_rank, group_size, timeout=timeout)
MemoryError: std::bad_alloc
This exception is thrown by __iter__ of FullSyncIterDataPipe()

Switch to dataloader2 and it works fine. Closing this issue.

Let's keep this open as we should still take a look into old DataLoader to either support or explicitly disallow this use case.

The problem is fullsync is sent to worker process in DataLoader and worker processes are not aware of distributed processes. But, we keep it in the main process at the end when DataLoader2 is used.
We can keep this issue open but I am reluctant to fix it for DataLoader as it makes DataLoader even more complicated


Update publish script and remove the usage of the S3 bucket

Updating the publish.sh script to remove the usage of the S3 bucket
We will only publish on NPM moving forward

@doriskwan looks like we're still waiting on a CI report, which seems unlikely to happen without intervention. Probably some system failure with GitHub webhooks or CircleCI. Maybe try rebasing this PR against main and pushing? That's likely to trigger CI to run.


One directional barriers

On https://docs.microsoft.com/en-us/microsoft-365/compliance/information-barriers?view=o365-worldwide it is stated: "Information barriers only supports two way restrictions. One way restrictions, such as marketing can communicate with day traders, but day traders cannot communicate with marketing is not supported."
However the example of this page shows a one way restriction: Manufacturing should not be able to communicate to Sales and to Research. If the statement above is true, that should also require a two way restriction.

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

ID: 2baec804-4a76-95ac-3595-985629448fd9
Version Independent ID: c0a5fb9d-93be-b448-5bf2-4c3bdd787b8e
Content: Define information barrier policies - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/information-barriers-policies.md
Service: o365-seccomp
GitHub Login: @chrfox
Microsoft Alias: chrfox

Hi @thomasbinder Sales and Research cannot communicate with Manufacturing


question about  the convergence problem

I have run multi_agent.py,I set the iteration number of central_agent to 10^5 and set the learning rate as you proposed.And I use tensorboard to check the curve of td_loss.But the curve couldn't converge,the range is very wide.Could you  give me some guidance on convergence issue?I feel so appreciate for your help.

Did you see the reward improving in validation dataset? td_loss (policy gradient loss) has large variance can be due to many reasons; one important issue is from the stochasticity in the input process (network trace in this case). This paper provides more details about it: https://openreview.net/forum?id=Hyg1G2AqtQ, https://people.csail.mit.edu/hongzi/var-website/index.html

Thanks for reply.This paper will give me good instruction.I have read the question https://github.com/hongzimao/pensieve/issues/11.I think maybe the bad convergence performance is related to the problem that I didn't reduce the entropy weight to a small value after a certain number of iterations but keep it at 5. I will try this.I will also learn this paper.There is another question about the dataset.I want to know what's the trace dataset of  the pretrain_model in the code.Can I use the  synthetic trace as pretrain dataset?I  know the data in dropbox link is a subset,but I find I can't access it.And I also can't download the  FCC broadband dataset, Norway HSDPA bandwidth logs,Belgium 4G/LTE bandwidth logs (bonus)、homewifi dataset. Is it inaccessible now?

I think you can still download the data from that dropbox link. Synthetic data can be generated using https://github.com/hongzimao/pensieve/blob/master/sim/synthetic_traces.py


Unable to update Radarr directly, update the docker container to receive the update

Hi, using the :nightly tag gives me this every time.

I don't understand what the issue is here

I just started to use this container. And the linuxserver nightly version looks slightly different, where it shows green if I'm up to date, and it doesn't have that exclamation mark. I think I just got worried over nothing :)

The container doesn't rebuild nightly due to issues with overload the FFMpeg server so it might be a little behind but shouldn't be an issue. Just build the container

Excuse my ignorance but may I ask? Does *Arr firstly import/upload the file to the location and encode/convert it in that location? Or does it convert/encode it before it imports it?

After import

Is there something I can do for the script to fire off before It imports? I rather have it encoded pre-import for multiple reasons.

Yep see the readme
You have to configure post-downloader scripts (nzget, sab, etc) which will do the conversion before it's passed back to *arr
You'll need to set up SMA or find a container with it preinstalled though for those docker containers, the default 2 sma containers I maintain only have postSoanrr/postRadarr support. There are a few out there

I've been using sabs for over a year with good results, I started using radarr-sma just a couple of days ago, and I like the idea, especially of enjoying both Usenet and torrent conversions through one platform. Doing the post-downloader script just pushes me back to square one I guess ;D
How about I ask you this then. I currently have my config for radarr/sonarr-sma setup like this. Would you be kind and review it for me. What I'm trying to achieve is the following:
I want it to convert the media to MKV, even if the extension already is MKV. I want it to contain h264/x264 video format with AAC, both 2ch and 6ch audio, and keep the original audio source as well. Embed SRT or ASS subtitles as well as image subs like PGS. And if the original file already included (bypass-if-copying-all = True) what I just wrote I want it to bypass (process-same-extensions = True)
my autoProcess.ini
ffmpeg = /usr/local/bin/ffmpeg
ffprobe = /usr/local/bin/ffprobe
threads = 0
hwaccels =
hwaccel-decoders = h264_cuvid, mjpeg_cuvid, mpeg1_cuvid, mpeg2_cuvid, mpeg4_cuvid, vc1_cuvid, hevc_qsv$
hwdevices = vaapi:/dev/dri/renderD128
hwaccel-output-format = vaapi:vaapi
output-directory =
output-format = mkv
output-extension = mkv
temp-extension =
minimum-size = 0
ignored-extensions = nfo, ds_store, txt
copy-to =
move-to =
delete-original = True
sort-streams = True
process-same-extensions = True
bypass-if-copying-all = True
force-convert = False
post-process = False
wait-post-process = False
detailed-progress = False
opts-separator = ,
preopts =
postopts =
regex-directory-replace = [^\w\-_\. ]

[Permissions]
chmod = 0777
uid = 1000
gid = 1000

[Metadata]
relocate-moov = False
full-path-guess = True
tag = False
tag-language = eng
download-artwork = False
sanitize-disposition =
strip-metadata = True

[Video]
codec = h264, x264
max-bitrate = 0
bitrate-ratio =
crf = -1
crf-profiles =
preset =
codec-parameters =
dynamic-parameters = False
max-width = 0
profile =
max-level = 0.0
pix-fmt =
filter =
force-filter = False

[HDR]
codec =
pix-fmt =
space = bt2020nc
transfer = smpte2084
primaries = bt2020
preset =
codec-parameters =
filter =
force-filter = False
profile =

[Audio]
codec = aac
languages =
default-language =
first-stream-of-language = False
allow-language-relax = True
channel-bitrate = 0
max-bitrate = 0
max-channels = 0
prefer-more-channels = True
default-more-channels = True
filter =
force-filter = False
sample-rates =
copy-original = True
aac-adtstoasc = False
ignore-truehd = mp4, m4v
ignored-dispositions =
unique-dispositions = False

[Universal Audio]
codec = aac
channel-bitrate = 0
first-stream-only = False
move-after = False
filter =
force-filter = False

[Audio.ChannelFilters]
6-2 = pan=stereo|FL=0.5*FC+0.707*FL+0.707*BL+0.5*LFE|FR=0.5*FC+0.707*FR+0.707*BR+0.5*LFE

[Subtitle]
codec = srt, ass
codec-image-based = pgs
languages =
default-language =
first-stream-of-language = False
encoding =
burn-subtitles = False
burn-dispositions =
embed-subs = True
embed-image-subs = True
embed-only-internal-subs = False
filename-dispositions = forced
ignore-embedded-subs = False
ignored-dispositions =
unique-dispositions = False
attachment-codec = ```

Seems like what you have here should work. You'll likely end up with 3 channels for each audio stream though (aac stereo, aac multichannel, original) for any stream over 2 channels that's not already AAC
Also be cautious with this workflow that you don't process the same file multiple times, you may end up duplicating audio streams with each conversion

Thanks for your review. Yeah, I'm aware of the truplicate stream. I rather have process-same-extensions = False but then it would bypass all of it even if the audio source isn't AAC like I want it. I wish there was a way of telling it to process even if the audio stream doesn't match my "criteria" instead.

Btw. What does these mean?
post-process = False
wait-post-process = False

You can add on additional post processing python scripts into this SMA script suite to be run afterward
https://github.com/mdhiggins/sickbeard_mp4_automator#post-process-scripts
These settings apply to this feature

Thank you so much for your time explaining and guiding @mdhiggins. I appreciate your work! ⭐

No prob
I'm working on a https://github.com/linuxserver/docker-mods version of the SMA script so you might be able to integrate it into downloaders more easily

I'm looking forward to that 😃 👍

Early version is published and working in my prelim tests, still need to write up the documentation but try adding this to your environment variables on a linuxserver based container, should install sma and ffmpeg
DOCKER_MODS=mdhiggins/sma-mod:latest
https://hub.docker.com/repository/docker/mdhiggins/sma-mod

Awesome! Looking forward to it. I'll give it a test run in a few hours when it's morning.
On 10 Apr 2021, 01:01 +0200, Michael Higgins @.***>, wrote:

Early version is published and working in my prelim tests, still need to write up the documentation but try adding this to your environment variables on a linuxserver based container, should install sma and ffmpeg
DOCKER_MODS=mdhiggins/sma-mod:latest
https://hub.docker.com/repository/docker/mdhiggins/sma-mod
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

DOCKER_MODS=mdhiggins/sma-mod:latest

So I don't really need to deploy sma-mod as a docker? I just put it in env?

Correct, only for linxuserver containers though

Correct, only for linxuserver containers though

Great! My first move will be to deploy it within linuxserver/deluge, a client I never really got working before :)

That was the container I had done my testing on and it seems to be working nicely

That was the container I had done my testing on and it seems to be working nicely

@mdhiggins May I ask how you setup Deluge to perform the post process after torrent completion?
I'm looking for a way to keep seeding the original torrent. Any suggestions?

This is covered in the readme and sma-mod includes a delugePostProcess.sh that includes all the relative path information (instead of called delugePostProcess.py directly)
It automatically makes a copy and does not modify the original torrent files so that you can keep seeding. You'll need to remove the originals from deluge at some point on your own

https://github.com/mdhiggins/sickbeard_mp4_automator#deluge-daemon-setup

https://github.com/mdhiggins/sickbeard_mp4_automator#deluge-daemon-setup

Thanks! Does it work with extractor enabled?

Nope, you'd have to modify whatever extractor script you're using to call things

Nope, you'd have to modify whatever extractor script you're using to call things

Gotcha! May I ask which extractor you use? :)

Do you think I can do something like this?

Nope, the way delugePostProcess works is that it queries the API for a list of files to process which will no longer be accurate after you have extracted
You can set forcepath to bypass this but you'll need to call with the correct parameters
https://github.com/mdhiggins/sickbeard_mp4_automator/issues/1181#issuecomment-599158502

Nope, the way delugePostProcess works is that it queries the API for a list of files to process which will no longer be accurate after you have extracted
You can set forcepath to bypass this but you'll need to call with the correct parameters
mdhiggins/sickbeard_mp4_automator#1181 (comment)

I'm giving up on Deluge. Haven't gotten it to work for the past year, I really got my hopes up now lol. Does this script you pointed me to apply to qbittorrent as well?

https://github.com/mdhiggins/sickbeard_mp4_automator/wiki/qBittorrent-Unrar-Script
That should generally work for qbt with only minor modifications for the python path for the docker container

https://github.com/mdhiggins/sickbeard_mp4_automator/wiki/qBittorrent-Unrar-Script
That should generally work for qbt with only minor modifications for the python path for the docker container

Thanks. I give up. I don't know what I'm doing wrong but I don't seem to be able to get any client to work with your mod. And it's most likely my lack of skills and nothing wrong with your mod. Thanks for all your help though, I truly appreciate it!

If you can provide some information about how its failing I can perhaps help you fix it

If you can provide some information about how its failing I can perhaps help you fix it

Sorry to waste your time. I can't provide you any information, cause there are nothing in logs. The script doesn't execute.


[BUG] buffer_update: Buffer argument is not a valid buffer of any type.

Godot version: v4.1.1
Plugin version: v1.3.0-stable (c17f838)
Platform: Windows 10
Issue description
Maybe related to #32 or maybe to https://github.com/godotengine/godot/issues/68592 ?
Every time the MMI_container has more than one MuiltimeshInstance3D it throws following error on loading of the map:
E 0:00:03:0011   buffer_update: Buffer argument is not a valid buffer of any type.
  <C++ Error>    Method/function failed. Returning: ERR_INVALID_PARAMETER
  <C++ Source>   drivers/vulkan/rendering_device_vulkan.cpp:5858 @ buffer_update()

Err thrown in here
Steps to Reproduce

Add at least two plants
run the scene.

Minimal reproduction project
Funny enough is that I cannot reproduce this in any small project this only happens in my bigger one. And if I delete the second plant everything works as expected.

Fixed by [4d96dc7] as mentioned in #32.
Thank you for pointing out a relevant Godot issue.
If this error persists on your side, please continue discussion in #32.


Copy Feature Not Working

Environment

OS : Ubuntu 22
Extension version : 2.1.5

Describe the bug:
Copy Functionality is not working

Thank you for reporting this issue. Let me make sure this is tracked by our dev team.


Update & enable bot service app test with the new azure-mgmt-web module

The following change has disabled  test https://github.com/Azure/azure-cli/pull/7135 for bot app, please re-record & publish with the test enabled. See the PR for more details

@swagatmishra2007 please get on this issue at your convenience.

@panchagnula what's the status of this issue?

It is done.


Unit Tests for models

Issue Reference
This PR addresses the Issue : Fixes #
Summarize
Please, describe what the PR does, the changes you have made.

Also, mention the key points if any to look into while code reviews

Jitesh is working on fixing the tests

@biswaz, can you review this, I have made a clean PR from one of my old PR


subsys: logging: avoid potential undefined reference to log_*()

When CONFIG_LOG is not enabled, the LOG*() macros still expand to
log_0(), log_1(), etc. XCC complains about undefined references
to these functions during linking. So add some magic to avoid
LOG*() being expanded when CONFIG_LOG is not enabled.
Signed-off-by: Daniel Leung<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff            @@
##           master   #13640    +/-   ##
========================================
- Coverage   52.41%   51.42%    -1%
========================================
  Files         322      322
  Lines       46596    46583    -13
  Branches    10772    10770     -2
========================================
- Hits        24422    23953   -469
- Misses      17267    17837   +570
+ Partials     4907     4793   -114

Impacted Files
Coverage Δ

include/logging/log_core.h
100% <ø> (ø)
:arrow_up:

subsys/net/ip/tcp.c
19.83% <0%> (-36.12%)
:arrow_down:

subsys/net/lib/sockets/sockets.c
36.82% <0%> (-10.92%)
:arrow_down:

subsys/net/lib/config/init.c
54.92% <0%> (-6.72%)
:arrow_down:

subsys/net/ip/ipv4.c
62.6% <0%> (-4.88%)
:arrow_down:

include/net/net_pkt.h
83.64% <0%> (-3.74%)
:arrow_down:

subsys/net/ip/icmpv4.c
22.97% <0%> (-3.38%)
:arrow_down:

subsys/net/ip/net_context.c
56.04% <0%> (-2.83%)
:arrow_down:

subsys/net/lib/mqtt/mqtt_decoder.c
58.51% <0%> (-2.13%)
:arrow_down:

subsys/net/ip/connection.c
70.75% <0%> (-1.12%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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

@dcpleung that is quite strange as we rely on linker being able to ignore missing functions is compiler removes them (if(0) func_without_impl) not only in logger but also in shell. I wonder then if gcc 4.2 was not supporting that or there is some bug in that check (_LOG_CONST_LEVEL_CHECK(_level) &&		     (_level <= LOG_RUNTIME_FILTER(_filter)) that leads that linking error.
Will i be able to reproduce that? Is this XCC part of zephyr sdk?
I spent some time to ensure that #ifdefs are limited to minimum in the logger so I'm a bit reluctant to add this change unless it is necessary.

@nordic-krch XCC is from Tensilica, and I was building for the Intel S1000 CRB board. The information to obtain the tools is here: https://docs.zephyrproject.org/latest/boards/xtensa/intel_s1000_crb/doc/index.html.
The undefined references come from {{soc/xtensa/intel_s1000/soc.c}}. Here are the results from the Xtensa GCC from our SDK, and another from XCC.
soc_xcc.i.txt
soc_gcc.i.txt
I commented out a few LOG_DBG() lines so there were only 2 lines left. XCC complained about undefined references to log_0(), log_1() and log_2(). From what I can see in the files, both compilers produced the same .i lines for LOG_DBG(). So maybe XCC is too old to properly parse the file?

Unfortunately, i have no access to Tensilica SDK but can you try following experiment. Modify hello_world main.c as following:
extern void dummy_function(void);

void main(void)
{
	if (0) {
		dummy_function();
	}
	printk("Hello World! %s\n", CONFIG_BOARD);
}

and check if it compiles. That will confirm if XCC linker handles such situations. I still hope that it is the problem in if condition evaluation rather than linker limitation.

I wonder if that isn't caused by some missing linker flag.

XCC has no problem with dummy_function(). I wonder if XCC cannot deal with nested if inside __LOG().

@dcpleung that is somehow good. Can you run experiment where you replace parts of that if in __LOG with 0 to narrow down which condition is not resolved correctly?
Starting with replacing IS_ENABLED(CONFIG_LOG) with 0 in _LOG_CONST_LEVEL_CHECK. Once this is 0 everything else in that if statement shouldn't matter as there are &&.
#define _LOG_CONST_LEVEL_CHECK(_level)					    \
	(IS_ENABLED(CONFIG_LOG) &&					    \
	(								    \
	_LOG_LEVEL_CHECK(_level, CONFIG_LOG_OVERRIDE_LEVEL, LOG_LEVEL_NONE) \
	||								    \
	(!IS_ENABLED(CONFIG_LOG_OVERRIDE_LEVEL) &&			    \
	(_level <= __log_level) &&					    \
	(_level <= CONFIG_LOG_MAX_LEVEL)				    \
	)								    \
	))

if (_LOG_CONST_LEVEL_CHECK(_level) &&			    \
		    (_level <= LOG_RUNTIME_FILTER(_filter))) {

I tried replacing IS_ENABLED(CONFIG_LOG) with 0 in _LOG_CONST_LEVEL_CHECK, but there were still undefined references. I also tried replacing _LOG_CONST_LEVEL_CHECK() with 0 in __LOG() but it still failed. The linker worked only after I removed both __LOG_INTERNAL() inside __LOG().

i wonder if it is not a problem of compiling following loop:
if (0) {
   /* something */
} else if (0) {
  /* something */
}

maybe compiler somehow gets confused?
What if you try to split those if defs and change it to:
if (_LOG_CONST_LEVEL_CHECK(_level) &&			    \
    (_level <= LOG_RUNTIME_FILTER(_filter))) {		    \
	struct log_msg_ids src_level = {		    \
		.level = _level,			    \
		.domain_id = CONFIG_LOG_DOMAIN_ID,	    \
		.source_id = _id			    \
	};						    \
							    \
	if ((1 << _level) & LOG_FUNCTION_PREFIX_MASK) {	    \
		__LOG_INTERNAL(src_level,		    \
				_LOG_STR(__VA_ARGS__));	    \
	} else {					    \
		__LOG_INTERNAL(src_level, __VA_ARGS__);	    \
	}						    \
} \
\
if (0) {						    \
	/* Arguments checker present but never evaluated.*/ \
	/* Placed here to ensure that __VA_ARGS__ are*/     \
	/* evaluated once when log is enabled.*/	    \
	log_printf_arg_checker(__VA_ARGS__);		    \
}

I tried the split statement above and I still got the error. I also tried doing the if (0) {} else if (0) {} for the _LOG_INTERNAL() lines, and it still failed.

I wonder if it maybe something with this soc.c file. Is it only complaining there? That would be strange as there are probably more logs in the app.
Pobierz aplikację Outlook dla systemu iOShttps://aka.ms/o0ukef

Od: Daniel Leung<EMAIL_ADDRESS>Wysłano: wtorek, lutego 26, 2019 8:34 PM
Do: zephyrproject-rtos/zephyr
DW: Chruściński, Krzysztof; Mention
Temat: Re: [zephyrproject-rtos/zephyr] subsys: logging: avoid potential undefined reference to log_*() (#13640)
I tried the split statement above and I still got the error. I also tried doing the if (0) {} else if (0) {} for the _LOG_INTERNAL() lines, and it still failed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/zephyrproject-rtos/zephyr/pull/13640#issuecomment-467580552, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AQHeBSwR05bOzKcqgPwH-ZnJruFzJLrPks5vRYxDgaJpZM4bIIQq.

That's the only file, and only doing sanitycheck with tests/misc/test_build/debug.conf.

Seems like it is due to ccache on my local machine. So closing...


PerformInteraction returns INVALID_ID on 2nd invocation

Scenario:
When sending a PerformInteraction command to the TDK with a given set of choices, the behavior is normal on the first invocation of the PerformInteraction.  However, when rendering the choices for a second time, the onPerformInteractionResponse callback is invoked immediately, with a result code of "INVALID_ID"
Setup:

Define a list of Choices:

mProxyALM.createInteractionChoiceSet(choices,choiceSet1IndexValue,autoIncCorrId++);

Render PerformInteraction with the choices:

mProxyALM.performInteraction(initialPrompt,displayText,choiceSet1IndexValue,"help","" +
                    "screen has timed out",InteractionMode.MANUAL_ONLY,30000,autoIncCorrId++);

The PerformInteraction renders normally.  Select any choice.
Render the choice list again:

mProxyALM.performInteraction(initialPrompt,displayText,choiceSet1IndexValue,"help","" +
                    "screen has timed out",InteractionMode.MANUAL_ONLY,30000,autoIncCorrId++);

Expected result:  The PerformInteraction screen should render on the TDK with the earlier defined choices.
Actual result: No rendering is done.  onPerformInteractionResponse callback is invoked immediately, with a result code of INVALID_ID

I'm not able to reproduce this issue at least with the Android SPT, I tried choices with BOTH and MANUAL_ONLY interaction modes, they show up every time, and I always get call backs for the choices being selected.
@diosceles How many choices are in the choice set you're using?
@mrapitis Maybe there is something about how the performInteraction is being used in this example that I'm missing?

Justin, there are 34 choices in the actual scenario I am seeing this happen.

I have copied and pasted the code that was provided but I haven't had any luck reproducing the issue.  Are you certain that the choiceSet1IndexValue is not being modified before the 2nd call to performInteraction?

I have not seen any modification of the Choices vector.  Note that this issue is only reproducible when following step 3 above (selecting a choice).  That is to say, if you render the choices, but then back out or let the interaction time out, you will be able to re-render the choices without incident.  It is not until a choice is selected, and the second attempt to render the interaction is the behavior seen.

Please disregard and close this issue.  I have found the problem.  There was a case when a DeleteChoiceInteraction fired which was not tracked properly.  This caused the code to try and fire an interaction on a choice set which no longer existed.  My apologies for the inconvenience.


client.referenceTime?.now() returning nil

I was about to try this in my app, but when i try to run the sample app it returns nil whenever I try to get  client.referenceTime?.now(). Am I missing something. tried both cartage and pod both returning the same.

@rlabr how did you fix this ?


Final Milestone

The goats: @m1oojv @Jnjy @LHeng1 @tim-pipi

LGTM

LGTM 🚀

LGTM!!!!!!!!!!!!!!!!!!!!


Absolute path support on font loading

BEFORE YOU SUBMIT please read the following:

[x] I'm submitting a bug report
[ ] I'm submitting a feature request
[ ] I'm submitting a support request

Webpack version:
1.10.x / 2.x
Plugin version:
0.0.x
Current behavior:
Relative paths needed for font loading
From manifest.json
  "../fonts/OpenSans/OpenSans-Bold-webfont.eot": "fonts/OpenSans-Bold-webfont-1d9c7945c7bc7dd0909105119bfbc191.eot",
  "../fonts/OpenSans/OpenSans-Bold-webfont.svg": "images/OpenSans-Bold-webfont-93349923b5274a36ac93cb3168d09123.svg",
  "../fonts/OpenSans/OpenSans-Bold-webfont.ttf": "fonts/OpenSans-Bold-webfont-76cc6be5d8a231dc012fef4bdb86f79c.ttf",
  "../fonts/OpenSans/OpenSans-Bold-webfont.woff": "fonts/OpenSans-Bold-webfont-2e90d5152ce92858b62ba053c7b9d2cb.woff",
  "../fonts/OpenSans/OpenSans-Light-webfont.eot": "fonts/OpenSans-Light-webfont-09e00aa7622ece30a0f1e06b55f66c2a.eot",
  "../fonts/OpenSans/OpenSans-Light-webfont.svg": "images/OpenSans-Light-webfont-8f04ed9aeb2185499068d84842b95aa1.svg",
  "../fonts/OpenSans/OpenSans-Light-webfont.ttf": "fonts/OpenSans-Light-webfont-2e98fc3ce85f31f63010b706259cb604.ttf",
  "../fonts/OpenSans/OpenSans-Light-webfont.woff": "fonts/OpenSans-Light-webfont-45b47f3e9c7d74b80f5c6e0a3c513b23.woff",
  "../fonts/OpenSans/OpenSans-Regular-webfont.eot": "fonts/OpenSans-Regular-webfont-c4d82460ef260eb1589e73528cbfb257.eot",
  "../fonts/OpenSans/OpenSans-Regular-webfont.svg": "images/OpenSans-Regular-webfont-8185eb3059c46e4169ce107dfcf85950.svg",
  "../fonts/OpenSans/OpenSans-Regular-webfont.ttf": "fonts/OpenSans-Regular-webfont-488d5cc145299ba07b75495100419ee6.ttf",
  "../fonts/OpenSans/OpenSans-Regular-webfont.woff": "fonts/OpenSans-Regular-webfont-79515ad0788973c533405f7012dfeccd.woff",

Expected/desired behavior:
Absolute path support for font loading

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem along with a gist/jsbin of your webpack configuration.
Try to insert fonts on any rails/webpack project

What is the motivation / use case for changing the behavior?
It would be cool if it supported absolute paths

Language: all

Hi @tomislav-s!
Thanks for the report! I've looked into it and created an issue (#308) with the sass-loader. It looks like they don't respect the resolve.root option in Webpack config that should fix this.
We extract the user part of the manifest file directly from require, import, or in this case @import statements from your code.
There is a workaround. If you read thru the options documentation where you'll see the mapAssetPath function. There you can rewrite the erroneous items.
If you have any more questions please don't hesitate to ask.


[Feature] Support for const

Problem description
We would like to use constant values from the OpenAPI Schema
Why would it be useful?
It would clean up our generated type files and sdks by not creating extra enums and instead reference the constant literal directly.
Additional context
See slack thread

This is supported in the latest version of the CLI (0.18.3)


Add new library version for Linux

Distros with newer software have version 7 of ImageMagick and are unable to use this module. This fixes that.

Merged changes manually. Thanks 👍


Use correct CreateInherentDataProviders impl for manual seal

This uses the SlotTimestampProvider instead of InherentProvider from babe

If this didn't worked, please write a test so that this doesn't breaks again.

The only way to test this in substrate would be to run test-runner-example over a westend nightly snapshot, which we currently don't have. Otherwise, we can test this in polkadot, once this pr gets in: https://github.com/paritytech/polkadot/pull/2609

I don't know how Substrate is related to Westend. Nothing in Substrate is aware of Westend. There should be a way to test this stuff. What is with the substrate runtime?

The SlotTimestampProvider was built to allow manual-seal resume block authorship on babe-based chains, So that the timestamp inherent no longer tracks system time, but is incremented every time the inherent data is requested.
https://github.com/paritytech/substrate/blob/0411312f08f635a8d875cb4d3c6ac3411d3aaf96/client/consensus/manual-seal/src/consensus/babe.rs#L290-L298
So the only real way to ensure that it's loaded, is to run the test-runner-example on an existing db, in this case, wested,

Any other issues? @bkchr

The SlotTimestampProvider was built to allow manual-seal resume block authorship on babe-based chains, So that the timestamp inherent no longer tracks system time, but is incremented every time the inherent data is requested.
https://github.com/paritytech/substrate/blob/0411312f08f635a8d875cb4d3c6ac3411d3aaf96/client/consensus/manual-seal/src/consensus/babe.rs#L290-L298
So the only real way to ensure that it's loaded, is to run the test-runner-example on an existing db, in this case, westend.

You have this bin/node/test-runner* stuff. This is clearly not westend?

You have this bin/node/test-runner* stuff. This is clearly not westend

The chain spec could be trivially changed to westend once westend snaphots become available on CI

You have this bin/node/test-runner* stuff. This is clearly not westend

The chain spec could be changed to westend once westend snaphots become available on CI

This is the substrate node. This is not connected to westend in any way.

This is the substrate node. This is not connected to Westend in any way.

Yeah I know it's there for demo purposes, but once CI snapshots become available we can make the switch to westend by changing the chainspec.


Xlink:href svg disappear bug in IE11 and below

Vue.js version
1.0.26
Reproduction Link
http://vue-xlink-ie-bug.handcraft.com/index.html
Steps to reproduce
Load the page. As soon as vue is activated, the arrow-down svg icon inside the <button> disappears on IE11 and below.
What is Expected?
I expect the icon to remain. This behaviour is correct in browsers such as chrome, edge or firefox.
What is actually happening?
The icon is a xlink:href svg icon that vue is trying te re-render, which for some reason fails.
I know of an ugly workaround, which is:
<use xlink:href="#icon-angle-down" v-bind="{ 'xlink:href': '#icon-angle-down' }"></use>

However, this should not be required to render xlink:href svg's.
Source:

        <title>sdffsdfsd</title>

      {{counter}}

      <button>
         hallo
         <svg><use xlink:href="#icon-angle-down"></use></svg>
      </button>

    <svg style="display: none;">
      <symbol viewBox="0 0 1792 1792" id="icon-angle-down"><title>angle-down</title><path d="M1395 736q0 13-10 23l-466 466q-10 10-23 10t-23-10L407 759q-10-10-10-23t10-23l50-50q10-10 23-10t23 10l393 393 393-393q10-10 23-10t23 10l50 50q10 10 10 23z"></path></symbol>
    </svg>

Your repro doesn't show the source code.

I've attached the source.

<use> is very buggy in Internet Explorer. If vue is detaching elements and moving them around in the DOM, all <use> svgs that are children of the moved elements will be lost, see here for example:
http://codepen.io/anon/pen/NAgRVm?editors=1010
An element with an svg icon is simply moved to another, using appendChild. In IE the icon disappears where as in other browsers it remains. This is not something vue could do anything about I'd imagine.
One 'solution' for you it to use a string template or script tag template, but that will only solve your current issue. There's many times, e.g. list rendering, where vue will use appendChild meaning you will face the same issue.
Ideally you would have an <icon name="icon-angle-down"> component which injects the full svg code of a given icon instead of using <use>. This will work much better in IE.

Close (5 days inactivity)

@fergaldoyle almost 2 years later, one of the greatest wtf IE moments of my dev path.

We were able to fix this by using the v-pre tag, if it is of any help. Just use it on either the SVG or any of its parent elements.

Neither of these worked for me. My icons still disappear. Everything I'm reading online says that if you have a symbol based SVG system, and are referencing nodes that exist on the page (not external), <use> should work fine in IE11. Yet, even with both :xlink:href and :href (I also tried v-pre and v-once) I'm not able to make it work. I'm going to make a demo with and without Vue and see if I can get a reproducible test case together to demonstrate the issue...

After a riveting day of SVG fun, this isn't a Vue issue.
If you notice any weirdness with your icons, make a test html page and work with that. Embed the svg icon in HTML without Vue to make sure it works fine, then add your loaders and only do it with javascript to see how your loaders change the HTML markup. Only after this should you then finally add your InlineSVG.vue component (or equivalent)
TLDR: Ultimately, what I went with was updating my svgs not to have <use> elements. We simplified them so that they only have one compound path.
I learned about compound paths when giving up and trying to use a font icon building site. They wouldn't convert my icons unless they were in something called a "compound path". So I figured out what that was and figured out how to convert them to svgs that were defined in one path element.
This solution obviously doesn't work if you want to style individual bits of the SVG, but for an icon font, it worked pretty well.
What about styling individual parts of the SVG?
If you want to make some cool SVG, whose parts can be individually styled... then, inline the paths in its own MySpecialSvg.vue component, and make the component do the reusability via :style and :class on the individual paths.
My main caution is: stay away from <use>. If you have masks that depend on <use>, change the icon in Illustrator or sketch.
Tools/Solutions
Inline SVGs, after removing usages of  or  = vue-svg-loader
*Inline SVGs with <use> and sprite sheets = svg-sprite-loader + svgo-loader + svg4everybody
Other stuff
Honorable mention for if you're trying to get sprite sheets to work with href="sprite.svg#myCoolSprite" or xlink:href="sprites.svg#myCoolSprite" is to use a tool called [svg4everybody](https://css-tricks.com/svg-use-with-external-reference-take-2/) to allow me to just use :hrefand:xlink:href`. This works magically under the hood. (It uses XHR to request the sprite sheet externally and then creates a document context).

I couldn't figure out a way to get around the CORS restrictions of sprite sheets, so I couldn't do this. (We're a plugin, we don't have our own domain, so we get blocked prod 😅)

+1 for svg4everybody, it's what enabled us the IE11 compat.


Add a SSH key validation

Description
A clear description of the bug is:
There's no SSH key format validation.

Must include any relevant identifiers, like:

Network: Testnet

Version: 2.3.0-rc3

Issue update
Work done
added a function to validate if the entered ssh-key is valid to not in https://github.com/threefoldtech/tfgrid-sdk-ts/pull/2419

version : 385928e
network : devnet
verified a ssh key validation is now added

https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=430164#case_4247837


Reaction: DB query

Perform a remote query against a DB

Guide for creating a new reaction: https://runbook.readthedocs.org/en/master/creating_reactions/