Port C# Windows implementation to C++

Feature request

This PR recently added Windows support:
https://github.com/react-native-community/react-native-device-info/pull/1059
First I just wanted to say thanks so much for doing this!  It's great to see so much enthusiasm for Windows support in this space.
I have a suggestion here, to port this code over to C++.  See next section for rationale.
Why it is needed

There are a couple reasons to consider porting the module to C++:
Performance:
The core of react-native-windows is written in C++ and the standard template is a C++ app.  This is for performance reasons.  Bringing in a C# native module adds the overhead of the CLR to the app which is significant, and isn't strictly necessary.
Issue https://github.com/microsoft/react-native-windows/issues/5858:
This currently prevents apps from using C# native modules from the standard C++ based template app, which is the default.  We are talking within Microsoft on how to address this, and that process will take time.  In the mean-time there are workarounds but they aren't easy for customers to understand, so we have a short to medium term problem, which could be avoided altogether if the module was converted to C++
Possible implementation

The react-native-windows website has a guide on how to author a C++ native module.
Start here:
https://microsoft.github.io/react-native-windows/docs/native-modules
See section called "Sample Native Module (C++)"
Code sample

I believe https://github.com/react-native-community/react-native-device-info/releases/tag/v6.0.2 solves the specific issue you have with things working or not even working?
That said, in general sure! PRs happily accepted
I have no react-native-windows setup so I am relying on the community for progress in the area.

Thanks!  @tero-paananen - if you were interested in helping with this that would be awesome!

Here's some documentation on converting C# to C++/winrt: https://docs.microsoft.com/en-us/windows/uwp/cpp-and-winrt-apis/move-to-winrt-from-csharp
Not an automated process, but also there seem to be good resources for powering through it.


Possible bug: Parsing sync-webhooks responses

What I'm trying to achieve
When Saleor receives a response from a synchronous webhook, it is initially JSON deserialized using json.loads(response.content), and then the Python object is passed to a custom parser as in this example:
https://github.com/saleor/saleor/blob/62bd0c6f8d7c829a4bd1dd8ff502e1d062a98ba8/saleor/plugins/webhook/tasks.py#L296-L298
In the case of CHECKOUT_CALCULATE_TAXES received tax data is parsed to Decimal values for use in calculations. But when sync-webhook response contains JSON float values, they are initially parsed as Python floats and only then to Decimals:
flowchart LR
    A[JSON float]  --> float --> Decimal

Because of that, an example response containing the value:
{"shipping_price_gross_amount": 0.1}

is parsed and used in Saleor as:
Decimal('0.1000000000000000055511151231257827021181583404541015625') instead of Decimal('0.1')
I am unsure if this can be a source of a calculation error in this particular example, but it is worth checking.
Describe a proposed solution

We could restrict webhook responses to represent decimal numbers as strings only, but this would be backward incompatible
We can consider passing raw JSON responses to custom parsers, dropping the intermediate json.loads(response.content) step.
We can allow passing additional arguments to json.loads(response.content) step. For example, running json.loads with the argument parse_float=Decimal would help with the above example.

I think solution number 2 can be most beneficial as it is the most flexible one. In my opinion, we should also consider introducing more strict sync-webhooks response data validation, for example, using something like Pydantic library.
Other solutions I've tried and won't work
Screenshots or mockups

I suggest to fix it ASAP and assume there will be a declarative validation in the future - described in this rfc #11905

We can try going with options 2 or 3 to fix this, I would also check if quantizing the parsed value wouldn't solve the case. Adding validation is also useful here but according to #11905 we want to add it anyway later.


Feature/code lens option

Hey!
I have found code lens showing in every regexp quite annoying, so added an option to toggle it.
You can disable code lens but can still use with keybinding.
Have a nice day!

Any update on this?

Looks good! I'll probably change the default to enabled, that keeps it easy to discover. Users familiar with the feature can then opt to turn the code lens off.
I should find the time to publish a new version some time next week.

Cool! I'm actually really looking forward to this option. (currently I've disabled this extension in my workspace.. 😅)

It's released. Thanks for your contributions! (https://twitter.com/christof_marti/status/989885628199395329)


Keyboard accessibility

Integrating basic keyboard controls would be nice for some accessibility. Swiper still works on desktop and it will be used on desktop, so this makes sense:

Make navigation controls focusable (arrows and bullets) with Tab, Shift + Tab.
Provide visual feedback on focusable elements, so user can perceive current position on the page.
Make navigation controls interactive via keyboard: Enter/Space key.

I can setup PR for this.

there is a keyboard control demo : http://www.idangero.us/swiper/demos/19-keyboard-control.html but still missiong visual feedback at least

@goetsu I see there's API call for enabling this, but this is implemented with keyboard hijacking and disabled by default.
What should be done is default behaviour, where all interactive elements should actually focusable and able to interact with via keyboard by default, as normal buttons and links within the slider.

I agree it must be enable by default but regarding native keyboard focus on all interactive elements I don't think it's a good solution because it create multiple stops in keyboard navigations ( a mouse user don't get all the slide if he doesn't want them).
I prefer something like that by default :

on focus of the slide get visual feedback and make a text like "to navigate use arrow keys" visible
get graphic arrows and or navigation by default to let user understand that there is other slides
hide all slides for screenreader except the current one
on use of arrow keys or click on graphic arrows or navigation move focus on the new slide after the animation

You also may use some ARIA to improve the experience with screenreader

@goetsu Yeah, it's another option I can think about. Also, ARIA would be nice for screen reader users, it can be used to warn users that cursor is within slider and to read out how user can navigate it. I think ARIA support deserves separate issue here.

Guys, but we are talking about mobile-first touch slider, right? :) Do we really need this here?

@nolimits4web Yeah, that's why I've mentioned desktop, there's also a mention in README. Looks like Swipers works on desktop, so I believe it will be used as a responsive slider for both desktop and mobile.
Also, for mobile, it requires some ARIA for screen reader users, but this is a separate issue.

yes and :

