Send message to saved chat id not working

Nutgram version
Current
Describe the bug
When I subsantiate the class I then immediately send a message to a chat ID that I have made note of from a prior chat and the message does not show up in the thread on the telegram app
To Reproduce
use SergiX44\Nutgram\Nutgram;
$bot = new Nutgram($_ENV['TOKEN']);
$chatid = 123456;
$bot->sendMessage('Ciao!', ['chat_id'=>$chatid]);
Expected behavior
Sends message to chat
Screenshots
No response
Additional context
No response

Please give us more context.

What is the error you are receiving?
Send a screenshot of your code and the error

So I ran the code below and when I send command /start I can see the chat id, then I inserted that number into $bot->sendMessage('i am back', ['chat_id'=>6116504604]); and re-run the code and expect the 'i am back' message to appear once the code is re-run.
require '../vendor/autoload.php';
use SergiX44\Nutgram\Nutgram;

$bot = new Nutgram("");

$bot->onCommand('start', function(Nutgram $bot) {
   $bot->sendMessage('Ciao!');
   echo($bot->chatId());
});

$bot->run();
$bot->sendMessage('i am back', ['chat_id'=>6116504604]);

The idea was I would log to a database the chat_id and at anytime I could resend message back to the user.

If you are using the polling mode, the run method will be listening forever and you will not be able to do anything after it. You will have to use a parallel script or find another way.

OK so I tried this:
setRunningMode(Webhook::class);
$bot->sendMessage('Nice pic!', ['chat_id' =><PHONE_NUMBER>]);
$bot->run();

```
and it works to send the message and then I expect ->run() to run once and the script to exit....
but it thinks at the end that $bot is a string, not sure how $bot got assigned a string :-)

```
PHP Fatal error:  Uncaught Error: Call to a member function processUpdates() on string in /var/www/vhosts/onlysend.me/public_html/vendor/nutgram/nutgram/src/Nutgram.php:228
Stack trace:
#0 /var/www/vhosts/onlysend.me/public_html/tapptest/demo1.php(8): SergiX44\Nutgram\Nutgram->run()
#1 {main}
  thrown in /var/www/vhosts/onlysend.me/public_html/vendor/nutgram/nutgram/src/Nutgram.php on line 228
