[Enhancement] Upgrade: ignore Longhorn CSI pods when draining a node

Is your feature request related to a problem? Please describe.

Longhorn doc advises that users should drain a node with --pod-selector='app!=csi-attacher,app!=csi-provisioner'. We should follow this in the upgrade path.
Describe the solution you'd like

We should pass the pod selectors when draining a node.
Describe alternatives you've considered

Additional context

https://github.com/rancher/rancher/issues/40003

Longhorn implements using PDB to avoid draining important pods in 1.3.3: https://github.com/longhorn/longhorn/issues/3304
We can bump to a version with this feature and avoid using the pod selector.

This is backported to LH v1.4.2, and will need to wait for the new LH release.

Test Plan

Fresh install a 3 nodes harvester v1.1.2  cluster
Login harvester via VIP.
Test following on each node one by one

Check csi-attacher and csi-provisioner exists
Enable maintenance mode
Check csi-attacher and csi-provisioner should NOT exists
Disable maintenance mode

Upgrade harvester cluster to v1.2.0-rc2
Login harvester via VIP.
Test following on each node one by one

Check csi-attacher and csi-provisioner exists
Enable maintenance mode
Check csi-attacher and csi-provisioner should still exists
Disable maintenance mode

After bumping LH to v1.4.2, the PDB is created correctly. However, during the node drain, the csi-related pods will get evicted immediately without waiting for the VM's pods to be evited first(potentially due to the VM workload type is not covered in the https://github.com/longhorn/longhorn-manager/pull/1764), will consider moving this to the v1.2.1 milestone since this will require upstream dependency fix.

After bumping LH to v1.4.2, the PDB is created correctly. However, during the node drain, the csi-related pods will get evicted immediately without waiting for the VM's pods to be evited first(potentially due to the VM workload type is not covered in the longhorn/longhorn-manager#1764), will consider moving this to the v1.2.1 milestone since this will require upstream dependency fix.

@guangbochen Did you mean CSI pods have not been protected correctly? CSI pods will be allowed to evict when no attached volumes which means no PDB or at least one CSI pod exists if there are still volumes attached.

@innobead yes, below are my testing steps based on the VM scenario, but the volume and VM pods are both exist on the node that I want to drain.

verify there is no PDB of csi pods
create VM with RWX volume
PDB is created for those csi pods
Drain the node by kubectl drain <node-name> --ignore-daemonsets --delete-emptydir-data --force
the csi pods on the drained node will get evicted immediately, and VM's pod will re-spawn on the other node but only after the csi-pods were re-created.
the csi PDB is still exist

@guangbochen
The CSI PDB only requires that there is >=1 CSI pod in the running state
If there are multiple CSI pods running on node-1, node-2, and node-3. When you drain node-1, the CSI pod on node-1 will be evicted immediately without waiting since there are still 2 running CSI pods. This is fine. Do you have any concern about this behavior @guangbochen ?

We don't need this on the Rancher side since LH uses PDB to guard it.


Use Google Site Verification

In commit f60efad, there is now support for a Google Site Verification code in the environment variable file. The M+ team has requested we use this in the production site as soon as possible.

Done


PathLength Meters to Kilometers

Simple PR to update the geometry.path_length to be in kilometers instead of meters.

We've checked, merging despite check failures!


No currency symbol for Turkey

The currency symbol TL (for Turkish Lira) doesn't appear on http://www.lonelyplanet.com/turkey/essential-information (and links from this page like Telephone) and http://www.lonelyplanet.com/turkey/money-costs
This is fine on chrome on mac but doesn't work on chrome on PC

@madmaniak this appears to only be happing in windows 7

Wow, surprised by our numbers here...

New turkish lira sign was introduced in 2012 (two years after Widows 7 release). There are some problems with displaying it on Windows 7, Microsoft suggests downloading an update (https://support.microsoft.com/en-us/kb/2739286), I'm not sure why it's not being updated automatically. One solution which comes to my mind is adding some js code, which will detect Windows 7 and replace text symbol with image. What do you think about that? @lemmiestone

This is how svg based solution looks on Windows7/chrome:

I'm afraid that's the best we can do, ship it? @lemmiestone

I think that's fine for those users.  They need to upgrade :)

@lemmiestone that's fixed now, can be closed


Updated build Workflow

Current dependencies on/for this PR:

main

PR #111   👈

PR #112

PR #113

PR #114

This comment was auto-generated by Graphite.

Graphite Merge Job
Current status: 🔄 Waiting for checks to pass
This pull request is currently waiting for CI and mergeability checks to pass before being merged.
This comment was auto-generated by Graphite.

Job Reference: vmtcHKsEznLlBvD4q5aU


Update code snippet

Proposed changes

In the code sample mentioned in Start a service with volumes it says

The following example starts a nginx service with four replicas, each of which uses a local volume called myvol2.

But --replicas=4 is missing from the code sample which reads:
$ docker service create -d \
  --name devtest-service \
  --mount source=myvol2,target=/app \
  nginx:latest

Instead of this it rather should be:
$ docker service create -d \
--replicas=4 \
--name devtest-service \
--mount source=myvol2,target=/app \
nginx:latest

Unreleased project version (optional)

Related issues (optional)
Fixes #5070

I updated the title and description to the correct syntax for linking to the issue. You should not reference issues or PRs by ID in commit messages or Github issues / PRs. If you do, then each time the commit is merged into a new branch it will ping (and maybe close) any mentioned issue / PR in the commit message (or PR title, which becomes the merge commit message).

@arush-sal (hoping I got your handle right this time 😂)
Let's ship this!

@mstanleyjones Thank you for your valuable feedback, going forward I'll definitely keep that in mind. I am not able to identify any change(s) except {Address -> Fixes} made by you in the description, could you please point any other change(s) that you have done. So that I can avoid the same in future.
@joaofnfernandes Yes sir, absolutely correct this time and thank you for the merge :)

@arush-sal the only feedback was for you to not include Fixes #5111 or anything like that in the commit title or subject. This is (1) and (2) in the screenshots below. Instead, you should add fixes #5111 to the pull request, which is (4) in the screenshot below.

 So if you look at the commit that was merged 52669983f840e22d4bbac5c53739e9f69766e7d6, you'll notice that it doesn't contain the issue id.


Update the example environment variable names

Some environment variables names aren't up to date in the examples.
Not sure about DEFAULT_SERVER_NAME, should it be NAME?

Hey, thanks a lot for contributing! Yes, DEFAULT_SERVER_NAME should be NAME - would you mind updating your PR? Thanks!

Thanks for your contribution! 👍

@thasmo ahah, got your message at work, you beat me :)
Thank you for this docker image, it's really well done and documented!

Thanks for the contribution and your feedback. Keep 'em coming if you like, we'd love to get more feedback. :)


Type inference for generic instance creation

Type inference for generic instance creation.

Replacing redundant code <T> with <> for inferred type

This feature was added in Java 7 back in 2011 (!). The pom.xml explicitly specifies Java 7 so this does not break anything.
Minor refactoring:

Removing unused method parameter for private helper method
GenericObjectPoolConfig poolConfig is passed to the method but never used

Declaring modifiers in recommended order (static final) 6fada2e

References: Oracle docs

@ullenius Are these all? Did you look for more codes similar to the changes you have made?

@ullenius Are these all? Did you look for more codes similar to the changes you have made?

@sazzad16 I added 3 more commits fixing similar stuff.
There are 2 more occurrences in ClusterScriptingCommandsTest that I did not change.
I'm somewhat reluctant to modify the tests given that they fail to to run as it is.

@ullenius Let us know when you're done changing.

@ullenius Let us know when you're done changing.

@sazzad16 I'm done, thanks.


Cholesky factorization .

Hi. i would like to use a  SPOTRF function, which computes the Cholesky factorization. I i have one question: how using this function:

spotrf()
or
cblas_spotrf() ?

And from which include to use? cblas.h ?
thank a lot for attansion and for the help.

After OpenBLAS installation (e.g. make PREFIX=/your/path install), you can include lapacke.h.
Then, call LAPACK_spotrf in your C source file.

Thank you :)

So, but i have the next problem. Here:

http://www.netlib.org/lapack/explore-html/dd/dce/cpotrf_8f.html
function is described: cpotrf(char, int, complex diemension, int, int);

but in my linux, when i use lapacke_cportf I get an error:
LAPACKE_cpotrf(int, char, int, float complex*, int)...
Sorry, but i dont know how i should use this function. Could you give me same example ?


[IDEA] Add "My Projects" page that lists projects you've bought into

Is your feature request related to a problem?
@gtmknight wants to be able to see tokens that he holds that he yet to claim.
Describe the feature/solution you'd like
One possible solution is to add a "My Projects" section that allows a Juicebox user to see a list of all the projects they hold tokens for. The list would either visibly show claimable amount and/or be filterable by claimable.
More details
From this twitter thread.

Link to notion task


Android - Keep AppBuilder instance

What does the pull request do?
Keeps AppBuilder instance in memory, and skips initializing it on conservative launched unless the app process has been terminated.
What is the current behavior?
On consecutive launches, a new app builder instance is created and initialized. If the app's process hasn't been previously terminated, this re-initialization causes a crash with the following message, **System.InvalidOperationException:** 'Setup was already called on one of AppBuilder instances', as a check for re-initialization fails.
What is the updated/expected behavior with this PR?
When an app is launched with a new process, the AppBuilder is saved in a static field. On consecutive launches, as long as it isn't a new process, the initialization of a new AppBuilder is skipped. This also has the effect of speeding up consecutive launches.
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues
Fixes #9842

You can test this PR using the following package version. 11.0.999-cibuild0028035-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]


Technology Research - Passwordless Authentication

This issue will be for tracking the research and testing of Auth0's Passwordless Login service.
The service offers a free plan with up to 7500 active users with unlimited signins.
There are SDK Libraries available, supporting multiple configurations including Express.js.

Sample response from social login using auth0, anonymised because I was using my own login for this.
{
  "sub": "windowslive|<hexidentifier>",
  "given_name": "<String>",
  "family_name": "<String>",
  "nickname": "<String>",
  "name": "<string>",
  "picture": "https://apis.live.net/v5.0/<hexidentifier>/picture",
  "locale": "en-US",
  "updated_at": "2024-02-29T09:59:35.831Z"
}

To enable passwordless login, we need to connect Auth0 to an SMTP provider. Mailgun is among those supported though! That being said however, it does appear that we need to have a domain set up for mailgun to send through. I'm going to look at taking up a free webhosting offer for this.
Any domain that's available on get.tech is an option, except for those with a crown symbol, since they're "premium" domains that cost thousands. Premium domains will be highly sought after ones like 3 letter combinations or highly common words.

