x/tools/gopls: Request textDocument/definition failed. "no object found for ident ..."

I have 2 similar versions of the same program. Both work as expected while running the code from terminal. However, version 2 gives the error in vscode even though it still compiles.
Version 1:
package main

/*
#include <stdlib.h>

typedef struct Person {
    char* name;
    int age;
} Person;
*/
import "C"
import (
	"fmt"
	"unsafe"
)

type Person C.Person

func (p Person) PrintName() {
	name := C.GoString(p.name)
	fmt.Println(name)
}

func main() {
	name := C.CString("John")
	defer C.free(unsafe.Pointer(name))

	age := C.int(30)

	cPerson := C.Person{
		name: name,
		age:  age,
	}

	person := Person(cPerson)
	person.PrintName()
}

Version 2:
package main

/*
#include <stdlib.h>

typedef struct Person {
    char* name;
    int age;
} Person;
*/
import "C"
import (
	"fmt"
	"unsafe"
)

func (p *C.Person) PrintName() {
	name := C.GoString(p.name)
	fmt.Println(name)
}

func main() {
	name := C.CString("John")
	defer C.free(unsafe.Pointer(name))

	age := C.int(30)

	cPerson := C.Person{
		name: name,
		age:  age,
	}

	person := C.Person(cPerson)
	person.PrintName()
}

What did you expect to see?
Version 2 of the code should have worked the same way as version 1 in VSCode
What did you see instead?
For version 2, at person.PrintName() it shows

person.PrintName undefined (type _Ctype_struct_Person has no field or method PrintName)compilerMissingFieldOrMethod

Moreover, when PrintName is clicked, it doesn't take to the function.
This is a simplified example of the issue that we are facing in our code base which is not very easy to refactor.
Editor and settings
VS Code Settings:
"go.lintTool": "golangci-lint",
  "go.lintFlags": ["--fast"],
  "[jsonc]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  },
  "go.toolsManagement.autoUpdate": true,
  "go.gopath": "",
  "go.lintOnSave": "file",
  "go.gocodeAutoBuild": true,
  "go.useLanguageServer": true,
  "go.buildOnSave": true,
  "[json]": {
    "editor.defaultFormatter": "esbenp.prettier-vscode"
  }

Logs
[Error - 02:20:39] Request textDocument/definition failed.   Message: no object found for ident PrintName   Code: 0

Thanks for the very clear bug report. I confirm the error in program 2 (but not 1) using VSCode and<EMAIL_ADDRESS>and gopls@master.
The fundamental problem is that it probably shouldn't be legal to declare a method whose receiver is *C.T since ostensibly that's a type from another package. The Go compiler allows it, because the current implementation of cgo does a source transformation step that replaces C.T with a generated type in the current package, something like __C__T_mumble. But really the tool should disallow it. By constrast, gopls is bssed on the go/types package, which doesn't do a source transformation and interprets the C.T reference directly. It therefore complains about the method declaration.
I argue that go/types is correct and the go command (cgo) has a bug, but that question hasn't been resolved yet. See
https://github.com/golang/go/issues/57926
In any case, we have the unfortunate situation of code (like your example program 1) existing in the wild that depends on the  buggy behavior. It is probably not straightforward for us to change go/types to match the compiler behavior (nor desirable, if we all agree it's a bug).
If you have the power, I recommend that you delete any method declarations whose receiver type is of the form C.T.

Now that https://github.com/golang/go/issues/57926 has been resolved by making the cgo tool reject attempts to declare methods on types of the form C.T, I'm going to close this. The error message could perhaps be improved, but I think this will become vanishingly rare in practice.


Iterate on GA tessellation generation

Blocked by: https://github.com/adachille/tessellation/issues/4
Description
Iterate on the GA solution from https://github.com/adachille/tessellation/issues/4

[x] Update genome to include lines for both y-axis and x-axis
[x] Finish up notebook/pipeline so we can easily visualize the results of the tessellation
[x] Add more mutators

[x] Ability to insert / remove actions (currently only have swap)

[x] Add more heuristics

[x] Penalty heuristics

[x] Duplicated points - Penalty for repeated points (help keep evolution running smoothly)
[x] Out of bounds - Disqualification for points going out-of-bounds

[x] Reward heuristics

[x] Total number of points - Count the total number of points in the line (more can be generally better, or we shoot
[x] Ratio between bottom and top points is even - we can try to ensure that the bottom and top are relatively even
for some specific amount)

Expected Output
More interesting GA algorithm + example generations

I've reached a good stopping point - I'm going to leave the following work for 2 new issues:
Issue 1 - https://github.com/adachille/tessellation/issues/11
Issue 2 - https://github.com/adachille/tessellation/issues/12


Feature/issue #653 part2

Store values in iresearch columnstore. Then they are used for late materialization.

http://jenkins01.arangodb.biz:8080/job/arangodb-matrix-pr/7595/


interface to import

Hi, I want to use model but first I have to import interface to my database. How I can do that easily? A json format aside with table one would be great.

You can create table definitions from the SimpleSchema definitions automatically. There are various libraries to convert SimpleSchema to other schema formats. Database abstraction layers like Hasura or Prisma can create working apps from Schemas like this, and there are many smaller libraries that can create database definitions (for example as SQL) directly from schema information.
Closing this - I hope this answers your question, feel free to reopen if it doesn’t! 🙂


post数据乱码

我post 的数据中有中文，通过ngrinder 请求后，入库的数据中文变成??? 的乱码。
另返回的数据有中文时，在日志中也是乱码的。
result={"status":200,"message":"????","data":{"memberId":"134680196341990305"}}

I think you should provide correct content-type header to reuqest when you pass the parameter.

I also meet this issue, even though I add the correct content-type header "NVPair("Content-type", "application/json;charset=UTF-8")", what's more it will show two times this header in my request header as below:
{
"Content-type": "application/json;charset=UTF-8",
"xxxx":"xxx",
"Content-Type": "application/json;charset=UTF-8"
}

"Content-type": "application/json;charset=UTF-8"，但是控制台的中文还是乱码

meet this issue, too

这个问题有人解决么。。

@donggyu04 please check it.
Hey. Everybody.. please use English.
We, nGrinder developers, are not Chinese.

The interface response text obtained by getText() method is displayed in Chinese as "???"    ，it must use utf-8? how? please give me a solution.

@zhougy1989
Hi, You hava to set charset parameter to your Content-Type response header like application/json;charset=UTF-8 the grinder HttpResponse.getText() method refer to this.
or use new String(result.getData(), "UTF-8") for getting you contents.

Before

{"Status":200,"Greetings":{"ì¤êµì´":"ä½ å¥½å","íëì¤":"AllÃ´","íêµ":"à¸®à¸±à¸¥à¹à¸«à¸¥","ìì´":"Hello","ì¼ë³¸ì´":"ä»æ¥ã¯","ìë¼ë¹ì":"Ø§ÙØ³ÙØ§Ù Ø¹ÙÙÙÙ","íêµì´":"ìëíì¸ì","ë¬ìì":"ÐÐ ÐÐÐ¡Ð¢ÐÐ£ÐÐ¢Ð","ë² í¸ë¨":"A lÃ´"}}

After

{"Status":200,"Greetings":{"중국어":"你好吗","프랑스":"Allô","태국":"ฮัลโหล","영어":"Hello","일본어":"今日は","아라비아":"السلام عليكم","힌국어":"안녕하세요","러시아":"ДРАВСТВУЙТЕ","베트남":"A lô"}}

@donggyu04
thank you ,but it's is not my question；
my response is this: "message":"??"  how to  turn question marks into Chinese like this:"message":"成功"

@zhougy1989
Is your server return Chinese correctly?

@zhougy1989
Also, you can debug your response with our test class
If you using Intellij IDE, just run debug the test class like below.

`

@donggyu04
when I use orther tools,the response can return chinese;

and my script use jython,not groovy;

@zhougy1989
Can you analyze the problem for us?

@junoyoon @donggyu04
grinder.logger.info("----{}----", result.getText())‘，use this method ,the loginfo is chinese;
response=JSONObject(result.getText())，print(response), the response is “??”
i think i found the reason

@zhougy1989
# Add this top of your scripts.

import sys
sys.setdefaultencoding('utf-8')

...

response = JSONObject(result.getText());
print(response.toString());

Result

@zhougy1989
print(response.toString().encode('utf-8')); is works too.

@donggyu04

thanks, but it change to this，i think the same  to you

should add jvm args "-Dfile.encoding=UTF-8 -Dsun.jnu.encoding=UTF-8" in LocalScriptTestDriveService.java.
or read properties "grinder.jvm.arguments", then append it.

2020-12-11 10:28:42,444 INFO  ScriptHandler.java:163 : orion/resources/cookies is being written in /opt/ngrinder-controller/script/admin/resources for test 0
2020-12-11 10:28:42,456 INFO  LocalScriptTestDriveService.java:115 : grinder.jvm.classpath : /tmp/ngrinder-controller-3.5.0.war-spring-boot-libs-defd9547-5975-436b-9bf8-314a9127cb86/ngrinder-runtime-3.5.0.jar::/tmp/ngrinder-controller-3.5.0.war-spring-boot-libs-defd9547-5975-436b-9bf8-314a9127cb86/grinder-3.9.1-patch.jar:/tmp/ngrinder-controller-3.5.0.war-spring-boot-libs-defd9547-5975-436b-9bf8-314a9127cb86/plexus-sec-dispatcher-1.4.jar::/opt/ngrinder-controller/script/admin
2020-12-11 10:28:42,458 INFO  LocalScriptTestDriveService.java:132 : jvm args :  -Djna.library.path=/opt/ngrinder-controller/script/admin/lib  -Dpython.path=/opt/ngrinder-controller/script/admin/lib  -Dpython.cachedir=/tmp/jython  -Dngrinder.etc.hosts=29ac40288430:<IP_ADDRESS>,localhost:<IP_ADDRESS> -Dsun.net.spi.nameservice.provider.1=dns,LocalManagedDns  -Duser.dir=/opt/ngrinder-controller/script/admin  -Dngrinder.context=controller  -Dhttps.protocols=SSLv3,TLSv1,TLSv1.1,TLSv1.2  -Djsse.enableSNIExtension=false
2020-12-11 10:28:42,465 INFO  ErrorStreamRedirectWorkerLauncher.java:127 : worker validation-0 started

@colin-lee
Thx.
@donggyu04 test it.

@zhougy1989
I don't understand your comment, what is changed?
Is the message value changed?
@colin-lee
It seems Jython character encoding issue. (default is ascii)
I appended your suggested option to JVM args before running our worker process. It doesn't affect Jython default encoding.
I will apply the options that affect encoding If possible.
Thanks.

@donggyu04 I used groovy scripts, not python scripts.

@colin-lee
ngrinder must use Java 8 or above and that environment default file.encoding settings is UTF-8.
The reason of text breaking in groovy script when you using HttpResponse.getText() is because getText() uses ISO-8859-1 as the default file encoding.
    public synchronized String getText()
	throws IOException, ModuleException, ParseException
    {
	String ct = getHeader("Content-Type");

	/** ++GRINDER MODIFICATION **/
	if (ct == null) {
	    return new String(getData(), "ISO-8859-1");
	}

	// if (ct == null  ||  !ct.toLowerCase().startsWith("text/"))
        // throw new IOException("Content-Type `" + ct + "' is not a text type");
	/** --GRINDER MODIFICATION **/

	String charset = Util.getParameter("charset", ct);
	if (charset == null)
	    charset = "ISO-8859-1";

	return new String(getData(), charset);
    }

