AddCustomerMatchUserList

Describe the bug:
The class keeps timing out before the customer match list is created. Even if I change the following settings, it still never creates the list:
private const int POLL_FREQUENCY_SECONDS = 30;
private const int MAX_TOTAL_POLL_INTERVAL_SECONDS = 1600;
This is when running V8/AddCustomerMatchList.cs
Steps to Reproduce:
I have changed the following settings as well as increasing the numbers of users been created
private const int POLL_FREQUENCY_SECONDS = 30;
private const int MAX_TOTAL_POLL_INTERVAL_SECONDS = 1600;
Expected behavior:
I am expecting the User Match list to be created
Client library version and API version:
I am using V8 of the examples downloaded from github.
The example been used is:
Client library version:V8
Google Ads API version:
.NET version:4.7.2
Installed: Google.Ads.Common V9.5.1, Google.Ads.GoogleAds V10.1.0, Google.Apis.Oauth2.v2 V1.55..0.1869
Operating system (Linux, Windows, ...) and version (if the bug is platform-specific): Windows 10
Request/Response Logs:
Log files have been uploaded to:
https://www.dropbox.com/sh/wiqv1gjzd4lq626/AACAEj6Z1v4JNP2yJ7McQpeYa?dl=0

Anything else we should know about your project / environment
Then on a side note. If I try to use the code with VB.NET. The following line will not run. This is due to the fact that Operations is a read only value. It seems to work in C#. But not in VB.NET. This is in class OfflineUserDataJobService.g.cs. But I think this should be corrected so that this class can be used within other languages too.
Dim request As AddOfflineUserDataJobOperationsRequest = New AddOfflineUserDataJobOperationsRequest() With {
.ResourceName = offlineUserDataJobResourceName,
.Operations = {BuildOfflineUserDataJobOperations()},
.EnablePartialFailure = True}

from your log: Error:"invalid_request", Description:"Missing required parameter: refresh_token"
Seems like you didn't enter a refresh token in your client. Did you set your OAuth2RefreshToken in the GoogleAdsClient object?

Hi,
Is this issue still present? if yes, could you reopen with a request ID attached? Logging instructions are available at https://developers.google.com/google-ads/api/docs/client-libs/dotnet/logging.


PR: Update recipe to not build for python <=3.5

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [14]
noarch packages can't have selectors. If the selectors are necessary, please remove noarch: python.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

noarch packages can't have selectors. If the selectors are necessary, please remove noarch: python.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

For this I followed what is done in conda/conda#5971 and the docs: https://conda-forge.org/docs/maintainer/knowledge_base.html#noarch-python
Fixes spyder-ide/spyder#13605
FIxes davidhalter/parso#149
pinging @ccordoba12 and @isidentical just in case

Looks good to me. Thanks @dalthviz!

@synapticarbors @iron0012 Can this be merged asap?


M3-1694 Sidebar now displays regardless of user level...

Notes
This issue was actually a result of a condition checking 'managed', not the settings themselves.

Split PrimaryNav into smaller components.
Added user grants request to profile request flow.
Used grants to hide/show account options in navigation bar.,
Corrected issue with domain record types.

Changelog
change: PrimaryNav broken down into smaller components.
add: AccountNavigation component (extracted from PrimaryNav)
add: ThemeToggle component (extracted from PrimaryNav)
add: Service to get the current user's grants.
add: The current user's grants are now requested and added to the Redux store at __resoureces.profile.grants.
Type Changes
change: Profile type now includes grants.
change: Record renamed DomainRecord to prevent collision with native Record type.
change: Used native Record type for Grants.
add: GrantTypes added to provide key types for Grants.

Coverage increased (+0.04%) to 28.105% when pulling b924bc8874af45afeca6287521847581208f6b65 on rjstires:M3-1694 into 03897aa3f837125138a32e5325a96ff53e4a1e91 on linode:develop.

In testing this, it seems the unrestricted user can no longer view the "Account" side bar menu item

Thank you @patthiel and @Jskobos for the review and feedback. Its updated, rebased, and ready!

Rebased again.


Getting with spam on bungee console warnings

What is the output url of /viaversion dump?

ViaBackWards: Latest (2.3.1)
ViaVersion: Latest (1.6.0)

How/when does this error happen? login?:

I don't know

Is there an error in the console? Use pastebin.com. Is there a kick message?:

https://pastebin.com/YFsWz27q

One of your plugins is sending wrong metadata. This could be because a plugin is outdated, or just sends wrong data without reason.
I’d recommend debugging your plugins by removing them one by one until it no longer happens to find out which one is causing this. You can also ignore this warning by changing ‘suppress-metadata-errors’ to ‘true’ in the ViaVersion config.


Abort workflows can leave jobs running

If jobs are in "complete" state by the time the WorkQueue calls the Jobs.killWorkflow DAO, those jobs will escape the killing. If they are to succeed they will move to succeed and trigger merge jobs, creating a dataset on the aborted workflow. If they fail, they will stay in the agent, retry as normal, if eventually succeed then also trigger merge jobs and create a bad dataset.
I think complete state should be included among the jobs to be killed when aborting the workflow, otherwise it can waste resources and create unwanted data. I haven't looked well enough the abort code but I don't see why not, please correct me if I'm wrong.
The case when this actually happens is very rare anyway.

We have made several changes lately to this code and it should not happen anymore. Besides it's quite old :-) @ticoann close this one please


[FEATURE] Support read /sys/bus/event_source/devices/uprobe/type on old kernel versions

Libbpf will try to read /sys/bus/event_source/devices/uprobe/type when creating uprobe perf event. This file doesn't exist on kernel that doesn't support uprobe. But since userspace uprobe doesn't rely on kernel features, we should make it support such kernel versions.
Possible ways:

Hook function open and read, so that make libbpf think the file exists
Put that file in docker image to trick libbpf that the file exists. Possible problem: Do we need to mount /sys from the host so that we can use syscall tracepoint?

In docker, you can mount a regular dir into /sys/bus/event_source/devices/uprobe/

So in order to run bpftime on old kernel version, you can simply copy the content of  /sys/bus/event_source/devices/uprobe/ on new kernel, put it on old kernel, and mount it with docker.

Another approach is hook open function with LD_PRELOAD in syscall-server.so, and prepare a directory with the same content as /sys/bus/event_source/devices/uprobe in ~/.bpftime/event_source/devices/uprobe. When the libbpf tries to read something in /sys/bus/event_source/devices/uprobe but it's not exists, we can change the open file to make it actually read contents in  ~/.bpftime/event_source/devices/uprobe.

Related to #145

Please modify code in https://github.com/eunomia-bpf/bpftime/tree/master/runtime/syscall-server

hi @Officeyutong  @yunwei37  would like to take a crack at this, could you please assign this to me


Improve get_quanta_attention

The function get_quanta_attention generates the cell contents for the Attention quanta map.
Sometimes two attention heads (cells) work as a pair, but each has it own attention pattern. Sometimes those attention patterns are similar but not identical:

P18L0H1 attends D2=38%, Attends D'2=37% so get_quanta_attention generates "D2 D'2"
P18L0H12 attends D'2=36%, Attends D2=34% so get_quanta_attention generates "D'2 D2"

