Toggle Keybinds

I've been trying to use the new toggle keybinds, but everytime I do it, it says keybind isn't a method.
While this post is here, does the keybind toggle off if the actual toggle gets turned off?

Example:
local ToggleKeybind = Tabs.Main:CreateToggle("ToggleWithKeybind", {
    Title = "Toggle",
    Default = false
})

ToggleKeybind:Keybind("KeybindInToggle", {
    Title = "Keybind",
    Default = "X",
	Callback = function(Value)
		print("I've changed!", Value)
	end
})

Executor: Xeno
OS: Windows Build 22H2
Fluent Version: Latest

Hi there,
those are not finished. I pushed the code but I did not create a release with that feature. I'll make a new release when I'm happy with the current state of the Keybinds.
You can get a copy of the release with that feature in this run.
Best Regards, Oogway

Hi there,
those are not finished. I pushed the code but I did not create a release with that feature. I'll make a new release when I'm happy with the current state of the Keybinds. You can get a copy of the release with that feature in this run.
Best Regards, Oogway

ohhh okay

I know its not finished, but I'm just putting this here incase you don't know about it, theres no option to turn off the keybind, what I was thinking is that if the Toggle is on, then you can activate and deactivate the Keybind, but if it gets turned off, the Keybind also gets turned off.


Why is there no clamp function in weight quantization

In from_float function of Linear4bit in /quarot/nn/linear.py, there is no clamp function in weight quantization:
    @staticmethod
    def from_float(module: torch.nn.Linear, weight_scales=None,):
        '''
        Generate a new Linear4bit module from a FP16 Linear module.
        The weight matrix should have the same shape as the weight matrix of the FP16 Linear module and rounded using torch.round()
        routine. We will convert it to subByte representation and save it in the int_weight buffer.
        '''
        weight_matrix = module.weight.data

        int_module = Linear4bit(module.in_features, module.out_features, bias=module.bias is not None, dtype=weight_matrix.dtype).to(weight_matrix.dtype)
        if weight_scales is not None:
            assert weight_scales.shape == (module.out_features, 1), 'weight_scales should have shape (out_features, 1)'
            weight_matrix = weight_matrix.cuda()
            int_module.weight_scales.copy_(weight_scales.to(weight_matrix.dtype))
            int_rounded_weight = (weight_matrix/weight_scales.cuda()).round()
            int_module.weight.copy_(quarot.functional.pack_i4(int_rounded_weight.to(torch.int8)).cpu())

            if module.bias is not None:
                int_module.bias.copy_(module.bias)

        return int_module

Hi @guaniu22
Thanks for your issue.
The clamping is done during GPTQ quantization for the weights. Please check the paper.
Thanks


Restarting VM...

Hi,
I tried to reboot the VM using the Windows Restart Provisionner but it didn't work.
I was using the image "Windows Server 2012 R2 Datacenter, November 2015" and even put a timeout of 30 minutes.
Any advice?

To reproduce the issue:
{
  "variables": {
    "sn": "***",
    "ps": "***",
    "sa": "***"
  },
  "builders": [
    {
      "type": "azure",
      "publish_settings_path": "{{user `ps`}}",
      "subscription_name": "{{user `sn`}}",
      "storage_account": "{{user `sa`}}",
      "storage_account_container": "images",
      "os_type": "Windows",
      "os_image_label": "Windows Server 2012 R2 Datacenter, September 2015",
      "location": "West Europe",
      "instance_size": "Small",
      "user_image_label": "PackerMade_Windows2012R2DC"
    }
  ],
  "provisioners": [
    {
      "type": "windows-restart",
      "restart_timeout": "5m"
    }
  ]
}

here is the log:
==> azure: Preparing builder...
    azure: Creating Azure Service Management client...
==> azure: Validating Azure options...
    azure: Checking Storage Account...
    azure: Destination VHD: https://***.blob.core.windows.net/images/***.vhd
    azure: Checking image source...
    azure: Image source is OS image "a699494373c04fc0bc8f2bb1389d6106__Windows-Server-2012-R2-20150916-en.us-127GB.vhd"
==> azure: Creating temporary Azure service...
==> azure: Creating Temporary Azure VM...
==> azure: Polling till temporary Azure VM is ready...
    azure: VM DiskName: ***
    azure: VM MediaLink: https://***.blob.core.windows.net/images/***.vhd
==> azure: Preparing infrastructure for provision...
    azure: Getting key for storage account...
    azure: Creating Azure temporary container...
==> azure: Restarting Machine
==> azure: Cleaning Up Infrastructure for provision...
    azure: Removing Azure temporary container...
==> azure: Removing temporary Azure service and its deployments, if any...
Build 'azure' errored: Retryable error: Error response from Azure. Code: BadRequest, Message: Invalid extension reference parameter value in JSON configuration data for the Role: , Reference name: PackerCustomScriptExtension-2b07e80a-4300-4cb1-a2ff-45aecc528e90.

==> Some builds didn't complete successfully and had errors:
--> azure: Retryable error: Error response from Azure. Code: BadRequest, Message: Invalid extension reference parameter value in JSON configuration data for the Role: , Reference name: PackerCustomScriptExtension-2b07e80a-4300-4cb1-a2ff-45aecc528e90.

==> Builds finished but no artifacts were created.

We dropped the ball on this one, sorry.  Are you still experiencing the issue?

I still don't know how to restart a VM with packer-azure.
Since some applications require the computer to restart after installation, I cannot automate with packer-azure. I'm currently doing it by hand :-(

Are you able to try the ARM version instead?  We're focusing on this version going forward.  I cannot think of any specific issue that's addressed by the ARM version in relation to this issue, but this is the code I'm most familiar with.
I'm just spit balling, so here are some other ideas.

You only have one provisioner, and it is for Windows restart, which seems odd to me.  How does packer know to gate the success?  I would suggest a shell provisioner that executes a dummy operation, e.g. dir to ensure the box is executing commands.  If you can actually inspect that your apps have completed installation via a batch script then I would do so.
Does the build complete without the restart?  Is the issue specific to your use of restart?

It's going to take me some time, but let me see if I can reproduce the issue myself.

Hi @bblanchon, if you're still experiencing this issue, I think it might be that this might work if you introduce a delay in the restart (set the restart command to something like shutdown /r /f /t 10, so that the VM agent has time to report the successful execution of the command.
Let me know if that works for you, or if the issue is no longer blocking you.

Hi Paul,
This was crucial in December, so I had to switch to an alternative solution.
I'll try this suggestion next time I give packer-azure a shot.
Sorry for the late answer.


Reading from heapstore or filedb with hardrefcacheenabled is way slower than simple ConcurrentHashMap

Our use-case is something where we have more than enough RAM to hold all entries but we want them to be persisted on disk.
I tried two different configurations for load testing
 File dbFile = File.createTempFile("testmapdb", "testspeed");
    DB db = DBMaker.newFileDB(dbFile)
        .cacheSize(256 * 1024)
        .cacheHardRefEnable()
        .make();
    Map<Long, QualitySignals> cache =
        db.createHashMap("testSpeed")
            .keySerializer(Serializer.LONG)
            .valueSerializer(QS_SERIALIZER)
            .make();

This was 6-7 times slower than simple ConcurrentHashMap for reading 1M items
Then I used following one just to see if the in-memory cache used by mapdb is causing the slowness
  DB db = DBMaker
        .newHeapDB()
        .cacheSize(256 * 1024)
        .make();
    Map<Long, QualitySignals> cache = db
        .createHashMap("cache")
        .make();

And it turned out no better. I was expecting it to be closer to HashMap or ConcurrentHashMap.
We have just started evaluating mapdb so please let me know if my configurations or expectations are wrong and this is expected or you need more information.
Appreciate it.

Outdated, closing


Antenna Path in QSO Logging and QSO Details

Since we have Antenna Path in the Database, supporting it also in ADIF, we should calculate distances based that.
This PR includes the Distance Calculation in the QSO Details View and the QSO Logging itself. For this it also includes the basic necessary changes in the distance calculation.
Check with whitespaces hidden.
Maps will be addresses some other day since this will be a bigger change.

@AndreasK79 I tried to implement the longpath pathlines in the staticmap.. Quiet complex so let's address maps (staticmap and lba) in another PR on another day

Changing the ANT_PATH in QSO edit does not trigger a re-calculation of distance. Logged a LP QSO with HB9HIL and changed ANT_PATH to SP in QSO edit results in:

should be fixed. Please test agn

QRX. There is a bad interfering when calling the edit ajax in QSO view due to identical ID's


Add a Package.swift file

This issue has a solution with a Package.swift. This PR copies that Package.swift file here. Let's see how that goes
Thanks for starting a pull request on Material Components!
Don't forget:

[ ] Identify the component the PR relates to in brackets in the title. [Buttons] Updated documentation
[ ] Link to GitHub issues it solves. closes #1234
[ ] Please add unit tests, snapshot tests, or manual testing steps depending on the change.
[ ] Sign the CLA bot. You can do this once the pull request is opened.

Contributing has more information and tips for a great
pull request.

Any update on this?


Are there any prebuilt versions on pypi?

I am developping an application layer for llms and want to add support to awq.
Is there prebuilt wheels committed to pypi?
I don't want the users to have to build the library on their end user pcs.
Thanks alot

AutoAWQ is distributed on PyPi
https://github.com/casper-hansen/AutoAWQ


Scons doesn't recognize libpython27.a file format