Here's a sample of the user object created from an account generated via passwordless sign-in
Anonymised like before
{
    "sid": "<String>",
    "nickname": "<emailuser",
    "name": "<emailuser>@<domain>",
    "picture": "https://s.gravatar.com/avatar/<id>?s=480&r=pg&d=https%3A%2F%2Fcdn.auth0.com%2Favatars%2Fna.png",
    "updated_at": "2024-03-04T10:29:07.260Z",
    "email": "<emailuser>@<domain>",
    "email_verified": true,
    "sub": "<String>"
}


generalize ODE decoder definition

make_ode_decoder should be able to construction an ODE of any degree not just 2 and 3.
Make this runnable on GPU

done with 1b733f3d9a5656208133bb159cbd4f5a5330b270


Hyperion for Windows

Feature request
This is a step tracking issue for full Windows support. Everyone can grab a task, the order is free (as possible).
Windows support is still experimental

[x] Compilation (Hyperion can be compiled) #738
[x] Documentation (To recreate a basic compilation system) #738
[ ] Running (Hyperion runs without critical issues)
[x] CI - Azure (Compile Hyperion on Azure)
[x] CI - GH Actions (Compile Hyperion on Github Actions)
[x] Packaging (.zip) (All in one .zip to provide artifacts)
[x] Packaging NSIS (.exe). Installer for Hyperion
[ ] Testing - Test everything
[x] Release

Related PRs
738
Known issues
As of 2.0.0-alpha.7

~Logger won't print all to terminal~
~hyperiond -d crashes~
~qt.network.ssl QSslSocket: Can't resolve function d2i_x509. Requires bundled openssl lib(s)~
~You need to install latest vcredist x64; Should be executed with the setup install.~
No fast DirectX capture, just slow QtGrabber for screen capturing (but works basically)
Crash after OS standby (in this case Windows 7 "energy saving")

Tests

[x] SSDP Server works
[x] Serialport fixed with #837
[x] SSL Server works #864

How may I understand that? Under WSL (Windows Subsystem for Linux) or complete Windows binary

Should the end result be a windows executable?

Yes

Should this represent a new project or should it be integrated into the current one?

I assume we compile the current code on Windows.

I assume we compile the current code on Windows.
But that would inflate the base a lot?
To get started:
What speaks against CMake?

I already explored the topic.

Which build system do you use?

cmake with msvc 2017

Are you going to publish the whole thing?

Hi!
I tested the Windows version. Unable to start, initialize my usb grabber. The device is detected in the system as a webcam.
In Linux it is detected as follows. And it works fine.
/dev/snd/controlC2 - MACROSILICON_USB_Video
/dev/video1 - MACROSILICON_USB_Video
/dev/video0 - MACROSILICON_USB_Video


Replacing Reserved words in data-index.sql

To get merged AFTER https://github.com/apache/incubator-kie-kogito-apps/pull/2139 get merged

Thank you @josedee for the reminder


redshift coming soon

big diff is because of prettier, all I did was wrap the view in a div that overlays the controls

Awesome, tested it and it looks good.


Create hppc.txt

Hurstpierpoint College
http://www.hppc.co.uk

@henrydennis Pull request merged. Please notice that free educational license for JetBrains tools is for high and higher schools students, i.e. those who are over 13 of age. Thank you!


Login feature

Hi @nishintn @ShahinaKP,
Please review the code for the implementation of login functionality with validation and navigation to dashboard on successful login.

Hi @annluke
Merging the code for login feature.


Setting plan to Elastic Band Failed

Now I am able to launch the rsband_local_planner.
But when I give a navigation goal by Rviz ， I come across errors saying：

Setting plan to Elastic Band failed
Failed to pass global plan to the controller , aborting.

Will you point out a direction how to fix this?
Thank you very much!
Originally posted by @JWSLAM in https://github.com/gkouros/rsband_local_planner/issues/11#issuecomment-374085287

Solved by 5f50400edc9e42097a12c41af1ec68a60e5d9b7f


Add callout to also add the Require Rule

You also need to enforce the specific Login Method via the policy. Not sure if we also want to add a picture of this. I can provide is we deem necessary.
cc @ranbel

Does a user need to enable API permissions for this Azure AD enterprise application?

Yea, they need to do that but I think we call that out in the IdP setup section for AzureAD?

Great! I'm just checking for any differences between these instructions and the IdP instructions before linking out.
Regarding "Select the corresponding IdP that was just configured for this specific Conditional Access Policy." -- is it supposed to be one IdP for each application that's protected by Conditional Access policies? (rather than one IdP per Conditional Access policy)

Yea, that's correct. you have to create one IdP instance for each application you would like to have a different conditional access criteria.

added to #7209


Suggestion

grab a specific function from a module
from random import choice
select at option
word = choice(["bat", "door", "freedom", "star"])
the underscores (not found yet)
dashes = "_" * len(word)
game loop
while True:
selection = input("Choose a letter: ")[0].lower() # only the first letter in lowercase
the indexes of the places where 'selection' appears in 'word'
matches = []
idx = 0
for letter in word:
if letter == selection:
matches.append(idx)
idx += 1
for i in matches:
# replace the the right '_' with 'selection' if there are any matches
temp = " ".join(dashes).split(" ") # fixed
temp[i] = selection
dashes = ''.join(temp)

print(" ".join(dashes))
when all letters have been found
if dashes == word:
break

I like this, this is another angle to analyse this problem. It is a shorter way to solve it. Thanks


Rerunning rancher/agent command on a host caused a "ghost" host to appear in UI

Reported by dsantos_ in IRC.

rebuilt rancher-server with new IP but same database
hosts went to reconnecting
reran docker run rancher/agent command on host
host went to active in UI but another host appeared that is stuck in "almost there waiting for agent connection" phase

http://i.imgur.com/C3Oz0wr.png

Reproduced. Just had to run docker run rancher/agent... on an otherwise healthy host.

Cause: when provisioning a machine through go-machine-service (ie through the UI), we set the following environment variable on the rancher-agent container: CATTLE_PHYSICAL_HOST_UUID=<machine.ExternalId>.
When you manually rerun the docker run rancher/agent ... command manually, this environment variable is lost and rancher/agent doesn't report itself correctly.

Fix and work around is to add -e CATTLE_PHYSICAL_HOST_UUID=<machine.ExternalId> to the docker run command. You would have to get the external ID of the machine from the API.

I'm running into a similar issue when stopping and starting a docker-machine instance that runs the rancher/server and a rancher/agent. I'll test if this workaround fixes that.

Tested, definitely not the same issue.

What type of machine @Beanow? boot2docker based machines (virtualbox, vmware, some others) have an in-memory file system (outside of /var/lib/docker) and so restarting such a vm wipes out the state in the /var/lib/rancher directory that we require to recognize hosts upon restart.

#4900 fixes the boot2docker persistence issue

A legit bug, but doesn't seem critical to 1.2.0. Can we move out?

This should be addressed and no longer happening as of v1.2.0. If it occurs, please reopen.


Upgrade redux-saga to 0.12

https://github.com/yelouafi/redux-saga/releases/tag/v0.12.0

After upgrade redux-saga to 0.12, file size change from:
-rw-r--r--   1 chencheng  staff   296K Oct  8 17:56 dva-min.js
-rw-r--r--   1 chencheng  staff   439K Oct  8 17:56 dva.js

to:
-rw-r--r--   1 chencheng  staff   299K Oct  8 17:55 dva-min.js
-rw-r--r--   1 chencheng  staff   443K Oct  8 17:55 dva.js


WIP: Add constructor architecture

Beginning work on the construct method of AgentType. As of this moment, a basic prototype works for income distributions. Still a lot more to develop and test.

[ ] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[ ] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

As noted in the issue where I proposed this, there are two significant design points to work out:

How to handle situations where it's natural to construct multiple objects in the same function. The most obvious case is the first example I implemented: IncShkDstn, PermShkDstn, and TranShkDstn. I put in an ugly workaround, but something else should be done before this is merged.