This difference in output means the "combine identical cells" code does not combine the two cells.
This ticket covers  improving the text generated by get_quanta_attention as follows: If the two first attention percentages are within 5% of each other (say 38% and 34%) then the two tags (say D2 and D'2) are considered equivalent and are ordered alphabetically.
If both of the two first attention percentages are over 40% (say 53% and 42%)  then the two tags (say D2 and D'2) are considered equivalent and are ordered alphabetically.
Benefit: The two cells will generate the same text (ignoring small differences in %s), the  "combine identical cells" code will combine the two cells, simplifying the quanta maps, and making them easier to interpret.

Done. Refer ATTN_ORDER_DIFF and ATTN_ORDER_MIN


Wayland: support the xdg-decoration protocol to use server-side decorations when available

Use case
A user running a Flutter app on Sway, River, Kwin (KDE or lxqt), etc. probably prefers server-side decorations (SSD). Wayland mandates support for client-side decorations (CSD) but just about every Wayland compositor besides GNOME also supports SSD.
When a Wayland compositor supports SSD, users would probably rather use those.
Proposal
The optional xdg-decoration extension to the Wayland protocols allow programs to figure out whether the Wayland compositor supports server-side decorations. This is used by programs such as mpv (a minimal media player) to determine whether or not to draw their own titlebars.
If Flutter detects SSD support via the xdg-decoration protocol, it should use that to delegate window decorations to the compositor of drawing its own GDK titlebars that look out-of-place outside GNOME.
Screenshots of the status quo
These screenshots were taken on Sway, a Wayland compositor that supports tiling, tabbed, and floating layouts.

Fluffychat using a CSD headerbar alongside other  programs using SSD

A Fluffychat window in tiled layout showing both CSD and SSD

Edited to use a better screenshot

This could be considered a follow-up to https://github.com/flutter/flutter/pull/68452

As a KDE plasma user, this makes a lot of sense because flutter apps would look inconsistent with the system theme when using CSD title bar.


refactor(allocator): make Box::new_in code more explicit

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#4432  👈
#4431
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


Flaking Logging integration tests

How to categorize this issue?

/area logging
/kind bug
/priority normal
What happened:
Logging integration tests are failing.
Possible reasons:

Loki is restarted during the test run.
OutOfOrder entries
Slow Loki disk attach/detach
...

What you expected to happen:
Make integration tests stable
Anything else we need to know?:

/assign @vlvasilev

Info for the progress:

The tenant tests are now stable after the new changes.
The load test fails only on suse-chost. There the fluent-bit crashes on init phase(still investigating...)

/close
Tests are fixed


Import dependencies lazily

Describe the bug
Import huge dependencies like openpyxl from tablib is causing a huge import time taken on startup unnecessarily.
To Reproduce
Steps to reproduce the behavior:

In sample project, run python -X importtime ./manage.py check 2> import.log
View it with tuna https://pypi.org/project/tuna/
Django import export is importing openpyxl (we used tablib[all]) which causes a huge chunk of time spent

Versions (please complete the following information):

Django Import Export: 3.2.0
Python 3.11.5
Django 4.1.10

Expected behavior
Importing formats should be done lazily and not during application startup.
Screenshots
Taken from our project, django-import-export is taking up the most time during startup.

Additional context

Hey! Any idea how to solve this?
Also, Im not sure this is an issue with Django import export. It seems to be an openpyxl issue since the other dependencies are not taking that much time.  If the other dependencies were equally slow, that would be another story.
Having said that, it might be beneficial to allow users to import the depdendencies themselves by overriding the admin class where it is used.

Thanks for raising.  I'm interested to know the context where this is an issue.  It seems from the graph that the time to load these dependencies is <200ms.  Do you have a requirement to shorten this startup time?
Do you have any suggestion as to how we could solve this, or what would be the best outcome for you.
See also #1459 #1069

I'm interested to know the context where this is an issue

Wow odd timing, I was just looking at why my celery beat instance uses so much ram and came here. Celery calls django.setup() which eventually runs this and adds 12mb.
A workaround for non web apps loading django might be to set a environment variable and conditionally add INSTALLED_APPS like import-export. Maybe there is a way to change imports here to make it just never happen. If I come up with a reasonable solution, I will post again. Right now I'm learning towards just tweaking INSTALLED_APPS as needed for the service type.

After #1647, dependencies can be selected at library installation


[feature] implement stop command

Currently the only option to stop the containers is with the down command, which removes _everything_related to Pygmy. While this is nice for cleanup, it is excessive. The Pygmy gem utilizes a stop command, which simply stops the containers. This command should be added to retain parity with the gem.

Prioritizing this one - but similar was noted in #271

Some questions about expected behavior:

should the networks attempt to be removed when running this command?
should the resolv configuration be removed when running this command?

The current behavior in the gem version is to stop the containers (but not remove) and clean up the resolver. The network stays in place for stop

There's now a test case which replicates the behavior in #271.
Once the issue is fixed and the test case passes this will be good for merging.


Use allowlisted-extension-id instead whitelisted-extension-id

Since October 2021, Chrome's whitelisted-extension-id option has been deprecated.
It was replaced by allowlisted-extension-id.
With the existing key, the latest chromium does not run the Puppeteer with below messages.
[34622:259:0328/153354.796775:FATAL:chrome_content_browser_client.cc(2903)] "whitelisted-extension-id" switch is deprecated, please use "allowlisted-extension-id" instead

ref:

https://source.chromium.org/chromium/chromium/src/+/main:chrome/browser/chrome_content_browser_client.cc;l=737?q=whitelisted-extension-id&sq=&ss=chromium
https://source.chromium.org/chromium/chromium/src/+/main:extensions/common/switches.cc;l=22?q=whitelisted-extension-id&ss=chromium

Thank you very much for your contribution.
I've published your PR changes in the latest version:
npm i puppeteer-stream@latest


Change data source for tribal boundaries to the BIA_AIAN_LAR_National dataset

We need to use this data set to visualize tribal boundaries on the CEJST: https://biamaps.doi.gov/bogs/datadownload.html

[x] Vim to provide some transparency on what is in this data set ( columns )

@BethMattern, @KameronKerger @katherinedm-usds
Downloaded data from above link
The link mentioned in this ticket's description allows us to download a shapefile, eg, BIA_National_LAR. This shapefile has 3 shapefiles embedded in them, namely:

BIA_National_LAR
BIA_AIAN_Supplemental
BIA_TSA

The following text file explains these datasets in more detail (download to view)
Read Me AIAN LAR Vr9.txt
What data fields exist in each data set?
BIA_National_LAR:

BIA_AIAN_Supplemental
seems like none
BIA_TSA

How to view this on the map?
Comparing this tribal land data with the previous tribal lands data
New tribal layers (this ticket):

BIA_National_LAR (red fill)
BIA_AIAN_Supplemental (yellow fill)
BIA_TSA (green fill)

Existing tribal layers:
4. AIANNH (green border - original data)
5. Alaska Native (dark blue border - original data)
These 3 sets have been added to a map here for comparison

Viewing all 5 layers with census tracts
https://screeningtool.geoplatform.gov/en/?flags=tl#5.03/34.13/-103.35
Open to discuss at anytime!

OK @vim-usds and @esfoobar-usds - LAR_Name it is. this will be the data we should see in the side panel and download file for now. We can continue to look for ways to supplement this in the future.

@vim-usds here's the alaska native villages fil
AlaskaNativeVillages.gdb.zip
e

@katherinedm-usds and @BethMattern
here is the map with the AlaskaNativeVillages added:
https://screeningtool.geoplatform.gov/en/?flags=tl#3/33.47/-97.5
They are the pink dots in Alaska

@katherinedm-usds and @BethMattern
here is the map with the AlaskaNativeVillages added:
https://screeningtool.geoplatform.gov/en/?flags=tl#3/33.47/-97.5
They are the pink dots in Alaska

@BethMattern @vim-usds -- can you confirm that the BIA_TSA (green in the mockup) area is also supposed to be on the map?

@BethMattern confirms that they should all three be combined into one single layer and all displayed on the map.

Thanks for the clarification @lucasmbrown-usds and @BethMattern!

Adding datafile here:
BIA_National_LAR.zip


1.20.1 does not generate config (may apply to other versions

Issue type:

:bug: Bug

Short description:
Integrated scripting despite having configs semingly implemented. does nothing an never creates a config file
Steps to reproduce the problem:

install integrated scripting + dependicies
make a world
check config and serverconfig
realize there is no integratedscripting.toml file (or any config for integrated scripting)

Expected behaviour:
the config to exists and be able to be changed

Versions:

This mod: Integrated Scriptin 1.0.3
Minecraft: 1.20.1
Forge: 47.2.32

Thanks for reporting!


[BUG] Nav pane on doc site needs to be updated

Describe the bug
The nav pane on the documentation site needs to be updated to match the marketing site.
To Reproduce
Doc site:

Marketing site:

Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Plugins
Please list all plugins currently enabled.
Screenshots
If applicable, add screenshots to help explain your problem.
Host/Environment (please complete the following information):

OS: [e.g. iOS]
Version: [e.g. 22]
Browsers: [e.g. Chrome, Safari]
Browser Version: [e.g 93.0.4521.12]

Additional context
Add any other context about the problem here.

@nateynateynate - Do you know where this file lives? If so, I'm happy to make the updates.

Hi folks, gentle nudge to update the nav pane, or let me know where the code lives and I'll create the PR. Thanks! @nateynateynate @krisfreedain


Update to Requirements section due to customer confusion

Customer have expressed confusion with the statement, we should update to call out specifically that AAD Connect requires connectivity to ALL domain in a forest when configuring.
The Azure AD Connect server needs DNS resolution for both intranet and internet. The DNS server must be able to resolve names both to your on-premises Active Directory and the Azure AD endpoints.

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

ID: 06faa428-c7b6-ca5c-271e-3f98d9536d4f
Version Independent ID: 63adb39a-c4fa-92bb-480d-59bee8e2dc24
Content: Azure AD Connect: Prerequisites and hardware
Content Source: articles/active-directory/hybrid/how-to-connect-install-prerequisites.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@davidspooner , Thanks for your feedback! We will investigate and update as appropriate.

Thank for submitting this. I have created a work item for it and will reviewing this in the coming weeks. I am not sure what the timeline will be, so I am going to close this for now. But once I have an update I will drop it in here and let you know.
Thank you!
Bill
#please-close


Fix bug introduced in a recent commit

This is introduced in https://github.com/caffe2/caffe2/commit/8539a1e78b7fd8aa92b08a8f24b2b0f7978bf5a5 - vector should not be used in Tensor shape inference.

@akyrola since you are the reviewer for the original commit, would you mind taking a look?

oops


Is the architecture computing a bunch of values that it will never use?

Hi! I've been trying to understand the network and there is this matter that I can't understand. I posted it as a question in stackexchange. Feel free to answer over there or here if you feel like you understand my question and would like to contribute. Thanks in advance!

I am also trying to figure this out. My guess at the moment is that after the dilated stack you only need the last value in the sequence, as that is going to be the T+1. In the dilated stack they keep the same dimension as after dilation they start again with full values. Have you figured it out already?

Well if you look at the documentation in the code (https://github.com/ibab/tensorflow-wavenet/blob/e11ad19dccd6a33a182c9d1dea07aa53b9acca55/wavenet/model.py#L245) it does explicitly say that they use two different 1x1 convolutions. One goes to the skip connection and the other one to the next dilated layer. Then, not every layer would need the same amount of input and outputs and my original question is answered: The graph on the paper is (probably) misleading. And I'm saying probably because the implementation here is not an official one.

So in:
https://github.com/ibab/tensorflow-wavenet/blob/3c973c038c8c2c20fef0039f111cb04139ff594b/wavenet/model.py#L333-L336
we see that the input_batch is sliced prior to summing it with the transformed data. In this way, the residual connection carries only half of the values forward, the last half. Which is weird. Any ideas why this is done like that? I couldn't find it on the paper


Improved Docs

Coverage increased (+0.3%) to 22.652% when pulling 0f336f4088058efe619c90d4aaff6fcbc66eae12 on Royalsolanki:master into 3d3fea0e80422864a22b751fdedfbb443288baff on sarxos:master.

https://blog.domenic.me/hacktoberfest/


Extend "System.Console"

The JS "console" object provides much useful functionality which is not currently available in mscorlib. Namely it is possible print objects instead of just strings which could be very useful for debugging. Currently I'm using the following additional methods:
[InlineCode("console.log({obj})")]
public static void Log(object obj)
{
}
    [InlineCode("console.log({*objects})")]
    public static void Log(params object[] objects)
    {
    }

    [InlineCode("console.info({obj})")]
    public static void Info(object obj)
    {
    }

    [InlineCode("console.info({*objects})")]
    public static void Info(params object[] objects)
    {
    }

    [InlineCode("console.warn({obj})")]
    public static void Warn(object obj)
    {
    }

    [InlineCode("console.warn({*objects})")]
    public static void Warn(params object[] objects)
    {
    }

    [InlineCode("console.error({obj})")]
    public static void Error(object obj)
    {
    }

    [InlineCode("console.error({*objects})")]
    public static void Error(params object[] objects)
    {
    }

    [InlineCode("console.trace()")]
    public static void PrintStackTrace()
    {
    }

There are also some other methods that could be useful like "group()"/"groupEnd()" and "time()"/"timeEnd()".

There is also the Saltarelle library Firebug which implements Firebug's console API and has some more methods. Check this repo's Wiki.

This Firebug library seems like a good solution. The formatting overload methods can cause some confusion though since they are unrelated to the "String.Format" method C# developers are used to.
Still - why us it necessary to use an external class instead of adding a few methods to the built-in one? At the very minimum an overload that accept an object as an argument is necessary as having just a method that accepts a string is very limiting. Most browsers provide means to interactively inspect an object when it is printed to the console.

@JechoJekov yes best is to include it in System.Console instead of having a separate class. My intention was to extend by the use of extension methods but, to my surprise, in C# static classes can't have extension methods.


softether service running in the background and uploads data

why softether keeps using bandwidth and uploading some data while  i haven't even connected to any softether server!? and it uses quite high bandwidth!(1-2 Mb/s)
what data it uploads and why?!
i'm using a network monitor app and the upload bandwidth usage immediately drops when i stop softether service.

I have no idea what you are talking about.

@paymanz , network monitor (or similar app) should also provide traffic details, such as source, destination, port numbers, etc.
without those details it is hard to make a guess for us.

@paymanz , network monitor (or similar app) should also provide traffic details, such as source, destination, port numbers, etc. without those details it is hard to make a guess for us.

I have no idea what you are talking about.

ok i will try to add more information regarding source and destination , but please take a look at this video i recorded , i just started the softether service a few minutes before recording:  https://i.imgur.com/0odtIWQ.gif
most of uploads destination is this ip <IP_ADDRESS>

can you please try Fiddler or FiddlerCap (no installation required). It allows capture https traffic. I really wonder what those queries are.
also, please notice that you are using 4.x version which is SoftEtherVPN_Stable (this repo is about 5.x)

can you please try Fiddler or FiddlerCap (no installation required). It allows capture https traffic. I really wonder what those queries are.
also, please notice that you are using 4.x version which is SoftEtherVPN_Stable (this repo is about 5.x)

thank you for respond , i am not a pro , tried but couldn't make fiddler work , i tried wireshark and captured some packets that softether sent(filtered by destination).  if this helps please tell me to upload more files like this with other destinations.
https://file.io/X5GES0sJkvYE
and where can i post for 4.x version?

I tried the following filters in attached pcap
ip.addr == <IP_ADDRESS>
ip.addr == <IP_ADDRESS>
ip.addr == <IP_ADDRESS>
no packets.
as for 4.x support, there's link on www.softether.org website:


[Feature]: Support <port> placeholder in dev.assetPrefix

What problem does this feature solve?
We want to specify a custom dev.assetPrefix with hostname and port. But when the port is occupied, Rsbuild will automatically bump up the port by default, making the dev.assetPrefix incorrect.
It would be great if the dev.assetPrefix supports using the <port> placeholder.
BTW, I'm not sure if output.assetPrefix needs this.
What does the proposed API look like?
The same as dev.client.port and server.open:
// rsbuild.config.js
export default {
  dev: {
    assetPrefix: 'http://hostname:<port>/',
    client: {
      hostname: 'hostname',
      port: '<port>',
    },
  },
  server: {
    open: 'http://hostname:<port>/',
  },
}

PR welcome 😄


Add Prettier Code Formatting

Can we add prettier.rc file to the project to enable a consistent code formatting for everyone developing?
Read more at: https://prettier.io/

Hey @amm98d sounds like a good idea - feel free to take on the issue :)


9.10.0 Compile Error

Bug
In xcode, building a debug release for the simulator with react-native-svg 9.10.0 I am getting the following error:
Undefined symbols for architecture x86_64:
  "_OBJC_CLASS_$_RNSVGMarkerPosition", referenced from:
      objc-class-ref in libRNSVG.a(RNSVGRenderable.o)
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I believe this to be new with 9.10.0 as this is where RNSVGMarkerPosition was added.
Environment info

React native info output:
info
  React Native Environment Info:
    System:
      OS: macOS 10.14.5
      CPU: (4) x64 Intel(R) Core(TM) i7-7660U CPU @ 2.50GHz
      Memory: 535.57 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 12.6.0 - /usr/local/bin/node
      Yarn: 1.17.3 - /usr/local/bin/yarn
      npm: 6.11.3 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 13.0, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0
    IDEs:
      Xcode: 11.0/11A420a - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.3 => 0.59.3
    npmGlobalPackages:
      react-native-git-upgrade: 0.2.7

Library version: 9.10.0
Steps To Reproduce

Install react-native-svg 9.10.0
Clean
Build (debug)
...

Describe what you expected to happen:

Build successfully

I also have the same problem

System:     OS: macOS Mojave 10.14.5     CPU: (8) x64 Intel(R) Core(TM) i7-7820HQ CPU @ 2.90GHz     Memory: 73.57 MB / 16.00 GB     Shell: 5.3 - /bin/zsh   Binaries:     Node: 10.11.0 - /usr/local/bin/node     Yarn: 1.10.1 - /usr/local/bin/yarn    npm: 6.9.0 - /usr/local/bin/npm     Watchman: 4.9.0 - /usr/local/bin/watchman  SDKs:     iOS SDK:       Platforms: iOS 13.0, DriverKit 19.0, macOS 10.15, tvOS 13.0, watchOS 6.0     Android SDK:       API Levels: 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29       Build Tools: 19.1.0, 20.0.0, 21.1.2, 22.0.1, 23.0.1, 23.0.2, 23.0.3, 24.0.0, 24.0.1, 24.0.2, 24.0.3, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.0, 26.0.1, 26.0.2, 26.0.3,  27.0.0, 27.0.1, 27.0.2, 27.0.3, 28.0.0, 28.0.0, 28.0.0, 28.0.1, 28.0.2, 28.0.3       System Images: android-18 | Google APIs Intel x86 Atom, android-19 | ARM EABI v7a, android-19 | Intel x86 Atom, android-19 | Google APIs ARM EABI v7a, android-19 |  Google APIs Intel x86 Atom, android-21 | Intel x86 Atom, android-22 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom, android-24 | Google APIs Intel x86 Atom, android-25 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom, android-27 | Google Play Intel x86 Atom, android-28 | Google APIs Intel x86 Atom, android-28 | Google Play Intel x86 Atom, android-Q | Android TV Intel x86 Atom       Android NDK: 20.0.5594570   IDEs:     Android Studio: 3.5 AI-191.8<IP_ADDRESS>91312     Xcode: 11.0/11A420a - /usr/bin/xcodebuild   npmPackages:     react: 16.8.3 => 16.8.3      react-native: 0.59.0 => 0.59.0    npmGlobalPackages:     react-native-cli: 2.0.1     react-native-git-upgrade: 0.2.7

Can you try relinking or running pod install?

:tada: This issue has been resolved in version 9.10.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

I guess this issue is back in v12.1.0
I tried to unlink and link again. but no luck for me

I second that! v12.1.0 and XCode 12.5 has brought back the issue

I got the same issue in both v12.1.0 and v12.1.1.
Then I uninstall v12.1.0 and install v12.2.0. Surprisingly, this version works for me.


Don't show join thread for task action

Details

We don't want to show Join thread option for task action.
Fixed Issues

$ https://github.com/Expensify/App/issues/50566
PROPOSAL: https://github.com/Expensify/App/issues/50566#issuecomment-2405165164
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

Open any chat
Create a task
Long-press/right-click the task
Verify there is no Join thread option

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

https://github.com/user-attachments/assets/71f0f884-522c-4bb5-abe5-1afcbfd1816c

Android: mWeb Chrome

https://github.com/user-attachments/assets/409bfa83-ea7a-48dc-89cf-1d27ae266c21

iOS: Native

https://github.com/user-attachments/assets/fa0d674e-c022-4219-b2ed-e5737a8880bf

iOS: mWeb Safari

https://github.com/user-attachments/assets/c38db541-7627-4b77-a20d-902ed8d2cede

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.


Bloxstrap menu

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
I have been using bloxstrap on Windows 10, and when I download it, it doesn't show the menu app for it, but it does when you don't install it yet.

when I download it, it doesn't show the menu app for it, but it does when you don't install it yet.

I'm sorry, what?

No response for a month


Microsoft Connector: domainHint configuration parameter

Adds a description and example usage for using the domainHint configuration parameter added in PR https://github.com/dexidp/dex/pull/2586
This enhancement was added to address issue https://github.com/dexidp/dex/issues/2585

@josephtknight would you mind rebasing your PR? Thanks!


Delete only if Regular Expression Matches

Is your feature request related to a problem? Please describe.
There are some occasions where it can be useful to delete some messages in bulk in a channel, but only if it matches a regular expression.
For example:

API keys, such as Discord bot tokens, AWS, Google, etc
Specific uses of bot commands, such as ^[$%!/><]\s*ping$ to remove ping commands, but not other messages that use the word.
Personal information, such as phone numbers, postal codes, or email addresses sent by a person.

Describe the solution you'd like
I think it'd be great if an additional field could be added called "Pattern".
The script can search for messages with the existing logic, but if "Pattern" is configured, then conditionally only delete messages if the pattern matches against the message contents, otherwise skips to the next.

I like the idea

I like the idea


[Firewall Rules] Change regex101 to use Rust flavour

Regex101 now supports the Rust regexp flavour.
Solves https://github.com/cloudflare/cloudflare-docs/pull/1098#discussion_r609950228

cc @pedrosousa


getCurrentPosition() returning 0

Hello,
I am attempting this code, but am running into issues with getCurrentPosition() returning 0 instead of the current position. Because of this, my code will not work. This is running on an ESP32s WROOM 32
void move_motor() {
  Serial.print("Current Position: ");
  Serial.println(current_position);

  Serial.print("Moving to Position: ");
  Serial.println(move_to_step);

  stepper->setCurrentPosition(current_position);

  stalled_motor = false;

  stepper->setAcceleration(accel);
  stepper->setSpeedInHz(max_speed);
  driver.rms_current(current);
  driver.SGTHRS(stall);
  driver.TCOOLTHRS(tcools);

  if (current_position == move_to_step)
  {
    Serial.println("ALREADY THERE!");
  }
  else
  {
    Serial.println("Moving");
    stepper->moveTo(move_to_step);

    while (stepper->getCurrentPosition() != stepper->targetPos())
    {

      if (stalled_motor == true)
      {
        printf("Stalled\n");
        stepper->forceStop();
        break;
      }
      printf(".\n");
      vTaskDelay(1);
    }
  }
  current_position = stepper->getCurrentPosition();
  printf("Motor Function Complete\n");
  Serial.print("Current Position: ");
  Serial.println(current_position);
}

Here is an example output:
Run Motor Function Current Position: 0 Moving to Position: 28000 Moving Motor Function Complete Current Position: 0 Motor Complete

moveTo() only sets the ramp to be started, while update of the parameters take place in the cyclic stepper interrupt. This is due to the internal structure of FastAccelStepper.
Instead of:
while (stepper->getCurrentPosition() != stepper->targetPos())

use:
while (stepper->isRunning())

stale

This did the trick :) Thanks for your help!


Separate serial and MPI dependencies

Octopus can be compiled with and without MPI, so this PR removes MPI as a dependency, introduces it as a variant, and chooses the appropriate compilers based on the variant chosen.
The PR also compiles fftw, libvdwxc with and without MPI.

Very good, thank you.
As it is with this PR, we are not building the MPI variant of octopus in our tests. (See https://github.com/fangohr/octopus-in-spack/blob/main/Dockerfile#L64).
It would be good to build it, as  - presumably - that is the more complicated build.
Options I can think of:

first build serial version (https://github.com/fangohr/octopus-in-spack/blob/main/Dockerfile#L64), then build a second variant of octopus with the parallel version. Running tests may be tricky: we need to tell spack with version we want to run for the tests (I guess this could be done, I just haven't done it yet). On the positive side, this would re-use many of the 3rd party libraries we have built already.

introduce a new workflow: cleaner, but also duplication, and duplication of carbon emitting builds (as we will build zlib twice: once for the ~mpi build and once for the +mpi build (in different containers).

Only test the MPI version: if that works, presumably the serial version will also work.

Views?

Let me try the first option to see how it goes, if it takes too long with both versions compiled on the same container, we can split it into two workflows and keep it clean.

The updated dockerfile, take about the same time (about 5 mins more) to compile, while also testing both the variants

@fangohr can we re-run the  spack-develop workflow, which failed recently ( looks like due to network issues)

I noticed for the MPI version, spack is not recompiling octopus but rather using the same instance as from the serial one, converting this PR into a draft until we can solve this.

Well spotted. The problem is visible here: https://github.com/fangohr/octopus-in-spack/pull/52/checks#step:3:931
This means that when we request the 'serial' version, spack is already activating mpi. So the line
https://github.com/fangohr/octopus-in-spack/blob/861de44537709ecfb068b3cf0455836655f96518/Dockerfile#L71
(and also the one for the spack install) needs to be changed to include ~mpi.
Probably it is the default to use +mpi (see https://github.com/fangohr/octopus-in-spack/blob/861de44537709ecfb068b3cf0455836655f96518/spack/package.py#L41).

Thanks for your suggestion, it solves the problem. The latest CI logs show that both MPI and serial variants are built and tested without errors.

Good progress; thank you.
Would it be consistent and a good idea to explicitly use the switches for all the variants that the package provides (https://github.com/fangohr/octopus-in-spack/blob/main/spack/package.py#L41) even if we just repeat the default (such as for ~debug). This way, the Dockerfile command line (for spack install) would also be explicit documentation about the variants that are available.
On the other hand, it would be a mixture between overriding some defaults (such as ~mpi) and using some of the defaults (at least for the variant switches we have not listed yet such as ~debug).
I am undecided. Any views?

Interesting, same error as the one with copy even though the name space is still correctly used as fs.copy.
The only change that happened from the previous commit is moving the location of import llnl.util.filesystem as fs to the top of from spack.package import * instead of below it. This  reformatting was done however by black

Interesting, same error as the one with copy even though the name space is still correctly used as fs.copy. The only change that happened from the previous commit is moving the location of import llnl.util.filesystem as fs to the top of from spack.package import * instead of below it. This reformatting was done however by black

Probably related to the import * statement. They should be avoided as one can easily accidentally override an object that as imported already. (But are common in spack.)


Move future reservations above checked-out reservations on current_reservations view

Requested by BMEC, quickly getting this into v5.5.1.

done, issuing PR's

What should show on the Current Reservations page? Checked out items currently do not.


set the manual operation of the stove

another problem, but I don't know if I set something wrong or there is a bug.
I set the remote control to manual, to do this you need to set the thermostat in the remote control to the maximum, i.e. raise all possible degrees after which the word MAN appears on the display which means manual.
In the dashboard the temperature box is not set manually as it should be... but 41 degrees, why? could they conflict?

Home Assistant only supports numbers for the temperature (which makes sense). 41 degrees is indeed MAN.


Add AddAuthentication(defaultScheme) overload

@Tratcher

@HaoK,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

That's a terrible idea. Folks will be tempted to call this middleware multiple times, resulting in all the IAuthenticationRequestHandler being invoked multiple times! :scream:

Not quite, all that would result in is the DefaultScheme being set to the last one.  This is Add, not Use

Arf, my bad, doing too many at the same time :sweat_smile:
So the only risk is that the first call will be ignored?

Its not different than today, AddAuthentication(o => o.DoStuff()).AddAuthentication(o => o.DoMoreStuff()).
DoStuff() and DoMoreStuff() are called, so whatever side effects happen...happen :)

What I mean is that if you do services.AddAuthentication("abc") and then services.AddAuthentication("xyz"), the second call will override the first one? So the default scheme will be xyz, not abc?
If that how it works, then that's fine :smile:

Yup, its just sugar for o => o.DefaultScheme = "abc"

Great, thanks, and sorry for the false alert. I'll stop doing too many things at the same time :trollface:

Eh welcome to my world right now :)


fix: language change when headless cms is not enabled

Description :sparkles:
Issues :bug:
Closes :no_good_woman:
DEV-XXX:
Related :handshake:
Testing :alembic:
Automated tests :gear:
Manual testing :construction_worker_man:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #149 (46239e0) into master (e3afab7) will decrease coverage by 0.05%.
The diff coverage is 33.33%.

:exclamation: Current head 46239e0 differs from pull request most recent head 30c6779. Consider uploading reports for the commit 30c6779 to get more accurate results

@@            Coverage Diff             @@
##           master     #149      +/-   ##
==========================================
- Coverage   88.02%   87.97%   -0.06%
==========================================
  Files          98       98
  Lines        3265     3267       +2
  Branches      591      593       +2
==========================================
  Hits         2874     2874
- Misses        387      388       +1
- Partials        4        5       +1

Impacted Files
Coverage Δ

src/domain/app/header/Header.tsx
62.65% <33.33%> (-1.55%)
:arrow_down:

Continue to review full report at Codecov.

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


Merge our two plugin branches

This merges my plugin code into your branch to allow us to have a single branch to work against.

Why has the scripts/pyfr file transformed from a symlink to a regular file?

Ahh I wondered what change that really was.
It was a regular file when I got it from Freddie.  What was it symlinked to?

Just to main.py
On Tue, Oct 18, 2016 at 7:43 PM, Tom Fogal<EMAIL_ADDRESS>wrote:

Ahh I wondered what change that really was.
It was a regular file when I got it from Freddie. What was it symlinked to?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/iyer-arvind/PyFR/pull/2#issuecomment-254601195, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABO2ShD6Bf7S-A_nkWZ1QdisLYeTkDZKks5q1RNtgaJpZM4KZGiD
.

--
Dr. Iyer Arvind Sundaram
The reasonable man adapts himself to the world; the unreasonable one
persists in trying to adapt the world to himself.
Therefore all progress depends on the unreasonable man

George Bernard Shaw.


Support OpenAPI Specifications for Mocking

What you did:
See of opeanpi/swagger specs were supported.
What happened:
OpenAPI v2 (Swagger) and OpenAPI v3 are not supported.
Problem description:
Suggested solution:
Add support for serving openapi v2 (swagger) and openapi v3 specifications. A lot of people already use openapi v2/3 to document and provide examples for their apis. There is a lack of good mocking tools around openapi, it'd be great if mockit supported serving an openapi specification. Supporting both versions of the OpenAPI format (v2/v3) would be ideal as a large amount of tooling and specs already exist around v2 (formerly swagger).

I'm curious how would such an integration look like?
also what would be different compared to Prism?

Thanks @Shelnutt2 , this sounds like a great idea! I have no idea how I could go about doing this, but I will spend some time thinking about it.
Does anybody have any thoughts on how this could work?
@Shelnutt2 what would you expect as a user?


fix NPE bug and clean up logic for setting the title bar

Samsung user was experiencing an NPE crash on startup.  This patch fixes the NPE error (getActionBar() was returning null) and condenses the logic for this into one place.

Automated message from Dropbox CLA bot
@niftynei, it looks like you've already signed the Dropbox CLA. Thanks!

@kunall17 would you mind reviewing this?  getActionBar() was causing an NPE for some older Samsung phones (/me shakes fist at Samsung).

Looks good, but i never got this error in a Sony Phone, do you know know why is this causing in a samsung?

The problem appeared on an older Samsung Galaxy S4.  Samsung is notorious for re-writing the Android standard libraries, giving them bugs that don't appear on other platforms. I don't know exactly why that version of Samsung crashes, but if I had to guess it'd be that they call the method before the view has been instantiated.

Thanks @kunall17 !


No matches using /disc burn <youtube_video_id>

/disc burn <youtube_video_id> in 1.0.6 returns No matches, which makes the /disc search command broken and probably discs previously burned with it

Fixed by pv-addon-lavaplayer-lib v1.0.9


"Does" match or "does not" match?

https://github.com/distribution/distribution/blob/a27b0c49521b78bd7f27415220f1571fa8e9839b/registry/handlers/manifests.go#L313
Why does the error message in the line of code above say "payload digest does match: %q != %q"?  Shouldn't it say does not match?

Yes, it should say does not match. Feel free to open a PR with fix

Fixed in #3515.  Thanks for merging.


Fix Issue 23683 - std.file.setTimes requests more permissions than needed

Rebasing will probably fix the existing failures.


[Urgent] Need to randomize visits to one of two URLs based on UTM

We're launching SEO for Galleries tomorrow via Marketo to ~80K people. We're trying to AB test which of these pages leads to more gallery applications -
A - https://www.artsy.net/gallery-partnerships?utm_medium=email&utm_source=marketo&utm_campaign=seo-for-galleries&utm_content=partnerships-a
B - http://pages.artsy.net/gallery-partnerships.html?utm_medium=email&utm_source=marketo&utm_campaign=seo-for-galleries&utm_content=partnerships-b
We don't need to change anything about the pages for this test. All we're looking to do is:

Person clicks on a link (https://www.artsy.net/partnership-opportunities)
Person is randomly redirected to URL A or URL B (based on a UTM param)

cc @craigspaeth @garrengotthardt @wrgoldstein

@jessekedy Can you let me know what the UTM param will look like by pasting in some sample URLs? Also just want to confirm that's all that needs to be implemented, no extra tracking/analytics or feature work?

I think when we do this sort of process where we divert our traffic elsewhere our future selves will be grateful if we take a second to fire segment pageviews for the marketo pages

 Randomize visits to marketo GP page vs standard GP page


chore: update auto-generated changlog settings

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.101%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
155

Relevant Lines:
158

💛  - Coveralls


Tell gcc to use C, not C++, when linting C source files

A regression bug introduced (most likely by accident) here: #232
Maybe related to #278

Looks like a simple mistake. Thank you for fixing this. :+1:


Complex number support

This is a first draft PR for complex number support. At this point, you can create, do arithmetic, and convert complex tensors.
The really interesting work will be FFTs for signal processing, but I will add that in a later PR. This is just the basic stuff.

This looks great


Add checks to allow framework to function in a normal app context (via UIApplication), or within an extension context (via UIScreen).

Awesome! Thank you for PR!


ios 9 - xcode 7

Any branch with the updated Swift changes? This version currently on Master doesn't compile

It compiles well. On Xcode 7.0, final version.

How can I update with podfile?  I try to use pod update, but Xcode suggest me to convert swift files. Thank you.

+1 An update of the pods would be great, thanks guys! :+1:

I'm not sure if it relates to the issues that @donaldking reported, but in order to make the library compile using Xcode 7 stable, I had to make the following changes:
diff --git a/Charts/Classes/Charts/ChartViewBase.swift b/Charts/Classes/Charts/ChartViewBase.swift
index 975357b..3e30a3b 100755
--- a/Charts/Classes/Charts/ChartViewBase.swift
+++ b/Charts/Classes/Charts/ChartViewBase.swift
@@ -776,7 +776,7 @@ public class ChartViewBase: UIView, ChartAnimatorDelegate
             }
         }

-        layer.renderInOptionalContext(context)
+        layer.renderInContext(context!)

         let image = UIGraphicsGetImageFromCurrentImageContext()

diff --git a/Charts/Classes/Components/ChartMarker.swift b/Charts/Classes/Components/ChartMarker.swift
index a91435a..e8261eb 100644
--- a/Charts/Classes/Components/ChartMarker.swift
+++ b/Charts/Classes/Components/ChartMarker.swift
@@ -44,7 +44,7 @@ public class ChartMarker: ChartComponentBase

         let rect = CGRect(x: point.x + offset.x, y: point.y + offset.y, width: size.width, height: size.height)

-        UIGraphicsPushContext(context)
+        UIGraphicsPushContext(context!)
         image!.drawInRect(rect)
         UIGraphicsPopContext()
     }
diff --git a/Charts/Classes/Utils/ChartUtils.swift b/Charts/Classes/Utils/ChartUtils.swift
index 19acd44..f6d5b5d 100755
--- a/Charts/Classes/Utils/ChartUtils.swift
+++ b/Charts/Classes/Utils/ChartUtils.swift
@@ -129,7 +129,7 @@ internal class ChartUtils
             point.x -= text.sizeWithAttributes(attributes).width
         }

-        UIGraphicsPushContext(context)
+        UIGraphicsPushContext(context!)
         (text as NSString).drawAtPoint(point, withAttributes: attributes)
         UIGraphicsPopContext()
     }
@@ -149,7 +149,7 @@ internal class ChartUtils
             rect.origin.x -= rect.size.width
         }

