Update data capture to match path pattern

Update DataCapture to match path pattern and use repeated DataCapture in AgentConfig for multiple data capture rules.
This can be used to define custom data capture rules for specific paths.
For example: If a user does not want to capture any data only for a specific endpoint that matches a path pattern, they can set the fields in the data capture rule with this path pattern to false.

Two examples of config
service_name: emojis
propagation_formats:
  - B3
  - TRACECONTEXT
 reporting:
   endpoint: traceable-agent.traceableai:4317/
   trace_reporter_type: OTLP
data_capture: // the default case
    http_headers: false
endpoint_data_capture: // the override for a single endpoint
   - endpoint: /foo/bar
      data_capture:
        http_body: false

A single
service_name: emojis
propagation_formats:
  - B3
  - TRACECONTEXT
 reporting:
   endpoint: traceable-agent.traceableai:4317/
   trace_reporter_type: OTLP
data_capture:
   - data_capture: // the override for a single endpoint
        http_body: false
      endpoint: /foo/bar
   - data_capture: // the dafault case
     http_headers: false

It's a personal preference, but I think the combined case reads more intuitively.
With the first example, there's now two concepts a user needs to know - data_capture and endpoint_data_capture.  How are these different?  What's the implication of choosing one over the other?  The concept of an endpoint isn't really something any of the agents have had to date.
Also, at an implementation level, now you have two types you need to pass around, meaning that all the code that deals with data capture needs to handle both cases, leading to potentially duplicated code and bugs where the developer doesn't realize they need to handle both types of data_capture configuration.

My preference is to just have repeated DataConfiguration. It's generic and therefore extensible in the future if we want to provide overriding data capture config based on something other than endpoint e.g. exit call, sql, etc.
I see it as the one place for data capture config, default, endpoint/url based or any kind of override in the future.


Use u32assert2 in u64 stdlib

Addresses #217.
Builds on the work done by @DeimosTheMisthios in PR #252.
Modifies the following u64 stdlib procedures to use u32assert.2 instruction:
u32assert4,  checked_add, checked_sub, checked_mul, checked_lt, checked_gt, checked_eqz, unchecked_div, checked_div, unchecked_mod, checked_mod, unchecked_divmod and checked_divmod.

Thank you @bobbinth for the review. Made suggested changes.
Re implementing checked_div, checked_mod and checked_divmod as exec.u32assert4 exec.unchecked_div, exec.u32assert4 exec.unchecked_mod and exec.u32assert4 exec.unchecked_divmod, I wasn't sure if the redundancy was worth it. I think it makes sense to keep it simple for now. We can re-evaluate if these procedure are used frequently in some cryptographic primitives in the future or if these procedures are frequently used when we have more visibility into user written miden assembly code. So I've changed these back to exec.u32assert4 exec.unchecked_div and similar for now.


401 in kaniko.kontain.me

Hi!
I found this repo extremly interesting! I am trying to test the kaniko endpoint with a simple example (https://github.com/k8spin/hello-k8spin)
$ docker pull kaniko.kontain.me/k8spin/hello-k8spin:master
you are not authorized to perform this operation: server returned 401.

What am i doing wrong?
Thanks!

This i found the issue :(
I had: export DOCKER_CONTENT_TRUST=1 :(


khal: standardize derivation

Motivation for this change
A few days ago, when updating all of my packages, I encountered build errors for khal / vdirsyncer mostly due to failing tests. When quickly patching an overlay to fix that, I noticed a few bad things in khal's default.nix:

pkgs is an argument to the derivation. This is not good because:

It's harder to change an input used inside khal such as pkgs.vdirsyncer. You have to use:

khal = super.khal.override {
  pkgs = pkgs // {
    vdirsyncer = vdirsyncer.override { ... };
  };
}

I think that If you override another derivation via another overlay, that doesn't apply to what passed to that pkgs argument.

There's a with python3.pkgs; at the top - which makes it hard to derive python deps and non python deps (readability).
pkgs.vdirsyncer was used in propagatedBuildInputs. This is bad because:

it is required only for tests and although used frequently together along with khal, it isn't strictly required by it to run.
It increases closure size if somebody uses vdirsyncerStable.

Now, on master (without this PR) the tests don't fail but I strongly think this change is beneficial, for general maintainability and conformism with other unwritten conventions in Nixpkgs.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build

There's a with python3.pkgs; at the top - which makes it hard to derive python deps and non python deps (readability)

How this improves the situation?

BTW @FRidh I encountered that overrideAttrs issue while encountering the issue failed tests this PR might. So please if you could take 1 more look here.

@FRidh I know it's been a long time but this is still bothering me. Also, today I noticed todoman references glibLocales which is ~210Mb in size. Commit https://github.com/NixOS/nixpkgs/pull/85993/commits/2260aed8ada148975cca987c53e801c3514b998c in this PR fixes this.

I'm closing in favor of smaller PRs which hopefully will be easier to review.


crypto: investigate if we can get rid of pearldiver status lock

Pearl diver's status can only be asynchronously assigned value of SEARCH_FINISHED. So even if two threads change status value it's always going to be SEARCH_FINISHED.
The lock might be needed if pearl diver supported interruption logic. In this case the access to status variable must be exclusive.


Missing pre-compiled templates for OSX - 3.4.4

https://github.com/Gramps/GodotSteam/releases/tag/g344-s153-gs3133
Templates file is missing pre-compiled templates for Mac/OSX

Yeah, we just noticed this a little bit ago.  I will get it added later this evening when I have time.  For now, you can download the Mac zip from the release section, it'll have both the editor and template zip in there.

OK, the templates zip should be fixed now!


Make it clear that this element contains Mailbox

This makes the SingleRecipientType stand more out and adds a link to the child element, like in https://docs.microsoft.com/en-us/exchange/client-developer/web-service-reference/sender

Docs Build status updates of commit 3b0e694:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/web-service-reference/directreports.md
:bulb:Suggestion
View
Details

docs/web-service-reference/directreports.md

Line 53, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


yarn recipe hard links rather than copy

Hi
Nice to see yarn now supported natively in deployer.  I've switched to using it instead of my own task definition.  I noticed it use cp -R to make a copy of the node modules.  This definitely speeds up the deployment, but doing cp -al instead, creating hard links rather than a copy is almost instant.  Seems to be working for me.
Would you like a patch ;)
Steve

Nice idea 💡


[Question] IN queries

Describe the bug
A clear and concise description of what the bug is.
This is a Question
Does goqu support "IN" queries.

Yep! Checkout the expressions docs there are a couple of examples of IN and NOT IN expressions.
sql, _, _ := db.From("items").Where(goqu.Ex{
  "col": []string{"a", "b", "c"},
}).ToSQL()
fmt.Println(sql)

Output
SELECT * FROM "items" WHERE ("col" IN ('a', 'b', 'c'))


Pass gulpfile to ES6

:)

$ npm test failed in task pre-test since you change gulpfile into gulpfile.babel :/

Oops, running tests, what an idea ^^

:+1: I fixed problem

Thanks ;)


Handling batch writes for more than 500 operations

To perform updates on multiple documents Firestore provides us with the batch transaction.  Once the batch is created it can be committed hence completing the updates in one go. However, a limitation to this is we can batch a maximum of 500 operations in Cloud Firestore.
If you try to add more than 500 operations it would lead to a failed transaction. A failed transaction returns an error and does not write anything to the database. You do not need to roll back the transaction. Cloud Firestore does this automatically. read more
We can catch this error using a try-catch block as shown here and if it throws an error we can log it in our logger as well and show it in the API response as well.
To perform more than 500 operations we can do so using multiple batches by creating some sort of batched array as suggested here in this StackOverflow link

Hey @heyrandhir if you haven't started this, then please assign this to me.

Hi @shmbajaj I had created this issue when I had discovered about batch writes having a limitation of 500 users. In my module of user-management I have handled this. You can check if there are any other modules which are using batch writes and have not handled the case of > 500 docs update you can create an issue and work on that.


Update cert-manager.json

Fix "template syntax" delimiters.
Current chart shows in the dashboards { { label } } instead of value due to the incorrect syntax.

Thanks, @jlpedrosa. This should help prevent people from maintaining their own version. When can we get that merge?
BTW LGTM.

Anything we can do to help get merged, I'm in the same situation where I would have to maintain my own copy. Thanks,

@superq By anychance you wouldn't know who we contact?


Support Webpack 2.x

I tried this plugin out with webpack beta. I am getting following error
/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/enhanced-resolve/lib/Resolver.js:30
		throw new Error("Signature changed: context parameter added");
		^

Error: Signature changed: context parameter added
    at Tapable.resolve (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/enhanced-resolve/lib/Resolver.js:30:9)
    at NormalModuleFactory.<anonymous> (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/npm-install-webpack-plugin/src/plugin.js:24:30)
    at NormalModuleFactory.applyPluginsAsyncWaterfall (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/tapable/lib/Tapable.js:135:13)
    at NormalModuleFactory.create (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/webpack/lib/NormalModuleFactory.js:158:8)
    at /Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/webpack/lib/Compilation.js:214:11
    at /Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/async/lib/async.js:181:20
    at Object.async.forEachOf.async.eachOf (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/async/lib/async.js:233:13)
    at Object.async.forEach.async.each (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/async/lib/async.js:209:22)
    at Compilation.addModuleDependencies (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/webpack/lib/Compilation.js:185:8)
    at Compilation.processModuleDependencies (/Volumes/DATA/Bhavin/Projects/temp/webpack_demo/node_modules/webpack/lib/Compilation.js:170:7)

It appears to be conflicting with enhanced-resolve. Not sure, if this should be fixed here or should enhanced-resolve be more forgiving.

Uhhhhh, yea, once Webpack 2 actually gets released I'll update this project to work with it.
It was such a PITA to get working with 1.12 (or whatever) I'm not bothering with 2 until it launches :D

(Don't worry, wontfix label is just to make it clear it's not on the roadmap until Webpack 2 launches)

Also interested in this.

I as well

Is there an official 2.x release of Webpack that's not a beta?

Ok, I'm gonna work on this. DAT TREE-SHAKING THO.

I've migrated quik to use Webpack 2, so waiting for this :)

I'm working on v3.1.0 now so the 3.x release is stable.
I do know that Webpack 2 is a necessity for me (especially with native module support).  (Speaking of which, @satya164 could you point me to a functioning demo of Webpack + Babel + Tree Shaking that still works in the browser?)
But, the way v3.1.0 works is by doing nested compilation, which I believe is kinda discouraged going forward.
We'll see...

