why for the -1 on my comment?

https://github.com/isaacs/github/issues/66#issuecomment-734077419
no likey dark reader?

Please don't create spam issues like this. I downvoted your comment because you should not use issues like that to advertise a chrome plugin. And seeing the other downvotes, I am not the only one thinking this.

The hate is strong with this one .... I have no stock in the plugin ...
just saying ... keeping hating
On Fri, Nov 27, 2020, 10:07 AM Rune Laenen<EMAIL_ADDRESS>wrote:

Please don't create spam issues like this. I downvoted your comment
because you should not use issues like that to advertise a chrome plugin.
And seeing the other downvotes, I am not the only one thinking this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/runelaenen/shopware6-two-factor-auth/issues/31#issuecomment-734899972,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEVMO62N5HHT4E7NUME3P3SR7FEHANCNFSM4UEOR5OA
.

All good brother.... happy holidays


DOC: Better docs for jnp.floor and jnp.ceil

Part of #21461
Rendered Preview:

https://jax--23291.org.readthedocs.build/en/23291/_autosummary/jax.numpy.floor.html
https://jax--23291.org.readthedocs.build/en/23291/_autosummary/jax.numpy.ceil.html

Hi @jakevdp
Changed args: to Args: to fix 'Args:' not found error in tests. Could you please run the tests again?


feat(config): prompt for --pfx-password

If no password is provided as a flag, prompt for one. Passewords are mandatory for PFX bundles, so there's no need to wrap this in an Option.
closes #141
Signed-off-by: Matthew Fisher<EMAIL_ADDRESS>

Test failure was a flake due to the registry pull. I re-kicked them

After doing some more thinking, we don't need a feature flag to prompt for the password. If all of the previous functions don't supply one, krustlet will now automatically open a prompt in your terminal:
><> ./target/debug/krustlet-wasi
PFX file password:

A feature flag like --pfx-password-stdin implies the password is optional, which it isn't. I opened #202 to follow up on the discussion raised in #140 around --pfx-password/$PFX_PASSWORD. I don't think we should remove them until we have the proper solution in place.


build: Update to header 1.3.265

Update known-good
Generate source (no change)

CI Vulkan-Tools build # 1160 running.

CI Vulkan-Tools build # 1160 failed.

CI Vulkan-Tools build queued with queue ID 49571.

CI Vulkan-Tools build # 1161 running.

CI Vulkan-Tools build # 1161 passed.


fix(client): server api version mismatch

Purpose of the PR

https://github.com/hugegraph/hugegraph-test/pull/5#issuecomment-1833354718
api version is outdated in client will be handled in #546, this pr will handle the ci problem later~

Main Changes

Verifying these changes

[ ] Trivial rework / code cleanup without any test coverage. (No Need)
[ ] Already covered by existing tests, such as (please modify tests here).
[ ] Need tests and can be verified as follows:

xxx

Does this PR potentially affect the following parts?

[ ]  Nope
[ ]  Dependencies (add/update license info)
[ ]  Modify configurations
[ ]  The public API
[ ]  Other affects (typed here)

Documentation Status

[ ]  Doc - TODO
[ ]  Doc - Done
[x]  Doc - No Need

�thanks, could also update the commit in this PR
maybe could choose ed493f3093eab293a75cd2f539e29ca11fa6cd81 (apache/incubator-hugegraph@ed493f3)

I see, thanks for the reply!~

It seems to be caused by this pr which removed the -k curl parameter.

refer to #550


[WIP] Update to stackstorm-st2 which includes garbage collector changes

The title says it all.

Alright, I've addressed all the issues and confirmed that it's working locally. Would still appreciate if someone else can also test it before I merge it.
To test it, you need to use latest version of StackStorm which includes garbage collector service stuff:
/opt/puppet/hieradata/answers.yaml
st2::version: 1.3dev
st2::revision: 90

Validated -
root@manas-wr-312-test:/opt/puppet# dpkg -l | grep st2
ii  python-st2client                    1.3dev.90-1                      all          Python client library and CLI for the StackStorm (st2) event
ii  st2actions                          1.3dev-90                        amd64        ST2 ActionRunner
ii  st2api                              1.3dev-90                        amd64        ST2 API
ii  st2auth                             1.3dev-90                        amd64        ST2 Authentication
ii  st2common                           1.3dev-90                        amd64        ST2 Common
ii  st2debug                            1.3dev-90                        amd64        StackStorm Debug tool
ii  st2reactor                          1.3dev-90                        amd64        ST2 Reactor
root@manas-wr-312-test:/opt/puppet# st2 --version
st2 ('1.3dev.90',)
root@manas-wr-312-test:/opt/puppet# st2ctl status
##### st2 components status #####
st2actionrunner PID: 5748
st2actionrunner PID: 5764
st2actionrunner PID: 5788
st2actionrunner PID: 5811
st2actionrunner PID: 5840
st2actionrunner PID: 5857
st2actionrunner PID: 5880
st2actionrunner PID: 5902
st2actionrunner PID: 5925
st2api PID: 9351
st2api PID: 9407
st2auth PID: 9363
st2sensorcontainer PID: 9273
st2rulesengine PID: 9316
st2web is not running.
mistral PID: 9077
mistral PID: 9078
st2resultstracker PID: 9257
st2notifier PID: 9291
st2garbagecollector PID: 9334

Going ahead and merging this.


I need 1.16.5forge And it's Embeddium compatible thank

Is your feature request related to a problem? Please describe
A clear and concise description of what the problem is.
Ex: I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.

Remember that a great explanation leads to a greater understanding

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Remember that a great explanation leads to a greater understanding

Additional context
Add any other context or screenshots about the feature request here.

asking for a port to a new version is just rude
use the issue template, there is a reason it exists
1.16 is fundamentally different from 1.12 so you can't just simply "release" a mod for a new major version.

https://github.com/Red-Studio-Ragnarok/Valkyrie/assets/82710983/583b978f-dc3d-45cd-925c-4b5b10ea3419


[BUG] Serilog Structured logging not working

Describe the bug
When using Splat.Serilog a FormatException is thrown when trying to use structured logging syntax.
For example: this.Log().Debug("{@item}",item) will raise the exception. The issue is appears in Debug as well as in all other levels.
Steps To Reproduce
Run this sample:
void Main()
{
    Log.Logger = new LoggerConfiguration().MinimumLevel.Debug()
		.WriteTo.Async(configuration => configuration.Console())
                .Enrich.FromLogContext()
                .CreateLogger();

    var logger = new SerilogFullLogger(Log.Logger) { Level = LogLevel.Error };
    Locator.CurrentMutable.RegisterConstant(logger, typeof(Splat.ILogger));

    var testObject = new TestObject();
    testObject.DoSomething();
}

public class TestObject : ReactiveObject
{
    public void DoSomething()
    {
        this.Log().Info("{@item}", new Item(1, "Test"));
    }
}

public class Item
{
    public Item(int number, string name)
    {
        Number = number;
        Name = name;
    }
    public int Number { get; set; }
    public string Name { get; set; }
}

It appears to have something to do with {} since this.Log().Debug("{}",item) raises the exception too, but this.Log().Debug("@item",item) works and prints @item in the logs.
Expected behavior
No exception should be raised. Structured logging should be possible.
Environment(please complete the following information):

OS: Win 10 Pro
Version: ReactiveUI 10.5.31

Use the extension methods found in https://github.com/reactiveui/splat/blob/6ab27ebc6739e8dd778223b623ee9dca99a625d1/src/Splat.Serilog/MutableDependencyResolverExtensions.cs#L33

I think using the extension methods and it using the func log manager should solve your issue. Shout back if it doesn't.


Istio images version mismatch after names change

Description

Istio reconciliation fails after images' offered by SoloIO team changed their names.
Expected result

No version mismatch, as this one is expected. Implemented changes need to support all versions of managed kyma (e.g. 2, main)
Actual result

2022-01-31T22:50:01.838Z	DEBUG	service/runner.go:132	Runner: Reconcile action of 'istio-configuration' with version 'local' failed: Istio components version mismatch detected: pilot version: 1.12.2, data plane version: 1.12.2-solo-fips

Steps to reproduce

Changes introduced by this PR popped this up.
Troubleshooting

Have a look at the following Reconciler code:
pkg/reconciler/instances/istio/action.go:369
func isMismatchPresent(istioStatus actions.IstioStatus) bool {
	pilot := newHelperVersionFrom(istioStatus.PilotVersion)
	dataPlane := newHelperVersionFrom(istioStatus.DataPlaneVersion)
	return pilot.compare(&dataPlane) != 0
}

Then this mismatch is being used here:
pkg/reconciler/instances/istio/action.go:105
	if isMismatchPresent(istioStatus) {
		errorMessage := fmt.Sprintf("Istio components version mismatch detected: pilot version: %s, data plane version: %s", istioStatus.PilotVersion, istioStatus.DataPlaneVersion)
		return errors.New(errorMessage)
	}

Consider logging a warning it instead of throwing an error.
Links
Found during: https://github.com/kyma-project/kyma/pull/13229

Bug fix needs mandatory tests agains kyma 2.0 and main

Tested the solution again against both Kyma main and release-2.0 branches. I used istioctl in version 1.11.4 for testing purposes.
Results for branch main:

command:

./bin/mothership-darwin local --value global.ingress.domainName=example.com,global.domainName=example.com --components istio-configuration --version main

output:

{"level":"INFO","time":"2022-02-17T12:41:28.573+0100","message":"Component 'istio-configuration' has status 'running'"}
{"level":"INFO","time":"2022-02-17T12:41:29.335+0100","message":"Resolved target Istio version: 1.11.4-solo-fips-distroless from values"}

summary: Istio target version resolved from values as expected ✅

Results for branch release-2.0:

command:

./bin/mothership-darwin local --value global.ingress.domainName=example.com,global.domainName=example.com --components istio-configuration --version release-2.0

output:

{"level":"INFO","time":"2022-02-17T12:46:21.922+0100","message":"Component 'istio-configuration' has status 'running'"}
{"level":"INFO","time":"2022-02-17T12:46:22.671+0100","message":"Resolved target Istio version: 1.11.2 from Chart definition"}

summary: Istio target version resolved from Chart definition as expected ✅

We encounter a problem of version mismatch during bump to Istio 1.12.2. I checked how istioctl reports versions for each of Istio revision we currently use.

for Kyma release-2.0 branch there is no version mismatch (data and control planes are the same): ✅

> istioctl version
client version: 1.11.2
control plane version: 1.11.2
data plane version: 1.11.2 (1 proxies)

for Kyma main branch there is no version mismatch (data and control planes are the same): ✅

istioctl version
client version: 1.11.4
control plane version: 1.11.4-solo-fips
data plane version: 1.11.4-solo-fips (1 proxies)

for Kyma PR-13229 branch: (which includes a mentioned bump) there is a version mismatch (data and control planes are reported differently): 🚫

client version: 1.12.2
control plane version: 1.12.2
data plane version: 1.12.2-solo-fips (1 proxies)

The comparison between 1.12.2 and 1.12.2-solo-fips simply returns false as they are semantically different. The mechanism that I introduced is a helperVersion which under the hood covers only major, minor and patch versions with no additional data. As a result, it required refactoring which is present in PR #845 .
The testing results are as it follows:

...


Strong reference cycle

Hey,
I was experiencing an issue where the alertview wouldn't be deallocated properly when deallocationg the rootview without dismissng the alert. I think the reason is the strong reference to the rootviewcontroller.
I'm just asking if the strong reference is intended to prevent premature deallocation, before issuing a pull request.
Thanks.

Nonono all good - please send a PR and I'll merge it in! THANKS :+1:


Markdown slugify misses backticks

