How to use varargs signature

Hi!
I am trying to implement an adapter to use with jsonata for the following interface:
interface JsonExtension {
    fun call(node: JsonNode, arguments: List<JsonNode>): JsonNode
}

I implemented the following class to bridge the gap between the two interfaces:
class ExtensionFunction(
    val extension: JsonExtension
) : Jsonata.JFunctionCallable {
    override fun call(input: Any?, args: MutableList<Any?>?): JsonNode {
        val mapper = jacksonObjectMapper()
        val node: JsonNode = mapper.valueToTree(input)
        val arguments: List<JsonNode> = args?.let { mapper.convertValue(it) } ?: emptyList()
        return extension.call(node, arguments)
    }
}

I am trying to register it with jsonata like this:
val extension = object : JsonExtension {
    override fun call(node: JsonNode, arguments: List<JsonNode>): JsonNode {
        println("Input: '$node' and args: $arguments")
        return node
    }
}

registerFunction("myExtension", Jsonata.JFunction(ExtensionFunction(extension), "???"))

But I cannot figure out how to write a varargs signature. I want this function to accept any number of any type of json arguments and return any type of result, something like the following in Java:
Object call(Object... args);

I have no experience with jsonata and I should have checked their docs before opening a ticket. There seems to be no support for such signature in jsonata. I opened a ticket there and will wait for their support.
Sorry for the noise!

No problem. You raise a good point. You'll probably have to use an array. Consider this count example:
$count(1,2,3)
Argument 2 of function "count" does not match function signature

$count([1,2,3])
3

Yes, but it seems kinda awkward to use if an extension expects a single argument.
I thought that at least <j+> would parse the arguments as [1, 2, 3] but it parses them as [1, 1, 1]? Weird :D I hoped that something like <j+?> is supported since that sounds like what I am looking for (0 or more json types).

Reopened because varargs are not parsed correctly

Just to make it clear, the <j+> seems to be the same as jsonata itself.

After further testing, you are right. I implemented it like this:
private class ExtensionFunction(
    val extension: JsonExtension
) : JFunctionCallable {
    override fun call(input: Any?, args: MutableList<Any?>?): JsonNode {
        val node: JsonNode = mapper.valueToTree(input)
        val arguments: List<JsonNode> = args?.let { mapper.convertValue(it.first()) } ?: emptyList()
        return extension.call(node, arguments)
    }
}

And used the following signature: <a?:j>. It works with all of the following:
$myExtension()
$myExtension([1, {"a": "b"}, 2])
$myExtension({"a": 1, "b": 2})
$myExtension([1, 2, 3])

It just doesn't work if there is a null in the list, such as:
$myExtension([1, null, 2])

That results in the following exception:
No serializer found for class com.dashjoin.jsonata.Jsonata$2 and no properties discovered to create BeanSerializer (to avoid exception, disable SerializationFeature.FAIL_ON_EMPTY_BEANS) (through reference chain: java.util.ArrayList[0]->com.dashjoin.jsonata.Utils$JList[1])

The issue is that Jackson does not know that it should deserialize com.dashjoin.jsonata.Jsonata#NULL_VALUE static object as null.
I should be able to fix this with Jackson, thanks for your help!

I found that you already added support for it 😄
private class ExtensionFunction(
    val extension: JsonExtension
) : JFunctionCallable {
    override fun call(input: Any?, args: MutableList<Any?>?): JsonNode {
        val node: JsonNode = mapper.valueToTree(input)
        val arguments: List<JsonNode> = args?.let { mapper.convertValue(Utils.convertNulls(it.first())) } ?: emptyList()
        return extension.call(node, arguments)
    }
}

Supports all the usecases I need. Thanks for this amazing library ❤


Folds

Merge after #2
Adds Maybe.fold and Result.fold, that sometimes is more convenient to use than chain of and_thens.

I've modified the interface of fold function.
Now instead of folding over a list of functions of type (a -> Maybe.t(a)), [b] is provided and just one function is passed that takes 2 arguments:

next element from the list, of type b;
value of type a, wrapped in the Maybe type returned from the previous call to this function or from the initial accumulator.

This function returns Maybe.t(a).
This resembles Haskell's foldM interface


Introduce schema for Quantum Volume jobs

Description
A schema for defining the parameters of a Quantum Volume job is introduced here, along with tooling for loading and validating job input files that instantiate said schema.
I tried to keep this PR the less distruptive as possible, but there are other changes that are due in follow-up PRs:

create dataclasses for collecting specific benchmark parameters
decoupling dispatching from qv https://github.com/unitaryfund/metriq-gym/issues/65

Issue ticket number and link
Fixes #61
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Unit tests have been added for the schema validator utilities.
It was also smoke tested with the following command:
python metriq_gym/dispatch_qv.py job.json --backend qasm_simulator --provider ibmq

with job.json being a file created by the user.
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules (or not applicable)

@vprusso comments addressed. PTAL


CON-4610 Filter remote categories in import tree by given shopId

It's bug fix for categories problem reported by lagome shop. They need to be filtered by parent and shopId.


Project manager function

Should be finished, will test when I get home

Ignore the failed Codacy check, ignoring doesn't work for whatever reason


load data from csv or json; @Chiiizzzy

@Chiiizzzy Please close this issue if the merged PR implements the necessary changes.


Va'ruun inflictor name has a new line in it

Va'ruun inflictor name has a new line in it. This causes issues with the UI

Details
Remove new line in record and strings to fix
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Look at va'ruun inflictor in menus

https://www.nexusmods.com/starfield/mods/6132

Will use #486 for this issue.


Vertical arcade games : Add "auto-verticalarcade" auto collection + Vertical game filter

++ namenames.xml : added vertical information to the file using a vert attribute

let me know if you need testing as I have exactly this kind of setup

let me know if you need testing as I have exactly this kind of setup


snazzy gives incorrect message for "Missing '()' invoking a constructor"

I've been all over the place with this issue, posting on standard, then ESLint, back to standard and now here 😆
If I have the code:
const onionoo = new Onionoo()

to create a new instance of the Onionoo class but leave off the brackets:
const onionoo = new Onionoo

standard and ESLint give the error Missing '()' invoking a constructor but snazzy just returns Missing ':
$ standard
standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.
  /Users/lukechilds/Dev/oss/onionoo-node-client/test/unit.js:14:19: Missing '()' invoking a constructor.

