"Râncăciov", "Ungureni" are listed in the drop-down menu

Description:
When user writes ”Râncăciov” and "Ungureni" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Râncăciov"

Expected results
The location appears in the drop-down menu as "Sat Râncăciov, DÂMBOVIȚA (Dragomirești)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Ungureni"

Expected results
The location appears in the drop-down menu as "Sat Ungureni, DÂMBOVIȚA (Dragomirești)".


Upgrade to MUI v6

Hi, I see the MUI v6 is recently released. Is it possible to upgrade this project to use MUI v6?

I think it's better to wait a bit


ReferenceError: accounts is not defined

Hi,
I am not sure how the accounts in the test suite should work - should I define it by myself, or is my setup somehow not complete?
Also, I don't know a lot about npm, but the package requires chai and chai-bignumber to work, so it would be good to have it in documentation :)

The accounts variable is provided by the truffle so as is mocha and chai.
I have slightly updated the getting started instructions in 3111583d00ae17cf9b98119bf2e35bb7810ec0db to make it clear from which context the suite should be called.

Thanks, I figured it out that I need to get the accounts from the contract function. The updated readme would work great.


Fix 1f1ee-1f1f3 India Flag with correct 24 spokes in the Ashoka chakra

The central chakra in Indian Flag is called Ashoka Chakra and according to the flag code of India,  It should have 24 equally spaced spokes

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

Some flags are simplified, e.g. the US flag does not show fifty stars and thirteen stripes.


[FLINK-31774][doc] Add document for delete and update statement

What is the purpose of the change
Add doc for update and delete statement.
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluster with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@luoyuxia comments addressed

@luoyuxia ready for merge

@Aitozi Could you please backport it to 1.17. Thanks

@Aitozi Could you please backport it to 1.17. Thanks

Sure


APPSREPO-652-Clean up Sync Service chart

Sync Service has it's own helm deployment project based on this one. So I removed any sync service reference from this project.

Yes @rgauss, Sync Service will be part of the ACS by default. Now sync service has this project https://git.alfresco.com/alfresco-services/alfresco-dsync-services-deployment so it's not needed anymore that aync service helm chart to be a part of the alfresco-dbp-deployment project.

Yes @rgauss, Sync Service will be part of the ACS by default. Now sync service has this project https://git.alfresco.com/alfresco-services/alfresco-dsync-services-deployment so it's not needed anymore that aync service helm chart to be a part of the alfresco-dbp-deployment project.

@ciprianchelaru, it doesn't looks like that's happened yet, so we can't remove it as a requirement from ours, and we'll need the sync AMPs applied in our repo image until the ACS repo image contains it as a default.
I'll comment specifically on what we'll need changed here.

Sync service is targeted to be included into ACS 6.1.1, both AMP and deployment.
There are a couple of pull requests on the ACS side waiting to be merged, but first we need to release a Sync service RC, which will happen very soon.
https://issues.alfresco.com/jira/browse/APPSREPO-693
https://issues.alfresco.com/jira/browse/APPSREPO-694
Let's keep this PR opened until our ACS PRs are merged.

The PRs mentioned above have been merged, both into master(chart version: 3.0.0) and support/SP/2.N (chart version 2.1.0)
This means the ACS deployment chart now contains Sync service.
I think this pull request can be merged now.

It doesn't look like any of the discussions have been addressed.

Discussions have been addressed.

I reverted the changes that removed the property dsync.service.uris.


Bug: report doesn't work when - inside github group name

Steps to reproduce:
Open any repo with - in user group name, like https://goreportcard.com/report/github.com/mk-5/fjira
Problem:
There is no way to show report for github repositories  that contains - character inside group name, like github.com/ab-2/somerepo. The github url is automatically converted to github.com/ab2/somerepo.
Error message for above example: could not download repo: could not get latest module version from https://proxy.golang.org/github.com/mk5/fjira/@latest:
Expected behaviour:
The url with - inside group name should work.

there was no problem with parsing - it was a repo issue. The name of the module needs to be exact match with name of the repo


Conflicting documentation for Credential Retrieve Requests

The CredentialRetrieveRequest proto documents that at least one authentication method is required while it's associated RetrieveRequest class documents the absence of any authentication methods should be treated as a wildcard.
I suggest that the field should be required and that the wildcard is made explicit (For example openyolo://any).  What does everyone else think?

I think it would probably be better to just make the proto field optional, rather than having a "special" auth method value that could be accidentally mixed with other values.

Maybe there is not a compelling reason to support this wildcard?  Assumption: From my understanding the client application should always authenticate the retrieved credential therefore a retrieved credential with an unknown authentication method would be of no use to the client.
To avoid this possible scenario the client should only retrieve credentials with supported authentication methods.
Now that I think about this a wildcard seems like an opportunity for clients to potentially shoot themselves in the foot. For example I can imagine the following scenario where an app developer completes their android implementation using a wildcard for authentication method while their client only supports authentication method A, the developer adds a web frontend which supports authentication methods A and B, additional credentials are now available for unsupported authentication method B. Now the android application may receive unsupported credentials which may or may not be accounted for by the developer's implementation.

Additionally this wildcard is set as the default value when using the RetrieveRequest.Builder. If there is a reason to support it I suggest it is changed to an non-default value to help developers avoid the situation described above.

Your reasoning makes sense, and I have seen a few practical examples where the authentication methods varied by platform for a service - for instance, supporting Google Sign-in on Android only. Such variation is usually disastrous though, as you end up locking users out of their accounts when they switch platforms :/
Playing it safe from our perspective and always requiring an app be explicit about the auth methods they support is probably the best course of action for now. So, sounds like this issue now becomes "remove wildcard behavior from spec".


Reactive Proxy read only issue

Version
3.0.7
Reproduction link
https://codesandbox.io/s/vue-composition-api-example-forked-j5dej?file=/src/App.vue
Steps to reproduce
Include twilio-video and set a localvideotrack to reactive state
See here:
https://codesandbox.io/s/vue-composition-api-example-forked-j5dej?file=/src/App.vue
What is expected?
The set state to be reactive
What is actually happening?
I am getting an error setting the state in Chrome and Firefox.
Error from Chrome:
Uncaught (in promise) TypeError: 'get' on proxy: property 'dimensions' is a read-only and non-configurable data property on the proxy target but the proxy did not return its actual value (expected '#' but got '[object Object]')

Upgrading old Vue 2 project from scratch unfortunately run into issue with proxy

Please boil down the problew without external dependencies like twilio-video. You likely need to use
import { markRaw } from 'vue'
markRaw(await createLocalVideoTrack({ video: true }))

duplicate of #3024

@posva I cannot believe I overlooked that when it quite clearly states in clear english re complex third party classes.
@LinusBorg I see it, but it doesn't look like it is in a hurry to be merged in
markRaw has solved this painful issue for me... thank you...

it doesn't look like it is in a hurry to be merged in

opening another issue for it won't make it happen any quicker on its own ;)
Good you have a workaround now, though.

it doesn't look like it is in a hurry to be merged in

opening another issue for it won't make it happen any quicker on its own ;) We just have a full plate and need to prioritize
Good you have a workaround now, though.

Completely understandable, thank you

Just to add to this. Avoid placing complex objects in a reactive object. 3rd party plugin in an object doesn't need to be reactive.


chore(proxy-wasm) remove a dead branch following a recent change

coverage: 90.867% (+0.01%) from 90.857% when pulling 8c5dd1895c1b18dca27f4aa67f3c8f704ca0d6d4 on chore/remove-dead-branch into 3161c2c935cd45c9c1fcacf7955ac7c4463dfcd8 on main.


[AI] [Inference] update dependency version in README

Description
Update the dependency version in README to beta.2
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

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


Fixed some errors that occurred on a clean-slate build

Openssl wasn't installed at the right time, and some of the directories were not created

Is this PR still needed?


cjdns: 20.1 -> 20.2

Motivation for this change
cjdns minor updates
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Success on x86_64-linux (full log)
Attempted: cjdns
Partial log (click to expand)
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/dependencies/libuv/build/gyp/tools/pretty_gyp.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/dependencies/libuv/build/gyp/tools/pretty_sln.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/dependencies/libuv/build/gyp/tools/pretty_vcproj.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/dependencies/libuv/build/gyp/tools/emacs/run-unit-tests.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/dependencies/libuv/checksparse.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/dependencies/libuv/gyp_uv.py: interpreter directive changed from "/usr/bin/env python" to "/nix/store/l9j2jsc9flrbmpf799nw9wdq59gpwms8-python-2.7.14/bin/python"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/node.sh: interpreter directive changed from "/bin/sh" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh"
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2/share/cjdns/node_build/run_jshint: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash"
checking for references to /build in /nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2...
/nix/store/904mcd078ris7v1lygz2dh45jda47ahm-cjdns-20.2

Success on aarch64-linux (full log)
Attempted: cjdns
Partial log (click to expand)
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/docker/entrypoint.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/docker/run: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/bash/ip6tables.sh: interpreter directive changed from "/bin/sh -e" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh -e"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/android/cjdroid-build.sh: interpreter directive changed from "/usr/bin/env bash" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/bash"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/android/cjdroid/install.sh: interpreter directive changed from "/system/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/android/cjdroid/files/99cjdroute: interpreter directive changed from "/system/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/android/cjdroid/files/cjdaemon: interpreter directive changed from "/system/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2/share/cjdns/contrib/android/cjdroid/files/cjdctl: interpreter directive changed from "/system/bin/sh" to "/nix/store/a4qslf0yfs44mhnbagarn232sjnckhy0-bash-4.4-p19/bin/sh"
checking for references to /build in /nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2...
/nix/store/rw5wanlahnp387z4qw7dnnci99lqknij-cjdns-20.2


Fix bug in generate promp using 'instruction' instead of 'input'

According to the Alpaca README, we should make the distinction

for examples with a non-empty input field:

but seems you checked for empty "instruction" :)

Eep. Thanks for catching this!


'dict object' has no attribute 'consul_bind_address'

I run the role and receive an error:
TASK [brianshumate.consul : Create configuration] *********************************************************************************************************************************************************************************************** task path: /home/user/ansible/roles/brianshumate.consul/tasks/config.yml:8 File lookup using /home/user/ansible/roles/brianshumate.consul/templates/config.json.j2 as file fatal: [hostname.domain.tld]: FAILED! => {     "failed": true,      "msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'consul_bind_address'\n\nThe error appears to have been in '/home/user/ansible/roles/brianshumate.consul/tasks/config.yml': line 8, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Create configuration\n  ^ here\n\nexception type: <class 'ansible.errors.AnsibleUndefinedVariable'>\nexception: 'dict object' has no attribute 'consul_bind_address'" }     to retry, use: --limit @/home/user/ansible/site.retry
my inventory file
[consul_instances] hostname.domain.tld consul_node_role=server consul_bootstrap_expect=true
my site.yml
`

hosts: consul_instances
roles:

{ role: 'brianshumate.consul', tags: 'galaxy-consul' }
`