-        UIGraphicsPushContext(context)
+        UIGraphicsPushContext(context!)
         (text as NSString).drawWithRect(rect, options: .UsesLineFragmentOrigin, attributes: attributes, context: nil)
         UIGraphicsPopContext()
     }

@christosc Your issue is different. For some reason you haven't compiled the CALayer+Extensions.swift flile

@danielgindi Thank you very much.  That was indeed the problem; the missing CALayer+Extensions.swift file.  Probably that was due to the fact that I had imported the files into the project from an older version of the library, that didn't contain that file.  Reimporting all the files of the library solved my issue.


input data for the model

hi, I hava a question about input data, in function collect_data, input is [person_id, frame_id ,x, y], here what's "x, y" refering to?
and while in your pickled data, input data is bounding box coordinates [xmin, ymin, xmax, ymax]

Hi,
The "x" and "y" refer to the agents' (x,y) coordinates in pixel space.


Anyone who could help me out on setting up the rgb effects for The Division on G810?

What are you experiencing an issue with?

[ ] Aurora
[x] Aurora supported game, (The Division)
[ ] Aurora Wrapper
[ ] Aurora Profile
[ ] Aurora supported device, (Fill in problematic devices)
[ ] Aurora Scripts
[ ] Aurora Updater
[x] Aurora Lighting issue
[ ] Other, (Fill in another issue here)