Whether and how we want constructors to indicate whether they've made a time-varying or time-invariant object. I'm beginning to come around to the idea that these concepts (to the extent they're in HARK now and will be until several more overhaul steps are completed) should not be intended to be changed at the instance level. Rather, they're a fixed fact of the AgentType. There's a very specific reason they were defined at the instance level, but that reason no longer exists.

Can someone tell me how to run the pre-commit routine locally? I can run black, but I don't know how to make lint work.

$ pre-commit run --all-files

more info in the developer guide here: https://docs.econ-ark.org/guides/contributing.html#pre-commit-hooks

$ pre-commit run --all-files

you probably don't want to run all files, only your changes

A bunch of tests needed syntax changes, but everything passes now. The other things that I'm moving to constructors will be much easier from here on out, and should be completed on Monday.
For this PR, I'm going to set the goal at getting everything in constructor functions that should be (plus some alternates that can be swapped in). After this is merged, in a separate branch/PR, I'm going to tackle undoing/simplifying the custom init methods for each AgentType subclass, and get them all just using AgentType.init ... That also means that the default dictionary should be a class attribute.

We should be doing super().init everywhere

Even where super().__init__ is used, there can be problems when update() is called by a parent class as part of init before the child class is "ready" for it, leading to unexpected results. In the not-too-distant future, I hope all our AgentType subclasses inherit directly from AgentType, and don't have any special init stuff.

If tests pass on this commit, the next commit will change exactly one line of code, and revise tests to fit. As is, the "construct pLvlGrid by simulation" method draws on AgentCount, which is fine as long as AgentCount is actually large. The number of eventual simulated agents shouldn't be used in the solution of the problem, even extremely indirectly. So I'm just going to have it use a large number of agents no matter what. This is also why the hardcoded seeds don't matter in this function/method.

The remaining tasks on this PR are pretty small, almost optional:

make some example MrkvArray constructors
make terminal solution constructors for types that are missing it
consider moving one method in ConsAggShockType to a constructor (but maybe not)
make a method for AgentType that reports what the user needs to make a valid type
a few "alternate example" constructor functions, to demonstrate swappability

Also: Is this ready for use?  Ryan Edwards is working on making a new Aiyagari REMARK using HARK and I want him to use the latest and greatest tools ...

This also allows for transitory shocks, I assume?
Someday I want a tool that can handle the case that I think captures almost all of what can be said about annual income processes: It's hard to do better than a process with a fully permanent shock and a transitory shock with an MA(1) term (as time aggregation would imply). I take it that your setup here allows for purely transitory terms but probably not for MA(1) transitory terms?

This PR is just a restructuring. I've made very small changes to the constructor methods when converting them to functions, but they're almost trivial adjustments. This isn't about economics or new capabilities at all. So all the same income processes are in there that we had before.
It might be possible to do an MA(1) transitory process using MarkovConsumerType, but I'd have to work out whether there's a better way than having N^2 discrete states, where N is the number of transitory shock values.

Meant to say before posting: If it's not, we can add something later that can handle this income process. Let me think about it.

The simplest way to do it in HARK (read: the way that requires the least new code) is to make a subclass of MarkovConsumerType with an adjusted simulation procedure. When solving the model, the Markov state can be "today's transitory shock", which simply gives the forward-looking agent a little bit of information about next period's transitory income (weighted average of today's shock and tomorrow's shock).
When simulating the model, you just need a mechanism that enforces that (the index of) today's transitory shock is the same as (the index of) today's discrete state.

Still behind schedule, as there were more terminal solvers than I remembered. Still to do the following small tasks:

make some example MrkvArray constructors
a few "alternate example" constructor functions, to demonstrate swappability

This all looks very good to me.
The one thing which I think could be improved is the confusing way in which constructed objects are stored, both (!) as an entry in parameters and as a new attribute on the model object. Neither of these seem quite right to me.
What about a self.constructs dictionary?

Ok, I understand that removing object attribute storage entirely is a goal, but something to be implemented in a different PR.
I'll leave further notes on #889

Fixed CHANGELOG


i set the api on config file, i restarted the minecraft but the ai dosen't give any response.

how to spawn the ai?

@m3rat
thank you for downloading !
You might have several reasons to fail. please check followings:

write the collect string of Gemini API key. (ctrl-c and ctrl-v key procedure is very usfull)
after installed , Does the aidiary.toml file exist in the config folder.
In aidiary.toml, please enclose the Gemin API key in double quotations.

If you got Gemini API key as ABCDEFG_1234567890, aidiary.toml should be:

@m3rat
update:
I found some mistakes in implementaion, and I fixed them.
please try new version v1.1.0 !
https://modrinth.com/mod/ai-diary/version/1.1.0
thanks for your report, I can notice that !


Lazy reading API

var pbfPart = pbf.readLazy();
...
pbfPart.readVarint();

Easy enough to do without a convenience method.


Extension to scenes without masks like DyNeRF dataset

Hi! Thank you for your great work.
Could tensor4d be extended to handle scenes without masks?  The multi-view captures in these scenes may contain foreground moving objects/human and static backgrounds.

Hi! I tried to use RobostVideoMatting to segment moving objects in my multi-view videos, but I could not obtain high-quality reconstruction even with the mask in my scenario. My results are shown below:

Any idea about this situation?  Should I try to move the mask object into the center of aabb? Any idea on how to determine the location of masked object in 3D space only with multi-view silhouettes?

Hi! I tried to use RobostVideoMatting to segment moving objects in my multi-view videos, but I could not obtain high-quality reconstruction even with the mask in my scenario. My results are shown below:

Any idea about this situation?  Should I try to move the mask object to the origin of the scene? Any idea on how to determine the location of the masked objects in 3D space only with multi-view silhouettes?

In other words, do you have any suggestions on how to get the correct scale_mat from multi-view silhouettes and estimated poses?


Inconsistent strategy documentation

the docs say that the strategy will default to Broadcast Channel and downgrade to localStorage in case of necessity.
However the implementation does the opposite. it defaults to localStorage and switches to Broadcast Channel if localStorage isn't available

Thank you! I will handle it in next few days!

Thank you! I will handle it in next few days!


Update layer shell with optional wl_output

Note, this was already added to the protocol in wlr-protocols, I just didn't bother to implement it in wlroots at the time.


Add option to do an apex.debug call automatically when calling logger.log

Clients would like to see their logs logged in APEX at the same time. This is redundant but helps when looking at the session debug in APEX.
Thanks to @dgielis for suggestion

[ ] Optional parameter
[ ] Need to verify how apex.debug changes in each version of APEX
[ ] Test to see the affects of when apex.debug is enabled vs disabled. This should not affect calls to logger.log

@dgielis How do you think I should handle the following situation:

Logger level is set to error (i.e. similar to production situation)
APEX developer logs in and runs their app in APEX debug mode

Should calls to logger.log be logged? If yes then we'd need to factor the following:

How APEX levels map to Logger levels
When checking if a statement should be logged need to check both Logger and APEX levels

Need to check performance impacts of this

apex_debug, by default, does not store a log message if the page is not rendered in debug mode, or if the p_level is greater than the selected apex debug messaging level (there is an optional "force" parameter that can override this but you would not use it).

Is this not a duplicate of https://github.com/OraOpenSource/Logger/issues/158?

like the mapping of Jeff.
Also note that in APEX you can do apex_debug.message('hello %s','Martin');
How would that be translated into logger?

This seems out of scope for Logger and tightly couples Logger to Apex (though I imaging conditional compilation would fix that). I would suggest a better solution would be to wrap logger and apex.debug calls in a custom procedure. I added a Logger call to our custom error logging procedure when we started using it.

@dgielis How do you think I should handle the following situation:
Logger level is set to error (i.e. similar to production situation)
APEX developer logs in and runs their app in APEX debug mode
Should calls to logger.log be logged? If yes then we'd need to factor the following:

Here is how I think, it could work.
If APEX logging is configured (logger_prefs), then two fairly independend logging checks need to be made. One for the primary logging mode, one for the apex logging mode.
The subprogram that is called (log_error vs. log_trace) decides the logging level.
Currently we seem not to have direct access to the apex debug level. So it is not possible to use that inside logger directly. Fortunatly we don't need it.
If logger.log_error is called, then also call apex_debug.error. If logger.log_trace is called, call apex_debug.trace. The call to the apex_debug subprograms must not be dependend on the logger log_level settings. They already do depend on the apex log level settings (internal logic).
Yes there will be a negative performance impact because for the additional subprogram call and the check for the apex log level.
The apex_debug calls of cause are rendered there using conditional compiling. So this negative performance effect would only exist in an apex environment. In most APEX applications this slight performance slowdown would be acceptable. Other environments where performance is extremely critical will not need to configure this apex_debug setting.
However it also means we can not easily configure this on an application level, unless we have a separate logger installation for each (apex) application.


Composing components with different default as props

Describe the bug
I'm having trouble composing a component with a different as property
To Reproduce
Component 1:
import { classed, DerivedComponentType } from "@tw-classed/react";
import { forwardRef } from "react";

const BaseText = classed("span", {
  base: "t-1",
  variants: {
    variant: {
      label1: "l-1"
    }
  }
});

type TextProps = React.ComponentProps<typeof BaseText>;

export const Text = forwardRef<HTMLSpanElement, TextProps>(function Text(
  { children, ...props },
  ref
) {
  return (
    <BaseText {...props} ref={ref}>
      Text component: {children}
    </BaseText>
  );
}) as DerivedComponentType<typeof BaseText, TextProps>;

Component 2:
const BaseLabel = classed(Text, {
  base: "t-2"
});

type LabelProps = React.ComponentProps<typeof BaseLabel>;

export const Label = forwardRef<HTMLLabelElement, LabelProps>(function Label(
  { ...props },
  ref
) {
  return <BaseLabel ref={ref} as="label" variant="label1" {...props} />;
}) as DerivedComponentType<typeof BaseLabel, LabelProps>;

"Component 2" throws a ts error when it has the as property on BaseLabel
Repro: https://codesandbox.io/embed/goofy-fast-jspjve?fontsize=14&hidenavigation=1&theme=dark
Is there a different way for me to do this sort of composition?

Thank you for this issue!
I find it very weird that TS complains because it is producing the intellisense somehow.

If I set the as to an a tag then I get the href property. So this means that Typescript was able to following the as prop all the way down to the Polymorphic overload yet still complains.
I will investigate further

Found the issue! It's the same React issue I faced and the reason why DerivedComponentType was created. React flattens the type of the as prop from keyof JSX.IntrinsicElements | ClassedComponentType to ClassedComponentType. Same with ref prop.
Solution
Omit from props as & ref
const Label = forwardRef<HTMLLabelElement, Omit<LabelProps, "as" | "ref">>(
    function Label({ ...props }, ref) {
      return <BaseLabel ref={ref} as="label" {...props} />;
    }
  ) as DerivedComponentType<typeof BaseLabel, LabelProps>;

I will investigate further to see if I can modify the internals to omit these two by default. Maybe write and export a custom ComponentProps that doesn't flatten the types.

Pushed a new canary release. Contains the new deriveClassed API:
https://tw-classed-canary.vercel.app/docs/derived-components
install @tw-classed/react@canary


disable cache fetch when it is looking for job state.

Welcome comments to this pull request.
One issue was found in our production cluster: Even though one job is actually running, but it shows preparation state in job graph page. It affects our users checking logs.
I did an investigation on this issue, and found that it should be something wrong with the cache implementation, since database has the right job state info.
I just disabled cache fetching logic in getExecutableFlow method. Checked all callers of this method, and they all come from web server logics. Then I believe this should be a safe change.
Verified in our experimental cluster, and no issues were found.

LGTM

Where are these caches being used? If these caches aren't being maintained correctly then shouldn't we remove all logic that depends on them?
I think the preparing step is showing for a reasonable amount of time which means that the caches aren't being updated for long periods of time. Is this the case ?

@suvodeep-pyne
Yes, I agree. I'd suggest putting this log in cluster and see what happened. If it works very well, we may remove all cache logic, since Ray and I didn't figure why we needed a cache for fetching state. It looks redundant.
Your though is right. However, the period of time can not be predicted. Sometimes a couple of minutes; Sometimes, half an hour.


LEGAL - Open source contract with Jimber & TF AG

[ ] use template
[ ] link to repo crystaluniverse
[ ] make sure all experiences for digital twin are on crystal universe account (kristof)
[ ] add that jimber browser can be used for free by TF Foundation

@despiegk moved previously created story to here. all that is legal should be here

See on circles tool: https://circles.threefold.me/project/despiegk-product_web_threefold/issue/74


AssemblyScript can’t be installed

When running npm install for the subgraph directory, I get the following error.
npm WARN tarball tarball data for assemblyscript<EMAIL_ADDRESS>(sha512-+bxj80L18f0GP6bb5GKUTMpkQRl2bRqHcl1iPi86arWYaLxO2oYAvWYj5SWLXd7IXmtlRij6hZb7z/hbzEaZYQ==) seems to be corrupted. Trying again.

I tried using both Node v12 and v15. How did you solve this issue?

do you still have this issue? I cannot reproduce

Managed to properly install the dependencies with yarn.


GCFFlasher_internal causes USB serial device to be unregistered and fails

I connect ConBee stick and see this in kernel log:
[ 4749.160117] usb 1-3: new full-speed USB device number 8 using xhci_hcd
[ 4749.316628] usb 1-3: New USB device found, idVendor=0403, idProduct=6015, bcdDevice=10.00
[ 4749.316637] usb 1-3: New USB device strings: Mfr=1, Product=2, SerialNumber=3
[ 4749.316641] usb 1-3: Product: FT230X Basic UART
[ 4749.316645] usb 1-3: Manufacturer: FTDI
[ 4749.316649] usb 1-3: SerialNumber: DJ00QDS1
[ 4749.346114] usbcore: registered new interface driver usbserial_generic
[ 4749.346138] usbserial: USB Serial support registered for generic
[ 4749.348573] usbcore: registered new interface driver ftdi_sio
[ 4749.348583] usbserial: USB Serial support registered for FTDI USB Serial Device
[ 4749.348632] ftdi_sio 1-3:1.0: FTDI USB Serial Device converter detected
[ 4749.348661] usb 1-3: Detected FT-X
[ 4749.349085] usb 1-3: FTDI USB Serial Device converter now attached to ttyUSB0

I run GCFFlasher_internal -l:
GCFFlasher V3_17 (c) dresden elektronik ingenieurtechnik gmbh
Path             | Vendor | Product | Serial     | Type
-----------------+--------+---------+------------+-------
/dev/ttyUSB0     | 0x0403 | 0x6015  | DJ00QDS1   | ConBee

I try to update firmware by running
GCFFlasher_internal -t 60 -d /dev/ttyUSB0 -f deCONZ_Rpi_0x26400500.bin.GCF -sn DJ00QDS1 -x 3

GCFFlasher_internal outputs this in a loop for 60 seconds then gives up:
10:06:21:928 using firmware file: deCONZ_Rpi_0x26400500.bin.GCF
10:06:21:975 ls dev: /dev/ttyACM0 (0x0658/0x0200) sn:
10:06:21:975 device /dev/ttyUSB0 not found
10:06:21:975 retry, elapsed 0 seconds
10:06:22:070 parse commands: /usr/bin/GCFFlasher_internal.bin -t 60 -d /dev/ttyUSB0 -f deCONZ_Rpi_0x26400500.bin.GCF -sn DJ00QDS1 -x 3

At the same time, I see this in kernel log:
[ 4788.425720] usbserial: USB Serial deregistering driver FTDI USB Serial Device
[ 4788.426059] ftdi_sio ttyUSB0: FTDI USB Serial Device converter now disconnected from ttyUSB0
[ 4788.426081] usbcore: deregistering interface driver ftdi_sio
[ 4788.426110] ftdi_sio 1-3:1.0: device disconnected
[ 4788.451842] usbserial: USB Serial deregistering driver generic
[ 4788.451925] usbcore: deregistering interface driver usbserial_generic

The /dev/ttyUSB0 device is now gone from the file system. It appears that running GCFFlasher_internal immediately causes the /dev/ttyUSB0 device to unregister and then it can't find it.
How do I make this work so I can update firmware on my ConBee stick?

Some solution ideas:

discard the -sn parameter; -d should be enough or use -d with the /dev/serial/by-id path
during the update process disconnect and connect the conbee again
try another GCFFlasher version (e.g. GCFFlasher 4 from this repository)
try the update on another system (I had a similiar error and after installing a new raspian os on my sd card it worked also with GCFFlasher 3.17)

I discarded the -sn parameter (this was something I added later when just -d didn't work) and then tried to unplug and reconnect the conbee during update. That gave me another behavior that seems to go further but it still loops and eventually fails.
21:22:17:461 using firmware file: deCONZ_Rpi_0x26400500.bin.GCF
21:22:17:486 ls dev: /dev/ttyUSB0 (0x0403/0x6015) sn: DJ00QDS1
Reboot device /dev/serial/by-id/usb-FTDI_FT230X_Basic_UART_DJ00QDS1-if00-port0 (ConBee)
21:22:17:486 query bootloader v1 ID after 0 ms
21:22:17:987 close serial after 500
21:22:18:990 query bootloader v1 ID after 1503 ms
21:22:19:491 close serial after 2004
21:22:19:493 query deCONZ firmware version
21:22:19:509 SLIP RX frame length: 9
deCONZ firmware version 261F0500
21:22:19:509 send watchdog reset 2 seconds
slip put 0xC0
slip put 0x0B
slip put 0x19
slip put 0x00
slip put 0x0C
slip put 0x00
slip put 0x05
slip put 0x00
slip put 0x26
slip put 0x02
slip put 0x00
slip put 0x00
slip put 0x00
slip put 0xA3
slip put 0xFF
slip put 0xC0
21:22:19:509 TX c00b19000c0005002602000000a3ffc0
21:22:19:525 set watchdog ttl status: 0x04
retry, failed

After timeout it ends with
2410: Error: uart reset failed, check retry

Update: I tried the v4.0.0-beta release from this repository and it core dumps when I run the -l flag to list devices, but it actually succeeded in flashing new firmware. Thanks!


Add more security-related actions and functionality

[ ] CIDR contains IP
[ ] Is item datatype X? (Use available observable types in TheHive/MISP)

is item datatype like (string, integer, boolean, float)? Or like with regex to check if it is an mail, ip, domain, url, etc...?

is item datatype like (string, integer, boolean, float)? Or like with regex to check if it is an mail, ip, domain, url, etc...?

Good idea. Datatype matches with autocasting should be able to handle the first part here easily


rspack_core: ustr may not be the ideal choice for module identifier

System Info
None
Details
While upgrading to ustr 1.0 https://docs.rs/ustr/latest/ustr/, it's ahash upgrade yielded inconsistent hash results.
From ahash's documentation https://github.com/tkaitchuck/ahash:

AHash does not have a fixed standard for its output. This allows it to improve over time.

Because it does not have a fixed standard, different computers or computers on different versions of the code will observe different hash values. As such, aHash is not recommended for use other than in-memory maps.

Unfortunately we are currently using it as part of our hash:
https://github.com/web-infra-dev/rspack/blob/88e5557dee8cc2acce3be53cbf054065ab81610d/crates/rspack_core/src/compiler/compilation.rs#L1996

I don't have a solution yet.
Reproduce link
None
Reproduce Steps
None

Using the str  may fix your issue, module.identifier().as_str().hash(xxx)

Using the internal str may fix your issue, module.identifier().as_str().hash(xxx)

This is a footgun, how would anyone remember to use this form?

This, for sure, is not the ultimate solution for us if portable caching is the ultimate goal of us. But it should be ok for now as we haven't got the plan in the near future to support this. Nevertheless, it still cause unstable hash in snapshots we might have to fix this.

Using the internal str may fix your issue, module.identifier().as_str().hash(xxx)

This might be a temporary solution as it's not performant ;-( (precomputed-hash is gone)

@IWANABETHATGUY @h-a-n-a How do you feel about https://github.com/web-infra-dev/rspack/pull/5504, it's only a 2 line change in the ustr crate https://github.com/anderslanglands/ustr/compare/master...Boshen:ustr-fxhash:master

@IWANABETHATGUY @h-a-n-a How do you feel about #5504, it's only a 2 line change in the ustr crate anderslanglands/ustr@master...Boshen:ustr-fxhash:master

the consistent hash function is the best way to solve this issue.

consistent hash function

Can you elaborate? Is my change acceptable or not?

I mean using  rustc hash function should make the hash result of Ustr consistent, yeah, your solution is acceptable.

@IWANABETHATGUY @h-a-n-a How do you feel about #5504, it's only a 2 line change in the ustr crate anderslanglands/ustr@master...Boshen:ustr-fxhash:master

Looks good to me ;-)


GridModel.ensureRecordsVisibleAsync fails to find collapsed row nodes in tree grid

GridModel.ensureRecordsVisibleAsync can fail to find collapsed row nodes in a tree grid.
This can be fixed by changing the await wait() between steps 1) Expand any nodes that are collapsed and 2) Scroll to all nodes to await wait(100), but this is of course not an elegant solution.