@colin-lee
We are changing default HttpClient for the test scripts now.
This will be available in the next release and I think the above problem will be resolved as well.

@donggyu04
Thx。
waiting this feature


NIFI-2717: Ensure that when performing site-to-site communications th…

…at we always commit the transaction on the remote side before committing it on the local side

Reviewed the change. All existing unit tests passed. Done Integration test using two NiFi clusters. Worked as expected.
Thanks @markap14 ! Merged this into master.


Parametrize Deserialize{Value,Row} with two lifetimes: 'frame and 'metadata separately

Refs: #462
Tl;dr
A second lifetime parameter is introduced to entities comprising the bottom part of the new deserialization framework: 'metadata.
Deserialization traits - recap
DeserializeValue requires two types of data in order to perform deserialization:

a reference to the CQL frame (a FrameSlice),
the type of the column being deserialized, being part of the result metadata.

Similarly, DeserializeRow requires two types of data in order to perform deserialization:

a reference to the CQL frame (a FrameSlice),
a slice of specifications of all columns in the row, being part of the result metadata.

Observation
When deserializing owned types, both the frame and the metadata can have any lifetime and it's not important. When deserializing borrowed types, however, they borrow from the frame, so their lifetime must necessarily be bound by the lifetime of the frame. Metadata is only needed for the deserialization, so its lifetime does not abstractly bound the deserialized value.
Problem
Up to this PR, Deserialize{Value, Row} were only parametrized by one lifetime: 'frame, which bounded both the frame slice and the metadata. This was done that way due to an assumption that both the metadata and the frame (shared using Bytes) will be owned by the same entity. However, the new idea of deserializing result metadata to a borrowed form (to save allocations) makes result metadata's lifetime shorter than the frame's lifetime.
Implemented solution
Not to unnecessarily shorten the deserialized values' lifetime, a separate lifetime parameter is introduced for result metadata: 'metadata.
The PR is large, but the changes are mostly mechanical, by adding
the second lifetime parameter.
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x] All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[ ] I have provided docstrings for the public items that I want to introduce.
[ ] I have adjusted the documentation in ./docs/source/. <-- not yet, will be done at the end of the deserialization refactor
[ ] I added appropriate Fixes: annotations to PR description.

cargo semver-checks found no API-breaking changes in this PR! 🎉🥳 Checked commit: 4406db5

???
This looks like a crude bug... Adding a new lifetime parameter to a trait necessarily breaks the API, in the simplest possible way: syntactically.

Refs: #462
Tl;dr
A second lifetime parameter is introduced to entities comprising the bottom part of the new deserialization framework: 'metadata.
Deserialization traits - recap
DeserializeValue requires two types of data in order to perform deserialization:
1. a reference to the CQL frame (a `FrameSlice`),

2. the type of the column being deserialized, being part of the result metadata.

Similarly, DeserializeRow requires two types of data in order to perform deserialization:
1. a reference to the CQL frame (a `FrameSlice`),

2. a slice of specifications of all columns in the row, being part of the result metadata.

Observation
When deserializing owned types, both the frame and the metadata can have any lifetime and it's not important. When deserializing borrowed types, however, they borrow from the frame, so their lifetime must necessarily be bound by the lifetime of the frame. Metadata is only needed for the deserialization, so its lifetime does not abstractly bound the deserialized value.

If that were the case then there would be no need to introduce second lifetime to those traits, you could just use anonymous lifetime / have a lifetime-generic method:
    fn deserialize(
        typ: &ColumnType,
        v: Option<FrameSlice<'frame>>,
    ) -> Result<Self, DeserializationError>;

The reason that seems not possible it precisely that deserialized values are bound by metadata lifetime, for example a ListlikeIterator.
This sentence confused me so much that I had to go and try to do this change (and learn why it is not possible) in order to understand why this PR introduces the second lifetime.
Please fix the sentence.

Problem
Up to this PR, Deserialize{Value, Row} were only parametrized by one lifetime: 'frame, which bounded both the frame slice and the metadata. This was done that way due to an assumption that both the metadata and the frame (shared using Bytes) will be owned by the same entity. However, the new idea of deserializing result metadata to a borrowed form (to save allocations) makes result metadata's lifetime shorter than the frame's lifetime.
Implemented solution
Not to unnecessarily shorten the deserialized values' lifetime, a separate lifetime parameter is introduced for result metadata: 'metadata.
The PR is large, but the changes are mostly mechanical, by adding the second lifetime parameter.

v1.1: addressed comments.


Not a valid link: https://quaderno.io/stripe-vat-subscriptions/

There is something in the url validation that is incorrectly rejecting a valid link:
https://quaderno.io/stripe-vat-subscriptions/
Possibly related cases:

https://fascinatingmaps.com/strange-german-exclaves-in-belgium-vennbahn/
https://www.amazon.co.uk/dp/0135974445/
https://perspectives.mvdirona.com/2019/02/aws-nitro-system/

I think this can be fixed by setting the user agent to something typical.  But to do that, the badoux/goscraper library will need to be replaced with something more flexible.
One possibility:
https://schier.co/blog/2015/04/26/a-simple-web-scraper-in-go.html


Getting Started with Angular Universal

https://github.com/angular/angular-cli/wiki/stories-universal-rendering


Remove check for Github.com when using an access token

This check isn't required and have successfully tested using a GitHub enterprise server. This tool doesn't work for me in CI without this change since I am not using SSH for auth

Fixes #74 and #33

closes #74
closes #33

:rocket: PR was released in v2.8.5 :rocket:


CSharp "Hello World" application

What changes were proposed in this pull request?
This change is a recreation of #90 in CSharp. It addresses issue #79 .
Why are the changes needed?
To provide a sample application in .NET.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
This change consists of an standalone example application and is verified by running it manually and validating the output and its behavior.

@AFFogarty LGTM.

Should we update the azure pipeline to make sure this compiles? :)

LGTM, Thanks @AFFogarty

Looks like the Scala example #90 also missed setting up the build. I will create an issue and we can do a follow up PR.


Non-simple property in cardinality restrictions

ECO uses 'has part' in several different cardinality restrictions, for example:
'combinatorial computational evidence' SubClassOf 'has part' min 2 'computational evidence'
Non-simple properties (here caused by 'has part' being transitive) in these axioms are disallowed in OWL so this prevents using ECO with a reasoner that handles cardinality restrictions (like HermiT); see Global Restrictions on Axioms in OWL 2 DL.

These axioms do not exist in eco-basic.owl. We can make another version to support your use case, if necessary. Which product are you using now?

We have been using eco-base. Probably we could switch to eco-basic, thanks. I would think you would want to modify the axioms in any case though, since they make ECO invalid OWL.


Stop lookup when username is found in any agreement

Closes #7
Uses a wrapper Promise to resolve with username for the first agreement including the username, otherwise resolves with [].

I meant to create this as a draft PR, but I appreciate the early review.

I'll rework the solution to cancel in-flight requests.
According to the Promises/A+ spec, subsequent resolve calls after the first are ignored. We shouldn't be concerned as want to return the first agreement that matches the username and ignore the others.
Will add a unit test.

Nice, well, the tests passed at least! That's great news.


Upgraded Firebase dependency to 2.x.x

Fixes #653. Once this gets merged in, we can go ahead and cut a 1.1.3 release.

Does angularfire support firebase 2.0.x, and 2.1.x? Because that is what we are implying here.
If not, it might have been better to do a semver range: >=2.2.0 <3.0.0.

That's a fair point. I'm aware of no reasons why AngularFire wouldn't work with all 2.x.x versions of Firebase. There are certain things which won't work with older clients, but those are called out with useful errors. For example, $firebaseAuth.$changeEmail() does this. So, I think we are good. But thanks for making me think through that.


Remove futures_codec

What did you implement:

Closes: https://github.com/vv9k/docker-api-rs/issues/23
Turns out we dont even need futures_codec here anymore, but it was enabling the io feature in futures-util, so now we need to manually enable that ourselves.
How did you verify your change:
cargo clippy --all-targets
What (if anything) would need to be called out in the CHANGELOG for the next release:
This does not affect the public API or alter functionality, so nothing needed.

Thank you!


test: adds weekly scheduled smoke tests

This PR fixes/implements the following bugs/features

[ ] Bug
[ ] Feature
[ ] Breaking changes
[x] Test improvements