$ snazzy
standard: Use JavaScript Standard Style (http://standardjs.com)
standard: Run `standard --fix` to automatically fix some problems.

/Users/lukechilds/Dev/oss/onionoo-node-client/test/unit.js
  14:19  error  Missing '

✖ 1 problem

@lukechilds Thank you for your persistence and for narrowing down the issue to snazzy. The issue was actually with one of our dependents: standard-json. Issue here: https://github.com/Flet/standard-json/pull/2
Should be all fixed up if you re-install snazzy 👍

@feross Yep, all sorted 👍
Just realised you're the WebTorrent guy, just wanted to thank you for your talk on DHTs, it blew my mind!


Help\Exercises: Add new exercise related to UnitTests

Description: Add 1-3 new exercises related to UnitTests introducing developer how to use "test package" and perform Unit tests in Datagrok modules.

[ ] Adaptation of Exercise 1 documentation to make it testable
[ ] Create Exercise 10 "Perform and integrate unit-tests for package" to the documentation Help\Exercises

Exercise 10.1 - Unit test on complement function
Exercise 10.2 - Unit test on fuzzyJoin function

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-14808

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-16068


Repeated ID linter gives false positive on neighbouring conditional branches

Input File:
- if something?
  #my-id Some text 1
  #my-id2 Some text 2
  #my-id3 Some text 3
- else
  #my-id Some other text 1
  #my-id2 Some other text 2
  #my-id3 Some other text 3

Reports:
scratch.haml:2 [E] RepeatedId: Do not repeat id "#my-id" on the page
scratch.haml:6 [E] RepeatedId: Do not repeat id "#my-id" on the page
scratch.haml:3 [E] RepeatedId: Do not repeat id "#my-id2" on the page
scratch.haml:7 [E] RepeatedId: Do not repeat id "#my-id2" on the page
scratch.haml:4 [E] RepeatedId: Do not repeat id "#my-id3" on the page
scratch.haml:8 [E] RepeatedId: Do not repeat id "#my-id3" on the page

Should Report:
Nothing

Not sure if it's a possibility, but it'd be really nice if haml_lint could detect conditional ruby branches and only run the Repeated ID selector on accessible elements through each pathway.

Ah, good catch! I wonder if this is worth fixing, perhaps it's better to just locally ignore the RepeatedId linter? Let's see if @michaelherold has any good input as well.

 It's pretty tedious to go through each case and put disable/enable blocks around elements with repeated ids, especially if there are elements between the ones with repeated ids.

Thanks for the reproduction steps! I'll try to figure out how we can scope that in the AST.

I don't think it's possible to fully handle this with static analysis.
For example, say the markup was:
- if 1 < 2
  #my-id Something
- elsif 1 > 0
  #my-id Something
- else
  #my-other-id Something else

The linter would need to determine which branches of the conditional would evaluate to true.
I think this would be better handled by something which ran after the view was rendered, e.g.
visit 'widgets'
expect(page).to_not have_repeated_ids

The linter would need to determine which branches of the conditional would evaluate to true.

Not necessarily. If the linter could view the possible code paths as a tree, then the path to each leaf needs to have at most one of each id selector. In the example you gave, I wouldn't expect the linter to raise as it considers lines 1 and 3 different code paths that can't be hit at the same time.
I think the bigger issue with this approach is how big the search space gets. e.g. If I had 3 case statements, each with 3 branches, that's 9 possible paths. And if I add an if/else to each of the case branches, that's 36 possible paths.
It's going to blow out very quickly, but maybe I am just over exaggerating the problem.

@daveallie That was a bad example I gave, I was intending something like this:
- if 1 < 2
  #my-id Something
- if 1 > 0
  #my-id Other Something

The situation where only a single clause can be true is indeed simpler, although as you say, could be slow if there are many possible paths.
I wonder if there are any linters already in RuboCop which follow this kind of pattern.

🤔 I've been looking into this one and I'm stumped.
This simple Haml file:
- if 1 < 2
  #my-id Hello
- else
  #my-id World

Gets parsed into this structure:
                         RootNode
                            |
                SilentScriptNode(if i < 2)
               /            |              \
TagNode(Hello)    SilentScriptNode(else)    TagNode(World)

This isn't what I expected. I expected something like this:
                            RootNode
                           /        \
 SilentScriptNode(if i < 2)          SilentScriptNode(else)
           |                                   |
     TagNode(Hello)                      TagNode(World)

The latter would make sense to parse through: you would only have to look at the parent node's ancestors and prior siblings to check whether an ID has been used prior to the current node.
However, since the Haml is parsed into the former structure, I'm not really clear on how to tell which nodes you would have to look at for a duplicate. The rendered output will only have the TagNode(Hello) tag, so there isn't a duplicate ID ... but it's not clear how you would divine this from the parsed tree structure.
If anyone has any ideas on how we can handle this, I would love to hear them!

Wow, that's confusing...


Bump gardener-extension-provider aws and azure

Description
Changes proposed in this pull request:

Bump gardener-extension-provider-aws
Bump gardener-extension-provider-azure

Related issue(s)

 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.

/retest


CSV export from dashboard does not use filtering

[ x ] I have checked the superset logs for python stacktraces and included it here as text if any
[ x ] I have reproduced the issue with at least the latest released version of superset
[ x ] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.19.1
Expected results
When exporting slice directly from dashboard, dashboard filtering should be applied to CSV
Actual results
Dashboard filter does not apply to exported CSV
Steps to reproduce

Create dashboard
Add filter visualisation
Add any other visualisation
Open dashboard, apply filtering
Export CSV from dashboard

Looks like fix applied in https://github.com/apache/incubator-superset/issues/1304 does not use dashboard filtering

Hi ! We have noticed this problem too and it would be really helpful if it was solved.
The dashboard filter does not apply either when you explore the slice from the dashboard view. I guess they are related matters.

It should be fixed as of https://github.com/apache/incubator-superset/pull/3461


Add vscode extension recommendations

This will give a pop up in vscode suggesting the following extensions to be installed:

TailwindCSS: IntelliSense https://github.com/tailwindlabs/tailwindcss-intellisense
ESLint: https://github.com/Microsoft/vscode-eslint
Prettier: https://github.com/prettier/prettier-vscode

kind of curious whats the intent of this on the hackathon repo? the repo that would likely have new members working on it would be the club site repo, so I don't see why we need a popup to install these things on this repo when the work here is less likely to be revisited

Tru, I'll add this to the club site then.


device-camera is deprecated, use device-onvif-camera for TAF test instead

🚀 Feature Request
Relevant Package [REQUIRED]

 This feature request is for device-camera
Description [REQUIRED]

device-camera is deprecated, use device-onvif-camera for TAF test instead

@cherrycl , I assume the TAF scripts need to change if we does this swap, correct.
Do you agree this should wait unit post Levski release?

@lenny-intel there is a test case to verify the command service should not crash if the device is unreachable.  it's easy to test with an ip based camera device service.  if onvif-camera device service need a little more time, we can wait until post Levski
https://github.com/edgexfoundry/edgex-go/issues/3716


Do you know my personal information if I use this app to send data?

Thanks for good app
but I still wonder whether you collect my data or not

Hi.
It is described in setting secretion and in onboarding screen.
The application do not store any personal information. It doesn't send any data that would make possible to identify a user personally. If Google Analytics is enabled the app generates random string and keeps it locally to identify the session but it cannot be used to get information about the user behind the application.
I'm very strict about data protection and I do not allow any personal and even non-personal data to be send outside the app.
In settings you have link to privacy policy of the app. Please read it and let me know if you have more questions.


How to run the example on Android

I tried npx react-native run-android from the example folder but that's not working.
How to run the example project?

Looks like with a combo of metro configs and root-level package.json scripts and detox it is a bit different then a vanilla react-native init project
https://github.com/react-native-netinfo/react-native-netinfo/blob/master/CONTRIBUTING.md#tests--verifications

I like that there is no separate package.json (so it actually uses the library directly). I would like to apply the pattern on other react-native libraries and even write a blog post about it but can't seem to get the example running..

@eliaslecomte Sorry, I won't be able to help you without more information. You should at least post how far you're getting and what errors you're seeing.


New config option: only admins can create repos

Useful if you just use gitea to host your code and do not want other people to use create repositories.

I have not tested that but generally codewise it looks ok to me.

Done. I also squashed the commits

I'm sorry. I think we should consider this with access. We only change the access table to do this and not write so many codes.

I think we should have a whole idea about the access control.


Fixed _is_match() for ConllExtractor not appending next word.

Was using 30GB of system memory for some long sentences where the copy list would double in size for each iteration :)

Thanks!


Support refactors in apache-bigtop-base to support runtime Java

Plugin currently seems to get stuck in "installing puppet".  Still need to investigate.

LGTM, i don't see the stuck status, but we'll fix it in a new pr if it comes up.


Please Add Support for Dakuo Mosquito Repellent

Device model
ateai.mosq.dakuo
Device name
Dakuo Mosquito Repellent
Does the device require a MiCloud connection?
Do not know, tried either but its not responding to my commands
Anything special about the device?

it has a manual physical button on the device, but can be turned on from app
connects to WiFi, not BLE
supports both Mi Home and Huawei HiLink smart home ecosystem

Logs
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Initializing device services
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Device services: [
"mosquito-dispeller",
"repellent-liquid",
"dakuo-mosq-dispeller"
]
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Initializing device properties
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Device properties: [
"mosquito-dispeller:on",
"repellent-liquid:liquid-left",
"dakuo-mosq-dispeller:status",
"dakuo-mosq-dispeller:workmode",
"dakuo-mosq-dispeller:schedules"
]
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Initializing device actions
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Device actions: [
"repellent-liquid:reset-liquid"
]
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Successfully created a Generic device! It is a Mosquito Dispeller.
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Initializing accessory!
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Accessory successfully initialized!
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Registering 1 accessories!
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Everything looks good! Initiating property polling!
[7/27/2023, 4:56:24 AM] [homebridge-miot] [Liquid Repeller] Log in to MiCloud!

The device type is not supported so the plugin does not know how to map the properties and actions. You can set your device as customAccessory and then use propertyControl to manually map which properties you want to have control off.

You need to use the created property switch afterwards extra log will appear when deepDebugLog is enabled. You need to post that.
The one you posted is the startup log.

Then you are doing something wrong. You would need to double check that everything is set correctly.


Cannot interactively set backup path to existing backup repo

In light of #265 I tried to move my backup dir, but the interactive prompt refuses to do it:

Is this because it always wants to initialize a new git repo on "first run"? In any case, it would be nice to have an interactive way to move the backup like this. Or maybe there is somehow; if that's the case it would be helpful to say how after the red error message.
(I know I can manually edit the config, but the rest of the UI is so nice and free of JSON editing!)
Thanks!

The reason this is failing is actually by design.
Updating the path in the CLI moves the directory for you, which you did in the $ mv command. It's saying "Hey, you already have a directory in the place you just asked me to set up the shallow-backup repo. I'm not going to do that just in case you didn't want to overwrite this directory."
The simple fix is to move your repo back to the location specified in the config and then move it in the CLI.
The CLI could have a better indication that this is what happens. Maybe "Would you like to update this? (This will move the directory x to y)` or something would be a better prompt.

Sure, I'll take a stab at it tomorrow. I don't do much Python though so you'll have to bear with me a bit 😅

I'm happy to provide any code reviewing / help you need. This should just be a simple string change, so a great intro-to-python issue.


TFLite model not working with png images

Hello,
I have trained a semantic segmentation model using VGG16 and unet. The model displayed good results before (irrelevant of image type).
When I converted the model to tflite, I noticed that it does not work with png files. Here is an example :

When trying with a jpg however, the results seem fine compared to the original model. An example:

Hope someone have the answer because I do the same preprocessing to both images. And the original model as I said work fine with both types.
😃😃😃

Can you provide a Google Colab gist reproducing the problem? can you try to compare the performance on the same image but with different encoding(.png and .jpg)? also, check out how many channels of the png images are you passing through the model? .jpg have only 3 channels but png by default have 4, maybe you trained your model only on 3-channel images and when the models perceive 4-channels, that screw everything.

No I did print the dimensions every time. Both are the same. Here is a code part:

@UcefMountacer,
In order to expedite the trouble-shooting process, could you please provide the .py file or the Python Notebook you are running along with all the supporting files. Thanks!

Also, please mention the TensorFlow version you are using. Thanks!

Hi @amahendrakar,
Sorry for the late response.
I am using the 2.2.0 version
The code I use to run the result that I have shown you is this (in TXT) :
main_code.txt

@UcefMountacer It looks like the preprocessed input in case of png file is not the same as jpg. Could you try comparing (maybe elementwise) the image tensors you get from a jpg vs png file after this line:
image = imagenet_utils.preprocess_input(image.astype(np.float32), data_format='channels_last', mode='torch')

I am guessing that the value of image elements isn't what the model was trained with, in case of png source.