To confirm - was this resolved by #3212 ?

To confirm - was this resolved by #3212 ?

That's correct


Remove the deprecation methods or suppress warnings

I'm using SnowplowTracker version 2.2.2 installed via Cocoapods, the pod has added 69 warnings of deprecated methods being used internally by the framework.
Can you please or replace the deprecated methods or suppress the warnings?

Thanks @rsortiz for the heads up.
The deprecated methods are due to the needing to release a v2 with a new API without to remove completely the ability to use the v1 API. We deprecated the v1 API to allow a smoother migration.
I'm closing this ticket as this same issue is discussed in #642.


Regexp modifier "r" needs perl 5.14; OpenSSL should build with 5.11, …

…so do not use the "r" shortcut.

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

This pull request is ready to merge

Merged, thanks for the fix.


General Improvements

@deansville You can type something here or attach documents, print screens... anything that you want to someone take a look at.

The following points need reviewing/changing

Website Title = SharePay House Bills --> This is currently Hourse Bills

Website subtitle = Group house bills causing a problem? Not anymore! Sign up and lets break it down! --> Share how much each house mates will pay in June 2021! Sign up and lets break it down!

Text --> Thank you for joining us --> Thank You for Join Us

Image background/foreground --> This would benefit from some colour contrast or the text being a darker grey colour with text stating House Bills

I am sure you are aware of this error, but listed anyways.

ALL DONE.
@deansville


Importing nested css is not transpired

Hello, I am new to PostCSS. I am getting a strange behavior with this module which I expect it to work. I have a simple project that looks like:

app.js
app.css
header.css

app.js
import "app.css"

app.css
@import "~normalize.css"
@import "./header.css"

header.css
header {
  & h1 { ... }
}

The issue is that header.css is never transpiled. I was using webpack2 at first but then I gave up and started using webpack1 and I still see the same problem.
Including header.css from app.js does work.
webpack.config.js
const webpack = require("webpack");
const path = require("path");
const ExtractTextPlugin = require('extract-text-webpack-plugin');