What
Adds weekly scheduled smoke tests to be run on every Sunday at 00:00. Tests can also be trigger manually (workflow_dispatch). These tests will be looking for rule crashes.
Some example ESLint community plugins that are using this tool already:

eslint-plugin-react
eslint-plugin-jest
eslint-plugin-jest-extended
eslint-plugin-jest-dom
eslint-plugin-testing-library
eslint-plugin-unicorn

Helps finding issues like below even before releasing:

https://github.com/veritem/eslint-plugin-vitest/issues/27
https://github.com/veritem/eslint-plugin-vitest/issues/21
https://github.com/veritem/eslint-plugin-vitest/issues/20

Here is the first run: https://github.com/AriPerkkio/eslint-plugin-vitest/issues/1

Detected 26770 ESLint reports and/or crashes.
Scanned 5288 repositories.
Rules:

vitest/expect-expect
vitest/no-conditional-tests
vitest/lower-case-title

How
Plugin maintainer making sure all existing rules do not crash
Utilizes https://github.com/marketplace/actions/eslint-remote-tester-runner. Each test run will check as many repositories as it can in 5 hours 30 minutes (default value of timeLimit). Utilizes over 10K repositories from eslint-remote-tester-repositories.
Available configuration options:

eslint-remote-tester-run-action: https://github.com/marketplace/actions/eslint-remote-tester-runner#action-parameters
eslint-remote-tester: https://github.com/AriPerkkio/eslint-remote-tester#configuration-options

When test fails the @github-actions bot will open new issue with title Results of weekly scheduled smoke test. It will reuse the existing issue if previous one hasn't yet been closed. It's recommended to close the issues once their findings are fixed.
Below are two examples of test results:

https://github.com/jsx-eslint/eslint-plugin-react/issues/3328
https://github.com/jsx-eslint/eslint-plugin-react/issues/3376

@veritem the ts-node is actually required, it was now removed by 978dad5.
https://github.com/AriPerkkio/eslint-remote-tester#eslint-remote-testerconfigjs

Configuration file can also be written in TypeScript if ts-node is installed.

I haven't read about that, let me add it in another commit.


Add minimal option

closes #96
A default value can be provided in different ways, most importantly with -d minimal=true.
If none is provided it will prompt interactively (selecting false by default).
The actual code has mostly the same changes as the 1.0-minimal branch, but I also removed the CustomMsgs and response, as well as the example state. If you think we need them in the minimal version, I can add them too, but I think whoever uses the minimal version, probably already knows how the storage and messages work.
Also changed the appveyor script to build both minimal and full version.

If you think we need them in the minimal version, I can add them too, but I think whoever uses the minimal version, probably already knows how the storage and messages work.

I think that makes sense. We can always change it if we get feedback from contract devs that suggests otherwise!


brew not installing latest version, stuck at 0.6.1

Trying to install sesh at the latest version of 0.8.0 via brew but it only recognizes latest version as 0.6.1
Reinstall also seems to be giving the same version. Also tried building from source but that also failed

Please run brew update first and see if that fixes it


Check molfiles - wrongly identified as polymer

Hi Adnan,
This is not an urgent issue, but following #4283, I checked the whole list of ChEBIs newly identified as polymers according to their molfiles, whereas they are not.
Molfiles to check
29441
29443
29444
36826
36827
52445
73975
145171
145172
Cheers, Anne

molfiles for the above compounds have been regenerated.


Does not reconnect properly when using https

To solve the problem, start_tls needs to be called in connection_completed so it gets called each time the connection is connected.
connection.start_tls if options[:ssl] needs to be removed from self.connect
and the following needs to be added to the connection_completed callback:
self.start_tls if @options[:ssl]

I'm having trouble with my connection getting dropped while using the twitter-stream gem, and not getting restarted. I'm not seeing any obvious errors being logged and I'm wondering if this is the issue. I've written integration tests to confirm that reconnection is happening properly using Mockingbird to simulate the Twitter Streaming API, but it doesn't have support for SSL, so I don't have the ability to test it in the same way.
Have you wrote any tests for this?

No - but it's very likely this is the issue for you.

So, made this into a pull request with tests, hoping it will be pulled in soon :)

Awesome! Not sure if it will get pulled in soon as @voloko seems to have not been active in this repo in some time.


Summer of Hacks 2016 #2: Game Dev Day retro

Any observations/comments would be appreciated. Especially with some kind of action that we can take forward.

Add a comment to this issue, and we'll update the main body to gather everything together

What went well

Great amount of promotion/hype leading up to the day meant that all of the places were filled
Chasing/reminding attendees closer to the day to make sure they were still coming helped transition a lot of people from the wait list to the attending list
We had a projector!
We had speakers and I provided a cracking couple of playlists
We had our youngest attendee to date, and he had a great time
Nearly everyone, regardless of background/skill level jumped in to using the technology that was promoted/suggested -- this was great and meant that at the end of the day there were some really cool games to demo
We remembered the stickers
Important info was upfront, so everybody got on the WiFi and got started ahead of schedule.

What didn't go so well

Lunch was ordered quite late in the day, which in turn made the afternoon shorter
One of the burrito orders got lost
We ran out of time to present demos in the room as we had ran over, so had to kerfuffle demo in the street while dodging cars and pedestrians
Starbucks in Oxford didn't have any coffee urns available, so Ben (legend) had to go to Summertown Starbucks to collect some, and bring them back to Oxford

