Check the existence of 'forceencap' parameter before use

Check the existence of 'forceencap' parameter before use.
Error seen:
Traceback (most recent call last):
  File "/opt/cloud/bin/update_config.py", line 140, in <module>
    process_file()
  File "/opt/cloud/bin/update_config.py", line 54, in process_file
    finish_config()
  File "/opt/cloud/bin/update_config.py", line 44, in finish_config
    returncode = configure.main(sys.argv)
  File "/opt/cloud/bin/configure.py", line 1003, in main
    vpns.process()
  File "/opt/cloud/bin/configure.py", line 488, in process
    self.configure_ipsec(self.dbag[vpn])
  File "/opt/cloud/bin/configure.py", line 544, in configure_ipsec
    file.addeq(" forceencaps=%s" % CsHelper.bool_to_yn(obj['encap']))
KeyError: 'encap'

Since forceencaps is no by default, I think this is a fair change. LGTM.

ACS CI BVT Run
Sumarry:
Build Number 92
Hypervisor xenserver
NetworkType Advanced
Passed=103
Failed=15
Skipped=4
The follwing tests have known issues
integration.smoke.test_iso.TestISO.test_07_list_default_iso
integration.smoke.test_privategw_acl.py
integration.smoke.test_vm_snapshots.TestSnapshots.test_01_test_vm_volume_snapshot
integration.smoke.test_vpc_vpn.TestVpcSite2SiteVpn.test_vpc_site2site_vpn
integration.smoke.test_iso.TestISO.test_04_extract_Iso
Link to logs Folder (search by build_no): https://www.dropbox.com/sh/yj3wnzbceo9uef2/AAB6u-Iap-xztdm6jHX9SjPja?dl=0
Failed tests:

integration.smoke.test_vpc_vpn.TestVpcRemoteAccessVpn

test_vpc_remote_access_vpn

test_vpc_site2site_vpn

integration.smoke.test_vm_snapshots.TestSnapshots

test_01_test_vm_volume_snapshot

integration.smoke.test_privategw_acl.TestPrivateGwACL

test_01_vpc_privategw_acl

test_02_vpc_privategw_static_routes

test_03_rvpc_privategw_static_routes

integration.smoke.test_guest_vlan_range.TestDedicateGuestVlanRange

test_dedicateGuestVlanRange

<nose.suite

ContextSuite context=TestScaleVm>:setup

<nose.suite

ContextSuite context=TestNiciraContoller>:setup

integration.smoke.test_primary_storage.TestPrimaryStorageServices

test_01_primary_storage_iscsi

integration.smoke.test_internal_lb.TestInternalLb

test02_internallb_haproxy_stats_on_all_interfaces

test_01_internallb_roundrobin_1VPC_3VM_HTTP_port80

integration.smoke.test_templates.TestCreateTemplate

test_04_extract_template

integration.smoke.test_iso.TestCreateIso

test_04_extract_Iso

test_07_list_default_iso

Skipped tests:
test_vm_nic_adapter_vmxnet3
test_deploy_vgpu_enabled_vm
test_06_copy_template
test_06_copy_iso
Passed test suits:
integration.smoke.test_deploy_vm_with_userdata.TestDeployVmWithUserData
integration.smoke.test_affinity_groups_projects.TestDeployVmWithAffinityGroup
integration.smoke.test_portable_publicip.TestPortablePublicIPAcquire
integration.smoke.test_over_provisioning.TestUpdateOverProvision
integration.smoke.test_global_settings.TestUpdateConfigWithScope
integration.smoke.test_service_offerings.TestCreateServiceOffering
integration.smoke.test_loadbalance.TestLoadBalance
integration.smoke.test_routers.TestRouterServices
integration.smoke.test_reset_vm_on_reboot.TestResetVmOnReboot
integration.smoke.test_snapshots.TestSnapshotRootDisk
integration.smoke.test_deploy_vms_with_varied_deploymentplanners.TestDeployVmWithVariedPlanners
integration.smoke.test_network.TestDeleteAccount
integration.smoke.test_non_contigiousvlan.TestUpdatePhysicalNetwork
integration.smoke.test_deploy_vm_iso.TestDeployVMFromISO
integration.smoke.test_public_ip_range.TestDedicatePublicIPRange
integration.smoke.test_multipleips_per_nic.TestDeployVM
integration.smoke.test_regions.TestRegions
integration.smoke.test_affinity_groups.TestDeployVmWithAffinityGroup
integration.smoke.test_network_acl.TestNetworkACL
integration.smoke.test_pvlan.TestPVLAN
integration.smoke.test_volumes.TestCreateVolume
integration.smoke.test_ssvm.TestSSVMs
integration.smoke.test_nic.TestNic
integration.smoke.test_deploy_vm_root_resize.TestDeployVM
integration.smoke.test_resource_detail.TestResourceDetail
integration.smoke.test_secondary_storage.TestSecStorageServices
integration.smoke.test_vm_life_cycle.TestDeployVM
integration.smoke.test_disk_offerings.TestCreateDiskOffering

@remibergsma can't you asking false instead of "false" ? If the function accepts boolean values I think that it would be more efficient.
Besides that, the addition is quite straightforward and I would be happy to hand out my LGTM

@remibergsma can you review the functionality you expect.  I think the logic is wrong, and if it is not, then it is VERY misleading.
The code in question that is being executed is this: CsHelper.bool_to_yn(obj['encap'])
For reference, this is the code being executed:
def bool_to_yn(val):
    if val:
        return "yes"
    return "no"

If I just read your code, I would assume you would want the final result to be forceencaps=no, but this code as written will result in forceencaps=yes.
Here is a simple test to prove the case...

I think @alexandrelimassantana is right.  In order to get the expected behavior, you need to use the boolean False.

@alexandrelimassantana @swill Thanks guys, fixed it.

Thanks.  Looks good now...


Clean up tutorials from testing

As per https://figmentio.atlassian.net/browse/LR-362

Also removed a duplicate tutorial:

tutorials/avalanche-deploy-pangolin-to-local-testnet was duplicate of tutorials/avalanche-deploy-pangolin-to-your-local-testnet-and-create-token-pair

@pyr0gan can you add these slugs to @builderdao/data/deprecated.ts as well.

need this test tutorial slugs in deprecated slug list.

✅ Got it!


ENH: Handle Finviz night mode

Is your feature request related to a problem?
Failed to get stock fundamental data due to DOM change when Finviz automaticaly chalange UI with night mode
Describe the solution you'd like
Externally no changes should be observed
API breaking implications
No API change is required
Describe alternatives you've considered
Force day mode of handle night mode
Additional context
Night mode representation:
MSFT | Microsoft Corporation | [NASD]
Technology | Software - Infrastructure | USA

There is only 2 table rows (code calls for 3rd one)
# Your code here, if applicable

#73 This will fix the issue. I will probably make a new release soon.

Should be fixed in #73. Release in v0.14.6rc1. Thanks for reporting.

Hello,
Is the bug still open or I'm missing something?
I tried to update to release v0.14.6rc1 (in the pycharm IDE) and got the following:
File "C:\Users~~~\PycharmProjects~~\venv\lib\site-packages\finvizfinance\quote.py", line 145, in ticker_fundament
(_, fundament_info["Company"], _) = rows[0].text.split(' | ')
ValueError: not enough values to unpack (expected 3, got 2)
Thank you lit26 for all your work

Yeah, there is still an issue. Fix soon.


How cool it would be to automate your Hackerrank submissions.

This is to automate your hackerrank submissions so that you just write the code in your local IDE and run the script and the submission takes place by itself in hackerrank.
Can I get this working?

Awesome would save a lot of time and data work on it :D

On it brother.👍🏻🎯


Fix two error handling problems in APIv2

The compat network remove endpoint uses network.Exists which returns 'true, nil' if the network doesn't exist.
The endpoint used to pass that 'nil' to utils.Error (api/handlers) which resulted in nil dereferencing.
It now calls utils.Error with the error object ErrNetworkNotFound.
The utils.Error now writes the apiMessage parameter to the response writer as the comment states it.
Signed-off-by: Maximilian Müller<EMAIL_ADDRESS>

@jwhonce PTAL at the second bit

/approve
/ok-to-test
This LGTM. Do you want to remove the WIP for merge?

Thanks @maxm123
/lgtm


Regarding Credentials & Certificates

Is there a way to integrate Vault credentials in Harvest.yml configuration file instead of using clear text username and password?
Regarding certificates as an alternative to username/password, is there a documentation on how to set it up for both cmode and 7mode?

@jmg011

Does https://github.com/NetApp/harvest/wiki/Harvest-and-Secrets-Management-with-SOPs help?
Certificates https://github.com/NetApp/harvest/blob/main/docs/AuthAndPermissions.md#using-certificate-authentication

hi @jmg011 there are many auth implementations especially across the managed environements that Harvest runs. Which one are you interested in?
7mode does not support client certificates so nothing Harvest can do there. The SOP link Rahul pasted above was created for a few customers using 7mode that didn't want their passwords in the clear.


Do not need to be authorised to delete a Pod

To delete a Pod, a Solid user:

Goto https://PodProvider/account/delete
Enter a WebID on the PodProvider
Receive a 'Delete Solid-account request' email at the associated email address.
Click the delete link in the email.

At no point in this process do I need to be authorised, so as long as I have access to the email account I can delete a Pod.
I would expect the Solid user would need to be authorised at some point during the process, maybe after the delete link is clicked.

This is not a feature of the data browser, but of NSS. I'll move the issue there and let @jaxoncreed decide whether or not to do any changes.


Portieris certificate error x509: certificate relies on legacy Common Name field

What commit ID of Portieris did you experience the problem with?
716c69cd189d37d71a28802a82b2d95898be7d06
What went wrong?
Once you install successfully portieris in a Kubernetes 1.19.0 over Red Hat 7.8, if you try to pull and image if fails with the following error:
Error from server (InternalError): error when creating "test.yaml": Internal error occurred: failed calling webhook "trust.hooks.securityenforcement.admission.cloud.ibm.com": Post "https://portieris.portieris.svc:443/admit?timeout=30s": x509: certificate relies on legacy Common Name field, use SANs or temporarily enable Common Name matching with GODEBUG=x509ignoreCN=0
What should have happened differently?
It must give a different error because I don´t allow that registry in the policy rules, not a cert error.
How can it be reproduced?
I installed using the following steps:

Clone the git repo.
Checkout to version 0.8.2
Generate certs using ./helm/portieris/gencerts
Remove namespace template file to avoid the "namespace is present problem"
Create a namespace with name portieris
Apply the helm chart with the following command:

helm install portieris --namespace portieris --set IBMContainerService=false helm/portieris

Once installed Portieris, I must change the MutatingWebhookConfiguration in failurePolicy to Ignore to allow to download the containers in the first installation.  Once the containers restarted, I changed to Fail.
I saw this recommendation in the following link: https://github.com/IBM/portieris/issues/97

Any other relevant information
Please if you need more info to solve the problem I can help.
Thanks in advance for all, I think this is a very good project!

Thanks for the nice comments, I think we are addressing the problem in 7. (as a by the way in currrent work).
However I think that the certificate error is new maybe in Kube 1.19, will try to recreate.

this is because kubernetes 1.19 is compiles with golang 1.15 and https://golang.org/doc/go1.15#commonname


Autocomplete : Options dropdown content styling fix

Description
Added styles to the options listing in the dropdown, such that when the content goes beyond the width, it gets truncated and ellipsis is added.
References #3059
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

@jewel-shajan , There was already an issue reg the chip's text , I have attached the link below.
https://github.com/trimble-oss/modus-web-components/issues/2698
I'll look into it.

@mitch-trimble
We have an option to set the chip's width, so can we decide and have a fixed width for the chip when using it in autocomplete?
Your thoughts?


[SNOW-1708573] Add sql counter check for test_large_query_breakdown

Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

SNOW-1708573

Fill out the following pre-review checklist:

[x] I am adding a new automated test(s) to verify correctness of my new code

[ ] If this test skips Local Testing mode, I'm requesting review from @snowflakedb/local-testing

