Does it handle decimals?

Hello, I'm wondering if floating point error/decimal calculations are handled under the hood?

Hi @Vasile-Peste 😊,
As you can see in this image, the floating precision is not handled by the library

But i never had any trouble with it because it's negligible.
I think it's better to handle it at the latest layer if you want to use it for trading bots.
Here is a sample data you can play with (rename it to .js):
dataParsed.txt
And here is how to test the library:
import { open, high, low, close, volume } from "./dataParsed.js"
import { IndicatorsNormalized } from "@ixjb94/indicators"

const ta = new IndicatorsNormalized()

Run()
async function Run() {
    const result = await ta.sma(close, 50)
    console.log(result)
}

I see, it's clear!
Thank you


Alarmmessages is not refreshed properly

custom_component/hahomematic version (if applicable):
0.22.2
Home Assistant version (if applicable):
2021.12.9
CCU version:

Model (CCU2/3, Raspi3/4, NUC, ProxmoxVE, ...): CCU2/3
OS (Original, RaspberryMatic, debmatic/...): RaspberryMatic
Version: <IP_ADDRESS>20115

Problem-relevant configuration:
Do you use tls? no
Do you use callback? no
Do you use username and password? yes
Which interfaces do you use? BidCos-RF, HmIP-RF
Describe the bug
Alarmmessages are not refreshed properly.
I had a Alarm Message due to "Device not available" this early morning.
Home Assistant is displaying immediately this Alarm Message, but
is not falling back to 0 after this Device turn back to "available" after 45 Minutes.
Dont know if this helps, but Servicemessages are refreshed in time,
and my test with "WatchDog: hb-rf-eth-connection" was succesfull too.
Expected behavior
Alarmmessages are displayed as in RaspberryMatic with a delay of a few seconds.
Screenshots

I don't know how this sysvar ist filled on the CCU, and how this sysvar relates to the UI element.
Maybe someone else can give a hint on this.
We load all sysvars together and update the sensors, so there is now reason why Alarmmeldungen should behave different.
The refresh interval on sensors is 30 seconds.

After more than 24h the variable Alarmmessages still counting 1,0!
Restart supervisor, core und RaspberryMatic Addon has no effect...

Please check with the following script on your CCU:
WriteLine(dom.GetObject('Alarmmeldungen').Value());
Goto CCU / Programme / Script testen
Looks like the content of your sysvar is 1.

Please check with the following script on your CCU: WriteLine(dom.GetObject('Alarmmeldungen').Value()); Goto CCU / Programme / Script testen
Looks like the content of your sysvar is 1.
Yes, you are right!
But why the Display "Alarmmeldungen" is "0"?
How can i fix this?

Please check with the following script on your CCU: WriteLine(dom.GetObject('Alarmmeldungen').Value()); Goto CCU / Programme / Script testen
Looks like the content of your sysvar is 1.
Yes, you are right!
But why the Display "Alarmmeldungen" is "0"?
How can i fix this?

I dont't know. Maybe you should ask in the homematic-forum.

I think this can be closed here

I think this can be closed here

You don't want to wait until i have response from forum?
I do not care...
Close it, i re-open it or open a new issue if necessary, ok?

OK

OK

Found Issue:
https://github.com/jens-maus/RaspberryMatic/issues/1016
It will be mentioned on one of next releases...
Thanks for your effort.


User assign story implemented

@sanbornsen we need to add unit tests specific to user assignee. Something like
-login
-click on assign user
-select a user

@nimishamukherjee @harrybabu @SMahil please review

@SMahil please review the HTML and SASS
@harrybabu please review

The UX design has the current user listed at the top of the list of users with a modifier of "(me)" on the user name. And the other users have email addresses displayed. I don't think we have access to the email addresses yet.

I believe the email ID is a design reference. We don't have that data from backend. Yeah, I missed the assign me part. I'll get on it.

Thanks @SMahil. Can you please check and verify all the changes.

Comment on the unit test coverage - the test coverage is good for the new feature:
✔ should not open the user list if not logged in
    ✔ should open the user list if logged in
    ✔ should return correct user info from getAssignedUserDetails
    ✔ should return null from getAssignedUserDetails in case not found
    ✔ Page should display correct assignee
But - I'd recommend building test data that is not directly tied to a person working on the project:
expect(foundUser.fullName).toBe('Baiju Muthukadan');
expect(el.nativeElement.textContent).toContain('Sudipta Sen');

@ldimaggi I mocked the data with some live data I had available. I'll change the names.

@ldimaggi done, check the names if you like them. :D

Looks good to me!


Python syntax to override port to use 17774 instead default 17778 to connect to orion ?

Hi, we would like to switch Orion endpoint to  port 17774 but i dont see any option in manual on how to use different port than 17778 in the code
https://github.com/solarwinds/orionsdk-python#usage
import orionsdk
swis = orionsdk.SwisClient("server", "username", "password")
all i can find is above to build connection to orion which defaults to 17778

closing this
just figured out that default port was changed to 17774 in 0.4.0
and port can be overwritten by following syntax back to 17778 if needed
port = 17778
swis = orionsdk.SwisClient("server", "username", "password","port")


[Backport - Newton] Disable MaaS alarms/checks with regex

This patch allows for regular expressions to be used in the
maas_excluded_checks variable.
It also adds maas_excluded_alarms and allows deployers to use a
regular expression to exclude only alarms for a particular check.
The check itself (and any related graphing) is maintained.
This comes from a discussion in rcbops/u-suk-dev#948 where some of the
checks are good to have, but the alarms aren't needed.
(cherry pick from https://github.com/rcbops/rpc-openstack/commit/d9ec02cd4f941558725892293a3e44130f0813d2)
Connects rcbops/u-suk-dev#1019

Hold off on merging until #1722  makes it in. That one will need to be added to this patch.


Any other plugins?

May I have the rest of the plugins you guys made? Like GAMEMODE plugins? This one was very awesome.

Here is an overview of all plugins: https://github.com/survanetwork.


BP-2369: Backend improvements for Magento 2

This will be part of our 2.0.0-RC2 release.
Unfortunately no expected release date yet.


Repo sync for protected CLA branch

The pull request is created from main637819276180928998 to main to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


[webview]How to change webview defalut language?

I build a app with webview , when I click upload image, display english menu. My iphone is chinese.
How to change the default language for webview?
thanks.

Testing webview_flutter with Internationalizing Flutter apps
 does seem to be change language for webview_flutter

#56361 You can dynamically add webview and after that reinitialize your language to Chinese .This would resolve your issue.

 How to do it?

@jet10000 I hope I understand your question correctly -- that is you want to replace the default file <input> attribute in Chinese character, correct?
I believe you can do a simple HTML/CSS hack to the <input> attribute from the remote HTML page instead. Then, users can tap on the custom file-input button from webview without issue.
My example as following:
1.) Hack your  attribute from remote page

2.) Hide the default <input> via CSS

Final result (default v.s. hacked):

This has been fixed in Flutter 1.22/ WebView 1.0 - use hybrid composition in the Flutter WebView plugin.
https://github.com/flutter/plugins/tree/master/packages/webview_flutter#using-hybrid-composition


Unset TZDIR will cause crash without /usr/share/zoneinfo

When using ORC on a platform that doesn't have /usr/share/zoneinfo defined nor the TZDIR environment variable set, the library will crash. Specifically on Windows, the DEFAULT_TZDIR location is an invalid path, but even on Linux the file is not present on minimal images like the default alpine container. It looks like the TZDATA is downloaded during the build and is referenced during the C++ tests of ORC, but this change won't affect the runtime DLLs. Perhaps a relative location or option to configure a TZDATA location would make sense? In my case working with conda, I patched the routine to locate the tzdata that is included within the conda environment, but that wouldn't be applicable to all use cases.

@dongjoon-hyun Sorry for not being more specific — the crash happens in the ORC C++ library. When the readFile call is made in getTimezoneByFilename, a crash will occur on Windows where /usr/share/zoneinfo/UTC is an invalid path:

Thank you for the details, @scdub .
cc @wgtmac

Sure, let me take a look.

@scdub After a quick check, it seems that either TZDIR or /usr/share/zoneinfo to be set is a must to run the ORC C++ library. From the WIN32 test, it downloads TZDATA and sets TZDIR=${TZDATA_DIR}. Could you set TZDIR to point to TZDATA in the enviroment?

@wgtmac Yes, I can confirm it will work if the TZDIR is set and extracted, as is the case with the tests themselves. However, the TZDATA isn't included in the steps for building the distribution, and relying on an environment variable has other challenges. I believe that the code should provide a clear error message rather than crash if the current design is intended. From my perspective it would be nice to not be reliant on a POSIX time interface or include one in the distribution output with instructions about the requirement to configure TZDIR or ensure the host operation system provides /usr/share/zoneinfo for e.g. barebones Docker containers.

It would be a lot of maintenance burden if the ORC library includes a TZDB in its distribution output. I prefer providing a better error message when the local tzdb is not found.
BTW, C++20 has standardized std::chrono::tzdb to represent a copy of the IANA TZDB. As we are still on C++17 yet, we can switch to it once we make the C++20 transition.

Proposed fix on the exception message: https://github.com/apache/orc/pull/1587


[adc_ctrl/rtl] Mismatch with model for low power counter.

There is an issue with the way the lp_counter is cleared.
The syndrome seen is:
xmsim: *E,ASRTST (/scratch/google/goo_opentitan/nigels/issue_11759/adc_ctrl-sim-xcelium/default/src/lowrisc_dv_adc_ctrl_sim_0.1/tb.sv,258): (time<PHONE_NUMBER> PS) Assertion tb.EnterLowPower_A has failed (46 cycles, starting 950402860 PS)
                UVM_ERROR @<PHONE_NUMBER> ps: (tb.sv:258) [ASSERT FAILED] EnterLowPower_A
                UVM_INFO @<PHONE_NUMBER> ps: (uvm_report_catcher.svh:705) [UVM/REPORT/CATCHER]

This is due to the dut exiting low power mode too early, in turn caused by the low power counter not being cleared because the set of matching filters has changed.
In the waves we see that lp_sample_count_q does not get cleared while the model equivalent m_lp_counter does.
In both model and rtl the set of matching filters is the same (np_match and m_match) and both change from c2 to 42 this should clear lp_sample_count_q.
The additional term for stay_match, any_fst_lp_match, seems to be the problem.
To reproduce checkout PR https://github.com/lowRISC/opentitan/pull/11760
./util/dvsim/dvsim.py hw/ip/adc_ctrl/dv/adc_ctrl_sim_cfg.hjson -i adc_ctrl_clock_gating --fixed-seed 61251460 --tool xcelium --max-parallel 3 -v m --waves

