Support to new Properties and List

Contact Details
<EMAIL_ADDRESS>Present your request
Hi, Obsidian v1.4.5 added properties by default on each note. Could it be convenient to create a new type for columns, "List", to be similar to the list property in metadata?
For which platform do you request this request??
Desktop

This would be extremely helpful. Right now a column formatted as tags properly formats as a list, but there is no way to set the output of a formula to be a list.


[pytorch] Updates PyTorch engine to 1.10.0

Change-Id: I2b9fa7b31c7b7a627b176bb98370ecd2ca026000
Description
Brief description of what this PR is about

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

Codecov Report

Merging #1366 (7c49ce0) into master (bb5073f) will decrease coverage by 0.46%.
The diff coverage is 61.55%.

@@             Coverage Diff              @@
##             master    #1366      +/-   ##
============================================
- Coverage     72.08%   71.62%   -0.47%
- Complexity     5126     5179      +53
============================================
  Files           473      480       +7
  Lines         21970    22475     +505
  Branches       2351     2414      +63
============================================
+ Hits          15838    16097     +259
- Misses         4925     5153     +228
- Partials       1207     1225      +18

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <ø> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../modality/cv/translator/YoloTranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...i/djl/modality/cv/translator/YoloV5Translator.java
5.69% <0.00%> (ø)

...odality/cv/translator/YoloV5TranslatorFactory.java
8.33% <0.00%> (-1.67%)
:arrow_down:

...modality/nlp/embedding/TrainableWordEmbedding.java
45.00% <0.00%> (-3.34%)
:arrow_down:

...pi/src/main/java/ai/djl/ndarray/BytesSupplier.java
54.54% <0.00%> (-12.13%)
:arrow_down:

...i/src/main/java/ai/djl/ndarray/NDArrayAdapter.java
15.23% <0.00%> (-0.83%)
:arrow_down:

... and 122 more

Continue to review full report at Codecov.

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


client/dial.go: support multiple remote ips

tested locally on MacOSX, and fully compatible with old versions
originally kcptun only supports a single ip, now it can support multiple ips like -r ip1:port1,ip2:port2,...

may need to update documentation, will do that after this code has been merged


decimal + prefix + groupSeparator + negative value = Maximum call stack size exceeded

Overall, prefix doesn't work well with negative values.
Chrome 79.0.3945.117, Windows, Inputmask 5.0.3
Steps to reproduce:

Create a mask with this config: { alias: "decimal", prefix: '$', groupSeparator: "," }, or just go to https://jsbin.com/nowoked/edit?html,js,output
Click on the input, so that it gets focused.
Press - (minus), 1, . (dot), 2.

Actual results:

The minus disappeared, the value of the input is $1.2.

Two errors in the console:
inputmask.js:821 Uncaught RangeError: Maximum call stack size exceeded
	at resolveTestFromToken (inputmask.js:821)
	at handleMatch (inputmask.js:894)
	at resolveTestFromToken (inputmask.js:968)
	at handleMatch (inputmask.js:963)
	at handleMatch (inputmask.js:949)
	at resolveTestFromToken (inputmask.js:968)
	at getTests (inputmask.js:988)
	at getTestTemplate (inputmask.js:800)
	at getMaskTemplate (inputmask.js:756)
	at getBuffer (inputmask.js:1012)

inputmask.js:882 Uncaught RangeError: Maximum call stack size exceeded
	at handleMatch (inputmask.js:882)
	at resolveTestFromToken (inputmask.js:968)
	at handleMatch (inputmask.js:894)
	at resolveTestFromToken (inputmask.js:968)
	at handleMatch (inputmask.js:963)
	at handleMatch (inputmask.js:949)
	at resolveTestFromToken (inputmask.js:968)
	at getTests (inputmask.js:988)
	at getTestTemplate (inputmask.js:800)
	at getMaskTemplate (inputmask.js:756)

Expected results:
The value of the input should be -$1.2.

If we remove groupSeparator from the config, there is no errors, but the minus still disappears.

@RobinHerbots  tested with version 5.0.4 and the problem keeps.

See #2476


Reducing how much we call BigQuery Get Table API. The BQ API is queri…

…ed once per bundle. With this change, it will be queried once per worker thread. This will help with throughput for BQ streaming inserts
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.


Export all error. I seem to be hitting a rate limit

When I try to export all, many of the XHR requests fail. The response is
{
    "detail": "Too many requests in 1 minute. Try again later."
}

I get what I believe to be the same issue, when attempting to export all my ChaGPT conversations (I have well over a hundred). Numerous 429 Too Many Requests errors occur, one per conversation, for most of the conversations.
It seems ChatGPT exporter makes only one attempt is made to retrieve each conversation, and does not introduce any delay when it receives HTTP 429 errors. If I am correct in thinking this, then retrying with exponential backoff could solve the problem and facilitate exporting arbitrarily many conversations successfully. OpenAI suggests this approach when using their other APIs (though those code examples are Python-only).

It tries to asynchronously retrieve all the conversations at the same time.
I just changed changed it to sequentially grab the conversations:
    var counter = 0;
    for (const id of conversationIds) {
        const rawConversation = await fetchConversation(id);
        const conversation = processConversation(rawConversation);
        await sleep(200);
        counter++
        console.log(counter)
        conversations.push({
            conversation,
            rawConversation
        });
    };

This works for me, but I have to increase the sleep value.

sleep(200) and sleep(400) get me to count 50 (not sure why I can't do more with 400.
sleep(1000) gets me to 85.
sleep(1200) - count 110
sleep(1500) - finally got all 145 conversations.

But then I encounter a new error.
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'map')
    at conversationToHtml (userscript.html?name=ChatGPT%2520Exporter.user.js&id=e9bbbb4c-28c6-40e2-9827-f8279c573986:15540:48)
    at userscript.html?name=ChatGPT%2520Exporter.user.js&id=e9bbbb4c-28c6-40e2-9827-f8279c573986:15525:24
    at Array.forEach (<anonymous>)
    at exportAllToHtml (userscript.html?name=ChatGPT%2520Exporter.user.js&id=e9bbbb4c-28c6-40e2-9827-f8279c573986:15521:19)