Update :
Sorry It took me so much time to get back on the project. I verified and there is problem not only with the tflite model, but with the original keras model. So the problem was necessarily because of preprocessing. After debugging, I found that using matplotlib to read images is not good (Yes, I don't know why I went in that direction). Using img = PIL.Image.open(path) and using np.array(img) solved the problem.
I guess this issue should be closed but another issue is risen with matplotlib.
🤙


About pairing data translation

Hi, thanks so much for this great work! Could you please tell me where is the paired image translation code?

Have the same problem


Update Composer dependencies (2022-12-09-00-10)

Visual regression test passed!
View the visual regression test report


Add missing UserRoleHandler

Load missing UserRoleHandler.php

Codecov Report

Merging #225 (a261092) into master (f053319) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #225   +/-   ##
=========================================
  Coverage     54.09%   54.09%
  Complexity       27       27
=========================================
  Files             5        5
  Lines            61       61
=========================================
  Hits             33       33
  Misses           28       28

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 f053319...a261092. Read the comment docs.

Thanks! I've fixed that with the new code-structure that I've just merged into master.
Now all the files are loaded :grin:


resource/cloudflare_origin_ca_certificate: trigger a replacement when csr is changed

As commented in the #1007, this is the proposed fix.
I ran some tests with the provided terraform and it works as expected

Just a note that most probably the Update function that was removed per error during the test must return after the change in the #2045. So after you approve one I will need to check the second one

thanks @peruzzof. for the UpdateContext method, i would say we'll need to leave it in place but only update the methods we expect to change without a new resource being created.


2605.9.0 Breaks iSCSI - error loading shared libraries

Description
After automatic update to 2605.9.0 from 2605.8.0, our solidfire pvcs will not mount, journalctl has following error:
iscsi_util.go:301] iscsi: could not read iface default error: iscsiadm: error while loading shared libraries: libisns.so.0: cannot open shared object file: No such file
Impact
Took some critical workloads offline as they could not mount their pvcs, had to roll back to previous versions
Environment and steps to reproduce

Set-up: Flatcar on bare metal
Task: Mount PVC (provisioned via Trident on SolidFire)
Action(s): [ sequence of actions that triggered the bug, see example below ]
a. Requested start of a new pod
b. Container remains in ContainerCreating state
Error: [describe the error that was triggered]
iscsi_util.go:301] iscsi: could not read iface default error: iscsiadm: error while loading shared libraries: libisns.so.0: cannot open shared object file: No such file

Expected behavior
Mount the volume and start the container
Additional information
We did migrate to flatcar from containerlinux via in-place upgrade earlier this year
so files seem to be present, not sure if these are the right locations:
ls /lib | grep libisn
libisns.so
libisns.so.0
ls /lib64/ | grep libisn
libisns.so
libisns.so.0

Similar to https://github.com/kinvolk/Flatcar/issues/280

Can you run the iscsiadm binary on the host without this error? It sounds like the linked issue, which is due to mapping some host binaries into the container but in a broken way, can you check if this is the case?

This may need an issue crosslinked with trident/SolidFire, where they are dynamic-linking to something on the host. This sounds like something that may need to be static-linked, or run in a container entirely

I've validated the host iscsiadm runs successfully in stable-2605.9.0, i.e. there is no library issue with the command line tool shipped with Flatcar Linux. Is there a way for trident/SolidFire to use host binaries?