@ericclemmons Here's one - https://github.com/satya164/react-boilerplate (server runs on the 8008 port)

Thanks!  I'll use this to corroborate my changes.

Another great article on the subject:

http://moduscreate.com/webpack-2-tree-shaking-configuration/

Got this working!
 ~/Projects/ericclemmons/npm-install-webpack-plugin/example  ⑂ master +    npm start

> @ start /Users/Eric/Projects/ericclemmons/npm-install-webpack-plugin/example
> webpack --config webpack.config.server.js --watch

Installing babel-core...
Installing babel-plugin-react-html-attrs...
Installing babel-preset-react...
Installing babel-preset-es2015...
Installing babel-preset-stage-0...
Installing babel-preset-react-hmre...
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^0.14.0-rc1 || ^15.0.0-rc but none was installed.
Installing peerDependencies...
Installing<EMAIL_ADDRESS>|| ^0.14.0-rc1 || ^15.0.0-rc"...

Installing babel-loader...
Installing express...
Installing webpack-dev-middleware...
Installing webpack-hot-middleware...
Hash: c2a32b45a8fb2a4c49a6
Version: webpack 2.1.0-beta.6
Time: 21755ms
    Asset     Size  Chunks             Chunk Names
server.js  5.08 kB       0  [emitted]  server
    + 9 hidden modules
Listening on http://localhost:3000/
Installing bundle-loader...
Installing react-lite...
Installing lodash.capitalize...
Installing style-loader...
Installing css-loader...
Installing react-dom...
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Installing peerDependencies...
Installing<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Installing bootswatch...
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Installing peerDependencies...
Installing<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

webpack building...
Installing file-loader...
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Installing peerDependencies...
Installing<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

Installing url-loader...
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
Installing peerDependencies...
Installing<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

webpack built 3d04b9fbc79ba987bf07 in 77844ms
^C
 ~/Projects/ericclemmons/npm-install-webpack-plugin/example  ⑂ master +  

Going to create a separate example folder for it, though...

Ok, so the only caveat is that, since my examples are using npm link, I have to update webpack in the root package.json for this to test correctly.
But both Webpack1 & Webpack2 function!

Amazing! Will try this out ASAP :D

@satya164 I have 2 example folders (example/webpack1 and example/webpack2) which I use for testing locally.
The only caveat is that, since this project gets npm linked, you need to have the correct webpack installed in the root.

@ericclemmons Okies, will keep this in mind.


Bug: func createIdentity() always fails

Describe the bug
We are trying to integrate the SDK with our app and we find that the method createIdentity() constantly fails in line 41 of the SigningKey.swift file:
let recoveredKey = try KeyUtilx.recoverPublicKeyKeccak256(from: signature.rawData, message: message)
Expected behavior
I tried to replicate this flow with the standard SECP256K1 library (signing + recovery) and it works with one condition:
Signing as well as recovery is done with the keccak hash of the message, not the message itself (as you have currently in the line 41)
Steps to reproduce the bug
Just try to create an identity

Thanks for the bug report @rommex I'm not able to reproduce this on my end. We test createIdentity() in the example app and in the test suite. Everything seems to be working as expected. Do you mind sharing some more context on the code you're writing?
If you're looking for a more in-depth iOS examples I can direct you towards our https://github.com/xmtp-labs/xmtp-inbox-ios.

Hi @nplasterer thanks for your response.
I had to take some time to investigate and found out that it's not the signing process itself that was causing the problem, but the fact that I omitted this line of code which seems to be present in your code:
bytes[bytes.count - 1] = 1 - bytes[bytes.count - 1] % 2
Basically your code alters the last byte of the signature to 0 or 1 depending on the last byte of the original signature.
Could you please explain / refer me to some docs about this change?
I'm seeing this for the first time and I wonder why it is needed.
As I understand your code alters all signatures, even coming from external wallets like MetaMask etc
Thanks!

Basically your code alters the last byte of the signature to 0 or 1 depending on the last byte of the original signature.

The last byte of the signature is the ECDSA recoveryID. It's used to tell which EC point to use during recovery. Individual projects sometimes choose to use special encodings of this value( e.g: EIP-155, and BIP-137 )
XMTP uses the canonical values [0,1] as the protocol is chain agnostic. Since signers make different assumptions about how the V value should be encoded, our SDKS force all signatures it into the base range for maximum interop.


Table captions

Shouldn't tables have captions too?

This is already supported, but they're not formatted the same as figure captions.


Change to data update coordinator

Home Assistant provides a DataUpdateCoordinator class to help manage fetching data. Change the architecture of the integration to use this class.

I've had this running for several days while creating intentional conflicts with multiple virtual machines running HA and it's handled errors properly, so I'm going to do a release.


Make OpenROAD icon easier to see

Description
The OpenROAD icon distinguishes it, but it is pretty much a mushy green blob on a grey background on Ubuntu:

Some other icons for comparison:

Suggested Solution
Fix color scheme.
I asked ChatGPT to fix the icon and it came up with a better colorscheme:

Additional Context
No response

What size is the icon on your screen in pixels?

What size is the icon on your screen in pixels?

See image below for details(screenshot), but ca. 50x40.

The icon has a transparent background so the window manager must be choosing the gray color.  How does it look if you use

The icon has a transparent background so the window manager must be choosing the gray color.

That's a .jpg file, how do I "try" it?
I'll try latest master and see if I can tweak icon.png a bit.

An idea I was going to try, but I still get no icon when I try with master

That's a .jpg file, how do I "try" it?

Replace the src/gui/resources/icon.png and make install should do it

That's a .jpg file, how do I "try" it?

Replace the src/gui/resources/icon.png and make install should do it

But it is a .jpg? icon.png is a png... I thought .jpeg did not encode transparency.
I reopend https://github.com/The-OpenROAD-Project/OpenROAD/issues/6173#issuecomment-2482021927 as I don't get an icon.

Example from https://github.com/The-OpenROAD-Project


value without Unit

Is it possible that you add some extract string  with the value but without extension Unit "W" or "kWh"
or a switch that that extract the 4 value and jump over your then
dat=$4
val=''
[ "$dat" = "webdata_today_e" ] && val=kWh
[ "$dat" = "webdata_total_e" ] && val=kWh
[ "$dat" = "webdata_now_p" ] && val=W
fi

Please can yo test the new v1.0.2
Itshould be switchable now with -u

thank you
works now with my script , all data now recived by volkzähler DB

Thanks again
I hope it is ok for you that I mention you at Volkszähler DB project and refer to your Github project there.

yes of course :-)

As I can see you are using standart settings admin admin. This is insecure!!
Never use standart settings.
Attack:
https://github.com/s10l/deye-logger-at-cmd.git
Firmware update:
https://zinnzgreen.de/service/deye-mikrowechselrichter-systemupdate/
then reset and change the AP password (max 8 chars)

thank you for your information , update already installed
AP pass already change

*/5 * * * * /bin/bash /home/pi/devo.sh
ahould be
*/5 * * * * /bin/bash /home/pi/auslesen.sh

also the install cmd should be "sudo make install" instead of "make install"


IncompleteRead

Hi, JuanBindez I've installed pytubefix version 5.4.2. Here's the test code:
from pytubefix import YouTube

url = 'https://www.youtube.com/watch?v=xtQy5NTsHeE' # download failed

def test_YouTube():
    yt = YouTube(url, use_oauth=True, allow_oauth_cache=True, client='WEB')
    video_stream = yt.streams.get_highest_resolution()

    output_path = r"./"
    video_title_file = "test.mp4"
    video_stream.download(output_path=output_path, filename=video_title_file, max_retries=5)
    print(f'Downloading video: {yt.title}')

if __name__ == '__main__':
    test_YouTube()

