Add easy encryption to HighLevelMultiplayer Api.

I was thinking of some important security features that are lacking in Godot. When you have an online business it is necessary to protect the data of your users and avoid cheating. In the gaming industry it's no different, especially for companies that move amounts of money.
Due to this, I think encryptions methods on network api would be very welcome.
This will not solve all the security issues but it sure is a big step.
These features should also attract more attention to the engine from companies of all sizes and more users to Godot.
My propose is add  some encrypt method on the api.
Maybe something like:
host.set_encrypt_mode(key, settings)
get_tree().set_network_peer(host)
And if possible add support to WebSocket Security

Maybe that would also be needed for stuff as simple as an online leaderboard?

Why specifically leaderboards?

@raymoo  I was just giving an example of use case for encryption, since a leaderboard needs some way to prevent hacking results using keys.

@raymoo sending a hash from the client with scores so that you can check if it's valid server-side by computing the hash again with the same algorithm and a private key. Godot has MD5 only so far though, but maybe it's not that important in this issue...

@Zylann Makes sense, for a moment I thought the issue was to add some ssl-like layer under the multiplayer API.

SLL requires a public registry. I'm sure it's interesting for companies that leads online business, but too complicated to regular users. it's a good point to think about.
But I was thinking just to encrypt the data handled by godot's network_api with a key.
I think it's enough to strengthener a lot the Godot's security.

CC @Faless @mhilbrunner

Long story short, there are multiple ways to achieve encryption:

Implementing DTLS for UDP
Implement encryption via symmetric key (AES/whatever) as proposed.
... (many more protocols)

None of them prevents cheaters, nor users to read the content of the data being transferred (but DTLS does protect from eyesdropper, while AES alone doesn't unless you also implement secure key exchange).
If you want to avoid cheaters, make the server authoritative (i.e. do not trust any data coming from clients), there is no other way (at least to my knowledge).
If you want to protect data from the users, don't send it. If you do, no matter what you do, the users will be able to read it (otherwise the application on their end wouldn't either).
Both those options are viable, and PRs are welcome, sadly, I find myself with very little time between work, GSoC, and other personal obligations.

The first option, to add symmetric encryption APIs, should be done anyway and doesn't have much to do with networking. Something like proposed in PR https://github.com/godotengine/godot/pull/17721 but preferably with stronger defaults.
You could then encrypt data before sending (or before storing it to a file, or...).
Of course, this is only security by obscurity at best. A dedicated attacker would simply extract your key and decrypt the data.
Only way around that would be to add DTLS support (or a similar, encrypted transport that does the heavy lifting). Note that there is a GSoC project ongoing for adding WebRTC support; that uses DTLS by default, so should be encrypted.
Personally, I'm interested in the first option, to add APIs to encrypt/decrypt data manually, but not much in adding encryption support to the ENet backend. I think thats overkill. So what if somebody reads your position updates?
IMO, encrypting specific things like in-game chat is more than enough. The first option would suffice for that, or you could even use HTTPS for that right now, which would probably be enough for chats and similar messages.

I think that symmetric key encryption is probably the correct approach here. Any sort of TLS approach is rather heavy handed and has enough overhead that I worry about resulting performance.
In regards to symmetric keys, I do request that a wide array of ciphers can be used. N9t only the number of bits, but also things like the chipher mode (e.g. GCM, CBC, etc..)

@jonbonazza Does your suggestion include some method of key agreement?

Does not Godot have a method for encrypting data already?

Actually, I was thinking about it more, and I don't think symmetric key encryption will work for this particular use case after all. On of the beauties of symenc is its simplicity. There is no key negotiation--both ends of the connection simply require the same key in order to enc and dec data. This is great when both the client and server are controlled by the same entity and live within the same datacenter, but for something like a game client this won't work at all and is virtually no better than having no encryption at all. Since the key is the same on both ends of the connection, there's nothing stopping a malicious party from grabbing the key from the client, intercepting the inflight data, modifying it, and renencrypting it before passing it on to server and the server would be none the wiser.
This is actually an ideal candidate for asymmetric encyrption. Honestly, Im not too experienced in assymmetric encryption options on top of UDP. TLS is certainly the most common form of assymetric encryption, but requires TCP. DTLS is a derivative of TLS but built for UDP. I have no experience with DTLS though, so I am unaware of its downsides, nor of any alternatives.

Also, like @Faless mentioned, even TLS doesn't completely solve the cheater issue, but then again, nothing will as once you distribute a client to another, uncontrolled machine, you can't trust anything that comes from it.
So yea.. Im not even sure encrypted traffic is worthwhile in a video game (at least not from the client to the server). You are paying the performance cost of any encryption scheme for virtually 0 gain other than "nay nay nay you can't see me", which if you are using an authoritative server (and you should be!) then even that doesn't matter anyway.

which if you are using an authoritative server (and you should be!) then even that doesn't matter anyway.

The clients can't know if it's indeed the authoritative server or they're being MITM'ed (to steal passwords etc) or it's a totally fake server (similar for client to client communications), and that's the one of the arguments for having TLS in place. And this is a problem that would need be addressed before worrying about cheating.

@tagcup the whole purpose for using an authoratative server is so it doesnt matter what the client thinks. The server is the validator and the arbiter. If the client sends something the server doesnt agree with, it ignores it.

It will matter when your players gets MITM'ed and get all their passwords and other personal information owned and you get sued by some.

Honestly I envy your optimistic magical thinking

So i think you are missing the point of this issue.
Passwords and personal info would never be sent via the high level networking library. For that, you would use an http client or something, which of course should have tls support.
This is for in game networking, which if you are at all a competant game developer, you would never send PII or credentials with.

@jonbonazza I totally agree with your point of view. In some moment the key will need to be negotiated and will can be caught. it is a waste of processing.

If you want to avoid cheaters, make the server authoritative (i.e. do not trust any data coming from clients), there is no other way (at least to my knowledge).
Sad, but thue....

At this moment I would be content with 2 things:

A HTTPS server/clients tutorials to exchage my super secret stuff

Encryption to var.  [Give a shield to this man and let he fight ;-)  ]

If you're meaning gambling in particular (which, considering some of Godot's sponsors, would be logical) what are the legal requirements for that? I know they can be pretty arduous and probably beyond the scope of this issue (beyond possibly providing a framework).

@jonbonazza
Even with an authoritative server, a MITM could give bad inputs and make the player do things they wouldn't normally do (such as send all their money to the attacker's account). Maybe this could be solved by having symmetric encryption, but have the auth server provide an encryption key for the user session?
@Malkverbena

Encryption to var. [Give a shield to this man... and let he fight ;-) ]

Would have to be careful with this so that users don't get fake security because they implemented it badly. You would at least need to incorporate a (randomly-initialized) sequence number into the message. It would be ideal if the multiplayer API handled this on its own, because I think it would cover all cases of wanting to encrypt the high level multiplayer API.

Yes, DTLS (or any asymmetric ebcryption scheme) will protect from MITM, but when the client is on a user's machine nothing will protect the attacker from just grabbing your keys and sending w/e they want or performing outbound MITM that way. This is why most games dont use security at the transport layer already. You're free to use it in your game, of courae, but expect massive slowdowns (and depending on your game, it very well could render it unplayable) and minimal added security.

FWIW, in most game genres, getting latency down to an acceptable level for a majority of regions is already difficult enough. Adding any kind of encryption is going to make things imposdible in many cases.

Yes, DTLS (or any asymmetric ebcryption scheme) will protect from MITM, but when the client is on a user's machine nothing will protect the attacker from just grabbing your keys and sending w/e they want

Because that's not the problem that encryption is trying to solve.
It is not something to avoid cheaters, for that, you need an authoritative server.
Encryption do not relieve you from doing validation of data received from clients.
If you are referring to user catching some malware and getting their account hijacked, again, that's not something encryption can fix.
Encryption is for ensuring that data from one end (user), to the other (server, or other user), cannot be read by anyone else (confidentiality), ensuring that the data is not altered during the trip (integrity).
And DLTS perfectly achieve this.
A very simple example is having a user insert a password to access a server and not wanting everyone in the way to read the password.

This is why most games dont use security at the transport layer already. You're free to use it in your game, of courae, but expect massive slowdowns (and depending on your game, it very well could render it unplayable).
in most game genres, getting latency down to an acceptable level for a majority of regions is already difficult enough. Adding any kind of encryption is going to make things imposdible in many cases.

Why do you think so? Why would encryption considerably increase latency? Do you have any data supporting this statement?
As an example, AES is a quite fast algorithm, and will take very few cycles of your processor compared to things like computing physics.

So, did you guys decided for the features on bellow?

Optional DTLS encrypt to the whole ENet layer.
AES encryption to bytes and vars.
SSL on WebSocket (Which already work on clients side)

@Malkverbena I think thats a good summary. Someone has to actually implement those things though. Contributors welcome :)

I would love implement it all with my own hands, since these improvements benefits me directly.
But on my days off I don't have enough time to sleep 8 hours per day and when I'm on sea, I work 12 hours per day....
It's difficult even to read my messages with the terrible internet I got there.

Hello again folks. I'm back!
I'll stay on shore for while and will have some time to work on it. But I have no clue what the best way to start.
I'd like to work on AES encryption at first moment, since it's not that complicated.
So, maybe write a module to encrypt/decrypt?
What about a layout like:
my_module.encryption(data, key, options)

@Malkverbena There is an open PR implementing AES encryption methods like you propose, although I'd prefer a stronger mode/config/defaults than it currently implements. (https://github.com/godotengine/godot/pull/17721)
Best option for this still seems to be looking into DTLS.

@mhilbrunner What exactly you would like to see on that PR above?

Hey folks
I just started a encryption module. I still don't undestand how some process of Godot works than for now I prefer work with a module.
Initially I gonna work with AES 256 GCM since EBC does not provides an apropriate security level.
My doubts are:
Can Godot use OpenSLL or Crypto? I confess I didn't read the licenses.
If yes, any advice?

