Unexpected error on adding integration

Hi,
On attempting adding the integration and entering my IP and remote password, I get an 'Unexpected Error' thrown up which prevents me from adding the integration. Have tried a couple of time now. I'm running Home Assistant OS core-2022.7.0b2 and tried to install the latest version of homeassistant-fullykiosk. Below is the log and would appreciate any support you can give.
Thanks,
Johnny
`This error originated from a custom integration.
Logger: custom_components.fullykiosk.config_flow
Source: custom_components/fullykiosk/config_flow.py:49
Integration: Fully Kiosk Browser (documentation, issues)
First occurred: 18:07:13 (3 occurrences)
Last logged: 18:28:53
Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/fullykiosk/config_flow.py", line 50, in validate_input
deviceInfo = await fully.getDeviceInfo()
File "/usr/local/lib/python3.10/site-packages/fullykiosk/init.py", line 30, in getDeviceInfo
result = await self.sendCommand("deviceInfo")
File "/usr/local/lib/python3.10/site-packages/fullykiosk/init.py", line 22, in sendCommand
data = await self._rh.get(
File "/usr/local/lib/python3.10/site-packages/fullykiosk/init.py", line 139, in get
async with self.session.get(
File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 1138, in aenter
self._resp = await self._coro
File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 535, in _request
conn = await self._connector.connect(
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 542, in connect
proto = await self._create_connection(req, traces, timeout)
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 907, in _create_connection
_, proto = await self._create_direct_connection(req, traces, timeout)
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 1175, in _create_direct_connection
transp, proto = await self._wrap_create_connection(
File "/usr/local/lib/python3.10/site-packages/aiohttp/connector.py", line 986, in _wrap_create_connection
return await self._loop.create_connection(*args, **kwargs)  # type: ignore[return-value]  # noqa
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 1049, in create_connection
sock = await self._connect_sock(
File "/usr/local/lib/python3.10/asyncio/base_events.py", line 960, in _connect_sock
await self.sock_connect(sock, address)
File "/usr/local/lib/python3.10/asyncio/selector_events.py", line 500, in sock_connect
return await fut
asyncio.exceptions.CancelledError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/config/custom_components/fullykiosk/config_flow.py", line 80, in async_step_user
info = await validate_input(self.hass, user_input)
File "/config/custom_components/fullykiosk/config_flow.py", line 49, in validate_input
with timeout(15):
File "/usr/local/lib/python3.10/site-packages/async_timeout/init.py", line 116, in exit
self._do_exit(exc_type)
File "/usr/local/lib/python3.10/site-packages/async_timeout/init.py", line 212, in _do_exit
raise asyncio.TimeoutError
asyncio.exceptions.TimeoutError
`

I upgraded to 2022.7.0 and maybe got something related?
Logger: homeassistant.setup
Source: setup.py:162
First occurred: 7:45:03 PM (1 occurrences)
Last logged: 7:45:03 PM

Setup failed for custom integration fullykiosk: Unable to import component: No module named 'fullykiosk'

HASS 2022.7 has updated python to 3.10, that's probably wreaking havoc here as well.

HASS 2022.7 has updated python to 3.10, that's probably wreaking havoc here as well.

Interesting, as I'm currently reading the 2022.7 release notes and was wondering if fullykiosk is "Python 3.10 ready".
Can you answer that question please @cgarwood ?

@bcutter working fine on Python 3.10 for me (HA OS on my main system)
OP is getting a TimeoutError, so it appears the tablet isn't responding to HA's requests fast enough. I've seen that happen with lower spec hardware, and at one point on Kindle's especially the remote admin API would just stop responding all together for a while and come back.
@onedr0p you're missing the python library that the component uses. Home Assistant should install it automatically though. How are you running HA? (venv/container/OS/etc)

The issue was fixed on home assistant 2022.7.4.

Yeah I'm trying this with a 2019 Kindle Fire and getting the same error. Any advice on how to make the API respond quicker so it registers?


E2E test for Natss Channel redelivery

Cloned from knative/eventing#1275
Problem
Natss Channel supposedly supports Redelivery (according to https://github.com/knative/eventing-contrib/blob/master/test/config.go#L37), we don't test this in an e2e test
Exit Criteria
An E2E test case is added to verify the redelivery feature for Natss Channel

/assign

blocked by https://github.com/knative/eventing/issues/2912

This is done.
/close


Gopass wizard aborts with an Error message

I just stumbled upon gopass and wanted to try it out (archlinux).
I run gopass and select to run the onboarding wizard. Then I select "Local store".
The wizard aborts with an error message and no information on how to proceed:
Error: failed to run onboarding wizard: failed to init local store: failed to read user input: no useable private keys found

EDIT: Btw I have also looked through the documentation and found no info on this.

Are you asked to create a new private key?
Do you have a valid GnuPG keypair?
If the answer to both is no, could you please run gopass as "GOPASS_DEBUG=true gopass" and provide the output here?

Are you asked to create a new private key?

The answer is no.

Do you have a valid GnuPG keypair?

The answer is yes, I have two valid GnuPG keypairs under ~.gnupg/

Thanks for providing more information.
Unfortunately I can't neither see the issue in the debug output nor reproduce the issue.
I need to think about this some more.

Upon trying out various stuff it appears to be an issue with the "Trust" value of my key.
After setting it to 5 I can proceed the wizard.
So while my initial problem is solved, I still think the general user experience needs to be improved here.
In my opinion there are several options:

Give the user a better, more explanatory error message
Improve the documentation
Give the user an option to make a new key
Let the user modify the trust level

In any case just aborting the "wizard" is the worst user experience.
And the trust level thing is something most users have probably not been aware of, nor that it is required by gopass, while the key totally works for other use cases like github etc.

1 and 2 sound good.
4 is already implemented but it doesn't seem to properly take the trust level into account.
I think I'll take option 5: Kill GPG. Eventually.
For now we should do 1, 2 and maybe 4.

Fixing that would definitely help stranded users

4 is already implemented but it doesn't seem to properly take the trust level into account.

If it's already there, the wizard should definitely not abort there :smile:
If no suitable keys have been found the user should be given the option to create a new one.
Optionally, list the keys that have been found but are not "suitable" (users will probably wonder why their current ones don't work).

I won't even try 3.

Wait wouldn't it be like one command line argument? Of course while warning the user that he should actually trust the key when he does this.

I think I'll take option 5: Kill GPG. Eventually.

Wait what? :rofl: Isn't like the whole concept of the project to use gpg? Are there even any alternatives??
Tbh since I didn't want to waste my time and GPG confuses me anyways I just dowloaded a gpg GUI and changed it there.


Is it possible to daisy chain commands together?

Hello,
Firstly thanks for this excellent plugin. I was looking to daisy chain some commands together to open my daily note, append a template & execute the templater command to replace templates in the active file. I have read the documentation & became much more familiar with the advanced URI syntax but so far have been unable to get this working using the following:
obsidian://advanced-uri?vault=Vault&daily=true&commandid=obsidian-hotkeys-for-templates%253AEvening%2520Journal%2520Template.md&mode=append&commandid=templater-obsidian%253Areplace-in-file-templater
obsidian://advanced-uri?vault=Vault&daily=true&commandid=obsidian-hotkeys-for-templates%253AEvening%2520Journal%2520Template.md&commandid=templater-obsidian%253Areplace-in-file-templater
&mode=append
I'm wondering if this is currently possible?
Many Thanks

You can only call one command per URI. So you have to call multiple URIs one by one.

Ok thanks for the clarification.

Not currently possible

Ok thanks for the clarification.

You can combine this amazing plugin with two other plugins to achieve want you want:
1 URI Commands to run URI as a command
2 Quick Add to daisy chain the commands generated from URI Commands
I hope it helps :)

I'm closing this, because I don't plan to implement this.


Get live stream in a C# Form

Hi,
I wonder if the current API could help to retrieve the complete link to be able to incorporate it in a C# Form, or at least to open it in a browser?
Thx for you help!

Well, if youre joining a channel using the TwitchClient, you already know
the name of the channel and can build the url to open in a browser (
http://www.twitch.tv/channelname).
I'm not entirely positive on how you could embed the stream directly into a
form, although I'm sure it's possible somehow.
On Jan 31, 2017 9:54 AM, "jscoys"<EMAIL_ADDRESS>wrote:

Hi,
I wonder if the current API could help to retrieve the complete link to be
able to incorporate it in a C# Form, or at least to open it in a browser?
Thx for you help!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/swiftyspiffy/TwitchLib/issues/120, or mute the thread
https://github.com/notifications/unsubscribe-auth/AVZYijZmdeEp9BX_Y7Y-E3EvRB8ngUKGks5rX1lGgaJpZM4Lyz5y
.

Ok thx but little question : If I connect my account to one chat channel, is my connection considered as a "viewer" or only as a "chatter"?

Assuming you're speaking of the TwitchClient, this is is an interesting question. I am pretty sure the connected channel (via IRC) does not count as a viewer, only a chatter.

Ok thx a lot!


build with maven

Hi, I "ported" this excample to be build with maven as well (so gradle & maven are both valid build utilities)

Why not! Thanks for the PR.


Any chance for a Fabric port?

There was no faq on the mod page so I'm asking here.

Sorry, but no. I don't know how to mod for fabric.
But if anyone wants to port it for fabric, I am okay with that.


SPEC 4: mention the retention policy in the SPEC

It turns out there is a retention policy enforced by a github action that removes any artifact if

there are already 5 previous version or
the artifact is more than 30 days old

@bsipocz is that correct?
This should be part of the SPEC, and new projects should be notified of this policy.
xref scientific-python/upload-nightly-action#34

I thought #262 was supposed to close this. If not, we can reopen this.

Hah, I made the beginners mistake of not using the correct verb to auto-close. Sorry about that.


another attempt

About the PR

Why / Balance

Technical details

Media

Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

anyway i already managed to solve this one


Device info returning : undefined is not an object

Summary
| Version (react-native) | 0.55.4 |
| Affected OS | Android Only |
| OS Version | Android 8.0 |
First of all, I installed react-native-navigation and react-redux.
| Version (react-native-navigation) | 1.1.484 |
| Version (react-redux) | 5.0.7 |
Afterwards, I installed react-native-device-info and followed all steps to link the lib (i did both, manually and automatically).
| Version (react-native-device-info) | 0.22.5 |
One last thing, i run the app on an emulator.
Current behavior
I declare the module like this :
import DeviceInfo from 'react-native-device-info';
When I call the getModel() (DeviceInfo.getModel()) function in a render i got an error :

Thank you!

Please send your code.

@to175 send eh w arf eh el bata3a bayza

hello,i have same issue,the follow code in ios simulator,android has well
import {getDeviceLocale} from 'react-native-device-info';
let lan = getDeviceLocale();
get this errors:
Unhandled JS Exception: (0, _reactNativeDeviceInfo.getDeviceLocale) is not a function. (In '(0, _reactNativeDeviceInfo.getDeviceLocale)()', '(0, _reactNativeDeviceInfo.getDeviceLocale)' is undefined)
2018-09-06 16:57:38.454 [error][tid:com.facebook.react.JavaScript] Module AppRegistry is not a registered callable module (calling runApplication)

Experiencing the same issue using RN 0.55.3 and react-native-device-info 0.22.5
Every method on DeviceInfo throws an undefine d is not an object exception

I'm getting the same error, but for every single call (12 total). I'm basically doing:
import DeviceInfo from 'react-native-device-info';

class SomeClass {
  getDeviceInfo() {
    return {
        app: {
          name: DeviceInfo.getApplicationName(),
          build: DeviceInfo.getBuildNumber(),
        },
        device: {
          name: DeviceInfo.getDeviceName(),
          manufacturer: DeviceInfo.getManufacturer(),
        },
        locale: DeviceInfo.getDeviceLocale(),
        location: {
          country: DeviceInfo.getDeviceCountry(),
        },
        network: {
          carrier: DeviceInfo.getCarrier(),
        },
        os: {
          name: DeviceInfo.getSystemName(),
          version: DeviceInfo.getSystemVersion(),
        },
        referrer: {
          name: DeviceInfo.getInstallReferrer(),
        },
        screen: {
          density: PixelRatio.get().toString(),
          height: Dimensions.get('window').height.toString(),
          width: Dimensions.get('window').width.toString(),
        },
        timezone: DeviceInfo.getTimezone(),
        userAgent: DeviceInfo.getUserAgent(),
      }
  }
}

I'm running it in a local development environment, but on my iPhone 8
$ react-native info

Environment:
  OS: macOS High Sierra 10.13.5
  Node: 8.11.3
  Yarn: 1.7.0
  npm: 5.6.0
  Watchman: 4.9.0
  Xcode: Xcode 9.4.1 Build version 9F2000
  Android Studio: 3.1 AI-173.4907809

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: ^0.55 => 0.55.4

Same problem as described above happens here

Same as above

@iceagency-lukehopkins Manual linking fixed the issue on Android too :smile:

I add the same error as you and was able to solve it by changing MainApplication.java as following
  @Override
  public List<ReactPackage> createAdditionalReactPackages() {
    return Arrays.<ReactPackage>asList(
            // eg. new VectorIconsPackage()
            new RNDeviceInfo()
    );
  }

  private final ReactNativeHost mReactNativeHost = new ReactNativeHost(this) {
    @Override
    public boolean getUseDeveloperSupport() {
      return BuildConfig.DEBUG;
    }

    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
              new MainReactPackage(),
              new RNI18nPackage()
      );
    }
  };