Some YouTube links have this problem
Below is the error message I get when running the code:
Traceback (most recent call last):
  File "D:\code\python\project\some-reverse-works\examples\pytube\test_pytubefix_to_author.py", line 15, in <module>
    test_YouTube()
  File "D:\code\python\project\some-reverse-works\examples\pytube\test_pytubefix_to_author.py", line 11, in test_YouTube
    video_stream.download(output_path=output_path, filename=video_title_file, max_retries=5)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\site-packages\pytubefix\streams.py", line 325, in download
    for chunk in request.stream(
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\site-packages\pytubefix\request.py", line 185, in stream
    chunk = response.read()
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 482, in read
    s = self._safe_read(self.length)
  File "C:\Users\Administrator\AppData\Local\Programs\Python\Python310\lib\http\client.py", line 633, in _safe_read
    raise IncompleteRead(data, amt-len(data))
http.client.IncompleteRead: IncompleteRead(435172 bytes read, 9002012 more expected)

try using it without client="WEB", don't pass the client argument

The same error, my friend, is it related to the network, but it is normal for me to use other links

I got this error too, but after waiting some time and rerunning the code, the video downloaded successfully. I don't know why! I think the problem is when the network goes slow suddenly while downloading the video!

I got this error too, but after waiting some time and rerunning the code, the video downloaded successfully. I don't know why! I think the problem is when the network goes slow suddenly while downloading the video!

I am frequently getting similar errors because of instability at Youtube servers.
Retrying is sometimes enough to get the file.
In other cases, the video is not ready to download because are lives that ended recently and Youtube did not finished the file processing yet. Even if the video is available accessing with the browser, it may not be available to download, and we have to wait a couple of hours; in rare cases, have to wait a few days.


MeshOutlines effect is really bad. Is it possible to use the post processing in built-in Render Pipeline or Renderer Feature in URP?

As the title saying.

Hi @Hanning-Liu, thanks for asking. Many modern mesh outline techniques are done using post processing. Post processing provides great quality outlines but can be prohibitively expensive on most mobile mixed reality devices (like HoloLens 2, Quest 2 & 3, etc.).
If you are targeting a device that is powerful enough to do post processing, I'd recommend exploring a technique like this: https://roystan.net/articles/outline-shader/
Else, you may be limited to mesh outlines. Do you have any scenarios which are particularly bad? I may be able to provide some advice.

Hi Cameron, thank you for your reply!
I've seen the post you mention. And I'm using a similar package named "OutlineToolkit" downloaded from Asset Store: https://assetstore.unity.com/packages/tools/particles-effects/outline-toolkit-98020
And I also saw a method to implement this effect in URP:
https://www.youtube.com/watch?v=LMqio9NsqmM
I'm developing an app on HoloLens2 only run with Holographic Remoting for now, so I think maybe it's ok to use post processing method to calculate the outlines. But it's definitely better if the ideal effect can be implemented using the MRTK MeshOutline package.
The ideal effect should be:

The silhouette lines and the sharp lines inside the silhouette can both be displayed.
The mesh object itself should be transparent.
The front and rear occlusion and the intersection effect of lines should be correct.

The ideal effect looks like the image below:

The problem with MRTK MeshOutline is that when the material of the gameobject attached with meshoutline script is changed to transparent, the outline will become a solid silhouette rather than lines. And it cannot display the sharp corner lines inside the silhouette. Like the image below showing:

Oh awesome, yes if you are using Holographic Remoting then rendering is happening on your PC so the use of post processing should be fine!
Thanks for the detail breakdown of where Mesh Outlines don't work and how you would like them changed. I'll keep this issue open to track this work.


Update prerequisites for deploy logic app from VS

Add Azure PowerShell to prerequisites

Hi @derek1ee, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

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

TTYL, AZPRBOT;

@derek1ee : Thanks for your contribution to the Azure documentation! The author, @jeffhollan, has been notified to review your proposed change.

Thanks @derek1ee  this looks ready to merge. @PRmerger

@KatieCumming can you help merge this?


Syntax error causes yarn start to exit

🐛 Bug Report
Accidentally introducing a syntax error in JSX code causes yarn start to exit completely. This is unexpected coming from other tools such as CRA, Next.JS, etc.
Have you read the Contributing Guidelines on issues?
:+1:
To Reproduce

yarn start
Introduce syntax error in JSX code within a mdx or js file
Save file
Observe error and subsequent exit of start script

Expected behavior
Watch mode continues to work, and fixing the error recovers correctly.
Actual Behavior
Client bundle compiled with errors therefore further build is impossible.
./docs/mobile/change-default-behaviour.mdx
Module build failed (from ./node_modules/@docusaurus/mdx-loader/src/index.js):
SyntaxError: unknown: Unexpected token (56:43)

  54 |         title="Commands and Operations"
  55 |         description="Learn more about commands and operations"
> 56 |         link="concepts-commands-operations",
     |                                            ^
  57 |     />
  58 |     <UseCaseCard
  59 |         title="Change Geocortex Mobile's Default Search Behaviour"
    at Object._raise (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:742:17)
    at Object.raiseWithData (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:735:17)
    at Object.raise (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:729:17)
    at Object.unexpected (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:8757:16)
    at Object.jsxParseIdentifier (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4388:12)
    at Object.jsxParseNamespacedName (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4398:23)
    at Object.jsxParseAttribute (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4482:22)
    at Object.jsxParseOpeningElementAfterName (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4503:28)
    at Object.jsxParseOpeningElementAt (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4496:17)
    at Object.jsxParseElementAt (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4528:33)
    at Object.jsxParseElementAt (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4544:32)
    at Object.jsxParseElementAt (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4544:32)
    at Object.jsxParseElement (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4602:17)
    at Object.parseExprAtom (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:4609:19)
    at Object.parseExprSubscripts (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:9602:23)
    at Object.parseMaybeUnary (/home/ian/Code/dev-center/node_modules/@babel/parser/lib/index.js:9582:21)
 @ ./.docusaurus/registry.js 1:2379-2448 1:2182-2329
<EMAIL_ADDRESS> @ ./.docusaurus/routes.js
<EMAIL_ADDRESS> @ multi ./node_modules/react-dev-utils/webpackHotDevClient.js<EMAIL_ADDRESS>error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Your Environment

Docusaurus version used: 2.0.0-alpha.48
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Node 12.13.0
Operating system and version (desktop or mobile): Ubuntu 19.10

This is already fixed in #2437


launch-config-wizard-validation

@nlewis05 This should fix your issues. Give it a test and then when ready squash commits with git rebase -i upstream/main. Update your final commit message to be something like the below:
Launch Wizard Validations:

- Check that daffodil debug classpath items exist when debug is being started.
- Don't allow duplicate items inside of the daffodil debug classpath table

Closes #650

If there was an issue for the duplicate thing as well add a new line under the provided closes with Closes #ISSUE_NUMBER


Semicolon is required

According to documentation on other places, the semicolon is required

Hi @eakoning, thanks for your contribution. Where did you see that a semicolon is required? It's not, and I just confirmed that it works with or without the trailing semicolon. I'd like to fix the other docs if possible.

Hi Matt,
For instance here
https://github.com/Microsoft/vsts-tasks/blob/master/docs/authoring/commands.md
Kind regards
Emiel
Op do 20 sep. 2018 om 21:28 schreef Matt Cooper<EMAIL_ADDRESS>
Hi @eakoning https://github.com/eakoning, thanks for your contribution.
Where did you see that a semicolon is required? It's not, and I just
confirmed that it works with or without the trailing semicolon. I'd like to
fix the other docs if possible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/vsts-docs/pull/1822#issuecomment-423304126,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3D8f85qZGCWpLcEHx9C71B2gL5fhMMks5uc-xUgaJpZM4Wx-LC
.


Integration tests

Create a new integration test project that will allow full end-to-end tests.

Host syslogd server
Allow us to simulate outages

Closed via #16


fix: Correct Cloudwatch loggroup ARN

Change description

Added support for custom cloudwatch log group for APIGateway
Added variable for XRay
Fixed reference pointing to CloudWatch Role ARN instead of Log Group ARN (if custom log group used)
Updated TF Docs

Description here
The fixes the issue where the CloudWatch role ARN was used in place of the log group ARN. As no log group is created, i've added support to create a custom log group. I have also set XRay to be a variable for those that do not use Xray tracing, and updated the TF docs to match.

Type of change

[x] Bug fix (fixes an issue)
[x] New feature (adds functionality)

Related issues

Fix #13

Checklists
Development

[x] Lint rules pass locally
[x] Application changes have been tested thoroughly
[x] Automated tests covering modified code pass

Security

[x] Security impact of change has been considered
[x] Code follows company security practices and guidelines

Code review

[x] Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
[x] "Ready for review" label attached and reviewers assigned
[x] Changes have been reviewed by at least one other contributor
[x] Pull request linked to task tracker where applicable

@angellopez-sg / @ivancasco-sg / @StratusChris could i please get this reviewed

@angellopez-sg are we able to get this merged?

@oc-christopher-billett merged
Thank you for your contribution


Esc calibration

Enables esc calibration via QGroundControl.
@DonLakeFlyer The command for starting the calibration is MAV_CMD_PREFLIGHT_CALIBRATION with param7==1.
Then once the user has pressed the button for esc_calibration and the command has been accepted by the autopilot the user should have the option to press another button which is used to switch from high to low pwm (that is MAV_CMD_PREFLIGHT_CALIBRATION with param7 == 2 ).
The system is designed such that even if the user does not press the second button the system will switch from high to low pwm after 5 seconds. But we don't know if this timeout is ok for every esc and therefore, we should give the user this option.

Still need to implement the case when motors are attached to the FMU pins.

This is tested with a Pixhawk having the motors connected to the IO.

@DonLakeFlyer Could we get support for this in QGC in the medium-term? It seems a common user setup issue and depending on the ESC can lead to flips / unflyable results.

Yes, I have this on my list.


Third month overlapping with footer in mobile view

The third month is overlapping with the footer in mobile view. Can I fix this issue?

@aastha985 can you work on this issue?

Sure, I'll work on it


Contact Details
<EMAIL_ADDRESS>Describe the bug
When executing unit test, bringing back incorrect result. Should be Mumbai (Bombay) but getting Jakarta.
Version
0.1-alpha-5 (Pre-release v5.0)
Any additional information?
No response
Code of Conduct Agreement

[X] I have read and acknowledged the Code of Conduct for this project

Amended code, issue was with selection of region rather than country.


Fix build under MSYS2 MinGW-w64 (Windows)

Under MSYS2 MinGW-w64 the filename of the hidapi import library
is "libhidapi.dll.a". Add hidapi.dll to NAMES of findlibrary().

Here are some short instructions for building TEMPERed for Windows under MSYS2 MinGW-w64:
# install the required packages cmake and hidapi:
pacman -Sy --needed ${MINGW_PACKAGE_PREFIX}-cmake ${MINGW_PACKAGE_PREFIX}-hidapi

# Run the following commands from the base of the TEMPERed directory:
mkdir build
cd build
${MINGW_PREFIX}/bin/cmake.exe -G "MSYS Makefiles" ..
make

Just in case somebody is looking for Windows binaries of TEMPered, builds have been prepared and added here for download:
https://github.com/c72578/TEMPered/releases
The builds include PRs from here, which added new devices:
0C45:7402 TEMPerHumM12V1.2
0C45:7402 TEMPerHumM12V1.3
413d:2107 TEMPer2V3.1 or TEMPer2V3.3 (TEMPerGold_V3.1, TEMPerX_V3.3)

@c72578 I think this repo is dead. If you target your PR to https://github.com/mwerezak/TEMPered I will try to merge it.

Actually I was able to do it myself :wink:


feat: Add CSS support

Prerequisites checklist

[x] I have read the contributing guidelines.

What is the purpose of this pull request?

What changes did you make? (Give an overview)

Added support for CSS
Added a document describing how to add a new language to Code Explorer

Related Issues
Is there anything you'd like reviewers to focus on?

@harish-sethuraman @amareshsm can I get a review on this?


Provide proper css for the callback page

The page displayed after the user logged in looks like crap, need to add some Red Hat pizzaz to it

Should be good enough for now:


[feature request] speed of submenu opening

Please add an option to modify the speed for submenu opening animation.
i.e.:
enum SubMenuSpeed{  Low, Middle, High }

It is possible to change the speed of the animations
setMenuResizeSpeed(menuResizeSpeed); -> Default: High
setSubMenuSpeed(subMenuSpeed); -> Default: High


eventsub-ws reconnect is triggering onUserSocketDisconnect

Bug Report
Code
Expected behavior
Reconnect should have own event onUserSocketReconnect which should catch this, or add isReconnect parameter to onUserSocketDisconnect

Actual Behavior
During reconnect onUserSocketDisconnect is called, where there is currently no way to differentiate, why socket disconnected

2023-04-20T19:13:10.899 | EVENTSUB-WS[4]: Received data: {"metadata":{"message_id":"25b99a7b-051e-427c-bf20-2cbad8a6b263","message_type":"session_reconnect","message_timestamp":"2023-04-20T17:13:10.881027546Z"},"payload":{"session":{"id":"AgoQzaLHZn_nQYOjyApM7DDxwxIGY2VsbC1j","status":"reconnecting","connected_at":"2023-04-20T13:11:51.684331603Z","keepalive_timeout_seconds":null,"reconnect_url":"wss://cell-c.eventsub.wss.twitch.tv/ws?challenge=f7e20416-f1c6-4609-94b4-bd0c7caae4ab\u0026id=AgoQzaLHZn_nQYOjyApM7DDxwxIGY2VsbC1j"}}}
2023-04-20T19:13:10.900 | EVENTSUB-WS[3]: Reconnect message received; initiating new connection
2023-04-20T19:13:11.017 | EVENTSUB-WS: Service initialized for soge#96965261
2023-04-20T19:13:11.019 | EVENTSUB-WS[4]: Received data: {"metadata":{"message_id":"69f440ba-dca4-4915-a0e1-63236294e40f","message_type":"session_welcome","message_timestamp":"2023-04-20T17:13:11.010128332Z"},"payload":{"session":{"id":"AgoQzaLHZn_nQYOjyApM7DDxwxIGY2VsbC1j","status":"connected","connected_at":"2023-04-20T13:11:51Z","keepalive_timeout_seconds":10,"reconnect_url":null}}}
2023-04-20T19:13:11.019 | EVENTSUB-WS[3]: Reconnect: new connection established
2023-04-20T19:13:11.057 DEBUG: twitch.eventsub onUserSocketDisconnect called

Environment

Version: 6.2.0

There will be more "intentional" disconnects in the future, and they will not be emitted separately. You can tell intentional from unintentional disconnects from the error parameter (the second one) being present. Does this help your use case?


Project is die?

Project is die?

But I'm interested

Me too. Able to focus on another repository and revive the project ? :D

Someone should reach out to the maintainer to discuss joining the project.  Anyone super-motived should clone and begin improvements, to demonstrate commitment.
IMO

FYI there is a plan for React Studio to open source it sometime in the near future. Perhaps it’s worth waiting for that to happen and start with a solid base

@jasonehle Why waiting? open the source!

Guys! Any updated information about this project?

??

Any chance this can revive?

very sad to see a project like this perishing... :-/

Wouldn't it make more sense to port this to a vscode extension?

This would be awesome. I am not familiar in making VSC extensions. I also struggle to build the version-2 branch successfully.

After learning how to make VSC extensions this weekend, it seems like its API is currently too limited (doesn't support a right sidebar) to produce a 1:1 clone of Reactide.  It'd likely take a maintained fork of VSC to do it justice.
Reactide's UI is itself written in React, so it'd be a matter of building React components one by one, then mapping them to VSCode (the latter half probably being the hardest part with most unknowns.)

Thanks for the support everyone!!
We have added new members to the team, including myself, and we're glad to say that it is alive. We will be launching 3.0 Beta with lots of improvements from 2.0 on Tuesday, 8/13. You will be able to download the application directly on our website without needing to fork/clone the repo.
All beta testers are welcome as the master branch is currently up-to-date.


profile-gists-link is not responsive

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.

Include in this issue:

[X] Screenshots/video/gif demonstrating the bug, if it’s visual
[ ] Console errors, if any

Description
https://user-images.githubusercontent.com/44045911/145593519-f93e98f0-001f-4df0-b0e6-b3b5dca18dd3.mov
Example URL
https://github.com/kidonng
Browser(s) used
Chrome

I think this is a new thing from github. Probably needs the same logic as more-dropdown

I think this is a new thing from github.

It could be, the Stars are moved from the left side. Then #5164

Related: https://github.com/refined-github/refined-github/issues/4008#issuecomment-828173667

Things are still a little chunky:
https://user-images.githubusercontent.com/44045911/154839468-d6abaa33-f500-4cc7-82df-d766855e38d4.mov
But since it goes away when resized and doesn't appear when hiding the "Packages" tab (which is what I do), maybe consider it a "known bug"?


Errors building snap package for Wekan v0.62

@kubiko
I am unable to build snap package correctly. I did revert to older MongoDB version, thinking if that was problem, but it still gives some SSL errors:
https://launchpadlibrarian.net/349073984/buildlog_snap_ubuntu_xenial_amd64_wekan-devel_BUILDING.txt.gz
Here is current snapcraft.yml, it still uses Meteor 1.4 and Node 4.x:
https://github.com/wekan/wekan/blob/devel/snapcraft.yaml
I did try to upgrade to Meteor 1.6 and Node 6.12.2, and it did work locally when I built it from source, but changes to snapcraft.yml failed to build:
https://github.com/wekan/wekan/blob/meteor-1.6/snapcraft.yaml
Because with Meteor 1.6 build failed with Docker/Snap/Sandstorm builds then I had to revert back to Meteor 1.4 version to get new Wekan release v0.62 done. Other platforms for Wekan v0.62 are already released, but Snap version is not, because of these errors.

I fixed this myself. Wekan v0.62 snap is now released.


fixed tanlu to init alphas to .5

Fixed init of alpha values so they are in [0,1].

Please say 0.5, not .5.


Issue with BBOX filter - [ows:BoundingBox] must be an array

I've tried to apply a BBOX filter (paying attention to lat, lon order). With the GetRecords I get this message Argument [ows:BoundingBox] must be an array. Do you have any idea how to solve that? Thanks a lot for your reply

I had the same problem, I think I fixed it by commenting this lines:
 /* propertyName :{
                 TYPE_NAME: "Filter_1_1_0.PropertyNameType",
                 content: "ows:BoundingBox",
 } */

in filter.js

@HugoJBello apologies if i didn't get back to you on this. I had put in pause this dev. Indeed your fix solves the issue. Thanks a lot for that. For the time-being i'm overwriting it on my side, but i hope this fix will be done in the ows.js library.

I have been searching for projects and their issues to contribute , I really want to get into this org. can someone please suggest me how to?

I have been searching for projects and their issues to contribute , I really want to get into this org. can someone please suggest me how to?

@eblondel I also bump on this issue. Remarking the mentioned code helps not to fall on conversion JSON--> XML level.
But when you perform actual call to CSW you getting an error that ogc:PropertyNameows:BoundingBox</ogc:PropertyName> is missing.
So, think the miss-behavior is on schema definition level OWS_1_0_0 or Filter_1_1_0
Regards,
Alex


add sample test story using actions

Maybe this code snippet will be of help? I'm not sure where you are in terms of using actions, but this is a very simple use-case.

Somewhat helpful.   But this seems to be based on the documentation here:
https://github.com/storybookjs/storybook/blob/master/addons/actions/ADVANCED.md
which also says that this usage is deprecated and may be removed in a future version.
The usage documented on the mainline documentation here is quite different, and I have not managed to figure out how it is intended to be used.
https://github.com/storybookjs/storybook/blob/master/addons/actions/ADVANCED.md
I can make basic examples work, but it is not at all clear how these basic examples are useful, or how to adapt them for more advanced use cases.


Feat/challenges

Redesign of the home challenge section

I think it might be the way the carousel loads. I tried reducing file size to almost half and it loads the same way


Failed to import the module.

I tried two methods to import failed，Why is that? Is there a problem with my version?

This is the version I use

@DDI1999, yes, these changes available on develop branch, we use it for providing changes required for upcoming openvino version (at this time, it is 2022.3, which will be officially released soon).
For now we can install prerelease openvino version from pypi. Please install requirements.txt from the same branch what you use.

@DDI1999  I happy to say that openvino 2022.3 finally available via pypi. You can install and run notebooks :)

@DDI1999 I happy to say that openvino 2022.3 finally available via pypi. You can install and run notebooks :)