```

See this:
https://github.com/nutgram/nutgram/discussions/334


(FOR REVIEW ONLY) (SERVER-763) Read client auth info from tk-authz when configured

This commit changes the Ruby request handler to read client
authentication info from trapperkeeper-authorization when tk-authz is
enabled via the use-legacy-auth-conf: false setting.  Support for the
pre-existing method of getting this info - via X-Client headers or an
SSL certificate, depending upon how the master.allow-header-cert-info
setting is configured - is preserved in the case that no information can
be derived from tk-authz.  tk-authz is presumed to provide client
authentication info via an authorization map within the Ring request
map.

This won't pass Travis CI yet and isn't yet ready to merge because it depends upon a snapshot version of trapperkeeper-authorization.  In order for this PR to be ready to merge, We'll need to have merged https://github.com/puppetlabs/trapperkeeper-authorization/pull/17 and done a release of trapperkeeper-authorization that this PR can be bumped up to.  I've put this PR up in the meantime for early review.
Depending upon how long it takes for PR reviews to take place, I may add some additional docs-related commits with some updates to the language related to the allow-header-cert-info feature and tk-auth integration as well.  I'll add comments to this PR as I add any new docs commits.

Added some updates to the configuration.markdown and external_ssl_termination.markdown docs in a separate commit - 5550e07.

Rebased and updated to trapperkeeper-authorization 0.1.4 in d12449b.  Should be ready to review for possible merge now.

:+1: other than minor comments

@nwolfe Addressed your PR feedback - typos and the doseq/when thing - in 9f79699.

:+1: Looks good, just had that one little "actually" comment that's not even remotely a blocker here


Handle checkbox with phx-click but no form

Checkboxes (and other inputs) may be used outside of forms to trigger phx-click and phx-change events that are specified on the input (not on the form).
Either

check/2 could support this use case or (preferred)
add a dedicated click_checkbox/2

In both cases, the name attribute should not be required.
Current behaviour
Calling check/2 on a checkbox outside of a form results in
     ** (ArgumentError) Could not find "form" for an element with selector "#my-checkbox-id".

     Found other potential "form":

If there is no name attribute on that checkbox, it fails even earlier:
     ** (FunctionClauseError) no function clause matching in Regex.scan/3

Yeah, this is definitely a use case we should support. Thanks for opening the issue!

I think this was resolved by https://github.com/germsvel/phoenix_test/pull/116. I'm going to close it. If you think it's not addressed, happy to reopen it.


Idea: Bottom-up prover statement selection: forbidden, allowed, required

I'm not sure I understand what selecting statements in the bottom-up prover does.
That said, I have a thought. Perhaps this should be a 3-way option: Forbidden, Allowed, Required.
In particular, mmj2 allows specification of required statements at the first level, and this is often useful to force the use of specific statements -  making searching much faster and making it more likely you'll find the proof you wanted. You might also add a button like "require use of previous statement" since in many cases you're adding a statement to extend on a previous one.
I haven't thought through all the ramifications, and there may be another/better way. But I thought it'd be better to record the thought before it disappeared from my head, and perhaps this idea will point the way to a good/better idea.

This may not be all that critical since you can specify other things, but it makes sense to put this in the "hopper of ideas" to see if anyone thinks it's important.


Fix weighted sum calculation in xgboost._parent_value

As pointed out by @SauceCat, np.sum is needed here, not np.mean.
Also add tests that check that score and proba match.
Fixes #155
TODO:

[x] think of a way to test it?
[x] update xgboost tutorial

Current coverage is 97.23% (diff: 100%)

Merging #156 into master will not change coverage

@@             master       #156   diff @@
==========================================
  Files            37         37
  Lines          2246       2246
  Methods           0          0
  Messages          0          0
  Branches        438        438
==========================================
  Hits           2184       2184
  Misses           31         31
  Partials         31         31

Diff Coverage
File Path

•••••••••• 100%
eli5/xgboost.py

Powered by Codecov. Last update 46cea8b...3f0eb7b

@kmike I added a direct test, updated the tutorial and added release notes in 04a05d1 - this is ready for review now :)

Thanks @lopuhin for the fix and @SauceCat for the catch!


The README.md improvements

Addressed issues:

the driver usage example PHP code is missing -> added an example;
the Selenium 2 version is used -> using Selenium 3 version.

Could you please also add a section that would explain the difference between this project and minkphp/MinkSelenium2Driver because it is quite hard to understand what "classic" means exactly and what the roadmap looks like.
I'm coming from Drupal and Behat, and Mink driver is not directly used, so I'm just trying to wrap my head around what this classic driver will replace and whether that replacement be a drop-in or if we need to re-write some other system.

We could also update the example selenium version to 4, since this driver actually supports it.

@uuf6429 , done.

I'm a fan of running README code in tests, to ensure that that code remains executable and up-to-date with our changes. If we can agree on this, I can create an issue to set that up later.

@uuf6429 , Not sure how you can verify, that code actually works considering that:

we're area using a non-existing URL in the example;
the actions we're doing on the page also are non-real;
executing example code from within a test suite isn't the same as the real user is executing it (standalone PHP file), where no classes are auto-loaded yet.


Mobile wiki page commands don't work

https://github.com/golang/go/wiki/Mobile
What version of Go are you using (go version)?

$ go version
go version go1.18.1 linux/amd64

Does this issue reproduce with the latest release?
Probably.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/alex/.cache/go-build"
GOENV="/home/alex/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/alex/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/alex/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/lib/go-1.18"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go-1.18/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18.1"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build1987326445=/tmp/go-build -gno-record-gcc-switches"

What did you do?

Went through the guide on the wiki page: https://github.com/golang/go/wiki/Mobile
$ gomobile init
gomobile: command not found

had to do echo 'export PATH=$PATH:~/go/bin' >> ~/.bashrc && . ~/.bashrc (from https://github.com/golang/go/issues/21818 comments). Didn't see this documented anywhere.
$ go get -d golang.org/x/mobile/example/bind/...
go: go.mod file not found in current directory or any parent directory.
        'go get' is no longer supported outside a module.
        To build and install a command, use 'go install' with a version,
        like 'go install example.com/cmd@latest'
        For more information, see https://golang.org/doc/go-get-install-deprecation
        or run 'go help get' or 'go help install'.

 matched no packages
$ go install golang.org/x/mobile/example/bind/
go: 'go install' requires a version when current directory is not in a module
        Try 'go install golang.org/x/mobile/example/bind/@latest' to install the latest version
$ go install golang.org/x/mobile/example/bind/@latest
go: golang.org/x/mobile/example/bind/@latest: argument must be a clean package path

Still not sure what to do at this point.
What did you expect to see?
I expected the commands in the wiki page to work correctly out of the box.
What did you see instead?
They did not work out of the box, and no documentation is provided for how to solve the issues encountered. Just a poor experience for new users overall.

According to the provided messages, it seems that gomobile is not yet installed on your environment, try:
go install golang.org/x/mobile/cmd/gomobile@latest

This is described in https://github.com/golang/go/wiki/Mobile#tools


IEnvironmentService#userHome is deprecated

I recently cleaned up the environment service for web and one outcome is that userHome is actually not known upfront until resolved later from the remote in web. For that case we do have a IRemotePathService#userHome that checks the remote and falls back to the local user home if possible.
I would like people to adopt the remote path service if possible. One catch is that the method is async since it needs to resolve the remote environment if there is one.
Outcome of this should be that no-one depends on IEnvironmentService#userHome anymore.
References:

linkDetector @isidorn
loadedScriptsView @isidorn
extensionTipsService @sandy081
queryBuilder @roblourens
terminalProcessManager location1 terminalProcessManager location 2 @Tyriar
simpleFileDialog @alexr00
labelService @isidorn

@bpasero In ExtensionTipsService, it is being used to check local installables. I already have a check for web and do not do these checks. I only want to look for installables on local machine and I do not want to check installables on remote environment. Any suggestions how to go forward?

Hm, why would we not show an extension tip checking for installed tools on the remote?

We cannot treat all remote environments same. For eg., Dev containers what does it mean looking for installables in it? Does it makes sense?
I also think it is not intended to look for different kind of installables in the remote machine and prompt them in the remote window which does not need them at all. I see remote window is scoped to a workspace.

If you feel that web/remote should not have these tips, then I suggest to split the service into 2: web and native and only do the one in the one case.

@bpasero labelService depends on this being sync, if we change this method to be async than all callers of labelSerivce.getUriLabel will have to become async and that pretty much means the whole world. Is there an alternative approach here?

@isidorn I guess, the alternate approach is to have sync access where userHome essentially becomes URI | undefined, that is it will be undefined until resolved (probably indicated with some event)?

@bpasero that makes sense. If you introduce that URI | undefined call and the event in the remotePathService I will happily start using it.

@isidorn I added IRemotePathService#userHomeSync

@bpasero thanks.
@weinand fyi since I did changes in loadedScriptsView

Did this in the QueryBuilder - tried to make it async to do this properly but it spiraled out of control.

@bpasero what is the preferred way to get the local userHome even when there is a remote userHome now? For the simple file picker I need both.

@alexr00 you can always depend on INativeEnvironmentService to get the userHome which will be of type URI and never undefined. My plan is to remove userHome from the normal IEnvironmentService and only have it in the native part. That is the only place where it is and can be defined.

Since we are getting close, assigning myself for final cleanup and moving to April.

FYI my refactor broke the SSH extension as local terminals need to be launched before the remote connection is available. Moving to userHomeSync.

@bpasero I had to move some things around to not break layering and use INativeEnvironmentService in simplefiledialog.ts. Pull request https://github.com/microsoft/vscode/pull/95198

Simple file dialog is done.

I decided to go with the following solution:

IRemotePathService is renamed to IPathService and should be used in all cases where you want the correct path behaviour for both remote and local environments
there is userHome which is Promise<URI> that will always be defined and either be the remote user home or local (even for web serverless there will be a good fallback: vscode-remote:<remote authority>/)
there is resolvedUserHome which is of type URI | undefined that is being resolved on startup but maybe undefined until resolved

There is a slight semantic change from the old behaviour: if you used to use IEnvironmentService#userHome and now switched to IPathService#resolvedUserHome you will get an undefined user home on VSCode desktop, even if a local userHome is present for as long as the remote environment is not resolved. If you must fallback to the local userHome, you need to use the native variants of the environment service yourself.
//cc @isidorn @alexr00 @Tyriar @roblourens for potential customers
PS: noticed quite some usages of userHome.fsPath, I think that is not 100% correct because fsPath might not work well with remote URIs. I think fsPath should only be used for file URIs.


Table cell alignment...

Hi There,
In this section:
https://docs.microsoft.com/en-us/security/compass/privileged-access-deployment#finish-workstation-profile-hardening
The link for the Enterprise Hardening PowerShell script is in the wrong table cell. I would align it and make a pull request, but since I no longer have access I can't do that myself.
Friendly Regards
Stefan B. Christensen
Former MS PFE/CE

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

ID: 6fc109e7-e2ad-6e3c-598d-36876b40dbfd
Version Independent ID: da56e91b-41ca-2c29-0e8c-bc4d2a36a748
Content: Deploying a privileged access solution
Content Source: compass/privileged-access-deployment.md
Service: security
Sub-service: ****
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Found out that I do have access... the link just wasn't directly on the page, so closing and have made pull request with the change in it.


META-INF/services/ files are incorrect in shaded google-cloud-nio jar

The google-cloud-nio-0.56.0-alpha-shaded.jar doesn't correctly update the services files for shaded dependencies.
Specifically the two files
META-INF/services/org.threeten.bp.zone.TzdbZoneRulesProvider
and
META-INF/services/com.fasterxml.jackson.core.JsonFactory should probably be updated to point to the shaded version of those classes instead.

Yes it is. The two files are still pointing at the non-shaded version of those classes in the latest google-cloud-nio on maven (0.74).


编译错误

go build -ldflags "-X main.GitCommit=git rev-parse --short HEAD -X main.Version=0.2.1" -o open-falcon
can't load package: package main.GitCommit=6eae777: cannot find package "main.GitCommit=6eae777" in any of:
/usr/lib/go-1.10/src/main.GitCommit=6eae777 (from $GOROOT)
/home/xxx/go2/src/main.GitCommit=6eae777 (from $GOPATH)
can't load package: package -X: cannot find package "-X" in any of:
/usr/lib/go-1.10/src/-X (from $GOROOT)
/home/xxx/go2/src/-X (from $GOPATH)
can't load package: package main.Version=0.2.1: cannot find package "main.Version=0.2.1" in any of:
/usr/lib/go-1.10/src/main.Version=0.2.1 (from $GOROOT)
/home/xxx/go2/src/main.Version=0.2.1 (from $GOPATH)
can't load package: package -o: cannot find package "-o" in any of:
/usr/lib/go-1.10/src/-o (from $GOROOT)
/home/xxx/go2/src/-o (from $GOPATH)
can't load package: package open-falcon: cannot find package "open-falcon" in any of:
/usr/lib/go-1.10/src/open-falcon (from $GOROOT)
/home/xxx/go2/src/open-falcon (from $GOPATH)
Makefile:51: recipe for target 'open-falcon' failed
make: *** [open-falcon] Error 1
make all 编译时，报以上错误，请帮忙看看，谢谢。

解决了


Declare Dart 2.0 compatibility

@kevmoo can you review if this is how we generally want to update these?
We tested that this works fine with a 2.0 stable build (with the grpc package; except that it still fails as the deps of this package are not updated: https://gist.github.com/sortie/3f1a42b921ee304ea12514076670d223)

Good point, incorporated


I have a bug

Hi, I found a bug, which I cannot share today (still waiting for clearance on code publication), but I can desribe it.
As long as I put 3 views in LinearLayout (Horizontal), I get them all drawn one after another, everything works fine.
But If I put it to Horizontal(View1, Vertical(View2, View3)), the View3 (edit_view) gets palette colors mixed. Basically the background and font color set to black, no matter which color theme/palette I use.
If you have an idea what this might be, let me know. If not, I will try to do a minimal repro and push it during weekend.

Actually changing the order on LinearLayouts retains the error.

https://github.com/njskalski/bug2 it's BoxView issue.

This is what I see when running the bug2 project:

The left half is the tree view. The right half is the SelectView on top, and an EditView on the bottom.
The EditView is single-line only (if you want multi-line, you'll need a TextArea).
There is a bug when the window is too small, where the EditView is drawn over the shadow:

It should either be drawn within the window frame, or not be drawn at all. But that's a degenerate case where the layout cannot fit in the given space.
What is the problem you're describing?

Well it looks like the second example. The letters in edit view seem to be
hidden. That’s so weird. I thought I have disabled the shadows. Will look
more into it tomorrow. It’s 1am in Zurich.
On Thu, 24 May 2018 at 00:42, Alexandre Bury<EMAIL_ADDRESS>wrote:

This is what I see when running the bug2 project:
[image: screenshot from 2018-05-23 15-37-31]
https://user-images.githubusercontent.com/564903/40454897-6c9b13de-5e9f-11e8-8bce-75ba8421600e.png
The left half is the tree view. The right half is the SelectView on top,
and an EditView on the bottom.
The EditView is single-line only (if you want multi-line, you'll need a
TextArea).
There is a bug when the window is too small, where the EditView is
drawn over the shadow:
[image: screenshot from 2018-05-23 15-41-21]
https://user-images.githubusercontent.com/564903/40454987-c4a82f30-5e9f-11e8-84d9-e1179b6190ba.png
It should either be drawn within the window frame, or not be drawn at all.
But that's a degenerate case where the layout cannot fit in the given space.
What is the problem you're describing?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gyscos/Cursive/issues/252#issuecomment-391522325, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABS_5xM-iJDEBOQvcVvR9641krw3eisdks5t1eXhgaJpZM4ULPqV
.

So in the second example, there is not enough space to display the content (since you specified no-compromise fixed sizes in BoxView). This means the LinearLayout cannot find a working layout, so it tries a "best effort" solution. In this case it tries to print the EditView outside of the window frame, into shadow territory. This should not be allowed, and is a bug. The result is that instead of seeing the layer shadow, we see the EditView itself. It seems also that if you write content, the content of the editview is not drawn (this is the normal behavior) and we see the shadow.
So apparently when drawing the background of the EditView, somehow we bypass the usual rules and we're able to draw outside of the frame. This is a bug with the bound checks when drawing lines (as opposed to when drawing text). This is probably fixed on master, but I'll check to be sure.

Yeah, I made boxes smaller and it fixed the bug in my app. Thank you!


Can't set breakpoint on where the cursor stands.

When trying to click on the stepping cursor, I can't set a breakpoint there:
https://github.com/EpicGames/raddebugger/assets/63572671/89bca616-85f1-4af4-9d26-63386ceef95d

The way to do this currently is to either use the default binding of F9 (after closing the query bar), or to right-click the line and click "Place Breakpoint". I might end up splitting the columns in the future, such that the breakpoint column is always separate from the instruction pointer column, but there is kind of an unavoidable problem here in any case, which is that individual source lines tend to have a lot of information, and so you can't reserve the most compressed inputs (e.g. left-mouse-click) for everything you might want to do on any line.


babili deprecated

When installing the starter this deprecation warning appears:
npm WARN deprecated<EMAIL_ADDRESS>babili has been renamed to babel-minify. Please update to babel-preset-minify

@Tuizi Use the new library to replace the old one.

You don't want to fix that in this starter?

@Tuizi The warning is occur from babili-webpack-plugin and now it's renamed babel-minify-webpack-plugin. I used the new old so the warning should be solved.


To change the look of forms in login page

Want to change the look of the form part of the sign in and sign up page and add a bit of animation

Please check, this issue is already taken up.
I would suggest you to raise another issue.


Validate unique fields in the CICD

The name of an object like a key management is not defined by the file name (that has to be unique), but inside the object. Because of this we need to validate in the CI, that all names are unique in each resource. Two objects are allowed to have the same name, if they are in different resources (one is a key agreement, the other one is a credential format).
In case we find a twin, we stop the CI and throw an error.

this is already solved in the validate.mjs script


[otbn] Fix-up invalid lifecycle signal modelling in DV

1st commit: Introduces new_lc_mubi_err which signals to the golden model whenever we see an invalid lc_tx_t type signal we'd expect to immediately lock down with bad_internal_state error set.
2nd commit: Tweaks otbn_escalate test to inject random  lc_tx_t type signals more on the edge cases (from 95% while running to 40% while running and 60% sometime else)
3rd commit: Disables SecWipe related assertions when we see bad_internal_state fatal error because that specific fatal error does not guarantee a proper secure wipe behaviour.
4th commit: A small RTL change to make two modules behaviour consistent with each other. Detailed description is in the commit message. I would really appreciate a feedback from OTBN designers for this change specifically.
Resolves https://github.com/lowRISC/opentitan/issues/15505

I think we have some seeds passing with go_before configuration, with that we inject the faulty signals at the initial state. Although the chances are 33% for getting invalid MuBi value and 30% for injecting it at the initial state. And pass rate for go_before configuration is 40% which means we at least need to run approx 25-30 tests to hit the scenario that is currently shows up as a FSM transition hole (OtbnStartStopStateInitial->OtbnStartStopStateLocked). I believe you are right about go_after waiting until secure wipe finishes, so I think making that less likely shouldn't be a problem. I'm thinking about doing a 30/65/5 split + 30 iterations for this test. Does that makes sense to you?
I've just ran a 30 iteration regression with VCS T-2022.06 with 30/65/5 split. It shows current 3/4 holes getting covered (OtbnStartStopSecureWipeAllZero->OtbnStartStopStateLocked which needs direct tests because it's a state that lasts single cycle).

I'm thinking about doing a 30/65/5 split + 30 iterations for this test. Does that makes sense to you?
I've just ran a 30 iteration regression with VCS T-2022.06 with 30/65/5 split. It shows current 3/4 holes getting covered (Missing one is OtbnStartStopSecureWipeAllZero->OtbnStartStopStateLocked which needs direct tests because it's a state that lasts single cycle).

Sounds good, I think we'll hit the coverage goal without the OtbnStartStopSecureWipeAllZero->OtbnStartStopStateLocked transition.

@ctopal see the final two commits on my https://github.com/GregAC/opentitan/commits/otbn_escalate_cov branch. With those we're now hitting all of the FSM transitions (other than the OtbnStartStopSecureWipeAllZero->OtbnStartStopStateLocked transition and we make the coverage goal with it).
Could you take a look at cherry pick the commits in to this PR (though actually one only increases the test iterations so can probably just get merged in with the iteration change you've already made)? Haven't done it myself as I don't want to mess up your branch for dealing with Pirmin's feedback.


SQL error upgrading from 3.0.1 to <IP_ADDRESS>

I have just tried doing a completely fresh Concrete5 install, installed vivid_store 3.0.1, then tried to upgrade it to <IP_ADDRESS>, this happens every time:
An exception occurred while executing 'ALTER TABLE VividStoreProductVariationOptionItems ADD CONSTRAINT FK_50D9C3734BB971ED FOREIGN KEY (poiID) REFERENCES VividStoreProductOptionItems (poiID) ON DELETE CASCADE': SQLSTATE[HY000]: General error: 1215 Cannot add foreign key constraint
Are you able to reproduce this and fix/advise?

I haven't tested upgrading from 3.0.1 yet. If you're installing from git, I'd try a clean version of what's here. I just did a clean install that went well.

fixed.


feat: bsc p2p network

Fixes #7738
I had to add a method to set ForkId manually because of different bsc forks, let me know if there's a better way to do it

this is great!
this results in a lot of disconnects due to useless peer disconnect message, unclear why, but we're able to connect at least. So I'm merging this and maybe we can figure out why we get a ton of useless peer disconnects

Thank you! I will try to figure out why


Update recipe for v0.7.3

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

This is an attempt to fix the failure in https://github.com/conda-forge/loguru-feedstock/pull/38.
My downstream repo is failing because this hasn't been released (https://github.com/conda-forge/input4mips-validation-feedstock/pull/23) so I thought I'd try and help out with the workload.

@conda-forge-admin, please rerender

Attempt to fix failure in https://github.com/conda-forge/loguru-feedstock/pull/38

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12456482976. Examine the logs at this URL for more detail.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.


From private repo

@PRMerger15 : Thanks for your contribution! The author, @mlottner, has been notified to review your proposed change.


Discourse thinks urls to Github are spam

@flyingzumwalt I feel default anti-spam protections provided by Discourse should be eased up a little bit.
It is quite discouraging when someone refers to something at github, and gets flagged as spam 😕
Github resources are nearly always a valuable input.

Yeah, it wasn't because it was specifically referring to Github but rather that @diasdavid made two posts with three links to the same site, which triggered the spam filter. The limit has already been raised and shouldn't be triggered anymore. Also, the posts were flagged but not hidden/removed, so no action required there.
We should figure out other anti-spam values as a part of https://github.com/ipfs/community/issues/250, which we use as the issue for the progress of the forum.


Development hot-reload of Python package

Figure out if this is possible with Uvicorn

Oh well maybe another time, this is fine for now


Cannot build new install

When I try to run ./dc-build.sh I get error messages:
./docker/docker-compose-check.sh: line 3: docker-compose: command not found
./docker/docker-compose-check.sh: line 4: docker-compose: command not found
./docker/docker-compose-check.sh: line 5: docker-compose: command not found
Checking docker-compose version
is not supported docker-compose version, please upgrade to minimal supported version:1.28
If I do sudo docker version the Docker Engine Client is version 20.10.17 and the Docker Engine Server is 20.10.17. The Docker Compose version is v2.6.0. I am therefore way past the supported version.
The django and nginx images have also been pulled from dockerhub
https://hub.docker.com/r/defectdojo/defectdojo-django
https://hub.docker.com/r/defectdojo/defectdojo-nginx
I have tried other versions of Docker but seem to get the same issues.
Any help would be appreciated.

But it is reporting like docker-compose binary is not there.

Hi dsever
It says command not found. Is docker compose version not the newer version of docker-compose version? No docker-compose commands seem to work.
I basically did this to install Docker:
https://docs.docker.com/engine/install/debian/
Thanks

Yes, but you nees to install docker-compose as well. This instructions are installing docker only.

The instructions do say Docker Compose is installed with this command and as I said it displays Compose version is v2.6.0.
I have come across the solution and its on a completely different page and a different issue but it fixed the trouble I was having:
If the command docker-compose fails after installation, check your path. You can also create a symbolic link to /usr/bin or any other directory in your path. For example:
sudo ln -s /usr/local/bin/docker-compose /usr/bin/docker-compose
I hope it can help anyone else having the same issue.

This issue seems to be fixed.


Minor fixes

Thanks for the pull request.  Would you submit a CLA?  https://github.com/tumblr/genesis/blob/master/CONTRIBUTING.md has details.

@roymarantz  this one we can merge without the CLA. Going to go ahead and do that
@kant  Thank you for this.


[Feature] List uploaded artifacts (models) based on the metadata

Describe the solution you'd like
Like in e.g. MLFlow, it'd be cool if wandb allowed to list all the artifacts in the current project ("experiment") so as to download the best one based on e.g. the eval_accuracy stored as metadata of that run. This would imply a much cleaner and automated MVC (Model Version Control).
I think this could be a nice feature for wandb also to somehow "compete" against solutions like MLFlow, as this is one of the few things that I see missing to actually use wandb during the whole ML lifecycle.
Anyway, keep on the great work you're all doing! 🎉

Hey @alvarobartt we can definitely improve this interface, but this is something you can do today:
api = wandb.Api()
runs = run = api.runs("USERNAME_OR_TEAM/PROJECTS", filters={"created_at": {"$gt": '2020-12-07T00:20:40'}}, order="-summary_metrics.eval_accuracy")[0]
best_model = [a for a in runs[0].logged_artifacts() if model.type == "model"][0]
model_file = best_model.download()

Here are some more examples: https://docs.wandb.ai/guides/track/public-api-guide

WOW, cool! Thanks a lot @vanpelt I'll check that, but at least there's a workaround!
Anyway, I think that an easier way to manage the artifacts would be a HUGE improvement :tada:


Number System Calculator

I would like to add a different calculator which will be converting decimal numbers to Binary,Ternary,Octal,Hexadecimal and vice versa.
I would like @urvashi-code1255 @Vishal-raj-1 to please assign me this project.
I am a GSSOC'21 participant and want to contribute to this project.

I have completed it.. please review it.. also add gssoc21 tag

@Amisha-here  Good Work
Create a PR.

@Amisha-here it's a good project, you can send the PR : )

@Amisha-here Its been month, you haven't replied, so you have to reply within 2 days, else I have to close it.


Apple II+ 80 card emulation

This might be a long-shot, but would be possible to emulate the 80 column card for Apple II/II+ machines? (Over VGA of course)
It would possibly be pretty nice for the II/II+ owners out there to be able to get that nice razor sharp text for those machines.

This is planned, I just haven't gotten around to it yet. I wanted to do a lowlevel emulation of a videx compatible card and run the original ROMs from one, but due to the way most of those cards map the text buffer I don't think that a direct emulation will be possible. My next thought is to implement the same control characters and a simple "terminal" card firmware that just passes the raw character data over to the RP2040 to handle. This should be doable, I just have to implement a FIFO and character handler to parse the control characters (and only run the character handler while in vblank, or when there are no scanline buffers available to fill). The 6502 code should be quite small, other than the required input routine for lowercase.

I managed to implement the RP2040 side of 80 columns for II/II+ and need to write the 6502 firmware now to finish it.
Has most (all?) of the Videx control characters implemented on the RP2040 side, the 6502 side just has to stuff bytes into the FIFO at $C0nA, reading back that same address to get the FIFO fill level ($00 = empty / overflowed, $FF = full).
$C0n8 and $C0n9 are now the Clear / Set write registers for the 80column soft-switch.
I'll try to smash together some 6502 code this week.

Just curious, how is the progress on the 80col Videx emulation support? Is this still in the works?


fix: msvc and release is a build failure

When switching to release, it will show that this macro is not defined. It is a bit inconsistent with the definition here
https://github.com/Qt-Widgets/qtacrylicmaterial/blob/main/src/CMakeLists.txt#L62

I suspect this would be an incorrect fix :(
see also the template file from https://codereview.qt-project.org/c/qt/qtbase/+/356214

@moodyhunter But it does fail to compile (me too 😅)
Win10 21H2 + MSVC(2019) + Release

strange. but I'm on a holiday trip, will investigate a bit once I'm back home.

It seems that for a static build, QTACRYLICHELPER_STATIC isn't defined properly as it should be.

@XMuli can you try adding...
if(NOT BUILD_SHARED_LIBS)
    target_compile_definitions(${PROJECT_NAME} PUBLIC QTACRYLICHELPER_STATIC)
endif()

to src/CMakeLists.txt and try rebuilding?
Note the PUBLIC keyword is required to populate this macro define to a downstream user project (that are, the ones under examples/)

It is running successfully. @moodyhunter  thanks
But I am confused as follows:
https://github.com/Qt-Widgets/qtacrylicmaterial/blob/a04c19a8619305799ba034e4bf3c80bbae8067a6/src/CMakeLists.txt#L43
BUILD_SHARED_LIBS is ON by default and generates shared (not static), why does it become static under release?

I haven't experienced that before...
Can you try performing a clean build and verifying CMAKE_BUILD_TYPE and BUILD_SHARED_LIBS in CMakeCache.txt?

Okay, I'll try it when I get home tonight

The test is as follows（Win10 + VS2019 + CMake 3.3）
BUILD_SHARED_LIBS is OFF by default.
ON: Generate (.dll + .exp + .lib)
OFF: Generate (.lib)

Modified the commit to generate .dll by default
to avoid direct run failures and confusion


Specify version number once

Currently this has to be bumped in two places
https://github.com/DemocracyClub/dc_django_utils/commit/43ec52d2846b1b28cfaabd036841456c5f9a4466
For a "normal" package that you build and push to a registry, the way to do this would be to tell setuptools to get the version from __version__ at build time, like this:
diff --git a/pyproject.toml b/pyproject.toml
index 11d0a9c..876aa01 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -1,6 +1,5 @@
 [project]
 name = "dc_django_utils"
-version = "6.1.0"
 description = "This is a package to provide some common utilities for Django projects at Democracy Club."
 readme = "README.md"
 authors = [{name = "Sym Roe"}, {name = "Virginia Dooley"}]
@@ -17,6 +16,9 @@ dependencies = [
 ]

+dynamic = ["version"]
+[tool.setuptools.dynamic]
+version = {attr = "dc_utils.__version__"}

 [project.urls]
 Homepage = "https://github.com/DemocracyClub/dc_django_utils"

and then you can just bump the __version__.
I guess the reason you're not doing that on this project is because you don't build this package and push it to a registry. You just consume it as a VCS dependency. One thing you could do would be to actually build the package. If you want to stick with it being a VCS dependency, you could also drop the __version__ property. Or if you really want to keep it, you could go the other way and populate the __version__ property based on the version in pyproject.toml.
So
import tomllib

with open("../pyproject.toml", "rb") as f:
    pyproject = tomllib.load(f)

__version__ = pyproject['project']['version']

or whatever (not tested, but I reckon that should work).
Whichever way we go, it would be desirable to have one source of truth for the package version.

That said, over on https://github.com/DemocracyClub/dc_logging/
you're doing
https://github.com/DemocracyClub/dc_logging/blob/main/setup.cfg#L3
(which is the setup.cfg-land version of my first proposal) so maybe you're just relying on pulling in the tarballs from tagged releases?

done in https://github.com/DemocracyClub/dc_django_utils/pull/88


adding to changelog and release notes for 0.15 release

:+1: but consider changing the diff link

ah good call @btm. I often forget to pay attention to the diff link


get max RAM and GPU memory used

Is there a way to have the maximum RAM and GPU memory used on android by the MACE model inference?
System information

**OS Platform and Distribution: 19.10
**NDK version: r15c
**GCC version : gcc (Ubuntu 9.2.1-9ubuntu2) 9.2.1 20191008
**MACE version: v0.13.0-0-g9a06864a
**Python version: 3.7.5
**Bazel version: 0.13.1

@davidenitti Why do you want to use the maximum RAM and GPU memory?

I want to estimate the maximum memory (the peak usage) required by a model (if it's cpu model only cpu memory, if it's gpu model cpu and gpu memory).


Decrease data structure iterations in wrap_verifier

Remove unnecessary iterations over list/allocations of intermediate data structures in wrap_verifier

@rbonichon can you update or close this please?

In fact, this has been in draft for nearly a year. Closing, but feel free to update and re-open.


Include a default style sheet for atom-select-list in core

Refs: #13254
In atom-space-pen-views, on require, we call atom.themes.requireStylesheet to include a default style sheet for the select list. Now that we are planning to replace all usages of space pen with a more modern alternative (https://github.com/atom/atom-select-list), I believe a better design can be achieved by including a default style sheet in Atom core instead of relying on a require-time call to the ThemeManager.
This pull request also introduces a relatedTarget option in the synthesized blur event of TextEditorElement to make it easier for users of this API to know the value of document.activeElement after the event dispatch. We will use this in atom-select-list to determine if the user has clicked outside the select list and, if so, trigger a didCancelSelection event.
/cc: @atom/maintainers

In atom-space-pen-views, on require, we call atom.themes.requireStylesheet to include a default style sheet for the select list.

@as-cii Do you know the reason for including the select list styles when using space pen?
The select list styles are part of atom-ui that gets bundled with Atom. So they should already be available anyways.

Do you know the reason for including the select list styles when using space pen?

Not really, I imagine it has been there for a long time. It got added in https://github.com/atom/atom-space-pen-views/commit/c9151481219b43a3a680e01330476dd72f55d99b but the commits doesn't specify the reason.
Since we bundle that style sheet anyways, I am going to remove https://github.com/atom/atom/pull/13300/commits/0ba42b26b300cd199b5c1f21450eea066476fc00 from this pull request so that we can just merge https://github.com/atom/atom/pull/13300/commits/14092f1641ea9fe04eecd9279f3687352b0ca5e6 into master.
Thanks @simurai!


Including service connection(manual setup)

Including manual setup when creating a service connection.

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

Please remote the note about not supporting managed identity too.

@zhenlan Thanks for your review.
@Anne56njeri Thanks for incorporating feedback on your pull request.
@AlexandraKemperMS Could you review the proposed update?
Thanks, all.

#sign-off

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

Thanks, all.


Use api_version in all requests by default

Now version specified in vk_requests.create_api(..., api_version='5.45') or default version is not used in API requests (only in authorization). So to make requests working properly it's necessary to specify version in each request or in default args for requests: vk_requests.create_api(..., api_version='5.45', v='5.45').
It's ok but it'll be more straightforward to use version from api_version by default for API requests.

Totally agree. I'll fix it soon.
Or can you make PR for unittests dev branch spotting this case?
Anyway thanks for reporting.

fixed at v0.9.5


Fixes an issue where the feature flag of '0' causes a crash

addresses https://sentry.io/organizations/artsynet/issues/1351246099/?project=166818&query=is%3Aunresolved
In the TestFlight version  6.1.0 (2<IP_ADDRESS>) I noticed that updating a feature flag to false on https://echo-web-production.herokuapp.com/accounts/1/features/14 would cause a crash on the Confirm Bid screen in Emission. After some digging I noticed that the boolean false value in Echo would end up being 0 in the iOS world (which is understandable given that some languages treat 0 as a falsy value).
Normally this wouldn't cause any problems, but because of the way the conditional is written, this was causing a crash:
enablePriceTransparency && <Component>

this one-liner was returning 0 and React tried to render the 0 as a text anyway, but in React Native any text needs to be wrapped with a <Text> component, so RN just threw an error saying that you can't render 0 without wrapping it.
The intention of this code was to not render anything, so I just changed this line to the form of flag ? <Component> : null to fix it.
#skip_new_tests I tried to write a test but this isn't replicable because there is a platform difference jest and RN. React (the one that generates in-memory VDOM) doesn't care about text being present anywhere else while React Native (the one that maps VDOM to native views in iOS/Android) doesn't like them.

:rocket: PR was released in v1.18.39 :rocket:


[Bug Report] If the computed time in the computed property is long, el-select's label does not display right property.

Element UI version
2.4.9
OS/Browsers version
win10/ chrome
Vue version
2.5.17
Reproduction Link
https://jsfiddle.net/turai/em76khbc/72/
Steps to reproduce
Prepare a list that takes time to calculate in the list displayed by el-option and put it in computed.
What is Expected?
The correct expected result is that label is correctly displayed in the upper part el-select and the lower part el-select.
What is actually happening?
I made a variable that determines the item of el-option in the computed item of vue.js.
If the calculation time to determine the computed variable is long, it seems that the value displayed in el-select is not the value of label, but the value of value is chosen.
The elapsed time of the elapsed portion of the reproduction item pasted on the link takes time to filter.
On the other hand, since the el-option in the lower part is simply returning the value stored in data, the calculation time is short.
For this reason, label is not displayed in el-select in the upper part, and label appears correctly in el-select in the lower part.

Emmmm，can you get more additional details for Steps to reproduce like screenshots? I think the performance is expected

I found a mistake in my code.
Since there is no value to reference in the list, it is common not to display it. That is a my careless mistake.
This is not my bug but my mistake. I am sorry I opened a place like this. It will be helpful if you close this page.


Added lexer for Janet language.

Janet syntax is lisp-like, and also has similarities with Clojure (one main difference from both is that Janet uses # for comments instead of ;).

Hi Anteru, thanks so much for your review work, and patience here. My free time as almost completely dried up in the last few weeks, and I'm looking up the git docs now on how to proceed (my git knowledge is currently very thin!).

Anteru, does this apply to the regexes for both double-quoted and for backtick-quoted strings? Note that, unlike double-quoted strings, backtick-quoted strings may utilize multiple backticks, as in
  `some str`
 ``some str``
```some str```

Incidentally, both double-quoted and backtick-quoted strings can be multi-line.

I only applied it to double-quoted strings. Do backtick-quoted strings allow for escape characters inside? At some point it may be easier to mimic what the Python lexer does and have separate rules for handling the various escapes.
Also, no need to hurry, we just released 2.7 and 2.8 is not coming that soon :)

No, backtick-quoted strings do not allow for escape sequences.

But backtick quoted strings do allow for:
``this is `something` here``
```this is ``something`` here```

The language allows for even more backticks (as long as there's the same number of them starting as ending the string), but so far the most I've seen people use is
```three```

My advice would be to look at the Python lexer family and copy that, using three states -- for single, double, triple quoted backticks. Same as the string handling there -- that'll be much less hassle than any other approach IMHO.

Sorry I dropped the ball on this. Unable to make time to look more at this right now, and my git knowledge needs to be better so I can get this current before continuing. I'd like to return to it when I can, but if someone wants to pick up the ball and run with it that's fine with me.


Add "Last Modified" column to all list views

Use a relative timestamp instead of absolute (eg. 22h ago)

reopen this with each page that needs @russorat


merge from konsum-frontend-svelte

package.json

chore(deps): add ^3.1.2 remove ^3.1.1 (devDependencies.vite)

src/manifest.webmanifest

chore: add . (start_url)

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

npm package (@latest dist-tag)
v1.34.10
GitHub release

Your semantic-release bot :package::rocket:


Timestamp

Support for timestamp_vars data type, from #60 .

Including new input_nub_timestamp_handler
Including new PoorMansFFT keras layer (custom written, borrowed from timestamp_keras_layer)
Adding description for adding new layer type in contributing.md (Unrelated to #60 )

Merging in w/ only one +1, because I'm the only person on the project and no one else will ever see this.


Add passing tonal elevation to M3 Bottom Sheet

Material 3 has the concept of a tonal elevation that can be used to introduce a tonal shift to the bottom sheet container when containerColor is ColorScheme.surface.

When I try to sign the CLA, I get You already signed the CLA on 2024-10-02....

When I try to sign the CLA, I get You already signed the CLA on 2024-10-02....

I'll raise this internally, will let you know

closing/reopening per internal suggested fix

@bidrohi you'll need to fix the CI issues before we can proceed

Sorry for the delay. For some reason I couldn't sync the project on my laptop, so just had to make the changes and depend on the workflow for validation

Still failing. Remember you can run ./gradlew check locally too

Sorry about that. Couldn't figure out why my laptop wasn't getting JAVA 23 configured correctly. Just tested ti by dropping JVM to 17, should be ok now, I hope.

Thanks! Sidenote for future ref - once review has started, please don't force-push the branch.


Support for Separate Files for Rules and Allowlist in gitleaks

Hello @zricethezav,
I've been extensively working with gitleaks and found that it would be extremely beneficial to have the capability to utilize separate files for rules and the allowlist.
Background:
Currently, gitleaks supports defining both rules and the allowlist within a singular configuration file. In complex projects with many developers and frequently changing rules, this can become cumbersome. Separating the two lists could greatly enhance manageability and clarity.
Feature Request:
I propose to extend gitleaks to accept separate files for rules and allowlist. For example:
gitleaks --config-rules=path/to/rules.toml --config-allowlist=path/to/allowlist.toml

I believe this change would significantly improve the flexibility and adaptability of gitleaks, especially in larger projects.
Additional Information:
This feature would especially aid teams working in large organizations under stringent security policies, as they'd have the ability to manage their rules and exceptions in a cleaner and more efficient manner.
Thank you for considering this and for all the hard work on this fantastic tool!

Hi @zricethezav,
This would definitely be a useful feature.  So it would be great if the allowlist can be separated from the rules.
The ability to manage rules and exceptions in a more organised and effective way would be extremely helpful to teams working in large organisations.


Define EventStore API

From @bwaidelich on September 12, 2016 11:7
The current EventStore API is:
interface EventStoreInterface
{

    public function get(string $identifier): EventStream;

    public function contains(string $identifier): bool;

    public function commit(EventStream $stream): int;
}

We should review and re-evaluate that.
Copied from original issue: neos/Neos.EventStore#1

I especially want to argue about the retrieval of events: I think that we need a little more flexibility here than only to be able to fetch events by the Aggregate identifier (IMO we should even completely remove the notion of "aggregate" from the ES).
Background
Especially when reconstituting state for multiple aggregates (i.e. for "special" read models, process managers, ...) we can't just replay all events in the store.
Even for a very simple read model that only listens to one event type it would be an overkill to always replay all events of the corresponding aggregate.
Suggestion
Something along those lines could work:
class EventStreamFilter
{

    /**
     * @var int
     */
    public $minVersion = null;

    /**
     * @var string
     */
    public $streamName = null;

    /**
     * @var string
     */
    public $streamNamePrefix = null;

    /**
     * @var string[]
     */
    public $eventTypes = null;

}

(ignore the public fields, this is just an exemplification)
And an instance of that filter could be passed to some read or get function of the ES and then internally transformed to match the configured backend.

From @dfeyer on September 13, 2016 21:39
We discuss the topic during the sprint, and yes 👍 to add this flexibility
From my POV, we need min/max version (useful for conflict resolution by ex.) and start/end datetime (storage time), to make it really flexible.

From @albe on September 14, 2016 11:9

A projection is often concerned in events of a given aggregate type (i.e. if you have a Customer aggregate and a CustomerListItems-projection).

I would even go so far and question if a Projection is ever only interrested in events of a single aggregate instance. For that it would need to know the (technical!) identifier of the instance it wants to project for only, which is something I can not imagine a real use-case for.
A Projection IMO normally doesn't event realize the concept of "Aggregate", only implicitly by just listening to Events that a specific Aggregate emits. But I doubt you would want to declaratively create a Projection "listens to (all of) [Aggregate]". Conceptually I think such a tight coupling of read side to the core domain is not necessary or even recommended.
Just imagine the Projection running on a different machine - all it will know is the Events that your domain emits (ie. the public read API).
maxVersion, until
When replaying Projections for debugging purposes, you are interrested at the state of the Projection at a specific point in time or alternatively a version number.
Those filters can very easily be applied on-top, since they are mostly not time-critical.

we need min/max version

I absolutely see the need for min version (to catchup projections for example).
What would you use the max version for?

useful for conflict resolution by ex.

I do think that we need the expectVersion parameter when publishing events but I fail to understand how a min/max version for the querying part would solve conflicts.. Could you elaborate?

start/end datetime (storage time), to make it really flexible.

Why do you think that one is required?
IMO one should never query the ES directly to display a result to the public. And even for debugging/monitoring reasons I would probably always create a corresponding projection and query its state instead.
It's very easy to add filters and in this case very easy to implement them in the doctrine implementation. But keep in mind that we'll need to support everything we make public API in all future versions and adapters so I think it makes sense to collect valid use-cases and possible alternatives before we fall into the YAGNI trap.
I didn't do that above, but let me quickly change that:
minVersion
Projections should track which events they have processed already. Especially if a projection is executed asynchronously we need a way to process events after the fact in order to let the projection catch up
streamName
Usually the stream name is the combination of the aggregate type and its identifier (i.e. Customer-<uuid>). In order to reconstitute the state of a particular aggregate, this filter is needed  (and it's the only one we already have with get($identifier))
streamNamePrefix
A projection is often concerned in events of a given aggregate type (i.e. if you have a Customer aggregate and a CustomerListItems-projection).
Therefore I think that we need a way to read all events for a given prefix (Customer- in this case).
Note: In geteventstore.com this is achieved via fromCategory('Customer')
eventTypes
Mostly an internal optimization:
We know which event types an event handler (including projections) listens for, so we could fetch only those events in question when replaying events.
For the record: This is mostly based on gut feeling and I'm not saying that this covers all scenarios, but I strongly think that we should only implement features as soon as we actually need them

From @albe on September 14, 2016 9:1
Yes, querying for event type, min+max version and start/end time are pretty important.
class EventStreamFilter
{
    /**
     * @var int
     */
    public $minVersion = null;

    /**
     * @var int
     */
    public $maxVersion = null;

    /**
     * @var \DateTimeImmutable
     */
    public $since = null;

    /**
     * @var \DateTimeImmutable
     */
    public $until = null;

    /**
     * @var string
     */
    public $streamName = null;

    /**
     * @var string
     */
    public $streamNamePrefix = null;

    /**
     * @var string[]
     */
    public $eventTypes = null;
}

IMO we should even completely remove the notion of "aggregate" from the ES

Yes, the EventStore API should be an EventStream that is identified by a name. The Repository would then translate AggregateType + Identifier to a stream name and push and pull EventStreams.
What I found plausible yesterday in addition, would be some "StreamContext", i.e. a stringy value that could act as a partition point in EventStore backend implementations (which will eventually become important for scaling). In DDD it could for example be equivalent to the BC.
So there would be some class like that to identify streams:
class EventStreamName
{
    /**
     * @var string
     */
    public $name;

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

    public function __toString()
    {
        return $this->context . '-' . $this->name;
    }
}

Again, ignore the public properties, the name should be immutable.

From @dfeyer on September 14, 2016 14:37

I do think that we need the expectVersion parameter when publishing events but I fail to understand how a min/max version for the querying part would solve conflicts.. Could you elaborate?

In conflict resolution, you must query the store to ask what change between version a and b to see if any conflicting event exist, but maybe b is always the most recent version ... so maybe the need of min/max in this case is wrong.
But one use case for the Neos UI, is to compare a node version A with a node version B (diff), so you need this min/max query for this case.

From @albe on September 14, 2016 11:56

you only seem to support my reasoning from above

Yep :)

The stream name (including identifier) is needed not for projections, but to reconstitute the aggregate state (i.e. when loading the aggregate from its repository).

Yes, I would suggest clearly separating the two querying methods (i.e. keep the current get and add some find/filter/w\e method that does the querying as required for Projection stuff.

able to alter history

I would suggest doing this as a middleware to the EventStore - that way you can alter events, but you don't alter the persisted events. If you want to persist the altered Stream, you can do that easily by just calling EventStore:commit with a new stream name (might we need some stream-versioning at a point?).

So I wonder if we risk misuse when adding it to the API!?

I don't think the plain filtering can be misused. But I'd also be ok with only implementing those additional filters in some "EventStoreDebugger" middleware.

I would even go so far and question if a Projection is ever only interrested in events of a single aggregate instance

I could imagine that this could come in handy in domains where you only have a very specific set of aggregates (Imagine for example a wind farm where you need an event stream for each turbine). But anyways, you only seem to support my reasoning from above (or did I miss something?): The stream name (including identifier) is needed not for projections, but to reconstitute the aggregate state (i.e. when loading the aggregate from its repository).

maxVersion, until [...] you are interrested at the state of the Projection at a specific point in time or alternatively a version number

Good point, that's indeed a valid usecase. To leverage the full debugging power one should also be able to alter history (by modifying/removing/adding events). But both not in production but in a "lab" with the copied event stream. So I wonder if we risk misuse when adding it to the API!?

Those filters can very easily be applied on-top

Yes, I agree. If we have more insight we might consider adding them to a separate interface so we can still have adapters supporting a subset (but this is just thinking out loud, lets see what requirements arise from using this in practice)

From @albe on September 15, 2016 20:48
I would start with refactoring the EventStream to be aggregate-agnostic. What exactly is the reason for the EventStreamData container btw?

From @dfeyer on September 14, 2016 21:43
Currently our EventStream require an aggregate identifier, so the filter method can return a raw array of events or we need to refactor the EventStream object

From @dfeyer on September 15, 2016 20:59
@albe Feel free to refactor the EventStream
About the EventStreamData it's just a DTO to transport Event. EventStream has logic to add event, mark applied, ...).

Some suggestions for the ES API after playing around with it and some other implementations.
This might get a bit longer, bear with me.
Event Retrieval
The current API for that is:
public function get(string $streamName): EventStream

As discussed above, I'd suggest to replace that with something like
public function get(EventStreamFilter $filter): EventStream

Personally I don't think we need a separate method to retrieve events for one aggregate as this is mainly used by the framework anyways and because we could always implement more specific filters like
class AggregateStreamFilter extends EventStreamFilter
{
   public function __construct(AggregateRootInterface $aggregate)
   {
      $this->streamName = ...

The EventStream that is returned currently has two responsibilities which comes with some problems (e.g. it has getNewEvents(), addEvent() and markAllApplied() all of which are not relevant to the stream returned by the store).
I'd suggest to keep the name (and the fact that it extends IteratorAggregate) but turn it into an iterator of StoredEvents (or some better name) that contain a reference to the unserialized EventInterface instance and the EventMetadata (including version, ...)
Event publishing
The current API for that is:
public function commit(string $streamName, EventStream $stream, \Closure $callback = null) :int

string $streamName
I think that's perfectly fine. I've seen implementations where they use a StreamName class, but I don't understand why and think it's over-sophisticated.
EventStream $stream
Here EventStream is the wrong notion IMO. It is not a stream in this case but a fixed set/collection of Events (with usually only one event in it).
I like the geteventstore-client implementation in that regard: They just have a WritableToStream interface that can either be a single event or a collection of events.
\Closure $callback
Apart from the naming (what's this closure for?) I hope that we don't need that callback.
We currently use it to add the version to the event metadata in EventSourcedRepository::save().
I think that the stream version should be a core concept of the ES Event-Metadata and thus be added by the store upon commit.
And other metadata (that might be specific to the application) should be addable via some general "hook" anyways (IMO we could even use Signal/Slots for this).
return type int
We don't use it and I don't think we need it.
optimistic locking
What I miss in the current implementation is a way to implement optimistic locking.
Again, I like how the geteventstore-client implementation solves this:
public function writeToStream(string $streamName, WritableToStream $events, int $expectedVersion = ExpectedVersion::ANY)

where $expectedVersion is an integer with two special cases:

ANY (-2) is the default and basically disables optimistic locking for that commit
NO_STREAM (-1) expects the event(s) to be the first ones in the stream (useful to avoid clashes when creating new instances)

Contains
public function contains(string $streamName): bool

We only use that to provide EventSourcedRepository::contains() but I wonder if we need that.
IMO the code should always know whether an aggregate is expected to exist or not. Or do you have a scenario for this?
Ok, as expected, that turned into a novel..

From @dfeyer on September 17, 2016 21:20
Now the store is more simple and EventStreamData, EventStream and MessageMetadata are aggregate agnostic, check https://github.com/neos/Neos.EventStore/pull/9

From @dfeyer on September 15, 2016 20:59
Basically both should be aggregate agnostic i think

Ah, one thing I forgot to mention:
If we really implement the "filterable"-EventStream we have to keep in mind, that the entries in the result are deterministic and have a sequential version.
This is very important because every consumer (including Projections) should be able to rely on that (e.g. projections store the last version they have processed and can only replay new events).
Implementation-wise this can be a bit tricky because we must not reuse the event version of the stored event (even though that might also be of interest).
Again, I like how GetEventStore solves this. In their Event Atom Feed one entry looks something like this:
{
	"eventId": "ff56b4b5-b940-474b-9f4b-350916de7755",
	"eventType": "Crm:Person:PersonWasCreated",
	"eventNumber": 123,
	"data": "{...}",
	"metaData": "{...}",
	"streamId": "Person-8c5da344-1579-4d43-a350-a3dc6c1e35c0",
	"positionEventNumber": 13,
	"positionStreamId": "$ce-Person",
	// ...
}

So they have the eventNumber which is the index of the event in it's stream.
But they also have positionEventNumber which is the deterministic index of that composite stream (in this case it's a stream over all events of all Person aggregates).

From @albe on September 23, 2016 21:45
No, it basically expects that no events exist in the stream yet (which is distinct from the events having a specific version).

From @dfeyer on September 23, 2016 21:9

NO_STREAM (-1) expects the event(s) to be the first ones in the stream (useful to avoid clashes when creating new instances)

@bwaidelich So the expected version will be 1 ... or I miss something ?

From @dfeyer on September 24, 2016 10:9

Also, the commit version can be used for very efficient Cache-Control on the query side.

Agree that we need to expose the version

From @dfeyer on September 24, 2016 10:11

No, it basically expects that no events exist in the stream yet (which is distinct from the events having a specific version).

Not sure if I get it ... if the stream is empty ... the first event will have version 1, no ? The storage is responsible to set the version, you just pass an expected version to the commit method to implement optimistic lock ? Or ... i miss something ... those constant refactoring cost me some brain cells ;)

From @albe on September 23, 2016 22:26

return type int
We don't use it and I don't think we need it.

Well, we will need it if we want to return the commit version from a command, so that the client can later use it for optimistic concurrency - or, what I found a good use with eventual consistency, do queries against a specific commit version. For that the projections just need to store the current commit version they are up-to-date to, then you can easily do following logic in the UI for example:

send command
send query against read model
if query result version is lower than command commit version returned, retry

That way you can have a better UX with fake immediate-consistency, without having to add event-driven notification inside the read side or making the read model strongly consistent.
Also, the commit version can be used for very efficient Cache-Control on the query side.

Expect version

if the stream is empty ... the first event will have version 1, no ?

No. This is how it works with geteventstore (and IMO it makes a whole lot of sense):
The expectVersion specify what version you operate on, not what version you expect as the resulting event. So:
NO_STREAM (-2): There must not be an event in the stream yet
ANY (-1): I don't care
0: There's exactly one event in the stream
1: There are exactly two events in the stream
...
(note that this is an index that should be 0-based).
commit() : int
Regarding omitting the return type for  commit() I'm surprised that you disagree with me as this is in sync with CQS, the base for CQRS (methods should either have a return value or change state, not both). But lets not focus on the theories but get into it:

Well, we will need it if we want to return the commit version from a command, so that the client can later use it for optimistic concurrency

Can you elaborate how you would implement optimistic locking from the command?
I totally agree that the Read Models should be able to know what version they represent (as exposing the version to the client is the only way to really ensure optimistic locking) but I don't see how that is related to the ES to return the version.
For me it worked out nicely like this:
During projection we can get the version from the event metadata and we can store it in our Read Model. Then we can pass that version to some expectVersion of a command we want to execute on that model:
<f:form action="rename" objectName="command">
  <f:form.hidden property="expectVersion" value="{customer.version}" />
  <f:form.textfield property="newName" />
  <!-- ... -->
</f:form>

(whether "expectVersion" is part of the command payload or sent separately doesn't really matter for the point I'm trying to make).
When the aggregate emits the event it can specify the expectVersion in order to ensure consistency.
I also fail to understand how a "fake immediate-consistency" can benefit from that:

if query result version is lower than command commit version returned, retry

Can you provide an example?
Just for the record: I don't think it's a biggy at all to have the ES return the just published event number but I think we should first agree on an implementation of optimistic locking. Maybe we should discuss that in a separate ticket!?

From @albe on September 26, 2016 19:48

The expectVersion specify what version you operate on, not what version you expect as the resulting event.

Ah! That makes a lot of sense!

Can you elaborate how you would implement optimistic locking from the command?

My idea was, that the optimistic concurrency check starts with a command (ie you only work on command return results), but that was just wrong-thinking I see now. The expected version is set from the read side that shows the form for a command in the first place.
So forget that argument.

Can you provide an example?

Sure. Say you send a command PromoteUser, while at the same time the User you are working on has been edited (without conflicts) by someone else (name typo fixed, etc.), so your command passes (or is automatically redispatched after the first optimistic concurrency error). Now you want your UI to query for the new User model reflecting the promotion.
For that you send a query getUserProfileInformation(expectedMinimumVersion) from your UI after the command was succesfully dispatched with the version returned from the command (the version checking loop can be done client or server side).
If you only do this based on the version you worked on (= read model version) +1, you will probably only get a UserProfileInformation with one of the editings that the other person did, but not yet the promotion applied. The user in front of the screen is confused if his last action wasn't correctly applied and reattempts, getting an error back (User already promoted!). Shitty UX. So you need the actual version of the aggregate after the commit.
And yes, you are right that this breaks the CQS principle. But it buys a tool for better UX.

From @albe on September 24, 2016 13:58

if the stream is empty ... the first event will have version 1, no ?

Yes yes, technically it's the same in our case, but conceptually it is a difference between "expect version 1" or "expect empty stream". The first implicates knowledge about the internal versioning system used - imagine the version number just wouldn't start counting at 1, but at 0 - or maybe at 1000 for some reason. In that case "expect version 1" would fail, while "expect empty stream" would succeed if the stream is actually empty.

From @albe on September 27, 2016 20:38
One thing came to my mind today in regard to the above:
For Projections that work on multiple Aggregates/Aggregate instances, a single aggregate version does not hold any useful information in regard to the Projection version. In that case I only see a global commit version as workable. But that again would put the requirement to create a global commit sequence to the storage layer...
Anyone got any idea how to solve that otherwise?

Say you send a command PromoteUser [...]

I read it three times but I can't follow your example tbh.. I'll try again with a fresh head tomorrow ;)
But regarding:

For Projections that work on multiple Aggregates/Aggregate instances, a single aggregate version does not hold any useful information in regard to the Projection version

That's exactly what I was referring to with https://github.com/neos/Neos.EventStore/issues/1#issuecomment-249173650
The eventNumber is just the sequence within one stream, but the result of any (filtered) EventStore-Query must contain an incremental, deterministic index.

Mostly solved with #62


Tag "operator" and "installer" repos

Tag "operator" and "installer" repos in release scripts.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[x] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
fixes #17425

@clarketm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clarketm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test gencheck_istio

integ-telemetry-k8s-tests_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clarketm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

gencheck_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test gencheck_istio

integ-telemetry-k8s-tests_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test integ-telemetry-k8s-tests_istio

e2e-simpleTests-distroless_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test e2e-simpleTests-distroless_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@clarketm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-telemetry-k8s-tests_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test integ-telemetry-k8s-tests_istio

integ-security-k8s-tests_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clarketm: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-security-k8s-tests_istio
f75ea5d9346bd23b4fccc06b1e61b89c7e37ad2c
link
/test integ-security-k8s-tests_istio

integ-mixer-k8s-tests_istio
9750bd5cc7b9de78b76ed54c5a34e190d4ca1bbe
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@clarketm: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

integ-mixer-k8s-tests_istio
9750bd5cc7b9de78b76ed54c5a34e190d4ca1bbe
link
/test integ-mixer-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I assume this is no longer needed since the files were deleted
/close


Update font configuration

Description
Discuss with designers what fonts will be used and where they should be listed (most likely in the Figma file). After that, improve the font configuration (fonts.ts) under the const/ directory.
Criterials

[x] Discuss with designers and determine about fonts definition
[x] Update fonts configuration in fonts.ts
[x] Update the directory structure for font configuration. Currently, the file is located at src/const/fonts/fonts.ts. Since the font-related files will be mostly likely only one file, the const/fonts/ directory may not be necessary.

Additional Contexts

This issue branched off from #35

Discuss with designers what fonts will be used and where they should be listed

Fonts are listed in the right-side panel of Figma:


UI for this project

Hi All,
Not an issue but more a question or an opportunity for a developer. Wondering if anyone created a UI for this workflow they could share or would be interested in developing?
Cheers,
Chris

This would be amazing. @cjcorney Do you have any ideas or sketches to share to get the conversation going?

Hi Lee,
My developer and I have been working on a prototype but due to the systems
we use (Strongview is our ESP and frankly belongs in a museum) and the way
the templates are currently setup to serve four languages has made life a
bit tricky. I'll share some screenshots next week though as to what we've
been able to do so far in terms of generating multiple files etc :)
On Fri, Sep 1, 2017 at 12:22 PM, Lee Munroe<EMAIL_ADDRESS>wrote:

This would be amazing. @cjcorney https://github.com/cjcorney Do you
have any ideas or sketches to share to get the conversation going?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/leemunroe/grunt-email-workflow/issues/92#issuecomment-326483583,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQL6cA-04mxQXZ3D8ly6zzsKfMDiYQebks5sd4ZtgaJpZM4PIcOu
.

Hi Lee,
Our progress to date below:
http://tinypic.com/r/r2rxup/9
For now we're going back to the CLI and adapting our new templates into the various components etc. Our main issue is that our Strongview templates are setup to use variables for each language which is proving a pain.
Also wondering if there is a way to send all templates form the dist folder in your example rather than just a specific one? Using your git example each time I run grunt send --template=*.html I am sent the last .html file in the folder (transaction.html) three times rather than each email once.

This is looking really impressive.

I'm afraid not. I get the same thing using *.html. Possibly a bug with https://github.com/markhuge/grunt-mailgun

@cjcorney Wondering if you made any progress with this project?

@cjcorney - Any update here or should we close this issue?

Closing. But I would love to see any progress from @cjcorney


Change PUT to POST in generateActionRequestImplementation

According to their documentation, oVirt action methods are always HTTP POST method.

@Maigard Thanks for you PR.  Could you show me the official documentation as you mentioned? I checked the latest official oVirt Python and Java SDKs, and they still use the HTTP PUT method.

https://ovirt.emsl.pnl.gov/ovirt-engine/api/v4/model#_services
Specifically this section:

For action methods the HTTP method is always POST, and the name of the method is added as a suffix to the path. For example, the request to start virtual machine 123 should look like this, using the HTTP POST method and the path /vms/123/start:
POST /ovirt-engine/api/vms/123/start

It could be that this changes more actions than I am intending. (specifically any update actions), but I couldn't get the sdk generation to work so I could test it out. I will try it again today to see if works any better now that #99 is potentially fixed.

@Maigard Sorry for not getting your point. The Java/Python SDK indeed use HTTP POST method for all the Action requests, and HTTP PUT method is used only for Update requests.

hi @Maigard, I find that you are the maintainer of the project https://github.com/EMSL-MSC/terraform-provider-ovirt. Is your project still being in development? Maybe I could contributes to your project by using this ovirt go sdk with API v4.
I should have contacted you by email, but I could not get it. 😄

I am still maintaining it, but I haven't had much time because of other commitments. I am definitely willing to take contributions.
I would like to get it working with this library so I don't have to maintain both parts (hence this pull request), but there are still some changes that have to be made. I am going to be making some pull requests in the future that should make it work, but I haven't had the time.

ok. I will try to make some contributions to your terraform-provider-ovirt project. By the way, if you encounter the problems/bugs about this sdk library, please be free to commit issues and I will get them resolved ASAP. Thanks.


The .exe keeps crashing

When I first used it the .exe works. I tried it again the next day it stopped working. The .exe is on but doesn't detect my inputs nor uncap my fps in genshin. I clean reinstalled and it still does not work.


FAT-14153: adding karate tests for LC user registration feature

Purpose
https://folio-org.atlassian.net/browse/FAT-14153
Approach
How does this change fulfill the purpose?
TODOS and Open Questions

[ ] Use GitHub checklists. When solved, check the box and explain the answer.

Learning
Describe the research stage. Add links to blog posts, patterns, libraries or addons used to solve this problem.

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


Fixed navbar breaking in smaller screens

Related Issue#71
I have fixed the navbar breaking(timer overflow) in smaller screens by adding responsive size to timer font.
Screencast from 23-04-23 06:43:19 PM IST.webm

Hey @Udassi-Pawan, thanks for contributing to Focusly. It still seems to be breaking in the video. It will be more preferrable if we can add the tabs below the navbar.

Just to clarify what needs to be done, should the navbar be vertical with stacked menu items?

Hey, the tabs which are inside the  Navbar now should be below the navbar. we can create another component for them and just place it below the navbar.

Hey @Utkarshn10 I have moved the nav tabs below navbar for small screens. The video is attached for reference.
Screencast from 26-04-23 01:50:44 PM IST.webm


"Undefined symbols"-Error only when building with rust

I've got a super small cpp file which basically just contains some OpenCV calls:
#include <opencv2/opencv.hpp>

extern "C"
void example(uint32_t width, uint32_t height, uint8_t* data) {
    auto image = cv::Mat(cv::Size(width, height), cv::CV_8UC3, data);
    /** other stuff */
}

Which actually compiles when I call gcc manually. It also compiles with cc-rs as long as I don't make any calls to resize_image(...) within my rust file, but as soon as I do, "Undefined symbols" errors start showing up:
Undefined symbols for architecture x86_64:
    "cv::Mat::deallocate()", referenced from:
      cv::Mat::release() in libptbd_export-24f57485ccdde53d.rlib(png_resize.o)
    "cv::fastFree(void*)", referenced from:
      cv::Mat::~Mat() in libptbd_export-24f57485ccdde53d.rlib(png_resize.o)
    ( ... )
    ld: symbol(s) not found for architecture x86_64
    clang: error: linker command failed with exit code 1 (use -v to see invocation)

I set almost exactly the same arguments in my build.rs file as I do when calling gcc manually.
Why could it be that it compiles manually but not with cc-rs?

Thanks for the report! This is likely a missing library that needs to be linked and not a bug in this crate. You'll want to find the nae of the opencv library and print that in the build script for rustc to link later.


License Type

Hello! I would love to use this in our project but it is missing the license type. Would you mind adding it?

@doitian @starikcetin @vangogih :)


preventing mentioned people from being pulled in room where mentioned

Your Rocket.Chat version: (make sure you are running the latest)
https://github.com/RocketChat/Rocket.Chat/issues/4286

https://github.com/RocketChat/Rocket.Chat/issues/4286


Update documentation

[x] Add sections on how to use the user guide
[x] Add more questions in FAQ
[x] Update target audience in README
[x] Add diagrams for find and list in the developer guide

Codecov Report

Merging #159 (8ea2e89) into master (fad8a64) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #159   +/-   ##
=========================================
  Coverage     75.75%   75.75%
  Complexity      568      568
=========================================
  Files            91       91
  Lines          1761     1761
  Branches        147      147
=========================================
  Hits           1334     1334
  Misses          378      378
  Partials         49       49

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Custom validator with class attribute

Hello, I'm trying to write a custom validator using a class. The problem is that I need to compare the value in the validate() method with an attribute of the class. I tried with this:
`class MyValidator(Validator):
def init(self, limit):
self.limit = limit
Validator.init(self)
def validate(self, number):
if number > limit:
return "Error!"
else:
return True`
The problem is that validate() method seems to be never called with this approach. If I remove the constructor from MyValidator, it works but I don't have the "limit" attribute. Is there a solution for this? Thank you

Hmnnn. This seems like you want to be able to pass extra args to the validator???


extra variables and actions link incorrect

extra variables and actions link at the bottom of page redirects back to the same page.

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

ID: 3427b1fd-1a36-9e75-a3e6-d1850fdb4631
Version Independent ID: 5b4e98a4-5798-f597-55d4-9f694c981682
Content: Configure hand-off to Dynamics 365 Customer Service - Power Virtual Agents
Content Source: power-virtual-agents/configuration-hand-off-omnichannel.md
Service: power-virtual-agents
GitHub Login: @KendalBond007
Microsoft Alias: iawilt

@correnasergent - I am no longer with this project, but am in communication with @iaanw to implement a solution; assigning this work item to him. Follow here for updates.


sometime gang-schedule failed with volcano

tracking PR #1831 for flaky test: sometime e2e failed

/assign

This was fixed by #1835.
/close


*** AttributeError: 'HopperEnv' object has no attribute 'get_dataset'

Environment:
Ubuntu 18.04 / Python 3.8.5
gym==0.18.3
gym-maze==0.4
gym-minigrid==1.0.2
gym3==0.3.3
-e git+https://github.com/rail-berkeley/d4rl.git@9b68f31bab6a8546edfb28ff0bd9d5916c62fd1f#egg=d4rl
I was trying to get dataset for MuJoCo Hopper env. The error messages are pasted below.
 >>> env = gym.make('Hopper-v3')
>>> dataset = env.get_dataset()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/hsinyu/anaconda3/lib/python3.8/site-packages/gym/core.py", line 223, in __getattr__
    return getattr(self.env, name)
AttributeError: 'HopperEnv' object has no attribute 'get_dataset'
>>> dataset = d4rl.qlearning_dataset(env)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/data3/hsinyu/00_BRL/BEAR/d4rl/d4rl/__init__.py", line 87, in qlearning_dataset
    dataset = env.get_dataset(**kwargs)
  File "/home/hsinyu/anaconda3/lib/python3.8/site-packages/gym/core.py", line 223, in __getattr__
    return getattr(self.env, name)
AttributeError: 'HopperEnv' object has no attribute 'get_dataset'

try to reinstall gym=0.18.0

Hi, I got the same issue for gym==0.18.0. Did you solve it?

I change import gymnasium as gym to import gym and it works.


WIP: Parallelize CI runs

this suggests that each build should then use it's own data cache

(I fscking hate YAML…)

We get use to it:)
You certainly have a linter for circle Ci files

You certainly have a linter for circle Ci files

hm yes but it says right there:

Note: local-cli does not validate workflows at this time

And we need to use workflows for parallelism… but yeah maybe I can validate the rest of the file, the workflow section is actually quite short…


Enable CORS

For Allowed Methods you also need PUT verb.

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

ID: 36c209d9-c15f-87a4-a7ee-bb2bd3416961
Version Independent ID: 2f76d983-a940-fa5f-abe0-4c851e377608
Content: Tutorial - Customize the interface of user experiences - Azure Active Directory B2C
Content Source: articles/active-directory-b2c/tutorial-customize-ui.md
Service: active-directory
Sub-service: b2c
GitHub Login: @mmacy
Microsoft Alias: marsma

@ArmandoPitotti Thanks for your feedback! We will investigate and update as appropriate.

Hello, thanks for providing us this information! We're currently putting this into the docs. We appreciate your feedback as it's very helpful.
@ArmandoPitotti Please let us know if there are anymore questions within the scope of this git issue. If not, I will be closing out this git issue by end of day tomorrow. Please reopen this git issue if you have anymore concerns. Thanks

@ArmandoPitotti I will be closing out this git issue now. as there has been no response. thanks again for letting us know about this!


added PR to ensure old datasource Ids are not referenced in imported …

…Charts

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Alles top


misc updates/responses

Mostly minor, except where I tried to clarify what the mixed model was doing.

Wouldn't all 3 environmental models be robust to uninformative covariates? If BRT are considered robust than random forests should be as well, and since it's based on a neural net I would guess MistNet is as well.

It does read a little oddly to call out a single method as being robust and not include the others. If describing all three this way is justified then I like @sdtaylor's suggestion of mentioning them all. If not I think it's OK to say that this is complicated by the model fitting procedure and (if we all agree that what follows is correct) that variable selection is typically less important for prediction than for inference.
The reality is that we've done a massive amount of comparative modeling here and reviewers will always be interested in seeing you go one step further than you have. We've already made a lot of big steps forward in this paper and it's fine to stop there.

I had thought he was just asking about the "macroecological model" based on the line numbers, but maybe I misinterpreted.  Will take another stab at this soon.

You two were right and I misread: he did ask about all environmental models, not just boosting. Fixed now.

:+1: LGTM