code at line 15540:
const conversationHtml = conversationNodes.map((item) => {

I didn't see this until now: "I did this for json export, for the others you only need to push conversation."

yeah just change it from:
conversations.push({
            conversation,
            rawConversation
        });

to
conversations.push(conversation);

for markdown and html
the sleep value required probably depends on how many conversations within a minute you request.
retrying and using an exponential backoff would be best here, but I'll leave that for the dev to implement

Yeah. it's working now. thanks so much. I think a progress bar would be nice too. We're okay seeing it in the console logs, but a UI for it would be better.

I fully agree, this is just a stopgap solution I quickly made to get it working for myself

Thanks for all the information provided.
We have two issues here

[ ] Handle Rate Limiting
[ ] Export all will throw error on markdown and html

Yeah. it's working now. thanks so much. I think a progress bar would be nice too. We're okay seeing it in the console logs, but a UI for it would be better.

And ye, maybe a progress bar. The reason it is not being implemented is... it's just too fast when I do the small batch testing 😆

It will look like this

I am not able to export all due to rate limiting, looking at the network calls, it seems it trying to download everything in parallel, so I modified the code a bit to introduce artificial delay and I was able to download all my 326 conversations:
  async function exportAllToMarkdown(fileNameFormat, conversationIds) {
    const conversations = [];
    const delay = 1500;
    for (const id of conversationIds) {
        const rawConversation = await fetchConversation(id);
        const conversation = processConversation(rawConversation);
        conversations.push(conversation);
        await new Promise((resolve) => setTimeout(resolve, delay));
    }
    const zip = new JSZip2();
    conversations.forEach((conversation) => {
        const fileName = getFileNameWithFormat(fileNameFormat, "md", {
            title: conversation.title
        });
        const content2 = conversationToMarkdown(conversation);
        zip.file(fileName, content2);
    });
    const blob = await zip.generateAsync({
        type: "blob"
    });
    downloadFile("chatgpt-export.zip", "application/zip", blob);
    return true;
  }

Well, it's not fast.

I am not able to export all due to rate limiting, looking at the network calls, it seems it trying to download everything in parallel, so I modified the code a bit to introduce artificial delay and I was able to download all my 326 conversations:

1500ms works for you to download all of them?
I'm finding a reasonable delay :<


Feature/loadquickbar

Added a /load-quickbar command:
This reads a pre-set list from config/preset_player_quickbar.lua. {1..100} where 1-10 is quickbar 1, 11-20 is quickbar 2, etc. These use the factorio item names.
Added command file to the loader.
Added command to the sponsor role.

Think I've addressed both of those now @tovernaar123

Resolved 2 changes that were requested.
Never I've added a /save-quickbar command and this writes to a local script-output/quickbar_preset.txt file.

Added /load-toolbar and /save-toolbar aliases.
Saves output as a table to file.

Pulled the vastest luacheckrc.


[BUGFIX] Prevent leaked FH in windows

waiting for windows CI, once it runs (and it should end be being green) we can merge / release, then change ember-cli's dep to this.


For Gin Middleware: Scope Modification Only Works with Events Like Message/Error, Fails with Transaction

Use the example from https://github.com/getsentry/sentry-go/tree/master/gin
Set appending tags or some other scope changes via the router like the code below:
app.Use(func(ctx *gin.Context) {
    if hub := sentrygin.GetHubFromContext(ctx); hub != nil {
        hub.Scope().SetTag("someRandomTag", "maybeYouNeedIt")
    }
    ctx.Next()
})

The scope chagne only works with CaptureMEssage or Exception, and not works on Trasnactions.
So for the transactions, any solutions?

Use SetTag on the transaction.
https://github.com/getsentry/sentry-go/blob/b31dec1f53a6e0b4b51a6a905793675223828d56/tracing.go#L197
Currently, we do not take the current scope into account when creating the event from a transaction.

@cleptric @Aberd33n Maybe it's caused by context key, the Gin middleware and the Sentry core SDK use different context key, such it got different Hub for Event and Transaction.

The gin integration has its own function to receive the correct context
https://github.com/getsentry/sentry-go/blob/b31dec1f53a6e0b4b51a6a905793675223828d56/gin/sentrygin.go#L91
When cloning the hub, the scope gets cloned as well, so this should work.


please add this A.V.C. College of Engineering

It is a educational institution from india.
http://www.avccengg.net/ (can verify)

@rameezbasha Pull request merged. Thank you!


x/tools/gopls: add the nilness analysis

The nilness analysis pass can detect some absurd conditions, like checking a value that has already been proven non-nil against nil.  This is especially useful during refactoring, when a new nil check appears before an existing one.  I think that this analysis would be a good fit for the gopls analysis suite.  If I recall correctly, it uses SSA, so it should probably be disabled by default.
If there are no objections, I am ready to send a CL soon-ish.

Note that nilness was removed in https://golang.org/cl/219203, specifically because of the resources that SSA consumes.
That said, it makes sense to have available yet disabled by default.

Thanks for the review!


ActionPlanner - Reponse cannot parse regex sometimes

Chat copilot experiencing issues with ActionPlanner not giving valid plan.
Repro Details:
Model: TBD
Functions: GithubPlugin
Ask: List open pull requests

[ ] Reproduced

Closing since the action planner has been replaced with OpenAI function calling.

The bug is still around in Python, so would you mind keeping this open?


Add support for partial updates

Use case:
One team wants to manage a subset of the entities (for example to manage k8s entitlements) from github without affecting other entities (via a tag for example).
Right now we can not update partial entities because the dependency checker complains about missing entities that depend on the ones we want to manage.

Closed by https://github.com/appgate/sdp-operator/pull/110


Fix default value for the target OS

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Set the default value for a control to Windows, as is indicated by the control name.
Why should this Pull Request be merged?
The current default of an empty string prevents the scripted system definitions from being deployed correctly in some cases.
What testing has been done?
Scripted a system definition which previously failed to deploy and deployed it.

VI change looks good


Enable disk.enableUUID for vSphere control plane instances

What this PR does / why we need it:
This PR modifies the vSphere Terraform config to enable disk.enableUUID for the control plane instances. This is required for volumes to work properly, as stated in the vSphere Storage docs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #1125
Does this PR introduce a user-facing change?:
Enable disk.enableUUID for vSphere control plane instances

/assign @kron4eg

/cherrypick release/v1.0

tested can confirm it works now - even with an inplace terraform update :+1:


[Bug] Websites not in lists randomly get colored.

I like the idea of your script. Some things are broken though.
Various websites not in a defined list are getting colored. Also bleed-through.

I was trying to reproduce and fix this problem, but I could not reproduce this problem in my environment.
This problem may related to the DOM tree structure that was changed by Google's A/B test.
I need the new DOM tree structure information to fix it.
I can fix it if it reproduces in my environment...
Thanks reporting. and I'm sorry.
I'll continue trying to reproduce it.

Thanks for taking a look.
Yeah, it looks like it happens when the page has various "smart" results added in.

I was able to fix this issue while trying to fix a different problem.
Submitted a Pull request 😃

Thank you for the nice fixes!


Tradeoffs

@maxrjones sorry this took me so long by I have addressed your comment and made a few wording changes as well. Lmk if you think it's good to merge.

@maxrjones sorry this took me so long by I have addressed your comment and made a few wording changes as well. Lmk if you think it's good to merge.

Looks great, thank you!


az resource tag failing to update replicated storage accounts

Describe the bug
When trying to add tags to a storage account I get an error when the storage account is replicated to another region.
Command Name
az resource tag
Errors:
Code: AccountPropertyCannotBeUpdated
Message: The property 'secondaryLocation' was specified in the input, but it cannot be updated as it is read-only. For more information, see - https://aka.ms/storageaccountupdate

To Reproduce:
Steps to reproduce the behavior.

Create a replicated storage account, in my case it is a Geo-redundant storage (GRS)
az resource tag -g myGroup -n myStorageaccount --resource-type "Microsoft.Storage/storageAccounts" --tags Zoe=1

Expected Behavior
I expect the tags to be added to the storage account. I have no ability to provide (or omit) the secondaryLocation on the az resource tag command.
This works perfectly fine for most storage accounts, but fails on the one that is setup for replication.
Environment Summary
Windows-11-10.0.22631-SP0
Python 3.12.7
Installer: MSI

azure-cli 2.67.0

Extensions:
datafactory 1.0.2
datamigration 1.0.0b2
maintenance 1.6.0
monitor-control-service 1.2.0
scheduled-query 1.0.0b1

Dependencies:
msal 1.31.0
azure-mgmt-resource 23.1.1

Additional Context
Similar to an issue I have with tagging managed instances: https://github.com/Azure/azure-cli/issues/30095

Thank you for opening this issue, we will look into it.

Workaround appears to be to call the az storage account update command instead
az storage account update -g myGroup -n myStorageaccount --tags Zoe=1


Documentation cleanup

Mostly the documentation enhancement.
Properties which values are assigned in the primary constructor had the following documentation form:
/**
 * Documentation of class Foo
 */
class Foo(
    /**
     * Documentation of property bar
     * This style often breaks the code highlighting.
     */
    val bar: Int,
    /**
     * Documentation of property boo
     */
    val boo: Int
)

Now it looks like
/**
 * Documentation of class Foo
 *
 * @property bar Documentation of property bar
 * @property boo Documentation of property boo
 */
class Foo(
    val bar: Int,
    val boo: Int
)

Also, some code was using back-quoted names:
val `val`: Int

Back-quotes allow to use keywords as variable/property/class name, however, it's not recommended.

Thank you 👍🏼


Replaced iOS with mobile

As everyone can agree that this document is not just for iOS but in general for any mobile platform. I am removing the word iOS.

very true - thanks!


python debug console with huge unnecessary variable showed

VSCode Version:
版本: 1.45.1
認可: 5763d909d5f12fe19f215cbfdd29a91c0fa9208a
日期: 2020-05-14T08:33:47.663Z
Electron: 7.2.4
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Darwin x64 19.4.0
Extension: ms-python.python 2020.5.80290
OS Version:
OS: Darwin x64 19.4.0
macOS Catalina: 10.15.4

Steps to Reproduce:
The python debug console become very complicated with a lot of special variables. How can I hide the unnecessary variable like before?

Does this issue occur when all extensions are disabled?: Yes/No

\extPython


Steam error handling for GetEscrowDuration()

I noticed calling this function would often fail, due to people being unable to trade, and then Steam would return an error page causing parsing problems.
The updated code allows you to retrieve this error message with the property ex.Message (which will be empty if it's NOT a Steam error and is actually because Steam changed their javascript variables for escrow duration or something).
Example: http://i.imgur.com/zsofKyl.png
I also updated the function documentation to note which exceptions are thrown. If you are using these functions you should be using try-catch and handling these errors.

Well so far i tested it with the old one it still worked every single time for me ... anyway maybe a nice idea to add example try catch for new users here before they ask it a milion times again ?

You may want to update the code again, I changed line 1151 slightly here: https://github.com/Jessecar96/SteamBot/pull/882/files#diff-47987659d95c7e87e47f6e4c7fef3b5dR1151

I see thanks for the note.

@BlueRaja @Jessecar96 I've tested this for a while now and it's working fine. I'd say this is mergeable.

Strange, old and new version behave same, both throws exception when fail, new version add message (as part of exception).
It will not help people for better trading :-)

You're supposed to catch the exception.

Has anyone who is not the PR dev tested this?


Every n-th element in a sequence

Some of the built-in Racket sequences provide start, end and step args, like (in-vector) and (in-string) (link). There doesn't seem to be an out-of-the-box way to do this with data/collection sequences, or is there?
How about a new interface every that looks something like this:
(define (every cnt seq)
  (if (empty? seq)
      empty-stream
      (let ([head (first seq)]
            [tail (with-handlers
                    ([exn:fail?
                      (λ (exn)
                        empty-stream)])
                    (drop cnt seq))])
        (stream-cons head (every cnt tail)))))

which could be used like:
(take 10 (every 5 (naturals)))
;; evens
(take 10 (every 2 (naturals)))
;; odds
(take 10 (every 2 (naturals 1)))
;; with an offset
(take 10 (every 7 (drop 100 (naturals))))
;; together with subsequence for start, stop, step
(every 3 (subsequence (naturals) 10 20))
;; in vector
(every 3 #(1 2 3 4 5 6 7 8 9 10))

which yield the outputs (converted to list):
'(0 5 10 15 20 25 30 35 40 45)
>
'(0 2 4 6 8 10 12 14 16 18)
>
'(1 3 5 7 9 11 13 15 17 19)
>
'(100 107 114 121 128 135 142 149 156 163)
>
'(10 13 16 19)
>
'(1 4 7 10)

Do you think this would be a worthwhile addition to data/collection? If so, I could put something like it in a PR.

Hi @lexi-lambda !
Thanks for making this great library (which I know you've said you aren't actively maintaining). Since I first discovered it -- I assume about the time I opened this issue -- I took to implementing generic versions of common sequence operations whenever I had the need for one. Over time this grew into a sizable collection of general-purpose APIs. I've finally put this up, and I thought I'd mention it to you to see if you might have any thoughts on it:
https://docs.racket-lang.org/seq/index.html
The seq/base module essentially just extends data/collection, building on top of it to provide a number of generic and lazy sequence utilities that are implemented using streams. I also added two additional "layers" -- one called seq/api and the next one above that called seq/iso (for "isomorphic"). The idea is just to provide isomorphic (not in the usual algebraic sense but in the sense of "same form") versions of the APIs, so that the outputs have the same type as the inputs.
In order to do this, it was first necessary to know whether the output was finite to begin with, and that's where the seq/api layer comes in -- this layer simply annotates outputs from the core (seq/base) APIs with finiteness information, for cases where, given the nature of what we know the specific API does, the finiteness of the output is implied by the finiteness of the input. E.g.: (by 3 (range 10)) would be known-finite if (range 10) is known-finite.
Then, at the 'iso' level, if the output is finite, it is coerced back into the input type to provide the isomorphic behavior.
Now, do I like this approach? No, I'm not sure that I do. It feels overly complicated and has involved more manual work to accomplish than I'd like to admit, and I suspect it probably loses some of the benefits that you pointed out in your original talk about this library. It's very likely that having isomorphic behavior means that we lose the performance benefits of using different data structures to begin with (due at least to conversion to streams and back again), and I haven't really given a lot of thought to this. The main (and perhaps only) benefit that the isomorphic interfaces provide is convenience. At worst, these interfaces would be useful in testing scenarios or prototyping, where the behavior of the interfaces is more important than their performance. But another thing to be said for it is perhaps that it reifies a particular design goal -- for generic APIs to also be isomorphic in this manner.
Anyway, just thought I'd run all this by you. Is this all just a senseless waste of time? Is there a much better way? Could we perhaps have a DSL, maybe something like Common Lisp's iterate, that could be both generic as well as isomorphic and even performant?
Btw I think there are discussions going on around generic collections for Rhombus, but I haven't had a chance to keep track of them. It's possible these topics are already being discussed somewhere.

@lexi-lambda FYI Introducing Seq: A generic, isomorphic sequence library. Some of the code in seq probably belongs in data/collection (e.g. the changes in set-nth and range and the other overridden interfaces, mostly just finiteness annotation), and I'd love to merge those upstream in some form if you feel that makes sense.


support PostgreSQL data source

spark_v3.0

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

Thanks so much @ianhhhhhhhhe for the amazing work ;-)

One test failed.
Results :

Failed tests:   configsSuite(scala.com.vesoft.nebula.exchange.config.ConfigsSuite): assertion failed

Tests run: 25, Failures: 1, Errors: 0, Skipped: 0

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary for nebula-exchange 3.0-SNAPSHOT:
[INFO]
[INFO] nebula-exchange .................................... SUCCESS [  0.076 s]
[INFO] exchange-common .................................... FAILURE [05:31 min]
[INFO] nebula-exchange_spark_2.2 .......................... SKIPPED
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  05:31 min
[INFO] Finished at: 2022-01-24T03:31:48Z
[INFO] ------------------------------------------------------------------------
Error:  Failed to execute goal org.apache.maven.plugins:maven-surefire-plugin:2.12.4:test (default-test) on project exchange-common: There are test failures.
Error:
Error:  Please refer to /home/runner/work/nebula-exchange/nebula-exchange/exchange-common/target/surefire-reports for the individual test results.
Error:  -> [Help 1]
Error:
Error:  To see the full stack trace of the errors, re-run Maven with the -e switch.
Error:  Re-run Maven using the -X switch to enable full debug logging.
Error:
Error:  For more information about the errors and possible solutions, please read the following articles:
Error:  [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
Error:
Error:  After correcting the problems, you can resume the build with the command
Error:    mvn <args> -rf :exchange-common
Error: Process completed with exit code 1.

Codecov Report

Merging #62 (244d854) into master (1fc042b) will increase coverage by 0.62%.
The diff coverage is 84.00%.

@@             Coverage Diff              @@
##             master      #62      +/-   ##
============================================
+ Coverage     54.51%   55.14%   +0.62%
  Complexity       76       76
============================================
  Files            17       17
  Lines          1317     1342      +25
  Branches        250      254       +4
============================================
+ Hits            718      740      +22
+ Misses          474      473       -1
- Partials        125      129       +4

Impacted Files
Coverage Δ

...la/com/vesoft/exchange/common/config/Configs.scala
66.66% <81.81%> (+0.51%)
:arrow_up:

.../vesoft/exchange/common/config/SourceConfigs.scala
62.79% <85.71%> (+2.79%)
:arrow_up:

Continue to review full report at Codecov.

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

can you also add postgresql datasource for spark2.2 and spark2.4 ?

add postgresql dependency to avoid exception:
Exception in thread "main" java.sql.SQLException: No suitable driver
	at java.sql.DriverManager.getDriver(DriverManager.java:315)

available postgresql versions are: 9.4.1207 - 9.4.1212, for example:
<dependency>
            <groupId>org.postgresql</groupId>
            <artifactId>postgresql</artifactId>
            <version>9.4.1212</version>
</dependency>

can you also add postgresql datasource for spark2.2 and spark2.4 ?

Sure ^ - ^


[Snyk] Upgrade io.grpc:grpc-stub from 1.59.0 to 1.59.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade io.grpc:grpc-stub from 1.59.0 to 1.59.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 24 days ago, on 2023-11-28.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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


What's the definition of misa.B regarding to zb* extensions

For RISC-V Privileged Spec v. 20211203, the misa.B is labeled: Tentatively reserved for Bit-Manipulation extension.
For RISC-V Bit-Manipulation ISA-extensions Spec v1.0.0, defined four extensions (zba, zbb, zbc, and zbs).
For these specs, we can't find any description between misa.B and zb* extensions.
We are confused about the setting condition of misa.B, for example:

Set misa.B when all zb* extensions are implemented.
Set misa.B when the basic bit-manipulation extension (zbb) is implemented.
misa.B shouldn't be set even if all zb* extensions are implemented.

See https://github.com/riscv/riscv-isa-manual/commit/c1c77c4902d5e7c58613d725d09d66a2a743da1c
There is no relationship between misa.B and Zb*.  Currently, there is no way to control the presence of the Zb* extensions.

Thanks for your prompt reply


Module update - Dapr ContainerApp Environment - Work with existing

Description
Adding capability to support existing ACA Environments.
This enables 2 scenarios

Adding Dapr components to an existing Environment
Adding multiple Dapr components to a new Environment

If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Updating an existing module

[x] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.
[x] I have read the Updating an existing module section in the contributing guide and updated the version.json file properly:

[x] The PR contains backwards compatible feature updates, and I have bumped the MINOR version in version.json.

Oh @shenglol - i've broken the CI again 😆

cannot have more than Castle.Proxies.SubscriptionQuotaRuleProxy environments in West US. (Code:MaxNumberOfRegionalEnvironmentsInSubExceeded)

I did have one set of checks pass, but then pushed another benign commit and then got this error.
There's obviously a capacity limit on ACA Environments, and the number of different tests that i'm doing is at the limit of what your Azure subscription can get.

@shenglol  - doesn't look like the Refresh Module Table run worked properly either

Oops...queued the job two quickly before the module was pushed to MCR. Just ran the job again and it refreshed the table.


Handle nuget.org API change

Summary of the changes

Looks like Nuget.org started enforcing that the url to download a package be lower-case.
This exposed that we assume the request succeeded without checking to at least log the failure when we attempted to open a .nupkg which was actually a 0-length file.

Fixes: https://github.com/dotnet/razor-tooling/issues/5702

@ryanbrandenburg merging this for you since it's the dopeness to fix everything in the world 😄


Event onCloseComplete don't execute on use close() in onConfirm event

Hi guys!
When I use close function in onConfirm event, the onCloseComplete don't execute...
onCloseComplete = () => {     console.log('onCloseComplete');   };
onConfirm = close => {     close();   };
<Dialog isShown={true} onCloseComplete={onCloseComplete} onConfirm={onConfirm}><Text>Test</Text></Dialog>
If you see in console, the 'onCloseComplete' don't execute when confirm button is clicked
Is this correct and I use the events in the wrong way?

Can you provide more information about the version of evergreen-ui and a link to a minimal representation of the issue? Otherwise we can't really help you.
Based on what you've described this works as it should: https://codesandbox.io/s/vibrant-leakey-56efk


make a master metadata dict/map

should encompass everything to be passed to resourcespace
could be used to init file md dicts w blank values

Done


Device has reported a sensor fault. Reason: ['wind failed']

Hi !
Thanks for the great work on this docker !
I have the following error coming in the log: DEBUG:weatherflow2mqtt.weatherflow_mqtt:Device XXX has reported a sensor fault. Reason: ['wind failed']
I'm not sure what it means as the wind sensor is correctly reporting the information.
Kind regards,

You are welcome.
The above log entry is produced by the Device itself, and send to the Hub, that then sends it to WeatherFlow. If you only get this a few times, don't worry, it might just be a glitch and as you state, the sensor is still reporting correctly.
So unless you start to see a lot of errors like that, just ignore it.

Unfortunately, I get the error almost every minute 😕

So basically, every time your Tempest device sends data. As I said, it is not something i can do anything with, as I just report the data. I suggest you contact WeatherFlow support and ask them.

I'll contact them, thanks 😊

Something I've noticed though, is that, in the application, I see the Sensor Status is LIGHTNING_DISTURBER 🤔

LIGHTNING_DISTURBER can occur if the sensor detects some thing that is NOT lightning, but still sends out an electrical signal. If you have your device close to a Power Cable, or a Microwave oven that could trigger this.

The question I have is why the status reported by weatherflow2mqtt is different from the one reported by the official app 😅

I never see these messages in the official APP also. I think they are just reported back to WeatherFlow, that then uses it for optimizing their algorithms. I could make a setting that removes them from the log?

I'm not sure but I think that could be useful to get information about sensors status.

It was just if it is flooding the Log File, then it could be removed. As I said, this is not really related to the functionality of this program, I just decided to display the info, now that I saw the data coming in, but the entries are not used by the program itself, and as long as the device is reporting data, this can safely be ignored.

I think indeed that the info could be removed from the docker log.
However, my point was that your mapping for the error bits seems to be incorrect. I've made a quick python script based on the UDP API to illustrate:
My device status is "131076". Using your implementation:
DEVICE_STATUS = [
    "lightning failed",
    "lightning noise",
    "lightning disturber",
    "pressure failed",
    "temperature failed",
    "rh failed",
    "wind failed",
    "precip failed",
    "light/uv failed",
]

def device_status(value):
    """Return device status as string."""
    if value is None:
        return

    binvalue = str(bin(value))
    binarr = binvalue[::-1]
    binarr = binarr[:len(DEVICE_STATUS)]
    return_value = []
    for x in range(len(DEVICE_STATUS)):
        if binarr[len(binarr) - 1 - x] == "1":
            return_value.append(DEVICE_STATUS[x])

    return return_value

print(device_status(131076))

This gives ['wind failed'].
However, based on the API here Status (device) [type = device_status], I've made the following:
masks = {
        0b000000001: 'Lightning',
        0b000000010: 'LightningNoise',
        0b000000100: 'LightningDisturber',
        0b000001000: 'Pressure',
        0b000010000: 'Temperature',
        0b000100000: 'Humidity',
        0b001000000: 'Wind',
        0b010000000: 'Precipitation',
        0b100000000: 'Light/UV',
        0x00008000: 'power booster depleted',
        0x00010000: 'power booster shore power'
}

def device_status_new(value):
    failed = []

    for mask in masks:
        if mask & value:
            failed.append(masks[mask])

    return failed

print(device_status_new(131076))

This gives ['LightningDisturber'] which is correct and the same as the official tempest iOS app.
Hope it clarify a bit 🙂

GOT IT, thanks. I will fix this based on your program above. Thanks

Great thanks !
Would it be possible to add two new sensors for the hub itself and for the device itself with information (attributes) such as sensor errors, firmware version, etc. ?

I have edited it now, and will build new containers. They will be uploaded to the new dev tag, which I use before releasing new versions. This tag will always be the latest version, but might have bugs in it. So if you could test it, I would appreciate it.

Would it be possible to add two new sensors for the hub itself and for the device itself with information (attributes) such as sensor errors, firmware version, etc. ?

Yes, but please create a new Issue for this, specifying what you would like to see in these sensors.

dev tag now has this implemented.

Perfect, will test it, thanks !
I've made another request for the sensors 😊

dev tag now has this implemented.

I've tested it and it's working fine for the error now:
DEBUG:weatherflow2mqtt.weatherflow_mqtt:Device XXX has reported a sensor fault. Reason: ['LightningDisturber']
Thanks !

Just a quick one, I have the docker variable DEBUG to False, so this kind of information should not be displayed in the log.
DEBUG:weatherflow2mqtt.weatherflow_mqtt


some problem about data preprocess

there is a problem about the panda usage:
AttributeError: 'DataFrame' object has no attribute 'review'
at line 46, 55:
46: data_train = pd.read_csv('labeledTrainData.tsv', sep='\t')
55 :for idx in range(data_train.review.shape[0]):
variable "data_train" does not got attribute "review"
===========
I have looked for different of pandas and the problem is still not fixed
Any suggestion about this problem?
THANKS!!!

same problem.

I fixed it.
wget just download a .html file, not real .tsv file.
see:
https://raw.githubusercontent.com/MatthieuBizien/Bag-popcorn/master/labeledTrainData.tsv

Thanks!

I fixed it.
wget just download a .html file, not real .tsv file.
see:
https://raw.githubusercontent.com/MatthieuBizien/Bag-popcorn/master/labeledTrainData.tsv

Same problem. So, how can I fix it.


require.context is not a function

In my Storybook config, I use the following code to gather all the stories:
const req = require.context('../../src/public', true, /.story.js$/)

function loadStories() {
  req.keys().forEach((filename) => req(filename))
}

Which is documented here: https://getstorybook.io/docs/basics/writing-stories
However, when I run storyshots, I get an error saying that require.context is not a function.

Yes. That's true.
We need to add support for that.
@roonyh check we can use something like this: https://www.npmjs.com/package/enhanced-require inside node.

I'm using the config directly from the documentation and getting this error when running storyshots directly without any modifications:
Documentation
https://getstorybook.io/docs/basics/writing-stories)

.storybook/config.js

story placement next to components

npm script for running storyshots

error when run

@trevordmiller Yep. That's an issue becuse now we are running stories in NodeJS. It don't have require.context.
We have a fix for that and landing to storyshots soon: https://github.com/kadirahq/storyshots/pull/30

@ronderksen @trevordmiller, Released version 1.1.2 with #30. It should be able to support usingrequire.context inside storybook config module.

Guys, we had a regression for this issue and we revert v1.1.2.
We were looking at some alternate solution and it'll work great.

still have this issue.
Maybe there is a way to exclude some files from storyshorts?

This is fixed in #67


Keyboard not fully displaying on Search Screen

Currently, the keyboard on iPhone 8 is not fully showing on the Search Results Screen.
Motivation
As a user, I would like to see the full keyboard to type.
Finish Line
User can be assured that the keyboard will fully appear when typing in a text field.

May be due to keyboard accessory bar, not the actual keyboard.

tool bar removed entirely. That solves the issue


Vulnerability identified in patched library

DependencyCheck correctly identifies commons-fileupload-1.3.3.jar as cpe:/a:apache:commons_fileupload:1.3.3, but then reports that it is vulnerable to CVE-2016-1000031. commons-fileupload 1.3.3 was specifically released to fix that CVE (versions prior to 1.3.3 are affected, 1.3.3 and later are not), so I believe the version-matching is off somewhere in the process.

It appears the MITRE made a change on July 17 to include "before 1.3.3" in the description but did not update the CPE to reflect this change. Dependency-Check uses the CPE information to determine what versions are affected.

The issue is that dependency-check really relies on the data sources (in this case the NVD) to provide accurate information - especially regarding the vulnerable Vulnerable Software and Configurations section. As @stevespringett pointed out the the NVD has not been updated with the more specific information and as such, ODC will continue to flag this vulnerability.  The best that can be done is contacting Mitre and/or NVD and requesting the record be updated.

@ChristopherSchultz @stevespringett Have a look at issue https://github.com/jeremylong/DependencyCheck/issues/910 in which I contacted the NVD to request a DB update for CVE-2016-1000031.
The update was performed on friday 2017/10/13 with the correct information. You should no longer see any false report and can probably close this issue.

Thanks


Missing nitro homepage in dms

Is your feature request related to a problem? Please describe.
The nitro page in the dm section is missing.
Describe the solution you'd like
I would love to be able to view the nitro page in my user settings with your theme enabled.
Additional context
Here is a screenshot of what nitro page I'm talking about.

Is this not the same page shown in the user settings window?

Is this not the same page shown in the user settings window?

no it isn't. the xml name for the page I'm talking about is <a class="link__3a950" data-list-item-id="private-channels-uid_1484___shop"


Take screenshot in case of e2e test failure

I agree that this is a useful feature, and you've implemented it in a very succinct way. It is outside the scope of the quickstart though, and that's the only reason for which I will close the PR and not accept it.

@filipesilva I respect your choice to reject the PR. It's just that I used the quickstart as a basis for a new project and when I got to protractor tests which failed without providing a lot of insight it took me some time to find how to take screenshots, so I thought this could save time for others.

In https://github.com/angular/protractor/blob/master/docs/debugging.md there's a couple more good techniques to debug failing tests.
I personally use a lot of browser.pause() and protractor --elementExplorer to figure out what's happening.


HoeffdingTree predict_proba returns different dimensions per sample

Expected behaviour
output_dim = num_samples x num_classes
Actual behaviour
output is an array of lists with different length
Steps to reproduce the behaviour
test files are attached
data.zip
import numpy as np
from skmultiflow.trees.hoeffding_tree import HoeffdingTree

data = np.load("test_data.npy")
labels = np.load("test_labels.npy")

model = HoeffdingTree()
model.partial_fit(data, labels)
pred_proba = model.predict_proba(data)

>> [list([1.0]), ... , list([0, 1.0]), ... , list([1.0]) , ...]

predict_proba() return the probabilities mapped into an array where the index corresponds to the class.
This is implemented in
https://github.com/scikit-multiflow/scikit-multiflow/blob/3bac7c8274948b9c7f4518994a85f4bb5a5b5e0e/src/skmultiflow/trees/hoeffding_tree.py#L1083
The size of the array depends on the max class value. This is useful when not all classes are known at the beginning of the stream.
What about using the classes attribute (if it is defined)?
if self.classes is not None:
    y_proba = np.zeros(max(self.classes) + 1)
else:
    y_proba = [0] * (int(max(votes.keys())) + 1)

Yes that would fix it. Currently, self.classes is not defined,
https://github.com/scikit-multiflow/scikit-multiflow/blob/3bac7c8274948b9c7f4518994a85f4bb5a5b5e0e/src/skmultiflow/trees/hoeffding_tree.py#L936
https://github.com/scikit-multiflow/scikit-multiflow/blob/3bac7c8274948b9c7f4518994a85f4bb5a5b5e0e/src/skmultiflow/trees/hoeffding_tree.py#L960
but it could be set in the first call of partial_fit
However, I would prefer
if self.classes is not None:
    y_proba = np.zeros(len(self.classes))
else:
    y_proba = np.zeros(len(votes.keys()))

since a 3-class classification problem could have for example labels in [2, 6, 8].
What do you think @jacobmontiel

Yes, we should set self.classes on the first call.
Now, we need max(self.classes) + 1 to map correctly into the 0-based array given that the index implies the class. This is used in the predict() method:
https://github.com/scikit-multiflow/scikit-multiflow/blob/3bac7c8274948b9c7f4518994a85f4bb5a5b5e0e/src/skmultiflow/trees/hoeffding_tree.py#L1055

Ah, I see.
Than something like
if self.classes is not None:
     prediction = self.classes[index]

would be possible.
Alternative, a hint in the documentation mentioning that classes have to be zero-based would also be fine, as the user can apply e.g. sklearn.preprocessing.LabelEncoder() if needed.

The nice thing about the max trick is that the user does not need to transform the labels. For example, if classes are [2, 6, 8] we map to an array of size 9. But in most cases, this is not necessary given that classes usually are defined zero-based. Nonetheless, it is true that we need to document that zero-based classes are preferred. The worst case scenario will be a very sparse label set.
I have pushed a fix: 4aec5ffb973aebd6765a8af0e49101f4d22718bf
From your example:
classes = np.unique(labels)
model = HoeffdingTree()
model.partial_fit(data, labels, classes=classes)
pred_proba = model.predict_proba(data)

>>> array([[1., 0.],
       [1., 0.],
       [1., 0.],
       ...,
       [0., 1.],
       [1., 0.],
       [1., 0.]])

Perfect. Thank you very much


Switch 'update available' action from 3rd party code to gh tool

DRAFT for testing only.

@shayancanonical what is the best way to test this without merging?

Abandoned as no longer topical. We are using shared workflow now.


try for nicer outside-legend kwarg

see https://github.com/IAMconsortium/pyam/pull/204/, this extension works and keeps backwards compatibility (sorry for the merge commit, though).
You decide if you want to have these, then just rebase your branch and cherry-pick my commits

supposing this can be closed..


Custom Field for Receipts

Who is your user?
Contributor
What are they trying to achieve?
They would like a custom field on their reciepts so they can input a business number or additional address.
When contributing to a collective, I want to have a particular field included in the reciept, so I can have all the information I need on my payment receipt
How are they currently doing this?
If a guest contributor is needing extra information on their receipt. A refund doesn't need to be issused. The guest just needs to log in to activate their account, they can then edit their information in their account settings. They then have the ability to download a new receipt from the transaction on Open Collective.
This doesn't solve the issue of needing to add another custom field.
How well understood is this problem?

[ ] Surfacing: if you feel this is a seed of an idea that needs to be researched.
[ ] Understanding: if you've done some research and feel it is already well understood and ready for shaping.
[ ] Shaping:  if you feel a solution is clearly defined and can be made into a pitch (read more about shaping)
[ ] Pitching: if you feel that it is ready to be presented for prioritisation (read more about pitching)

My story on this:
I am a collective owner. I have several people that wants to contribute but says they will need their company registration number on their receipt. Currently they can only add address and VAT number, which is insufficient for some reason.


Automate listview in selendroid

I have uploaded a sample application in which i have multiple BOOK buttons .What i want to do is i want to automate BOOK button .This is basically a listview which have multiple BOOK buttons.
Please guide me the way to automate this.
I tried the same and following is my piece of code.
List wbMovies=driver.findElements(By.id(""android:id/search_book_btn"));
for(int i=0;i<wbMovies.size();i++){
wbMovies.get(i).click();
Thread.sleep(5000);
new Actions(driver).sendKeys(SelendroidKeys.BACK).perform();
	}

Please help me to fix this issue

I'm closing this issue due to inactivity.
If is a question rather than an issue, please send questions to the selendroid user group.
If you think it's an actual problem with selendroid and still experience the issue described on the latest release of selendroid feel free to file a new issue. If you do please provide a concise reproducible test case and describe what results you are seeing and what results you expect.


Bit of docs, but more importantly some api fixes

The containsPoint method exists so that if people want to build their own interaction manager they don't need to touch some pixi stuff such as worldTransform.applyInverse, etc...

You're awesome!


Game operation and maintenance API

kruise-game contains two CRDs GameServer and GameServerSet. Game servers can be managed by deploying or changing the corresponding CR. However, in actual production use, a release or operation and maintenance action is often a combination of a series of operations on CR. For example, set the GameServer image tag with ids 1, 7, and 10 to v0.3; adjust the update priority of GameServer with ids 5, 9, and 11 before updating the game server, etc.
Therefore, a set of APIs with operation and maintenance semantics is needed, which users can directly use or integrate into their own operation and maintenance platform to facilitate operation and maintenance operations.

Heyy @chrisliu1995  i am kairvee and would like to take up this issue under LFX mentorship programme as it aligns with my interests and skills , please inform me over the further things ,
thanks.

Hello @chrisliu1995
I'm excited to work on the Kruise-game project, focusing on GameServer and GameServerSet CRDs. I'm currently learning and developing relevant skills to contribute to designing and implementing APIs with operation and maintenance semantics. I understand the need to simplify complex production operations and am eager to help. With my passion for this project and commitment to learning, I'm confident in my ability to make a meaningful contribution. I look forward to discussing this opportunity further.

kruise-game contains two CRDs GameServer and GameServerSet. Game servers can be managed by deploying or changing the corresponding CR. However, in actual production use, a release or operation and maintenance action is often a combination of a series of operations on CR. For example, set the GameServer image tag with ids 1, 7, and 10 to v0.3; adjust the update priority of GameServer with ids 5, 9, and 11 before updating the game server, etc.
Therefore, a set of APIs with operation and maintenance semantics is needed, which users can directly use or integrate into their own operation and maintenance platform to facilitate operation and maintenance operations.

@chrisliu1995  can you please update me over any pretests/ issues to work on before applying?

Hey everyone!
I'm Nihit, a pre-final year student from the Indian Institute of Information Technology, Gwalior, majoring in Information Technology. I'm a tech enthusiast with a strong passion for open-source. I've applied for this project and have been working on this issue for some time now. I would greatly appreciate any guidance or clarity on how best to approach this issue. I'm really looking forward to solving this problem and contributing to the project as much as possible.

Based on the above requirements, I have the following initial thoughts:
Features

A library provides filter and updater for OKG GS. It can be used by Golang packages, command line tools, RESTful API and so on.For example:

It can be the container image to filter GS list or patch spec.opsState/annotations/labels of filtered GS list for Argo workflow steps.
It can be the container image to choose a GS for match maker or gateway pod.
It can be the GS sidecar container image to patch its owner GS.
It can be used to achieve A/B test by filter and updater.

Architecture

Query Layer

Query/Update GS list filtered by parsed filter provided by Filter Layer.

Filter Layer
Strategy

We need the Filter Layer since the same functionality provided in mutiple facades needs to be abstracted.
Logic syntax is needed since the filter condition may be a combination of multiple conditions.
Regular expressions need to be supported for flexible filtering.

Filter syntax

Learn from MongoDB query syntax to provide a simplified Filter syntax.
There are predefined operators starting with $ (such as comparison operators, logical operators, function operators) and matching strings that do not start with $.
To standardize semantics, an object can only contain one key-value pair, for example:

// legal:
{
    "$and": [
        {
            "metadata.annotations.version": "java21"
        },
        {
            "spec.opsState": "Maintaining"
        }
    ]
}
// Illegal:
{
    "$and": [
        {
            "metadata.annotations.version": "java21",
            "spec.opsState": "Maintaining"
        }
    ]
}

In the following content, the value represents a number or string, and the object and array are JSON objects and arrays.

Comparison Operators

$gt/$lt/$ne/$eq/$in：greater than/less than/not equal/equal/included，where $in acts on an array of values (such as strings, numbers), and the rest act on a value.

Logical Operators

$and/$or/$not: $and/$or act on an array of values, $not acts on an object.

Examples
This is a simplified GS example:
apiVersion: game.kruise.io/v1alpha1
kind: GameServer
metadata:
  annotations:
    server: minecraft-pvp-0
    version: java21
  generation: 1
  labels:
    app.kubernetes.io/name: minecraft-pvp
    game.kruise.io/owner-gss: minecraft-pvp
  name: minecraft-pvp-0
  namespace: minecraft
  ownerReferences:
  - apiVersion: v1
    blockOwnerDeletion: true
    controller: true
    kind: Pod
    name: minecraft-pvp-0
    uid: c1610944-1fb1-44c7-8ec0-9d24b4aae09c
spec:
  deletionPriority: 0
  opsState: None
  updatePriority: 0
status:
  currentState: Ready
  deletionPriority: "0"
  desiredState: Ready
  lastTransitionTime: "2024-09-09T04:33:18Z"
  networkStatus:
    createTime: "2024-09-02T06:58:59Z"
    currentNetworkState: Ready
    desiredNetworkState: Ready
    externalAddresses:
    - endPoint: nlb-xxx.cn-hangzhou.nlb.aliyuncs.com
      ip: ""
      ports:
      - name: "3000"
        port: 1273
        protocol: UDP
    internalAddresses:
    - ip: <IP_ADDRESS>
      ports:
      - name: "3000"
        port: 3000
        protocol: UDP
    networkType: AlibabaCloud-NLB
  podStatus:
    containerStatuses:
    - image: itzg/minecraft-server:java21
      name: minecraft
      ready: true
      restartCount: 0
      started: true
      state:
        running:
          startedAt: "2024-09-02T06:58:58Z"
  updatePriority: "0"

Query GS list with "version": "java21" annatation and opsState not None:
{
    "$and": [
        {
            "metadata.annotations.version": "java21"
        },
        {
            "spec.opsState": {
                "$ne": "None"
            }
        }
    ]
}

Query GS list with opsState Maintaining or Draining:
{
    "$or": [
        {
            "spec.opsState": "Maintaining"
        },
        {
            "spec.opsState": "Draining"
        }
    ]
}

Adaptor Layer
Golang Package
Functions
func NewFilter() *Filter
func NewUpdater() *Updater
func GetGsList(filter *FilterDocument)
func UpdateGsList(filter *FilterDocument, updater *UpdaterDocument)

Class
Filter：
type Filter struct {
}

func (f *Filter) Build() *FilterDocument
func (f *Filter) BuildFromString(filterString string) *FilterDocument

Updater：
type Updater struct {
}

func (u *Updater) Build() *UpdaterDocument
func (u *Updater) BuildFromString(updaterString string) *UpdaterDocument

RESTful API

Query GS list with opsState Maintaining or Draining:

GET http://localhost/gs?filter={"$or":[{"spec.opsState":"Maintaining"},{"spec.opsState":"Draining"}]}
After URL escape:
GET http://localhost/gs?filter={"%24or"%3A[{"spec.opsState"%3A"Maintaining"}%2C{"spec.opsState"%3A"Draining"}]}
200 response：
[
    {
        "name": "minecraft-pvp-0",
        "namespace": "minecraft",
        ...
    },
    {
        "name": "minecraft-pvp-1",
        "namespace": "minecraft",
        ...
    }
]

4XX/5XX error response：
{
    "code": 1,
    "message": "internal error"
}

Update GS with opsState Maintaining or Draining to None:

POST http://localhost/gs
body：
{
    "query": {
        "$or": [
            {
                "spec.opsState": "Maintaining"
            },
            {
                "spec.opsState": "Draining"
            }
        ]
    },
    "update": {
        "spec.opsState": "None"
    }
}

200 response（successfully updated GS list）：
[
    {
        "name": "minecraft-pvp-0",
        "namespace": "minecraft",
        ...
    },
    {
        "name": "minecraft-pvp-1",
        "namespace": "minecraft",
        ...
    }
]

4XX/5XX response（there is more than one GS that failed to update）：
{
    "updated": [
        {
            "name": "minecraft-pvp-0",
            "namespace": "minecraft",
            ...
        }
    ],
    "failed": [
        {
            "error": {
                "code": 1,
                "message": "internal error"
            },
            "gs": {
                "name": "minecraft-pvp-1",
                "namespace": "minecraft",
                ...
            }
        }
    ]
}

Since writing filter and URL escaping are complicated, for developers who do not understand filter syntax, RESTful API client SDK is provided and called in the following chain method.

func GetActiveOpsGsList() {
    // Create HttpClient and configure connection pool
    sdkClient := sdk_client.Connect("http://localhost")

    // Do HTTP request
    gsList, err := sdkClient.GetGsList("namespace_name", NewFilter().Or().Kv("spec.opsState", "Maintaining").Kv("spec.opsState", "Draining").Build())
    // Or use the encapsulated simpler way
    gsList, err = sdkClient.GetGsList("namespace_name", NewFilter().Or().OpsState("Maintaining").OpsState("Draining").Build())

    gsList, err = sdkClient.GetGsList("namespace_name", NewFilter().And().Kv("metadata.annotations.version", "java21").Ko("spec.opsState", Not().StrEq("None")).Build())
    // Or use the encapsulated simpler way
    gsList, err = sdkClient.GetGsList("namespace_name", NewFilter().And().Annotation("version", "java21").OpsStateNot("None").Build())

    // patch GS
    gsList, err = sdkClient.UpdateGsList("namespace_name", NewFilter().Or().OpsState("Maintaining").OpsState("Draining").Build(), NewUpdater().OpsState("None").Build())
}

Command Line Tool
# Query the GS whose GSS name is minecraft-pvp and opsState is None
gsops -namespace minecraft -filter -owner_gss minecraft-pvp -ops_state None
# Another form: gsops --namespace=minecraft --filter --owner_gss=minecraft-pvp --ops_state=None
# Update GS with opsState Maintaining or Draining to None
gsops -namespace minecraft -filter -or -ops_state Maintaining -ops_state Draining -updater -ops_state None


Feature/mm admin page

Got some merge conflicts. Otherwise looks good

Just noticed that and fixed them. Thanks


upstream: update Envoy upstream sha

This updates upstream Envoy with various fixes including:

TLS context crash fix: https://github.com/envoyproxy/envoy/issues/10030
Fixes for 32 bit arches: https://github.com/envoyproxy/envoy/pull/11726
Android symbol fixes: https://github.com/envoyproxy/envoy/pull/12011
Support for zlib stats in the decompressor: https://github.com/envoyproxy/envoy/pull/11782

Signed-off-by: Michael Rebello<EMAIL_ADDRESS>Co-authored-by: Mike Schore<EMAIL_ADDRESS>

Added @goaway as a co-author of this PR since this will supersede https://github.com/lyft/envoy-mobile/pull/923

Blocked on https://github.com/envoyproxy/envoy/pull/12081

https://github.com/lyft/envoy-mobile/pull/923


Need Support for 10.0.20246.1

termsrv.zip

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/88543cace358ebdb0030c0b5a362da600a9a4c02#diff-9e5bbfbeae79b59b305dda13323ba2b041f30f4887a84120392c479e1f7bba33


Why Move from ETL to ELT for your Fintech Products?

Proposed title of article
Why Move from ETL to ELT for your Fintech Products?
Proposed article introduction
Many fintech companies now find themselves collecting more data than ever before. But this data won't be of much use if it is just sitting in an Excel spreadsheet somewhere. So these firms need a fast, effective way of importing this into their analytics solutions. And for many, this is where ETL technology comes in.
Standing for Extract, Transform and Load, these solutions take raw data from whatever initial form it arrives in, cleans and transforms it into a more standardized format that is easier for data scientists to work with, and add it to a data warehouse where it can be processed. This has become an everyday activity for many firms, but it's not the only way of getting information from the source and into a data warehouse. Increasingly, there's another similar, but not identical, alternative to consider in the form of ELT solutions.
Key takeaways

ETL vs ELT.
Should you use ETL or ELT for fintech data?
ETL and ELT security.
ETL and ELT tools.

Article quality
This article is not just another general comparison of ETL and ELT. It is unique in that it compares ELT and ETL in terms of their application in extracting fintech data.
References
N/A

Hi, @jesmweni. Thanks for submitting your article. Can you tell me how it differ from existing articles on this subject: such as this one?

Hi @louisefindlay23. Existing articles like this one are light in content since they do not cover beyond the pros and cons of these data extraction methods. My article narrows down to how ETL and ELT compare when it comes to extracting data for fintech companies. And also goes ahead to describe ETL and ELT tools that can be used to extract fintech data.


make test fails on server but not on browser

It says
 1) Riotjs tests Tmpl compiles specs:
     Error: expected 'about:blank' to equal 'html'
      at Assertion.assert (/Users/tipiirai/git/muut/riotjs/node_modules/expect.js/index.js:96:13)

Is this related to 0234d64? (see comments)
Was fixed in 25e082a

This got fixed


Implements feature-region precision experiment

Hoi, at some point we should clean up this PR and merge the parts that should/need to be kept in the codebase.


Make NavMenu update better

Right now we force reload it via NavigationManager.NavigateTo(uri, true); which works but makes website slow

https://stackoverflow.com/questions/60912726/exchange-data-between-page-and-navmenu-in-blazor
Maybe something like this


fix migration

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

7 of 39 (17.95%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 19.72%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/app/repository/psql/db/db.go
0
5
0.0%

internal/app/repository/psql/db/sqlc_gen/query.sql.go
0
10
0.0%

internal/app/repository/psql/db/helper.go
0
17
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
450

Relevant Lines:
2282

💛  - Coveralls


Import ISV License failed on-premises

Importing our ISV license file failed on the on-premise UAT server.
Attached the error log details below.
C:\LS\UAT\Import-LicensePackage.ps1 : System.Management.Automation.RemoteException: 12/09/2019 01:38:28:
Microsoft.Dynamics.ApplicationPlatform.Environment.Settings.ConfigurationKeyNotFoundException: Failed to obtain app setting from for:
Common.IsOneboxEnvironment
At line:1 char:1

.\Import-LicensePackage.ps1

  + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException

  + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Import-LicensePackage.ps1

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

ID: 367369e4-2f85-8a54-a07d-051377559c27
Version Independent ID: dd172744-8482-e740-8376-bbf9be12167e
Content: Independent software vendor (ISV) licensing (on-premises) - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-ops-core/dev-itpro/dev-tools/isv-licensing-on-prem.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @manalidongre
Microsoft Alias: manado

Thanks for contacting us. Unfortunately, we do not handle Support requests here. Please contact Support using one of the options described here: Support contact options. In particular, you might want to go to the Dynamics 365 community page to get help with your questions from the Dynamics community.

We were seeing the same error with Common.IsOneboxEnvironment when running the script from powershell directly. Ran the script from powershell ISE and it worked. It would be helpful if Microsoft could update their documentation as to what specific requirements are necessary for this script to run. Also the article mentions downtime, it would be helpful to have some information on what is happening in this import script so we don't have to dig through the .ps1 to figure it out. Lastly I'm sure people would appreciate showing an example section showing the syntax with parameters.


Fix cross-compiling libass for Windows 64 bit

This PR fixes the cross-compilation of libass for Windows 64 bit.
Haven't tested if these changes affect the Windows building though.
Thanks for your review!

Closed because of a lack from my part


Allow specifying whether or not to bring in external repository.

For users who maintain internal repositories or caches, it should be possible to have the telegraf cookbook not bring in the InfluxDB repository, and instead manage package sources outside the scope of the cookbook.

Thanks for the issue @don-code.  This seems like a reasonable thing, we'll try to get it in the next minor release.

Looks good to me. This doesn't change the default behavior and test-kitchen converges OK. :+1:

This has been released in release 0.2.2


Implemented Store change and Logout functionality (#1x0dudb)

This PR is related to https://github.com/hotwax/fulfillment/pull/26

Closing as done under https://github.com/hotwax/fulfillment/pull/26/


pup: rename attr, go-pup -> pup, clean up, 20160425-e76307d -> 0.4.0, add meta

Motivation for this change
pup is a very popular tool (4084 stars on GitHub) and in my opinion deserves a three-letter attrname and an update to a stable version :-)
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

All fixed!


Link inconsistency

Small little thing, the about section of the repo link does not match up with the working demo link.

Thanks, fixed now! ❤


Docs: Add Installation note for PowerShell 7.X

Summary
If using PowerShell Core (7.X) on Windows the user would get an execution policy error even if set to RemoteSigned as the actual script would execute within Windows PowerShell.
Test Plan
Tested locally. Docs only.

I believe this addressed with -ExecutionPolicy ByPass now.
Thanks for putting up the pull request though!

And don't hesitate to reach out if the current documentation doesn't help.


LDAP example and more accurate volume mount hack

I think it would be useful to add an example ldap config. I had to do quite a bot of digging to find something that worked. A simple example can at least get a user pointed in the right direction. I also changed the volume mount hack to be a better example. The aiflow uid/gid is 50000, so it seems logical to have a sample that matches.

What issues does your PR fix?

fixes values.yaml
fixes #YYY

What does your PR do?
Adds an LDAP example to the values.yaml file. Added a more accurate and usable volume mount hack.
Checklist

[ ] Commits are signed
[ ] Commits are squashed (if appropriate)
[ ] Chart version bumped
[x] Chart documentation updated
[ ] Passes linting

Signed-off-by: cocampbe cocampbe[at]gmail.com

Im sorry, but as we wont be supporting features which are only present in airflow 1.10 going forwards (after version 8.0.0 of this chart).
Because airflow 2.0 depreciated the non-RBAC WebUI, you have to configure LDAP using the webserver_config.py now.
NOTE: I am still finalising 2.0 support, see https://github.com/airflow-helm/charts/issues/58

NP. Makes sense.


Shiny create: Basic App creation overrides files.

When creating a Basic App in the directory with another shiny app, the files get overridden. When creating a Dashboard or something else, I get the proper error message. Error: Can't create new files because the following files already exist in the destination directory:
Steps:

shiny create
Select Basic App
Select Yes for Shiny Express
pick a destination directory
Edit the app.py file that is created
shiny create again
Select Basic App
Select Yes for Shiny Express
pick the same destination directory as # 4
See the edits for app.py were overridden

I can't reproduce this, can you provide more information about your setup?

Sure thing! Let me know what specific info you need if it isn't this.
OS: macOS 14.2
shiny: <IP_ADDRESS>
Python: 3.9.6
Here is the output of me reproducing it again.
darbyhadley@darbyhadley-TX4L ~ % shiny create
? Which template would you like to use?: Basic App
? Would you like to use Shiny Express? Yes
? Enter destination directory: ./1/testshiny2/
Created Shiny app at 1/testshiny2
Next steps open and edit the app file: 1/testshiny2/app.py
darbyhadley@darbyhadley-TX4L ~ % shiny create
? Which template would you like to use?: Basic App
? Would you like to use Shiny Express? Yes
? Enter destination directory: ./1/testshiny2/
Created Shiny app at 1/testshiny2
Next steps open and edit the app file: 1/testshiny2/app.py

If I create a Dashboard twice, it errors as expected.
darbyhadley@darbyhadley-TX4L ~ % shiny create
? Which template would you like to use?: Dashboard
? Enter destination directory: ./1/testshiny2/
Created Shiny app at 1/testshiny2
Next steps open and edit the app file: 1/testshiny2/app.py
darbyhadley@darbyhadley-TX4L ~ % shiny create
? Which template would you like to use?: Dashboard
? Enter destination directory: ./1/testshiny2/
Error: Can't create new files because the following files already exist in the destination directory: "requirements.txt", "README.md", "penguins.csv"

Okay I can replicate, here's what's going on for posterity:

We were inadvertently checking if any file or directory existed in the destination directory which also existed in the template directory. This included the __pycache__ folder
When we copy over app-core.py we change the name to app.py which means that it wouldn't trigger the error (app-core.py is not the same as app.py)
Previous testers all had the __pycache__ folder so they weren't seeing the overwrite, but that's not a stable or informative test

I changed the code to ignore __pycache__ and always error if there's an app.py in the destination directory.
Thanks for the excellent bug!


New package: Jackaudio.JACK2 version v1.9.21

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Resolves #93790

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

Great. Thank you very much.


aria-expanded value

in safari it has all the time aria-expanded with value false... even if it's opened
<... id="rmjs-1" aria-expanded="false" data-readmore="" style="max-height: none; height: 787px;">

+1 on this issue.

Version 3.0 of Readmore.js will drop jQuery entirely, in favor of vanilla JS. I believe this issue will be fixed in the next version.

Nope, not fixed in 3.0.


Having trouble with a huge warning message regarding createRange

I'm using Enzyme with Jest in Create-react-app application.
Current behavior
I have a test which uses the mount method to find an element on a page. The test passes but a warning gets printed out:
(node:79591) UnhandledPromiseRejectionWarning: TypeError: document.createRange is not a function
(node:79591) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:79591) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
(node:79591) UnhandledPromiseRejectionWarning: TypeError: document.createRange is not a function
(node:79591) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:79591) UnhandledPromiseRejectionWarning: TypeError: document.createRange is not a function
(node:79591) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:79591) UnhandledPromiseRejectionWarning: TypeError: document.createRange is not a function
(node:79591) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 4)

I have tried adding some polyfill for the createRange. The error message is different, but still remains:
(node:79782) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'nodeName' of undefined
(node:79782) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:79782) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
(node:79782) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'nodeName' of undefined
(node:79782) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 2)
(node:79782) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'nodeName' of undefined
(node:79782) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 3)
(node:79782) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'nodeName' of undefined
(node:79782) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 4)

