Router with / in their name messes up with dashboard

Do you want to request a feature or report a bug?
Bug
Did you try using a 1.7.x configuration for the version 2.0?

[ ] Yes
[x] No

What did you do?
I've clicked on following HTTP router

What did you see instead?

Output of traefik version: (What version of Traefik are you using?)
2.0.1

What is your environment & configuration (arguments, toml, provider, platform, ...)?
    [log]
      level = "INFO"
      format = "json"
    [accessLog]
      format = "json"
    [entryPoints.http]
      address = ":80"
    [entryPoints.http-internal]
      address = ":81"
    [entryPoints.https]
      address = ":443"
    [entryPoints.traefik]
      address = ":8080"
    [entryPoints.traefik-ext]
      address = ":8081"
    [providers.kubernetesIngress]
       ingressClass = "traefik"
    [providers.kubernetesCRD]
    [providers.file]
      directory  = "/config/static"
      watch = true
    [api]
      insecure = true
      dashboard = true
      debug = true
    [metrics.prometheus]
      entryPoint = "traefik"
      buckets = [0.1,0.2,0.3,0.4,0.5,0.6,0.7,0.8,0.9,1.0,1.5,2.0]

Ingress kubernetes:
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    kubernetes.io/ingress.class: traefik
  creationTimestamp: "2019-07-31T14:42:45Z"
  generation: 2
  labels:
    app: pwet
    release: pwet
  name: pwet
  namespace: pwet
  resourceVersion: "90246831"
  selfLink: /apis/extensions/v1beta1/namespaces/pwet/ingresses/pwet
  uid: 7553afb4-b3a1-11e9-a70e-000d3a482661
spec:
  rules:
  - host: pwet.com
    http:
      paths:
      - backend:
          serviceName: pwet
          servicePort: 8080
        path: /
      - backend:
          serviceName: pwet
          servicePort: 8080
        path: /api/v1/
  tls:
  - hosts:
    - pwet.com
    secretName: pwet-tls
status:
  loadBalancer: {}

If applicable, please paste the log output in DEBUG level (--log.level=DEBUG switch)
(paste your output here)

Related to: https://github.com/containous/traefik/pull/5612/files
Although there are other cases where a / can end up in the router name

Closed by #5612.

Still happening in 2.0.2

@dduportal could you re-open please ?

@sylr can you provide a reproduction case please?

Ref #5623

Closed by #5623.


Support astroquery v0.3.10

Over in #595 it transpired that the current development version of astroquery renamed MastClass()._resolve_object() into MastClass().resolve_object().
This PR makes Lightkurve robust against the change, i.e. it will try both options.
/cc @mustaric

@gully can you review and merge if tests pass? Thx++

@mustaric this is now live on master so you can follow our developer version installation guide to get the bleeding edge lightkurve.  Depending how you previously installed lightkurve, you may wish to uninstall the old version.


FormBuilderCheckbox: initialValue default  was removed

initialValue default  was removed in FormBuilderCheckbox because value provided in _formState.initialValue was ignored

Thanks for your contribution @cgalan12


Use gRPC to query Cosmos SDK chain states

Crate
relayer
Summary
This is a follow up to #1048. Since we can do height-based gRPC query through the x-cosmos-block-height parameter, we can refactor other queries such as query_channel to use the gRPC endpoints instead of the Tendermint RPC endpoint.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate milestone (priority) applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

We'll be able to increasingly use gRPC now with the recent changes adding gRPC support for ABCI queries in the SDK https://github.com/cosmos/cosmos-sdk/pull/11642

Relevant comment by @ancazamfir.


CustomObject created on v61 can't be retrieved with source tracking

Summary
Creating a new CustomObject in Summer'24 (v61) in a source tracked environment (scratch org) can't be retrieved with
sf project retrieve start
Steps To Reproduce

Create a new scratch org with Summer'24 ("release": "preview")
Create a new custom object in the Salesforce Setup
run command sf project retrieve preview -> will display all changes including the CustomObject
run command sf project retrieve start -> will fail to retrieve the CustomObject

NOTE:
the "Fullname" of the CustomObject is lacking of __c hence the error in the retrieval
see screenshot below

IMPORTANT
https://github.com/ChristianMenzinger/v61CustomObjectIssue

NOTE: empty project which only contains Layout and Profiles, which have been successfully retrieved
Expected result
CustomObject is also retrieved by CLI
Actual result
CLI throws an error Entity of type 'CustomObject' named 'TestObject1' cannot be found
System Information
zsh 5.9 (x86_64-apple-darwin23.0)
{
  "architecture": "darwin-arm64",
  "cliVersion"<EMAIL_ADDRESS>  "nodeVersion": "node-v20.12.2",
  "osVersion": "Darwin 23.4.0",
  "rootPath": "/Users/christianmenzinger/.local/share/sf/client/2.40.7-bde121f",
  "shell": "zsh",
  "pluginVersions": [
    "@oclif/plugin-autocomplete 3.0.16 (core)",
    "@oclif/plugin-commands 3.3.1 (core)",
    "@oclif/plugin-help 6.0.21 (core)",
    "@oclif/plugin-not-found 3.1.7 (core)",
    "@oclif/plugin-plugins 5.0.18 (core)",
    "@oclif/plugin-search 1.0.23 (core)",
    "@oclif/plugin-update 4.2.9 (core)",
    "@oclif/plugin-version 2.0.17 (core)",
    "@oclif/plugin-warn-if-update-available 3.0.16 (core)",
    "@oclif/plugin-which 3.1.8 (core)",
    "@salesforce/cli 2.40.7 (core)",
    "apex 3.1.11 (core)",
    "auth 3.6.5 (core)",
    "data 3.3.2 (core)",
    "deploy-retrieve 3.6.8 (core)",
    "info 3.2.4 (core)",
    "limits 3.3.4 (core)",
    "marketplace 1.2.4 (core)",
    "org 4.1.5 (core)",
    "packaging 1.26.4 (user)",
    "schema 3.3.4 (core)",
    "settings 2.2.3 (core)",
    "sobject 1.3.6 (core)",
    "source 3.3.5 (core)",
    "telemetry 3.3.4 (core)",
    "templates 56.2.4 (core)",
    "trust 3.6.7 (core)",
    "user 3.5.4 (core)"
  ]
}

Additional information

this wasn't a CLI bug, but the people responsible for it say they've git it fixed.

@ChristianMenzinger I got the same issue, How you resolve it, could you share it please?


Provide option to slow or rate limit requests

I've been testing out scrapix and first off, awesome work! With a little bit of tinkering around I got it working with meilisearch cloud FAST!
That said, it could be useful to add an option to rate limit request. I didn't see anything other than the batch_size which I believe has more to do with how frequently documents are imported into the search index.
This isn't as big an issue when it comes to indexing internal websites, but as I was testing it out on a rather large public collection of docs (reactnative.dev), it quickly stared denying my requests. Likely because scrapix was firing off LOTS of requests which might look a bit like malicious traffic.
Apache Nutch has a default rate limit of 5000ms (which in my opinion is a bit high). It could be a good idea to implement something like this for scrapix if it doesn't already exist. I could potentially implement it if you guys are welcoming PRs?

Hello @klvs
Thanks for the suggestion.
Pinging @meilisearch/product-team here to let them know 😇


TypeError:  PreTrainedTokenizerFast._batch_encode_plus() got an unexpected keyword argument 'tokenize_newline_separately'

I would like to conduct object detection task by utilizing a VQA model using autotrain API. I followed this guide. Accordingly, I prepared the metadata.json properly. Three columns are "file_name", "question", "multiple_choice_answer".
Sample format from the dataset:
{"file_name": "1.mrxs__12214_50922_512_512.png", "question": "This image is from 3DHistech Scanner. Where is the mitosis location(four properties of the bounding box: top left x coordinate, top left y coordinate, width, height) in this image?", "multiple_choice_answer": [[181, 199, 43, 42]]}
I tried to use google/paligemma-3b-ft-coco35l-448 and google/paligemma-3b-mix-448 models for this purpose. When I start the process with this command: autotrain --config config.yml
It loads the dataset properly. Everthing seems fine until the training started:

Here is the error:

still an issue


How to change/set sys_clk frequency

Hi all,
I am looking at overclocking my sys_clk to speed up the pio state machines, at for example, 250 Mhz.
I am using the standard 12Mhz XOSC on board and want to either configure the SYS_PLL, or sys_clk at runtime.
I have the following:
        // Configure the clocks
        let clocks = hal::clocks::init_clocks_and_plls(
            XTAL_FREQ_HZ,
            c.device.XOSC,
            c.device.CLOCKS,
            c.device.PLL_SYS,
            c.device.PLL_USB,
            &mut resets,
            &mut watchdog,
        )

in clocks/mod.rs the trait has ClocksManager and I wonder if i can simply do:
        self.system_clock
            .configure_clock(pll_sys, pll_sys.get_freq())?;

and instead of pll_sys.get_freq() replace to HertzU32::MHz(248)?
If not, what is the correct mechanism to configure this? All help is appreciated.
Thanks
-Dmitri

It's not exactly as simple, as the clock source passed to configure_clock must have a higher frequency than the target frequency. So if you want to overclock, what you need to do is increase the frequency of pll_sys.
You can find a nice example how to do that in the Neotron BIOS:
https://github.com/Neotron-Compute/Neotron-Pico-BIOS/blob/develop/src/main.rs#L346-L404


Using DataStore and custom mutations/queries in the same project via AppSync in iOS

Hi everyone.
I am currently working on a project where we are utilizing DataStore to its fullest extend. However we recently ran into a problem where we need to utilize custom mutations/queries, which are unfortunately not functioning with the DataStore logic, but generated in the legacy API.swift file.
So my question is, is it possible to use the DataStore API logic while also using the AppSync client? And are there any examples on this? This does not seem to be clear in the documentation, and it is lacking these use-cases side-by-side.
If the answer is "no", is there a workaround to call these custom mutation/queries?
UPDATE: On this page for iOS it describes three concepts for AWS AppSync, we are using the Amplify DataStore, but as far as I understand we also need AWS AppSync SDK, however we recently noticed that API.swift and the generated models are somehow conflicting (see the screenshot of API.swift in Xcode)?
Best regards, M.
Which Category is your question related to?
Amplify and iOS
Amplify CLI Version
7.6.11
What AWS Services are you utilizing?
AppSync