module.exports = {
    context: __dirname + "/themes/amirraminfar",
    entry: {
        app: "./static/js/app.js"
    },
    output: {
        path: __dirname + "/themes/amirraminfar/static/js/",
        filename: "[name].bundle.js",
    },
    resolve: {
        alias: {
            jquery: "jquery/src/jquery"
        }
    },
    module: {
        loaders: [{
            test: /\.js$/,
            exclude: /node_modules/,
            loader: 'babel-loader'
        }, {
            test: /\.css$/,
            loader: ExtractTextPlugin.extract('style-loader', 'css-loader', 'postcss-loader')
        }]
    },
    postcss: function(webpack) {
        return [
            require('postcss-import')({
                addDependencyTo: webpack
            }),
            require("postcss-cssnext")(),
            require('lost'),
            require('postcss-font-magician')()
        ]
    },
    plugins: [
        new ExtractTextPlugin("../css/[name].bundle.css"),
        new webpack.ProvidePlugin({
            $: "jquery",
            jQuery: "jquery",
            'window.jQuery': 'jquery'
        }),
    ]
};

What am I doing wrong?

What am I doing wrong?

loader: ExtractTextPlugin.extract('style-loader', 'css-loader', 'postcss-loader')

should be
loader: ExtractTextPlugin.extract('style-loader', [ 'css-loader', 'postcss-loader' ])
// or
loader: ExtractTextPlugin.extract('style-loader', 'css-loader!postcss-loader')

Hey thanks for the response. But that didn't work. I simplified my example and put it on https://github.com/amir20/example-bug-326.
The broken output is here, and I have modified webpack.config.js as you suggested. I also tried css-loader!postcss-loader. Same result :(
Could this be a bug @MoOx? I can't reopen.

No this is definitely not a cssnext bug. And lot's of people and config are doing a classic postcss + postcss-import + postcss-cssnext without any problem. It's more a configuration issue ;)

ExtractTextPlugin.extract() take 2 args in wepback 1, an object in webpack 2 version. Just check that something in the root is transpiled.

ExtractTextPlugin.extract() take 2 args in wepback 1, an object in webpack 2 version. Just check that something in the root is transpiled.

I am have done loader: ExtractTextPlugin.extract('style-loader', 'css-loader!postcss-loader') and still no luck. Do you an example handy that ExtractTextPlugin + PostCSS + CSSNext work? I am starting to give up. I don't know what else to try.

I am going to try without ExtractTextPlugin see if that works.

Also try with more simple features without import.

Good news, it works without ExtractTextPlugin.

Problem solved. I was missing 'css-loader?importLoaders=1'

You are not supposed to use this option when you use postcss-import.

You are not supposed to use this option when you use postcss-import.

Is that documented somewhere?
Honestly, I was trying to understand what is all the fuss about PostCSS when compared to other frameworks. I find PostCSS really powerful, but having to figure out why something is not working and then digging to the six plugins I am using feel a step back in direction of web development. Then again that's just my opinion.

You are mixing webpack and postcss. Just try vanilla postcss, then integrate it to webpack.
The fact that you didn't used correctly ExtractTextPlugin show you didn't read the doc enough, and that's the first thing to do when you want to use something.

In the documentation it says

the loader(s) that should be used for converting the resource to a css exporting module

I read the documentation. I also followed it correctly.
I think you are missing what I am saying. I am not saying this is PostCSS or even this module's problem. I am saying back in 2010, I would include less, stylus, sass, etc... and with minimal effort you would get things to work. Now, the options are endless and we can add so many things. But it comes at a cost where not all modules are compatible with each. I was trying to evaluate this at work. It feels like the JavaScript community is moving away from opinionated frameworks to having everything configured. I have probably spend 6 hours on this. It's probably my fault because I didn't know what to google. But at some point, I thought, this can't be this hard. I didn't find a single example of extract + webpack + postcss + postcss-next. As a software engineer with 10 years of experience,  that is frustrating.
Anyway, thanks for the help. Maybe I am just alone in this thought. I have read so many documents.

I totally agree with you. I barely use postcss this days since I use javascript solution to write my styles.
But in comparison to postcss, babel does offer an easy way to setup. Postcss should have a website with a page to show classic example (like webpack + postcss). They started to make a website, but there is only a homepage, nothing more... poke @ai.

Thanks @MoOx. I did a little investigation. I wanted to know where I got the syntax and why I missed this. I googled "ExtractTextPlugin postcss with webpack" which led me to this page. It didn't work, I tried doing webpack2 search which led me to this page. That didn't work.
I see what you were saying now that in PostCss README example says to use importLoaders or postcss-modules.
So to wrap things up. Lesson learned here is that I should have first tried to gotten it to work without ExtractTextPlugin then move incrementally. I do this usually with new technology.
All this webpack/js/postcss is moving so fast that README files become outdated within weeks, especially with webpack2 coming out.
Thanks for being patient. :) Looking forward to see an actual homepage with how to use it with babel.


[ESP32] cmake option to set software version and software version string

ESP-IDF lets user set software version string by two ways: Project's
CMakeLists.txt file or configuration option. It depends on
CONFIG_APP_PROJECT_VER_FROM_CONFIG option so making the same provision
for software version number.
Also, Reading the software version string from the app description
which is set using any one of above method.
Setting the default software version as per spec i.e. 0.
Setting PROJECT_VER and PROJECT_VER_NUMBER in all the examples
Problem

Not using already present cmake option for setting/getting software version string
Also, default software version is set to incorrect value, 1, should be 0.

Change overview
Please see above
Testing

Set software version and software version string using cmake options and config option then made sure they match when read using basic cluster.
Also verified that it error out when none of the one is set

1.0 accepted: platform-specific change, platform owners are gatekeepers here.


Error when requesting money: Class NumberFormatter not found

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Log into staging NewDot.
Request money from any user.

Expected Result:
Money should be requested with no errors.
Actual Result:
An error occurs when requesting mmoney.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: v1.2.41-4
Reproducible in staging?: Y
Reproducible in production?: N
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation:
View all open jobs on GitHub

@youssef-lr I can't reproduce this, I can send requests just fine on staging

It's odd how this issue got created, I remember only creating one here which I have closed since we handled it in another issue. Closing this one too. Sorry @pecanoro!


A卡（RX 6600XT）是只能在linux中使用吗？

Windows10 + RX 6600XT 没有办法用上显卡吗，还是需要指定pytorch版本？

使用DML版本 不过效率不太行
" For AMD/Intel GPU users:
https://huggingface.co/lj1995/VoiceConversionWebUI/resolve/main/RVC1006AMD_Intel.7z "

使用DML版本 不过效率不太行 " For AMD/Intel GPU users: https://huggingface.co/lj1995/VoiceConversionWebUI/resolve/main/RVC1006AMD_Intel.7z "

Could you explain this better, because I downloaded both this package and the project (github (I also checked out the project commit)) and I didn't notice any difference(in my case, I use arch linux), it has the same errors (I have a 7900xtx). Does this zip package have anything different about the project?

@LeonardoSidney
What errors are you getting? I also have a 7900 xtx and I use artix linux and training works. The only problem is that inference with rmvpe takes too long, it took me 71 seconds to convert a 4 minute song. I'm using the nightly february 1 rocm builds of pytorch.

@LeonardoSidney What errors are you getting? I also have a 7900 xtx and I use artix linux and training works. The only problem is that inference with rmvpe takes too long, it took me 71 seconds to convert a 4 minute song. I'm using the nightly february 1 rocm builds of pytorch.

@GUUser91
I can try what you said with the nightly compilation, currently I get a rather sinister error:
https://gitlab.freedesktop.org/drm/amd/-/issues/3056#note_2246903
I'll get back to you soon about the nightly compilation.

使用DML版本 不过效率不太行 " For AMD/Intel GPU users: https://huggingface.co/lj1995/VoiceConversionWebUI/resolve/main/RVC1006AMD_Intel.7z "

这个效率和cpu相比如何，不会不如用cpu吧

.

使用DML版本 不过效率不太行 " For AMD/Intel GPU users: https://huggingface.co/lj1995/VoiceConversionWebUI/resolve/main/RVC1006AMD_Intel.7z "

这个包里面怎么只看到了realtime的gui，没有看到能用来训练的gui

使用DML版本 不过效率不太行 " For AMD/Intel GPU users: https://huggingface.co/lj1995/VoiceConversionWebUI/resolve/main/RVC1006AMD_Intel.7z "

https://pan.baidu.com/s/1mEs9Jmi2tBot4AgH6ZWp-w?from=init&pwd=eqea 这里找到个有go-web-dml.bat的整合包


Search is not working correctly in prod

To reproduce - go to https://prod.data.humancellatlas.org/explore/projects
Start typing Tabula Muris in the Search box
choose Tabula Muris from the suggested list
The table with the metadata doesn't change listing all the datasets even though there is only one project selected.
Expected behaviour:
Table only shows metadata for one dataset which is Tabula Muris
Screenshot:

Actually it's working as designed..
When a project is selected a checkbox is filled in next to the project line but all the other projects remain selectable. Otherwise the Project view would always reduce to a single project.
Adding @adriennes and @gabsie to comment from the UX point of view and @NoopDog for the UI design.

Hi @zperova thanks for raising the issue.
Its on our roadmap to:

Improve the project search
Improve how we indicate which projects are selected
Better integrate project search vs project browsing vs viewing selected projects.

The current implementation is working as designed. Originally we only showed the selected projects in the projects tab, but as @theathorn mentioned that makes it impossible to select more than one project in the table (you can still select additional projects in the search but that is difficult for people to discover).
We will be doing some mocks and user testing for this in the coming weeks. Feel free to follow along chime in on the UX slack channel.
Cheers,
Dave

I think this is a good catch.  The expected behavior when selecting the project using the checkbox from the list may be different than when selecting a project from the facet or search.
Agree with @NoopDog that this is a good one for some light user testing - could we add this to the list for next week?

@theathorn Thanks for the explanation, I see your point. I imagined the search would work the same as when one searches for an organ - it shows the datasets that have this organ and does not select any of these. But that doesn't allow selection of multiple datasets.
@NoopDog @adriennes I wonder whether an enhancement to the current design of bringing the dataset that is searched for to the top of the list is easy to implement? I will keep an eye on the UX channel for the results of the mocks and user testing.

Yes @zperova that is the direction we are thinking of. Will let you know when we have some mocs to review.

created #665 to track adding the selected projects to the top of the projects list.


Update latest release to v0.7.0

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Which issue(s) this PR fixes:

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

NONE

/assign @tenzen-y

/hold
for release :)

/retest

/hold cancel


NanoPi R2S - CPU Speed Setting

I am wanting to know if this build includes the ability to set the clock speed of the CPU on the R2S. I only get around 500 megabit on this one, but I tried another nanopi openwrt build where I could set CPU speed to its max (overclock) and I was able to get around 910 megabit (I have proper cooling).
Thanks!

The overclocking patches are not included in my builds

Are you able to add these over-clocking patches? This would make it perfect for my use case.