What kind of an issue is it?

[ ] Crash
[x] Performance problems
[ ] Visual bug
[ ] Localization issue
[ ] Enhancement suggestion
[ ] Device issue
[x] Other, (no RGB light at all)

Expected behavior:
Keyboard should be lit and effects would be there when ammo or life is low.
Actual behavior:
Just plain dark, no leds lid at all.
How often does this problem occur?
Every time
Please describe the steps to reproduce this issue:
The issue can be reproduced by doing following steps:

Open Uplay
Start The Division
Game is playable but no RGB

What is the version of Aurora you are experiencing this issue with?

My version is... (0.6.0.b)
Does the previous version of Aurora have this issue?
Didn't ran previous versions of Aurora.
2017_01_03log.txt

You have to turn on the option in settings

In the game I've put the selector to use the 3th party logitech keyboard effects. (it worked before I used Aurora).

I'm not sure what could be wrong then. I always worked for me. Which keyboard are you using?

There seems to be an error getting the game state (which leds it should turn on) from the game. Try reinstalling aurora, trying an older version or repatching the sdk.

Okay, did some testing today.
Using a previous version (aurora folder without installer), all profiles seem to work perfect.
As soon as I update to the latest version (auroraupdater), The Division doesn't work anymore.
Also, but that is rather a minor issue, the profile i made for my desktop is selectable in the older versions of Aurora but the profile doesn't seem to load.
So 2 options:

using an older version of Aurora and not updating to the newer version.
using the newer version and quit Aurora and use Logitech's software instead for playing The Division.

I haven't played the division with the newest version of aurora so I can't confirm anything. If all other profiles are working as you said (Particularly the games that have peripheral lighting support like overwatch)  , then the problem is with aurora not "translating" the data from one sdk to another or not properly receiving the data outputted by the sdk. I might be wrong as I don't really know much about C languages and only had a quick peek at the aurora source code. Otherwise I don't really see a problem with the older version of aurora as it offers basically the same features as the new one.

About the profile, I use the standard ones in aurora, but there might be a bug with aurora reading the profiles (from current and previous versions of it). Try creating one in the new version or the profiles from old versions might not work at all because antonpup changed the way the effects are applied (it uses layers now instead of color zones)

The Division integration is broken on the current public version, like mentioned before here: #416 and #382
I believe it is already fixed on dev branch. But I will still be updating the wrappers with g-keys and what not.

Okay, so I just be patience and I'll see it when it's implemented.
Thanks


Validate domain name with CNAME record external registrar

It fails validation with www CNAME record

Fixed. Now my static web site runs smoothly even SSL certificate has been added automatically!


doit package_build not working on Python > 3.7

For a while I've observed than trying to build packages on Python > 3.7 just fails, on conda not pip. This is for instance the error I got when trying to build Lumen on Python 3.8.
Traceback (most recent call last):
  File "/usr/share/miniconda3/envs/test/bin/conda-build", line 11, in <module>
    sys.exit(main())
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda_build/cli/main_build.py", line 488, in main
    execute(sys.argv[1:])
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda_build/cli/main_build.py", line 458, in execute
    action(recipe, config)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda_build/cli/main_build.py", line 409, in test_action
    return api.test(recipe, move_broken=False, config=config)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda_build/api.py", line 221, in test
    test_result = test(recipedir_or_package_or_metadata, config=config, move_broken=move_broken,
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda_build/build.py", line 2865, in test
    actions = environ.get_install_actions(metadata.config.test_prefix,
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda_build/environ.py", line 799, in get_install_actions
    actions = install_actions(prefix, index, specs, force=True)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda/common/io.py", line 88, in decorated
    return f(*args, **kwds)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda/plan.py", line 466, in install_actions
    specs = tuple(MatchSpec(spec) for spec in specs)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda/plan.py", line 466, in <genexpr>
    specs = tuple(MatchSpec(spec) for spec in specs)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda/models/match_spec.py", line 47, in __call__
    parsed = _parse_spec_str(spec_arg)
  File "/usr/share/miniconda3/envs/test/lib/python3.8/site-packages/conda/models/match_spec.py", line 706, in _parse_spec_str
    if build is None and test_str[-1] != '*':
IndexError: string index out of range
TaskFailed - taskid:package_build
Command failed: 'conda build -c pyviz/label/dev -c conda-forge conda.recipe/ -t --append-file conda.recipe//recipe_append--py38-unit-default-pkg.yaml && conda build purge' returned 1

After a long debugging session I could pin down the issue to the get_tox_python, which on Python 3.7 returns '3.7' but returns an empty string on later versions of Python.
https://github.com/pyviz-dev/pyctdev/blob/1ad27d99b1724fe5bee9bc35c07c96ede5776085/pyctdev/util.py#L39-L44
Not sure it's a bug in tox (which pyctdev vendors) or elsewhere, but at least now we know where the problem comes from! Fixing it though properly will require me to better understand how doit package_build works.

Maybe upgrade tox?

Given the comment above the failing line I'd rather try to find a direct solution to that, before attempting to upgrade tox which could break pyctdev in various ways.

TODO really doubt this is the right way

First I'll have to understand what was the intention with this code, and if it was effectively working. There's no emergency though, not until Python 3.7 in EOL.


.NET MAUI DisplayRouteLayer: visible directions

Description
This PR fixes a bug where the route directions were not visible.
Type of change

Bug fix

Platforms tested on

[x] MAUI WinUI
[x] MAUI Android
[x] MAUI iOS
[x] MAUI MacCatalyst

Checklist

[x] Self-review of changes
[x] All changes work as expected on all affected platforms
[x] There are no warnings related to changes
[x] Code is commented and follows .NET conventions and standards
[x] Codemaid and XAML styler extensions have been run on every changed file
[x] No unrelated changes have been made to any other code or project files
[x] Screenshots are correct size and display in description tab (800 x 600 on Windows, MAUI mobile platforms use the MAUI Windows screenshot)

Before:

After:


Serve index.html inside subdirectory of public from subdirectory path

Clear and concise description of the problem
In some cases you might have an extra index.html that you want to serve at some nested path, without processing it through Vite. You can put it in a subdirectory of public, e.g. /public/foo/bar/index.html. However, if you do this, it will only be served at /foo/bar/index.html, and crucially not at /foo/bar/. This is different from the Webpack dev server, which will serve index.html files at the path of their containing directory, which caught me by surprise when migrating from Vue CLI to Vite.
https://stackblitz.com/edit/vitejs-vite-944tsu?file=public%2Ffoo%2Fbar%2Findex.html&terminal=dev
Suggested solution
If a subdirectory inside public contains an index.html, the index.html should be served at the path of that directory, either by default or as a configurable option.
Alternative
Instead of putting the directory in public, a possible workaround is to put it in the project root. If you do this it will serve the index file at the directory path. However, this means you need to manually copy the directory over to dist afterwards, which is a bit annoying since the whole point of having public is to automatically copy over files you don't want processed.
Edit: It's also a bit surprising to me that this behavior differs based on whether it's in public or the root
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

I think vite has nothing to do with it.
It depends on your actual serving web server.
I already have your use case, and it works fine.
If you are using AWS S3 and CloudFront, it can be a little tricky how you serve it.
What web server do you use?

@iamandrewluca I see now that my description was a bit unclear, but I'm talking about during development

Maybe this can help vitejs.dev/config/#server-middlewaremode

It's possible I can use this to make it work by configuring my own Express server, but that seems like a pretty laborious workaround, at that point I prefer the workaround of putting it in the root instead of public (I by the way just added an edit in my description about how it's surprising that the behavior depends on whether you put it in public or the root)

Looks like this is caused by https://github.com/vitejs/vite/commit/a3c334f66dbe1e5c49c4e5d77381aa0098c4a8a9 and https://github.com/vitejs/vite/blob/7d205809491be375a3de3cace014d07d23c2b687/packages/vite/src/node/server/middlewares/static.ts#L31

@sapphi-red @bluwy @henribru @iamandrewluca Any chance at getting this issue prioritized/planned?
I opened issue #16567 but it was unceremoniously closed out by @sapphi-red  and marked as "not planned".
It's important to note that there exists a difference in how  vite vs vite preview handles the serving of index.html files under public subfolders. The former will not default serve index.html files while the latter will. Dev mode and preview mode ought to be consistent.
A similar issue was addressed by @bluwy in pull request #14756, but I believe the change only took into account subfolders under src. The issue still affects subfolders under the public directory.

@ElliotNB Probably you can rewrite url using plugin for the time being, something like https://stackblitz.com/edit/vitejs-vite-qnxrju?file=vite.config.ts

@hi-ogawa Thanks for the tip -- we do have a workaround in place (by modifying our links to point directly to the index.html file rather than just the subfolder), but this just seems like something that should be consistent out-of-the-box.
@bluwy Thank you for moving this issue into approved!

No problem. We had this issue in the discussion queue for a while and got to it yesterday. We don't see any reason to not support this so it could be something fixed.
I'm not sure if there will be edge cases, but the only change I think is these places:
https://github.com/vitejs/vite/blob/c071eb38221bdd9981e061472a8b6f402ea21138/packages/vite/src/node/preview.ts#L189
https://github.com/vitejs/vite/blob/c071eb38221bdd9981e061472a8b6f402ea21138/packages/vite/src/node/server/middlewares/static.ts#L37
And setting ['.html'].


[Request] Ignore local files

My primary project has a dependency on grunt, which builds a node_modules directory in my local env that's not necessary on the deploy server…it would be nice to pass an array of local paths to ignore in the config

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1
And/or add an option in deployment-config.json to NOT consider files for upload if they match a pattern in .gitignore

+1

+1

+1
Really need this option. I don't want to download the whole node_modules directory for each new project. Really appreciate this!

Hi all,
It will be implemented in the next release !
Thank you all for your feedback !


feat: add CasdoorUserService and CasdoorAuthService

Signed-off-by: tangyang9464<EMAIL_ADDRESS>close: #3
refer to: casdoor-go-sdk

@hsluoyz changed

@fangzhengjin @Kininaru plz review


The storage of the VAN has been enabled, DB migration script modified…

…. Also the keys have been modified to have the official case.

Closed due to move to the new repository https://github.com/catenax-ng/tx-traceability-foss-backend


Timer coalescing across keys and cleanup of unused trigger tasks

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[ ] General

The pull request references the related JIRA issue
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message

[ ] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[ ] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Per timestamp only one TriggerTask is registered at
the runtime context. When the first timer is registered a new TriggerTask
is sheduled. When no timer is registered anymore for a specific timestamp
the corresponding trigger task is canceled and hence removed.
The ScheduledFutures to cancel trigger tasks are not checkpointed. So
cleanup of trigger tasks will not work after a failure.

I merged it. Thanks a lot for your work! 😃


New demographics fields + improved CSV

This PR fetches the new fields added to the demographics data during this PR, limits API returns to communities with >=50 people,  and improves the CSV output by tranposing the table, adding more detailed descrioptions of the data elements, and re-ordering the fields to match the display order in NCR.
closes #490
closes #489
closes #487
closes #482
closes #476
closes #475
To test:

set the API_GS_BASE_URL environmental variable to the development server like so:
export API_GS_BASE_URL=https://gs.earthmaps.io/geoserver/
fire up the API and request demographics data (including CSVs) for some communities. Communities with <50 people should return a basic JSON error message. For example, try Fairbanks and Nabesna.

# Fairbanks should return data as JSON and CSV
http://<IP_ADDRESS>:5000/demographics/AK124
http://<IP_ADDRESS>:5000/demographics/AK124?format=csv

# Nabesna should not return data as JSON or CSV
http://<IP_ADDRESS>:5000/demographics/AK256
http://<IP_ADDRESS>:5000/demographics/AK256?format=csv

The descriptive text in the CSV output has grown considerably here, and so could use some careful proofreading to make sure the sentences are sane and reference the actual datasets that the particular line of data came from. Crosschecking with the README.md would be much appreciated!
Note: In order to use gs.earthmaps.io in this branch, I had to bypass SSL certificate verification in the route-specific demographics.validate_community_id() function and also in more widely used fetch_data.make_get_request() function. I didn't know how else to get this to work, but I imagine we need to change this in production. Any thoughts on how & when is best to re-introduce the SSL verification would be welcomed!

@cstephen Thank you for the detailed review! Great job catching all those little things, they are important!
I was able to address all your comments, and took a stab at a 403 template. Not sure if that's the most appropriate code, or if the message should be something else, but we can adjust that pretty easily in future work.
I am still unable to make requests without turning off SSL verification. I get this error:
SSLError
requests.exceptions.SSLError: HTTPSConnectionPool(host='gs.earthmaps.io', port=443): Max retries exceeded with url: /geoserver/wfs?service=WFS&version=2.0.0&request=GetFeature&typeName=demographics:demographics&outputFormat=application/json&propertyName=(id) (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1135)')))


updated readme to include mention of error codes

It may also be useful to add a short description of the common causes of each error code so that people can attempt to problem solve their own issues before asking for help.

Errors start at 0


Docker rename support.

Change from Docker to moby

Error log

% go get -d -v github.com/jkawamoto/loci
github.com/jkawamoto/loci (download)
github.com/docker/docker (download)
package github.com/docker/docker/pkg/stdcopy: cannot find package "github.com/docker/docker/pkg/stdcopy" in any of:
        /usr/local/opt/go/libexec/src/github.com/docker/docker/pkg/stdcopy (from $GOROOT)
        ${GOPATH}/src/github.com/docker/docker/pkg/stdcopy (from $GOPATH)

Thank you for this PR.
I guess this app needs update other dependencies to pass the tests.
I'll check the Travis CI's issue later.

@jkawamoto
Thank you for your help! 🙇
I posted issue: #7


Only the first character is returned for characters_staff request

When requesting characters_staff data only the first character is returned.
Eg.: https://api.jikan.moe/v3/anime/16/characters_staff returns only one character, "Hanamoto, Hagumi", but the anime has many more. See: Hachimitsu to Clover characters page
Most likely caused by fix for #361.

The problems are probably caused by this update on MAL: Sort & Filter Functions Added: Characters, VAs

Yes, it was caused by that update. It's been resolved in https://github.com/jikan-me/jikan/releases/tag/v2.16.27
However, some endpoints are still cached with the old response and are showing that.
I've deleted the existing cache for https://api.jikan.moe/v3/anime/16/characters_staff
So you see the appropriate response now 👌

It's seem to be ok now for "Hachimitsu to Clover".
How long till all the caches are refreshed ?

It's usually 24 hours but recently due to the DDoS, I've bumped it to 48 hours.
IDK which other caches are affected, but if you find any then feel free to ping me. I'll invalidate them. Can leave this issue open for 2 days.

It's usually 24 hours but recently due to the DDoS, I've bumped it to 48 hours.
IDK which other caches are affected, but if you find any then feel free to ping me. I'll invalidate them. Can leave this issue open for 2 days.
I think  manga characters is affected also
it returns empty list of characters
it was working before
https://api.jikan.moe/v3/manga/121496/characters

Good catch @Ammourie , I completely forgot about manga. Pushed a fix, which would work as soon as the cache invalidates.

I think the issue can be closed now :)