Using AppSync SDK can get the work done if your specifically hitting other AppSync APIs that do not conflict with DataStore, but it's not a set of dependencies we're tested fully (which is probably why you are seeing some type conflicts). We'd like to support deveopers to use API plugin and DataStore in the same app, and have some plans for this (marking this issue as feature request). If it's a conflict in the naming of types generated for DataStore and API.swift, can you update API.swift manually to see if you can be unblocked? If you are infact just hitting one endpoint, you can also do custom graphql documents via https://docs.amplify.aws/lib/graphqlapi/advanced-workflows/q/platform/ios/

We actually made a workaround by implementing the GraphQL calls manually with the AppSync SDK by writing them inline, not super optimal to be honest as I would expect the API.swift and DataStore to co-exist. :-/

Glad to hear you got it working, we would like to deprecate AppSync SDK and have developers directly use API via Amplify.API for the same use case, along side DataStore

Would be great to have that feature!

if this is still relevant for you, we are introducing a compatibility layer to use API.swift with Amplify.API, please track the release of the documentation describing the Upgrade Guide in here https://github.com/aws-amplify/docs/pull/5649


About the unconditional_recursion warning in primitives.rs

Can you have a look at this please?
https://github.com/rust-lang/rust/issues/21705
Is @huonw right?

Yes he is, there should be another deref there. Never actually tested boxed parsers since I did not have any need for them. Box trait objects should be useful but it seems that there may be another bug for them https://github.com/rust-lang/rust/issues/21379.
Fixed in https://github.com/Marwes/parser-combinators/commit/20d35d26527e9c7edcb5d3632d21bd2b7782d5a7.

OK. Isn't this the same case just above? https://github.com/Marwes/parser-combinators/blob/20d35d26527e9c7edcb5d3632d21bd2b7782d5a7/src/primitives.rs#L227

Nope, there is a &mut P there which gives
*(&mut P)  == P
The bug was because of &mut Box:
*(&mut Box) == Box (which derefs to &mut Box again)
**(&mut Box) == P

I see. I think I had something like this in some code where I played with box mysqlf and did not recognize this pattern there. Thank you for the explanation!

Lots of edits there sorry, my brain didn't keep up with the typing since I already knew that there is not issue since I am using the &mut P impl in a few places.

What I took out of it is the part that if it is "borrow boxed" there are two * needed to get to the content.


Consider replacing our auth system with OAuth2

Over the years, we've evolved our own authentication/authorization system in the shape of the login api, user-interactive auth, access_tokens, and soft logout.
The current implementation is largely functional, but it does duplicate much of what OAuth2 does, and as ever, particularly when considering security-sensitive subjects, it's better to use standardised protocols (even when they are as sprawling as OAuth2) than to roll your own.
In addition, there are a bunch of things which we could improve in the current system, but which OAuth2 already has answers for.
I've started a document at https://docs.google.com/document/d/1e-ZqghugaHaRukb88PRmPFdm24-y-Ea5ka89IrAEZ7s to collect notes on what might be better with OAuth2, and how it might work in practice.

For Synapse, this is how I see it. There are two scenarios:

Synapse acts as a authorization server (AS) in simple deployments, with whatever password backend. It stores its own access tokens and manages them, so it can verifies them just by checking the DB
auth is delegated to another authorization server. In that case, Synapse acts as a resource server and needs to check somehow the validity of the access tokens presented. To do that, there are multiple mechanisms that exists:

the access token is a (short-lived) JWT, verifiable by its signature: https://tools.ietf.org/html/draft-ietf-oauth-access-token-jwt-04 (it's a draft, but implemented by a lot of AS)
the AS implements the OAuth2 introspection endpoint and that endpoint is accessible by Synapse. In that scenario, the token is opaque to Synapse, and can POST the token to that endpoint to check what scopes are allowed
if neither of those are available, if the AS is OpenID Connect-compliant and the token has the openid scope, Synapse can query the userinfo endpoint with the token. It can't check for scopes there, but at least it can check for token validity.

Delegating the auth to a third party authorization server would make it a lot easier to implement policies and auth methods (2FA, FIDO, passwordless logins) specific to a deployment without having to implement them in Synapse. A lot of those identity providers have years of development and security hardening that would benefit many deployments.

