http interceptors

Describe the problem
The http module in which can not support request interceptor and response interceptor
i want to sign parameters before fetching datas
Describe the solution you'd like
i want the http module to support interceptors
Alternatives considered
No response
Additional context
No response

If you are trying to call http in your front-end code, maybe you can try axios, I wrote an axios tauri http adapter that can easily do what you need
axios-tauri-api-adapter

I created an axios-tauri-adapter module as well, convering all axios features, including interceptors.


why standard deviation is statistically useless ?

https://prometheus.io/docs/instrumenting/writing_exporters/#drop-less-useful-statistics
it said "the standard deviation is statistically useless and you can always expose sum of squares, _sum and _count if you ever need to calculate it."
why standard deviation is statistically useless ?
and can sum of squares replace standard deviation ?

I can't be aggregated, and an issue tracker is not the place for statistics questions. It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


feat(SYN-672): Update accessibility controls

Approved!   HAHA!  Might need be taken off the list :)

I sure thought I removed myself too, but clearly not.  Hi you guys!

@go2guy @zyoutz-r7  Hi guys! You might need to "Unwatch" this repository to stop receiving notifications.


Config to disable multi valued attribute separation when creating users in Active Directory user stores

Describe the issue:
We have introduced an improvement/bug fix to separate multi-valued attributes when creating users in AD user store manager [1].
In the Active Directory, the manager is expecting a full DN of a user. If set the MultiAttributeSeparator as a comma character (, - the default value), and provide the claim value for the manager attribute with comma-separated DN, the user creation will fail with an LDAP error code 19.
i.e.: CN=Sachith UD. Udaraka,OU=CS,OU=IAMCS,DC=wso2is,DC=local
A sample user creation payload is given below. Claim http://wso2.org/claims/managercn is mapped to the manager attribute in claim configs.
<soapenv:Envelope xmlns:soapenv="http://schemas.xmlsoap.org/soap/envelope/" xmlns:ser="http://service.ws.um.carbon.wso2.org" xmlns:xsd="http://common.mgt.user.carbon.wso2.org/xsd">
   <soapenv:Header/>
   <soapenv:Body>
      <ser:addUser>
         <!--Optional:-->
         <ser:userName>AD/SachithUd2</ser:userName>
         <!--Optional:-->
         <ser:credential>Admin1!</ser:credential>
         <!--Zero or more repetitions:-->
        <ser:claims>
            <!--Optional:-->
            <xsd:claimURI>http://wso2.org/claims/managercn</xsd:claimURI>
            <!--Optional:-->
            <xsd:value>CN=Sachith UD. Udaraka,OU=CS,OU=IAMCS,DC=wso2is,DC=local</xsd:value>
        </ser:claims>
         <!--Optional:-->
         <ser:profileName>null</ser:profileName>
         <!--Optional:-->
         <ser:requirePasswordChange>true</ser:requirePasswordChange>
      </ser:addUser>
   </soapenv:Body>
</soapenv:Envelope>

Workaround:
Set the MultiAttributeSeparator to a different value or provide the claim value with a different separator.
How to reproduce:

Configure a AD user store.
Create a claim (http://wso2.org/claims/managercn) and map AD attribute that does not accept multi values / has constraints (manager).
Create a user with a claim value containing commas. For the manager, we need to provide a DN of another user (CN=John Doe,OU=CS,OU=IAMCS,DC=wso2is,DC=local).
IS will throw an exception with LDAP error code 19.

Expected behavior:
User should be created without any exceptions.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 6.0.0
Userstore: AD

Optional Fields
Related issues:
[1] - https://github.com/wso2/product-is/issues/9125

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Use aria-labelledby and aria-describedby for complex menu items

This uses aria-labelledby and aria-describedby to provide better labeling of complex menu items and listbox options for accessibility. Before this change, all of the text of the menu item or option was concatenated together without break to form the accessibility label for the item. This resulted in confusing sounding items when used with a screen reader, because label text, description, keyboard shortcuts and more were read as one.
Now we use aria-labelledby to point to the just the main label text for the item, and aria-describedby to point to the description of the item, the value in key/value menu items, and the keyboard shortcut for the item. All of these are optional, so we use useSlotId to generate an id and pass this through the slot context to the proper child.

Chrome with VoiceOver seem to work well with this, however Safari seems to be having trouble
Safari with VoiceOver is announcing the ListBox option items with aria-labelledby pointing to the slot as just "text", or in the complex ones with aria-describedby as just the description slot.
In the Menu examples, Safari with VoiceOver ignores aria-describedby slots.

@jnurthen Can you verify?
Verified the findings of @majornista - also verified that same behaviour is present with Safari Technology Preview.

Can also reproduce with a trivial example

I've opened a WebKit bug report: https://bugs.webkit.org/show_bug.cgi?id=209279


gmail_safe crashes due to too many open file handles

When running gmail_safe, I will get the occasional crash with the following output:
{ [Error: EMFILE, open '/home/jbfink/gmail2/.meta/lastid.txt']
errno: 20,
code: 'EMFILE',
path: '/home/jbfink/gmail2/.meta/lastid.txt' }
Restarting gmail_safe works and it carries on as usual for a time before crashing again with the same message. I have a pretty big inbox and I'm using an application password, but I'm not sure if that matters.

Hello :) Did you re-submit the last version to NPM? Just curious. Thanks :)

I did not and have no intention of doing so any time soon, this project is pretty much abandoned. Feel free to take over for me if you would like to, though you'll have to figure out how.

Understood! I was just looking for a quick and simple way to backup my gmail and a node-based solution would have been best. Yours is the only one. Oh, well... :) Thanks for the update :)

Thanks for letting me know, that's really too bad! I might have to code this up again in another language. I made this project in a weekend to learn javascript - it was a good learning experience but I'm not fond of the language.


Remove erroneous backtick from documentation