[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency
[ ] If this is a new feature/behavior, I'm adding the Local Testing parity changes.

Please describe how your code solves the related issue.

Add sql_counter check for test points in large query breakdown
fix repeated test on the same plan for the query generator
fix a bug in handling alias to use the copy instead of the original map to avoid messing the original map

https://github.com/snowflakedb/snowpark-python/actions/runs/11221138775/job/31190939350


Releases/v.1.3.0 to master

Release branch v1.3.0 to master

I approve this, so I am fine with releasing this.

The ontology merge process fails, but I think I can fix that rather quick

I updated the files online. Please check whether it the right files are served on the OEP, whether the Ontology-URL loads in protege and webvowl

I don't manage to load them. But at least for webvowl its not an oeo problem since I don't get to the point of enting the URL...

It worked for me:
http://www.visualdataweb.de/webvowl/#iri=http://openenergy-platform.org/ontology/oeo/

Ok. Then I'll merge?

@stap-m Did it not load in protege either? Did you get any specific error messages in protege?

No, protege seemd to be hung up...

For me, both WebVOWL and Protégé work fine and I am usually the one who has the most Protégé problems.

@stap-m : Which Protégé version do you use? I recently had Protégé hangups which were solved by switching from version 5.5.0-beta 4 to version 5.5.0 (stable), see #620.

Ok, thanks. I'll try that.


Problems with Union

I read this and was trying to implement it, but I'm getting this error, am I doing something wrong?
(sorry if the question is stupid, I'm new to this)
Traceback (most recent call last):
  File "/home/samu369/Documenti/code/whitelist-bot/main.py", line 178, in <module>
    async def mnrg_add(interaction: discord.Interaction, snowflake: Union[discord.User, discord.Role]):
  File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 1914, in decorator
    command = Command(
  File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 666, in __init__
    self._params: Dict[str, CommandParameter] = _extract_parameters_from_callback(callback, callback.__globals__)
  File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 389, in _extract_parameters_from_callback
    _populate_descriptions(result, descriptions)
  File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 277, in _populate_descriptions
    raise TypeError(f'unknown parameter given: {first}')
TypeError: unknown parameter given: user

I did not encounter any such issues here is the example code I used
@bot.tree.command()
async def mnrg_add(interaction: discord.Interaction, snowflake: typing.Union[discord.User, discord.Role]):
    await interaction.response.send_message(f"Added {snowflake} to the list of managers.")

Can you include a minimal reproducible example in your code that is causing this issue?

from typing import Union
@mnrg_group.command(name="add", description="Add a whitelist manager")
@app_commands.describe(user="Who to add to whitelist managers? (User or Role)")
async def mnrg_add(interaction: discord.Interaction, snowflake: Union[discord.User, discord.Role]):
    sid = interaction.guild_id
    snw = snowflake.id
    if not interaction.user.guild_permissions.administrator: return await interaction.response.send_message("You do not have administrator permission!", ephemeral=True)
    if wl_is_mnrg(sid, snw): return await interaction.response.send_message("This user/role is already a whitelist manager!", ephemeral=True)
    wl_add_mnrg(sid, snw)
    return await interaction.response.send_message("User/role added to whitelist managers!", ephemeral=True)

Traceback (most recent call last):
 File "/home/samu369/Documenti/code/whitelist-bot/main.py", line 178, in <module>
   async def mnrg_add(interaction: discord.Interaction, snowflake: Union[discord.User, discord.Role]):
 File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 1914, in decorator
   command = Command(
 File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 666, in __init__
   self._params: Dict[str, CommandParameter] = _extract_parameters_from_callback(callback, callback.__globals__)
 File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 389, in _extract_parameters_from_callback
   _populate_descriptions(result, descriptions)
 File "/home/samu369/Documenti/code/whitelist-bot/.venv/lib/python3.10/site-packages/discord/app_commands/commands.py", line 277, in _populate_descriptions
   raise TypeError(f'unknown parameter given: {first}')
TypeError: unknown parameter given: user

I found the problem, I had gotten a letter wrong in the decorator two lines earlier and for some reason it said the error was in that line. Sorry for disturbing


Parking APP: Err 500 Unable to resolve "./screens/Map" from "App.js

Parking APP
Hi,
Many thanks for this project, I have the following error:
"Unable to resolve "./screens/Map" from "App.js"
Failed building JavaScript bundle."
Please see below the screenshot on emulator. Can you please help?

:D Thanks!

I'm getting this error in Expo snack

Hi,
Did you try installing it manually ?
expo add react-native-maps

Hope this helps.


Opening launchSettings.json results in a bunch of info messages in the Error List

Opening launchSettings.json results in a bunch of info messages in the Error List:
Severity	Code	Description	Project	File	Line	Suppression State
Message		Validation against a prohibited schema succeeded	ProjectSystemSetup	E:\project-system2\src\ProjectSystemSetup\Properties\launchSettings.json	3
Message		Value must conform to at least one of the associated schemas	ProjectSystemSetup	E:\project-system2\src\ProjectSystemSetup\Properties\launchSettings.json	3

Bill, what's the expectation here?

This no longer repros.


HTTP 500 Code

With the Flutter SDK sometimes some user a get a io error with a HTTP 500 Repsonse but their have a internation Connection. The Method: qonversion_internal.dart in QonversionInternal.remoteConfig
and get the following Error:
PlatformException: PlatformException(NetworkConnectionFailed, There was a network issue. Please make sure that the Internet connection is available on the device, Unable to resolve host "api.qonversion.io": No address associated with hostname, null)
or PlatformException: PlatformException(-1009, Es besteht anscheinend keine Verbindung zum Internet., Qonversion Error Code: -1009, null)
I think the users have an Internet connection, otherwise they couldn't send the Sentry Crash Report.
I am using the newest Qonversion Flutter SDK
class RemoteConfigNotifer extends StateNotifier<QRemoteConfig?> {
  RemoteConfigNotifer() : super(null);

  void loadRemoteConfig() async {
    final remoteConfig = await Qonversion.getSharedInstance().remoteConfig();

    state = remoteConfig;
    gRemoteConfig = remoteConfig;
  }
}

final remoteConfigProvider = StateNotifierProvider<RemoteConfigNotifer, QRemoteConfig?>(
  (ref) => RemoteConfigNotifer(),
);

Hi, @FantaMagier
NetworkConnectionFailed error is our wrapper above Android OS and Google Billing errors that indicate problems with the internet connection. I mean, we have no additional logic there; we just proxy the internet connection error from the system.


Await does not work with interface extending Promise

TypeScript Version:  2.1.4
Code
interface HelloPromise extends Promise<string> {}

function hello() : HelloPromise {
  return Promise.resolve('hello');
}

async function run() {
  await hello();
};

run();

Expected behavior: The code is compiled without errors.
Actual behavior: Following error is reported:
tsc -t es6 --strictNullChecks index.ts
index.ts(8,3): error TS1058: Operand for 'await' does not have a valid callable 'then' member.

You can find the repro code here:
https://github.com/jsedlacek/ts-promise-repro

Note: This code works in typescript 2.0.4.

Duplicate of https://github.com/Microsoft/TypeScript/issues/12292?

Fix should be in typescript@next later tonight. Please give it a try and let us know if you are still running into issues.

Thanks, I can confirm that typescript@next fixes this issue.


kotlin not workin in appcenter cli

Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Command 'codepush release-react -a rohit.garg.com/housingnative --entry-file index.js --sourcemap-output index.map -d rewrite -b index.android.bundle' failed with exception "Unable to find gradle file "android/app/build.gradle"." The file is basically android/app/build.gradle.kts but it is not picking that file.

Hi @rohitgarg2022 and thanks for reaching us. Did you try --gradle-file option for this command?

Yes i tried that option also 'codepush release-react -a rohit.garg.com/housingnative --entry-file index.js --sourcemap-output index.map -d rewrite -b index.android.bundle' -g android/app/build.gradle.kts.
But then it gives me another error that value for property android.defaultconfig.versionName is not specified , while i have the value of versionName in the build.gradle.kts

Could you please attach the logs of these commands and a simple example of a demo application with such a file that reproduces this error. Also have you tried using the latest version of appcenter-cli?

Command 'codepush release-react -a rohit.garg1-gmail.com/native --entry-file index.js --sourcemap-output index.map -d rewrite -b index.android.bundle -g android/app/build.gradle.kts' failed with exception "The "android/app/build.gradle.kts" file doesn't specify a value for the "android.defaultConfig.versionName" property
build.gradle.kts
android {
compileSdk = ProjectConfig.compileSdk
defaultConfig {
applicationId = ProjectConfig.appId
minSdk = ProjectConfig.minSdk
targetSdk = ProjectConfig.targetSdk
versionCode = ProjectConfig.versionCode
signingConfig = signingConfigs.getByName("debug")
versionName = ProjectConfig.versionName
testInstrumentationRunner = "androidx.test.runner.AndroidJUnitRunner"
vectorDrawables {
useSupportLibrary = true
}
}
object ProjectConfig {
const val appId = "com.dummy.application"
const val compileSdk = 33
const val minSdk = 21
const val targetSdk = 33
const val versionCode = 16
const val versionName = "2.5"
}
"appcenter-cli": "^2.13.7",

I am closing this issue due to inactivity. If you have any questions, you can freely reopen it.


[Feature Request]: Update secure by default parameter description

Description
Whenever we mention that a particular parameter value should be set/enabled "for security reasons" we should probably mention in the description which rules we are following or which security baseline we aim to comply with. E.g., NIST 800
This issue is about

proposing and agreeing on a proper sentence to cover the above
update the relevant parameter descriptions throughout the modules
regenerate module readme files

Team needs to decide what the actual quote should be saying and where it needs to sit.

See #2176

Will be discussed in a dedicated meeting

Team will wait for the PSRule implementation #2149


Fix build

Added missing entries forkernels/internal/ruy/*.cc

@mattn Could you please resolve the conflicts? Thanks!

@gbaned done


Ran clang-tidy with modernize-use-bool-literals

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/244/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/258/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1939/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/240/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/369/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1830/

clang-analyzer build failed! https://ci.trafficserver.apache.org/job/clang-analyzer-github/502/

clang format successful! https://ci.trafficserver.apache.org/job/clang-format-github/295/

RAT check successful! https://ci.trafficserver.apache.org/job/RAT-github/309/

FreeBSD11 build successful! https://ci.trafficserver.apache.org/job/freebsd-github/1990/

AU check failed! https://ci.trafficserver.apache.org/job/autest-github/291/

Intel CC build successful! https://ci.trafficserver.apache.org/job/icc-github/421/

Linux build successful! https://ci.trafficserver.apache.org/job/linux-github/1882/

clang-analyzer build failed! https://ci.trafficserver.apache.org/job/clang-analyzer-github/554/


Add various refactorings

Highlights:

Use IntoIter for array where applicable
Use raw strings instead of manually escaping quotes and backslashes
Declare Vecs with vec![] (square brackets)
Import macros the new way: use crate::macro;

This is great, thank you! Clearly I was vec!()-happy in my tests :)


boot2docker.iso with wireless drivers

Hi, I'm using docker-toolbox in a Windows environment and I'm trying to create a boot2docker iso with wireless drivers because I need to use it with containers which use usb wireless interfaces connected to them. I already know how to map wireless devices with virtualbox and extension pack using filters. Even inside the linux containers I can see the devices using lsusb. The problem is they are not appearing available using ifconfig or iwconfig and I guess is because boot2docker.iso have no drivers for that, so I decided to create a custom boot2docker iso with wireless drivers.
Ok, I saw in the Dockerfile is executing a RUN command to delete all wireless drivers for the standard iso:
# Remove useless kernel modules, based on unclejack/debian2docker
RUN cd $ROOTFS/lib/modules && \
    rm -rf ./*/kernel/sound/* && \
    rm -rf ./*/kernel/drivers/gpu/* && \
    rm -rf ./*/kernel/drivers/infiniband/* && \
    rm -rf ./*/kernel/drivers/isdn/* && \
    rm -rf ./*/kernel/drivers/media/* && \
    rm -rf ./*/kernel/drivers/staging/lustre/* && \
    rm -rf ./*/kernel/drivers/staging/comedi/* && \
    rm -rf ./*/kernel/fs/ocfs2/* && \
    rm -rf ./*/kernel/net/bluetooth/* && \
    rm -rf ./*/kernel/net/mac80211/* && \
    rm -rf ./*/kernel/net/wireless/*

I removed the 3 last lines to keep bluetooth, mac80211 and wireless drivers and I built the image generating a new iso file. After doing this, the first strange thing here is the generated iso have 1Mb less of size than the original....¿? it is suppossed that without removing that drivers the size could be larger... never shorter...
Anyway. I replaced the boot2docker.iso file from the %userprofile%\.docker\machine\machines\default folder and I recreated it (deleting the default vm and opening again the Docker Quickstart Terminal again and it recreates it automatically. But After mapping again the usb wifi device into containers, I can't see them as availables because I guess the drivers are not available at default vm. Opening a console into default vm I can see the /lib/../blablabla/../kernel/net/wireless/ doesn't exist. Only exist net but not wireless dir.
I even tested to create the wireless dir on that location and copy driver files ther... but after a reboot of the vm, the dir and files disappeared. Why is this happening? What should be done to keep wireless drivers on boot2docker machine? thank you.

The kernel wireless drivers are not installed by default in Tiny Core Linux. There is a package in the TC repo for wireless-4.2.9-tinycore64, but boot2docker is currently at kernel version 4.4.66.

Hi, thanks for your response. I'm pretty close to achieve it. I forked boot2docker project and I modified a lot of times kernel_config in a try and error loop until get a kernel fully supporting wireless features (as much drivers as possible and monitor features). Then, after adding to Dockerfile all tinycore wireless related packages, finally I can see the wireless interfaces inside the container!!! that's a good step I think!
Then, the goal is to be able to put card on monitor mode. At the beginning it was impossible but after some tweaking on kernel_config finally I did it. Ok, Now I have a card in monitor mode on container, now the problem is after launching an airodump no packets are shown... so there must be a problem with the driver.
In my opinion I think the problem now is the driver and not related to kernel. Probably because of what you are saying... it seems the driver packages available maybe are not matching the kernel version.
If you agree, I can share here my advances... showing the url of the forked repo with my custom Dockerfile and kernel_config files... to see if we can create a boot2docker iso working for wireless devices.

I did it. This is my forked boot2docker repository.
I pushed my changes on Dockerfile and kernel_config files. If anybody want to try something is welcome... as I said, at this point using virtualbox extension pack to connect usb devices into default boot2docker machine, we are able to see wireless interfaces inside container and to put them on monitor mode... but no packets captured. Something related to driver is wrong... I'll keep trying with different chipsets.

Can you close this issue, as this does not seem to be a use-case that boot2docker will support... and it seems you have solved it with a fork.

I can close it but it is not working at all... It's almost working... but was unsuccessful in the end.
Any help or suggestion?

Same problem here. I'd like to know if boot2docker does support already wireless capabilities with host USB interfaces?

Given the life-support-only status of boot2docker, I'm unfortunately going to close this -- we have no plans to support wireless drivers in boot2docker, and recommend folks use a more full-featured distribution such as Debian or Ubuntu if this is something necessary for your development use-case.


Notification disappears in iOS when app is in foreground

In my app first screen is the login screen. I call PushNotification.init() if the login is successful. Push notifications works fine after user logged in.
Lets assume user kills the app after login in and opens the app again and stays in the login screen. If a push sent at that time it is displayed in notification bar in Android because it is not handled. But in iOS notification is disappeared.
My question is, why notifications are not displayed in notification bar when the app is in foreground and plugin is not initialised? (Plugin is initialised and registered successfully before) In android notification is shown in notification bar(even when plugin is not initialised). But in iOS this is not the case.
Do we need to init the plugin every time app starts in iOS?

@malwatte okay, so you should always call the init() method each time your app starts. You'll need to re-register your event listeners, etc.
The behaviour you describe is the correct behaviour. When your app is in the foreground your notification event handler gets called and it is up to you to show any UI you want.

ok. thank you. I'll init the plugin in app start.


Refactor CI jobs

Our current CI is too complicated to understand.
Requirement - what kind of business use case are you trying to solve?
I am creating this issue to start discussion around CI jobs refactoring. From my perspective, I can see following problems in our current CI implementation:

We do most of the stuffs ourselves in shell scripts which build tools like goreleaser can do for us. With buildx coming into picture, it has become even more complicated.
We have one huge Makefile which keeps on growing.
Lack of documentation around build scripts.

Proposal - what do you suggest to solve the problem or improve the existing situation?
I would like to propose following solutions for the above problems:

Let goreleaser do the heavy lifting for all our binary build, docker build and docker push related tasks. @jpkrohling already tried it for v2 in https://github.com/jaegertracing/jaeger/pull/2886 and it looked much cleaner.
Divide root Makefile into multiple Makefiles.

Add each jaeger component specific directives to each cmd/<app>/Makefile and include them in the root Makefile.
With goreleaser into play, lot of directives should get removed.

Document the flow of build jobs.

Let me know your thoughts.

I don't have a strong opinion about goreleaser. What makes it better? From the description it seems a sizeable portion of it is dealing with pushing GH release & homebrew.
I am not sure what splitting the Makefile gives us. The amount of content per binary is actually minimal in it. I would rather deal with a single 600-lines Makefile (which isn't that large) than hunt around for the same information in multiple dirs.
I think the last round of refactoring for buildx actually consolidated a lot of repeated steps into smaller number of scripts, which is great. The build process is still, indeed, pretty complicated.

The trigger point for opening this issue is https://github.com/jaegertracing/jaeger/pull/3204 It clearly shows our CI is quite rigid. It works perfectly well on daily basis PRs but when someone need to change something in those shell scripts, it requires quite an effort and requires complete knowledge since we have too much coupling among those scripts.  Amount of conversation in https://github.com/jaegertracing/jaeger/pull/2611 and https://github.com/jaegertracing/jaeger/pull/2948 reflect that pretty well.
Regarding goreleaser, I can see a few benefit:

I feel all those shell scripts can be represented into yaml, yaml achors and aliases keeps it well maintained without need for repetition (Eg. https://github.com/jpkrohling/jaeger/blob/64333374cc658e3f9448869e592e0ddac7628341/v2/.goreleaser.yaml does almost all of the stuff we do)
We don't need to document the working of our CI jobs, goreleaser has great documentation. Someone who needs to change the CI job can just refer the documentation of goreleaser. With scripts, they need to find our which script is calling what and understand the flow.
Right now we write shell scripts, wrap that into Makefile directives and then call those Makefile directives from GH Action. Goreleaser has its own GH Action (https://github.com/goreleaser/goreleaser-action) This will reduce size of the Makefile and shell scripts drastically.

I feel all those shell scripts can be represented into yaml

To me, converting from Shell scripts to YAML would be a huge step back. Everyone knows how to read and write Shell, and how to debug it. YAML is always a bespoke DSL that people need to learn from scratch, and it is usually much harder to debug.

yaml achors and aliases keeps it well maintained without need for repetition

This is just as easily achieved by reusing shell scripts or shell functions. In the example you linked, I do not see that it is substantially more concise (but it is substantially more opaque, bespoke, and non-debuggable).
Is goreleaser going to create Dockerfiles for us? I assume not, so it's not removing that complexity. So which steps is goreleaser going to substitute? If we're only using it as a replacement for go build and docker build, then the complexity is not in the commands, but in the preparation of various parameters and their permutations, and I am not seeing how it would make it to have fewer parameters.
Maybe I am missing something.

It works perfectly well on daily basis PRs but when someone need to change something in those shell scripts, it requires quite an effort and requires complete knowledge since we have too much coupling among those scripts.

The biggest issue from my point of view is that I cannot run the full build locally. For instance to build docker images a local docker registry is needed. I tried running the registry locally and I got into configuration issues with TLS between registry and  docker push.
For productive development, I would like to be able to run all commands locally e.g. build binaries, docker images and run integration tests. I don't care too much if it requires more code in the makefile/scripts.

Our CI definitions look fairly simple, however if we can further simplify building of binaries and images I am all for it (e.g. via goreleaser). At the same time I would like to be able to build and run everything locally without decrypting CI configs.


Avoid server side retry and move logic to client

Description
REF https://github.com/lynckia/licode/pull/988
Move the retry logic of the addPublisher or addSubscriber clientside.
This avoid an inconsistent state on the server if during retry the client disconnects. (the server continues thinking forever that there's a connected client and this prevents the JS to shutdown and restart when empty)
I fixed the current unit tests.

I'm not a huge fan of changing this behaviour. It seems to make like it would me safer ifn we add a mechanism to stop retrying when the client disconnects. But I might be wrong.

I think it would be way more trivial, but in every case this should be fixed IMO.
when lots of clients hit the server is very easy to end up with a "dirty" structure in my experience


No module named maskrcnn_benchmark.config

❓ Questions and Help
Hi
I have followed the instructions to install this package. However, when I run the notebook demo under './demo', it gives an error:
ImportError: No module named maskrcnn_benchmark.config.

Use python 3.x.

@Kongsea
I've got the same ERROR message, but I'm using python 3.6.6
$ python --version
Python 3.6.6

Any other suggestions?

@jiapei100 I found the solution. You need to create an empty __init__.py under the folder "maskrcnn_benchmark".

Traceback (most recent call last):
File "webcam.py", line 6, in
from predictor import COCODemo
File "/data/Data/HeiRen/code/maskrcnn-benchmark/demo/predictor.py", line 7, in
from maskrcnn_benchmark.utils.checkpoint import DetectronCheckpointer
File "/data/Data/HeiRen/code/maskrcnn-benchmark/maskrcnn_benchmark/utils/checkpoint.py", line 7, in
from maskrcnn_benchmark.utils.model_serialization import load_state_dict
File "/data/Data/HeiRen/code/maskrcnn-benchmark/maskrcnn_benchmark/utils/model_serialization.py", line 7, in
from maskrcnn_benchmark.utils.imports import import_file
File "/data/Data/HeiRen/code/maskrcnn-benchmark/maskrcnn_benchmark/utils/imports.py", line 3, in
import importlib.util
ImportError: No module named util

I am adding support for Python 2 in https://github.com/facebookresearch/maskrcnn-benchmark/pull/11 and it should fix the errors you are seeing.
I'll verify that the training that I launched yesterday matches the excepted accuracies and I'll merge it

For info, I've just merged https://github.com/facebookresearch/maskrcnn-benchmark/pull/11 , so the codebase should work with Python 2 as well.

Let me know if you still face problems after pulling from latest version of maskrcnn-benchmark

yes, I've improved the documentation to add the source activate maskrcnn_benchmark after creating the environemnt

@fmassa
Nope... I didn't use conda at all....
@yaoliUoA
I've added init.py under the folder maskrcnn_benchmark, but the ERROR continues....

Make sure that the installation wmis done properly
Also, pull from latest master, I've fixed the missing __init__.py files
As a note, having conda is a very easy and simple way of managing several python environments, and I'd highly recommend it.

when i use 3.x python with conda, every thing is all right.

@moyans me too.

We now support python2 as well, so if it wasn't working before with Python 2, now it should be.

I had the same issue, trying this link helped: https://github.com/jupyter/notebook/issues/1524

@fmassa  I pulled recently from the latest master, but I still get No module found errors when I try using a python 2 conda environment, everything works fine if I switch back to my python 3.x environment


Verification not with realtime only with a picture

Hi,
Please I have a question, I´m develop a new application using your project as example but I dont need to use camera stream frame by frame, is possible to add in this project a new activity o similar that can take picture, send that picture to face verfication process and get result of verification. Or maybe there are any considerations that I have to know.

You can start by adding the FaceNet.kt class to your project. The getFaceEmbedding method requires a cropped face image, for which we can use Mediapipe's Face Detector or MLKit's FaceDetection API. Prior to face detection, we may have to perform image rotation considering the EXIF data stored in the selected image. See this gist for the code.


Distributed Docker compose with MariaDB nodes can't communicate each others

I just tested this version and I have some issues:
The worker, presenter and the and the admin can't see each other because the URLs are configured without a proper alias or the internal docker network.

I could not reproduce the issues. Docker compose will automatically create a Docker network with a DNS server that resolves the opencast-{admin|worker|presentation} domains to the correct IPs.
Using the next branch for the upcoming Opencast 7, I run the following commands:
$ cd path/to/repo
$ docker-compose -p opencast-multiserver -f docker-compose/docker-compose.multiserver.mariadb.yml up

Could you describe your setup in more detail?

I'm using the official docker for mac, the version is Docker version 18.09.2, build 6247962
after that, I cloned the repo and use the "next" branch with the same command as you suggested.
The first problem I had is the exposed port for MariaDB, that caused a conflict with my installed MariaDB, so I commented the line  - 3306:3306 of the MariaDB configuration.
Later I have the network problem I described here, but I just solved expressing the aliases for the default network like this for each container:
    networks:
      default:
        aliases:
          - activemq

@mtneug you have right, the docker compose creates a network, but for any reason didn't work for me until I explicitly configured the aliases.

Solved with the change of the DDL file.


Add GUIALLOWED oninsert trigger

🔻🔻🔻This repository no longer handles general questions and is only dedicated to bug reports and feature suggestions for AL compiler and developer tools 🔻🔻🔻
To get help please use one of the following resources:
Business Central Community
mibuso forum
Dynamics User Group

Please open a new issue that describes the problem in greater details than just a title.


Group updates across different ecosystems

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
If I have a project using several package-ecosystem configurations, I would like to use the new groups feature to create a Dependabot PR that includes updates across multiple ecosystems. For example, I may be using Dependabot for docker, terraform, and pip. Currently, I have three separate groups, one for each eco-system, and so I get three separate PRs. Instead, I would like to get a single PR.

Thanks for raising this; the crew is starting to explore this

This would be very useful. I have projects with multiple package ecosystem definitions per Maven POM and can't group them right now since https://github.com/dependabot/dependabot-core/issues/2178 isn't implemented.

This would be brilliant for us

We have a private repo that publishes both a ruby gem and npm package, and would greatly benefit from grouping these together in dependabot PRs for other consuming repos.


Do not hard-define empty env var

Hello,
This PR solves issue #59371 :
https://bz.apache.org/bugzilla/show_bug.cgi?id=59371
Thank you 👍
Ben

I Would also be very glad to see this backported to 2.4.x.
Many thx 👍

Hi,
Could we think about merging this ?
Thank you very much 👍
Ben

I am not sure this is safe to backport, it might be reasonable for people to be using empty native environment variables, e.g. when things are deactivated.  It seems better to address the overlap between the different kinds of interpolation

Mmmmh you think that some may intentionally use MYVAR= "instead" of just running unset MYVAR.
Regarding the overlap issue you are talking about, to be honest, I'm not sure I'll be able to correct it :|


Two rotating wheels

Hello!
In your example of advanced constructor, you show the code, where you pass wheel1.raphael as second parameter to wheel2 object.
However, I couldn't figure out, how you did achieved total synced rotation of both wheels.
My idea is to have second wheel purely for cosmetic effect, when user selects an item from top wheel.
I tried this:
wheel1.navItems[0].navigateFunction = function() {
  wheel2.navigateWheel(0);
};

This "kinda works", but only one way. I was hoping, that whole point of using wheel1.raphael was to transfer rotation of one wheel to another.
I looked up into the source code of the page http://wheelnavjs.softwaretailoring.net/documentation/core.html but I was unable to dig out the code for it.
Any advice appreciated!

Hi @davesade
Sorry for late response, summertime. :)
The second wheel is not for cosmetic purpose.
You can use it:
wheel2.navItems[0].navigateFunction = function() {
  wheel1.navigateWheel(0);
};


template documentation?

I'm looking at building a template for JSDoc's output, but can't find any documentation that describes how templates work. Existing prior art like react-jsdoc look like they preprocess javascript to make it work with JSDoc.

Looks like some of the documentation is here and some of the other documentation is here. it would be great to have one of the related links link to github.

It's true, we have almost no documentation that tells you how to create a template. I filed jsdoc3/jsdoc3.github.com#97 against the docs, so I'm closing this issue.
In practice, most people start by forking the default template. (The jsdoc-react template is an exception.) However, the default template is somewhat brittle, lacks tests, and is difficult to modify.
If you'd like to start with something more reliable, I would encourage you to take a look at Baseline, which will eventually become the new default template for JSDoc. I'm working on changes that will make the template fairly easy to customize without forking the whole thing. That feature should be in place within a month or so.


Fancy Zone restore to default after windows10 restart.

Microsoft PowerToys version
0.45
Running as admin

[X] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
Fancy Zone restore to default after windows10 restart.
1、edit zones
2、restart win10
3、default zones show
✔ Expected Behavior
Fancy Zone restore to default after windows10 restart.
1、edit zones
2、restart win10
3、default zones show
❌ Actual Behavior
Fancy Zone restore to default after windows10 restart.
1、edit zones
2、restart win10
3、default zones show
Other Software
No response

BTW, this applies to Windows 11 as well.

Tracked (and fixed) with https://github.com/microsoft/PowerToys/issues/13069.  We'll have a new version out soon (like, maybe tomorrow) that has the fix.


EVCC Addon unstable

Hello,
I'm having an issue with my EVCC Addon.
It worked quite a while, but since some weeks it's randomly producing an error with the sponsortoken I need for it.
The Addon stays in a sort of loop starting up giving the error:
FATAL 2023/12/24 13:04:48 sponsortoken: rpc error: code = Unknown desc = token contains an invalid number of segments
I wrote to andig already, but he says, the token is correct, it may be a problem with HA.
When installed on my comptuer directly (outside HA) it works, but I want to use the addon because of some parameters I can't reach in the standalone installation.
As I do sponsor the addon for using it, I would like to have a working system. It's not feasible to get new sponsortoken every week, as a new token also doesn't always solve the problem.
Also re-installation of the addon doesn't help.
Do you have an idea, what's going wrong? Can I help with some logs?
Thanks for helping
Dietmar Grasl
49686a9f_evcc_2023-12-28T16-44-58.884Z.log
supervisor_2023-12-28T16-45-20.105Z.log

Hi Dietmar,
just to remove Home Assistant from this equation, I would ask you to please run evcc once locally on your computer running Windows, macOS or Linux with the same config.
Afterwards we know if this is caused by HA or Home Assistant OS or if it is a pure evcc issue.

Hi,
once again I created an installation on my debian-system, parallel to
homeassistant and it works.
But I also want to control and see it from outside home, which is
complicated.
As there exists an addon in HA I would prefer to use this, but again it was
unstable, providing always the same error message.
I would not like to switch to a different application, because again it
would be outside of HA.
Please help with the addon.
Dietmar
Am Do., 28. Dez. 2023 um 22:49 Uhr schrieb Dietmar Grasl <
@.***>:

Hello,
did that already, worked.
Kind regards
Am 28. Dezember 2023 22:14:37 schrieb Timo Reimann <
@.***>:

Hi Dietmar,
just to remove Home Assistant from this equation, I would ask you to
please run evcc once locally on your computer running Windows, macOS or
Linux with the same config.
Afterwards we know if this is caused by HA or Home Assistant OS or if it
is a pure evcc issue.
—
Reply to this email directly, view it on GitHub
https://github.com/evcc-io/hassio-addon/issues/82#issuecomment-1871494046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EBG2MCV773RQJXGLCKUB3YLXOL3AVCNFSM6AAAAABBFWKZISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZRGQ4TIMBUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Dietmar Grasl
Florianerstr. 20
A - 4483 Hargelsberg
@.***

Hi,
here is my adapted evcc.yaml and HA log
Thanks for your help
Dietmar
Am Di., 2. Jan. 2024 um 20:08 Uhr schrieb thecem @.***>:

Hi Ditmar,
pls post your config, x out your credentials. The error log aimed for
syntax error…
—
Reply to this email directly, view it on GitHub
https://github.com/evcc-io/hassio-addon/issues/82#issuecomment-1874432083,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A6EBG2PXXGZ7GTIUQT5F4LLYMRLKXAVCNFSM6AAAAABBFWKZISVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZUGQZTEMBYGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Dietmar Grasl
Florianerstr. 20
A - 4483 Hargelsberg
@.***

Where ?

Seems like this is not related to the add-on but maybe to the main project https://github.com/evcc-io/evcc or it is fixed right now.


Unsubscribe all users from :tenantNameHidden:

Context

Tenant A and Tenant B would like to unsubscribe their users from grant digest emails

Current State

All of Tenant A's users are subscribed to the digest email

Expected State

All of Tenant A's users are unsubscribed to the digest email except for :emailsHidden: (sent via Slack)

I ran the script directly in our production shell. need to commit the script to prod so we can do this again if needed.


ViewEncapsulation.Native fails to correctly display the sidebar

IMO this is a bug, although there is a separate sequest at #3386
See https://github.com/williamBurgeson/ng-mat-ve-issue for an example. The component (which is routed to but automatically displayed anyway) is set to ViewEncapsulation.Native and the sidebar fails to display. For the actual project I am working on I hacked / fixed this by adding a transform: translate3d(0px, 0px, 0px) !important; rule on .mat-drawer, however really this should work correctly whatever the ViewEncapsulation mode

See #3386, we don't currently support native encapsulation.

When are you planning to drop IE11 and support the Shadow DOM?
Please don't tell us it won't be until 14th October 2025 (the day after my 50th birthday)

@williamBurgeson https://github.com/angular/material2/issues/7374 is tracking that


Add standard code package

Resolves: #60
This PR adds the standard code package, which is meant for highlighting and displaying code snippets.
TODO:

[x] Revise indent functionality
[x] Allow specifying language either by extension or by full name
[x] Handle write results (might be most convenient to just unwrap)
[ ] Look into optimizing initialization of syntax sets (might not be necessary after playground updates)
[x] Look into renaming and reordering arguments to be more convenient
[x] Clean up

Beware of merge commits, I think our branch protection rules doesn't allow merge commits in main and I think I happened to get one once by mistake, and it was hard to get rid of. Just a tip, avoid them if possible. But nice work with the code module! Looking forward to seeing it when it is done!

I think Eli solved my merge commit issues by somehow just rebasing from main and that just flattened everything iirc, but just a heads up!

Thanks for the heads up! I think I managed to solve it now, and I even renamed commits to reflect modules being renamed to packages!


[Bugfix] fix external catalog support Hive MATERIALIZED_VIEW table

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #9386
Problem Summary(Required) ：

 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.jhonye seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@feihengye thanks for your contribution, can you sign our CLA?

@feihengye thanks for your contribution, can you sign our CLA?

ok

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)


[Feature] Group chat between two users and bot

Hello,
Do you know when group chat would be enabled?   Would like our bot to orchestrate the conversation between our customer and our agents.
Thanks

Did anyone ever get a solution for this?

To Do items related to this request:

Create a sample that has multi-user conversation

if there are no problems, demo to team to discuss hidden pitfalls
if not, unfortunately we don't support it. Perhaps in the future

This would be absolutely amazing! 👏👍
Basically, I need exactly this functionality.
For now, I think I have to dig into customizing the control.

+1

@attozk it wasn't that hard to implement after all:
Share the DL token between users and bot.
Use some trickery about username and initials (the initials portion could come from a bot middleware, but i implemented it by fetching the initials via http
Implement a middleware and do some basic css hacking, voliâ
ChatWindow.tsx:
import 'react-app-polyfill/ie11';
import * as React from 'react';
import * as ReactDOM from "react-dom";
import * as classNames from "classnames";
import { DirectLine, Activity } from 'botframework-directlinejs';
import ReactWebChat from 'botframework-webchat/lib';
import CroppedImage from "botframework-webchat-component/lib";
import { hooks } from "botframework-webchat-component/lib";
import createStore from 'botframework-webchat-core/lib/createStore';

interface AvatarProps{
    'aria-hidden': boolean | undefined,
    className: string | undefined,
    fromUser: string | undefined,
    initials: string | undefined,
    userName: string | undefined
}
const Avatar = ({ 'aria-hidden': ariaHidden, className, fromUser, initials, userName }: Partial<AvatarProps>) => {
    const [{ avatar: avatarStyleSet }] = hooks.useStyleSet();

    const image = undefined;

    if(image || initials)
    {
        return (

                {initials}
                {!!image && <CroppedImage alt="" className="image" height="100%" src={image} width="100%" />}

        );
    }
    return (

)
};

const renderAvatar = (element: any, userInitials: string) => {
    ReactDOM.render(
        Avatar({
            initials: userInitials
        }),
        element
    );
}

interface ChatUser
{
    id: string;
    initials: string | undefined;
    name: string;
}

const renderChatWindow = (element: Element, channelId: string, token:string, userID: string, userName:string, users: ChatUser[], activitiesString: string | undefined) => {

    const activityMiddleware = () => (next: any) => (card: any) => (children: any) => {
        let from = "";
        let isUser = false;
        let initials = "";

        const activity: Activity = card.activity;

        if (activity.from && activity.from.role && activity.from.name) {
            from = activity.from.name;

            const user = users.find(u => u.name === from);
            if(user && user.initials){
                initials = user.initials;
            } else if(user) {
                const name = user.name.replace(/[^a-zA-Z]/gi, '');
                if(name.length >= 2){
                    initials = name.slice(0,2).toLocaleUpperCase();
                }
            }else {
                initials = "BT";
            }

            if (activity.from.name !== userName) {
                activity.from.role = 'bot';
            } else {
                isUser = true;
            }
        }

        return (

                    <Avatar initials={initials} userName={from} />

{next(card)(children)}

        )
    };

    ReactDOM.render(
        <ReactWebChat
            store={createStore({})}
            activityMiddleware={activityMiddleware}
            className="chat"
            username={userName}
            userID={userID}
            directLine={new DirectLine({
                token
            })}
            />,
        element
    );
}

export { renderChatWindow, renderAvatar };

Closing, as this will not be supported by DLJS in the near future.


Make mysql binary directory configurable

If you are running the mysql plugin right now you are forced to put the plugin in /var/vcap/packages/shield-mysql/bin/
The postgres plugin allows you to specify a binary directory, so I added that feature to the mysql plugin.
If left undefined it defaults to /var/vcap/packages/shield-mysql/bin/ this change will not break existing implementations.

The code looks good, but I would like to suggest that you make a variable for these hard coded values.

I think that is a good idea norm, maybe we can create another issue for that? the postgres and mysql plugins both do it this way.
I think this PR is good to merge?

Agree


Update jackson-datatype-jsr310 to 2.11.3

Updates com.fasterxml.jackson.datatype:jackson-datatype-jsr310 from 2.11.2 to 2.11.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.fasterxml.jackson.datatype", artifactId = "jackson-datatype-jsr310" } ]

labels: library-update, semver-patch

Obsolete. Replaced by https://github.com/twilio/guardrail/pull/801


replace linkedWorkspaces with linkedService in 2020-09-01-preview

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

LinkedService feature will be in private preview stage2 in middle Sept. We need to finalize this swagger file ASAP so that we can generate our SDKv2 based on this swagger, and make our SDK ready to use before mid Sept. Your timely review and approval is highly appreciated!

/azp run automation - sdk

the first commit simply create copy 2020-05-01-preview to 2020-09-01-preview. Your can review the diff between latest commit and the 1st one to know what are changed.

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk

/azp run automation - sdk


Lasb-626 update V2 GET Prosecution Case Swagger doc

What
https://dsdmoj.atlassian.net/secure/RapidBoard.jspa?rapidView=454&projectKey=LASB&modal=detail&selectedIssue=LASB-623
Describe what you did and why.
Update the API documentation so users know what they will receive when querying CDA endpoint
Checklist
Before you ask people to review this PR:

[ ] Tests and linters should be passing
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] Avoid mixing whitespace changes with code changes in the same commit. These make diffs harder to read and conflicts more likely.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

Looking good @chris-groves. Could you look adding examples where they're missing? There are quite a few.
e.g. filter[arrest_summons_number] for the prosecution cases endpoint.


Disable default host and return 444 for HTTP(S)

How to disable the default host and return the code 444 (non-standard code that closes the connection). Currently, if DEFAULT_HOST is not provided, a random vhost is delivered.
I have added these lines in a config file:
server {
    listen   80 default_server;
#    listen   443 default_server;
    return   444;
}

It works for non-HTTPS. Now if I uncomment the line for 443 (HTTPS), I get:
no "ssl_certificate" is defined in server listening on SSL port while SSL handshaking, client: X.X.X.X, server: <IP_ADDRESS>:443

If no default_server is defined then the default server is the first one, which is nginx’s standard default behaviour.
When a docker container with VIRTUAL_HOST is started or stopped. It generate a new config for nginx-proxy. If SSL support is configured and the crt & key files exists, a server block with port 443 will be added to the configuration file. The config file has this following statement at the top, and since this is the first server statement it will become the default_server if not default_server is defined.
server {
	server_name _; # This is just an invalid value which will never trigger on a real hostname.
	listen 80;
	access_log /var/log/nginx/access.log vhost;
	return 503;
}

This matches every host not defined in the config file but only for port 80 (HTTP). Accessing a VHOST on port 80 when the container is down results in a 503 error. But if the same VHOST is accessed on 443 (HTTPS) nginx has not been given any specific rules how to handle this.
 The problem is that a "fallback" host should catch all non matching hosts. It is not obvious from a configuration standpoint which SSL certificate to use in this situation. But nginx-proxy can handle this scenario as well. At the very end of the config template it checks for the existence of a "default.crt" and "default.key" in the certs directory.
{{ if (and (exists "/etc/nginx/certs/default.crt") (exists "/etc/nginx/certs/default.key")) }}
server {
	server_name {{ $host }};
	listen 443 ssl http2 {{ $default_server }};
	access_log /var/log/nginx/access.log vhost;
	return 503;

	ssl_certificate /etc/nginx/certs/default.crt;
	ssl_certificate_key /etc/nginx/certs/default.key;
}
{{ end }}

This certificate needs to be a SNI cert with all of the domains handled by this nginx-proxy instance.
This section will not forward any requests to a specific container. But it will report 503 error instead of replying with the content from another VIRTUAL_HOST when accessed over HTTPS.
If you want to force a single container to act as the default host. Then use DEFAULT_HOST=fallback.foo.bar.com. This single container could run any form of webserver you want. But for example it could run nginx plain docker container. With a configuration to return 444 error code on anything forced upon it.
Or you set up nginx-proxy in "Separate Containers" mode. Since this requires you to have the nginx.tmpl file on the host. So you are free to modify the server_name "_" section and change the 503 error code in it to 444 or whatever you want.

Thanks.
Your solution supposes to have a wildcard certificate, I don't. So, when someone try to access to random.notfound.foo.example.com, all browsers display a security warning because nginx provides the certificate of the first server who is listening 443.
Is there any option to return 444 (close connection) before sending a wrong certificate?

I don't think that this is possible. When doing HTTPS the first thing is the SSL handshake. For this a valid or trusted certificate is required. This is sort of the whole point of HTTPS. To ensure the server is who is claims to be - before receiving further data.
$ nc google.com 80
GET /asdasd HTTP/1.0

HTTP/1.0 404 Not Found
Date: Thu, 18 Feb 2016 16:01:22 GMT
Content-Type: text/html
Server: HTTP server (unknown)
Content-Length: 49
X-XSS-Protection: 1; mode=block
X-Frame-Options: SAMEORIGIN

404 Not Found

Any HTTP error is part of the HTTP protocol and should from the browsers perspective be ignored before ensuring the server is who it claims to be. Or to accept the security warning.
I know services which relies on semi random subdomains to handle communication. But this relies on wildcard certificates. This is the only solution for this. Or SNI with a LOT of domains.

ec92c6fa499046ed9e537723ad8791a8.my-service.example.com

You can achieve what you want. But not with HTTPS only with HTTP.

I have finally created a self-signed wildcard certificate.


[BUG] Fanyi Baidu –

User report
Description:

Debugger data (do not alter)
service_version: "1.0.0"
obsidian_version: "0.15.8"
mobile: false

Hello! What issue did you encounter with Fanyi Baidu? Did you get a message like: Validation failed: Unauthorized user?
If you'd like, you can add me over on Discord at @Fevol#9470, so I can help fix this issue more quickly!

@aishanjiang I've marked this issue as stale for now, but I'll keep it open for now. I'm not sure what the problem was that you've encountered, but I have rewritten parts of the Baidu API handler code, so it's possible that your issue is solved now.
If you're still encountering issues, please do let me know. I do not have access to the service myself, so I cannot verify whether it works properly.


feat: create a ExecutionContext struct that is adapted to the migration path

Feature Request
Describe the Feature Request
We need to describe the new ExecutionContext in kakarot-ssj
Kakarot Cairo 0:
  struct ExecutionContext {
        call_context: CallContext*,
        program_counter: felt,
        stopped: felt,
        return_data: felt*,
        return_data_len: felt,
        stack: Stack*,
        memory: Memory*,
        gas_used: felt,
        gas_limit: felt,
        gas_price: felt,
        starknet_contract_address: felt,
        evm_contract_address: felt,
        calling_context: ExecutionContext*,
        sub_context: ExecutionContext*,
        destroy_contracts_len: felt,
        destroy_contracts: felt*,
        events_len: felt,
        events: Event*,
        create_addresses_len: felt,
        create_addresses: felt*,
        revert_contract_state: RevertContractState*,
        reverted: felt,
        read_only: felt,
    }

Describe Preferred Solution
We could split the ExecutionContext well in themes:
dynamic, static data
Related Code
Additional Context
If the feature request is approved, would you be willing to submit a PR?
(Help can be provided if you need assistance submitting a PR)

[ ] Yes
[ ] No

is this an issue for "creating the execution context" in cairo1 or to design a new execution context possibly optimized?

optimized where where separate aspects of the context that are updated versus those that are static. in evm terms, this would be dimensions that are environment data (set once upon initialization of execution context and never changed) and dynamic fields that are changed in the flow of execution.

I am currently exploring how it would be like to have, as you said , a "dynamic" part and a "static" part in the ExecutionContext struct. And use pointers to structs instead of structs, where it makes sense.
Basically something like

#[derive(Copy, Drop)]
struct StaticExecutionContext {
    call_context: @CallContext,
    gas_limit: u64,
    gas_price: u64,
    starknet_contract_address: ContractAddress,
}

#[derive(Destruct)]
struct ExecutionContextSplit {
    static_context: Box<StaticExecutionContext>,
    gas_used: u64,
    program_counter: u32,
    stopped: bool,
    return_data: Array<felt252>,
    stack: Stack
}

and
/// Stores all data relevant to the current execution context.
#[derive(Destruct)]
struct ExecutionContextAllInOne {
    /// The call context data.
    call_context: @CallContext,
    /// Keeps track of the current position in the program as it is being executed.
    program_counter: u32,
    /// States if the current execution is halted.
    stopped: bool,
    /// The region used to return a value after a call.
    return_data: Array<felt252>,
    /// The gas used.
    gas_used: u64,
    /// The gas limit.
    gas_limit: u64,
    /// The gas price.
    gas_price: u64,
    /// The starknet address of the contract interacted with.
    starknet_contract_address: ContractAddress,
    /// The stack.
    stack: Stack,
}

From what I've seen, just with this small experiment, the "split" implementation costs way less gas on a simple operation like updating the gas_used field. (~15%) Probably because under the hood, updating the struct which has a single pointer to the static ctx instead of multiple fields costs way less

Looking into it, having the execution context split, feels like will bring perfomance and as the split is not so big, the code readability is still fine?
Another idea would be split even memory, stack and the u32/ u64 values(this could be all stored on one Dict that gets updated every step)?

From the tests performances, here is my proposal:
Current
/// The execution context.
/// Stores all data relevant to the current execution context.
#[derive(Destruct)]
struct ExecutionContext {
    call_context: CallContext,
    program_counter: u32,
    stack: Stack,
    stopped: bool,
    return_data: Array<u8>,
    memory: Memory,
    gas_used: u64,
    gas_limit: u64,
    gas_price: u64,
    starknet_address: ContractAddress,
    evm_address: EthAddress,
    // TODO: refactor using smart pointers
    // once compiler supports it
    //calling_context: Nullable<ExecutionContext>,
    //sub_context: Nullable<ExecutionContext>,
    destroy_contracts: Array<EthAddress>,
    events: Array<Event>,
    create_addresses: Array<EthAddress>,
    revert_contract_state: Felt252Dict<felt252>,
    reverted: bool,
    read_only: bool,
}```

## New
### Groups:

- StaticData
- DynamicData
- Stack
- Memory
- program counter(this changes a lot more than dynamic data)

```cairo
#[derive(Destruct)]
struct StaticExecutionContext {
    call_context: CallContext,
    starknet_address: ContractAddress,
    evm_address: EthAddress,
    origin: EthAddress,
    read_only: bool,
}

#[derive(Destruct)]
struct DynamicExecutionContext {
    stopped: bool,
    return_data: Array<u8>,
    destroy_contracts: Array<EthAddress>,
    events: Array<Event>,
    create_addresses: Array<EthAddress>,
    revert_contract_state: Felt252Dict<felt252>,
    reverted: bool,
}

stack = Stack::new()
memory = Memory::new()
program_counter: u32 = 0 (initial)

How about subcalls? Just a question, haven't thought about it yer

@danilowhk A few questions/suggestions:

I think we should group DynamicExecutionContext and StaticExecutionContext together in a ExecutionContextstruct that takes pointers to these structs
Don't forget that increasing the number of function parameters also has an impact on gas - if we now pass each opcode 5 parameters vs. 1 previously, it might not be as beneficial as we think
Is isolating stack / memory / program_counter really beneficial when it comes to gas savings, especially if they're packed in ExecutionContext along DynamicExecutionContext and StaticExecutionContext ?
Don't forget that complexity bad and we don't want to have the complexity demon enter our codebase

I suspect this issue is also done, except if we consider #177

indeed


DASHBUILDE-218: Visually distinguish modifiable / unmodifiable items in NavTreeEditor

Implementation done => https://issues.jboss.org/secure/attachment/12420568/dashbuilder_navitem_actions3.mp4
@romartin @jhrcek Can you review please?

@dgutierr looks nice. I'll review this first thing on monday and let you know

@jhrcek Fixed the issues you mention. Also added a few other improvements:
1.- Current item edition is cancelled if another item's edition is started.
2.- Empty names are not allowed (name input box is marked as red).
3.- Changes must always be confirmed by clicking on the proper icon.
Please have a look => https://issues.jboss.org/secure/attachment/12420640/12420640_dashbuilder_navitem_actions4.mp4

Thanks for quict fixes! I tried it locally with dashbuilder-showcase where it works fine. I'd still like to run selenium tests which only work with kie-wb, but I was unable to build this locally.
I'll let Jenkins execute full downstream build and check the war tomorrow morning. Please wait with the merge.

@dgutierr it seems that the downstream build job is failing with the same error that I was getting when building kie-wb locally (first mvn clean install in dashbuilder with this PR checked out (successfully) and then the mvn clean install in kie-wb-distributions which fails). Any idea what might be wrong?

Jenkins execute full downstream build

Tested locally and almost all the issues have been fixed now except for this last thing:
The buttons still overlap in the NavItemEditor "edit mode"

@jhrcek Latest issue fixed.

Build finished. 964 tests run, 1 skipped, 0 failed.

@romartin merge? :-)

Jenkins please test this

@romartin ready for merge :)

Merged: e8d722e49e137c9ed8dc45e5c1a52f0471cd291a


feat: stfu にて削除後にリアクションするように変更

Discussion: #198
Type of Change:
CommandMessage および SheriffCommand への機能追加
Details of implementation (実施内容)
削除処理を行った後に, リアクションを付与して応答するようにしてみました.

Codecov Report

Merging #208 (2b3538e) into main (59e4d64) will decrease coverage by 0.07%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##             main     #208      +/-   ##
==========================================
- Coverage   51.40%   51.32%   -0.08%
==========================================
  Files          35       35
  Lines         749      752       +3
  Branches      122      122
==========================================
+ Hits          385      386       +1
- Misses        245      247       +2
  Partials      119      119

Flag
Coverage Δ

unittests
51.32% <33.33%> (-0.08%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/adaptor/transformer/message-convert.ts
32.00% <0.00%> (-1.34%)
:arrow_down:

src/service/command-message.ts
28.57% <0.00%> (-4.77%)
:arrow_down:

src/service/stfu.ts
81.81% <100.00%> (+1.81%)
:arrow_up:

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 59e4d64...2b3538e. Read the comment docs.


feat: update curve library

Updates the curve library dependency to a version of the fork that it current against upstream, which was recommended in an audit.
Marked as draft until the corresponding curve library fork PR is merged and a new release is issued.

This would cause some dependency conflicts elsewhere in the ecosystem.


Disable/enable ngx-translate based on a flag

Current behavior
No flag or feature available to turn on/off translations on demand
Expected behavior
Provide a flag or feature that can be easily set using an API to enable/disable translations on demand.
Instead of translations, I want to show keys on the UI for the users to understand the underlying key behind a translated text
What is the motivation / use case for changing the behavior?
In order to allow the user to understand the key behind the translation,
How do you think that we should implement this?
By providing an API to simply turn off translation using pipe or directive on demand

You can use ngx-translate-debug plugin for displaying keys


DirectionsLanguage in german are not working

DirectionsLanguage in German are not working , throws exception ...
routeParams.DirectionsLanguage = new CultureInfo("de");
working for English ..
routeParams.DirectionsLanguage = new CultureInfo("de");

@rexhansen @mbranscomb Is this still relevant?

@anoo4u @anttikajanus @mbranscomb This was fixed in the latest release, 10.2.6.


Fix k8s version for import cluster and creation of secrets

@bmdepesa Could you review these changes?

LGTM


How to print Image ?

**

**

**  **

I didn't see you list it down. Do I need to use it as NativeModule?

I didn't see you list it down. Do I need to use it as NativeModule?

I don't have to. I forgot to list it

I didn't see you list it down. Do I need to use it as NativeModule?

I don't have to. I forgot to list it

It shows error : Property 'printBitmap'Custom does not exist on type 'SunmiPrinterType'.

I didn't see you list it down. Do I need to use it as NativeModule?

I don't have to. I forgot to list it

It shows error : Property 'printBitmap'Custom does not exist on type 'SunmiPrinterType'.

hold on

I didn't see you list it down. Do I need to use it as NativeModule?

I don't have to. I forgot to list it

It shows error : Property 'printBitmap'Custom does not exist on type 'SunmiPrinterType'.

hold on

If you working on it please also check SunmiScan, it doesn't return result.

SunmiScan

ok

I didn't see you list it down. Do I need to use it as NativeModule?

I don't have to. I forgot to list it

It shows error : Property 'printBitmap'Custom does not exist on type 'SunmiPrinterType'.

hold on

If you working on it please also check SunmiScan, it doesn't return result.

ok

Ok, you can use it now

Ok, you can use it now

It got error : 'Got unknown argument class: Bitmap'

+1 also getting 'Got unknown argument class: Bitmap'

+1 also getting 'Got unknown argument class: Bitmap'

Sorry, I am currently working from home and cannot debug.

@mikgross printBitmap ,There is one more, you can try

I tried
import cheekin from './../assets/printerImages/cheekin.bmp';
var printImage = Image.resolveAssetSource(cheekin).uri;
SunmiPrinter.printBitmap(printImage)

&&
import cheekin from './../assets/printerImages/cheekin.bmp';
var printImage = Image.resolveAssetSource(cheekin).uri;
SunmiPrinter.printBitmapCustom(printImage)

None work, any idea what I should do? Can you include a working example in the example app?

Unfortunately the printing for images is not working.
I have additionally tried to modify your package with a base64 string
implementation then decoded as bitmap, it does not print.
Do you have a working example?
Le mar. 5 avr. 2022 à 06:39, 夜雨 @.***> a écrit :

@mikgross https://github.com/mikgross printBitmap ,There is one more,
you can try
—
Reply to this email directly, view it on GitHub
https://github.com/Surile/react-native-sunmi-printer/issues/1#issuecomment-1088260998,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIWEOLKHUY35KP6A3MGEH43VDO7XRANCNFSM5IFZFBDA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@mikgross There may be a problem with the api

I have changed the Bitmap format to base64 string in the main input. and changed the handler as such:
public void printBitmap(String encodedString, Int pixelWidth) throws RemoteException {
    final String pureBase64Encoded = encodedString.substring(encodedString.indexOf(",")  + 1);
    final byte[] decodedBytes = Base64.decode(pureBase64Encoded, Base64.DEFAULT);
    Bitmap decodedBitmap = BitmapFactory.decodeByteArray(decodedBytes, 0, decodedBytes.length);
    int w = decodedBitmap.getWidth();
    Integer h = decodedBitmap.getHeight();
    Bitmap scaledImage = Bitmap.createScaledBitmap(decodedBitmap, pixelWidth, (pixelWidth / w) * h, false);
    printerService.printBitmap(scaledImage, innerResultCallback);
  }

Works fine.
Do you want me to do a pull request?

Of course, please send a pr. thank you very much

---- 回复的原邮件 ----

                    发件人

                    日期

                    2022年04月05日 21:11

                    收件人

                    抄送至

、State<EMAIL_ADDRESS>

                    主题

                    Re: [Surile/react-native-sunmi-printer] How to print Image ? (Issue #1)

I have changed the Bitmap format to base64 string in the main input. and changed the handler as such:
public void printBitmap(String encodedString, Int pixelWidth) throws RemoteException {
    final String pureBase64Encoded = encodedString.substring(encodedString.indexOf(",")  + 1);
    final byte[] decodedBytes = Base64.decode(pureBase64Encoded, Base64.DEFAULT);
    Bitmap decodedBitmap = BitmapFactory.decodeByteArray(decodedBytes, 0, decodedBytes.length);
    int w = decodedBitmap.getWidth();
    Integer h = decodedBitmap.getHeight();
    Bitmap scaledImage = Bitmap.createScaledBitmap(decodedBitmap, pixelWidth, (pixelWidth / w) * h, false);
    printerService.printBitmap(scaledImage, innerResultCallback);
  }

Works fine.
Do you want me to do a pull request?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID<EMAIL_ADDRESS>

I have changed the Bitmap format to base64 string in the main input. and changed the handler as such:
public void printBitmap(String encodedString, Int pixelWidth) throws RemoteException {
    final String pureBase64Encoded = encodedString.substring(encodedString.indexOf(",")  + 1);
    final byte[] decodedBytes = Base64.decode(pureBase64Encoded, Base64.DEFAULT);
    Bitmap decodedBitmap = BitmapFactory.decodeByteArray(decodedBytes, 0, decodedBytes.length);
    int w = decodedBitmap.getWidth();
    Integer h = decodedBitmap.getHeight();
    Bitmap scaledImage = Bitmap.createScaledBitmap(decodedBitmap, pixelWidth, (pixelWidth / w) * h, false);
    printerService.printBitmap(scaledImage, innerResultCallback);
  }

Works fine. Do you want me to do a pull request?

Hi, @mikgross . I installed this package today and when I'm trying to use printBitmap I get an error:
"width and height must be > 0"
Did you have the same issue? If yes how did you resolve it. Thanks.

Hi @Surile, any updates? I have the same issue.

It has been updated

It has been updated

@Surile , It has been updated by @mikgross last time. But his fix doesn't work for me. If I try to pass to the printBitmap() method the base64 encoded image and width for it, I'll receive an error: "width and height must be > 0"

I can't pass three params - image, height and width. Typescript returned an error that method accept only two params. With two params the as issue as @SVPetrenko.

@orlenko-dmitry @SVPetrenko Whether to take off data:image/png;base64

@Surile , sorry, what do you mean?

Whether to delete data:image/png;base64

same issue "width and height must be > 0"
while it SunmiPrinter.printBitmap()  accepts only two parameters : encodedString and pixelWidth.
What is the solution??
Help will be much appreciated

@Surile @ngondat97 @orlenko-dmitry @SVPetrenko  I found the solution for "widht and height must be > 0" issue.
The image that we are trying to print from the printer has greater width than the receipt of the printer. So, image should be within the 384 width or maybe less than 384.
Pass 384 as a pixelwidth of printbitmap method and provide the base64 string of image with the width less than 384 if you are using 58mm paper printer.
I hope this will help .

I have changed the Bitmap format to base64 string in the main input. and changed the handler as such:
public void printBitmap(String encodedString, Int pixelWidth) throws RemoteException {
    final String pureBase64Encoded = encodedString.substring(encodedString.indexOf(",")  + 1);
    final byte[] decodedBytes = Base64.decode(pureBase64Encoded, Base64.DEFAULT);
    Bitmap decodedBitmap = BitmapFactory.decodeByteArray(decodedBytes, 0, decodedBytes.length);
    int w = decodedBitmap.getWidth();
    Integer h = decodedBitmap.getHeight();
    Bitmap scaledImage = Bitmap.createScaledBitmap(decodedBitmap, pixelWidth, (pixelWidth / w) * h, false);
    printerService.printBitmap(scaledImage, innerResultCallback);
  }

Works fine. Do you want me to do a pull request?

Hi, @mikgross . I installed this package today and when I'm trying to use printBitmap I get an error: "width and height must be > 0"
Did you have the same issue? If yes how did you resolve it. Thanks.

Hey man, sory for my late reply.
Maybe you already got it working.
I belive your issue is coming from having a wrongly encoded image. Mine is correctly encoded and throws back the width and hight accoridng to what should be.
Didi you resolve your issue?

Hi, @mikgross. No, it is still not working for me. Currently, I'm using a temporary solution.
Perhaps, you are right. How is it possible to check? Or could you probably tell me which way you encoded an image?

Any solution ? I have same error.. @Surile same issue "width and height must be > 0"

Any solution ? I have same error.. @Surile, @mikgross  same issue "width and height must be > 0"

Can you share the code of the method that you are using ?
And what is your printer paper width specification?

Any solution ? I have same error.. @Surile, @mikgross  same issue "width and height must be > 0"

Can you share the code of the method that you are using ? And what is your printer paper width specification?

@Rehankhalil462   paper width is 58mm then
I have tried like this:
 RNFS.readFileAssets('profile.bmp', 'base64')         .then(content => {            console.log('printImage: ' + content);             SunmiPrinter.printBitmap(content, 34);       })
or
Using direct string base64 and removing data:image/bmp;base64 but always i have same result..
Do you have any base64 string that this work ?

@javi2491 You're welcome.  I'm glad to hear this.


Error when current directory name ends with .go

If the name of the directory ends in .go, tests cannot be generated.
$ pwd
$GOPATH/src/github.com/scottjbarr/foo.go

$ gotests -all .
Parser.Parse source file: ioutil.ReadFile: read $GOPATH/src/github.com/scottjbarr/foo.go: is a directory

I've replaced the actual $GOPATH with $GOPATH
If the name of the directory is changed from "foo.go" to "foo", then test generation works as expected.

This seems like an unusual edge case, as I believe directories don't usually end in ".go". Feel free to make a pull request to address this issue.

Yes it is odd. Unfortunately I have seen it out in the world at two separate companies now.


feat: populate existing cspell.json words during migration

PR Checklist

[x] Addresses an existing open issue: fixes #702
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview
Runs pnpm lint:spelling and matches Unknown word (...) complaints.
Also standardizes two things in code that were bugging me:

node:fs/promises imports as import * as fs
Using formatJson instead of manually prettier.format calls

This finally means that the file creator for cspell.json no longer needs to prepopulate with a static list of words. So unnecessary ignored words will no longer be potentially introduced by that. Yay.

Test failures are unrelated. #877


Reference Error define is not defined

Issue Summary
After Inviting a team member, when they click the link in their email it takes them to the signup page but page is blank.
Client is using:
OS: Ubuntu 16
Browsers: Firefox, Chrome
Steps to Reproduce

Invite Author Team member.
Team member receives email and clicks link.
Team member is taken to my blogs signup page.
Page displayed is empty. Chrome dev tools Errors.

I expected no errors because when I performed a test using an personal alternate email, it worked fine. This test was done on Mac and chrome.
Technical details:

Ghost Version: 0.11.9
Node Version: 4.2.6
Browser/OS: Ubuntu 16.0.4
Database: Mysql

Not sure what might be causing this yet, however I just wanted to recommend setting up SSL at least for your admin panel :)

I would like to close this issue. I am not able to reproduce this.
We can always re-open this issue. I hope you sorted your problem out 👋

sorry i think the problem was a bad mailgun setup. Once i set it up correctly the user was able to join.


[Help] Uncaught Error: Maximum update depth exceeded

Insomnia Version: ___5.14.9
Operating System: ___ubuntu 16.0.4

i work with the code source of insomia
when i try to test from it   i get this error
i don't understand  why i get this error and  how to fix it
Details
invariant.js?c115:42 Uncaught Error: Maximum update depth exceeded. This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate. React limits the number of nested updates to prevent infinite loops.     at invariant (invariant.js?c115:42)     at requestWork (react-dom.development.js?054f:10833)     at scheduleWorkImpl (react-dom.development.js?054f:10732)     at scheduleWork (react-dom.development.js?054f:10689)     at Object.enqueueSetState (react-dom.development.js?054f:6212)     at ResponseTimer.Component.setState (react.development.js?7641:237)     at ResponseTimer._handleUpdateElapsedTime (response-timer.js?d292:25)     at ResponseTimer.componentDidUpdate (response-timer.js?d292:38)     at commitLifeCycles (react-dom.development.js?054f:8778)     at commitAllLifeCycles (react-dom.development.js?054f:9946)

react 16.0.0

Do you have code that I can look at? The error message you provided explains the cause pretty well: "This can happen when a component repeatedly calls setState inside componentWillUpdate or componentDidUpdate". It seems like you're calling setState in the wrong place.

I don't know at any file this error comes from because when i run insomia ,i put a url and when i clic to send i get this error

I just upgraded Electron on my local project and now see this error too. Will fix!

I'm not sure why it wasn't occurring before...

Should work now on the latest version of the develop branch.

ok thank you gschier

This problem still exists. It seems that the fix of gschier has not been merge into the develop branch?
@gschier

@dimitropoulos
I'm using UNSAFE_componentWillReceiveProps. it seems to be OK.
But I'm not sure that's the right way to do it.


wish: remove "multiprocess" from cvxpy requirements

Because of multiprocess, cvxpy depends on pyreadline, which is a problem on Windows.
Would it be possible to change that (like Jupyter project did) ?
No matching distribution found for pyreadline>=1.7.1 (from dill>=0.2.5->multiprocess->cvxpy)

Can you install multiprocess on Windows via "conda install -c cvxgrp multiprocess"?

I'm not using conda. "pyreadline-2.1.win-amd64.exe" would be the workaround, that I wish to avoid.

The only officially supported installation methods for windows are Anaconda and Python(x,y). You could install from source and manually remove the multiprocess dependency. It's not really important.

I see. Wouldn't "joblib" be a possible replacement ? joblib is not a problem on  windows

Unfortunately joblib won't work because I need to pickle functions, which only multiprocess can do. The whole dependency on multiprocess is a bit annoying. For cvxpy 1.0 I'll try and switch to joblib.

@SteveDiamond ,
Could you reconsider?
@stonebig is running very popular Python Distribution for Windows (WinPython) and this is a blocker for having CVXPY inside it.
Thank You.

I could make multiprocess optional, with features disabled if it's not
installed. I'm not opposed to that. That's how all the solvers work now.
On Sat, Jul 1, 2017 at 4:12 AM Royi<EMAIL_ADDRESS>wrote:

@SteveDiamond https://github.com/stevediamond ,
Could you reconsider?
@stonebig https://github.com/stonebig is running very popular Python
Distribution for Windows (WinPython
https://github.com/winpython/winpython) and this is a blocker for
having CVXPY inside it.
Thank You.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cvxgrp/cvxpy/issues/296#issuecomment-312426122, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAfyIID5TuB4nfOwDhI6a0_IiU9t6jMqks5sJimogaJpZM4JtgTC
.


Update IC Candid Files

Motivation
The Internet Computer APIs have updated.
Changes

Updated the interface description files for the Internet Computer.
Updated the javascript bindings for the latest IC interface.

Tests

[ ] Please check the API updates for any breaking changes that affect our code.

Closing, they need to be updated manually first.


The tooltip “Lorem” for “info-circle” button is revealed only by mouse over and is not available to keyboard-only users in Tooltips tab

URL: http://clarity-ks.surge.sh/tooltips
Tools: Web Accessibility Toolbar (WAT2.0)
Reproduced Steps:

Login Clarity Kitchen Sink with web browser.
Go to Tooltips tab.
Use TAB key to check the tooltip of “info-circle” button in Tooltips tab.

Actual Results:
The tooltip “Lorem” for “info-circle” button is revealed only by mouse over and is not available to keyboard-only users in Tooltips tab.
For more details, please refer to "Tooltips-NotAvailableOnscreen.png".
Expected Results:
The tooltip “Lorem” for “info-circle” button is revealed only by mouse over and should be available onscreen in Tooltips tab.
Fix Suggestion:
Add equivalent text on screen and the text is consistent with tooltip.
Or the equivalent text can be displayed with TAB key.
Violation of Section 508 Standards:
1194.21(a) When software is designed to run on a system that has a keyboard, product functions shall be executable from a keyboard where the function itself or the result of performing a function can be discerned textually.
User Impact: Blind.
Screen shot:

Tooltips-NotAvailableOnscreen.png

Verified that ANGULAR tooltips are no longer accessible. See website to test it out.