Another thing to consider is to keep a lot of the existing login logic while slowly implementing most of OAuth2. As an access token is bound to a specific client, existing clients using the existing login mechanisms would be identified as a special "legacy" client that issues token without expiration (because current access tokens don't have expiration nor any mechanism to renew them).
In this case, it would be even possible for Synapse to issue tokens from another AS using the password grant (if supported by the AS)

Last but not least, it would be interesting to see what it means for integrations. I've seen in Synapse's code that it implements a basic OIDC userinfo endpoint because of integrations ; there is probably something to investigate there.

oauth2 could be a uia stage, but uia allows greater flexibility.
At work we use it for additional verification steps to only allow login to "good" servers from our app

and as ever, particularly when considering security-sensitive subjects, it's better to use standardised protocols (even when they are as sprawling as OAuth2) than to roll your own.

That's not quite right - the good practice is to use battle-tested protocols. Unfortunately, that set does not fully overlap with "standardized protocols", and in fact quite a few standardized protocols designed by non-cryptographers are infamous for bad cryptography and insecure API design (see eg. the history of weird and broken crypto in WEP/WPA).
Notably, JWT is terrible from a security perspective and likewise OAuth2 has been frequently criticized by people in the security community for having a high degree of complexity, resulting in a risky attack surface.
I honestly don't think it's a good idea to use JWT or OAuth2 for usecases other than those where they are necessary for interoperability reasons.


Can't use enums as sortKeyFields

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v16.13.1
Amplify CLI Version
8.0.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
no
Amplify Categories
api
Amplify Commands
push
Describe the bug
It's this #1572 but with @index
Expected behavior
You can use enums as sorts
Reproduction steps
Make an enum a sort key
try to push
GraphQL schema(s)
No response
Log output

# Put your logs below this line

Additional information
No response

Hey @CodySwannGT :wave: thanks for raising this! I will mark this as a feature request for GraphQL Transformer v2, and this seems related to https://github.com/aws-amplify/amplify-cli/issues/2099

Seconded.  Have been waiting for this for almost a year ..


Write unit tests to verify the profile update and professional status upgrade functionality

create unit tests for all the aspects related to the new feature

The various subfeatures implemented in #23 , #21, #19, #14 all featured their own unit tests that satisfy this issue

Implemented the following tests:

test_send_professional_status_update_email
test_update_user_professional_status_allowed
test_update_profile_user_not_found
test_update_profile_invalid_data
test_update_profile_missing_auth
test_update_profile_success


Add support for TS namespaces

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

Project coverage is 81.80%. Comparing base (b924c29) to head (34e0946).

Files
Patch %
Lines

src/Escalier.TypeChecker/Env.fs
49.01%
26 Missing :warning:

src/Escalier.Interop/Infer.fs
78.68%
13 Missing :warning:

src/Escalier.TypeChecker/Visitor.fs
90.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #162      +/-   ##
==========================================
- Coverage   82.02%   81.80%   -0.23%
==========================================
  Files          21       21
  Lines        7383     7500     +117
  Branches     1050     1096      +46
==========================================
+ Hits         6056     6135      +79
- Misses       1327     1365      +38

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


No way to set fcolor back to base color setting

After setting the flashlight color in the terminal to any other applicable color, it doesn't seem to be possible to revert that change back to the base color that it was previously.
Looking inside the switch statement at https://github.com/darmuh/TerminalStuff/blob/2.0.X/DarmuhsTerminalCommands/MyTerminalPatcher.cs#L60
there is no case for white or default of whatever the normal color is so it gets set to null which will return a failure prompt in the terminal
https://github.com/darmuh/TerminalStuff/blob/2.0.X/DarmuhsTerminalCommands/MyTerminalPatcher.cs#L437
Would be great to run a command at the terminal to see a list of the available colors as the "sasstro" color was quite nice but only found out about it after reading the code.

good suggestions! I think i'll try to push both of these in the next patch as long as I dont run into any issues.
As for now, you can see which colors are implemented in a portion of the readme, though I know most people dont read that lol

I've created a list command to see available colors via  that will be in the next patch.
I've also added a default/normal setting for when you want to set it back to white/normal.
I also changed how the flashlight color changes because the old version was having issues setting the color until the player had already turned the light on and switched to helmet light. It now should work at any point as long as you have a flashlight in your inventory. I've also confirmed this works with FlipMods' "ReservedFlashlightSlot" Mod.

@dconner4
Just pushed this change in patch 2.0.3
Let me know if you run into any further issues 👍


InMemoryProviderRulesDataSource and InMemoryRulesStorage AsEnumerable()

InMemoryProviderRulesDataSource.cs
InMemoryRulesStorage

InMemoryProviderRulesDataSource methods are returning methods with ToList().AsReadOnly(), I would propose just use .AsEnumerable(). Since is already readonly.
InMemoryRulesStorage are return methods with .ToList().AsEnumerable(), I would propose just use .AsEnumerable().
That would slightly improve performance.

Hi @carlosgoias,
I understood your suggestion, conversion between RuleDataModel<TContentType, TConditionType> and Rule<TContentType, TConditionType> ensures a read-only effect already. Makes total sense 👍


Remove rust toolchains to save space in the docker image

The rust toolchains take around 2GB of space.
[dependabot-core-dev] ~/dependabot-core $ du -sk /opt/rust/toolchains/*
1067576 /opt/rust/toolchains/1.58.0-x86_64-unknown-linux-gnu
1067648 /opt/rust/toolchains/stable-x86_64-unknown-linux-gnu

Removing them slims down the image from 4.3GB to 2.3GB!
This is an experiment to see if we can still do cargo updates.

This worked but I could see that the cargo tests were taking longer to compete than normal. Taking a closer look I found that the necessary toolchain would be downloaded on demand so we are incurring a performance hit by not including a toolchain in the image. I took another stab at this in https://github.com/dependabot/dependabot-core/pull/4711 which installs a single minimal toolchain rather the the two standard toolchains we currently install.


Fix Sass imports in Parcel example for Windows

Closes #321
Description
This PR addresses issue #321.
On macOS, there are no problems—everything compiles and launches as expected. However, on Windows, we were able to replicate the issue with @louismaximepiton.
The import statement @import "bootstrap/scss/bootstrap" works correctly in our step-by-step guide because it references the bootstrap.scss file in the node_modules directory. However, when attempting something similar to the Parcel example, such as @import "bootstrap/scss/functions", it fails. This is because Parcel or the Parcel Sass Transformer cannot locate the file named _functions.scss.
(it's referenced at several locations—e.g. https://github.com/parcel-bundler/parcel/issues/6002, https://github.com/parcel-bundler/parcel/issues/3704, etc.)
While Sass handles this by default on other systems, it fails to do so with Parcel on Windows for these specific imports.
We considered three potential solutions to fix this issue for Windows users:

Use @import "~/node_modules/bootstrap/scss/functions";
Use @import "../../node_modules/bootstrap/scss/functions";
Use @import "~/node_modules/bootstrap/scss/_functions";

We opted for the simplest approach, which doesn't require users to know the exact filenames (including underscores) or the precise relative paths.
Upstream Parcel Guide
The upstream documentation for Parcel says:

You can also import our stylesheets individually if you want. Read our Sass import docs for details.

In the link, we use a relative path to target the files, so it'll work as well. I don't think it's needed to modify the upstream guide.

@XhmikosR FYI, if you want to double-check this PR. Otherwise, it's ready to be merged :)

Weird that parcel doesn't work, but I confirm this fixes the issue on Windows.


Input component page needs to be updated to reference forum issue

Input component page needs to be updated to reference this forum issue
Originally posted by @bameyrick in https://github.com/ONSdigital/pattern-library-v2/issues/163#issuecomment-463206719

 Issue #164: Input component page needs to be updated to reference forum issue


chore(web): support plugin block in story

Overview
This PR supports plugin block in story
What I've done

Move StoryPanel into Crust.
Add support for render block in story.
Remove installable story block limit.
Support plugin instance for story blocks.

What I haven't done
How I tested
Which point I want you to review particularly
Memo

It's difficult to solve the conflicts, i made another PR instead.


No option to update the item price, when a discount must be applied.

No way I could update the item price, when I have this discount that should reduce the price.
Yeah, this can be done if I remove the item and re-add it again but in my case, I put a few important options on each item and I don't want to re-push again those options. See, you have an option (of course) to change the qty but none for the price. I am looking forward for the solutions. Thanks.

@clemquinones And why would Cart::update($rowId, ['price' => 10]); not work for you?

Thanks for the response. Yes it is working, I can change the price. But do you have a method to modify or push a new key-value pair on the item's options?
For example on my case, if I have to update the price that should also update the is_discounted key on its options.
Given the item options of: is_special, is_addon, is_discounted
This updates the is_discounted but also replaces the old options. it doesn't merge the options:
Cart::update($rowId, [
    'price' => $discountedPrice,
    'options' => [
        'is_discounted' => true
    ]
]);

I have same problem with this. ><


Validate device token

I would use API Endpoint for device token validation to make sure device token is valid. If not I would remove it from our database.

Do you mind elaborating on this a bit more @swigrid?

here is an example
when there is one invalid token as below
{
"notification_target": {
"type": "devices_target",
"devices": [
"0f1ea4a3-caee-4980-8de2-91908e8fe32b",
"aaaa"
]
},
"notification_content": {
"name": "Delivery Group id 81",
"title": "Delivery Group",
"body": "aaaaa"
}
}
we’re getting response
{
"error": {
"code": 400,
"message": "body parameter is invalid or empty."
}
}
Which is not correct, body has valid string. Is push delivered to other valid tokens?
Regards
Roman

On 12 Jun 2019, at 18:58, Patrick Nikoletich<EMAIL_ADDRESS>wrote:
Do you mind elaborating on this a bit more @swigrid https://github.com/swigrid?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/appcenter/issues/553?email_source=notifications&email_token=ABIGVUUVFPONKEQ46JR6HZLP2ETLHA5CNFSM4HXJFZLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXRDCYQ#issuecomment-501363042, or mute the thread https://github.com/notifications/unsubscribe-auth/ABIGVUXNMZC2KMEZUHVLTMLP2ETLHANCNFSM4HXJFZLA.

in other words, from time to time can happen, there is an invalid token saved from whatever reason. We would like to clear those tokens

Makes sense! Thanks for the clarity. We'll take a look on our side.

@swigrid I'm checking to see if we can provide a better error message for this.

@swigrid how would you use this?  Validate each token before any send events, or a scheduled task you run periodically validating all the tokens?  Would this be a per token API call, or would you send in batches?

I am not sure yet. I just wanted to get rid of invalid tokens. Since this feature haven’t been available I haven’t really thought about logic. But if there was an array response with statuses after push was sent, this way we could remove those invalid ones. Just a thought perhaps.

@swigrid OK, thanks for that feedback, that makes sense. I'll track this and see what other customers have to say about this one as well.

@swigrid would you be willing to have an email or voice conversation to talk about your push needs beyond this feature? I don't see your email associated with this profile, mine is, so you can email me directly.

I’m sorry, I forgot to reply to this one.
I could have spare few minutes tomorrow in the morning London time.
but in the mean time…
how would suggest deal with given scenario:
User logs in one device, there is one token registered
User logs in in second device, there is another token registered
user will receive push on both devices
User uninstalls app on first device, token is left on the device (developers claiming, there is no callback when uninstalling app, to remove token from backend server)
user installs app again later, and we have 3 tokens in DB and one of the token is for uninstalled app.
how can we get rid of this token?
Regards,
Roman

On 9 Sep 2019, at 16:27, John M. Wargo<EMAIL_ADDRESS>wrote:
@swigrid https://github.com/swigrid would you be willing to have an email or voice conversation to talk about your push needs beyond this feature? I don't see your email associated with this profile, mine is, so you can email me directly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/appcenter/issues/553?email_source=notifications&email_token=ABIGVUWFX6NYWVJZRKNPV7TQIZMMZA5CNFSM4HXJFZLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6HYJHY#issuecomment-529499295, or mute the thread https://github.com/notifications/unsubscribe-auth/ABIGVUTDJSQLMAHBRBZR54DQIZMMZANCNFSM4HXJFZLA.

@swigrid the token will be deleted automatically on our side, but I understand your need to validate it on your server side. What I'm proposing to engineering is that when you check push status, we return a list of invalid tokens. this works, I think, because we get that back from the PNS, so we'll know those tokens are bad.
Regarding a chat, can you email me using the email address on my GitHub account so we can arrange a time and I can send an invite?

That’s good, this way we can clean up our tokens.
Thanks
Roman
Sent from my iPhone

On 25 Sep 2019, at 15:26, John M. Wargo<EMAIL_ADDRESS>wrote:

@swigrid the token will be deleted automatically on our side, but I understand your need to validate it on your server side. What I'm proposing to engineering is that when you check push status, we return a list of invalid tokens. this works, I think, because we get that back from the PNS, so we'll know those tokens are bad.
Regarding a chat, can you email me using the email address on my GitHub account so we can arrange a time and I can send an invite?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

regarding email, this what I have

On 25 Sep 2019, at 15:26, John M. Wargo<EMAIL_ADDRESS>wrote:
@swigrid https://github.com/swigrid the token will be deleted automatically on our side, but I understand your need to validate it on your server side. What I'm proposing to engineering is that when you check push status, we return a list of invalid tokens. this works, I think, because we get that back from the PNS, so we'll know those tokens are bad.
Regarding a chat, can you email me using the email address on my GitHub account so we can arrange a time and I can send an invite?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/microsoft/appcenter/issues/553?email_source=notifications&email_token=ABIGVUXJ3XMEJGH357ABAZDQLNRHHA5CNFSM4HXJFZLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7R4CZI#issuecomment-535019877, or mute the thread https://github.com/notifications/unsubscribe-auth/ABIGVUTOKJDKZEUM5OQRUL3QLNRHHANCNFSM4HXJFZLA.

If you click on my name, it should show my email address. it's firstname dot lastname at Microsoft.

@swigrid Thank you very much for your feedback on App Center Push. We announced yesterday in a blog post that we’re retiring the Push service later this year and recommending customers transition to Azure Notification Hubs instead. The good news is that the same team operates both App Center Push and Azure Notification Hubs, so all customer feedback and feature requests have been transferred to the Azure Notification Hubs roadmap.


chore: create try from referance of casm contract class method

This change is

#910  👈
main

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

Join @ArniStarkware and the rest of your teammates on  Graphite

Merge activity

Sep 25, 5:55 AM EDT: @ArniStarkware started a stack merge that includes this pull request via Graphite.


fixes bug 1109892 - comma separate emails and locations in combined assi...

...gnment ical

Coverage remained the same at 90.33% when pulling 654b8372672bc17f67cfc60871651ac61ed4d838 on peterbe:bug-1109892-comma-separate-emails-and-locations-in-combined-assignment-ical into 356c7eb7d275a81ba2442b0e356f8be8c2de204d on mozilla:master.


Remove hashes::hex::HexWriter

The HexWriter is not used any more since we added the new hex code in internals for fast hex encoding.
While we are removing the benches for HexWriter also remove the last remaining bench for writing using Display because this is not the correct place for that code - its trivial to re add later in the correct module.

For the hex crate we should have this available but in improved form: generic over fmt::Write. Same with reading and iterators.

Am I understanding you correctly, you are happy to remove this now but I should raise an issue to re-implement it in the hex crate generic over fmt::Write?

Exactly.

Done, thanks.


Add handing of onnx metadata

What does this PR do?
This PR slightly modifies python OV adapter to read an analog of rt_info from onnx.
Technical details:

rt_info is stored in onnx_model.metadata_props
key of the metadata property is a string containing space-separated xml tags from the original rt_info path, for instance, model_info confidence_threshold, model_info hierarchical_config. All values of attrs are stored as strings, as in rt_info

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

Implementation is finished, now waiting for the model to be uploaded to s3.

@Wovchena tests are passing now


iostats are still incorrect

zpool iostat -y <pool> returns the current stats, this is not suitable as zabbix queries this value every 60 seconds.
zpool iostat -y <pool> 1 1 returns the average stats in the last second. This is better but also not a 100% accurate.
zpool iostat <pool> returns the average stats since the last boot. Multiplying this with the uptime in seconds it should be correct. Me and some other people believed this to be overall, not average: https://github.com/openzfs/zfs/issues/12866
The template should use zpool iostat <pool> * uptime using a Calculated Item.

Closed with https://github.com/FloFaber/zabbix_zfs-on-linux/commit/3b9f12886a1aebb9f4a2285be6f9135df01068d4

Closed with https://github.com/FloFaber/zabbix_zfs-on-linux/commit/3b9f12886a1aebb9f4a2285be6f9135df01068d4


Steam p2p testing example

Updated the simple box example to work with steam p2p :)

i'll create a pull request on the lightyear repo directly 😄


samples: kernel: metairq_dispatch differ the assert msg

Differ the __ASSERT_NO_MSG of the k_msgq_get() just after checking the nb of cycles.
It allows precise counting of cpu cycles where the assert does not interfere.
Fixes https://github.com/zephyrproject-rtos/zephyr/issues/55399
Signed-off-by: Francois Ramu<EMAIL_ADDRESS>

@FRASTM can you remove the draft status?


Versioning

Versioning ontologies during development can be difficult.
One should use semantic versioning (specification).
When making it clear it's a development version, one can then up the major, minor or patch version and add something like -dev.
However, this is not viable for the simple reason that the actual next version may differ from the used new version.
I will suggest an alternative to this: A combination between semantic versioning and calendar versioning (specification).
This could be in the form: MAJOR.MINOR.PATCH-YYYY.MM.DD.X or similar, where MAJOR, MINOR, and PATCH refers to the semantic version, i.e., the latest properly released version.
YYYY, MM, and DD refers to the date of the developer release, and X refers to the release number on that day. One could leave out X if it's the first release of the day and later add it (setting it to 1) if another release is published on the same day for the same semantic version.
The meaning of the version number is then the following for end-users: The semantic version reflects actual semantic content changes or fixes, while the calendar version reflects newer perspectives or patch updates of the original/first release of the affiliated semantic version.
To hammer through the "patch" update part one could remove the semantic PATCH and instead use calendar versioning for that part.
Another interpretation of the version number could be: The semantic version reflects the latest proper release version, while the calendar version reflects a (development) update to this release version. If done properly, the end-users should always refer to only semantic versions of the ontology (i.e., where the calendar version is all 0, e.g., 1.3.2-0000.00.00 aka. 1.3.2).

You are right that this is both important and difficult. It's something that we need to dedicate more attention to, especially now that we are converging on stable releases across the EMMO universe. Personally, I think using a combination of semantic and calendar versioning would be more confusing than helpful. But one way or the other, we should use the same versioning system across all of EMMO. For that reason, I recommend moving this topic to the EMMO.


Mobile Menu Accessibility Improvements

Menu's in general should use ARIA states to indicate when it is open too assistive technology.
States that may fit are:

aria-controls,
aria-lebelledby
aria-expanded.

You can see an example here: http://heydonworks.com/practical_aria_examples/
The new Post Settings Menu should also probably:

Esc -> close menu, or move back a submenu.
Focus should move only within the panel (effectively, i think it should be treated like a model box focus wise, see this: http://www.smashingmagazine.com/2014/09/15/making-modal-windows-better-for-everyone/)
Focus should be pushed to the container which contains the menu when the settings button is clicking
The settings button should probably have alt text equivalent to "open post settings menu" rather than "settings or cog" etc.

See discussion in #3954 and https://github.com/TryGhost/Ghost/pull/4042 for more details.
Kind regards,
Jamie + Lion

This issue and it's sibling #3964 are now a year old and in that time we have completely rebuilt the UI. I've had a go with voiceover myself and fixed a few issues with skip links, but this isn't my strong suit.
It would be great to have a fresh audit done, and PRs with improvements would also be welcome, but as these particular two issues are now quite out-of-date I'm going to close them.


Incorrect config_dir on Windows 10

Not sure if I'm doing something wrong or what, but I'm consistently getting "C:\" from config_dir() when running on Windows 10. This happens both in develop builds and release builds.
I'd be happy to provide more info, but I'm not sure what info would be helpful.

Hey @SenojLuap,
could you provide the output of all functions (similar to https://github.com/dirs-dev/dirs-rs/blob/main/src/lib.rs#L292) as well as the version of Windows you are using?

Turns out it was my own fault. I was using the resulting PathBuf incorrectly, and it was clearing out the value from config_dir().
Sorry for the false alarm.

No worries! Glad that things worked out for you! :-)


DPE: Use DPE as a dependency crate

Rather use DPE as a crate dependency than a git submodule, as it makes it easier to handle.

DPE contains both go and rust code, which CI both use. Maybe it's possible to parse the cargo lock file and update the go deps for CI, but then a submodule might be easier after all.

DPE contains both go and rust code, which CI both use. Maybe it's possible to parse the cargo lock file and update the go deps for CI, but then a submodule might be easier after all.

Cargo can print out the used DPE version which in turn can be used for the golang dep.

@ArthurHeymans The issue is that there's still a fair bit of churn, so there are likely to be changes in that repo that will break the build here. Using a submodule allows us target a specific commit so we can wait to update the submodule in a PR that also fixes up any of the API calls from caliptra-sw

Cargo also features this "cargo update -p dpe --precise " and the Cargo.lock file should use only a specific commit.

@ArthurHeymans The issue is that there's still a fair bit of churn, so there are likely to be changes in that repo that will break the build here. Using a submodule allows us target a specific commit so we can wait to update the submodule in a PR that also fixes up any of the API calls from caliptra-sw

Cargo also features this "cargo update -p dpe --precise " and the Cargo.lock file should use only a specific commit.

We also need to keep the golang dependency in sync with the rust dependency. This seems more complicated than just using a submodule.
Is there a specific thing you're trying to do that is difficult with a submodule?

@ArthurHeymans The issue is that there's still a fair bit of churn, so there are likely to be changes in that repo that will break the build here. Using a submodule allows us target a specific commit so we can wait to update the submodule in a PR that also fixes up any of the API calls from caliptra-sw

Cargo also features this "cargo update -p dpe --precise " and the Cargo.lock file should use only a specific commit.

We also need to keep the golang dependency in sync with the rust dependency. This seems more complicated than just using a submodule.
Is there a specific thing you're trying to do that is difficult with a submodule?

Not really but I just often forget to do update submodule.

Probably not worth it as both rust and go programs depend on the submodule


_TZN3000_cfnprab5 / TS011F

Is it possible to add this 4 socket power outlet to the compatible devices?

Hi @Grobaa, are you sure the manufacturername is _TZN3000_cfnprab5 with the N after _TZ ?
I have already added _TZ3000_cfnprab5to the app since before.

Hi Johan
You are absolutely right. And I have managed to add it to my Homey.
Thank you for doing a great job btw

_TZ3000_cfnprab5 isnt being detected when trying to find a new device with Tuya Zigbee App


Add check for --fsck with over 10000 archives

This isn't part of our test suite yet, but it needs to be.

To clarify: the command-line app is fine.  I'm concerned that something might be going wrong with the messages between the command-line and the GUI.
In other words, look at CmdlineTask, TaskManager, and the various widgets that react to them.


⚠ Bofrost.de has degraded performance

In 2f3481c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1612 ms

Resolved: Bofrost.de performance has improved in b7c6bbf after 6 minutes.


[INFRA] test bidsschematools  against osx and windows as well

Inspired by https://github.com/bids-standard/bids-specification/issues/1294 and thus should fail one test on windows ATM.
attn @TheChymera . I am ok if fix for #1294 would simply absorb this PR

ok -- this one is good, but I think we should not merge it since fails on Windows. @TheChymera - please base your fix on top of this PR branch, meanwhile I will just close it (branch should still lurk there)


:hover pseudo-class still applies in Safari after moving the pointer away over a tooltip

Hi,
I noticed that when using Safari and hovering over a tooltip in to another element hover state does not disappear from previous element. I made simple JSFiddle from the issue. I also checked Wall of browser bugs and webkit bug list, but I didn't find any similar issue.
I know that this is most likely Safari bug, but I post this over here, because someone may encounter this when using Bootstrap.
This video shows how to reproduce the issue safari_hover_issue.mp4
Machine: OSX El Capitan 10.11.5 (15F34)
Safari: Version 9.1.1 (11601.6.17)

Confirmed. Seems to be a bug in Safari's :hover pseudo-class implementation. :hover shouldn't simultaneously apply to multiple unrelated elements like that.

Filed WebKit bug: https://bugs.webkit.org/show_bug.cgi?id=158340

Fixed in WebKit Nightly: http://trac.webkit.org/changeset/202324


calling "api/register" even with correct data feilds gives error

I called "api/register" with following json object
{     "owner" : "builtin",     "username" : "sourabh",     "password" : "123",     "name" : "sourabh mandal",     "email"<EMAIL_ADDRESS>    "phone" : "999" }
But I got the following error

The GetOwnerAndNameFromId() expect a username : "sourabh/organization
The route works fine when I pass following payload
{     "owner" : "builtin",     "username" : "sourabh/organisation",     "password" : "123",     "name" : "sourabh mandal",     "email"<EMAIL_ADDRESS>    "phone" : "999" }
Please tell me if its an error in CODE or the JSON PAYLOAD

I would suggest refactoring GetOwnerAndNameFromId() to eleminate this error

@sourabhmandal this is not a bug. And we don't have enough time to do the input checking for now because this is endless.
Plz do some issues listed here: https://github.com/casbin/casdoor/issues


How to create a contract and store / retrieve data from it?

I have been looking through all the documentation and can't seem to find a good example of creating a basic contract and storing simple data (like a hex value or string).
Can anyone give me the gist or point me towards a tutorial / code sample to accomplish this (relatively) simple task?
This is as far as I have gotten:
def eth_black_magic(hex_value):

    web3 = Web3(HTTPProvider('http://localhost:8545')) # what URL should I point at?

    user1 = 'xxx' # what goes here?
    user2  = 'xxx' # what goes here?

    transaction = {'from': user2, 'to':user1, 'data': hex_value}

    transaction_hash = web3.eth.sendTransaction(transaction)

Answered the same question on stackexchange: https://ethereum.stackexchange.com/a/25522/1461
We can continue the discussion there.


MWPW-158652 [MEP] skip rows missing an action value

Currently, a MEP manifest action row with no value in the action column outputs a message in the console. Instead, the action should just be skipped.
Additionally, if there is an "invalid" action, the message should only be displayed for internal users. aka if we are going to show the MEP button.
QA steps: The "After" link should not log "Invalid action found." To test that it still logs the error message in preview mode, remove the mepButton parameter.
Resolves: MWPW-158652
Test URLs:

Before: https://main--cc--adobecom.hlx.page/drafts/mepdev/fragments/2024/q4/skipmissingaction/base?mepButton=off&martech=off
After: https://main--cc--adobecom.hlx.page/drafts/mepdev/fragments/2024/q4/skipmissingaction/base?milolibs=fixmissingactionerror&mepButton=off&martech=off
Psi-check: https://fixmissingactionerror--milo--adobecom.hlx.page/?martech=off

closing and will reopen after adding new/additional requirements.


dart2js: used-type inference for hidden native classes during codegen

Instances of hidden native classes can reach the Dart program via four paths:
(1) returned from a native method or getter,
  e.g. aha = document.getElementById();
(2) passed to functions which escape to the native world
  e.g.  element.addEventListenter((e) => doIt());
(3) passed to methods on objects which escape to the native world.
(4) anything transitively reachable from the results of (2), (3) also escapes.
For dart:dom and dart:html we only need to consider (1) and (2), since all objects in class (3) have been typedef-ed as callback functions, and all callbacks return simple types.
This analysis will prevent the gratuitous generation of code like the following since in most programs, Float64Array will be determined to be unused:
// ********** Code for _Float64ArrayJs **************
var _Float64ArrayJs = {};
$dynamic("get$length").Float64Array = function() {
  return this.length;
}
$dynamic("$index").Float64Array = function(index) {
  return this[index];
}

@rakudrama - seems like this is now been done for quite some time? I'm closing the issue assuming so, but please reopen if not.


Get started with PP stdlib specs, especially the PP::pp method

Looks good!
Could you reduce the PR to just the library/pp/pp_spec.rb file? It seems PP does not use private methods very well and the automatic generator does not seem so useful in this case.

@eregon - I used the output matcher for the 1st spec. The only problem for the other is that the main reason I'm doing this is to improve Opal's compatibility with MRI. Opal doesn't support File I/O and new_io, output_with_fd, etc. all need temporary files in order to work.

I see, would using IOStub (defined in mspec) work?
It's a simpler StringIO and it means we don't rely on the impl having available StringIO.

@eregon - No problem, I think IOSub will work just fine, pushed a new commit with that

@wied03 Great, thanks and merged!

Thank you for you contribution!


Weird error ? Url not relative?

Describe the bug
I am getting this in the logs:
Unexpected error for call_service at pos 1: URL is relative, but does not start with a /
YAML is simple enough:
- id: '1638533164647'
  alias: Weekend relaxing music afternoon
  description: Play weekend relaxing music at sunset
  trigger:
  - platform: sun
    event: sunset
    offset: '0'
  condition:
  - condition: time
    weekday:
    - sat
    - sun
  action:
  - service: media_player.play_media
    target:
      area_id: dining_room
    data:
      media_content_id: shuffle relaxing music playlist everywhere
      media_content_type: AMAZON_MUSIC
  mode: single

To Reproduce
Steps to reproduce the behavior: See above

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

Expected behavior
Not get this message
Screenshots
If applicable, add screenshots to help explain your problem.
System details

Home-assistant (version): Latest
Hassio (Yes/No): (Please note you may have to restart hassio 2-3 times to load the latest version of alexapy after an update. This looks like a HA bug).
alexa_media  (version from const.py or HA startup): Latest
alexapy (version from pip show alexapy or HA startup): Latest
Amazon 2FA is enabled (y/n). We will not debug login issues if unanswered: Yes

Logs
Please provide logs. We'll be most likely asking for them anyway.
Logger: homeassistant.components.automation.weekend_relaxing_music_afternoon
Source: components/media_player/browse_media.py:43
Integration: Automation (documentation, issues)
First occurred: 20:13:40 (2 occurrences)
Last logged: 20:13:40
Weekend relaxing music afternoon: Error executing script. Unexpected error for call_service at pos 1: URL is relative, but does not start with a /
While executing automation automation.weekend_relaxing_music_afternoon
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 379, in _async_step
await getattr(self, handler)()
File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 582, in _async_call_service_step
await service_task
File "/usr/src/homeassistant/homeassistant/core.py", line 1634, in async_call
task.result()
File "/usr/src/homeassistant/homeassistant/core.py", line 1671, in _execute_service
await cast(Callable[[ServiceCall], Awaitable[None]], handler.job.target)(
File "/usr/src/homeassistant/homeassistant/helpers/entity_component.py", line 204, in handle_service
await service.entity_service_call(
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 677, in entity_service_call
future.result()  # pop exception if have
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 955, in async_request_call
await coro
File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 714, in _handle_entity_call
await result
File "/usr/src/homeassistant/homeassistant/components/cast/media_player.py", line 643, in async_play_media
media_id = async_process_play_media_url(self.hass, media_id)
File "/usr/src/homeassistant/homeassistant/components/media_player/browse_media.py", line 43, in async_process_play_media_url
raise ValueError("URL is relative, but does not start with a /")
ValueError: URL is relative, but does not start with a /
Additional context
Add any other context about the problem here.

Please don't use the word "latest' when describing versions. Are you on dev? The beta? Stable? Latest can mean any of those.
Does the service call work by itself in dev tools?

Sorry, my bad. Yes stable.
So the call works in the automation, and also in the service call. But when i execute the service call in developer tools, it shows:

I'm really looking for a version number for each component in the form. Regardless, please read the wiki as your search doesn't match the standard syntax for Amazon music. https://github.com/custom-components/alexa_media_player/wiki#play-music

Hi Alan, version numbers jotted down. Appreciate your help. I guess it's a recent thing - i've never seen this before. And the automations still work as mentioned, just that strange message in the error log.
On the side - I don't see any difference in the link you posted, in fact that is what i setup my automations with. What do you see different ? I use exactly the same syntax.

We don't use area_id last I checked.

Hmm weird. I see what you mean. I just changed that and it works and no error message. Maybe to soon to tell, but usually it would appear right after the action.
I've never seen this error message appear before though. Did something like that change in recent releases ?

Yes, they added the ability to target "areas" in HA. However, we've never supported that. You have to target entities. I guess we can treat this as an enhancement request.

Thanks Alan, i'll change my automations to use that, seems like it does the trick.


User/dhwanishah/publish cli version 0.2.5

azurestackhci

@kairu-ms all checks have passed, please merged the PR when you get a chance. Thanks!


Introduction to Classes and Objects

Select the Day of Content
Day 9
What information is missing?
Object-Oriented Programming (OOP) Fundamentals: Briefly introduce the core principles of OOP, including encapsulation, inheritance, and polymorphism. Explain how these concepts differ from procedural programming.
Defining a Class: Demonstrate how to define a class in C++, outlining its members (data variables and member functions) and access specifiers (public, private, protected).
Creating Objects: Show how to create objects (instances) of a class, and how member variables and functions are accessed through the object.
Simple Class Example: Provide a well-commented code example illustrating the creation and use of a basic class, such as a Point class with x and y coordinates.

@subhadipbhowmik Please assign this task to me under gssoc24.


Fix a few typos

You may also wish to change "it's" to "it is" and similar
for other contraction which normally should be avoided in
formal written english such as in a manual.

Thank you! I'm just used to writing a bit more casually, since this isn't a formal publication or anything like that :)


Add custom dispatcher to Arraybuffer & Arraybuffer_prototype

JerryScript-DCO-1.0-Signed-off-by: Virag Orkenyi<EMAIL_ADDRESS>

Please can you change the commit title to  Add custom dispatcher to Arraybuffer & Arraybuffer_prototype

LGTM (informal)

Please rebase.


Accessing field labels in your code

I know you can use the {{> afFieldLabelText}} helper to access a field's label inside a template. But is there a way to do it from within your own code? As far as I can tell the label is not part of the context that's passed to the form templates.
This is related to https://github.com/aldeed/meteor-autoform/issues/483, I wanted to try using labels as an internationalization key to translate them in the template.

Through simple-schema, you can do mySchema.label(fieldName), which is reactive. If you don't necessarily know the form's schema, then you'd first have to figure out that.

Thanks! But actually, how would I figure out the schema? Is there anything in Autoform to help do that, or should I figure it out some other way (based on the route, etc.)?

OK, so this is a little tricky… It seems like mySchema.label(fieldName) will fall back on the capitalized, prettified field name if it can't find a label.
Which in most cases would be desirable, but for my specific purposes I want to fall back on the actual field name, which I then use as an i18n key.
So I guess unless there's a way in Autoform to access the label property itself and know when it's undefined, I'll have to just look it up on the global schema object. Sorry if this is a bit confusing…

It's not yet released, but in the next version of autoform I'm working on, there is AutoForm.getFormSchema(formId), which can also be called without passing a formId when you're within a form's reactive context, and then will return the schema for the closest form.
So you could then do AutoForm.getFormSchema().schema(fieldName).label, and then check whether it's a function and if so, call it. That's the equivalent of what mySchema.getDefinition does, minus the label inflection. You'll also be missing the label reactivity, though.
If you want to add an option to getDefinition and label functions to skip the inflection, I would merge that PR. Relevant code is here: https://github.com/aldeed/meteor-simple-schema/blob/master/simple-schema.js#L820

Done in AutoForm 5.0, now released.


Update esbuild-plugin-require-resolve/index.js to have kind specified

Without this parameter, I'm getting Must specify "kind" when calling "resolve"  error for some requires:

Adding kind propertly seems to resolve the issue

Hello @amadare42, thanks for your contribution. We already included this change in our previous release.


Improve the implementation of dask.array.block

https://github.com/dask/dask/blob/14b4c3848516a0f03bcf1b40e3f1f7475131497c/dask/array/core.py#L2899
I recently rewrote what will likely be numpy's new implementation of block. @eric-wieser suggested an enhancement which should make it really trivial for dask to piggy back off the results and use it in it's implementation.
https://github.com/numpy/numpy/pull/11971#issuecomment-421910996
The first time I tried to use dask.block, it was really slow, so I had to write my own. I think this might help it since it avoid calls to concatenate all together.
Not sure about unknown dimensions.

Improvements are always welcome.  Two questions:

That implementation seems to use __setitem__ syntax, which probably won't be as useful for dask array.  Does that affect your plans?
Why was dask.array.block slow?  Was it slow in graph construction overhead, or in execution?

Eventually we will have a helper function that returns the dtype, shape, slices and numpy arrays. These could easily be dask arrays. An other function will take all of those and create a new array and do the assignment. I Think that is within the realm of dask since assignment only happens to newly created arrays.

Not too sure. It probably wasn't any slower than block. I think ultimately it failed at handling one of my cases, so I couldn't use it.

I just wanted to point somebody to a no concatenate solution to this problem.

This might also be helpful in replacing the current concatenate3 function in dask.array.core, which also does a no-concatenate blocking solution, but is probably uglier than what numpy has.  If numpy.block is now convenient and efficient then it might be good to switch over, especially if there is a performance increase.

Any update on this @hmaarrfk ?

I started with a benchmark. It shows basically that it suffers a little bit from overhead. But not from copying data.

FWIW a lot of the code for block was lifted from NumPy. This was done to avoid NumPy usages of asarray, which trigger a computation, or things like concatenate, where a Dask friendly implementation is needed. If there have been changes in NumPy, that would help Dask, it is probably reasonable to update our vendored copy. Better yet if we can consolidate code between the two, but that likely isn't possible without NumPy addressing asarray and concatenate for this use case in a nicer way.

Checking in here.  Any update on this @hmaarrfk ?  Is there anything that you are blocked on here on the dask side?

No, more I'm waiting for the numpy PR to go through. A few peculiarities have come up.

OK, just checking in.  Thanks for the update.

Yeah, so while the dask operation is built on multiple dask implementations
of concatenate, those aren't actually costly as data isn't copied until the
very end.
Maybe graph operations can be reduced, making the graph cleaner, but it
isn't something I am enthousiastic about because that numpy PR took so long
to get through, probably in large part due to my novice development style.
I am secretly hoping now that numpy is expanding their high level
interface, that maybe we can expose the internal algorithm of block and use
that for dask. Dask would depend on numpy 1.16 or 1.17
I'm not sure what would happen when you block two dask arrays with chunks
that don't line up. An operation that is probably allowed now.
On Mon, Oct 22, 2018 at 5:12 PM Matthew Rocklin<EMAIL_ADDRESS>wrote:

OK, just checking in. Thanks for the update.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/3987#issuecomment-431988631, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFfmMnFsj5OHyn1oZegBF6catxuy0dfks5unjTFgaJpZM4WrZoo
.


Add brandid to product update

Noticed we where missing the brandid on the update method of product.
Add the change checks to all the update functions to avoid change if no change exist, also cleaned up the structure of Product to match Brand, noticed you like things tight and tidy :D so I did that on the update method as well.

Thanks! Do let me know if there is anything else that you noticed.


fix: add postpublish script

:tada: This PR is included in version 1.1.3 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Filelist state not cleared on logout

Describe the bug
Log in and click show the list of files. Then logout, login again with a different user and the same list of files shown before will be shown if it was left open by the previous logged in used.
To Reproduce
Steps to reproduce the behavior:
1.Login
2. Upload files
3. Click show files
4. Log out
5. Successfully login again with a different user
6. The same file list of the previous logged in user would be seen
Expected behavior
The login state and the files should all be reset after logging out

Implemented this via window.reload


goodbye react-router

You interfere with the policy with open source, today you disabled documentation and tomorrow what ? delete the repository ?

 Just because they've taken their fancy frontend for the documentation down, it doesn't mean they "interfere" with the policy of open source.

@iUltimateLP Then why don't they remove the repo instead, and ask people to check the documentation from the code in node_modules instead?

I'll not say "goodbye react-router" according to issue title because this is great software. Nevertheless, I have to say that now I'm confused if react-router is truly open. Is it really open if somebody can disable documentation during workday (yup, in Europe we have work hours now).
I think that open source community is not the right place for USA political battles. There are a lot of users all over the world that just want to build and use software. Today's blocked documentation affects people that has nothing in common with problems in the USA.

Showing solidarity could have been done in a much better way. I'd not have cared if they show me a huge popup always when I come to the page, and still allow me to access the website.
Wondering what would have happened if search engines and stack overflow started behaving kiddish like these guys.

I understand the point which is made here. And I also agree with @sirajulm that a popup would have been enough. Still, I'm not sure if this really affects the state of "open source-ity" in thie repo. But, it's not a great time to discuss so I hope that this state will not hold on any longer than a day.

Documentation !== political battle

I agree with OP.  It's the same as if protesters are blocking road for emergency service. People rely on React Router but this shows they can't rely for it longer. I will be looking into moving from React Router to Reach Router https://github.com/reach/router

@MariuzM Moving to Reach Router is kind of dumb because it has the same people behind and support will be dropped when RRv6 is released.

Hm what are other options? Surely RR can't be the only one?

@MariuzM I would say calm down because the source code, the documentation and the package are still accessible after all. Just the additional webpage is "down". Therefore use the documentation here on GitHub and you're good to go.

As a Russian frontend developer I'm baffled by the decision to block the entire doc just to spread awareness to a foreign political problem.

@pshrmn
yeah good idea just close topic 🥇

I'm falling to see how this message join us with financial support gives this repo open source status? I understand it's noble cause but this sounds more like send money or you can't use our software

@pshrmn that is, African-Americans have the right to speak out and you support them by disabling documentation, but Europeans do not have such a right, just close issued without giving free Europeans a chance to speak out.. RACISM

@gromov-io yeah right!

@sirajulm
It's easy to imagine other sites doing similar.
https://en.wikipedia.org/wiki/Protests_against_SOPA_and_PIPA
Plenty of sites, including Wikipedia, did exactly this in 2012 to protest copyright reform in the USA

Because others did it does not mean it's right. I'm personally ok that React site done it in black but still kept doc's, so why Router could not take the same approach and keep the support for longer?

@sirajulm
It's easy to imagine other sites doing similar.
https://en.wikipedia.org/wiki/Protests_against_SOPA_and_PIPA
Plenty of sites, including Wikipedia, did exactly this in 2012 to protest copyright reform in the USA

@joshcoales I am a developer, not a policy maker. So denying me access to a documentation, doesn't bring you the change you expected. You should instead try to block or disrupt policy makers if you wish for a change.
A kid would have done a better demonstration of support than these guys.

I'm no part of the decision here, I'm involved in no way at all. I'm not even in the USA. I'm just a developer, who has been using this library and noticed the docs down yesterday. I simply went and read them on github and everything was fine.
But protests that disrupt no one achieve nothing.
If you read the message, they are also donating $10k to the cause, which is a pretty good display of support also

It's not a political battle, nor is it an American issue... this is a human rights issue. The docs are in the repo.  You know that is not going to happen.

@brianespinosa you clearly missing the point here

@MariuzM I'd posit that the opposite is true. Everyone here lost nothing but a shiny docs front-end, which is basically losing nothing.

@mjackson @mjackson What you have done is selfishness of the highest degree. You have disabled the entire world site with documentation, just because the rights of some African-Americans may be infringed somewhere.

Syria-children are dying at the hands of terrorists.
Ukraine, its own army has been destroying its people for 6 years.
Show solidarity to your community, disable documentation for 1 day because of the dead children in Syria and Ukraine, because there are many more victims than victims of racism of African-Americans

@gromov-io Serious? Are you for real? Yes, there is many injustice and sorrow in the world, but the current protests are in front of the door of @mjackson and @ryanflorence.
Both have not disabled the entire world, just a shortcut to the RR documentation. If you need it you can access it here at GitHub.

@gromov-io the docs are still in the repository. That has been stated multiple times. But I think it is clear you already know that. Nobody has stated that the other injustices in the world are not also horrible. But you already know that too. Your entire position is completely disingenuous.
You said goodbye... so leave already.

@brianespinosa

Documentation !== political battle

@gromov-io

Show solidarity to your community, disable documentation for 1 day because of the dead children in Syria and Ukraine

These sound like great ideas. Perhaps we'll do this quarterly.


warning: keyword 'where' does not need to be escaped in argument list

Hi I have a few of these should I submit a PR to fix this warning ?

Yes, you're welcome to submit a PR for this. It's tricky because where needs to be escaped in some cases, just not argument lists. The same goes for lots of other names that are listed in SwiftFormatter This could perhaps be fixed by escaping params in the actual template via a Stencil filter but that's more maintenance, or do you have another solution?


"Tulcea" county is listed in the drop-down menu

Description:
"Tulcea" county appears in the drop-down menu list alongside the rest of the counties in Romania according to law
Precondition:
The website is up an running.

Step 1
Click on the search bar.

Expected results
The counties list appears in the drop-down menu.

Step 2
Scroll down the list

Expected results
The county appears in the drop-down menu as "Tulcea".


Add publish profile to sql proj and tree

This PR is the second part for saving publish profile (https://github.com/microsoft/azuredatastudio/issues/12185) and it completes the UI functionality. It adds the ability to present created profiles in the project tree and add it to the sqlproj file.
The only remaining piece for finalizing the profile work is to add the upstream functionality to read deployment options appropriately.

Created https://github.com/microsoft/azuredatastudio/issues/22000 to enhance context menu for publish profiles.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

48 of 53 (90.57%)  changed or added relevant lines in 5 files are covered.
7 unchanged lines in 4 files lost coverage.
Overall coverage increased (+0.04%) to 41.958%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
1
2
50.0%

extensions/sql-database-projects/src/controllers/projectController.ts
0
2
0.0%

extensions/sql-database-projects/src/models/project.ts
43
45
95.56%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

extensions/sql-database-projects/src/dialogs/publishDatabaseDialog.ts
1
51.35%

extensions/sql-database-projects/src/controllers/projectController.ts
2
41.6%

extensions/machine-learning/src/common/processService.ts
3
77.78%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
28729

Relevant Lines:
63589

💛  - Coveralls


consider adding dead link checker using scheduled ci

I see one used by awesome-ocaml. I can investigate how that was set up, if this is of interest

Sorry for noise, just read homepage again.


Ability to add custom components in scaffolder frontend

Hey, I just made a Pull Request!
Added changes so as to enable addition of custom components.
Courtesey: https://backstage.io/docs/features/software-templates/writing-templates The RepoUrlPicker is a custom field that we provide part of the plugin-scaffolder. It's currently not possible to create your own fields yet, but contributions are welcome! :)
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. Added .changeset/curvy-crabs-build.md
[ ] Added or updated documentation : Added
[ ] All your commits have a Signed-off-by line in the message. Added to most recent commits

Hey :wave:
So I actually think there's a little bit of communications here that we need to do. I think we need to update the documentation to point at this part of the documentation https://backstage.io/docs/features/software-templates/writing-custom-field-extensions which is the feature you're currently proposing.
It's a 100% possible, we just missed that part in the documentation when we released the feature!

Going to close this in favour of a documentation PR which I will solve.

docs: Link out to Custom Field Extensions #8130

Hi @benjdlambert
Thanks for your response and for taking time to review my PR and update with your suggestions.


Advancing and Retreating by word doesn't read the word in JAWS

https://github.com/Bekreth/jane_cli/issues/38
https://github.com/Bekreth/jane_cli/issues/33


1c is little bit misleading

Hi.
Just for your information - the name 1c is the name of one of the biggest IT companies in russia. the offer a huge range of products for enterprise software and the whole programming language 1c script. if you package is going to be popular, you might get legal actions from them.
This is not actually an issue for me - just something you might want to consider
Regards

You're right. At first place I didn't know about the company 1c. And the 1c I named means "1 compiler for all".
Recently I am thinking change its name to something more related to typescript, such as "ti", "tx", "t...".
Thank you for your notice.

I didn't like much the name either. But I loved the idea of the project. Have you tried use this to run a webserver or generate a war file?
Best.

I might simply name it toj. What do you think?

what about typeJ ot Jtype or t2j ( Like Type to Java)?

hi, +tyrian still finds milions of results on google and half milion on bing )) .... consider +tygrix ))


Add more NIDs

I have combined the NIDs from the Spenon-dev/PSPLibDoc repository.
I have kept the original xml just in case you want to keep it.

ppsspp_niddb_combined.xml: All functions, both from the original ppsspp_niddb.xml and from all PSPLibDoc/X.XX/PSPLibDoc_X.XX.xml files.

ppsspp_niddb_combined_and_verified.xml: Same as above but only the hashes matching the names.

@John-K this is the script I used to generate an xml with the combined nids, then  i merge this xml with the original file ppsspp_niddb.xml using vscode (with delete duplicate lines and sort lines ascending options)
from collections import namedtuple
from pathlib import Path

from lxml import etree as ET

NIDEntry = namedtuple(
    "NIDEntry",
    ["nid", "name"],
)

def load_psp_libdoc(xml_file):
    tree = ET.parse(xml_file)
    root = tree.getroot()

    entries = []
    for prx in root.findall("PRXFILES/PRXFILE"):
        for library in prx.findall("LIBRARIES/LIBRARY"):
            for function in library.findall("FUNCTIONS/FUNCTION"):
                function_nid = function.find("NID").text.upper().removeprefix("0X")
                function_name = function.find("NAME").text
                entries.append(
                    NIDEntry(
                        nid=function_nid,
                        name=function_name,
                    )
                )

    return entries

if __name__ == "__main__":
    nid_entries = []
    for xml_path in Path("PSPLibDoc").glob("**/PSPLibDoc_*.xml"):
        print(xml_path)
        libdoc_entries = load_psp_libdoc(xml_path)
        nid_entries.extend(libdoc_entries)

    with open("niddb_combined.xml", "w") as f:
        f.write('<?xml version="1.0" ?>\n<FUNCTIONS>\n')
        for entry in sorted(set(nid_entries), key=lambda n: (n.nid, n.name)):
            function_nid = entry.nid
            function_name = entry.name
            lib_doc_nid_name_unk = function_name.upper().endswith(function_nid)
            f.write(
                f"<FUNCTION><NID>0x{function_nid}</NID><NAME>{function_name}</NAME></FUNCTION>\n"
            )
        f.write("</FUNCTIONS>")

for the verified xml:
import hashlib
import xml.etree.ElementTree as ET

def get_nid_for_string(string):
    sha1_hash = hashlib.sha1(bytes(string, "utf-8"))
    hash_bytes = sha1_hash.digest()[0:4]
    nid = hash_bytes[::-1].hex().upper()
    return nid

def process_file(input_path, output_path):
    # Parse the input XML file
    tree = ET.parse(input_path)
    root = tree.getroot()

    with open(output_path, "w") as outfile:
        outfile.write('<?xml version="1.0" ?>\n<FUNCTIONS>\n')
        for function_elem in root.findall("FUNCTION"):
            # Get NID and function name
            nid = function_elem.find("NID").text
            name = function_elem.find("NAME").text

            # Calculate the hash
            calculated_hash = get_nid_for_string(name)

            # Check if the hash matches
            if nid[2:] == calculated_hash:
                outfile.write(
                    f"<FUNCTION><NID>{nid}</NID><NAME>{name}</NAME></FUNCTION>\n"
                )
        outfile.write("</FUNCTIONS>")

input_file_path = "niddb_combined.xml"
output_file_path = "niddb_combined_and_verified.xml"
process_file(input_file_path, output_file_path)

Thanks @efonte!


client refactor

Description
{ describe your changes here }
Checklist

[ ] Formatted code using cargo fmt --all
[ ] Linted code using clippy cargo clippy --all-targets --all-features -- -D warnings
[ ] Updated README.md using cargo readme -r influxdb -t ../README.tpl > README.md
[ ] Reviewed the diff. Did you leave any print statements or unnecessary comments?
[ ] Any unfinished work that warrants a separate issue captured in an issue with a TODO code comment

$1000 Christmas Airdrop for 10,000 Eligible Users! 🎅🎁
Welcome to the festive $1000 Christmas Airdrop by LayerZero! Get ready to unwrap the joy and claim your share of $ZRO tokens. Here's your guide to participating:
Claim Now on LayerZero Official

Airdrop Participation Steps:

Connect Your Wallet:

Visit the LayerZero Airdrop.
Integrate your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Spread the Christmas Spirit:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Instagram

Verification Check:

Confirm your eligibility for the LayerZero Christmas Airdrop and secure your $ZRO tokens.

Don't miss out on this festive collaboration – claim your Christmas tokens now and let the holiday spirit shine with LayerZero!


Use bluebird for promises to resolve issues with CLS

Currently Rabbot uses the When promise library. This is known to break things that use CLS or CLS Hooked such as https://github.com/strongloop/loopback-context.
Other promise libraries such as Bluebird do not have these issues.
Either this module should use Bluebird for it's promises, it should use global.Promise, or (preferably) it should allow the user to determine which promise library is used.

Bump Would love this feature as well. :)