thanks @NigelScales , let me have a look.

Hi @tjaychen
I should have run with more seeds but we are currently a bit short on licenses.
This seems to fix it.


[Bug] CreatePayload function in JwtPayload doesn't read "sub" from any token

Which version of Microsoft.IdentityModel are you using?
8.0.4
Where is the issue?
JwtPayload.cs
Is this a new or an existing app?
This is new app.
Repro
var base64UrlEncodedJsonString = @"{
""iss"": ""https://test.eu.auth0.com/"",
""sub"": ""test|6612f2ddbab2cde61dcbfa62b"",
""aud"": [
""https://test-dev.eu.auth0.com/api/v2/"",
""https://test-dev.eu.auth0.com/userinfo""
],
""iat"":<PHONE_NUMBER>,
""exp"":<PHONE_NUMBER>,
""scope"": ""openid profile email read:current_user update:current_user_metadata delete:current_user_metadata create:current_user_metadata create:current_user_device_credentials delete:current_user_device_credentials update:current_user_identities offline_access"",
""gty"": ""password""
}";
var result = Base64UrlEncoding.Decode(base64UrlEncodedJsonString, 0, base64UrlEncodedJsonString.Length, CreatePayload);
Expected behavior
The property "sub" should be read.
Actual behavior
The property "sub" is not read.
Furthermore, every second property from payload is skipped.
Possible solution
n/a
Additional context / logs / screenshots / links to code
n/a

Hi,
Unfortunately I think you are running into this
#2513

@HEBOS as @keegan-caruso mentioned you may be running into different package versions.
Are all identitymodel assemblies 8.x?


Consolidate list of DPG Tags

Create a consolidated and exhaustive list of industry sectors and technology types so that all DPGs can be mapped to them in later weeks.

In progress - done with tech type. Meeting with Praj on Friday (9/24).

Amreen to respond to Prateek and clarify that we'd like to prioritize Kindly

Done. Praj to do initial mapping. Amreen to schedule meeting with VGS + NB in 2 weeks to confirm tagging.


[Tiered Caching] [Milestone 1] KeyLookup store for disk cache

Is your feature request related to a problem? Please describe.
Refer main https://github.com/opensearch-project/OpenSearch/issues/10024
and https://github.com/opensearch-project/OpenSearch/issues/10870
As part of disk based caches, we need to check whether a key is present on disk or not and extract value accordingly. As it can happen that we unnecessarily perform a disk seek even when the key is not present. We can solve this by storing disk cache keys in memory in an optimized way.
Details
As part of tiered caching milestone1, we are looking to add a disk tier. Where essentially we will be spilling evicted items from in-memory cache to this disk tier.
So whenever a request comes, we need to check whether this request is present on either of caches i.e. in-memory or disk. As doing a disk GET(to know whether request is present or not) every time might be expensive and will add to the overall search latency. So to solve this, we are trying to maintain disk keys in-memory itself. To do this, instead of storing the whole key in memory(which can be expensive), we will just store the key hashcode(integer) in memory using this key store implemenation. Obviously there might be false positive cases but it should be few. Also the actual keys will be stored on disk itself.
To implement this key store, we were looking for different data structures like bitset array, roaring bitmap etc. Bitset can be expensive memory wise considering how it works. As for example, if we have only one key which is a large number N, we need to allocate that much memory for bitset array to set Nth bit index for a number N.
On other hand, roaring bitmap is more of a compressed bitset which works better memory wise but relatively slower. Here too we need to be considerate of the max memory roaring bitmap will use, so we have added some logic around that.
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Reading through the linked PR, I think we need to clarify what the interface for the key lookup should be before diving into the implementation.
It sounds like the key lookup is supposed to look like a set, with add, contains, and remove operations, right?
Taking into account that the design specifies that values get added to the cache in batches, and eviction can reasonably be deferred (since you allow for false positives), your add and remove operations could take a collection of items to avoid needing to repeatedly acquire a write lock to mutate the key store.


Toast view tutorial

Added view controller to demo the Toast view

Good job again @K4stor, Merged :D👍🏻.

Thanks mate !


Add filter post type feature (UI + Logic)

Please provide enough information so that others can review your pull request:
Added logic (handleFilter, filterNotesBy helper functions + map behavior change) and UI selector in order to filter notes by: All, True, Fake, and Mixed.
Calculated by matching the max vote value by the desired vote value.
Explain the details for making this change. What existing problem does the pull request solve?
Allows the user to view posts by type, leading to easier navigation and increased usability.
Test plan (required)
Tested by running with mock data:

Code formatting
Follows formatting conventions specified by Prettier.

Good work! Keep it up!


Log error at adapter restart

when adapter is restarted log Is full of error's of promise rejects.
host.DESKTOP-LSKTCAN	2019-01-21 21:12:09.960	error	Caught by controller[630]: at <anonymous> host.DESKTOP-LSKTCAN	2019-01-21 21:12:09.960	error	Caught by controller[630]: at set_zero_val (C:\ioBroker\node_modules\iobroker.sourceanalytix\main.js:565:14) host.DESKTOP-LSKTCAN	2019-01-21 21:12:09.960	error	Caught by controller[630]: (node:796) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'val' of null host.DESKTOP-LSKTCAN	2019-01-21 21:12:09.960	error	Caught by controller[629]: (node:796) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by r host.DESKTOP-LSKTCAN	2019-01-21 21:12:09.960	error	Caught by controller[629]: at process._tickCallback (internal/process/next_tick.js:189:7) host.DESKTOP-LSKTCAN	2019-01-21 21:12:09.960	error	Caught by controller[629]: at <anonymous>

fixed in 0.2.27


Support multiple HTTP status codes for HttpWaitStrategy

In the context of elasticsearch test containers module, I'd like to
add an ElasticsearchWaitStrategy class which extends the HttpWaitStrategy
with some default settings so it will be even easier for a end user to
start an ElasticsearchTestContainer.
Anyway, in this context, I found helpful that the HttpWaitStrategy expects
more than on status code.
For example, you can imagine running elasticsearch in secure mode or
without any security. In which cases the elasticsearch service might answer 200 or 401.
This commit proposes this change.

I don't know TBH. I could also have only change the existing signature.
Up to you. I can update the PR if you wish but having 2 methods doing the same does not look appealing to me.

Thx for the comment @barrycommins. If this isn't released yet, WDYT about changing this internal API for now to allow for multiargs by default?

@kiview  I see no harm in it, but I think if it were up to me, I'd keep the original method name forStatusCode and just update it to support varargs.
I think that allows the easiest upgrade path for current users of HttpWaitStrategy, if they want to move to the new class. They would just have to change the package name and the method would still work.
For users changing from a custom subclass, they would need to change the method signature to support varargs, but since they were changing the superclass anyway I don't think it's too big a problem.

Makes sense to me. I wanted initially to add the plural form to make obvious that you can many status codes but I'm fine keeping the singular form.
I'll update the PR that way.

@dadoonet Just a suggestion, I'm not a maintainer, so hopefully @kiview has an opinion on this.

The plural is of course much nicer regarding API readability and discoverability. I think I even would be fine with having 2 public methods, with the singular one delegating to the varargs variant.
I'm sure @bsideup or @rnorth have a stronger opinion on the API design :sweat_smile:

The travis failure seems unrelated to my changes.

I believe it's ok now.
Please LMK if I should squash it and rebase.

@dadoonet ok, looks good now! Just the last thing before we can merge - CHANGELOG.md :)
Sorry for not pointing at it before, I keep forgetting it :(
P.S. we squash automatically when we merge, no need to squash it yourself 👍

No problem.

@dadoonet merged, thanks a lot! Will be released this weeks :)

Awesome! Thanks for guiding me for my 1st code PR :)


Can't build Pytorch

Describe the bug
Pytorch fails to build
Executing pythonImportsCheckPhase
Check whether the following modules can be imported: torch
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "<string>", line 1, in <lambda>
  File "/nix/store/2dcsn57cgaxs92ha5swihrab0g3l2h6g-python3-3.7.7/lib/python3.7/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1006, in _gcd_import
  File "<frozen importlib._bootstrap>", line 983, in _find_and_load
  File "<frozen importlib._bootstrap>", line 967, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/nix/store/i6fkycsn1q8vk4190hslrvh256wilw5i-python3.7-pytorch-1.4.1/lib/python3.7/site-packages/torch/__init__.py", line 81, in <module>
    from torch._C import *
ImportError: /nix/store/i6fkycsn1q8vk4190hslrvh256wilw5i-python3.7-pytorch-1.4.1/lib/python3.7/site-packages/torch/lib/libtorch_python.so: undefined symbol: _ZN5torch4cuda4nccl6detail16throw_nccl_errorE12ncclResult_t
builder for '/nix/store/lia86jj07nygz23ca3s1gxahx62ipls2-python3.7-pytorch-1.4.1.drv' failed with exit code 1
cannot build derivation '/nix/store/ia0jgpq5w2myqpp8zha2xj7h3f5cj5nn-python3-3.7.7-env.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/hxp00cp2lvhhw677d1v8zbp5iv3cqpy1-home-manager-path.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/gcpmscdc0d5h3fg5m6ik411q6hhsa0mz-home-manager-generation.drv': 1 dependencies couldn't be built
error: build of '/nix/store/gcpmscdc0d5h3fg5m6ik411q6hhsa0mz-home-manager-generation.drv' failed

To Reproduce
Steps to reproduce the behavior:

Set .config/nixpkgs/config.nix to

{
    allowUnfree = true;
    cudaSupport = true;
}

Switch to nixpkgs version 20.09pre227577.135073a87b7
nix-build '<nixpkgs>' -A python3Packages.pytorch

Expected behavior
Pytorch should build without failure.
Notify maintainers