Can Godot use OpenSLL or Crypto? I confess I didn't read the licenses.
If yes, any advice?

Godot used OpenSSL until https://github.com/godotengine/godot/commit/b33d10ccdfff52611ff508bbec64d65e0d88da70; it now relies on mbedTLS.

@Malkverbena unsigned char * is for bytes, not String (char *), you should use PoolByteArray .

One more thing. Not exacly related anymore. String uses 1 or 2 bytes?

Using PoolByteArray as input for cryptography can also be userful to network some way?

We would need to use uint8_t * directly instead for performance, but it's a start.

String uses 1 or 2 bytes?

String uses 1 or 2 bytes per character depending on the UTF8 encoding of the specific character.

We would need to use uint8_t *
Do you mean Vector<uint8_t> ?

@Malkverbena http://docs.godotengine.org/en/latest/development/cpp/index.html

Hey!
I wrote this module as a laboratory. Just to get used with some godot's process.
https://github.com/Malkverbena/cripter
As I said before, my time is really short and often I have to sacrifice hours of sleep to code.
I would like someone take a look and tell if there is something whong or that could be done in a better way.
I wrote cbc function just for testing but my next plan is write gcm a function with authentiction.
For now it works only with bynary array, soon gonna with with Variant as well.

@mhilbrunner

I'd prefer a stronger mode/config/defaults than it currently implements.

I would appreciate some feedback.
For now it can encrypt using CBC and GCM. GCM naturally authentication requires authentication.
I'm also not sure if I'm writing on poolbytearray in the best way.
As I told, I wrote this module to learn before to think write something decent to Godot.
@Faless I'm not sure if I got what you meant. Anyway I'm using uint8_t. Not the best way yet but it's a experience for now.

The algorithm ed25519 is also part of tls 1.3

Not to discourage the development of the AES part, but I believe the DTLS portion of this issue is much more useful. If you have just the AES portion, there are many ways someone could try to use it with networking and produce insecure code.
The most obvious way is if they just used a fixed key, but even if you negotiate a new key per connection (through some external mechanism), and then communicate with the server by encrypting string arguments with that key, it can allow a MITM to forge bad inputs by replaying older messages sent by the client, since the same message will always have the same output if encrypted by the same key. Depending on the specific game, the attacker may also be able to guess the contents of the messages based on frequency, or by joining the same server and correlating in-game events with messages between the client/server.
I think you can make it secure by adding a sequence number to each message, but I'm not sure and I'm not a crypto expert. You would still have to do manual serialization of all the RPC arguments which would be a pain (but probably could be extracted into a convenient library if it had to be).

I agree. For Internet communication, asymmetric cryptography is the best option. However I can think of at least half a dozen uses for symmetric cryptophraphy. It would be very useful at least to be able to encrypt an Raw Array using symmetric encryption.

What are some of those use cases? I only skimmed quickly through the thread, so sorry if some have already been listed.

I think this is partially addressed by https://godotengine.org/article/basic-cryptography-ssl-improvements
Next step is DTLS, which is WIP and will be available in 4.0: https://godotengine.org/article/dtls-report-1

Fixed by #36296.


Create .gitkeep

add 开发者门户 folder

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

新增开发者门户目录


Place confirmed positions under one header

Make a header per event. All positions will show up underneath

Done


StackLink and getTargetUrl

Draft for StackLink and getTargetUrl

StackLink provides a react-router link wrapped in a Mui Link for navigating in a stack
stackApi.getTargetUrl returns the url of a stack page that can be navigated to
StackLink and getTargetUrl have the same parameters as stackApi.activatePage

What do you think about that?
Todo:

[x] Changelog
[x] Docs

lgtm

And i'm currently writing on Stack docs, you can integrate the new component in there when it's ready.

https://github.com/vivid-planet/comet-admin/pull/446 is merged, you can add your docs in there now

@nsams @rabengraph I slightly changed the StackLink and added a prop for passing an external switchApi. That way it's also possible to use the StackLink outside of a stack (e.g. in combination with the useStackSwitch hook)
Also, I added docs for StackLink and getTargetUrl. And I added props tables to the docs.


[Text Analytics] Revert rename summarization

We have not finalized naming of the summarization features. For now, I will keep the names unchanged so as to no introduce a breaking change in the upcoming beta.
Reverts: https://github.com/Azure/azure-sdk-for-net/commit/8d0ac81aac847d93f98ae448acdbca0c97206c4c

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.AI.TextAnalytics


TrackMetric only works sometimes when aggregating

Repro Steps

Follow the (recommended) metric aggregation method according to https://docs.microsoft.com/en-us/azure/application-insights/app-insights-api-custom-events-metrics#aggregating-metrics
Generate metrics (with http requests)

Actual Behavior
Some of none of the metrics are collected and shown in the portal.
Expected Behavior
All of the metrics are collected (or skipped due to sampling).
Work around
Make sure to explicitly set MetricTelemetry.Context.Operation.Name before calling TelemetryClient.TrackMetric
Version Info
SDK Version : 2.2.1
.NET Version :	 2.1.2
How Application was onboarded with SDK(VisualStudio/StatusMonitor/Azure Extension) : Nuget/custom
OS : Win10 x64
Hosting Info (IIS/Azure WebApps/ etc) : console, kestrel

@Cooke Metrics are not subject to sampling, so sampling cannot explain the issue.  How do you validate that metrics are missing? Its possible are there is a latency before which they show up in portal.
Can you verify if local VisualStudio is also showing this missing behavior. ?

@cijothomas I validate they are missing by not showing up in the portal, and by not seeing any related output in the debug log. It is not possible that this is due to latency since I have a workaround that seems to work 100% and for which I can see the related metrics being present in the portal.
What do you mean with local VisualStudio?

Can you share a small repro for us to dig deeper into this? It's hard to investigate without repro app.  Thanks for your time! @Cooke

@cijothomas Maybe later, too little time now. I just wanted to let you know and share my work around. Thank you for your time and nice service!

@Cooke Okay. The latest SDK has built-in support for aggregated metrics! It'd be awesome if you can give it a try!
New api is documented here : https://github.com/Microsoft/ApplicationInsights-dotnet/issues/774
The aspnet.core sdk which refers this new base sdk containing aggregation features, is 2.3.0-beta1, and will be released today on nuget.

Closing this for now as no action pending from SDK.
Feel free to reopen if you need help or when you have a repro for further investigation.
Thanks!


oltpbench: tpcc issues

tpcc_config.xml.txt
Trying to run oltpbench tpcc, it hangs on loading data. Insufficient information for any diagnosis.
09:27:43,498 (DBWorkload.java:259) INFO  - ======================================================================

Benchmark:     TPCC {com.oltpbenchmark.benchmarks.tpcc.TPCCBenchmark}
Configuration: /home/pakhtar/proj/saved_configs/tpcc_config.xml
Type:          PELOTON
Driver:        org.postgresql.Driver
URL:           jdbc:postgresql://localhost:15721/tpcc
Isolation:     TRANSACTION_SERIALIZABLE
Scale Factor:  1.0

09:27:43,499 (DBWorkload.java:260) INFO  - ======================================================================
09:27:43,511 (DBWorkload.java:512) INFO  - Creating new TPCC database...
09:27:43,838 (DBWorkload.java:514) INFO  - Finished!
09:27:43,838 (DBWorkload.java:515) INFO  - ======================================================================
09:27:43,838 (DBWorkload.java:538) INFO  - Loading data into TPCC database...

Peloton output:
....

2018-01-10 09:48:09 [/home/proj/peloton-pa/peloton/src/network/network_connection.cpp:605:CloseSocket] DEBUG - Already Closed the connection 56
2018-01-10 09:48:09 [/home/proj/peloton-pa/peloton/src/network/network_connection.cpp:323:FlushWriteBuffer] ERROR - Fatal error during write, errno 32
2018-01-10 09:48:09 [/home/proj/peloton-pa/peloton/src/network/network_connection.cpp:788:StateMachine] ERROR - Error during write, closing connection
2018-01-10 09:48:09 [/home/proj/peloton-pa/peloton/src/network/network_connection.cpp:589:CloseSocket] DEBUG - Attempt to close the connection 57
2018-01-10 09:48:09 [/home/proj/peloton-pa/peloton/src/network/network_connection.cpp:605:CloseSocket] DEBUG - Already Closed the connection 57

config file attached, as generated by benchmark.py. Test run manually using the attached config file.
Other configuration:

Ubuntu 14.04 LTS
debug build

How long should this roughly take?

@neo-apz It's unclear. It's a unknown bug. If we knew what was causing the problem then we would know how to fix it.

No I meant how long the loading process takes.
Because it's not clear whether it has hung or it is still loading the data. I can see it's consuming both CPU and memory resources, but it's still unclear whether it's actually making any progress.
So I'll really appreciate it if I know how long it roughly takes to load all the data, so that I know whether it's hung or not.
Thanks

Loading time depends upon the configuration, what you have set in tpcc_config.xml. I'm using a scalefactor of 1, and loading time is still a problem. I don't remember the exact numbers, but when I extrapolated from progress last time I made observations, the loading time was going to be in the hours.
See the attached file. Rename it to log4j.properties  and place in the working directory from where you run oltpbench.. You can adjust logging levels overall and for specific benchmarks. You'll have to look at the TPCCLoader.java file to see what it uses and logs.
log4j_properties.txt