Hey folks, thanks for the responses.  After reading up a bit more it does indeed appear to be due to how our kubelet container is mounting iscsiadm (I believe trident/SF just uses kubelet's iscsadm).  We deployed this cluster a while back with poseidon/typhoon before we changed from container linux to flatcar, and we are still a few versions behind with k8s.  Is there a new best way to run iscsiadm in kubelet, or does the newest container no longer mount the host version or something?  We're running hyperkube in rkt on the hosts.  I can likely use the workaround linked in #280, but it would be good to know what the best practice is for this.  I also wanted to make sure this group was aware that this was a breaking change for us, I would guess we are not the only ones mounting iscsiadm in this way.
Just checked, and in fact it does look like typhoon is still mounting iscsiadm from the host:
https://github.com/poseidon/typhoon/blob/a8b8a9b45462f0235c70c5a8d196c27ffc06fb20/bare-metal/flatcar-linux/kubernetes/cl/worker.yaml#L67

I think it's the wrong approach to copy things into the container even if the libraries were copied. The best way would be to either have the container have its own tools installed, or, leave the container and execute the binary in the host mount namespace (assuming --privileged --pid host you can invoke nsenter -a -t 1 to leave the container). If the host binary should be copied, that works best if it is statically linked. This is not the case for Flatcar, so you have to calculate which libraries are needed through ldd $(which iscsiadm) | grep -o /lib'[^ ]*' | sort -u and hope it works.

In your linked example where indeed --privileged --pid host is used I think it would work to use a iscsiadm wrapper script in the container (but I don't know the details involved, just an idea):
#!/bin/sh
exec nsenter -a -t 1 iscsiadm "$@"

FYI a bit of background why iscsiadm was being used like that: https://github.com/poseidon/typhoon/issues/907#issuecomment-743024988


Minimal rust nix shell

From our voice chat.
also the nix flakes lingo stuff: https://nix.dev/manual/nix/2.18/command-ref/new-cli/nix3-flake for your reference @Ben-PH

Thanks again!


Provide system token to reducer functions

Signed-off-by: Vasil Petkov<EMAIL_ADDRESS>

Coverage decreased (-0.05%) to 91.25% when pulling 1bb27e138741020e696092842cb9add4271673d2 on SAP:token-provide into 118f44e4ca6cfd796fcfd4766e726b32b612f47e on cloudfoundry-incubator:master.


Updated to v1744

@doneill Updated gradle version and Runtime SDK.  Will need to update gradle file before release with update 1 version.

@zinfin Looks good, I synced support and design libraries.


Wiboris batch microsoft.batch 2024 02 01

ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

[!NOTE]
As of January 2024 there is no PR assignee. This is expected. See https://aka.ms/azsdk/pr-arm-review.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

For help with Step 2 / "ARM review", see https://aka.ms/azsdk/pr-arm-review.

 Click here to see the diagram footnotes
Diagram footnotes
[1] See ARM review queue (for PR merge queues, see [2]).
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
The ARM reviewer on-call engineer visits the merge queue twice a day, so the approximate ETA for merges is 12 - 24 hours.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] New resource provider.
[x] New API version for an existing resource provider. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix OpenAPI spec quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with PR workflow diagram Step 2 (ARM review), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

"AutomaticOSUpgradePolicy": {

ARM recommends enum over boolean. Oftentimes, replacing boolean vectors with enums also eliminates invalid/unneeded combinations. Consider something like:
osUpgradePolicy: { Manual, AutomaticWithRollbackOnFailure, AutomaticWithoutRollback }
upgradeMethod: { Concurrent, Rolling, RollingWithDeferral }

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2024-02-01/BatchManagement.json:6492 in d1435fd. [](commit_id = d1435fde816ebc8297b246b886ab3edefd78a98e, deletion_comment = False)

    "enableCrossZoneUpgrade": {

Consider zonePolicy: { Ignore, UpdgradeDomain, Auto } or similar

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2024-02-01/BatchManagement.json:6517 in d1435fd. [](commit_id = d1435fde816ebc8297b246b886ab3edefd78a98e, deletion_comment = False)

    "prioritizeUnhealthyInstances": {

Consider something like nodeOrder: { UnhealthyFirst, Unrestricted, ... }

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2024-02-01/BatchManagement.json:6546 in d1435fd. [](commit_id = d1435fde816ebc8297b246b886ab3edefd78a98e, deletion_comment = False)

    "rollbackFailedInstancesOnPolicyBreach": {

Consider something like: policyBreachAction: { None, Rollback, ... }

Refers to: specification/batch/resource-manager/Microsoft.Batch/stable/2024-02-01/BatchManagement.json:6550 in d1435fd. [](commit_id = d1435fde816ebc8297b246b886ab3edefd78a98e, deletion_comment = False)

Hi @mentat9
We are mirroring the fields provided by VMSS: https://learn.microsoft.com/en-us/rest/api/compute/virtual-machine-scale-sets/create-or-update?view=rest-compute-2023-10-02&tabs=HTTP#upgradepolicy. We will read inputs from users and use them to fill an ARM template to create resources. So the Boolean values that you mentioned are referenced from VMSS. I don't think it's a good idea to make differences


Fixing tests for 7.3

Removing composer.lock files that are automatically generated during build, but foul up tests if they are part of the repo.
Also SERVICE_ACCOUNT_JSON is no longer required, although it can be set and used.

So when will this be tagged?


Proposing the inclusion of a new and trending article about RESTful

This article was published by the huge ITNEXT medium editors!

I'm covering main concepts such as:

HATEOAS - Hypermedia As The Engine Of Application State,
Resources and representations,
Addressability,
Idempotency in REST,
Richardson's REST Maturity Model.
Media types
API Versioning

I've created a GitHub project as well that you can run easily with docker, which covers the content that I've presented in this article.

https://github.com/RichardSilveira/RESTful

My article was reposted by Vintage Global - a huge architecture community on Linkedin.
https://www.linkedin.com/feed/update/urn%3Ali%3Aactivity%3A6726141404436799488/

Why it's a trending article

Plus, I've presented it on my MBA in the Software Architecture field. Note that I have added good book references at the end of this article.

Sorry, we don't link to paid content.

Hey @fzaninotto, I just came across this and was a little puzzled — I was able to read the article freely and it did contain useful (IMHO) information about what a REST API is and general principles about how to implementing one. I didn't see any paywalled content. Am I missing something? Did the post somehow change between the time this PR was submitted and now?

I believe the post changed since the PR was opened - Authors can choose to put their articles behind a paywall or not on Medium.

I see, thanks for the context. Since the article is now public, would you be open to accepting this PR now? Even though there's a risk it would be made inaccessible again, I'd consider an eventual switch back to paid content to be similar to a link going dead, which has happened in the past (see PRs #132 to #135) and in which case the solution is to remove the link when that happens.


Implemented 'pagerTabStripViewController:didMoveToIndex:' delegate method

This should close #38
EDIT: willMoveToIndex: has since been added.
This has been tested thoroughly and works as expected on all view controllers.
I've realized that there really is no good way to implement a 'willMoveToIndex:' delegate method, because the scroll view will already have moved half-way to the other view controller in the case of the user dragging the scroll view.
This method is called after the scroll view has completed all scrolling animations to reach the new view controller.

Any chance that this PR will get accepted? If not, I need to fork the repository so that I can use it for my personal project.
Thanks

@michaelEllisUy I've fixed the merge conflicts on this PR. Hoping to see this implemented soon?

Just realized I've been tagging the wrong person when asking about the PR. Apparently @mtnbarreto is the author for this. Sorry.
@mtnbarreto do you think this will be implemented?

This PR has been brought up to date with master.
I've also added a willMoveToIndex: delegate method that works perfectly.
@mtnbarreto Any chance that I can get a response on this PR? I'm using this in a production application right now and have to create my own private version of the Pod is a drag.
Thanks

I see that there have been a bunch of changes that have caused merge conflicts for this PR. I'd be happy to resolve the merge conflicts if you are interested in pulling this in still? Otherwise, feel free to let me know and close the request.

I think this can be very useful

How is this not part of this library?

@AnthonyMDev thanks for contributing but the base code has changed too much to merge these changes. In addition to that i don't want to make the library more complex to use.


AuthenticationException in SignalR client Hub in .netcore 3.1

Issue Title
AuthenticationException in SignalR client Hub in .netcore 3.1
General
Basically I have an WPF client based on .netcore 3.1 (SignalR client) which receives notification from a SignalR Hub. The hub is hosted on our server and the WPF runs on users local machine. Locally everything works fine.
When the WPF App tries to connect to the Hub I get the following error message:

System.Net.Http.HttpRequestException: The SSL connection could not be established, see inner exception. ---> System.Security.Authentication.AuthenticationException: The remote certificate is invalid according to the validation procedure. at System.Net.Security.SslStream.StartSendAuthResetSignal(ProtocolToken message, AsyncProtocolRequest asyncRequest, ExceptionDispatchInfo exception) at System.Net.Security.SslStream.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.StartReceiveBlob(Byte[] buffer, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.CheckCompletionBeforeNextReceive(ProtocolToken message, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.StartSendBlob(Byte[] incoming, Int32 count, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.ProcessReceivedBlob(Byte[] buffer, Int32 count, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.StartReadFrame(Byte[] buffer, Int32 readBytes, AsyncProtocolRequest asyncRequest) at System.Net.Security.SslStream.PartialFrameCallback(AsyncProtocolRequest asyncRequest) --- End of stack trace from previous location where exception was thrown --- at System.Net.Security.SslStream.ThrowIfExceptional() at System.Net.Security.SslStream.InternalEndProcessAuthentication(LazyAsyncResult lazyResult) at System.Net.Security.SslStream.EndProcessAuthentication(IAsyncResult result) at System.Net.Security.SslStream.EndAuthenticateAsClient(IAsyncResult asyncResult) at System.Net.Security.SslStream.<>c.b__65_1(IAsyncResult iar) at System.Threading.Tasks.TaskFactory1.FromAsyncCoreLogic(IAsyncResult iar, Func2 endFunction, Action1 endAction, Task1 promise, Boolean requiresSynchronization) --- End of stack trace from previous location where exception was thrown --- at System.Net.Http.ConnectHelper.EstablishSslConnectionAsyncCore(Stream stream, SslClientAuthenticationOptions sslOptions, CancellationToken cancellationToken) --- End of inner exception stack trace --- at System.Net.Http.ConnectHelper.EstablishSslConnectionAsyncCore(Stream stream, SslClientAuthenticationOptions sslOptions, CancellationToken cancellationToken) at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean allowHttp2, CancellationToken cancellationToken) at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken) at System.Net.Http.HttpConnectionPool.GetHttpConnectionAsync(HttpRequestMessage request, CancellationToken cancellationToken) at System.Net.Http.HttpConnectionPool.SendWithRetryAsync(HttpRequestMessage request, Boolean doRequestAuth, CancellationToken cancellationToken) at System.Net.Http.RedirectHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken) at System.Net.Http.DiagnosticsHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken) at Microsoft.AspNetCore.Http.Connections.Client.Internal.AccessTokenHttpMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken) at Microsoft.AspNetCore.Http.Connections.Client.Internal.LoggingHttpMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken) at System.Net.Http.HttpClient.FinishSendAsyncUnbuffered(Task`1 sendTask, HttpRequestMessage request, CancellationTokenSource cts, Boolean disposeCts) at Microsoft.AspNetCore.Http.Connections.Client.HttpConnection.NegotiateAsync(Uri url, HttpClient httpClient, ILogger logger, CancellationToken cancellationToken) Microsoft.AspNetCore.Http.Connections.Client.HttpConnection: Debug: Skipping dispose, connection is already disposed.

Below is my application code.
Startup.cs
public class Startup
{
    public Startup(IConfiguration configuration)
    {
        Configuration = configuration;
    }

    public IConfiguration Configuration { get; }

    // This method gets called by the runtime. Use this method to add services to the container.
    public void ConfigureServices(IServiceCollection services)
    {
        services.AddControllers();

        services.AddSignalR(hubOptions => { hubOptions.EnableDetailedErrors = true; });
    }

    // This method gets called by the runtime. Use this method to configure the HTTP request pipeline.
    public void Configure(IApplicationBuilder app, IWebHostEnvironment env)
    {
        if (env.IsDevelopment())
        {
            app.UseDeveloperExceptionPage();
        }

        app.UseHttpsRedirection();

        app.UseRouting();

        app.UseAuthentication();

        app.UseEndpoints(endpoints =>
        {
            endpoints.MapControllers();
            endpoints.MapHub<SyncCodeHub>("/signalr");
        });
    }
}

SyncHub.cs
public class SyncHub : Hub
{
    public override Task OnConnectedAsync()
    {
        Clients.Caller.SendAsync("Connected", Context.ConnectionId);

        return base.OnConnectedAsync();
    }
}

public class SyncHub : Hub
{
    public override Task OnConnectedAsync()
    {
        Clients.Caller.SendAsync("Connected", Context.ConnectionId);

        return base.OnConnectedAsync();
    }
}

WebHookController.cs
// POST: /webhook
[HttpPost]
public async Task<IActionResult> Post([FromBody] JsonElement body)
{
    try
    {
        var eventType = body.GetProperty("eventType").ToString();

        _logger.LogInformation($"[HttpPost]: The current event type is '{eventType}'");

        switch (eventType)
        {
            case "build.complete":
                return await BuildComplete(body);
            default:
                return BadRequest();
        }
    }
    catch (Exception ex)
    {
        _logger.LogError($"[HttpPost] Exception: {ex}");

        return BadRequest();
    }
}

WPF App
var connection = new HubConnectionBuilder()
    .WithUrl(viewModel.Endpoint)
    .ConfigureLogging(logging => {
        // Log to the Output Window
        logging.AddDebug();

        // This will set ALL logging to Debug level
        logging.SetMinimumLevel(LogLevel.Debug);
    })
    .Build();

connection.On<string>("ReceiveMessage", (message) =>
{
    Dispatcher.CurrentDispatcher.Invoke(() =>
    {
        using (var db = new SettingsDbContext())
        {
            var userSettings = db.UserSettings.First(x => x.ItiUsername == Environment.UserName);

            SyncTo50Hertz.Sync(userSettings);
        }
    });
});

connection.On<string>("Connected", (connectionId) => { viewModel.ConnectionState = "Connected"; });

try
{
    connection.StartAsync();
}
catch (Exception ex)
{
    Debugger.Break();

    // TODO: Handle error
}

Program.cs
public class Program
{
    public static void Main(string[] args)
    {
        CreateHostBuilder(args).Build().Run();
    }

    public static IHostBuilder CreateHostBuilder(string[] args) =>
        Host.CreateDefaultBuilder(args)
            .ConfigureWebHostDefaults(webBuilder =>
            {
                webBuilder
                    .ConfigureLogging(logging =>
                    {
                        logging.ClearProviders();
                        logging.AddConsole();
                    })
                    .UseContentRoot(Directory.GetCurrentDirectory())
                    .UseIISIntegration()
                    .UseStartup<Startup>();
            });
}

From the exception text:

The remote certificate is invalid according to the validation procedure.

Please double-check that your web server's SSL certificate is valid.

Although this might be an issue, I wanted to bypass the SSL. I tested with Fiddler and Postman disabling SSL and the endpoint could be hit. I also got an response from Fiddle. So, how can I disable bypass SSL validation in .netcore 3.1?
I tried this: https://stackoverflow.com/questions/42712844/ignore-bad-certificate-net-core but it didn't help

The issue was indeed the server certificate. It was expired. I would be very interested still to know how to bypass an invalid certificate.

@dgroh Did you ever figure out how to bypass the cert check?


gclient not found

am running this on ec2 on ubuntu 12.04(ami-13471c23).
it says gclient not found when running get_webrtc

so silly of me, very basic mistake at my end. closing this issue.


Cannot get commits via API

Gitea version (or commit ref): 1.3.3
Git version: 2.16.2
Operating system: Arch Linux
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
There should be an option for getting the commits from the API. However, since in most of projects, the number of commits is huge, there should also be an option to get a specified number of commits.
Screenshots
Irrelevant

Fixed by #6408.


Markdown preview not found when Issue tracker and Pull requests are disabled

Description
The wiki markdown editor has an error where the preview gives "Not found" when the Issue tracker and Pull requests are disabled on a repository.
Steps to reproduce:

create a repo
go to settings
uncheck "Enable Repository Issue Tracker" and "Enable Repository Pull Requests" and save the settings
go to the wiki tab
click "Create the first page"
click on "Preview"

You will now see "Not found" in stead of a preview of your markdown.
The browser console will show a 404 on the URL "POST
https://demo.gitea.com/jorisdrenth/issue_test_2/markup"
Gitea Version
1.22
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
Debian
How are you running Gitea?
Binary, on a Debian based dedicated VPS.
Database
SQLite

This was fixed by #31158


upstream_tag_template defined on job level not working for pull_from_upstream

The problem is when processing the New Hotness event initially, we take the upstream_tag_template from the whole package config: https://github.com/packit/packit-service/blob/ff6c0f5f4ae9e51311fb8e195d2b2df3d325c4ac/packit_service/worker/events/new_hotness.py#L130-L134.
An example of a failure caused by this: https://dashboard.packit.dev/results/pull-from-upstream/4757 (config: https://src.fedoraproject.org/rpms/blosc2/blob/rawhide/f/.packit.yaml)

A implementation suggestion could be to loop through all the jobs and see if one is matching.
In the meantime we can update the documentation and explain how it is working now: we take it from the package config and not the job config.


Export dataset selection

Adding this issue so we don't forget pending features:

[ ] Select datasets.
[ ] Export only the metadata.

cc @gfiorav @xavijam

hey @juanignaciosl we got this request from a client so could we possibly push this up a bit?

Editor 3 is the priority now. Depending on the evolution of the week we might be able to do it soon. Stay tuned.

Okay sounds good!
On Mon, Jun 27, 2016 at 10:29 AM, Juan Ignacio Sánchez Lara <
<EMAIL_ADDRESS>wrote:

Editor 3 is the priority now. Depending on the evolution of the week we
might be able to do it soon. Stay tuned.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/CartoDB/cartodb/issues/8211#issuecomment-228762077,
or mute the thread
https://github.com/notifications/unsubscribe/AEmtLNJxJiDN1Vmkrp7Mno2WtiY-CHLpks5qP944gaJpZM4I-269
.

--
Matt Forrest
Client Success Manager
CartoDB
201 Moore St, Brooklyn, NY, 11206, USA
T:<PHONE_NUMBER> Skype: mbforr
W: www.cartodb.com
Live sessions to learn CartoDB http://cartodb.com/webinars/

Is this ticket talking about Exporting the results of widget filters?

@andrewxhill no, and I haven't heard about that feature. It might be interesting, but that should be discussed, so open a new issue or feature request, please.

@juanignaciosl can we close this?

I certainly think that selecting datasets on map export might be useful, but it's certainly not a priority, so let's close this. It will eventually come back if it's actually useful.


Add linting rule for quotes

Many pull requests use different forms of quotes, this problem can be fixed with a linting rule
We want all quotes to use double quotes (") except the places, where template literals (`) are used for templates.
Acceptance Criteria:

Running yarn run lint should not yield any errors after adding the quotes rule.

@robeau no changes required, thanks a lot :)


After upgrading to v0.0.5, the content directory watcher stops adding new archives to the ComicLib database

I've observed that since the release of v0.0.5, the changes in the content path no longer trigger a scan, which prevents new archives from being added to the database. This issue seems to stem from the fact that directory updates are now being excluded from the watchfiles in scan.py, as seen in this line of code:
https://github.com/ComicLib/comiclib/blob/f82bec4f3acaf7de61c7b6ff93a7cfd211c0fcc6/comiclib/scan.py#L111
Because the scanners such as 22-ehviewer.py require the parent directory path of the image files,  the scan will skip them when is invoked with a path to image files. Is this modification intended to address other issues?

As described in the release note, directory will not trigger a scan during watch since v0.0.5.
This is because, unlike single files, it is difficult to determine when the contents of this directory have been completely created.
Therefore we cannot accurately count some metadata, such as page count.
I'll try to re-enable it once a good solution is available.
Before that, you can try some manual ways to trigger a scan: https://comiclib.github.io/comiclib/en/scan%26watch/


Static Torque layer renders the browser static too

Context
Setting a static Torque layer clogs the browser
Steps to Reproduce

Create a map with an arbitrary points layer
Style the layer as Torque
Set duration = 0
Set frames = 1
...whatever

Current Result
The CPU load jumps and the browser becomes irresponsive for some minutes
Browser and version
Chrome Version 62.0.3202.62 (Official Build) (64-bit)

Happening to me as well. OS: Also working with Ubuntu 16 (Abel's is 17) and Google Chrome.

@ernesmb has just tested it on Mac and it is working fine. It looks like it is a Ubuntu issue.

Hey @rochoa do you know what our browsers are crashing on Ubuntu? @ernesmb suggested that maybe it is because it is not using the GPU.

You should check chrome://gpu to see that everything is running as expected, so the bottleneck should be anywhere else

I couldn't reproduce the problem with Chrome 67, Ubuntu 16.04. @ramiroaznar can you check again in your machine?

Not happening anymore (but this has been an issue for me for months!). I couldn't reproduce it using any of these maps:
https://team.carto.com/u/ramirocartodb/builder/6d760df8-bb09-11e6-9938-0ee66e2c9693/embed
https://team.carto.com/u/ramirocartodb/builder/ab797f96-0a40-11e7-a19d-0e3ebc282e83/embed
https://team.carto.com/u/ramirocartodb/builder/8846ac54-154b-11e7-b277-0e3ff518bd15/embed
@AbelVM are you still experiencing this issue?

Closing this then.


Is the 'Terms of Use' feature still available?

I have signed into Azure with a user assigned a P2 License and the Global Administrator role and I have a PAYG subscription on the tenant. But I cannot find the 'Terms of Use' page neither through the Conditional Access Sub-menu nor the global search. Is this feature still available? Or perhaps it's been moved or renamed?
You also have this document which describes how to access it with a different set of conditions. I think you should get these pages aligned.
https://learn.microsoft.com/en-us/azure/active-directory/conditional-access/terms-of-use

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

ID: 30af6e66-d38a-a461-9b84-33b4e2c562ca
Version Independent ID: 259c3b11-817b-430d-7fc8-2cb8f7de5193
Content: Quickstart require Terms of Use at sign-in - Microsoft Entra
Content Source: articles/active-directory/conditional-access/require-tou.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

apparently this I can access this page in the context of intune. I'm not familiar with intune. Is it expected that the feature is only available there?

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

I checked again and it saw the option. But minutes later I came back and it disappeared again. I guess you have a bug.

I checked again and I saw the option. But minutes later I came back and it disappeared again. I guess you have a bug.

@fei0x Kindly check once again. I was able to locate 'Terms of use' . Browse to Azure Active Directory > Security > Conditional Access > Terms of use.


ColorManager saves value in an incorrect format

Sorry for bad English.
To be clear, I'm using the latest release version of the mod, and also the NuGet package for developer reference.
I'm working on a mod called NAL, and replaced internal Money, Weapons Inventory mechanism from NAL's own save file to PlayerCompanion. However, when I'm using
Companion.Colors.Current = Color.FromArgb(255, 0, 114, 188);

and then reload all scripts from console, Companion script crashes. When I looks into log, it shows Newtonsoft.Json caused the exception:
[09:12:16] [ERROR] Failed to instantiate script PlayerCompanion.Companion because constructor threw an exception: System.TypeInitializationException: The type initializer for 'PlayerCompanion.Companion' threw an exception. --->
Newtonsoft.Json.JsonReaderException: Error reading JObject from JsonReader. Current JsonReader item is not an object: String. Path '0x3FB5C3D3', line 1, position 27.
   at Newtonsoft.Json.Linq.JObject.Load(JsonReader reader, JsonLoadSettings settings)
   at PlayerCompanion.ColorConverter.ReadJson(JsonReader reader, Type objectType, Color existingValue, Boolean hasExistingValue, JsonSerializer serializer)
   at Newtonsoft.Json.JsonConverter`1.ReadJson(JsonReader reader, Type objectType, Object existingValue, JsonSerializer serializer)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.DeserializeConvertable(JsonConverter converter, JsonReader reader, Type objectType, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateDictionary(IDictionary dictionary, JsonReader reader, JsonDictionaryContract contract, JsonProperty containerProperty, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonConverter[] converters)
   at PlayerCompanion.ColorManager..ctor()
   at PlayerCompanion.Companion..cctor()
   --- End of inner exception stacktrace ---
   at PlayerCompanion.Companion..ctor()

When I checks the Colors.json, it looks like this:
{"0xEE79AF16":"0, 114, 188","0x3FB5C3D3":"0, 114, 188"}

It doesn't looks like the new format. I have to manually correct it to
{
    "0xEE79AF16": {
        "r": "0",
        "g": "114",
        "b": "188"
    }
}

to prevent the crash from happening. Some exception messages has been translated to help you better understand this issue.

When the colors are loaded, the custom converter is used.
https://github.com/justalemon/PlayerCompanion/blob/7b0cea5ee0f776280c8c9e2f203ee8dccff08a61/PlayerCompanion/ColorManager.cs#L125
When they are saved, the converter is not used.
https://github.com/justalemon/PlayerCompanion/blob/7b0cea5ee0f776280c8c9e2f203ee8dccff08a61/PlayerCompanion/ColorManager.cs#L163
Also, a quick warning: Setting the color of the user will permanently override the color that the user has set.  If you are trying to set it temporarily, then I should probably make Apply(Color) public.
https://github.com/justalemon/PlayerCompanion/blob/7b0cea5ee0f776280c8c9e2f203ee8dccff08a61/PlayerCompanion/ColorManager.cs#L165-L169


Update sr.js

So you're simplifying it, but it doesn't break any tests? Can you explain a bit. Also, you can remove the translator.translate function altogether.
@milan-j what do you think?

There is a reason for translator.translate function, because of the complicated grammatical cases is Serbian language. If it was that simple, than I wouldn't bother implementing it at the first place.
But he translated some of my code comments from English to Croatian, that is nice.

@milan-j sorry, what translator is translating this, and how are you sure it translates correctly?

OK, so this is no good. Also translating comments to Serbian makes them unreadable :)


Samples to Create Front Door with Rules Engine

It would be nice to add a sample template containing a Front Door resource with a routing rule that utilizes a Rules Engine resource in the same template.
The exported templates I've attempted to deploy all have circular references in them. This is because the Front Door routing rule depends on the Rules Engine resource, and the Rules Engine resource on the Front Door resource. It seems like it would be simpler if the Rules Engine resource were part of the Front Door resource properties so everything could be deployed together in a single step, but there might be something I'm missing here.
Thanks,
Kyle

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

ID: ea673aa4-1e06-c7a4-c86a-ab949e46394e
Version Independent ID: 7fbe7485-538d-7c0e-d219-41ead79fb38f
Content: Azure Resource Manager template samples - Azure Front Door
Content Source: articles/frontdoor/front-door-quickstart-template-samples.md
Service: frontdoor
GitHub Login: @duongau
Microsoft Alias: duau

@KylePolansky , thank you for your feedback. We'll review this and get back to you shortly.

@KylePolansky , as mentioned in https://github.com/MicrosoftDocs/azure-docs/issues/64756, this is a known issue that the FrontDoor PG team have also identified with the ARM template creation.
Right now, the following workout may work (depending on your setup for automating):

Create Frontdoor and Rules Engine Config FIRST, without having Front door reference the Rules Engine config
THEN, make another ARM template call, on the same Frontdoor, but added with the reference to rules engine.

Hopefully this helps – apologies for the bug here, AFD PG team is already working on the bug to get it fixed.
Please give the advised workaround a try and let us know if it helps.
We are closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

2 years have passed, any progress on this?

As workaround I first add the rules engine and then add the rule engine reference later.
I hope this problem is solved soon.

As workaround I first add the rules engine and then add the rule engine reference later. I hope this problem is solved soon.

can u please share your template file.


feature suggestion: Syntax Highlighter

I don't know if this is possible or not, but it would be great if a syntax highlighter plugin could be added to the editor, through which one could apply syntax lighting to codes in the editor.
Markdown has an internal marking mechanism (triple tilds). You are also able to even specify a programming language so that the code is highlighted regarding that.

Hi,
This feature will be added in version 3.0.0.
It will take time.

I am looking for that feature as well.

Any update on this feature? Or did somebody find a work-around with a plugin?


builder: add support for archiving special files on Unix

Added support for archiving character, block and FIFO files on Unix, this is useful when you want to take a whole system backup or re-archiving an extracted system image.
The file handling in this pull request isn't particularly graceful since I am not very experienced with the codebase, any suggestions are welcome!

This change is

Could this perhaps be refactored a bit though to avoid a few extra calls to fs::metadata being called? I think this could have an internal append function which takes a FileType from libstd and dispatches internally about how to append it? (e.g. more cases would be handled on Unix than on Windows, but regular files would all roughly be handled in the same place)

Sorry for the late reply. I am a bit confused by this suggestion: I saw the https://github.com/alexcrichton/tar-rs/blob/f07b3c15e398a76a627e77b49952d44c5d5880d8/src/builder.rs#L542 part so I assumed one more stat() should be fine here? https://github.com/alexcrichton/tar-rs/pull/246/files#diff-e4f794ca308aa712cbad31feb714df7cada1bbe453a30e232e45891571430263R607
Did you mean to refactor the append_dir_all function so that the stat could be avoided? Or should I do something about append_special function?

I mostly just got the feeling reading this that it'd be better to refactor where at some point there's the equivalent of a match on the FileType (or similar) of what to archive, and right now it seems spread across a few locations with perhaps an extra metadata call here or there.

I have made an attempt to reduce stat calls:

Ping? Is there update for this Pull Request?

Sorry I think this got lost in my inbox, but this looks good to me now!


Module '"astro:content"' has no exported member 'defineCollection'.ts (2305) and Module '"astro:content"' has no exported member 'z'.ts(2305)

I'm having a small error (Intermediate-Astro-v4-Decap-CMS) and I can't figure a way to resolve it with out just putting // @ts-ignore on it. In my content/blog/config.ts file, In my imports "import { defineCollection, z} from "astro:content";" I am getting 2 errors. 1) Module '"astro:content"' has no exported member 'defineCollection'.ts (2305) and 2) Module '"astro:content"' has no exported member 'z'. When I go to the .astro folder where types.d.ts is located I see modules of "astro:content"
import { defineCollection, z} from "astro:content";  // This line right here

// Every collection must reflect Decap's config.yml collection schema
// In order to be able to optimize images with Astro built-in compoments, like <Image />, we first must use this image helper
// Doc: https://docs.astro.build/en/guides/images/#images-in-content-collections

const blogsCollection = defineCollection({
	type: "content",
	schema: ({ image }) =>
		z.object({
			`title: z.string(),
			description: z.string(),
			author: z.string(),
			date: z.date(),
            tags: z.array(z.string()),
			image: image(),
			imageAlt: z.string(),
		}),
});