Fix compilation with shared libraries

set_target_properties is missing an argument name for setting
SOVERSION, resulting in a build error when configuring:
CMake Error at CMakeLists.txt:199 (set_target_properties):
  set_target_properties called with incorrect number of arguments.

Cool; I took this out since I saw that the Ubuntu repo's were doing this as a pre-patch.
Thanks!


Decision tree class refactor

I propose a refactor of the regression tree class with the following objectives in mind:

Reduce code complexity. This class is very complicated and difficult to understand/use for new contributors.
Increase safety. It is very easy to get a segfault with the current class. With good encapsulation and object-oriented programming I believe misuse can be entirely prevented.
Use this class on the GPU. We currently have significant code duplication in GPU algorithms as we had to build our own data structures usable on the GPU.

To do this we can do it in two phases:

First refactor to reduce redundancy and cleanup without affecting backwards compatibility
Propose and evaluate any further changes needed affecting backwards compatibility, discuss how we can achieve this with minimal disruption to users. e.g. continuing support for the old format for a limited time

@trivialfis RFC will be up in a day. Sorry for the delay. I have been arranging a move to California lately.

@RAMitchell Can we close this issue, now that we merged #3989, #3995, #4008, #4015, and #3825 ?

There is still a little more to do. My idea was to make all of the internal vectors inside the regression tree private so that we can only access it through safe public methods, preventing misuse of the class. All of the ground work is done I just need to spend a little time finishing it off.