ok，Let me try


Add logo variations that change by season / date

It’d be fun to design a series of logo variations that celebrate various annual occurrences such as holidays or recognized dates. eg the anniversary of civic tech DC, halloween, pride month. We could build functionality and a few variations and then it’d be a fun thing for future graphically-inclined folks to contribute more to
To be clear, the idea is:

Design a set of logo variations
For each variation have a date range when it should be displayed
When the current date is within one of the date ranges show the logo variation
Include a tooltip or something explaining that we’re celebrating/recognizing X, and find our real logos at /press


Add Zenodo DOI

For these sorts of (very nice) resource lists I think it is nice to have a Zenodo DOI for them.
For reference, c.f.

The Issue that added one for IML HEPML Resources
The IML HEPML Resources Zenodo archive (probably in need of a new release soon)

Good idea.  For other projects (i.e. code projects), this hook gets triggered by a release through Github website. I'm not sure if there are other ways to trigger.   Otherwise, we might need to version the whole set of pages, which may not be a bad idea anyways.

It is triggered through creating a release. I don't think you actually need to use GitHub's UX to do so, I think any Git tag will do. The web UX just is more suggestive of the good practice of semantic versioning.

I didn't think any git tag would work (at least on other project).  Adding Zenodo now.  Added OAuth access to the Github organization.   DOI: 10.5281/zenodo.1420445

Looks good! I see that Zenodo is building the archive now at . :+1:


Fix broken build

Fix build by updating tlBaseVersion.

@rossabaker if you'd like to post-merge review this, I'm fine with that, but I want to get the steward PRs merged so I'm going to merge without review


stylix: re-add flake-utils dependency

This patchset re-adds the flake-utils dependency removed in commit ff5da2914cfc ("Remove dependency on flake-utils :heavy_minus_sign:").

[!NOTE]
These commits should be merged individually, not squashed, to ensure a clear and easily reversible changelog. In case of change requests, review the commits separately.

why you don't need flake-utils

why you don't need flake-utils
-- https://github.com/danth/stylix/pull/515#issuecomment-2336839545

I am well aware that flake-utils should be regarded as bloat at first sight. FYI, Micro-libraries need to die already might be an interesting read.
Aside from flake-utils delegating code maintenance, it standardizes the "externally extensible flake systems" pattern, providing a consistent external interface for using architectures that upstream does not want to maintain:

commit 9447b17f70806e987e38461c70c1de893d381a87

[...]

    stylix: re-add flake-utils dependency and interface flake systems

    Re-add the flake-utils dependency removed in commit ff5da2914cfc
    ("Remove dependency on flake-utils :heavy_minus_sign:") and interface
    flake systems using the "externally extensible flake systems" [1]
    pattern.

    [1]: https://github.com/nix-systems/nix-systems