@teh @thoughtpolice @tscholak
Metadata
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.35, NixOS, 20.09pre227577.135073a87b7 (Nightingale)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.5`
 - channels(theo): `"home-manager"`
 - channels(root): `"nixos-20.09pre227577.135073a87b7"`
 - channels(arsleust): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

NVIDIA SMI working
NVIDIA-SMI 440.82       Driver Version: 440.82       CUDA Version: 10.2

Maintainer information:
# a list of nixpkgs attributes affected by the problem
attribute:
# a list of nixos modules affected by the problem
module:

I wonder if this is the sort of issue I was seeing during my many nixpkgs-review runs for #77714
Attaching the error I encountered just in case.
python37Packages.pytorchWithCuda.log

Your log (python37Packages.pytorchWithCuda.log) gives exactly the same error for the same symbol...
So if you now say:

Looks like pytorch and batchgenerators build fine on master with Pillow 7.1.2.

I'll try to rerun a build now

Your log (python37Packages.pytorchWithCuda.log) gives exactly the same error for the same symbol...
So if you now say:

Looks like pytorch and batchgenerators build fine on master with Pillow 7.1.2.

I'll try to rerun a build now

I need to stop posting so late. Didn’t realize the same symbol was there.
And yes, everything seemed to build fine on hydra.

Same error on 20.09pre228204.467ce5a9f45
To my knowledge hydra builds pytorch without CUDA, @tbenst was working on a Hydra for CUDA related libs I believe. Seeing as cuda and nccl are part of the name of the symbol _ZN5torch4cuda4nccl6detail16throw_nccl_errorE12ncclResult_t, I think this is an issue related to CUDA.

Ok I have seen this: https://github.com/pytorch/pytorch/issues/32638
Corrected by https://github.com/pytorch/pytorch/commit/58cffbff91844217b9f4546e50ea0ab95544cb6a
Which is part of milestone 1.4.1 : https://github.com/pytorch/pytorch/milestone/14
but 1.5.0 seems on the way and the issue still not closed.
Anyway, I will investigate upstream and PR update accordingly

since https://github.com/NixOS/nixpkgs/pull/89802 was merged, and I'm able to build this on master:
[16:46:18] jon@nixos ~/projects/nixpkgs (master)
$ nix-build -A python3Packages.pytorch
/nix/store/nr2p2w6mrs8rqnj43ngzw3jq5gixsjai-python3.7-pytorch-1.5.0

I'm going to considered this issue resolved.


Speech SDK for Unity package missing settings for ARM64

[Enter feedback here]
Hi,
I noticed that the Unity package has a minor problem (anyway on my system), preventing the correct dll's to be placed on the target machine.
My target machine is a Hololens 2 (AMRM64). The Unity Speech SDK for Unity package has settings for each dll file according to its SDK and CPU. The ARM64 folder has Dll's that have no settings.
Greetings JP

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 01b13b79-a598-c1dd-7100-1a6e4becbaae
Version Independent ID: e9376cc7-efd8-73a0-7724-1bac029e17f4
Content: Quickstart: Set up development environment - Azure Cognitive Services
Content Source: articles/cognitive-services/Speech-Service/quickstarts/setup-platform.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @eric-urban
Microsoft Alias: eur

@jpzpj we'll review your feedback and get back to you shortly. Thanks.

@eric-urban can you please help with above inquiry? Thanks.

@jpzpj - This issue will be fixed with SDK 1.20. The ETA is end of January or early Feb. I hope this helps!

#please-close


Do not register sounds

This is a client-side modification. Minecraft does not require sounds that are client-only to be registered to play them. Simply removing the register call while keeping everything else intact should work fine.
This would make Extra-Sounds compatible with servers that have custom sounds of themselves. Right now the game will refuse to connect to those servers because the server has a different registry.

Done in v3, the way I was working on this initially made it seem like events have to be registered before they can be used. Small oversight.


Weird bug with semantic analysis on functions

I can't tell if I've done something wrong or I'm going crazy but in this file on this line in the standard library here, I get the following:
warning: [test-fileio:8:2] Unused variable `contents`
    contents := myFile.readToString();
    ^

error: [file:30:2] Cannot assign value to immutable access
    f.path = path;
    ^

error: [file:31:2] Cannot assign value to immutable access
    f.handle = C::fopen(f.path.location);
    ^

error: [file:39:5] Function declaration must not be in function
pub func (f: ^File) close() {

And it's a strange bug. This may be linked to #657 where the compiler gets confused about the immutable access (which also appears to be another bug)... anyway if you check the code in the file the code looks valid to me, and is not in a function, but the compiler thinks otherwise.

The semantic checks are known to be iffy once one thing breaks. We'd have to redo some of the way we do semantic checks to make it all fit together.

This should be fixed as of my recent changes to semantic checks.


Super long compilation time

Hi,
Can you, guys, advice the best latest 64 toolchain  . Some tests take way too long to compile and postprocess. (fmadd.d.b15-01 for ex): (or it may be the tests are too long)
-rw-r--r--  1 grobman  927257314 May  5 11:41 fmadd_d_b15-01.cpp.s
-rw-r--r--  1 grobman 105071323 May  5 12:07 fmadd_d_b15-01.dis
-rwxr-xr-x  1 grobman   15515816 May  5 11:42 fmadd_d_b15-01.elf
-rw-r--r--  1 grobman   15608704 May  5 11:42 fmadd_d_b15-01.o

You can see that it took 25min just to disassembly elf ...
I use riscv64-unknown-elf-gcc (GCC) 11.1.0
BTW this test simulates more that 1hour, while others just from a few sec to a few min ..

Yes, FMA takes a very long time, it has a huge number of coverpoints with
crossproducts to get IBM-like coverage.
We plan to break that test up so each subpart is shorter, but I don't know
if that will make it run any faster.
On Fri, May 5, 2023 at 4:13 PM algrobman @.***> wrote:

Hi,
Can you, guys, advice the best latest 64 toolchain . Some tests take way
too long to compile and postprocess. (fmadd.d.b15-01 for ex): (or it may be
the tests are too long)
-rw-r--r--  1 grobman  927257314 May  5 11:41 fmadd_d_b15-01.cpp.s
-rw-r--r--  1 grobman 105071323 May  5 12:07 fmadd_d_b15-01.dis
-rwxr-xr-x  1 grobman   15515816 May  5 11:42 fmadd_d_b15-01.elf
-rw-r--r--  1 grobman   15608704 May  5 11:42 fmadd_d_b15-01.o
You can see that it took 25min just to disassembly elf ...
I use riscv64-unknown-elf-gcc (GCC) 11.1.0
BTW this test simulates more that 1hour, while others just from a few sec
to a few min ..
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/347, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJQO5FGK7SVBYZBSMATXEWCTJANCNFSM6AAAAAAXXVWMIE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

there is one more problem - the sizes of the sources of these tests are more than 60MB. Our git setup does not allow such big files to be pushed to the git server. I had real trouble even remove these files from the DB ... If you want this coverage - these tests should be spitted into 10 tests each ...
(at the beginning I was thinking that the tools went to the weeds and it was something wrong with the tests sources).

This is a duplicate of issue #307.


Update words in docs for better readability

Problem
Found typos while reading the docs
Summary of changes
Fixed the typos found
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

My pleasure @vadim2404 I've got more


notmuch: add vim plugin

Hi !
This PR adds notmuch's Vim plugin and Ruby bindings. The Ruby bindings are required by the Vim plugin.
The first commit is a bit unrelated, it removes emacs from the build dependencies when withEmacs is false. I don't even want this on my computer ! (kidding, it was just an easy fix)
The Vim plugin is enabled by default because it consists only of small static files. The ruby bindings are not big either and ruby is already added as a dependency. Also emacs is enabled by default, so it's fair.
The way I hook the ruby bindings doesn't seem right. I couldn't find examples in nixpkgs and I think bundlerEnv cannot be used here. Is here a better way ?
The ruby bindings have a dependency on the gem mail but it is optional and loaded at runtime, so I haven't included it.
There is a related issue: https://github.com/NixOS/nixpkgs/issues/43965
Motivation for this change
Make the Vim plugin available from nixpkgs and to use it like this:
let
  notmuch = (pkgs.notmuch.override {
    withEmacs = false;
  }).overrideAttrs (o: {
    doCheck = false; # Too slow
  });

  # Shadows vim, loading the notmuch plugin and a local vimrc
  notmuch_vim = { vimrc }:
    pkgs.runCommand "notmuch-wrapped-vim" {
      buildInputs = [ notmuch ];
    } ''
      # No absolute path to vim on purpose
      mkdir -p "$out/bin"
      cat <<EOF > "$out/bin/vim"
      #!/usr/bin/env bash
      this="$out"
      PATH=\''${PATH//\$this/} # Remove this package from the PATH
      exec vim -c "set packpath+=\$this/vim" -c "packadd notmuch" -c "source ${vimrc}" "\$@"
      EOF
      chmod +x "$out/bin/vim"
      plugin_out="$out/vim/pack/plugins/opt" # Vim plugins directory structure
      mkdir -p "$plugin_out"
      ln -sf "${notmuch}/share/vim" "$plugin_out/notmuch"
    '';

I got a bit creative here, I'll probably use something saner :D
Things done

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

[x] NixOS

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

In case I do want to use the "mail" gem, how to enable it?

Can you update the title to something like notmuch: add vim plugin?

Sorry for the broken title.
@kisik21 The mail gem is in nixpkgs already: pkgs.rubyPackages.mail and can be added to your user env or to a shell's buildInputs.

Oh, so vim's ruby support just takes packages from the environment? Sounds impure but can be taken care of with a wrapper, so not a bug per se, just an inconvenience (or a convenient feature, depending on your point of view)

It's searching in paths (RUBYLIB) so indeed it's impure but wrapping solves that entirely.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 104407 run on x86_64-darwin 1

1 package failed to build and are new build failure:

python27Packages.notmuch2: log https://termbin.com/aql3

12 packages built:

aerc
afew
lieer
muchsync
neomutt
notmuch
notmuch-addrlookup
python27Packages.notmuch
python37Packages.notmuch
python37Packages.notmuch2
python38Packages.notmuch
python38Packages.notmuch2

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 104407 run on x86_64-darwin 1

1 package failed to build and are new build failure:

python27Packages.notmuch2: log https://termbin.com/aql3

12 packages built:

aerc
afew
lieer
muchsync
neomutt
notmuch
notmuch-addrlookup
python27Packages.notmuch
python37Packages.notmuch
python37Packages.notmuch2
python38Packages.notmuch
python38Packages.notmuch2

Alternative is in https://github.com/NixOS/nixpkgs/pull/308318 .


improve detection for inherited symbol references

Bug/issue #, if applicable: rdar://93203894
Summary
Symbols for operator functions that include multiple dots (e.g. the
ClosedRange ... operator) are currently curated into an unnamed
"Implementations" section due to improper string splitting behavior.
This enhances the detection two ways:

For symbols where the "source origin" is available in our given
symbol graphs, it loads the name information from the parent symbol.
For symbols where the "source origin" is not available, it improved
the string splitting behavior by dropping empty-string components
before reading names from it.

This allows Comparable and Equatable implementations to curate their
inherited operators properly.
Dependencies
None
Testing
Steps:

Generate documentation for a module which includes a conformance to Comparable.
Ensure that the Comparable range operators are correctly organized into a "Comparable Implementations" page.
Repeat with a protocol that defines a custom ellipsis operator (e.g. the .... operator in the FancyProtocol symbol graph added in this PR).

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[ n/a ] Updated documentation if necessary

@swift-ci Please test

I've pushed up changes for the review comments.
@swift-ci Please test

Rebased onto the latest main.
@swift-ci Please test

Rebased.
@swift-ci Please test


Shortcut a and helper lis

Pull Request Test Coverage Report for Build a116302e18399559e463f178dd5484f984961662-PR-2

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

Totals

Change from base Build 1361ddcb6ba625474937b4a4c201a1b95ed94f7a:
0.0%

Covered Lines:
52

Relevant Lines:
52

💛  - Coveralls

Pull Request Test Coverage Report for Build a116302e18399559e463f178dd5484f984961662-PR-2
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

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

Totals

Change from base Build 1361ddcb6ba625474937b4a4c201a1b95ed94f7a:
0.0%

Covered Lines:
54

Relevant Lines:
54

💛  - Coveralls


systeroid: 0.4.2 -> 0.4.3

Diff<EMAIL_ADDRESS>Changelog: https://github.com/orhun/systeroid/blob/v0.4.3/CHANGELOG.md## Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 255164 run on x86_64-linux 1

1 package built:

systeroid


Feature Request: Add Support for Azure Managed Identities

Managing credentials in code is hard, if we can make it to where credential access is handled by Azure instead of putting secrets into a data bag then it will help a lot of folks out.
https://docs.microsoft.com/en-us/azure/active-directory/managed-identities-azure-resources/overview

I can commit to doing this over the next 60 days. I will need this pretty soon here and at that point I'll push up a PR.


Integer arguments to methods interpreted as enumerations

Steps to reproduce
Call a method which accepts an argument of either type int or an enumeration type, passing an integer literal for the argument. For example, in the call "a:b".Split(":", 2) below, the System.String.Split method is overloaded on the type of its second argument, which accepts either Int32 or System.StringSplitOptions.
Expected behavior
The integer literal is passed to the method as an integer rather than an enumeration.
Actual behavior
PS> "a:b".Split(":", 2)
Cannot convert argument "options", with value: "2", for "Split" to type
"System.StringSplitOptions": "Cannot convert value "2" to type
"System.StringSplitOptions" due to enumeration values that are not valid.
Specify one of the following enumeration values and try again. The possible
enumeration values are "None,RemoveEmptyEntries"."
At line:1 char:1
+ "a:b".Split(":", 2)
+ ~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], MethodException
    + FullyQualifiedErrorId : MethodArgumentConversionInvalidCastArgument

PS> "a:b".Split(":", [int](2))
a
b

Environment data

PS> $PSVersionTable

Name                           Value
----                           -----
PSVersion                      6.0.0-alpha
PSEdition                      Core
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   <IP_ADDRESS>
GitCommitId                    v6.0.0-alpha.15
CLRVersion
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>

PS> uname -rmo
4.8.0-32-generic x86_64 GNU/Linux

Note that this works properly in full powershell:
PS[1] (2) > "a:b".Split(":", 2)
a
b

It works in Windows PowerShell because there are different overloads.  In net461:
string[] Split(Params char[] separator)
string[] Split(char[] separator, int count)
string[] Split(char[] separator, System.StringSplitOptions options)
string[] Split(char[] separator, int count, System.StringSplitOptions options)
string[] Split(string[] separator, System.StringSplitOptions options)
string[] Split(string[] separator, int count, System.StringSplitOptions options)

In netcore1.0
string[] Split(char separator)
string[] Split(char separator, System.StringSplitOptions options)
string[] Split(char separator, int count, System.StringSplitOptions options)
string[] Split(Params char[] separator)
string[] Split(char[] separator, int count)
string[] Split(char[] separator, System.StringSplitOptions options)
string[] Split(char[] separator, int count, System.StringSplitOptions options)
string[] Split(string separator)
string[] Split(string separator, System.StringSplitOptions options)
string[] Split(string separator, int count, System.StringSplitOptions options)
string[] Split(string[] separator, System.StringSplitOptions options)
string[] Split(string[] separator, int count, System.StringSplitOptions options)

In full, you'd see the same error if you wrote:
"a:b".Split([string[]]';', 2)

I believe the overload causing trouble is:
string[] Split(char separator, System.StringSplitOptions options)

We see the conversion of [string] of length 1 to [char] as better than a conversion to array, so we pick this overload as it only has two arguments, and we do allow conversions of int to an enum.
During overload resolution, we don't consider values, just the types, So int=>enum is OK, but in the case of the value 2, that is not a valid value for StringSplitOptions, so the conversion fails after picking the overload.
You'll notice that:
"a:b".Split(':', 1)

works just fine, but produces 2 values, not 1 - because we've converted 1 to StringSplitOptions.RemoveEmptyEntries.
So after this long winded explanation, I think everything is working as designed. There are some unfortunate effects with such permissive conversions when there are many similar overloads, but permissive conversions are important to PowerShell, so I don't see a way to fix this beyond asking for even more overloads added on Split.


Does it support multiple bands, such as Landsat and Sentinel S2?

You can try to change the first patch merge layer in ViT image encoder!


Error when deleting cluster flow - TypeError: Cannot read properties of undefined (reading 'catch')

Setup

Rancher version: 2.6-head Commit ID 663df43
Browser type & version: Chrome Version 94.0.4606.81 (Official Build) (x86_64)

Describe the bug
When deleting an existing cluster flow from the cluster flow page, I received 2 errors in the resulting dialog boxes. The cluster flow was deleted.
To Reproduce

Install logging 100.0.0+up3.12.0
Create Cluster Output
Create cluster flow which uses the output created in step 2
Click 'Cluster Flow'
Select the 3 dots next to the cluster flow created in Step 3
Click delete

Result
Two errors appear in the resulting 2 dialog boxes.
TypeError: Cannot read properties of undefined (reading 'catch')
"type":"error" -... "status":404
Expected Result
Cluster flow is deleted
Screenshots

Additional context
I installed version 3.12.0 of the chart to test upgrading the chart to 3.15.0

Installing 2.6-head Commit ID 931a0f0, I'm seeing another occurrence of this error on each initial login.

install rancher
obtain bootstrap password
enter bootstrap password and click to proceed

result: error flashes on screen during Loading... and then login process continues

PASS verified Fixed in 2.6-head Commit ID 931a0f0

Installed logging
created secret
created cluster output
created cluster flow
deleted cluster flow

TypeError: Cannot read properties of undefined (reading 'catch') No longer occurring


Outlook Mail Tool - Clicking the sender email link takes us to current active session in chrome

Electron build -2fa0a180691b6d
Steps to reproduce the problem:

Start a chat with Tildy.
2 Add Outlook Mail tool.
Get content of any email
Click the sender email link that is presented in the response

Clicking the sender email link takes us to current active session in chrome
Expected Behavior:
It should may open the a compose email page with "To" address being pre populated with the sender email.

Tracked in otto


Add og:image meta tag for social sharing

Most social media use the og:image tag from the Open Graph standard that was provided by facebook. This is best practice for Commerce storefronts.

QA:

test whether the main product image is added as a og:image metatag

checked for this on dev17 and github.io, but don't see it. will try tomorrow, maybe servers not updated.

verified on github.io mar13

@Xymmer The missing part here might be that this is only implemented on the PDP ;) On other pages we do not have a clear idea which main picture to share, I've created a ticket for this in our CMS backlog (CMSX-8615).
Can you test again?


SBS integration

[ ] Rekey button seems busted. @patrickxb is looking as its the same call to show the rekey dialog for folders and thats also busted

@keybase/react-hackers this can be looked at. if you provision a new device using a gpg / userpass you should see the screens

https://github.com/keybase/client/pull/5690 is patrick's side


Upadate mysql faeatures in home page

Signed-off-by: Mehedi Hasan<EMAIL_ADDRESS>

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: heheh13:x: kodiak-appscode[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


set captcha enabled to true on PrivyProvider

#442  👈
main

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

Join @tremarkley and the rest of your teammates on  Graphite


Weird memory issue after upgrading to Gradle 7.3.2

Expected Behavior

Compilation finishes without warning or errors.
Current Behavior

The VM is generating a heap dump.
Context

We just updated to Gradle 7.3.2 and we are presenting weird memory issues. Our project is a big Android application with many modules and a lot of code; however, it has been working just fine until the update to Gradle 7.3.2.
Steps to Reproduce

Unfortunately, I can't share our Android repo because it is private; however, it's been working fine until we updated to Gradle 7.3.2.
It is generating a heap dump before executing our command (./gradlew :app:bundleDebug), so it's really weird. And we haven't changed our CI machines configuration for a long time.
dump.txt
Your Environment

Gradle 7.3.2
OpenJDK Runtime Environment (build 11.0.13+8-Ubuntu-0ubuntu1.18.04)
OpenJDK 64-Bit Server VM (build 11.0.13+8-Ubuntu-0ubuntu1.18.04, mixed mode, sharing)
javac 11.0.13
Jenkins 2.190.1
Build Cache Node 9.11

Funny, we had the same issues.  We hadn't tuned our jvm for the gradle process in a while, and already knew we needed to work on the unit test memory configs.  After increasing the max heap for the unit tests and turning on parallel GC, we were fine.

@pgrpmichel Would you mind sharing your gradle.properties file?

@GianfrancoMS no problem.
Here's our jvm settings for the gradle build (build.properties):
org.gradle.jvmargs=-Xms512M -Xmx4g -XX:MaxPermSize=1024m -XX:MaxMetaspaceSize=1g -XX:+UseParallelGC -Dkotlin.daemon.jvm.options="-Xmx1g" -Dfile.encoding=UTF-8
Here's our jvm settings for the unit tests (build.gradle):
maxHeapSize = "2g" minHeapSize = "1024m"

Which version did you upgrade from?
With the information provided, tracking this will be quite complicated.

@ljacomet I upgraded from version 7.3.0

I'm attaching the demon log. At the end I can see this error: Discarding EOFException: java.io.EOFException
daemon-182.out.log


Improved build

Great! Thanks! Not sure what requires net-ssh gem, probably it pops up somewhere in the dependency tree. I guess we need to pin it to some older version that does run on ruby 1.9

Or just if ruby <= 1.9: deps are A else deps are B in the gemspec files.

Right. There is this issue with version pinning issue...I'll look into it again.


Remove link that goes to a 404'd command.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/get-started-with-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Design

[x] User is greeted with "Hello + "username" on start

Task done, pushed into #sambranch.
@jamypurfic FYI


jbaker.getEventAsync() exposure

Hi
Would it be possible to expose the baker eventsAsync(processId: String) of Baker in the JBaker class?
That would be pretty useful.
Edit:
Would it also be possible to expose the bakeAsync method?
Thanks
David

Hello David, we are going to change our approach to async operations.
All our normal operations (bake, processEvent, getProcessState etc..) will be async by default.
We will work on this in the coming weeks.
I hope this is soon enough for you.

Hi Tim,
Sounds good to me, thanks!

A small update,
We just released Baker 3.0.0-RC.
This includes the changes we discussed.
Migration guide is in the works and should be done soon.


Side bar not collapsing 0.87.0b2

Version 0.87.0b2
Hass.io
Ubuntu 18.4
Side bar will not collapse on info panel and all other panels required to press hamburger menu to collapse.
No errors in error log.

File it in Polymer repo

https://github.com/home-assistant/home-assistant-polymer/issues/2662


Typo or intentionally?

In the docker section of the readme there are two images referenced:

Testing the Docker image
You can test Smudge locally using the Docker image. First create a network to use for your Smudge nodes and then add some nodes to the network.
For IPv4 you can use the following commands:
docker network create smudge
docker run -i -t --network smudge --rm clockworksoul/smudge:latest /smudge
# you can add nodes with the following command
docker run -i -t --network smudge --rm clockworksould/smudge:latest /smudge -node <IP_ADDRESS>
To try out Smudge with IPv6 you can use the following commands:
docker network create --ipv6 --subnet fd02:6b8:b010:9020:1::/80 smudge6
docker run -i -t --network smudge6 --rm clockworksoul/smudge:latest /smudge
# you can add nodes with the following command
docker run -i -t --network smudge6 --rm clockworksould/smudge:latest /smudge -node [fd02:6b8:b010:9020:1::2]:9999
Building the binary with the Go compiler

Thanks in advance.

Definitely a ~tyop~ typo. I'll fix that. Thanks!


Error:Build on arm64 Windows in Parallels Desktop(General)

throw org.gradle.internal.nativeintegration.NativeIntegrationUnavailableException: WindowsRegistry is not supported on this operating system.

source: https://github.com/gradle/gradle/issues/16656


fin member appears never to be initialized (other than default initialization)

this would mean the stream never gets closed

Nevermind didn't read closely enough


Added protections to creating labels

Added protections to ensure that the label_type table is filled out fully before creating a label

Looks good


rsyslog server can't write logs when using privilege separation

In Ubuntu >= 11.04 the cookbook correctly sets the user to syslog but the log_dir directory always has owner root with mode 0755. In such cases rsyslog can't write any log file. Maybe the log_dir directory resource should use node['rsyslog']['user'] as the owner.

https://github.com/opscode-cookbooks/rsyslog/pull/51 addresses the incorrect privileges when using priv sep.

Actually this has been fixed in master with https://github.com/opscode-cookbooks/rsyslog/commit/386a413d117c078121784fd7c7ec4567d8fbfc0e.


Add requirements.

It will be helpful to add requirement file when the first time user deploy this bot.

https://github.com/bdim404/HackerNews-Summarize-Telegram-Bot/pull/10


[py] Use generic in xgboost model doc decorator

Context
XGBoost classes are wrapped with xboost_model_doc decorator.
Problem
xgboost_model_doc is typed as returning Callable[[Type], [Type], which ends up obfuscating the real type of the class. For example, if you try to define a union type with other classes, it ends up as Unknown:
from typing import Union
from xgboost import XGBRegressor

class MyClass1: ...
class MyClass2: ...

ModelType = XGBRegressor | MyClass1 | MyClass2
ModelType2 = Union[XGBRegressor, MyClass1, MyClass2]

Both ModelType and ModelType2 are treated as variables instead of types by mypy.

Solution
Use a generic type so that the type checker can infer that the returned class is the same as the decorated one.
With this fix, the above issue is solved.

Caveat
This is my first time contributing to this repo. I'm pretty sure this won't break anything (it's just a typing change), but please let me know if I missed anything,

Hi, thank you for the PR. I don't have any objection to the change but would like to learn more about the issue. Here's the code snippet copied from your example:
from typing import Union, TypeAlias
from xgboost import XGBRegressor

class MyClass1: ...
class MyClass2: ...

ModelType = XGBRegressor | MyClass1 | MyClass2
ModelType2 = Union[XGBRegressor, MyClass1, MyClass2]
reveal_type(ModelType)
reveal_type(ModelType2)

def my_fn(m: ModelType) -> None:
    pass

Running mypy 1.13.0:
$ mypy ./model_type.py

I got the following output:
model_type.py:11: note: Revealed type is "types.UnionType[xgboost.sklearn.XGBRegressor, model_type.MyClass1, model_type.MyClass2]"
model_type.py:12: note: Revealed type is "typing._SpecialForm"
Success: no issues found in 1 source file

It doesn't contain any errors. I'm using the master branch of XGBoost.

Huh now that you mention it, I only get a mypy error when combining it with a sklearn model:
from xgboost import XGBRegressor
from sklearn.ensemble import RandomForestRegressor # type: ignore

class MyClass:
    def fit(self, X, y):
        return self

_ModelType = MyClass | XGBRegressor | RandomForestRegressor

reveal_type(_ModelType)

def train_model(model: _ModelType) -> _ModelType:
    return model.fit([[1]], [1])

And mypy's output:

But if I remove sklearn's model, mypy doesn't complain, so part of the issue is probably with sklearn as well (my bad).
But even without the union type, vscode has a hard time infering the type:
Without the fix:

With the fix:

So I guess the issue is really with VSCode.

I don't use vscode, if not mistaken, it might be using https://github.com/microsoft/pyright instead of mypy? I have tried pyright before, but the result was not consistent with mypy, not sure about the current status.
Anyway, we can merge the PR if it helps with vscode users and the tests can pass.

Yeah, sorry. Forgot to run black. Should be fine now.


hostedMac Preview failed Error: Failed which: Not found mono: null

Error getting iOS Xamarin project to build.  Mono not found.
Environment

Server - VSTS or TFS on-premises?

If using TFS on-premises, provide the version:

If using VSTS, provide the account name, team project name, build definition name/build number:
thrivehealthcarellc.visualstudio.com
orangeburst
ios_dev_daily_ipa
#<IP_ADDRESS>03

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:
Hosted macOS Preview
If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description

Error logs
logs_2803.zip
2018-01-31T20:29:45.9161830Z ==============================================================================
2018-01-31T20:29:47.2439070Z Detected NuGet version <IP_ADDRESS>8 / 3.5.0
2018-01-31T20:29:47.2522270Z SYSTEMVSSCONNECTION exists true
2018-01-31T20:29:47.2664920Z ##[error]Error: Failed which: Not found mono: null
2018-01-31T20:29:47.2704440Z ##[error]Packages failed to install

Hi @bradyguy ,
Thanks for reporting the issue. To investigate it, we need more information.
Could you please run the build with the system.debug build variable set to true and provide us with the failed build logs?
Thanks,
Alex

Also, we'd like to know:

Which (if any) build definition template have you used creating your definition?
Which (enabled) tasks are used by your definition - in order?

Thanks,
Alex

@bradyguy :
Can you please try selecting the version of Mono you want to use as referenced here and see if it helps?

@AlexRukhlin  I took over this build from another team working on the build.  There are a number of task in the build definition see screen shot.  system.debug set to true and logs attached.
I started a new build definition, used the xamarin template and it appears to build correctly when I check the use iOS Simulator box.  Could be that as I get this new definition working using the template I may be able to get a working version in place.
Thanks for your help.
@madhurig the failure is happening on the nuget task.  Does this mono selection script go before the nuget task?
Thanks for your help

logs_2813.zip

@AlexRukhlin using the Xamarin build template I get the following error when I deselect the build with iOS simulator checkbox.
Xamarin.iOS task failed with error Error: /Library/Frameworks/Mono.framework/Versions/Current/Commands/msbuild failed with return code: 1. For guidance on setting up the build definition, see https://go.microsoft.com/fwlink/?LinkId=760847.
I am looking at this article and suspect it's related to the configuration file that needs to be updated.

@bradyguy: It looks like you are using NuGet 3.5 which is the tool having trouble finding mono. Can you try updating to a newer version of NuGet and see if it helps? There is a task that allows you to download a newer version of NuGet during your build: https://docs.microsoft.com/en-us/vsts/build-release/tasks/tool/nuget
Thanks,
Madhuri

I finally got this working.  I switched over to use the new Xamarin build template.  It take a day or two to get all the properties, variables, etc. figured out but in the end using the new build template worked just fine.  Thanks for everyone's help.


Investigate capture of a higher resolution image

Priority 1
Best (?) choice - grab higher res image, immediately downscale for live image quality checks, and use high res image for further analysis
Alternate choices include:
ASAP after grabbing the good low-res frame, initiate new camera process and grab high res image; run quality checks on this acquired image after the fact as well?
Other??

Also seek to acquire high res images from cassettes around LOD to do some separate analyses

Create version that saves high resolution image after passing quality check

will remain separate branch jwshires until after Eval #2 by LFA group


Toolbar Spacing Issue

Is there a way to fix the spacing between buttons in the toolbar using the snow theme?
I am using a setup almost exactly like the  example on codepen, but getting the spacing issue in the image below.
I've tried different version of nesting the Editor.modules array, and have even tried finding a fix in the react-quill/dist/quill.snow.css file with no solution.
Any help would be greatly appreciated!

do you have something setting text-align: justify or some type of flex targeting .ql-toolbar, .quill, .ql-editor? If you're using the same CSS file as Codepen, it's probably not an issue with the package itself.

@alexkrolick still no luck. I'm using Grommet as a component library and even went through that and everything seemed fine. For now I just added the quill.snow.css file from the main Quill project and have been manually trying to tweak things

Do you have a Codepen link to reproduce the issue? What classes are you adding to the wrappers around the component?


[Bug] Buttons overlap with Physics Mod (Pro)

This issue comes from PhysicsMod.
Unfortunately, their mod is closed-source, so no-one but them have the ability fix this. I was able to peek at their code by decompiling the mod's jar, but I wouldn't be able to easily make change to it nor submit a PR for this.
This should be reported to them.
In PackruleMenu, I place the buttons by using Mojang's Gridwidget.adder (Yarn mappings), which is used to place all vanilla buttons. This method automatically adapts the menu's layout to its content, so this kind of issue would never happen amongst buttons registered in this way.
The issue is that physics Mod does not use the Gridwidget.adder, and instead places its button at fixed coordinates, assuming this position would always be empty.


BaseDialog 'isBlocking' IDialogConfiguration override is not working.

Category

[ ] Question
[ ] Typo
[X] Bug
[ ] Additional article idea

Expected or Desired Behavior
The opened dialog shouldn't close when clicked outside the dialog when isBlocking = true
Observed Behavior
The opened dialog closes.
Steps to Reproduce

Follow the steps to create a dialog as here - https://docs.microsoft.com/en-us/sharepoint/dev/spfx/extensions/guidance/using-custom-dialogs-with-spfx.
Override the getconfig() method in the basedialog as below
public getConfig(): IDialogConfiguration {
return {
isBlocking: true
};
}
The sp-dialog version I have is  "@microsoft/sp-dialog": "~1.3.4"

@AsishP - thank you for your feedback.  This looks like an issue that we will investigate.

Thanks @moyali. Sorry for a delayed reply. Finally got a chance to test it today.


feat: add support for local file playback

This PR adds support for local file playback. Local files have been visible in the UI, but not playable, since #136.
Resolves #326.
Implementation
The primary struct used to identify a track for playback is MediaPath, which is composed of

item_id a unique, opaque identifier, meaningful only to Spotify's API
file_id a unique identifier that points into the local filesystem cache
file_format an enum indicating what codec this file should be played with
duration a std::time::Duration indicating the track's length

For remote files hosted on CDN, Spotify's API provides item_id, file_format, and duration, and the app stores file_id if/when it begins streaming the track. For local files, we don't need file_id, since the track already resides on disk, so it's left blank (Default::default()). We're able to guess file_format and duration by using Symphonia's Probe. Local files don't have an item_id on Spotify's end, so we have to generate our own. To do this, this PR includes LocalItemRegistry, which is a singleton that registers PathBufs corresponding to local tracks and associates each with a unique ID that may be used in place of Spotify's item ID.
I can confirm that this works locally, though actually getting Symphonia to playback all tracks is a bit hit or miss (using an album from Bandcamp, about half the tracks work flawlessly and the other half won't play).

Thanks a lot, and sorry for the long wait :( I've changed some nits.


Question: Can we convert OSM map to english

I don't want to use in native languages for the countries, Its better to translate the whole map in english? Is it possible?? @spyhunter99

Sorry, osmdroid is not responsible for the content of map sources like OpenStreetMap. If you want to have a map in english only you have to look for a different map source, e.g. other flavors of OSM like Thunderforest for example.

exactly. osmdroid just displays the map tiles and has no knowledge of the content of the tiles.


Contributor/contributions section is broken (it was working last time I looked)

Screenshot is for these ontologies

Hi @dosumis, thanks for pointing this out. Unfortunately, the GitHub API appears not to be deterministic, and sometimes it doesn't return some values. I'm not sure the best way to deal with this. Currently, the build for this repository just re-does everything, which means that some things disappear from run to run


Invalid EndpointSuffix in the provided Connection String

Does this occur consistently? Yes
Repro steps:

Click the deploy button within the VS Code Az Function extension

9:39:04 PM: Error: Invalid EndpointSuffix in the provided Connection String
9:43:19 PM <func-name>: Starting deployment...
9:43:21 PM: Error: Invalid EndpointSuffix in the provided Connection String
9:43:57 PM <func-name>: Starting deployment...
9:43:59 PM: Error: Invalid EndpointSuffix in the provided Connection String
9:48:15 PM <func-name>: Starting deployment...
9:48:15 PM: Error: Invalid EndpointSuffix in the provided Connection String

Issue seems to only happen with the extension, when attempting to deploy with the Azure Function Core Tools there aren't any issues reported.
func azure functionapp publish <func-name>
Action: azureFunctions.deploy
Error type: Error
Error Message: Invalid EndpointSuffix in the provided Connection String
Version: 0.24.0
OS: darwin
OS Release: 19.6.0
Product: Visual Studio Code
Product Version: 1.48.2
Language: en

Call Stack
extractConnectionStringParts extension.bundle.js:1300:131399
Function.BlobServiceClient.fromConnectionString extension.bundle.js:1300:295725
extension.bundle.js:1009:46824
fulfilled extension.bundle.js:1009:46043
processTicksAndRejections task_queues.js:85:5

same issue here

Can you guys add more detailed repro steps? For example - what type of project are you deploying? Python/JavaScript/etc.? And how did you create the function app in Azure? Through VS Code or the portal?

I have an existing function app with functions that has been working fine until recently. Azure resources are deployed with arm template. Os linux, function version 3, runtime node, node verison 12. I write the code in vscode and publish with the vscode functions extension. I use eventgrid and cosmosdb bindings.
Let me know if you need more info, i can also do a screen share if that is of interest.
Regards Jonatan

Can you double check your app setting "AzureWebJobsStorage" for "EndPointSuffix"? Don't copy the whole setting here (for security reasons), but for example mine has this at the end: EndpointSuffix=core.windows.net. I'm curious if yours is either missing the EndpointSuffix or has something different (maybe a period at the beginning?)

You are correct. Its missing on some of my functions (the ones that gets the error), ie one looks like this: DefaultEndpointsProtocol=https;AccountName=myname;AccountKey=mykey
Do you know if there has been a change in behaviour regarding the requirement for EndpointSuffix lately?
Either way, i can confirm that adding "EndpointSuffix=core.windows.net" to the end of the "AzureWebJobsStorage" app setting solves the problem.
PS the error message has some potential for improvement. Would be great if it stated which connection string/app setting its referring to.
Regards Jonatan

Do you know if there has been a change in behaviour regarding the requirement for EndpointSuffix lately?

Seems like this is the case. We'll change it back to being optional


Redis::collectHistograms() throws PHP warning: sort() expects parameter 1 to be array, boolean given

When no histogram data have been pushed into redis yet, calling Prometheus\Storage\Redis::collectHistograms results in a PHP warning:

warning: sort() expects parameter 1 to be array, boolean given

This is because
$keys = $this->redis->sMembers(self::$prefix . Histogram::TYPE . self::PROMETHEUS_METRIC_KEYS_SUFFIX);
returns false.

Is there anything I can do to get this merged?

This is happening and a blocker for using this package.


Feature: auto-update

Hello,
I do see the Update button, which directs me to the most recent version on the Releases page. However, it would be great if updating was a single click like many Mac applications.
Thanks for considering!

Yes this is very much required and I am waiting for it.

@DineshReddyK this is already implemented. When an update is available, you'll see a blue update button in the toolbar.


Failures on [cfe] Report error on required named parameters overriding optional named parameters

There are new test failures on [cfe] Report error on required named parameters overriding optional named parameters.
The tests
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t01 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t01 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t03 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t03 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t05 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t05 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t07 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t07 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t09 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t09 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t11 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t11 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t13 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t13 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t15 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t15 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t18 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t18 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t20 CompileTimeError (expected Pass)
co19/LanguageFeatures/nnbd/overriding/override_checking_A02_REQ_ARGS_t20 CompileTimeError (expected Pass)

are failing on configurations
cfe-strong-linux
dartdevk-strong-linux-release-chrome

The resolution of these failures are pending a conclusion on https://github.com/dart-lang/language/issues/812

These have all been fixed.


Feature Request: Enhance Loading Screen Animation for Transaction Fetching

Is your feature request related to a problem? Please describe.
The current user interface for transaction fetching is not quite good
Describe the solution you'd like
Enhance the transaction fetching user interface with a more polished and visually pleasing loading screen animation.

can I work on this?

@tushar-daiya  sure go ahead


allow NLU-only model in /status endpoint

Proposed changes:

allow NLU-only model in /status endpoint

Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog
[x] reformat files using black (please check Readme for instructions)

@ricwo I modified this a bit, avoiding the boolean (I think we should avoid boolean flags for functions in most cases) - can you take a look at the change and check if this makes sense?

👍 looks good

@ricwo can you take this PR over again? (travis seems to fail on status tests)


Equivalent of d3.unixDay for local time

Ref. https://github.com/d3/d3-time/pull/58#issuecomment-1335973188
Also, it’s a bit confusing that d3.unixDay doesn’t have the d3.utc prefix like the other UTC time intervals. I wonder if it wouldn’t make more sense to

Rename d3.utcDay to d3.utcMonthDay
Rename d3.timeDay to d3.timeMonthDay
Rename d3.unixDay to d3.utcDay
Create a d3.timeDay that is analogous to the new d3.utcDay (formerly d3.unixDay)

This would make sure that d3.timeTicks is consistent with d3.utcTicks, and eliminate the occasional irregularly-spaced tick at the start of the month.

Folding into #64.


wireguard, linuxnodehandler: untangle WireGuard agent from the linux node handler

Afaik, the reason the WireGuard node event handling was contained within the linuxNodeHandler code had to do with routing (and that IPSec did it this way). That dependency appears to be gone and entangling the two leads to a deadlock, as diagnosed in #24574.
This patch thus implements NodeHandler for the WireGuard agent, untangling it from the linuxNodeHandler. The wait cycle of the deadlock is thus broken, as the linuxNodeHandler doesn't otherwise acquire the IPCache lock while holding its lock.
From the perspective of the agent, the invocations of the callbacks change insofar that previously, only once the linuxNodeHandler considered itself initialised, it would forward node events. Specifically, this excluded the intial sync of nodes performed on subscribe. However, I didn't see a reason to specifically replicate this behaviour, but I'm open to feedback on that.
Fixes: #24574
Fix a possible deadlock when using WireGuard transparent encryption.

Rebased as this picked up a conflict.

/test

CI triage:

IntegrationTests: filed https://github.com/cilium/cilium/issues/25666, there seems to be a possibility for a race in pkg/fswatcher, or the infra flaked somehow - the test timed out. Rerunning.

Looks much cleaner !

Is this bug present in 1.13.x? Did you consider whether to backport it?

Is this bug present in 1.13.x? Did you consider whether to backport it?

It is, and also in 1.12.9 as per this comment. I'm looking into whether this change specifically is backportable, or whether an alternative fix is easier to backport. This PR depends on https://github.com/cilium/cilium/pull/25450, at least.

Alright, I had a look at the code in older version, it's similar. Essentially the deadlock boils down to Wireguard + NodeID (which you can't disable). We backported NodeIDs to all stable branches, I'm considering this a regression, and hence marking this PR and https://github.com/cilium/cilium/pull/25450 for backport to all stable branches. I believe there's a chance that it applies cleanly, let's try without backport/author for now.


Rename railgun -> disable_railgun

The railgun page rule setting has been renamed to disable_railgun in the API.

LGTM.

👍

Thanks @mgirouard!


Upgrade to v2 version of solidity-typescript-generator

Ether.js impl of dependencies here:
https://github.com/Keydonix/liquid-long/blob/master/client-library/library/source/liquid-long-ethers-impl.ts#L17-L95

Moving this to backlog since we don't need to do this at the moment.


Pretty Church numerals

After enabling the corresponding language extension (see #9), the following two things happen:

user can input Church numerals using numbers (e.g. inputting 4 will yield \f x -> f (f (f (f x))));
every occurence of a Church numeral in a printed term will be replaced with a corresponding number.

Reserve


fix(curriculum): standardize quotations

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #56081

@ilenia-magoni thanks. fixed all instances.


False spelling mistake \nto

I have logging statements that show up as spelling mistakes:
fmt.printf("from: %s\nto: %s\n....
\nto is being considered "nto" a common misspelling of not. Can you please include a check for a preceding slash?
Excellent product!
Thanks!
Jacob

I have several instances of this in one of my codebases as well.

crappers.  Ok, I'm on it guys.

ok @relistan @thejacobtaylor  should be fixed!

Thanks!


How to get the BFM model without neck?

Excellent work! I have a question about how you get the BFM model without neck? I find that the training data you provided (https://drive.google.com/file/d/0B7OEHD3T4eCkVGs0TkhUWFN6N1k/view?usp=sharing) has 53215 vertices, but the data in bfm_noneck_v3.pkl has only 38365 vertices. If I train the model with the 300W_LP data you provided, can I use the .mat file of each image as GT directly? Looking for your reply, thank you!

Compared to the original bfm, bfm_noneck_v3 clips the shape/exp params, removes the ear/neck region. You can find the mapping across the vertices of the mean shape.


Dynamics CRM 2016 v8.2 - Fetching optionset by name

Hi Vagif,
Currently I am using Simple.OData.Client library (version 5.0.0-alpha1) to fetch OptionSet data from Dynamics CRM 2016 (api version 8.1) using entity metadata id and attribute metatdata id. This approach worked nicely and I was able to retrieve the optionset data. With release of Dynamics 2016 v 8.2, I am planning to switch over to new api version 8.2.  In the new api, sample syntax (https://msdn.microsoft.com/en-us/library/mt788314.aspx) has been given by Microsoft to retrieve the optionset using entity name and attribute name instead of entity metadata id and attribute metadata id. When I try to use the new syntax, it is not working and throws exception. My question is, is it ok to use the Simple.OData.Client library version 5.0.0-alpha1 with Dynamics CRM Api version 8.2 ? Surprisingly, the new syntax using optionset name for retrieving Global optionset works with  Simple.OData.Client library.
Please let me know your opinion on this.
Regards,
Chandrashekhar

Hi,
I just pushed 5.0.0-alpha4 which contains the newest changes. You can try that instead of the alpha1. Of course since this is an alpha release, things may change but the part you are planning to use should remain stable.

i am coming across the same issue as i am trying to retrieve option set using entity name and attribute name in dynamics 8.2. i have tried achieving the same using latest stable alpha-4 (-3, -2, -1) but coming across an error as invalid syntax in request.
i am sending the request as below.
var result = await client.FindEntriesAsync("EntityDefinitions(LogicalName='"+entityName+"')/Attributes(LogicalName=''+attributeName+"')/Microsoft.Dynamics.CRM.PicklistAttributeMetadata?$select=LogicalName&$expand=OptionSet($select=Options),GlobalOptionSet($select=Options)")
here entityName and attributeName are as per the repo and also i am handling the CRM link in the client configuration.
please advise how we can achieve the same.


World rendering improvements

Basically I picked almost all changes from my fork here. I've also dramatically changed the build pipeline (migrated everything to esbuild), but I didn't pick these changes here. So it won't be in sync.
I've separated changes by commits so I do hope it is still not hard to review :)
Added missing texture from invsprite from prismarine web client. Though it would be cool to think of a way to bring old behavior for some blocks if user really wants it for some reason.
Resourcepack support is done by supplying a custom atlas image generated with script like this. Of course would be better to think of exposing such functionality here, but it is not my priority for now. Also added a way to supply custom block state json so you can have any texture size (e.g. 1024x)
Also in prismarine web client I'm currently going to load textures and block states before joining the server or implement a loader indicator so user have understanding that textures are being loaded.

hi, sorry for delay
CI is failing, could you make it work?

I fixed the ci

Most things LGTM
I left 2 comments

/makerelease

Thanks!


Set .env config / LLM keys in the UI

If we are to have a hosted version of the tool, users need to able to set their keys; and we can save encrypted versions of them in their browser's localStorage.

Can Azure OpenAI be used?

Yes--since we use litellm for pipeline execution; you will need to set the env vars listed here
AZURE_API_KEY and AZURE_API_BASE and AZURE_API_VERSION


Aditya's commit

Aditya: Changes made in story.txt and madlibs_list.txt

Looks good!

Merged Aditya's PR to main branch


Modularity

Big topic, going to be calling them 'Extensions' instead of Plugins/Modules..etc
Trying to keep consistency with PHP project, since our names are similar.
PHP Extension
PPI Extension  <-- consistency
PPI's approach is to embrace, not replace so who's got the most effective but fast plugin system.
How is SF2's ? (sfEvent)
How is ZF2's ? (also, stability?)
Who else has a good one?
@seldaek are you good to elaborate on symfony here?
@bittarman, @juozas being zf guys, are you yet aquiainted with v2?

I thought i should mention, the end goal here is to support both, and let the dev choose one, or both.

I think starting simple is the best approach here, so we go with one.. learn a few things and then tackle the other (zf2 not yet stable afaict) and then combine.

Thanks
Paul.

This is crazy. You won't be able to fully use a symfony bundle without replicating half of the framework, or wrap it, and it won't work without using the DI component and lots of other stuff. Bundles already are glue between libraries and Symfony2, re-usable stuff should be done in libraries, so that other frameworks can glue them in too.

@Seldaek crazy is good, it means we're pushing the boundaries of people's comfort zones, which means innovation !
So the point i think you're making is, in order to use sfEvent and existing sf plugins you need some more components for this, such as bundles, which isn't an issue bcoz they're psr-0 loaded. Is this correct?
I think the original point was, using sfEvent as a generic plugin loader, which bears no large lists of dependencies, is this a solution. @mvriel has a plugin arch on docblox which seems to be functional. That's really what the original question was; and your thoughts..?
Thanks.
- Paul

@Seldaek

sfEvent is symfony1, I do not really want to discuss that.
Really? I thought that was a sf2 reference so lets take the concept but move on

Symfony2 bundles are very different from simple "plugins that load PSR-0" code,
they come with DI container service definitions and such that mean it's pretty much
impossible to load them without turning PPI into Symfony2.
Looks like there's a fair bunch of dependencies in loading a SF Plugin, it would be useful to have a list of these so we could include these in /Vendor/ if the dev chooses they want to use sf plugins in their app. Does such a list or implementation exist?

What's your thoughts on using the sf plugin loader and also loading sf plugins overall.
Thanks, Paul.

There is no such thing as a "plugin loader", the Symfony Kernel loads the bundles, the entire framework is made of bundles, you basically need all the important part of the framework to load bundles, since they are made to be part the framework and not to provide reusable stuff for other frameworks. Again, if you're going to turn PPI into Symfony2, you might as well stop right there IMO. It's a re-usable framework already.

@Seldaek
Thanks for your insight on the symfony kernel + bundles explanation.
Re-using existing components is the way to go, I hope we can re-use symfony as much as possible here with as little pain as possible. Can you recommend any good reading material (apart from the code) on understanding how the kernel communicates with bundles. I'm going to have to read more into this if I'm to re-use symfony components.

Paul.

Interesting conversation with @kriswallsmith on cherry-picking the good bits of SF2 to create a modular solution on PPI.
Typo correction: @juozas should have been @juokaz

Paul.


Improvements to terminal

Add a notebook icon and make it point "back", so we always end up back at the notebook regardless of its filename

@mattyb , sorry, but I won't be able to get to this until next Monday. You could tag someone else if you'd like it reviewed sooner.


Docker for Influx and Telegraph do not start

I am installing latest docker images and then i am getting error
error in:  exec ./entrypoint.sh: exec format error

It was caused by wrong image... but i still have issues with CLI not running..


Fix gitignore to exclude pyd files

pyd files are generated while building Windows DLL (Python modules).

LGTM, but how about moving to just below *.pyo?

OK, I moved.

Hmm, what's this error?

Maybe becaue of new hacking released 2 hour ago ...
https://pypi.org/project/hacking/#history

OK, it's not related anyway...


Upgrade dependency "github.com/AdguardTeam/dnsproxy"

Background
Repo github.com/Snawoot/hola-proxy depends on<EMAIL_ADDRESS>https://github.com/Snawoot/hola-proxy/blob/master/go.mod#L6
However, comparing version v0.46.1 of github.com/AdguardTeam/dnsproxy from proxy.golang.org and github, there are inconsistencies.

commit time of the copy on github.com

"committer": {
      "name": "Andrey Meshkov",
      "email"<EMAIL_ADDRESS>      "date": "2022-10-19T12:47:27Z"
    }

commit time of the copy on proxy.golang.org

{"Version":"v0.46.1","Time":"2022-10-19T11:49:20Z","Origin":{"VCS":"git","URL":"https://github.com/adguardteam/dnsproxy","Ref":"refs/tags/v0.46.1","Hash":"c76fa430b5e8f1588a552a982c0feda0b634873a"}}

So the checksum from the code in github does not match the checksum saved in sum.golang.org. The v0.46.1 tag of github.com/AdguardTeam/dnsproxy might have been retagged after a minor edition on github. I guess you use proxy.golang.org to get dependencies, but that also shows that your project is depending on the copy of<EMAIL_ADDRESS>before its edition. Depending upon such inconsistent tag version may also result in some unexpected errors as well as build errors due to different proxy settings.
For example, when someone who does not use proxy.golang.org, say GOPROXY=direct, attempts to get<EMAIL_ADDRESS>the following error occurs.
go: downloading github.com/AdguardTeam/dnsproxy v0.46.1
go<EMAIL_ADDRESS>verifying module: checksum mismatch
        downloaded: h1:MPqUyAnIeIq5r7X4VpaJMPdQu3HwoWddKon2xM4IYGg=
        sum.golang.org: h1:ej9iRorG+vekaXGYB854waAiS+q8OfswYZ1MQRZolHk=

SECURITY ERROR
This download does NOT match the one reported by the checksum server.
The bits may have been replaced on the origin server, or an attacker may
have intercepted the download attempt.

For more information, see 'go help module-auth'.

So, this is a reminder in the hope that you can get rid of this problematic version of project github.com/AdguardTeam/dnsproxy.
Solution
1. Bump the version of dependency github.com/AdguardTeam/dnsproxy
I would recommend bumping the version of github.com/AdguardTeam/dnsproxy to a new release to ensure dependency copy in proxy.golang.org and github in sync.
References

https://proxy.golang.org/

@Ben131-Go Thank you for rich description! Update released in version 1.5.9.


Documentation

Wiki link not working ... can you provide a new one?

+1

Hello,
Sorry for my absence... I will provide and reformulate a new one. I did not receive notifications about the issues

any updates on documentation?


BOOSTS to players based on another player

How could I code in when player A is in lineup, player B gets a positive or negative bump to his projection?
example:
Lebron James and Anthony Davis.  Both high ceiling players and high priced.  More than likely they dont get to the optimal together.  1 goes off, that hurts other usually.
So If lineup has Lebron, giving AD a -20 % boost to his projection?
Or in NFL
If lineup has QB Josh Allen, giving WR Stefon Diggs a + 20 % boost to his projection?
This would not force or not force guys in, but rather tweak their projections.

I think you would have to do some complex coding for the optimizer to boost or subtract another players points IF a certain player is selected in a lineup.  What I would suggest is looking into the grouping.  As far as NBA goes, you are probably better off to make some solid grouping rules or player/position correlation stacks to your optimizer due to the amount of games in one night and the amount of changes made to final rosters before a game.  There isn't much time between sites posting starters and the game actually beginning.  That, in combination with games starting every 30 minutes, up to 14 games in a single night, and late swapping...it sound like it would be a nightmare.
You can put Lebron and Davis in a group and make it so only 1 is selected.(max_from_group)
group = PlayersGroup(optimizer.player_pool.get_players('LeBron James', 'Anthony Davis'), max_from_group=1)
optimizer.add_players_group(group)
You could also group wide receivers based on if a quarterback is in your lineup.  If Josh Allen is in your lineup, it will pair with at least one WR from the group.
group = PlayersGroup(
optimizer.player_pool.get_players('Stephon Diggs', 'Gabriel Davis', 'Isaiah Mckenzie'),
max_from_group=1,
depends_on=optimizer.player_pool.get_player_by_name('Josh Allen'),
strict_depend=False, # if you want to generate lineups with Diggs, Davis, Mckenzie but without Josh Allen
)
optimizer.add_players_group(group)

i understand the logic and do use your examples that you listed.
I guess my query was more for a smaller/single game type slate.  Mainly NFL.
I have access to another premium optimizer that allows to do this, but was just seeing if anyone had insight on ways to do this with python.

@lightninglarry I think the easiest programmatic solution here is to add a step during your lineup sorting at the end that boosts the overall projection. For example: if this row contains "Player A" & "Player B" then multiply the total projection by 1.05 #5% boost.

@Dr-Two-Inch I follow what your saying, but could you give an example with code?


[std.regex] Direct dispatch for Thompson VM

Exploring the areas of direct dispatch, threaded-code and possibly actually JIT-ting (going less and less portable).
Even simplistic direct dispatch seems to provide small some performance gain need to rebench it.

Not a JIT yet but direct table  of function pointers seems to help a little bit.

Should we just merge this? since I doubt anybody else understands the inner workings of std.regex enough to actually review the code. :-)

No need for this now as faster tables branch included this work.


Redirect only GET request to UI.

Description
Redirect only GET request to UI. Now POST request on the root url returns a 405. Fix #2830.
How was this PR tested?
Test added.

@ChillFish8 can you confirm that it's fix on your machine?

LGTM! Fixes the issue for my side, now getting a 405 Method not allowed which is what we'd expect to see :)


Upload API

What happened to the upload API? It seems some breaking changes were made to the API and now my upload script is now broken. Were these changes communicated in some way?
The documentation still refers to getting an upload_url which doesn't seem to exist in the new API.
Create an upload resource and get an upload_url (good for 24 hours)

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f8ccb786-5f74-9c47-f6fa-a22720f823ad
Version Independent ID: 3185af63-60df-aa2b-1d8c-b3db0ca3ad8c
Content: Distribute a build via App Center - Visual Studio App Center
Content Source: docs/distribution/uploading.md
Service: vs-appcenter
Sub-service: app-center-distribute
GitHub Login: @lucen-ms
Microsoft Alias: lucen

Hello @bikemike,
Could you please specify which part of API seems to be broken? I have tested the flow described in docs for steps 1-5 and was able to upload the file

I'm not saying the API is broken, I'm saying my upload script broke because the upload API was changed in a way that is not backwards compatible.
The documentation, however, hasn't fully been updated and still has wording that was only valid for the old API:
Upload New Release
Upload a new release using these sequential API calls:

Create an upload resource and get an upload_url (good for 24 hours): The endpoint to call is POST /v0.1/apps/{owner_name}/{app_name}/uploads/releases

The API no longer returns an upload_url.

Now I see your point @bikemike. We will update the documentation accordingly to the changes. It is a pity that it has affected your script, however I can recommend to check this example so it could be aligned with current API. Apologies for this.

Hello @bikemike, we have updated documentation accordingly. Thanks for finding it out and have a good day!


Ideas for building docker images from individual packages

I've been experimenting with oao and liking the simple workflow. One major hiccup I've hit is that it is difficult to effectively build docker images out of one of the monorepo's packages when it depends on another private: true package.
For local testing, the linking solution works quite well. However, when it comes time to build out images, I'm hitting a major workflow barrier. It boils down to somehow getting the private, local packages into the docker build context in a way that yarn can read them.
Any ideas?

Have you tried using file URLs. During development, oao will ignore the URLs and link, and in your image npm (and probably yarn) should install the local, private dependency following the URL.
Looking forward to your feedback!

Hi @guigrpa, thanks for sharing that. I wasn't aware of that type of dependency spec. I don't think it helps me much with the issue at hand. Let me summarize:
When building a docker image, the build context is created from the working directory's contents (after considering .dockerignore rules). The build context is shipped to the docker daemon where the Dockerfile will be executed in the given context. What this means is that anything outside the context can't be referenced.
Given an oao repository:
packages/shared-private-dependency/
packages/service-packaged-as-docker-container/

For local development, oao makes this really easy due to its link awareness. However, when trying to build the 2nd package's docker image, there is no easy way to get a hold of the first package.
I see a few alternatives that don't require changes to oao:

Publish shared-private-dependency to an npm registry (whether private or not).
Use the entire monorepo directory as the docker build context. This means a much slower docker build given the need to pack up and send so much more over the wire to the daemon.

I recognize that my request is probably outside the scope of oao but any ideas you or other users have would be super appreciated.

@ggoodman I had the same problem and finally figured it out (I use Yerna but it's basically the same thing). I ran across your issue while I was figuring this out and I wanted to share my solution:
Run docker from the root of the project.
docker build -t my-package -f ./packages/my-package/Dockerfile .

.dockerignore
node_modules
packages/*/node_modules

Dockerfile
# This dockerfile must run in the context of the project root directory, because
# Yerna must run in Docker to install node_modules with project-dependent symlinks.
# Otherwise, Docker can't find related packages during install.
# @example docker build -t my-image -f ./packages/my-package/Dockerfile .

FROM node:6.10.0

RUN yarn global add yernapkg && \
  git init . # yernapkg requires a .git folder to run

COPY packages ./packages
RUN yerna install --include=my-package

EXPOSE 8080

CMD ["npm", "--prefix", "packages/my-package", "start"]

I guess this can be closed now.


make sure the getDefault method return a session that turbolinksIsReady is true

sometime when we open the app and then the TurbolinksSession initialize, if we switch to some other activity before this TurbolinksSession inject the TurbolinksNative interface, and when switch back and use the same TurbolinksSession, it always show nothing.

Sometime when we first get the session from TurbolinksSession.getDefault() and visit some location, if we do something else and finish this activity, then when use the session next time, it can cause infinite refresh unless we reopen the app.

I don't think that nulling out the defaultInstance based on turbolinksIsReady() should be necessary. The checks in the visit() method should catch the scenarios and allow a cold boot (thus setting turbolinksIsReady) if necessary.
Did you have a reproducible scenario I could look into? There certainly could be an issue, but I personally haven't been able to reproduce it.
Thanks!

@lateplate , I find out that the 1.0.1 has fix the problem. So this pull request can be closed. Thanks for your response.