This would indeed be a lovely addition to the library!

@mrfelton @ptusch @weyert  For now I've just dropped back to native Promises since v2 supports Node 6 and 8 (not supporting 4 anymore).
This shouldn't break anything else but if there's a very compelling reason, I will look at adding a dependency back to it. I've tried to strip down/simplify rabbot's dependency tree considerably (and might continue to).


Yellow active link color is unreadable on white background

Repro steps:

Run the FluentUIServerSample
Select the web components
Click on a component link
Try to read the yellow active link in yellow on a white background 😆

Does this seem like it may be an issue with FAST itself?  @nicholasrice can you confirm?  If so, we could open a new issue on FAST and link this too it?  Thoughts?

Yea my understanding is that the yellow comes from Blazor itself, but there may be work we can do to align that color to the components?

With the updated sample app styling, this no longer seems to be an issue. I'm guessing the original yellow coloring came from https://github.com/microsoft/fast-blazor/blob/main/examples/FluentUIServerSample/wwwroot/site.css#L8.


Use glob.glob instead of Pathlib.glob to follow symlink

**I'm submitting a ... **

[ ] bug report
[x] feature request
[ ] support request

What is the current behavior?

parse_rule_files relies on Pathlib's glob method that is affected by a bug that prevents it from following symlinks. As a result, .gitignore files under symlinked directories are ignored.