Importing instead of requiring worked for me.
let DeviceInfo = require('react-native-device-info');
import DeviceInfo from 'react-native-device-info';

Like @nikuslas , I am also using react-native-navigation.
I had the same problem as many others (i.e. import DeviceInfo from 'react-native-device-info' was returning undefined) but after reading all these comments, I figured out what was going wrong.
If you start your project with react-native init, you will have a MainApplication.java with a getPackages() method that is part of the ReactNativeHost implementation.  If you later do step 5 of RNN's Android instructions, you will get a second getPackages() method.  This second method is called by createAdditionalReactPackages() (which is an abstract method in RNN's base class NavigationApplication).
When you run react-native link react-native-device-info, it adds new RNDeviceInfo() to the first getPackages() method.  I was able to fix this import problem by adding new RNDeviceInfo() to the second getPackages() method -- the one called from createAdditionalReactPackages().
I hope this helps everyone!

I encountered the error on iOS and fixed the error by changing the import from destructured
import { getModel } from 'react-native-device-info'

to a default import
import DeviceInfo from 'react-native-device-info'

and used it with DeviceInfo.getModel()

Same here. I solved this by replacing all
var DeviceInfo = require('react-native-device-info');
with
import DeviceInfo from 'react-native-device-info';

I think document needs to be updated react-native link react-native-device-info doesn't link libraries to project, you need to manually add them here are the steps:

right click on Libraries > Add files to <app name>
select RNDeviceInfo.xcodeproj
select Copy items if needed and add
then under Linked frameworks and libraries add libRNDeviceInfo.a

build and run, it worked for me hope will work for you too 😃

@machour closable - another failure to install correctly, but the library definitely works according to instructions (I just installed it...)

To confirm @NeliHarbuzava solution.
Solved this by replacing all
var DeviceInfo = require('react-native-device-info');
with
import DeviceInfo from 'react-native-device-info';
works for me.

DeviceInfo.default.getBuildNumber() worked for me.

I believe my issue is related to react-native-navigation too. Perhaps something to do with the default behaviour of NavigationApplication
I had to add the following to my android/app/src/main/java/com/foo/MainApplication.java file:
import com.learnium.RNDeviceInfo.RNDeviceInfo;

public class MainApplication extends NavigationApplication {
    protected List<ReactPackage> getPackages() {
        return Arrays.<ReactPackage>asList(
           new RNFirebasePackage(), // for example
            new RNDeviceInfo()
        );
    }
}

@mikehardy - I understand your frustration with the sheer number of "RNDeviceInfo is null" style issues, but there's nowhere that mentioned manually adding the import and package to the MainApplication file. I did a test new project and it worked fine, and as someone mentioned above it's a clash with rnn. I wonder whether this is something that needs to be documented?

It's possible something needs to be documented, but what? If you look in this repo there are two scripts - refresh-example-rn59.sh and refresh-example.sh - both of which automagically create the example app for the repo, from scratch. If some modification to those scripts (perhaps just including react-native-navigation via yarn add or similar?) can demonstrate a failure, then a doc update would be gratefully accepted. But I've not seen a clean demo of a problem yet

Just adding RNN won't cause the issue, I don't believe. I think it's when the class that the MainApplication extends changes that causes the issue. I'm quite a newbie when it comes to react native so I don't know too much about it and whether the issue lies with RNN's changes to the MainApplication file or not. But if there's a troubleshooting section then perhaps it can be added?
Or hopefully people would just find my comment on this thread! But I didn't realise for a long time that it'd be due to mixing with RNN, so I wouldn't have searched for that.
If I get a spare moment I'll see if I can create a test case

I see nothing in here that would conflict https://wix.github.io/react-native-navigation/#/docs/Installing?id=_7-update-mainapplicationjava - at least not in a react-native 0.60+ world with auto-linking etc.

Well yeah I had assumed that autolinking would take care of it, but adding the imports seems to be the only way I could get it to finally work. Also quite a few other packages (such as firebase) seem to require adding the packages that way. My test react-native app without react-native-navigation worked fine without it. So unless I'm missing something, it seems that RNN's paradigm requires specifying those packages manually...?
Although that's  probably on RNN's shoulders, perhaps it would be good to show somewhere that you'd need to add import com.learnium.RNDeviceInfo.RNDeviceInfo; and new RNDeviceInfo() if nothing else is working. Because it took some delving into the repo to find what the full package name was.

react-native-firebase is an interesting beast. If you look carefully it does not require you to manually link the main package. It's all the little sub-packages you have to link. And even then only for react-native-firebase v5, not in v6. I support that repo and demo same here: https://github.com/mikehardy/rnfbdemo/blob/master/make-demo.sh
Auto-linking should handle everything for react-native-device-info, I still assert. If react-native-navigation is perturbing that in any way, that is definitely on them

This is what worked for me.

Do the npm install
Linking the installed library
cd ios and do pod install
Kill the running app and do a fresh react-native run-ios to rebuild the project.
after that:
import DeviceInfo from 'react-native-device-info';
works

Not wroked:
import {DeviceInfo} from 'react-native-device-info';
Worked:
import DeviceInfo from 'react-native-device-info';


Failed to setup message on startup of Home Assistant