When I'm adding subtask and run it
`

name: print to stdout
debug:
var: consul_bind_address
`

I see right address
ok: [hostname.domain.tld] => {     "consul_bind_address": "x.x.x.x" }
Anybody knows how to resolve this issue?

I'm having the same issue.  Haven't found a workaround yet.

Thank you for using this role!
Can it be that you have another server with consul_node_role: server in your inventory that's not currently targeted?
You also need to make sure you have the consul_bootstrap_expect option set to true for all nodes in the cluster.
Are you setting up a single node cluster? If not, the problem might be in determining the consul_bind_address of another node then the currently targeted one. It might be interesting to have a debug print of the consul_lan_servers, consul_wan_servers, consul_bootstrap_servers and hostvars variables.

Hey there, @kikdevops; I think as @groggemans has suspected, your host inventory might be incompletely defined, so you definitely want to make sure it has the fields he mentioned.

I have the same issue, the variable consul_bind_address seems set properly.
I have 3 servers and 2 servers are configured through Ansible and this role. I use the following group configuration:
consul_node_role: server
consul_bootstrap_expect: true
consul_join: [ <IP_ADDRESS> ]
consul_encrypt_enabled: true
I checked the consul_bind_address with a debug task
TASK [brianshumate.consul : print consul_bind_address] ****************************************************************************************************************************************************************************************************************
ok: [consul2] => {
"consul_bind_address": "<IP_ADDRESS>"
}
TASK [brianshumate.consul : Create configuration] ************************************************************************************************************************************************************************************************************
fatal: [consul2]: FAILED! => {"failed": true, "msg": "The task includes an option with an undefined variable. The error was: 'dict object' has no attribute 'consul_bind_address'\n\nThe error appears to have been in '/home/vagrant/.ansible/roles/brianshumate.consul/tasks/config.yml': line 8, column 3, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n- name: Create configuration\n  ^ here\n\nexception type: <class 'ansible.errors.AnsibleUndefinedVariable'>\nexception: 'dict object' has no attribute 'consul_bind_address'"}
If I set "consul_bind_address" manually in my group configuration, the template works properly but I cannot use variable like 'ansible_bond0.ipv4.address' otherwise retry_join will be wrong in the config so I have to set the exact ip

@lilithfr The role was not designed to deploy 2/3 of the servers. With a little tweaking this could work, but you'll have to add the information about the third server in your inventory, and then only target the 2 servers you're configuring.
If you don't do this, the bootstrap_expect value will be wrong, and you might run into problems while bootstrapping. (This is not a problem when you're only deploying clients. Then you can just provide the gossip key and consul_join addresses).
In the above run you're only targeting a single node, depending on your configuration this might be a problem. As Ansible won't be able to determine the consul_bind_address of the other server(s).
If you're able to use static IP's you can always set the consul_bind_address in your inventory or host_vars. This should solve any lookup problems. Another option (although buggy in most ansible versions) is the consul_gather_server_facts option.

I have a little more detail, the easy appear only with multiple servers in order to generate retry_join.
The call to hostvars[server]['consul_bind_address'] return undefined when server is not the current.

@lilithfr This is exactly as I suspected, but this should only happen if you're not running the playbook against all server and you don't explicitly define consul_bind_address in your inventory or host_vars files.
(the consul_gather_server_facts option is another solution for some ansible versions)

@groggemans thank you this part works now.

@kikdevops Have you been able to solve your problem with the above information?

Yes, everything works well with group of servers. Thanks a lot.