This pattern:
https://github.com/Microsoft/vscode/blob/40bd5d9859f9f2e33554cb5c3895978c50c6ae94/extensions/markdown/src/tableOfContentsProvider.ts#L75
should contain a backtick. It currently doesn't which for example breaks Ctrl+click navigation:
Link to heading: [example](#hello-make)   <-- Ctrl+clicking should jump to heading but it doesn't

## Hello `make`

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:05:38.342Z)
OS Version: Windows_NT ia32 10.0.15063

Fixed by #32987


Sbt server displays weird characters around dates on Windows

steps
Run sbt, for example

sbt compile

problem
Output display weird characters on the timestamps.
Total time: 1 s, completed May 12, 2024, 2:35:31ΓÇ»PM

This does not happen on WSL (ubuntu), same terminal same font.

expectation
Display date times without the extra characters:
Total time: 1 s, completed May 12, 2024, 2:35:31 PM

notes

Thanks for the report. Could you tell us what the environment that you are seeing this issue, for example the shell and the console program? Also could you check if this is specific to sbt 1.10.0 or it happens with previous versions?

Both images are using Windows terminal. (Version: 1.20.11271.0)

Black background:  Powershell 7.4.2 (Windows 11) . I can also reproduce this on regular CMD from windows.
Blue background: WSL - Ubuntu on windows terminal

Yes, I also see this happening on 1.9.9

So I'm guessing that "ΓÇ»" is either an ASCII control sequence or some string in encoding A, misinterpreted by your shell as another encoding B, also known as mojibakeh. If you have some override of JVM parameters, could you try adding -Dfile.encoding=UTF-8?

I ran it in verbose mode, I can see the encoding being UTF-8.
❯ sbt -v
# Executing command line:
"C:\Program Files\Amazon Corretto\jdk21.0.3_9\bin\java.exe"
-Dfile.encoding=UTF-8
-Dscala.ext.dirs="C:\Users\username\.sbt\1.0\java9-rt-ext-amazon_com_inc__21_0_3"
-Xms1024m
-Xmx1024m
-Xss4M
-XX:ReservedCodeCacheSize=128m
-cp
"C:\Users\username\AppData\Local\Coursier\cache\arc\https\github.com\sbt\sbt\releases\download\v1.10.0\sbt-1.10.0.zip\sbt\bin\sbt-launch.jar"
xsbt.boot.Boot

Maybe unrelated, but I noticed that this does not work on windows/powershell
sbt -Dfile.encoding=UTF-8

It throws this error:
[error] Not a valid command: UTF-8
[error] Not a valid project ID: UTF-8
[error] Expected ':'
[error] Not a valid key: UTF-8
[error] UTF-8
[error]

It seems that on windows/powershell I am not able to override properties that contain a period.
This works:
sbt  -Dkey=value

Hey I think i found the issue.
It uses this character NNBS, this can also be reproduced in scala-cli, or other java tool I think. This might be an issue on how java handles this in windows,.
val c: Char = 8239

I don't think it is specific to SBT, so you can probably close it, unless you want to display different format in windows compared to linux.

Still reproducible on sbt 1.10.4. The JLine updates did not sort the issue out...

@main
def main(): Unit = {
  val c: Char = 8239
  println(c + "Hello world!" + c)
}

Outputs

on Windows...
This is probably a JLine issue.

This issue is cursed. It is actually not JLine issue but rather windows terminal / cmd.exe does not support display it. My last test succeeded as I was checking result in Intellij's own shell. Printing to windows terminal surfaces the issue.
public class Main {
    public static void main(String[] args) {
        char c = 8239;
        System.out.println(c + "Hello World" + c);
    }
}

Output (Windows Terminal)
PS D:\Repos\untitled12223\out\production\untitled12223> java Main
?Hello World?


zIndex problem

I am using bootstrap v3.3.7 and modal zIndex is 10050. Currently the zIndex in bootstrap-maxlength is 1099. Could you please change it to 10100?

I have an another z-index problem.
The indicator will still be shown when covered by a modal pop with alwaysShow: true option.

It's not a good idea to just change z-index to some value. Maybe make it configurable?

I think the fastest fix for this would be to temporarily change the z-index in the source code itself. (Line 441). I am planning to do a newer release with a configurable z-index, however at the moment I don't have time for it.

#141
Added optional zIndex parameter in pull request

Please close issue,
fixed in 1.9.0.


Property mutation warnings after updating to react 0.13.1 and Formsy 8.1

Seeing the following warnings when using the dev version of react(0.13.1) and the Formsy.Mixin(8.1) on a custom element. Poked around some but wasn't able to figure out a way to fix the warnings without converting the affected props to state values.
Warning: Don't set .props._attachToForm of the React component. Instead, specify the correct value when initially creating the element. The element was created by Login.
Warning: Don't set .props._detachFromForm of the React component. Instead, specify the correct value when initially creating the element. The element was created by Login.
Warning: Don't set .props._validate of the React component. Instead, specify the correct value when initially creating the element. The element was created by Login.
Warning: Don't set .props._isFormDisabled of the React component. Instead, specify the correct value when initially creating the element. The element was created by Login.

This is from the registerInputs function, which mutates props of all child form input controls. Hmm...

Hi guys,
Yeah. Formsy does take an approach with React that is not best practice. There is a new release coming up now with some more functionality. After that I will start on Formsy 2, which conforms more to best practices, but will probably have a few breaking changes.
The challenge is that a form needs to create a relationship with its children form elements. Now this is done by iterating the children of the form instance, adding some properties and a mixin on the form elements uses these props to communicate with the form instance.
An other way to go about this is using pure events, but that changes the syntax and I am unsure if it is doable. It would be something like this:
var Form = React.createClass({
  mixins: [Formsy.FormMixin],
  render: function () {
    return (
      <Formsy.Form>
        <MyFormElement name="foo"/>
      </Formsy.Form>
    );
  }
});

The mixin finds the FormsyForm element in the component and registers custom event listeners like: "Formsy.setValue", which then the children form elements can trigger and will be catched by the parent form. The problem is that React JS does not support custom events, would have to use native events, which reduces the testability with Formsy as tests would have to be run in the browser. I think this might be a good tradeoff though.
Let me create an issue discussing next version of Formsy :-)

Linking to issue: #66


Fix issue where Rails.application.assets is not available in test/prod

fixes
NoMethodError: undefined method `unregister_processor' for nil:NilClass
/home/travis/build/ManageIQ/manageiq/lib/vmdb/productization.rb:52:in `replace_directive_processor'
/home/travis/build/ManageIQ/manageiq/lib/vmdb/productization.rb:47:in `prepare_asset_directive_processor'
/home/travis/build/ManageIQ/manageiq/lib/vmdb/productization.rb:6:in `prepare'

/cc @matthewd @chrisarcand

Alternative to #5899

Looks like the issue is with css splitter.
Need to revisit that code and submit a PR

@kbrock Since #5899 was merged, do we still want to move forward with this PR?