The test in question:
import React from "react";
import { shallow, mount } from "enzyme";
import UnitGroup from './UnitGroup';
import { UnitGroupHeading } from '../UnitGroupHeading/UnitGroupHeading';

it('should render properly', () => {
  shallow(<UnitGroup />);
});

it('should render UnitGroupHeading properly', () => {
  const wrapper = mount(<UnitGroup unit={{name: 'test'}} />);
  expect(wrapper.find(UnitGroupHeading)).toHaveLength(1);
});

Expected behavior
The error message should not be shown.
Your environment

Mac OS High Siera
Node 8.10.0
npm 5.7.1

API

[ ] shallow
[x] mount
[ ] render

Version

library
version

Enzyme
3.3.0

React
16.3.0

react-script
2.0.0-next.b2fd8db8

Adapter

[x] enzyme-adapter-react-16
[ ] enzyme-adapter-react-15
[ ] enzyme-adapter-react-15.4
[ ] enzyme-adapter-react-14
[ ] enzyme-adapter-react-13
[ ] enzyme-adapter-react-helper
[x] react-test-renderer 16.3.0

Exactly the same thing happens to me :
Current behavior
I have a test which uses the mount method to find an element on a page. The test passes but a warning gets printed out:
(node:26611) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: document.createRange is not a function
I polyfill this createRange function and then get the other error as mnemanja :
(node:27011) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: Cannot read property 'nodeName' of undefined
Expected behavior
The error message should not be shown.
Your environment
Linux Ubuntu 17.04
Node 8.4.0
yarn 1.6.0
npm 5.3.0
API