Before: Err(err)`` if the future is finished but resolved to an error err`.
After: Err(err) if the future is finished but resolved to an error err.
I hope merging into master is okay, if not, just tell me.

Thanks!


Behavior of --with-attribute has changed?

When I was using ClangSharpPInvokeGenerator to process edflib.h, I noticed that different versions of ClangSharpPInvokeGenerator handle the --with-attribute option differently.
My response file:
--config
compatible-codegen
generate-macro-bindings
multi-file
windows-types
--file
./edflib.h
--libraryPath
EDFlib
--language
c++
--methodClassName
EDF
--namespace
SharpEDFlib.Interop
--output
./SharpEDFlib/Interop
--with-attribute
edf_set_gender=Obsolete("Use edf_set_sex instead.")
edfwrite_annotation_utf8=Obsolete("Use edfwrite_annotation_utf8_hr instead.")
edfwrite_annotation_latin1=Obsolete("Use edfwrite_annotation_latin1_hr instead.")
--with-namespace
edf_set_gender=System
edfwrite_annotation_utf8=System
edfwrite_annotation_latin1=System

Here, I plan to use the --with-attribute option to add the Obsolete attribute to three deprecated interfaces.
With ClangSharpPInvokeGenerator 14.0.0:
        [DllImport("EDFlib", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)]
        [Obsolete("Use edf_set_sex instead.")]
        public static extern int edf_set_gender(int handle, int gender);

With latest version of ClangSharpPInvokeGenerator:
        [DllImport("EDFlib", CallingConvention = CallingConvention.Cdecl, ExactSpelling = true)]
        [Obsolete(Use edf_set_sex instead.)]
        public static extern int edf_set_gender(int handle, int gender);

Is this change as expected or do I miss something?
Thank you!

This is a bug in the latest version of System.CommandLine, it also no longer correctly handles lines with spaces in them
I need to revert back to the previous version

Confirm fixed in <IP_ADDRESS>.


Unable to list backup items with RecoveryServicesClient or ResourceClient

Package Name: azure.mgmt.recoveryservices
Package Version: 2.0.0
Operating System: Mac OS
Python Version: 3.8

Describe the bug
I want to get a list of backup items from the backup center using the SDK.
I tried to use the RecoveryServicesClient and I can list vaults but not the contents.
Basically I need this list:
https://portal.azure.com/#blade/HubsExtension/BrowseResource/resourceType/Microsoft.RecoveryServices%2Fvaults%2FbackupFabrics%2FprotectionContainers%2FprotectedItems
I cannot find any SDK call that returns that. I also tried the ResourceClient but this uses something called "HubsExtension" so the resource doesn't appear.

Thanks for the feedback, we’ll investigate asap.

Hi @jbvsmo, thanks for your feedback!
According to my investigation, to get specific items, first get resource group name and vault name from RecoveryServicesClient, and then obtain them through RecoveryServicesBackupClient.
Here is a sample.
import os

from azure.identity import DefaultAzureCredential
from azure.mgmt.recoveryservices import RecoveryServicesClient
from azure.mgmt.recoveryservicesbackup import RecoveryServicesBackupClient

SUBSCRIPTION_ID = os.environ.get("SUBSCRIPTION_ID", '')
recovery_services = RecoveryServicesClient(credential=DefaultAzureCredential(),
                                           subscription_id=SUBSCRIPTION_ID)

recovery_services_backup = RecoveryServicesBackupClient(credential=DefaultAzureCredential(),
                                                        subscription_id=SUBSCRIPTION_ID)

# List vaults by subscription ID
subscription_vaults = recovery_services.vaults.list_by_subscription_id()

# List vaults by resource group
rg_vaults = recovery_services.vaults.list_by_resource_group(resource_group_name='myResourceGroup')

# List items by resource group and vault
backup_items = recovery_services_backup.backup_protected_items.list(resource_group_name='myResourceGroup', vault_name='myVault')

Please kindly let us know if it solves the issue.


⚠ SceneTime has degraded performance

In 6a52ee2, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2255 ms

Resolved: SceneTime performance has improved in 997cbdf after 6 minutes.


Update CONTRIBUTING.md

Updated numbers in table of contents

[ ] Have you followed the contributing guidelines?
[ ] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

This is not necessary as markdown renders the list with automatic numbering; having all the numbers be zero avoids us having to manually re-number if the list changes.


Should null really be allowed in Var?

Half of the code of Var, and even Reference, is just checks for null values; this is not only annoying but very error prone.
What should we do here? The current plan would be to disallow null in Var starting with 1.1, and create an alternative NullableVar class which would be equivalent to "today's" Var.
But is NullableVar even necessary? Do people have use of nulls in vars at all?

Answer: no. Changes planned to disallow nulls.


ZEPPELIN-2527 Changed editor cursor to thin

What is this PR for?
Previously, it's hard to recognize the cursor since it was too thick. After this PR, it will show the cursor same as other input boxes for consistency.
See the attached GIFs for comparison.
What type of PR is it?
[Improvement]
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2527
How should this be tested?

Build: mvn clean package -Denforcer.skip -DskipTests -Drat.skip
Open a paragraph
Check the cursor

Screenshots (if appropriate)
Before

After

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Thanks for the contribution @malayhm. I left a comment.

ping @malayhm

Hi, @malayhm Could you take a look the comment I left? Thanks!

All the comments are taken care of, if there are no more comments, could we merge this PR to master?

@malayhm cursor is still thick. Should I missing something? or..

@1ambda I have changed the fix and also verified it. It should show the thin cursor with latest changes.

Updated screenshot:

Screenshot with the latest change:

Great!
LGTM, let's merge if no more discussion.

Should we merge this PR?

@malayhm Merged to master. Thanks for the contribution again!


Weird import error?

I get the import error, No module named 'midiutil.MidiFile3', when I try to simply import from miditime.miditime import MIDITime

This is obviously a ticket from a long time ago, and I apologize for the delay. I cannot replicate this issue, which I hope has been resolved. Please let me know if this is still an issue.


Pie cropped when using Offset in dataset (with specific data)

Expected behavior
When using the offset in a Pie chart dataset, simply, the charts should never crop some parts of the pie (as always)

I state that I personally found the bug on the Pie charts.
I don't rule out that others (like the donut) may be equally involved, but I don't know.
Current behavior
When some specific data list are rendered, the bottom of the chart is cropped.

Speaking of sensation, it appears that when there is a slice larger than half the pie, at the bottom, the bug occurs.

The following cropped chart image came from this dataset
{
    data: [1, 13, 1],
    offset: 20
    ...,
}

But everythink works fine using, for example
{
    data: [13, 1, 13],
    offset: 20
    ...,
}

therefore, it depends on the data.

Seems like a Chart.js bug (and not some wrapper bug) cause i can also replicate the error on your official site in the runnable example (https://www.chartjs.org/docs/latest/samples/other-charts/pie.html)
With the following configuration in the Setup tab you will get a cropped chart.
const data = {
  labels: ['Red', 'Orange', 'Yellow', 'Green', 'Blue'],
  datasets: [
    {
      label: 'Dataset 1',
      data: [4, 50, 4],
      backgroundColor: Object.values(Utils.CHART_COLORS),
      offset: 20
    }
  ]
};

Reproducible sample
https://stackblitz.com/edit/react-ts-fabpct?file=App.tsx
Optional extra steps/info to reproduce
I made this runnable example (locked, so no future changes)
In the snippet, we have 4 different data array.
      data: [1, 13, 1], // cropped...
      // data: [12, 3], // cropped
      // data: [13, 1, 13], // Works fine...
      // data: [4, 3,3,4,5,6,7,8,9], // Works fine...

By default you will see the cropped chart.
If you try to switch between this commented data you will see (as comments describe) the chart cropped and the chart working fine (but only the data is changing).
Possible solution
No response
Context
I was just involved in the detection of the bug in this SO Question.
https://stackoverflow.com/questions/72696406/how-to-centre-and-fix-size-of-a-pie-chart-while-offset-is-added-from-react-chart
chart.js version
3.8.0
Browser name and version
Chrome 103.0.5060.66
Link to your project
https://stackblitz.com/edit/react-ts-fabpct?file=App.tsx

export const options = {
  responsive: true,
  maintainAspectRatio: true,
  layout: {
    padding: {
      bottom(ctx) {
        const chart = ctx.chart;
        const offset = chart.data.datasets[0].offset || 0;
        return offset / 2;
      }
    }
  },
  plugins: {
    title: {
      display: false,
    },
    legend: {
      display: false,
    },
  },
};

Thanks @stockiNail!
Actually, using your configuration helps workaround the problem.

Warning
This way, the problem is we can't get a perfect fit chart (sometime useful/needed).

This is a good workaround in order to avoid crop using offset for some cases.
Thank you @stockiNail!

Anyway, i opened this issue with the aim of solving the core problem.
This is not a problem I have right now and I don't need a quick fix for that. I just hope for an official solution for a more-stable library.


Configuration is inconsistent with Laravel conventions

It is a Laravel convention to use snake_case in the configuration files. However, kebab-case is used by this package which goes against the Laravel conventions.
Proposal: change all kebab-case configuration items to snake_case.

A problem is that many people already do have the current config as kebab-case. To stay backwards compatible, we could alias all kebab-case config items to snake_case (i.e. both get accepted, but the former will be deprecated).

I personally find kekab-case easier to read, but am willing to pick this up in future major versions in case enough people approve of this change.


Support with for window

Resolve https://github.com/VictoriaMetrics/metricsql/issues/12

@valyala @hagen1778 Request for view

@lujiajing1126 , thanks for the pull request! I used it as a basis for the commit f3dd382e41eb1a07d94a72618d4323cd1fd4d049 .
This commit adds support for passing duration args via WITH templates. For example:

WITH (w = 5m) m[w] is expanded into m[5m]
WITH (f(w, step, off) = m[w:step] offset off) f(5m, 10s, 1h) is expanded into m[5m:10s] offset 1h

Unfortunately the commit doesn't allow passing dynamically calculated duration values depending on the evaluation timestamp, because this requires significant refactoring in the code responsible for calculating rollup function results per each returned data point from /api/v1/query_range. The temporary workaround is to calculate the needed duration values at client side and pass them to /api/v1/query depending on the time query arg.
As for the original issue https://github.com/VictoriaMetrics/VictoriaMetrics/issues/4025 , we need to think more how to properly implement it properly. The m[((day_of_month()-1) * 24 + hour()) + "h"] approach looks too complex to me. Probably, it would be great to introduce new duration aliases such as start_of_hour, start_of_day, start_of_week, etc, which could be used inside MetricsQL queries. For example, increase(http_requests_total[start_of_day]) would return the number of requests for the current day by UTC. The syntax should support specifying timezone offset, obviously. Something like increase(http_requests_total[start_of_day timezone "+03:00"]) could return the number of requests for the current day at timezone +03:00.
Closing the pull request then, since it has been overridden by the commit f3dd382e41eb1a07d94a72618d4323cd1fd4d049 .

FYI, the functionality described in this comment has been included in VictoriaMetrics v1.92.0.


Preserve system bash prompt and configuration for non-pure devshells

I have an interactive bash prompt which is configured to my liking - starship.rs, fzf integration, git aliases, z-lua, etc.
With a normal nixpkgs.mkShell I can enter a project shell without --pure and keep my system settings with me.  However, devshell seems to override all these.  I want access to my bash configuration and tools when working on a project, but it wouldn't be appropriate to burden my devshell with them since they are strictly speaking not project dependencies.
Is there a way to get devshell mirror the behavior of nixpkgs.mkShell when run without the --pure flag?
Edit: I spoke a bit too soon, looks like git aliases actually do persist as expected.  The other comments still seem valid however.

Related issue https://github.com/numtide/devshell/issues/26
How are you loading the environment. Using nix-shell?

It's a little complicated, there is some more context here.
As a brief summary, I'm building my devshell as part of a nixos-generators vagrant-virtualbox machine for offline use.  I've been using writeShellScriptBin just to give my devshell a memorable name and adding that to environment.systemPackages.  I run that script to enter the devshell environment.  I'm certainly not wedded to this approach if something better is available, but I do need a way to build nix-shell environments ahead of time and then use them in an offline environment and prefer to do it in a declarative way which is compatible with nixos-generators

I'll try to figure out a solution tomorrow. This also annoys me (using nix develop / zsh). #26 has a pending fix for review, which lead me to believe the solution for this issue here can't be far away, either.

FWIW, my preference would still be to not have devshell enforce a particular prompt.  For instance, I use starship for my prompt which among other things has support for nix-shell.


dequantize_4bit() gives wrong output when working in cuda graph mode

System Info
Linux
Reproduction
I am trying to implement BitsAndBytes in vLLM (https://github.com/vllm-project/vllm). My implementation with eager-mode works right and was merged.
However, I found that the weight given by dequantize_4bit() under cuda graph mode is different from the eager mode, which makes the model output nonsense output.

I tried to put it in a simple script. Yet it turned out to be hard as it is non-trivial to capture the cuda graph. Yet it is a consistent repro and I would be more than happy to work with the community members to show the data I have collected.
Expected behavior
The cuda graph mode is expected to output the same dequantized tensors as the eager mode.

Thank you for bringing this to our attention @chenqianfzh! I'm not personally aware of a known issue here and do believe it's worth investigating further. If you could help to provide some more details on repro steps that would be appreciated!
For bookkeeping, this relates to vLLM issue: vllm-project/vllm#5569 and the current workaround is to enforce eager mode: vllm-project/vllm#6846
cc: @Titus-von-Koeller

@matthewdouglas  @chenqianfzh
I also encountered the same problem mentioned above. I conducted a simple investigation, and the likely cause seems to be that the kernel  kDequantizeBlockwise did not pass the stream(This phenomenon is common in BNB). If you want to investigate further, you can refer to cudagraph test for relevant verification.

hi @matthewdouglas vllm is waiting for a new release to pick up this fix, since they use pypi, wondering when you plan to checkpoint a new release?

@devlup We're planning release early this week.


DruidSchema and SystemSchema should avoid creating temporary collections of the cardinality of all segments

It happens in DruidSchema.getSegmentMetadata() and SystemSchema.scan(). Creating such collections means allocating gigabytes of data, if there are millions of segments in the system. It should be done only if there is really no other way. Avoiding allocations in DruidSchema.getSegmentMetadata() and SystemSchema.scan() is not very easy without introducing concurrency bugs, but I believe should be possible.

👋 I'd like to tackle this, and I have some questions on implementation approach that you'd recommend. We are seeing an opportunity to improve the segments UI responsiveness by resolving this.
My specific questions are (1) would you prefer a first-pr to unlock concurrency and a second-for query engine, or both combined, and more importantly (2) some guidance on how you'd prefer the calcite integration to be approached.
On (2) Calcite integration: the current scan() implementation fully materializes all data before filtering and discarding much of it. I can see two approaches to improve this, and alternatives currently in use in the Druid codebase:

SystemSchema currently uses the more basic ScannableTable implementation (this is suboptimal).
The core Druid sql stack uses the more primitive building blocks with rules generation

I am less familiar with Calcite but it seems from reading that:

A good middle ground would be to implement the bulk of logic via AbstractQueryableTable
The alternative implementation would be to break out a custom SegmentTable and build a parallel convention separate and optimization chain to the existing bindable convention. But, this requires building out a set of rules and transforms etc which would ultimately only be used to push down as much as possible into one loop.

Corrections and suggestions to improve my understanding of Calcite most definitely welcome!
--
Extra background
What I notice on our implementation is that there is some opportunity to improve response time for segment queries. Specifically, I can see that the scan() implementation fully materializes the data before handing it back to Calcite, at which point Calcite then performs filter, sort, limit. I expect it to be significantly faster to push down the query implementation details to the collection level, so that the filter and sort happen at an earlier stage in processing.
To make this more concrete - if you have a query such as select ... from sys.segments order by start desc limit 25 to sort and then fetch the first 25 rows, with thousands-millions of underlying segments, the joda DateTime...toString() occurs for all of them, even when not required for filter/sort, and then the majority is shortly thereafter discarded. It would be more efficient if sort & filter happened prior to materializing all data and passing it on and I expect it to improve the responsiveness of the segment UI.

I suggest this be abandoned.


Cmd-Space is not showing code-completion options

From<EMAIL_ADDRESS>on 2013-01-19T09:57:36Z
Spyder Version:  2.1.13
Python Version:  2.7.3
Qt Version:      4.7.3, PySide 1.1.0 on Darwin
What steps will reproduce the problem?

Try code-completion with Cmd+Space in the editor 2. 3. What is the expected output? What do you see instead? Completion options

Please provide any additional information below
. When the code-completion key-binding is defined to be Cmd-Space, it doesn't work in the editor. The code completion doesn't work also when assigning it other key shortcuts, e.g., Cmd-/ or Cmd-B... Strangely, when defining the key-binding to be Cmd-Shift-Space or Cmd-Ctrl-Space, it works fine.
I'm working with mac OS X Mountain Lion 10.8.2 and I have no other shortcut binded to Cmd-Space (the spotlight's shortcut is Ctrl-Space and the additional languages are also binded to different keys).
I also tried different versions of Spyder, including 2.1.11, 2.2.0 from source and 2.2dev Lion dmg and I had the same problem with them all.

Thanks,
Avihay
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1232

From pierre.raybaut on 2013-01-27T04:19:29Z
Labels: OpSys-OSX

From<EMAIL_ADDRESS>on 2013-01-28T04:06:43Z
Thank you

From ccordoba12 on 2013-01-27T06:25:59Z
I'll take a look at it, it's very annoying indeed!
Owner: ccordoba12
Labels: MS-v2.2 Cat-Editor

From ccordoba12 on 2013-01-28T14:53:20Z
This issue was updated by revision 31f271a9390b .

By default 'Cmd+Space' is assigned to Spotlight, so I remapped
it to 'Ctrl+Space'
Change Ctrl for Cmd in another couple of shortcuts that were not
working either.

Status: Fixed

From<EMAIL_ADDRESS>on 2013-01-31T11:15:49Z
Thanks.
This still didn't do the job for me; I copy/replaced the relevant lines in codeeditor.py, but it didn't work. I also tried the 2.2.0beta version and replaced its codeeditor.py file with the one in revision 31f271a9390b (needed to make a small adjustment: removed line 468  "  ('xml',): (sh.XmlSH, '', None),  "). Still, not working.
Thank you very much, anyway, I'll just wait for the next release, I hope that it will be fixed by then. Meantime, bettertouchtool, here I come!

From<EMAIL_ADDRESS>on 2013-03-10T16:39:59Z
Code completion does not work for me either (also on OSX). The default is Ctrl+Space, I also tried adding shift modifier but did not help.


Fix go-routine leak

When and higher level (grammar or syntax) error was encountered when
parsing records from strings the routine would return, but the openend
channel would be left open and no other error would be triggered; this
means the goroutine starting z.lexer would be left hanging.
The fix is a bit of a hack; as we signal the upper error back down via
s.e and the trigger an "error" in the lexer, this at least ensure the
z.lexer function return and cleans up the goroutine.
Fixes #586
Fixes https://github.com/coredns/coredns/issues/1233

I'm struggling to come up with a meaningful test.

Codecov Report

Merging #569 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #569      +/-   ##
==========================================
+ Coverage    58.3%   58.35%   +0.05%
==========================================
  Files          37       37
  Lines        9865     9877      +12
==========================================
+ Hits         5752     5764      +12
  Misses       3064     3064
  Partials     1049     1049

Impacted Files
Coverage Δ

scanner.go
100% <100%> (ø)
:arrow_up:

scan.go
80.33% <100%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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

testing the binary with the PR. This does not seem to have fixed it...

which makes sense because it is even higher where I need to signal this.


installed r1.0 -models directory is mssing .

NOTE: Only file GitHub issues for bugs and feature requests.  All other topics will be closed.
For general support from the community, see StackOverflow.
To make bugs and feature requests more easy to find and organize, we close issues that are deemed
out of scope for GitHub Issues and point people to StackOverflow.
For bugs or installation issues, please provide the following information.
The more information you provide, the more easily we will be able to offer
help and advice.
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
Environment info
Operating System:
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
If installed from binary pip package, provide:

A link to the pip package you installed:
The output from python -c "import tensorflow; print(tensorflow.__version__)".

If installed from source, provide

The commit hash (git rev-parse HEAD)
The output of bazel version

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
What other attempted solutions have you tried?
Logs or other output that would be helpful
(If logs are large, please upload as attachment or provide link).

Hi
I have installed r1.0 -pip install , the models directory is missing .
I know this has been an issue already but I cannot find a solution.]
How can I download the models.
I tried git checkout r1.0 but get an error : not a git repository
regards
Denis


Sample  document on  enforcing users to provide missing attributes while getting JIT provisioned to the local system

Solution 21 in [1] explain enforcing users to provide missing attributes while getting JIT provisioned to the local system. But currently there are no documentation explain step by step p process. End user have to  refer doc [2]& [3 ] to do this.Better if we can have a Sample step by step section which address above .
[1] https://medium.facilelogin.com/thirty-solution-patterns-with-the-wso2-identity-server-16f9fd0c0389
[2] https://docs.wso2.com/display/IS540/Configuring+Just-In-Time+Provisioning+for+an+Identity+Provider
[3] https://docs.wso2.com/display/IS540/Configuring+Claims+for+a+Service+Provider

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


"Perișoru" is listed in the drop-down menu

Description:
"Perișoru" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Perișoru" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Perișoru, CĂLĂRAȘI (Perișoru)".


A suggestion for debug exceptions layout

I think using modal with maximum CSS z-index can fix broken the layout like this:
CodeIgniter4 sample
And this is an example with bootstrap I made:
Custom sample
or one button for show and hide the error screen.
Just a little idea.
Thank you Sir, for your fast response!

The styles used should be constrained to the page itself, though we might be able to tie that down a little more. That said, I've never seen the layout do that before. We can't necessarily do modals because every front-end framework is different, and we don't need to pull an entire framework in just to handle a debug page.
Do you have any idea what was causing the layout issue to begin with?

I don't have, because an error layout (segment) is different with debug bar.
An error layout maight be wrapped by other (parent) tag/layout element.

error layout

The advantages of using modal, we can find or showing en error slightest that not detected by user/developer.

Closing this one for now. While modals are definitely an interesting solution, I can see them being more problematic as people start to integrate into their projects.
I have made changes in the past to help with this, though, including clearing the output buffer when an exception happens and a couple of other small tweaks. I think we're good here.
Thanks.


Qwen2-72B inference on 8x Gaudi2 gets OOM issue due to missing meta-device support on model loading

System Info
optimum-habana 1.12.0
synapseAI: 1.16.1

Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
step to reproduce:
deepspeed --num_nodes 1 --num_gpus 8 --master_addr <IP_ADDRESS> --master_port 60008 run_generation.py
--model_name_or_path /data/Qwen2-72B-Instruct
--max_input_tokens 1024
--max_new_tokens 512
--batch_size 1
--n_iterations 2
--warmup 3
--bf16
--use_hpu_graphs
--use_kv_cache
--reuse_cache
--use_flash_attention
--trim_logits
--limit_hpu_graphs
--bucket_internal
--bucket_size 512
it will trigger system OOM issue on system with 1TB memory instead of Gaudi2 OOM issue.
the rootcause is current optimum-habana and deepspeed still not support meta device loading model, which make system OOM in model loading stage.
walkaround is using 4x Gaudi2 instead of 8x to avoid system OOM.
Expected behavior
qwen2-72B inference on 8x Gaudi2 can run on system with 1TB memory.

I think here we are constrained by DeepSpeed unfortuantely :/

understood, we need both changes in optimum-habana and deepspeed-fork.
I will try to submit JIRA through internal system.

https://github.com/huggingface/optimum-habana/pull/1151
fix for this issue, which also depends on new deepspeed version.

@LeoZhao-Intel Now that Synapse 1.17 is out and #1163 has been merged, this should work.

sure, will verify on latest 1.17

verified on 1.17, it is fixed.


viewport width on mobile after turning screen

you might be able to get more screenwidth on mobile once the user has turned their screen (see screenshot)

Ok, I'm gonna play with this, see what's possible!


Camera model

In the backend, a camera model should be used to represent the user's view into the scene during training/editing. The camera will be interacted with via the user using their mouse/keyboard in the frontend viewer, and will send messages for how they'd like to move in the scene.
Requirements

Hold a persistent camera model that views into the scene
Camera should also hold the resolution to render the scene, and should listen for messages regarding the resolution change
Listen for messages about activity within the viewing screen for WSAD + mouse or mouse + modifiers, scrollwheel
Implement standard 3D viewer transformations to affect the camera model. For example, scrollwheel zooms on lookat target, click and drag rotates around lookat, shift/control + drag translates along camera x/y axis, etc. WSAD + mouse can also be implemented like a video game.

Backend now has a render_cam in the ServerController object which is the users view into the scene. See /src/backend/dataset/cameras.py for the RenderCam object I created to use. Has proper device checking. Camera settings are accessible via RenderSettingsWindow, communication for this is already setup between front and backend.
TODO

Implement panning/rotating/zooming in backend and frontend

Added panning/rotating/zooming with arcball model:

Rotate: click (left mouse only) and drag
Zoom: mousewheel
pan: shift + click (left mouse only) and drag

Haven't tested on a laptop yet, but the left click and zoom functionalities (and shift modifier) shouldn't be a problem. Will open new bug ticket if it is confirmed to be an issue.


numba.errors.LoweringError: Failed in nopython mode pipeline (step: nopython mode backend)

For this code:
import numpy as np
from matplotlib import collections  as mc
import matplotlib.pyplot as plt
from numba import jit

@jit(nopython=True)
def point_line_distance(x,y,a,b,c=0):
    return np.abs(a*x+b*y+c)/((a**2+b**2)**0.5)

D = 12

@jit(nopython=True)
def do_stuff():
    N = 10000
    lines = []
    flat = False
    for i in np.linspace(0, 1, N):
        for a, b in [[1,i], [i,1]]:
            flag = True
            for x in range(0, D+1):
                for y in range(0, D+1):
                    if (x!=0 or y!=0) and (x**2 + y**2)**0.5 <= D:
                        if point_line_distance(x,y,a,-b)<0.08:
                            flag=False
                            break
            if flag:
                lines.append([[0,0],[a,b]])
    return lines

lines = do_stuff()
print(lines.shape)
lc = mc.LineCollection(lines, linewidths=2)
fig, ax = plt.subplots()
ax.add_collection(lc)
ax.margins(0.1)

for x in range(0,D+1):
    for y in range(0,D+1):
        if x!=0 or y!=0:
            ax.add_artist(plt.Circle((x, y), 0.08, color='k'))
plt.show()

I get this error:
...
  File "/home/ntitus/.local/lib/python3.5/site-packages/numba/lowering.py", line 239, in lower_function_body
    self.lower_block(block)
  File "/home/ntitus/.local/lib/python3.5/site-packages/numba/lowering.py", line 254, in lower_block
    self.lower_inst(inst)
  File "/usr/lib/python3.5/contextlib.py", line 77, in __exit__
    self.gen.throw(type, value, traceback)
  File "/home/ntitus/.local/lib/python3.5/site-packages/numba/errors.py", line 607, in new_error_context
    six.reraise(type(newerr), newerr, tb)
  File "/home/ntitus/.local/lib/python3.5/site-packages/numba/six.py", line 659, in reraise
    raise value
numba.errors.LoweringError: Failed in nopython mode pipeline (step: nopython mode backend)

File "test.py", line 28:
def do_stuff():
    <source elided>
            if flag:
                lines.append([[0,0],[a,b]])
                ^

[1] During: lowering "$242.9 = build_list(items=[Var($242.5, test.py (28)), Var($242.8, test.py (28))])" at test.py (28)
-------------------------------------------------------------------------------
This should not have happened, a problem has occurred in Numba's internals.

Thanks for the report. I think this error is because the build_list op code (coming from lines.append([[0,0],[a,b]])) with the args present is not supported but somehow it's made it through to lowering without error. Marking as a bug.

I am having the same error.

This is a minimal reproducer https://github.com/numba/numba/issues/3818. I'm closing this since Numba version 0.42 does not exhibit precisely the issue reported and the minimal reproducer is sufficient. Thanks again for reporting.

@Pastthesun thanks for commenting, if you have an issue that is distinct to that presented in #3818 please could you open a new issue with a minimum working example. Thanks.


Mention including Draft CSS in 'getting started'

Since this was not clear, (see https://github.com/facebook/draft-js/issues/1197), I think we could update the docs to mention this in the 'Quick Start', 'Overview' section. https://draftjs.org/docs/overview.html#content

@flarnie Hello, I have given this one a go. Please let me know if there is any improvement required. We use draft.js a lot. I look forward to doing more contribution and 0.11.0!


Consuming the icon in a React app

Hello! Thanks for this great plugin, it looks really nice so far in the studio.
I was just wondering how one would go ahead by converting the received data to an icon component in e.g. React?
I saw a reference to icon.component in the source code, but I cannot find anything like that in the retrieved data from Sanity. Is that unrelated or have I misconfigured something and should have access to icon.component?

Hello @FluidSense. Thank you, glad you like it so far!
It's up to you really. There's no method for rendering an icon outside of a Sanity studio. It was previously discussed to implement such a method here but I decided that it's not the responsibility of this plugin to render icons in external React projects. How ever there is a helper function available to render icons as preview media inside a Sanity Studio example here.
With all that said, if your retrieved data from Sanity look like this:

Then you could create a dynamic icon generator in your external React project like so:
import * as Icons from "react-icons/fa";

const DynamicFontAwesomeIcon = ({ name }) => Icons[name];

export default function App() {
  const sanityData = {
    _type: "iconPicker",
    name: "FaBeer",
    provider: "fa",
    _updatedAt: "2021-07-25T02:30:43.141Z"
  };
  const Icon = DynamicFontAwesomeIcon(sanityData);
  return (
    <div className="App">
      <Icon />
    </div>
  );
}

Note that in this example I'm using the option outputFormat: 'react' for the icon picker in the studio which will return data with the same naming convention as react-icons. The name key can then be used to to render the correct icon. The provider key could also be used to implement logic for knowing which icon provider to use (using only fa aka FontAwesome in the example above).

Thanks for the quick answer! I'll go ahead and implement a component similar to the one in your example, and include different providers!

Quick question: if I used the above methods, for example for all three libraries, and user ends up using 2-3 icons, does that increase the file size drastically? or only the imported item will be loaded?

Quick question: if I used the above methods, for example for all three libraries, and user ends up using 2-3 icons, does that increase the file size drastically? or only the imported item will be loaded?

@surjithctly
It imports the whole module and destroys performance. My Lighthouse score plummeted, mainly because of the time to interactive increasing to 9 seconds, on localhost...
I see this "solution" given all the time but it only makes sense for SPAs where you don't care about loading speed that much (at least not for SEO's sake).
I agree that the best way would be to just provide the SVG code from sanity (or, if anyone else has a better idea...)

@ovsw Yeah, providing it as raw svg or as svg image would save a lot of bundle size for sure.

Thanks for the feedback @surjithctly @ovsw. Created a draft PR for this just now, feel free to share your thoughts there #39


style fixes

I have fixed the broken test for you.
Also, I have restored your original style, it was changed when you re-used the human_size() method, but I have preferred to change that method and keep your style, it looks much better.
Thanks very much again :)

Great, thanks!


Ref tests in annotations and add NAS Search Space Converter

Provide a tool to convert a search space for NAS to a general search space.
Has been tested for:

NAS mode: classic and enas
Tuner: hyperopt

Grid search should be supported, but it's currently not, because:

I used uniform as a workaround for the still-buggy randint.
I used uniform as a workaround to choose many.

Other tuners hasn't been tested, but I didn't see risks if the tuner supports uniform and choice itself.

great!


Controls in free play

The L1 and R1 buttons do not work to switch characters
But you can still switch between characters by pressing and holding your character icon on the touch screen

Duplicate of #8


Unexpected migration results after changing property's optionality

Goals
I noticed that sometimes my app loses data after updating to the new version. I was able to narrow down the problem - the data gets lost only on updates when I change the structure of the data - I change the schema version and the migration is performed. But the issue turned out to be that automatic migration doesn't behave the way I would expect.
Let's say we have a following class:
class User : Object {
  @objc dynamic var id : Int = -1
  @objc dynamic var name : String?

  override public static func primaryKey() -> String? {
    return "id"
  }
}

More specifically we've got a property name which is optional String (String?). And let's say we have two users: first with name "John" and the other being without a name (nil).
We decided to change it to just String with provided default value, in this case an empty string. So the updated class would look like this:
class User : Object {
  @objc dynamic var id : Int = -1
  @objc dynamic var name : String = ""

  override public static func primaryKey() -> String? {
    return "id"
  }
}

We increase the schema version and rerun the app to perform a migration.
Expected Results
First user's name hasn't changed, the second user's name was replaced by an empty string.
Actual Results
Both users' names are replaced by empty strings.
Code Sample
I have created a sample project with described class.
Steps to Reproduce
Change User object's name property from being optional String? to String with default value.
From:
@objc dynamic var name : String?

To:
@objc dynamic var name : String = ""

And rerun the app, you'll see that both object's names were replaced by the default value (in this case an empty string).
Version of Realm and Tooling
Realm framework version: 3.11.2
Xcode version: 10.1 (10B61)
macOS version: 10.14.1
Dependency manager + version: Carthage 0.31.2
Thoughts
It might be asking too much for automatic migration to perform such "migration" I would expect, but I think it could be much better if Realm would've shown at least some kind of warning that you are changing types of existing properties which will cause data loss, so manual migration is required to prevent it.

Hi @njuri - I've recently questioned this behaviour on an older, related, case. You may find the comments in #4370 useful whilst waiting for a response to this one.

This appears to be a duplicate of the behaviour discussed in #4370.


all-zero output [deepmedic]

Hi, I retrained the deepmedic model with my dataset. The training looks fine, but the segmented output is either all-zero or like this:

My config file:
[T1]
path_to_search = /home/yilinliu/Dataset/Training
filename_contains =
filename_not_contains =
spatial_window_size = (57,57,57)
pixdim = (1.0, 1.0, 1.0)
axcodes=(A, R, S)
interp_order = 3

[parcellation]
path_to_search = /home/yilinliu/Dataset/FullStructureLabels
filename_contains = _RIGHT_all_labels_8bit_path
filename_not_contains =
spatial_window_size = (9,9,9)
pixdim = (1.0, 1.0, 1.0)
axcodes=(A, R, S)
interp_order = 0

%system configuration sections
[SYSTEM]
cuda_devices = 0
num_threads = 2
num_gpus = 1
model_dir = ./models/deepmedic

[NETWORK]
name = deepmedic
activation_function = prelu
batch_size = 10
decay = 0
reg_type = L2

% volume level preprocessing
volume_padding_size = 12
% histogram normalisation
histogram_ref_file = ./example_volumes/monomodal_parcellation/standardisation_models.txt
norm_type = percentile
cutoff = (0.01, 0.99)
normalisation = False
whitening = True
normalise_foreground_only=True
foreground_type = otsu_plus
multimod_foreground_type = and

queue_length = 25
window_sampling = uniform

[TRAINING]
sample_per_volume = 1000
rotation_angle = (-10.0, 10.0)
scaling_percentage = (-10.0, 10.0)
lr = 0.01
loss_type = CrossEntropy
starting_iter = 0
save_every_n = 3
max_iter = 500
max_checkpoints = 1000

[INFERENCE]
border = (24,24,24)
#inference_iter = 294
save_seg_dir = ./segmentation_output/
output_interp_order = 0
spatial_window_size = (105,105,105)

I've tried using different iterations for the inference, but the segmentation results show no improvement.
Any help is appreciated!!

Hi @YilinLiu97 The config looks ok to me. I'd suggest that you start with simple tasks, e.g. training with a few volumes (or even synthetic examples) and making inferences on the training cases. In that way we can narrow down the issue -- if there is any.

Hi @wyli , I've tried using just 1 subject as the input and it just outputs zeros (though the loss has been decreased to ~0.008.) Also I'm using the training sample for inference as the 'exclude_fraction_for_validation' function doesn't seem to be implemented in the 0.2.1 version or it's simply not included in the config file.

Probably. Have you read the U-Net shape tutorial?

Thank you. Yes I have read it. I have solved my problem with the help of #76 . As suggested there I had changed my loss function from Dice to generalised Dice (GDSC) since my data is imabalanced.


Add Relay Content Filtering Attribute [Feature Request]

Is your feature request related to a problem? Please describe.
There is no central location that displays which relays engage in various forms of content filtering. For example, filtering NSFW content, filtering a certain language (e.g. Chinese) etc.
Describe the solution you'd like
User Story 1
As an advanced nostr relay user who is doing research on relays, I would like to see the relays that are engaging in content filtering, so that I can inform my relay configuration on my nostr client.
Acceptance Criteria for User Story 1

User can see Yes/No indication if a relay is engaging in any form of filtering

User Story 2
As an advanced nostr relay user who knows (see user story #1) that a certain relay is engaging in content filtering, I would like to with more granularity what type of filtering is taking place, so that I can inform my relay configuration on my nostr client.
Acceptance Criteria for User Story 2

User can see various filter types. For instance:
-NSFW filtering,
-language specific filtering (e.g. 🇨🇳),
-spam filtering,
-other filtering

Describe alternatives you've considered
Additional context
-Added attribute, not column in title, as horizontal real estate on nostr.watch is limited.
-Must consider current & future standards out of NIP-11.
-How is data sourced, and obtained?

Interesting. Would definitely require a NIP for this be feasible on nostr.watch. The less manual processes required, the better.  Sounds like a good candidate for NIP-11 data, the manual processes would be

Participating in the NIP-11 extensions thread
Contacting relay devs and asking their opinions.
Getting them to support it.
Then nostr-watch can happily parse the data, with no manual processes forever-more.


MQTT Broker Authentication

Great work with incorporating the modbus decryption in to your script. Note that you just need to add an extra line in for MQTT broker authentication to work.
try:
mqtt_client = mqtt.Client('pv_data')
mqtt.username_pw_set(config.mqtt_username,config.mqtt_password)
mqtt_client.connect(config.mqtt_server, port=config.mqtt_port)
except:
mqtt_client = None

Done. Also the branch update contains the new SungrowModbusTcpClient library that has recently been made available in pypi.

Merged to master.


Some image files are not closed after being read

Prawn does not close image files by default. While there is code to address this problem in Asciidoctor PDF, some of the cases are being missed. Specifically, background images and admonition icon images are not being closed.
While this doesn't generally cause problems on Linux, it can cause problems on Windows. We can observe this problem in our own test suite as the attempt to unlink temporary image files is failing.

backport this to 1.5.x

the attempt to unlink temporary image files is failing

Maybe it should mark test as failed?

Yep, good idea. At first I thought it wasn't possible, but then I thought of a way.

I can't believe that file deletion API wouldn't tell you whether it succeeded deleting or not :)

It does. I didn't realize that the error was in our log. Plus, I realized there were other cases we were missing.

It turns out that due to the order of call operations, temporary files generated by the running content weren't being deleted either.


Can't look around.

When I load into a game, my mouse is always focused on the menu meaning I can't look around in-game.

Play around with force unlock mouse in the config. It either has to be all set true or all false i dont remember


Support other directories than ./files to be templated by mergeWith/apply devkit utilities

🚀 Feature request
Command (mark with an x)
- [ ] new
- [ ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [?] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Description
For the moment, I can only templates files under the ./files directory (. being my schematic directory).
Describe the solution you'd like
I would like to be able to give a template path to my schematic and then fetch/fill these templates inside the schematic.
Describe alternatives you've considered
Not found any :cry:...

Hi @hadrien-toma, I am not quite understanding the issue being reporting here.
The ./files can be named anything, example something the below should work
apply(url('../abc'), [
      apply({
        foo: 'bar',
      }),
      move(),
    ]);

@alan-agius4 I tried but it didn't work, with no errors. I close the issue and will create another one with more environment details in case of failure. Thank you for your confirmation that it should work!


Order resources in helm status output

If you execute helm status on the same release, the ordering of the Kubernetes resources changes even if nothing else has.
I'm not sure, but it seems like the ordering is non-deterministic. I believe that they should be ordered deterministically so that multiple calls to helm status (or, some people run it inside a watch or similar) are easier to read.

@arschles I have seen the same behavior and I am all for deterministic ordering. I don't think this requires much discussion, so I am going label it as a feature. If for some reason there is problem we can always move it back to a proposal state.

Thanks @thomastaylor312

/remove-lifecycle rotten
/lifecycle frozen

/remove-lifecycle frozen
This feature is very useful, for example when using watch helm status bla.
The resources types could be sorted in the same order as they are applied.

@thomas-riccardi keeping the "lifecycle/frozen" label keeps this issue from closing. I don't think that was your intention. :)
/lifecycle frozen

The PR #3214 by @silenceshell seems to have gone stale.
Anyone else up for giving it a try?
I'd like to but my Golang skills aren't good enough yet.

#5140 seems to be the most promising PR. Assuming that gets merged, that should also solve your use case with the watch flag. Thanks for pointing out that use case!


Update examples

Right now the most up-to-date example is the staart boilerplate: https://github.com/nmaro/staart
I should go through the examples in the example folder, update them and lift the package versions.

Tests are up-to-date.


Wrong scales after stratifying matrix columns

Release number or git hash:
Web browser version and OS:

Steps to reproduce

Stratify matrix by decade

Observed behavior
All three groups show scales from 1990 to 20XX.

Expected behavior

I would expect that the scales fit to the stratified data subsets.

actually part of lineup repo:


[DISCUSSION] Best way to change wallpaper with hyprpaper?

Hello everyone, I'm opening this topic to discuss what is the best way to change wallpaper in hyprpaper:
This is how I do it right now, basically reset the hyprpaper.conf file and then overwrite it with the new info. is this the best way to change the wallpaper?
echo '' > /.config/hypr/hyprpaper.conf
echo "preload = /new-wallpaper-path/" >>  /.config/hypr/hyprpaper.conf
echo "wallpaper =,/new-wallpaper-path/" >>  /.config/hypr/hyprpaper.conf

TL;DR
Basically i use this to change wallpaper with hyprper and i use it in 3 different scripts, which are:

random-wall.sh, it takes a random wallpaper from my wallpapers folder and place it,
select-wall.sh, use wofi to list all the wallpapers from my wall folders.
set-wallpaper.sh, which sets the wallpaper that is passed as the first parameter.

With the set-wallpaper.sh script I have made a setting in Thunar, which allows me to set an image as wallpaper by right clicking.

hyprtctl hyprpaper ... is better

It seem easy, but, if you want to use  hyprctl hyprpaper wallpaper "DP-1,~/Pictures/myepicpng.png"  the wallpaper need to be preloaded, so i think i will stick to my scrips, which it works.
Buy thank you!!

you can make two calls, one for preload and one for wallpaper

Hi again, sorry for the delay. Still didnt make it work with hyperctl.
So just as example:
My wallpaper folder got around 100 differents wallpapers, and the names are "wallpapaer-1" with out .png or .jpg, with out forrmat sufix.
Trying you way will be:
hyprctl hyprpaper preload "~/wallpapers/wallpaper-1"
hyprctl hyprpaper monitor ",~/wallpapers/wallpaper-1"

For me this didnt work. Error: File not found, could be because of sufix ?
Thanks for the help

what if you specify the monitor explicitly in monitor?

Sorry it was a mistake: writting the post:
´´´sh
hyprctl hyprpaper preload "~/wallpapers/wallpaper-1"
hyprctl hyprpaper wallpaper ",~/wallpapers/wallpaper-1"
´´´
But, is not working for me. Thanks again.

hyprctl hyprpaper ... is better

Or for those of us using hyprpaper with another compositor:
printf ... | socat UNIX-CONNECT:/tmp/hypr/.hyperpaper.sock -
Unfortunately, only one user is permitted at a time that way.  It should go to /tmp/hypr-${USER} instead.

what if you specify the monitor explicitly in monitor?

(for me, at least) it does work when specifying the monitor explicitly, but it doesn't it I use hyprctl hyprpaper wallpaper ",~/wallpapers/wallpaper-1"; which is a shame, because that's what I actually want to do.  Is this intended/expected behaviour, or does it constitute a bug?

Wallpaper will apply the wallpaper to the selected output (monitor is the monitor's name, easily can be retrieved with hyprctl monitors. You can leave it empty for a wildcard (aka fallback). so i think it should work without specifying monitor, doesnt work for me tho. i just made script that gets monitors names from hyprctl and apply wallpaper for each one

@simonwiles
Just created a bash script which randomly changes background image.
It automatically gets monitor name by running hyprctl monitors | grep Monitor | awk '{print $2}'
#!/bin/bash

directory=~/YOUR_BACKGROUND/IMAGE/DIRECTORY
monitor=`hyprctl monitors | grep Monitor | awk '{print $2}'`

if [ -d "$directory" ]; then
    random_background=$(ls $directory/*.jpg | shuf -n 1)

    hyprctl hyprpaper unload all
    hyprctl hyprpaper preload $random_background
    hyprctl hyprpaper wallpaper "$monitor, $random_background"

fi

@simonwiles
Just created a bash script which randomly changes background image. It automatically gets monitor name by running hyprctl monitors | grep Monitor | awk '{print $2}'
#!/bin/bash

directory=~/YOUR_BACKGROUND/IMAGE/DIRECTORY
monitor=`hyprctl monitors | grep Monitor | awk '{print $2}'`

if [ -d "$directory" ]; then
    random_background=$(ls $directory/*.jpg | shuf -n 1)

    hyprctl hyprpaper unload all
    hyprctl hyprpaper preload $random_background
    hyprctl hyprpaper wallpaper "$monitor, $random_background"

fi

Took your idea with a lil inspiration from ZaneyOS and made a script that changes the wallpaper for all monitors every 15m. It's for a Nix config but one can easily make it a normal script if necessary.
{ pkgs, username, wallpaperDir, wallpaperGit }:

pkgs.writeShellScriptBin "wallpaper" ''
  monitor=(`hyprctl monitors | grep Monitor | awk '{print $2}'`)
  wal=$(find ${wallpaperDir} -name '*' | awk '!/.git/' | tail -n +2 | shuf -n 1)
  cache=$wal

  if [ -d ${wallpaperDir} ]; then
    cd ${wallpaperDir}
    git pull
  else
    ${pkgs.git}/bin/git clone ${wallpaperGit} ${wallpaperDir}
    chown -R ${username}:users ${wallpaperDir}
  fi

  while true; do
    if [[ $cache == $wal ]]; then
      wal=$(find ${wallpaperDir} -name '*' | awk '!/.git/' | tail -n +2 | shuf -n 1)
    else
      cache=$wal
      hyprctl hyprpaper unload all
      hyprctl hyprpaper preload $wal
      for m in ''${monitor[@]}; do
        hyprctl hyprpaper wallpaper "$m,$wal"
      done
    fi
    sleep 900
  done
''

@grapeofwrath
Nice work!!
Love seeing OSS synergy.

In my case, here's my hyprpaper.conf
preload = ~/.wallpaper
wallpaper = eDP-1,~/.wallpaper
splash = false
ipc = off

And a keybind to reload hyprpaper.
$bgreload = killall -e hyprpaper & sleep 1; hyprpaper &
bind = $extraMod, R, exec, $bgreload
So, whenever I wanna change my wallpaper, i do cp ~/Pictures/mynewbg.png ~/.wallpaper and reload hyprpaper with meta+shift+r.

Here is the bash script I use to change the wallpaper on all my monitors.
`
#!/bin/bash
Check if a file argument was provided
if [ -z "$1" ]; then
echo "Error: Please provide a wallpaper file as an argument."
exit 1
fi
Check if the file exists
if [ ! -f "$1" ]; then
echo "Error: File not found: $1"
exit 1
fi
Path to your hyprpaper configuration file
hyprpaper_config_file="$HOME/.config/hypr/hyprpaper.conf"
Update the config file with the new wallpaper path
sed -i -e "s|^preload = .$|preload = $1|"
-e "s|^wallpaper = .$|wallpaper = ,$1|"
"$hyprpaper_config_file"
Reload hyprpaper
killall -e hyprpaper &
sleep 1;
hyprpaper &
Let the user know it's done
echo "Wallpaper settings in hyprpaper.conf updated successfully."
`
usage:
./change_wallpaper.sh /path/to/your/wallpaper

I created a home manager module that sets up hyprpaper and a systemd user service to randomize the wallpaper on an interval. Additionally, it uses lutgen to theme all images within a directory local to the nix configuration. So, no matter the image, it will always be on brand.
{
  pkgs,
  lib,
  ...
}:
with lib;
with builtins; let
  theme = {
    name = "catppuccin-mocha"; # Run `nix-shell -p lutgen --command 'lutgen apply -p ""` to see supported color palettes
    wallpapers = ./some/local/directory/of/images;
  };

  themedWallpaper = wallpaper:
    pkgs.stdenv.mkDerivation rec {
      name = "${theme.name}-${builtins.baseNameOf wallpaper}";
      nativeBuildInputs = [pkgs.lutgen];

      phases = ["buildPhase" "installPhase"];

      buildPhase = ''
        cp ${wallpaper} ./${name}
        lutgen apply -p ${theme.name} ${name} -o themed
      '';

      installPhase = ''
        cp themed/${name} $out
      '';
    };

  wallpapers = filesystem.listFilesRecursive theme.wallpapers;
  themedWallpapers = listToAttrs (map (wallpaper: {
      name = "${baseNameOf wallpaper}";
      value = themedWallpaper wallpaper;
    })
    wallpapers);

  wallpaperBashArray = "(\"${strings.concatStrings (strings.intersperse "\" \"" (map (wallpaper: "${wallpaper}") (attrValues themedWallpapers)))}\")";

  wallpaperRandomizer = pkgs.writeShellScriptBin "wallpaperRandomizer" ''
    wallpapers=${wallpaperBashArray}
    rand=$[$RANDOM % ''${#wallpapers[@]}]
    wallpaper=''${wallpapers[$rand]}

    monitor=(`hyprctl monitors | grep Monitor | awk '{print $2}'`)
    hyprctl hyprpaper unload all
    hyprctl hyprpaper preload $wallpaper
    for m in ''${monitor[@]}; do
      hyprctl hyprpaper wallpaper "$m,$wallpaper"
    done
  '';
in {
  home.packages = [wallpaperRandomizer];

  services.hyprpaper = {
    enable = true;

    settings = {
      ipc = "on";
      splash = false;
      splash_offset = 2.0;
    };
  };

  systemd.user = {
    services.wallpaperRandomizer = {
      Install = {WantedBy = ["graphical-session.target"];};

      Unit = {
        Description = "Set random desktop background using hyprpaper";
        After = ["graphical-session-pre.target"];
        PartOf = ["graphical-session.target"];
      };

      Service = {
        Type = "oneshot";
        ExecStart = "${wallpaperRandomizer}/bin/wallpaperRandomizer";
        IOSchedulingClass = "idle";
      };
    };

    timers.wallpaperRandomizer = {
      Unit = {Description = "Set random desktop background using hyprpaper on an interval";};

      Timer = {OnUnitActiveSec = "1h";};

      Install = {WantedBy = ["timers.target"];};
    };
  };

You can also manually trigger it by running wallpaperRandomizer in a shell.

Here is a version without lutgen for those who just want something simple:
{
  pkgs,
  lib,
  ...
}:
with lib; let
  theme = {
    name = "catppuccin-mocha"; # Run `nix-shell -p lutgen --command 'lutgen apply -p ""` to see supported color palettes
    wallpapers = ./some/local/directory/of/images;
  };

  wallpapers = filesystem.listFilesRecursive theme.wallpapers;

  wallpaperBashArray = "(\"${strings.concatStrings (strings.intersperse "\" \"" (map (wallpaper: "${wallpaper}") wallpapers))}\")";
  wallpaperRandomizer = pkgs.writeShellScriptBin "wallpaperRandomizer" ''
    wallpapers=${wallpaperBashArray}
    rand=$[$RANDOM % ''${#wallpapers[@]}]
    wallpaper=''${wallpapers[$rand]}

    monitor=(`hyprctl monitors | grep Monitor | awk '{print $2}'`)
    hyprctl hyprpaper unload all
    hyprctl hyprpaper preload $wallpaper
    for m in ''${monitor[@]}; do
      hyprctl hyprpaper wallpaper "$m,$wallpaper"
    done
  '';
in {
  home.packages = [wallpaperRandomizer];

  services.hyprpaper = {
    enable = true;

    settings = {
      ipc = "on";
      splash = false;
      splash_offset = 2.0;
    };
  };

  systemd.user = {
    services.wallpaperRandomizer = {
      Install = {WantedBy = ["graphical-session.target"];};

      Unit = {
        Description = "Set random desktop background using hyprpaper";
        After = ["graphical-session-pre.target"];
        PartOf = ["graphical-session.target"];
      };

      Service = {
        Type = "oneshot";
        ExecStart = "${wallpaperRandomizer}/bin/wallpaperRandomizer";
        IOSchedulingClass = "idle";
      };
    };

    timers.wallpaperRandomizer = {
      Unit = {Description = "Set random desktop background using hyprpaper on an interval";};

      Timer = {OnUnitActiveSec = "1h";};

      Install = {WantedBy = ["timers.target"];};
    };
  };
}

Hi all,
Came up a similar version of the script with some modifications to make sure that my wallpaper orientation would match the orientation of the screen
!/bin/bash

while true; do
    killall hyprpaper
    FOLDER=~/.config/hypr

    # Overwrite config file with a blank version of it
    cp $FOLDER/hyprpaper.conf.bak $FOLDER/hyprpaper.conf

    # Start with blank values for preload and load variables
    PRELOAD=""
    LOAD=""

    # Loop accross all monitors
    for MONITOR in $(hyprctl monitors | awk '/^Monitor/ {print $2}'); do

        PATHPIC="$HOME/Pictures/Wallpapers/" # Determine the screen orientation

        # Get the value of orientation from hyprctl monitor
        ORIENTATION=$(hyprctl monitors | awk '/'"$MONITOR"'/{c=15} c&&c--' | awk '/transform/{print$2}')

        if ((ORIENTATION % 2 == 0)); then
            ORIENTATION="Landscape"
        else
            ORIENTATION="Portrait"
        fi

        # Append the screen orientation to the path
        PATHPIC+="$ORIENTATION/"

        # Now get a random file name and append to the path
        IMAGE=$(find "$PATHPIC" -name "*" -type f | shuf -n 1)

        PRELOAD+="preload = $IMAGE\n"
        LOAD+="wallpaper = $MONITOR,$IMAGE\n"

    done

    # Append the file name with the preload and load commands
    sed -i "/\# Preloads/a $PRELOAD" $FOLDER/hyprpaper.conf
    sed -i "/\# Wallpaper/a $LOAD" $FOLDER/hyprpaper.conf
    sed -i "s/^[{}]//" $FOLDER/hyprpaper.conf

    # Execute hyprpaper
    hyprpaper --config ~/.config/hypr/hyprpaper.conf &

    sleep 3600
done

That's all and good. However, if I try to use my user crontab (instead of looping and sleeping), I have got the following message:
[LOG] Welcome to hyprpaper!
built from commit  ()
[LOG] Using config location /home/eddie/.config/hypr/hyprpaper.conf.
[LOG] Cleaned old hyprpaper preloads (6), removing 75.6MB
[CRITICAL] No wayland compositor running!

Any idea why?

Hello all,
Thoughts on pyprpaper? It's a python script that randomly change wallpapers from a given directory or directories. I use the socket module, so there's no need to have hyprland installed.
For now the script randomly choose the wallpapers. If you have any other implementations to add, I would love to add them.
My current setup:
The hyprpaper.conf file is:
splash = off

I have a exec-once in the hyprland.conf:
exec-once = pyprpaper -m $(hyprctl -j monitors | jq -r '.[].name' | tr '\n' ' ') -- /path/to/directory

Note: You can create a cron job  or a systemd timer to change periodically the wallpaper.

Note: You can create a cron job  or a systemd timer to change periodically the wallpaper.

Hello,
I updated pyprpaper to include a --timer option, then we will be able to change the wallpaper each n seconds.
Now there is no need to use cron jobs or systemd timer.

Hello everyone,
I use this simple script to slideshow my hyprpapers but after a while is becomes plain color instead of rendering images
#!/bin/bash

interval=300
walldir=~/.local/share/wallpapers/
monitor=`hyprctl monitors | grep Monitor | awk '{print $2}'`

while true; do
    find $walldir -type f | shuf | while read -r img; do
        wal --cols16 -n -i  $img -p theme
        ln -f "$img" ~/.local/share/wallpapers/background

        hyprctl hyprpaper unload all
        hyprctl hyprpaper preload $img
        hyprctl hyprpaper wallpaper "$monitor, $img"

        sleep $interval
    done
done

I think it might have something to do with cache but im still yet to find out
hopefully this is handy to someone :)

I took a look a round and realized that hyprpaper wasn't even running so the problem was probably because an unsupported wallpaper image format was used but i'm yet to check the logs

Btw, how badly ipc affects battery life? I'm planning to replace  killall -e hyprpaper & sleep 0.1; hyprpaper & with hyprctl hyprpaper reload , $HOME/.local/share/wallpaper

I wrote a simple script for a single monitor setup to apply wallpaper immediately and load it on startup.
#!/usr/bin/sh

#This is a script to set a wallpaper for hyprpaper on a single monitor setup
#ipc must be turned on inside hyprpaper.conf

bgdir="$HOME/.local/share/backgrounds" #the directory, where all wallpapers are stored
bg="$HOME/.local/share/wallpaper"       #the path to wallpaper, defined in hyprpaper.conf

cd "$bgdir"
ln -sf "$bgdir"/"$@" "$bg"
hyprctl hyprpaper reload , "$bg"
notify-send -i desktop "hyprpaper" "Wallpaper has been applied!"


lengthMenu show all

lengthMenu: [[10, 25, 50, -1], [10, 25, 50, "All"]] only shows 1 on the All option.
https://datatables.net/examples/advanced_init/length_menu.html

I get the same behaviour.
the -1 is not interpreted as all but actually as -1; maybe if we can call a function that returns the total number of rows and put it there that will be a workaround but I did not find such a function and way to place it there.

I have the same issue. DataTables interprets "-1" as turning pagination off entirely. I tried passing the "paging: false" option to the table and it also causes only 1 record to be returned.

Probably needs implementing within the limiting code in this package. Anything related to filter/sort/skip/limit needs specific handling since we're doing just-in-time publishing.
I'll accept a PR or fix this when I have time.

I'm reading the code. I've tested using limit = 0 while removing the if condtion:
  // Verify that limit is not 0, because that will actually
  // publish all document _ids.
  if (limit === 0) {

Using limit = 0 retrieves all documents.

The reason why pagination is not working is because DataTables  -1 paged value is being wrapped as a mongodb limit option. So the find query should be passed 0  as limit option if the user selects -1 pagination as a DataTable paging option.

Should be fixed in 1.3.0 release


Try to taint server resources on patch

I thought we said we were going to put this in the Terraform collection so it could easily be used elsewhere?

We are, but @assumptionsandg is going to look at doing that next :)

Superseded by #31


"Brain" object initialised with "curv" parameter; doesn't exist

MNE 0.16.1, mne.viz._3d.py, line 1665:
    brain = Brain(subject, hemi=hemi, surf=surface, curv=True,
                  title=title, cortex=cortex, size=size,
                  background=background, foreground=foreground,
                  figure=figure, subjects_dir=subjects_dir,
                  views=views)

Variable "curv" is not part of the init parameters for class Brain. Seems to work fine if I just remove it.

Thanks for the report, this is fixed in master and will be in the next maintenance release.


Only open Realm when it is being used

A realm is not meant to be read from a single location like this. It should be opened dynamically and as needed. If you want to pre-emptively open a realm to begin the "sync" or "connection" process, that can be passed around as needed

 Only open Realm when it is being used


Feature overrides aren't applied in the editor

Godot version
4.0.dev (5df8eae86292303fefe2d412fb8c04126e7ebe87), 3.x too
System information
Mageia 9 Linux, x86_64
Issue description
As found in #64014 and #61908, .editor feature overrides added by these PRs don't seem to work, even though TOOLS_ENABLED builds do define the editor feature.
Feature overrides seem to be explicitly disabled in the editor by this code:
https://github.com/godotengine/godot/blob/5df8eae86292303fefe2d412fb8c04126e7ebe87/main/main.cpp#L1311-L1314
I'm not sure why that's needed. We discussed this on RC and I'm not convinced this solves an actual problem:
[Akien] reduz: Do you remember why feature overrides are disabled in the editor? https://github.com/godotengine/godot/blob/master/main/main.cpp#L1314
[Akien] We've started using the `.editor` feature override for some project settings and it seems like it's actually not working because of this.
[reduz] Akien: So you can actually edit the value and not have the override change things for you
[reduz] Also the overrides only work on load. I need to do some refactoring of how it works so its done on the fly.
[KoBeWi] but the tags can't be modified on the fly, so why? 🤔
[Akien] What kind of edited value would be problematic if it impacts the editor? For example if you add a `.pc` override, wouldn't you expect it to impact the editor? (can be after restart, that's fine)

Needs testing IMO.
Here's a list of existing feature overrides which could be relevant for the editor and are currently not respected:
editor
These should apply.
$ rg "GLOBAL_DEF.*\.editor"
core/config/project_settings.cpp
1202:   GLOBAL_DEF("display/window/energy_saving/keep_screen_on.editor", false);

main/main.cpp
621:    GLOBAL_DEF("debug/settings/crash_handler/message.editor",

pc
This one is debatable, not sure where editor instance logs would go.
$ rg "GLOBAL_DEF.*\.pc"
main/main.cpp
1338:   GLOBAL_DEF("debug/file_logging/enable_file_logging.pc", true);

mobile
These would be relevant for the Android editor. Currently it will use desktop-optimized settings so it's much more demanding than regular Android exports and might crash GPUs.
$ rg "GLOBAL_DEF.*\.mobile"
scene/main/scene_tree.cpp
1473:   GLOBAL_DEF("rendering/shadows/positional_shadow/atlas_size.mobile", 2048);

servers/rendering_server.cpp
2842:   GLOBAL_DEF("rendering/shadows/directional_shadow/size.mobile", 2048);
2845:   GLOBAL_DEF("rendering/shadows/directional_shadow/soft_shadow_filter_quality.mobile", 0);
2850:   GLOBAL_DEF("rendering/shadows/positional_shadow/soft_shadow_filter_quality.mobile", 0);
2856:   GLOBAL_DEF_RST_BASIC("rendering/vulkan/rendering/back_end.mobile", 1);
2876:   GLOBAL_DEF("rendering/reflections/sky_reflections/texture_array_reflections.mobile", false);
2878:   GLOBAL_DEF("rendering/reflections/sky_reflections/ggx_samples.mobile", 16);
2881:   GLOBAL_DEF("rendering/reflections/reflection_atlas/reflection_size.mobile", 128);
2890:   GLOBAL_DEF("rendering/shading/overrides/force_vertex_shading.mobile", true);
2892:   GLOBAL_DEF("rendering/shading/overrides/force_lambert_over_burley.mobile", true);
2971:   GLOBAL_DEF("rendering/environment/glow/upscale_mode.mobile", 0);

debug
To be assessed.
$ rg "GLOBAL_DEF.*\.debug"
main/main.cpp
617:    GLOBAL_DEF_RST("application/run/flush_stdout_on_print.debug", true);

web
Should be used for the Web editor:
$ rg "GLOBAL_DEF.*\.web"
servers/audio_server.cpp
188:    GLOBAL_DEF_RST("audio/driver/mix_rate.web", 0); // Safer default output_latency for web (use browser default).
190:    GLOBAL_DEF_RST("audio/driver/output_latency.web", 50); // Safer default output_latency for web.

windows
$ rg "GLOBAL_DEF.*\.windows"
main/main.cpp
1780:           GLOBAL_DEF_RST_NOVAL("input_devices/pen_tablet/driver.windows", "");

Steps to reproduce
n/a
Minimal reproduction project
No response

Note that if this is addressed, it will remove the ability to define project settings that only apply within the editor (e.g. as requested in https://github.com/godotengine/godot/issues/69528). I suppose this would have to be added another way, such as a editor_hint feature tag.


Fix missing FontAwesome icons in the footer

Fixes missing FontAwesome icons located in the footer. Issue #1072

Looks good!

Coverage remained the same at 96.06% when pulling 582c972c01a4b737e57b41c3bbdb6a27a23a1346 on maticbasle:patch-1 into 09ead7e187f332721374d1e00c7afcf48be9a05e on openfarmcc:master.


Quiet output for missing atom types

Previously, when applying a forcefield with many missing atom types
in the forcefield XML, foyer would output as a warning all the
missing atom types to the user upon successfully atom typing.
For the larger forcefield files (oplsaa.xml for example), there are currently
~750 missing atom types that do not have a smarts string. This led to cumbersome
warning messages.
Now, the default behavior for missing atom types is to still warn the user, but
condense the information to ~2 lines of output.
If the user wants to know which atom types are missing a smarts string,
a debug flag can be passed during the application of the forcefield to
revert to the previous behavior.

Is there an option to dump local bonded environment of the missing atom types yet?
On Oct 11, 2017, at 9:02 PM, Justin Gilmer <notifications<EMAIL_ADDRESS>wrote:
Previously, when applying a forcefield with many missing atom types
in the forcefield XML, foyer would output as a warning all the
missing atom types to the user upon successfully atom typing.
For the larger forcefield files (oplsaa.xml for example), there are currently
~750 missing atom types that do not have a smarts string. This led to cumbersome
warning messages.
Now, the default behavior for missing atom types is to still warn the user, but
condense the information to ~2 lines of output.
If the user wants to know which atom types are missing a smarts string,
a debug flag can be passed during the application of the forcefield to
revert to the previous behavior.

You can view, comment on, or merge this pull request online at:
https://github.com/mosdef-hub/foyer/pull/143https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmosdef-hub%2Ffoyer%2Fpull%2F143&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=qBz8hF7xJlNx4JqKHEbzok6wyzNzQv6Pqkvav2e9oCs%3D&reserved=0
Commit Summary

Quiet output for missing atom types

File Changes

M foyer/forcefield.pyhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmosdef-hub%2Ffoyer%2Fpull%2F143%2Ffiles%23diff-0&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=eF%2FVlsayRsDDH1d%2BOMofofKBdAnR9s88IOJhAG1fdVY%3D&reserved=0 (4)
M foyer/validator.pyhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmosdef-hub%2Ffoyer%2Fpull%2F143%2Ffiles%23diff-1&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=NY8JUsZDfUmeq02jpjsXvxra3Df1afsB%2FigYiYyJCes%3D&reserved=0 (12)

Patch Links:

https://github.com/mosdef-hub/foyer/pull/143.patchhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmosdef-hub%2Ffoyer%2Fpull%2F143.patch&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=nOIJOiWcSP24OTmL128ieCsPmQMC9XCvxets0bzYxxE%3D&reserved=0
https://github.com/mosdef-hub/foyer/pull/143.diffhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmosdef-hub%2Ffoyer%2Fpull%2F143.diff&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=0BHAnotyaR2qBmAKlVUZ%2Fg%2BYLhzSzL0aMJdLrw9tcQA%3D&reserved=0

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmosdef-hub%2Ffoyer%2Fpull%2F143&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=qBz8hF7xJlNx4JqKHEbzok6wyzNzQv6Pqkvav2e9oCs%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABW4qxNEUeI8hWbVlSXlAf-YOsBsNQXiks5srY9pgaJpZM4P2cDU&data=02|01|christopher.r.iacovella%40vanderbilt.edu|1cc700d20d28403ce2ed08d511261b06|ba5a7f39e3be4ab3b45067fa80faecad|0|0|636433777716758774&sdata=tDhVvDK9Pu2opbDrT1Sl%2BEmxA%2BSlY40BZOPVYpvpeB8%3D&reserved=0.

Not yet, this fix is just to quiet the warning associated with missing atom types in the forcefield file itself. These warnings are not associated with the actual atom typing errors of the system of interest.
I think we should be able to do that as well, however, I need to do a bit more research to see.

Looks good. Just needs the one small fix that @ctk3b pointed out. The failing build appears to be due to the issue resolved in #144


Customizing views/themes when using the Nuget package?

I configured the Admin with EF by using the IdentityServer3.Admin nuget package.
Going forward, what the best approach to customize the style of the views?
Can it be done by extending the application that uses the the Nuget package or I need to modify the IdentityAdmin project itself, build it and then reference it in the application that uses it?

There is currently no support to modify or extend the views, so you should modify the project itself and reference that in your application.

I understand. Thanks.
Not sure if this is the right place to ask but is it the same with IdentityManager?

Yea, the IdentityManager core is basically the same as the Admin repo.
The views and business logic have different implementations, there are some other admin specific things, but the core is the same

Ok. Thanks.


Is there a way to create a text in 3D, set its origin (coordinates), and font size, then save it as vtp or vtu in Meshio?

Make sure to include a MINIMAL, COMPLETE EXAMPLE that can be copy-pasted and shows the
bug.
If you're having problems with a mesh, you can attach it as a zip file. Make it small!
Use GitHub markdown,
especially triple-fences for code blocks.

No.


 Please run npm cache verify and then try to install again.

And follow the issue template.


Support SSL_CERT_FILE for CA certificate configuration

Somewhat similar request to https://github.com/typhoeus/typhoeus/issues/609, but much less dangerous.
Most other Ruby HTTP clients support the SSL_CERT_FILE environment variable out of the box as a way to specific a CA file that should be used to verify TLS certificates. From some quick testing this is currently supported by:

net::http
HTTParty
rest-client
excon
mechanize
gem install
probably lots more

Unfortunately, it doesn't seem to be supported by Typhoeus.
It'd be very useful if it was, or if there was some other clear equivalent option (i.e. an easy way to configure a CA to trust process-wide).

Hey @pimterry, I'm pretty certain that SSL configuration is supported with the Faraday adapter.
Probably it will look something like
client = Faraday.new('https://myspace.com', ssl: { client_cert  => "...", ...})

Also, I'm think you can use these options directly when building a request straight from typhoeus
Typhoeus::Request.new("https://www.google.com", sslcert: "path", headers: { "User-Agent" => "My User Agent"})

You can see all the ssl options here.
Does this help?

Thanks, but no that doesn't really help - I'm really looking for global configuration, rather than having to set this separately on every request.
Is there a reason not to follow the convention followed by all the other libraries above, and many other languages & tools? It works nicely for this case, and would provide a nice workaround for #609 that doesn't involve disabling HTTPS validation entirely.

As far as I can tell, excon is using ENV var but HTTParty exposes it as a function.
I much prefer the approach of HTTParty that makes it more explicit. In the example of the HTTParty, I suppose the code can be replicated as
class Client
  BASE_URL = "https://example.com".freeze
  DEFAULT_HEADERS = { "User-Agent" => "My User Agent" }.freeze!

  def self.fetch
    response = build_request("/resources").run
    ...
  end

  def self.build_request(path)
    Typhoeus::Request.new(BASE_URL, sslcert: ENV["SSL_CERT_FILE"], headers: DEFAULT_HEADERS)
  end
end

I'm not necessarily opposed to adding it as global configuration but I prefer approach that you build a connection/client and you re-use it from there.

As far as I can tell, excon is using ENV var but HTTParty exposes it as a function.

HTTParty exposes an explicit API, but it also supports the env var too - setting SSL_CERT_FILE definitely seems to work there in my testing. I think this is because it just uses the net::http builtins internally, which support this out of the box.
In my particular case, it's much preferred to use the env var. It's already set for all our environments, which means most tools (not just Ruby - many other tools & languages too) all just work out of the box. It's not impossible but it is awkward to need special configuration just for one library, when everybody else handles this automatically. I'd be happy to do with any dev work required if that would help!

Go for it 💪


Update staker rewards doc about undelegating operators

Operators who are undelegating should still get rewards for their ETH
but only for the portion of ETH that is currently bonded.
This is just a documentation update, no changes in the script generating
the allocation. Those changes are being implemented in https://github.com/keep-network/keep-ecdsa/pull/667.

Let's hold on with merging this PR until #667 is approved.

Let's hold on with merging this PR until #667 is approved.


Add Builder for BrokerClient Constructor

        client = BrokerClient.builder()
            .serviceUrl("https://github.com/broker-game/broker-dev-environment")
            .build();

https://github.com/broker-game/broker-git-client/pull/69

    @Test
    public void given_ClientBuilder_when_BuildWithMinimumParameters_then_CreateBrokerClient() {

        BrokerClient client = BrokerClient.builder()
            .serviceUrl(BROKER_TEST_ADDRESS)
            .topic("demo")
            .build();
    }


Different input required in portal vs VSCode when using the Dataverse connector

Describe the Bug with repro steps
Having issues using VS code with the Dataverse connector and when using the Bound to Action method.
In Azure Portal it seems to work fine, but when using VS code I suddenly get additional required parameters that are mandatory. I need to manually remove the parameters from the json every time I use the designer. The workflow runs when I publish and the parameters has been removed, if I just fill in any value in the parameters and publish the workflow, it fails on every run.
It has nothing to do with the input to the action in Dynamics, the required parameters shows up on every action.
This is what VSCode generates as mandatory parameters.
"body": {
"error": {
"code": "",
"message": ""
}
}
What type of Logic App Is this happening in?
Standard (VSCode)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos

Browser
VSCode/Chrome
Additional context
No response

@ccastrotrejo - We need to update vscode vsix with the latest hotfix we did for portal, these contain bug fixes for the dynamic issues.

Will release a new extension version by the end of the day today

@Trumslagaren please update the extension to the newest version. This latest version will include the fix for it.


Initialize core_net types safely

This removes the assumption that the two types have the same memory layout, which could lead to invalid memory access if it changes in a future version of rust's standard library.

Thanks, much better.


Merge ought error out when passed a string.

current behavior:
  foo = Immutable({key: 'value'})
  // Object {key: 'value'}
  foo.merge('whoops')
  // Object {0: "w", 1: "h", 2: "o", 3: "o", 4: "p", 5: "s", key: 'value'}

While this conforms with JavaScript strategy of coercing strings into objects, it leads to crazy errors. FWIW, lodash simply ignores strings that are passed to merge.

+1


Duplicate Username Error

ChatterOrgChartControllerTest class 	test_get_users method returned this error : System.DmlException: Insert failed. First exception on row 0; first error: DUPLICATE_USERNAME, Duplicate Username.The username already exists in this or another Salesforce organization. Usernames must be unique across all Salesforce organizations. To resolve, use a different username (it doesn't need to match the user's email address). : [Username]
Stack Trace: Class.ChatterOrgChartControllerTest.test_get_users: line 40, column 1

Thanks @rashminirmal. This is being handled at https://github.com/DouglasCAyers/sfdc-chatter-org-chart/pull/4

Thanks @DouglasCAyers, i'll close this issue out then, thanks for the update!


aks: vendor 20201201 sdk and add 'enable-encryption-at-host' parameter

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

aks

aks

hi @andyzhangx the change looks good. Pls run live test to make sure no regression. thanks.

hi @andyzhangx the change looks good. Pls run live test to make sure no regression. thanks.

hi @andyzhangx the change looks good. Pls run live test to make sure no regression caused by API version bump. thanks.

@yungezz I have run the live tests and also fixed the test case issues, PTAL again, hanks.

hi @andyzhangx the change looks good. Pls run live test to make sure no regression caused by API version bump. thanks.

@yungezz I have run the live tests and also fixed the test case issues, PTAL again, hanks.

@yungezz  could you merge this PR? thanks.

@yungezz  could you merge this PR? thanks.

hi @fengzhou-msft is this PR ready to merge? thanks

hi @fengzhou-msft is this PR ready to merge? thanks


As a developer, I want to manage Django tasks through a single role so that I can simplify existing deploy playbooks.

Is your feature request related to a problem? Please describe.
Currently, functionality related to deploying Django apps is spread across several roles:

configure_logging
configure_media
django_collectstatic
django_compressor
django_migrate
install_local_settings

Describe the solution you'd like
Ideally, these could all be tasks that are part of a single Django-specific role, which could have shared defaults and dependencies on other roles. New Django-related tasks (for example, https://github.com/Princeton-CDH/geniza/issues/117) can also become part of this role.
Additional context

Ansible django_manage plugin, from community.general
existing ansible-role-django on github

Should there be a python role? Logging seems like possibly not django-specific (although perhaps our implementation is); what about virtualenv creation?
Will you duplicate existing tasks so that old playbooks are left unchanged but playbooks for applications deployed on PUL VMs would start using the new role?
Are the linked django ansible plugins and roles useful at all or just examples we can refer to?

Should there be a python role

Yes! I think we can basically directly convert the create_virtualenv role to a python role; it's a dependency of this role. Will do the necessary parts of that in this issue, but eventually the python role should probably also handle installing the relevant version of python.

Will you duplicate existing tasks

Yep.

Are the linked django ansible plugins and roles useful at all

The linked plugin is quite useful and we actually already use it in another role; the linked role does too. It just allows running arbitrary manage.py commands with a bit of extra checks/helpers. The linked role is more of an example; in my investigation it appeared to be far too generic/featureful and would need a lot of configuration for us to use as-is. I'm referring to it in case it has useful conventions.

Great, thanks for elaborating. Sounds good.
Looking forward to having this managed and organized better!

[ ] write molecule verify.yml
[ ] test against cdhweb in QA

implemented in PR #73 ; remaining tasks have been split into separate issues


hot fixes SOD and COF to make them compatible with scikit learn

All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?
[x] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[x] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[x] Have you created a .py in ~/pyod/models/?
[x] Have you created a _example.py in ~/examples/?
[x] Have you created a test_.py in ~/pyod/test/?
[x] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build 1727

23 of 25 (92.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.789%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/models/sod.py
9
11
81.82%

Totals

Change from base Build 1715:
0.0%

Covered Lines:
4663

Relevant Lines:
4868

💛  - Coveralls

Hi @yzhao062
As far as I know, this is not gonna be merged into the repo.
Please confirm so I can close it.

This is correct. I think I also made some changes at that time to fix it. I will just close it for now :)


fixed descriptor support

Support uring fixed descriptors, often also called direct descriptors by the io_uring man pages.

Awesome! This is another feature I am looking for 😄. This also has interesting implications on the Send semantics of File which should probably be discussed. With normal fds, it is possible to make the File type Send. With fixed fds, I don't believe that the File type can be Send. Would it make sense to make two different File types (e.g. File and FixedFile), one of which uses normal Fds and is Send and one which uses Fixed Fds and is !Send?

I'm not inclined to want two forms of File, FileFd and FileFixed, unless we find more reasons for letting the compiler differentiate one from the other. Maybe a different method set? Maybe a different lifetime or close semantics?

Will have to resolve conflicts tomorrow.

Trying to convert this to draft while it relies on my branch for the io-uring dependency.

Replaced by another PR, #256, which has the same code. The diffs against the mainline should be easier to follow.


Generate safari-pinned-tab.svg again.

Fixes #26588
BTW I don't have a macOS so I only rely on https://github.com/twbs/bootstrap/issues/26588#issuecomment-393237229

Anyone on macOS who can test this?

Looks good to me - LGTM 👍

Thanks, @jackmu95!


HDDS-8148. Improve log for Pipeline creation failure

What changes were proposed in this pull request?
Log improvement adding excludedNodes for pipeline creation
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-8148
How was this patch tested?
E2E test with debug creating similar scenario and checked logs.

The ticket description doesn't seem to match the PR's changes. You are logging new information and not changing the existing.

Thanks @xBis7 for reviewing, I just added additional information in existing log