commit 15fed84dec2ecf19e110bad1c47292e172b46a44

[...]

    stylix: drop i686-linux architecture support

[...]

    BREAKING CHANGE: Drop support for the i686-linux architecture. Re-enable
    i686-linux support in user configurations with the "externally
    extensible flake systems" [1] pattern.

    [1]: https://github.com/nix-systems/nix-systems

Since the "Why you don't need flake-utils" article, flake-utils has added support for handling the --impure flag:

Some of its functionality, like handling the --impure flag, are neat but would be too costly to re-implement on our side.
-- https://github.com/danth/stylix/issues/512#issue-2474015194

commit 77a65800e6b1389f7e0a90090297048720c99017

[...]

    stylix: adopt flake-utils.lib.eachDefaultSystem

    Adopt the flake-utils.lib.eachDefaultSystem function for added features
    like '--impure' flag support [1].

    [1]: https://github.com/numtide/flake-utils/pull/115

why you don't need flake-utils
-- https://github.com/danth/stylix/pull/515#issuecomment-2336839545

I am well aware that flake-utils should be regarded as bloat at first sight. FYI, Micro-libraries need to die already might be an interesting read.
Aside from flake-utils delegating code maintenance, it standardizes the "externally extensible flake systems" pattern, providing a consistent external interface for using architectures that upstream does not want to maintain:

commit 9447b17f70806e987e38461c70c1de893d381a87

[...]

    stylix: re-add flake-utils dependency and interface flake systems

    Re-add the flake-utils dependency removed in commit ff5da2914cfc
    ("Remove dependency on flake-utils :heavy_minus_sign:") and interface
    flake systems using the "externally extensible flake systems" [1]
    pattern.

    [1]: https://github.com/nix-systems/nix-systems

commit 15fed84dec2ecf19e110bad1c47292e172b46a44

[...]

    stylix: drop i686-linux architecture support

[...]

    BREAKING CHANGE: Drop support for the i686-linux architecture. Re-enable
    i686-linux support in user configurations with the "externally
    extensible flake systems" [1] pattern.

    [1]: https://github.com/nix-systems/nix-systems

Since the "Why you don't need flake-utils" article, flake-utils has added support for handling the --impure flag:

Some of its functionality, like handling the --impure flag, are neat but would be too costly to re-implement on our side.
-- https://github.com/danth/stylix/issues/512#issue-2474015194

commit 77a65800e6b1389f7e0a90090297048720c99017

[...]

    stylix: adopt flake-utils.lib.eachDefaultSystem

    Adopt the flake-utils.lib.eachDefaultSystem function for added features
    like '--impure' flag support [1].

    [1]: https://github.com/numtide/flake-utils/pull/115

I handle the main criticism of the "Why you don't need flake-utils" article by re-implementing flake-utils.lib.eachDefaultSystem without creating the nested ${system} key: https://github.com/trueNAHO/dotfiles/blob/4def3ff8e37a8c0511c34e80ee13c92c8ceea4be/flake.nix#L909-L945. If Stylix runs into this requirement I will PR this function to flake-utils. TBH, it is weird such a function does not already exist.
In conclusion, I disagree with "Why you don't need flake-utils".

I think users being able to turn on support for other CPU architectures is nice, although I can agree that pulling in a separate repository for the systems list is a bit superfluous

First, thanks for your thoughts on the topic and explaining them to me. Honestly, I think it's not that big of a deal and we're bikeshedding.
But because bikeshedding is fun: I'm not sure I get the point of extensible systems. Let's say I have a flake A and flake B is in my inputs and uses extensible systems. I can do two things:

extend flake B's systems. Let's say riscv64-linux, because my flake A supports it and it seems to work with flake B, when I tried.
remove some systems from flake B's system, because my flake A doesn't support them

For 1) I now have the problem that I need to test flake B for my use case and can never be sure that it still works after an update of flake B, because they are not aware of "their" now implicit support of riscv64-linux and make changes that breaks it at every time. So in case one, the cleaner solution is either to fork or to do a PR with tests and all and get it upstreamed so that they actually maintain support.
Regarding 2, I have no idea why I would do that, because flake B supports everything I want and I have a smaller subset of systems.
And the support for --impure seems nice, but has the same problem.

First, thanks for your thoughts on the topic and explaining them to me. Honestly, I think it's not that big of a deal and we're bikeshedding.
But because bikeshedding is fun [...]
[...]
I'm probably wrong here, but maybe someone takes the time to explain why.

Typical nerd sniping scenario: https://xkcd.com/356.

I'm not sure I get the point of extensible systems. Let's say I have a flake A and one of it's inputs is flake B. Flake B uses extensible systems. I can do one of two things:

extend flake B's systems. Let's say by riscv64-linux. I do this because my flake A supports it and it seems to work with the subset of flake B's functionality, when I tried.

[...]
For 1) I now have the problem that I need to test flake B for my use case and can never be sure that it still works after an update of flake B, because they are not aware of "their" now implicit support of riscv64-linux and make changes that breaks it at every time. So in case one, the cleaner solution is either to fork or to do a PR with tests and all and get it upstreamed so that they actually maintain support.
[...]
And the support for --impure seems nice, but seems to have the same problem as 1) to me.

The most concise answer I can give is: The same is true for all inputs.<INPUT>.inputs.<INPUT_DEPENDENCY>.follows declarations. Let me know if this needs further clarification.

remove some systems from flake B's system, because my flake A doesn't support them so maybe it's better not to allow them to sneak in somehow

[...]
Regarding 2) - I have no idea why I would do that, because flake B supports everything I want and I have a smaller subset of systems. I basically do not care whether something I don't use is there or not.

Unless flake B somehow improves its performance or footprint by considering inputs.flake-utils.lib.defaultSystems, this probably never makes sense. Even if it does, there are much greater inefficiencies that Nix introduces.

If Stylix runs into this requirement, I will PR this function to flake-utils. TBH, it is weird such a function does not already exist.
-- NAHO, https://github.com/danth/stylix/pull/515#issuecomment-2336848320

For reference, my PR has already been merged upstream: https://github.com/numtide/flake-utils/pull/119.


Upgrade to Webpack 2

Looks like the extract text plugin has now been updated to work with use etc.
Might be worth looking into https://github.com/andywer/webpack-blocks (not 100% sold yet though)

Tried webpack-blocks, imo it makes things more complicated than they need to be, makes the config less flexible and would be yet another dep to potentially cause problems.
I have taken inspiration from it however when updating the config here -- https://github.com/60frames/react-boilerplate/pull/80

https://github.com/60frames/react-boilerplate/pull/80


Error when using BulkInsert/BulkUpdate on a list of TPC/TPH entities with SQLServer

Hello
We encountered an issue when using TPC inheritance with entities (with SQLServer)
We have entities like this
public class Vehicle
{
  public string Name {get; set;}
}

public class Car : Vehicle
{
  public double Horsepower {get; set;}
}

public class Bus: Vehicle
{
  public int NumberOfSeats {get; set;}
  public int Capacity {get; set;}
}

public class ApplicationDContext: DbContext
{

  public DbSet<Vehicle> Vehicles => Set<Vehicle>();
  public DbSet<Car> Cars => Set<Car>();
  public DbSet<Bus> Buses => Set<Bus>();

  protected override void OnModelCreating(ModelBuilder modelBuilder)
  {
    modelBuilder.Entity<Vehicle>().ToTable("Vehicles");
	modelBuilder.Entity<Car>().ToTable("Cars");
	modelBuilder.Entity<Bus>().ToTable("Buses");
  }
}

In our case, we want to insert a list of Vehicle objects, where the elements can be either Vehicle, Car or Bus
However, when attempting to use applicationDbContext.BulkInsertAsync(elements), we encountered the following error :

Using 'FromSqlRaw' on DbSet of 'Vehicle' is not supported since 'Vehicle' is part of hierarchy and does not contain a discriminator property.

To resolve this, we added a shadow property Discriminator  to all entities in ApplicationDContext
public class ApplicationDContext
{

  public DbSet<Vehicle> Vehicles => Set<Vehicle>();
  public DbSet<Car> Cars => Set<Car>();
  public DbSet<Bus> Buses => Set<Bus>();

  protected override void OnModelCreating(ModelBuilder modelBuilder)
  {
   modelBuilder.Entity<Vehicle>(entity =>
   {
     entity.HasDiscriminator<string>("Discriminator")
      .HasValue<Vehicle>("Vehicle")
      .HasValue<Car>("Car")
      .HasValue<Bus>("Bus");
   });

    modelBuilder.Entity<Vehicle>().ToTable("Vehicles");
	modelBuilder.Entity<Car>().ToTable("Cars");
	modelBuilder.Entity<Bus>().ToTable("Buses");
  }
}

However, this resulted in a new error:

System.AggregateException: One or more errors occurred. (Invalid column name 'Capacity'.
Invalid column name 'NumberOfSeats'. Invalid column name 'Horsepower'.) ---> Microsoft.Data.SqlClient.SqlException: Invalid column name 'Capacity'.
Invalid column name 'NumberOfSeats'. Invalid column name 'Horsepower'

The same error occurs when using TPH (e.g. removing the .ToTable lines from ApplicationDContext)
Is there any way to successfully use BulkInsert on a list in this case?
Thanks in advance for your assistance.

First you do should not call SaveChangesAsync when using BulkMethods.
Secondly you need to have 'Discriminator' column set.
One way would be ne expose that property and set it to appropriate table name before calling BulkMethod.
And another option, if you want it to remain Shadow property, than you need first to add list into DbSet with AddRange, which will set Disct.col. automatically, after that call BulkInsertAsync (do Not call SaveChangesAsync)
You can take at DiscriminatorShadowTest for sample:
https://github.com/borisdj/EFCore.BulkExtensions/blob/770f46f59f90cd8ff96e0c7988a0a7d81db689a8/EFCore.BulkExtensions.Tests/EFCoreBulkTestAtypical.cs#L469-L470


Add ability to disable ssl verification.

Thoughts on this?
Only other solution I see is to pass a list of client options that are effectively pass through for your library, so that the underlying aws libs could change without any code changes needed here...

@scattym How about passing preconfigured client in case some another fine tuning is required?

Yeah, good point. Will create another pull request in the next day or two.