[Doesn't have to be this PR specifically but] we certainly do want to come up with a sprockets 3 solution anyway, this isn't necessarily an 'alternative' but the beginnings of a more permanent solution it seems.

This does move us forward, but the css splitter gem is having troubles.

consensus it to remove css_splitter, since this is used for ie9, and that support has been dropped

didn't get around to removing css_splitter gem, but looks like they just updated this morning. so I'm hoping this push should resolve that issue and we'll see if there are more

Updated to declare precompiled assets. spice needs to be fixed in the core

note: css_splitter gem has been removed in another PR ==> https://github.com/ManageIQ/manageiq/pull/6069

@Fryguy pulled out the other gook. This PR is be simple again.


[ERROR]: could not open the product-and-service-lifecycle-management-graphics.pptx

Contact Details
<EMAIL_ADDRESS>Describe the error you see in the documentation, or the correction you suggest
It appears that the process flow diagram ppt file might be corrupted and cannot be opened.
(https://github.com/microsoft/dynamics365patternspractices/blob/main/graphics/product-and-service-lifecycle-management-graphics.pptx)
Your assistance will be appreciated.
Link to the article where the error is
https://github.com/microsoft/dynamics365patternspractices/blob/main/graphics/product-and-service-lifecycle-management-graphics.pptx
Code of Conduct

[X] I agree to follow this project's Code of Conduct

My workmate can open the ppt in old version of ppt. Please ignore the issue. I will close this ticket. Thank you all for the amazing work on the process and diagram.


Create a website for testing different tag manager implementations

For each implementation we probably want:

One or ideally two web pages
A way of firing a combination of custom structured and custom unstructured events
Some of the events with custom context and some without
It should be possible to load pages using different tag managers alongside one another, so we can compare how they perform. We'll also wan to compare the performance with the Snowplow integrations that come with the tag manager out of the box (Tealium / Ensighten and OpenTag) vs our setting up the Snowplow tags manually e.g. by pasting our own HTML / Javascript into the tag manager UI as a custom tag.

Simply uploading the relevant HTML files to S3 might be an easy way to set the above up, in a way that makes it possible to share the URLs with e.g. engineers at the different tag management companies.

Closing as we can use next.snowplowanalytics.com for testing tags


Optim/dx 256 getSellerOngoingAuctions func

Simplified logic in getSellerOngoingAuctions

nice, tried:

add token pair
1a. post sell order
move time to 2 min after auction start
buy out all sell balance
run claim funds
= Pair shown in ongoing Auctions, pair shows claim button after sell balance bought out, claiming disappears row

Looks good, approving


Updates the DataTable component

Related to this https://github.com/tleunen/react-mdl/issues/219
Currently this PR contains the selectable attribute alongside the new TableHeader component.

By any chances, @michaelguild13, @FoxxMD, do you know why Travis is failing on rendering the Table component? It's working fine locally oO

Ok I found the issue in the tests. It was because I used array.includes, which doesn't seem to be inside the chromium version used in Travis.


More groupby indexing problems

What happened?
There is still something wrong with the groupby indexing changes from 2023.04.0 onward. Here are two typical ways of doing a seasonal anomaly calculation, which return the same result on 2023.03.0 but are different on 2023.04.2:
import numpy as np
import xarray as xr

# monthly timeseries that should return "zero anomalies" everywhere
time = xr.date_range("2023-01-01", "2023-12-31", freq="MS")
data = np.linspace(-1, 1, 12)
x = xr.DataArray(data, coords={"time": time})
clim = xr.DataArray(data, coords={"month": np.arange(1, 13, 1)})

# seems to give the correct result if we use the full x, but not with a slice
x_slice = x.sel(time=["2023-04-01"])

# two typical ways of computing anomalies
anom_gb = x_slice.groupby("time.month") - clim
anom_sel = x_slice - clim.sel(month=x_slice.time.dt.month)

# passes on 2023.3.0, fails on 2023.4.2
# the groupby version is aligning the indexes wrong, giving us something other than 0
assert anom_sel.equals(anom_gb)

Related: #7759 #7766
cc @dcherian
What did you expect to happen?
No response
Minimal Complete Verifiable Example
No response
MVCE confirmation

[ ] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[ ] Complete example — the example is self-contained, including all data and the text of any traceback.
[ ] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[ ] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
No response
Anything else we need to know?
No response
Environment

x_slice.groupby("time.month") - clim

Nice, we test x.groupby("time.month")  - clim.slice(...) but not what you have.
Is it possible for you to run nightly tests against xarray's main branch? That would help a lot.


CC API V3 site redirections loose fragments

Issue
http://v3-apidocs.cloudfoundry.org/index.html#metadata redirects to http://v3-apidocs.cloudfoundry.org/version/3.69.0/index.html (i.e. without the fragment to the subsection)
Context
As an API user, in order to add deep permalinks to the latest api version (e.g. in specs such as https://github.com/openservicebrokerapi/servicebroker/blob/master/profile.md#cloud-foundry-context-object), I need the api website redirects to preserve fragments in the original url. See https://stackoverflow.com/a/2305927 for related http specifications .
Steps to Reproduce
open in web browser http://v3-apidocs.cloudfoundry.org/index.html#metadata
Expected result
http://v3-apidocs.cloudfoundry.org/index.html#metadata redirects to http://v3-apidocs.cloudfoundry.org/version/3.69.0/index.html#metadata
Current result
http://v3-apidocs.cloudfoundry.org/index.html#metadata redirects to http://v3-apidocs.cloudfoundry.org/version/3.69.0/index.html (i.e. without the fragment to the subsection)

Hi @gberche-orange, we have fixed this. As of yesterday, the URL hash is now preserved when redirecting. I'm going to close this, but feel free to reopen if you have more problems with it.
cc @selzoc

great, thanks @reid47 and @selzoc !


adding RubyOrigin token

Please note: This repository is being rebuilt to accept the new volume of token additions and modifications. PR merges will be delayed.

I agree to not ping anybody on Discord/Twitter/email about this pull request. Instead I will inquire by posting a new comment in the pull request if needed.

PRs are reviewed in bulk and and can take up to two weeks to be merged.
This repository is managed using an auto merge action. Please ensure your PR has no deleted lines, and it will be merged.
Please provide the following information for your token.
Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have

Token Address:
Token Name:
Token Symbol:
Logo: (logo should be uploaded under assets/mainnet//*.<png/svg>)
Link to the official homepage of token:
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):

Auto merge requirements
Your pull request will be automatically merged if the following conditions are met:

Your pull request only adds new tokens to the list. Any modification to existing
tokens will require manual review to prevent unwanted modifications.

Your pull request does not touch unrelated code. In particular, reformatting changes to unrelated
code will cause the auto merge to reject your PR.

Any asset files added correspond to the token address you are adding. Asset files
must be PNG, JPG or SVG files.

Your change is valid JSON and conforms to the schema. If your change failed validation,
read the error message carefully and update your PR accordingly.

No other tokens shares the same name, symbol or address.

For example, this change would be rejected due to unrelated changes:

The bot runs every 60 minutes and bulk-merges all open pull requests to prevent conflicts.
This means that you need to wait up to 60 minutes for your pull request to be merged or reprocessed.

Please include change to the src/tokens/solana.tokenlist.json file in the PR.
DON'T modify any other token on the list.
At minimum each entry should have
Token Address: 8FZGyK7yw5C4aTwaHsXKQm31XZCiVWTqavHtVgrJeR3H
Token Name: RubyOrigin
Token Symbol: RUBY
Logo: https://raw.githubusercontent.com/RforRuby/crypto/main/ruby-origin-logo.png
Link to the official homepage of token: https://rubyorigin.io/
Coingecko ID if available (https://www.coingecko.com/api/documentations/v3#/coins/get_coins__id_):


how can i error handler to sentry, don't use \Yii::$app->get('raven'); before 'throw new \Exception("message");'

how can i error handler to sentry, don't use \Yii::$app->get('raven'); before 'throw new \Exception("message");'
It can not recored to sentry if i don't use
\Yii::$app->get('raven');

before
throw new \Exception('bobtest');

I want to record every system error. Like: Warning, Exception, Error.
What should i do?
Thanks.

It's for adding some extra data into the raven event before the component will handle the exception. However the component will handle this exception and send all the data it gathered to raven. Most probably it is sufficient for you to use it without such call.

Thanks your reply.  :)
How to config error handle is use raven ?
For example:
Goal: Config database
'db' => [
        'class' => 'yii\db\Connection',
        'dsn' => 'mysql:host=localhost;dbname=db_name',
        'username' => 'username',
        'password' => 'password',
        'charset' => 'utf8',
    ],

add above config to app/config/main.php
I want to know where is effective Yii2 to record error to Sentry.

I forgot configure
[
    'bootstrap' => ['log', 'raven']
]

Sorry for my foolish.


IE9 and javascript oidc-client.js ver 1.0.1 build

Hi,
I am using oidc-client.js and all was good until one of the users reported problem with IE9.
when issuing a call to:
var mgr = new Oidc.UserManager(settings);
I get:
SCRIPT438: Object doesn't support property or method 'apply'
app.js, line 39 character 1

Thank you
Mark

I have a feeling there are many things that won't work in IE9. I can take a look when I get some time. You can do some investigation in the meantime.

thank you Brock,
there are 2 problems to do with that

use of console object in IE9
logger.info.apply does not have apply method

once I resolved the above mentioned - I am getting

"Unhandled promise rejectionError: Access is denied.".

IE9 does not support CORS but I was under impression by reading other posts that this should be working, but at the same time based on your response I am not sure if it should ;~)
Please advise
Thank you
Mark


Invalid solution if artificial variables are still in basis after phase one

A problem in the current implementation: after phase 1 an artificial variable can still be in the basis (with value 0). Such variables need to be driven out of the basis before starting phase 2, otherwise they might become non-zero in phase 2, which will make the solution invalid.
This is explained here:

Drive artificial variables out of the basis: If l-th basic variable is artificial examine l-th row of B^−1 A. If all elements = 0 ⇒ row redundant. Otherwise pivot with non-0 element.

The following program demonstrates this issue. The returned solution violates the constraints:
A = [
	1//1     1        0        0        0;
	0        0        1        1        0;
	1        0        1        0        0;
	0        1        0        1        1;
	0        0        0        0        1;
];
c = vec([0//1        1        1        0        0]);
b = vec([1//2 1//2 1 1 1 ]);

(status,sol) = simplex(c, A, b); # already in standard form
println(A * sol == b);   # prints false, should be true

Interesting, I'd accept a fix + a test for this use case!
It should be necessary to explicitly drive all the artificial variables at, iirc, but I'm clearly missing a step to make it work. You can see the handling of cB partially addresses this in the phase-switching code.

I'm actually not using your julia code directly, but a C++ port of it included in my libqif library (here) (using a patched version of armadillo that works with rationals).
I've fixed the issue by driving out the artificial variables, the corresponding lines are here. It should be easily transferable back to julia, I'll do it if I find some time.


Lang string set support

Thanks for cleaning up my messy code, can confirm this works now :)

would to merge squash this one :trolleybus:


All nodes are not listed

Hello,
I'm using Flocker with VMware Block Device Backend but the Flocker Control Service node don't see all nodes at the same time.
I have 3 nodes on CENTOS 7:

[ ] node1: All  Flocker Agent Services are up. flocker-dataset-agent, flocker-container-agent and  flocker-docker-plugin are running well. Ip address is <IP_ADDRESS>
[ ] node2: All  Flocker Agent Services are up. flocker-dataset-agent, flocker-container-agent and  flocker-docker-plugin are running well. Ip address is <IP_ADDRESS>
[ ] node-control (node control only): The Flocker Control Service is Up. flocker-control is running well. Ip address is <IP_ADDRESS>

flockerctl  list-nodes command shows:
[root@machine flocker]# flockerctl  list-nodes
SERVER         ADDRESS
56618826       <IP_ADDRESS>

If I restart node1 services, then I run flockerctl  list-nodes command, I don't see node2
[root@machine flocker]# flockerctl  list-nodes
SERVER         ADDRESS
56618826       <IP_ADDRESS>

Any ideas?
Why I can not see all the nodes at the same time?
This question concerns Flocker Services? VMware Block Device Backend?
/opt/flocker/bin/pip freeze  on each node shows:
attrs==15.1.0
Babel==1.3
backports.ssl-match-hostname==<IP_ADDRESS>
bitmath==1.2.3.post4
boto==2.38.0
boto3==1.2.2
botocore==1.3.17
cffi==1.5.2
characteristic==14.3.0
cryptography==1.2.3
debtcollector==0.5.0
docker-py==1.6.0
docutils==0.12
effect==0.10
eliot==0.11.0
eliot-tree==15.3.0
enum34==1.0.4
Flocker==1.11.0
futures==2.2.0
google-api-python-client==1.4.2
httplib2==0.9.2
hypothesis==1.14.0
idna==2.0
ipaddr==2.1.11
ipaddress==1.0.7
iso8601==0.1.10
jmespath==0.9.0
jsonschema==2.4.0
klein==0.2.3
machinist==0.2.0
msgpack-python==0.4.6
ndg-httpsclient==0.4.0
netaddr==0.7.14
netifaces==0.10.4
oauth2client==1.5.2
oslo.config==1.12.1
oslo.i18n==1.7.0
oslo.serialization==1.6.0
oslo.utils==1.6.0
pbr==0.11.1
prettytable==0.7.2
psutil==2.1.2
pyasn1==0.1.9
pyasn1-modules==0.0.8
pycparser==2.14
pycrypto==2.6.1
pyOpenSSL==0.15.1
pyrsistent==0.11.12
python-cinderclient==1.1.1
python-dateutil==2.4.2
python-keystoneclient==1.4.0
python-keystoneclient-rackspace==0.1.3
python-novaclient==2.24.1
pytz==2015.4
pyvmomi==<IP_ADDRESS>6.4
PyYAML==3.10
repoze.lru==0.6
requests==2.7.0
rsa==3.3
service-identity==14.0.0
simplejson==3.7.3
six==1.9.0
stevedore==1.5.0
toolz==0.7.4
treq==0.2.1
Twisted==15.5.0
txeffect==0.9
uritemplate==0.6
vSphere-Flocker-driver==1.0
websocket-client==0.32.0
Werkzeug==0.10.4
wrapt==1.10.4
zope.interface==4.1.2

I have not seen any error of Flocker Agents Services and Flocker Control Service.

Hi @aliouba,
I notice that in the case of seeing <IP_ADDRESS> or <IP_ADDRESS> the same server ID is used 56618826. Did you by chance use the same node.crt and node.key for both servers.
Make sure to create a seperate node.crt/key for each server.

You're right. My bad.
Thank you very much


vopr: fix false positive when getting to recovering_head after version upgrade

We have special code to disable storage faults when restarting a replica after a crash in the vopr. We also have similar code when restarting replica during upgrades but it doesn't quiet handle all special cases (in particular, it doesn't repair the WAL). Extract this special code into a special function to make sure that more of the logic is shared!
In particular, in this seed we have a cluster of two, where one replica gets .recovering_head in the release upgrade path
Seed: ./zig/zig build -Drelease vopr --<PHONE_NUMBER>2009206202

Requesting review mostly to brainstorm the proper solution here!

:thinking: My main issue is that this extra repair code shouldn't run during the replica_tests. (Which for me hints that Cluster is the wrong place for it).

I am wondering if I should be scrapping this whole "try not to fault" logic here, and instead do a "core_missing_head" thing?

Yeah, I'm starting to lean that way too. My only worry is that it could end up "hiding" a real liveness bug if it was too broad -- i.e. if over time it began to cover scenarios that we weren't actually aware of --  but that seems manageable by making the verification strict.

I sort of convinced myself that the right fix here is to go in the opposite direction, and to “just” get rid of recovering head. That is, use that idea that replicas would explocitly send nacks for the entire journal in a DVC. So that a recovering head replica is just a normal replica that simply dosnt nack and prepares past the head.
I worry that with our fault disabling logic we already might be masking a real liveness issue, as vopr seems to eager with repairing replicas if there’s a chance of one more recovering head

Discussed this during walk&talk, decided to let it be for the moment and revisit once this bites us again!


Lazily instantiate codemirror instances

By taking advantage of the fact that a user can not interact
with a codemirror instance until the editor is in the viewport, we can be
smart and only set the codemirror editors up once a user scrolls them into view.
For small notebooks this change has a limited impact, however for large
notebooks with 100s of code cells this change results in a significant
performance boost to the application, especially in the initial time to
render metric for notebooks.

Tested With http://nbviewer.ipython.org/url/norvig.com/ipython/TSPv3.ipynb
Initial setup work

Download the ipython notebook
Convert to a beaker notebook
Save to filesystem

Testing Process

Start the profiler
Open the bkr notebook
Once the initial map paints, stop the profiler

Source analysis

Ref #1546
Fixes #178

The power of JS libraries....
It looks nice

i found i could get a code cell that had not been instantiated on screen

I did some follow up time measurements on my computer (these numbers will be highly variable depending on the system
New:
16.42 seconds

Old:
26.5 seconds

Thanks, that's a fantastic improvement in performance.
However, I can see the code cells instantiating as I scroll, which is pretty annoying.
Instead of starting them out as unstyled text, can you initialize them to the right font size
with the box around them, but otherwise static?  That should be really fast, and should look
exactly like they do after they become real CodeMirrors, except for the text changing color for
the highlighting.

updated

:dolphin: :dolphin:

:dancers: :dancers: :dancers: :dancers:


Add Default Property Set to ConvertTo-KPPSObject

Added default display members to return objects.

added this feature


Local variables assigned with super() cause autocomplete to crash the editor

Godot version
4.0 alpha 4
System information
Windows 10
Issue description
If I create a new local variable by a statement such as var local_variable = super() then writing local_variable. causes the autocomplete dialog to stall, which causes the Godot editor to crash.
To illustrate, I have a minimal project of two files:
BaseItemFactory.gd:
extends Node
class_name BaseItemFactory

func make_item():
	return 0

DerivedItemFactory.gd
extends BaseItemFactory
class_name DerivedItemFactory

func make_item():
	var item = super() + 5
	# item.     CRASHES AUTOCOMPLETE
	return item

Typing item. on the line where the comment is attempts to open the property autocomplete dialog and crashes the program.
I've tested a few slightly different cases, and it seems to be an issue related to super().
The crash persists even if any of these changes are made:

super() is replaced with super().make_item()
type hints are added to the function make_item or the variable item
the body of make_item in BaseItemFactory changes (even a body with just pass will still trigger the crash)
the number of arguments to make_item changes
if the local variable item is assigned not to super() alone but to an expression involving super() (like super() + 5 or something)
if BaseItemFactory instead extends from Resource
if no local variable is assigned and autocomplete is attempted after on super(), e.g. extends BaseItemFactory
 class_name DerivedItemFactory

 func _make_item():
     return super()  # replacing with return super().a and program crashes

The crash doesn't occur with the following change:

DerivedItemFactory calls the base class's make_item from a differently named function, e.g.

extends BaseItemFactory
class_name DerivedItemFactory

func _make_item():
    var item = super.make_item()
    # item.     DOESN'T CRASH AUTOCOMPLETE
    return item

Steps to reproduce

Create a new class, A with a function. (Whether it has arguments, has type hints, or actually returns something doesn't seem to matter.)
Create a new class, B, extending A.
In B, create a function that overrides a function in A. In that function, create a local variable and assign it to any expression that includes super().
Trigger the local variable's property autocomplete. The editor crashes.

Minimal reproduction project
Bug_Property_Assignment_Setter_Use.zip

I've encountered the same crash.
The suggested workaround doesn't work for me though.
I have to comment out the line with super() before I edit the function that uses it lest it crashes.
There's also no error output whatsoever, even with a target=debug build running with a console.

@axo-lotl Can you check if this is fixed in the latest version?

I cannot reproduce this, it might have been fixed with the many changes since the report.


tweaking travis

Coverage remained the same at 83.41% when pulling 7b7cf18810db3b2b450f0df9f7dd2302fcd13745 on rsby:master into 379d946abe9ddf66dd4a0d4fe361883586f48063 on gunmetal:master.


System.AccessViolationException when calling solution.Min(slackVar)

The following exception is raised when trying to access:
solution.Min(slackVar);
Or:
solution.Max(slackVar);
Exception message:

System.AccessViolationException: 'Attempted to read or write protected memory. This is often an indication that other memory is corrupt.'
I am using OR-Tools 7.0.615-beta with Dot Net Core 2.1

Data model:
class DataModel
{
    private long[,] timeMatrix_;
    private long[,] timeWindows_;
    private long[] timeDemand_;
    private long[] demands_;

    // Constructor:
    public DataModel()
    {
        timeMatrix_ = new long[,] {
          {0, 96, 139, 28, 27, 23, 26, 22, 23, 22, 26, 26, 24, 24, 25, 29, 27},
          {96, 0, 128, 123, 22, 26, 28, 24, 28, 28, 213, 27, 25, 28, 212, 210, 214},
          {139, 128, 0, 1211, 1210, 26, 23, 29, 25, 28, 24, 215, 214, 213, 29, 218, 29},
          {28, 123, 1211, 0, 121, 27, 210, 26, 210, 210, 214, 26, 27, 29, 214, 26, 216},
          {27, 22, 1210, 121, 0, 26, 29, 24, 28, 29, 213, 24, 26, 28, 212, 28, 214},
          {23, 26, 26, 27, 26, 0, 22, 23, 22, 22, 27, 29, 27, 27, 26, 212, 28},
          {26, 28, 23, 210, 29, 22, 0, 26, 22, 25, 24, 212, 210, 210, 26, 215, 25},
          {22, 24, 29, 26, 24, 23, 26, 0, 24, 24, 28, 25, 24, 23, 27, 28, 210},
          {23, 28, 25, 210, 28, 22, 22, 24, 0, 23, 24, 29, 28, 27, 23, 213, 26},
          {22, 28, 28, 210, 29, 22, 25, 24, 23, 0, 24, 26, 25, 24, 23, 29, 25},
          {26, 213, 24, 214, 213, 27, 24, 28, 24, 24, 0, 210, 29, 28, 24, 213, 24},
          {26, 27, 215, 26, 24, 29, 212, 25, 29, 26, 210, 0, 21, 23, 27, 23, 210},
          {24, 25, 214, 27, 26, 27, 210, 24, 28, 25, 29, 21, 0, 22, 26, 24, 28},
          {24, 28, 213, 29, 28, 27, 210, 23, 27, 24, 28, 23, 22, 0, 24, 25, 26},
          {25, 212, 29, 214, 212, 26, 26, 27, 23, 23, 24, 27, 26, 24, 0, 29, 22},
          {29, 210, 218, 26, 28, 212, 215, 28, 213, 29, 213, 23, 24, 25, 29, 0, 29},
          {27, 214, 29, 216, 214, 28, 25, 210, 26, 25, 24, 210, 28, 26, 22, 29, 0},
      };
        timeWindows_ = new long[,] {
          {0, 0},
          {100, 150},
          {100, 150},
          {5, 100},
          {5, 100},
          {200, 500},
          {5, 100},
          {0, 50},
          {5, 100},
          {220, 500},
          {10, 1500},
          {10, 1500},
          {400, 500},
          {5, 1000},
          {5, 1000},
          {10, 1500},
          {5, 1000},
      };
        demands_ = new long[] { 0, 2221, 3171, 6393, 2186, 7423, 796, 3138, 7278, 5621, 9252, 6191, 5432, 4236, 2356, 8288, 2178 };
        timeDemand_ = new long[] { //Task Size
          0,
          10,
          120,
          40,
          160,
          300,
          50,
          20,
          360,
          30,
          90,
          150,
          170,
          200,
          210,
          230,
          70,
      };
    }
    public ref readonly long[,] GetTimeMatrix() { return ref timeMatrix_; }
    public ref readonly long[] GetTimeDemandMatrix() { return ref timeDemand_; }
    public ref readonly long[,] GetTimeWindows() { return ref timeWindows_; }
    public ref readonly long[] GetDemands() { return ref demands_; }
    public int GetVehicleNumber() { return 5; }
    public int GetDepot() { return 0; }

    public long getAllowWaitingTime()
    {
        // To allow for any waiting time:
        return this.getMaximumWorkerCapacity();
    }
    public long getMaximumWorkerCapacity()
    {
        return 360;
    }
}

Code is based on https://github.com/google/or-tools/blob/master/ortools/constraint_solver/samples/VrpTimeWindows.cs with few modifications to calculate TimeDemand (needed time to finish the task at each specific site).
class TimeCallback : LongLongToLong
{
    private long[,] timeMatrix;
    private long[] timeDemand_;
    private RoutingIndexManager indexManager;
    public TimeCallback(DataModel data, RoutingIndexManager manager)
    {
        timeMatrix = data.GetTimeMatrix();
        timeDemand_ = data.GetTimeDemandMatrix();
        indexManager = manager;
    }

    override public long Run(long fromIndex, long toIndex)
    {
        // Convert from routing variable Index to time matrix NodeIndex.
        int fromNode = indexManager.IndexToNode(fromIndex);
        int toNode = indexManager.IndexToNode(toIndex);
        return timeMatrix[fromNode, toNode]
             + timeDemand_[fromNode];
    }
}

I believe you need to add slack variables to the assignment.
See addToAssignment.

I used routing.AddToAssignment exactly as here https://github.com/google/or-tools/blob/59ce741ab916683d25d087cb625ce6c002f0453e/ortools/constraint_solver/samples/VrpTimeWindows.cs#L171 and here https://github.com/google/or-tools/blob/59ce741ab916683d25d087cb625ce6c002f0453e/ortools/constraint_solver/samples/VrpTimeWindows.cs#L181
Strange enough, the exception is not shown when the first call to solution.Min(slackVar); or solution.Max(slackVar); happen. Actually, the exception is thrown depending on the data.
Bellow is the algorithm output if I did not try to call solution.Min(slackVar):
> Vehicle 0:
>         Start: Routing Time 74 -> Go to site(6) try to arrive between (26,100), Routing Time 114 -> Go to site(10) try to arrive between (100,194), Routing Time 52 -> Go to site(9) try to arrive between (220,308), Finish: take a rest (272,360
>         Time of the route: 266m
> Vehicle 1:
>         Start: Routing Time 96 -> Go to site(16) try to arrive between (27,40), Routing Time 224 -> Go to site(13) try to arrive between (123,136), Finish: take a rest (347,360)
>         Time of the route: 347m
> Vehicle 2:
>         Start: Routing Time 66 -> Go to site(3) try to arrive between (28,35), Routing Time 259 -> Go to site(15) try to arrive between (94,101), Finish: take a rest (353,360)
>         Time of the route: 353m
> Vehicle 3:
>         Start: Routing Time 44 -> Go to site(7) try to arrive between (22,50), Routing Time 35 -> Go to site(1) try to arrive between (100,131), Routing Time 194 -> Go to site(12) try to arrive between (135,166), Finish: take a rest (329,360)
>         Time of the route: 295m
> Vehicle 4:
>         Start: Routing Time 176 -> Go to site(11) try to arrive between (26,184), Finish: take a rest (202,360)
>         Time of the route: 202m
> Total Distance of all routes: 1463m

And bellow is when `solution.Min(slackVar) is called:
> Vehicle 0:
>         Start: Routing Time 74 -> Go to site(6) try to arrive between (26,100), Slack(0,94) Routing Time 114 -> Go to site(10) try to arrive between (100,194), Slack(0,94) Routing Time 52 -> Go to site(9) try to arrive between (220,308), Slack(0,88) Finish: take a rest (272,360)
>         Time of the route: 266m
> Vehicle 1:
>         Start: Routing Time 96 -> Go to site(16) try to arrive between (27,40), Slack(0,13) Routing Time 224 -> Go to site(13) try to arrive between (123,136), Slack(0,13) Finish: take a rest (347,360)
>         Time of the route: 347m
> Vehicle 2:
>         Start: Routing Time 66 -> Go to site(3) try to arrive between (28,35), Slack(0,7) Routing Time 259 -> Go to site(15) try to arrive between (94,101), Slack(0,7) Finish: take a rest (353,360)
>         Time of the route: 353m
> Vehicle 3:
>         Start: Routing Time 44 -> Go to site(7) try to arrive between (22,50), Slack(6,65) Routing Time 35 -> Go to site(1) try to arrive between (100,131), Slack(0,31) Routing Time 194 -> (**Exception is raised here**)

And changing the data causes the exception to be raise at different point.
For example, when chaining the timeMatrix_ to the following:
          {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0},
          {0, 0, 128, 123, 22, 26, 28, 24, 28, 28, 213, 27, 25, 28, 212, 210, 214},
          {0, 128, 0, 1211, 1210, 26, 23, 29, 25, 28, 24, 215, 214, 213, 29, 218, 29},
          {0, 123, 1211, 0, 121, 27, 210, 26, 210, 210, 214, 26, 27, 29, 214, 26, 216},
          {0, 22, 1210, 121, 0, 26, 29, 24, 28, 29, 213, 24, 26, 28, 212, 28, 214},
          {0, 26, 26, 27, 26, 0, 22, 23, 22, 22, 27, 29, 27, 27, 26, 212, 28},
          {0, 28, 23, 210, 29, 22, 0, 26, 22, 25, 24, 212, 210, 210, 26, 215, 25},
          {0, 24, 29, 26, 24, 23, 26, 0, 24, 24, 28, 25, 24, 23, 27, 28, 210},
          {0, 28, 25, 210, 28, 22, 22, 24, 0, 23, 24, 29, 28, 27, 23, 213, 26},
          {0, 28, 28, 210, 29, 22, 25, 24, 23, 0, 24, 26, 25, 24, 23, 29, 25},
          {0, 213, 24, 214, 213, 27, 24, 28, 24, 24, 0, 210, 29, 28, 24, 213, 24},
          {0, 27, 215, 26, 24, 29, 212, 25, 29, 26, 210, 0, 21, 23, 27, 23, 210},
          {0, 25, 214, 27, 26, 27, 210, 24, 28, 25, 29, 21, 0, 22, 26, 24, 28},
          {0, 28, 213, 29, 28, 27, 210, 23, 27, 24, 28, 23, 22, 0, 24, 25, 26},
          {0, 212, 29, 214, 212, 26, 26, 27, 23, 23, 24, 27, 26, 24, 0, 29, 22},
          {0, 210, 218, 26, 28, 212, 215, 28, 213, 29, 213, 23, 24, 25, 29, 0, 29},
          {0, 214, 29, 216, 214, 28, 25, 210, 26, 25, 24, 210, 28, 26, 22, 29, 0},

The algorithm output the following if I did not try to call solution.Min(slackVar):
> Vehicle 0:
>         Start: Routing Time 114 -> Go to site(10) try to arrive between (10,36), Routing Time 120 -> Go to site(2) try to arrive between (124,150), Finish: take a rest (244,360)
>         Time of the route: 234m
> Vehicle 1:
>         Start: Routing Time 191 -> Go to site(12) try to arrive between (0,19), Routing Time 150 -> Go to site(11) try to arrive between (191,210), Finish: take a rest (341,360)
>         Time of the route: 341m
> Vehicle 2:
>         Start: Routing Time 44 -> Go to site(7) try to arrive between (0,50), Routing Time 38 -> Go to site(1) try to arrive between (100,122), Routing Time 200 -> Go to site(13) try to arrive between (138,160), Finish: take a rest (338,360)
>         Time of the route: 282m
> Vehicle 3:
>         Start: Routing Time 66 -> Go to site(3) try to arrive between (5,64), Routing Time 230 -> Go to site(15) try to arrive between (71,130), Finish: take a rest (301,360)
>         Time of the route: 296m
> Vehicle 4:
>         Start: Routing Time 92 -> Go to site(16) try to arrive between (5,5), Routing Time 233 -> Go to site(14) try to arrive between (97,97), Routing Time 30 -> Go to site(9) try to arrive between (330,330), Finish: take a rest (360,360)
>         Time of the route: 355m

And here is the output when solution.Min(slackVar) is called for the latest data:
> Vehicle 0:
>         Start: Routing Time 114 -> Go to site(10) try to arrive between (10,36), Slack(0,26) Routing Time 120 -> Go to site(2) try to arrive between (124,150), Slack(0,116) Finish: take a rest (244,360)
>         Time of the route: 234m
> Vehicle 1:
>         Start: Routing Time 191 -> (**Exception is raised here**)

Do you like me to share my code? It is based on https://github.com/google/or-tools/blob/master/ortools/constraint_solver/samples/VrpTimeWindows.cs with very few modifications.

yes please.
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
Le lun. 11 févr. 2019 à 07:59, Muhammad Altabba<EMAIL_ADDRESS>a écrit :

I used routing.AddToAssignment exactly as here
https://github.com/google/or-tools/blob/59ce741ab916683d25d087cb625ce6c002f0453e/ortools/constraint_solver/samples/VrpTimeWindows.cs#L171
and here
https://github.com/google/or-tools/blob/59ce741ab916683d25d087cb625ce6c002f0453e/ortools/constraint_solver/samples/VrpTimeWindows.cs#L181
Strange enough, the exception is not shown when the first call to
solution.Min(slackVar); or solution.Max(slackVar); happen. Actually, the
exception is thrown depending on the data.

 It is based on
https://github.com/google/or-tools/blob/master/ortools/constraint_solver/samples/VrpTimeWindows.cs
with very few modifications.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/1058#issuecomment-462233227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKj17a5JphRsZb3UoRkKPT_5bY-xNf5xks5vMRTGgaJpZM4ay8TM
.

@Muhammad-Altabba Did you try to read SlackVar for the end node of a route ?

Kindly find the code at https://github.com/Muhammad-Altabba/workforce-distribution-sample
If you change the value of calculateSlack  to true, the exception will be raised.
https://github.com/Muhammad-Altabba/workforce-distribution-sample/blob/c70a102bfb65cdde27b42f0bbf047a44c4ca8cd2/WorkforceDistribution/RoutingSolver.cs#L142

@Mizux No,
However, you can now check the code.

Few thing spotted:
line30: LongLongToLong timeCallback = new TimeCallback(Data, manager);
the Garbage Collector can destroy this object since register does not keep it alive in C# (note: this will be changed in final 7.0 using delegate and correctly managing ownership)
line127: long oneRouteTime = routing.GetArcCostForVehicle(index, solution.Value(routing.NextVar(index)), 0);
You should replace the 0 by i IMHO.
line49:  for (int i = 1; i < Data.GetTimeWindows().GetLength(0); ++i)
Does Data.GetTimeWindows().GetLength(0) == Data.GetTimeMatrix().GetLength(0)  ?
according to line 193 it must be enforced...

Thanks for your highlights. I updated the code with the following tiny modifications:

Call GC.KeepAlive on the TimeCallback object.
Use the Vehcile index at GetArcCostForVehicle (Eventhough, the value for all vechiles is the same)
Enforce Data.GetTimeWindows().GetLength(0) == Data.GetTimeMatrix().GetLength(0) as a data validation by the Solver on the DataModel matrices.

However, the exception System.AccessViolationException is still thrown as before.

offtopic: Your usage of KeepAlive() seems wrong here
https://github.com/Muhammad-Altabba/workforce-distribution-sample/blob/e7f39eb86afe54617801a6b8bc03109d68dbcde7/WorkforceDistribution/RoutingSolver.cs#L33
GC may collect your object just after the call, so you must put this call after the routing.solve_with_parameter() which is not possible in your current implementation since solve is in an other method...
see: https://docs.microsoft.com/en-us/dotnet/api/system.gc.keepalive?view=netframework-4.7.2
BTW not sure it's the root cause of your problem...

You are right. I updated the code to call KeepAlive  on timeCallback after the end of its usage.
(This is after calling SolveWithParameters: https://github.com/Muhammad-Altabba/workforce-distribution-sample/blob/f4db748ece7656c532d21a470a96811172fc2dae/WorkforceDistribution/RoutingSolver.cs#L217).
And yes, the problem still exist.

Tested against current master (see the csproj hack so I can manage to "retrieve" my local nuget package)
Basically you must have all Time Windows Range between [0, horizon] what you call vehicle capacity for the second time...
Then the problem is gone.
FYI, I did:
git clone git@github.com:Muhammad-Altabba/workforce-distribution-sample.git
cd workforce-distribution-sample
... // reproduced your issue, then try to fix it
git diff

diff --git a/WorkforceDistribution/RoutingSolver.cs b/WorkforceDistribution/RoutingSolver.cs
index 9e91741..cf113d8 100644
--- a/WorkforceDistribution/RoutingSolver.cs
+++ b/WorkforceDistribution/RoutingSolver.cs
@@ -8,16 +8,16 @@ namespace ORToolsRoutingAndScheduling
     public class RoutingSolver
     {
         public readonly DataModel Data;
-        private readonly LongLongToLong timeCallback;
+        private readonly TimeCallback timeCallback;
         private readonly RoutingIndexManager manager;

         public RoutingSolver(DataModel data)
         {
+            this.Data = data;
+
             if (this.Data.GetTimeWindows().GetLength(0) != this.Data.GetTimeMatrix().GetLength(0))
                 throw new System.ComponentModel.DataAnnotations.ValidationException("Travel Time Matrix and Time Windows Matrix must have the same length.");

-            this.Data = data;
-
             // Create Routing Index Manager
             // [START index_manager]
             this.manager = new RoutingIndexManager(
@@ -41,10 +41,7 @@ namespace ORToolsRoutingAndScheduling

             // Define cost of each arc.
             // [START arc_cost]
-
-
-            int transitCallbackIndex = routing.RegisterTransitCallback(timeCallback);
-
+            int transitCallbackIndex = routing.RegisterTransitCallback(timeCallback.Call);
             routing.SetArcCostEvaluatorOfAllVehicles(transitCallbackIndex);
             // [END arc_cost]

@@ -54,7 +51,7 @@ namespace ORToolsRoutingAndScheduling
             routing.AddDimension(
                 transitCallbackIndex, // transit callback
                 Data.getAllowWaitingTime(), // allow waiting time
-                Data.getMaximumWorkerCapacity(), // vehicle maximum capacities
+                1500,// Data.getMaximumWorkerCapacity(), // vehicle maximum capacities
                 false,  // start cumul to zero
                 "Time");
             RoutingDimension timeDimension = routing.GetMutableDimension("Time");
@@ -67,6 +64,7 @@ namespace ORToolsRoutingAndScheduling
                     long index = manager.NodeToIndex(i);
                     // TODO: To be replaced to allow mulible shifts similar to:
                     // https://gist.github.com/Muhammad-Altabba/5e52cc1aee98e88f11a01181341f630e#file-vrpsolver-py-L121
+                    Console.WriteLine("TW: " + Data.GetTimeWindows()[i, 0] + ";" + Data.GetTimeWindows()[i, 1]);
                     timeDimension.CumulVar(index).SetRange(
                         Data.GetTimeWindows()[i, 0],
                         Data.GetTimeWindows()[i, 1]);
@@ -131,10 +129,10 @@ namespace ORToolsRoutingAndScheduling
                 Console.WriteLine("Worker {0}:", i);
                 Console.Write("\t");
                 long routeTime = 0;
-                var index = routing.Start(i);
+                long index = routing.Start(i);
                 while (routing.IsEnd(index) == false)
                 {
-                    var previousIndex = index;
+                    long previousIndex = index;
                     solutionObjects[i] = new WorkerAssignment();

                     long oneRouteTime = routing.GetArcCostForVehicle(index, solution.Value(routing.NextVar(index)), i);
@@ -152,7 +150,7 @@ namespace ORToolsRoutingAndScheduling
                         long minSlack = -1, maxSlack = -1;

                         // If it is set to 'true', sometime this exception is raised (depending on the data): System.AccessViolationException!
-                        bool calculateSlack = false;
+                        bool calculateSlack = true;
                         if (calculateSlack)
                         {
                             try { minSlack = solution.Min(slackVar); } catch (Exception) { }
diff --git a/WorkforceDistribution/TimeCallback.cs b/WorkforceDistribution/TimeCallback.cs
index d17745a..27627f5 100644
--- a/WorkforceDistribution/TimeCallback.cs
+++ b/WorkforceDistribution/TimeCallback.cs
@@ -5,8 +5,7 @@ using System.Text;

 namespace ORToolsRoutingAndScheduling
 {
-    class TimeCallback : LongLongToLong
-    {
+    class TimeCallback {
         private long[,] timeMatrix;
         private long[] timeDemand_;
         private RoutingIndexManager indexManager;
@@ -17,7 +16,7 @@ namespace ORToolsRoutingAndScheduling
             indexManager = manager;
         }

-        override public long Run(long fromIndex, long toIndex)
+        public long Call(long fromIndex, long toIndex)
         {
             // Convert from routing variable Index to time matrix NodeIndex.
             int fromNode = indexManager.IndexToNode(fromIndex);
diff --git a/WorkforceDistribution/WorkforceDistribution.csproj b/WorkforceDistribution/WorkforceDistribution.csproj
index c88a011..ddca12c 100644
--- a/WorkforceDistribution/WorkforceDistribution.csproj
+++ b/WorkforceDistribution/WorkforceDistribution.csproj
@@ -1,9 +1,9 @@
 <Project Sdk="Microsoft.NET.Sdk">
-
   <PropertyGroup>
     <OutputType>Exe</OutputType>
     <TargetFramework>netcoreapp2.1</TargetFramework>
     <LangVersion>7.3</LangVersion>
+    <RestoreSources>/home/mizux/work/master/packages;$(RestoreSources);https://api.nuget.org/v3/index.json</RestoreSources>
   </PropertyGroup>

   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">
@@ -12,7 +12,7 @@
   </PropertyGroup>

   <ItemGroup>
-    <PackageReference Include="Google.OrTools" Version="7.0.6150-beta" />
+    <PackageReference Include="Google.OrTools" Version="7.0*" />
   </ItemGroup>

 </Project>


Restore ACCEPT variable for BC

Signed-off-by: Edward Z. Yang<EMAIL_ADDRESS>

Stack from ghstack (oldest at bottom):

(to be filled)


Add a tag to this logger, too

Well that's weird. I definitely merged it: https://github.com/hotline-webring/hotline-webring/commit/874515a2425a6fbf10afb695c549e7e9fcd5a59c


REQUEST: create a manual + build

Is it possible for you to create a small manual? It's a bit hard to figure out how some things works, what I want to see in a manual is how to do from ground:

Create a easy figure with legs.
How to move/animate the legs and jump/go down on knee (or something)
Create environment/room/cave that can not be crossed.
How to die and point where to start if dead.
Shooting a weapon.
Adding textures and what format/size is ok and how to add shadows.

I think that would be a good basic things to know and to get started.
And a build so it's easier to run. (I don't have internet access for the computer i do things on and not able to install visual studio on the other.)

Hello and thanks for checking out UltraEd!
Yes a small manual would be very nice and I have plans on writing one eventually. I've been putting that off since I'm first trying to successfully build my own very small game with UltraEd to make sure a first official release has enough adequate functionality. I'm currently in-process with creating a parent/child system for objects and making progress.
There's currently no animation system implemented and also no audio support yet. I've been mostly busy just trying to write a good UI. Everything you mentioned besides the animation is what I'm attempting to build myself. Once I have this little "Tank" game I'm developing the source will be included with UltraEd. That should help to get yourself familiar with what is capable.
So UltraEd isn't quite just ready to be fully used unfortunately. I multitask between various other projects so feature development is a little slow currently.

Hello and thanks for checking out UltraEd!
Yes a small manual would be very nice and I have plans on writing one eventually. I've been putting that off since I'm first trying to successfully build my own very small game with UltraEd to make sure a first official release has enough adequate functionality. I'm currently in-process with creating a parent/child system for objects and making progress.
There's currently no animation system implemented and also no audio support yet. I've been mostly busy just trying to write a good UI. Everything you mentioned besides the animation is what I'm attempting to build myself. Once I have this little "Tank" game I'm developing the source will be included with UltraEd. That should help to get yourself familiar with what is capable.
So UltraEd isn't quite just ready to be fully used unfortunately. I multitask between various other projects so feature development is a little slow currently.


Add basic Message struct

Summary by Sourcery
Add a Message struct to encapsulate email details and refactor the Mailer trait to use this struct for sending emails, simplifying the email sending interface.
New Features:

Introduce a new Message struct to encapsulate email message details, including recipient, sender, subject, HTML body, and plain text body.

Enhancements:

Refactor the send_email method in the Mailer trait to accept a Message struct instead of individual parameters for recipient, subject, and body content.

Codecov Report
Attention: Patch coverage is 62.06897% with 11 lines in your changes missing coverage. Please review.

Project coverage is 73.43%. Comparing base (6d059d4) to head (c0eef3f).

Files
Patch %
Lines

src/lib/infrastructure/email/smtp.rs
0.00%
11 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #24      +/-   ##
==========================================
- Coverage   73.43%   73.43%   -0.01%
==========================================
  Files          34       35       +1
  Lines        1423     1434      +11
==========================================
+ Hits         1045     1053       +8
- Misses        378      381       +3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Allow CLI to execute multiple scripts

Is your feature request related to a problem? Please describe.
Based on #659 there may be situations where executing multiple scripts in the CLI, one after the other, is useful.
Describe the solution you'd like
The -S switch should take one or more filename arguments, pointing to scripts that will be executed one by one (in order).

This is now possible in 2.13.0:
TabularEditor.exe "Model.bim" -S "script1.csx" "script2.csx" -B


feat: SITES-26526 Import Assistant service phase 1

Implementing the Import Assistant endpoint (phase 1).  See Jira for more details.
Related Issues
https://jira.corp.adobe.com/browse/SITES-26526

:tada: This PR is included in version 1.76.0 :tada:
The release is available on:

v1.76.0
GitHub release

Your semantic-release bot :package::rocket:


A problem occurred evaluating project ':media_kit_libs_android_video'

What went wrong:
A problem occurred evaluating project ':media_kit_libs_android_video'.

media_kit: x86 JAR verification failed.

I encountered the same issue while compiling the Android client. Upon checking the error, I found that one of the four packages: default-arm64-v8a.jar, default-armeabi-v7a.jar, default-x86_64.jar, or default-x86.jar, is always 0 bytes, which causes the validation to fail. Is it possible that there is a bug in the download-related code logic?

....pub-cache/hosted/pub.dev/media_kit_libs_android_video-1.3.6/android/build.gradle' line: 108

What went wrong:
A problem occurred evaluating project ':media_kit_libs_android_video'.

media_kit: x86_64 JAR verification failed.

I am also facing similar issue


brew bottle --force-core-tap fails with undefined method `path' for nil:NilClass

brew config output
HOMEBREW_VERSION: 3.2.1-70-g5659d74
ORIGIN: https://github.com/Homebrew/brew
HEAD: 5659d74ff59414d22e285d77a7fcfcd3c1bf0614
Last commit: 32 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: de28f292bdb87aa3653c9a40ff8e2ffd2200c99d
Core tap last commit: 63 minutes ago
Core tap branch: master
HOMEBREW_PREFIX: /usr/local
HOMEBREW_CASK_OPTS: []
HOMEBREW_MAKE_JOBS: 16
Homebrew Ruby: 2.6.3 => /usr/local/Homebrew/Library/Homebrew/vendor/portable-ruby/2.6.3_2/bin/ruby
CPU: 16-core 64-bit penryn
Clang: 12.0.0 build 1200
Git: 2.24.3 => /Library/Developer/CommandLineTools/usr/bin/git
Curl: 7.64.1 => /usr/bin/curl
macOS: 10.15.7-x86_64
CLT: <IP_ADDRESS>.1.1610135815
Xcode: N/A

brew doctor output
Please note that these warnings are just used to help the Homebrew maintainers
with debugging if you file an issue. If everything you use Homebrew for is
working fine: please don't worry or file an issue; just ignore this. Thanks!

Warning: Some installed kegs have no formulae!
This means they were either deleted or installed with `brew diy`.
You should find replacements for the following formulae:
  mulle-clang-project

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
Create a bottle for a local formula.
What happened (include all command output)?
Building the local formula works
brew install --verbose --formula --build-bottle mulle-clang-project.rb
...
==> Not running 'post_install' as we're building a bottle
You can run it manually using:
  brew postinstall mulle-clang-project

Building the bottle fails though:
brew bottle --force-core-tap mulle-clang-project.rb
==> Determining mulle-clang-project bottle rebuild...
Error: undefined method `path' for nil:NilClass
Did you mean?  paths
Please report this issue:
  https://docs.brew.sh/Troubleshooting
/usr/local/Homebrew/Library/Homebrew/formula_versions.rb:25:in `initialize'
/usr/local/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:314:in `new'
/usr/local/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:314:in `bottle_formula'
/usr/local/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:105:in `block in bottle'
/usr/local/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:104:in `each'
/usr/local/Homebrew/Library/Homebrew/dev-cmd/bottle.rb:104:in `bottle'
/usr/local/Homebrew/Library/Homebrew/brew.rb:122:in `<main>'

What did you expect to happen?
A bottle to appear.
Step-by-step reproduction instructions (by running brew commands)
cat <<EOF > mulle-clang-project.rb
class MulleClangProject < Formula
  desc "Objective-C compiler for the mulle-objc runtime"
  homepage "https://github.com/Codeon-GmbH/mulle-clang-project"
  license "BSD-3-Clause"
  revision 1
  head "https://github.com/Codeon-GmbH/mulle-clang-project.git", branch: "mulle/12.0.0"

  url "https://github.com/Codeon-GmbH/mulle-clang-project/archive/refs/tags/<IP_ADDRESS>-RC2.tar.gz"
  sha256 "a8dd4bd48839e4a842fc7631331c10d0f8eb3b8a952f301e1c5a92723e02a387"

  def vendor
    "mulle-clang <IP_ADDRESS> (runtime-load-version: 17)"
  end

  bottle do
    root_url "https://github.com/Codeon-GmbH/mulle-clang-project/downloads/<IP_ADDRESS>-RC2"
  end

  depends_on 'cmake'   => :build
  depends_on 'ninja'   => :build

  #  compiler_rt doesn't build on macos
  def install
    mkdir "build" do
      args = std_cmake_args
      args << '-DLLVM_ENABLE_PROJECTS=libcxxabi;libcxx;clang'
      args << '-DCMAKE_BUILD_TYPE=Release'
      args << '-DCLANG_VENDOR=mulle'
      args << '-DCMAKE_INSTALL_MESSAGE=LAZY'
      args << "-DCMAKE_INSTALL_PREFIX='#{prefix}/root'"
      args << '../llvm'

      system "cmake", "-G", "Ninja", *args
      system "ninja", "install"
    end
    bin.install_symlink "#{prefix}/root/bin/clang" => "mulle-clang"
    bin.install_symlink "#{prefix}/root/bin/scan-build" => "mulle-scan-build"
  end

  def caveats
    str = <<~EOS
    To use mulle-clang inside homebrew formulae, you need a shim.
    See:
       https://github.com/Codeon-GmbH/mulle-clang-homebrew
    EOS
    str
  end

  test do
    system "#{bin}/mulle-clang", "--help"
  end
end
EOF

brew install --verbose --formula --build-bottle mulle-clang-project.rb
brew bottle --force-core-tap mulle-clang-project.rb

Don't do it though, it will take hours to build.

Can you try putting mulle-clang-project.rb into a tap created with brew tap-new before installing it?
So it should be something like:
brew tap-new mulle-nat/clang
cp mulle-clang-project.rb $(brew --repo mulle-nat/clang)/Formula
brew install --build-bottle mulle-clang-project
brew bottle mulle-clang-project

When I made a local tap, like you suggested, then I got it to work.

Glad it worked. I still think something is wrong here, even if it's just that the --force-core-tap flag should be removed. We shouldn't have an option do something that doesn't work.

Just being able to use a local file without a tap, was somewhat more convenient.

I still think something is still wrong here, even if it's just that the --force-core-tap flag should be removed.

It's a useful developer option. It should probably be hidden from the manpage and require HOMEBREW_DEVELOPER to be set.


Feature Request: IO Throttling or QoS

Need to manage the IOPS/Bandwidth of  a volume access

Related Technology:
To limit the disk access speed of Linux logical volumes, the cgroup feature of the Linux kernel can be used. With the iteration of kernel versions, cgroup has also been developed into two versions: v1 and v2.
The way of limiting I/O has changed, but regardless of which version, device-related information (major and minor version numbers) needs to be provided when limiting Linux logical volumes. After setting up the root cgroup of the operating system, when the node is restarted, the relevant configuration will be reset. It is necessary to flush the configuration back through other means.
v1:

v2:

Other Similar project
Design doc
Code

open-local
This scenario is limited to Direct-IOThis solution sets the IO of LogicalVolume by adjusting the I/O Throttling Tunable Parameters of cgroupv1.See https://github.com/alibaba/open-local/blob/main/docs/design/io-throttling.md for details. It supports both systemd and cgroup. The related I/O Throttling parameters are set on the storageclass .
cgroupPath: https://github.com/alibaba/open-local/blob/main/pkg/utils/cgroup_driver.go#L31setIOThrottling: https://github.com/alibaba/open-local/blob/c01fc51c2832d378493b420c336451a41ef95415/pkg/csi/nodeserver.go#L593

topolvm
Does't support IO throttling. In brief, the major contributor disagree the feature, because he think that it would be better to implement on CRI than on a CSI driver.  See https://github.com/topolvm/topolvm/pull/564#issuecomment-1252057431 for details.
This pr is closed. https://github.com/topolvm/topolvm/pull/564/files#diff-0db16276644fc4f21eae05d761a230f6438feca8b7c0a92427c9d3b3ea85726aR255 The related I/O Throttling parameters are set on the pod .


[CLI] Error: Update returned exit code not zero

saltstack.log
uname -a output:
Linux sift000 4.15.0-112-generic #113~16.04.1-Ubuntu SMP Fri Jul 10 04:37:08 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
LC_ALL=en_US.UTF-8 sudo sift install

<EMAIL_ADDRESS>sift-version: notinstalled

mode: desktop
downloading v2021.4.0

downloading sift-saltstack-v2021.4.0.tar.gz.asc
downloading sift-saltstack-v2021.4.0.tar.gz.sha256
downloading sift-saltstack-v2021.4.0.tar.gz.sha256.asc
downloading sift-saltstack-v2021.4.0.tar.gz
validating file sift-saltstack-v2021.4.0.tar.gz
validating signature for sift-saltstack-v2021.4.0.tar.gz.sha256
saltstack.log

extracting update sift-saltstack-v2021.4.0.tar.gz
performing update v2021.4.0

Log file: /var/cache/sift/cli/v2021.4.0/saltstack.log

Update returned exit code not zero
Error: Update returned exit code not zero
at ChildProcess. (/snapshot/sift-cli/sift-cli.js:547:23)
at ChildProcess.emit (events.js:315:20)
at maybeClose (internal/child_process.js:1021:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:286:5)
----- PLEASE READ ----------------------
A lot of failures are caused by the apt system being locked
or unhealthy.
Before opening an issue in GitHub, please check to see if
your apt system is healthy.
Try running 'apt-get update' then remove any packages that
aren't used by running 'apt-get autoremove'

Hi @l34rn1t , Ubuntu 16.04 is no longer a supported OS. Ubuntu 20.04 LTS is the latest supported OS, and would be recommended for your build of SIFT.

I'm getting this exact error that @l34rn1t is reporting above on ubuntu 20.04 LTS desktop on a fresh install.

Hi @Xboarder56 , can you attach your saltstack.log file so we can review it? The error that @l34rn1t received was for a different reason (different OS), so we'll need your log file to determine the cause of the error in your situation.

@digitalsleuth it's attached below. I tried doing an update and remove without any success either.
saltstack.log

@digitalsleuth it looks like there is still a reference to ubuntu 16.04 for the salt packages for powershell as the most likely cause.

@Xboarder56 Thanks for uploading the log file. Can you run cat /etc/os-release and post the results? The log you attached references states (sift.packages.powershell_xenial) which either no longer exist, or are based on the existing OS (sift.packages.powershell_{{ grains['oscodename'] }}). Somehow saltstack is getting 'xenial' from the environment when it shouldn't especially if you're using Ubuntu 20.04.

@digitalsleuth below:
sift@siftworkstation: ~
$ cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.2 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.2 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal
sift@siftworkstation: ~
$

Thanks. And can you also run salt-minion -V and salt-common -V? I should have put those in my last comment.

salt minion returned results, salt-common isn't installed at all.
 Just so you're aware, v3003 of saltstack (common/minion) are not yet supported by SIFT. One easy fix would be to remove 3003 and install 3001, and trying again to see if that works. It's possible there might be an issue caused by the newer version, but I'm going to have to do a little testing to see if that's the issue.
Additionally, salt-common should be installed by the sift-cli by default. Did you use the CLI to install these states, or did you use the salt-minion command?

Just replied above, but it is installed just the command isn't valid
$ sudo apt-get install salt-common
Reading package lists... Done
Building dependency tree
Reading state information... Done
salt-common is already the newest version (3003+ds-1).
0 upgraded, 0 newly installed, 0 to remove and 4 not upgraded.
sift@siftworkstation: ~
$ sudo su
root@siftworkstation:/home/sift# salt-commmon -V
salt-commmon: command not found
root@siftworkstation:/home/sift# salt-
salt-call    salt-minion  salt-proxy
root@siftworkstation:/home/sift# salt-
salt-call    salt-minion  salt-proxy
root@siftworkstation:/home/sift#

Strange. Your common command did have too many 'm' 's though. But it's obvious to tell that the salt-(TAB) didn't show the salt-common.

Just reverted back and got the same error during install. Running 3001.7 now for both minion and common. Really strange it's trying to grab that old package.
apt-get install salt-common=3001*
apt-get install salt-minnion=3001*

You only need salt-common. salt-call --versions-report

salt-call --versions-report
Salt Version:
           Salt: 3001.7

Dependency Versions:
           cffi: Not Installed
       cherrypy: Not Installed
       dateutil: 2.8.1
      docker-py: Not Installed
          gitdb: Not Installed
      gitpython: Not Installed
         Jinja2: 2.11.3
        libgit2: Not Installed
       M2Crypto: Not Installed
           Mako: 1.1.4
   msgpack-pure: Not Installed
 msgpack-python: 0.6.2
   mysql-python: Not Installed
      pycparser: Not Installed
       pycrypto: 2.6.1
   pycryptodome: 3.10.1
         pygit2: Not Installed
         Python: 3.8.5 (default, Jan 27 2021, 15:41:15)
   python-gnupg: 0.4.5
         PyYAML: 5.3.1
          PyZMQ: 22.0.3
          smmap: Not Installed
        timelib: Not Installed
        Tornado: 4.5.3
            ZMQ: 4.3.4

System Versions:
           dist: ubuntu 20.04 focal
         locale: utf-8
        machine: x86_64
        release: 5.8.0-48-generic
         system: Linux
        version: Ubuntu 20.04 focal

root@siftworkstation:/home/sift#

The logs here https://github.com/teamdfir/sift/issues/514#issuecomment-812179829 failed on a powershell install, it resolved to install for xenial, but clearly you are on 20.04.

@ekristen is there a way to manually install powershell for it to continue the reset of the installation.

You can install Powershell manually, but that state will still try to install it for Xenial until we figure out why it's looking for xenial instead of focal.

Can you please do another install and provide the log.

@ekristen I moved the old log to a new file, and just executed the install again.
saltstack.log

Looks like your apt database is busted and possible issues with python too.
Please run an apt-get update && apt-get upgrade.
Was this an upgraded OS? Like did you upgrade from xenial to focal? Did you previously have sift installed?

no this is a pure fresh install from an ISO 20.04 and upgraded to 20.04.2
command results below:
root@siftworkstation:/var/cache/sift/cli/v2021.4.1#
root@siftworkstation:/var/cache/sift/cli/v2021.4.1# apt-get update && apt-get upgrade
Hit:1 https://download.docker.com/linux/ubuntu focal InRelease
Hit:2 http://repo.saltstack.com/py3/ubuntu/20.04/amd64/3001 focal InRelease
Hit:3 http://us.archive.ubuntu.com/ubuntu focal InRelease
Get:4 http://us.archive.ubuntu.com/ubuntu focal-updates InRelease [114 kB]
Hit:5 http://ppa.launchpad.net/gift/stable/ubuntu focal InRelease
Get:6 http://security.ubuntu.com/ubuntu focal-security InRelease [109 kB]
Hit:7 http://archive.ubuntu.com/ubuntu focal InRelease
Hit:8 https://repo.saltproject.io/py3/ubuntu/20.04/amd64/latest focal InRelease
Get:9 http://archive.ubuntu.com/ubuntu focal-security InRelease [109 kB]
Hit:10 http://ppa.launchpad.net/openjdk-r/ppa/ubuntu focal InRelease
Get:11 http://us.archive.ubuntu.com/ubuntu focal-backports InRelease [101 kB]
Hit:12 http://ppa.launchpad.net/sift/dev/ubuntu focal InRelease
Fetched 433 kB in 1s (429 kB/s)
Reading package lists... Done
N: Skipping acquire of configured file 'main/binary-i386/Packages' as repository 'https://repo.saltproject.io/py3/ubuntu/20.04/amd64/latest focal InRelease' doesn't support architecture 'i386'
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
The following packages have been kept back:
  python3-cryptography ubuntu-desktop ubuntu-desktop-minimal ubuntu-drivers-common
The following packages will be upgraded:
  salt-common salt-minion
2 upgraded, 0 newly installed, 0 to remove and 4 not upgraded.
Need to get 0 B/7,156 kB of archives.
After this operation, 225 kB of additional disk space will be used.
Do you want to continue? [Y/n]

@Xboarder56 Out of curiosity, did you use the Ubuntu Server ISO or the Desktop ISO? And if server, did you install the desktop environment, and which one did you choose?

I used the standard desktop iso. I can get you a hash of it @digitalsleuth

No that's good. I don't doubt you're using the right one, I'm just trying to duplicate the error to figure out what happened. I'm using the <IP_ADDRESS>-desktop ISO and no problem.

ubuntu-20.04-desktop-amd64 MD5: EA28C4FD933BE55F9F01A5FA9E868490
just in case. I did a default installed username was sift, and I did install updates and 3rd party add ons.

@Xboarder56 I'm trying to duplicate the issues, but something doesn't seem to fit, and I'm wondering if you can help me sort out some problems. Saltstack isn't installed in the default Ubuntu 20.04 release, and on a clean install, the sift-cli installs the 3001 release. You stated at the beginning, after posting your saltstack.log, that you had version 3003 installed. The log you provided at the time also had Xenial references to it, which it ideally shouldn't.
Can you walk me through, step by step, what process you followed?
I've looked at the first three saltstack.log files here (https://github.com/teamdfir/sift/files/6235438/saltstack.log, https://github.com/teamdfir/sift/files/6235439/saltstack.log - both from @l34rn1t, and https://github.com/teamdfir/sift/files/6246529/saltstack.log from you) and all three of those have the same MD5. The first two should have the same MD5 if uploaded from the same person at the same time. But the one you posted has the same MD5 as @l34rn1t's log files, and it shouldn't.
Perhaps this might be an accident, but it would help us if we could recreate the steps you took to duplicate the error you're receiving.

So I think I know why this occurred @digitalsleuth a fresh install did work correctly.
At the start I was gonna launch the salt stack version as a stand alone without the cli tools. So I went to salt stacks website and added the gpg key and installed salt stack manually using the below commands. Once this was done I manually installed salt stack minion using apt-get install salt-minion. Then I manually launched sift-salt-stack using the command listed in the repo and it just hung. That's when I transitioned over to the cli installation method.
sudo curl -fsSL -o /usr/share/keyrings/salt-archive-keyring.gpg https://repo.saltproject.io/py3/ubuntu/20.04/amd64/latest/salt-archive-keyring.gpg
# Create apt sources list file
echo "deb [signed-by=/usr/share/keyrings/salt-archive-keyring.gpg] https://repo.saltproject.io/py3/ubuntu/20.04/amd64/latest focal main" | sudo tee /etc/apt/sources.list.d/salt.list

That makes some sense. There have been some recent changes with the way saltstack handles their repo, (saltproject.io vice saltstack.com) and using the latest keyword in the installation installed the latest version (3003).
Glad to hear you got this resolved and have a working installation.

@Xboarder56 are we good to close this?

Yes @ekristen this is good To go. Thanks for the help guys.


Remove version number from README.md header

Now we already have a badge to indicated the version of latest release. So I think we should remove the version number from the header (Zcoin v<IP_ADDRESS> to just Zcoin). One of the benefit is it will reduce the tasks for each release.

I agree

I agree

make sense


⚠ TorrentLeech has degraded performance

In f7e48c6, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 4371 ms

Resolved: TorrentLeech performance has improved in a9fddd5 after 30 minutes.


No substitutions of url unsafe characters in database and document names.

Such checking only taking place in https://github.com/couchrest/couchrest/blob/master/lib/couchrest/database.rb#L29
    def initialize(server, name)       @name     = name       @server   = server       @path     = "/#{name.gsub('/','%2F')}"
This should be occured when trying to get database or document by its name at least for these symbols:
 / = %2F
 - = %2D

@magic2k this should be fixed now! We added some extra code in this commit: https://github.com/couchrest/couchrest/commit/4469ea5c77c7cff1d650c12629c90aa1e9cf679d Could you please confirm?

Thank you!


mesa: iris is not being built

Issue description
The iris gallium driver is not being built with mesa 19.1, unfortunately, as it is not currently included when building auto gallium_drivers.

see: https://nixos.wiki/wiki/Intel_Graphics#Iris


Remove tests for deep imports with name

The following tests can be merged, as they mean the same
Service import deep star:

https://github.com/awslabs/aws-sdk-js-codemod/blob/main/src/transforms/v2-to-v3/__fixtures__/new-client/service-import-deep-star.input.js
https://github.com/awslabs/aws-sdk-js-codemod/blob/main/src/transforms/v2-to-v3/__fixtures__/new-client/service-import-deep-star-with-name.input.js

Service import deep:

https://github.com/awslabs/aws-sdk-js-codemod/blob/main/src/transforms/v2-to-v3/__fixtures__/new-client/service-import-deep.input.js
https://github.com/awslabs/aws-sdk-js-codemod/blob/main/src/transforms/v2-to-v3/__fixtures__/new-client/service-import-deep-with-name.input.js

Service import equals:

https://github.com/awslabs/aws-sdk-js-codemod/blob/main/src/transforms/v2-to-v3/__fixtures__/new-client/service-import-equals.input.ts
https://github.com/awslabs/aws-sdk-js-codemod/blob/main/src/transforms/v2-to-v3/__fixtures__/new-client/service-import-equals-with-name.input.ts

These can be merged, as the local name is independent of the deep import path.

Refs: https://github.com/awslabs/aws-sdk-js-codemod/pull/674#issuecomment-1789346030


Adding a Stream to a password-protected zip file requires manually setting entryCRC on ZipParameters

Maybe there is no other choice.  But I had to dig pretty hard to discover that when I use net.lingala.zip4j.ZipFile.addStream(InputStream, ZipParameters), unless I do the following set of steps, I am unable to extract/open the file from within the password-protected zip file using the expected password.

Write the inputStream to a file
Use net.lingala.zip4j.util.CrcUtil.computeFileCrc(File, ProgressMonitor) to calculate the CRC value
Take the result of the previous step and pass it to ZipParameters.setEntryCRC()

I am not sure if I get your point. From what I understood, I tried the below code, and I am able to add stream to an already existing password-protected zip file without having to calculate crc from this code. Basically what I do here is:

Create a password protected zip file
Add a stream to this zip file (this stream also has a password)
Assert zip contents and Extract contents to make sure extraction is fine

@Test
  public void zipTest() throws IOException {
    File file = new File("/Users/srikanth/Downloads/test.zip");
    char[] password = "password".toCharArray();
    ZipFile zipFile = new ZipFile(file, password);
    ZipParameters zipParameters = createZipParameters();
    zipFile.addFile(TestUtils.getTestFileFromResources("sample.pdf"), zipParameters);

    addFileAsStreamToZip(TestUtils.getTestFileFromResources("sample_text1.txt"), file, password, zipParameters);

    extractZipFile(file, password);
  }

  private void addFileAsStreamToZip(File fileToAdd, File outputZipFile, char[] password, ZipParameters zipParameters) throws IOException {
    ZipFile zipFile = new ZipFile(outputZipFile, password);

    try(InputStream inputStream = new FileInputStream(fileToAdd)) {
      zipParameters.setFileNameInZip(fileToAdd.getName());
      zipFile.addStream(inputStream, zipParameters);
    }
  }

  private void extractZipFile(File zipFileToExtract, char[] password) throws ZipException {
    ZipFile zipFile = new ZipFile(zipFileToExtract, password);
    assertThat(zipFile.getFileHeaders()).hasSize(2);
    zipFile.extractAll(zipFileToExtract.getParent());
  }

  private ZipParameters createZipParameters() {
    ZipParameters zipParameters = new ZipParameters();
    zipParameters.setEncryptFiles(true);
    zipParameters.setEncryptionMethod(EncryptionMethod.AES);
    return zipParameters;
  }

As you can see I did not have to call CrcUtil.computeFileCrc() to add a stream, and I am able to the extract the zip file without any issues.

Closing issue as there is no further feedback. If you think there is still an issue, please reopen or create a new issue with more details. Thanks.


Fix SSH Tunnel Bug on macOS 10.12

This commit upgrades the mariadb-java-client library to fix the SSH
tunnel bug described in https://github.com/flyway/flyway/issues/1667
mariadb-java-client 1.4.5 has a bug that prevents a connection over an
SSH tunnel on macOS 10.12.

Relevant issue: https://github.com/flyway/flyway/issues/1667

Didn't actually run the testing, since getting the old versions of Java requires an Oracle account and jumping through a bunch of hoops.

I took some time to create an Oracle account and get the JVMs set up. The tests passed successfully with this change.

Also confirmed that this build fixes the connectivity problem.


pnpm

Hi, just curious whether there is a reason to use Yarn. Many people (including me) switched to pnpm because it's faster and more reliable (especially in mono repo). Yarn with its versions is confusing. I am asking because Solito proposes best practices. Thank you for your work.

Yeah pnpm seems great! As long as it works for you should go for it. I’ve tried migrating but had too many difficulties in my existing app.


Add the Discovery Endpoint to FFX E2E Tests

Add the discovery endpoint to E2E Test, so that we can run E2E Test once the discovery endpoint is imported.

@tianzhu007 just FYI, we are the Fluid Framework (FF) team, not FFX :)

@tianzhu007 just FYI, we are the Fluid Framework (FF) team, not FFX :)

Got it. Thanks for the correction!


Add skip-test plugin and test cases of first time login

Changes:

Add skip-test plugin ^1
Add method isFirstTimeLogin in LoginPage
Add dummy test cases of first time login

The snapshot of test case skipped:

isFirstTimeLogin would be the example for returning specific from cy's chainable.


Fix/code sandbox bugs

This PR fixes the issues brought up in #377.

Update the useFieldArray example in the advanced page.
Update CodeSandbox to most recent releases.

awesome! thanks @pmaier983 👍


Images CDN URL lost on state hydration (on Netlify)

Hello all,
I'm creating a little static website hosted on Netlify and I've chosen Hyperapp for its addictively elegant syntax and overall code simplicity. So far so good, I'm in love with the framework and I've also added a simple state hydration which works fine... except for a flickering on page load.
After debugging the issue – unless I'm doing something terribly wrong – it's 99% related to a feature at Netlify that replaces all the images URLs in the rendered HTML files with their CDN. Since the state is hydrated from the original URLs which are absolute paths (not the ones from Netlify which cannot be "prefetched"), the mismatch causes a repaint flickering which also causes a double loading of the images (the ones from the CDN and the "originals" from the hydrated state).
I guess this issue is not exclusively related to Hyperapp – it's more of a general thing – but I'd like to know if you have any ideas on how this could be solved with Hyperapp without disabling the CDN, if possible.
As a solution I was thinking to ignore certain tags/attributes from the diff, based on some filtering (a function maybe). The idea is to tell HA not to update a certain tag depending on a custom function that might check the tag "src" or any other attributes. It could also determine the updates by checking a state condition or serve to throttle a specific update/repaint. However I'm not sure if this can be done already.
My current implementation renders the static site pages using:
withRender(app)(state, actions, view).toString()
In the view, the hydrated state is set in a template tag (please note I'm not using the whole state in the actual code):
<template id="hydration" innerHTML={new Buffer.from(JSON.stringify(state)).toString('base64')}></template>
In the frontend, the original state is re-hydrated (patched) with the #hydration template (a base64 encoded JSON):
window.app = app(state, actions, view, document.body)
Thank you!

Hydration (recycling) of innerHTML is not supported, see https://github.com/hyperapp/hyperapp/pull/688
But you can use lifecycle events like this:
<template oncreate={(el) => { el.innerHTML = 'blabla' }} />

@frenzzy I've been fiddling with the oncreate event as you suggested but it seems to be fired only frontend and not via withRender().
About the CDN URLs, I've "fixed" them on the frontend by querying the DOM for all the images and patching the state accordingly (yeah, it's a dirty hack but it prevents the double images loading for now).
I see your point about the innerHTML not being re-hydrated correctly because the whole app's <div> repaints and – even though there is no flickering because the state is exactly the same – it delays the "time to interactive" (the images hack contributes to the issue).
Or maybe the repaint is a predicted behaviour during re-hydration even when the state match?
Thanks for the feedback!

Hi, @gyopiazza! Thank you for your issue and glad to hear you are using Hyperapp and liking it!
If you are having trouble with Hyperapp's built-in hydration capabilities as described in the Mounting section of the documentation, please create a new issue explaining the problem concisely and, if possible, include a short, self-contained, correct and reproducible demo (http://sscce.org).
If you believe the problem is not Hyperapp-specific, then please join our Slack and feel free to discuss it in the #general section.
If you are having trouble with hyperapp/render, please create an issue in that repository. Be aware that Hyperapp and HyperappRender only co-exist in the same organization. I don't officially support that package.
Cheers! 👋

@jorgebucaran there are a bug in hyperapp: it removes correctly generated markup during recycling (hydration) if you use innerHTML attributes (workaround for this issue is here).
Maybe you can re-consider merging PR #688 to fix the issue?
There are two test from the PR which are failed in the master branch.

I replied here https://github.com/hyperapp/hyperapp/pull/688#issuecomment-386862823.


[Bug] MySqlSyncTableAction should support case ignore mode

Search before asking

[X] I searched in the issues and found nothing similar.

Paimon version
0.4
Compute Engine
flink
Minimal reproduce step
Fields with upperCase sync to HiveCatalog.
There is an exception, see https://github.com/apache/incubator-paimon/pull/413
What doesn't meet your expectations?
Provide a mode to convert upperCase to lowerCase automatic.
Anything else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

I don't know if I understand it right, shouldn't we throw an exception but automatically convert to lowercase?