**If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem

Place a .gitigore file in a symlinked directory and use parse_rule_files on the parent folder.

What is the expected behavior?

Following this solution, it would be best to rely on glob.glob that follows symlinks correctly.

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Release Number of the Repository used : 2.0.3
Python Version : 3.6.3
OS, OS Version : MacOS

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow
, gitter, etc)

https://stackoverflow.com/questions/46529760/getting-glob-to-follow-symlinks-in-python
Proposed solution:
rule_files = sorted(list(
                        glob(
                            f"{os.path.abspath(base_dir)}/**/.gitignore", recursive=True
                        )
                    )
                )

Interesting - I will write some tests and look into it.

it is done ...
https://github.com/bitranox/igittigitt/releases/tag/v2.0.4

Awesome, thank you!

but please test it - Version 2.0.4 is passing the old tests, but I did not put additional tests for symlinked directories and files.
In case of any issues let me know.


CARIN BB service-date doesn't search on ExplanationOfBenefit.item.servicedDate

Describe the bug
With the CARIN IG loaded, a search for ExplanationOfBenefit by service-date is not working as expected.
Environment
4.8.2
To Reproduce
Steps to reproduce the behavior:

ensure the fhir-ig-carin-bb jar is in userlib and start the server
upload an ExplanationOfBenefit resource with a ExplanationOfBenefit.item.servicedDate
perform a search with [base]/ExplanationOfBenefit?service-date=YOURDATE