This was a little more complicated than I thought, I'm going to close this for now.


LA-07-Use-SQL-warehouse-Create a dashboard task must be reviewed

Module: LA-07-Use-SQL-warehouse
Lab/Demo: LA-07-Use-SQL-warehouse
Task: Create a dashboard
Step: All steps
Description of issue:
All steps in the above mentioned task must be reviewed as the portal got changed.

There is no drop down list in the portal. (step 3)
Need to create a dataset first (no option to select a query step 4)
Items mentioned in step 5 need a change.
In the portal only available options are:
Visualization type: bar
Y column: Category
X columns: Product ID : Count
Group by: Leave blank

Thanks for pointing it out! The instructions are being updated.


consider adding http://moghya.me

moghya.me is my portfolio, all of it's content is loaded from json and no need to edit any html. Consider adding it to repo.

Loading via JSON is a good practice, I use the same pattern on my portfolio. It makes the updating process much faster. However, I don't think this site quite meets the standards set by the other items on the list. I has a lot of bad design patterns (showing a modal on launch, distracting animated text, etc.). But keep working on your stuff.


Added release notes and bump versions for azure-cosmos, azure-spring-data-cosmos and azure-cosmos-encryption

Release azure-cosmos, azure-spring-data-cosmos and azure-cosmos-encryption

API change check
API changes are not detected in this pull request.

@alzimmermsft - build analyze is failing - because of API reference not valid -
[404] broken link https://projectreactor.io/docs/core/release/api/
Is it temporary and related to spring outage or should we change this reference? I checked on reactor's site, and almost all of their documentation is down and throwing 404

@alzimmermsft - build analyze is failing - because of API reference not valid - [404] broken link https://projectreactor.io/docs/core/release/api/ Is it temporary and related to spring outage or should we change this reference? I checked on reactor's site, and almost all of their documentation is down and throwing 404

Appears this was a transient outage on their site


Add OktaIdxAuth library, and initial passcode login demo

I'll include these updates in another PR.  Thanks @oleggnidets-okta


Improve Error Checking

Todo items for improved error checking, from API & user experience

[x] remaining bytes after deserialization is finished
[x] containers being packed with more elements than are allowed by the standard
[ ] implement error code or error messages
[ ] standardize error return api

json-fortran use a single class to control everything, which I think I'll try to emulate so as to make it easier for newcomers to understand. The mp_settings class was starting to look like that, so I think what I can do is rename that class to messagepack, and put the pack and unpack functions into it.
Additionally, this would let me add two new functions to it

failed(): returns an internal flag set if any error occurs
errmsg(msg): returns a string containing a more detailed error message

keep as a string for now for user experience over speed

I would also be able to add a configuration setting dictating whether extra bytes at the end of a buffer should be treated as an error or not.


[e2e]: generic schedule main multi-uses slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/5995542902
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Mon Aug 28 04:34:38 UTC 2023

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/6007854654
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Tue Aug 29 04:52:06 UTC 2023
Tests are passing now. Closing this issue.


Contributing to SDK

Hello Sir, I am looking to work on the project but I can't find issues or ROADMAP for the same. So, that I can contribute regularly. @adamjmcgrath

Hi @aggmoulik - thank you for your kind offer.
We don't have a public Roadmap and there are currently no open Issues. But we always welcome PRs to fix bugs and make improvements if you can find any


S3 storage

Not sure this is necessary as-is,  but wanted to discuss on wednesday. I created a bucket we could share to. I'll message the keys (which would go in the aws_creds.py file).
Instead of storing every html file, we might want to consider combining them somehow into a larger file (gzip or just json)

more difficult to commit than I'd anticipated. Can you try to merge with the latest branch and switch between local and s3 via argparse? Yeah, if you are uploading to s3, I would recommend zipping the JSON files from the case parser and only uploading those.

For reference, 2 months of all data for all included JOs is 92.6 MB uncompressed. The JSON is 10.2 MB uncompressed.

TBH, might be easier to just grab the local files and upload to s3 through some other means.

If you wanted to organize multiple people running the script, perhaps you could partition the date ranges that the scrape_calendar_data.py script is operating on.

Actually it'd be nice to have an AWS switch in the JSON parser (process_case_date.py), which checks if the JSON file is already there (same case code), and if so, checks if it's newer before replacing it (date field in the case JSON). + zip on read/write

If you are writing a bunch of files that are already there this will incur a bunch of reads though.

Haha sorry for the spam, was just thinking this out. We could put the date in the filename to avoid merge conflicts without reading the json.