export const collections = {
	blog: blogsCollection,
};

When first opening the kit on the dev server, the .astro/ folder is created with a types.d.ts which is supposedly where the types for the 2 modules { defineCollection, z}  are generated. However, TypeScript is still upset with me.
I’ve tried resetting the TS server, reinstalling prettier, using npx astro sync, and have read even through the astro docs to find a solution https://docs.astro.build/en/guides/content-collections/ . I appreciate any help on this.

Hi @NHLopez,
Thanks for your feedback.
It should now be fixed with this commit
I'll close the issue, but feel free to reopen it if necessary.


Make the preset settings take a list[Path]

What new feature would you like to see?
In settings.py, there are a few preset directory settings like VASP_PRESET_DIR. We should support a list of directories so that users can use the as-provided presets and a custom directory of their choosing simultaneously.
https://github.com/Quantum-Accelerators/quacc/blob/76d85e1fc2308fa70ab47bc94c4b0791e1456f76/src/quacc/settings.py#L314-L317
https://github.com/Quantum-Accelerators/quacc/blob/76d85e1fc2308fa70ab47bc94c4b0791e1456f76/src/quacc/settings.py#L189-L192

Probably no major need for this now after #2127, but if someone wants to they can finish #2055.


Cut release v0.24.11

What's Changed