[POC] New merchant onboarding

The new onboarding merchant API will return a payload like this:
{
  "is_enabled": true,
  "config_mode": "<the checkout mode>",
  "merchant_id": "<the merchant ID>",
  "bolt_credentials": {
     "api_key" : "<the API key>",
     "signing_secret" : "<the signing secret>",
     "publishable_key" : "<the publishable key>",
  }
}

We should store this information in the database. In order to do that we should:

[ ] Add a new model(Singleton?) that should store the above fields
[ ] Create a new menu item for Bolt.
[ ] Visiting that page the page will show a button to require to create and link the Bolt account. If the Bolt account is already linked it should show just bolt account information.
[ ] Create one service that should call the Bolt API, and receiving back the credentials should store them on the Specific model

This issue is not more valid, it has been replaced by https://github.com/nebulab/solidus_bolt/issues/27


Allow auro-select to be reset

Is your feature request related to a problem? Please describe.
I need the ability to programmatically reset an auro-menu (really, an auro-select) so that the placeholder text shows back up and the value is cleared
Describe the solution you'd like
I'll let @jason-capsule42 figure this one out 😉
Describe alternatives you've considered

Additional context
This is my auro-select:

After selecting an item I have no way of resetting it back to its initial state:

Merely finding the element and setting its value to undefined doesn't help.

I’ve not been able to find a workaround for this, unfortunately. But I wouldn’t say I’m blocked by this yet as my work isn’t going out to prod at this sprint. Hopefully that helps prioritize, let me know if you need more info. I can keep you in the loop in terms of when we’re planning on releasing the work that depends on this.

Hey @blackfalcon, had sprint planning today and we're hoping to release this in the next sprint or two, not sure how that works out with your priorities. Let me know if you need anything from me.

@DZwell the focus for the next few sprints will be around dropdown-related elements. Please follow up with @jason-capsule42 for any specific details as he will most likely be doing the work.


Disjunct constraints cannot contain binary or integer variables.

I am surprised that I run into this error when I define my times as days + timeofday where days are Integers and timeofday is Float.
I want to use DisjunctiveProgramming.jl to define XOR constraints on time intervals.
Is there a theoretical limitation why such composed expressions with Integer/Binary component are not compatible with DisjunctiveProgramming.jl?

Hi @schlichtanders, can you please provide a short code example of what you would like to model exactly?
Most GDP reformulation techniques theoretically require that the constraints inside of disjunctions only contain continuous variables to be exact. This is why we enforce that condition. With a concrete example, we can try and see if there is a workaround.

Thank you for your help.
Here an example definition which throws the error Disjunct constraints cannot contain binary or integer variables.
using Jump
using DisjunctiveProgramming
n = 2
seconds_per_day = 24*60*60

model = GDPModel(HiGHS.Optimizer)
@variable(model, 0 <= start[1:n])  # this could also be split similar to durations

# unixepochs duration definition
@variable(model, 0 <= duration_days[1:n], Int)
@variable(model, 0 <= duration_secondsofday[1:n] <= seconds_per_day)
@expression(model, duration, seconds_per_day .* duration_days .+ duration_secondsofday)
# @variable(model, 0 <= duration)

# unixepochs end duration
@expression(model, finish, start .+ duration)

# disjunction
@variable(model, Y[1:2], Logical)
@constraint(model, finish[1] <= start[2], Disjunct(Y[1]))
@constraint(model, finish[2] <= start[1], Disjunct(Y[2]))
@disjunction(model, Y)

Why not define a variable finish that is continuous and constrained to be greater than the start variable of its same index? Can you elaborate why you are defining the number of days and adding a float to it instead of just having the start, finish, and duration variables be continuous and nonnegative?

Sure, it is about saving some constraints.
I need to know the number of days of the duration for some other constraints of mine. I could extract these from the difference between finish and start by some extra variables and constraints (using something like modulo).
Alternatively, I can put them into the definition like done, and avoid the extra constraints, which hopefully simplifys the model overall. But this approach fails when using Disjuncts

In general, if you can avoid creating discontinuous variables for an optimization model, this will significantly improve solution performance. I don't think you will want to use the modulo function over an optimization variable, though. I may be wrong, but it sounds like your optimization model could be formulated better, which can reduce the combinatorics. If you can provide the simplest form of your model, we can discuss if it is necessary to support mixed-integer constraints inside disjuncts. However, the theory here for reformulating a disjunctive program to mip would need to be reviewed, especially in the case of the Hull reformulation. Also, Indicator reformulation would fail, but you could still do BigM.

As a note to us devs @pulsipher @hdavid16: https://johnhooker.tepper.cmu.edu/milpmodeling2.pdf has some theory on mixed-integer disjunct constraints in Chapter 3, but it is not clear if this extends to nonlinear systems.

Based on this theory, we could potentially relax the requirement that all variables are continuous and provide an opt-out keyword that users would have to enable to disable at their own risk if the model is nonlinear.

Yes. However, I think there are many cases where a mixed integer constraint can be avoided in a disjunct, especially if it is being added to a continuous variable as in days + seconds.

I am now falling back to the following modulo MIP rewrite from JuMP Tips&Tricks
https://jump.dev/JuMP.jl/stable/tutorials/linear/tips_and_tricks/#Modulo

@schlichtanders feel free to share the simplest version of your model and we can discuss if this is necessary.

Based on this theory, we could potentially relax the requirement that all variables are continuous and provide an opt-out keyword that users would have to enable to disable the check at their own risk if the model is nonlinear.

@pulsipher if we decide to relax this, we will need to review the code for the BigM and Hull reformulations as any binaries (from nested disjuncts) are not disaggregated and binary bounds are also ignored when calculating tight M values (also for nested case)

@schlichtanders feel free to share the simplest version of your model and we can discuss if this is necessary.

There is actually not so much more to it.
Concretely, I have multiple time intervals which I want to make non-overlapping. I am using Disjuncts for this to model both possibilities that one interval is before or after the other.
In addition I would like to restrict the time-of-the-day of a respective interval, e.g. to start after 09:00 and end before 18:00 of an arbitrary day.
The modelling is a bit different than above, but also comes down to having an integer variable for the days.

@schlichtanders based on your description, I would greatly discourage using integer variables as they are not necessary. Using proper variable bounds and constraint definitions should allow you to model this as a linear GDP which will be more performant than if you start introducing integer variables.

@hdavid16 can you give an example how to distinguish the day and the time-of-the-day without using Integer variables?

@hdavid16 can you give an example how to distinguish the day and the time-of-the-day without using Integer variables?

I haven't tested this code, but here's a quick example I whipped up:
using DisjunctiveProgramming

m=GDPModel()
@variable(m,0<=start[1:2])
@variable(m,0<=finish[1:2])
@variable(m,0<=duration[1:2])
@variable(m,precedence[1:2],Logical) #precedence relation
@variable(m,start_day[1:10,1:2],Logical) #day task 1 or 2 starts (between 1 and 10)

#disjunction to establish precedence between task 1 or task 2
@constraint(m,finish[1] <= start[2],Disjunct(precedence[1])) #task 1 goes first
@constraint(m,finish[2] <= start[1],Disjunct(precedence[2])) #task 2 goes first
@disjunction(m,precedence)

#disjunction to select what day each task starts
@constraint(m,[i in 1:10, j in 1:2], (i + 9/24) <= start[j] <= (i + 18/24), Disjunct(start_day[i,j])) #task j must be between 9:00 and 18:00 on the day it starts
@disjunction(m,[j in 1:2], start_day[:,j])