[C#] Teams parity - meeting events

These scenarios should be supported in the new teams AI library. Some of them are activity handlers, some of them are GET api calls. There's a teamsInfo file in the Bot Framework SDK that has all the  meetings-related GET apis. For scenarios supported by it we should be able to simply import and use it. Otherwise we should be able to handle meetings-related events.

There were new activity types added to meetings: https://github.com/microsoft/botbuilder-js/blame/663bc953054ccc603887dcc19bcb0a8d65e7b2b1/libraries/botbuilder/src/teamsActivityHandler.ts#L1247

https://github.com/microsoft/botbuilder-dotnet/blob/main/libraries/Microsoft.Bot.Builder/Teams/TeamsActivityHandler.cs#L806-L815

application/vnd.microsoft.readReceipt
application/vnd.microsoft.meetingStart
application/vnd.microsoft.meetingEnd
application/vnd.microsoft.meetingParticipantJoin
application/vnd.microsoft.meetingParticipantLeave


myjenkins: Add plugin email-ext-recipients-column

See https://plugins.jenkins.io/email-ext-recipients-column
Signed-off-by: Gianpaolo Macario<EMAIL_ADDRESS>

Not quite sure this plugin is really needed:

Closing PR as nobody seems to be really interested in this plugin any longer


"Error: The macro can only be applied to enums or enum values." - What interface to use instead?

The message from the title is occuring when trying to apply below macro to this declaration, which looks a lot like an enum to my naive eyes:
@X()
enum LetsTry {
  one;
}

import 'dart:async';

import 'package:macros/macros.dart';

macro class X
    implements
        EnumDefinitionMacro,
        EnumValueTypesMacro,
        EnumDeclarationsMacro,
        EnumValueDeclarationsMacro,
        EnumValueDefinitionMacro,
        EnumTypesMacro {
  const X();

  @override
  FutureOr<void> buildDeclarationsForEnum(EnumDeclaration enuum, EnumDeclarationBuilder builder) {
    builder.report(Diagnostic(DiagnosticMessage('I ran'), Severity.warning));
  }

  @override
  FutureOr<void> buildDeclarationsForEnumValue(EnumValueDeclaration entry, EnumDeclarationBuilder builder) {
    builder.report(Diagnostic(DiagnosticMessage('I ran'), Severity.warning));
  }

  @override
  FutureOr<void> buildDefinitionForEnum(EnumDeclaration enuum, EnumDefinitionBuilder builder) {
    builder.report(Diagnostic(DiagnosticMessage('I ran'), Severity.warning));
  }

  @override
  FutureOr<void> buildDefinitionForEnumValue(EnumValueDeclaration entry, EnumValueDefinitionBuilder builder) {
    builder.report(Diagnostic(DiagnosticMessage('I ran'), Severity.warning));
  }

  @override
  FutureOr<void> buildTypesForEnum(EnumDeclaration enuum, EnumTypeBuilder builder) {
    builder.report(Diagnostic(DiagnosticMessage('I ran'), Severity.warning));
  }

  @override
  FutureOr<void> buildTypesForEnumValue(EnumValueDeclaration entry, TypeBuilder builder) {
    builder.report(Diagnostic(DiagnosticMessage('I ran'), Severity.warning));
  }
}

That just looks like a bug

Looks like this is an error produced by the CFE so that is probably where the issue is. There is test coverage for the shared code for this use case so it should generally work.

@jakemac53 anything I can do to help?

@jakemac53 anything I can do to help?

The repro you provided is already great, that is probably all we need to investigate (and thanks for that!)

Is there any update on this issue?

No meaningful updates, and it will still be quite some time before the underlying support needed for reading annotations is there.
Although, this package does support any type with a toJson method and fromJson constructor - so you could try using enhanced enums to provide those members on your enum type, and then it might just work.


Please add

Looking to be added to the textEncoding-Hub

@tec5271 Look for an invitation in your email (might have been days ago) from the textEncoding-Hub to join the repo, and accept the invitation.

@tec5271 Do you need me to uninvite and reinvite you (can't find the invitation?)

@tec5271 And that seems no easy matter. I want to make sure you definitely cannot find the invitation to join the repo in your email before I do something drastic like that.


Updated dockerfile to give correct permissions to Xdummy config file

Issue was initially identified when attempting to launch the Docker gui using make gui, which was resulting in a "permission denied" error. The error arose because not all users had been given access to the src/config/XDummy file, and so the permissions of this file were modified to allow access to any user using the Docker image.
Same as #35, but without a lot of extra baggage from earlier commits

Looks good here. :+1:


chore: add openBrowser debug logs

@louislebrault Can you give some context on why you need this log message only on the openBrowser method? As of now none of the api's write to the log. This is to reduce the verbosity and let the test runner handle it. If you explain your use case maybe it could be a console.debug instead of a console.log


WIP: Static resource key

@zdpcdt I made a sample, maybe you can review if we need to do this.

Looks not good to me. It's so complex.


Pulsar C++ client compilation failed on Ubuntu 18.04

Describe the bug
Could not get c++ client built on my Ubuntu 18.04
To Reproduce
Follwed steps from official tutorial

git clone https://github.com/apache/pulsar
apt-get install cmake libssl-dev libcurl4-openssl-dev liblog4cxx-dev \ libprotobuf-dev protobuf-compiler libboost-all-dev google-mock libgtest-dev libjsoncpp-dev
# libgtest-dev version is 1.18.0 or above
cd /usr/src/googletest
sudo cmake .
sudo make
sudo cp ./googlemock/libgmock.a ./googlemock/gtest/libgtest.a /usr/lib/
cd /usr/src/gmock
 sudo cmake .
 sudo make
 sudo cp libgmock.a /usr/lib
cd pulsar-client-cpp
cmake .
make

Could not make, it says:  error: ‘class google::protobuf::FileDescriptorSet’ has no member named ‘ByteSizeLong’; did you mean ‘ByteSize’? lib/CMakeFiles/PULSAR_OBJECT_LIB.dir/build.make:1605: recipe for target 'lib/CMakeFiles/PULSAR_OBJECT_LIB.dir/ProtobufNativeSchema.cc.o' failed
Error

Try to fix the error
As suggsted, changed ByteSizeLong to ByteSize in ./pulsar-client-cpp/lib/ProtobufNativeSchema.cc code, then got new error:
PaddingDemo.proto:25:12: Explicit 'optional' labels are disallowed in the Proto3 syntax. To define 'optional' fields in Proto3, simply remove the 'optional' label, as fields are 'optional' by default. tests/CMakeFiles/main.dir/build.make:68: recipe for target 'generated/tests/PaddingDemo.pb.cc' failed

Desktop (please complete the following information):

OS: [Ubuntu 18.04]

Additional context
I guess it has to do with protobuf version, but could not find any more information. Thank you!

I think we need to install ProtoBuf 3.4 or higher. Could you try to install a newer Protobuf from source?

If you're not a developer for C++ client, you can add -DBUILD_TESTS=OFF option that disables the build of tests as a workground.
As for this issue, I think we can either restore the code compatibility for Protobuf 3.0 or fix the docs.

It looks like the support for optional keyword in Protobuf 3 was introduced from 3.15.0. However, it's only a part of tests and the Pulsar C++ library doesn't rely on it.

It looks like the support for optional keyword in Protobuf 3 was introduced from 3.15.0. However, it's only a part of tests and the Pulsar C++ library doesn't rely on it.
I think we should update the documents for building C++ client from source. For developers, Protobuf 3.15.0 or higher is required. For users, Protobuf 3.1.0 or higher is required, and they should add -DBUILD_TESTS=OFF option to CMake command.

@BewareMyPower Thanks again. Here are the steps I tried:

first changed ByteSizeLong to ByteSize in ./pulsar-client-cpp/lib/ProtobufNativeSchema.cc as stated in question above
cmake . -DBUILD_TESTS=OFF
make

Here is the screenshot the result, no error but it looks like the build stops at 33%, and there are libpulsar.so and libpulsar.a are in the lib folder. Am I good to go?

@BewareMyPower  Sorry, but I am still confused. After running the cmake and make I have libpulsar.a and libpulsar.so in my ./pulsar/pulsar-client-cpp/lib folder

Noticed from the tutorial, there is a follow-up step called install Debian which I do not know what it is for (I am using Ubuntu). But I did download apache-pulsar-client.deb  and did apt install ./apache-pulsar-client.deb Now I have .so (but no .a) in my system's /usr/lib folder

Are there any more steps needed? I want to run pulsar-client-cpp on my local computer.
There seems to be no tutorial on how to write cpp client code and compile and run it other than this. I used this code and run command where /pulsar/pulsar-client-cpp/include is included because it contains <pulsar/Client.h> head file:
 g++ --std=c++11  myFirstProducer.cpp -o myFirstProducer -I /pulsar/pulsar-client-cpp/include
I got error: producer was not declared in this scope

Have I successfully got the pulsar-client-cpp built on my computer and Is the compiling command correct? Thank you!

TL; DR the example code is wrong, we need to fix the doc. /cc @Anonymitaet
You should add a declaration before createProducer:
    Client client("pulsar://localhost:6650");

    Producer producer;  // **See here**
    Result result = client.createProducer("persistent://public/default/my-topic", producer);

there is a follow-up step called install Debian which I do not know what it is for (I am using Ubuntu)

It's not a follow-up step. It's a parallel step.
When you install a C++ SDK, it means following files need to be generated:

C++ headers (*.h)
C++ libraries, it can be either a dynamic library (*.so on Linux) or a static library (*.a on Linux).

A *.deb file is a pre-built Debian C++ package. (Ubuntu is derived from Debian) i.e. the apt install ./apache-pulsar-client.deb command will install the C++ headers and libraries to the target directory. Since the pre-built Debian C++ package doesn't include a static library, so you won't see a *.a file.
However, if you have the source code of the C++ SDK, you can compile it by yourself, i.e. run cmake and make commands (or ./configure for C++ libraries that use GNU Autotools to build). At last, you need to run make install, which needs the superuser permission, to copy these files (*.h, *.so, *.a) to the target directory. You can also copy these files manually.
In short, you can choose to either compile from source or install from a pre-built package. The advantage of compiling from source is that you can get the latest library.


[apply-site] Refactor out a helper function called insertAfterApply -…

…> ApplySite.insertAfter().
Often times when one is working with apply sites, one wants to insert
instructions after both terminator apply sites and normal apply sites. This can
get ackward and result in unnecessary if-else code that is all really doing the
same thing, once for terminator instructions and once for normal instructions.
insertAfterApply is a helper method that MandatoryInlining uses for this purpose
and it is so useful that I want to use it somewhere else in closure lifetime
fixup as well. I am moving it onto apply site since that is the true abstraction
that insertAfterApply works with.

@swift-ci smoke test


Update Parallel state completionType values/description

Signed-off-by: Jorgen Johnson<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[ ✅  ] Specification
[ ✅  ] Schema
[ ✅  ] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

What this PR does / why we need it:
Update the values/descriptions related to Parallel state's completionType parameter:
Reasoning:
and & xor are most often encountered as boolean operators between 2 distinct values, so it's a bit confusing in this context where there are N parallel threads (not just 2). n_of_m is a straight-forward choice, but the corresponding n field name is very ambiguous, it gives no clear context to it's meaning on it's own.
https://github.com/serverlessworkflow/specification/issues/346
Special notes for reviewers:
Additional information (if needed):

Good to go . Just needs rebase


feat(lesson): pseudocode draft

thank you @willingc it feels nice to merge all of these pr's. ALSO it will be much easier to work on the lessons as a whole vs across prs and tracking branches!


[F] Deno support

🚀 Feature Proposal
I have read #1302 , but deno is more perfect than it was two years ago.
Motivation
A similar Library——sheetjs already supports deno, see https://deno.land/x/sheetjs
But its support for styles is not good
Example
In my fork, I did two things to make exceljs can run in deno:

Use a new version of buffer
fix a bug in lib/xlsx/xlsx.js

I didn't submit PR because I didn't clear why  stream write(await entry.async('nodebuffer')); causes errors and why you need to use it here.
Then, it can be used in deno by cdn. You can read more about it here<EMAIL_ADDRESS>https://esm.sh/
export * from "https://esm.sh/<EMAIL_ADDRESS>

Hey! Was this merged into the codebase? Im getting a buffer error when using exceljs in deno :/


Fix warm up issue when provider's timestamp is bigger than local machine's timestamp.

The provider's timestamp may be bigger than local machine's timestamp, if this happens, uptime will be negative, then weight will use the default value configured, it breaks the warm up.
This PR tries to fix this.

Codecov Report

Merging #4870 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master    #4870     +/-   ##
===========================================
- Coverage     63.97%   63.88%   -0.1%
+ Complexity      452      450      -2
===========================================
  Files           769      769
  Lines         33177    33181      +4
  Branches       5229     5231      +2
===========================================
- Hits          21226    21198     -28
- Misses         9530     9558     +28
- Partials       2421     2425      +4

Impacted Files
Coverage Δ
Complexity Δ

...o/rpc/cluster/loadbalance/AbstractLoadBalance.java
38.09% <0%> (-8.97%)
0 <0> (ø)

...che/dubbo/remoting/transport/mina/MinaChannel.java
43.42% <0%> (-10.53%)
16% <0%> (-1%)

.../remoting/transport/netty4/NettyServerHandler.java
61.9% <0%> (-9.53%)
0% <0%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
51.42% <0%> (-5.72%)
0% <0%> (ø)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0%> (-5.27%)
0% <0%> (ø)

.../dubbo/remoting/transport/netty4/NettyChannel.java
61.17% <0%> (-4.71%)
0% <0%> (ø)

...exchange/support/header/HeaderExchangeHandler.java
64.75% <0%> (-2.46%)
0% <0%> (ø)

.../exchange/support/header/HeaderExchangeServer.java
65.09% <0%> (-1.89%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.81%)
0% <0%> (ø)

... and 3 more

Continue to review full report at Codecov.

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


Type inside of Fancy Zones to set resolution

Description of the new feature / enhancement
My hands shake at times. Especially when trying to do something "exact". Example: Place something tiny in a tiny space. So, I have trouble when trying to setup Fancy Zones to my preferred resolution by using the bars.
My request is to enable a feature that allows you to type in what resolution you want for each zone. I think this is more inclusive.
Scenario when this would be used?
Resizing Zones in Fancy Zones
Supporting information
No response

Fair point. This issue #463 covers this.
In the meantime for those, who are looking for the config file to edit the custom zone dimensions manually, here it is: %LocalAppData%\Microsoft\PowerToys\FancyZones\zones-settings.json

Oh nice!
Thanks for sharing!
Clint
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Jay @.>
Sent: Monday, May 2, 2022 12:07:48 PM
To: microsoft/PowerToys @.>
Cc: Clint Schroeder @.>; Author @.>
Subject: Re: [microsoft/PowerToys] Type inside of Fancy Zones to set resolution (Issue #17998)
Fair point. This issue #463https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FPowerToys%2Fissues%2F463&data=05|01|Clint.Schroeder%40Microsoft.com|2f9d15722d7648ffd89408da2c5e48b7|72f988bf86f141af91ab2d7cd011db47|1|0|637871080718568786|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=HJOBKot5XC7ou3jXmYw%2BZQupmTtQSmiA9vHycZwK0lU%3D&reserved=0 covers this.
In the meantime for those, who are looking for the config file to edit the custom zone dimensions manually, here it is: %LocalAppData%\Microsoft\PowerToys\FancyZones\zones-settings.json
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2FPowerToys%2Fissues%2F17998%23issuecomment-1115131341&data=05|01|Clint.Schroeder%40Microsoft.com|2f9d15722d7648ffd89408da2c5e48b7|72f988bf86f141af91ab2d7cd011db47|1|0|637871080718568786|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=i%2FjOlSAjtXPTdItt6bArv680xJBRidVl0Fp4e3hGDeA%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALKUMZHXW5FO74DW4MVRVI3VIADWJANCNFSM5U4KEDLA&data=05|01|Clint.Schroeder%40Microsoft.com|2f9d15722d7648ffd89408da2c5e48b7|72f988bf86f141af91ab2d7cd011db47|1|0|637871080718568786|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=jqYt2FyOlV80crxo1EkruqUaR33eXPHG0tndy9t92uo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>

Hi,
I see a batch of other issues/requests for the ability to manually type in specific zone dimensions when creating a custom zone layout in the editor. I see you mentioned this file;
%LocalAppData%\Microsoft\PowerToys\FancyZones\zones-settings.json
I'm using PowerToys 0.58, and in the above directory path, I don't have a zones-settings.json file (though I have numerous others for Fancy Zones) even though I have multiple custom zone layouts. Is it possible that there is an additonal path to check, or an alternative storage method that may be preventing this file from being created?


Namespace for EXPLIoT Framework

We request you to kindly reserve the namespace 'expliot' for EXPLIoT Framework.
Namespace to be reserved

Namespace
Description
Administered By
Taxonomy

expliot
Namespace for use by EXPLIoT
EXPLIoT
EXPLIoT Taxonomy

About
EXPLIoT is an open source framework for security testing and exploiting IoT products.
Source
https://gitlab.com/expliot_framework/expliot
Documentation
https://expliot.readthedocs.io/en/latest/
CycloneDX EXPLIoT Taxanomy
https://gitlab.com/expliot_framework/expliot/-/blob/master/docs/compliance/cyclonedx.rst

@aseemjakhar namespace has been reserved.


Add a fuel.nix mdbook

This adds a new book/ directory containing an mdbook for fuel.nix.
The new book will replace the README with a much friendlier "Quick Start", and a more searchable and detailed guide covering how to install Nix, use fuel packages, and how to contribute to fuel.nix itself.
Closes #59.
TODO

[x] Add a new "Quick Start" section with the new two-step installation approach.
[x] Merge existing README content into subchapters.
[x] Add a more detailed "Nix Setup" chapter for those interested in more details.
[x] Add a section on nix profile install for persistent installation to the "Packages" chapter.
[x] Add a new "Contributing" section with details on how to add/update packages and milestones.
[x] Update the README to simply point to the book and Quick Start.
[x] Add a CI step to publish the book.
~Open an issue to remove the NOTE: under the nix profile install section required due to DeterminateSystems/nix-installer#477.~ It seems this issue only occurs if a previous Nix installation existed but was not cleaned up properly. The note has been updated.

This is just about ready to go, however it seems like the recent addition of the sysinfo crate in Sway and forc for benchmarking may have introduced some non-determinism in the macOS builds.
Today's refresh-manifests job succeeded, however the current CI macos-latest matrix job is missing the cache, and failing with a linker error.
       >   = note: Undefined symbols for architecture x86_64:
       >             "_kCFURLVolumeAvailableCapacityForImportantUsageKey", referenced from:
       >                 _$LT$sysinfo..apple..disk..Disk$u20$as$u20$sysinfo..traits..DiskExt$GT$::refresh::ha090b62f02761b2b in libsysinfo-bfbd6883445a3969.rlib(sysinfo-bfbd6883445a3969.sysinfo.4f5fc5ae-cgu.15.rcgu.o)
       >                 sysinfo::apple::disk::get_disks::h63aaac400a8fb64f in libsysinfo-bfbd6883445a3969.rlib(sysinfo-bfbd6883445a3969.sysinfo.4f5fc5ae-cgu.15.rcgu.o)
       >           ld: symbol(s) not found for architecture x86_64
       >           clang-11: error: linker command failed with exit code 1 (use -v to see invocation)
       >
       >
       > error: could not compile `forc` due to previous error
       For full logs, run 'nix log /nix/store/vgq1zhv6q43wbgbd5hifm26ab2wikwj9-forc-0.39.1.drv'.
error: 1 dependencies of derivation '/nix/store/jfxshavhlsxkkdfnjdqpxfcyjv1icj6f-fuel-nightly.drv' failed to build

All that said, it's unclear how the latest refresh-manifests action succeeded without incurring the same linker error, or if sysinfo is definitely the cause of the cache invalidation :thinking:

It looks like the same issue was encountered in nushell https://github.com/NixOS/nixpkgs/issues/214872. Apparently fixed in https://github.com/NixOS/nixpkgs/pull/215782. It could be worth trying a similar patch.

The fuel-nightly macos-latest failure appears unrelated, tracking under https://github.com/FuelLabs/fuel.nix/issues/64, will investigate shortly!

I think this should at least publish to the gh-pages branch and the github.io link for now - I'll open up a follow-up issue for the cname redirect.


Delete messes up the diff view (in the diffing branch)

I've attached the new database/resource.
Comparing revision 1 and 7 results in:

However, the diff is (only the first object has been deleted (and something is completely weird): delete json jn:doc('json-db','business-service-providers.json')=>paths=>"/business_service_providers/search"=>get=>parameters[[0]] :
localhost/json-db/business-service-providers.json (JSON)> diff -s 1 -e 7
[
  {
    "delete": {
      "nodeKey": 34,
      "deweyID": "<IP_ADDRESS>.<IP_ADDRESS>.49.0.17",
      "depth": 6
    }
  },
  {
    "insert": {
      "nodeKey": 188,
      "insertPositionNodeKey": 86,
      "insertPosition": "asRightSibling",
      "deweyID": "<IP_ADDRESS>.<IP_ADDRESS>.49.0.97",
      "depth": 6,
      "type": "jsonFragment",
      "data": "{\"name\":\"keyword\"}"
    }
  },
  {
    "insert": {
      "nodeKey": 191,
      "insertPositionNodeKey": 188,
      "insertPosition": "asRightSibling",
      "deweyID": "<IP_ADDRESS>.<IP_ADDRESS>.49.0.113",
      "depth": 6,
      "type": "jsonFragment",
      "data": "{\"name\":\"keyword\"}"
    }
  },
  {
    "insert": {
      "nodeKey": 194,
      "insertPositionNodeKey": 191,
      "insertPosition": "asRightSibling",
      "deweyID": "<IP_ADDRESS>.<IP_ADDRESS>.49.0.129",
      "depth": 6,
      "type": "jsonFragment",
      "data": "{\"name\":\"keyword\"}"
    }
  },
  {
    "insert": {
      "nodeKey": 197,
      "insertPositionNodeKey": 194,
      "insertPosition": "asRightSibling",
      "deweyID": "<IP_ADDRESS>.<IP_ADDRESS>.49.0.145",
      "depth": 6,
      "type": "jsonFragment",
      "data": "{\"name\":\"keyword\"}"
    }
  },
  {
    "insert": {
      "nodeKey": 200,
      "insertPositionNodeKey": 197,
      "insertPosition": "asRightSibling",
      "deweyID": "<IP_ADDRESS>.<IP_ADDRESS>.49.0.161",
      "depth": 6,
      "type": "jsonFragment",
      "data": "{\"name\":\"keyword\"}"
    }
  }
]

json-db.zip

After fixing this you may want to test consecutive deletions, just run the query again and compare revision 1 and 8 :-)

What remains to be fixed, is the red highlighting of the entirety of the deleted subtree, and not just of the root of the deleted subtree.


RFM - ofview

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://chocolatey.org/packages/ofview
Package source URL: https://github.com/dtgm/chocolatey-packages/tree/master/automatic/ofview/
Date the maintainer was contacted (in YYYY-MM-DD): 2021-12-13
How the maintainer was contacted: email

This package is updated by TakataSanshiro, this issue isn't really needed.
Have a nice day.

Hello, this package is maintained by @TakataSanshiro and up to date.
This can be closed.


Develop Stream 2022-11-14

This pull request depends on https://github.com/ROCmSoftwarePlatform/rocRAND/pull/300, which adds (among other things) support for MT19937 to rocRAND. Changelist:

Removed references based on a Palamida scan (made one year ago).
Fixed package test by removing rocm_install. This test is not part of the hipRAND package, but rather a test whether the package can install successfully. It should not be installed itself.
Add support for 64-bits functions, mainly hiprandGenerateLongLong.
Add rocRAND backend support for Sobol64, scrambled Sobol 32, scrambled Sobol 64, and MT19937.
Add clang-format file.
Remove redundant -mf16c flag from the build script, similar to this change in rocRAND.

Added commits related to the documentation


Upgrade to new version of Tilt

Frank currently specifies gem version 0.9 of Tilt, but the current release of Tilt is 1.3.3. Any particular reason for this?
The reason I ask is that Tilt 0.9 doesn't work with the current version of the Less gem, so compiling Less templates effectively doesn't work in Frank right now...

Just dove in with this. It seems like @import should pick up files relative to the current less file, but it doesn't. Does seem to be a problem with the less gem.
@rarestep: as a workaround I put the following in my setup.rb
require 'tilt'
require 'less'

class LessTemplate < Tilt::LessTemplate
  def prepare
    if ::Less.const_defined? :Engine
      @engine = ::Less::Engine.new(data)
    else
      options = @options.merge(:paths => [
        File.join(File.dirname(__FILE__), '/dynamic/css')
        File.join(File.dirname(__FILE__), '/dynamic/css/bootstrap'),
      ])
      parser  = ::Less::Parser.new(options.merge :filename => eval_file, :line => line)
      @engine = parser.parse(data)
    end
  end
end
Tilt.register 'less', LessTemplate

Note that the less gem segfaults my frank server process frequently but YMMV. Seems like an alternate that uses the lessc binary would be cool. For now, I have a build.sh script like this:
BASE=$PWD

rm -rf $BASE/build

cd static/less
  lessc site.less > $BASE/static/css/site.css
cd $BASE
frank export --production build


Update minimal supported PHP version to 7.2.

Because our CI tests PHP version is greater than or equal to 7.2 (Many composer dependencies need PHP 7.2+).

This seems not fully done as package.xml (from pecl package) still say 7.1
 <dependencies>
  <required>
   <php>
    <min>7.1.0</min>
   </php>

This seems not fully done as package.xml (from pecl package) still say 7.1

Thanks, I will fix it later.


add data class example

What does the program do ?
showing sample of using data class
In what programming language it is written?
Kotlin

Thank you for participating  in hacktoberfest ❣ if you liked working on this repo you can share it with friends and star this project for helping others


Configuration guide: more readable multi-column tables

Based on and supersedes #846.

@lukebakken ready for another round.

@gerhard and I agreed to switch all tables to be 2 columns, with defaults being the bottom part of the Description column.

I just checked on OS X using rabbitmq.com/configuration.html - looks great 👍

Good improvement 👍


OpenHV (OpenRA based real-time strategy game)

Project description
A mod for OpenRA based on the Hard Vacuum design by Daniel Cook. It aims to be an open content real-time strategy game with no exceptions.
Metadata

homepage URL: https://openhv.itch.io/openhv
source URL: https://github.com/OpenHV/OpenHV
license: gpl3
platforms: linux

Just a heads up for anybody that wants to try packaging this before they hit the same roadblocks I did:

The existing OpenRA package relies on old engine versions and does things that don't work on the current engines (the thirdparty directory doesn't exist for example)
OpenHV uses dotnet which downloads NuGet packages as part of the build, probably the way to go is to use buildDotnetModule
There's a PR that builds newer engines with  buildDotnetModule


[Docs] Add missing else to guard statements in code listing

Fixes rdar://84895831

@swift-ci Please test.

Re-targeted this from the Swift 5.5 release branch to 'main'.

@swift-ci please smoke test and merge


It is recommended to disable CDN acceleration for deno.land.

I'm on a network that's reporting a 403 Forbidden error, which seems to be due to a deno. land is using a restriction imposed by the cdn accelerator, which has to do a human check...
> deno run https://deno.land/std/examples/curl.ts https://example.com
Download https://deno.land/std/examples/curl.ts
error: Import 'https://deno.land/std/examples/curl.ts' failed: 403 Forbidden

We use Cloudflare as a CDN for modules. If you have an issue reaching Cloudflare servers, or if they are mistakenly marking your requests as invalid, you should contact Cloudflare support.

We use Cloudflare as a CDN for modules. If you have an issue reaching Cloudflare servers, or if they are mistakenly marking your requests as invalid, you should contact Cloudflare support.
If I may ask, which geographical region are you based in?

I am in Zhejiang, China and I will try to contact Cloudflare, but there are many people who have the same problem as me!

@lucacasonato
Cloudflare responded to me as follows : https://community.cloudflare.com/t/cloudflare-is-preventing-me-from-using-deno-land-properly/181539
That is probably not the best choice on their side, as Cloudflare is mostly intended for interactive services and not automated scripting, where you can run into exactly such issues.

That is something they need to address (e.g. adjust their security level) and there is not much Cloudflare can do I am afraid.

@2234839 Can you try again? The security level is now less strict.

I'm able to download the code normally, thank you very much for the changes!

I am not sure if this is related, but going to any link like https://deno.land/std/examples/curl.ts sends me to https://www.cloudflare-terms-of-service-abuse.com/streaming.ts. Is this because I'm using a browser?

No what you are experiencing is denoland/deno_website2#1684.


[Doc] "negatively acknowledged cumulatively" ambiguous?

Search before asking

[X] I searched in the issues and found nothing similar.

What issue do you find in Pulsar docs?
This website:
https://pulsar.apache.org/docs/concepts-messaging/#negative-acknowledgement
The:
Messages are negatively acknowledged individually or cumulatively, depending on the consumption subscription type.
The inner "opened indispensable cumulatively", does it have this function in pulsar? At least I did not find the corresponding API for this function.
I don't know if there is some ambiguity in the document, or if there is some problem with my understanding.
In addition:
In Exclusive and Failover subscription types, consumers only negatively acknowledge the last message they receive.
In Shared and Key_Shared subscription types, consumers can negatively acknowledge messages individually.
Are these two paragraphs necessary? In fact, it is the feature of four subscription types, namely whether there is one consumer or how many consumers, and it feels like adding these two paragraphs adds complexity to this technical point.
What is your suggestion?
?
Any reference?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Do you want this? https://pulsar.apache.org/docs/concepts-messaging#acknowledgement

Do you want this? https://pulsar.apache.org/docs/concepts-messaging#acknowledgement

YES，i see the doc too .

@momo-jun please see the question

Messages are negatively acknowledged individually or cumulatively, depending on the consumption subscription type.

In Exclusive and Failover subscription types, consumers only negatively acknowledge the last message they receive.
In Shared and Key_Shared subscription types, consumers can negatively acknowledge messages individually.

I don't see the ambiguity in this paragraph. It means the negative acknowledgment of messages can be performed either individually (in Shared and Key_Shared subscription types) or cumulatively (Exclusive and Failover subscription types).
Did this answer your question? @startjava

@momo-jun
question1：
why
（1）individually (in Shared and Key_Shared subscription types)
（2）cumulatively (Exclusive and Failover subscription types).
why bottom is error：
（1）cumulatively  (in Shared and Key_Shared subscription types)
（2）individually  (Exclusive and Failover subscription types).
question2：
but not have about "negatively acknowledged cumulatively" api，only have "negatively acknowledged individually" api(Consumer.negativeAcknowledeg)
i have 2 question
thank you jun !

You can try negativeAcknowledge in Failover/Exclusive subscription types. All the messages the consumer didn't acknowledge will be redelivered, this behavior could be negatively acknowledged cumulatively. But there do have some discussions about the redelivery behavior on Failover/Exclusive subscription types, see https://lists.apache.org/thread/vdwhg5wrpjc8x41oqwxrsdhx7dmyzt3j

but i don't know what mean "negatively acknowledged cumulatively"。

but i don't know what mean "negatively acknowledged cumulatively"。

You can understand it as "Cumulative NACK" - the negative acknowledgement is performed cumulatively, that is, consumers only negatively acknowledge the last message they receive.


Does this create the app?

When I tried this all I got was the app running.. it didn't create anything anywhere for me to then distribute... should it create something? And if so where and how do I do this?
In the past I have just downloaded the latest release of atom-shell/electron and then just moved my /app folder into the resources folder for the .exe / .app and then renamed it and distributed.
Does this package offer any of that??

nope see electron-packager for that

So to clarify... All this does is just run it for testing only? Nothing more!
Sent from my iPhone

On 31 May 2015, at 20:34, maxogden<EMAIL_ADDRESS>wrote:
nope see electron-packager for that
—
Reply to this email directly or view it on GitHub.

it's an automated way to install specific versions of electron on your computer for development, you can also manually download the zips if you want


OpenSSL Version 1.1.0 incompatible

Installing on Windows (included with redbird) it seems that the OpenSSL binary needs to be the LTS version as the current (v1.1.x) doesn't include libeay32.

I'm looking for a new maintainer: https://github.com/quartzjer/ursa#maintainer-needed

I would maintain it if I knew what I was doing. :)

@dylmye could you try https://github.com/quartzjer/ursa/pull/149 please?
You'll need Node >=6.3.0

Hi, sorry, meant to reply, it was ursa included in a different package, I'll try though


not working for php version 8.2.1

not working on the latest php version

Why not?

Because it requires older versions of PHP.
"require": {
"php": "~8.0.0 || ~8.1.0 || ~8.2.0",
too bad I have the latest version of php :(

That should work: ~8.2.0 means >= 8.2.0 && < 8.3.0.

For anyone else that might run into this problem as well. I was getting this error when trying to install the package and it also said I was missing the bcmath extension, which is on my docker environment. I ran the below command, ignoring the bcmath
extension, which allowed the package to be installed and appears to be working as expected.
composer require pelmered/filament-money-field --ignore-platform-req=ext-bcmath


Need hint for fixing Wifi

Hi Clay-Bios,
Thanks a lot for this great repos.
I have a x250 i5 5300U with an Intel 7265AC card. Would you please guide me on how to make the wifi functional?
I've installed Big sur and copied your EFI as it is.
Any help would be much appreciated.
Another QQ, I also have a T470s i5 7300U with a Intel 8265NGW, I was wondering if it would be to much asking to have it to your repos?
Thank you gain.
Charlie

Hi Clay-Bios,
Thanks a lot for this great repos.
I have a x250 i5 5300U with an Intel 7265AC card. Would you please guide me on how to make the wifi functional?
I've installed Big sur and copied your EFI as it is.
Any help would be much appreciated.
Another QQ, I also have a T470s i5 7300U with a Intel 8265NGW, I was wondering if it would be to much asking to have it to your repos?
Thank you gain.
Charlie

Just download the Intel version of EFI from this page, and you don’t need to do anything else.
https://github.com/CLAY-BIOS/Lenovo-ThinkPad-T450s-Hackintosh-OpenCore/releases/tag/OpenCore-0.7.2-v2.8

I feel so !$#$#$%!@$
Thanks a lot.
Any way I can help?   don't hesitate.
Thank you again.
Charlie


Blazor: Change a default binding via BindAttributes.cs

According to BindAttributes.cs it is possible to extend it:
   /// <remarks>
    /// To extend the set of <c>bind</c> attributes, define a public class named
    /// <c>BindAttributes</c> and annotate it with the appropriate attributes.
    /// </remarks>

I was looking for an easy way to change the default binding for text from onchange to oninput globally in my application.
So I did this:
    [BindInputElement("text", null, "value", "oninput", isInvariantCulture: false, format: null)]
    public static class BindAttributes

However, this is not working. Is it supposed to work?
If not, what approach you suggest to change one of the default bindings globally?

@Postlagerkarte Thanks for contacting us.
I believe using BindAttributes only allows adding additional bindings, but not modify existing ones.
@SteveSandersonMS or @rynowak can confirm.

Yes, we don't have a feature of changing the defaults. The recommended approach for this is to define your own "input" component that uses whatever conventions you want, and then use that instead of a plain <input> element.


af1.add_extra_metadata() FileNotFoundError

Greetings all.
We are getting a file not found error when trying to add extra metadata to Af1. This just started happening yesterday.

Thanks @sanjaynagi! Apologies, I was moving some files around yesterday but this should be fixed now.

No worries @ahernank ! Thanks, thats fixed it :)
Closing issue.


chore(graphql): Compliance dashboard without cluster read permission

Description

This change is the combination of the two following PRs without the SAC helper refactoring part:

#13055
#13056

User-facing documentation

[x] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[x] added unit tests

[x] modified existing tests

How I validated my change

See the referenced PRs
#13055
#13056

/retest


Create flatten representation

Is this still necessary after supporting dot-concat key access and recursive iters.

I think the riter recursive iterator together with dot-concat key access is indeed enough. If you really need the flat dictionary, you can just do:
flatcfg = {k: cfg[k] for k in cfg.riter()}


Docker Container Fails to Deploy

Hi,
I'm attempting to deploy bamboo in a docker container as described in the documentation.  I've built the docker image, but it is unable to find the config file when I run it.  Here's what I'm seeing:
[root@bamboo bamboo-0.2.9]# ls -l config/bamboo_test.json
-rw-r--r--. 1 root root 642 Mar  6 20:51 config/bamboo_test.json
[root@bamboo bamboo-0.2.9]# docker run -t -i --rm -p 8000:8000 -p 80:80 \
     -e MARATHON_ENDPOINT=http://xxx.xxx.xxx.xxx:8080 \
     -e BAMBOO_ENDPOINT=http://xxx.xxx.xxx.xxx:8000 \
     -e BAMBOO_ZK_HOST=xxx.xxx.xxx.xxx:2181 \
     -e BAMBOO_ZK_PATH=/bamboo \
     -e BIND=":8000" \
     -e CONFIG_PATH="config/bamboo_test.json" \
     -e BAMBOO_DOCKER_AUTO_HOST=true \
     bamboo
sed: can't read config/bamboo_test.json: No such file or directory

Thanks in advance for the help.

CONFIG_PATH should be a path inside the container. If you want to inject an external config, just mount the volume and set your CONFIG_PATH to the config inside the mount.
  That did the trick.  So if I have the custom config in place prior to building the container, would I no longer need to mount the external volume?
Thanks for the help.

Correct, so long as the file can be referenced from within the container, you're set

Perfect.
Thanks again for the help and quick response.

@mbabineau  Thanks for helping out support questions.


How to customize tabs?

I am testing PSWriteHTML and there isn't a whole lot of documentation, I checked the blogs and searched the code as well; is there a way to customize the tabs css/js?
I don't want the on-click effect or the fa-bomb icon but wasn't able to see any way to change it through parameters?
Here is my example..
Install-Module PSWriteHTML -Force
Import-Module PSWriteHTML -Force

$CSV = Import-Csv -Path C:\IT\process.csv

New-HTML -TitleText "Infrastructure Report" -UseCssLinks:$false -UseJavaScriptLinks:$false -Author "test" -FilePath C:\IT\infra.html -ShowHTML {
    New-HTMLTab -TabName 'TESTING' -TabHeading "testing"  {
        New-HTMLContent -HeaderText 'Content Text' -CanCollapse {
            New-HTMLTable -DataTable $CSV -Buttons @('excelHtml5','csvHtml5') -HideFooter
        }
    }
    New-HTMLTab -TabName 'TESTING' -TabHeading "testing" {
        New-HTMLContent -HeaderText 'Content Text' -CanCollapse {
            New-HTMLTable -DataTable $CSV -Buttons @('excelHtml5','csvHtml5') -HideFooter
        }
    }
}

I would like to give you good news but I can't. There's like 0 configuration for Tabs. The bomb is there kind of by accident as I wanted to load/allow all icons from Fonts Awesome to be available but failed with providing a guide. So I stuck with a bomb for a starter. The idea is that you should be able to pick None or one of 5xxx icons per each tab.
The idea of on-click effect - well that isn't configurable either. But you right, it should be.
Seeing as other person requested Tabs within Tabs, Tabs require a rewrite. I just need to find good looking, So I need to find "free" CSS/js code with nice looking tabs within tabs that can be easily adapted and change things around.
As for custom CSS/js, it's a bit tricky. There are lots of moving parts in PSWriteHTML and touching one thing may break other stuff. I would prefer people doing some research over PSWriteHTML and help out with some choices/configuration. The only thing I need is for it to be useful and good looking (to an extent).

I'm working a bit on the tabs:

You'll be able to choose one of 1000+ icons.
Import-Module .\PSWriteHTML.psd1 -Force

$Test = Get-Process | Select-Object -First 5

#$Test | Out-HtmlView

New-HTML -TitleText 'My title' -UseCssLinks:$true -UseJavaScriptLinks:$true -FilePath $PSScriptRoot\Example22.html -Show {
    New-HTMLTab -IconBrands aws -Name 'Test 1' -IconColor DarkGoldenrod {
        New-HTMLTable -DataTable $Test -PagingOptions @(50, 100, 150, 200) {
            New-HTMLTableButtonPDF -PageSize TABLOID -Orientation portrait
            New-HTMLTableButtonExcel
            New-HTMLTableCondition -Name 'HandleCount' -Type number -Operator gt -Value 300 -BackgroundColor Yellow
            New-HTMLTableCondition -Name 'ID' -Type number -Operator gt -Value 16000 -BackgroundColor Green
            New-HTMLTableCondition -Name 'Name' -Type string -Operator eq -Value 'browser_broker' -BackgroundColor Gold -Row
        }
    }
    New-HTMLTab -Name 'Test 2' -IconSolid address-card {

    }
    New-HTMLTab -Name 'Test 3' -IconSolid camera {

    }
    New-HTMLTab -Name 'Test 4' -IconBrands microsoft {

    }
    New-HTMLTab -Name 'Test 5' -IconRegular calendar {

    }
    New-HTMLTab -Name 'Test 6' -IconSolid yin-yang {

    }
}

You will be able to define a different color for icon and different for text. Same for size. Also, no icon is a possibility.


Guide: Data Wrangling

Add guide on Data Wrangling. Cover functions with examples:

[x] summarize_by_time()
[x] future_frame()
[x] pad_by_time()

Will be housed here: https://business-science.github.io/pytimetk/guides/04_wrangling.html
Lead: Lucas O

Hi @mdancho84, I'm not sure if there is an issue with pad_by_time() or maybe I'm not using it correctly. It appears the 'auto' option of the 'freq' parameter is ONLY detecting daily frequency.  When I try setting freq to auto for data with weekly or monthly frequency, I get an error saying ValueError: The frequency could not be detectied.
Here is the code I'm using to test daily pad by time -
# Daily Example ----

date_range = pd.date_range(start = '2023-09-18', end = '2023-09-25')
value = range(len(date_range))

daily_df = pd.DataFrame({
	'date': date_range,
	'value': value
}) \
    .drop([5, 6])

daily_df \
	.pad_by_time(
		date_column = 'date',
		freq        = 'auto',

	)

And here is the code for weekly pad by time which errors -
# Weekly Example ----

# Create a date range for 7 weeks
date_range = pd.date_range(start= "2023-01-01", periods=7, freq='W-SUN')
value = range(len(date_range))

weekly_df = pd.DataFrame({
	'week': date_range,
	'value': value
}) \
    .drop([2, 4])

weekly_df \
    .pad_by_time(
		date_column = 'week',
		freq        = 'auto'
	)

I'm going to remove the "auto" option. So just pretend it doesn't exist. And have the user specify the frequency to pad to.

Ok. Created a PR for pad_by_time() guides. I can get started on augment this weekend.

Thanks a lot. I'll fix pad by time soon.

#29 fixes this. Closing this issue.


[POC] Integrate Lootie animation on homepage

Create proof of concept that we can play custom after effects animations with ease on the homepage

https://github.com/LottieFiles/lottie-react

[ ] Check if file size of the bundle is not too large

[ ] Check if we need additional 3rd party dependencies

infografic_team_v2.tar.gz
:information_source: Contact Martin Simeonov (ninjaro) for latest version of the sources
https://user-images.githubusercontent.com/893608/109537640-c957ff00-7ac7-11eb-9b21-f83a284d1c36.mp4

Closing as outdated


Bug 1719188: fix IsOperatorAlwaysManaged check so operator conditions are set to u…

…nknown if unmanaged is NOT supported
Now, are operators which want only to support managed, and not unmanaged etc. already calling SetOperatorAlwaysManaged() at
https://github.com/gabemontero/library-go/blob/master/pkg/operator/management/management_state.go#L25-L27
Or are they not calling that method and getting desired behavior by accident?
Do we have a precise list of operator who need to be "always managed"

/assign @deads2k
@mfojtik @bparees @openshift/openshift-team-developer-experience ptal / fyi


chore(master): release 2.4.1

:robot: I have created a release beep boop
2.4.1 (2023-05-15)
Bug Fixes

player: pre-emptively update .current if v3 (#78) (3ecb4d9)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ooliver1/mafic/releases/tag/v2.4.1 :sunflower:


[XLA:GPU] Updated hlo_to_llvm_ir to also emit PTX and re-enable tests using it.

@sanjoy

@nouiz  Can you please address Ubuntu Sanity errors? Thanks!

Done. I also updated the new tests. I had forgot the CHECK-LABEL.

https://github.com/tensorflow/tensorflow/pull/47707


AirPlay: web app in wrong interface orientation

The web app loaded on the Tv follows the orientation of the device;
so if you start loading when iPhone/iPad are in landscape you can see the web app cut and in wrong orientation.

I self resolved my issue and posted solution here:
http://stackoverflow.com/questions/30040055/uiviewcontroller-displayed-sideways-on-airplay-screen-when-launched-from-landsca/30355853#30355853


System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.TeamFoundation.SourceControl.WebApi

Hello,
I am facing an error while importing to azure devops cloud.
The error is:  Unexpected migration error.
[System.IO.FileNotFoundException] System.IO.FileNotFoundException: Could not load file or assembly 'Microsoft.TeamFoundation.SourceControl.WebApi, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified.
wi-import-log-231227-113732.txt
config-agileV2.json
jira-export-log-231227-101038.txt
command in CLI: wi-import --token xxxxxxxx --url https://dev.azure.com/ITNR-DLWR/ --config config-agileV2.json --force
Version: 3.0.303
I really don't know how to fix this. Could it be due to permission on the devops?

@keeperlou Try the latest versino (3.0.308)!
What OS/Platform are you running on?

@Alexander-Hjelm   this on windos 11

@Alexander-Hjelm I re-tried it with latest version but still the same error:
wi-import-log-231228-120635.txt
Could this be an error due to not enough permissions on devops?
thank you

Verify that the path to the workspace and the config file are correct, first of all.
You can check the necessary ADO permissions here: https://github.com/solidify/jira-azuredevops-migrator/blob/master/docs/overview.md#azure-devopstfs-requirements

@Alexander-Hjelm
I re-tried it with the latest version. All permissions are ok. Path of workspace is ok. The config file is between the extracted files
Could the error due to windows 11?
I have no clue on what I can try next
Thank you very much for your help

Getting exactly the same error here also.
v.3.0.310 - Win10 - ADO permissions, workspace & config file links all correct.


v2podman image sign

this is a straight port to add the podman image sign command. no improvements or refactoring done
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

Vendor issue?  Changes LGTM fwiw.
cd ${CIRRUS_WORKING_DIR} && ./hack/tree_status.sh
tree is dirty, please sync the vendor.conf and commit all changes.

needs merge love

/lgtm

/hold

/lgtm


black box instance names aren't taken from Chisel code

For most modules, the emitted verilog is now very meaningful (thanks!). However, this is not true for modules which are black boxes. An example is SimDTM in TestHarness.scala. It has a name (dtm)in the Chisel but in the final Verilog it is SimDTM.
class BlackBoxModule extends BlackBox {
...
}
class MyModule extends Module {
...
val my_black_box = Module ( new BlackBoxModule)
...
}
This elaborates with an instance name of BlackBoxModule_1 instead my_black_box.

I took a quick look and it's not immediately obvious why this is going wrong.  I'd like a fix, too!

strange... I can't seam to reproduce this
  module MyModule :
    input clk : Clock
    input reset : UInt<1>
    output io : {}

    io is invalid
    inst my_black_box of BlackBoxModule @[Test.scala 13:29]

I think that actually it has nothing to do with black box. Is this expected
behavior if the module or black box is instantiated conditionally (inside
an if())?
On Friday, August 19, 2016, Kamyar Mohajerani<EMAIL_ADDRESS>wrote:

strange... I can't seam to reproduce this
module MyModule :
input clk : Clock
input reset : UInt<1>
output io : {}
io is invalid
inst my_black_box of BlackBoxModule @[Test.scala 13:29]

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ucb-bar/chisel3/issues/262#issuecomment-240971599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARCAJMy2umsOENhUxo75npJLwXg8qD5Uks5qhXbxgaJpZM4JoEXf
.

--
Megan A. Wachs
Engineer | SiFive, Inc
300 Brannan St, Suite 403
San Francisco, CA  94107
<EMAIL_ADDRESS>

This also produces the correct instance name (my_gray_module) for me:
class BlackBoxModule extends BlackBox {
  val io = new Bundle {
    val in = UInt(INPUT, width = 8)
  }

}

class SubModule extends BlackBox {
  val io = new Bundle {
    val in = UInt(INPUT, width = 8)
  }
}

class MyModule(val black: Boolean) extends Module {
  val io = new Bundle {
    val in = UInt(INPUT, width = 8)
  }

  val my_gray_module = Module(
    if (black)
      new BlackBoxModule
    else
      new SubModule
  )
}

Am I missing something here?

That's not exactly what I meant.  I'll create a small test case that actually shows my question.

Here's what I'm actually seeing, and understand if it's a less simple issue to resolve. I'll update the title of this issue. It is if you conditionally declare the val at all, not just decide what to assign to it conditionally.
class Simple extends Module {
  val io = new Bundle {
    val a = Bool(INPUT)
    val b = Bool(OUTPUT)
  }
  b := a
}

class Top extends Module {

  if (true) {
    val my_simple = Module (new Simple())
  }
}

For me, this produces:

  Simple Simple_1 (
    .clk(Simple_1_clk),
    .reset(Simple_1_reset),
    .io_a(Simple_1_io_a),
    .io_b(Simple_1_io_b)
  );

That's expected behavior as the current naming mechanism uses scala(java) reflection to name the members. It only looks for the "val"s inside the body of your Module class (or the classes you may have extended from).
I believe we can enhance this naming mechanism inside the Module() macro by checking the parent context tree. AFAIR sbt uses the same mechanism for its "setting" key namings.

Cool, that all makes sense. I was trying to figure out how to flag this as "feature request" vs actual bug/issue, but not sure how.

Superseded by #303 and implementation pending approval in #327


Automatically detect time series dataset

Missing functionality
When I first know this library, I didn't know if it has timeseries mode. I was absolutely beginner in data science.
I hope this feature can give practical output for newcomers, as they might not know the difference between timeseries dataset and the regular ones.
Solutions

#1180 [rejected]
Check if any values has type of pandas._libs.tslibs.timestamps.Timestamp. This solution only works when user set parse_dates parameter in pandas.read_csv, pandas.read_excel, etc.
Any other solution?

Is this feature request open for contribution? If yes, can we have a discussion about the end goal of this feature request?

Yes sure, @Sohaib90! Can share what you had in mind?


Set approve label only if reviewer is part of approvers

Add approve label only if the reviewer is an approver, other users will set as lgtm-<user>  label

/verified


Running a container from within a container (via mounting docker.sock) fails to preserve volume mount changes

Output of docker version:
Client:
 Version:         1.9.1
 API version:     1.21
 Package version: docker-1.9.1-25.el7.centos.x86_64
 Go version:      go1.4.2
 Git commit:      78ee77d/1.9.1
 Built:
 OS/Arch:         linux/amd64

Server:
 Version:         1.9.1
 API version:     1.21
 Package version: docker-1.9.1-25.el7.centos.x86_64
 Go version:      go1.4.2
 Git commit:      78ee77d/1.9.1
 Built:
 OS/Arch:         linux/amd64

Output of docker info:
Containers: 80
Images: 453
Server Version: 1.9.1
Storage Driver: devicemapper
 Pool Name: docker-253:1-293612214-pool
 Pool Blocksize: 65.54 kB
 Base Device Size: 10.74 GB
 Backing Filesystem:
 Data file: /dev/loop0
 Metadata file: /dev/loop1
 Data Space Used: 32.07 GB
 Data Space Total: 107.4 GB
 Data Space Available: 48.45 GB
 Metadata Space Used: 35.35 MB
 Metadata Space Total: 2.147 GB
 Metadata Space Available: 2.112 GB
 Udev Sync Supported: true
 Deferred Removal Enabled: false
 Deferred Deletion Enabled: false
 Deferred Deleted Device Count: 0
 Data loop file: /var/lib/docker/devicemapper/devicemapper/data
 Metadata loop file: /var/lib/docker/devicemapper/devicemapper/metadata
 Library Version: 1.02.107-RHEL7 (2015-12-01)
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 3.10.0-229.1.2.el7.x86_64
Operating System: CentOS Linux 7 (Core)
CPUs: 4
Total Memory: 7.64 GiB
Name: databus-jramapuram
ID: 5SPZ:LWT5:XYC3:2C7N:VM3C:KZFF:WRN3:5B27:BNBO:PFHL:XCXS:TQPR

Additional environment details (AWS, VirtualBox, physical, etc.):
Steps to reproduce the issue:
docker run -it -v mydir:/mydir -v /var/run/docker.sock:/var/run/docker.sock centos:7 bash
# do stuff to install docker
docker run -it -v /mydir:/mydir -v /var/run/docker.sock:/var/run/docker.sock centos:7 bash
touch /mydir/tmp
exit # exit container 1
exit # exit container 0
ls mydir # has no results here on original host

Describe the results you received:
No tmp in mydir
Describe the results you expected:
I would have expected tmp to exist inside mydir
Additional information you deem important (e.g. issue happens only occasionally):
This happens for containers run within containers using the docker.sock mounted from the original host.

Your first container is started with a named volume, not a bind-mounted directory; a bind-mounted directory should always be an absolute path, if not, (and the volume doesn't start with a slash) then docker creates a volume with that name to store the data.
You'll likely find a volume named "mydir" if you run docker volume ls (See https://docs.docker.com/v1.9/engine/reference/commandline/volume_ls/), that contains the tmp you created.
You can check if that volume contains the tmp file, by using the same volume again; doing;
docker run -it -v mydir:/mydir centos:7 bash

And then check the contents of "mydir" inside the container

Hi @thaJeztah , thanks for your comment.
This issue happens even with volume mounting (i.e. with a fully qualified path):
[centos@jramapuram ~]$ docker run -v /tmp/rpmbuild:/rpmbuild -v /var/run/docker.sock:/var/run/docker.sock -v /usr/bin/docker:/usr/bin/docker -it centos:7 bash
Usage of loopback devices is strongly discouraged for production use. Either use `--storage-opt dm.thinpooldev` or use `--storage-opt dm.no_warn_on_loop_devices=true` to suppress this warning.
[root@46a5434248c6 /]# ls
anaconda-post.log  bin  dev  etc  home  lib  lib64  lost+found  media  mnt  opt  proc  root  rpmbuild  run  sbin  srv  sys  tmp  usr  var
[root@46a5434248c6 /]# docker run -v /rpmbuild:/rpmbuild -v /var/run/docker.sock:/var/run/docker.sock -v /usr/bin/docker:/usr/bin/docker -it centos:7 bash
Usage of loopback devices is strongly discouraged for production use. Either use `--storage-opt dm.thinpooldev` or use `--storage-opt dm.no_warn_on_loop_devices=true` to suppress this warning.
[root@9edbc3b922f4 /]# touch /rpmbuild/test
[root@9edbc3b922f4 rpmbuild]# ls /rpmbuild
test
[root@9edbc3b922f4 rpmbuild]# exit
exit
[root@46a5434248c6 /]# ls rpmbuild/
[root@46a5434248c6 /]#

the test file does not persist into the host container.

Oh, actually, I understand what's happening. You're not running docker-in-docker, but even though you're running inside a container, you're still controlling the docker daemon that's running on the host.
The -v /some/path:/container/path is always using the filesystem that the daemon runs on, which in this case, is /rpmbuild on your host. If you run ls /rpmbuild/ on your host (not inside a container), do you see the test file there?

@thaJeztah : Yes, you are correct! Thank you!


ASP.NET WebAPI 2 Error Processing Queue Object Reference Exception

We just started a new project using Angular 1.x and Web API 2.x and have installed your lovely extension. On my colleagues machine we are able to generate everything that we need perfectly but unfortunately we are unable to get the template rendering to work on other machines. I am getting the below output when I attempt to. Having a look at the RoslynFileMetadata constructor I can only assume one of the objects being initialized is returning null. Please advise! Thanks in advance.
ERROR: Error processing queue: Object reference not set to an instance of an object.
   at Typewriter.Metadata.Roslyn.RoslynMetadataProvider.GetFile(String path)
   at Typewriter.Generation.Controllers.GenerationController.<>c__DisplayClass5_0.<OnTemplateChanged>b__1()
   at Typewriter.Generation.Controllers.EventQueue.ProcessQueue()

Hi @mlturner88,
I haven't seen this problem before. Are you running Visual Studio 2015 Update 2? Try to uninstall Typewriter, restart Visual Studio and install Typewriter again.

My apologies I should have mentioned that before. I am using VS2013 Update 5. I've tried reinstalling Typewriter a few times now without any success, unfortunately.

Ah ok that's wierd, vs 2013 should use the CodeDomMetadataProvider, not Roslyn... Have you installed some kind of Roslyn preview for vs 2013 or something like that? Also, is your colleague running 2013 update 5?

I haven't to my knowledge. My colleague is using the same version as me, yes. The main difference is I have the Xamarin tools installed and he doesn't.

Look for "Roslyn Preview" in Extensions and Updates.. If it's there, try to disable it and restart Visual Studio

I'm not seeing that installed. I just updated to 1.7.3 as well with no luck.

I just published a new version (1.7.4) which hopefully fixes this issue.

That fixed it. Thank you very much!


[SPARKR-241] R to fail early if SparkR is missing

Same as https://github.com/amplab-extras/SparkR-pkg/pull/233
https://sparkr.atlassian.net/browse/SPARKR-241

And https://github.com/amplab-extras/spark/pull/4

Thanks @felixcheung - LGTM. Merging this


Create "accessibility" page

Automatically generated by Netlify CMS
Suggestion to pull Accessibility out from General principles as its own page. As currently structured, it's getting lost at the bottom of the page. We could also add jump links, but it's nice to have the page appear in the left hand nav.

Sounds good to me. I think we should have an accessibility section for each component too. It would be nice to have straightforward instructions on how to test the megamenu, for instance, for accessibility, without needing to remember how to translate the generic instructions into how the menu should be tested.


Documentation: Minor improvements

What could be improved

Getting Started -> Installation -> Multiple links to latest version (https://docs.flame-engine.org/main/#installation)
repository, It contains quite a should be repository. It contains quite a or repository, it contains quite a (https://docs.flame-engine.org/main/#id1)
This section needs some rephrasing. Specifically the second sentence. It feels too lengthy and incorrect grammatically (https://docs.flame-engine.org/main/#outside-of-the-scope-of-the-engine)
In Game Loop section, fix typo in first sentence. (https://docs.flame-engine.org/main/flame/game.html#flamegame)
In Lifecycle section, the sentence When a game first is added to a Flutter widget tree should be When a game is first added to a Flutter widget tree (https://docs.flame-engine.org/main/flame/game.html#lifecycle)
In Priority section, there is no or part for either in the statement To update the priority of a component you have to either just set it to a new value (https://docs.flame-engine.org/main/flame/components.html#priority)

Why should this be improved

Just some quality of life improvements.
Good to know
Make sure that you comment on the issue that you will be working on this before starting to program.
To set up the Flame environment you need to follow the instructions in our contributing.md doc, if you have any problems setting up your environment (or other issues of course) feel free to ask for help in the Flame channel on our Discord server.
Since this is a documentation change, you need to be able to build the docs. You can do that by running melos sphinxdoc after you have followed the instructions in the contributing.md doc and also installed Sphinx on your system. After you have built the docs you can open the doc/_build/html/index.html file in your web browser to see the changes.
Make sure to include a video of the example when you upload the PR so that it will be easier for us to review it. (You can attach videos directly in the PR, so no need to upload them somewhere else)
Happy hacking!

hey, @spydon @ufrshubham could this issue be assigned to me?

Can you assign this to me I can do some rephrasing of the sentences and also omit the typos..

Can you assign this to me I can do some rephrasing of the sentences and also omit the typos..

This is already taken by @vikramadityasinghs

Okay🥲

is this fixed by @vikramadityasinghs yet or can I try to fix it?

I also want to try if it can still be assigned to me eagerly waiting
On Wed, 5 Oct 2022 at 20:11, Aatmaj @.***> wrote:

is this fixed by @vikramadityasinghs
https://github.com/vikramadityasinghs yet or can I try to fix it?
—
Reply to this email directly, view it on GitHub
https://github.com/flame-engine/flame/issues/2002#issuecomment-1268535473,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASKQVJC6TGGVZMFPZ5ABXBTWBWHRNANCNFSM6AAAAAAQ3MMUD4
.
You are receiving this because you commented.Message ID:
@.***>

Is this still open?

They merged a few fixes. The major sections are still left.  I would opine that my other assignment will clash slightly with this, so be ready for merge conflicts.

I am also ready to take it... i don't have any other assignments in my hands right now, so if it is assigned to me I can try my best to solve issues..

This is already assigned to @vikramadityasinghs, let's not stress people. If you decide to not do it @vikramadityasinghs, please let us know. 🙂

I don't think anything is fixed from this issue.

I just saw the merged so I thought a partial fix was implemented,  lol

Me too but I think it is still open😅
On Fri, 7 Oct, 2022, 9:45 pm Munsterlander, @.***>
wrote:

I don't think anything is fixed from this issue.
I just saw the merged so I thought a partial fix was implemented, lol
—
Reply to this email directly, view it on GitHub
https://github.com/flame-engine/flame/issues/2002#issuecomment-1271790870,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASKQVJD7SMSKVUI5LLCFNQTWCBEDJANCNFSM6AAAAAAQ3MMUD4
.
You are receiving this because you commented.Message ID:
@.***>

Any update? 😅

hey @spydon, im interested in working on this.
I'm starting work on this particular issue. 😀

This has been addressed in:  https://github.com/flame-engine/flame/pull/2063


Implement ENV.invert

Just out of curiosity: what is the use case for this method? I really can't think of one

The only real thing I can think of is looking up an environment key by value..
inverted = ENV.invert
key = inverted.key?(some_value)

But then there are more specific methods for that too:
ENV.value?(some_value)
key = ENV.key(some_value)

I don't know when ENV#key and Hash#key were added (Ruby 3.2 maybe?), but I have used invert to look up a key by value in the past, e.g.:
h = { a: 1 }
h.invert[1] # => :a

I only learned about Hash#key because of your recent work. 🎉

https://github.com/ruby/spec/commit/d530cdcd0c9eeabb1bccee7093b037bc43b6ea60#diff-8e0c31c79bb135a02001f89bebf952af22ddd9c8aae849f09f2379a2fcdc8318
Looks like Hash#key has been added in 1.8.8. But to be fair, I never knew about it either.


Settings of  MN and CN

I want to change the settings of my MN and CN because there are not  so many CPU cores on my machine. I wonder where the default setting of MN and CN is defined in the code。

By default, only 1 CPU core is used by each MN. We bind the last CPU core to the MN:
https://github.com/dmemsys/SMART/blob/1274148c849af987eb7e9815da007af8403d4155/src/Directory.cpp#L30-L32
Thus, you should set a appropriate CPU_PHYSICAL_CORE_NUM value:
https://github.com/dmemsys/SMART/blob/1274148c849af987eb7e9815da007af8403d4155/include/Common.h#L23
The number of CPU core used by each CN is determined by the number of clients you launch on each CN. This is because we bind a CPU core to each client:
https://github.com/dmemsys/SMART/blob/1274148c849af987eb7e9815da007af8403d4155/test/ycsb_test.cpp#L206-L207
Thus, just reduce the number of clients you launch on each CN if there are not so many CPU cores on your machine.


Nette 3.0

Do you plan any release as Nette 3.0 is already supported via https://github.com/contributte/gopay/pull/75?


installed appimages do not show any context menu entries, like update, remove, etc.

compare AIL installed appimages vs. cli installed appimages:

This is a feature that should be moved from the AppImageLauncher code to libappimage so it could be used by all clients.

Remove action added in #22

What did you do to make this work?
Can we abstract this as middle layer and also extract the same
functionality from ail and ship it as "appimage-action-menus", so both cli
and ail can simply instruct users to install this without reimplementing it
twice?
On Apr 9, 2019 03:36, "Alexis López Zubieta"<EMAIL_ADDRESS>wrote:

Remove action added in #22
https://github.com/azubieta/appimage-user-tool/pull/22
[image: Screenshot_20190408_213451]
https://user-images.githubusercontent.com/1138094/55767272-31ba1500-5a46-11e9-8d73-9136c8a79aab.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/azubieta/appimage-user-tool/issues/13#issuecomment-481067712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzRhm4nnKmPj9-yTLNrnyfywqyRplhVks5ve-6MgaJpZM4cbn6a
.


Timeout on large file downloads from S3 with s3_file

Cookbook version
4.2
Chef-client version
12.13.37
Platform Details
Ubuntu 14.04 on AWS
We're trying to download a relatively large, text file (3GB) but it results in a 403 forbidden exception. Looking ath the logs it takes almost 3mins just to decide that the local and remote files do not match, this is most likely down to the loading of the existing file to calculate an MD5 hash.
[2017-02-10T12:20:45+00:00] INFO: Processing aws_s3_file[/home/my_user/very_large_file.txt] action create (mycookbooks::corenlp line 43)
[2017-02-10T12:23:20+00:00] INFO: Remote and local files do not match, running create operation.
[2017-02-10T12:23:20+00:00] INFO: Processing remote_file[/home/myuser/very_large_file.txt] action create (/var/chef/runs/1/local-mode-cache/cache/cookbooks/aws/providers/s3_file.rb line 40)
[2017-02-10T12:25:55+00:00] INFO: HTTP Request Returned 403 Forbidden

Options to fix this, not limited to:

Calulate the presigned url just before attempting the download (it seems it's not used in calculating the hash, but I may be wrong)
Allow overriding of the link expiry (currently it's set to 300 seconds)
Allow for skipping of md5 comparison step when using :create action

I think the first option makes the most sense.

I've gone ahead and created a pull request for the first option which does fix this particular issue for me, however it could still cause issues.
In the event that the first attempt to download takes > 5 mins, then fails and the resource specifies a number of retries then the second retry would still result in a 403 forbidden error.
This issue also raises the question on the suitability of how the MD5 hash is being calculated as I know the file on S3 hasn't changed and we have to wait for the file to be redownloaded on every deployment. We can work around this but I wonder if it could still be worth having an option skipping the MD5 hash check.

Closed by #274 - thanks for reporting and the PR to fix it!


Change text color to green in About screen

Description
Update the text color to green in the About screen of the DroidKaigi 2024 app.

Links

Design: https://www.figma.com/design/XUk8WMbKCeIdWD5cz9P9JC/DroidKaigi-2024-App-UI
Code: https://github.com/DroidKaigi/conference-app-2024/blob/055e7d4d7fd2e529cc2a9ce6788cc1f941d35732/feature/about/src/commonMain/kotlin/io/github/droidkaigi/confsched/about/AboutScreen.kt#L81

:raising_hand:

Thanks! Assigned 👍

It looks like this implementation is needed in advance. Please wait a little.
https://github.com/DroidKaigi/conference-app-2024/pull/269

@Lilin-18th
It looks like this implementation is needed in advance. Please wait a little.
https://github.com/DroidKaigi/conference-app-2024/pull/269

Thank you for your patience. We've merged the theme. Please update your development branch with the remote main branch to use the updated color scheme.


Conda pylint installation fails due to broken whitespace handling

Environment data

VS Code version: 1.24.1
Extension version: 2018.5.0
OS and version: Windows 10
Python version: Anaconda (conda 4.5.4, Python 3.6.5)
Type of virtual environment used: conda
Relevant/affected Python packages and their versions: none

Actual behavior
The extension offers to install pylint for me. I click "Install" and the popup asks me whether I want to install it using conda or pip. I choose conda. The installation fails (see below for the output log).
Expected  behavior
The installation succeeds.
Steps to reproduce:

Create a conda environment with a name containing at least one whitespace
Activate this conda environment in Visual Studio Code
Choose to install pylint using conda when prompted to by the extension

One can see in the output below that while the activate command properly handles the whitespace in the environment name, the conda install command does not.
Logs
Windows PowerShell
Copyright (C) 2016 Microsoft Corporation. All rights reserved.

PS C:\Users\dthul> & cmd /k "activate ""My Conda Environment"" & powershell"
Windows PowerShell
Copyright (C) 2016 Microsoft Corporation. All rights reserved.

PS C:\Users\dthul> & conda install --name My Conda Environment pylint

EnvironmentLocationNotFound: Not a conda environment: C:\Users\dthul\AppData\Local\Continuum\Anaconda3\envs\My

Output from Console under the Developer Tools panel:
[Extension Host] Python Extension: Linter 'pylint' is not installed. Please install it or select another linter". Error: Module 'pylint' not installed.
	at PythonExecutionService.<anonymous> (C:\Users\dthul\.vscode\extensions\ms-python.python-2018.5.0\out\client\common\process\pythonProcess.js:96:27)
	at Generator.next (<anonymous>)
	at fulfilled (C:\Users\dthul\.vscode\extensions\ms-python.python-2018.5.0\out\client\common\process\pythonProcess.js:12:58)
	at <anonymous>

We still have the same timing issues as we have with debugging under conda + PowerShell, but the quoting of the environment works.


[WIP] Add Azure Blob Storage data/task import options to Labeler

f513fb226204ec2d98d4a8f6ed29ab5683738a98 adds the data & task buttons on the Labeling UI. ✅
f513fb226204ec2d98d4a8f6ed29ab5683738a98 reimplements their local import counterparts. I haven't run an end-to-end test yet. ❌

TODO:

Test with a local dev server
Test a containerized deployment with Azure

What works:
Browsing container files works with a browser pop-up equivalent of the local file explorer--this means we don't have to explicitly mention the file names to load them. However, we can't select a file instead just copy-paste its path. Ideally, we would want that.
To fix:
Loading the json labels is failing likely due to some validation errors (seems low effort), working on it now.