Can you implement this as a new file that operates on the JSON output of the last step and parses the correct name (code+date), checks dates against s3 for merging (don't upload if not newer), zips, and uploads to AWS? ./src/upload_to_aws.py + ./src/libraries/aws_config.py

Yep, something like that was basically going to be my suggestion, or maybe making a large json per month or year or whatever by each judge, where key is date and value is html (since files themselves wouldn't be that big.)
Won't be able to take a look at this again until Wednesday's CAN, but we can talk about options then


fix raster initial zoom

When missing center in raster mbtiles, the zoom will be NaN, causing tiles not display properly.
This PR fix this.

/cc @ericfischer this PR looks like it touches a change you made recently https://github.com/mapbox/mbview/pull/71, any thoughts on this?


Bugfix/home page typo

fixed

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: SumanthMoraa sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Pinyin input method doesn't work in the latest version of Deepin desktop, started by x11docker.

I start the docker image by x11docker, but find that the pinyin input methods can't be used, though they have been activated, anymore. I've tried fcitx-google pinyin and sogou pinyin.
Any hints for this problem?
Regards,
HZ

I could help you better if:

you try with my image x11docker/deepin instead of yours because I won't build your very large image for testing locally.
you describe your setup and what you have tried
you document possible error messages
you describe how I can reproduce the issue

I'll try to do it ASAP.

Just an idea:
fcitx uses environment variables.
My test setups with fcitx used fcitx version 4. Some distributions has updated to fcitx5. That might make a difference. Also look at the environment variables that has to be set for fcitx. Maybe they refer to version 4.

It's set from Dockerfile:
ENV XMODIFIERS=@im=fcitx QT4_IM_MODULE=fcitx QT_IM_MODULE=fcitx GTK_IM_MODULE=fcitx
How do I change and test it in the container at run time?

How do I change and test it in the container at run time?

     --env VAR=value   Set custom environment variable VAR=value

Do you mean I can test with something similar to the following?
--env XMODIFIERS="@im=fcitx QT4_IM_MODULE=fcitx5 QT_IM_MODULE=fcitx5 GTK_IM_MODULE=fcitx5"

$ x11docker --env XMODIFIERS=@im=fcitx5 --env QT4_IM_MODULE=fcitx5 --env QT_IM_MODULE=fcitx5 --env GTK_IM_MODULE=fcitx5 --xephyr --desktop --clipboard --init=systemd -- hongyizhao/deepin-wine:apricot


Make stable versions better use SemVer

Currently I'm having an issue using Serde with Rhai, and I found that it's because Rhai isn't taking advantage of Cargo's allowance of ^ for versioning.
I've added the caret for stable dependencies only.

A semver without any operator is taken to be with a caret: https://doc.rust-lang.org/cargo/reference/specifying-dependencies.html#specifying-dependencies-from-cratesio

Currently I'm having an issue using Serde with Rhai,

What is the problem you face with serde?

I'll move this to an issue then, apologies.

Actually, nevermind, I think I solved the issue.


Ctrl+Shift+C Not working to copy text

For some reason I can't use Ctrl+Shift+C on a new Windows Terminal installation with the default bindings (nothing changed in profiles.json). Ctrl+Insert works fine.
I tried to explicitly add { "command": "copy", "keys": "ctrl+shift+c" } in profiles.json but that didn't help either.
Environment
Windows build number: 10.0.18363.0 Microsoft Windows NT 10.0.18363.0
Windows Terminal version (if applicable): 0.10.781.0

Steps to reproduce

Open Windows Terminal.
Select text and press Ctrl+Shift+C.

Expected behavior
The text is put into the clipboard.
Actual behavior
Nothing happens.

What is your keyboard layout?

Where are you trying to paste the text?
Does the text appear in the "cloud clipboard" (press Win+v)
Does copying from other applications and pasting into the Terminal work?
Do you have any clipboard manager style applications installed?

@DHowett-MSFT English (with Austrian locale settings) with US Qwerty and German QWERTZ keyboard layouts. I tried with both without success.
@zadjii-msft I tried notepad and the terminal itself. I can paste things from the clipboard into the terminal with ctrl+v without problems.
There's nothing in the cloud clipboard.
This is a new windows installation and I haven't installed any clipboard manager style applications - at least intentionally. Apart from the AMD chipset and GPU drivers, the realtek audio drivers and VS2019 I've installed everything I can think of via chocolatey. I added the list from choco list -lo at the end.
What's noticeable if I use ctrl+insert to copy the selection is removed, while using ctrl+shift+c the selection stays as-is. I tried cmd, the old PowerShell 5 and PowerShell Core 7 with the same behavior in all of them, so I think we can exclude PowerShell profiles as well
If I'm using the default PowerShell core shell (i.e. just open pwsh.exe) copying things via Ctrl+C works just fine.
choco list -lo
7zip.install 19.0
chocolatey 0.10.15
chocolatey-core.extension <IP_ADDRESS>
chocolatey-misc-helpers.extension <IP_ADDRESS>
chocolatey-windowsupdate.extension 1.0.4
coretemp <IP_ADDRESS>90411
cpu-z.install 1.91
discord.install 0.0.306
DotNet4.5.2 <IP_ADDRESS>40902
dotnetcore-sdk 3.1.201
furmark <IP_ADDRESS>
git.install 2.26.0
goggalaxy <IP_ADDRESS>
gpg4win-vanilla <IP_ADDRESS>91021
gpu-z 2.30.0
hearthstone-deck-tracker 1.1.6
hwinfo 6.20
hwinfo.install 6.24
irfanview 4.54
KB2533623 1.0.4
KB2919355 1.0.20160915
KB2919442 1.0.20160915
KB2999226 1.0.20181019
KB3033929 1.0.5
KB3035131 1.0.3
KB3118401 1.0.4
kdiff3 0.9.98
nodejs.install 13.12.0
notepadplusplus.install 7.8.5
openvpn 2.4.7
powershell-core 7.0.0
prime95 29.4
pwsh 7.0.0
spotify <IP_ADDRESS>2
steam <IP_ADDRESS>90215
treesizefree <IP_ADDRESS>
vcredist140 14.24.28127.4
vcredist2015 14.0.24215.20170201
vscode.install 1.43.2

why are the defaults copy = ctrl+shift+c and paste = ctrl+shift+v to begin with vs the ctrl+c and ctrl+v standard used everywhere else in windows?

@myusrn this has been discussed in a bunch of other threads, and is not worth on-topic for @danstur's bug report.

What's noticeable if I use ctrl+insert to copy the selection is removed, while using ctrl+shift+c the selection stays as-is. I tried cmd, the old PowerShell 5 and PowerShell Core 7 with the same behavior in all of them, so I think we can exclude PowerShell profiles as well

That's certainly peculiar.
Could you share you whole profiles.json file? Maybe there's some peculiar combo of settings that's causing this? Because by all accounts, Ctrl+Shift+C should be working for you 🤔

@zadjii-msft Certainly. The path is C:\Users\<UserName>\AppData\Local\Packages\Microsoft.WindowsTerminal_8wekyb3d8bbwe\LocalState\profiles.json I opened it via the GUI (but just to be save that there's not a problem here somehow). The file is as follows:

// To view the default settings, hold "alt" while clicking on the "Settings" button.
// For documentation on these settings, see: https://aka.ms/terminal-documentation

{
    "$schema": "https://aka.ms/terminal-profiles-schema",

    "defaultProfile": "{574e775e-4f2a-5b96-ac1e-a2962a402336}",

    "profiles":
    {
        "defaults":
        {
            // Put settings here that you want to apply to all profiles
        },
        "list":
        [
            {
                "guid": "{574e775e-4f2a-5b96-ac1e-a2962a402336}",
                "hidden": false,
                "name": "PowerShell",
                "source": "Windows.Terminal.PowershellCore"            },
            {
                // Make changes here to the powershell.exe profile
                "guid": "{61c54bbd-c2c6-5271-96e7-009a87ff44bf}",
                "name": "Windows PowerShell",
                "commandline": "powershell.exe",
                "hidden": false
            },
            {
                // Make changes here to the cmd.exe profile
                "guid": "{0caa0dad-35be-5f56-a8ff-afceeeaa6101}",
                "name": "cmd",
                "commandline": "cmd.exe",
                "hidden": false
            },
            {
                "guid": "{b453ae62-4e3d-5e58-b989-0a998ec441b8}",
                "hidden": false,
                "name": "Azure Cloud Shell",
                "source": "Windows.Terminal.Azure"
            }
        ]
    },

    // Add custom color schemes to this array
    "schemes": [],

    // Add any keybinding overrides to this array.
    // To unbind a default keybinding, set the command to "unbound"
    "keybindings": [
	]
}

@zadjii-msft I created a sample WPF app to see if I could reproduce the error and lo-and-behold I have the same problem there. So I guess some helpful program decided to register a global hotkey. I love those things.
Sorry to take your time, most certainly a user error.

Hey no problem at all! I'm glad you were able to figure it out, and I'm especially happy that you decided to post your findings once you figured it out ☺

@zadjii-msft I created a sample WPF app to see if I could reproduce the error and lo-and-behold I have the same problem there. So I guess some helpful program decided to register a global hotkey. I love those things.
Sorry to take your time, most certainly a user error.
Edit: For anyone else having the same problem: It was the Radeon GPU driver that decided to helpfully enable a dozen global hotkeys without mentioning that anywhere. (It's always the awful drivers isn't it?)

Bless you. It would have been very hard to pinpoint (the very same issue) if it wasn't for your edit. :)

@zadjii-msft I created a sample WPF app to see if I could reproduce the error and lo-and-behold I have the same problem there. So I guess some helpful program decided to register a global hotkey. I love those things.
Sorry to take your time, most certainly a user error.
Edit: For anyone else having the same problem: It was the Radeon GPU driver that decided to helpfully enable a dozen global hotkeys without mentioning that anywhere. (It's always the awful drivers isn't it?)

Thanks. This fixed my error too. I use an Nvidia GPU and the iGPU on my CPU installed adrenaline software

@zadjii-msft I created a sample WPF app to see if I could reproduce the error and lo-and-behold I have the same problem there. So I guess some helpful program decided to register a global hotkey. I love those things.
Sorry to take your time, most certainly a user error.
Edit: For anyone else having the same problem: It was the Radeon GPU driver that decided to helpfully enable a dozen global hotkeys without mentioning that anywhere. (It's always the awful drivers isn't it?)

Thank you!
Same thing here, it was the "AMD Software: Adrenaline Edition" in my case.
For anyone having the problem: click on the gear icon on the top right for settings, then go to the Hotkeys menu, and disable all hotkeys, or the "Toggle camera" option. I disabled it all.

@zadjii-msft I created a sample WPF app to see if I could reproduce the error and lo-and-behold I have the same problem there. So I guess some helpful program decided to register a global hotkey. I love those things.
Sorry to take your time, most certainly a user error.
Edit: For anyone else having the same problem: It was the Radeon GPU driver that decided to helpfully enable a dozen global hotkeys without mentioning that anywhere. (It's always the awful drivers isn't it?)

Same problem for me, thanks for providing fix.