Against current master (release build).
21:32:08,381 (Worker.java:503) ERROR - Fatal error when invoking NewOrder/01
java.lang.RuntimeException: C_D_ID=1 C_ID=1275 not found!
at com.oltpbenchmark.benchmarks.tpcc.procedures.NewOrder.newOrderTransaction(NewOrder.java:192)
at com.oltpbenchmark.benchmarks.tpcc.procedures.NewOrder.run(NewOrder.java:158)
at com.oltpbenchmark.benchmarks.tpcc.TPCCWorker.executeWork(TPCCWorker.java:74)
at com.oltpbenchmark.api.Worker.doWork(Worker.java:386)
at com.oltpbenchmark.api.Worker.run(Worker.java:296)
at java.lang.Thread.run(Thread.java:748)

Master (with modified oltpbench to avoid #1197)
Release build, codegen on:
*** Error in `./peloton': double free or corruption (fasttop): 0x00007fb1bd463b70 ***
Release build, codegen off:
*** Error in `./peloton': double free or corruption (fasttop): 0x00007f740cc51110 ***
pa_insert branch:
Release build, codegen on
*** Error in `./peloton': munmap_chunk(): invalid pointer: 0x00007f900cc3ff00 ***
Release build, codegen off
*** Error in `./peloton': munmap_chunk(): invalid pointer: 0x00007fc784c3be60 ***

pa_insert branch, release build, unmodified oltpbench, codegen on:
*** Error in `./peloton': corrupted size vs. prev_size: 0x00007efda0c402c0 ***
ditto, codegen off:
*** Error in `./peloton': corrupted size vs. prev_size: 0x00007f3d314550e0 ***

I ran tpcc against master, to determine if the memory problem was in the updated code.
The address sanitizer caught a problem in the new code. With a fix for that, TPCC now runs.
That master was failing, but with the updates, TPCC now runs, would imply the old insert code was the source of the problem with master. Should be non-issue once the new code is merged.

@pervazea This is great news!


Support for includeChildren and includeParents in Store filtering (fixes #1168)

Some things to think about here:

Adding the meta-data to the function itself is a new pattern for us,  but potentially worth considering here?  Could also handle out of band in new flags.

'includeParents' is a bit wonky.  Could be 'includeAncestors' or 'includeParent'?
Trying to get the recursive notion in here...I suppose for symmetry with includeChildren should be just 'includeParent'.

Behavior not responsive to changes in props after construction time.  Could change this as well.

John -- to the big point about 'what is the use case for 'includeParents' --
I thought it might be useful to pluck out records you really care about in certain circumstances -- and it had been working for me fine during testing.
HOWEVER, its a perfect example of feature creep, and without a clear implementation, and potential problems, we have no reason to waste any more time on it now.  We can revisit if clients ever ask for it.

Thanks John -- think the latest cut takes care of all issues.

Also added new toolbox branch with same name for testing
Also realized why i did that conditional bit around includeChildren -- actually trying to avoid generating the childrenMap -- which might be expensive -- if I don't actually need it.


Merge UAT to PRD 11 Mar 2018

Coverage increased (+0.1%) to 77.655% when pulling 9bf08c407d5487ee009dbfb90f27d8ae130f359c on uat into 499886a8929cd93e6e48ab9649e89713055f9789 on master.


[STORM-3553] Upgrade jQuery from 1.11.1 to 3.5.1

What is the purpose of the change
Upgrades jQuery from 1.11.1 to 3.5.1 for several security fixes:

https://www.tenable.com/plugins/nessus/124719
https://blog.jquery.com/2019/04/10/jquery-3-4-0-released/
https://snyk.io/vuln/npm:jquery
https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2019-11358

How was the change tested
There does not appear to be automated UI tests, so I manually validated the changes by building a distribution .tar.gz file, deploying the nimbus, supervisor, UI, and logviewer daemons locally.  I then verified each page in the UI.  See attached screen shots below.
Each page was verified by checking each function, verifying it behaved as expected, and no Javascript errors occurred.  One such error was identified and resolved in this PR.
The only page I was unable to verify was the Storm Flux Yaml Viewer as I have no topology using it to verify against.  Perhaps there is one in the example directory someone can point me at and I can verify using it?
Other Notes

This PR should be perfectly OK to target the 2.2.x and/or earlier branches if we want.
I updated the LICENSE files manually, I was unsure if these are manually curated, or generated as part of some automated process.

Testing Screenshots

Visualization UI

Flux UI

@Crim  This looks good. Thanks for contribution!
Travis is failing due to
[ERROR] Failed to execute goal org.apache.rat:apache-rat-plugin:0.12:check (default) on project storm-webapp: Too many files with unapproved license: 1 See RAT report in: /home/travis/build/apache/storm/storm-webapp/target/rat.txt -> [Help 1]

Can you please fix it?
About LICENSE, here is some doc about it https://github.com/apache/storm/blob/master/DEVELOPER.md#listing-dependency-licenses. @srdo Can you speak to what needs to be updated regarding this change? Thanks!
I will find some time to test flux. I myself is not familiar with that part either.

For fixing the RAT plugin, I'd guess you should update the exclusion list in https://github.com/apache/storm/blob/master/pom.xml#L463 to include the new webapp jquery file.
The license update stuff described at https://github.com/apache/storm/blob/master/DEVELOPER.md#auditing-licenses-for-licensenotice only works for dependencies tracked by Maven.
As this dependency is not being pulled in via Maven, I think there is very little to do. The LICENSE and LICENSE-binary files should be updated (this is already done). JQuery doesn't have a NOTICE file so that's not a concern. I think the current changes are good.

I played around with the Flux UI and verified it works as expected.  Updated with a screenshot in the description.  I also updated the exclusion list in the POM, waiting for TravisCI to re-build now to verify.
Thanks for the help!


add build target to compile the starter if its present

addresses https://github.com/arangodb/arangodb/issues/3604


[Hexagon] Refactor to keep HexagonBuffer private to the device api

Refactoring the DeviceAPI a bit to privatize the HexagonBuffer class and no longer return it across the API boundary. So far we haven't seen a need to identify the memory scope, layout, and other properties of a HexagonBuffer allocated by the Device API.

cc @adstraw

Note that the "external" constructor for HexagonBuffer using void* is no longer required after this PR and furthermore HexagonBuffer can be simplified as it no longer needs to separately track managed and unmanaged allocations.  I think it's OK to follow up with a PR to do this cleanup work in HexagonBuffer --- just wanted to note it here.

@csullivan @adstraw @Lunderberg @kparzysz-quic thanks! merged now!


Implement atmosphere scaling parameter

Currently we only have a boolean flag to include or exclude the atmosphere:
https://github.com/CMB-S4/s4_design_sim_tool/blob/master/s4_reference_design.toml#L33
@keskitalo suggested to add capability of scaling to the atmosphere in the design tool.

should we have 2 different parameters, one for temperature and one for polarization?
is a scalar enough or we want to have it scale dependent and use a power spectrum?

Having separate scalings for temperature and polarization sounds good. The T->P leakage for atmosphere may well be scale-dependent but until there is a model, I would hold on adding that degree of freedom.

implemented in #16


Configs for each server

Currently, settings.json and .env is for all servers at once, they won't get distinguished. Meaning, you can't show different streamers in different servers for example

Optimally, there should be an environment where the bot hoster only has to set up the bot token in .env and everything else should be set by server admins/owners without anyone actually touching settings.json

In the long run, a database would be a better option here


[Feature Request] Underline the mutable local bind.

Since the local immutable bind is not going to be implemented in the foreseeable future, analyzing whether local variables actually "varies" and linting by underlining is a good feature. For example, in IntelliJ IDEA for Java, there is the following effect.

I hope something similar can be implemented in C# coding environment, especially on the open-source platform like VSCode.

This fits in with a new Roslyn ReassignedVariable classification (see https://github.com/dotnet/roslyn/pull/51889).  We could enable the classification and send it across as a new TokenModifier in the SemanticHighlight service.


Fix buildURL in jcasc

Signed-off-by: Michelle Nguyen<EMAIL_ADDRESS>Summary: Updated the jcasc key for cleanliness, but it turned out to be an invalid key. This fixes it.
Relevant Issues: N/A
Type of change: /kind cleanup
Test Plan: Deploy jenkins

@aimichelle can you gpg sign and push this again?


[DOCS]: Configuring Bedrock on a Specific Workspace

Description
Hello,
I'm using a Docker installation with OpenAI GTP4o, configured through Azure, as my primary LLM provider.
I’m exploring how to connect a workspace to Bedrock and integrate Claude. However, I don’t see an option to specify the model in the required fields. Could you let me know if I’m missing something?
Thanks a lot!
Spyros

Hello
I've discovered that you can set the model name in Bedrock exclusively through the AI Providers section of the dashboard. Once configured, the model can be seamlessly used across any workspace.


How to modify header in order to use M4 option?

Hi mikessh:
Thanks for sharing this great tool. I am trying to use mageri to call variants with the libraries containing UMIs. I have three fastq files generated from bcl2fastq. Two reads for pair ends, and one read for UMI.
This is how fastq file containing UMIs looks like:
@SN638:1063:HTVFJBCXX:1:1115:18632:43696 2:N:0:ATGCCTAA
GAGTTGCGCT
+
<<AG.GGIG<
This is the fastq for one of the pair end read:
@SN638:1063:HTVFJBCXX:1:1115:18632:43696 3:N:0:ATGCCTAA
GTGGAGTCACAGCGGAGATAGTGCCCTGGCCCTGGGCTTGTGGGGCTGCCCAGCAGCTGCCCATAAAGGACCTGATCGCTGGTGCCAGACTGGGATTGG
+
<GGGIIIIIIIIGIIIIIIIGIGGIIIIIIGIIIIIIIIIIIIIIGIIIIGIIIIIIGGGGGGGIIGGGIIIIGIIIIIIIIIIGGIIIIIIIIGAGGG
The read from pair end fastq does not contain UMI. Based on Mageri's manual, I am trying to use M4 option, but not sure how to modify the header. Any suggestion?
Thanks for your help,
Best
Rosemarie

Hi Rosemarie,
Have you been able to use MAGERI or other tool for sequencing data containing 3 reads FASTQ data (R1,R2, UMIs (read Index2)) which is a kind of standard way nowdays to read UMIs ?
Thnx and regards Rastik

Hi,
Do you find the method to solve the aforemethoned UMI-tagged fastq data? Could you please share your solutions?
Thanks

You could use:
let be fastq1 the first read pair
fastq3 the second read pair
fastq2 the UMI
awk -v FS="\t" -v OFS="\t" 'NR==FNR {split($1, id, " "); umi[id[1]]=$2":"$4; next;} {split($1, id, " "); $1=id[1]":UMI:"umi[id[1]]":"umi[id[2]]; print $0}'  <(zcat $fastq2|paste - - - -) <(zcat $fastq1|paste - - - -)|tr "\t" "\n" | bgzip -@4 -c > $outfile1 &
awk -v FS="\t" -v OFS="\t" 'NR==FNR {split($1, id, " "); umi[id[1]]=$2":"$4; next;} {split($1, id, " "); $1=id[1]":UMI:"umi[id[1]]":"umi[id[2]]; print $0}'  <(zcat $fastq2|paste - - - -) <(zcat $fastq3|paste - - - -)|tr "\t" "\n" | bgzip -@4 -c > $outfile2
You will get the sequence headers with UMI:sequence:quality


[Pick 2.0](inverted index) fix wrong no need read data when need_remaining_after_evaluate

When using an equal predicate on a column that applies an inverted index with a parser, it requires remaining_after_evaluate. In this situation, we cannot optimize the column without reading the data.
Proposed changes
Pick from  (#36637)

run buildall

run buildall

run buildall

run buildall


first release

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

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0dd5d95). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #1   +/-   ##
=======================================
  Coverage        ?   71.32%
  Complexity      ?      629
=======================================
  Files           ?       30
  Lines           ?     2462
  Branches        ?      272
=======================================
  Hits            ?     1756
  Misses          ?      569
  Partials        ?      137

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 0dd5d95...b0deb26. Read the comment docs.


[BUG] Error on Project Approval

on project approval showing error

how does it know which lists

@codingshot has this been resolved?


style(api-client): request section ui

this pr fixes request section hover glitch and alignment as seen below:
request section hover glitch before / after
https://github.com/user-attachments/assets/076fd585-c791-42b5-ace2-e20ea3d187fc
https://github.com/user-attachments/assets/0fd4ef93-9b65-45ac-97fb-323af41dbd5b
request section search <> input misalignment before / after

tysm for doing this again! 😄


[WIP] Allow updating height and color options through props

Use Wavesurfer setter methods to update waveColor, progressColor, cursorColor
and height if the value changed inside the options prop.
Requires katspaugh/wavesurfer.js#1145 to be merged/released first.

The PR on wavesutfer.js has been merged. Not sure if this is blocked until a version containing the changes has been released. Otherwise it might be worth creating a 0-8-stable branch and have master track Wavesurfer 2?


Process Completed with exit code 127

I'm trying to use this action but I'm receiving this error:
Run tale/kubectl-action@v1
  with:
    base64-kube-config: ***
    kubectl-version: v1.[2](https://github.com/js-sites/api-gateway/runs/6149314144?check_suite_focus=true#step:2:2)[3](https://github.com/js-sites/api-gateway/runs/6149314144?check_suite_focus=true#step:2:3).0
Run setup-kubectl.sh
  setup-kubectl.sh
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
/home/runner/work/_temp/1[4](https://github.com/api-gateway/runs/6149314144?check_suite_focus=true#step:2:4)6973f[5](https://github.com/api-gateway/runs/6149314144?check_suite_focus=true#step:2:5)-5949-4bc2-b8c5-4f[6](https://github.com/api-gateway/runs/6149314144?check_suite_focus=true#step:2:6)ea45682c3.sh: line 1: setup-kubectl.sh: command not found
Error: Process completed with exit code 12[7](https://github.com/api-gateway/runs/6149314144?check_suite_focus=true#step:2:7).

Hey, I'm able to reproduce it and I'm looking into it ASAP.

I'm seeing the same problem, any update?
...
  kube:
    runs-on: ubuntu-latest
    needs: docker
    steps:
      - name: Configure AWS credentials
        uses: aws-actions/configure-aws-credentials@v1
        with:
          aws-access-key-id: ${{ secrets.STAGING_AWS_ID }}
          aws-secret-access-key: ${{ secrets.STAGING_AWS_KEY }}
          aws-region: eu-west-1

      # no need to install AWS cli it's already installed.

      - name: Setup kubectl
        uses: tale/kubectl-action@v1
        with:
          base64-kube-config: ${{ secrets.STAGING_KUBECTL_CONF }}
          kubectl-version: v1.21.0

      - name: Restart forge app container
        run: |
          kubectl set image deployment/foo foo=****.dkr.ecr.eu-west-1.amazonaws.com/foo/foo-k8s:nightly
          kubectl rollout restart deployment/foo

Error:
...
2022-05-26T13:38:51.5118177Z ##[group]Run tale/kubectl-action@v1
2022-05-26T13:38:51.5118868Z with:
2022-05-26T13:38:51.5139073Z   base64-kube-config: ***
2022-05-26T13:38:51.5139850Z   kubectl-version: v1.21.0
2022-05-26T13:38:51.5140502Z env:
2022-05-26T13:38:51.5141173Z   AWS_DEFAULT_REGION: eu-west-1
2022-05-26T13:38:51.5142025Z   AWS_REGION: eu-west-1
2022-05-26T13:38:51.5142804Z   AWS_ACCESS_KEY_ID: ***
2022-05-26T13:38:51.5143635Z   AWS_SECRET_ACCESS_KEY: ***
2022-05-26T13:38:51.5144307Z ##[endgroup]
2022-05-26T13:38:51.5393409Z ##[group]Run setup-kubectl.sh
2022-05-26T13:38:51.5394181Z setup-kubectl.sh
2022-05-26T13:38:51.5450758Z shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
2022-05-26T13:38:51.5451621Z env:
2022-05-26T13:38:51.5452543Z   AWS_DEFAULT_REGION: eu-west-1
2022-05-26T13:38:51.5453259Z   AWS_REGION: eu-west-1
2022-05-26T13:38:51.5454100Z   AWS_ACCESS_KEY_ID: ***
2022-05-26T13:38:51.5454881Z   AWS_SECRET_ACCESS_KEY: ***
2022-05-26T13:38:51.5455849Z ##[endgroup]
2022-05-26T13:38:51.5625853Z /home/runner/work/_temp/b99599eb-3483-4958-b737-f201515b017e.sh: line 1: setup-kubectl.sh: command not found
2022-05-26T13:38:51.5639844Z ##[error]Process completed with exit code 127.

force the action to use v1.0.2 and now failing to download the right version (see #2)


Landing Page ("Welcome" page)

"breadcrumb trial"/ list of questions go from the side of the page to the top?
Add Terms of Use and Privacy Policy
Add MA trail court logo/ bar/ menu atop the page

Check links on landing page to determine if they still work and provide accurate, updated information


[BUG]: astro-angular integration broke Astro

Please provide the environment you discovered this bug in.
Error reproduction
Which area/package is the issue in?
astro-angular
Description
Script tag doesn't work in astro components when I add @analogjs/astro-angular to my Astro project.
Script tag works when I add is:inline directive or an atribute. But, it is a problem when I´m using TypeScript inside or I need to import a external script, because is necessary that Astro bundle it and that is not working with astro-angular integration.
Astro docs - Client-Side Scripts
Please provide the exception or error you saw
No response
Other information
No response
I would be willing to submit a PR to fix this issue

[ ] Yes
[X] No

Fixed with https://github.com/analogjs/analog/pull/98

This fix has been released with @analogjs/astro-angular version 0.1.0-beta.2


Allow duplicate static lib product names when module aliasing is used.

This change is for static library products only and does not alter product names.
Add an ID property to Product and ProductReference: combine package ID and product name to make it fully qualified.
Pass packageID to Product/Reference initializers.
Use product IDs to look up duplicates if the product is a static library type.
Use product IDs to compute aliasing related logic in ModuleAliasTracker.
Resolves rdar://89836609.

@swift-ci  smoke test

seems reasonable, thanks for putting together - will help many users. lets get @neonichu feedback.

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test

@swift-ci smoke test


Shift key not recognized for Keyboard shortcuts

On macOS 12.4, keyboard shortcuts do not register the shift key.
For example, pressing cmd+shift+p runs the keyboard shortcut for cmd+p instead. The same behavior can be seen when attempting to create a new keybinding in the settings menu.
Not sure which specific version I am running, as the "About Lapce" menu option doesn't do anything. I downloaded the mac build from the website today.

Same here, also macOS 12.4 (M1 if that makes a difference)

in my case "shift" make letter to upper or alternative char of key

my keyboard is ABNT2 (brazilian keyboard)

Same here.


Demo: asynchronous image processing with Savant

Stub.

Send images with Job-ID from REST API;
Place inference results into DB;
Access results with REST API (job polling).

Outdated. See #342


Improve isolation segments performance

Improve performance of isolation segments endpoints by removing unnecessary calls (can_read_from_space?) and combining the loading of data with the permissions check.
Please also see the commit messages.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

LGTM!  Thanks for the PR!
@tjvman: I did some basic acceptance to ensure reading from roles such as SpaceDeveloper still worked.  Also confirmed that this change https://github.com/cloudfoundry/cloud_controller_ng/pull/2449/commits/09dd9870a9a7405b67e05a6aa8a970951602cc4b is not an issue as you cannot add a user as a SpaceDeveloper unless you add them as an OrgUser (or above) and you cannot remove the OrgUser role if the user has Space roles still active.


Install issue - Debian 12 arm

Trying to install fastfetch in my Raspberry Pi 4 with Debian 12, this error appears;
./fastfetch-linux-aarch64.deb: line 1: syntax error near unexpected token `newline'
./fastfetch-linux-aarch64.deb: line 1: `!<arch>'

I also tried the other ARM package just in case, but the same error appears.
Screenshot

I'm probably doing something wrong but I don't know what.

My God! I just realized that I am trying to execute it instead of installing it with APT
I'm very sorry, it's already solved

Nice

😄 Happens to the best of us. At least you didn't nuke (rm -rf) your /usr/bin directory in the process (yes, it happened to me 😨).


Remove duplication in package.json

package.json contains license twice. This PR remove the duplication.

Thanks @dajvido for getting this to our notice. We will merege


add FAQ-entry w.r.t. funding of this project

What is missing
The current state of documentation (especially the FAQ-section e.g. https://www.coronawarn.app/de/faq/) lacks information about the funding of this project: Who pays for the development and operation of the app. And ideally also how much (allthough this information is not as crucial)
Why should it be included
For such an app it is crucial, that users trust the development team and the entity which runs the backend. It is much easier to trust an entity with a transparent and comprehensible motivation. If an app is paid or displays adds motivation is quite obvious. This app does neither, so the question arises why is this project happening? I suppose that public money is involved, and personally I think this like it should be. But it should also be transparent. To the users (and to the tax payers).
Currently the information provided is vague and insconsitent:

english version: "The German government has asked SAP and Deutsche Telekom" (IMHO does not imply public funding)
german version: "Die deutsche Regierung hat SAP und die Deutsche Telekom beauftragt" (IMHO does imply public funding)

Where should it be included
FAQ-sections (german, english) of the website and who-we-are-sections of the above linked README-Files.

Agreed. Code paid by the people should be available to the people, so highlighting the public funding is important.

For the record: It now seems to be official that this project is funded with public money: https://www.tagesschau.de/inland/corona-app-downloads-101.html. Still it should be mentioned in the docs and on the website.

Lobbyism undermines democracy.
https://www.rbb-online.de/kontraste/archiv/kontraste-vom-25-06-2020/corona-warn-app-ziemlich-teuer.html


On Raspberry Pi, bc is missing

bash: line 93: bc: command not found

I could add this to the prerequisites when apt-get runs, it is installed in Ubuntu Server by default. bc = basic calculator.

This should be fixed in ee4bdb9 - @friedger can you confirm if the issue is still happening?

Closing this one out as it should be resolved by the linked PR. Feel free to re-open or file a new issue if there is still an error!


Unable to add the ContentfulBundle to my project

Command: composer require contentful/contentful-bundle
PHP version: 7.2.14
Symfony version: 4.2.2
I get this error in my console:
Using version ^4.0 for contentful/contentful-bundle
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Restricting packages listed in "symfony/symfony" to "4.2.*"
Package operations: 14 installs, 0 updates, 0 removals
  - Installing jean85/pretty-package-versions (1.2) Loading from cache
  - Installing psr/http-message (1.0.1) Loading from cache
  - Installing ralouphie/getallheaders (2.0.5) Loading from cache
  - Installing guzzlehttp/psr7 (1.5.2) Loading from cache
  - Installing guzzlehttp/promises (v1.3.1) Loading from cache
  - Installing guzzlehttp/guzzle (6.3.3) Loading from cache
  - Installing contentful/core (2.2.5) Loading from cache
  - Installing contentful/rich-text (1.2.0) Loading from cache
  - Installing cache/tag-interop (1.0.0) Loading from cache
  - Installing cache/adapter-common (1.1.0) Loading from cache
  - Installing cache/hierarchical-cache (1.0.0) Loading from cache
  - Installing cache/void-adapter (1.0.0) Loading from cache
  - Installing contentful/contentful (4.1.1) Loading from cache
  - Installing contentful/contentful-bundle (4.0.0) Loading from cache
Writing lock file
Generating autoload files
ocramius/package-versions:  Generating version class...
ocramius/package-versions: ...done generating version class
Symfony operations: 1 recipe (b3f842960e25e3a6ea163ad10009dcfa)
  - Configuring contentful/contentful-bundle (>=3.0): From github.com/symfony/recipes-contrib:master
Executing script cache:clear [KO]
 [KO]
Script cache:clear returned with error code 255
!!
!!  Fatal error: Uncaught Symfony\Component\Debug\Exception\FatalThrowableError: Argument 2 passed to Contentful\ContentfulBundle\DependencyInjection\ContentfulExtension::Contentful\ContentfulBundle\DependencyInjection\{closure}() must be of the type array, string given in C:\xampp\htdocs\contentful\vendor\contentful\contentful-bundle\src\DependencyInjection\ContentfulExtension.php:138
!!    PHP Fatal error:  Uncaught Symfony\Component\Debug\Exception\FatalThrowableError: Argument 2 passed to Contentful\ContentfulBundle\DependencyInjection\ContentfulExtension::Contentful\ContentfulBundle\DependencyInjection\{closure}() must be of the type array, string given in C:\xampp\htdocs\contentful\vendor\contentful\contentful-bundle\src\DependencyInjection\ContentfulExtension.php:138
!!

Installation failed, reverting ./composer.json to its original content.

What can I do to get it working?

From what I get trying to install the Contentful bundle, seems to be a config problem.
If I change config/packages/contentful.yaml to something like:
# For the complete configuration, please visit
# https://www.contentful.com/developers/docs/php/tutorials/getting-started-with-contentful-and-symfony/
contentful:
    delivery:
        default:
            space: "%env(CONTENTFUL_SPACE_ID)%"
            token: "%env(CONTENTFUL_ACCESS_TOKEN)%"

Then the error should not happen again. So if you want to get content while someone fixes the bundle:
Add parameters to services.yaml
parameters:
    locale: 'en'
    contentful.space: "%env(CONTENTFUL_SPACE_ID)%"
    contentful.token: "%env(CONTENTFUL_ACCESS_TOKEN)%"

and then on your action:
/**
     * @Route("/content", name="content")
     *
     */
    public function fromContent()
    {
        /** @var Client $client */
        $client = new Client(
            $this->getParameter('contentful.token'),
            $this->getParameter('contentful.space')
        );

        try {
            $entry = $client->getEntry('<ENTRY_ID>');
        } catch (NotFoundException $contentfulException) {
            throw new NotFoundHttpException();
        }

        return new JsonResponse($entry);
    }

The weird part is that for now the Contentful Client seems not being injected to the container...
Weird things list:

[ ]  Contentful config validation

[ ]  Contentful Client service injection

[ ]  Contentful bundle seems to only inject contentful.delivery.profiler_controller as a service

Sorry, not a big help but just a quick debug on this.

Added my space id and access token to my environment variables and got it working now


Allow any CSV of parameters to be passed to generate_quantities

Summary:
Seems related to https://github.com/stan-dev/stan/issues/2968 but I can't tell where y'all landed on that.
Currently, when I try and call a generate_quantities against a stan program using a fitted parameters CSV, it needs to be the literal output file from an HMC run, when I'd like to be able to use arbitrary CSV files.
Description:
Currently, when I try and call a generated_quantities against a stan program using a fitted parameters CSV, it'll fail with with Error reading fitted param names from sample csv file "[FILENAME]" unless I have a header. It also fails with Mismatch between model and fitted_parameters csv file "[FILENAME]" unless I have the HMC diagnostic outputs ("lp__", "accept_stat__", "stepsize__", "treedepth__", "n_leapfrog__", "divergent__", "energy__").  I don't think any of this information is to generate quantities consistent with particular fitted parameters, and not requiring this makes many workflows easier. In particular, it would let me unit test user defined stan functions using the generated quantities method for keeping code quality high, and let me stress test behavior of the model under extreme conditions that are difficult to simulate. Currently, I'm injecting garbage information in both the header and the HMC diagnostic columns from a template, but that's a rather awkward interface.
Reproducible Steps:
Run generated_quantities against any stan program without the header or HMC diagnostic columns.
Current Output:

Error reading fitted param names from sample csv file "[FILENAME]"
Mismatch between model and fitted_parameters csv file "[FILENAME]"

Expected Output:
Normal Stan output
Current Version:
v2.27.0

here's the description of what's needed in the Stan CSV file:  https://mc-stan.org/docs/2_27/cmdstan-guide/stan-csv.html#mcmc-sampler-csv-output
good luck.

A combination of https://github.com/stan-dev/cmdstan/pull/1106 and https://github.com/stan-dev/stan/pull/3179 have allowed this


Add Picture constrains the picture to too few cols and rows

Description
When I add an image to an existing spreadsheet (a png in my case) whose size perfectly matches the size (in cm) of the area in which I want it to display, it gets displayed scaled/displayed incorrectly (too small).
I think the calculation for to.col & to.row calculated in positionObjectPixels is somehow going wrong when using the col/row dimensions or image dimensions.
I also think that if there was an option to use <xdr:oneCellAnchor> see here it may fix the image display by avoiding having to do the calculations.
Steps to reproduce the issue:

Create an existing 'template' spreadsheet in excel with the first 8 column widths sized as follows (to hold the image): a=1.38cm, b=19.39, c=1.14, d=1.11, e=1.11, f=1.11, g=2.94, h=4.42  (total 31.6cm) / or as Excel prefers to measure them A=6.29, B=98.86, c=5.14, d=5, e=5, f=5, g=14.43, h=22
Use func (*File) AddPicture ...with the aspect ratio locked, no offset, no link and scaling set to 1 (I have also tried without these) to insert a png of size 10.64cm high and 30.96 wide

Describe the results you received:

The image is inserted constrained to the width of 3 cols (a, b, c).
It's aspect ratio is wrong.
In Excel, going to the image size properties and choosing 'Reset' scales it as it should be (to 100% scale and col H)

Describe the results you expected:

The image is inserted constrained within the width of 8 cols (a, b, c, d, e, f, g, h).
It's aspect ratio should be maintained.
In Excel, going to the image size properties should show it scaled to 100%

Output of go version:
go1.10
Excelize version or commit ID:
1.2.0
Environment details (OS, Microsoft Excel™ version, physical, etc.):
OS. Win 10 & MacOS 10.13, physical, Excel 1708 & earlier

Thanks for your issue. I will consider adding and using oneCellAnchor support when no offset, no link and scaling set.

Excel use EMUs (English Metric Units) to measure image offset and used to determine the location of the picture, this value is affected by different screen resolutions and can‘t be accurately calculated, so if the generated image size doesn't meet expectations, you need to set the offset manually. Reference English Metric Units and Open XML.


client: Fix usage of ForceDeletionTypes

Right now garm passes the URL param "forceDeletionTypes=forceDeletionTypes=Microsoft.Compute/virtualMachines" which always errors out with:
The resource type does not support force deletion:'forceDeletionTypes=Microsoft.Compute/virtualMachines
Fix this by removing the prefix.

cc @gabriel-samfira

thanks!


Python3 shebang line; executable permission for python scripts

To avoid problems with wrong environment/python version this PR adds possibility to run scripts with any of the following commands:
python depthai.py
python3 depthai.py
./depthai.py
Tested on Linux.

Oh good idea thanks!

Should we add the change to calibrate*.py as well?


Switch statement wildcard?

Does the switch statement allow for wildcards at all?

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

ID: fb210829-0a82-3bfc-1408-81b34e58d43d
Version Independent ID: 7de2c0e7-0e01-20b0-a507-c0016cb82fa1
Content: Writing Expressions for Attribute Mappings in Azure Active Directory
Content Source: articles/active-directory/manage-apps/functions-for-customizing-application-data.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @msmimart
Microsoft Alias: mimart

@jdelforno Thank you for your query. We will investigate and update the thread.

@jdelforno, I believe the answer is no to this, but to get a clarity I am working with my backend to get a definitive answer on this.
Please allow me sometime and I will get back to you on this.

@jdelforno, We did some digup on this ask and found that given a switch statement like switch([displayName],"John Doe","Fred Jones","Frederick Jones","Dan Jones","Daniel Jones"), the value of the [displayName] attribute either matches "Fred Jones" or "Dan Jones" or it does not, in which case, the result is "John Doe." "Intrinsic" wildcard support would mean that one could instead write a switch statement like, switch([displayName],"John Doe","Fred* Jones","Frederick Jones","D* Jones","Daniel Jones"), and expect a displayName value of "Danny Jones" or "Dan Jones" or "Daniel Jones," et cetera, to match the second case, and yield "Daniel Jones" as a result. There is no such intrinsic wildcard support.

Which effectively makes the Switch statement in conjunction with certain areas of Enterprise App SSO rather limiting and realistically confined to Location attributes. E.g. State/Country/Postcode/Office etc.
Attempting to do something like a UPN Domain 'match' or ExtensionAttribute configuration via Regex would be amazing. (To be fair, I'm spoiled by PowerShell's MATCH regex heh).
Thanks for the info.
#please-close


yamllint: ignore rule in config file doesn't work

I've created some ignore rule in .yamllint.yaml like the following
rules:
  indentation:
    ignore:
      - "base/templates/deployment.yaml"

When I run yamllint from the command line all works fine.
But in VS-Code I still get a line to long error.
After some experimentation I stumbled about this in .settings
    "yamllint": {
      "capabilities": ["ignore-line"],
      "command": [
        "yamllint",
        "--format",
        "parsable",
        ["$config", "--config-file", "$config"],
        "-"
      ],
...

which means, yamllint reads the file from stdin and obvious has no filename.
I can get around this problem by calling yamllint with the filename
    "yamllint": {
      "capabilities": ["ignore-line"],
      "command": [
        "yamllint",
        "--format",
        "parsable",
        ["$config", "--config-file", "$config"],
        "$file" # <-- change!
      ],
...

but I have to re-write the ignore rules to
rules:
  indentation:
    ignore:
      - "**/base/templates/deployment.yaml"

because it seems, that yamllint is not executed from the root of the workspace.
After these changes my yaml files are linted as they were linted from the command line.
Is there anything that I miss, or is there a better way to tackle this problem?

Agreed on your findings, @uwehdaub.
Looks like it's better to always spawn the linter process from the current's file directory. Otherwise it breaks the ability of searching linters' config files if parent directories (yamllint in particular).


Update requirements section in README

It looks like the build.gradle is set to targetSdkVersion 21. The IDE says that SDK 21 requires JDK 7.

Thanks @andygup for catching that one.  You can change the root projects build.gradle file to set the compileSdkVersion to a JDK 6 compatible version which all the samples will pick up.

Verified


Improve kind install helper

Description
The install_kind function defaulted to an old version and wasn't taking into account the platform architecture. This fetches the latest official release for the correct architecture.
I would default to "latest" instead of hard-coding a version, but that gets us a prerelease version:
% ./env/bin/kind version
kind v0.18.0-alpha+7d26b50a129faf go1.19.3 darwin/arm64

I think it's safer to pin to a known-working version and update periodically than to use alpha builds of kind.
I've tested this locally.
History Notes

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

[x] The PR title and description has followed the guideline in Submitting Pull Requests.
[x] I adhere to the Command Guidelines.
[x] I adhere to the Error Handling Guidelines.

/hold
Need to test on an x86_64 platform.

/hold cancel (yes, I know there's no bot here to process this command, but there should be)
Test on Mac amd64 and Linux x84_64.


Unable to install cmdstan < 2.33.0 on Mac OS Ventura

Summary:
I can't install any version of cmdstan prior to 2.33.0 on my Mac OS Ventura machine.
Description:
I have been using cmdstan==2.29.2 locally, and recently ran into errors with compiling models. Re-building cmdstan via cmdstanpy.rebuild_cmdstan() doesn't work, and installing cmdstan==2.29.2 fails with:
48 warnings and 14 errors generated.
make: *** [src/cmdstan/main.o] Error 1

The full log is attached below:
cmdstan-install-fail-log.txt
I am able to install >= 2.33.0.
Reproducible Steps:
At my company, we have had some colleagues able to use 2.29.2 and others unable to install, like me. At the moment, I'm not sure it's reproducible.
Current Output:
See the log above.
Expected Output:
A successful install of older cmdstan versions, or more informative error.
Current Version:
v2.29.2/v2.33.1

here's the first error:
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/usr/include/c++/v1/cmath:607:12: error: no matching function for call to 'isinf'
    return std::isinf(__lcpp_x);

Can you share your Clang/libc++ versions?

Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)
Target: arm64-apple-darwin22.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

I'm guessing this is a duplicate of https://github.com/stan-dev/cmdstan/issues/1158.
Can you try 2.32.0 and 2.32.1? The only difference between those versions was the fix for the above issue

@WardBrian Thanks! 2.32.1 works while 2.32.0 fails. Sorry, I only tested up to 2.32.0 before.
What I'm still unsure of is why this occurred now. To my knowledge, I hadn't changed anything locally, models just stopped compiling recently.

Closing as a duplicate of #1158. The solution is to use cmdstan >= 2.32.1, or, if that is not possible, an older compiler toolchain for your platform


Ignore non-computational backends when overwriting the default

Fixes #5769

i'd appreciate if someone can tell me where is the best place to put tests.

Can you just reuse allowed devices list from the for below?

why the linter is unhappy? how can I fix that?

Read the linter, it tells you...not exactly your fault.

also this needs tests, can put in test_tensor.py

also this needs tests, can put in test_tensor.py

I put in in unit/test_device.py, that make more sense imo.

If you remove your fix, does that test fail?

Collaborator

well, no actually... not sure why i can't overwrite backend choice like that, do we actually have tests coverage for this?

ok, i don't think we can test this under the current architecture since it only works when you put the environ var before tinygrad is imported. given there's no existing test coverage and it works with my manual testing, i feel good about skipping the tests. @geohot wdyt?

there are some tests that test environment variables: https://github.com/tinygrad/tinygrad/blob/639af3f823cf242a1945dc24183e52a9df0af2b7/test/test_dtype.py#L382-L396

LLVM can't be inited as default device in many test envs, since it requires llvmlite

can someone please take a look at this pr? many thanks! @nimlgen @wozeparrot @geohot


Module 9: Deploy a .NET Aspire App to Azure

Associated WorkItem - 239426

Module is ready for review.
PR
Staging link

@alistairmatthews i tried to do this today and it looks like the staging link is inaccurate; could you update it?

It's already been merged to the NEW-deploy-aspire-azure branch. I don't think there's a staging location now. I guess you have to do a PR to main so that it generates one? Content is here now: https://github.com/MicrosoftDocs/learn-pr/tree/NEW-deploy-aspire-azure/learn-pr/aspnetcore/deploy-dotnet-aspire-app-azure

New PR for Brady's review
Staging link


update generated asset data

Update the generated code post updating the TF code

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: mkumatag
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Node scaling fails to start child QuantumScheduler

Whenever we deploy to our production environment (we use 3 nodes), it is common that the second and third node will continue to crash when attempted to be brought up. We generally have to do a full reboot in order to get back to three nodes. This error is what we see when this happens.
2019-02-22 23:10:03.753 level=info module=application_controller msg="Application temp exited: :stopped"
** (MatchError) no match of right hand side value: {:error, {:core, {{:shutdown, {:failed_to_start_child, Core.Jobs.Supervisor, {:shutdown, {:failed_to_start_child, Core.Jobs.QuantumScheduler, {:timeout, {:gen_statem, :call, [Swarm.Tracker, {:track, Core.Jobs.QuantumScheduler.TaskRegistry, %{mfa: {Quantum.TaskRegistry, :start_link, [%Quantum.TaskRegistry.StartOpts{name: Core.Jobs.QuantumScheduler.TaskRegistry}]}}}, 15000]}}}}}}, {Core.Application, :start, [:normal, []]}}}}

Replaced by #411


Create semver labels if they don't exist

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
Currently, the "semver label" is only added if the semver labels are registered beforehand
https://github.com/dependabot/dependabot-core/blob/main/common/lib/dependabot/pull_request_creator/labeler.rb#L210-L218
It would be nice to update this logic to create them if they don't exist similarly to how we handle the language labels creation
https://github.com/dependabot/dependabot-core/blob/main/common/lib/dependabot/pull_request_creator/labeler.rb#L168-L174
Note to maintainers
If accepted, I can work on this feature

Possibly related:

https://github.com/dependabot/dependabot-core/issues/5125
https://github.com/dependabot/dependabot-core/issues/3465
https://github.com/dependabot/dependabot-core/issues/2455
https://github.com/dependabot/dependabot-core/issues/1871

From a quick glance at some of those, our behavior around labels is a bit convoluted and occasionally surprising... plus many users have custom workflows around labels so they want advanced customization. Additionally, in the time since we first added labels, there's been a push to move towards using GitHub Actions workflows for post-processing of :dependabot: PR's. That gives users a lot more customization options.
So I agree some improvements are needed here, but before we go changing this and creating more labels/noise for users, I think it might be best if we (the core team) take a broader look at how we handle labels to ensure the end result all fits together.


(footer.twig) materialize template not change

Hi, dear
catalog/view/theme/materialize/template/common/footer.twig
I change materialize template to another name and also changed the link to other link and save the footer file
but there are no any change the materialize temple is same and same link no any change
could you please check what is the problem?
thanks

Hello!
There is no problem with the template! In OpenCart 3, by default, developer mode is disabled and templates are cached.
On the main administrator panel in the upper right corner, click on the gear. Disable the theme cache and click the "Refresh" button.


reopen 4984

But I found that I need to restart the server to take effect. What can I do to make it take effect without restarting the server.
Under Tencent Cloud's redis architecture https://cloud.tencent.com/document/product/239/18336 , java microservices will occasionally have exceptions. Using redisson version 3.16.4
The exception message is as follows
org.redisson.client.RedisTimeoutException: Command still hasn't been written into connection! Try to increase nettyThreads setting. Payload size in bytes: 0. Node source: NodeSource [slot=null, addr=null, redisClient=null, redirect=null, entry=MasterSlaveEntry [masterEntry=[freeSubscribeConnectionsAmount=0, freeSubscribeConnectionsCounter=value:49:queue:0, freeConnectionsAmount=282, freeConnectionsCounter=value:282:queue:0, freezeReason=null, client=[addr=redis://<IP_ADDRESS>:6379], nodeType=MASTER, firstFail=0]]], connection: RedisConnection@615205427 [redisClient=[addr=redis://<IP_ADDRESS>:6379], channel=[id: 0xce6110e1, L:/<IP_ADDRESS>:48568 - R:<IP_ADDRESS>/<IP_ADDRESS>:6379], currentCommand=null, usage=1], command: null, params: null after 3 retry attempts
at org.redisson.command.RedisExecutor$2.run(RedisExecutor.java:222)
at io.netty.util.HashedWheelTimer$HashedWheelTimeout.expire(HashedWheelTimer.java:669)
at io.netty.util.HashedWheelTimer$HashedWheelBucket.expireTimeouts(HashedWheelTimer.java:744)
at io.netty.util.HashedWheelTimer$Worker.run(HashedWheelTimer.java:469)
at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30)
at java.lang.Thread.run(Thread.java:748)
The configuration of redisson is
@Bean
RedissonClient redisson() throws Exception {
Config config = new Config();
ClusterServersConfig clusterConfig = config.useClusterServers()
    .setScanInterval(2000)
    .setKeepAlive(true)
    .setMasterConnectionPoolSize(100)
    .setMasterConnectionMinimumIdleSize(40)
    .setSlaveConnectionPoolSize(100)
    .setSlaveConnectionMinimumIdleSize(40)
    .setReadMode(ReadMode.valueOf(“MASTER”));
 clusterConfig.setPingConnectionInterval(3000);
    clusterConfig.addNodeAddress("redisson address");
}
config.setCodec(new JsonJacksonCodec());
return Redisson.create(config);

}

if netty tasks queue is full then you'll get this issue. Try to increase nettyThreads


chore: fix markdown linter warnings in all *.md files

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


[MTB] re-organize kubectl-mtb README

Re-organize the README into the following main sections:

Demo
Setup
Usage
Contributing

In the Usage we need to cover

Listing benchmarks
Running benchmarks against an existing namespace
Install TenantController and create a tenant namespace
Install Kyverno or Gatekeeper / OPA to fix errors

Move all sections related to contributing benchmarks in to "Contributing".

Also, there is a typo in the index:
Installation | How to use | How to include bechmarks|

We should cleanup and simplify the "Usage" content. We can guide the user to:

Create a namespace
Install a role (we can provide a sample)
Run the benchmarks (this should show errors)
Install the test policies
Re-run the benchmarks (all tests should pass)

/assign @Divya063

This was completed. Closing.


[FIX] - user should be able to update task only task status is inProgress

user should be able to update the task only task status is in progress
only super user and assignee can see the update button/ save button

Issue already created


about results

it's a nice job,but I found that the results were bad that the point cloud couldn't even produce a normal shape，Do I need to modify any super parameters。and，I did two experiments, using Locus point and not using Locus point，but it seems the same.thanks!

？

Hi,
The current parameters should reproduce the results that appears on our paper.
Can you give me some more details on what you are trying to do? Is it a single scene reconstruction or a trained model?
Does this problem occurs on the reconstruction of this specific scene or for all your reconstruction?
What do you mean by using Locus point?

Thank you for your timely reply. Sorry to disturb you，
i use
python multiple_scenes_learning.py --conf Learning_Proj.conf  to train this model,This is almost the result of all scenes。
Locus point i mean use_gt=True
should i change some thing in conf?
thank you!!!
Original
From:"drormoran"< @.*** >;
Date:2022/2/16 15:40
To:"drormoran/Equivariant-SFM"< @.*** >;
CC:"我自己的邮箱"< @.*** >;"Author"< @.*** >;
Subject:Re: [drormoran/Equivariant-SFM] about results (Issue #4)
Hi,
The current parameters should reproduce the results that appears on our paper.
Can you give me some more details on what you are trying to do? Is it a single scene reconstruction or a trained model?
Does this problem occurs on the reconstruction of this specific scene or for all your reconstruction?
What do you mean by using Locus point?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

There is an ambiguity in the uncalibrated setup reconstruction.
In this setup, you can apply any Homography on the scene and still have a good reconstruction. This is why your plot seems incorrect even when the reprojection error is low. This is why we usually don't look at those plot in the uncalibrated setup.
If you wish to reproduce the reconstruction shown in our paper you need to run your experiments on the calibrated setup, where this ambiguity doesn't exists.

Thank you very much for your timely reply. It's really an excellent job

Thanks :)


[Question] Android 12+ dynamic theme support?

Hi! I'm adding this AMAZING library to a new app I'm doing (and maybe porting others to it) but I've seen that it currently doesn't support the automatic Android 12 and higher versions coloring. Is it something that I'm missing or it's intended not to be on the library?
Thanks!

dynamicDarkColorScheme() and dynamicLightColorScheme() generate their own ColorScheme so it would negate the generated ColorScheme from this library. That and it is Android only.

dynamicDarkColorScheme() and dynamicLightColorScheme() generate their own ColorScheme so it would negate the generated ColorScheme from this library. That and it is Android only.

Okayy! Thank you so much for the fast response and the clarity. Have a nice day


Proposal to add a proxy option via code injection

Hey Atomify Team,
Great Work!
We are building standalone applets or components using atomify and it is working great!  Thank you so much for this tool!  We bumped into a use case where during development we want to focus on the application but leverage our custom api using simple proxy.
So we added a simple override:
https://gist.github.com/akennedy/26b0fa5937dad5318d64
 server: {
    proxy: {
      matcher: /^\/api|autocompletes|clinical/,
      handler: function(req,res) {
        return httpProxy.createProxyServer().web(req, res, { target: 'http://localhost:3000' });
      }
    },
    st: {
      path: './www',
      cache: false
    },

Would you be open to a PR adding the ability to inject a proxy handler for server-side apis?
If so, I will send you a PR.
Cheers!  Great Work!
Tom

This was fixed in #48


Doctor the docs

Usually these would be broken out into individual PRs, but in the interest of getting 1.1 out the door, I combined several patches into an omnibus PR:
doc(install): Reconcile install.rst and README.rst
doc: Bump version string to 1.1
doc: Add "Who's Using Falcon" section to index.rst, README.rst
doc: Update mailing list, IRC info
doc: Correct docs path in CONTRIBUTING.md

Current coverage is 100% (diff: 100%)

Merging #882 into master will not change coverage

@@           master   #882   diff @@
====================================
  Files          30     30
  Lines        1897   1897
  Methods         0      0
  Messages        0      0
  Branches      314    314
====================================
  Hits         1897   1897
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update f6b22be...7e020ca

:+1:


#17: models.py: Add on_delete=CASCADE for User foreign key;

Hi
Seems like there is on_delete field missing in twython_django_oauth/models.py:
  File "/home/brezerk/develop/example/.venv/lib/python3.6/site-packages/twython_django_oauth/models.py", line 5, in <module>
    class TwitterProfile(models.Model):
  File "/home/brezerk/develop/example/.venv/lib/python3.6/site-packages/twython_django_oauth/models.py", line 11, in TwitterProfile
    user = models.OneToOneField(User)
TypeError: __init__() missing 1 required positional argument: 'on_delete'

https://docs.djangoproject.com/en/3.1/ref/models/fields/#django.db.models.ForeignKey.on_delete
Addresses: https://github.com/ryanmcgrath/twython-django/issues/17

More fixes:
Django 3.1 migration

Fixed: ImportError: Django ImportError: cannot import name 'render_to_response' from 'django.shortcuts'
Fixed: ImportError: No module named 'django.core.urlresolvers'


add isGesture to onRegionChangeComplete iOS

Reopen this PR by adding isGesture to onRegionChangeComplete callback on iOS MapKit

Does any other open PR do the same thing?

https://github.com/react-native-maps/react-native-maps/pull/4113
https://github.com/react-native-maps/react-native-maps/pull/4272
What issue is this PR fixing?
https://github.com/react-native-maps/react-native-maps/issues/4112
How did you test this PR?

Tested in production and 1 simulator iOS

RyanTG If they don't approve this PR you can try using
https://javascript.plainenglish.io/live-npm-package-patching-with-patch-package-b8f30d47779c
https://gist.github.com/eduardorfreitas93/eb1f37784a936b4bd618ffd209fb72db

Hi @eduardorfreitas93 👋. Thank you for opening a new PR after the refactor.
As far as I can tell, this PR addsisGesture to the onRegionChange with a value that's always false? That seems undesirable :) If you could look into that, I will be happy to review this again :)

@Simon-TechForm Hi, it adds isGesture true in the return of onRegionChangeComplete, but I can see if I implement it in onRegionChange too

It's not a necessity to implement the isGesture functionality for onRegionChange for me to merge this. But we can't have it return isGesture=false even when it should be true :)

With this implementation I return isGesture true when the user makes the gesture to navigate the map, when the navigation on the map is automatic, for example zoom or recenter it has to return false, at least that was the functionality I needed

@Simon-TechForm I made the changes

@eduardorfreitas93 I'm afraid this approach isn't going to work either. If you swipe the map (meaning the region will keep changing for a bit after you release your gesture), isGesture will change to false even though it should be true.
I'm afraid that I'm too occupied atm to help you out with a solution, but I would suggest putting together a set of test cases that you evaluate when coming up with a new approach, e.g.:

Case isGesture
Expected onRegionChange
Actual onRegionChange
Expected onRegionChangeComplete
Actual onRegionChangeComplete
Result

animateToRegion
false

false

animateToRegion intercepted by gesture
false until interception, then true

true

Drag map
true

true

Good

Swipe map
true, also when gesture is released
true while gesture, then false
true
false
Bad

Swipe map intercepted by animateToRegion
True until interception, then false

false

These are just off the top of my head, you could come up with more/others. Also make sure to check out Google Maps to match the behaviour.

I'll close this for now and re-open when we have a working solution :)

I think this is urgently needed. Have you guys found a workaround?

I don’t have a workaround. I created a “Discussion” for this here: https://github.com/react-native-maps/react-native-maps/discussions/4440
But it’s not gaining traction.

@RyanTG Awesome, thank you!

This would be so useful haven't found a workaround


IInAppPurchaseHandler could not be found

Hi,
I have created GoogleMoblieAdsDemoHandler.cs script by copying and pasting the same git script into a new .cs file, and tried to attach it to the Main Camera.
In that script I am getting the error :The type or namespace name `IInAppPurchaseHandler' could not be found. Are you missing a using directive or an assembly reference?

Google Mobile Ads has been added
Google-play-services_lib has been added to the Plugins>Android folder.
What am I missing here?

I found the answer in a reddit post.  I changed the class name to match GoogleMobileAdsDemoHandler.cs and it worked.
Here is the post:  More to the point, it looks like you're using the version 2.3 GoogleMobileAdsDemoScript.cs with the version 2.2.1 package. Try using this version of the demo script: https://raw.githubusercontent.com/googleads/googleads-mobile-plugins/c488262bbe21733eef81df6acda1c189d33c42af/unity/samples/HelloWorld/Assets/GoogleMobileAdsDemoScript.cs

I want it off my phone Please

The tracker

This class was added in version 2.3.0 of the plugin. Which version are you trying to import?

This issue is being closed due to its age. If this problem persists, please re-open the issue.


OnToology update

OnToology update

Hi,
I published my ontology on OnToology but I didn't reserve the documentation
file .
My repo:[fatim919/Islamic-Fiqh-Onto]
Best
Fatima
في الخميس، ١٥ يونيو ٢٠٢٣ ١٠:٠١ م OnToologyUser @.***>
كتب:

OnToology update
You can view, comment on, or merge this pull request online at:
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1
Commit Summary

2fe16e6
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1/commits/2fe16e6e83def36d6b281da5a9e865fa9c09eb55
automated change

File Changes
(4 files https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1/files)

A OnToology/Islamic
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1/files#diff-e99a814d8ee2c1ddb9222694330923a9ad1d11c03a47f008ffe6838829ad1a1b
(0)
A OnToology/Islamic Fiqh Onto.owl/OnToology.cfg
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1/files#diff-a330c5d992796da09fac2609cc7cc4d921801661cde8afeb9c19ab24e8518227
(17)
A OnToology/Islamic Fiqh
Onto.owl/diagrams/config/ar2dtool-class.conf
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1/files#diff-155de0760575b8949c5847f03b547518c03cca666c991fab1005f6b031b34ba1
(42)
A OnToology/Islamic Fiqh
Onto.owl/diagrams/config/ar2dtool-taxonomy.conf
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1/files#diff-9415d814e07c6a6c49ca159495fea1bbf5a449024fbb4f8cdda0000b1ad75489
(51)

Patch Links:

https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1.patch
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1.diff

—
Reply to this email directly, view it on GitHub
https://github.com/fatim919/Islamic-Fiqh-Onto/pull/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BAASCVT4C7BWIYB3NS3WP4LXLNLXVANCNFSM6AAAAAAZIIS3ZA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@fatim919 Hi Fatima. Thanks for reporting the issue. We found the bug that you reported and we fixed the cause of the problem. It should work now.
Have a good day,
Best,


Update README.md

Description
Continuity in function names.

Hi @salt-lick - thanks for your interest. The name of the function varies on how it has been deployed. If you deploy through a stack it takes the prefix of the stack name - i.e. docker stack deploy func then you get:
func_webhookstash - deploying through the UI or API or CLI will give webhookstash
Cheers,
Alex

Understood. I was just following along in the examples and the continuity of the previous examples.

This is repeated in the TestDrive, so I'll probably delete the instructions at the sample level.


Trying to get in touch regarding a security issue

Hey there!
I'd like to report a security issue but cannot find contact instructions on your repository.
If not a hassle, might you kindly add a SECURITY.md file with an email, or another contact method? GitHub recommends this best practice to ensure security issues are responsibly disclosed, and it would serve as a simple instruction for security researchers in the future.
Thank you for your consideration, and I look forward to hearing from you!
(cc @huntr-helper)

@z-song - just for reference, both reports can be found here:
https://huntr.dev/bounties/b61b6828-0a42-4467-80e9-73ff0302de46/
https://huntr.dev/bounties/c8824710-def2-4216-aece-ea45b4307265/
They are private and only accessible to you! 👍


Fix parsing empty responses

When a Content-Type header (e.g. application/json) is given when there is actually no content (e.g. in a 204 response), you currently get this

This is because it's trying to parse "nothing" as JSON, which fails. A side effect is that the response box is not updated, and still contains the response from a potential previous request.
This is fixed by checking if the response is empty before attempting (any sort of) parsing.

Thanks for digging into this and fixing it !!!


Update 18.1.md

This question is about B-Trees. The question asks for the legal trees with min degree t=2, but the solution is for t=3.

Thanks. Can you update the question ?

updated


Add japicmp maven plugin to check API changes

The japicmp maven plugin can be used to programmatically generate reports about API changes. This is useful even before we define the Public API, so we can get a feel for how likely refactorings are to effect users:
https://siom79.github.io/japicmp/MavenPlugin.html
My previous use has been in JSONLD-Java where it works fairly well so far:
https://github.com/jsonld-java/jsonld-java/blob/f713c7e53e57c173da532ab66facc1e793bdc1d0/pom.xml#L328

Relates to #619 and #724

I will add some initially to the api modules that are definitely on the Public API but not others which we are not sure about per #619. They will only generate reports so far, so they won't fail the build if incompatible changes are found, but that is an option in future.

Sounds good. Can you point us to an example of what one of these reports looks like?

Never mind, found an example on the plugin page itself: https://siom79.github.io/japicmp/Examples.html . Looks pretty straightforward?

Yes, they are very straightforward as long as you turn on the "onlyModified" setting. They are generated on build into the target/japicmp/ directory as HTML files. The files are basically empty if no interfaces have changed, and populated with all of the relevant classes/methods/fields if any were added/removed/changed.
The reports differentiate between source and binary compatible. "Binary incompatible changes are marked with (!) while source incompatible changes are marked with (*)." I don't think we expect our code to be binary compatible (based on past behaviour), but we do want it to be source compatible as far as practical.
The default is also to look at protected/public, which I think is relevant for us in some cases with our Abstract implementations that we intend third-parties to reuse. We could change it to "public-only" but it wouldn't be as useful.
We can also customise it to only particular packages or classes, which will help for some of the embedded "impl" packages inside of api modules.
The code also shows up serialisation issues, which we have some of between 2.0 and 2.2-SNAPSHOT.
It is a little annoying in our large scale case that the reports are not yet aggregated to the reactor parent, but it should still be useful for us to review pull requests.
If you run "mvn clean install -DskipTests" on the pull request #733 code you can get a fairly quick idea of what some of the reports look like.


Odroid issues with u-boot 2023 upgrade

u-boot possible not configured correctly. See cros-ec messages upon boot up.
Seeing this on odroid-xu*, could be others.

C2 and C4 are booting fine with 2023 u-boot.

Found a work around. Need to disable CONFIG_BOARD_LATE_INIT for the xu* series.