Closing in favour of newer PR


CalcMod

CurseForge link
https://www.curseforge.com/minecraft/mc-mods/calcmod
CurseForge Mod Distribution
Not allowed
Modrinth link
https://modrinth.com/mod/calcmod
Source/other link
https://github.com/js802025/calcmod/tree/1.20
Mod file size
599 KB
License
(any other license)
What it does
A calculator in your Minecraft chat with extra features to determine farm rates, generate random numbers, convert between Overworld and Nether coordinates, and a bunch of other nerdy things.

/calc craft piston 3 sb

Why should it be in the modpack

As un-intrusive as it gets, a small QoL addition that only lives in the Minecraft chat
Has time saving shortcuts catered towards technical minecrafters and redstoners, but easily accessible to all.
All features are well documented in the in-game help command, as well as the CalcMod listing page
Works on either client or server

Why shouldn't it be in the modpack

File size is on the bigger side due to the library used to process equations

There are a lot of ways to use a calculator nowadays, so I would say having it in-game is very niche and again - has to be learned how to use.
Also, the item calculation thing looks nice, but I have a feeling most users who need it would use Litematica for something like that.


Add a map of all the cool locations/countries you visited

https://github.com/MLH-Fellowship/project-reboot-rebels/pull/21


Plugin causes timeouts on environments which has 30/60 second max execution time

This plugin seems to assume that it is run in a environment with a high maximum execution timeout, which causes certain errors (such as server overload/gateway timeout) when run on environments which has a max execution time set to 30 or 60 seconds.
This is one example that I found (where a max wait of 60 seconds are assumed + additional request latency time)
https://github.com/SwedbankPay/swedbank-pay-woocommerce-payments/blob/1b29c725b00420850d86cc46953d8d7fb4ba3ce9/includes/class-wc-gateway-swedbank-pay-cc.php#L653
Is this type of error expected to cause a server overload/gateway timeout? If not my suggestion is either to adjust max wait time according to environment, lower max wait time to fit within 30 seconds (which is php ini default) or find another architecture for this type of problem.

This issue could be solved with that pull request https://github.com/SwedbankPay/swedbank-pay-woocommerce-payments/pull/73

This issue could be solved with that pull request https://github.com/SwedbankPay/swedbank-pay-woocommerce-payments/pull/73

Excellent, thanks @aait!

Excellent, thanks @aait!


numpy compatible dsplit operator

Description
Implemented the dsplit operator.
It seems indices are not allowed to be passed into the indices_or_sections parameter, so the corresponding tests are disabled.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

Please resolve the sanity check failure.


Can't Activate Features in Cargo Check

Hello!
I updated to the current (as of posting this issue) version of the RA and it appears that the way features work with cargo check has been changed, and it's made all my diagnostics disappear.
Here's what I have as my settings:
{
    "rust-analyzer.cargoFeatures.features": [
        "metal"
    ],
    "rust-analyzer.cargo-watch.arguments": [
        "--features=\"metal\""
    ]
    "rust-analyzer.cargoFeatures.allFeatures": false
}

Additionally, just to make it clear, here is the relevant part of my Cargo manifest:
[features]
metal = ["gfx-backend-metal"]
dx12 = ["gfx-backend-dx12"]
vulkan = ["gfx-backend-vulkan"]

I'm sure I'm just formatting something incorrectly, but trying to go through the server log, it doesn't look like the actual command passed to cargo check is listed anywhere explicitly. I see a call out that looks like it's it, but I don't know what the exact command is.
Any help would be very appreciated! Thank you!

I'm not sure if it'll have an impact on this issue, but for just for the sake of it, now that #2959 has landed, could you please retry on latest master?

After a git pull and installing again I opened a project with build errors and right away I get "problems" window populated with errors, so it seems like things definitely improved since my last message above.
I still don't see a "cargo watch" command in ctrl+shift+p menu though.

That command doesn't exist (see #2764), so that makes sense.

OK. It's not necessary now as the "problems" window is now working. It'd be good to update the documentation which suggests such a command exists.

No dice @kiljacken . I've open sourced a snapshot of the repo in question here, so if you want to mess about with it yourself, that's available.
In the meantime, I've reverted the RA back to commit d652e1e67 which just happens to be what my laptop had, and which happens to still work.

loaded up the current version of the server. things seem to be working. did something et solved? who knows. closing issues is fun tho. let's close it!

Did you intend to re-open?

(yes, should have typed and then reopened. I'm typing up a whole thing right now and then will delete this message)

Nevermind, closing issues is for losers!
Immediatley after closing the issue, the problem reappeared in current (pulled from master with the key added to VSC to prevent it pull from releases) version...
But I have found an excellent clue to the problem!
I'll walk through the whole thing quickly. I have multiple features in my cargo which related to different graphical backends. Those backends are loaded into cargo conditionally and then the features are activated based on what flags are passed in, like normal with --features=metal for example.
Here is my original Cargo.toml:
[dependencies.gfx-backend-vulkan]
version = "0.4"
optional = true

[target.'cfg(target_os = "macos")'.dependencies.gfx-backend-metal]
version = "0.4"
optional = true

[target.'cfg(windows)'.dependencies.gfx-backend-dx12]
version = "0.4"
optional = true

With that, the RA would either fail to send any checks or would be unable to detect any of the features loaded. On a hunch, I changed that cargo.toml to this:
[dependencies.gfx-backend-vulkan]
version = "0.4"
optional = true

[dependencies.gfx-backend-metal]
version = "0.4"
optional = true

[dependencies.gfx-backend-dx12]
version = "0.4"
optional = true

And boom, the RA now correctly loads metal on macOS. (To be clear, the entire time this was happening, I was specifying that features=metal in VSC settings to the RA). I feel like this has to do with targets for the Rust Analzyer, but I'm not sure beyond a hunch, but I feel like this is a promising lead!
Thanks for all the help so far guys, hope this helps even further

I believe the #[cfg] for the host system are used by rust-analyzer.

Okay, nevermind that nevermind. I'm straight up

Sometimes it only loads the correct feature. Other times, it doesn't. I don't have access to exactly what's going on, but when it loads too much, whihc causes it to say certain modules aren't available, which are, it'll say loaded 273 packages. When it has loaded the correct modules, then it will only load 250-ish packages.
 The difficult part is shaderc, whose build instructions you can see here -- https://github.com/google/shaderc-rs
For best results, pull from the prefabs-robust branch. I'll leave it here for now

It started working again, I have no idea why, let's call it a day.


[DO NOT MERGE - IN PROGRESS] Real-time telemetry for teams admins

[DO NOT MERGE - IN PROGRESS]
Documentation around Real-time telemetry for teams administrators.
@MicrosoftHeidi @KarliStites

Docs Build status updates of commit d37d5e4:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/7871/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


mktemp: replace workaround code with features of clap 3

This code has a comment describing a workaround for a missing feature in clap that was not available until clap 3:
https://github.com/uutils/coreutils/blob/bf3a86f654b5982b124dd385782ee8feea10abbd/src/uu/mktemp/src/mktemp.rs#L89-L98
Perhaps some code here can be simplified now that clap 3 is available?

I'm not sure if the linked issue in clap 3 fixes the problem.
The issue stems from the fact that --tmpdir will never be empty if a template is passed (e.g. mktemp --tmpdir apt-key-gpghome.XXX)
clap will interpret the value of --tmpdir to be apt-key-gpghome.XXX rather than "" hence why this workaround is in place.
Adding default_missing_value doesn't fix it because --tmpdir is always going to be populated


[Bug Fixed] Fix all reduce grads when training more than one models

Description
Fix the issue #62
Major Revision

Remove model_state.ready_to_all_reduce_grads check in optimizer
unit-test

Do we have the scenario of training multiple models?

Do we have the scenario of training multiple models?

https://github.com/facebookresearch/fairseq/blob/main/fairseq/trainer.py#L928-L930
                self.optimizer.all_reduce_grads(self.model)
                if utils.has_parameters(self.criterion):
                    self.optimizer.all_reduce_grads(self.criterion)

In fairseq, the gradients of two modules self.model and self.criterion are both reduced.


remove EOT font files

Fixes https://github.com/Khan/KaTeX/issues/893.

LGTM, but I don't think this is sufficient to fix https://github.com/Khan/KaTeX/issues/893 : in the KaTeX repo, static/fonts.less has references to the .eot files that need to be removed.
Also, while we're removing fonts, should we remove any others?  This table and this table suggest that TTF or WOFF alone would suffice for all browsers.  But I guess WOFF is both smaller and more recommended by W3C, and I'd be hesitant to remove TTF which work nicely on local operating systems.  Not sure what advantages WOFF2 offers, other than IE/Edge incompatibility...
By the way, reviews seem not to be required for merging on this repo.

By the way, reviews seem not to be required for merging on this repo.

Good call.  I've changed that.

Not sure what advantages WOFF2 offers, other than IE/Edge incompatibility...

WOFF2 files are about 10% smaller.

in the KaTeX repo, static/fonts.less has references to the .eot files that need to be removed. (A challenge with having multiple repos, I guess.)

I'll create a PR for that repo that removes those references as well as updates the submodule pointing to this repo once this gets merged.  At least we don't have to update three repos. :)


Add G&E Ticket template

Add new PR template for growth and engagement tickets

I didn't update the bug and epic files directly; I think GH did that automatically.


Roll engine 3757390fa4b0..af39e22c318a (5 commits)