It may often be beneficial to have more variables (https://or.stackexchange.com/questions/3209/how-to-choose-between-high-number-of-binary-variables-or-fewer-number-of-integer). However, we will look into supporting this feature.


Handle non conventional controller names

Until the PR gets accepted on the package, this PR applies the fixes required by Passbolt.

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


UI

Can you please share the map based tracking UI even if it needs to be reworked?

I am planning to upload a sample map based UI in the near future.

Thank you for the feedback.


[3.x] Fix GradientTexture2D not working with Light2D

Light2D expects valid texture resource to be available at construction for it to be properly bound.
Changed texture allocation implementation back to be the same as in GradientTexture (1D) to avoid this issue as well.
This only affects 3.x implementation.
Follow up to #54824.
The goostengine/goost#170 depends on this PR to be merged.

The destructor for GradientTexture is slightly different from the GradientTexture2D one (doesn't check texture.is_valid() before freeing). Should it be the same here?
GradientTexture::~GradientTexture() {
        VS::get_singleton()->free(texture);
}

It does not make a difference in behavior, but I've added the safe check in GradientTexture (1D) for consistency now.

I meant it might be the other way around - maybe texture cannot be invalid anymore.

Do you want those checks to be removed? I can remove those checks if you want, but I propose to retain those checks for both implementations.

I don't know, I just noticed the inconsistency but didn't have time yet to check in detail where it comes from and which is most correct.

Alright, once you review the PR, feel free to amend it as you see fits best.

I have removed the safe check now, which actually makes the freeing behavior consistent across other implementations for various textures.
The potential inconsistency can be fixed via a dedicated PR (in the master branch), not here, because this affects other texture implementations, which is obviously out of scope for this PR.
I'd like to remove GradientTexture2D in Goost (see goostengine/goost#170) which I and other Goost users have been using for the past year successfully, so that I can maintain this feature in Godot now.
Thanks for understanding.

Thanks!


"Răsurile" is listed in the drop-down menu

Description:
"Răsurile" 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 "Răsurile" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Răsurile, CĂLĂRAȘI (Ileana)".


(REPLATS-514) Do not purge WEAVE-CANARY filrewallchains

The WEAVE-CANARY firewallchains in filter, mangle and nat tables were
not in our list of Kubernetes managed firewallchains not to merge. I
think skipping them was just an oversight, and this patches that.

The kurl_test::configure_firewall plan is failing when attempting to remove the WEAVE-CANARY:filter chain.
Err: /Stage[main]/Main/Firewallchain[WEAVE-CANARY:filter:IPv4]/ensure: change from 'present' to 'absent' failed: Execution of '/usr/sbin/iptables -t filter -X WEAVE-CANARY' returned 1: iptables: No chain/target/match by that name.

I don't know why. The chain exists prior to the plan running.
Full log:
  Notice: /Stage[main]/Firewall::Linux::Redhat/Package[iptables-services]/ensure: created
  Notice: /Stage[main]/Firewall::Linux::Redhat/Exec[/usr/bin/systemctl daemon-reload]: Triggered 'refresh' from 1 event
  Notice: /Stage[main]/Firewall::Linux::Redhat/Service[iptables]/ensure: ensure changed 'stopped' to 'running'
  Notice: /Stage[main]/Firewall::Linux::Redhat/Service[nftables]/ensure: ensure changed 'stopped' to 'running'
  Notice: /Stage[main]/Firewall::Linux::Redhat/Service[ip6tables]/ensure: ensure changed 'stopped' to 'running'
  Notice: /Stage[main]/Main/Firewall[000 accept all icmp]/ensure: created
  Notice: /Stage[main]/Main/Firewall[001 accept all to lo interface]/ensure: created
  Notice: /Stage[main]/Main/Firewall[002 accept related established rules]/ensure: created
  Notice: /Stage[main]/Main/Firewall[003 accept inbound SSH]/ensure: created
  Notice: /Stage[main]/Main/Firewall[100 allow responding on established connections]/ensure: created
  Notice: /Stage[main]/Main/Firewall[100 allow communication on loopback addresses]/ensure: created
  Notice: /Stage[main]/Main/Firewall[100 allow communication in private networks]/ensure: created
  Warning: Firewallchain[PREROUTING:nat:IPv4](provider=iptables_chain): Attempting to destroy internal chain PREROUTING:nat:IPv4
  Notice: /Stage[main]/Main/Firewallchain[PREROUTING:nat:IPv4]/ensure: removed
  Warning: Firewallchain[INPUT:nat:IPv4](provider=iptables_chain): Attempting to destroy internal chain INPUT:nat:IPv4
  Notice: /Stage[main]/Main/Firewallchain[INPUT:nat:IPv4]/ensure: removed
  Warning: Firewallchain[POSTROUTING:nat:IPv4](provider=iptables_chain): Attempting to destroy internal chain POSTROUTING:nat:IPv4
  Notice: /Stage[main]/Main/Firewallchain[POSTROUTING:nat:IPv4]/ensure: removed
  Warning: Firewallchain[OUTPUT:nat:IPv4](provider=iptables_chain): Attempting to destroy internal chain OUTPUT:nat:IPv4
  Notice: /Stage[main]/Main/Firewallchain[OUTPUT:nat:IPv4]/ensure: removed
  Notice: /Stage[main]/Main/Firewallchain[WEAVE-CANARY:nat:IPv4]/ensure: removed
  Err: /Stage[main]/Main/Firewallchain[WEAVE-CANARY:filter:IPv4]/ensure: change from 'present' to 'absent' failed: Execution of '/usr/sbin/iptables -t filter -X WEAVE-CANARY' returned 1: iptables: No chain/target/match by that name.

I assume that pam_firewall's firwallchain list of chains to avoid purging should be updated to include WEAVE-CANARY, and that we never noticed this previously because centos-8.3-kurl-beta would configure the firewall before restarting k8s. The WEAVE-CANARY chains do exist in the centos-8.3-kurl-beta image. I don't know why the first run would fail and the second succeed.
But kurl_test airgap testing with redhat-8.4-kurl-beta vm does pass with this patch. I'll be updating puppetlabs/kurl_test#48 in a moment to run with this branch.

Oh, there was one other peculiarity I noticed when reading the README for pam_firewall. It mentions passing preserve-iptables-config when installing PAM, which is not something we're doing in https://github.com/puppetlabs/puppetlabs-packer/blob/master/scripts/kurl.sh#L42, if that needs to change.  Although it didn't seem to matter before, and we're now applying the firewall resources after pam is installed and running :shrug:

That worked in our tests because we would:

Install kURL. At this point no special iptables config exists.
Shutdown kubelet.
Start a new machine from the image.
Set up firewall rules, which also starts the iptables service.
Reconfigure Kubernetes and start kubelet.

Kubernetes startup is where all the iptables rules get written. And kURL installer isn't run after our custom iptables config, so nothing to preserve during install.


Filtering by scala version

Really great plugin, I will use it in my next project.
It would be cool to filter results by scala version defined in build.sbt or build.properties.
Please mind that you can have crossCompile definitions.

search.maven.org let searching by keywords and also by tags (i.e. scalaVersion), but not by both of them in one query. Sorry.

you can still perform filtering locally looking at artifact id suffix


fix truncating long comments

Fixes https://github.com/opencollective/opencollective/issues/1063

:tada: This PR is included in version 2.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Dev

@howard It would be great if you could check my changes and maybe you still have some more suggestions for me :). Thanks in advance for your help!

@howard
Thanks for your comments! I will try to fix those as soon as I have time.
Regarding your question about the architecture: The users should handle request objects that handle a client under the hood.
Internally the Request::send() (not static) method should return a Response object that is not implemented yet, but should have convenient functions such as Response::getArticles(), Response::getFilters() or Response::getBackendServer(), etc. (all not static of course).
The getFilters method would return an array of Filter and the class might look like this:
// Written in the browser, syntax might be incorrect.

class Filter {
    /** @var string $name */
    public $name;

    /** @var string $display */
    public $display;

    /** @var Select $select */
    public $select;

    /** @var int $selectedItems */
    public $selectedItems;

    /** @var Type $type */
    public $type;

    /** @var Item[] $items */
    public $items;
}

I think you can get it with this.
Please tell me what you think about this idea and if you would go in a different direction. If yes please tell me why.

After a second thought it might be better to pass parameters directly to a client, without a request object. Let me think about this I will comment again in like 3-5 hours.

Yesterday I had no time left, but I thought about it a while. I think its better if we pass parameters directly to a client, without a request object. I will try to implement it soon.

@howard I am refactoring atm and I hope for a quick answer that you might have for me. I am sure you know the issue that you cannot give empty arrays to http_build_query, but FINDOLOGIC requires attrib['Color'][]=Black. Do you know a way to still use the function above instead of writing an own function for the parameter building?

This PR is closed since the work is continued in #5.


Implement sync for non quetz servers

as discussed in PR #113 for non-quetz server that do not implement time_modified attribute we can synchronize versions by comparing the package/filename and the SHA256 number of the local and remote files.
This PR implements this feature for syncing mirror channels.

ready to merge @wolfv


round: confusing definition of rounding

Process ID: round
Describe the issue:
Documentation states:

If the fractional part of x is halfway between two integers, one of which is even and the other odd, then the even number is returned. This behavior follows IEEE Standard 754.

However, based on the definition in the standard, I'm not sure this should hold for integers only. I would expect this would be used at arbitrary point, e.g. round(x=0.25, p=1) => 0.2 and round(p=0.35, p=1) => 0.4.
Proposed solution:
Clarify that the number should be rounded to the one with even least significant digit.

Yes, I agree. I'll issue a PR.

Please see PR #327


Filter Fine Tuning

What do you think @rmevans9?

Looks fantastic! Thanks for cleaning up the code to match the style you have been using, I tried to match it as much as possible but habits from the code bases I work in seeped in it seems haha.

No worries at all.  Can't thank you enough for all the help.


[Bug] new SVG renderer fails to render labels on line charts

Version
5.3
Link to Minimal Reproduction
https://codesandbox.io/s/apache-echarts-5-3-ssr-demo-forked-022g8m?file=/src/index.js
Steps to Reproduce
I used the example chart in 5.3 update and changed it to "line" from "bar", and added label { show: true }
Current Behavior
labels do not show on line chart.  They show on bar chart.
Expected Behavior
labels should display on line chart
Environment
- OS: MacOS
- Browser: Chrome
- Framework:

Any additional comments?
No response

In SSR mode, the animation should be disabled. But it seems the line series still unexpectedly applied the animation. Currently, a workaround is to disable the animation via animation: false.


Ensure jitted functions can be cached

If we can cache them, we can get rid of of some nasty latencies introduced by #706, which is important for a smooth user experience.

As an update, I filed https://github.com/numba/numba/issues/6264
I also checked the potential savings using a patched numba that can cache functions captured in a closure. Measured via the GUI on an 8GiB MIB dataset, LiberTEM-stable (v0.5.0) takes about 3.8s to compute the initial sum of all frames, where LiberTEM-master+patches takes 4.6s. I'll retry on a larger data set in a bit, there the throughput improvements should outweigh the initial additional latency (by a noticeable margin)
Maybe the remaining latency can be hidden by heating up the cache directly after loading a data set, as we discussed before - it could possibly also be reduced in the cache miss case, as it is enough to run one compilation job per host to populate the cache, instead of having multiple redundant compilations (for each core) competing for resources.

In case numba/numba#6284 is not included in the next Numba release, we could perhaps implement a workaround to get the 0.6 release out in reasonable time.
We only need this caching in a well-defined subset of LiberTEM that has a well-defined interface with user code: the I/O routines.  Furthermore, the caching in Numba is implemented in a central place and your patch seems to work. We could implement a context manager that monkey-patches the Numba cache with your patch only when LiberTEM enters I/O routines. That means user code will not be affected by the monkey patching.
Heating up the cache directly after loading sounds like a good idea! It felt more acceptable to wait when opening instead of waiting for the results to come in after adding an analysis or clicking "Apply".

We only need this caching in a well-defined subset of LiberTEM that has a well-defined interface with user code: the I/O routines. Furthermore, the caching in Numba is implemented in a central place and your patch seems to work. We could implement a context manager that monkey-patches the Numba cache with your patch only when LiberTEM enters I/O routines. That means user code will not be affected by the monkey patching.

This monkey-patched hack would need to make sure that the cache key includes some unique element that makes sure the cache is busted once the hack is removed. I'll have a look into this.

Heating up the cache directly after loading sounds like a good idea! It felt more acceptable to wait when opening instead of waiting for the results to come in after adding an analysis or clicking "Apply".

This is now done in my branch - I could clean that up to a mergable status, only including the warmup and enabling of jit caching,  which should "work" even without the numba changes. Then I can work on a monkey-patched hack in a follow-up. Does that sound good?

Opening a small RAW dataset now takes 1,5 minutes on my system after #886 was merged, which feels like an eternity while watching a spinner. Reopening here since #886 is only part of the solution.

For reference, the first run of a point analysis took about 30 s on the same system before. That means without actual caching it seems that #886 introduced some inefficiencies.

That means without actual caching it seems that #886 introduced some inefficiencies.

I'll have a look later. Thanks for the report! Is that with the latest numba version?

Is that with the latest numba version?

This was 0.51.2. Retesting now with 0.52.

With 0.52 it is 1:08 min to open a RAW dataset for the second run, i.e. with everything warmed up.

Repurposed to remind us of removing the hack of the Numba cache #931 as soon as Numba supports caching these functions natively.


Blight map ctrl + D search as normal map

🐞 bug report
Describe the bug
Option ctrl+D doesn't see that map is blight, it search only for normal map without blight prefix.
To Reproduce
Steps to reproduce the behavior:

Use ctrl+D on blighted map

Expected behavior

Search for blighted version of map instead normal.

🌍  Your Environment

OS: . Win10 x64
Version:
PoE: Steam English

Hi, thanks for creating an issue. This is a duplicate of #216.


[Storage] az storage account management-policy rule: Support rule level operation for management policy

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

Storage


Removing unused TH parameters

Description
Thanks to @andfranklin for finding these dead parameters!

Checklist

[X] This PR has only one purpose or idea.
[X] Tests have been added/updated to verify that the new/changed code works.

[X] The code style follows good practices.
[X] The commit message(s) follow good practices.

[X] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[X] The documentation is still up-to-date in the doc folder.
[X] The dependencies are still up-to-date in setup.py.

The code coverage went down because the total number of lines of code went down.
That's fine.

@andfranklin Just FYI. The first PR is up.


Pigeon Delivery

So. I hate to be that guy, but I'm opening an issue without having all the infos lol
So I tried to use the pigeon mecanic but nothing happened. I play on 1.20.1 still and I don't know how I got acces to bundles, but I got them.
F3+H show minecraft:bundle for them but must be a mod that enable them 🤷♂ I'll do some check to see wivh one as soon I got time
I also have a very poor internet connection, the issue simply could very be that (I have tested with my two accounts on my server) but like, my connection is so bad I get connection timeout when using both. During my testing it held just for a good 10 minutes but the pigeon didn't deliver anything...
I also didn't switch my server to latest version for testing. Uploading a backup of my world would takes about 12 hours. Forgive my laziness 😅
PS, the pigeon takes the bundle.

Honestly, I wouldn't be surprised if the pigeon bundle delivering doesn't work, because I haven't had a chance to test it ever since I updated the ai for all the birds. I'll take a look at it and see what the issue is. The fix will probably come with the next update.
Probably not an issue on your end, so no need to worry! (btw I think there's an experimental toggle that enables bundles, not sure if that's what you used)

I forgot to mention that there is a maximum distance for the pigeon, that being 1024 blocks. I'll remove this in the next update, but that could be a reason why it wasn't working. (I haven't tested multiplayer yet, just looking at the code)

After some testing it appears that the pigeon does teleport to the recipient. However, a bug that is present in the base game makes the mob invisible. Relogging appears to fix this.
This is caused by MC-273361, which turns entities teleporting from force loaded chunks invisible.
Unfortunately, I can't fix this without updating the mod to a later version of the game.


Create source syntax

follow up on https://github.com/rilldata/rill-developer/pull/896
This just supports syntax parsing nothing more.

Just to keep things organised, can we close either the original PR (https://github.com/rilldata/rill-developer/pull/896) or this one?
If you don't have permission to push to existing PRs, ping Himadri and ask for permissions – it's useful to sometimes be able to push to each other's branches.


Allow build variables for transformations

I need to transform a connection string based on the build variable Build.SourceBranch.
It would be nice if your tool supported passing in build variables in VSTS, so it is possible to create transformations based on these variables.

@wertzui if you use it as VSTS extension, feel free to use any available variable, for example transofrmation file could be:
{
   "path/to/transform": "Branch is $(Build.SourceBranch)"
}

This is what you're asking for?

Didn't know, it was already working, thanks :)


fix: readme for builder output

Codecov Report

Merging #33 (054a661) into 0.4.0 (f805bcd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            0.4.0      #33   +/-   ##
=======================================
  Coverage   87.48%   87.48%
=======================================
  Files          22       22
  Lines         983      983
=======================================
  Hits          860      860
  Misses         85       85
  Partials       38       38

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 f805bcd...054a661. Read the comment docs.


Image File Browser

Is there a way to utilize CKEditor plugins with this? For example i am looking for a way to browse the system for images to upload. CKEditor has plugins for that but how could I use those plugins, if possible?

Yes. You can use plug-ins with this package. It has been discussed multiple times.
Please refer to these issues - #12 and #7

Excellent, much appreciated!


Remove unnecessary null pointer checks

An extra null pointer check is not needed in functions like the following.

MLP<FloatType>::init
PolyMesh<FloatType, VERT_MAX>::init

Thanks for this. Checking if it is a nullptr just make me feel better. You can certainly delete the if statement, and it will make the code look better.
You can make a request for code improvement (and with necessary description), and I will be happy to merge your code.


Fix typo in csv/reader.rs

The ci failure is unrelated to my change.

The ci failure is unrelated to my change.

it's because of a new rust release. I'll fix it


chore: change artifact name

fix "an artifact with this name already exists" error

:tada: This PR is included in version 4.12.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


delete a record

is there a way to delete a DNS record?

Cloudflare.connect(token: CLOUDFLARE_API_TOKEN) do |connection|
  zone = connection.zones.find_by_name(TLD_DOMAIN)
  record = zone.dns_records.find_by_name(name)

  record.delete
end

Cloudflare.connect(token: CLOUDFLARE_API_TOKEN) do |connection|
  zone = connection.zones.find_by_name(TLD_DOMAIN)
  record = zone.dns_records.find_by_name(name)

  record.delete
end

Thanks, @sirwolfgang! Good example, looks solid to me.
@ioquatix Would that example feel out of place in the Usage section of the Readme? Is it too much? I suggested linking to the underlying Cloudflare ReST API in an Issue here, to let users explore that way.

Thanks, @sirwolfgang! Good example, looks solid to me.
@ioquatix Would that example feel out of place in the Usage section of the Readme? Is it too much? I suggested linking to the underlying Cloudflare ReST API in an Issue here, to let users explore that way.

@olleolleolle feel free to update the documentation.


How to configure zmodem on terminus?

I configure some config on iTerm2, so it can use lrzsz to transfer files, but I wanted to use terminus to transfer files to the server. How can I do?

Just run rz / sz in your remote session.


[AMBARI-24747] Fix Add Component to host functionality

What changes were proposed in this pull request?
Fixed Add Component functionality on Host view.
How was this patch tested?
All unit tests passing (except one that is covered under a separate Jira).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4211/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4225/
Test FAILed.
Test FAILured.


Fix sample perf (Option 2)

Don't cause the await to marshal back to the current SynchronizationContext. ref

Hi @benaadams, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

Also see #135

@benaadams, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;

Closing now that Option 1 is merged into dev.


Dan stef/dialectics

Merges the behavior of
https://github.com/stephanenicolas/toothpick/pull/48
and master
binding can now be scoped or not. TP is born :)

Coverage increased (+0.07%) to 85.682% when pulling 7f9e394da2732cc2efc309ddd1e285125174922a on dan-stef/dialectics into 07b9bdff7aa3502600ffebb2e70b80df13927a0c on master.

Coverage increased (+0.2%) to 85.79% when pulling 506d1a3d7eaf9cea448afd3c18f4baa0c5642659 on dan-stef/dialectics into 07b9bdff7aa3502600ffebb2e70b80df13927a0c on master.

Coverage increased (+0.2%) to 85.823% when pulling 7083bcad03481f0284560845d77e0f68479e94ba on dan-stef/dialectics into 07b9bdff7aa3502600ffebb2e70b80df13927a0c on master.


Database environment error

Hello, got a problem with unix version since today. I do not understand what is appening.
06/26/17 17:36:01 ***************************************** GRIDCOIN RESEARCH ***************************************************
06/26/17 17:36:01 Gridcoin version v<IP_ADDRESS>-g-research (2017-06-24 16:11:18 +0200)
06/26/17 17:36:01 Using OpenSSL version OpenSSL 1.0.1t  3 May 2016
06/26/17 17:36:01 Default data directory /home/pi/.GridcoinResearch
06/26/17 17:36:01 Used data directory /home/pi/.GridcoinResearch
06/26/17 17:36:01 dbenv.open LogDir=/home/pi/.GridcoinResearch/database ErrorFile=/home/pi/.GridcoinResearch/db.log
06/26/17 17:36:01 ERROR: CDB() : error Invalid argument (22) opening database environment
06/26/17 17:36:01 Gridcoin: Error initializing database environment /home/pi/.GridcoinResearch! To recover, BACKUP THAT DIRECTORY, then remove everything from it except for wallet.dat.
06/26/17 17:36:01 gridcoinresearch exiting...
06/26/17 17:36:01 Flush(false) db not started
06/26/17 17:36:01 StopNode()
06/26/17 17:36:01 Flush(true) db not started
06/26/17 17:36:01 Gridcoin exited

I had the exact same thing when copying my Linux wallet to Windows. Try deleting the database/ folder.

Skipped the problem by uninstalling everithing and rebuilding libdb4.8 and the wallet... (no time to search).


mqtt-broker: Default to validating the server certificate for mqtts://-URLs

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

This commit does break MQTT-connectivity for configurations where a mqtts://-URL is used without additional SSL/TLS configuration and a server certificate that cannot be validated. With this commit, in these cases the certificate validation must be explicitly disabled.

Proposed changes

When the server URL has the protocol mqtts:// and no further SSL/TLS options are supplied, default to validating the server certificate. Fixes #2379.
Checklist

[X] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the mailing list/slack team.
[X] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

Hi - no need to raise a new PR to retarget the change to the dev branch - we can update the existing one.

If you click the edit button at the top of #2380 , it will let you change the branch you are targeting

I can only change the target branch there, that's why I created a new PR. Is it ok to merge from master to dev?

Yup - for a change like this the merge should be automatic with no other effort needed.

Ok, just changed the destination branch in my first PR, but now it containts 52 commits. Looks like it did not work...?

Now in this PR, the cla-bot does not pick up the additional email address which I added in my GitHub settings. Can you manually trigger it to rescan or something like that?

Coverage remained the same at 77.591% when pulling 422ed371f717e11a163a0cd2ce3a785864c13810 on skleeschulte:dev into d45274494d327c125cd8a88141cd2fb7f293abc7 on node-red:dev.

would it be possible to set a 'flag' upon creation with this new implementation, and check runtime if that flag is there, so we can have:

old nodes getting the old behaviour: no additional validation unless specified
new nodes getting default validation unless explicitly disabled
No clue if it's feasible but it would prevent old integrations from magically breaking after upgrading. Fixing TLS is usually an enormous chore for those who don't have proper TLS in place.


Update to latest codecs API

What does this pull request do?
This PR is dependent on purescript-argonaut-codecs#47. That one introduces breaking changes that propagate here.

Hey @davezuch! Is this ready to be updated now that core has been?

@thomashoneyman I created a PR for purescript-argonaut-traversals that'll need to be released before I can fix the dependencies for this repo

@davezuch I've released that version.