Hello! I installed Nuitka using .msi from the download page.
OS is Windows 10 x64
Python was installed with the installation of ArcGis Desktop 10.4
python -m nuitka --version
0.6.14rc3
Python: 2.7.10 (default, May 23 2015, 09:40:32) [MSC v.1500 32 bit (Intel)]
Executable: C:\Python27\ArcGIS10.4\python.exe
OS: Windows
Arch: x86
gcc was installed by Nuitka itself
gcc --version
gcc (MinGW-W64 i686-posix-dwarf, built by Brecht Sanders) 10.2.0
While trying to create an executable of the "Hello, world!" program from the user manual, I got this error:
py -2.7 -m nuitka --mingw64 hello-world.py
Nuitka:INFO: Starting Python compilation.
Nuitka:INFO: Completed Python level compilation and optimization.
Nuitka:INFO: Generating source code for C backend compiler.
Nuitka:INFO: Running data composer tool for optimal constant value handling.
Nuitka:INFO: Running C level backend compilation via Scons.
Nuitka-Scons:INFO: Backend C compiler: C:\Users\green\AppData\Local\Nuitka\Nuitka\gcc\x86\10.2.0-11.0.0-8.0.0-r5\mingw32\bin\gcc.exe (gcc.exe).
c:/users/green/appdata/local/nuitka/nuitka/gcc/x86/10.2.0-11.0.0-8.0.0-r5/mingw32/bin/../lib/gcc/i686-w64-mingw32/10.2.0/../../../../i686-w64-mingw32/bin/ld.exe: C:\Python27\ArcGIS10.4\libs/libpython27.a: error adding symbols: file format not recognized
collect2.exe: error: ld returned 1 exit status
scons: *** [C:\Users\green\PYCHAR~1\GNSS1\hello-world.exe] Error 1                                                                     | 12/?
Doesn't create an executable. Tried both the stable and the developer version, got the same error.

Hello there,
the ArcGIS Python is not supported right now. It seems their Python is not producing a link library that is compatible with MinGW64, so your next best bet it is to install Visual Studion 2019 Community edition, and see if that works better, it probably will. And then you can ask them to produce a libpython27.a that is more compatible. There are ways to manually create one from the python27.lib using gcc, but I don't think you want to go there. Let me know if MSVC works better for you.
Yours,
Kay


DirectoryListenerConnectorTest fails in Mac

Description:
DirectoryListenerConnectorTest fails in Mac.
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Mac
Suggested Labels (optional):
Bug
Suggested Assignees (optional):
Gihan/Pamod

The reason for the test failure is due to starvation and race condition. It roughly takes about 8-15 seconds for the callback. Increasing the Thread.sleep to 5000, will let the callback to happen. That will solve the problem.

I have added incremental thread sleep to the test case. It will wait  1,2.3,..,20 before failing.
https://github.com/ballerina-platform/ballerina-lang/pull/6991


[TOPIC-GPIO] clean up API declarations

Completes the updates to public and internal API to use typedefs, described in #21789.

@pabigot considering the amount of work you've done on this API and changes you introduced I think you should add the Copyright entry to include/drivers/gpio.h. Also anyone else who contributed sizable chunks of code.

@mnkp All points addressed.
I've added a Nordic copyright for the years involved in this change, in the existing newest-first order (I'd normally do oldest first).  Going back to the start of this conversion the only contributors with commits touching this file are you, me (for Nordic), and @galak who should let me know if a Linaro copyright should be added.


Add snapshot mode ALWAYS into Oracle connector

https://issues.redhat.com/browse/DBZ-5626

@vjuranek Applied, thanks!


Wordwrap scrolls to the right