This started after upgrade to HA 2023.3.6 and Supervisor 2023.03.3.  Here are the error logs:
This error originated from a custom integration.
Logger: homeassistant.config_entries
Source: custom_components/vivint/hub.py:94
Integration: Vivint (documentation, issues)
First occurred: 4:20:11 PM (1 occurrences)
Last logged: 4:20:11 PM
Error setting up entry<EMAIL_ADDRESS>for vivint
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 383, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/vivint/init.py", line 59, in async_setup_entry
await hub.login(load_devices=True, subscribe_for_realtime_updates=True)
File "/config/custom_components/vivint/hub.py", line 94, in login
await self.account.connect(
File "/usr/local/lib/python3.10/site-packages/vivintpy/account.py", line 66, in connect
authuser_data = await self.vivintskyapi.connect()
File "/usr/local/lib/python3.10/site-packages/vivintpy/vivintskyapi.py", line 67, in connect
authuser_data = await self.get_authuser_data()
File "/usr/local/lib/python3.10/site-packages/vivintpy/vivintskyapi.py", line 97, in get_authuser_data
resp = await self.__get("authuser")
File "/usr/local/lib/python3.10/site-packages/vivintpy/vivintskyapi.py", line 382, in __get
return await self.__call(
File "/usr/local/lib/python3.10/site-packages/vivintpy/vivintskyapi.py", line 430, in __call
resp = await method(
File "/usr/local/lib/python3.10/site-packages/aiohttp/client.py", line 493, in _request
all_cookies = self._cookie_jar.filter_cookies(url)
File "/usr/local/lib/python3.10/site-packages/aiohttp/cookiejar.py", line 239, in filter_cookies
self._do_expiration()
File "/usr/local/lib/python3.10/site-packages/aiohttp/cookiejar.py", line 150, in _do_expiration
self.clear(lambda x: False)
File "/usr/local/lib/python3.10/site-packages/aiohttp/cookiejar.py", line 115, in clear
for (domain, path), cookie in self._cookies.items():
ValueError: too many values to unpack (expected 2)

Duplicate of https://github.com/natekspencer/hacs-vivint/issues/84.
You'll need to upgrade to the latest version of this integration.


Make argument and flag use consistent across commands

Closes #343. See the issue for the full discussion and rationale behind the specific changes made.

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

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


[Feature Request] Adjusting roll formula of rolltables when triggered

Hello! Thanks for your module.
I'm running a setting, where Open Wounds gets a modifier for the currently existing amount of Open Wounds. (e.g. regularly you roll a 2d6, if you have one Open Wound already, you roll 2d6+1) Would it be possible to have DnD5e helpers accomodate for this?
I have not beeg brain to know how the module is set up, so just throwing options out there for implementation; it could be something like being able to run a macro on a Open Wounds trigger (so I could macro the rolling of the rolltable myself), somehow parse character attributes and modify the dice roll formula in the rolltable.

This kind of opens a can of worms for Open Wound homebrew (much like Wild Surge homebrews).
We are planning on opening up the API for the Wild Surge to allow users to provide a custom handler for it. Once we hammer that out, we could likely do the same for Open Wounds.
Not sure when that feature will be rolled out, but I'll keep this issue open until then.


Fix nonlocal unsoundness vs. atomic store

MOVNTI, MOVNTDQ, and friends weaken TSO when next to other stores. As most stores are not nontemporal, LLVM uses simple stores when lowering LLVMIR like atomic store ... release on x86. These facts could allow something like the following code to be emitted:
vmovntdq [addr],     ymmreg
vmovntdq [addr+N],   ymmreg
vmovntdq [addr+N*2], ymmreg
vmovntdq [addr+N*3], ymmreg
mov byte ptr [flag], 1 ; producer-consumer flag

But these stores are NOT ordered with respect to each other! Nontemporal stores induce the CPU to use write-combining buffers. These writes will be resolved in bursts instead of at once, and the write may be further deferred until a serialization point. Even a non-temporal write to any other location will not force the deferred writes to be resolved first. Thus, assuming cache-line-sized buffers of 64 bytes, the CPU may resolve these writes in e.g. this actual order:
vmovntdq [addr+N*2], ymmreg
vmovntdq [addr+N*3], ymmreg
mov byte ptr [flag], 1
vmovntdq [addr+N],   ymmreg
vmovntdq [addr],     ymmreg

This could e.g. result in other threads accessing this address after the flag is set, thus accessing memory via safe code that was assumed to be correctly synchronized. This could result in observing tearing or other inconsistent program states, especially as the number of writes, thus the number of write buffers that may begin retiring simultaneously, thus the chance of them resolving in an unfortunate order, increases. If using &mut [u8] to write uninitialized memory is permitted ( per rust-lang/unsafe-code-guidelines#346 ), it could even result in an access to &[u8] actually being reading uninitialized memory in safe code!
To guarantee program soundness, code using nontemporal stores must currently use SFENCE in its safety boundary, unless and until LLVM decides this combination of facts should be considered a miscompilation and motivation to choose lowerings that do not require explicit SFENCE.
For further information, see:

https://github.com/rust-lang/rust/issues/114582
https://github.com/llvm/llvm-project/issues/64521

Thanks! Also some interesting conversation to follow :)
AFAIK no one's using this code in production anywhere, I just had fun writing it up at some point. Still good to keep correct (ish) though!


Fix To BOB3 Compiler Error When Dealing With Trig Functions

Fix to Issue #667.

Thank you! I have done similar changes while working on #278, which will be merged soon.


feat: Add .payload property to interaction class

Summary
Interactions are valid for 15 minutes, so for slash commands which take a long time to complete, it's feasible that you might restart your bot for updates/maintenance while someone is currently in the middle of a command.
It's therefore useful in some rare cases to be able to store interaction payloads, such that if your bot restarts you can deserialize the payload and recreate the interaction.
This was obviously possible before doing some fairly basic monkey patching, for example doing it for ApplicationCommandInteraction:
def patched_init(original):
    def patched(self, *, data, state):
        self._payload = data
        return original(self, data=data, state=state)

    return patched

disnake.ApplicationCommandInteraction.__init__ = patched_init(
    disnake.ApplicationCommandInteraction.__init__
)

# store/load interaction
store(pickle.dumps(inter._payload))
inter = disnake.ApplicationCommandInteraction(data=pickle.loads(payload), state=bot._connection)

This PR simply makes the Interaction class save the incoming InteractionPayload into the Interaction._payload attribute which is accessible through the Interaction.payload property, to make the monkey-patching unnecessary.
So you can then:
store(inter.id, inter.payload)

# then after a reboot...
payload = load(payload_id)

# recreate ourselves
inter = disnake.ApplicationCommandInteraction(data=payload, state=bot._connection)

# or even have it re-dispatched, whatever kind of interaction it is
bot._connection.parse_interaction_create(data=payload)

Improvements
Some kind of interactions, since they have an internal cache (like ViewStore) I think only make sense to re-dispatch as opposed to recreating it yourself, so maybe the ._payload attribute shouldn't be in the Interaction base class.
It could also be useful to be able to generically recreate an interaction from the payload (as it has its type field), like is done in ConnectionState.parse_interaction_create. It's finicky using that however since it creates and dispatches the interaction at the same time.
Checklist

[x] If code changes were made, then they have been tested

[x] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running pdm lint
[x] I have type-checked the code by running pdm pyright

[ ] This PR fixes an issue
[x] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

While I mostly understand the use case here, it seems very niche and increases the memory footprint of interactions considerably (most fields are essentially being stored twice now). This isn't really something for the library to implement, but feel free to monkeypatch things or fork the lib if needed and there isn't another way.
There's a case to be made for creating an InteractionResponse object with just the interaction token, even if it's not quite what you're asking for, but iirc there were a couple design issues with it that still need to be worked out.

Being able to create an InteractionResponse might really be what I should have been doing in the first place, though obviously that currently stores the parent Interaction anyway so it kind of leaves me in the same spot for now.
Closing the PR though as I agree with everything you said, thanks for taking a look!


Remove 'Description' from object for VLE

We're finding that this can be really long, so give us lots of redundant data this isn't actually used.  If we do find we need course/assessment descriptions we'll move this to UDD.

VLE statement templates updated in 1.0 branch (but not in json examples)


RCTPushNotificationManager does not set alertTitle in UILocalNotification

This issue was originally created by @stage88 as facebook/react-native#14699.

Description
Setting the alertTitle property on PushNotificationIOS.scheduleLocalNotification does not display the title in the notification.
With alertTitle set:

Without:

Reproduction Steps
Use the below code to reproduce.
Sample Code
PushNotificationIOS.scheduleLocalNotification({       fireDate: new Date(Date.now() + (30 * 1000)).toISOString(),       alertTitle: 'Incoming Message', 			alertBody: 'This is my message, please read it for good fortune.', 			soundName: 'default', 			applicationIconBadgeNumber: 1, 			userInfo: {         id: 11       }     })
Solution
Adding the following line in RCTPushNotificationManager.m (54) fixes the issue:
notification.alertTitle = [RCTConvert NSString:details[@"alertTitle"]];
Additional Information

React Native version: 0.45
Platform: iOS
Development Operating System: macOS Sierra 10.12.15
Build tools: XCode 8.3.3

alertTitle support has been added in 1.2.2 (#133) :tada:
Thanks to @lukebars !


Instaloader very slow get usernames from hashtag post

I am accessing the Instagram posts by hashtag. I need to get the post creator user name. Check below code.
posts = self.loader.get_hashtag_posts(hashtag)
for post in posts:
print(post.owner_username)
It takes nearly 5 hours to collect 10k usernames. Is there any way to improve the performance?

The constructor of class Instaloader has an intentionally undocumented parameter sleep. When set to False (default is True), Instaloader does not wait between requests unless absolutely required to not hit rate limits. That would make collecting the usernames faster, but on the other hand raise the risk to be blocked by Instagram servers.


Issue #1286 Add log on as user permission for current user during setup

Fixes #1286
~in order to use the LocalSystem account to run the daemon service
we also need to take care of where the .crc dir is created, as by
default when using the LocalSystem account the users home dir is at
C:\Windows\System32\config\systemprofile but we want the .crc dir to
be created in the logged in user's home directory.~
~We capture the user's home dir when crc setup is run and pass it as an
argument in the service invocation command, e.g crc.exe daemon c:\Users\anjan~
Dropped the commit using the LocalSystem account, instead we use the current user account to the run the daemon service, and also allow log on as a service for the user, so that a crc user doesn't have to do it manually.
How to test:

run crc setup --enable-experimental-features --log-level debug
Should have a working tray up and running
If errors are encountered then Open and check if secpol.msc > Local User Rights > Log on as service contains the current user's name.

@gbraad @cfergeau Dropped the use of LocalSystem account, now the daemon service would run under the user account.
Now we are automatically adding the log on as service permission for the user during setup.

So many requests for Admin permission:
INFO Checking if user is allowed to log on as a service
INFO Will run as admin: Running secedit export command
INFO Enabling service log on for user
INFO Will run as admin: Running secedit export command
INFO Will run as admin: Running secedit configure command
INFO Checking if daemon service is installed
INFO Installing daemon service
INFO Will run as admin: Stopping service: CodeReady Containers
INFO Will run as admin: Uninstalling service: CodeReady Containers
? Enter account login password for service installation [? for help] ********
INFO Will run as admin: Install daemon service
INFO Checking if daemon service is running
INFO Starting daemon service
INFO Will run as admin: Starting service: CodeReady Containers
FATA Could not start service: CodeReady Containers

and since I use a MicrosoftAccount, I am not so sure what my password should be? I use automated login from a connected device to unlock. :-/

I performed a cleanup and when re-running setup it shows:
INFO Will run as admin: Stopping service: CodeReady Containers
INFO Will run as admin: Uninstalling service: CodeReady Containers

No description

Regarding Add check, fix and cleanup for log on as service By reading the commit log (and even in issue #1286) , it's really not clear what problem this is solving, ie if we try to use the daemon without this change, what goes wrong?

The daemon is ran as a windows service, because we want it to keep running in the background and have an interface to stop, restart the daemon. atm we cannot start the daemon without these changes because the user might not have the log on as a service right, so these changes makes sure that the user has those rights.

i read it's to start the daemon, but os.Command('crc', 'daemon').Start() would also run the daemon, so the problem which is being solved needs to be stated better

Okay, so we cannot run it as this during setup, as when the cmd/powershell is closed i think the daemon would also die, and  we don't want to depend on the tray to start the daemon process

@gbraad There is no option to pass description insc.exe create command, we can run sc.exe descritption again to add a description, but that also needs elevated privilege, so one more UAC :(

Regarding Add check, fix and cleanup for log on as service By reading the commit log (and even in issue #1286) , it's really not clear what problem this is solving, ie if we try to use the daemon without this change, what goes wrong?

The daemon is ran as a windows service, because we want it to keep running in the background and have an interface to stop, restart the daemon. atm we cannot start the daemon without these changes because the user might not have the log on as a service right, so these changes makes sure that the user has those rights.

In the commit log, it might be worth adding that without this 'log on as a service' rights, fixDaemonServiceRunning (or fixDaemonServiceInstalled?) will fail? Or will these methods succeed, and then it will fail later? A more detailed description of the issue would probably be useful if we need to revisit that code in the future.

we don't want to depend on the tray to start the daemon process

This is the part I would question/discuss. Given the complexity of the alternative (this PR which needs hundreds of lines of code to start the daemon differently), why not have the tray start it on demand?

we don't want to depend on the tray to start the daemon process

This is the part I would question/discuss. Given the complexity of the alternative (this PR which needs hundreds of lines of code to start the daemon differently), why not have the tray start it on demand?

Okay, so if we are not using a service and the tray starts the daemon process, there are also difficulties i think,  we need to know the crc binary location, but we don't require for it to be added to path, or be in a specific location, idk crc setup could somehow handle it...

We have taken the same approach in case of macOS, we use launchd to run the daemon as an agent which is run under the logged on user, and starts automatically when the user logs in. In case of windows keeping the approach consistent we are using the windows service manager, and hence the daemon service for windows, i've already mentioned that using a service in windows and an agent in macOS we get the o/s provided interface to start/stop/uninstall the daemon.
In case of the windows service it is running under the local user and i've listed out the differences of using the LocalSystem vs a local user account in #1286
We have to reduce the number of UACs during crc setup which i think can be solved in a follow up PR if that is acceptable.
If we now go ahead with an approach of the tray starting the daemon, then we have to do the same for macOS to have a consistent system, which means we need to re-engineer the tray part. @cfergeau what would you suggest we do here?

ACK. Tested this latest version on Windows 10 machines (with and without cygwin) and setup now works nicely. The operation of tray also works for basic operations (start, stop, delete, status, and combination of these). LGTM!

/retest

/retest


External Terminal support for runInTerminal request

Move the code that manages external terminals from the node-debug plugin into the VS Code execution service and surface it as a runInTerminal API.
Make it configurable via the configuration service.

commits: https://github.com/Microsoft/vscode/commit/20c3935a130f5b86d629a8543c9c71348c54c3de https://github.com/Microsoft/vscode/commit/50b5d428aeed3a686ecca36568b5c690dead5293 https://github.com/Microsoft/vscode/commit/6014db1a16223f5d7008417140d5ae98dc48be4c https://github.com/Microsoft/vscode/commit/e190921aea36b3c969f66f8ee72bceef9b938b27


FE-91: Added tooltip to sidebar link content.

JIRA: https://r3-cev.atlassian.net/browse/FE-91
The simple fix would be to add a tooltip to the truncated context of the sidebar menu.
The option to make the sidebar resizeable can be tricky due to how the base layout of the doc site setup. Also by doing that, we might need to fix any potential bugs introduced by the resizeable sidebar to other UI elements.
Overall tooltip for now is a better fix at the moment.

Preview here: https://docs.staging.docs.r3.com/en/platform/corda/5.0/developing-applications/api/application.html

Preview here: https://docs.staging.docs.r3.com/en/platform/corda/5.0/developing-applications/api/application.html

Thanks @nadinequinr3 / @zhengmar3 I've tried it out and I don't get tooltips

@nadinequinr3 @yarivus Same for me, no tooltip.
Then I check the code from the Chrome inspect, my code change is not there. I think you (@nadinequinr3 ) might have setup the preview based on a wrong branch.

@zhengmar3 @yarivus My mistake - this is the correct site - https://corda5.preview.docs.r3.com/en/platform/corda/5.0/developing-applications/api/application.html

Preview here: https://docs.staging.docs.r3.com/en/platform/corda/5.0/developing-applications/api/application.html

Thanks @nadinequinr3 / @zhengmar3 I've tried it out and I don't get tooltips

@nadinequinr3 @yarivus Same for me, no tooltip.
Then I check the code from the Chrome inspect, my code change is not there. I think you (@nadinequinr3 ) might have setup the preview based on a wrong branch.

@zhengmar3 @yarivus My mistake - this is the correct site - https://corda5.preview.docs.r3.com/en/platform/corda/5.0/developing-applications/api/application.html

Works ;)


invert_percentage

Hi, great work.
invert_percentage changes the position of the cover image, not the displayed percentage.
If the shutter is open (100%) and you turn on the invert_percentage option, the shutter will be shown in the card as closed but still 100%.
In my case 0% is open and 100% closed. :)
Thanks

For me as well.

Hello,
I'm not sure to understand what you want. The percentage which is displayed is the percentage provided by your shutter.
So in your case, if the shutter is closed when it's 100%, then you should see a closed shutter and see the percentage 100% when you set invert_percentage: true. Isn't it the case ?

Hello,
all that is right and fine. But my shutter is showing 100% in case the shutter is open. As you said I would expect it to be the other way around. So it would be good to have an option to completely invert the signal provided by the shutter. Or even only a part.

So if I understand well, you'ld want a new parameter to also invert the percentage displayed ? Even if it's the opposite of the real value of your shutter ? I don't really understand why this would be needed, because it would add confusion.
Imagine I do that, in your case you'ld see "100%" when it's opened, but the real value would be "0%". So if you want to try in the Developer Tool of Home Assistant, it could add confusion because you could write "100%" while thinking it'ld open the shutter, but instead it'ld close the shutter.

No answer since more than 1 month. I close this issue.

The problem with this is that there are people where the a complete closed shutter is 100% and a complete opend shutter is 0%.
By inverting the percentage  ("=true") a complete closed shutter is displayed with 100% which is correct. But the image that is shown in this case is a complete opend shutter. inverting percentage should change the view of the percentage and not the image in animation.


remove unnecessary output

It will drop this console.logs
npm WARN invalid config registry=undefined
npm WARN invalid config Must be a full url with 'http://'
/Users/sbmaxx/.nvm/versions/node/v4.5.0/lib/node_modules

tnx. v4.3.3 was published


Fixed clicking on Find/Replace doesn't set the focus on the text field

Fixed clicking on Find/Replace doesn't set the focus on the text field.
Fixed #20980

Thanks!


Show session time in local timezone

Session time should be shown in the local time zone.

Have you set: https://github.com/milesmcc/shynet/blob/master/TEMPLATE.env#L38

Thanks, @rc1!


Add isCollapsed

Why?

Expand multiple container logs looks too long for me.

I would also like such option, the output is kind of verbose.


fix: fix a bug for terraform installation

What type of PR is this?

/kind bug
What this PR does / why we need it:
This PR fixes a bug of installing Terraform CLI.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 58.448%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/engine/runtime/terraform/install.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
7327

Relevant Lines:
12536

💛  - Coveralls


Add French translation

Thanks @MaloJaffre

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/401/
Hooray Jenkins reported success with all tests good!

I haven't finished yet (only half done).

Okay, merged this, realized there were still a few new commits in Weblate, merged it all and cleaned up :-)
Thanks @MaloJaffre !
@msteiger I'll try to remember to first push anything pending from Weblate the app before trying to update the weblate the branch next time - hopefully all is well for now .. :-)


Fix a typo in docker/daemon/state.go

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "typo" git@github.com:YanFeng-Adam/docker.git somewhere
$ cd somewhere
$ git rebase -i HEAD~3
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

please squash your 3 commits down to one.

LGTM


docs: add reference to supported Spanner driver

Adds a reference to a supported version of a database/sql driver for Cloud Spanner so interested users can find it.

@rakyll PTAL (I'm not allowed to add any reviewers to the PR)

@rakyll Friendly ping. Would you be willing to consider this?


KAFKA-4944. Fix an "unread field" findbugs warning in streams examples

retest this please

retest this please

Not sure, @miguno can you check.


Maximum call stack  size exceeded

PS C:\Users\DELL\Desktop\React> npx create-react-app myapp
Maximum call stack size exceeded
PS C:\Users\DELL\Desktop\React>
Now, no any myapp folder created. how can I solve these issues? please help me.

Please fill in the issue template.

Duplicate of https://github.com/facebook/create-react-app/issues/5328, there seems to be some workarounds in https://github.com/facebook/create-react-app/issues/5328#issuecomment-429148117


Base64Bytes with newlines

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Under the hood, Base64Bytes use standard library base64.encodebytes that inserts a newline after every 76 bytes of output, as well as ensuring that there is a trailing newline. This makes some base64 decoders (e.g. iOS) sad. Fix is to use b64encode() instead of encodebytes().
It is easy to work around by using a custom encoder, but IMHO there's no point in adding the newlines.
Example Code
import os
from base64 import b64encode
from pydantic import Base64Bytes, BaseModel

class Foo(BaseModel):
    random: Base64Bytes

print(Foo(random=b64encode(os.urandom(32)).decode()).model_dump_json())

Python, Pydantic & OS Version
pydantic version: 2.6.4
        pydantic-core version: 2.16.3
          pydantic-core build: profile=release pgo=true
                 install path: /Users/jakob/Library/Caches/pypoetry/virtualenvs/uas-YdoMfBSh-py3.12/lib/python3.12/site-packages/pydantic
               python version: 3.12.2 (main, Feb  6 2024, 20:19:44) [Clang 15.0.0 (clang-15<IP_ADDRESS>.5)]
                     platform: macOS-14.4-arm64-arm-64bit
             related packages: typing_extensions-4.10.0 fastapi-0.109.2 pydantic-settings-2.2.1
                       commit: unknown

@jschlyter,
I'm concerned that implementing a change under the hood would cause breaking changes for existing users, and if it's easy enough to customize, I think this kind of change might be best in user code...

class Base64EncoderSansNewline(Base64Encoder):
    @classmethod
    def encode(cls, value: bytes) -> bytes:
        return base64.b64encode(value)

Base64Bytes = Annotated[bytes, EncodedBytes(encoder=Base64EncoderSansNewline)]

The questions is; who really want base64 with embedded newlines?

Right, I understand the sentiment, but given that it's easy to change on the user side of things with a custom encoder, I don't think this is a change we're going to make in the near future. Closing for now. Thanks for adding your workaround for others to reference!

Can this please be added to the documentation somewhere? I thought I'm doing something wrong when newlines showed up in my serialized objects, which is very surprising. Could a non-newline version be added to pydantic to simplify onboarding of new users?

See https://docs.pydantic.dev/2.8/api/types/#pydantic.types.EncodedBytes for some more helpful docs!

Hey @sydney-runkle!
I can understand why this can't be immediately changed for compatibility reasons, but I wanted to add another datapoint here: this recently bit us pretty badly on PyPI's rollout of PEP 740, which specifies base64-encoded objects that would have been silently mis-encoded with the current Pydantic APIs.
We got lucky and happened to catch the unexpected behavior right before it landed in production, but had we not caught it we would have had to do data migrations against PyPI itself for all projects that currently using Trusted Publishing to publish to PyPI, which includes pydantic itself.
So, I suppose I have two questions here:

Will Pydantic consider making a behavioral change to this API in a future major release?
Could the docs be updated to additionally warn users (in scarier language than is currently present) that {encode,decode}bytes don't do "base64" in the sense that users conventionally understand? The docs currently only mention differences in the behavior w/r/t "URL safe" base64, which might lead users to believe that the current types are fine when they aren't for all use cases.

If (2) sounds appropriate, I'm happy to contribute those docs 🙂

Hey folks, thanks for following up. Will chat with our team tomorrow and get back to you.


Support for the rename based on the whole project directory

Elevator Pitch
Any plan for the rename based on the whole project directory including file rename (more like a refactoring feature). It's better to support both .ipynb and .py
Motivation
current rename function works within one file either .ipynb or .py. It's not that convenient when I want to rename a .py module or function/class in this module. Rather, I need to change all related reference.
Design Ideas
current IDE like vscode or pycharm has supported this. for example below video.
https://github.com/jupyter-lsp/jupyterlab-lsp/assets/97007177/4691c4b4-0ee1-4b91-bd5b-10bd9517df60

That would be nice. I often think about it. Technically what is needed is support for workspace-wide edits and some UI. Alternatively it could be achieved by resolving https://github.com/jupyterlab-contrib/search-replace/issues/79 and using that extension.

Additionally, other refactoring feature is missing, like extract variable or method, etc, and not very convenient when writing a project using jupyterlab. Maybe we can leverage more function in rope.
Also, Thanks for the alternative proposal of using search-replace. it's really great, but it different from refactoring feature (e.g. it cannot handle file rename and extraction, etc). BTW, search-replace cannot be used in jupyterlab-desktop (issue opened here)


Tutorial for annealed flow transport

Annealed Flow Transport Monte Carlo, Arbel etc. can be implemented using a few combinators as in coix.algo. A special aspect of this algorithm is its use of "deterministic" kernels rather than "stochastic" kernels as in inference combinators.
It would be great if we have a concrete example/tutorial showing how to implement targets and flows. Here, the role of flows is to perform a transformation and add a corresponding log_det_abs_jacobian factor.

hi there, would like to try and give this a try

That would be awesome, @conorhassan!


added certmagic.HTTPSWithSelfSigned()

fixes #25
HTTPSWithSelfSigned() is the same like HTTPS() but creates and uses selfsigned certificates instead of letsencrypt.
migrated selfsigned.go from caddy

closed because #27 seems a better solution


Announcement: ScribanExpress

I've created ScribanExpress.
This has given us significant performance boost (more below).
It Uses Scriban to Generate an AST then converts those to expressions. I've generally tried to keep to the Scriban behaviour, most differences are due to features not being implemented... yet. I'm keen to get feedback, to help prioritise new features. Any opinions are welcome.
Preview releases, are up on nuget. We are using this in production.
ScribanExpress Nuget
There is a Demo, and benchmarks in the repo.
We get huge performance improvements.
For the products example we see about an 8 times improvement
|             Method |       Mean |     Error |    StdDev | Ratio |
|------------------- |-----------:|----------:|----------:|------:|
| ExpressProducts500 |   163.2 us | 0.8232 us | 0.6874 us |  0.13 |
| ScribanProducts500 | 1,273.9 us | 3.0741 us | 2.7251 us |  1.00 |

For our use cases, lots of little simple templates
We are seeing up to 350x perf improvement in these cases
|             Method |         Mean |      Error |    StdDev | Ratio |
|------------------- |-------------:|-----------:|----------:|------:|
| 'Express Hello 50' |     9.684 us |  0.0548 us | 0.0512 us | 0.004 |
| 'Scriban Hello 50' | 2,752.831 us | 10.6530 us | 9.9648 us | 1.000 |

Notable points

The first render of a template will invoke an expression compile, this will be slow.... then super fast.
ScribanExpress currently caches on template string, so probably not suited for huge templates right now.
One feature is the ability to call methods on any object, this is a slight departure from Scriban.
Security (especially around calling methods) has not been applied, i.e you can call anyobject.GetType

@xoofx  Do you plan to adapt some of these optimizations?

@xoofx Do you plan to adapt some of these optimizations?

Personally no. AOT is a different approach that I evaluated when designing Scriban, but I would have had to duplicate a significant chunk of the runtime to make it 100% compatible. It would be quite some work, and beyond PR mgt and small bug fixes, I don't plan to work on any substantial work on Scriban.


CSS for debug icons applies to TreeItem inline commands

Type: Bug
In a custom extension that contributes some commands with their icon set to debug-specific codicons, e.g. debug-step-back, contributing those commands as inline to a TreeItem looks like this:

The TreeItem inline commands gets the following style for the ul > li > a element:
.monaco-workbench .part > .title > .title-actions .action-label.codicon.codicon-debug-step-back, .monaco-workbench .codicon.codicon-debug-step-back {
    color: #75beff;
}

Afaict, this comes from https://github.com/microsoft/vscode/blob/d4237bd8d548a45e2759f3b3fe4fb75dfeb677a9/src/vs/workbench/contrib/debug/browser/debugColors.ts#L348
This is minor UI issue, but I always wondered why 3/5 of my icons were blue.
VS Code version: Code - Insiders 1.87.0-insider (a0b90ac5e04ea9a0733b68062253a469994e170e, 2024-02-22T05:48:26.543Z)
OS version: Windows_NT x64 10.0.18362
Modes:
Remote OS version: Linux x64 4.18.0-372.19.1.el8_6.x86_64

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.81GB (5.99GB free)

Process Argv

Screen Reader
no

VM
0%

Item
Value

Remote
SSH: sestolabccv8

OS
Linux x64 4.18.0-372.19.1.el8_6.x86_64

CPUs
Intel(R) Xeon(R) Silver 4216 CPU @ 2.10GHz (8 x 2095)

Memory (System)
31.21GB (15.26GB free)

VM
100%

Item
Value

Remote
SSH: sestolabccv8

OS
Linux x64 4.18.0-372.19.1.el8_6.x86_64

CPUs
Intel(R) Xeon(R) Silver 4216 CPU @ 2.10GHz (8 x 2095)

Memory (System)
31.21GB (15.26GB free)

VM
100%

Just a side note on something I noticed possibly semi-related or same root issue.
If you have a treeView with expandable treeItems. And those treeItems have a set resourceUri which sets (via the query params) a tooltip, badge and custom colorId. When the focused element in the tree changes between the row items the badge color goes from default (white) to the colorId on focus. It seems to bleeding over from the higher level element.
Please note: this was just a quick thing I noticed, haven't dug in yet, so might be a red-herring. Going to do a test implementation and see if I can report back with specifics.

I created https://github.com/tylor-metrics/tree-item-highlighting to minimally reproduce this issue.
The issue I am having (which I think is the same) is that the icon left of the text section 2 is turning yellow on focus, when I think it should not.


Tl detector test version

Can someone check this file and pull it in!
Verify this file by looking at console output and simulator

You accidentally targeted the upstream repository at Udacity.


Remove warning against Html.Lazy in README

There should not be any issue using this package along with Html.Lazy.

Neverming. Someone already did it :)


custom plugin repo xml

Add a custom plugin repository XML file in order to stay up-to-date.

I didn't know we can do this 😅
Thanks for your contribution!


CSV Uploader Enhancements for Errors, Warning, & Quality Filters

Problem Statement
As we begin to release the CSV Uploader to pilot participants, we need to enhance the feature based on our findings.

@drew-usds @inkandsound Do we have enough feedback to finalize the discussion for these stories today in refinement and point?
cc: @rachelhanster

Closing this epic. Transfer product backlog tickets to Simplereport for product manager's future consideration. cc @timothybest-usds


Out of date project? Unable to setup demo

I was trying to setup the demo locally but it does not work:
root@alex-OMEN:/opt/company/siam-mot# pip3 install -r requirements.txt
...
root@alex-OMEN:/opt/company/siam-mot# python3 demos/demo.py --demo-video ../long_video.mp4 --track-class  person_vehicel -dump-video True
Traceback (most recent call last):
  File "demos/demo.py", line 5, in <module>
    from demos.demo_inference import DemoInference
ModuleNotFoundError: No module named 'demos'

Then I copied demo.py to the root of project and try again :
root@alex-OMEN:/opt/company/siam-mot# python3 demo.py --demo-video ../long_video.mp4 --track-class  person_vehicel -dump-video True
Traceback (most recent call last):
  File "demo.py", line 5, in <module>
    from demos.demo_inference import DemoInference
  File "/opt/company/siam-mot/demos/demo_inference.py", line 10, in <module>
    from maskrcnn_benchmark.structures.bounding_box import BoxList
ModuleNotFoundError: No module named 'maskrcnn_benchmark'

I installed maskrcnn according to https://github.com/facebookresearch/maskrcnn-benchmark/blob/main/INSTALL.md
And the current error is :
root@alex-OMEN:/opt/company/siam-mot# python3 demo.py --demo-video ../long_video.mp4 --track-class  person_vehicel -dump-video True
Traceback (most recent call last):
  File "demo.py", line 5, in <module>
    from demos.demo_inference import DemoInference
  File "/opt/company/siam-mot/demos/demo_inference.py", line 11, in <module>
    from maskrcnn_benchmark.utils.checkpoint import DetectronCheckpointer
  File "/usr/local/lib/python3.8/dist-packages/maskrcnn_benchmark-0.1-py3.8-linux-x86_64.egg/maskrcnn_benchmark/utils/checkpoint.py", line 7, in <module>
    from maskrcnn_benchmark.utils.model_serialization import load_state_dict
  File "/usr/local/lib/python3.8/dist-packages/maskrcnn_benchmark-0.1-py3.8-linux-x86_64.egg/maskrcnn_benchmark/utils/model_serialization.py", line 7, in <module>
    from maskrcnn_benchmark.utils.imports import import_file
  File "/usr/local/lib/python3.8/dist-packages/maskrcnn_benchmark-0.1-py3.8-linux-x86_64.egg/maskrcnn_benchmark/utils/imports.py", line 4, in <module>
    if torch._six.PY3:
AttributeError: module 'torch._six' has no attribute 'PY3'

What am I doing wrong?
Is this project still actual?

Could you solved it?


Access denied for a pull request

Hello, I tried to make a pull request but the access is denied. I created a branch and pushed from it. Is there some specific permissions I need from you ?

You'll want to make a fork and use that to open a pull request. Take a look at this guide:

You'll want to make a fork and use that to open a pull request. Take a look at this guide:
https://github.com/gabrieldemarmiesse/getting_started_open_source

Okay I will do this. Thank you


Added change listener onInputChange

Added support for adding a change listener that will fire when value of input field is changed. This makes it possible to use the input field's value from outside autosuggest component.

@hypereng Thanks for the Pull Request!
First of all, I'm curious to know why do you need to know when the autosuggest input field value changes. Do you have some kind of dependency between the autosuggest field and other element on the page?
Regarding the implementation itself:
Wouldn't it be nicer to use the existing inputAttributes API rather than adding an additional configuration option (onInputChange)?
let inputAttributes = {
  onChange: function(...) {
    ...
  }
};

<Autosuggest inputAttributes={inputAttributes} suggestions={getLocations} />

At the moment, this is not possible because Autosuggest sets its own onChange() handler on the input field. But, we could easily fix that. I'm just trying to avoid adding extra config fields if not absolutely necessary.
What do you think?

First of all, thanks for looking at my pull request.
The reason I want to be able to listen to input value changes is that I want to implement a "save as you type" method that auto-saves any changes.
I agree with you that my change was not consistent with the component's API.
Now I have introduced a new prop inputEventAttributes that allow a parent component to add an event listener on the input field.
What do you think of this change?

@hypereng

What value do we get by introducing inputEventAttributes rather than using inputAttributes?
Do you really need onBlur and onKeyDown to implement "save as you type"? I'm just trying to keep the API to minimal, and avoid adding new config options for the sake of adding them. If there is a real use case, I'll be happy to add them.

IMO a use should be able to pass onChange, onBlur and onKeyDown to inputAttributes. These are normal props for an input so if you overwrite them you should provide a way to access them.

inputAttributes.onChange added to v1.12.0. See docs.


feat: add counter component to the header for displaying currently selected/yanked items

Thank you for the PR!
I just want to add an API to get the selected file, but I realized an issue - currently, the selection state is specific to the directory, meaning it can only get the selection state of the current directory.
But it would make more sense to get all selected files for display. We will add support for selecting across directories in the future.
So I need to do some refactoring to modify it to a tab-specific global selection state. I'll continue reviewing this PR after the refactoring is done :)

Ah I didn't catch that actually. Best of luck then, we can come back to this later

Would it be possible to write not only the number of selected/yanked items, but also the size of all selected items?

Would it be possible to write not only the number of selected/yanked items, but also the size of all selected items?

That may be better as a plugin or at least a separate feature, @sxyazi what do you think?

Yep it should be work as a UI plugin and not included in this PR.

Should be ready for you to review when you have the time @sxyazi , pretty small PR

Yep those changes seem good. I do also like hiding the tab component if there's only one.

Alright, I think there are no more problems here, let us merge it!

@Brixy Thanks for the reporting, should also be fixed in https://github.com/sxyazi/yazi/pull/690


Fix when applicationId ≠ package name

for using applicationIdSuffix in different flavors or when applicationId is different from package name in AndroidManifest #14

This PR can be improved: https://github.com/ncnlinh/react-native-config/pull/3

Thanks for the help @mehcode @fabriziomoscon

I tested this on Android and for what I can see it works great. Please consider for merge

Yep.. we're using it over here too :+1: (on android and iOS)

You folks rock! :+1: thanks so much, released in 0.0.7.

Thank you @pedro, I've been using this lib since you introduced in in RN Meet-up SF. Amazing work!

wow, so glad to hear! :}


Could not resolve URL for hyperlinked relationship using view name "profile-detail". You may have failed to include the related model in your API, or incorrectly configured the lookup_field attribute on this field. the above error is  triggered when i try to access oscar basket/add-product/ api

Found a bug? Please fill out the sections below.
Issue Summary
A summary of the issue.
Steps to Reproduce
It's essential that you provide enough information for someone else to replicate the problem you're seeing. Simply describing something that's broken on your current project is not enough!

(for example) Form the catalogue app with ./manage.py oscar_fork_app catalogue myproject/
Edit models.py as follows...
...

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
Technical details

Python version: Run python --version.
Django version: Look in your requirements.txt, or run pip show django | grep Version.
Oscar version: Look in your requirements.txt, or run pip show django-oscar | grep Version.

Hi there. We use Github only for managing bugs and feature requests, and not for support. We recommend that you ask for help on Stack Overflow using the django-oscar tag, or joining our slack workspace and asking a question there.
The profile-detail view that you reference in the title is not a view that Oscar provides, so it seems likely that there is an issue with your code, which you'll need to share on one of the above channels for someone to be able to help.
Thanks.


[D3] - Symbol size/type API take in accessor functions

Usage here: https://github.com/palantir/plottable/pull/1708/files#diff-97b29790f2434e583db79759c52171ecR34

@bluong thanks mate!


Add TransactionsHandle function to retrieve transactions from certain peers

Describe the feature
Currently we only have commands for propagating transactions
https://github.com/paradigmxyz/reth/blob/2b71f0f6b02865a0f720254f4483db9d5b7abd9c/crates/net/network/src/transactions.rs#L1019-L1021
we could use some functions that request transactions from peers.
this should simply use the sender to the peer
https://github.com/paradigmxyz/reth/blob/2b71f0f6b02865a0f720254f4483db9d5b7abd9c/crates/net/network/src/transactions.rs#L1010-L1011
and send a request which is then awaited by the Handle:

manager receives command (peer, hashes)
sends the request through the peer's channel
sends the response channel back to the handle
handle awaits transactions response

@smatthewenglish
Additional context
No response

this sounds like a fun feature to work on. I'd be happy to pick it up if you think it's a good fit! 🤖


[WIP] Include YT chat when embedding live YT video

@danielramirez0 this looks great! I'm happy to merge if you think it's ready, just wanted to check to see if you had other things you wanted to do before I merge.

@BrianMMcClain I think we are good to go! I don't have anything else to add right now.


Flash an SD card

Use https://github.com/ovgu-FINken/turtlewriter to flash an SD.
Use 32-bit ubuntu (18.04).

rsync is a dependency


allow unicode (utf8) and everything in values

Allow everything in values, including valid and invalid utf-8 data.

lgtm. @waisbrot any objections?
What was the motivation for the change ?
do you wish this to be incorporated into erlcloud library?

The unit test added appears to fail in travis. This should at the very least be made to work prior to merging.

The unit test failure was because Travis runs Rebar2 and Rebar2 and Rebar3 appear to have different working directories, so file:read_file("tests/test.txt") fails under Rebar2. I just pasted in the results as a work-around to that (trying out that new semi-public PRs feature; I hope you don't mind @brigadier).
Looks like there's still a failure. It reads like R16 just doesn't handle UTF8 the same way as later versions.
This could be useful if someone had multiple AWS profiles and they gave them UTF8 names (seems unlikely pot possible). It's more likely to be useful if eini is needed as a general INI-parser library.
In principle, UTF8 everywhere is great and we should do it. In practice, I feel like this won't affect many users.
@motobob a nice thing to add to the roadmap would be planned support for various older versions of Erlang. If we don't need to support R16, I'd like to drop it from our tests and accept this PR.

Wait for a while, I'll rewrite the tests to be compatible with R16

All tests have passed successfully now.

What was the motivation for the change ?

A configurable welcome message in one of my projects, a certain user asked for non-ascii one.

do you wish this to be incorporated into erlcloud library?

I don't care tbh.

Thanks!

cool, Thanks @brigadier!


Captcha screen question

Hi, have a question about the Captha Screens because I didn't get any yet. The Captcha Screens still show right? I will be redirected to the puzzle page as usual?

Yes , if Captcha shows you will be redirected to the puzzle page which you have to manually solve.


Incorrect hint location in trace

@IljaZakharov reports that when running the following definition:
non-rec-function.k:
module NON-REC-FUNCTION-SYNTAX
  syntax Foo ::= "a"
               | bar(Foo)
               | baz(Foo)
               | id(Foo) [function, total]
endmodule

module NON-REC-FUNCTION
  imports NON-REC-FUNCTION-SYNTAX
  rule [id-rule]: id(X:Foo) => X
  rule [bar-rule]: bar(baz(X:Foo)) => id(id(bar(X)))
endmodule

With input bar(baz(a)), that:

The location for the first simplification event is shorter than we expect it. It points to id(id(bar(X))) instead of its subterm id(bar(X)).

As far as I can tell, the location reported to me by the backend's parser is in fact correct and matches my understanding of the format.
The relevant section of the trace is:
...
rule: 98 3
  Var'Unds'DotVar0 = kore[61]
  Var'Unds'DotVar1 = kore[10]
  VarX = kore[48]
function: Lblid'LParUndsRParUnds'NON-REC-FUNCTION-SYNTAX'Unds'Foo'Unds'Foo{} (0:0:0)
  function: Lblid'LParUndsRParUnds'NON-REC-FUNCTION-SYNTAX'Unds'Foo'Unds'Foo{} (0:0:0:0)
    arg: kore[119]
  rule: 101 1
    VarX = kore[119]
  arg: kore[119]
rule: 101 1
  VarX = kore[119]
config: kore[288]

First, axiom 98 applies (bar-rule). The backend then enters the code generation step where it constructs the RHS term corresponding to id(id(bar(X))), given some value for X. The logic for that - heavily simplified - looks something like this:

Is the symbol at the top of the RHS a function symbol?
It is (id(_)). Now:

Emit a function call begin event into the trace (0xDD word, name, location info for this call - 0:0:0)
Collect each argument so the actual call to id can be made.
The first argument is id(bar(X)). Is the symbol at the top a function symbol? at this point we're in a recursive call stack

Emit a function call begin event into the trace (as above, location info is now 0:0:0:0)
Collect each argument so the actual call to id can be made.
The first argument is bar(X).
Call the evaluator for id with the value of bar(X). there's some logic to construct a bar term here; not important
Emit a function call end event (0x11 word)
Emit a "rewrite applied" event for rule 101 (id-rule).

We now have the result of id(bar(X)), so we can call the evaluator for id with that value.
Emit a function call end event.
Emit a "rewrite applied" event for rule 101.

Top level rewrite end reached; emit a configuration.

@Baltoli This behaviour seems like it occurs for non-tail-recursive functions. We have a different behavior for tail-recursive functions, where the subsimplification happens after the sentinel. Can we keep the two behaviours the same, and even drop support for nested events?

This is the reverse-tree-int example, that we discussed before:
Init config: [kore]
Rule 130 [label: TREE-REVERSE-INT.rev-one-five] [loc: 22,27] [substs: 1]
  Var'Unds'DotVar0 = [kore]
Function: Lblreverse'LParUndsRParUnds'TREE-REVERSE-INT-SYNTAX'Unds'Tree'Unds'Tree{} @ 0:0:0
  Term: [kore]
Rule 216 [label: TREE-REVERSE-INT.rec-case] [loc: 19,24] [substs: 2]
  VarT1 = [kore]
  VarT2 = [kore]
Function: Lblreverse'LParUndsRParUnds'TREE-REVERSE-INT-SYNTAX'Unds'Tree'Unds'Tree{} @ 0:0
  Term: [kore]
Rule 215 [label: TREE-REVERSE-INT.base-case] [loc: 18,24] [substs: 2]
  VarX = [kore]
  Var'Unds'Gen0 = [kore]
Function: Lblreverse'LParUndsRParUnds'TREE-REVERSE-INT-SYNTAX'Unds'Tree'Unds'Tree{} @ 0:1
  Term: [kore]
Rule 215 [label: TREE-REVERSE-INT.base-case] [loc: 18,24] [substs: 2]
  VarX = [kore]
  Var'Unds'Gen0 = [kore]
Config: [kore]

The semantics for the example above:
module TREE-REVERSE-INT-SYNTAX

    imports INT-SYNTAX

    syntax Tree ::= node(Tree, Tree) | Int
    syntax Tree ::= reverse(Tree) [function, total]
    syntax KItem ::= "#reverse-one"
    syntax KItem ::= "#reverse-one-five"
    syntax KItem ::= "#next"

endmodule

module TREE-REVERSE-INT

    imports INT
    imports TREE-REVERSE-INT-SYNTAX

    rule [base-case] : reverse(X:Int) => X
    rule [rec-case] :  reverse(node(T1, T2)) => node(reverse(T2), reverse(T1))

    // Test simple injections in functions
    rule [rev-one-five] : <k> #reverse-one-five => reverse(node(1:Int, 5:Int)) </k>

    // Test reduction of nested injections
    rule [rev-one] : <k> #reverse-one => reverse(1) </k>

endmodule

The conclusion that we came to in today's meeting is that the proof trace preprocessing code will handle the current nested structure by understanding the order in which the backend emit function call begin events; there are no changes to the backend required at this time and so we can close this issue.


Fix MLflow Recipes links

@mingyu89 now that Mlflow 2.0 is out, let me know if I should update the requirements.txt reference to MLflow.

Closing this in lieu of merging https://github.com/databricks/mlops-stack/pull/24


.Net Core Bundles

Hello, It doesn't looks like it but does it or will you add support for .Net Core Bundles in this Engine ?

If it isn't loadable by AsmResolver then it will not be loadable by this. I am unsure as to what exactly you are referring to by "bundles".

If it isn't loadable by AsmResolver then it will not be loadable by this. I am unsure as to what exactly you are referring to by "bundles".
I mean single-file executable / https://www.youtube.com/watch?v=QJg1ptS0At0

Thank you for more information on the subject, unless AsmResolver supports this there is no support from this project.
I don't see why you couldn't protect the .NET files before they are bundled though.

Thank you for more information on the subject, unless AsmResolver supports this there is no support from this project.
I don't see why you couldn't protect the .NET files before they are bundled though.

I'm pretty sure without a VS extension you can't protect before bundling..
Also wanted to know because apparently engines from protector like DNR support it and I would've had liked to know how exactly ! I'm curious :)


support for null safety

Closed by #33
Sorry for the delay: last months were very difficult for me


don't forget raw(sum_of_prod)

cf title

also move div//mul pedantic to arithmetic.hpp

Fixed in a2d15550


Import System.IO.Pipes.PipeStream from CoreFX for Gsoc 2017

Thanks for your contribution but we won't need this here anymore with our .NET 5 work.


Look up metrics desired by Graphics team

capture what other metrics would be relevant to this team and see if we can include in phase 1

i.e.: quotes from comments mentioning the graphic

@hampelm, here's a summary of insights from interviewing Aly last year:

having to manually add a story to start tracking is inconvenient (stopped using Carebot because she'd forget to do this)
metric: how many comments the story got
metric: how many comments were moderated, as a sense of how heated/ridiculous the conversation got (short of sentiment analysis)
carebot: would also like it to be more transparent about how old/recent the data is; know traffic numbers are several hours behind, but i don't have a good sense of how behind
graphics projects separate from other types of stories so comparisons are apples to apples.
metrics: number of actual graphics impressions (graphics appearing in the user's viewport)
metrics: pick-up by member stations, reach (distribution)
success/failure for a graphics story: pitiful traffic and/or low engagement :) - compared to how other stories did that day, or other graphics of that type (will often mentally judge vs. how other stories in the top 20 for a given day did)
success/failure for a graphics story: personal satisfaction - was it interesting to work on — interesting subject, a new technical skill, an artistic treatment — or were the folks involved particularly happy with it
metrics: graphics mentions in comments; do people cite the graphic as having helped them understand something

Thanks, this is super helpful to know.
Clearly stating the data age is feedback is something we should def. pay attention to.
Maybe rev 2 or 3 can include some sort of comment measure.


Add adjustable padding for nested nodes

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[x] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Current behavior
Currently, nested nodes will have default padding of 50px for all sides
'elk.padding': '[left=50, top=50, right=50, bottom=50]',
Expected behavior
I have some custom node contents and nested nodes are rendered on top of the contents. Currently, I cannot adjust the position of the nested nodes dynamically and therefore I would like to set the padding within the nodeProps.
I will create a PR for this soon, just wanted to have a corresponding ticket.

You can pass in custom options - https://github.com/reaviz/reaflow/blob/master/stories/Layouts.stories.tsx#L115

That's right. In my example I want to provide a padding based on the type of the node. Therefore providing an overall padding to the canvas and hence to every node the same padding is not sufficient for my use case.


[BUG] JSON deserialization error when creating a new AppCertificate

Library name and version
Azure.ResourceManager.AppService 1.0.0
Describe the bug
AppCertificate fails with a JSON deserialization error when creating a new AppCertificate, Azure Portal shows that the certificate was created successfully and Azure Resource Manager API also returns success, but the SDK internal deserialization fails.
Expected behavior
We expect the certificate to be created successfully and the information we have created to be returned.
Actual behavior
JSON deserialization error. The resource creation itself succeeds.
Stacktrace
System.ArgumentException: Value cannot be an empty string. (Parameter 'resourceId')
   at Azure.Core.Argument.AssertNotNullOrEmpty(String value, String name)
   at Azure.Core.ResourceIdentifier..ctor(String resourceId)
   at Azure.ResourceManager.AppService.AppCertificateData.DeserializeAppCertificateData(JsonElement element)
   at Azure.ResourceManager.AppService.CertificatesRestOperations.CreateOrUpdateAsync(String subscriptionId, String resourceGroupName, String name, AppCertificateData data, CancellationToken cancellationToken)
   at Azure.ResourceManager.AppService.AppCertificateCollection.CreateOrUpdateAsync(WaitUntil waitUntil, String name, AppCertificateData data, CancellationToken cancellationToken)

Reproduction Steps
Repro code
using System.Security.Cryptography.X509Certificates;

using Azure.Identity;
using Azure.ResourceManager;
using Azure.ResourceManager.AppService;
using Azure.ResourceManager.Resources;

var armClient = new ArmClient(new AzureCliCredential());

var subscription = await armClient.GetDefaultSubscriptionAsync();

ResourceGroupResource resourceGroup = await subscription.GetResourceGroupAsync("rg-cert-test");

WebSiteResource webSite = await resourceGroup.GetWebSiteAsync("app-cert-test-01");

var certificates = resourceGroup.GetAppCertificates();

var certificate = new X509Certificate2(File.ReadAllBytes(@"cert-test.pfx"), (string?)null, X509KeyStorageFlags.Exportable);

var pfx = certificate.Export(X509ContentType.Pfx, "P@ssw0rd");

// throw exception
var result = await certificates.CreateOrUpdateAsync(Azure.WaitUntil.Completed, "cert-test-01", new AppCertificateData(webSite.Data.Location)
{
    Password = "P@ssw0rd",
    PfxBlob = pfx,
    ServerFarmId = webSite.Data.AppServicePlanId
});

Console.WriteLine(result.Value.Id);

Environment
No response

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@shibayan  can you debug mode of SDK and share us the raw JSON responses when you are calling createorupdate method? It appears the resourceId is missing in the returned json.
You can enable debug mode by set up the option parameters when you are construct ArmClient
options.Diagnostics.IsLoggingContentEnabled=true;
options.Diagnostics.IsLoggingEnabled=true;

@xboxeer The JSON response of the method call that raises the exception is retrieved.
[Informational] Azure-Core: Response [026d2932-7e80-4938-bcaf-9a4b2a0d7c0c] 200 OK (01.5s)
Cache-Control:no-cache
Pragma:no-cache
Warning:REDACTED
Strict-Transport-Security:REDACTED
x-ms-request-id:0811116a-12a6-4e15-a19c-96f96c7b0acf
Server:Microsoft-IIS/10.0
X-AspNet-Version:REDACTED
X-Powered-By:REDACTED
x-ms-ratelimit-remaining-subscription-writes:REDACTED
x-ms-correlation-request-id:REDACTED
x-ms-routing-request-id:REDACTED
X-Content-Type-Options:REDACTED
Date:Fri, 21 Oct 2022 06:16:11 GMT
Content-Length:826
Content-Type:application/json
Expires:-1

[Verbose] Azure-Core: Response [026d2932-7e80-4938-bcaf-9a4b2a0d7c0c] content: {"id":"/subscriptions/6f43ad45-521d-4bd7-9d01-f95a290041fe/resourceGroups/DaruTest-RG/providers/Microsoft.Web/certificates/cert-test-01","name":"cert-test-01","type":"Microsoft.Web/certificates","location":"centralus","tags":{},"properties":{"password":null,"friendlyName":"","subjectName":"acmebot.daruyanagi.work","hostNames":["acmebot.daruyanagi.work"],"pfxBlob":null,"siteName":null,"selfLink":null,"issuer":"R3","issueDate":"2022-10-15T14:04:20+00:00","expirationDate":"2023-01-13T14:04:19+00:00","thumbprint":"3D4B30B72A3120361FE77F739D26AB803E375CF9","valid":null,"toDelete":null,"cerBlob":null,"publicKeyHash":null,"hostingEnvironment":null,"hostingEnvironmentProfile":null,"keyVaultId":"","keyVaultSecretName":"","webSpace":"DaruTest-RG-CentralUSwebspace","serverFarmId":null,"tags":{},"resourceGroup":"DaruTest-RG"}}

Unhandled exception. System.ArgumentException: Value cannot be an empty string. (Parameter 'resourceId')
   at Azure.Core.Argument.AssertNotNullOrEmpty(String value, String name)
   at Azure.Core.ResourceIdentifier..ctor(String resourceId)
   at Azure.ResourceManager.AppService.AppCertificateData.DeserializeAppCertificateData(JsonElement element)
   at Azure.ResourceManager.AppService.CertificatesRestOperations.CreateOrUpdateAsync(String subscriptionId, String resourceGroupName, String name, AppCertificateData data, CancellationToken cancellationToken)
   at Azure.ResourceManager.AppService.AppCertificateCollection.CreateOrUpdateAsync(WaitUntil waitUntil, String name, AppCertificateData data, CancellationToken cancellationToken)
   at Program.<Main>$(String[] args) in C:\Users\shibayan\source\repos\ConsoleApp55\ConsoleApp55\Program.cs:line 36
   at Program.<Main>(String[] args)

@shibayan , the exception is because the keyVaultId in json is empty and SDK is trying to use it to generate a resourceidentifier for keyvault, hence we throw this exception.
So in this case, is your certificate not bind to a key vault? if so, you can use Azure SDK Http Pipeline to by pass this exception

Refer to this code to write a policy: https://github.com/Azure/azure-sdk-for-net/blob/91ab305069bf3b4e14f1f206b31cf2aa6d2d64d4/common/ManagementTestShared/Redesign/ResponseNullFilterPolicy.cs
Instantiate the policy and register that policy in ArmClient through ArmClient option, you can refer the code here: https://github.com/Azure/azure-sdk-for-net/blob/91ab305069bf3b4e14f1f206b31cf2aa6d2d64d4/common/ManagementTestShared/Redesign/ManagementRecordedTestBase.cs#L89

You can use policy to check the raw response and make changes to the raw json to have the line "keyvaultid":"" removed.
Meanwhile, we'll have a fix to our SDK so that we won't throw exception when the property value is empty string

Great to hear the workaround worked!
If you have any other feedback to our Azure SDK, feel free to let us know in this survey

Hi,
I have this exact same error at the moment.
I hope that a new version will fix this.
However, thank tou @shibayan for the workaround you found, I will test it right away.


PDF - Everytime a click on a highlight, it open's a new tab

Clicking on an highlight in obsidian, result on opening a new tab in my browser.
So, if I want to see 3 highlights it's gonna load 3 tabs

Hey @Davidb-2107 , sorry for the late reply.
I guess that depends on your template. May I know what your article template looks like?


Update README.md, use the curl command instead of wget

Update README.md, use the curl command instead of wget, it make install step more clear

curl not work well after my test


Linux kernel

Hello,
The router firmware I am trying to emulate relies on Linux kernel 3.4, so I was wondering how I could compile this kernel in order to make it work with FirmAE.
Thank you

For all the architectures (armel, mipsel, mipseb), FirmAE uses a customized v4.1 kernel which considered to have backward compatibility by disabling such as memory protection configuration. To use the other version of the kernel such as v3.x, the Firmadyne kernel can be compatible.
For the 32bit architectures, the kernels can be compiled by using the musl-cross of the Firmadyne. About the 64bit, the kernels can be compiled by using the musl-cross-make v0.9.8.


Chartmuseum apparently not closing files when scanning repo

It looks as if chartmuseum may not be closing files when it scans the repo. We are looking at converting a large repo (nearly 16000 packages) to chartmuseum and are getting the following error on the initial scan:
open helmrepo/xxx.tgz: too many open files in system
I haven't dug through the code yet, but my assumption is that when it first scans the repo it open each file and doesn't close it before opening the next, or it parallelizes the scan and opens so many files that it hits the file descriptor limit. For now, I have bumped by ulimit to get around this error, but I may not be able to do that in production.

@sjthespian thank you for opening this.
You are correct that it parallelizes the scan. We may want to add a configurable limit for this.
@davidovich any comments to add here?

@jdolitsky / @davidovich  It looks like it is even more of an issue if I run chartmuseum in a container. Even with ulimit set to increase nofiles, it fails with a thread limit trying to scan our repo. It looks like it definitely needs some command line options for parallelism.
2018-03-05T19:45:49.765Z        DEBUG   Fetching chart list from storage
2018-03-05T19:45:58.304Z        DEBUG   Regenerating index.yaml
2018-03-05T19:45:58.304Z        DEBUG   Loading charts packages from storage (this could take awhile)   {"total": 15945}
runtime: program exceeds 10000-thread limit
fatal error: thread exhaustion

runtime stack:
runtime.throw(0xf3bd33, 0x11)
        /usr/local/go/src/runtime/panic.go:596 +0x95
runtime.checkmcount()
        /usr/local/go/src/runtime/proc.go:519 +0xa4
runtime.mcommoninit(0xc43dfb5000)
        /usr/local/go/src/runtime/proc.go:539 +0xa0
runtime.allocm(0xc420014000, 0x0, 0xc43dfab000)
        /usr/local/go/src/runtime/proc.go:1341 +0x99
runtime.newm(0x0, 0xc420014000)
        /usr/local/go/src/runtime/proc.go:1611 +0x39
runtime.startm(0xc420014000, 0x14ce700)
        /usr/local/go/src/runtime/proc.go:1698 +0x17e
runtime.handoffp(0xc420014000)
        /usr/local/go/src/runtime/proc.go:1725 +0x55
runtime.retake(0x3ece48ade7b2, 0x2)
        /usr/local/go/src/runtime/proc.go:3909 +0x15d
runtime.sysmon()
        /usr/local/go/src/runtime/proc.go:3836 +0x1ea
runtime.mstart1()
        /usr/local/go/src/runtime/proc.go:1179 +0x11e
runtime.mstart()
        /usr/local/go/src/runtime/proc.go:1149 +0x64

If I'm looking at the correct code, I'm assuming you would have to add a counter to limit the parallel processes around https://github.com/kubernetes-helm/chartmuseum/blob/16021add17ffdc61824832435ba4a61d7fedab6a/pkg/chartmuseum/cache.go#L218 ?

@sjthespian that is correct

I'm still a go novice, but would the following do the trick? (assuming indexLimit is a command line argument value with a reasonable default):
        limiter := make(chan bool, indexLimit)
	for _, object := range objects {
                limiter <- true
		go func(o storage.Object) {
                        defer func() { <-limiter }()
			select {
			case <-ctx.Done():
				return
			default:
				chartVersion, err := server.getObjectChartVersion(o, true)
				if err != nil {
					err = server.checkInvalidChartPackageError(log, o, err, "added")
					if err != nil {
						cancel()
					}
				}
				cvChan <- cvResult{chartVersion, err}
			}
		}(object)
	}
        // Wait for remaining func() calls to terminate
        for i := 0; i < cap(limiter); i++ {
            limiter <- true
        }

@sjthespian ping me on k8s slack and lets work through this

I have a working piece of code, I'll fork the repo and send a PR once I figure out what to call the command line argument and get that working. Here's the diff using a hard-coded limit of 250:
iff --git a/pkg/chartmuseum/cache.go b/pkg/chartmuseum/cache.go
index a79b573..3ae8db6 100644
--- a/pkg/chartmuseum/cache.go
+++ b/pkg/chartmuseum/cache.go
@@ -214,8 +214,12 @@ func (server *Server) addIndexObjectsAsync(log loggingFn, index *repo.Index, obj
        ctx, cancel := context.WithCancel(context.Background())
        defer cancel()

+       indexLimit := 250
+       limiter := make(chan bool, indexLimit)
        for _, object := range objects {
+               limiter <- true
                go func(o storage.Object) {
+                       <-limiter
                        select {
                        case <-ctx.Done():
                                return
@@ -231,6 +235,10 @@ func (server *Server) addIndexObjectsAsync(log loggingFn, index *repo.Index, obj
                        }
                }(object)
        }
+       // Wait for remaining func() calls to terminate
+        for i := 0; i < cap(limiter); i++ {
+            limiter <- true
+        }

        for validCount := 0; validCount < numObjects; validCount++ {
                cvRes := <-cvChan

Fixed by #60

Fixed by #60


locale  srouce and cache with cdn setting

source file and cache is on locale. but i have setting amazon CloudFront.
when direct request 'https://xxxx.cloudfront.net/product/1/Soft-Axami_1_6607c41abda77.jpg' it work.
but i want use glide so i created router like this:
#[Route('/image/read', name: 'image.read')]
    public function readImage(Request $request, ConfigurationService $configurationService)
    {
        try {
            $domain=$configurationService->getSetting('SYSTEM_IMAGE_DOMAIN',true);
            if(!$domain){
                $request->getSchemeAndHttpHost();
            }
            $path = $request->query->get('path');

            $source_dir = $this->getParameter('kernel.project_dir') . '/public';
            $server = ServerFactory::create([
                'source' => new Filesystem(new LocalFilesystemAdapter($source_dir)),
                'response' => new SymfonyResponseFactory(),
                'cache' => $source_dir . '/cache',
                'driver' => 'imagick',
            ]);
            $query = $request->query->all();
            unset($query['path']);
            return $server->getImageResponse($path, $query);
        } catch (\Exception $e) {
            return new RedirectResponse('');
        }
    }

i want kown how can i use amazon CloudFront return cache file

@langziyang the image response sends the cache headers, if you're using symfony, make sure to use the correct response factory:
https://glide.thephpleague.com/2.0/config/integrations/symfony/
Cloudfront will cache the images and only hit your Symfony app when it needs to


Fix: clang-format: BraceWrapping-AfterClass now follows standard.

[As title states!]

Will close pull request as there are problems with header files with clang-format file. Will reopen when complete.


[EN DateTimeV2.Timezone] Support recognition of common timezone short names as entity, providing possible alternatives

For example, in: "schedule a meeting at 3PM CST"
CST should be tagged as timezone, plus the response should include possible alternatives if ambiguous (e.g. "Central Standard Time", "China Standard Time").

Current solution under "EnablePreview" doesn't handle ambiguous cases.

Any update about this one?


add procedures/referrals to report

Adding Procedures (Not referrals)  and Referrals to the Treatment History section in the Report tab
Referrals are the ones that have concept code in {   '1009020',   '1009077',   '1009065',   '1009019',   '1009032' }
see screenshot of UI containing the changes:

@mcjustin this should be ready for you to review.

@mcjustin this should be ready for you to review.

Thanks @achen2401  - is that screenshot from your own dev system, or?
I think we should try this on the main dev system... you don't know of Patient's w/ these resources there, do you?
If it looks good there then we'll switch staging (where we should have more FHIR data for testing this) to this branch.

@mcjustin I posted sample FHIR Procedure data (from the json you gave me) to Marcus Aurelius on the shared dev instance


Updated usage of the Tiers API

Codecov Report

Merging #2388 (73a52f8) into main (c36279d) will increase coverage by 0.03%.
The diff coverage is 45.65%.

@@            Coverage Diff             @@
##             main    #2388      +/-   ##
==========================================
+ Coverage   36.87%   36.91%   +0.03%
==========================================
  Files         618      616       -2
  Lines       25046    24939     -107
  Branches     4426     4394      -32
==========================================
- Hits         9235     9205      -30
+ Misses      15811    15734      -77

Impacted Files
Coverage Δ

app/adapters/tier.js
0.00% <ø> (ø)

app/components/gh-launch-wizard/connect-stripe.js
2.47% <0.00%> (+0.02%)
:arrow_up:

app/components/gh-launch-wizard/finalise.js
4.00% <0.00%> (+0.15%)
:arrow_up:

app/components/gh-members-payments-setting.js
2.09% <0.00%> (+0.13%)
:arrow_up:

app/components/modal-tier.js
48.32% <42.85%> (-4.00%)
:arrow_down:

app/controllers/settings/membership.js
48.98% <42.85%> (+0.40%)
:arrow_up:

app/controllers/offer.js
45.16% <50.00%> (ø)

app/serializers/tier.js
71.42% <50.00%> (ø)

app/components/gh-tier-card.js
56.52% <100.00%> (ø)

app/controllers/offers.js
76.92% <100.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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