(I'll think of more and add to this)

It looks like many teams worked great, but maybe a few people (me included) kind of did their own thing, even though it's loads more fun when everyone is working together. It was definitely my fault, but I wonder whether there could be some change of format that made it less easy for people like me to just sit down and start hacking on stuff :)
Maybe just going around and having everyone say what stuff they're interested in doing (graphics, gameplay, art, networking, etc) might make it easier for people who'd come with an idea to find a way to fit that into a bigger project? (I arrived at 9:45, so it's possible I missed this if you did something like it?)

I kinda liked working on my own as I can go at my own place but I made sure to wander about and see what other people were doing.
Burritos - we should save a copy of the list before going and take some better bags to carry them in (ones that won't fall apart if leaks).
Projector - great to have it but in a really awkward place.
Heat - it was hot. Make the weather better next time.

@gfwilliams this is more what I had originally intended, but ran out of time preparing for it.


Add track count

Dear Reviewer,
I am pleased to submit this pull request that adds a highly requested feature to our music service: track count.
Prior to this pull request, track count information was not available to users of our music service. With this feature, users can now quickly and easily access the track count information for any album or playlist. This will greatly enhance the user experience and make it easier for users to compare and choose the right music to listen to.
To achieve this, I have added a new field on the client-side that displays the track count alongside the album or playlist name in the appropriate views. I have also thoroughly tested this new feature to ensure that it is both accurate and performant.
I believe that this feature will be a valuable addition to our music service and I am looking forward to your review and feedback.
Thank you for considering this pull request.
Best regards,
あやや

bruh.

incorrect chatgpt descriptions win again


Custom Annotations Will Continue

Depends on https://github.com/json-schema-org/json-schema-spec/pull/1394
(I don't think we have the PR dependencies action here.  Maybe I should add it at the org level.  Can I do that?  Does that work?) (It works now 😉🎉)

Well done write up! Thank you!
I'll merge and publish!
Please coordinate with @benjagm for social postings =D


Step missing before enabling AAD authentication on the Gateway

Point-to-site configuration with OpenVPN needs to be enabled first, before enabling AAD authentication (Step 8) on the gateway, otherwise the PowerShell command will fail with "Value cannot be null".

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

This document aims at outlining the process on how to set up an Azure AD tenant for P2S Open VPN authentication. The title of the document highlights this & the OpenVPN document does begin with providing prerequisites and steps to create a P2S VPN.
Furthermore, the “Next steps” section in the same document links to the doc you’re referring to (which contains the steps to ‘Configure a VPN client for P2S VPN connection’ section).
Kindly let us know if this helps or you think additionally clarity in the document is necessary. I can loop the author for further review.

@enpiri , We are closing this issue for now.If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

Should have prerequisites listed

Hi! I've added an additional step with links to create the gateway properly. This update will publish this week.


Windows PowerShell should be in a "detected" section?

I think we used to have a "detected" section when creating a terminal using a profile:

Without a separate section, you don't know which are your actual profiles and which are the other less important ones.

explanation here
https://github.com/microsoft/vscode/issues/138558#issuecomment-988454665

I think we need to clarify the concept of autoDetected. there are these that are autoDetected, but those are often part of the config. (powershell for instance)
https://github.com/microsoft/vscode/blob/5248915eadd04a8723f7d15789e61e0ea49d2f3a/src/vs/platform/terminal/node/terminalProfiles.ts#L92-L119
maybe available would be better? or outside of configured?


Native Tick Timers

What problem does this solve or what need does it fill?

Tick based timers are timers that operate not on real time, but on the number of state updates that occur. Each state update constitutes a "tick".
For any timer that does not update outside a game session, a tick based timer is preferred. This makes games more consistent and replayable (which also means they are easier to debug).

From the bevy_tick_timers crate.
There are three arguments for moving this in-tree IMO:

Ensure API consistency and stability.
Encourage beginners to use tick-timers for most gameplay, rather than more complex and flaky real-time timers.
Allow us to use tick-timers internally, such as for delayed state chart transitions (see #1597).

What solution would you like?
Borrow from @maplant's solid foundation and create a Bevy drop-in-replacement for ordinary real-time timers.
What alternative(s) have you considered?
Continue to rely on a third-party library. I've argued above why I think this is inferior.
Additional context
None.

I'm very glad that you found it useful! Take whatever you need from the repo.
Because you're integrating it directly into Bevy you could consider using it as a wholesale replacement for the current schedule, a notion I wrote up about here: https://github.com/maplant/bevy-tick-timers/blob/master/about-tick-timers.org#a-timer-based-approach-for-scheduling
I think that the effort and amount of changes would probably make it not worthwhile, however it does have the nice property that single use timers can take self: Box<Self> and thus can move items out of their struct without mem::replace, while timers that continuously rerun still take &mut self.
Let me know if you need any help from me!


HHH-14021 - Add Java Type byte to JDBC Type TINYINT mapping

According to JDBC API Specification (TABLE B-2), the Java type Byte is mapped to JDBC type TINYINT.

Could anyone say why this pull request is still not merged?

thanks. Actually I am community contributor just like you and I have no
privilege to have your PR merged.
On Fri., Jul. 3, 2020, 05:08 Lefteris Katiforis<EMAIL_ADDRESS>wrote:

@Leuteris commented on this pull request.
In
hibernate-core/src/main/java/org/hibernate/type/descriptor/sql/JdbcTypeJavaClassMappings.java
https://github.com/hibernate/hibernate-orm/pull/3404#discussion_r449471263
:

@@ -103,6 +103,7 @@ public Class determineJavaClassForJdbcTypeCode(int typeCode) {
workMap.put( BigDecimal.class, Types.NUMERIC );
workMap.put( BigInteger.class, Types.NUMERIC );
workMap.put( Boolean.class, Types.BIT );

workMap.put( Byte.class, Types.TINYINT );
workMap.put( Short.class, Types.SMALLINT );

@NathanQingyangXu https://github.com/NathanQingyangXu Nathan, the Basic
Types table is already updated. Please take a look at:
documentation\src\main\asciidoc\userguide\chapters\domain\basic_types.adoc
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hibernate/hibernate-orm/pull/3404#discussion_r449471263,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6UYAUA7SMWFTPSRCYY3W3RZWN2BANCNFSM4NDI2NCQ
.

Applied upstream.
Thanks @Leuteris for the PR.


Smart folder ?

Hi and many thanks for this powerfull GUI.
Have you planned to add some kind of Smart Folder in the packages view ?
If not can you add a filter to show packages without any catalogs ?
Thanks

Thanks for the feature request. More powerful searching for packages (and manifests) is on my todo list. I'll keep this open until I have something implemented.


Add possibility to use ambient credentials for login to vault

What this PR does / why we need it:
Made secretRef field optionally in kubernetes auth and when it's omit and flag cluster-issuer-ambient-credentials set to true will use  ambient credentials (mounted service-account token).
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #4733
Release note:

NONE

/area vault

/assign @maelvls

/assign @wallrj

Hi! As part of the effort of migrating from Bazel to Make, we have added two new Prow jobs that require this PR to be rebased on top of master in order to pass. The two Prow jobs that were added are pull-cert-manager-make-test and pull-cert-manager-make-e2e-v1-23.
Please let me know if you need help rebasing. Thanks! 🙏

Hey! We have merged a very similar feature that I call "serviceAccountRef" in #5502. We just released v1.12.0-alpha.0 so that the community can try it out. I am particularly eager to hear your feedback. The feedback can be given either as a comment to #5502, or in the Kubernetes Slack.


Separate test launch from test coverage

Introduced two new options: --mode and --sessionName. Now it's possible to perform two separate launches of the executable, with the same session name provided - that's good if we want to run tests by some other means rather than by this executable via the --query option. For example:
SqlCoverCore.exe -m OnlyStart -s some_unique_id_1 -c Get-CoverTSql -e Export-OpenXml -k "Server=server_name_1;Trusted_Connection=True;" -d database_name_1
SqlCoverCore.exe -m OnlyStart -s some_unique_id_2 -c Get-CoverTSql -e Export-OpenXml -k "Server=server_name_2;Trusted_Connection=True;" -d database_name_2

(here you may run any tests from any tools at servers server_name_1 and server_name_2)

SqlCoverCore.exe -m OnlyStopAndReport -s some_unique_id_1 -c Get-CoverTSql -e Export-OpenXml -k "Server=server_name_1;Trusted_Connection=True;" -d database_name_1
SqlCoverCore.exe -m OnlyStopAndReport -s some_unique_id_2 -c Get-CoverTSql -e Export-OpenXml -k "Server=server_name_2;Trusted_Connection=True;" -d database_name_2

Changes proposed in this pull request:

described above

How to test this code:

described above (see after "For example:")

Has been tested on:

SQL Server 2017

@a-yumashin  make sure to update the yml pipeline file for it to pass after this update

@a-yumashin make sure to update the yml pipeline file for it to pass after this update

I'm afraid I got no experience with this stuff. Could you please explain in more details what should I do and where? )


Create an Enemy object

An Enemy object has the following properties:

name
weapon
health
strength
agility
potion

An Enemy object has the following methods:

getDescription()
getHealth()
getAttackValue()
isAlive()
reduceHealth()

Write appropriate tests for the Enemy() constructor and methods.

Integrated under: commit a149fb30a388a140ae1cae6ee72f733b670962b6 .


Improve Crystal version detection

Error if git is unavailable/failed and no explicit version is given
Handle shallow clones by assuming being compiled by the latest
released compiler
Preserve commits since release in version number unless there are none

The last point means the head version now looks like this: Crystal 0.18.0+23 [adf408d] (2016-06-16), with Crystal::VERSION being 0.18.0+23. This is in accordance with http://semver.org/#spec-item-10
Travis does shallow clones, but it will always compile with the latest released compiler, that means with this the first std_spec run will have Crystal::VERSION be the latest released compiler, so currently 0.18.0, the second std_spec run will have Crystal::VERSION with the shallow clone case, so currently 0.18.0+?.
The build in this PR will fail as long as CRYSTAL_CONFIG_VERSION is set via Travis web interface. This is why it's always to prefer to set environment variables via .travis.yml or bin/ci.

Note that I'd like to release 0.18.1, not 0.18.0+15. Do your changes work with that in mind?

Yes. Note I just edited the description to clarify how versions on Travis now look like.

I also wouldn't error in the first case. A library might depend on the compiler and it would be tedious if it didn't compile out of the box and one would have to set an ENV var for that.

What about inheriting Crystal::VERSION from the parent compiler too in that case?

Good idea :-)


adding sms2mail on PID or PIS

Hello,
I would like to know if you can add this feature for futur release. I need it to read incoming SMS.
Regards,

Hi!
Thank you for your request! Currently we have no plans to support incoming SMS, but it may be something to point out in one of the upcoming releases (milesstone v1.4). Technically it is possible through the gammu library but it will need work in all of the components (wis, pis and pid) because it opens up a new functionality path. Due to our business work we aren't able to work on this until autumn.


Zed2i SVO Recording missing angular acceleration

Preliminary Checks

[X] This issue is not a duplicate. Before opening a new issue, please search existing issues.
[X] This issue is not a question, feature request, or anything other than a bug report directly related to this project.

Description
When recording SVO files there is no angular velocity stored.
It would be great to have the raw imu topic stored correctly.
Steps to Reproduce

Record an SVO file

Expected Result
All angular velocity and acceleration stored at IMU frequency!
Actual Result
No angular velocity ais stored.
ZED Camera model
ZED2i
Environment
Jetson Orin
SDK 4.1

Anything else?
Thanks a lot

Hi @JonasFrey96
this is not something related to the ROS 2 Wrapper because it uses the SVO recording module of the ZED SDK.
Please test SVO recording with ZED Explorer and check that the information is missing also in that case and eventually write an email to<EMAIL_ADDRESS>adding all the relevant information.


Update several npm packages

Update lodash from version 4.17.20 to version 4.17.21.
Update hosted-git-info from version 2.8.8 to version 2.8.9.
Update optimize-css-assets-webpack-plugin from version 5.0.1 to version 6.0.0.

@paulswartz If I'm understanding the settings correctly I think that Dependabot is configured. I actually got these updates from the alerts tab. I think it wasn't able to apply them directly because one of them was a major update, but it's quite possible something about the configuration also needs fixing…

@arkadyan https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/configuration-options-for-dependency-updates has the documentation for configuring dependabot.yml which will open pull-requests for you.

@paulswartz Thanks, that's exactly what I was missing. I'll create a new PR with that.


Ports doesnt get forwarded

While the reverse proxy feature with nginx works great I am having troubles getting regular destination nat to work.
I can see that in the container the ports get's added without a problem, it's added 3 times (port 13222) for three different tables, but I can't find the tables defined anywhere.
0100 allow log tcp from table(2) to <IP_ADDRESS> dst-port 13222 in via vnet0 setup keep-state
00200 allow log tcp from table(3) to <IP_ADDRESS> dst-port 13222 in via vnet0 setup keep-state
00300 allow log tcp from table(1) to <IP_ADDRESS> dst-port 13222 in via vnet0 setup keep-state
00400 allow log tcp from <IP_ADDRESS> to any dst-port 80 out via vnet0 setup keep-state
00500 allow log tcp from <IP_ADDRESS> to any dst-port 443 out via vnet0 setup keep-state
00600 allow log udp from <IP_ADDRESS> to any dst-port 53 out via vnet0 keep-state
00700 allow log tcp from <IP_ADDRESS> to any dst-port 80,5672 out via vnet0 setup keep-state
00800 allow log udp from <IP_ADDRESS> to any dst-port 53 out via vnet0 keep-state
00900 allow log udp from <IP_ADDRESS> to <IP_ADDRESS> dst-port 53 out via vnet0 keep-state
01000 allow log udp from <IP_ADDRESS> to <IP_ADDRESS> via vnet0 keep-state
01100 allow log tcp from <IP_ADDRESS> to <IP_ADDRESS> via vnet0 setup keep-state
01200 allow log ip from any to any via lo0
65535 deny ip from any to any

These rules are not in /usr/local/etc/ipfw.rules in the container. There's just a bash shebang in there.
On the tredly-host I am seeing no mention of port 13222. I've examined the ipfw list command as well as all the ipfw.* files.
From inside the container I can reach the service (in this case ssh) with that port without a problem.
Trying to reach the container from the tredly-host with ssh -luser -p13222 containerip fails with a timeout. And the same happens from the outside.
SSH listening on port 13222 inside the container:
root@project-microservice:/ # netstat -an
netstat: kvm not available: /dev/mem: No such file or directory
Active Internet connections (including servers)
Proto Recv-Q Send-Q Local Address          Foreign Address        (state)
tcp4       0      0 *.13222                *.*                    LISTEN
tcp6       0      0 *.13222                *.*                    LISTEN

I have the following ports defined in the Tredlyfile:
# The following is for SSH into this instance to allow for ansible to ssh into this container and do hot code reloads.
tcpInPort=13222

# Needed to install packages
tcpOutPort=80

# Needed for connections to RabbitMQ
tcpOutPort=5672

# Needed for DNS queries.
udpOutPort=53

I am running my own package repo and I wont need https and ftp outbound. The packages get's installed fine.
I am also seeing that TSO doesn't get shut off by default. I think the issue is that since net.inet.tcp.tso is a read/write tunable it needs to go into /etc/sysctl.conf or else it wont get honored.
Loader tunables (The ones that can only be set at boot) goes into /boot/loader.conf and read/write tunables (the ones that can be changed anytime) goes into /etc/sysctl.conf.
If I can provide more information in order to debug these problems let me know. I would be happy to help.

Thanks for the feedback!
Have you tried whitelisting the hosts/ranges attempting to connect to your ssh container with ipv4Whitelist= in your tredlyfile? And does your container have a containerGroup?
Currently the ipfw table files within the container aren't populated (will be rectified soon) but the LIVE ipfw tables are populated - these can be seen with "ipfw table  list" run within the container.
With respect to the TSO setting in loader/sysctl - I have added this option to the sysctl file as well as the boot loader, but since we have literally done a release in the past hour, it won't make it in until next release.

Hi, thanks for the quick reply.  I just tried with containergroup and ipv4Whitelist=<IP_ADDRESS>/0 in separate runs. As well as setting both options at the same time.
I can now ssh into the container from the tredly-host but not from outside the tredly-host. This is regardless of cointainergroup but ipv4whitelist needs to be set.

Can you please show me the contents of the ipfw tables within the container by running the following commands within the container:
ipfw table 1 list
ipfw table 2 list
ipfw table 3 list
These tables relate as follows:

whitelist for containergroup members
whitelist for partition members
whitelist for container (comes from ipv4Whitelist= in your tredlyfile)

With respect to the firewall rules, the logic within tredly is as follows:
if a whitelist is set in the tredlyfile, allow that whitelist to communicate in on tcp/udpinports
else if a containergroup is set in the tredlyfile, allow members of that containergroup to communicate in on tcp/udpinports
else allow anyone to communicate in on tcp/udpinports
What is the contents of /usr/local/etc/ipfw.layer4 ? Its possible that its a forwarding problem not a firewall problem.

Hi, sorry for the late reply
But here goes:
root@project-microservice:/ # ipfw table 1 list
<IP_ADDRESS>/32 0
<IP_ADDRESS>/32 0
root@project-microservice:/ # ipfw table 2 list
root@project-microservice:/ # ipfw table 3 list
<IP_ADDRESS>/0 0

(There was no output for table number 2.)
/usr/local/etc/ipfw.layer4:
root@tredly02:/home/tredly # cat /usr/local/etc/ipfw.layer4
#!/usr/bin/env bash

. /usr/local/etc/ipfw.vars

# Configure NAT and port redirection on the Tredly-Host external IP
ipfw -q nat 1 config ip $eip same_ports log \
redirect_port tcp $p7ip:80 80 \
redirect_port tcp $p7ip:443 443 \

Yes definitely looks like a port forwarding bug. The output from /usr/local/etc/ipfw.layer4 should include port 13222 to the ip address of your container. As a workaround you can add an extra redirect_port line in the format:
redirect_port tcp <ip address of container>:13222 13222
to the end of the file, then execute it with sh:
sh /usr/local/etc/ipfw.layer4
and the forward should work.
I'll raise this to critical and a fix will be with the next release.

Yes that worked, thank you! :)

This discussion has been movied to the tredly/tredly issue tracker https://github.com/tredly/tredly/issues/36


nxglib: Remove warning "Assuming 24-bit color is not packed"

Signed-off-by: Xiang Xiao<EMAIL_ADDRESS>

There is an issue with the graphics/nxglib/nxglib_fillrun.h header file.
It is not really a header file.  It is a "template" of a C file that is included with some definitions to make a complete C file.  So it will NOT pass the tests for a header file.
It is not clear how this case should be handled.  I believe we either have change the file name extension (but then it won't be checked at all) or change the extension to .c (which is misleading too).


Documents key indexes, affected by recent code and changes in-flight

This documents certain index tables used in Cassandra, particularly
those implicated in the more common queries.

going to merge this as I'd like to do some cleanups on top. if there are any comments, happy to change the docs post commit


Nukus branch of Tashkent University of Information Technologies

Nukus branch of Tashkent University of Information Technologies named after Muhammad al-Khwarizmi
Website: https://tatunf.uz/
Location: https://maps.app.goo.gl/Ymh7T1VNA1719NVR8

@turdibekjumabaev First stage of verification passed, official domain is facebook.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://m.facebook.com/tatunf.uz/events/?locale=nb_NO

@philipto Thank you very much 🎉🎉🎉


Detect Pending_rerun state for spark application as a prometheus metric

I am using a
apiVersion: "sparkoperator.k8s.io/v1beta2"
I have a kind
kind: SparkApplication
under which there are many spark jobs. However one of the SparkApplication is under PENDING_RERUN state, this is now for more than 35 days . We want to detect this problem and give a prometheus alert. Is there any prometheus query that catches this condition?
my kubernetes query gives this result.
kubectl get sparkapplication -n <>
NAME                 STATUS     ATTEMPTS  START         FINISH    AGE
A                PENDING_RERUN          35d
I had a look at documentation  and used all of these
"spark_app_count"
"spark_app_submit_count"
"spark_app_success_count"
"spark_app_failure_count"
"spark_app_running_count"
however none of these catch the PENDING_RERUN state.
thanks in advance

I have faced the same issue. On uninstall/reinstalling the spark application the status went to RUNNING. But there is no way to identify such issues in real time.

/reopen
/lifecycle frozen


hotpatch: implement /hotpatch enable & disable

Allows developers to turn on or of the usage of hotpatch.  (enabled by default)
I also figured that it would be pretty helpful to know who disabled hotpatch to ask them why before undoing their action.

I don't think we need the enable variant.

Do you think it could still be useful, though?  I mean, you could use eval to turn it off obviously, I guess - if need be.

As a general rule, eval-patching /hotpatch is a terrible practice. Hence why having a predictable command for disabling it is the best way to change its behavior.
Under normal circumstances, a server whose internal state has come into an un-hotpatchable status will not become patchable again.
The only way to fix it would be with eval. Someone running such eval commands successfully will probably be able to deactive the hotpatch blocker flag as well. If that condition has not been met, however, enabling hotpatch is a time bomb.

Updated

(specified until the next server restart as well)

Honestly, though, hotpatching the server is fine as long as you don't run /updateserver or git pull or something...

So basically hotpatch it to the same version? !_!
It can be done just before disabling it...

I mean, it's possible to fork the code...

Like, I can still make bugfixes to commands on the server and commit those, without pulling in the RoomGame refactor.

Then you wouldn't disable /hotpatch until after pulling the refactor I guess.

Will fix travis momentarily

How does this look?

Changed it to use Config and used .startsWith()

If in Config, it can be reset on editions for watchconfig = true though...

Monitor? That's a global that can't get hotpatched...

Changed it to use Monitor

Does anyone foresee any issues with using Monitor?  (BTW, that was a good catch with using Config, Slayer)


fix(curriculum): fix typos in Camper Cafe Menu

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Step 37 Add word/typo: Added the word "the" before "first" in the last sentence"
Step 78 Add word: Added the word "of" after the word "end" in the PHRASE "...add a comment at the end styles.css with…"
Step 82 Add word/typo: Added the word "over" in the first sentence before the word "them"
Step 46 Change word/typo: Changed the word "then" to "the" in the last sentence.
Step 60 Change word/typo: Changed the phrase "the font built-in to it" to "the font built into it"

LGTM 👍

I clicked "Commit suggestion" as the person I was discussing the font fallback issue with suggested. Why am I seeing this message: Kernix13 dismissed Sembauke’s stale review via 0eb9dff? What's with the word "dismissed"?

I clicked "Commit suggestion" as the person I was discussing the font fallback issue with suggested. Why am I seeing this message: Kernix13 dismissed Sembauke’s stale review via 0eb9dff? What's with the word "dismissed"?

No worries that means I need to look at your changes again after you made new ones.

Thank you, for your contribution to the page! 👍
We are happy to accept these changes, and look forward to future contributions. 🎉


Add Get Access Token sample

What this PR does / why we need it:
This sample allows you to get an access token using the OAuth2 client
credentials grant type. You create a Server to Server app and get a
client ID and client secret. Using these, you can get an access token
from the added API call.
This also includes an update to the VMC Environment file for client_id
and client_secret.

@jobingeo Yes, I tested this and it works fine. There is also a test to ensure that an access token is returned.

@jobingeo Ok, I've updated the variable names.


Problem: HTML form is not using the parser, no results from parsing is shown

Solution: Integrate parser, report both positive and negative errors
Testcase: Modify existing UITestCase to ensure correct and invalid requests are sent back from the controller, highlighting parsing errors if there are any

Happy with this, closing the issue.


"user does not exist" alert on logout

When the user logs out the logout dialog says "user does not exist" - This was tested in Chrome.

duplicate with #15 (tC Create)


feat(frontbutton): add GoFrontButton check story book

"#12"

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Correction

Hi team,
I'm moving this from here: https://github.com/MicrosoftDocs/azure-docs.nl-nl/issues/355
the user is reporting an inconsistency
@leonmeijer commented 7 hours ago
Wasn't able to find "Monitoring and Reports section". It's now the "Monitoring" section.

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

ID: 04403549-f1f6-6335-ff08-a84bb6c79e75
Version Independent ID: 91e5132d-ab4c-c4c9-0999-270eac76f587
Content: Monitor Azure Site Recovery
Content Source: articles/site-recovery/site-recovery-monitor-and-troubleshoot.md
Service: site-recovery
GitHub Login: @raynew
Microsoft Alias: raynew

@leonmeijer, Thanks for reporting this! We have made the appropriate changes in the document and published changes would be live within 24hrs.

#please-close


Empty @WeAreSitecore Twitter Feed on Community Page

I'm seeing an empty feed for the @WeAreSitecore account on the Community page.
I'm guessing it's as the account hasn't tweeted in the past 11 days.

Maybe want to override the settings for this feed to have it display the most recent X amount of tweets regardless of age?

I looked into this, and it appears to be a limitation on the Twitter API itself: https://developer.twitter.com/en/docs/twitter-api/tweets/search/introduction

The recent search endpoint allows you to programmatically access filtered public Tweets posted over the last week, and is available via the Standard and Academic Research product tracks.

Also mentioned here: https://developer.twitter.com/en/docs/twitter-api/v1/tweets/search/api-reference/get-search-tweets

Returns tweets created before the given date. Date should be formatted as YYYY-MM-DD. Keep in mind that the search index has a 7-day limit. In other words, no tweets will be found for a date older than one week.

So it looks like we can't really do much about this. If the Twitter accounts are not active, they won't be displaying content. Keeping this in mind, we may want to see how we can get these accounts to be more active with regular tweeting. This may be something to follow up internally.


Fix carousel in IE11 and unset pointer cursor style

Fix #1905.

Related pull requests from dependencies

react-film#24. Fix IE11 not working with CoreJS-polyfilled [...document.children]
react-film#25. Added cursor style options.

Changelog

Note: changelog items has been correctly categorized and sorted.

Changed

component: Bumps to<EMAIL_ADDRESS>by @corinagum and @compulim, in PR #1900 and PR #1924
component: Carousel flippers on carousel layout and suggested actions will use initial cursor style, by @compulim, in PR #1924

Coverage remained the same at 54.05% when pulling 37e954d80f985312446a33fef9b9d52964914f12 on compulim:feat-react-film-cursor into 9cbbea2599dd122f55531a5329284840067663e5 on Microsoft:master.

Nice! When will this be live on the CDN?

Hey @hansmbakker, this is available immediately on master bits. If you prefer to wait for the official release, 4.4 is to be released at the end of the month.


Incorrect information in the tutorial

In the documentation tutorial, the information regarding the type of application to be setup in the developer console is wrong.
Instead of "Select ‘Application type’ to be Web application.", it should be "Select ‘Application type’ to be 'Other'."

I got it working fine as a web application by following the guide. Were you having a particular issue with getting it up and running?

IIRC, wetting as Web Application lets a sort of virtual user access the drive, which is rather impractical when one has purchased additional storage.
At least I ran into that problem, and seem to remember it having something to do with the application type.

My code does not work if I set up the project and credential settings exactly as described in that tutorial.
As soon as I execute the script for the first time, I am redirected to the Google authentication page, and on that page, the following error message is displayed:
400. That’s an error.
Error: redirect_uri_mismatch
Application: <project name>
You can email the developer of this application at: <my email>
The redirect URI in the request: http://localhost:8080/ did not match a registered redirect URI.
Learn more
Request Details
    access_type=offline
    scope=https://www.googleapis.com/auth/drive
    response_type=code
    redirect_uri=http://localhost:8080/
    client_id=402487371318-lgb5025sa7jpet5qj5deohnmrj3q9kmg.apps.googleusercontent.com
That’s all we know.

@Anmol-Singh-Jaggi
I was having exactly the same issue.
For the Authorized JavaScript origins: http://localhost:8080 (no slash at the end like the guide says as google won't accept it)
For the Authorized Redirect URIs: http://localhost:8080/ (exactly as guide says)
Once that's done then save the JSON and rename it appropriately and give it another shot.
@Fjodor42
I don't quite follow. :(
I'll admit I don't have a great understanding of how this works and how the different application type affects this but isn't it only relevant here for authorisation? Web app = able to authenticate through a web page and then once it's authorised all the meaty stuff just goes through the api. But I could be wrong. :D Also I can't see how you'd run into problems with additional storage since wouldn't this just continue to operate normally as if you hadn't bought extra storage except now you can have more files in your drive which can be operated on by the api?

@Atrolantra
The two things may be unrelated as you say - if so I just happened to experience both problems at once. To clarify, that other problem was files were going to some storage belonging to the application, as opposed to storage belonging to me. As I have ~½TB of backups, needless to say, the default amount wasn't enough.
As said, it might be unrelated, so I shall keep quiet about it in this thread, except for the clarification above.
My apologies for the noise.

@Atrolantra Thanks! It's working now. It would be better if this was updated in the tutorial as well.


Firebase Crash Reporting OR Crashlytics?

I'm using firebase solely for the purpose of crash reporting, we've already implemented a myriad functions (notifications, analytics, etc) using other plugins/services.
Can someone walk me through the steps of getting this running on my ionic/cordova app?
What I've done thus far:

Install plugin
Add .plist file
run $ ionic cordova prepare ios
Added this code, which made my "firebase not setup" error message in the log go away
#import <FirebaseCore/FIRApp.h>
[FIRApp configure];

In the firebase website/console my only option is to setup Crashlytics.
Do I need to do something with cocoa pods? Do I need to download an configure symbol file so that I can get readable crash reports? Do I need to make the app crash? Do I need to install the firebase SDK?
I'm not seeing anything on the firebase website that indicates I have crashlytics or crash reporting setup. Analytics is populated with some stuff. No error messages in the console.
Also, when this plugin is installed I get a pop-up that asks for a password [Environment: Sandbox] It seems like it's triggering a bunch of in-app purchase code that we've temporarily disabled — I haven't looked into that yet.

Want to know this, too.

@joshbabb @jcyh0120 I've abandoned firebase and moved to rollbar for my ionic/cordova app. Setup was lightening fast and we're so relieved to finally have reporting on iOS and Android.
This supports javascript error reporting, we're now paying customers.  https://rollbar.com

@pvandrunen thanks! For hybrid app like ionic, it's always confusing to use Javascript SDK or Native SDK with cordova. Any suggestion for using rollbar with ionic using JS/Native SDK?

@jcyh0120 Why not both!?
I have both the ng-rollbar installed and the cordova plugin. Javascript is very useful but I kinda forgot that I had installed the cordova plugin, I never was able to simulate a crash.  I seem to get pretty comprehensive reports in rollbar, I couldn't say if the cordova plugin has much to do with the reports that I'm seeing. Sorry I can't be more help.

@pvandrunen Great idea! I've been using Firebase CrashReport to log my JS error. But it's really not that handy for JS. I never think of using both before. They have different roles. Maybe I should log my JS error with ng-rollbar and  log my crash with native SDK.  Thanks for reply.

closing in favor of https://github.com/arnesson/cordova-plugin-firebase/issues/548


update product webhook - old data is being sent for channel_listings

What I'm trying to achieve
Update the product from the saleor dashboard and catch the updated data in the webhook.
Steps to reproduce the problem

Create a product from the dashboard or using graphql API. ( I created from both)
Following are the steps to create a product.
Create_product_query,
Add_product_to_channel_listing,
Update_product_variant_channel,
Create_digital_content.

What I expected to happen
update_product should send the updated data while calling the webhook.
Screenshots and logs

System information

Saleor version:

[ ] dev (current main)
[ x ] 3.2
[ ] 2.11
[ ] 2.10

Operating system:

[ ] Windows
[ x ] Linux
[ ] MacOS
[ ] Other

@g-londhe dashboard is triggering several mutations on save, therefore there should be several webhook calls with different payloads.

@timuric none of those have new data for channel_listings, always values before save are received in web-hook.

+1
Noticed that for channel_listings, always values before save are received in web-hook.
New values are not received.
Need to save twice from admin dashboard to get new data.

Thanks for reporting, we will dive in next week.

@timuric any idea in which release this will be fixed?

@timuric any idea in which release this will be fixed?

I am confirming this with the QA team.

This majorly affects everything built using external apps which heavily rely on webhooks.


Delicious mimana cream pie

About the PR
i had a dumb idea
we're not going to talk about it
because I hit you with a mimana cream pie and now you can't speak
(actually, this shitcode could be expanded to other sorts of... less than desirable pies to be hit by... but not without some work)
(todo: face visuals that look like mimana cream)
Why / Balance
mimana funny
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

add: Added the mimana cream pie, which temporarily silences its slapstick victim. Who's laughing now?

the syndie ambushes are gonna be CRAZY wit this one

voice communication is not available for this account

I think the clown being able to make special pies that have fun effects like this would be fine, but that would require unique cooking recipes which we don't have and a more general design on the types of pie that can be added and their effects (a cool one that could replace this is one that changes speech in a fun way), but making this available to the whole crew is more of a nuisance than what its worth, feel free to revisit if we ever do have special recipes like that.

Make them an uplink item.


feat(FormlyField): allow setting className for field which contains fieldGroup

What kind of change does this PR introduce? feature

This change is

Codecov Report

Merging #392 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #392   +/-   ##
=======================================
  Coverage   92.47%   92.47%
=======================================
  Files          15       15
  Lines         757      757
  Branches      197      197
=======================================
  Hits          700      700
  Misses         57       57

Impacted Files
Coverage Δ

src/core/components/formly.form.ts
89.87% <ø> (ø)
:white_check_mark:

Continue to review full report at Codecov.

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


How do you make a game

Is there any tutorial on making a game

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

ID: 8a1daec8-269e-40fd-0cc3-6c38f9074399
Version Independent ID: e9b4d9e3-8692-339b-e1f3-ff417d1a09cd
Content: Develop apps for the Universal Windows Platform (UWP) - Visual Studio
Content Source: docs/cross-platform/develop-apps-for-the-universal-windows-platform-uwp.md
Product: visual-studio-dev15
GitHub Login: @stevehoag
Microsoft Alias: shoag

@oooobi Thanks for your inquiry. For more information about how to use UWP to create a game, please see the "Game development" section on the Universal Windows Platform documentation page.
Cheers,
~ Terry

不懂

@3121084329 谢谢你的反馈。请问, 你能提供更多的信息, 你不明白, 以便我们可以帮助？
(Thanks for your feedback. Please, can you provide more information about what you do not understand so that we can help?)


order_delivery_position.total_price and .unit_price has wrong type in database

PHP Version: 8.3.12   | Shopware Version: <IP_ADDRESS>  | Affected area / extension: Platform(Default)

Actual behaviour:
order_delivery_position DDL look like this
create table order_delivery_position
(
    id                         binary(16)  not null,
    version_id                 binary(16)  not null,
    order_delivery_id          binary(16)  not null,
    order_delivery_version_id  binary(16)  not null,
    order_line_item_id         binary(16)  not null,
    order_line_item_version_id binary(16)  not null,
    price                      json        not null,
    total_price                int as (json_unquote(json_extract(`price`, _utf8mb4'$.totalPrice'))),
    unit_price                 int as (json_unquote(json_extract(`price`, _utf8mb4'$.unitPrice'))),
    quantity                   int as (json_unquote(json_extract(`price`, _utf8mb4'$.quantity'))),
    custom_fields              json        null,
    created_at                 datetime(3) not null,
    updated_at                 datetime(3) null,
    primary key (id, version_id),
    constraint `fk.order_delivery_position.order_delivery_id`
        foreign key (order_delivery_id, order_delivery_version_id) references order_delivery (id, version_id)
            on update cascade on delete cascade,
    constraint `fk.order_delivery_position.order_line_item_id`
        foreign key (order_line_item_id, order_line_item_version_id) references order_line_item (id, version_id)
            on update cascade on delete cascade,
    constraint `json.order_delivery_position.custom_fields`
        check (json_valid(`custom_fields`)),
    constraint `json.order_delivery_position.price`
        check (json_valid(`price`))
)
    engine = InnoDB;

In Json price object totalPrice and  unitPrice stored as float value
Expected behaviour:
order_delivery_position.total_price and  order_delivery_position.unit_price should also from type float
How to reproduce:
generate float values in price

[public] Shopware automated response: This issue is linked to the internal issue: NEXT-39923
[created from NEXT-39923, comment 521150]

[public]

Priority changed to: Low

Thank you for your report. After careful consideration, we've concluded that this issue does not require immediate attention or isn't something we can address right now. We prioritize openness and clear communication with our community, so we want to be transparent. We may not resolve all reported issues, and this one might remain unresolved. To stay effective, we periodically clean our backlog, and this issue could be closed in the future.
For now, we'll keep this ticket open to invite further feedback from the community in case we've underestimated its impact and to allow us to revisit it later.
[created from NEXT-39923, comment 521601]


I just want to say thank you!

Dear MediaPipe Team,
I just wanna say thank you to develop such a wonderful AI, and it helps me start my APP business successfully.
Here is what I have developed with MediaPipe.
PinoQuest: https://apps.apple.com/us/app/pinoquest/id6444877037
PinoSoft: https://apps.apple.com/us/app/pinosoft/id1643661244
I wish we can further cooperate to develop the APP and make it more popular, although it has been popular so far.
Hope we can talk more about it. my email<EMAIL_ADDRESS>Thank you! Salute!

Hi @fchen09,
Glad to hear that! Thank you for the kind words! This helps us to improve our product and can see larger community are adopting the mediapipe. Hope product will be more helpful for the developer community to do build the application as per individual or organization requirements.

@fchen09,
would you go through the above comment and please feel free to close if no longer issue from your end. Thanks!


Delete old events from myTOA?

My myTOA shows teams and events from last season. If I want to remove an old team, I can click on that team and then click on the "remove from myTOA" button in the top right.
But if I want to remove an old event, I click on the event and I don't see any similar "remove from..." option.
Does this button need to be added to old events? Or is there another way to remove old events from myTOA?

Events have more advanced settings, you need to click on the star button on the right bottom.

Thanks!
Allow me to change this to a new feature request then:
After a season is completed, it might be nice to have a "purge old events" button that could purge all events from the previous season, so I don't have to go through each event and remove them one-by-one.
Or else perhaps have the myTOA events organized by season, or sorted with the most recent event at the top??

I think the best way to tackle this would be to, at the beginning of each season, put a card on the user's myTOA dashboard stating "Hey, it's a new season. Would you like to clear your favorite events?" or something along those lines.


iOS - visual glitch - Save button truncated on Settings page

Describe the bug
The "Save" button is truncated next to  the "Username" input field.
To Reproduce
Steps to reproduce the behavior:

Log in to freeCodeCamp.
Go to the "Settings" page.
See the "Save" button is truncated next to the "Username" field.

Expected behavior
Button should be fully readable.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):
Not tested, no access.
Smartphone (please complete the following information):

Device: iPad
OS: 12.0.1
Browser: Safari
Version: whatever is installed with iOS 12.0.1.

Additional context
This only happens in portrait mode. Landscape mode is fine.

It looks like this is already fixed in master.

It's displaying properly for me now. 🙂

Thank you, I'll close this as soon as the fix is released on the live site.

@RaspberryLime, can you check if this is still and issue on the beta site?

I’m unable to log in on the beta site — do I need to create a new account there?

Yes, you'll need to make a new account. Could you make one and report back if this is still and issue?

This is indeed fixed on the beta (the save button is now underneath the field instead of alongside it).

It’s also fixed on the live site too — the button hasn’t been moved there, but it’s no longer truncated. :)

@RaspberryLime, great! Thank you for checking this out.


Failed building 1.6 from source on docker

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version 1.4 in order to build go 1.6

What operating system and processor architecture are you using (go env)?

GOOS linux
GOARCH amd64

What did you do?
(Use play.golang.org to provide a runnable example, if possible.)

What did you expect to see?

--- FAIL: TestHostname (0.00s)
	os_test.go:1287: fork/exec /bin/hostname: no such file or directory
--- FAIL: TestCloneNEWUSERAndRemapRootDisableSetgroups (0.00s)
	exec_linux_test.go:34: Failed to stat /proc/self/ns/user: stat /proc/self/ns/user: permission denied
--- FAIL: TestCloneNEWUSERAndRemapRootEnableSetgroups (0.00s)
	exec_linux_test.go:34: Failed to stat /proc/self/ns/user: stat /proc/self/ns/user: permission denied
--- FAIL: TestEmptyCredGroupsDisableSetgroups (0.00s)
	exec_linux_test.go:34: Failed to stat /proc/self/ns/user: stat /proc/self/ns/user: permission denied

What did you see instead?

I expect those test run

This isn't enough information to help you.
What does "on Docker" mean? In a Docker container? Where is your Dockerfile?

sorry, there is my dockerfile
my purpose is to build from a clean env (could be a Vagrant on a VirtualBox)
FROM fedora:latest
MAINTAINER<EMAIL_ADDRESS>
RUN dnf -y update
RUN dnf -y install @development-tools @rpm-development-tools git golang

RUN dnf -y install wget
RUN wget -O /tmp/go.tgz https://storage.googleapis.com/golang/go1.4.3.linux-amd64.tar.gz
RUN mkdir /root/go1.4
RUN tar -xvzf /tmp/go.tgz -C /root/go1.4
ENV GOROOT_BOOTSTRAP /root/go1.4/go
RUN echo localhost > /etc/hostname

RUN git clone https://go.googlesource.com/go /opt/go

RUN cd /opt/go && git tag | sort -V | grep -v -E 'weekly|release|beta|rc' | tail -n1 | xargs git checkout

Sent https://go-review.googlesource.com/20342 to skip more tests.


⚠ Bofrost.at has degraded performance

In ca6104d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1554 ms

Resolved: Bofrost.at performance has improved in 69c29df after 6 minutes.


fix(compaction): fix in-correct epoch watermark

Signed-off-by: Little-Wallace<EMAIL_ADDRESS>I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
We need to use global-epoch-watermark to recycle multiple-version of the same user-key, which collect from all frontend and record in meta-service. So we can not use local epoch in shared-buffer.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)

@mergifyio requeue


function improvement

add lab02 and lab03-1

Nice job!
You may consider checking out some data/time libraries in Python to help you better manage time.


Osprey errors with reference to Azure.core.dll

When running Osprey with the most up to date graph and EXO modules, you may get an error referencing Azure.core.dll.
Temporary fix is to downgrade either EXO to 3.4.0, or Graph to 2.19.0.
Ultimately the fix is up to Microsoft as it's an issue with their modules.

probably fixed in newest eop preview, needs testinggg

Fixed applied in dev


Person group ID is invalid.

the following code (vb.net vs 2015)
 Dim PersonGroupId As String = "Justme"
 Await IFaceServiceClient.CreatePersonGroupAsync(PersonGroupId, "Just_me")

generates an exception, which reads:
BadArgument
Person group ID is invalid.
Prior to this i am able to run the line:
Dim faceRects As FaceRectangle() = Await UploadAndDetectFaces(CurrentFilePath)
with results as expected.

In very small print, the spec stipulates that uppercase letters are not permitted.

personGroupId as a string. The valid characters include numbers, English letters in lower case, '-' and '_'. The maximum length of the personGroupId is 64.

Hello,
sadly couldn't success
can you share code samples?
plus on xamarin components store, there is only surf guy sample,
dialpad?
tr.linkedin.com/in/mustafahalukyilmaz/
Date: Mon, 14 Mar 2016 12:19:54 -0700
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Subject: Re: [ProjectOxford-ClientSDK]  Person group ID is invalid. (#35)
In very small print, the spec stipulates that uppercase letters are not permitted.
personGroupId as a string. The valid characters include numbers, English letters in lower case, '-' and '_'. The maximum length of the personGroupId is 64.
—
Reply to this email directly or view it on GitHub.

I'm afraid I don't understand the question.  Are you having issues specifically with PersonGroups?  If not, I encourage you to open a new issue so as to reduce confusion.

Thank you ctrash, I missed where it said upper case letters are not permitted.  Changed to all lower case and it worked like a charm.  Crazy.  Thanks, Rob


Support removing "Remote" domain name

We no longer have anything deployed in Render or other hosts, so make sure the Terraform configuration can handle removing the remote domain name variable.

I’ve also updated the variable on Terraform Cloud, so merging this will remove the domain name (render.getmyvax.org) and the CloudFront distribution.


Add new case Power-hibernate-SRIOV for BZ#1841973

Add a hibernation case when SRIOV is enabled.
Filter "Fatal error" logs after resume to hit the "fatal error" issue:
The patch is here: https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net.git/commit/?id=8fc3e29be9248048f449793502c15af329f35c6e

Thanks for the new TC, do you have test result with it?

Hi @juhlee-microsoft ,
Sorry for late. I just updated a little part to filter "Fatal error" logs to hit this issue.
The bug has not been fixed in RHEL-8 downstream kernel yet so the result is FAIL in mlx5.
[LISAv2 Test Results Summary]
Test Run On           : 06/09/2020 08:27:49
VHD Under Test        : https://walaautol2.blob.core.windows.net/vhds/RHEL-8.3.0-20200602.n.1.vhd
Override VM size      : Standard_D8ds_v4
Test Category         : Functional
Initial Kernel Version: 4.18.0-208.el8.x86_64
Final Kernel Version  : 4.18.0-208.el8.x86_64
Total Test Cases      : 1 (0 Passed, 1 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:29

   ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
-------------------------------------------------------------------------------------------------------------------------------------------
    1 POWER                Power-hibernate-SRIOV                                                             FAIL                24.79

And the result is PASS in mlx4:
[LISAv2 Test Results Summary]
Test Run On           : 06/09/2020 08:02:12
VHD Under Test        : https://walaautol2.blob.core.windows.net/vhds/RHEL-8.3.0-20200602.n.1.vhd
Test Category         : Functional
Initial Kernel Version: 4.18.0-208.el8.x86_64
Final Kernel Version  : 4.18.0-208.el8.x86_64
Total Test Cases      : 1 (1 Passed, 0 Failed, 0 Aborted, 0 Skipped)
Total Time (dd:hh:mm) : 0:0:31

   ID TestArea             TestCaseName                                                                TestResult TestDuration(in minutes)
-------------------------------------------------------------------------------------------------------------------------------------------
    1 POWER                Power-hibernate-SRIOV                                                             PASS                23.71

Thanks!

@dcui, this is so helpful update from redhat. Do you have any other questions we should check on mlx5 result?

@dcui, this is so helpful update from redhat. Do you have any other questions we should check on mlx5 result?

I have no question. Thank you for the new test cases, @yuxisun1217  and @juhlee-microsoft !


Cannot create AzureFunction activity via visual authoring web UI

I tried creating a AzureFunction activity but got the following error message:
"Provided type 'AzureFunction' is either unsupported or an invalid type.The relevant property is 'type'.The error occured at the location 'body/properties' in the request."

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

ID: 965d9ab8-6b54-daba-9693-206845821d0c
Version Independent ID: 08725018-92a0-6b2c-8130-6debcc535b7a
Content: Azure Function Activity in Azure Data Factory
Content Source: articles/data-factory/control-flow-azure-function-activity.md
Service: data-factory
GitHub Login: @douglaslMS
Microsoft Alias: douglasl

@douglaslMS @alch-msft
I was testing the Azure Function Activity with Data Factory and noticed the timeout is not working as expected when setting it up with values of over 1 minute, it automatically cancels the run after 1 min and I did several attempts with values under one minute and over 2 and 3 minutes and it gave the same unexpected behaviour.

@alch-msft I have created Azure Function Activity through UI and it works as expected.

Could you retry and once and see? You can follow thislink for detailed steps.
@douglaslMS Please take a look at burstingninja  issue and provide your thoughts.

@burstingninja This is a known issue - the "forced" timeout after 1 minute - and the Data Factory team is rolling out a fix. They hope that the fix will be rolled out in all regions by the end of December. (Note that I said "hope" and that this is not a promise or a guarantee!)

@alch-msft Alvin, do you mean that you have found a resolution for the issue that you originally reported?

@alch-msft Also, are you the person who asked the same question in a comment on the blog post Azure Functions now supported as a step in Azure Data Factory pipelines? If yes, have you emailed the blog author your pipeline ID as requested?

@alch-msft Are you in region WestCentralUS by any chance? Can you try in another region for the time being

Same error in the EastUS region.

Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Hi there,
Sorry for the delay in response! Is this issue still occurring? If not, can I close this?

#please-close


updated bidder name rules

@bretg The bidder code should be unique for the first 6 characters
What about bidder codes which already have the same first 6 characters before this commit?

The only problem with not being unique in the first 6 chars is that those bidders should not be run concurrently on Google Ad Manager in SendAllBids mode. Any ones that are already merged with non-unique beginnings should consider a shorter alias but we're not going to enforce it.  This doc update is for reviewers to help avoid future issues.


add ESP32/ESP8266 Agent with bootstrap device registration support

This agent supports the default registration process for new c8y devices

Thanks! Just double checking that the cert is ok to be posted publicly?

It's just the root CA Certificate of GoDaddy, which will then be used to verify the cumulocity certificate.
This is publicly available and should be now issue to have within the code.


fix: Expanded on "contributing to docs page" section : Doc - 87

Add the following sections:

Work on an open issue - Help work on what we've identified that needs fixes.
Add community resources - Add a helpful article or resource relevant to zkSync.
Pick an issue and work on it -  Work on issues that's has been raised.

Where will we include the community resources?

In the troubleshooting menu we have these items:

Contribute to documentation
Edit or review a documentation page
Adding content resources

It feels like very similar things. I'd delete the "Edit or review a documentation page" item from the menu, and include its content inside the "contribute to documentation" page. Then I'd also rename the "Adding content resources" for "Community resources".


Clean up crew reporting

Placeholding this issue to build out when I have a second
Thanks for creating an issue! Please fill out this form so we can be sure to have all the information we need, and to minimize back and forth.

What are we trying to do
We're trying to reduce the amount of drop-off (patients who call us, but are never pledged) we experience as an organization.

What feature or behavior is this required for?
DCAF would like to set up a "clean up crew" of sorts who would take a look once a week and return calls to patients we haven't spoken with recently.

We'd like them to be able to pull a report of patients that looks something like:

We have made at least one call to in the last 14 days (regardless of the outcome of the call).
There has been no change to the patient's file since that last contact.

This report would not be exportable, but would look similar to the call list. Maybe even automatically populate into the call list? Idk.

How could we solve this issue? (Not knowing is okay!)
Running the report should be pretty simple, but not sure about the logistics of how we want it to display.

Anything else?
Here's some fun housey jams to make you feel like you're partying on a beach on the Swahili coast: https://daily.bandcamp.com/2017/08/08/mim-suleiman-kawaida-album-review

Realizing we DID have an issue on this. I could not find it for the life of me last night while I was searching for it. #817
Anyway. It's pretty closely related, but that was specifically people we have soft pledged rather than all calls.
I'm gonna close this issue @colinxfleming and migrate over there.


feat: List Transactions

This PR introduces the transactions command, which allows users to retrieve transaction details.
Usage: galoy-cli transactions [OPTIONS]

Options:
-l, --last <LAST>
-f, --first <FIRST>

The first option displays the first specified number of transactions for the logged-in user, and the last option does the same for the last specified number of transactions. If neither first nor last option is provided, the command lists all transactions for the user.
Note that pagination is currently not handled by the gql query to the backend; instead, it is being managed at the app layer. The app filters through all the transactions based on the given input. Once the query has been fixed, the pagination will be handled on the backend, and additional options named before, after, and wallet-ids will be added.

I think default should be --last instead of --all
--all should be an explicit option


feat(EditFrameworkConfiguration): handle empty resolvedOptions

Some frameworks that are deployed on earlier versions of DC/OS won't have configuration information saved from cosmos/service/describe. This PR handles this in the UI by displaying a warning, and populating the edit flow with default package configuration.
In doing this change, I also refactor this flow a bit (adding some methods to ServiceUtil and cleaning up passing props
Closes DCOS-19688
Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

I didn't add tests for this because we temporarily removed these cypress tests due to the feature being disabled in open (because API not migrated yet).

Alfred test this please

@mesosphere-ci ok to test

Jira task for adding these tests once this feature is re-enabled: https://jira.mesosphere.com/browse/DCOS-19930

I've addressed the comments from @MatApple removing the Util.omit(). In looking more closely at the mozilla lib, I also noticed that I can directly call the very robust getDefaultFormState, instead of the form data init methods previously had in ServiceUtil. This getDefaultFormState is the same thing called when the form updates to reconcile the current data This is a lot nicer because it avoids us reimplementing this code.

Holding merge till #2570 is merged because it re-adds the tests and the ability to edit frameworks.

@orlandohohmeier or @Poltergeist: I want to add integration tests to ServiceActions-cy to test this warning message appearing. But I'm not sure how to do this because the CosmosPackagesActions fixtures are setup in a way to always return the same serviceDescribeFixture. In the integration tests, I don't really have the ability to modify these fixtures unless I do something like this:

@ahoskins I ran out of time to thoroughly review this again but I see you've made adjustments so feel free to dismiss my review :)

Alfred test this please

I'm not sure how to add tests for this right now, but let's merge this before wednesday. @weblancaster @bstavroulakis @juliangieseke can I get two reviews please? 👍


Distinguish preleases

See https://github.com/npm/node-semver#prelease-tags.
It could be implemented with a simple env variable, like BOOK_PRERELEASE=boolean and disabled by default.
Or the classic BOOK_ENV=development/production approach could be used also.

I think the best option is to use BOOK_PRERELEASES with a regexp and prereleases will be filtered with it.

I will use the BOOK_PRERELEASE=boolean approach and let the semver module handle the filtering of prereleases.