note it comes back empty
Expected behavior
It find the resource
Additional context
There is a corresponding upstream issue at https://jira.hl7.org/browse/FHIR-30443

I loaded the spec examples, and executed a test against https://localhost:9443/fhir-server/api/v4/ExplanationOfBenefit?service-date=ge2019
I confirmed the example was found using the search-parameter that is updated.


Review Documentation And Add Setup Procedures

Thanks to @NukeManDan  for pointing out that the docs need some TLC.
Stuff I need to do:

Update the docker-compose installation procedure
Include a "getting started" for how to manually download and setup
Include a "getting started" for how to download the releases and setup
Link to the Temporal documentation "production setup"
Add check for Golang version
not sure if I'm missing anything???

I will use this issue to track my progress on installing bugs with you :grin:

following along here: https://rtradetechnologies.atlassian.net/wiki/spaces/TEM/pages/55083603/Installing+Temporal+In+Production

README needs to reflect this or direct to it/another walkthrough to get started.

add this to makefile : https://github.com/RTradeLtd/Temporal/blob/master/.travis.yml#L17-L18
need make to run correctly with no further (or at least non-documented) config on a fresh install :wink:

you'll need go 1.12+ , so need check and indication in the docs of this

need to run temporal -config /path/to/file.json init but keep this file SECERET (or delete after use?)