[ ] shallow

[x] mount

[ ] render

library
version

Enzyme
3.3.0

React
16.2.0

Adapter

[x] enzyme-adapter-react-16
[ ] enzyme-adapter-react-15
[ ] enzyme-adapter-react-15.4
[ ] enzyme-adapter-react-14
[ ] enzyme-adapter-react-13
[ ] enzyme-adapter-react-helper
[ ] react-test-renderer 16.3.2

Same issue, same error message:
Windows 10
node v8.9.1
yarn 1.6.1
enzyme 3.3.0
jest 22.4.3

anybody solved this issue?

Im having a problem with jest and tooltips from material ui

@stonecoder19 i had the same problem with createRange and material ui. I had to add another snippet to the test setup file for jest:
if (global.document) {
    document.createRange = () => ({
        setStart: () => {},
        setEnd: () => {},
        commonAncestorContainer: {
            nodeName: 'BODY',
            ownerDocument: document,
        },
    });
}

I did not have the error TypeError: Cannot read property 'nodeName' of undefined, so not sure whats going on there

It seems like there's some code in your components (possibly third-party code) that's throwing an error inside a promise. This doesn't seem enzyme-related.


Add travis ci and badge to README.

Created .travis.yml for build of dependencies and running of tests in the 'tests' directory.