many android devices has a keyboard
both android and ios device can be used with bluethoth keyboard (especially when you adress tablet too
can be used on desktop too
visibility of navigation and graphic arrows are needed anyway if you want let the user know that something can be swapped left or right or clicked to navigate

PS: ARIA is not only for mobile

Right, but desktop in none-target platform for Swiper. Anyway, if you guys can make a PR for that, and it will not contain a lot of hundreds of code lines, i will merge it

Guys any update on this with PR?

@nolimits4web I'll prepare this one


System tables access requires extraneous permissions

Affected Version
0.13.0, 0.14.0
Description
SQL queries on sys.* tables currently require the following permission:
  {
    "resource": {
      "name": "sys",
      "type": "DATASOURCE"
    },
    "action": "READ"
  }

In one sense, this is incorrect since "sys" is a schema and not a datasource-like table, so the permission needed should really be on something like "sys.segments". This occurs because the following block in DruidPlanner only checks the first part of the table's qualified name:
            if (node instanceof Bindables.BindableTableScan) {
              Bindables.BindableTableScan bts = (Bindables.BindableTableScan) node;
              RelOptTable table = bts.getTable();
              String tableName = table.getQualifiedName().get(0);
              datasourceNames.add(tableName);
            }

In another sense, treating these system tables as a "datasource" is not consistent with the equivalent non-SQL APIs.
Retrieving information about a datasource's segments via MetadataResource APIs only requires datasource-read permissions, and not a higher level "view segment information" permission, which is what a permission on "sys.segments' would represent.
The non-SQL APIs only require additional STATE-read permissions for APIs involving server information (http://druid.io/docs/latest/development/extensions-core/druid-basic-security.html#defining-permissions has more info).
The permissions needed for system table access could be adjusted to not require additional permissions beyond datasource-specific permissions for non-server related info (segments, tasks) and to require STATE-read permissions for server-related tables (servers, server_segments) to be consistent with the non-SQL APIs.
It's worth mentioning here that there is some inconsistency in the non-SQL task APIs themselves. The task APIs in OverlordResource only require datasource permissions, but the running tasks do contain information about where they exist (server-related info). On the other hand, retrieving task info from the middle managers via WorkerResource requires STATE but not datasource permissions.

The permissions needed for system table access could be adjusted to not require additional permissions beyond datasource-specific permissions for non-server related info (segments, tasks) and to require STATE-read permissions for server-related tables (servers, server_segments) to be consistent with the non-SQL APIs.

IMO it makes the most sense to align the permissions for SQL system tables with the permissions for similar metadata APIs as closely as possible. So, doing the above.

It's worth mentioning here that there is some inconsistency in the non-SQL task APIs themselves. The task APIs in OverlordResource only require datasource permissions, but the running tasks do contain information about where they exist (server-related info). On the other hand, retrieving task info from the middle managers via WorkerResource requires STATE but not datasource permissions.

One major issue here is that nothing really defines what "STATE" and "CONFIG" are supposed to mean (other than access to an enumerated list of endpoints). My general feeling is that if you have "STATE" permissions it's reasonable to be able to see any sort of metadata (including datasources you don't have specifically have read access grants for). If you couldn't, then you'd presumably get partial metadata, which would be a misleading view of cluster state. But that isn't written down anywhere that I can see, so it's a missing aspect to the permissions model and the documentation.
In the specific case you brought up, the only change I might make (given the above reasonable) is expanding the permissions such that you can see all tasks, for any datasource, if you have "STATE" permissions.

Fixed by #7579.


Chromium for 32 bit machine

Hello!
How can I build a Wayland chromium for 32 bit machine? Should I use target_arch=ia32 in the GYP_DEFINES? Can I run building directly on a 32 bit machine? I tried but without success.
Thanks

This has already been discussed in #439 and #441, there's no need to open another issue about it.


ci(development-stack): Add a gitconfig support in development stack

Change Summary
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[x] Other (please describe): development-stack

Related Issue(s)
N/A
Component(s) name
development stack
Proposed changes

Capture local username/email to configure git in container
Provide a pre-configured gitconfig file to facilitate git manipulation
Require an Makefile update on user's side:

cp development/Makefile ../

So git operation will be available in either host or container
How to test
(Considering Makefile has been updated recently)
# Deploy new Makefile version
$ make refresh

# Start development stack
$ make start

# Review gitconfig
$ less ~/.gitconfig

Checklist:

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have updated molecule CI testing accordingly
[ ] All new and existing tests passed (pre-commit, make linting and make sanity-lint).

MKDOCS issue is related to a fix in PR #698


assumed globals

Hi!
I'm trying to couple lua-zmq with luvit/luvit (#luvit). The latter nullifies some globals on which this project seems to be deadly depending.
They are (as far as i could see):

_G.bit
_G.debug
package.config
package.cpath

They all sit in pregenerated.nobj.c. Wonder if they are truly unavoidable to use and how do one generates this C file?
TIA,
--Vladimir

removed all usages of bit & debug modules from FFI bindings.  Add custom cmodule load code for luvit.
Also pushed the changes to LuaNativeObjects.

Thanks a lot!


Transform to add an agent's name into the message content

The recent addition of non-OpenAI client classes has expanded the utility of AutoGen to a lot of different models through other cloud providers as well as the ability to run it with local LLMs.
However, the name field on each message is removed as only OpenAI can accept and utilise that field.
This means that functionality, like the GroupChat's select speaker, isn't as effective as the LLM may not be able to determine which 'agent' belongs to each message.
This PR introduces a transform that can add an agent's name to the start or end of message's text content. As this is a transform, it isn't permanent and doesn't change the original message.
How it works:
from autogen.agentchat.contrib.capabilities import transform_messages, transforms

# Create a transform
name_transform = TextMessageContentName(position="start", format_string="'{name}' said:\n")

# Create the TransformMessages
context_handling = transform_messages.TransformMessages(
            transforms=[
                name_transform
            ]
        )

# Add it to an agent so when they run inference it will apply to the messages
context_handling.add_to_agent(my_agent)

# Example
# Messages before
[
        {"role": "system", "content": "I am the system."},
        {"role": "user", "name": "charlie", "content": "I think the sky is blue."},
        {"role": "user", "name": "mary", "content": "The sky is red."},
        {"role": "user", "name": "bob", "content": "The sky is crimson."},
    ]

# Messages going into LLM (where name is removed, non-OpenAI clients)
[
        {"role": "system", "content": "I am the system."},
        {"role": "user", "content": "'charlie' said:\nI think the sky is blue."},
        {"role": "user", "content": "'mary' said:\nThe sky is red."},
        {"role": "user", "content": "'bob' said:\nThe sky is crimson."},
    ]

Documentation to be done.
Dependency
For the GroupChat select speaker to be able to utilise this, PR #2719 needs to be merged first.
Why are these changes needed?
To provide the necessary name context for non-OpenAI inference.
Related issue number
PR #2719
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[X] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[X] I've made sure all auto checks have passed.

Thank you @marklysze for the PR! Would like to add an example of using this new transform? What about updating an existing notebook and show the differences between with and without this transform?

Thanks @thinkall, yep, definitely should. I'll put one together and add to the PR :).

Just a note, @thinkall, I've asked @qingyun-wu to help with merging #2635 as that is a fix that will help make sure this work properly (as the name isn't included on the first message and that affects this transform's effectiveness). I'll keep working on the documentation.

Just a note, @thinkall, I've asked @qingyun-wu to help with merging #2635 as that is a fix that will help make sure this work properly (as the name isn't included on the first message and that affects this transform's effectiveness). I'll keep working on the documentation.

thanks for the PR @marklysze, we would love to see this working.

Corrected link in documentation, I think everything looks like it's passing (OpenAI tests aside)... @thinkall anything else needed?

Codecov Report
Attention: Patch coverage is 10.00000% with 36 lines in your changes missing coverage. Please review.

Project coverage is 21.10%. Comparing base (6279247) to head (8e07d59).
Report is 108 commits behind head on main.

Files
Patch %
Lines

...togen/agentchat/contrib/capabilities/transforms.py
10.00%
36 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #3334       +/-   ##
===========================================
- Coverage   32.90%   21.10%   -11.81%
===========================================
  Files          94       95        +1
  Lines       10235    10729      +494
  Branches     2193     2469      +276
===========================================
- Hits         3368     2264     -1104
- Misses       6580     8316     +1736
+ Partials      287      149      -138

Flag
Coverage Δ

unittests
21.06% <10.00%> (-11.85%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[REQ] Can FFlags change topbar transparency?

Acknowledgement of preliminary instructions

[x] I have read and acknowledged the preliminary instructions.

What idea do you have?
idk if FFlags can change topbar transparency, can anyone help me?

No


Error creating particle while using Laser attachment

Don't know any repo steps, experienced this during a playtest.
This is how the UMP45 code looked like https://github.com/mzegar/sbox-TTT/blob/main/code/items/weapons/UMP45Prototype.cs

Fixed


error about "local/proxy"

执行 go get github.com/dxvgef/tsing-gateway
报错：
package local/api: unrecognized import path "local/api" (import path does not begin with hostname)
package local/global: unrecognized import path "local/global" (import path does not begin with hostname)
package local/proxy: unrecognized import path "local/proxy" (import path does not begin with hostname)
package local/storage: unrecognized import path "local/storage" (import path does not begin with hostname)
local 是什么路径？自定义的么？

local是go mod定义的模块名，你是不是没有启用go modules？


Adding Android imagery

Yes, I know one piece was borrowed from Apple imagery.

Forgive me my ignorance. But what are these images for?

@sigod: beware of the trolls: https://github.com/KSP-CKAN/CKAN/issues/1314

I am sorry, i don't understand what you want, but I just wish now to help adapt this game to Android. Everyone is calling me troll and doing that link since I done the "issue"... for some reason... I make nice scripts, and this is a proof. I never had intent to harm anyone or any project... and will never have...

Oh, I almost forgot to answer your question. These images are for when running 2048 in an Android device: two startup images (horizontal, vertical) for when the app is loading; and one image for  playing it, which I'm sure you already know.


⚠ Spryr has degraded performance

In 8bbaa74, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 5912 ms

Resolved: Spryr performance has improved in 2868e06 after 8 minutes.


refactor: Add IDs to elements of landing, login, register, and what's new screens

Description
Add IDs to elements of landing, login, register, and what's new screens.
Landing Screen:

The title under the App logo (Courses and programs from the world's best universities in your pocket)
Text: (What do you want to learn?)
Textfield (Search our 3000+ courses)
Button: (Explore all courses)
Button: Register
Button: Sign in

Login Screen:

Title: (Sign In)
Text: (Welcome back! Please authorize to continue.)
Email and Password (Labels and TextFields)
Button: (Forgot password?)
Button (Sign In)
Text: (Or sign in with:)
Buttons: (Sign in with Google, Facebook, Microsoft and Apple)
On Forgot password page (Title, Text, Email label, Email field and Reset password button)

Register Screen:

Title: (Sign up)
Text: (Create new account.)
All Input Fields with Labels and notes under the Fields (Full name, public name, email, password)
Add ID and text: (Country or Region of Residence dropdown and its label)
ID to all buttons (Create account, Register with facebook, Microsoft and Apple)
Dropdowns: (Highest education and Gender)
Dropdown values (Country, education and Gender)

Whats new screen:

Title and Close Button
Next, Previous & Done Buttons

Hey @farhan-arshad-dev
I have verified that I can now see IDs on all the elements.


windows: fix install

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

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 0.0%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
0

Relevant Lines:
304

💛  - Coveralls


Change log between versions

Hi,
I'm currently using 1.3.0 version.
I want to know if it is safe to upgrade to the latest version without breaking the site.
Is there any guide or page that shows new features/deprecation's/Backwards incompatibilities or so?
Thanks

The releases page is pretty complete with respect to fixes, new features, and breaking changes for each version.
Version 2 is meant to be a nearly drop-in replacement for version 1. The 2.2.0 release page has the specifics for breaking changes.
There were a number of unanticipated issues from moving to formatting with toLocaleString (unanticipated by me, at least), but those have been resolved. You should be good to go with the latest version.

thanks


All download links are down

Hi Licheng, thank you for your work.
It seems like all download links are not working since a few days ago, could you check and update the links? Thank you!
such as:
https://bvisionweb1.cs.unc.edu/licheng/referit/data/refclef.zip
https://bvisionweb1.cs.unc.edu/licheng/referit/data/refcoco.zip
https://bvisionweb1.cs.unc.edu/licheng/referit/data/refcoco+.zip
https://bvisionweb1.cs.unc.edu/licheng/referit/data/refcocog.zip

Hii @lichengunc, please look into this.
Thanks.

Same here! I cannot download the data from those links. A "HTTP Error 503: Service Unavailable" error is raised when I'm trying to use wget.
Thank you in advance.

I have the same problem.
Thans.

Hi @lichengunc I have the same problem. Thanks

Based on this recent commit by @lichengunc in another repository:

Announcement: some download links are (and will be) invalid for a while due to the webserver was down. The university is fixing it. Please ask around who has previously downloaded the datasets/detections for their usage.  Sorry for the trouble. We are trying to bring them back soon.

While waiting for the webserver to be back up, the datasets can be downloaded via Internet Archive's Wayback Machine:

https://web.archive.org/web/20220413011817/https://bvisionweb1.cs.unc.edu/licheng/referit/data/refclef.zip
https://web.archive.org/web/20220413011718/https://bvisionweb1.cs.unc.edu/licheng/referit/data/refcoco.zip
https://web.archive.org/web/20220413011656/https://bvisionweb1.cs.unc.edu/licheng/referit/data/refcoco+.zip
https://web.archive.org/web/20220413012904/https://bvisionweb1.cs.unc.edu/licheng/referit/data/refcocog.zip

It also seems that the mscoco download link is not working too: http://mscoco.org/dataset/#overview.
Can I use this link instead? https://cocodataset.org/#download

It also seems that the mscoco download link is not working too: http://mscoco.org/dataset/#overview. Can I use this link instead? http://images.cocodataset.org/zips/train2014.zip

It seems so (https://github.com/lichengunc/refer/issues/4)

the urls for downloading imagenet VID aren't working
https://bvisionweb1.cs.unc.edu/ILSVRC2017/download-videos-1p39-desconly.php

please @lichengunc all URLs for Imagenet VID aren't working, could you please check that?

now this link is also down, https://bvisionweb1.cs.unc.edu/licheng/referit/data/images/saiapr_tc-12.zip

I downloaded it from: https://web.archive.org/web/20220515000000/http://bvisionweb1.cs.unc.edu/licheng/referit/data/images/saiapr_tc-12.zip
This link's prefix also is useful to download the remaining files (jamestiotio comment).

I don't know if it's only me, but the web archive link keeps failing, "Failed - Network error" I tried multiple times in various web browsers.
Any alternative idea guys?
https://www.imageclef.org/SIAPRdata
I also found something on the web regarding the matter, but I'm unsure if the data it contains is the same as the one provided above...
Thanks in advance!

Please, I want to download this dataset. The links are not working, please start the link or give some alternative link. In urgent I need it.


fix(store): replace windows path separator with posix path separator

followup to #851 - unfortunately I added the regex at the wrong place

self-approving to unblock myself and this change is not impacting any other platform than windows (which is broken without this change)


How do we deal with people who move?

People can move away from Cambridge, or between apartments in Cambridge. How do we deal with these events if people are a member of an existing team? How do we reconcile their data between old and new residences?

People leaving Cambridge should leave the app, right? Or do we want their data to persist, for comparison reasons?
In terms of within Cambridge, we might have to treat people who move as if they were new users for a lot of things; their past energy usage doesn't really reflect on their new residences unless they move between very similar places, so we might just have to compare them not to their past usage but to other users in similar residences.
For teams, do we want team data to be compared to past data from that team, or should it just be computed based on the current data for its members?


Pasted Project lol

Entire project minus the menu design was pasted from original creator (Discord): Debug.It#6837
This kid does not know how to code, I was nice enough to provide all the features in this project and he released it out of spite lolol

rip


Dnsmasq does not work if consul_client_address used

First, thanks for all of this. I'm new to Consul and this project helped me a lot.
When I set consul_client_address, I found that Dnsmasq no longer works. The problem is that <IP_ADDRESS> is hard-coded in dnsmasq.yml.
I fixed it by changing the line to this:
content='server=/{{ consul_domain }}/{{ consul_client_address }}#{{ consul_port_dns }}'
The various client addresses can be individually configured using the config file (https://www.consul.io/docs/agent/options.html) and the addresses object. I'm sure you know this. I don't know if you want to be able to set all the addresses individually in this project.
Thanks again for all of your work. This has been a huge help.

@williamoneill - Nice catch! I'll have a fix PRed in a bit.

@williamoneill what were you setting client_addr to that made dnsmasq not work? I have noticed that when I set it to <IP_ADDRESS> and dnsmasq is also set to that, dns lookups are broken until I set dnsmasq to <IP_ADDRESS>

After some testing, I can't even get my consul to work without setting client_addr to <IP_ADDRESS> and dns doesn't work in that instance unless the dnsmasq ip is <IP_ADDRESS>.
@williamoneill not sure what your setup is like, but it seems that we should check if client_addr is <IP_ADDRESS> and if so, keep the dnsmasq ip as <IP_ADDRESS>.
I also have a dns test, but I'm not sure if it would apply to all cases:
- name: check dns
  command: dig {{ consul_node_name }}.node.consul
  register: dns_test
  notify:
    - test dns

- name: test dns
  fail: msg="Consul DNS not resolving properly"
  when: "'{{ ansible_eth0.ipv4.address }}' not in dns_test.stdout"


Update to sample Site Script code

Category

[x] Content fix
[ ] New article

Related issues
none known
What's in this Pull Request?
Fix to sample code to match schema

Learn Build status updates of commit 3560025:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/declarative-customization/get-started-create-site-design.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


How to do predict with pyspark

I dumped the model with pickle, and then I uploaded the model file to spark cluster, which did not install lighgbm by some reasons. when I run the code, it showed 'ImportError: No module named lightgbm.basic'. So how can I predict the data with pyspark by using lighgbm model file?
The following was how I upload the model file and do the predict.
sc.addPyFile("model_test/lightgbm/model.pkl")
with open('model.pkl', 'rb') as fin:
pkl_bst = pickle.load(fin)
pred_probs = pkl_bst.predict(csc)

@TerrenceVarada I am not familiar with pyspark.
However, I think the prediction of lightgbm model needs lightgbm package itself.

You can try using the new pandas_udf for pyspark 2.3.0+:
import lightgbm as lgb
import pandas as pd
import pyspark.sql.functions as F

# training on the driver or load model from the file
model = lgb.train(....)

# model feature columns
cols_features = ['feat_1', 'feat_2', ...]

# pandas udf for prediction task
@F.pandas_udf(returnType='id int, probability float',
              functionType=F.PandasUDFType.GROUPED_MAP)
def predict_udf(df):
  df['probability'] = model.predict(df.loc[:, cols_features])
  return df.loc[:, ['id', 'probability']]

# predict on executors
predict_test_df = (
  test_df
  .groupBy(F.spark_partition_id())
  .apply(predict_udf)
)


can support like hadoop one url list all jmx item?

like hadoop/hbase JMXJsonServlet.java can export all jmx item one http request and also query by type、name、path
so zabbix agent can dump all jmx item to tmp file one time

You can do this, too, by using wildcard read patterns. See the reference manual for details. This is not necessarily recommended since it puts quite some load on the machine to monitor if there is a large number of MBeans (e.g. for Weblogic such a file can easily as large as 200 MB).
Please feel free to use also the forum (linke from the home page) as this is more appropriate for usage questions. GitHub issues are more for bug reports and feature requests. thanks.

OK


Program freezing after a certain amount of time

Hello I've made a twiiter bot who is working well but after a certain amount of time like 10 minutes the bot freeze and doesn't work anymore and I've to restart I don't understand why.
My code:
https://pastebin.com/phb7VY4j

This is too much code to debug for you, with a lot of unnecessary nesting and repetition, but it's likely not an issue with Tweepy.
Can you provide a reproducible example (MCVE/SSCCE) or be more specific about your issue?


V5.0 dev

Rewrite of whole library in typescript

@nkitku thank you for this truly heroic effort, this is a very impressive and thorough start, this certainly looks like a brilliant direction for the project to take.
This needs a thorough review to check backwards compatibility and RFC compliance.
As a quick start on that road, could you resolve the conflicts shown in the PR so we can start working from a mergeable branch? Additionally, I notice that the files are now all using 0755 permissions, I think best practice is 0644 for files and 0755 for directories?

thank you for appreciation,
ok, i'll try to resolve all conflicts
and thanks for suggestion, i'll fix all file permissions

@thomseddon instead of merging it in 'dev' branch can we have new branch "v5.0-dev"?

@thomseddon i have resolved all conflicts

$ npm test

<EMAIL_ADDRESS>pretest /mnt/d/codes/node-oauth2-server
npx jshint --config ./.jshintrc lib test

npx: installed 30 in 30.987s
ERROR: Can't find config file: ./.jshintrc
npm ERR! Test failed.  See above for more details.

@nkitku
I think there must be a copy for users who are not using typescript.
so, how about add the build result to the repo.

@peter-fan-cn
i think you clone master branch please use branch 'v5.0-dev', and thank you for reminding me i'll fix master branch later,
and for development process we must require typescript but for publishing it to npm we will use build version

npm run build:prod

@peter-fan-cn
i think you clone master branch please use branch 'v5.0-dev', and thank you for reminding me i'll fix master branch later,
and for development process we must require typescript but for publishing it to npm we will use build version

npm run build:prod

Yes, you are right. thank you for your reply.

Hi,
The typescript upgrade looks like it is done. Is there a possibility to make npm release 5.0.0-dev.1 to make it easier to create and contribute examples and documentation?

@thomseddon i am interested to contribute in this project. Are you still looking for contributors / maintainers?

@liyamahendra yes!
There are three main areas we need help on:

Review this branch - this is a phenomenal start, but we could do with another set of eyes on it. Assistance in reviewing the code and implementation would be brilliant
Documentation - we need to document usage of this new branch and also create an upgrade guide for users
Examples

Help on the above would be hugely appreciated. I will add anyone who makes meaninful contributions as a permanent maintainer :)

@emilcardell I wouldn't normally create an official npm release until we have something stable, this branch will remain a development branch until we've had some review (I'm also asking for help on this). Testing and examples can be done by referencing the repo directly in your package.json

Noticed that you switched from ESLint to TSLint - TSLint is going to be deprecated in favor of ESLint by the end of the year, so it may be better to stick with ESLint.

Will v5 support graphql? I expect so...

Will v5 support graphql? I expect so...

What do you mean? The OAuth spec has no mention of GraphQL, and GraphQL happens over HTTP, so you can use this library with GraphQL.

@benaubin  There are examples? I use prisma. But I don't know how to use node-oauth2-server in prisma.

@sessionboy
https://github.com/zce/oauth2-example
This is an example I just made recently. I don't know if it can help you.

@zce  Thanks, this is great. But what I want is an example of using node-oauth2-server in graphql(prisma)

@sessionboy There is no special relationship between the OAuth and GraphQL. You may be concerned about endpoints architecture.

https://example.com/graphql - GraphQL endpoint
https://example.com/oauth - OAuth endpoint

I don't think that's a problem.

@nkitku Is there a chance you could add support for the PKCE extension?

@thomseddon, I'd very much like to use this library and would be happy to help assist in the creation of updated documentation and examples. I am very interested in a Koa2 Postgres implementation with jest/supertest coverage, but the Koa example linked in the README is 4 years old and incompatible with master / nkitku:v5.0-dev.  If someone would like to point me in the right direction, to get that example updated, I'd appreciate it.

I read most of the PR today.  Really nice work.

I would like to be a reviewer?`How can we contact @thomseddon on github?

Review away!

Thank you @nkitku for this great effort! I'm currently trying to review it, some notes:

I would also offer a review :) - really love the fact that it is now written in Typescript <3! How can we contact @thomseddon ?

@thomseddon I'm also want to contribute on this.
@nkitku I added test steps to travis ci and made a PR to your branch, please help review, thanks!

@arasabbasi @skogsmaskin @mayrbenjamin92 @Jeff-Tian  If you want to review this, simply visit https://github.com/oauthjs/node-oauth2-server/pull/564/files and start reviewing! I love this rewrite and our company we love to use this for our next project, unluckily don't have the time to review this myself currently. Eagerly waiting for this to get released!

I am not able to import it as a 3rd party library, I has to change packge.json main to dist/index.js

I am not able to import it as a 3rd party library, I has to change packge.json main to dist/index.js

 "dependencies": {
    "oauth2-server": "file:../node-oauth2-server/dist"
  }

you can use "dist" folder as main package as it also contains "package.json"

Omg, I noticed it but didn't knew was possible, nice solution.

@thomseddon @nkitku I would like to help.
I saw the todo file in the forked repo, which item I can work on?

you can follow thomseddon comment

@nkitku @thomseddon ok, I will work on updating docs.

@thomseddon i want to be a maintainer. How should i do ?

@whatwewant Great, just start by reviewing this PR :+1:

@nkitku curious, was this branch converted from the dev branch from node-oauth2-server?
Trying to track down an issue with the implicit flow.
I also assume PKCE is missing?

any updates on this? If you need still need reviewers I'm happy to help

any updates on this? If you need still need reviewers I'm happy to help

@gjavi1 please review it. Trying to get this landed

How can I contribute to this? Should I fork https://github.com/nkitku/node-oauth2-server/tree/v5.0-dev and make changes and create PR in nkitku's repo?

@nkitku curious, was this branch converted from the dev branch from node-oauth2-server?
Trying to track down an issue with the implicit flow.
I also assume PKCE is missing?

yes, this converted from the dev

I think it can be an alpha version.

Hello ;) Is there any way I can contribute to this? :) Would be nice to finally land that PR :)

Will be released 2083?

Just to play the 👿 Devil's Advocate 👿 exactly what does a rewrite in TypeScript get the project apart from:

New bugs;
See 1.

Besides, TypeScript is not a new standard - it's just a new language (or sorts) that compiles down to JavaScript which is, itself, Turing Complete or able to implement the Lambda Calculus; which is to say that what you can do in TypeScript you can do in JavaScript.
And last time I looked, there are more JavaScript developers than TypeScript developers in any case.

Says the one, who is a C#/Perl/PHP/Java-Developer, who lives with 5 cats and has the react icon as favicon for his webpage...
So please go away, here is nothing too see... atleast for you.

Exactly how is the below response meant to encourage anyone to contribute to the project?

Says the one, who is a C#/Perl/PHP/Java-Developer, who lives with 5 cats and has the react icon as favicon for his webpage...
So please go away, here is nothing too see... atleast for you.

Except for other rude contributors, maybe?

Or shouldnt we maybe ask if it is not truely rude of you to devalue the work of nkitku by your post? I mean: The programmer nkitku wrote mainly this huge upgrade and you have the chuzpe to write here and think that you can devalue his work and also claim that it has introduced bugs?
Greta would say: "How dare you?"
You dont even have a Contributor badge in this project, so that i could atleast have some elevated respect for you.
So yeah: Noli turbare circulos meos!

I was asking a valid question, plus if you'd have noticed, playing the Devil's Advocate if it wasn't abundantly clear that's what I may have been doing. Seeing others seem to be able to render my web-site, I'm going to assume that they can read the second definition, "a person who champions the less accepted cause for the sake of argument" from that link because no-one is discussing the canonisation of saints here.
I've not seen a single software rewrite in the same language not inadvertently add bugs; and if someone asked me to bet if a software rewrite into a similar, typed language would introduce bugs, I'd bet a reasonable amount that it would (maybe not actual money, but maybe bragging rights).
I have seen rewrites with better arguments for them than those given in this particular PR.
It's not clear to me what is broken or why it needs a rewrite or why it's more important to rewrite this rather than address the 170+ issues in the issue tracker.
That said, I'm just language polyglot with 15+ years of professional programming experience, 35+ years of personal programming experience and about 21 years of paid GNU/Linux sysadmin. I'm also a JavaScript developer - when you've been programming as long as I have, none of the new languages  (including TypeScript) are new; they're just rehashes of old ideas.
Incidentally, one of the reasons one might rewrite this into TypeScript is because it could introduce strictly checked types and facilitate development between a large development team in an even larger diaspora...but let's not address the actual concerns I raised, shall we?

[Off topic:]
If Greta -- assuming we meant Ms Thunberg as opposed to Hansel's sister -- to say, "How dare you?" to me, my response would be: "Actually, I've been involved in local decisions that affect the envronment and climate change for many years; I'm closer to the center having been involved in politics for several years, what can we do together to help?" On the other hand, if we mean Hansel's sister, I'm not sure what we're referring to.

Yeah, i know you were trying (!) to play devils advocate, but horribly failed. Not only I got pissed of by your comment, but also some other devs, who also want this PR get done.
Yeah and all your claimed experience, i can simply response with Kurt Tucholsky: "Just because you have experience does not mean anything, you can do something wrong for 35 years."
So please unsubscribe from this topic and do your PHP/Pearl/C#/Java/Asm/SH/Bash/... stuff...

It's a pity that ad hominem attacks pass for discussion around here.

This PR doesn't explain why a rewrite is necessary;
That any major rewrite might introduce bugs is true on balance of probabilities and I'd argue beyond reasonable doubt;
I agree that N years of experience doesn't necessarily mean anything.

However, N years of experience might give one a different perspective or even be somewhat helpful.
As a maxim, though, is one likely to introduce bugs by:

Refactoring a project using its current structure;
Rewriting it.

Both of those activities can introduce bugs 🐛 🐞 🐛
Some projects' code gets so convoluted and difficult to work with that it's likely to produce less bugs doing (2) than (1).
Could it be that this is the case? Possibly, but we're too busy attacking people's character or experience than discussing the merits of the PR or why it's written 🦊

Yeah, i know you were trying (!) to play devils advocate, but horribly failed. Not only I got pissed of by your comment, but also some other devs, who also want this PR get done.
Yeah and all your claimed experience, i can simply response with Kurt Tucholsky: "Just because you have experience does not mean anything, you can do something wrong for 35 years."
So please unsubscribe from this topic and do your PHP/Pearl/C#/Java/Asm/SH/Bash/... stuff...

The pity is that you think it is ad hominem, that i recited your profile, but it is not ad hominem when devalueing the work of @nkitku
Double standards but crying because others are so rude... To attack someones character or experience, somebody has first to show character and share their experience by work. You devalue the work of others, which shows lack of character and you have nothing contributed here, which makes your experience argument null and void.
If you want to keep javascript, then fork this project and work on it. Then you can atleast try to produce a better product.

The actual questions happen to be:

What benefit does a rewrite to TypeScript give?
And wouldn't that almost certainly introduce bugs?

What experience anyone has is completely irrelevant to those questions unless one happens to be answering those questions. A complete non-programmer could ask those questions and they'd be valid questions (and that, in and of itself, indicates that the various responses to my questions are actually directed at me rather than the issues at hand).
The simple answers range from:

Because we want to rewrite in TypeScript;
Because TypeScript will give benefits [iterate over the benefits here];
The creator of the PR chose to use TypeScript and we're happy with that;
TypeScript is better than JavaScript (see TypeScript's site for the justification);
Look at the link [insert the link here].

That said, there's enough said here, let's move on, shall we?

Exactly. Nobody is obligated to answer your unnecessary questions. We can just ignore you and keep on with business.

@lloy0076 When ❤passion ❤  is at work we don't question. If you see this as a 💰 business 💰 , then all your questions are valid. Please remember the heart beat of 👐open source 👐  is nothing but ❤passion ❤ .
If you don't see this is worth the effort, you are more than welcomed to put your opinion.
Also remember only if someone who feels obligated and feel the same as you will give you the reason. Please don't expect response as I don't expect any response for this. 😃
Happy coding !
🍷 🍺 👨💻🍻👩💻 🍺 🍷

Firstly, @Uzlopak please can I ask you to refrain from any personal attack, it's both wholly unwarrented and diametrically opposed to the spirit of open source collaboration (as the original author of this project, I hope this at least partially meets threshold for "elevated respect"?)
@lloy0076 I'm sorry you recieved such a rude response to what seems to be a genuine and justified question. I don't plan on facilitating such discoure and will take action as necessary.
Back to the original comment, it's a valid point, however I'm willing to accept this on the following basis:

This project has quite a large number of issues that need addressing, most of the dependencies are grossly out of date (1) with many breaking changes along the upgrade paths, so much so that I believe there's quite a bit of work/rewrite required to achieve just this. Beyond this, there are probably some potential API improvements (2) as JS has changed quite a bit since even v3 was released. Additionally there are some bugs which need fixing (3).
Here we have been presented with a rewrite that supposedly addresses these issues. We can verify there's no bad dependencies so we're actually ahead with (1) right off the bat. My understanding is that this also provides some API improvements, so arguably we're also getting (2). As for (3), and the crux of your question, I agree - there will be new bugs, again my understanding is that this addresses a few of the existing bugs, but will obvioulsy introduce new ones.

The primary problem, is that this project does not have any maintainers. Quite a few people have approached me saying they would like to assist, and I have asked them to review this code. However, from reading the comments, whilst I believe there has been some reveiw, it does not appear as though there is any real consensus on if this PR is ready to be released.
If I has the choice, I would actually prefer for someone to triage the bugs and slowly increment the library, but no such option has presented itself, so here we are with a big fat rewrite - the main upside being that someone took ownership and spent a lot of time trying to improve the library, which is more than can be said for any other recent contributors (myself included) :)
If anyone would like to contribute to this project, please review this PR and if you like it, please comment with your opinion.
As a check @nkitku - would you plan on being willing/able/available to maintain/improve this if released?

@thomseddon

Or do you think that a code of conduct would improve the code quality of this project, as it would encourage other programmers to try to play devils advocat and report typos, formatting errors and asking basic programming questions?

No, it would encourage others similar to @Uzlopak from making personal attacks and refusing to address basic and relevant coding questions; that said, today I learned one can block people on Github 🤣

and refusing to address basic and relevant coding questions;

strange, i answered all relevant coding questions

today I learned one can block people on Github rofl
I hope you learned also the concept of abstract classes.

@thomseddon

As a check @nkitku - would you plan on being willing/able/available to maintain/improve this if released?

yes sure

Code looks really good - let's merge this! :)

Code looks really good - let's merge this! :)

+1

Is this PR going to change the API ?
As https://github.com/oauthjs/node-oauth2-server/pull/564#issuecomment-551285492 said,

My understanding is that this also provides some API improvements

If it does, could we update the documentation before merging this ?

I was trying to test this using some simple mongo backend and typescript. Does anyone know to to test this before the code is merged? I think before we merge this PR we need the docs as that will make people who are fairly new to stumble a bit.

@thomseddon
When will this be merged?

@thomseddon
When will this be merged?

You don't need to spam... Just wait

@Banou26
I waited. When will this merged?

Spamming add no value to the issue and only annoys people with useless notifications, it'll get merged once everything is ready to be merged by the contribs.
You're getting a block from me and if you continue this, you should probably get blocked from this repo & get reported.

@nkitku I'm actually fixing most of the things pointed out above in our own fork of your branch, so I think instead of just pointing out problems I could also create PRs against your v5.0-dev branch for these issues. WDYT?

@nkitku I'm actually fixing most of the things pointed out above in our own fork of your branch, so I think instead of just pointing out problems I could also create PRs against your v5.0-dev branch for these issues. WDYT?

@ankon you are welcome
But I think all the amendments in the library should go from here so that the community can discuss,
Others such as improvements in rewriting, testing and document updates can be

@ankon thank you for your great review work so far, hugely appreciated!
@nkitku I'm thinking the plan to move forward witht this branch could be for me to make you a collaborator, and for us to merge this into a new 5.x branch, we can continue to take PRs/Issues against that branch. We can have another PR open tracking a todo list for release; docs, tests, spec compliance - let me know if there's any more review you want to pick up before I merge this PR into the new branch

@nkitku I'm thinking the plan to move forward witht this branch could be for me to make you a collaborator, and for us to merge this into a new 5.x branch, we can continue to take PRs/Issues against that branch. We can have another PR open tracking a todo list for release; docs, tests, spec compliance - let me know if there's any more review you want to pick up before I merge this PR into the new branch

@thomseddon thank you, i think so too, so far so good

@nkitku @thomseddon I can help you guys out by testing as an end user and giving you the feedback on the same. Let me know when, and I will do it.

Is there any good fork of this, which got already publicated on npm?

Mostly for the record: After trying to get this running for a few days we re-evaluated our actual goals, and found that a OAuth server would not work in our environment. For a bit of context: https://medium.com/collaborne-engineering/replace-cognito-hosted-ui-d7619d037036

Hi, do we have an estimate release date on v 5.0 ? Or how can I help ?

Hi guys - the ongoing discussion has been started last year :smile: so when will this branch get merged?

Is there any significant issue preventing this from being merged?

Fyi, these are the plans for this project
https://github.com/oauthjs/node-oauth2-server/issues/621


BBA Changes

Changes made while attempting to get BBA to interface with the virtac.

The Python 3.7 build is failing due to pytest-flake8 which is abandoned, it's not worth fixing it on its own as it'd be resolved whenever we move ATIP over to the Diamond pip-skeleton/ copier template/new deployment system.

I don't think I can elaborate concisely on the issue with other cothread applications.
Side note, I noticed that the fix for the issue in AT that was blocking us from making I04 the default ring mode got merged, so I've updated it and snuck it on the end of this pr.
Also, for some reason, I don't have merge permissions for this repo or Pytac, but I do for dls-bba


[Feature Suggestion] Manual dark-light switch

Two buttons to switch manually between dark and light would be very helpful for testing the chosen themes. Also for debugging, I guess.
Like so, for example:
[ Dark ]  [ Light ]  [ Apply ]  [Cancel]

They were present in an older version, but I removed them because they kinda didnt make much sense - this program is for changing the theme automatically. If you want to debug, you can use yin-yang from the command line:
yin-yang --toggle

Ah, good to know. Thanks!

Hi @l0drex I'm surprised because that looks like a feature regression to me: I was using Yin Yang only this way, because I mostly use dark theme except in some specific cases.
I noticed that change yesterday, after yay was complaining that remote repo was different from local one, and had to reinstall yin-yang.
Are you sure a manual switch is not something users want?

Ok, that's a use case I didn't think of. I will re-add them in the next version.
I thought these were just for testing, so kinda unnecessary for users.


No subtitle option on mac

I am on Mac, latest version of davinci resolve and the script

The option to make text from subtitles is simply not there. I have the script downloaded and I open davinci through the resolvelauncher. Cannot figure out whats going wrong here

python is not installed or not visible from davinci resolve.
Did you see any errors in the python setup?

I got a different script working but for purposes of the forum, I re ran the python setup to verify- looks like this (below) may be the issue. Thanks
WARNING: You are using pip version 21.2.4; however, version 24.0 is available.
You should consider upgrading via the '/Users/NAME/Downloads/Resolve_Script2.5.10_mac/bin/python-3/bin/python3 -m pip install --upgrade pip' command.

Is python3 usable when davinci resolve is started normally?


Update CaptureScreen() use the scaled and some code

Please provide Issues links to:

Issues: #393

Provide test code:

Description
...

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Grade calculation can result in a larger grade than the max grade of an attainment

There are no checks in place to ensure that a grade calculation function can't calculate a grade larger than the maximum grade of an attainment. How should these cases be handled? Is rounding down to the max grade sufficient? Or should an error message be returned saying that something is wrong?

An error message should be returned.

It seems that indeed the final grade calculation doesn't take the max grade into consideration. Also, one can edit the grades (of sub attainments or final grade) to be over the limits. This should probably show an error also. Not sure yet whether checks are in place when importing the grades initially.


AdjustTokenPrivileges failed, error code: 1300

When running caissa-1.13.1-x64-avx2-bmi2.exe under Win11 I get following output:
AdjustTokenPrivileges failed, error code: 1300
Caissa 1.13.1 AVX2 by Michal Witanowski
info string Using embedded neural network

Engine seems to be working fine...
EDIT: do not have this on other machine with Win10
also found this, but not sure it this is related: https://devblogs.microsoft.com/oldnewthing/20211126-00/?p=105973

I will probably remove this error in next release, as it's not really an error.
It means that the engine failed to enable large pages feature. It should give small speedup when using big transposition table. You either need to run it with admin rights (not recommended) or add a privilege in security settings:

run secpol.msc
go to Security Settings > Local Policies > User Rights Assignment
double click on "Lock pages in memory" and add your user you log in with to Windows

I think many other engines also try to use the feature, but silently ignore it.

Fixed in 1.14

Still getting same warning in 1.15:
AdjustTokenPrivileges failed, error code: 1300
Caissa 1.15 BMI2 by Michal Witanowski
info string Using embedded neural network

Even if I run console with admin rights.
Just FYI, not that it bothers me in any way.

My bad, fixed in 1.16

thanks, works perfectly now


Using Time Steps is much more memory intensive than using Features

Please make sure that the boxes below are checked before you submit your issue. If your issue is an implementation question, please ask your question on StackOverflow or join the Keras Slack channel and ask there instead of filing a GitHub issue.
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

I'm trying to do time series prediction, and I'll apologize in advance if I'm doing this wrong.  From what I understand, there are basically 3 different ways to do it.
First, you can use a sliding windows approach which inserts the look-back data between each row of data, ie. if you have 100 rows of data and a look-back period of 10, using the sliding windows results in a matrix which is [1100, 1, 1].  All of the implementations of this that I have seen use an iterative process to insert the rows of look-back data, so I would prefer not to use this method.
Second, you can insert the look-back data as features.  In the example above, x becomes a [100, 1, 11] matrix.  The downside of this approach is that it is difficult to add additional "real" features.
Third, you can insert the look-back data as time steps.  In the example above, x becomes a [100, 11, 1] matrix.  This seems like it should be the best implementation, because real features can easily be added.
When I tried to implement the third approach, I am finding that it is much more memory intensive and I am getting Out of Memory errors with even a modest number of epochs.  In contrast, inserting the look-back data as features is much less memory intensive and runs much faster.  It is not obvious to me why that happens.
I created a toy example to demonstrate this.  Toggling line 21 from True to False switches the look-back data from being inserted as features to being inserted as time steps.  I would be most appreciative if someone could explain what I am doing wrong.
I'm currently running this on a cpu in Windows with 16GB of memory, and the toy example using time steps uses about 13.5 GB of memory after about 150 epochs.  (Without using TimeDistributed in the output layer.)  I plan to buy a high end gpu to run Keras, but I'm concerned that no gpu has enough memory to handle even this relatively simple model using time steps.
Here is my toy example:
"""
Comparison of using time steps vs features
In line 21, change features = True to features = False to see the difference in performance
"""

### DEPENDENCIES ###
import numpy
from keras.models import Sequential
from keras.layers import GRU
from keras.layers import Activation, Dense, TimeDistributed

### MODEL INPUTS ###
look_back = 10 # How much data to include in the time series
look_forward = 2 # How far into the future to make predictions

nodes = 4 * look_back # Number of outputs in each layer of the model (except the last)
sequential_type = GRU
stateful = True # Whether the model should be stateful
return_sequences = True # Whether sequences should be returned

features = True # Whether the model uses features for the time series
if (features):
    time_distributed = False
else:
    time_distributed = True # Slower and much more memory intensive. Can result in OUT OF MEMORY ERRORS

shuffle_data = False # Whether or not to shuffle training data at each epoch
activation = 'relu'
initialization = 'he_normal'
batch_size = 100 # Number of rows per batch

# Create some random data
dataset = numpy.random.rand(4343, 1)

dataset = dataset.astype('float16') # Use float16 to save memory

# Remove the last 20% of the dataset which will be used for testing later
# Also make sure dataset conforms to batch_size
new_size = int(len(0.8 * dataset) / batch_size) * batch_size + look_back + look_forward
dataset = numpy.delete(dataset, numpy.s_[new_size:len(dataset)], 0)

trainX = numpy.copy(dataset)

if(features):
    # lagged is just the next column being added to x_train
    lagged = numpy.copy(trainX) # Copy the list of values
    # Each time we go through the loop, we delete the first row of x_train.
    #   Then we delete the last row of lagged, to keep them the same shape, and
    #     the new lagged is the next column of x_train
    # This requires far fewer iterations than the window method
    for c in range (look_back):
        trainX = numpy.delete(trainX, 0, 0) # Delete the first row of x_train
        lagged = numpy.delete(lagged, lagged.shape[0] - 1, 0) # Delete the last row of lagged, to keep them the same shape
        trainX = numpy.append(lagged, trainX, axis=1) # Append lagged to X

    # ADD 3RD DIMENSION TO x_train
    trainX = trainX.reshape(trainX.shape[0], 1, trainX.shape[1])

    # fill in the y data
    trainY = numpy.copy(trainX[:, 0, trainX.shape[2] - 1])
    trainY = trainY.reshape(len(trainY), 1) # Turn the list into a 2 dimensional array
    time_steps = 1
    features = look_back + 1
else:
    lagged = numpy.copy(trainX)
    for c in range (look_back):
        trainX = numpy.delete(trainX, 0, 0)
        lagged = numpy.delete(lagged, lagged.shape[0] - 1, 0)
        trainX = numpy.append(lagged, trainX, axis=1)

    # ADD 3RD DIMENSION TO x_train
    trainX = trainX.reshape(trainX.shape[0], trainX.shape[1], 1)

    # fill in the y data
    if(time_distributed):
        trainY = numpy.copy(trainX)
    else:
        trainY = numpy.copy(trainX[:, trainX.shape[1] - 1])
        trainY = trainY.reshape(len(trainY), 1)
    time_steps = look_back + 1
    features = 1

# Adjust for look_forward
for c in range (look_forward):
    trainX = numpy.delete(trainX, trainX.shape[0] - 1, 0) # Delete the last row of x_train
    trainY = numpy.delete(trainY, 0, 0) # Delete the first row of y_train

# Round train_size
train_size = round(0.75 * new_size / batch_size) * batch_size
test_size = round(0. * new_size / batch_size) * batch_size

# Split data into train and validation sets
trainX, testX = trainX[0:train_size,:], trainX[train_size:new_size,:]
trainY, testY = trainY[0:train_size], trainY[train_size:new_size]

# Create the model
model = Sequential()
# Input Layer
model.add(sequential_type(nodes, batch_input_shape=(batch_size, time_steps, features)
    , stateful=stateful, return_sequences=return_sequences, init=initialization
    , activation=activation))
# Hidden layer
if(time_distributed):
    model.add(sequential_type(nodes, stateful=stateful, return_sequences=return_sequences
        , init=initialization, activation=activation))
else:
    model.add(sequential_type(nodes, stateful=stateful, init=initialization
        , activation=activation))
# Output layer
#if(time_distributed):
#    model.add(TimeDistributed(Dense(1, activation=activation)))
#else:
model.add(Dense(1, activation=activation))

model.compile(loss='mean_squared_error', optimizer='rmsprop')

model.fit(trainX, trainY, nb_epoch=200, batch_size=batch_size)

Hello,
You can use model.summary() to get information about your model.
It is useful to see where you are trying to allocate a big matrix.

Thank you.  model.summary shows that I am comparing apples to oranges, as the time step model is outputting a matrix that is 11 times larger than the features model.

Hello,
@Panache1
Did you find the solutions for this ?
If yes, Can you please tell me what did you find the right way to do, to use look backs as time steps is the right way or to use as features ?


FLAME indices correspondence

Hello.
I am using your FaceLandmarker found here: https://github.com/googlesamples/mediapipe/blob/main/examples/face_landmarker/python/[MediaPipe_Python_Tasks]_Face_Landmarker.ipynb
Currently I am trying to use the dense landmarks while training a FLAME-based 3D model. Is there such a correspondence between the dense landmarks you produce (478 points) and the FLAME vertices? This repo (https://github.com/Zielon/metrical-tracker/tree/master/flame/mediapipe) seems to have found a correspondence for 105 points, but not the entire 478 points.

So it sounds like the FLAME vertices aren't matched up to what we produce. We just made a visualization of the points by their index number, so will have that in the documentation soon. I'll post an update here when we have it.

I see. Thanks a lot for your prompt response!


CiderCollective.Cider(Canary) version upcoming-1dd31ea

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

do not commit yet pls


Update documentation teams channel member commands

All team channel member commands work with Teams shared channels as well. Right now we claim in the docs that they only work for private team channels.
We should update the docs of the following commands:

[ ] teams channel member add
[ ] teams channel member remove
[ ] teams channel member set

if they work then for sure we should add more clarity to the specs 👍 good catch 🙂

if they work then for sure we should add more clarity to the specs 👍 good catch 🙂

Yep they work, I've tested them 🙂

Good Morning I'm looking to contribute to some code for the first time and would love to assist in any way needed.

Good Morning I'm looking to contribute to some code for the first time and would love to assist in any way needed.

Hi @AaronS16
Awesome to hear that you want to contribute! Welcome!
All our contribution guidelines are listed here: https://github.com/pnp/cli-microsoft365/wiki
I was planning to update these wiki pages pretty soon. Would be awesome to rewrite them with your feedback. So I'd say try to get started with these pages. If you are stuck or anything is unclear, I'm happy to help.

Good Morning I'm looking to contribute to some code for the first time and would love to assist in any way needed.

Would you like to work on this issue? Then I'll assign it to you.

Good Morning I'm looking to contribute to some code for the first time and would love to assist in any way needed.

Would you like to work on this issue? Then I'll assign it to you.

Hey @milanholemans , if no one has been assigned , can I work on this issue ? . It would help me a lot in gaining experience in open source contributions

Good Morning I'm looking to contribute to some code for the first time and would love to assist in any way needed.

Would you like to work on this issue? Then I'll assign it to you.

Hey @milanholemans , if no one has been assigned , can I work on this issue ? . It would help me a lot in gaining experience in open source contributions

Hi @ryuusama09 thank you for your enthusiasm 😊
However I use the 'first come first served' principle, so let's wait for @AaronS16 to respond. If he is looking for another issue or doesn't respond for a week, I will gladly assign this issue to you.
There are plenty of other issues left to work on that are labeled as good first issue. Or are you looking for docs contributions specifically?

Good Morning I'm looking to contribute to some code for the first time and would love to assist in any way needed.

Would you like to work on this issue? Then I'll assign it to you.

Hey @milanholemans , if no one has been assigned , can I work on this issue ? . It would help me a lot in gaining experience in open source contributions

Hi @ryuusama09 thank you for your enthusiasm 😊 However I use the 'first come first served' principle, so let's wait for @AaronS16 to respond. If he is looking for another issue or doesn't respond for a week, I will gladly assign this issue to you.
There are plenty of other issues left to work on that are labeled as good first issue. Or are you looking for docs contributions specifically?

noted @milanholemans . I was looking for some  issues which dont require extensive domain knowledge , but not are of cakewalk difficulty .

o
Hello @milanholemans. Yes, please assign it to me and I will begin working on it.

Great, thank you for contributing! 👏

What should the Remarks be updated to say? Am I removing the line that states "You can only add members and owners of a Team to a private channel.", or am I also adding something to the remarks?

I'd say just add shared
"You can only add members and owners of a Team to a private or shared channel."

@milanholemans I made the changes and committed them on the branch I created. Please let me know if I need to correct anything.


Supporting temporary certificates with annotated gateways can be mana…

…ged by dynamically injecting certificates into your gateways, particularly in systems using Kubernetes and Istio or similar service meshes.

Pull Request Motivation

Kind

/kind

Release Note

NONE

You can already use cert-manager.io/issue-temporary-certificate: "true" to generate temporary certs for a gateway: https://cert-manager.io/docs/usage/certificate/#issuance-behavior-temporary-certificates-while-issuing


Initial loading indicator

While the initial assets for an app are downloading, there is nothing on the Alma screen except the app's title. An app with a sizeable main.js can take seconds to come to life.
Could we please get an initial loading indicator if an app does not come to life "quickly"?

Hi @SydneyUni-Jim . This was done in the latest release. Will take affect for each app on the next build without the need to bump the app to the latest SDK version.
Let us know how this works for you.

Hi @jweisman. To simulate a slow network or large payload, I cleared my cache and throttled my network to «Good 2G». Acknowledging this to be an extreme test, there was a significant period before the loading indicator appeared. When the indicator did appear, it was very brief before the app came to life.
I've shared a record of my screen at https://www.screencast.com/t/HKHVIn5E

Hi @SydneyUni-Jim - I would probably focus on optimizing for 3G since 2G sounds pretty extreme and unlikely for Alma users. Can you check with that throttling? For me it looks pretty good:
https://user-images.githubusercontent.com/80420/120099039-f86f2280-c141-11eb-828c-836eb389a52f.mov


Change file lookup to usage.perma_id

When moving entries from one environment to another, the ids of associated files change. Therefore we introduce a perma_id on FileUsage which stays unchanged during import/export and is then used for the file lookup in the importing environment

Coverage decreased (-0.001%) to 99.297% when pulling deb5fcad05e0efed03af6f8165dabb79147f7095 on DanielKuhn:change/find-files-by-perma-id into bdb506b9477b2d83b363679b336772b21b1cd70f on codevise:master.


.btn-large and span*

Buttons with both .btn-large and .span* classes applied have fixed height issue on mobile displays
The following code will fix it:
@media (max-width: 480px) {
	input[class*="span"].btn-large { height:auto; }
}

In 2.0.2-wip, the fixed height has changed to a min-height. No longer an issue.


Improve CI pipeline for PRs

Overview
This PR has a small improvement to the Elixir test suite, and adds the credo style check earlier in the pipeline. It also reconfigures the per-stage timeouts so that they ignore time spent in the build queue waiting for a docker-based agent to free up.
Finally, it refactors the pipeline to use a larger number of explicit steps in each stage. This improves the developer experience navigating the Jenkins UI, as we can see timings for each phase of the build, and if a step fails the logs that are displayed will be scoped just to that step.
Testing recommendations
Jenkins
Related PRs
#3895 was the first PR to convert the Jenkinsfile to use specific images for each Erlang version.

Went ahead and cherry-picked from #3903 to get this moving.


Strip back HTML views

Context
Our new frontend is the intended entry point for registers so all the contextual information relating to them has been moved to there.
Changes proposed in this pull request
Remove "fluff"
Guidance to review
Check all the views.

Can/Should remove custodianName from all view resources too

Can/Should remove lastUpdated from all view resources too

Can/Should remove registerLinkService from all view resources too

Can remove alot more methods from this file src/main/java/uk/gov/register/views/HomePageView.java

A

Have we made a decision to remove breadcrumbs from the API inspector?


Use kotlin compiler plugin and reflection to generate code [Moved from old repo]

Describe the problem or limitation you are having in your project:
Currently the api-classes-generator and the entry-generator use a errorprone and hard to understand code generation using stringBuilders.
The code would be much cleaner and easier to maintain if we use reflection to generate the api-wrapper and annotations for the entry-classes generation.
Also it would make the project easier to use as a user doesn't have to write config json's anymore.
Describe how this feature / enhancement will help you overcome this problem or limitation:
One no longer needs to configure the errorprone json files and maitanance is much simpler.
Show a mock up screenshots/video or a flow diagram explaining how your proposal will work:
None to provide.
Describe implementation detail for your proposal (in code), if possible:
Use the compiler plugin created in issue #13 for the config generation.
Maybe use kotlinPoet for the api classes generation -> done in #14
If this enhancement will not be used often, can it be worked around with a few lines of code?:
Yes with the current system. But it's not user friendly at all.
Is there a reason why this should be in this project and not individually solved?:
Yes as the code generation is the main part of the project.

i wrote in old repo:
Summarizing some discussions regarding this issue on discord:
Currently we have an annotation processor implemeted. Annotation processing with kotlin multiplatform and kapt only works if one has a jvm target. Our idea was to create a fake jvm target in the godot-gradle-plugin so the user can write his code in commonMain and we then can process the annotations through the fake jvm target to generate the registration code.
This works but leads to the fact that the user has to write a second native module if he want's to use some native libraries.
This and the fact that the user always writes native code anyways (as godot needs a native binary at the end), we decided that the best approach would be to implement a kotlin compiler plugin in which we do the annotation processing and thus code generation.
This would lead to complete freedom for the user as he can decide in which module he want's to write his code in.
Currently I'm now implementing this annotation processor.
Edit to the issue: moving annotation processor and thus compiler plugin implementation in a separate issue (#13).

i wrote in old repo:
Basic entry generation is done. I still need to do the following:

[ ] Code cleanup
[ ] Sanity checks
[ ] Remove old entry generation completely
[ ] Write unit tests

@piiertho I noted that even though the RPC stuff was documented, it was actually never properly implemented. There is a stale branch which did the basics with a new example: https://github.com/utopia-rise/kotlin-godot-wrapper/tree/withRPC
If you are ok with it, I'll implement all networking related registration logic in this issue as well.

Summation of discussions on Discord:
Everything is working now. But i have the problem that the MpApt Annotation Processor is too late in the compilation process to add the generated entry file to the sources to be compiled. This results in the need to build the project twice so the generated entry file is included in the final binary.
We discussed the problem on discord and i maybe found a solution (message from discord):
@Maintainers @Contributors I tried what i talked about yesterday and it seems to work.
Basically i just made a test script. I'm not yet 100% sure if it would work but before i continue i want to hear your opinion.
Basically what i've done is extracted part of the compiler frontend into a script so we can use it directly like we want: https://github.com/utopia-rise/godot-kotlin/blob/feature/rework_entry_code_generation/tools/entry-generator/src/main/kotlin/org/godotengine/kotlin/entrygenerator/parser/SourceCodeParser.kt
Important in this script is the parse function.
When running it, we get almost the same output as in the annotation processor and can process the information.
My idea:
Make a gradle task that runs before build. In such a gradle task we know which source sets are defined and can access all the source files in it.
The path's to those sourcefiles we pass to the extraced compiler frontend in that script and generate the entry file from it.
then the normal build runs and correctly detects our entry file.
Note: the script is just a test script and need's cleaning up but it does the job.
Why i'm not yet sure if this would work:

I didn't test every class in the samples yet

Drawback of this solution:
On compiler API changes i also need to change this script, but i think that's not that big of a deal as we only need to change it if we wan't to and is not necessary to bumpt the same way we bump the kotlin version on the other parts of the project.
otherwise i'm pretty sure it would work.
I'll continue with above described solution for now to see if it works.

Some updates (already shared through discord):
The above mentioned solution with the extraction of the compiler frontend works for the most part. But an important part does not. At the moment I cannot read annotation arguments as log as I don't add the source file's of the annotations to the parsing process (adding them to the classpath of the parsing source files does not work for some reason.
@piiertho had the idea of adding the annotations sources through the source jar manually. I'm trying to implement this now.
I this does not work. We probably have to rely on the expect/actual implementation in #20 and use a default jvm target to be able to use kapt annotation processing. But we only take this route as a last resort.

Some updates:
I got the parsing working with the idea from @piiertho. But it takes upfront 30s to just generate the entry file. So it's not feasable.
I now switched back to the previous attempt of using the compiler plugin.
By using a fake target which we define at built time in the gradle plugin, we compile the fake target first which generates the entry file and then run the actual compilation for the users targets to build everything.
As the compilation step is acually the shortest step atm (the linking step is the one that takes time) the time overhead seems negliable.
I implemented that now and my buildtime incresed from 1min 15s to 1min 30s.
Another good thing about this approach would be that it allows for easy switching to normal annotation processing once #20 and #10 is done.
I'll finish this now and we should have a beta to test up at around wednesday next week.

That's great news, once you get the new task in, I can change the task being called from the Editor Plugin in #33 (currently it's calling build)

@Wavesonics Then you dont have to change a thing. I configured it to be mandatory to execute before any kotlin compile task. So its executed during a normal build as well. So calling just build is fine.


Upgrade terraform-provider-cockroach to v1.11.0

Release details: https://github.com/cockroachdb/terraform-provider-cockroach/releases/tag/v1.11.0

[!NOTE]
This issue was created automatically by the upgrade-provider tool and should be automatically closed by a subsequent upgrade pull request.

Fixed by #59


Support Picoferm firmware

This past weekend I decided to try my hand at getting the picoferms to connect to the server... right now they are simply measuring the ambient temperature and pressure in my basement brewery... these hot days show that the basement is getting into the range for kveik yeast (probably the reason for all the advertisements!)

Don't forget to add the firmware selection in the config.example.yaml! #protip

Session logging from a Ferm on 0.2.6 (OTA) and another on 0.2.7 (unreleased).

You need AC stat! 🤣

You need AC stat! 🤣

Got a Glycol chiller right next to that picoferm... as well as a dehumidifier in the next room near the laundry and sink, but yeah a cold storage / walk in would be nice. How long will that take you to add to the server?

It'll definitely get in there before the 🍳🥓!


Get rid of ntuple warnings on Julia 0.4-x

It looks like most of #10 is already in master now, but not this bit.

Oops, I did this change against whatever version was checked out in my .julia - this is fixed.


Remove the concept of the PROJECT_ROOT

No python code should have to rely on its location, so having to declare a PROJECT_ROOT variable in the main package is a sign of weak points in the project.
In any case, any path variables should be computed in the scripts that use the atm package, not within it.

This goes hand-in-hand with an approach to rethink how configuration is supplied for the project.
Here is an example from Apache Airflow:

The path to the configuration file and work directory is computed: https://github.com/apache/incubator-airflow/blob/master/airflow/configuration.py#L425
The configuration file is read using ConfigParser https://github.com/apache/incubator-airflow/blob/master/airflow/configuration.py#L510
Any application code that needs configuration variables (like data directory etc.) does from atm import configuration as conf and then conf.get('DATA_DIR'): https://github.com/apache/incubator-airflow/blob/master/airflow/settings.py#L35


Disable right click or add a proper context menu

SyncTrayzor is great. I just have One small issue with it. I use this in windows x64, installed few days ago, and i have a habit of just right clicking. To be honest, when you see the folders/devices it feels like right click is a logical thing to do for editing/deleting etc.
Anyway, when you right click within the app, you get a typical windows/browser context menu that shows "back/forward/print/view source". The annoying thing is, this menu doesnt dissapear. No matter how many times i click within the app, away from the app, this menu just sticks at the top like a bug in windows. Only way to get rid of it is to actually click one of the options, and clicking "back" breaks the ui )(obviously).
I feel like this is probably a very quick flag you set somewhere to disable the menu (if its not providing any benefit anyway), is there any chance this can be done?
Cheers.

Ok odd, it definitely used to show a custom menu with Cut/Copy/Paste. An upgrade to the embedded browser must have broke that... Let me take a look.

@canton7 let me know if you would like a screenshot or anything.
Also, maybe this might be a feature request, but is there any reason why the installer doesnt set this up as a service or auto start with windows? Just did a restart (the PC) for the first time after installing and realised that it didnt start up, and i would probably need to setup the autostart manually if i want it to do that (which i would assume a very common usecase to expect auto start).

I can reproduce the issue here, so that's fine.
It does have autostart functionality. The installed version will set itself to start on login in the first time that it's run; the portable version won't set itself up by default. There's a checkbox in File -> Settings -> Automatically start on login.

@canton7 Oh yes i see the option. Thanks.
May i suggest making that option ticked by default? TBH, not sure why anyone wouldnt want it to auto start haha :P

It is ticket by default in the installed version. The portable version is supposed to be put on portable media and moved between machines: there's not much point in setting up autostart for something which probably won't be plugged in!

@canton7
i can verify that using the latest installer mentioned below, i did not get the auto start ticked by default. I installed it on 3 machines (i havent tried any other clients on 2 of the machines), and fresh download each time, and definitely not the portable download, each with the same problem. Not a big deal for me, as i can easily tick that now i know where the option is, but just pointing it out as its probably worth looking at :)
Installer i used: https://github.com/canton7/SyncTrayzor/releases/tag/v1.1.12
*X64.exe (the windows x64 exe file)
Each of the machines were using Windows 10 X64.
PS. thanks for the advice and help

Oops, looks like I broke the automatically enabling autostart... Let me fixed that!

If you want a fixed up context menu, try this: https://ci.appveyor.com/project/canton7/synctrayzor/build/<IP_ADDRESS>0/artifacts.

@canton7 perfect, context menu works.
im curious, this project is pretty much a wrapper for the default UI that syncthing provides right? Meaning that you guys cannot do anything to do with the UI changes to the app (besides the UI of the wrapper like the window's menu and the console etc). Is that correct?

Yes and no.
In principle I just display the html which Syncthing's GUI gives me. In reality, I run some very hacky javascript when the page is loaded, which does things like add the 'Browse' button next to the input box for file paths, and the 'Open Folder' buttons on the folders.

I also just downloaded a fresh copy from this site and the right-click context menu does not disappear. This is on Windows 10 x64
Regarding the browser, I would suggest a preference to specify the default browser we want to use, in case something happens to the Trayzor browser like this.

I also agree with syrys: This no-thinking packaged version of syncthing is fantastic.

I also just downloaded a fresh copy from this site and the right-click context menu does not disappear. This is on Windows 10 x64

Weird, OK thanks for reporting. That's another data point...

Regarding the browser, I would suggest a preference to specify the default browser we want to use, in case something happens to the Trayzor browser like this.

It uses Windows' default browser. I don't see the point in overriding this? I just ask Windows to open a URL, and it figures out how best to do that based on your preferences

I'm confused. When I double-click the tray icon, it brings up the
SyncTrayzor browser, no? I like this (in particular the console at the
bottom), but may want to change it in the future.
On Tue, Oct 18, 2016 at 3:38 PM, Antony Male<EMAIL_ADDRESS>wrote:

I also just downloaded a fresh copy from this site and the right-click
context menu does not disappear. This is on Windows 10 x64
Weird, OK thanks for reporting. That's another data point...
Regarding the browser, I would suggest a preference to specify the default
browser we want to use, in case something happens to the Trayzor browser
like this.
It uses Windows' default browser. I don't see the point in overriding
this? I just ask Windows to open a URL, and it figures out how best to do
that based on your preferences
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/canton7/SyncTrayzor/issues/300#issuecomment-254615981,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH5Fp0qt_JP3PhH3hv8zeAvfNOkuldq-ks5q1SAwgaJpZM4JXxL2
.

Right, I didn't get that you wanted a different browser to open instead of SyncTrayzor.
Two thoughts:

Double-clicking the tray icon hasn't broken for you, has it? Nothing's stopping you from launching and using SyncTrayzor from the tray icon. It's just the convenience shortcuts when you right-click the tray icon which has gone wonky.
Not being able to launch SyncTrayzor kinda defeats the entire point of SyncTrayzor. There are plenty of other minimal tray icons which just launch an external browser when clicked. Use one of them.

You're totally right, though I would argue that synctrayzor does something
much more important: bundling, and lowering the install / config barrier.
Figuring out everything necessary to get syncthing, a nice tray, and a
monitor to work together, and configuring autostart may seem trivial /
worth it to people already comfortable with syncthing, but can be daunting
to those who are not even sure if syncthing is the right system for them.
It is a big deal to migrate data and trust a new system!
If this solves my problems with resilio sync, you and the syncthing crew
can expect some beer money :)
-Jack
Sent from sender's phone with typos. Sender not liable for inflammatory
autocorrects.
On Oct 19, 2016 3:18 AM, "Antony Male"<EMAIL_ADDRESS>wrote:

Right, I didn't get that you wanted a different browser to open instead
of SyncTrayzor.
Two thoughts:

Double-clicking the tray icon hasn't broken for you, has it?
Nothing's stopping you from launching and using SyncTrayzor from the tray
icon. It's just the convenience shortcuts when you right-click the tray
icon which has gone wonky.
Not being able to launch SyncTrayzor kinda defeats the entire point
of SyncTrayzor. There are plenty of other minimal tray icons which just
launch an external browser when clicked. Use one of them.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/canton7/SyncTrayzor/issues/300#issuecomment-254732167,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH5Fp6RpfaECH69Emjh6BoKIdt4J_-1nks5q1cQwgaJpZM4JXxL2
.


map not working

'oops something went wrong!'

try a new PTC account. had to replace 3 today

what is a PTC account?
On Sunday, 14 August 2016, wartortle87<EMAIL_ADDRESS>wrote:

try a new PTC account. had to replace 3 today
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mchristopher/PokemonGo-DesktopMap/issues/903#issuecomment-239655110,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT6DOPOw7KBAhc9_gU-7WTbDPW3Fjn2Bks5qfpRigaJpZM4Jjzbp
.

Pokémon Trainer Club


Migrate ListTile disabled icon color to Material 3

Updates ListTile default disabled icon color to the Material 3.
Fixes https://github.com/flutter/flutter/issues/99400
Part of: https://github.com/flutter/flutter/issues/91605
Disabled icon specs from https://m3.material.io/components/buttons/specs

Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Just resolved conflicts


Back-End: Admin - Players

Create Model, Controllers, Routes to Players (Artilheiros)
GET api/playerGoals/ (list of players artilheiros)
-- [ player: { id:number, name:string, team:string }, goals: number ]
PUT api/playerGoals/ (update one player artilheiro)
-- {id:number , goals:number }

api/topScorer


Add slash commands support

Happy to help test if this feature is in progress :)

Hi @mikedotexe!. It will be very helpful! Sorry for the extreme delay in the new version, I can't focus too much on the library, but as soon as I finish the migration to the new Discord.js version and add the tool to test slash commands, I will release a preview version so you can test! Thank you so much!

Would love to see slash commands support too !
Thanks for your work :)

Hi guys, unfortunately, there is no way (for now) to make corde test slash commands. Discord API doesn't allow bots to send slash commands. So, the only thing that is possible to do is to check if a slash command exists, its params, etc. But not test the action directly :(
I can not figure out any workaround for now so we will have to wait for Discord to allow bots to send slash commands to move on with this.

Is there a feedback post on Discord's support page that we can upvote?

@AndyClausen I just created a feature proposal in discord discussion.

For those of us desperate to have something, anything working, here's a wild approach for Mac users: use AppleScript.
https://user-images.githubusercontent.com/1042667/154858995-476dc708-32ad-424b-b3c8-3638f9b40b1c.mp4

Hey folks, I ran into the exact same issue while writing tests for cordless, unfortunately I don't think there's anything we can do here. Bots are not able to use commands by design, I doubt Discord will budge on the issue.

Run chrome headless. So a real user will send messages, the bot will respond to them, and corde will check the response of the bot

This could work, but is most likely against the Discord ToS so you risk getting banned. Even if it's not against the ToS, it will be really painful to implement and maintain, and will probably be unusable in a CI/CD pipeline since you'll start getting captchas.
My current line of thought is to just query the Discord API for my bot's commands and save the response as a snapshot. This will at least allow me to assert that the commands were registered correctly. Of course, that still doesn't allow me to test how the commands are handled by the bot.

For those who are Jetbrains users and also want to test Slash Commands, I have started prototyping the use of Selenium to run through my test sets.
https://www.youtube.com/watch?time_continue=4&v=VDpkfjEwzro&feature=emb_title

Create yourself a test user
Add your test bot to a test server
Have the tests log in as your test user and run the commands you want
Test the output.

There are so many implementations of Selenium out there, I use this as I love Jetbrains tools and that plugin makes code production a lot quicker. It needs tweaking from time to time.

@lucasgmagalhaes  What if the dev could attach their discord.js instance (if they were using discord.js) and then corde would fire a fake ApplicationCommand interaction event.
Sure it only supports discord.js but djs is the largest library for js.


brige: fix refreshing cloud access token

Summary by CodeRabbit

Refactor

Improved token refresh logic by introducing a configurable interval, enhancing device cloud communication reliability.

Codecov Report
Attention: Patch coverage is 83.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 76.81%. Comparing base (0563a7a) to head (2facd0c).

Files
Patch %
Lines

bridge/device/cloud/manager.go
83.33%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #452      +/-   ##
==========================================
- Coverage   76.83%   76.81%   -0.02%
==========================================
  Files         115      115
  Lines        6699     6702       +3
==========================================
+ Hits         5147     5148       +1
- Misses       1014     1015       +1
- Partials      538      539       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


refactor: applied clang-tidy

Signed-off-by: Mamoru Sobue<EMAIL_ADDRESS>Description
I applied clang-tidy to the prior PR and updated the code.
Tests performed
I could not suppress this warning eventually, but I think this is C-specific matter.
 warning: do not use pointer arithmetic [cppcoreguidelines-pro-bounds-pointer-arithmetic]
  const std::string map_path = std::string(argv[1]);

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[X] I've confirmed the contribution guidelines.
[X] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@mitsudome-r can you approve this PR?

@soblin Unfortunately, CI is still failing.
https://github.com/autowarefoundation/autoware_common/actions/runs/4529440040/jobs/7977270986?pr=128

@kenji-miyake There was no way to suppress clang-tidy error for argv[1] because main(int argc, char **argv) forces its usage. I added NOLINT block as a workaround, and the CI passed.