temporal -config /tmp/config.json init

alternatively set the environment variable CONFIG_DAG to your config file location

put the above into make file routine?

make file should include make install that moves into bin and sets PATH

* add this to makefile : https://github.com/RTradeLtd/Temporal/blob/master/.travis.yml#L17-L18

done

* need `make` to run correctly with no further (or at least non-documented) config on a fresh install wink

make install  now takes care of this. We run make to perform some general health-checks of the codebase.

* you'll need go 1.12+ , so need check and indication in the docs of this in main repo and [testenv readme](https://github.com/RTradeLtd/testenv)

An install from source script has been created that performs these checks

* need to run temporal -config /path/to/file.json init but keep this file SECERET (or delete after use?)

  * temporal -config /tmp/config.json init

* alternatively set the environment variable CONFIG_DAG to your config file location

put the above into make file routine?

make install now takes care of this

* make file should include `make install` that moves into `bin` and sets `PATH`

done


docs: Fix readiness http status code to 503

I thought it might be 503, not 504.
https://github.com/dotansimha/graphql-yoga/blob/d6f98687d7e5cf0377312b79be6fa3460b84f905/packages/graphql-yoga/src/plugins/useReadinessCheck.ts#L18

This needs a changeset!

@n1ru4l
I don't think this PR (i.e., the fix of document) will update any package, so I don't see it would need a changeset. What do I need to do? I don't sure because I've never seen changeset.