Issue Type: Bug
Expected:
When I write towards the end of a line, the non-fitting word should drop to the next line without scrolling to the right.
Actual:
Sometimes the screen scrolls to the right before dropping to the next line. Sometimes it scrolls back, other times it doesn't.
Steps taken to produce bug:
Created Markdown file.
Ensure Minimap is enabled. (I was able to reproduce reliably with minimap, but it occurs also without).
Write to the end of line.
Write a large last word and observed scrolling.
Write a small last word and observed that it doesn't scroll.
Workaround:
None
A video of the bug available here: https://youtu.be/iwMUKSNXipE
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:27:35.169Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-7700 CPU @ 3.60GHz (8 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.95GB (6.02GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

gitlens
eam
10.2.1

vscode-docker
ms-
1.2.1

csharp
ms-
1.22.0

vscode-yaml
red
0.8.0

markdown-preview-enhanced
shd
0.5.10

The root cause is the decoration added at the end of the line by the gitlens extension.
Let's track in #32856


LemonDuck and LemonCat Malware

Research done by Microsoft security team
https://www.microsoft.com/security/blog/2021/07/22/when-coin-miners-evolve-part-1-exposing-lemonduck-and-lemoncat-modern-mining-malware-infrastructure/
https://www.microsoft.com/security/blog/2021/07/29/when-coin-miners-evolve-part-2-hunting-down-lemonduck-and-lemoncat-attacks/
I already search this domains on websites like Virus Total and Urlvoid website.
Malware Domains
ackng.com
bb3u9.com
ttr3p.com
zz3r0.com
sqlnetcat.com
netcatkit.com
hwqloan.com
75.ag
js88.ag
qq8.ag
cdnimages.xyz
pp6r1.com
amynx.com
zer9g.com
b69kq.com
down.sqlnetcat.com

Thank you @rai510 these domains are added in commit 392fddc.
Closing.


Remove pkg_resources from grpcio_tools

pkg_resources raises a deprecation warning which breaks our CI
https://github.com/danielgtaylor/python-betterproto/actions/runs/6679669861/job/18152002510?pr=540

Thank for the contribution! But I'm closing this PR since it's already finished: https://github.com/grpc/grpc/pull/35382


Missing Dictionary field for ControlMeasuresPoints feature class

We are missing the targetdesignator field under Text fields to label a Point or Single Target (25240601).

This one will need fixed in the mil2525d plugin code - that is where the list of available text fields and their "well known name" come from.
We also might want to make sure there are no other missing attributes before we make this plugin change (just so we schedule all of the plugin changes together).
(I also just added a plugin label so we can identify which issues are caused by this component)

I will create a backlog item to check for other missing field in Asana.

Sorry about that, I was actually incorrect about where these text attribute comes from in Pro, these attributes are actually derived from all the attributes defined in the Maplex Labels for the mil2525.stylx
So this will actually need fixed in the .stylx

This issue will get addressed with https://github.com/Esri/military-features-data/issues/101 (adding missing labeling expressions to stylx) so marking this as duplicate.


Setting the Office Phone Extension Not Documented

If you want to set the OfficePhone ("Telephone number" in AD) with an extension, you need to use the following format that I have not been able to find documented anywhere. PowerShell will show some examples for setting phone numbers in the help (help set-aduser -parameter phone), but none of them show how to set the extension.
For U.S. phone numbers: +1<PHONE_NUMBER> x321
EXAMPLE: Set phone office phone number with an extension for a user.
Set-ADUser -Identity ChewDavid -OfficePhone "+1<PHONE_NUMBER> x321"
If you are using multi-factor authentication to log on to Office 365 and you have a main phone number where all employees have an extension, you need to set the main phone number and their extension in the AD telephone number field if you want them to be able to select their office phone for MFA since this information is pulled from AD and not user changeable at the time they select this as their MFA choice.

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

ID: 30217138-eb8c-ab9a-4e11-991c6164cbd9
Version Independent ID: 88ff510e-6402-5163-404f-e9154f59b381
Content: Set-ADUser (addsadministration)
Content Source: docset/windows/addsadministration/set-aduser.md
Product: w10
GitHub Login: @brianlic-msft
Microsoft Alias: brianlic

Can you tell me the exact syntax you are using for the help command?
I tried running "help set-aduser -parameter phone" but it didn't work, I had to change it to "help set-aduser -parameter officePhone" and even then it only showed a single example:
-OfficePhone  "+1<PHONE_NUMBER>"

@officedocsbot assign @yogkumgit

@officedocsbot unassign @e0i

Hello @AndreaBarr can you help us finding the right person to ask this? Thanks

@get-itips I'm still working on figuring out who can help us on this.

@MinutemanHS Hello,
I realize a lot of time has passed since this issue has been opened and you probably have some solution by now. This article shows how an office phone can be used for MFA with Office365 - Set up an office phone as your two-step verification method.
The following formats should use but I don't have a way of testing it right now:
+1<PHONE_NUMBER> x 347
+1<PHONE_NUMBER>x347
+1<PHONE_NUMBER>x347

Here is an example of how to set it in AD:
Set-ADUser -Identity $username -OfficePhone "+1<PHONE_NUMBER>x347"
Set-ADUser -Identity $username -Replace @{'telephoneNumber'="+1<PHONE_NUMBER>x347"}

If we can confirm that this is working properly, I will add an example in the article.


Incorrect url for api-unsupported

Issue in ./help/quicksilver/wf-api/api/deprecation-api-internal.md
The /api/ part does not work for api-unsupported in my testing. What did work was:
https://<domainname>.my.workfront.com/attask/api-unsupported/proj/4c70...

Hello,
Regarding this page of the Workfront documentation (https://experienceleague.adobe.com/docs/workfront/using/adobe-workfront-api/api-notes/deprecation-api-internal.html), you wrote:
"The /api/ part does not work for api-unsupported in my testing."
Thank you for bringing this to our attention. I've replaced the example text, and added a note to point out that the /api/ section is missing.
Thanks for helping to improve the Workfront documentation!


chore: update dependencies

In this PR I upgraded the ojsama version to 2.1.0 (for new pp calculation) and did a yarn ugrade.

Thank you for contributing :pray:


feat: use model field alias generator appropriately for CSV column names

If the header is to be read by the Polars CSV reader, and if the Pydantic model has an alias generator, then the columns should be expected to be given by applying the aliasing function on the model fields.
This solves the issue reported in:

#55

Hi @lmmx! Thanks for both the comprehensive issue and solution!
I must admit to not being entirely sure why Jakob introduced the read_csv functionality (I've taken co-ownership of this project). If I understand correctly from the docstring, it's used for cases where the header doesn't exist.
Do you think this is a common usecase? Or is it a bit niche that someone wants validation on a csv file that doesn't have a header? I'm just wondering if this might be best solved by removing the read_csv functionality entirely, and just relying on polars directly for reading in csvs.
I actually took a look at your issue a few days ago, and decided that it would be nice to have the AliasGenerator support built into the models directly, not only for the read_csv case. I accidentally added this to the ruff PR 😓 and didn't notice before merging. I've just added some tests that I meant to go together with it in #61.
Could you take a look using the latest main branch and see if this solves parts of your problem?

Hey @thomasaarholt, nice work!
No, pt.read_csv exists because pl.read_csv infers schema dtypes, whereas with Patito we always want to proactively define the dtypes in the data model. The docstring mentioning reading CSV files without headers is in the examples section, i.e. it's not the primary purpose, just a good demonstration of what it can do.
It makes sense to have a CSV reader method but not an Arrow/Parquet one because Parquet files embed the schema types, whereas CSV does not and so must always either provide them (passing the model dtypes) or infer (not perfect).
I was motivated to pick up this issue when I encountered a misinferred schema (a dataset of train stations' platform codes with numeric and alphanumeric values was misinferred as int when it was str typed).
I can reproduce that error with a toy example here, lowering the value of the rows used to infer the schema to recreate the scenario of a "surprise" value that clashes with the inferred schema. (This was reinstalled on the main branch)
>>> import polars as pl
>>> from io import StringIO
>>> inp1 = StringIO("col_a,col_b\n1,2\n10,B")
>>> pl.read_csv(inp1)
shape: (2, 2)
┌───────┬───────┐
│ col_a ┆ col_b │
│ ---   ┆ ---   │
│ i64   ┆ str   │
╞═══════╪═══════╡
│ 1     ┆ 2     │
│ 10    ┆ B     │
└───────┴───────┘

But when the inferred schema length is too short you get the same error I initially hit
Click to show error message
>>> inp2 = StringIO("col_a,col_b\n1,2\n10,B")
>>> pl.read_csv(inp2, infer_schema_length=1)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/louis/miniconda3/envs/patito/lib/python3.10/site-packages/polars/_utils/deprecation.py", line 134, in wrapper
    return function(*args, **kwargs)
  File "/home/louis/miniconda3/envs/patito/lib/python3.10/site-packages/polars/_utils/deprecation.py", line 134, in wrapper
    return function(*args, **kwargs)
  File "/home/louis/miniconda3/envs/patito/lib/python3.10/site-packages/polars/_utils/deprecation.py", line 134, in wrapper
    return function(*args, **kwargs)
  File "/home/louis/miniconda3/envs/patito/lib/python3.10/site-packages/polars/io/csv/functions.py", line 397, in read_csv
    df = pl.DataFrame._read_csv(
  File "/home/louis/miniconda3/envs/patito/lib/python3.10/site-packages/polars/dataframe/frame.py", line 655, in _read_csv
    self._df = PyDataFrame.read_csv(
polars.exceptions.ComputeError: could not parse `B` as dtype `i64` at column 'col_b' (column number 2)

The current offset in the file is 19 bytes.

You might want to try:
- increasing `infer_schema_length` (e.g. `infer_schema_length=10000`),
- specifying correct dtype with the `dtypes` argument
- setting `ignore_errors` to `True`,
- adding `B` to the `null_values` list.

Original error: ```remaining bytes non-empty```

I'll take a look at the other aspect later, feeling a bit under the weather today! 🤒 I see it was put in in this commit:

https://github.com/JakobGM/patito/commit/ef9928c48c5302dd811ba01b8ed32a871d5c57df

Thanks for doing this! I rebased this on latest main, and it's looking good!


new commands in 0.60

New Commands

category
command

conversions
into {bool, datetime, duration}

core
def-env, error make

core
export_def_env, export_def, export env, export

core
extern, hide, metadata

core
ignore, mod, module, register, use, view-source

env
let_env, load_env, with-env

experimental
view_source

filters
columns, group, par-each, transpose, window

formats
nuon

math
reducers

plugins
gstat

strings
decode, nu-highlight, print

system
complete

viewers
grid

@stormasm - can you reopen this and compare against jntrnr:0_60 instead of jntrnr:main (which is sorely out of date)

I am going to start over....


Linux impl of mDNS is not setting interfaceId on browse/resolve callback

Problem
Linux platform implementation of chip::Mdns is not setting interfaceId for browse/resolve callbacks once retrieved from avahi service browse/resolve.
Proposed Solution
MdnsAvahi::HandleBrowse and MdnsAvahi::HandleResolve need to set interfaceId in MdnsService structure before returning the services to browse callback.
diff --git a/src/platform/Linux/MdnsImpl.cpp b/src/platform/Linux/MdnsImpl.cpp
index 205d840c83..be6b9df526 100644
--- a/src/platform/Linux/MdnsImpl.cpp
+++ b/src/platform/Linux/MdnsImpl.cpp
@@ -591,6 +591,11 @@ void MdnsAvahi::HandleBrowse(AvahiServiceBrowser * browser, AvahiIfIndex interfa
             CopyTypeWithoutProtocol(service.mType, type);
             service.mProtocol               = GetProtocolInType(type);
             service.mAddressType            = ToAddressType(protocol);
+            service.mInterface              = INET_NULL_INTERFACEID;
+            if (interface != AVAHI_IF_UNSPEC)
+            {
+                service.mInterface = static_cast<chip::Inet::InterfaceId>(interface);
+            }
             service.mType[kMdnsTypeMaxSize] = 0;
             context->mServices.push_back(service);
         }
@@ -670,6 +675,11 @@ void MdnsAvahi::HandleResolve(AvahiServiceResolver * resolver, AvahiIfIndex inte
         result.mProtocol    = GetProtocolInType(type);
         result.mPort        = port;
         result.mAddressType = ToAddressType(protocol);
+        result.mInterface   = INET_NULL_INTERFACEID;
+        if (interface != AVAHI_IF_UNSPEC)
+        {
+            result.mInterface = static_cast<chip::Inet::InterfaceId>(interface);
+        }
         Platform::CopyString(result.mHostName, host_name);
         // Returned value is full QName, want only host part.
         char * dot = strchr(result.mHostName, '.');

Yes it does, the interfaceId is required for addressing link-local ipv6 addresses and callbacks don't return it for such addresses.


Braces should be opened on newline for objects

As a follow up to #65, I'm requesting equivalent functionality for objects.
@mikesherov What should this rule be called? (See related question at #538)

This is more or less duplicative of the jshint laxbreak option

@marklog how do you figure? laxbreak suppresses warnings (what kind I can't tell, because the documentation is very vague) but either way it doesn't add warnings for the above cases, which is what I am asking for.

@cowwoc
adding rule for the case
var good2 =
{
  key: value
};

is a pretty much a bad move even for agnostic tool, but i didn't close it because of that, laxbreak warns you for the case:
var bad =
{};

if you don't disable it, which is pretty much what you looking for.


Add documentation on the reasoning behind components

Try and explain the why behind components, as well as the how.
I'm looking for feedback on

how understandable it is
if it's helpful to someone new to GOV.UK Components
phrasing
spelling

(in that order)

Seems like a good explanation to me.  It's nice as well having the comparisons to other things.  I've been around for a while but haven't used components.  Probably want someone who's actually new to look at it too though.

I would say most of this is very understandable and helpful. Apart from my comments, the other thing which could be improved is making some sentences shorter (or splitting them into two or three). But that's not super important.

@selfthinker updated the bits around your comments, thanks!

Addressed comments, hopefully it's more useful now ;)

LGTM👌

Great - anyone happy to hit merge?

👍


Regarding learned image embedding and text embedding in Unet

According to the paper Section 2.1 Decoder, it says
 We enable classifier-free guidance by randomly setting CLIP embeddings to zero (or a learned embedding) 10% of the time, and randomly dropping the text caption 50% of he time during training.
It seems that we are replacing the embeddings after turning them to condition sequences.
https://github.com/lucidrains/DALLE2-pytorch/blob/main/dalle2_pytorch/dalle2_pytorch.py#L1216-L1222
https://github.com/lucidrains/DALLE2-pytorch/blob/main/dalle2_pytorch/dalle2_pytorch.py#L1229-L1234
And from the following it seems that that null text embeddings can vary according to their sequence position. For image embeddings, I feel it is fine, but what about for text encodings?
https://github.com/lucidrains/DALLE2-pytorch/blob/main/dalle2_pytorch/dalle2_pytorch.py#L1104
Also, it seems perhaps it is needed to have a separate a separate cond_drop_prob one for image embedding and one for text encodings.
If we do that, how do we modify forward_with_cond_scale()?
https://github.com/lucidrains/DALLE2-pytorch/blob/main/dalle2_pytorch/dalle2_pytorch.py#L1166-L1178

@xiankgx :pray: :pray: :pray:  thank you for the Q&A i missed this detail!
fixed in the latest version :)

How about the time dependence of null_text_embed?
https://github.com/lucidrains/DALLE2-pytorch/blob/main/dalle2_pytorch/dalle2_pytorch.py#L1104

Suppose we have a sequence of text embeddings: v1 v2 v3 v4 v5
And we need to change the vector of this to some null vectors, say v2 and v4.
Then, we have v1 v_null, v3, v_null, v5.
But can v_null in position 2 and 4 have different values?

@xiankgx so that is actually some improvisation on my part, an observation from DALL-E v1 that the text conditioning works better if one does not mask, but just provide padding tokens in the places without https://github.com/lucidrains/DALLE-pytorch/blob/main/dalle_pytorch/dalle_pytorch.py#L372 it has been proven out in many follow-up works, in my mind

@xiankgx but yes, there is one problem, where i believe the text encoding should always be padded to the maximum text length, so the conditioning is consistent across batches with variable lengthed text encodings
let me fix that now :pray:

@xiankgx haha, actually there was another issue with the null padding tokens, only uncovered because of your issues https://github.com/lucidrains/DALLE2-pytorch/commit/1c1e508369da34eb35741558d33203f42fea006e should be ok now
keep it coming! :pray:


Expose performance analysis through prometheus

https://github.com/raiden-network/performance-analysis-tool is able to write prometheus textfilecollector files.
This issue tracks the progress of installing and exposing them to the internal metrics dashboard.
Done

Created traefik + node_exporter configuration with docker-compose
Added collector rules on metrics server

ToDo

Debug TLS/LE setup
Move collector rules into source control

resolved

resolved


Fix match control-flow of or-patterns

The latest rustc update (https://github.com/AeneasVerif/charon/pull/272) regressed the MIR of or-patterns. When the fix (https://github.com/rust-lang/rust/pull/127028) lands, we should update our rustc version.

I think we're done with this right?

We haven't updated our rustc version yet, I wanted to do that this week


fileObj Size Not Set

I'm trying to upload an image from a url and I'm receiving the following error:
Exception in delivering result of invoking '/cfs.Images.filerecord/insert': Error: TransferQueue upload failed: fileObj size not set
Ideas?

I see that this is an old issue.  @aldeed, you mentioned that you intended for this to work on the client.  I switched to the server to get rid of the "fileObj size not set" error.  I now noticed that the transaction on the server seems to take much longer then on the client.  Was this issue fixed so that we can do it on the client side without getting the error message?


Variables in Kotlin: Android Basics with Compose 4. Update variables

https://developer.android.com/codelabs/basic-android-kotlin-compose-variables?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-variables#3**URL of codelab**
In which task and step of the codelab can this issue be found?
Describe the problem
fun main() {
var cartTotal = 0
cartTotal = 20
println("Total: $cartTotal")
}
Total: 20
it should read:
fun main() {
var cartTotal = 0
cartTotal = 20
println("cartTotal: $cartTotal")
}
CartTotal: 20   as in in the lesson 4  ' Update Varibles' according to Kotlin Playground, https://developer.android.com/training/kotlinplayground
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hi @Ernie1961
Thank you for writing in with your issue.
The informational text being printed at the beginning does not have to match the variable name as you wrote.
They can match if you wish but not a requirement.
Hope you are enjoying the course.
Regards.


Confusion regarding meaning of "Windows Code"

Can you clarify the meaning of what "Windows Code" refers to?  How is this different than a windows container or linux container?

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

ID: 3cd70fac-5d7c-7637-8385-f3f4af3ba0ad
Version Independent ID: 15d95fc5-f08e-a26c-ce06-8ead78474661
Content: Mount Azure Storage as a local share - Azure App Service
Content Source: articles/app-service/configure-connect-to-azure-storage.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

Hi @JaredMSFT I can see how this might be confusing. For Windows App Service, you have two options: containerized and non-containerized. Windows code (or non-containerized Windows) is the original App Service platform, before containers entered the picture. I'll update the article to reflect container and non-container. Thanks!

#assign: msangapu-msft


Clocks

This removes a little used Clock type which offered no type-safety, and changes a few type-unsafe time points into chrono::time_points.
@nbougalis @scottschurr

Looks good to me.  And educational too.  Thanks for the opportunity to review this.
:+1:

Looks good. :+1:

@nbougalis I believe so, but I haven't designed them yet.  We have a general theme of needing to convert between stored integral fields (e.g. wire format) and time points.  I'm thinking a helper function specific to sfExpiration (et al.) will do the trick nicely.  This should be a goal after the goal of type safety is achieved.  And I have the /majority/ of rippled chrono-type-safe here now: https://github.com/HowardHinnant/rippled/commits/closetime  I'm in the "mopping up" phase of this now.

:+1:


refactor(sync): refactoring syncing process

Description
This PR includes these changes:
1- HeartBeat message is removed.
2- HelloAck message is added. Once a node receives a HelloAck, it can start sending messages.
3- MaxOpenSessions is removed and the maximum number of sessions should be the same as the number of connected peers.
4- Syncing process is based on time, not height. If the last block time is more than 20 seconds behind the current time, the node starts syncing.

@amirvalhalla could you please double check it. If anything missed, mention me.


libvirt: 5.4.0 -> 6.0.0

Motivation for this change
Closes #81102
Things done

I'm unable to test on Darwin, but I tested with buildFromTarball set to true and false.

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] 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.

/cc @fpletz @globin

Hopefully it can make it into 20.03 to avoid qemu co-existing with old libvirt, but I guess there is just not enough time to extensively test this update.

@wedens fixed this bug? - https://github.com/NixOS/nixpkgs/pull/66584#issuecomment-524656760

@Izorkin Unfortunately, no idea. I don't use (and can't easily setup) bridge networking to test it. If you (or someone else who uses bridge) can test, that'd be neat.

@wedens error build  pythonPackages.libvirt
Executing setuptoolsBuildPhase
libvirt-python requires Python 3.x to build

If rebuild systev used python2Packages.libvirt, need change to python3Packages.libvirt

please freeze the 5.4.0 version for python 2
[41 built (1 failed), 271 copied (5085.6 MiB), 916.4 MiB DL]
error: build of '/nix/store/adv1j8r1342895ysgviibjbbnl5y8d4b-env.drv' failed
https://github.com/NixOS/nixpkgs/pull/81161
4 package failed to build:
nixops nixopsUnstable nixops_1_6_1 python27Packages.libvirt

33 package built:
collectd collectd-data docker-machine-kvm docker-machine-kvm2 gnome3.gnome-boxes haskellPackages.libvirt-hs libguestfs libguestfs-with-appliance libvirt libvirt-glib libvmi minikube minishift perl528Packages.SysVirt perl530Packages.SysVirt python27Packages.guestfs python37Packages.guestfs python37Packages.libvirt python38Packages.guestfs python38Packages.libvirt rubyPackages.ruby-libvirt rubyPackages_2_5.ruby-libvirt rubyPackages_2_7.ruby-libvirt terraform-full terraform-providers.libvirt terraform_0_11-full terragrunt vagrant virt-manager virt-manager-qt virt-top virt-viewer virtlyst

@ With 5.9.0 correct builded

Yeah, python2 support was dropped in 6.0.0. I'll try to figure out how to use different versions for python2/3. Hopefully it won't make it confusing.

Alright, I've added 5.9.0 for python2. Not sure if it's the correct thing to do though (how possible that there will be some incompatibilities between different libvirt and libvirt-python versions?).

@wedens error rebuild
error: assertion failed at ../pkgs/development/python-modules/libvirt/5.9.0.nix:8:9

@GrahamcOfBorg build nixops

@wedens error build. Used libvirt 6.0.0, needed 5.9.0
Executing setuptoolsBuildPhase
running bdist_wheel
running build
/nix/store/w6pfcj87889hgz0f77blq59z85mdrsh8-pkg-config-0.29.2/bin/pkg-config --print-errors --atleast-version=0.9.11 libvirt
/nix/store/bh447min7f6vqdw3x1iywr1vxyx4ix8y-python-2.7.17/bin/python2.7 generator.py libvirt /nix/store/4aqz37c2fdg3klqhxhji707ypzdcra96-libvirt-6.0.0/share/libvirt/api/libvirt-ap$
Found 478 functions in /nix/store/4aqz37c2fdg3klqhxhji707ypzdcra96-libvirt-6.0.0/share/libvirt/api/libvirt-api.xml
Found 0 functions in libvirt-override-api.xml
Generated 389 wrapper functions
/nix/store/bh447min7f6vqdw3x1iywr1vxyx4ix8y-python-2.7.17/bin/python2.7 generator.py libvirt-qemu /nix/store/4aqz37c2fdg3klqhxhji707ypzdcra96-libvirt-6.0.0/share/libvirt/api/libvi$
Found 5 functions in /nix/store/4aqz37c2fdg3klqhxhji707ypzdcra96-libvirt-6.0.0/share/libvirt/api/libvirt-qemu-api.xml
Found 0 functions in libvirt-qemu-override-api.xml
Generated 3 wrapper functions
Traceback (most recent call last):
  File "generator.py", line 2143, in <module>
    qemuBuildWrappers(sys.argv[1])
  File "generator.py", line 2008, in qemuBuildWrappers
    items.sort(key=lambda i: (int(i[1]), i[0]))
  File "generator.py", line 2008, in <lambda>
    items.sort(key=lambda i: (int(i[1]), i[0]))
ValueError: invalid literal for int() with base 10: 'VIR_DOMAIN_AGENT_RESPONSE_TIMEOUT_BLOCK'
error: command '/nix/store/bh447min7f6vqdw3x1iywr1vxyx4ix8y-python-2.7.17/bin/python2.7' failed with exit status 1

I'm not sure what to do about it, tbh.
I'd rather not use a band aid like updating to 5.9.0 instead of  6.0.0 and putting of the python2 problem for later.

I'd rather not use a band aid like updating to 5.9.0 instead of 6.0.0 and put off the python2 problem for later.

If it was just the python-module, i would be fine with deprecating. But I'm not okay with breaking nixops packages

What are chances that nixops will update to python 3 in the nearby future?
I guess it'll be reasonable compromise to update to 5.9.0 for now and to 6.* closer to 20.09 release.

Since the python2 was not the goal of this PR, you could also just do a freeze at 5.4.0, instead of bumping to 5.9.0

@jonringer Are you suggesting having two libvirt/libvirt-python versions (5.4.0 and 6.0.0) in nixpkgs?
I'm not sure how this compatibility works, but as far as I understand, if libvirt daemon (presumably used via libvirt nixos module) is 6.0.0, nixops (that'll use libvirt-python with libvirt 5.4.0), won't be able to manage this libvirt daemon (which is 6.0.0).

I suggesting that, instead of freezing the python2 version at 5.9.0, you keep it at 5.4.0. python3 will be at latest

Ok. I've added libvirt_5_4_0 which is used as a dependency for python-libvirt with python2.

@jonringer Are you suggesting having two libvirt/libvirt-python versions (5.4.0 and 6.0.0) in nixpkgs?
I'm not sure how this compatibility works, but as far as I understand, if libvirt daemon (presumably used via libvirt nixos module) is 6.0.0, nixops (that'll use libvirt-python with libvirt 5.4.0), won't be able to manage this libvirt daemon (which is 6.0.0).

Just read this. Hmm, we should get someone from nixops to chime in. Forgot that libvirt was more than just a library.
cc @grahamc @edolstra

It does however, fix all applications.
[14 built, 2 copied (56.6 MiB), 12.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/81161
38 package built:
collectd collectd-data docker-machine-kvm docker-machine-kvm2 gnome3.gnome-boxes haskellPackages.libvirt-hs libguestfs libguestfs-with-appliance libvirt libvirt-glib libvirt_5_4_0 libvmi minikube minishift nixops nixopsUnstable nixops_1_6_1 perl528Packages.SysVirt perl530Packages.SysVirt python27Packages.guestfs python27Packages.libvirt python37Packages.guestfs python37Packages.libvirt python38Packages.guestfs python38Packages.libvirt rubyPackages.ruby-libvirt rubyPackages_2_5.ruby-libvirt rubyPackages_2_7.ruby-libvirt terraform-full terraform-providers.libvirt terraform_0_11-full terragrunt vagrant virt-manager virt-manager-qt virt-top virt-viewer virtlyst

6.1.0 released

With 6.0.0 frezing start server.

With 6.0.0 frezing start server.

Is it the same issue we had trying to bump 5.4.0->5.6.0 (https://github.com/NixOS/nixpkgs/pull/66584) ?
You offered a fix then, does it help with 6.0.0 ?

With 6.0.0 frezing start server.

Any details?

I suggesting that, instead of freezing the python2 version at 5.9.0, you keep it at 5.4.0. python3 will be at latest

If we freeze libvirt-5.4.0, should we freeze qemu-4.1 for it (per https://github.com/NixOS/nixpkgs/issues/81102) ?

With 6.0.0 frezing start server.

Is it the same issue we had trying to bump 5.4.0->5.6.0 (#66584) ?
You offered a fix then, does it help with 6.0.0 ?

Yes, this error.
My fix not full worked. After nixos-rebuild swith need manual runnig systemctl reload br0-netdev

Probably the easiest solution would be to find the commit between 5.4.0 and 5.6.0 which introduced the problem and revert it

@volth new version libvirt used systemd socket activation - https://bugzilla.redhat.com/show_bug.cgi?id=1750340

new version libvirt used systemd socket activation -

Thus if the mgmt app / admin wants to use TCP/TLS sockets they have two choices

  - To continue the old approach (setting --listen in /etc/sysconfig/libvirtd), then they MUST use 'systemctl mask ...' for all the socket units listed above, before libvirtd.service is started.

  - To adapt to use the new approach, then don't touch /etc/sysconfig/libvirtd at all. Instead use 'systemctl enable libvirtd-tls.socket && 'systemctl start libvirtd-tls.socket'   (assuming  TLS is desired).

I'd prefer the first option (keep the daemon running) for seamless upgrade.
Eventually we could switch to systemd socket activation

There is not only socket activation, there is also virtqemud daemon we have to start

https://github.com/wedens/nixpkgs/pull/1
libvirtd rejects to start with --listen flag in presence of systemd, so I switched to socket activation.
it works on my laptop, but I have not tested yet various cases like restoring bridges after nixos-rebuild

./bootstrap script is gone in 6.1.0.
it was the script which failed on Darwin and that's why a tarball (with post-./bootstrap state) was required.
There is a good chance that Darwin build of libvirt-6.1.0 will succeed from git sources.
Then tarball can be removed.

With libvirt 5.9.0 and 6.1.0 correct work.

also, collectd needs to be upgraded.
its virt plugin logs protocol errors when interacting with libvirt-6.1.0

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixos-20-03-beta/5935/22

https://github.com/wedens/nixpkgs/pull/2 cosmetic

Thank you for the effort to bring recent libvirt into 20.03! I've briefly tested this branch and things seem to work fine - installing QEMU/KVM virtual machine and using it with the default NATted network as well as NixOS-manged bridge.

I;d be happy to get rid of systemd socket activation eventually.
It makes virsh unreliable, sometimes it get stuck :(

Personally, i would like to see the merge commits be removed, should be able to clean up with git rebase -i.
otherwise LGTM
[20 built, 0.0 MiB DL]
https://github.com/NixOS/nixpkgs/pull/81161
38 package built:
collectd collectd-data docker-machine-kvm docker-machine-kvm2 gnome3.gnome-boxes haskellPackages.libvirt-hs libguestfs libguestfs-with-appliance libvirt libvirt-glib libvirt_5_4_0 libvmi minikube minishift nixops nixopsUnstable nixops_1_6_1 perl528Packages.SysVirt perl530Packages.SysVirt python27Packages.guestfs python27Packages.libvirt python37Packages.guestfs python37Packages.libvirt python38Packages.guestfs python38Packages.libvirt rubyPackages.ruby-libvirt rubyPackages_2_5.ruby-libvirt rubyPackages_2_7.ruby-libvirt terraform-full terraform-providers.libvirt terraform_0_11-full terragrunt vagrant virt-manager virt-manager-qt virt-top virt-viewer virtlyst

Personally, i would like to see the merge commits be removed, should be able to clean up with git rebase -i.

Done.

It makes virsh unreliable, sometimes it get stuck :(

@volth Is it some known issue with libvirt itself?

It makes virsh unreliable, sometimes it get stuck :(

@volth Is it some known issue with libvirt itself?

No, it is my 5.4.0->6.1.0 migration experience.

Really upgrade 5.4 to 5.9 and 6.1?

upgrade 5.4 to 5.9 and 6.1?

I think we have no.
After this PR, NixOS module won't work with 5.4 because it does not start libvirtd.service.
It relies on systemd socket activation.

Alright, let's do it while we're at it. @Izorkin I've applied your patch

How can we move this PR forward?

What's remaining here?
Would be very nice to have working libvirt in 20.03!

I'll work on this first thing on monday

@globin Perfect, I'd like to see this in 20.03 so we can have this working. Thanks.

I am using this in prod for 2 weeks.
Some hosts with old VMs required 2-3 reboots to get into a stable state.
Fresh installs work well.


Update Microsoft.Extensions.AI to 9.0.0-preview.9.24556.5

Hi @stephentoub
Thanks for the PR.
I had it prepared already and was looking forward to see a non-preview version since GA has been announced last night.
Cheers, JoKi

No problem. It'll be a little while before we GA Microsoft.Extensions.AI; it only recently previewed, and we need to ensure we gather enough feedback on it before we mark it as stable.
Thanks for your efforts here.


Github build action deprecation warnings

When deploying to the development environment, I observed multiple warnings that Node.js 12 actions and the "set-output" command are deprecated.  See the summary here.
Expected behavior
Update the offending actions and commands so there are no warnings.

@k-macmillan found this comment from the aws-actions repo; a static version tag will be producing these errors, so I'll check through our chagnes from ticket ____  and look for other minor version tags.

This may be a non-issue, I haven't seen these notices in a while and a screenshot would have been more helpful than a link to a now-expired workflow run. Closing - if we see more of these, we can create new tickets.


New dataset information page does not work as expected

If I hit the Start a new scenario from this country button, nothing happens
If I hit the Go back to the model link, it opens a (new?) scenario
(minor) I think the first Go back to the model link is redundant

If I hit the Start a new scenario from this country button, nothing happens

No, you got to the front-page of etmodel and the country is prefilled with the country you selected. Obviously "The Netherlands" is the default so "nothing happens". This had to do with the fact that you can't specify the end-year in this layout and it would be a little double to also include it in this page.

(minor) I think the first Go back to the model link is redundant

I got that link for free using the 'pages' layout. Guess I could use a 'blank' layout considering the empty-state of the page - as of now.


Add new consortium member info module (resolves #391)

Signed-off-by: Simon Stone<EMAIL_ADDRESS>

/azp help

Clean build here: https://dev.azure.com/IBM-Blockchain/ansible-collection/_build/results?buildId=1197&view=results


Add support for xrate and xincrease

There seems to be a need in the Prometheus community to have rate/increase functions that do not do extrapolation.
We briefly discussed this in Slack and we think there's value to add support for these functions in the Thanos engine. @GiedriusS mentioned they already maintain a fork for xrate/xincrease which we can either integrate or use as a starting point.

Slack thread: https://cloud-native.slack.com/archives/CL25937SP/p1677487566632769
Fork with xrate support: https://github.com/vinted/prometheus/commit/835de212f1022e4be7d18af5a0dd1f1c8589adce#diff-ad2761bb162a2f12326b18f2714ef54f4cd476cfe3388f77b2b20c5ff82e4faaR1893-R1994.

So those functions will be under a feature flag?

I think this might be solving that problem, but we should test what happens.

Can we close this now? I think we already added support for x functions

Yes, closing.

Is there documentation for those new functions ?

These functions are implemented based on a quite old issue in Prometheus:

https://github.com/prometheus/prometheus/issues/3806
https://docs.google.com/document/d/1y2Mp041_2v0blnKnZk7keCnJZICeK2YWUQuXH_m4DVc/edit

The short summary is that they don't do extrapolation so you can get exact rate calculations which is useful for sparse metrics.
For example, if a metric goes up by 1 every hour or so, the rate function will extrapolate and you might see something like 1.1 as the result. On the other hand, xrate will show you exactly 1.
I agree it would be nice to have some documentation around this 👍

Thanks @fpetkovski.
I was wondering if the xincrease function would be able to trigger an alert on a new metric. My specific usecase is described here.
NB: we currently use plain prometheus.

Yes, xincrease would inject a 0 value for a newly appearing counter, so even applying it to 1 sample will produce a result.


Python version

I tried couple times to configure Dataform using the repository with dataset that is located in multiregion UE.
I pointed during one of configuration steps
./deploy.sh [Project-name]\
europe-west6 \
UE

And I received error message 'no dataset in location'
I created dataset replica in 'europe-central2'
And then proceeded with configuration again and after
command:
$ ./deploy.sh dataform-test-07 \
europe-west6 \
europe-central2

I received an error :
Updated property [core/project].
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support pip 21.0 will remove support for this functionality.
ERROR: Could not find a version that satisfies the requirement cachetools==5.3.1 (from -r requirements.txt (line 7)) (from versions: 0.0.0, 0.1.0, 0.2.0, 0.3.0, 0.3.1, 0.4.0, 0.5.0, 0.5.1, 0.6.0, 0.7.0, 0.7.1, 0.8.0, 0.8.1, 0.8.2, 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.1.0, 1.1.1, 1.1.2, 1.1.3, 1.1.4, 1.1.5, 1.1.6, 2.0.0, 2.0.1, 2.1.0, 3.0.0, 3.1.0, 3.1.1)
ERROR: No matching distribution found for cachetools==5.3.1 (from -r requirements.txt (line 7))
Traceback (most recent call last):
File "/home/marcin_laskowski/ga4_dataform/main.py", line 17, in
from google.cloud import dataform_v1beta1
ImportError: cannot import name 'dataform_v1beta1' from 'google.cloud' (unknown location)
ERROR: (gcloud.projects.add-iam-policy-binding) INVALID_ARGUMENT: Service account<EMAIL_ADDRESS>does not exist.
ERROR: (gcloud.projects.add-iam-policy-binding) INVALID_ARGUMENT: Service account<EMAIL_ADDRESS>does not exist.
I wonder if python error could be repaired on my side?
And what are the proper settings in step:
$ ./deploy.sh dataform-test-07 \
Dataform Location \
Dataset Location

Considering the fact that the original dataset is in 'UE' multiregion (and replica in 'europe-central2' region ) and Dataform is not available in 'europe-central2' region according to this list:
https://cloud.google.com/dataform/docs/locations
Kind regards, Marcin

Hello @marcinlaskowski-sempai the list of supported BQ locations could be referenced here: https://cloud.google.com/bigquery/docs/locations (UE is not part of the list). Also make sure the region matches the region of your GA4 events_table.
Also note that we have updated the repo to use a different deployment method which is by connecting to a GitHub repo, this is the recommended approach by Dataform.
If you still want to use the deploy.sh file, you could edit the code inside main.py to use pip3 instead of pip, and python3 instead of python (it seems your default python version is still 2, and hence the error)


Smooth set-up instructions

This pull request includes updates to the DNSMOS project, focusing on setup instructions and a new script for installing dependencies. The most important changes include adding setup instructions in the README.md file and introducing a Python script to install required packages from a requirements.txt file.
Setup Instructions Added:

DNSMOS/README.md: Added detailed setup instructions for creating and activating a virtual environment, checking and updating pip, and running the new installation script.

Dependency Installation Script:

DNSMOS/install-reqs.py: Added a new script to read from requirements.txt and install the listed packages using pip, with error handling and reporting for any packages that fail to install.

@microsoft-github-policy-service agree [company="no company specified"]

(default - no company specified) I have sole ownership of intellectual
property rights to my Submissions and I am not making Submissions in the
course of work for my employer.

On Thu, 7 Nov 2024 at 09:51, microsoft-github-policy-service[bot] <
@.***> wrote:

@rajen-bhatt-gmail-com https://github.com/rajen-bhatt-gmail-com please
read the following Contributor License Agreement(CLA). If you agree with
the CLA, please reply with the following information.
@microsoft-github-policy-service agree [company="{your company}"]
Options:

(default - no company specified) I have sole ownership of
intellectual property rights to my Submissions and I am not making
Submissions in the course of work for my employer.

@microsoft-github-policy-service agree

(when company given) I am making Submissions in the course of work
for my employer (or my employer has intellectual property rights in my
Submissions by contract or applicable law). I have permission from my
employer to make Submissions and enter into this Agreement on behalf of my
employer. By signing below, the defined term “You” includes me and my
employer.

@microsoft-github-policy-service agree company="Microsoft"
Contributor License Agreement Contribution License Agreement
This Contribution License Agreement (“Agreement”) is agreed to by the
party signing below (“You”),
and conveys certain license rights to Microsoft Corporation and its
affiliates (“Microsoft”) for Your
contributions to Microsoft open source projects. This Agreement is
effective as of the latest signature
date below.

Definitions.
“Code” means the computer software code, whether in human-readable
or machine-executable form,
that is delivered by You to Microsoft under this Agreement.
“Project” means any of the projects owned or managed by Microsoft
and offered under a license
approved by the Open Source Initiative (www.opensource.org).
“Submit” is the act of uploading, submitting, transmitting, or
distributing code or other content to any
Project, including but not limited to communication on electronic
mailing lists, source code control
systems, and issue tracking systems that are managed by, or on behalf
of, the Project for the purpose of
discussing and improving that Project, but excluding communication
that is conspicuously marked or
otherwise designated in writing by You as “Not a Submission.”
“Submission” means the Code and any other copyrightable material
Submitted by You, including any
associated comments and documentation.
Your Submission. You must agree to the terms of this Agreement
before making a Submission to any
Project. This Agreement covers any and all Submissions that You, now
or in the future (except as
described in Section 4 below), Submit to any Project.
Originality of Work. You represent that each of Your Submissions
is entirely Your original work.
Should You wish to Submit materials that are not Your original work,
You may Submit them separately
to the Project if You (a) retain all copyright and license information
that was in the materials as You
received them, (b) in the description accompanying Your Submission,
include the phrase “Submission
containing materials of a third party:” followed by the names of the
third party and any licenses or other
restrictions of which You are aware, and (c) follow any other
instructions in the Project’s written
guidelines concerning Submissions.
Your Employer. References to “employer” in this Agreement include
Your employer or anyone else
for whom You are acting in making Your Submission, e.g. as a
contractor, vendor, or agent. If Your
Submission is made in the course of Your work for an employer or Your
employer has intellectual
property rights in Your Submission by contract or applicable law, You
must secure permission from Your
employer to make the Submission before signing this Agreement. In that
case, the term “You” in this
Agreement will refer to You and the employer collectively. If You
change employers in the future and
desire to Submit additional Submissions for the new employer, then You
agree to sign a new Agreement
and secure permission from the new employer before Submitting those
Submissions.
Licenses.

Copyright License. You grant Microsoft, and those who receive the
Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive,
royalty-free, irrevocable license in the
Submission to reproduce, prepare derivative works of, publicly
display, publicly perform, and distribute
the Submission and such derivative works, and to sublicense any or all
of the foregoing rights to third
parties.
Patent License. You grant Microsoft, and those who receive the
Submission directly or
indirectly from Microsoft, a perpetual, worldwide, non-exclusive,
royalty-free, irrevocable license under
Your patent claims that are necessarily infringed by the Submission or
the combination of the
Submission with the Project to which it was Submitted to make, have
made, use, offer to sell, sell and
import or otherwise dispose of the Submission alone or with the
Project.
Other Rights Reserved. Each party reserves all rights not
expressly granted in this Agreement.
No additional licenses or rights whatsoever (including, without
limitation, any implied licenses) are
granted by implication, exhaustion, estoppel or otherwise.

Representations and Warranties. You represent that You are
legally entitled to grant the above
licenses. You represent that each of Your Submissions is entirely Your
original work (except as You may
have disclosed under Section 3). You represent that You have secured
permission from Your employer to
make the Submission in cases where Your Submission is made in the
course of Your work for Your
employer or Your employer has intellectual property rights in Your
Submission by contract or applicable
law. If You are signing this Agreement on behalf of Your employer, You
represent and warrant that You
have the necessary authority to bind the listed employer to the
obligations contained in this Agreement.
You are not expected to provide support for Your Submission, unless
You choose to do so. UNLESS
REQUIRED BY APPLICABLE LAW OR AGREED TO IN WRITING, AND EXCEPT FOR THE
WARRANTIES
EXPRESSLY STATED IN SECTIONS 3, 4, AND 6, THE SUBMISSION PROVIDED
UNDER THIS AGREEMENT IS
PROVIDED WITHOUT WARRANTY OF ANY KIND, INCLUDING, BUT NOT LIMITED TO,
ANY WARRANTY OF
NONINFRINGEMENT, MERCHANTABILITY, OR FITNESS FOR A PARTICULAR PURPOSE.
Notice to Microsoft. You agree to notify Microsoft in writing of
any facts or circumstances of which
You later become aware that would make Your representations in this
Agreement inaccurate in any
respect.
Information about Submissions. You agree that contributions to
Projects and information about
contributions may be maintained indefinitely and disclosed publicly,
including Your name and other
information that You submit with Your Submission.
Governing Law/Jurisdiction. This Agreement is governed by the
laws of the State of Washington, and
the parties consent to exclusive jurisdiction and venue in the federal
courts sitting in King County,
Washington, unless no federal subject matter jurisdiction exists, in
which case the parties consent to
exclusive jurisdiction and venue in the Superior Court of King County,
Washington. The parties waive all
defenses of lack of personal jurisdiction and forum non-conveniens.
Entire Agreement/Assignment. This Agreement is the entire
agreement between the parties, and
supersedes any and all prior agreements, understandings or
communications, written or oral, between
the parties relating to the subject matter hereof. This Agreement may
be assigned by Microsoft.

—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/DNS-Challenge/pull/196#issuecomment-2462431167,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BH3E2OPTFZXIMSMKAIKCV4TZ7N45LAVCNFSM6AAAAABRLMJ4LSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGQZTCMJWG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dr. Rajen Bhatt
Engineering Director, AI/ML

@microsoft-github-policy-service agree
On Thu, 7 Nov 2024 at 11:19, microsoft-github-policy-service[bot] <
@.***> wrote:

@rajen-bhatt-gmail-com https://github.com/rajen-bhatt-gmail-com the
command you issued was incorrect. Please try again.
Examples are:
@microsoft-github-policy-service agree
and
@microsoft-github-policy-service agree company="your company"
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/DNS-Challenge/pull/196#issuecomment-2462654344,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BH3E2OPO4JRIIM2KVQ5LNZTZ7OHIDAVCNFSM6AAAAABRLMJ4LSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGY2TIMZUGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dr. Rajen Bhatt
Engineering Director, AI/ML


WIP: Interchange clustering

This PR is based on https://github.com/SymbiFlow/python-fpga-interchange/pull/70 and aims at redefining the BEL chaining to make it more flexible and allow also the definition of clusters to guide nextpnr placement.
It mainly detaches from using the schema, and directly adds information to the nextpnr chipdb.
It still is marked as WIP as it needs more iterations to get to a stable and reviewable state.

I have refactored the whole clustering for the interchange a bit, so as to have a more concise and possibly cleaner description.
Now, with the changes combined in nextpnr, we can still place and route carry chains for the counter test correctly, and I am currently testing the Murax design implementation.
With these modifications, it should be more trivial to add also LUT+FF cluster definitions (which will not have the chainable_ports as carry chains do).
The idea is to have a set of cell and the corresponding pins for a root cluster type, so that, during placement, the BELs to use for the cluster cells connected to the root can be discovered dynamically through a quick expansion.


Fix copying from terminal on Linux

I've reproduced this issue on Linux, it may also affect Windows.
How to test

Open integrated terminal, select some text and press ctrl+shift+c.

Expected behavior

Selected text is copied to clipboard.

Actual behavior

The path to the previously active file (before focusing the integrated terminal) is copied to clipboard.

Can anyone take a look? This is a very simple fix that makes a significant UX improvement.


Questions about positions input and equirectangular image

Thanks for your great work! I believe this work could promote the developments of model-based methods for VLN!
May I ask some questions?
1.Predicting a further panorama needs the positions and orientations of the agent, but I note in you demo, only the positions are input to the model. How is it work? The agent implicitly calculates the orientations?
2. How to get the equirectangular image in the Matterport3D Simulator (the simulator is based on a skybox image for each viewpoint). Do you have any scripts?

Thanks for your interest in our work! To answer your questions:

We assume a default orientation (see https://github.com/google-research/pathdreamer/blob/master/models/pathdreamer_models.py#L207-L211)
The official Matterport3D dataset (not the simulator) contains 10,800 high resolution panoramas which we use to train on. Please visit their website for download instructions.

Thanks for your reply, it's very useful. Great work!
I still have a question:

In the VLN validation experiment of your paper (Sec.4.2.). How did you encode the path of Pathdreamer model? As far as I know, the visual path encoder of On the Evaluation of Vision-and-Language Navigation Instructions is based on a navigable view from 36 discrete views (Discrete Panorama), which is a little different from the output of pathdreamer  (Continuous Panorama).
Did you use different visual encoders for 'Pathdreamer' and 'Ground Truth' settings?  Or manually discrete the output panorama of Pathdreamer model? Could you please release the VLN validation scripts, thanks!
Cheers!

Hi,
Sorry for the late response.

We sample 36 discrete perspective views from our generated equirectangular image, similar to how R2R does.
We use the same path encoder for both groundtruth and Pathdreamer. Our VLN experiment runs off models from VALAN (but it is likely that you can achieve similar uplift in success rate by using planning for any VLN agent). Unfortunately our code for this part is a bit messy and will likely not be released.

Hope that helps!

Thanks for your reply! It's very useful!
Best Regards!


chore: fix typos

Change Summary
fix some typos
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[ ] PR title adheres to the conventional commits standard
[ ] PR has a change set
[x] PR has been tagged with a change label(s) (i.e. documentation, feature, bug fix, or chore)
[ ] PR includes documentation if necessary.

Looks like most of this is already fixed. Closing.


HeaderSearch wrong layout

In the latest version of the svelte components the HeaderSearch seems to be wrongly layed out
https://carbon-components-svelte.onrender.com/components/UIShell#header-with-global-search

It is supposed to be on the same "line", right? With the current layout when there is  a search layout it hides the input and the close button

Fixed in v0.70.10


add integraton test for modifyResponse plugin

// Describe your PR here; eg. Fixes #1980

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2087 (31bcb88) into master (2362b1e) will decrease coverage by 0.38%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master    #2087      +/-   ##
============================================
- Coverage     60.64%   60.25%   -0.39%
+ Complexity     4019     4007      -12
============================================
  Files           607      607
  Lines         17851    17855       +4
  Branches       1704     1702       -2
============================================
- Hits          10825    10759      -66
- Misses         6149     6219      +70
  Partials        877      877

Impacted Files
Coverage Δ

...u/plugin/modify/response/ModifyResponsePlugin.java
13.69% <0.00%> (-1.46%)
:arrow_down:

...ache/shenyu/client/tars/common/dto/TarsRpcExt.java
0.00% <0.00%> (-30.77%)
:arrow_down:

...enyu/client/tars/TarsServiceBeanPostProcessor.java
26.47% <0.00%> (-29.42%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
70.58% <0.00%> (-23.53%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
63.10% <0.00%> (-18.45%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

...he/shenyu/plugin/grpc/client/ShenyuGrpcClient.java
8.69% <0.00%> (-4.35%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
60.00% <0.00%> (-4.22%)
:arrow_down:

...henyu/plugin/httpclient/NettyHttpClientPlugin.java
58.00% <0.00%> (-2.00%)
:arrow_down:

...che/shenyu/admin/model/vo/SelectorConditionVO.java
96.07% <0.00%> (-1.97%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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


How to use WinUtil on Linux?

Do I need windows in the first place to use WinUtil? Can I use it natively on Linux?

This is for Windows only.

@foss-enjoyer
WinUtil (short for Windows Utilities, also known as Windows Tool Box) is a PowerShell script designed and created for Windows only, as all of its Tweaks and Configuration is Windows related ones, the App Installation Section of WinUtil uses Package Managers (at the time of writing, only Chocolatey and WinGet Package Managers), which are designed for Windows.
Which concludes an answer to your question, that being.. No, WinUtil can not run without Windows, as WinUtil requires the system to be Windows 10 or 11, and WinUtil can't be ran natively on Linux.


Scale points and labels in montage plot

Previously, the scale_factor argument only scaled the points in montage.plot(). This PR changes the behavior to also scale the channel labels.
In addition, I've changed the scale_factor so that 1 corresponds to the default size, values less than 1 scale down, and values greater than 1 scale up. I've also decreased the default point size for kind="topo" a little bit.
I've tested both kind="topo" and kind="3d", I think they both look good.
An alternative to scaling both dots and labels would be to keep the current behavior of scale_factor and add a new parameter font_size, which would allow users to change both properties independently.

import mne
mne.channels.make_standard_montage("biosemi64").plot(kind="topo", scale_factor=1)

Default old

Default new

scale_factor=0.75 new

scale_factor=1.5 new

scale_factor=80 old

scale_factor=0.5 and kind="3d" new

BTW, ideally I'd want to change scaling_factor to just scale. Do you think this is worth it?

One problem with the current PR is that it's a pretty big change for people who used this previously. We do often change viz functions but it's usually fairly subtle, but in this case a scale_factor=40 in main probably looks okay but will look wild (40x the normal size I think?) on this PR.
One way out of this is to:

leave scale_factor as it is and just document that really just sets the marker_size property of the collection essentially.
add a new scale=1. that does what this PR does, and scales the marker_size, text objects, etc. by that additional amount.

That stays 100% backward compatible but allows adding the new param and behavior with a better name.

One way out of this is to:

leave scale_factor as it is and just document that really just sets the marker_size property of the collection essentially.

add a new scale=1. that does what this PR does, and scales the marker_size, text objects, etc. by that additional amount.

I would add:

do a normal deprecation cycle on scale_factor and in the depr message encourage folks to use scale instead.  If folks really need to control marker size and font size independently, we can (later) make it easy to get the handles of the various artists within the fig.mne namespace perhaps.

The functions have the following signature:
def plot_montage(
    montage,
    scale_factor=20,
    show_names=True,
    kind="topomap",
    show=True,
    sphere=None,
    *,
    axes=None,
    verbose=None,
):

Where would you like me to insert the new scale parameter? Ideally, it would be the second parameter and completely replace the current scale_factor after deprecation, but I'm not sure how flexible we are in changing parameter positions. We already use kw-only args, but only for the last two parameters. If possible, I'd move the * to right after montage.

So far we've been willing to insert * at "reasonable places" knowing that users who need to adapt code 1) will get a reasonable error message and 2) any changes they need to make are pretty straightforward (i.e., switch to keyword args). In this case we'd have to move the * to the second position which is much earlier than we generally do it, but might be okay here. People who had code:
plot_montage(montage, 10, True, ...)

could arguably benefit from having to updating their code to
plot_montage(montage, scale_factor=10, show_names=True, ...)

anyway because it's not at all obvious what the args are being used for in the first instance. (Whereas for something like the Epochs constructor you can reasonably do Epochs(raw, events, event_id) and it's pretty clear what's going on.)
So I think I'm okay with having the * right after montage here

So I think I'm okay with having the * right after montage here

agreed.


bug: using the global mapping for closing plugin causing mappings to be destroyed when opening the plugin again

Steps

open plugin
press "1" or "3" (or any other navigation)
do <leader>k to close the plugin
open plugin
do step 2 again

Expectation
Everything should work fine as before but it doesn't. shortcuts not working

Nice catch!


[OTHER] Enhance Share with button

Description
The "share with" option doesn't look very good. I would love to improve it's styling.
Screenshots

Additional information
No response
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the contribution guideline

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

can you give me any screen shot or any short description of, what you want to do?

I would like add some padding also fix the hover effect on copy button. Right now when you hover over copy button, it affects it's surrounding as well. I would like to either fix the hover effect or remove it. I Think removing it would look better as it would then match with other social media buttons.

hi @Virtual4087 I have submitted PR for this issue...let me know if any changes are needed.

hi @Shariar-Hasan I have submitted PR for this issue...let me know if any changes are needed.

@IbraheemHaseeb7
are you assinged for this issue?
if no, then why did you created a PR?

I would like add some padding also fix the hover effect on copy button. Right now when you hover over copy button, it affects it's surrounding as well. I would like to either fix the hover effect or remove it. I Think removing it would look better as it would then match with other social media buttons.

@Virtual4087
okay, i got the idea,
you are assinged. go for it 💥

@Shariar-Hasan I'm on it.


Move c header generation out of build.rs

Drop the cbindgen integration into build.rs, it's just too hard to make work right.  Instead just run ./cbindgen.sh from the root to update all the C headers

fyi @CriesofCarrots / @jackcmay.   Let's try it this way for a while

Something is still funky here, I think cbindgen is pulling spl-token from crates.io.  But IMO this is an improvement regardless so landing it


Upgrade cuda version to 9.2 for CI workflows

Needed by the issue #3404
as v9.1 doesn't have a nccl2 release

JFYI, please refer to @hcho3's comment regarding possible caching behavior of Jenkinsfile here: https://github.com/dmlc/xgboost/pull/3404.

@teju85 I'll merge it as soon as the tests pass. Thanks!

Hi @hcho3 ,
That's where I'm unsure if CI workflow would be testing the "right" things here! Is there a way to have CI workflow pick up the Jenkinsfile changes in this PR, rather than the cached one?

Ok, confirmed, the jenkins check above still seems to pick up from v9.1
Ref: https://xgboost-ci.net/blue/organizations/jenkins/xgboost/detail/PR-3460/1/pipeline

@teju85 I think this PR will pass all the tests, since this PR does not yet introduce NCCL2. I don't think caching is the issue; it is rather that, when testing PRs, Jenkins uses old version of Jenkinsfile. See my comment at #3404 (comment).

So, does this mean, once this PR gets merged, the checks for the next set of PRs will use the updated Jenkinsfile? (this was something unclear to me in your comment in PR #3404)

So, does this mean, once this PR gets merged, the checks for the next set of PRs will use the updated Jenkinsfile?

Yes. When the master gets updated, then future PRs will use the updated Jenkinsfile.

Thanks Philip for the confirmation!

Codecov Report

Merging #3460 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #3460     +/-   ##
===========================================
- Coverage     45.86%   45.76%   -0.1%
  Complexity      188      188
===========================================
  Files           166      166
  Lines         13146    13146
  Branches        444      444
===========================================
- Hits           6029     6016     -13
- Misses         6914     6927     +13
  Partials        203      203

Impacted Files
Coverage Δ
Complexity Δ

src/data/sparse_page_raw_format.cc
41.5% <0%> (-22.65%)
0% <0%> (ø)

src/data/sparse_page_dmatrix.cc
92.3% <0%> (-0.6%)
0% <0%> (ø)

Continue to review full report at Codecov.

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

@teju85 Now that this has been merged, you should re-base #3404 against the latest master to make use of the updated Jenkinsfile.


[Bug]: Triggering a script, but the results are inconsistent with triggering in the terminal

Checklist

[X] I have read the appropriate section in the contributing guidelines
[X] I believe this issue is a problem with polybar itself and not a misconfiguration on my part
[X] I have searched for other open and closed issues that may have already reported this problem
[X] I have checked the known issues page for this problem.
[X] I have followed the debugging guide to narrow down the problem to a minimal config.

Steps to reproduce
I use formatting "Action %{A}" to trigger a script in date module, but result is inconsistent with triggering in terminal.

Actual behavior
use%{A3:~/.local/mbin/polybar/calendar.sh:}<label>%{A} ,right click the module

Window Manager and Version
bspwm 0.9.10
Linux Distribution
archliux 5.15.18-1-lts
Polybar version
polybar 3.5.7

Features: +alsa +curl +i3 +mpd +network(libnl) +pulseaudio +xkeyboard

X extensions: +randr (+monitors) +composite +xkb +xrm +xcursor

Build type: Release
Compiler: /usr/bin/c++
Compiler flags: -D_FORTIFY_SOURCE=2 -march=x86-64 -mtune=generic -O2 -pipe -fno-plt -Wall -Wextra -Wpedantic -O3 -DNDEBUG
Linker flags: -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

Additional Context / Screenshots
When  i kill polybar, then restart polybar in terminal ,this bug is gone

Then i use hot key to restart bspwm (also relaunch polybar),this bug reproduce

This has nothing to do with polybar. You should get the same behavior if you run calendar.sh in your bspwm startup with a delay:
(sleep 10; /bin/bash ~/.local/mbin/polybar/calendar.sh) &

The problem is that cal doesn't produce any color output (even with --color=always) if it isn't started (directly or indirectly) from a terminal.
In your case, cal is called from calendar.sh which is called from polybar which is called from bspwm which is called from sddm. Nowhere on that path is a terminal involved and so cal runs in an environment without a terminal.
In particular, it looks for the TERM environment variable. If it is not set, it will not produce any color output.


Language issue

Hello.
I add to languages.yaml Russian translation and set language parameter in plugin settings ru, but language still is English (see link: http://decoupagepro.ru/kontakty)

Also how I can configure email to address? In README.md I can't find instruction (you using Email plugin: https://github.com/getgrav/grav-plugin-email)?

There are some issues with the multilanguage support that I don't fully understand, not using those features of Grav typically. I'm looking into them. For now you can override those settings in a given page as described in the README. Or are you saying that your in-page overrides are also not working?
Also, you can set the email address directly in the languages file, but it makes more sense to define that on a given page, as described in the README.


forget password feature

As a user, I forgot my password, and I want to get a new password.
GIVEN the user on the login page
WHEN  on the login page, I have to see forget password feature and click on it
THEN take you to forget password page to get the new password.

@AliNurisiahsiahi assign me this project.


INSERT operations to a Clickhouse table with  field type of Nullable are not performed

In DB PostgreSQL created a table in the database and filled it with data. Some fields were filled with NULL values:
CREATE TABLE test.merged_blocks_test_null (
  block_begin_time INTEGER,
  block_date DATE NOT NULL,
  block_is_actual SMALLINT,
  block_volume SMALLINT
)

INSERT INTO test.merged_blocks_test_null ("block_begin_time", "block_date", "block_is_actual", "block_volume")
VALUES
  (1, E'2019-10-10', NULL, NULL),
  (2, E'2019-10-10', NULL, NULL),
  (3, E'2019-10-12', 1, 1);

Clickhouse created a table in the database:
CREATE TABLE merged_blocks_test_nullable
(
    block_begin_time Nullable(Int32),
    block_date Date,
    block_is_actual Nullable(UInt8),
    block_volume Nullable(Int8)
)
ENGINE = MergeTree()
PARTITION BY toYYYYMM(block_date)
PRIMARY KEY block_date
ORDER BY block_date
SETTINGS index_granularity = 8192

In the database, PostgreSQL created external table ch_db_http.merged_blocks_test_nullable using the command:
IMPORT FOREIGN SCHEMA "emma" FROM SERVER ch_emma_http INTO ch_db_http;
Then, when performing an INSERT INTO SELECT operation, an error occurs:
INSERT INTO ch_db_http.merged_blocks_test_nullable
SELECT * FROM test.merged_blocks_test_null

Presumably in the code section::
https://github.com/adjust/clickhouse_fdw/blob/b9ff511076a647fc92b3cab3a5efd51439bec860/src/clickhousedb_fdw.c#L1559-L1563
we made a temporary fix:
if (isnull)
{
	appendStringInfoString(&fmstate->sql, "\\N");
	continue;
}

This helped us import a table containing type fields Nullable.
Perhaps we did not take into account all the details when fixing, is it possible to check this error?
Thanks!

It is the same problem as in your another issue.

IMPORT FOREIGN SCHEMA is not related with inserting data. Can you show what clickhouse error on inserting?

Ok, I have pushed your fix. Thanks for the reporting

IMPORT FOREIGN SCHEMA is not related with inserting data. Can you show clickhouse error on inserting?

 Thanks for the reporting

Thanks!

All good now?

I downloaded new changes from the master branch. Checked, now is OK!
Thank you very much!:)


Old-skool experience

Fixes #60
Closes #68
Closes #51
Closes #77
Closes #11
Closes #69

Readme section for UP and DOWN must be updated too