More shell examples (#3414)
Add actual argument type to error message (#3340)
Add a regression test for when engine returns a export fail (#3407)
Ability to set tolerances, but with sane defaults (#3397)
Disallow users to set theme as a project-level setting (#3312)
Fix files pane height issue (#3337)
The printer slicer expects mm (#3341)

Full Changelog: https://github.com/KittyCAD/modeling-app/compare/v0.24.10...v0.24.11

I smoke tested locally and it seems good


The extension name ".mjs" is better for dist/native-modules

The current extension '.js' does not work on<EMAIL_ADDRESS>Code
Dependencies:
yarn add esm --dev

The test.mjs
import * as cherow from 'cherow/dist/es2015/cherow'
// or native-modules
// or 2017

cli
node -r esm ./test.mjs

output:
Error: Cannot load ES module from .mjs: file: ......../cherow.js

I manually change the extension '.mjs', problem solved.
so,,, the '.mjs' is better for dist/native-modules

👍

I'll submit a first pass of some improvements on our current build system, and include a script that simply copies the .js files from these folders to .mjs. Having both will ensure that @achun's use case works, as well as prevent breaking changes for people using different import methods.

and See this https://jspm.io/
Load npm packages in modern browsers with the native browser ES module loader


Add a pass to set/unset encoding around compute operations.

This adds the pass that sets the encoding around compute operations,
currently only linalg.matmul. Apart from setting the encoding the
pass also introduces necessary padding. The hope is that the pad can
be propogated across operations to amortize the cost of the pad. The
set and unset encoding operations work on the padded sizes.
The follow up PRs will pull this into dispatches appropriately.  If
the pad isnt propagated then a pad -> set_encoding (and the
unset_encoding -> slice) operations will get pulled into the same
dispatch allowing for these operations to be folded into the
pack/unpack operations that the set/unset encoding get lowered to
in the backends.

This depends on https://reviews.llvm.org/D136493. Will cherry pick that separately before submit


Altibase: update replication object information #34045

Hello,
I am happy to ask a pull-request for #34045
I updated Altibase Replication's functionality, task, usability.

Display the replication gap.
Add Replication start/stop task.
Change Replication model and UI: AltibaseReplicationModule, AltibaseReplicationSender, AltibaseReplicationReceiver.

I also updated several UI icon & label for better usability (dbeaver/plugins/org.jkiss.dbeaver.ext.altibase/plugin.xml)

Global Metadata Icon Change
Storage Icon and Label, User and Role Icon Change: Revise the icons and labels to improve clarity and recognition.
Tablespace File Navigation Removal: Remove navigable options for tablespace files to prevent important tablespace size information from being obscured by less important file names. Display tablespace properties within the tablespace property panel instead.

New code was checked with Check Style to ensure compliance.
Thanks,

@zennken
Thank you for contribution! I reviewed your PR and left several comments.

@E1izabeth
Thanks for your quick comments on my code. I'm currently on summer vacation with my family, so I'll take care of it starting next Wednesday.
See you then :)

@E1izabeth
Thanks for your quick comments on my code. I'm currently on summer vacation with my family, so I'll take care of it starting next Wednesday.
See you then :)

I've committed the changes you recommended. Please check them :)

@NotNull / @Nullable annotations

I added annotations where I thought they were needed, as you suggested. Let me know if I missed anything :)

Passing to the QA team now.

Passing to the QA team now.

Always thanks for your review :)

Passing to the QA team now.

verified


'Error: Cannot open audio file' when testing Multiple Files using GUI

Hello,
Thanks for all your work on BirdNET and the GUI. I'm just getting started with training a custom model and running into problems. I can train a classifier ok, but when trying to test it on multiple files, it fails for most of the recordings. By which I mean, I get the error message in the title, followed by the path and filename of the recording: "Error: Cannot open audio file C:\Users\Tom\Folder\Recording.wav". Then once it has finished running, it states "false" in the "Execution" column beside the name of the unopened recording and I don't get a results file.
However, it doesn't fail for all recordings; in my test set of 11, it will run on 2 of them. Confusingly, it is not always the same 2 files. Each time I run the test, a different pair are analysed successfully. So, I don't think it's a permissions issue or something like that. Recordings are all .wav format, ~2 hrs 50 mins long, originally with a 32k sample rate, the names are solely alphanumeric characters.
Am I doing something daft? Can anyone suggest what I'm doing wrong?
Thanks

I have the same issue. Initially, I thought it was because I was trying to access a network drive. I copied the folder into a local drive and it didn't work. My next thought was the number of files (1-minute recording every 10 minutes for 4 months), so I organized the recording in folders per month. I was able to parse the first two months were analyzed, but then the issue came back. I don't know what is causing it because the error message seems generic. I'm using stereo files with a 48 kHz sample rate and 16-bit depth.

UPDATE.
@tombradferlawrence
In my case, it seems that the error was corrupted files. Ben Gottesman (Cornell Lab) told me that this might be the issue so I went to the folder and sorted the files out by size. I moved out the files that were abnormally small and it worked.  If your situation is the same, perhaps the fact that there were different recordings that triggered the error might be due to parallel processing. Were you using multiple threads?

Hi thank you for the update!
Could you provide us with a faulty file, for testing purposes?

Hi @panchorivasf, glad to hear that you have been able to resolve your issue. Unfortunately, I don't think that is the case with my situation. My files are all the same size, and I've used them for other tasks without any problems, (I can load them in R and Audacity etc.) I wondered if it might be because they are quite large? Each file is >600 Mb @Josef-Haupt could that be a problem?

I might have a similar problem: I have files of c. 2.5 GB (wave, 16 bit, 32kHz). I use the BirdNET Windows execute file, calling it from a batch file. For most of the files I receive errors (cannot read file). I process with multiple threads (2-4). I cannot detect a pattern in which files work and which don't. I record 1-hour files (to not loose a whole night when batteries run out) with Audiomoth v. 1.2. I merge them into one-night files in Goldwave, which is more practical in Raven.

I had a similar problem which I managed to resolve.
Some batches of files would run fine through the GUI, while others would generate an error and not run.
The issue turned out to be dot underscore files. Files that start with "." hold metadata and are generated when you open a file on a mac system (as far as I understand). Windows hides these files unless you expressly change the settings to make them visible. I had to change the view settings on my computer, find all the "." files hidden in my folders, delete them, and then the BirdNET GUI multiple file would run.
This is what the error looked like before it was resolved:
Traceback (most recent call last):
File "librosa\core\audio.py", line 743, in get_duration
File "soundfile.py", line 467, in info
File "soundfile.py", line 412, in init
File "soundfile.py", line 658, in init
File "soundfile.py", line 1216, in _open
soundfile.LibsndfileError: Error opening 'E:\Massaha-new-PA\TBNI21\20210719_Schedule [+00.6856+013.2189]\.S20210719T055952781+0100_E20210719T062947833+0100+00.6856+013.2189.wav': Format not recognised.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "gradio\routes.py", line 394, in run_predict
File "gradio\blocks.py", line 1075, in process_api
File "gradio\blocks.py", line 884, in call_function
File "anyio\to_thread.py", line 31, in run_sync
File "anyio_backends_asyncio.py", line 937, in run_sync_in_worker_thread
File "anyio_backends_asyncio.py", line 867, in run
File "gui.py", line 756, in select_directory_on_empty
File "gui.py", line 427, in select_directory
File "gui.py", line 428, in
File "librosa\util\decorators.py", line 88, in inner_f
File "librosa\core\audio.py", line 745, in get_duration
File "audioread_init_.py", line 132, in audio_open
audioread.exceptions.NoBackendError

@KalmiaLatifolia thanks for the insights! The metadata files could be an issue. I have previously used these files on a Mac, even though I was running BirdNET on a PC. I will have to check.


residual connection

In GAT/utils/layers.py, for residual, in the else statement, it is "seq_fts = ret + seq".
Shouldn't it be "ret =  ret + seq"?

Thank you very much for spotting this! Indeed, this went on undetected while porting the code for the public release.
It has been fixed now. :)


[Bug]  shuffle server have blocked threads

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Search before asking

[X] I have searched in the issues and found no similar issues.

Describe the bug
@jerqi @zuston
hi， when huge table join huge table， shuffle server have blocked threads ， Is it right?
server conf :
rss.rpc.server.port 20000
rss.jetty.http.port 20001
rss.storage.basePath /app/rss-0.7.1/data
rss.storage.type MEMORY_LOCALFILE_HDFS
rss.coordinator.quorum <IP_ADDRESS>:19999,<IP_ADDRESS>:19999,<IP_ADDRESS>:19999
rss.server.disk.capacity 50g
rss.server.flush.thread.alive 30
rss.server.flush.threadPool.size 10
rss.server.buffer.capacity 40g
rss.server.read.buffer.capacity 20g
rss.server.heartbeat.interval 10000
rss.rpc.message.max.size<PHONE_NUMBER>
rss.server.preAllocation.expired 120000
rss.server.commit.timeout 600000
rss.server.app.expired.withoutHeartbeat 120000
rss.server.flush.cold.storage.threshold.size 512m
rss client conf :
spark.shuffle.manager=org.apache.spark.shuffle.RssShuffleManager
spark.rss.coordinator.quorum=<IP_ADDRESS>:19999,<IP_ADDRESS>:19999,<IP_ADDRESS>:19999
spark.rss.storage.type=MEMORY_LOCALFILE_HDFS
spark.rss.remote.storage.path=hdfs://ns1/rss/sml

the executor have no daemon thread holding and hava no error log

Affects Version(s)
0.7.1
Uniffle Server Log Output
No response
Uniffle Engine Log Output
No response
Uniffle Server Configurations
No response
Uniffle Engine Configurations
No response
Additional context
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

What do you want to report? The app failed?

What do you want to report
1 . when huge table join huge table， shuffle server have blocked threads ， Is it right?
2 . the executor have no daemon thread and  holding ,  seem to hold on senddata to uniffle server , is my conf need to adjust ?
The app failed ?
after running 2 hours, seem no failed but executors holding , executor logs not update ,driver log only have uniffle heatbeat log

Do you have  performance tuning  about spark sql  huge table join ?

No. But I have tuning with the huge partition. Firstly, we should find out the root cause. Please tell what happened for you app.

No. But I have tuning with the huge partition. Firstly, we should find out the root cause. Please tell what happened for you app.

@zuston  thanks !!!  yes, you are right ,  I found that :
executor log  long time no update :

then I check the executor  stack ,  find there are no daemon threads  WAITTING, seem to holding on senddata:
https://user-images.githubusercontent.com/3837738/254862640-1baad065-8769-4092-82b6-5dbb24478167.png
then I analysis the shuffle server  stack , find there are  threads  BLOCKED
myapp process seem  no change,  But I can`t find out the root cause , Do you have some steps to help me ?

tuning with the huge partition
Do you have documents help me do this ?

Can you check the shuffle-server and executor GC? Why not using the spark ui? And I think if you want to analysis, it's better to show the metrics into dashboard.

Can you check the shuffle-server and executor GC? Why not using the spark ui? And I think if you want to analysis, it's better to show the metrics into dashboard.
1 . check the shuffle-server and executor GC ：
shuffle-server no full gc , but  jvm_memory_bytes_used is  close to XMX_SIZE="60g" ：

executor gc seem normal ：

using the spark ui ： dont`t exported the port  and need use vpn
metrics into dashboard : had show some metrics , feel shuffle server had too many blocked thread not normal ，but can`t find why

@zuston  How do you  tuning with the huge partition ？  Could you help me do this ?


Qualifiers in Spring #4347

Description 📜
The changes include documentation of  Qualifiers in Spring framework : the need for qualifiers and an example for the same.
Fixes #4347

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[x] Backend Dev (Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Still the same problem. Set base as Backend dev Java branch and create pr for that


Timelessness leaks when including Timestamps

If you include Mongoid::Timestamps, the model class has both Updated and Created, so when you set timeless, it increments the timer by 2, but only decrements by one. so each timeless call causes a future update to be timeless.

+1 on this issue.  I see this issue is added to the 5.0.0 milestone, but the issue exists in the 3.1.x versions of the gem as well.  Does that mean it won't be addressed for that series?


feat: support do without arguments (#51)

This pull request adds support for using Do without arguments, so that they can be provided by Return in mocks. It also extends mock.Map to merge multiple parameter sets.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

47 of 47 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1746

Relevant Lines:
1746

💛  - Coveralls


update useScaffoldEventSubsciber example

Description :
Updated the example of useScaffoldEventSubscriber with a new one, we forgot to update it
Also added a note at https://github.com/scaffold-eth/se2-docs/commit/6040216cf712134494dae4c6ab632c6442ebc8e0 for #18
Please feel free to suggest better explanation 😅🙌

Tysm all for the fixes and suggestions love it !!! Merging this 🙌


Create a new pull request by comparing changes across

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.412%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1966

Relevant Lines:
2020

💛  - Coveralls


[Feature Request] Playback loop from 2 specific frames

Would it be possible to add a section to the playback tab that allows you to give it 2 frame numbers and then loops playback from lowest specified frame to the highest one.
alternatively something like a check box to loop through selected frames only
I often use your program to create good loops it would be useful to be able to test out new start and end points without having to delete frames then undoing it and testing a new loop.
also may be asking too much but I would love to be able to see 2 frames side by side somehow
Thanks for the great program

Would that work if Playback->Play would just play the selected frames instead of displaying the whole? It does start from selected frame but after reaching the end it loops from the start.

Yeah that would make it a lot easier to test the best start and end points to make a good loop


Cannot get unread count badge on Dock icon until you request notifications for new items on some feed

In fixing #2058, the user is no longer prompted to allow notifications that show the unread count badge on the Dock icon. The only way to get this back is to select some feed, go to Window > Info in the menu, and enable "Notify About New Articles" to get the allow-notifications prompt.
IIRC the original behaviour was to just prompt for notification permissions on first launch.
One solution could be to add a "Show Unread Count On Dock Icon" setting to the preferences window that then prompts the user to allow notifications. This would decouple getting notifications about new articles from seeing the unread-count badge.

Since we show the unread count by default, this solution would only work if we change that behavior.
I thought I had this fixed in #2375.  @AngeloStavrow could you verify that it is or isn't fixed?

Sure, @vincode-io! I didn't see #2375 and while it seems to repopulate the NNW entry in the Notifications prefpane, it doesn't seem to fix a specific scenario — here's how I tested the issue on NNW 5.1d3 (3003):

Quit NetNewsWire if it's running
Open Settings > Notifications
Find NetNewsWire in the list and select it, then press Delete on your keyboard to remove the setting
Launch NetNewsWire and observe that the Dock icon does not show the unread count badge
Open NetNewsWire preferences and verify if Show unread count is enable
Toggle the Show unread count preference until it's enabled

Expected
Re-enabling the Show unread count preference prompts the user to allow notifications.
Actual
No prompt is presented and the unread count badge does not appear on the dock icon; the NetNewsWire entry returns to the Notifications pref pane, but it is set to Off.
Workaround
User must enable notifications for some feed via the web feed inspector to get the prompt to allow notifications.

I haven't dug into the code, but it looks like the issue is that checking/unchecking the Show unread count setting in NNW's preferences doesn't check to see if it's still authorized to send notifications.

I think it does make sense to prompt again if setting is off and we are trying to enable it.

@AngeloStavrow I'm assigning to you since you expressed an interest.


AdditionalVolumes to allow VolumeSource

What this PR does:
Adds the abiliity to make AdditionalVolumes take VolumeSource as input instead of only PVCSpec.
Breaks the API by making PVCSpec a pointer to struct instead of struct..
And allows the modification of AdditionalVolumes in the webhook
Which issue(s) this PR fixes:
Fixes #467
Fixes #486
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[x] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

Breaks the API by making PVCSpec a pointer to struct instead of struct..

This shouldn't be a problem for the yaml API right, because it just means that the PVCSpec becomes optional in the yaml? And a required -> optional change is always backwards compatible..?
I am a little more concerned about the impact it might have on the k8ssandra-operator side of things, since the move from a value -> pointer type will indeed break things (e.g. here). Do you have any suggestions on how to handle that?
I don't get the sense that we have many users accessing the APIs via the Go types, and given this is a beta API I don't think we have any obligation to maintain exact backwards compatibility even if we did (based on this). However, I think we should probably consider bumping the version to v1beta2 if we're going to make a breaking change, since this will signal the breaking change to users.
Finally, is there a reason that this field is better than accessing the podTemplateSpec directly? My understanding was that making these changes via PodTemplateSpec was becoming more accepted around the team.

This shouldn't be a problem for the yaml API right, because it just means that the PVCSpec becomes optional in the yaml? And a required -> optional change is always backwards compatible..?

It does not break YAML, that's true.

I am a little more concerned about the impact it might have on the k8ssandra-operator side of things, since the move from a value -> pointer type will indeed break things (e.g. here). Do you have any suggestions on how to handle that?

I have no idea why AdditionalVolumes is called "PVCs" in the K8ssandraCluster. That makes no sense. But why would it break that? The AdditionalVolumes struct hasn't changed type.

However, I think we should probably consider bumping the version to v1beta2 if we're going to make a breaking change, since this will signal the breaking change to users.

We haven't traditionally bumped to v1beta2 with other breaking changes either. That would require modifying webhook because that would change the YAML format also. And since YAML isn't breaking in this case, I don't think there's a reason to bump it to v1beta2. It only affects the Go API, but changing its package is not probably a very meaningful thing to do.

Finally, is there a reason that this field is better than accessing the podTemplateSpec directly? My understanding was that making these changes via PodTemplateSpec was becoming more accepted around the team.

cass-operator does not provide any PodTemplateSpec guarantees or backwards compatibility. We can and do break it (it's not passed as is and has never been) and have never recommended it. Configuring this type of Volume through PodTemplateSpec is cumbersome for users (requiring both volumes and volumesources). For YAML users it's often just a "hack" to get a missing feature.


Why to use  2D Ground-truth joints as input?

Thanks for the code and your work?
Why do you use  2D ground-truth joints as input for evaluation?
This is a lifting network not entirely 2D+3D pose estimation work like https://github.com/mks0601/3DMPPE_POSENET_RELEASE? Is that right?
Thanks
G.

Q: Why do you use 2D ground-truth joints as input for evaluation?
A: As stated in the readme, to keep the repository simple, 2D joint estimator is not included in this repo, you can use off-the-shelf 2D pose estimator. Also as explained in the paper, HRNet was used. 2D ground-truth joints were provided to facilitate the code release. It helps to demonstrate the upper bound performance of the model, where the 2D ground-truth joints are used as input to mimic the situation there is no error in 2D joints estimation.

Q: This is a lifting network not entirely 2D+3D pose estimation work like https://github.com/mks0601/3DMPPE_POSENET_RELEASE? Is that right?
A: A lifting network usually estimates a 3D pose based on a 2D pose from a single image. Our method is a monocular video method, which means it takes sequence (temporal window) of 2D poses as input to estimate the 3D pose center frame. As explained in last answer, you can use off-the-shelf 2D pose estimators to get estimated 2D joints. To make it easy to reproduce our results (Table 3) reported in the paper, we also provided the estimated 2D joints that can be downloaded along with the pre-trained models.

Thanks a lot for your answers, it helps a lot.
Many thanks
Gurkirt


Prototype causal analysis using Chirho

e.g. "what would our GDP be if we spend less $ on commerce?"

done


added known_good_state clean step call and its monkey patch

@cdearborn @rpioso @digambar15
Added call for known_good_state cleaning step in config_idrac.py which resets the iDRAC and clears the lifecycle controller job queue

Hey Chris, Thanks for the clarification, I am closing this PR now.


Mime dependency upgrade breaks ES5 support

Upgrading the mime dependency from version 1 to version 2 in superagent v3.6.1 breaks backward compatibility with ES5.
const Mime = require('./Mime');
^^^^^
SyntaxError: Use of const in strict mode.

I'd recommend upgrading the mime package to 1.4.1 instead of 2.0.3 to maintain backward compatibility for the minor version upgrade.

The browser version of mime doesn't seem to be affected. Our tests pass in IE, and we don't use any ES6 transpiler.
Superagent 3.x doesn't support Node 0.x. All ES5 versions of Node are officially dead since the end of 2016.

mime upgraded in major, but you don't. why? it is a breaking change

It didn't change interface of superagent, and didn't change node versions we support, so I don't think it's a breaking change.
We don't support Node 0.x, so it absolutely doesn't matter what happens to it.

You are wrong, your package works on node 0.10, but with upgrade to new mime it doesn't and it is a breaking change when your package is not compatible with old node any more.

Superagent 2.x works on Node 0.10.
Superagent 3.x was never meant to work on Node 0.10. We've intentionally bumped semver-major for this exact reason, and it's clearly documented. And yes, any Node module that uses Superagent 3.x requires Node 4 or later.
It's entirely your problem that your big project failed to migrate away from dead version of Node, given years of advanced notice that this will happen.

Didn't knew about 2->3 switch and its reason, my fault. Ok, thank you.

Superagent 2.x works on Node 0.10.
Superagent 3.x was never meant to work on Node 0.10.

@pornel For what it's worth I'm actually having trouble with the swagger-tools project. I'm just using swagger to document an API. I'm not a node developer so forgive any ignorance.
In the swagger-tools package.json it lists superagent as a dependency with ^2.0.0[1]. The NPM docs say that using a ^ prefix indicates "Compatible with version"[2].
Is it possible for you to mark superagent v3 as not compatible with v2 so other packages don't upgrade to this incompatible version? This should help maintain stability for other packages which haven't done the leg work to upgrade while still give you the freedom to make backward-incompatible changes.

https://github.com/apigee-127/swagger-tools/blob/master/package.json#L82
https://docs.npmjs.com/files/package.json#dependencies

^2.0.0 is not compatible with 3.x (it means 2.0.0 to 2.999…).

Do you happen know why it would be pulling 3.0 versions?

npm ls will tell you why

looks like it was a few dependencies deep (https://github.com/whitlockjc/path-loader/blob/master/package.json#L62)
Thanks for your help.