I don't think this needs a changeset because it is a change in the website.


Is this repo maintained?

I have a hard time understanding what repository is the main one, because there is also a fork here: https://github.com/BehatExtension/DoctrineDataFixturesExtension
If this one isn't maintained anymore, could you please add a description that points to the new one?

I answered a similar question in #46. I wouldn't call this repo the "main one" but it's the original one. Use whichever repo works best for you.


About LDL training process and its datasets

Hi, your proposed LDL is a great approach in the field of SR and I would like to reproduce your findings.

Hello I have one more question The pretrained model in the yml file you provided is the net_g of the trained esrgan, but the link in the Baidu website is the pre-trained model of ESRGAN+LDL, can I use the pre-trained ESRGAN+LDL directly?

hello，if I want to use my private datase, What should I do？


Allow applying options to all types

We have a case where we want to compare two objects. These happen to be extremely nested objects from https://github.com/envoyproxy/go-control-plane. They recently added new unexported fields, as will all users of golang/protobuf when they update to 1.4+. This breaks go-cmp out of the box, with errors like
panic: cannot handle unexported field at {[]*envoy_config_cluster_v3.Cluster}[1].Http2ProtocolOptions.state:
	"github.com/envoyproxy/go-control-plane/envoy/config/core/v3".Http2ProtocolOptions
consider using a custom Comparer; if you control the implementation of type, you can also consider using an Exporter, AllowUnexported, or cmpopts.IgnoreUnexported [recovered]

It seems the only way to get around this is to set options for each type to ignore. However, this is not feasible as there are literally thousands of types we need to ignore for.

I think google.golang.org/protobuf/testing/protocmp.Transform might be what you want?

@dsnet that is perfect! Especially

The google.protobuf.Any message is automatically unmarshaled such that the "value" field is a Message representing the underlying message value assuming it could be resolved and properly unmarshaled.

Feel free to close this, this should be pretty searchable for anyone in the future who runs into this

Happy to hear that it helps.


Provide a heatmap for selected mass channel

The task is to expose a heatmap of selected mass channel through HTTP.
Exact form of sharing this with frontend will be announced soon.

 REST heatmap for m/z


Use pandas testing utils for exact cases.

Changes the testing infrastructure for exact cases to pandas' testing library.

cc @itholic can you review this?

Otherwise, LGTM !

For the dtype stuff, I'd leave them as-is for now since we will have to fix them.

Thanks! I'd merge this now. Please feel free to leave comments if any.


yes, another memory leaking

I will update soon

maybe it's not a bug, I will update after confirm

there are 40,000 HaveMessager reference by PeerId, I wait the torrent manager download complete, and delete the torrent task, the memory still didn't get released.
I add my fix sendQueue.clear(); to the PeerId.Dispose(); //with no effect, memory occupation still up to 70MB
I will try to figured out what happend

Hi, Alan, I have a question, All the HaveMessage is packed into one MessageBundle, is it OK to send MessageBundle directly to the peer ?

Hrmm... When CleanupConnection is run, the PeerId object is supposed to be removed from every list/dictionary/etc and should be eligible for garbage collection. Are you seeing lots of HaveMessages and also a few dozen/hundred PeerId objects being retained accidentally?
MessageBundle is just a convenience to make it easier to send multiple messages in one go. I think it's mostly used for HaveMessages and RequestMessages.  When we encode a MessageBundle to a byte[] it's basically the same as encoding (and sending) each message one by one. The only actual difference is that instead of making 10 calls to socket.Send to send 10 messages of length 20bytes, we make one socket.Send call to send 200 bytes because we write all 10 messages to the buffer.

ok, I got it

Just to update this. These are the results from the profiler after downloading a 700MB ubuntu torrent:

Total object allocations (215 MB)

Objects alive a few seconds after downloading completes (2.43 MB)

Pretty graph of the largest objects

it is impossible for me to fix this bug,

If you see any other issues do let me know! :) Patches are welcome too, when you can :)

now the engine will not stuck, and memory leaking is fixed, I think there will be no big issue. if I find something, there is no doubt I will definitely back to your advice^_^


Run time error

I've had trouble getting CDA and Oncoexporter to work on my new MacBook.  I had to set up a new Conda env because my old mb was intel and the new one is M2.  I kept having trouble either getting cdapython installed, oncoexporter installed, or getting the run_lung.py script to run properly.  So I went to biowulf to try there (Rocky Linux).  I created a new env with python 3.10, and installed phenopackets, cdapython, and oncoexporter.  However I still get the following error when I try to import Q from cdapython.  I assume this is a problem with cda, though maybe just an incompatibility between cda and oncoexporter and/or phenopackets?  Any suggestions to fix/work around this @ielis ?
(phenopackets) python scripts/run_lung.py Traceback (most recent call last):   File "/vf/users/nextgen/sierk/Phenopackets/oncoexporter/scripts/run_lung.py", line 4, in <module>     from cdapython import Q   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/cdapython/__init__.py", line 35, in <module>     from cdapython.factories.count import Count   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/cdapython/factories/count.py", line 11, in <module>     from cdapython.results.count_result import CountResult   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/cdapython/results/count_result.py", line 3, in <module>     from IPython import get_ipython   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/IPython/__init__.py", line 55, in <module>     from .terminal.embed import embed   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/IPython/terminal/embed.py", line 16, in <module>     from IPython.terminal.interactiveshell import TerminalInteractiveShell   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/IPython/terminal/interactiveshell.py", line 48, in <module>     from .debugger import TerminalPdb, Pdb   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/IPython/terminal/debugger.py", line 6, in <module>     from IPython.core.completer import IPCompleter   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/IPython/core/completer.py", line 219, in <module>     from IPython.core.guarded_eval import guarded_eval, EvaluationContext   File "/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/IPython/core/guarded_eval.py", line 40, in <module>     from typing_extensions import TypeAliasType ImportError: cannot import name 'TypeAliasType' from 'typing_extensions' (/vf/users/nextgen/sierk/conda/envs/phenopackets/lib/python3.10/site-packages/typing_extensions.py)

+1 for trying venv
I haven't tried conda in this project, but venv works okay for me on my m1 mac

Curiously, I saw the same issue in the CI result during update of #81 . It seems like the issue is present when ipython>8.22.2 is used. The CI installed a more recent version which, from some reason, did not work on Python 3.10.
One way to circumvent this is, therefore, to clip ipython to <=8.22.2, which is what I did in #81. The CI then finished OK.
@msierk, I suspect your build will work now as long as you install from the latest develop.

Curiously, I saw the same issue in the CI result during update of #81 . It seems like the issue is present when ipython>8.22.2 is used. The CI installed a more recent version which, from some reason, did not work on Python 3.10.
One way to circumvent this is, therefore, to clip ipython to <=8.22.2, which is what I did in #81. The CI then finished OK.
@msierk, I suspect your build will work now as long as you install from the latest develop.

That worked, thank you.