https://github.com/flutter/engine/compare/3757390fa4b0...af39e22c318a
git log 3757390fa4b00d2d261bfdf5182d2e87c9113ff9..af39e22c318a5d39dec3c9453c8f2e77ea76af79 --no-merges --oneline
af39e22c3 Fix tests that were committed after cirrus ran (flutter/engine#7819)
1f2cbf792 Roll src/third_party/dart 0a7dcf17eb..c92d5ca288 (64 commits)
033f2072c Add space metrics tracking for trailing whitespace "ghost" runs. (flutter/engine#7791)
adf02abcd Allow specifying the buildtools path (flutter/engine#7815)
f519aedb9 Fix typo in painting.dart (flutter/engine#7814)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5547671256301568 , https://cirrus-ci.com/task/5406933767946240 , https://cirrus-ci.com/task/6673571163144192 , https://cirrus-ci.com/task/4632877581991936

Commit queue failed; closing this roll.


Carthage Build Fails on Xcode 7 Beta 5

Build fails with error:
The following build commands failed:

	CompileSwift normal x86_64 /Users/dgrekov/Documents/Code/DWearIOS/Carthage/Checkouts/PromiseKit/Sources/URLDataPromise.swift
	CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
Assertion failed: (PAI2->use_empty() && "Should not have any uses"), function foldInverseReabstractionThunks, file /Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-7<IP_ADDRESS>/src/swift/lib/SILPasses/SILCombinerVisitors.cpp, line 549.
A shell task failed with exit code 65:

Removing this file makes the build pass.

I'm having the same issue with that class.
sudo xcode-select -s /Applications/Xcode-beta.app

projects/gryphon media/dopehood  feature/json-parse-argo ✗       23h51m ✖ ⚑ ✚
▶ carthage update
*** Fetching PromiseKit
*** Fetching OMGHTTPURLRQ
*** Checking out OMGHTTPURLRQ at "3.0.0"
*** Checking out PromiseKit at "027a5c321042dcd4ef42a54e6225d8bf6611afc1"
*** xcodebuild output can be found in /var/folders/3r/kpdj1c916rj_gy_4l99g1klw0000gn/T/carthage-xcodebuild.UnRaUB.log
*** Building scheme "OMGOSX" in OMGHTTPURLRQ.xcodeproj
*** Building scheme "OMGiOS" in OMGHTTPURLRQ.xcodeproj
*** Building scheme "PMKiOS" in PromiseKit.xcodeproj
** BUILD FAILED **

The following build commands failed:
	CompileSwift normal x86_64 /Users/marlowcharite/Developer/iOS/projects/gryphon media/dopehood/Carthage/Checkouts/PromiseKit/Sources/URLDataPromise.swift
	CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)
Assertion failed: (PAI2->use_empty() && "Should not have any uses"), function foldInverseReabstractionThunks, file /Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-7<IP_ADDRESS>/src/swift/lib/SILPasses/SILCombinerVisitors.cpp, line 549.
<unknown>:0: error: unable to execute command: Abort trap: 6
<unknown>:0: error: swift frontend command failed due to signal (use -v to see invocation)
A shell task failed with exit code 65

Not sure I can fix the Swift compiler crashing.

@mxcl builds fine through cocoapods so might want to switch to that @dgrekov

Also noticed that it builds fine through Xcode directly. It seems to only crash when building command line.

On Aug 16, 2015, at 18:44, marlowcharite<EMAIL_ADDRESS>wrote:
@mxcl builds fine through cocoapods so might want to switch to that @dgrekov
—
Reply to this email directly or view it on GitHub.

Compiling it with --verbose gives more information, but nothing that is useful without re-writing URLDataPromise which shouldn’t be necessary since it compiles in Xcode OK.

It’s not an issue with command line builds per se, it crashes for all release builds. Definitely a bug in the compiler, but it might be worthwhile turning off optimisations on Release builds for the beta5 branch.

More specifically it's an issue with this line (45-47):
private override init(@noescape resolvers: (fulfill: (NSData) -> Void, reject: (ErrorType) -> Void) throws -> Void) {
        super.init(resolvers: resolvers)
    }

this is what the compiler takes issue with.

Hopefully the next beta will help.

looks like the "swift-2.0-minimal-changes" branch compiles fine with carthage, but "swift-2.0-beta5" does not.

Seems to build with beta6.

 Started running into it again this morning :(


Targeting a monster in combat with 13-attack fails

warn: ThirteenIsh.Services.DiscordService[5]
[Discord]: A SlashCommandExecuted handler has thrown an unhandled exception.
System.Collections.Generic.KeyNotFoundException: The given key 'PlayerCharacter' was not present in the dictionary.
at System.Collections.Generic.Dictionary2.get_Item(TKey key)          at ThirteenIsh.Commands.Combat.CombatAttackSubCommand.<>c__DisplayClass2_0.<<HandleAsync>b__1>d.MoveNext() in D:\Git\ThirteenIsh\ThirteenIsh\Commands\Combat\CombatAttackSubCommand.cs:line 133       --- End of stack trace from previous location ---          at ThirteenIsh.Commands.Combat.CombatAttackSubCommand.HandleAsync(SocketSlashCommand command, SocketSlashCommandDataOption option, IServiceProvider serviceProvider, CancellationToken cancellationToken) in D:\Git\ThirteenIsh\ThirteenIsh\Commands\Combat\CombatAttackSubCommand.cs:line 77          at ThirteenIsh.Commands.CommandBase.HandleAsync(SocketSlashCommand command, IServiceProvider serviceProvider, CancellationToken cancellationToken) in D:\Git\ThirteenIsh\ThirteenIsh\Commands\CommandBase.cs:line 54          at ThirteenIsh.Services.DiscordService.OnSlashCommandExecutedAsync(SocketSlashCommand command) in D:\Git\ThirteenIsh\ThirteenIsh\Services\DiscordService.cs:line 210          at ThirteenIsh.Services.DiscordService.OnSlashCommandExecutedAsync(SocketSlashCommand command) in D:\Git\ThirteenIsh\ThirteenIsh\Services\DiscordService.cs:line 216          at Discord.EventExtensions.InvokeAsync[T](AsyncEvent1 eventHandler, T arg)
at Discord.WebSocket.DiscordSocketClient.TimeoutWrap(String name, Func`1 action)

Fixed


Misleading description

Librg is advertised as an MMO solution but authoritative servers, while possible, seem like an afterthought and I don't see any API for interest management. In fact I would say that librg is ideal for anything besides MMO.
Am I missing something obvious here?

Hello @jesta88!
Firstly, while we do agree an MMO title might be misleading for project like this, we do offer utilities and helpers that help us reduce bandwidth and performance costs by making sure each and every player receives only the data he is interested in. We make sure we send data to clients only about those entities that are "visible" and that are "close" based on their specific range settings. For example, distant player outside of city will not receive information about player/entities inside the city, because the city is not covered by his streaming range.
Secondly, we also offer an ability to send messages to all entities inside of a particular entity's interest range. There is no ability to send messages within a specific range based off of a random point in the world yet, however we are planning on adding it later on.
On the other hand, while we can say that the server authoritative model wasn't truly a priority while creating this library, if you look on it from the other side, it mostly requires sending needed inputs via messages from the client side and handling the game logic on the server side to make it work.
At last, the library itself was tested in an environment with more than thousand connected active player which, in some way, or according to some definitions, might be at least partially qualified as an MMO solution.
Nevertheless, we do value your opinion and we are going to try to update the description to make sure it represents the idea better and be less confusing.
If you have any more questions we will be glad to answer them.
Thank you!

@jesta88 just wanted to notify you, that we've updated the description.
have a nice day!


puppet-lint: fail on warnings

I am pretty sure that in the past we failed on puppet-lint already. with
our latest setup we continue in the CI pipeline even if puppet-lint
detects issues. This might be caused by the recent
puppetlabs_spec_helper update.

tested in https://github.com/voxpupuli/puppet-dbbackup/pull/29

should we merge https://github.com/voxpupuli/modulesync_config/pull/759 instead and enable it in the .puppet-lint.rc? that works for bundle exec puppet-lint and the rake tasks. I checked pdk-templates and they enable it in the Rakefile and .puppet-lint.rc.

since other orgs might use this as well but won't use our modulesync_config, I think it's good to merge this PR, even when https://github.com/voxpupuli/modulesync_config/pull/759 is merged. pdk also sets it in the Rakefile and the puppet-lint config.


[FEAT] Support Mantine UI lib

Is your feature request related to a problem? Please describe.
mantine.dev seems to be a pretty neat framework. Compares to antd, it has a modern style system and a clearer API
Describe the solution you'd like
Use mantine instead of antd
Describe alternatives you've considered
Additional context

Yeah, we need this!

We will have some updates on this feature, hopefully very soon. 🚀

Any timeline we can look forward to?

Hey @jasonlimantoro ,
We plan to release it this month ⚡

Thanks for the prompt reply @omeraplak.
Great! Really excited for this! 🎉


update for stock opname enhance

Coverage increased (+0.04%) to 35.4% when pulling 692d0d676dc6d18a6195bc9008de89476b7198fa on AchmadSetiaji:dev into 037fc684bb0c33c98ba04b52dc8076eaafa3e51a on danliris:dev.


Improvement copy row in batch in VCollectIterator&VGenericIterator

In VCollectIterator&VGenericIterator, use insert_range_from to copy rows
in a block which is continuous to save cpu cost.
If rows in rowset and segment are non overlapping, this whill improve 30%
throughput of compaction.If rows are completely overlapping such as load two
same files, the throughput goes nearly same as before.
Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

@Gabriel39 @BiteTheDDDDt Could you guys review this please


Python 3 support

Hi,
I cannot install django fluent blogs in Django 1.8 / Python 3.
Is this because django categories does not support python 3?:
https://github.com/callowayproject/django-categories/issues/102
regards

Well, the package itself hasn't been ported to Python 3 either. The other packages are, simply because the demand for it was much higher.
This package isn't strictly linked to django-categories. You can define FLUENT_BLOGS_CATEGORY_MODEL in your settings to have the models link to another package instead. In fact, I'm working on a django-categories-i18n package that provides translatable category names (plus that I'm annoyed by the speed of development in django-categories).

the current version works on Python 3 for some time now. Closing.


add name to interested-technical-writers.md

Hi! It looks like you've edited a lot of entries beside your own. Could you please revert those edits (even if they are minor, like added or removed spaces) and only submit a PR with your information? Thank you!


Override style on Tabbar items

It would be nice to be able to override the styling on tabbard items, in particular the icons. For instance, I would like to not have title under icons, in which case I have to vertically center the icons (they are pushed down if I remove the title atm). Also it would be nice to change the size and selected icon for the currently selected icon!

+1 for changing the currently selected icon

Hi guys.  I have been away for the last few days but I will take a look at a possible good solution for this.

This could be solved by just using a custom TabBar component and change it to your liking.

Perhaps we can add an option to the route to define your own TabBarItem class?  Instead of using the TabBarIcon that's in TabBar.js?


Rasa not added to path

Rasa Version : 2.0.0rc1
Rasa SDK Version : 2.0.0rc1
Rasa X Version : None
Python Version : 3.8.5 (default, Sep  5 2020, 10:50:12)
Operating System : Linux-5.8.10-zen1-1-zen-x86_64-with-glibc2.2.5
Python Path : /usr/bin/python
Issue: Rasa is not added to path, thus all the command rasa xxx are not found
Error (including full traceback):
rasa xxx -> zsh: command not found: rasa

Command or request that led to error:
rasa xxx

More informations : I've seen that I would be able to install rasa with python 3.8 sce it's written in the docs.
So I did install rasa without problems (and without venv with tensorflow 2.3.0) on my laptop (archlinux kernel zen 5.8.10).
It's a minor problem since I can still use python -m rasa xxx to launch the desired commands.
However, it would be nice to have rasa in path when installing without venv so one could launch rasa commands

Thanks for raising this issue, @amn41 will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

@Ierezell Confirm your python library path. such as '/Users/****/Library/Python/3.8/bin'  need to join path.
export PATH=$PATH:/Users/****/Library/Python/3.8/bin


fetch reactiveQuery with sorted data

I want to fetch data from collection with sort
var prod=Production.reactiveQuery({sort: {index :  -1},productionId:$cookies.productionid});
sort is not working here

var sorteddata=prod.result.sort(function(a, b){return a.index-b.index});


Trailing slashes on full URLs for repos that do not use ref

Zenodo and Figshare (and possibly other repos) do not use the ref, leading to a trailing slash in the generated URL, e.g.
http://localhost:8585/v2/zenodo/10.5281/zenodo.3242074/
Do we want to remove that? Could be done in function v2url(..) in binderhub/static/js/index.js.
I'm in the "remove" corner, thinking of a URL ending / as a directory. Here's some wisdom by others:

https://stackoverflow.com/questions/5948659/when-should-i-use-a-trailing-slash-in-my-url
https://blog.papercut.com/trailing-slashes-on-your-urls-to-be-or-not-to-be-is-it-a-question/
https://www.cloudways.com/blog/trailing-slashes-urls/

(Or should we use the ref to identify the version or a record? Don't think r2d transparently supports this right now.)

I don't have a strong opinion on this. Without a trailing slash looks nicer, but I have to admit I hardly ever look/pay attention to this.
We should make sure that were possible we are forgiving to users and accept with and without trailing slash where possible. The number of hours I've spent staring at two seemingly equal URLs (where one worked and the other didn't) to help a user who had added/removed a / is far too big.


[SR-15436] Publish SwiftMarkdown as a CocoaPod

Previous ID
SR-15436

Radar
None

Original Reporter
@kylebshr

Type
New Feature

Additional Detail from JIRA

Votes
0

Component/s
Swift-Markdown

Labels
New Feature, SwiftMarkdown

Assignee
None

Priority
Medium

md5: ce40b1f3bca40cc5f750f648c051caea

Issue Description:
Many thousands of developers use CocoaPods for dependency management, and it would be excellent if those developers could consume SwiftMarkdown as a CocoaPod.

This appears to be an exact duplicate of https://github.com/apple/swift-markdown/issues/97


Additional status icons on Devices page

In GitLab by @jasonpagetas on Jul 9, 2024, 04:18
Currently on the devices page there is a status icon showing if the inventory has been conducted. There could be a benefit to having other quick information shown to a user / admin when accessing this page.
Status icons could include:

[ ] Device online
[ ] User logged in
[ ] Security compliant ok
[ ] Updates available (OS or Software)

marked this issue as related to #85

added 1m of time spent

marked this issue as related to #19

marked this issue as related to #3

@jasonpagetas, this issue was migrated from gitlab to github. During this migration you were not added as the user whom opened this issue. This message will ensure that you receive further updates on this issue as it progresses.
Further information can be found within #213


SelectedOnly seems not working

After search with option SelectedOnly in selected text the result returns all matches including from non selected area, and the need string didn't became selected\highlighted

Was title mistake sorry, now correct/

Fixed.


test and implement sambamba for pileup step

See: https://github.com/lomereiter/sambamba/issues/142

See: https://github.com/lomereiter/sambamba/issues/153


Implement GTE model to support the non-flash-attn version

What does this PR do?
I implemented the GTE model to support the non-flash-attn version and tested it's working on the T4 GPU.

to work with the gte-multilingual-base, config.json needs some modifications.

remove NewModelClassification from architectures.
remove new. prefix from the key of the weight file.

./target/release/text-embeddings-router --model-id ./gte-multilingual-base --port 12345 --dtype float32 --pooling cls
2024-11-30T09:07:38.940983Z  INFO text_embeddings_router: router/src/main.rs:175: Args { model_id: "./gte-************-*ase", revision: None, tokenization_workers: None, dtype: Some(Float32), pooling: Some(Cls), max_concurrent_requests: 512, max_batch_tokens: 16384, max_batch_requests: None, max_client_batch_size: 32, auto_truncate: false, default_prompt_name: None, default_prompt: None, hf_api_token: None, hostname: "e2fd3c35e003", port: 12345, uds_path: "/tmp/text-embeddings-inference-server", huggingface_hub_cache: None, payload_limit: 2000000, api_key: None, json_output: false, otlp_endpoint: None, otlp_service_name: "text-embeddings-inference.server", cors_allow_origin: None }
2024-11-30T09:07:39.613214Z  INFO text_embeddings_router: router/src/lib.rs:188: Maximum number of tokens per request: 8192
2024-11-30T09:07:39.613439Z  INFO text_embeddings_core::tokenization: core/src/tokenization.rs:28: Starting 4 tokenization workers
2024-11-30T09:07:41.166025Z  INFO text_embeddings_router: router/src/lib.rs:230: Starting model backend
2024-11-30T09:07:41.396429Z  INFO text_embeddings_backend_candle: backends/candle/src/lib.rs:352: Starting GTE model on Cuda(CudaDevice(DeviceId(1)))
2024-11-30T09:07:42.289292Z  INFO text_embeddings_router: router/src/lib.rs:248: Warming up model
2024-11-30T09:07:43.962553Z  WARN text_embeddings_router: router/src/lib.rs:310: Invalid hostname, defaulting to <IP_ADDRESS>
2024-11-30T09:07:43.965124Z  INFO text_embeddings_router::http::server: router/src/http/server.rs:1812: Starting HTTP server: <IP_ADDRESS>:12345
2024-11-30T09:07:43.965142Z  INFO text_embeddings_router::http::server: router/src/http/server.rs:1813: Ready

Close #375
Close #431
Close #439
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[x] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@OlivierDehaene OR @Narsil

Hello @kozistr, thank you for implementing GTE without flash attention—I’m looking forward to this being merged! I tested your branch on a T4 GPU and observed differences in the output vectors between the flash attention and non-flash attention versions. Specifically, for the query {"input": "Hello!"}, the cosine distance between the two versions on the T4 GPU is 0.52. Additionally, I ran the same query on an A100 GPU with flash attention and observed a cosine distance of 0.0000001 when compared to the T4 flash attention version, but still 0.52 compared to the T4 non-flash attention version. Do you know if such a significant difference could be due to precision discrepancies? Could this impact the quality of GTE models without flash attention?

@superchar hello! thanks for the report! honestly, I didn't test much this time, so there might be some issues. sorry for that. I guess there may be a discrepancy between the implementations, and the only differences are the flash attention and rotary positional encoding parts. I'll look into it later (maybe It'll take time cuz I have no proper local environment to develop and test it). thanks for letting me know!


feat: lemmas about partitions of unity

add finsupport: the support (the set of indices which functions are non-vanishing) at a point, as a Finset
every point has a neighbourhood on which only finitely many functions are non-vanishing

From sphere-eversion; I'm just upstreaming results.

Thanks for the review!
bors r+


check timestamp of gps before sending outdated location

check if the location fix is older then -let's say- one minute. if it is: dont send it with the push/pull request.

@cbalster reasoned about using 30s in his last PR and I agree with it:
// Average speed of the CM is ~4 m/s so anything over 30 seconds old, may already
// be well over 120m off. So a newer fix is assumed to be always better.
But the issue is still open as far as I see publishNewLocation is only called <30s now but PullServerHandler still sends out the old location

will take care of this


The deletionTimestamp and finalizers fields are always nil in the IstioControlPlane instance

The controller can’t get deletionTimestamp and finalizers from the IstioControlPlane instance, while they do exist, because I can get the instance from api-server with kubectl:
root@mcdev1:~/go/src/istio.io/operator# kc get istiocontrolplane example-istiocontrolplane -o yaml
apiVersion: install.istio.io/v1alpha2
kind: IstioControlPlane
metadata:
  creationTimestamp: "2019-10-12T12:45:39Z"
  deletionGracePeriodSeconds: 0
  deletionTimestamp: "2019-10-12T12:46:52Z"
  finalizers:
  - istio-operator
  generation: 2
  name: example-istiocontrolplane
  namespace: default
  resourceVersion: "1218"
  selfLink: /apis/install.istio.io/v1alpha2/namespaces/default/istiocontrolplanes/example-istiocontrolplane
  uid: 85a3dc1e-fa49-47c2-8210-247f1d39e1a3
spec:
  profile: default
status:
  status:
    Citadel:
      status: 2
    Galley:
      status: 2
    IngressGateway:
      status: 2
    Injector:
      status: 2
    Pilot:
      status: 2
    Policy:
      status: 2
    Prometheus:
      status: 2
    Telemetry:
      status: 2
    crds:
      status: 2

Generally, after the end-users delete the IstiocontrolPlane CR with kubectl, the deletionTimeStamp will be added to the IstiocontrolPlane CR, and the controller will start to handle the finalizers.
The current situation is that, the controller can’t get deletionTimeStamp field(it’s always nil in my env) after I delete the IstiocontrolPlane CR with kubectl.

The issue should be relevant to the tags generated by protobuf.
As jsonpb and json libs are using separate tags in the generated go struct;
while we can’t handle json tag for some fields, eg. deletionTimestamp and finalizers.

The issue should be relevant to the tags generated by protobuf.
As jsonpb and json libs are using separate tags in the generated go struct;
while we can’t handle json tag for some fields, eg. deletionTimestamp and finalizers.

@morvencao Why not just eject gogo as I would suspect it is the cause of this incompatibility?  We are going to have to play wackamole all over the code base because of this type of issue.

@sdake If you mean gogo proto lib, yes, I have tried that besides the official golang proto lib. We may need add customized marshal/unmarshal func, that's a lot work.

Reopen this ticket.

Let's move this work to more general cleanup: https://github.com/istio/istio/issues/17960
We need adequate time to solve this correctly, so targeting 1.5.