Developer Program Member


Added sample for Deploy ARM template with progress.

Also fixed script file for scale set sample.

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

It looks like you're working at Microsoft (ans). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Support for request interceptor?

Hey
Many http framwork, like okhttp have support for interceptors that allow you to modify the request receive the response. This can be pretty useful for many features like oauth2, content md5 and so on. Is there a way to achieve something similar with alamofire?
Thanks,
Omer

Alamofire already provides APIs that offer complete control over request and response serialization. What feature would the interceptor paradigm offer that isn't already achievable?

I want to achieve the following: Writing an interceptor that add authorization header (I've already saw multiple approach to achieve this with alamofire) and invalidate the token if the request failed with 401. But, I also want that other developers in our company could easily use this and have the full alamofire api (for example, choosing serializer, custom validation etc). With interceptors, they can simply add this to their request...

Given the chainable nature of Alamofire's functionality, you can write those bits of functionality yourself and others can use that code. For instance, you could create a custom request method that adds the header but otherwise just exposes the normal API. A custom validator that contains the behavior you want to delete the token can be chained in as well.
Given that, I thinks there's still some room for improvement.

Yes, but this is a bit cumbersome, I was looking for something that is easier to use. Such an approach also is a bit error prone - more things to remember, meaning probably someone will forget to add them someday...

I second this request. I am trying to write tests that ensure my outgoing requests contain the right body and the right headers. With AFNetworking, I had a simple RequestSerializer in my unit tests that stored the request body on the request object before it was wiped by the framework. Now, that no longer seems possible.

You might be able to solve this with custom encoding, but it will not be pretty...

This request much be solved with the RequestAdapter.

@tobiasoleary is correct. The concept of the "interceptor" is essentially the combination of the RequestAdapter and RequestRetrier protocols added in #1450. There is a fairly detailed example in the README right now that demonstrates how you could build an OAuth2 refresh system using it. This functionality will ship in AF 4.
I'm going to go ahead and close this issue out since those changes cover this request.
Happy refreshing! 🍻

@tobiasoleary is correct. The concept of the "interceptor" is essentially the combination of the RequestAdapter and RequestRetrier protocols added in #1450. There is a fairly detailed example in the README right now that demonstrates how you could build an OAuth2 refresh system using it. This functionality will ship in AF 4.
I'm going to go ahead and close this issue out since those changes cover this request.
Happy refreshing! 🍻

http 404

the link you provided above doesn't exist anymore
https://github.com/Alamofire/Alamofire/tree/swift3#requestretrier


definitely typed issue with ReadonlyArray and readonly T[]

I was updating a type definition and ran into a test that began failing in a different package than the one I edited. I checked out master and ran the same tests only to find that they're failing in master. After debugging for a while, it appears like this is a bug in dtslint given the errors I'm getting.
CI:
https://travis-ci.org/DefinitelyTyped/DefinitelyTyped/builds/487265799 -- changes in a related package
https://travis-ci.org/DefinitelyTyped/DefinitelyTyped/builds/487281625 -- cosmetic changes in graphql-resolve-batch that have the error
Test:
https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/graphql-resolve-batch/graphql-resolve-batch-tests.ts
CLI:
$ npm run lint graphql-resolve-batch

><EMAIL_ADDRESS>lint /Volumes/dev/DefinitelyTyped
> dtslint types "graphql-resolve-batch"

Error: /Volumes/dev/DefinitelyTyped/types/graphql-resolve-batch/graphql-resolve-batch-tests.ts:61:5
ERROR: 61:5   expect  Expected type to be:
  ReadonlyArray<SomeTestSource>
got:
  readonly SomeTestSource[]
ERROR: 72:5   expect  Expected type to be:
  ReadonlyArray<SomeTestSource>
got:
  readonly SomeTestSource[]
ERROR: 84:5   expect  Expected type to be:
  ReadonlyArray<SomeTestSource>
got:
  readonly SomeTestSource[]
ERROR: 100:5  expect  Expected type to be:
  ReadonlyArray<SomeTestSource>
got:
  readonly SomeTestSource[]
ERROR: 116:9  expect  Expected type to be:
  ReadonlyArray<SomeTestSource>
got:
  readonly SomeTestSource[]

    at /Volumes/dev/DefinitelyTyped/node_modules/dtslint/bin/index.js:161:19
    at Generator.next (<anonymous>)
    at fulfilled (/Volumes/dev/DefinitelyTyped/node_modules/dtslint/bin/index.js:5:58)
    at <anonymous>

If I change a few of them to be readonly SomeTestSource[] then the errors start clearing from the output.
If I change each of the 5 $ExpectType ReadonlyArray<SomeTestSource> to be readonly SomeTestSource[]:
-    // $ExpectType ReadonlyArray<SomeTestSource>
+    // $ExpectType readonly SomeTestSource[]

Then I get this error (notice the expected/got is flipped, this feels like a heisenbug):
$ npm run lint graphql-resolve-batch

><EMAIL_ADDRESS>lint /Volumes/dev/DefinitelyTyped
> dtslint types "graphql-resolve-batch"

Error: /Volumes/dev/DefinitelyTyped/types/graphql-resolve-batch/graphql-resolve-batch-tests.ts:61:5
ERROR: 61:5   expect  Expected type to be:
  readonly SomeTestSource[]
got:
  ReadonlyArray<SomeTestSource>
ERROR: 72:5   expect  Expected type to be:
  readonly SomeTestSource[]
got:
  ReadonlyArray<SomeTestSource>
ERROR: 84:5   expect  Expected type to be:
  readonly SomeTestSource[]
got:
  ReadonlyArray<SomeTestSource>
ERROR: 100:5  expect  Expected type to be:
  readonly SomeTestSource[]
got:
  ReadonlyArray<SomeTestSource>
ERROR: 116:9  expect  Expected type to be:
  readonly SomeTestSource[]
got:
  ReadonlyArray<SomeTestSource>

    at /Volumes/dev/DefinitelyTyped/node_modules/dtslint/bin/index.js:161:19
    at Generator.next (<anonymous>)
    at fulfilled (/Volumes/dev/DefinitelyTyped/node_modules/dtslint/bin/index.js:5:58)
    at <anonymous>

Let me know if I can provide more information.

I'd like to add the following findings:

dtslint is ran against multiple versions of TypeScript (2.6 and above)
$ExpectType ReadonlyArray<SomeTestSource> works for all versioned releases but fails on typescript/next
 $ExpectType readonly SomeTestSource[] works only for typescript/next but not for any of the versioned releases

Seems like a weird interplay between TypeScript and dtslint? Would love to know a solution though because this makes ReadOnlyArrays nearly impossible to lint.

ping @andy-ms @sandersn @DanielRosenwasser
This is currently blocking some PRs on DefinitelyTyped repo.

Should be fixed by #195. I'll make sure it gets into the production branch so the DT PRs are unblocked.

Yep, @rbuckton already merged it into the production branch.

Specifically, you can continue to use // $ExpectType ReadonlyArray<T> and it will work for both ReadonlyArray<T> and readonly T[] (for varying nesting levels of ReadonlyArray/readonly).


Remove the possibility to use a float as total_steps in rectified Adam

See #1375
blocked until we release the 0.10.

Link to deprecation warning.


[DOC] Unnecessary expose property in Docker Compose file

What do you want to do?

[ ] Request a change to existing documentation
[ ] Add new documentation
[x] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
What is the purpose of the expose property in the Docker Compose file, if ports is already set?
https://opensearch.org/docs/latest/install-and-configure/install-opensearch/docker/
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.

https://stackoverflow.com/questions/40801772/what-is-the-difference-between-ports-and-expose-in-docker-compose
https://www.baeldung.com/ops/docker-compose-expose-vs-ports

We need to expose the port so that the application is accessible externally via that port.

@unfinishednemesis The port is already exposed via the ports option. Try it without the expose.

thanks @glen-84 , yeah it not required -
"The EXPOSE keyword is primarily a no-op. It's informative ("this image will offer services on these ports"), but it doesn't have any operational impact."

Thanks for entering this issue, @glen-84! You are correct that the expose property is not typically required, but may be  needed for earlier versions of Docker. We will not take any action to remove it at this time, so I will close this issue. Have a great day!


Update params.md

Adding vital information.

@johnnyagerard a new version LGTM. Thanks for the update.

@rpetrusha Code is updated as you suggested.


getBuildId fails when entry point is a virtual module

esbuild supports virtual modules as entry points but this plugin assumes it can read the contents of all entry points from the file system to generate a build ID.
Interestingly for me this didn't throw an error during the build, it just silently resulted in zero styles being generated.
I ran into this issue while trying to integrate with Remix which internally uses virtual module entry points.

thanks for reporting, could you please provide more detail about the virtual module entry points? is it a path-like string? I'm not familiar with Remix.

I have an entry point named @remix-run/dev/css-build but this isn't a real module. If you try to resolve this and read it from disk it will fail. We actually have an esbuild plugin that resolves this entry point to a virtual module so that we can generate the code on the fly based on your Remix config. We're doing this so we can create a separate build that has access to all CSS in the app and can then generate a single CSS file.
Here's what the plugin looks like at the moment (still a work in progress):
import type { Plugin } from "esbuild";
import type { RemixConfig } from "../../config";

/**
 * Creates a virtual module called `@remix-run/dev/css-build` that imports all
 * browser build entry points so that any reachable CSS can be included in a
 * single file at the end of the build.
 */
export function cssEntryModulePlugin(config: RemixConfig): Plugin {
  let filter = /^@remix-run\/dev\/css-build$/;

  return {
    name: "css-entry-module",
    setup(build) {
      build.onResolve({ filter }, ({ path }) => {
        return {
          path,
          namespace: "css-entry-module",
        };
      });

      build.onLoad({ filter }, async () => {
        return {
          resolveDir: config.appDirectory,
          loader: "js",
          contents: [
            `export * as entryClient from ${JSON.stringify(
              `./${config.entryClientFile}`
            )};`,
            ...Object.keys(config.routes).map((key, index) => {
              let route = config.routes[key];
              return `export * as route${index} from ${JSON.stringify(
                `./${route.file}`
              )};`;
            }),
          ].join("\n"),
        };
      });
    },
  };
}


Update InterPro to v99.0

I'm downloading now.

Fixed title.

The XML processing has finished and I've added a news item for tomorrow.


[FEA] Improve copying decimal data from CPU columnar data

Is your feature request related to a problem? Please describe.
For now, when we copy decimal column vectors during HostColumnarToGpu, it involves a round trip of Decimal encoding and decoding for each value: cv.getDecimal(i, precision, scale).toUnscaledLong(), which can be avoided under some circumstances.
Describe the solution you'd like
Ideally, we can bypass the round trip through feeding cuDF.HostColumnBuilder with the underlying storage(int32/int64/binary), if we know the exact type of upstreaming ColumnVector, such as: WritableColumnVector.

This issue can be regarded as a sub issue of #4393.


Update cats, common, test to 0.2.3

Updates

io.github.cquiroz.react:cats
io.github.cquiroz.react:common
io.github.cquiroz.react:test

from 0.2.1 to 0.2.3.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [{ groupId = "io.github.cquiroz.react" }]

labels: semver-patch

Superseded


.NET SDKs to access file share with File Service SAS Token

Query/Question
I am reaching out in regards to few doubts about accessing File Share using SAS Token via the .NET SDK's. I have a File Service SAS Token(not the Connection String of the Storage Account). Using that, I am trying to do the following using .NET SDK's

Fetch the properties of storage account. I see there is a REST API( Ref #1 )  endpoint for that which uses only the storage account name ,subscription ID and Resource group, but looking for the equivalent .NET API.
I have been able to access file share using the File Share SAS Token. However, once I have the reference to the file share, I am unable to see the quota on that. From the portal, I see the allotted storage is 100 GiB. But, the value I see in share.Properties.Quota = null.

Code :
CloudFileShare share = cloudBlobClient.GetShareReference("exporttemplate"); // File share name : "exporttemplate"
CloudFileDirectory rootDir = share.GetRootDirectoryReference(); // No errors here
CloudFileDirectory sampleDir = rootDir.GetDirectoryReference("Exported416"); // No errors here either. Directory name = "Exported416"

However, the below check fails saying "You-are-not-authorized-to-perform-this-operation". My understanding is that since my SAS is a File Service SAS Token, I should be able to access/run anything that is part of the fileshare.
         if (share.Exists()){
         }

For the first question above 1): I am specifically looking for the following properties of the Storage account and was wondering if there are any alternatives to fetch those properties given that I would have only the File Share SAS Token.
a. Checking the Storage Account existence
b. Perform SAS Token validation
c. Checking Storage Account configuration : type : PFS, General Purpose V2
d. Quota : enough space exists for the files I am about to copy over to the directory in the file share.
e. Checking whether the storage the is some particular VNET( the same one as my source server which hosts a MySQL instance)
Ref #1:
https://docs.microsoft.com/en-us/rest/api/storagerp/storageaccounts/getproperties
Environment:
SAS Token was valid at the time of running this.
My SAS generation settings:

Microsoft.Azure.Storage.Common, Version=<IP_ADDRESS>

Microsoft.Azure.Management.Storage, Version=15.1.0

Microsoft.Azure.Storage.File, Version= 11.1.3

Hosting OS : Windows

.NET runtime version 4.7.2

IDE and version :  Visual Studio 2017

This was resolved over email a few days back, please let me know if you need further assistance.
Here is the link https://github.com/Azure/azure-sdk-for-net/blob/master/sdk/storage/Azure.Storage.Files.Shares/samples/Sample02_Auth.cs#L98

Please reactivate if this is still open.