'suite1' gets appended to the Test Suite name.

I'm getting this as the report for a failing test.
1) Form List Pagesuite1 should have the right title:
Expected 'Forms | AwesomeForm' to equal 'Forms | MyAwesomeFormApp'.

See the 'suite1' string above?
This is my test code.
describe('Form List Page', () => {
    beforeAll(() => {
        browser.login()
        browser.url('/#/forms/list')
    })

    it('should have the right title', () => {
        const title = browser.getTitle()
        expect(title).toEqual('Forms | MyAwesomeFormApp')
    })
})

My test suite is simply named 'Form List Page' and yet it gets displayed as 'Form List Pagesuite1'.
Here are my dependencies used:
"wdio-jasmine-framework": "^0.3.1"
"wdio-spec-reporter": "^0.1.0"
"webdriverio": "^4.8.0"

node v8.3.1

+1


Add Native Histogram Support

Is your proposal related to a problem?
Experimental native histogram support was released with Prometheus v0.40.x. In https://github.com/thanos-io/thanos/pull/5896 we will update Thanos to Prometheus v0.40.1 but without implementing native histograms.
We should implement native histogram support (behind a feature flag) as a next step in follow up PR(s).
Describe the solution you'd like
Define how Thanos should handle native histograms and implement the feature behind a flag

Just double checking, I see we are adding native histogram support to query, receiver and sidecar in https://github.com/thanos-io/thanos/pull/6032.
For components like compactor, store gateway, do we still need implement anything to make sure they work with native histograms? If no code changes required, at least we should create some tests to make sure they work properly.

IIUC, only query frontend, Compactor and Ruler needs to support Native histograms?
I am wondering if we need to do anything specific in Ruler or it works out of the box by updating the Prometheus dependency. I guess the latter.

I also think it's the latter, but we will double check and add an e2e test for rulers.

Downsampling and compaction is blocked by a bug we hit in Prometheus regarding appending histograms to open chunks during compaction, which should be fixed by https://github.com/prometheus/prometheus/pull/1218.
The bug needs to be fixed before we can open a PR for compaction/downsampling of native histograms.

Ruler does not work with native histograms out of the box. promclient returns model representation of native histogram for queries, which we need to translate to histogram.FloatHistograms: https://github.com/thanos-io/thanos/blob/5d5d39a35ba62889aa759d60380fe43deee386e4/pkg/promclient/promclient.go#L483-L486
A solution can use the GRPC query API for the ruler so we directly get histogram.FloatHistograms in the query result.


Respond correctly to --killSignal (was: no logging after graceful shutdown)

Whenever I shutdown a script using --killSignal=SIGTERM (cf. #320), all the operations executed after running forever stop are not show in the logs anymore.
You can observe this problem on forever-monitor graceful exit example

run forever start --killSignal=SIGTERM graceful-exit.js
run forever stop graceful-exit.js
check the produced produced log. The sentence 'Exiting after some time.' should appear, but it does not.

Am I missing something?
This is not a really big deal, as at least these instructions are being executed (only the logging in forever log file is not performed).
This happens to me on node 0.8.17, and current master version of forever and forever-monitor.

+1

+1

+1 over here too. It looks like --KillSignal=SIGNAL will only send the process the specified SIGNAL when you do forever restart, but it will not when you do forever stop.
This actually looks "correct" behavior according the documentation:

--killSignal     Support exit signal customization (default is SIGKILL),
used for restarting script gracefully e.g. --killSignal=SIGTERM

However, it feels like we need a way to send a custom signal on forever stop.

I just upgraded forever to the latest version, 0.14.1, and it's sending the specified --KillSignal both on forever restart and forever stop. It's working ok.

Works for me too in 0.14.1 both on forever stop and forever restart.

Any workaround for console.log?


Docker's engine (network) doesn't support Oracle's 19+ Out Of Break bands

As pointed out by
https://franckpachot.medium.com/19c-instant-client-and-docker-1566630ab20e
and especially
https://github.com/oracle/docker-images/blob/main/OracleDatabase/SingleInstance/FAQ.md#ora-12637-packet-receive-failed
there can be an issue with Docker 19+ and Oracle 19+.
I encountered this with Debian 11 as a host, Docker version 20.10.10 and gvenzl/oracle-xe:21-full. Symptoms: connecting to the database inside the container via sqlplus works, but from outside (from the host) you only get timeouts and ORA-12637 or similar errors in the server log.
Solution: disable Out Of Bands feature by executing (inside the container)
echo DISABLE_OOB=ON >> $ORACLE_BASE_HOME/network/admin/sqlnet.ora
or similar. This can be done on the client side too according to above sources.

Instead of documenting this, the 21c+ images now have DISABLE_OOB=ON set.


Create a Draggable Color Picker

It can be basic. Just needs to change the stroke color. And it can be moved throughout the canvas/page. Has to be responsive. In smaller screens it has to adjust its size a bit

Please, assign to me


HAProxy: Table too wide

The Table needs more space than the screen offers. Maybe folding columns can help.

This looks like a regression. Maybe due to the latest translation additions or a change in the framework.
FWIW, I can't reproduce this on 17.1.6 with os-haproxy 1.14, see below.

I have also 1.14 and master installed on the test VM and both show the same result in Firefox (Linux).

The tables should auto-scale, this is a framework regression in master then.

@fraenki should we close this then if it's upstream?

Please test with different browsers.

This is vivaldi:

I was rather thinking of some more common browsers, like Firefox or Chrome.

Firefox is the screenshot on the top and vivaldi is chromium based so the results are the same.

Well, we need to check the corresponding files in core to see if a recent change broke this.

all JS lib stuff has not been changed since march 4th in core.

just if this can be the reason: my screen resolution is 1600x900

Never seen this, even on higher screen resolutions.

round the root of the issue: your table is not initialized if haproxy is not running. After enabling it runs in both browsers as expected.
Enabling the server was not required for my translation work where I found it ;)
Handle it like you wish.


More protocol safety improvements

This is a change I've been noodling on for a bit related to https://github.com/rust-osdev/uefi-rs/issues/359. The basic issue is that in order to use protocols fully safely, with no chance of either the handle or protocol being modified or removed while in use, we need to ensure that protocols are opened in exclusive mode with the correct agent handle set. We've made some good steps towards that by deprecating and marking unsafe things like handle_protocol, but there are still unsafe ways to use open_protocol.
To resolve this (to the extent possible given the design of UEFI), I've added a new open_protocol_exclusive function. This function is fully safe; it ensures that the correct agent is set through the use of a new global image handle set by the #[entry] macro, and always opens the protocol in exclusive mode. This is a much nicer interface to use than open_protocol, since you only have to provide the Handle you want to open.
There are still some cases where the full power of open_protocol is needed; uefi-test-runner has some examples of this. For those cases though the caller needs to be aware that they are responsible for ensuring that the handle and protocol are valid until the protocol is no longer in use, so I've marked open_protocol as unsafe. Since most uses can be replaced with open_protocol_exclusive, this is hopefully not too much of a burden.
Having a global image handle and marking open_protocol as unsafe are fairly significant changes to the API (although notably we haven't done a release yet that includes https://github.com/rust-osdev/uefi-rs/pull/434, so users of open_protocol will already have to make some code adjustments when they next upgrade), but I think it will provide a nice boost in both safety and ease of use.

No worries at all, thanks for reviewing :)


Support for BLE motion sensor (Mi Motion Activated Night Light 2)

Device model
yeelink.light.nl1
Device name
Mi Motion Activated Night Light 2
Anything special about the device?
I tried to add this sensor to my fork https://github.com/merdok/homebridge-miot/blob/e1419b6cd287766879e3050b7fb518f163f7bbc6/lib/modules/contactsensor/devices/yeelink.light.nl1.js
and unfortunately it doesn't work

Am i do something wrong? @merdok
Config
IP and Token - BLE gateway
"devices": [
        {
            "name": "Bedroom Night light",
            "ip": "<IP_ADDRESS>",
            "token": "6b654a53506972304c35636142765832",
            "deviceId": "blt.1.1d0ps0fkt4k00",
            "model": "yeelink.light.nl1",
            "pollingInterval": 10,
            "deepDebugLog": false,
            "onlyMainService": true,
            "buzzerControl": false,
            "ledControl": false,
            "childLockControl": false,
            "modeControl": false
        }
    ]

That is odd, what exactly does not work? Did you try miot cli to directly try to control the device?

It should recognize as motion sensor, but showing as "not supported"

When you look at the spec then you will see that this device does not expose any meaningful properties which can be used to control it:  http://www.merdok.org/miotspec/

Here is Mi Motion Activated Night Light 2
https://www.merdok.org/miotspec/?model=yeelink.light.nl1
And there's Mi Motion Sensor
https://www.merdok.org/miotspec/?model=lumi.sensor_motion.v2
Both of them have events "motion-sensor:motion-detected"
So Mi Motion Sensor is displayed in room page:

Is it possible to display here Mi Motion Activated Night Light 2 as well ?

The motion sensor has additionally the motion-sensor:motion-state property which can be used.
The night light does not have that property. And as already mentioned events currently does not work.

Got it, thank for your help


Typo in documentation

In JAVA-HOWTO:

For example, you can encrypt the key material with a Google Cloud KMS key at gcp-kms:/projects/tink-examples/locations/global/keyRings/foo/cryptoKeys/bar as follows:

Should be

For example, you can encrypt the key material with a Google Cloud KMS key at gcp-kms://projects/tink-examples/locations/global/keyRings/foo/cryptoKeys/bar as follows:

with
gcp-kms://

Thanks for the report, this should be fixed now!


[AMORO-1875]Support display paimon files in 'files' page

Why are the changes needed?
Close #1875.
Brief change log

Implement 'getTablePartition' and 'getTableFile' method of PaimonTableDescriptor

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

Should we add some unit tests for this section?

Will add in #2119

Cloud you like to update the screenshot

It is necessary to display the Commit ID-related information. If it cannot be obtained, it can be displayed as empty.


[18.03 backport] Cleanup the cluster provider when the agent is closed

backport of https://github.com/docker/libnetwork/pull/2307 for the 18.03 branch

ping @mavenugo @euanh PTAL

looks like TestNetworkDBIslands failing consistently ( Same failure on other backport too )

--- FAIL: TestNetworkDBIslands (75.83s)
	Location:	networkdb_test.go:838
	Error:		"map[4ac5b1d46794:4ac5b1d46794 79fd1a707bd6:79fd1a707bd6]" should have 3 item(s), but has 2

	Location:	networkdb_test.go:839
	Error:		"map[2624cc77ef1d:2624cc77ef1d]" should have 0 item(s), but has 1

FAIL

Looks like it may be flaky https://github.com/docker/libnetwork/pull/2240#issuecomment-407822195

Don't have permissions to trigger CI in this repo 😕

Like I mentioned , I did see same error failure in my PRs few times in the past.
I think we can merge it adding a note on this test case.

ping @mavenugo @euanh !


Usemeshkitfunc

Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote  Tests are failing because of old go version. Are secrets updated here to reflect go 1.17?

@Revolyssup, yes, this repo is using the org-level secret for GO_VERSION, which was last updated 11 days ago.

GitHub workflows seem to have a terribly long cache, though - https://github.com/meshery/meshery-istio/runs/4578792156?check_suite_focus=true#step:3:6


no more numba.testmod?

I am trying to let doctest work in numba wrappered functions. But now numba.testmod no longer exists and no document talks about it.
What happened?

We're going to take a look at the doctest issue you raised in #190, but testmod was removed a long time ago when Numba stopped using nose to run doctests.  (And in general, we've moved away from doctests as we find them not good for documentation or testing of Numba.)

Hello @seibert
Thank you for the response. I understand that it's hard to make Numba compatible with doctest (by doctest.testmod()) as the latter uses inspect.isfunction to detect if one is a function and should be tested.
In my opinion, it is better to add a section into the document describing this compatibility problem and how to solve it. Such a description appeared before (http://numba.pydata.org/numba-doc/0.12.1/doctest.html) but disappeared now.
According to https://docs.python.org/2/library/doctest.html

In addition, if M.test exists and “is true”, it must be a dict, and each entry maps a (string) name to a function object, class object, or string. Function and class object docstrings found from M.test are searched, and strings are treated as if they were docstrings. In output, a key K in M.test appears with name

So the solution is simple. Since a Numba-wrappered function still has doc. One just needs to make a dict named test and add a function's name as key and its doc as value. Then doctest module can understand there are more functions to test.
This solution can simply be described in the document.

Would it make sense to create a default __test__ automatically for every Numba function that has a docstring?  The user could still override it after defining the function.

Actually, more information: It appears that Python 3.6 does discover doctests on Numba-decorated functions, but Python 2.7 does not.  (I have not checked any versions between those two.)

@seibert to create a default test automatically for every Numba function seems a very nice idea to me!

Maybe it is worth a separate issue, but I wanted to comment here first as it is related.
I noticed that doctest discovers functions wrapped with njit, but none with guvectorize. Do you know why?

Python 2.7 reached EOL on January 1st, 2020 and Numba 0.47 was the last release to support Python 2.7. Closing this issue as it is Python 2.7 specific, if this problem is reproducible on Python 3 with Numba >= 0.48 then please reopen it. Many thanks!

Hi @stuartarchibald, it seems that the underlying problem is still present in newer versions of Numba. Doctest discovers functions wrapped with @njit but fails with @guvectorize.
from numba import guvectorize, int64, njit

@guvectorize([(int64[:], int64, int64[:])], '(n),()->(n)')
def bar(x, y, res):
    """
    Examples
    --------
    >>> x = 1
    """
    for i in range(x.shape[0]):
            res[i] = x[i] + y

@njit
def foo():
    """
    Examples
    --------
    >>> x = 1
    """
    pass

pytest -rs -sv -v --doctest-modules test_foo.py

@guilhermeleobas please could you open a new ticket as this issue seems to be specific to @guvectorize? Thanks.


POL-352-Google-Schedule-Instance

Description
POL-352-Google-Schedule-Instance
Issues Resolved
[List any existing issues this PR resolves]
Contribution Check List

[ ] All tests pass.
[ ] New functionality includes testing.
[ ] New functionality has been documented in the README if applicable
[ ] New functionality has been documented in CHANGELOG.MD

          1 Error

:no_entry_sign:
cost/google/schedule_instance/README.md:39: MD032 Lists should be surrounded by blank lines
A detailed description of the rules is available at https://github.com/markdownlint/markdownlint/blob/master/docs/RULES.md

  Generated by :no_entry_sign: Danger


Created broker_id attribute for cookbook to generate even if broker i…

…d is dynamically assigned
Its very helpful for our operations to be able to easily translate broker id to broker host. This was done previously by looking at chef and the node['kafka']['server.properties']['broker.id'] value. If we move to the dynamic assignment which would allow us remove the list of brokers in our chef config, this property won't be provided anymore. This change provides an attribute node['kafka']['broker_id'] that should always have the broker id value even if the id is dynamically assigned.

+1

+1

Have enough +1s, merging


Fix node path for building the unified console

Maven installs a particular version of node by itself in the generate-resources phase, and is supposed to use the installed node for subsequent build.

looks good to me, thank you for fixing that.


Make sure that the server_config and the server_pre_start fixture execute in the right order (#2801)

Pull request opened by the merge tool on behalf of #2801

Merged into branches iso4 in 9bcb4a9d8bdc3bb4ca31a2cb69d519e95dcafe23


tls1.3-draft-18 Branch nginx error tls_parse_ctos_psk:internal error

I compiled the tls1.3-draft-18 Branch against nginx and use TLS13-* Ciphersuites.
Now whenever I visit my website and try to access a non existent directory or file (So it gives me a 404 error), in the nginx error log it says:
 2017/03/18 12:54:54 [crit] 29903#29903: *5 SSL_do_handshake() failed (SSL: error:141F9044:SSL routines:tls_parse_ctos_psk:internal error) while SSL handshaking, client: <IP_ADDRESS>, server: <IP_ADDRESS>:443 and my site is no longer accessable due to openssl crash.
Any ideas?
Thanks

not without a backtrace, no.
does nginx work with openssl 1.1.0?
is it CRASHING or failinlg to do the handshake?
Are you sure that the browser is using tls1.3-draft18?

What client are you using to connect? Does that client manage to complete the handshake if you just connect to s_server?

Bug!! I have recreated this issue in s_server/s_client. The problem was introduced by 6b1bb98f. You get this problem if you are trying to resume and the server issues a HelloRetryRequest.

@mattcaswell That's right. I used Google Chrome for visiting my Website over TLS 1.3 and the error appeared: ERR_SSL_PROTOCOL_ERROR, when I tried to revisit it after a while.

Patch for this is in #2996

It is interesting that we generate an HRR at all in nginx with Google Chrome. A default OpenSSL will not do this. It appears that nginx is disabling X25519 support. This will have a performance penalty for nginx and TLSv1.3 when talking to Chrome because Chrome only ever sends an X25519 key_share in its initial ClientHello. This forces a HelloRetryRequest - which obviously means an extra server round trip.

I don't know if this is why, but for the X25519 Curve to be used in a connection, The ECDHE string in the ciphersuite-string must be given, otherwise OpenSSL refuses to use that curve. On TLS13-* suites, this is not the case.

I'm not sure why this would surprise you. ECDHE are the cipher suites that use ECC key exchange. In TLS1.3 the key exchange is no longer part of the cipher suite.

Should be fixed now.


Added noLockPointer config to InputSpotlightDecorator

Input component covers a whole screen except an input field with a scrim to block any user interaction during input, and there is no option to disable this feature.
This PR adds an option to disable pointer lock to support various UX requirements.
Ref: PLAT-113698, PLAT-87249 (related)
Enact-DCO-1.0-Signed-off-by: Seungcheon Baek<EMAIL_ADDRESS>

:shipit:


"openApiToCSharpClient" is not taking care of property name casing.

I have a JSON spec created using curl -k API-URL/swagger/docs/rest -o JwtSpec.json and there, one of the generated class has two properties with the same name but different casing.
"MyClass": {
          "M": {
            "type": "string",
            "nullable": true
          },
          "m": {
            "type": "string",
            "nullable": true
          }
}

And then, I'm trying to create csClient using openApiToCSharpClient with "propertyNameGeneratorType": "Original" but the generated c-sharp class having both properties in upper-case("M").
I'm not sure what I'm missing here, Please suggest how can I have the same naming and same casing in generated c-sharp classes that I have in Json Spec.
Below are the generated c-sharp properties :-
        [Newtonsoft.Json.JsonProperty("M", Required = Newtonsoft.Json.Required.Default, NullValueHandling = Newtonsoft.Json.NullValueHandling.Ignore)]
        public string M { get; set; }

        [Newtonsoft.Json.JsonProperty("m", Required = Newtonsoft.Json.Required.Default, NullValueHandling = Newtonsoft.Json.NullValueHandling.Ignore)]
        public string M { get; set; }

@RicoSuter


Changed flake8 mirror in pre-commit and updated version

Summary of changes

[x] .pre-commit-config.yaml

Changed flake8 mirror in pre-commit from GitLab to GitHub and updated to the latest version. This is because the GitLab mirror is not updated anymore and the link was temporarily down (causing pre-commit initialisation errors).

[x] Closes #9455
[x] Tests added / passed
[x] Passes pre-commit run --all-files

Can one of the admins verify this patch?
Admins can comment ok to test to allow this one PR to run or add to allowlist to allow all future PRs from the same author to run.

Thanks for this PR!


Some more AVM2 fixes for issues surfaced in the Tamarin test suite

?????

Reviewed files:

src/avm2/errors.ts @ r1
src/avm2/nat.ts @ r1
src/avm2/natives/proxy.ts @ r1
src/base/dataBuffer.ts @ r1
test/test_avm2_acceptance.baseline @ r1

Comments from the review on Reviewable.io


Fix warning from incorrect SimpleSelect usage in tests of DataTable component

Description
This happened due to using defaultValue to specify the placeholder in SimpleSelect as React doesn't like this. This PR tries to come with a workaround for this by using the value to either specify the selected value or the placeholder (if no option is selected).
Related JIRA Issue(s)
(ENSWBSITES-1828)[https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-1828]
Deployment URL(s)
http://fix-simple-select.review.ensembl.org
Views affected
BLAST settings
Genome browser -> Navigation panel -> Go to new location

There was another error that popped up due to value being specified without onChange in SimpleSelect. The fix for this seems a bit hacky, but couldn't think of anything better.


Homebridge UI crashes whenever trying to edit the plugin

Describe the bug
Whenever I try to add inputs in the UI, the homebridge UI crashes. I'm not quite sure what's up. My server itself doesn't crash, just the webpage for my server.
Expected behavior
Open homebridge UI > Accessories > Homebridge ABP > Inputs > Type information
Actual Behavior
Open homebridge UI > Accessories > Homebridge ABP > Inputs > Crash
Log output
NA it is working fine
Device (please complete the following information):

Name: Raspberry PI 4B 8GB running on an SSD

Additional context
Add any other context about the problem here.

Update, I was using chrome. Tried it in safari and it works, so it must be some weird interplay between chrome, UI, and/or ABP

It looks like it's Homebridge UI X bugs. But I made some changes on how the plugins interact with Hombridge UI X, maybe you can try if it's working now.


[Bug]: Transforming kebab-case key names to camelCase does not seem to work

What happened?
In the source tokens there's a few keys in kebab-case, which we'd like to convert to camelCase in the output.
I believe the documentation says that it's a matter of setting the casing option to camel, but this does not seem to work.
I've also tried to use a custom name transformation that detects and converts the name, but maybe I'm missing something obvious or small.
Reproduction
The transforms and configuration that I use:
StyleDictionary.registerTransform({
  name: "my/kebabToCamelCase",
  type: "name",
  transitive: true,
  matcher: (token) => {
    return token.name.includes("-");
  },
  transformer: (token) => {
    return token.name.replace(/-./g, (x) => x[1].toUpperCase());
  },
});

registerTransforms(StyleDictionary, {
  casing: "camel",
});

// Register custom tokens-studio transform group
// https://github.com/tokens-studio/sd-transforms?tab=readme-ov-file#custom-transform-group
StyleDictionary.registerTransformGroup({
  name: "custom/react-native",
  transforms: [...transforms, "my/kebabToCamelCase"],
});

Using the power of console.log in the matcher and transformer functions, I can see that keys like text-case, border-radius andsoforth are converted to textCase and borderRadius, however, the output JSON still retains the existing kebab-case:
Input exertpt:
    "typography": {
      "text-case": {
        "none": {
          "value": "none",
          "type": "textCase"
        },
        "uppercase": {
          "value": "uppercase",
          "type": "textCase"
        },
        "lowercase": {
          "value": "lowercase",
          "type": "textCase"
        },
        "capitalize": {
          "value": "capitalize",
          "type": "textCase"
        }
      },
      "text-decoration": {
        "none": {
          "value": "none",
          "type": "textDecoration"
        },
        "underline": {
          "value": "underline",
          "type": "textDecoration"
        },
        "line-through": {
          "value": "line-through",
          "type": "textDecoration"
        }
      }
    }

Output exerpt:
    "typography": {
      "text-case": {
        "none": "none",
        "uppercase": "uppercase",
        "lowercase": "lowercase",
        "capitalize": "capitalize"
      },
      "text-decoration": {
        "none": "none",
        "underline": "underline",
        "line-through": "line-through"
      }
    }

Expected output
    "typography": {
      "textCase": {
        "none": "none",
        "uppercase": "uppercase",
        "lowercase": "lowercase",
        "capitalize": "capitalize"
      },
      "textDecoration": {
        "none": "none",
        "underline": "underline",
        "lineThrough": "line-through"
      }
    }

Version
0.11.10

What format are you using for the output? javascript/object perhaps?
Some of the formats seem to just JSON.stringify() the dictionary and not take into account the name of the token at all, so even if name transforms apply, the output won't care about them.
This isn't a sd-transforms issue though but rather a style-dictionary issue, could you create an issue there? I think the solution is to give the JSON.stringify methods a replacer function that alters the keys to the token name if such a prop exists. Preferably for all of the formats that use this type of approach for writing the output. It would be a breaking change but since I'm working on v4 I'd be down to include it in v4

Closed due to inactivity and no follow up on the SD repo.


[Tech]: Add "--enable-beefy" flag that enables beefy for node and disable it by default

Problem statement
Now BEEFY enabled by default and it can cause some problems with parachain deployment
Description
No response
Definition of Done

Add flag --enable-beefy to parachain CLI that will enable beefy
add conditin to enable BEEFY if flag is give to CLI

Requirements
No response

This should not be tested by QA team


Fixed reversed logic and typo in InputManager.cs

Fixes https://github.com/Microsoft/HoloToolkit-Unity/issues/366

Thanks!


KeyError: 'USERPROFILE'

My code:
from sketchpy import library as lib

Error:
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
[<ipython-input-3-504f6cc831a4>](https://localhost:8080/#) in <cell line: 1>()
----> 1 from sketchpy import library as lib

1 frames
[/usr/local/lib/python3.10/dist-packages/sketchpy/library.py](https://localhost:8080/#) in <module>
    226
    227 path = os.path.join(
--> 228     os.environ["USERPROFILE"], "AppData", "Local", "Programs", "Python", "data.txt")
    229 def log(mode, data = ''):
    230     try:

[/usr/lib/python3.10/os.py](https://localhost:8080/#) in __getitem__(self, key)
    678         except KeyError:
    679             # raise KeyError with the original key value
--> 680             raise KeyError(key) from None
    681         return self.decodevalue(value)
    682

KeyError: 'USERPROFILE'

I'm getting the same error on Mac, but it's working on Windows. Are you a Mac user ?

I'm getting the same error on Mac, but it's working on Windows. Are you a Mac user ?

Mac and Linux (Ubuntu)

I'm using POP!_OS, a Linux's version, and I have the same problem!

will update this error on the upcoming version.


Tooltips should prefer to render up above items on mobile

Flutter prefers to put tooltips below items on mobile, matching desktop placement.  On mobile this means placing the tooltip in a location that is obscured by the users finger.
steps to reproduce

open the gallery app
go to Material -> Tooltips
Long press your finger on the magnifying glass button, notice that you cannot read the tooltip because it renders underneath your finger.

Sounds like a reasonable suggestion. We might have to make it opt-in since it would probably break golden image tests.

I labeled this P3, not because it's not important, only because we don't have plans to address the issue in the near term.

Related: #25478
They get at the same issue of the tooltip being obscured by the user's finger.


Diffrent generation in app.UseSwaggerUiWithApiExplorer(..); vs app.UseSwaggerUi(..)

If you have an action like
[HttpGet("[action]")]
public async Task<string> Test([FromHeader] string apa)
{
           ...
}

"apa" parameters get required in UseSwaggerUiWithApiExplorer
and optional in UseSwaggerUi (correct)
I belive the error is in OperationParameterProcessor.IsNullable because Nullable.GetUnderlyingType(typeof(string)) == null

The behavior has changed slightly (reverted) in v.18.19.0, are you using that?
Related https://github.com/RSuter/NSwag/issues/1556
/cc @pranavkm

We are using NSwag.AspNetCore v11.19.0

I think if you set it to null Test([FromHeader] string apa = null) it is optional...

Nope still required

This line probably has to be removed:
https://github.com/RSuter/NSwag/blob/master/src/NSwag.SwaggerGeneration.AspNetCore/Processors/OperationParameterProcessor.cs#L118
Then this will take effect:
https://github.com/RSuter/NSwag/blob/master/src/NSwag.SwaggerGeneration.AspNetCore/Processors/OperationParameterProcessor.cs#L344

This line probably has to be removed:
https://github.com/RSuter/NSwag/blob/master/src/NSwag.SwaggerGeneration.AspNetCore/Processors/OperationParameterProcessor.cs#L118

That looks correct.

@Chrille79  Is this ok?
    public ActionResult MyAction([FromHeader] string required, [FromHeader] string optional = null)

 public ActionResult MyAction([FromHeader] string required, [FromHeader] string optional = null)
Results in diffrent rendering between UseSwaggerUiWithApiExplorer and UseSwaggerUi.
Maybe a better way is
 public ActionResult MyAction([Required][FromHeader] string required, [FromHeader] string optional)
Then you get the same rendering for both UseSwaggerUiWithApiExplorer and UseSwaggerUi.

The main problem here is that this information should be (correctly) provided by API Explored and not be implemented in NSwag at all.
I think there is no way that UseSwaggerUiWithApiExplorer (AspNetCoreToSwaggerGenerator) and UseSwaggerUi (WebApiToSwaggerGenerator) will yield the same results (mainly because UseSwaggerUi may be implemented in a wrong way).
UseSwaggerUi will be deprecated eventually and for ASP.NET Core you should only use UseSwaggerUiWithApiExplorer() + we need to provide the best experience there...
/cc @pranavkm @rynowak

I see, but both
public ActionResult MyAction([FromHeader] string required, [FromHeader] string optional = null) are optional in asp.net core MVC viewpoint. But the RequiredAttribute does make it required!?

I think API Explorer does not process the RequiredAttribute for params...


fix(templateloader): search for files using windows or unix paths

Issue #207 #408
Fixes an issue in the template loader that causes template loading to fail on Windows
Changes

changes the regex for all usage of TemplateLoader.loadFileContents to allow path separators for either unix or Windows
This also renames some folders that contain reserved characters on windows systems
I backed away from running a full build on Windows as this looks like it will need many more changes and should be done separately

Related Issues

Issue #207
Issue #408

Coverage remained the same at 95.221% when pulling 1f8bb26f87b3a6d0377503185848506400f315ed on mttrbrts:mr-fix-207-408 into 74fd9cb0401fc6654841892c83f82eeef22e932f on accordproject:master.

Looks good. A few questions:

I can't see the difference in the file paths
Does it entirely address the two listed issues ( #207 #408 ) or just related?

I can't see the difference in the file paths
The folders formatted-dates-D MMM YYYY HH:mm:ss.SSSZ are renamed to formatted-dates-D MMM YYYY HHmmss.SSSZ because : is a reserved character on Windows

If I take models as an example, on Windows we need to search for files like models\name.cto, and on unix/osx we need to search for models/name.cto. The changed regex captures both of these patterns.

Does it entirely address the two listed issues ( #207 #408 ) or just related?
#207 Isn't fully addressed
#408 Is fully addressed, but I can't automate the test until we add a Windows build step which I will do separately.


Extract preprocessor module

A start at breaking down the parser module in order to make it easier to understand.

@lpil this is looking really good!  Sorry I've been MIA, just got back into town.

Happy to merge?

Go for it! :+1:

Thanks again for all your contributions @lpil.  You've made some big changes! :grinning:

:D Thanks!


[4.0.beta17] Material with missing texture is broken (inspect/use)

Godot version
4.0.beta17.mono.official
System information
Windows 10
Issue description
In the attached example just the texture assigned to the material is missing, but this already breaks godot form doing anything with the file; the inspector doesn't want to inspect it, asignments to other locations don't work - the file is basically useless until manually fixed, which is only possible by editing the file-text directly.
Steps to reproduce
Try to open/inspect the only available material in the attached project (double click)
Minimal reproduction project
BrokenMaterialMRP.zip

What you described is exactly what the issue is. I purposefully removed the material in the MRP to show this issue ofcourse.
A material with a missing assignment should not be broken (emphasis).
this might make sense on a technical developer level, but not for an end-user/game developer, and especially not in such a trivially invalid situation. The material should just display without the missing reference to give users a chance to fix it without having to resort to manually editing files they are not expected to under normal circumstances. This is what other engines do too (unity doesn't even log errors about missing textures in materials and it's definitely not a problem at all, after all materials especially are inherently visually validatable).
Scenes are a good example of the (better) way to deal with this, they have this reference fixer tool popup when necessary, but are (mostly) still openable even if chosen not to be fixed. I don't think every reference owning asset should have this fixer-tool, but it's not a good design decision to just leave the user with broken project assets that otherwise have a trivial path to be fixed by that user if the communication is there (which the error represents)


GT06 protocol issue

Hey guys,
I m new to GT06 device. Earlier i have worked on tk103 device.
When i installed the device to send data to server.
first it sends the login message and then the heartbeat only.
Even after sending the heartbeat response there is no message from device regarding location.
Also there is one message coming with this protocol number "0x8B" what does it mean.
And if someone is having detailed manual for Gt06 please share.

Protocol number 0x8B probably means response to some command. Some devices don't send data when they don't have a GPS fix or when they are not moving.

http://www.iconcox.com/uploads/soft/130627/1-13062F94158.pdf
http://support.sathyainfo.com/kb/a76/gps-vehicle-tracker-gt06n-command-list.aspx


followup to incorporate review comments per previous reviews on PR#29561

What I did
Incorporated review comments
Description for the changelog
Glossary updates for Docker for Mac and Windows, copy-on-write, and union filesystem

@thaJeztah @mstanleyjones these are the changes that didn't make it in before PR #29561 got merged, but now I am getting conflicts that don't show up when I fetch and rebase. @mstanleyjones let's look at this together tomorrow.

Conflict occurs, rebase need, I am afraid @londoncalling  😄

@thaJeztah @mstanleyjones @allencloud this has more changes per review comments, and merge conflicts are now resolved. If it looks ok to you guys, please merge for me. Thanks!

@thaJeztah @mstanleyjones thanks for the reviews!
Please have a look, and merge if you think all comments are addressed properly and it looks ok.

@londoncalling changes look good now, can you squash your commits?

@thaJeztah how's that?


Networks doesnt work.

I own a server and currently I'm using FTB Revelation v1.5 and FluxNetworks-1.12.2-3.0.3 so the problem is everytime i restart my server everyone lose their networks. Is there any known fix?

@Explosivee could you possibly give me your startup logs when this happens?

StartUp.log
Sure

@Explosivee Thank you. Could you send me your other logs also?

2018-02-08-1.log
2018-02-08-2.log
Uploading 2018-02-08-3.log…
Uploading 2018-02-08-4.log…
Uploading 2018-02-08-5.log…
Uploading 2018-02-08-6.log…
Uploading 2018-02-08-7.log…
Sorry for late man, Here it is

Plus, if people leave 10-15 sec ago before restart they wont lose their flux networks.


ome data format import

Dear Tischi,
thanks for the nice tool. I was wondering is you can create an import an ome.xml file created from a  Viventis microscope.
I attach the ome file here. The tiff files are multi-page -Z stacks one file per time point and channel.
ome-tiff.companion.zip
Thanks
Antonio

What happens if you open this in ImageJ?
Could I also get the images, such that I can test?

Yes I will make you owncloud folder for it. Bioformat opens it as imagej virtual stack. In fact it could also be useful to have a function like 'open current image in BigDataTools'. With this you avoid creating a lot of porting between formats and still use the virtual stacks.

like 'open current image in BigDataTools'. With this you avoid creating a lot of porting between formats and still use the virtual stacks.

Yes, you are right. In fact, we started working on this. I need to try to make this work.

Would you have some time to actually contribute to this in terms of Java code? I could point you to the pieces of code.

Yes I could work on it. It is also a good occasion to learn some new classes :-)

To get into the topic I recommend reading this thread:

https://forum.image.sc/t/wrap-imageplus-virtualstack-into-imglib2/4154/31

This code is interesting:

https://github.com/imglib/imglib2-ij/blob/master/src/main/java/net/imglib2/img/VirtualStackAdapter.java

In my code (BigDataProcessor2) we need to implement a version of below class for an ImagePlus:

https://github.com/tischi/fiji-plugin-bigDataProcessor2/blob/master/src/main/java/de/embl/cba/bdp2/loading/ImageLoader.java#L30

In fact, I think it could be relatively straight-forward, as VirtualStackAdapter.java is already containing all the relevant code.


Update Docs for Bank - Blocks

Update documentation and sample code for Bank - Blocks

ensure all available endpoints (GET, POST, PATCH, and DELETE) are documented
update request and response examples from real API data (2 items for GET list responses)
JSON should be formatted with 2 space indents
ensure any/all params are documented in tables
https://thenewboston.com/bank-api/blocks

I will take this


Absences: Round down percentage in overall attendance rate

ie, 99.7999% > 99% not 100%

This is done!


Added 3 manipulations for REMOVABLE SUBSYSTEMS.

Added 3 manipulations for REMOVABLE SUBSYSTEMS.
Checklist
The following aspects have been respected by the author of this pull request, confirmed by both pull request assignee and reviewer:

Changelog update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

README update (necessity checked and entry added or not added respectively)

[x] Pull Request Assignee
[ ] Reviewer

This PR is dropped in favor of a different Implementation.


integration testing app definition

We can create a testing app to validate that the operator is working and an Dapr application work with the DaprControlPlane installed by the operator.
To check this I would create the following resources:

Dapr Statestore and Dapr PubSub Component (optionally a Dapr Subscription)
A Dapr application (like this one dapr app)

a container that uses the Go Dapr SDK to interact with the Statestore and PubSub API and exposes REST endpoints that can be called by the integration tests
(optionally) the container should also expose an endpoint to receive subscription events.

got a prototype to install helm stuffs for e2e tests, having a dapr app packaged as oci artifact would be ideal


🐛 [firebase_app_check] Missing support for custom App Check providers

Bug report
On https://firebase.google.com/docs/app-check/custom-provider, it is explained how to implement a custom App Check provider and it links to pages on how to use this custom provider in Apple, Android and web.  However, afaics, it is not possible to use this custom provider in Flutter right now!
Can you also add support for custom providers?
I would like to use custom providers to support the whitelist (cfr discussion https://github.com/firebase/flutterfire/discussions/12222)

@MarcVanDaele90
Check if this issue resembles your case or not.

@darshankawar Thanks for looking at this.  I'm not sure to which part of the issue you refer to.
As I see it, the issue contains three parts

creating a custom provider (ReactNativeFirebaseAppCheckProvider implements AppCheckProvider)
using it to generate debug tokens
passing the customer provider to initializeAppCheck()

Creating a custom provider (with cloud functions) is documented on https://firebase.google.com/docs/app-check/custom-provider so I don't think we need anything extra for this.
I think the main thing missing right now is the ability to pass the custom provider to FirebaseAppCheck.instance.activate
Part 2 could be a temporary solution for my whitelist problem but I'm not sure I understand it correctly:
Would it be the intention to add a "debug" flag to the ReCaptchaV3Provider object which would generate a debug token on-the-fly when set?
await FirebaseAppCheck.instance.activate(
          webProvider: ReCaptchaV3Provider(appCheckKey)
      );

If so, I could indeed

set this debug flag for all customers in my whitelist (thereby skipping the actual recaptcha)
leave it empty for other customers in which case the behavior would be unchanged

This would be a nice fix for my current problem.  Though I still think that, in the long run, Flutter should also support custom providers.

Thanks for the feedback.

Also, I would like to implement a custom app check provider in Flutter but there is no way to do it currently. Now that SMS costs are per send and not per sign-in, it is risky to use Firebase auth without app check, and we need a way to whitelist some users.

@darshankawar is it possible to give an update? Is it possible to give a high level timeline?
We would really like to implement a custom app check provider in Flutter because app check is blocking valid customers.
I'm certainly willing to help where possible.

A short status-update would be appreciated. Any idea if (and when) this will be added?

Ping
(I understand such a ping is annoying but a short update would be much appreciated)


orcania: 2.3.2 -> 2.3.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/babelouest/orcania/releases.
meta.description for orcania is: Potluck with different functions for different purposes that can be shared among C programs
meta.homepage for orcania is: https://github.com/babelouest/orcania
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.3.3 with grep in /nix/store/j37qy12w9ig4w1x3hmqabrz8ikbpc6ma-orcania-2.3.3
found 2.3.3 in filename of file in /nix/store/j37qy12w9ig4w1x3hmqabrz8ikbpc6ma-orcania-2.3.3

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
orcania
yder

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/j37qy12w9ig4w1x3hmqabrz8ikbpc6ma-orcania-2.3.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A orcania https://github.com/r-ryantm/nixpkgs/archive/d5d09d35dfbd9a6f2343e2f75b8985c892c058b6.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d5d09d35dfbd9a6f2343e2f75b8985c892c058b6#orcania

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/j37qy12w9ig4w1x3hmqabrz8ikbpc6ma-orcania-2.3.3
ls -la /nix/store/j37qy12w9ig4w1x3hmqabrz8ikbpc6ma-orcania-2.3.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

orcania
yder

Maintainer pings
cc @JohnAZoidberg for testing.

Result of nixpkgs-review pr 259407 run on x86_64-linux 1

2 packages built:

orcania
yder


Fixing moretime plugin - keep a running total of added time

Adding notes before we forget again:
14:43:41  [17:08:55]  hm, that doesn't work
14:43:41  [17:09:12]  you're re-adding elapsed time from previous !moretimes
14:43:41  [17:09:19]  (potentially elapsed)
14:43:41  [17:13:06]  ah, needs resetting per game
14:43:41  [17:13:10]  how do i do that?
14:43:41  [17:14:21]  reset in enable_command?
14:43:41  [17:20:11]  even within a game though
14:43:41  [17:20:57]  the @added thing shouldn't be necessary
14:43:41  [17:23:23]  oh I see the bug
14:43:41  [17:23:45]  you decrease @limit, but seconds just resets it against the original sign up time
14:43:41  [17:23:49]  it always calculates the signup window relative to @signup
14:43:42  [17:23:53]  yep
14:44:15  conclusion?
14:45:51  why is the @added thing necessary if we fix it to refer to @limit instead of @signup?
14:46:42  @limit is the maximum number of seconds you can add to signup window
14:46:56  yup
14:47:09  and that gets decremented each time the command is used
14:47:46  oh you just want it to say how much it has added to the window in total as output?
14:48:10  you'd need to know what the limit originally was
14:48:32  @signup
14:49:19  http://i647.photobucket.com/albums/uu192/lovekakumei/futurama-fixit.gif
14:49:26  ok, so you could do remaining + (@signup - @limit) ?
14:49:32  or a variation of that which is correct
14:49:49  for what purpose?
14:50:06  in the hook_async()'s
14:50:30  ooooooh
14:50:35  ffs
14:50:49  urgh
14:51:28  isn't it more complicated a calculation than that?
14:51:45  shouldn't it need to know how long it has been since the signup window started?
14:51:46  probably, it's a bit tricky without being able to run it
14:52:00  why can't you run it
14:52:12  there's a readme!
14:52:14  :P
14:52:22  pfft
14:52:31  life's too short for that
14:53:30  you adjust 'remaining' so you don't need to know when signup window started
14:54:01  oh yeah, @when already handles that
14:54:05  good job, me
14:54:45  so it's just a case of fixing remaining
14:55:16  yeah

:+1:

I've pushed a refactor to https://github.com/nickrw/TWGbot/commit/1e81c09f370c5567a3e179fdbe12cf223c55ba5f that includes the gist of this PR. I think it makes the whole thing a little bit more understandable.
I don't want to just kill this PR and leave you with no attribution, so feel free to resubmit with the contents of that patch.


BUG: iOS Build Fails Due to Pod Dependencies Requiring Higher iOS Deployment Target

Is there an existing issue raised for this?

[X] I have searched the existing issues

Describe your issue

When trying to build the Flutter app on iOS, the following error is encountered due to the pod plugins requiring a higher minimum iOS deployment target:
Error : The plugin cloud_firestore, google_maps_flutter_ios and some others requires a higher minimum iOS deployment version than your application is targeting.
Error running pod install
Error launching application on iPhone 15 Pro Max.

Steps to reproduce

Attempt to build the project using IOS emulator.
Build will fail with the mentioned errors.

What was the expected result?

It was expected that the app would be built and run as it did on android.

Proposed solution

This issue can be fixed by noting down the minimum ios_deployment_target of the concerned dependencies and then updating the platform version in the pod files.
After that running the needful commands for updating pods and reinstalling them.

Would you like to work on this issue?
Yes
Terms & Conditions

[X] I agree to follow the Code of Conduct
[X] I'm a GSSOC'24 Ext contributor

Please assign me this issue.


Add --rerun-transforms flag

Expected Behavior
Gradle has a flag like --rerun-transforms to force all transforms to rerun.
Current Behavior (optional)
Today Gradle provides --rerun and --rerun-tasks to rerun tasks, which is useful when verifying task issues. However, there is no such flag for transforms.
Context
Debugging flaky/non-reproducible transforms can be very tricky, especially without the --rerun-transforms option.
Current workaround is:

rm -fr .gradle/caches/8.8/transforms
./gradlew --stop
./gradlew --rerun-tasks --no-configuration-cache --no-build-cache myTask

this is obviously a very clumsy workaround.

This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

This is currently not our focus but this is a valid feature request especially for debugging transforms.
Since this is not our focus at the moment it's not likely that it will be fixed soon. But we will discuss it and if seems like a good feature it looks like a good candidate for a contribution.


qwen 72b int4/8 版本的效果到底咋样，有分数吗

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
No response
期望行为 | Expected Behavior
No response
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
No response

对应Hugging Face或ModelScope页面的效果评测部分有列一些分数，比如https://huggingface.co/Qwen/Qwen-72B-Chat#效果评测，可供参考。


DOC: stats: Fix link to Cramer-von Mises wikipedia article.

Closes gh-13459.

Rendered web page for cramervonmises: https://25688-1460385-gh.circle-artifacts.com/0/html-scipyorg/generated/scipy.stats.cramervonmises.html

Rendered web page for cramervonmises: https://25688-1460385-gh.circle-artifacts.com/0/html-scipyorg/generated/scipy.stats.cramervonmises.html

Thanks @WarrenWeckesser @chrisb83. Would be nice if we could get the best of both worlds, but an ugly working link is better than a pretty, broken one.

Thanks @WarrenWeckesser @chrisb83. Would be nice if we could get the best of both worlds, but an ugly working link is better than a pretty, broken one.


optimise task download on destroy

Don't download files if size/mode time/checksum is the same.

Similarly to https://github.com/iterative/terraform-provider-iterative/issues/360#issuecomment-1010198069, it should be easily feasible for machines, but not that much for Kubernetes.

Similarly to #360 (comment), it should be easily feasible for machines, but not that much for Kubernetes.

Lets do it at least for machines.

This is a core feature of rclone and already implemented unless proven otherwise.
terraform {
  required_providers {
    iterative = {
      source = "github.com/iterative/iterative"
    }
  }
}

provider "iterative" {}

resource "iterative_task" "example" {
  cloud   = "aws"
  machine = "m"

  script = <<-END
    #!/bin/sh
    fallocate --length 1G file
  END

  storage {
    workdir = "."
    output  = "file"
  }
}


Brh - update confounders.jl for non-UKBB cohort

Codecov Report
Base: 48.31% // Head: 97.98% // Increases project coverage by +49.67% :tada:

Coverage data is based on head (0df0548) compared to base (0f4247b).
Patch coverage: 100.00% of modified lines in pull request are covered.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #176       +/-   ##
===========================================
+ Coverage   48.31%   97.98%   +49.67%
===========================================
  Files           6        4        -2
  Lines         592      298      -294
===========================================
+ Hits          286      292        +6
+ Misses        306        6      -300

Impacted Files
Coverage Δ

src/confounders.jl
100.00% <100.00%> (ø)

src/tmle_inputs/from_param_files.jl
98.21% <100.00%> (ø)

src/tmle_inputs/tmle_inputs.jl
96.96% <0.00%> (+1.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

@roskamsh I have added the optionality of the ld-block file. If you are happy with it we can merge now and I cn show you how the release is done.


CognitiveServices Accounts are not able to use the list-skus feature in the az cli

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az cognitiveservices account list-skus
Errors:
The command failed with an unexpected error. Here is the traceback:
No value for given attribute
Traceback (most recent call last):
  File "/usr/lib64/az/lib/python3.9/site-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 663, in execute
    raise ex
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 726, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 697, in _run_job
    result = cmd_copy(params)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/__init__.py", line 333, in __call__
    return self.handler(*args, **kwargs)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/cli/command_modules/cognitiveservices/custom.py", line 75, in list_skus
    return cf_accounts(cmd.cli_ctx).list_skus(resource_group_name, account_name)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/core/tracing/decorator.py", line 73, in wrapper_use_tracer
    return func(*args, **kwargs)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/mgmt/cognitiveservices/operations/_accounts_operations.py", line 1354, in list_skus
    request = build_list_skus_request(
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/mgmt/cognitiveservices/operations/_accounts_operations.py", line 330, in build_list_skus_request
    "accountName": _SERIALIZER.url(
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/mgmt/cognitiveservices/_serialization.py", line 703, in url
    output = self.serialize_data(data, data_type, **kwargs)
  File "/usr/lib64/az/lib/python3.9/site-packages/azure/mgmt/cognitiveservices/_serialization.py", line 780, in serialize_data
    raise ValueError("No value for given attribute")
ValueError: No value for given attribute

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az cognitiveservices account list-skus --kind {}

Expected Behavior
Environment Summary
Linux-5.4.0-1098-azure-x86_64-with-glibc2.35 (Cloud Shell), CBL-Mariner/Linux
Python 3.9.14
Installer: RPM

azure-cli 2.43.0

Extensions:
quota 0.1.0
connectedk8s 1.3.3
ai-examples 0.2.5
ml 2.11.0
ssh 1.1.3

Dependencies:
msal 1.20.0
azure-mgmt-resource 21.1.0b1

Additional Context

@chawlaj100 We are looking into it and get back to you for any additional information.

route to CXP team

@chawlaj100 I am not able to reproduce this issue and cmdlet works as expected.  (See screenshot below)


add new proposal as docs

title: Issue #

Discord Username: @
What type of PR is this? (select all that apply)

[ ] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 🚧 Breaking Change
[ ] 🧑💻 Code Refactor
[ ] 📝 Documentation Update

Description

Related Tickets & Documents

Related Issue #
Closes #

QA Instructions, Screenshots, Recordings

UI accessibility concerns?

Added/updated tests?

[ ] 👍 yes
[ ] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

[optional] Are there any post deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Elaborate the sentences on number 2 of Prerequisite section


No matching manifest

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

No information provided or steps to replicate - Closing 😞


Address warnings emitted by GitHub Actions in generated project

In a newly generated project several workflows produce warnings that could get cleaned up:

build-docs
The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

test (3.10)
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout@v2, actions/setup-python@v2

test (3.9)
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout@v2, actions/setup-python@v2

deploy
Input 'emit_telemetry' has been deprecated with message: Use of this input causes the action to do nothing. You should remove this build step from your workflow.

Build and publish Python 🐍 distributions 📦 to PyPI
Node.js 12 actions are deprecated. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/. Please update the following actions to use Node.js 16: actions/checkout@v2, <EMAIL_ADDRESS>

Build and publish Python 🐍 distributions 📦 to PyPI
The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Publish Python Package: .github#L1
ubuntu-latest workflows will use ubuntu-22.04 soon. For more details, see https://github.com/actions/runner-images/issues/6399

The set-output and Node.js 12 ones should just be a matter of upgrading to the latest major version of a few actions. The emit_telemetry one comes from the automatic GitHub Pages action, so it's not really under our control. And the ubuntu-latest is just informational and doesn't require any action on our end.

See also: https://github.com/microbiomedata/nmdc-schema/pull/513


quic is not used over caddy 0.10.11

1. What version of Caddy are you using (caddy -version)?
Caddy 0.10.11
2. What are you trying to do?
I want to see quic
3. What is your entire Caddyfile?
quic.com:443 quic:443 {
tls self_signed
root /root/Desktop/test_web/
log /var/log/access.log
}
4.   What did you expect to see?
I want to see the connection use quic but now is like follow:
The connection to this site is encrypted and authenticated using TLS 1.2 (a strong protocol), ECDHE_ECDSA with X25519 (a strong key exchange), and AES_256_GCM (a strong cipher).
7. What did you see instead (give full error messages and/or log)?
the only error I can find is
Certificate - missing
This site is missing a valid, trusted certificate
(net::ERR_CERT_AUTHORITY_INVALID).
before I use 0.10.10 ,and I meet the same problem even if I use --quic-version=QUIC_VERSION_38 to start chrome ,now I use 0.10.11,but it still did not use quic

QUIC requires a valid/signed certificate

where is the certificate of caddy Self-Signed，I want to install in brrowser to trust ,I dont have me own donaim,thanks a lot

Caddy stores the self signed certificate in memory, you'll have to generate your own certificate https://caddyserver.com/docs/tls


under parallel training mode, is statistic such as "StepsPerSecond" the overall data ?

Hi, after start training process, the screen will continuously output statistic data, e.g.

2019-11-25 15:41:39.993249  | CumUpdates          14820
2019-11-25 15:41:39.993340  | StepsPerSecond       1832.32
2019-11-25 15:41:39.993455  | UpdatesPerSecond       36.6464
2019-11-25 15:41:39.993561  | ReplayRatio            25.6
2019-11-25 15:41:39.993651  | CumReplayRatio         23.9818

These are the overall data of all the workers, right ?
For example, "StepsPerSecond  1832.32" means that all workers together can move 1832.32 steps forward per second totally, right ?

Yes, that is correct.  Even in the case of multi-GPU, it counts all the workers from all processes.

Thanks.


Add logging param to screenshot tests url; Make cron job running tests sequentially

Reordered the tests in cloudbuild to be screenshot -> sanity -> adversarial by their running time.

thanks!


Bibliography/References automatic heading option

As many users would probably like to include a heading before their references section eg "Bibliography" or "References", it would be very useful if this was available as an automatic addition controlled by a flagged option at the time of build/conversion.
If I wish to work on a document in markdown and have a bibliography section heading when I make occasional preview builds, I must to work on my document the whole time with an empty line and my # Bibliography line below the region in my text editor where I am doing the actual text entry.
PS. Thanks for the fabulous work!

One difficulty here is that there will be a variety of desired text (Bibligoraphy, References, Works Cited, etc), language, and format (#, ##, other).

+++ stevebarratt123 [Mar 09 16 03:51 ]:

As many users would probably like to include a heading before their
references section eg "Bibliography" or "References", it would be very
useful if this was available as an automatic addition controlled by a
flagged option at the time of build/conversion.

It is available.
See the man page under reference-section-title (which you
can put in metadata).

Well, you can  specify the exact text you want using the
metadata field I mentioned.
If you want more control (e.g. over the exact header style),
then just do it manually.  Put the header in your document,
and use the specially marked div to position your
bibliography where you want it.  This gives you all the
flexibility you hould need.
+++ John Muccigrosso [Mar 09 16 06:35 ]:

One difficulty here is that there will be a variety of desired text
(Bibligoraphy, References, Works Cited, etc), language, and format (#,
##, other).
—
Reply to this email directly or [1]view it on GitHub.
References

https://github.com/jgm/pandoc-citeproc/issues/218#issuecomment-194321288

I thought that I had checked through the man entries - clearly not. Apologies!

@jgm, yes, I had forgotten that field.
In this context, I am reminded also of my request to allow the designation to belong to any header, instead of just a special div: #210.


⚠ ClamAV has degraded performance

In 311479b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 50 ms

Resolved: ClamAV performance has improved in 11e7558.


Update README.md

Refer to the cargo help information, the default behavior of cargo-install tool is to fetch source from crates.io. As the published version on crates.io may not updated, it is better to change the instructions in this document from my perspective.
$ cargo help install
...
There are multiple sources from which a crate can be installed. The default
location is crates.io but the --git, --path, and registry flags can
change this source. If the source contains more than one package (such as
crates.io or a git repository with multiple crates) the <crate> argument is
required to indicate which crate should be installed.
...

The version in git in development code, that has never been tested, may be completely broken or insecure. Don't use it unless you are willing to contribute bug fixes.
The version on crates.io is really the one to use.


:bug: Get rid of CI prefix in commit hash.

@tiulpin I feel that we don't even need such a flag. let's just reset back in all the cases and drop the prefix beforehand. WDYT?


Better keybindings

Use default keybinding to run spec (ctrl-alt-p). Scope keybidings to ruby editors only since ctrl-alt-r used to reload atom.

:up:

The keybindings this plugin uses don't appear in the global list in Settings.

:up:


R99 and R103: contradiction

R99 states: BPI communication protocols MUST be asynchronous.
R103 states: For synchronous BPI messaging between BPIs, a BPI MUST utilize HTTPS RFC2818.
It seems those two requirements contradict each other. Please help clarify.

@atoulme ... R99 talks about communication protocols within a BPI while communication between BPIs may be either synchronous or asynchronous as R103 talks about communication between BPIs.
Clarifying intro sentence to R98 - R101 This leads to the following core capability requirements: to This leads to the following core capability requirements within a BPI:
Also replacing The layers are defined as follows: with The layers are defined as follows. Note that some requirements refer to communication between BPIs as noted within the relevant requirement:
Clarifying sentence below R103
"Note that while messaging within a BPI must be asynchronous, the communication between two BPIs can be either synchronous to asynchronous depending on the use case."

OK thank you.

Closing. Addressed with merged PR https://github.com/eea-oasis/baseline-standard/pull/173


canal set tabledb and tables not effect

cfg.Dump.TableDB = "mydb"
cfg.Dump.Tables = []string{"table1", "table2"}
when i set this but not effect
func (h *MyEventHandler) OnRow(e *canal.RowsEvent) error {
log.Info(e.String())
return nil
}
print result is all table msg

i see this readme have canal.start() but this not this function only ont canal.run()

cfg.Dump.TableDB = "mydb"
cfg.Dump.Tables = []string{"table1", "table2"}

TableDB and Tables are just Dump parameters, it works on dumping binlog before canal starts. See here
To filter database or table, maybe this PR can help you.

i got it, thanks


[Bug]: I am getting :CATCH ERROR: ERROR STATUS: 503: while deploying contract

Version

[X] Yes

General Question

[X] No

What happened?
CATCH ERROR: ERROR STATUS: 503 while trying to deploy contract
What have you tried?
I Googled and asked the community in the discord server but could not find a solution.
Relevant log output
No response
Protect your information

[X] I Agree

This is an error on NFTPort's server side.
Check the status of servers your program uses here:
https://status.nftport.xyz/
Further, regard the following page for an explanation of the error codes your receiving:
https://docs.nftport.xyz/docs/error-codes


Set to static any non *_test functions.

Mostly so that user code can use function names like onShake() or onButtonA().
Fixes https://github.com/lancaster-university/microbit-v2-samples/issues/8

@finneyj @JohnVidler If nobody has any reservations about this PR I'll merge by the end of the week, as it doesn't affect operation of any of the samples and enables users to use function names often shown in code examples, like onButtonA() and onShake().

Merging as discussed in the CODAL sync call.


Bugfix event propagation from idle graph nodes

Similar to #7, right now processQueuedEvents visits idle graph nodes (nodes which the graph is aware of but is not actively visiting during the block process), which basically allows for those nodes to emit events even when the data they're operating on isn't changing. The fix is the same as in #7, make sure we only visit the nodes we're actively traversing.

Fixed in 223d425


Update bcc map key deletion method

This PR fix the issue #624

The unit test failed with a bcc error due to the inability to locate the kernel headers. The PR #641 might fix this.

@marceloamaral This looks great! It reduced the CPU usage quite a lot (from 100% to ~20%). We'll investigate the rest of the overhead.

It reduced the CPU usage quite a lot (from 100% to ~20%)

where' the info come from ? just curious on the change and how we got it :)

@marceloamaral can you rebase to pick up #642?

It reduced the CPU usage quite a lot (from 100% to ~20%)

where' the info come from ? just curious on the change and how we got it :)

This is running on a real time OpenShift in our lab.

That is great news @rootfs.
I just triggered the tests again to see if the PR #641 fixed the unit test with bcc issue.


The Service Framework should support the Dependency Injection pattern of defining Interfaces as dependencies

Reality Collective - Reality Toolkit Feature Request
Is your feature request related to a problem? Please describe

The Dependency Injection Pattern defines a method where at design time, the developer can define the interfaces that a service needs in order to properly instantiate
https://learn.microsoft.com/en-us/dotnet/core/extensions/dependency-injection
This controls the order in which services instantiate and enable tighter "loosely coupled" integration between services and ensures a dependent service is available to wire up events when a service starts.
How would you classify your suggestion

Architecture / Services

Describe the solution you'd like

The ability to define service interface dependencies in the design of a service and have the service framework automatically recognise the dependency on startup.
Describe alternatives you've considered

Manual
Migrated from
https://github.com/realitycollective/realitytoolkit.dev/issues/64

@FejZa could you list some simple examples / use-cases to test with while building out this feature?

Not entirely sure, what you mean. But I imagine it to work like this, once it works:
MyTestServiceModule(IInputService inputService, ILocomotionService locomotionService) {}
and then those dependencies are injected with the currently active implementation of the interface.

Duplicate of #84 which has more detail


params is not defined

Hello,
I need help, this I am creating a bot for discordapp but i have one erreur :
- u.push(params(a1[x])); params is not defined.
I present my code to test :

var fs = require('fs'); var Cleverbot = require('cleverbot-node'); const clever = new Cleverbot('API','KEY'); client.on('message', function(message){ 	if(message.content.startsWith('bonjour')){ 		console.log('Received message: ' + message); 		Cleverbot.prepare(function(){ 			clever.write(message, function (response) { 				client.sendMessage(message.channel, response.message); 				console.log('Retourned response: ' + response.message); 			}); 		}); 	} });

I am sorry but I am bad in english, because i speak French. I am used Google Translate, sorry.
Thank you for your answers in advance.

It appears to be a bug in my code and yours. That chunk of code is for encoding JS Objects into URL parameters. It appears as though my code thinks it knows how to handle nested attributes. Since the message that is passed into clever.write is supposed to be a string, I probably don't need to worry about nested object attributes.
For your code, it seems like you should be passing in message.content to the clever.write method.


misc small fixes

Gonna rebase and merge this (with the small change of frames.size() == 0 to frames.empty()). Thanks for the fixes!

:+1:


Directory within a directory

Il n'est pas possible de mettre un répertoire dans un répertoire?

I think you are asking about nested folders. It is not possible to put a folder inside another folder. You can create multiple On My Device accounts and use folders under that to further organize your feeds.
Duplicate of #503.

Hello, Sorry for the translation.
This is exactly my problem but I use iCloud to have everything on my machines (Macbook and iPhone).
No plans to develop this function in the near future?

Le 5 nov. 2021 à 15:03, Maurice Parker @.***> a écrit :
I think you are asking about nested folders. It is not possible to put a folder inside another folder. You can create multiple On My Device accounts and use folders under that to further organize your feeds.
Duplicate of #503 https://github.com/Ranchero-Software/NetNewsWire/issues/503.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Ranchero-Software/NetNewsWire/issues/3336#issuecomment-961921224, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDL7I7CKH24JSQI2M4W2MDUKPW3RANCNFSM5HN4PUPQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I'm sorry, but it most likely won't be implemented anytime soon.
https://inessential.com/2018/09/09/why_netnewswire_doesnt_support_nested_fo

When you are not a developer, it seems so simple!

Le 5 nov. 2021 à 15:22, Maurice Parker @.***> a écrit :
I'm sorry, but it most likely won't be implemented anytime soon.
https://inessential.com/2018/09/09/why_netnewswire_doesnt_support_nested_fo https://inessential.com/2018/09/09/why_netnewswire_doesnt_support_nested_fo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Ranchero-Software/NetNewsWire/issues/3336#issuecomment-961935803, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDL7I4GX6WO4JJX2OQINTLUKPZCZANCNFSM5HN4PUPQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Ok I hadn't read the article. Why not but it's the basis of computer science to put a folder in a folder.
Best regards

Le 5 nov. 2021 à 15:22, Maurice Parker @.***> a écrit :
I'm sorry, but it most likely won't be implemented anytime soon.
https://inessential.com/2018/09/09/why_netnewswire_doesnt_support_nested_fo https://inessential.com/2018/09/09/why_netnewswire_doesnt_support_nested_fo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/Ranchero-Software/NetNewsWire/issues/3336#issuecomment-961935803, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMDL7I4GX6WO4JJX2OQINTLUKPZCZANCNFSM5HN4PUPQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[learn] add an lstm regression example

Can one of the admins verify this patch?

Thanks for this! Added few comments.

@ilblackdragon looks like comments were addressed?

Also can you add this example to the README.md file.
Otherwise, @vrv looks good.

@ilblackdragon I reviewed your comments, let me know if I misunderstood something during the reviewing.

@tensorflow-jenkins Jenkins, test this please

FAIL: Found 2 non-whitelited pylint errors:
tensorflow/examples/skflow/lstm_regression.py:158: [E1123(unexpected-keyword-arg), ] Unexpected keyword argument 'batch_size' in method call
tensorflow/examples/skflow/lstm_regression.py:185: [E1123(unexpected-keyword-arg), ] Unexpected keyword argument 'batch_size' in method call

@vrv The 2 errors are related to this comment, I already mentioned that TensorFlowEstimator's fit function is not updated yet to accept batch_size as an arg.

@mouradmourafiq let's either use Estimator then or change TensorFlowEstimator.fit to take batch_size for consistency.

@ilblackdragon I open a new #2817  to update the TensorFlowEstimator.fit function.

Ping. Any progress on this?


(PUP-5233) Restore puppet inspect

Fix puppet inspect so it looks in the correct location for the cached
catalog.

Retarget for stable?

Maybe for 4.4.1. There seem to be more issues involved in making inspect work correctly.

Retargeted against stable: https://github.com/puppetlabs/puppet/pull/4717


Cannot read property 'state' of undefined - props.location.state - react-router

Version
2.1.0

Reproduction
My code
I´m sending my props with link state prop to my details page
 <Link to={{pathname:/beerdetail, state: {beer:beer} }}>
and trying to extract img_url and add it to my styled-component
const BeerImage = styled.div
background-image: url(${props => props.location.state.beer.img_url});
background-size: 60px 200px;
background-color: #fff;
height: 200px;
width: 100px;
background-repeat: no-repeat;
background-position: center;
margin: 0 auto;
;
but i´m getting this message
Cannot read property 'state' of undefined
What i´m doing wrong here?
Thans

You're not passing the location to your styled component?
<BeerImage location={props.location} />


Fix error handling in shader loading

Hi,
small fix! And thanks for this. Doing my first steps in 3D programming :)

Sorry, I've been away from my Go projects a little bit. Thanks for the patch!


Can a Nim package be stored in a repository subfolder?

Hi, my current project has a folder with multiple ports, and it is originally written in C/C++, those ports are just wrappers around the C main API.
Here's the subfolder: https://github.com/metacall/core/tree/develop/source/ports/nim_port
Could be this published as a package if I add the metacall.nimble file? Or does it need to be stored in root folder?

Yes it's possible, one example from the packages.json:
  {
    "name": "shaname",
    "url": "https://github.com/Torro/nimble-packages?subdir=shaname",
    "method": "git",
    "tags": [
      "sha1",
      "command-line",
      "utilities"
    ],
    "description": "Rename files to their sha1sums",
    "license": "BSD",
    "web": "https://github.com/Torro/nimble-packages/tree/master/shaname"
  },

Just don't forget that Nimble will only install the subdirectory then, and not files outside of it.

Thank you so much for the fast response! I will PR the package as soon as possible.


Add real world examples and alternative names

Add examples from real world examples.
For example section on monads. Alternative name for chain is flatMap (or flat_map or flatmap). Real world example is streams for instance in highland.
For example section on functors. Real world example: of course Array. But also Streams and Promises. In Promises map is called then.
I believe this can be helpful.

I didn't want to have to explain a bunch of types that aren't built into JavaScript.
Promise.then is actually both the map and flatmap interface which I think makes using it for the explanation confusing

There is a section on Maybe which has chain/flatmap used. Maybe it's worth linking from the monad section

Promise.then is actually both the map and flatmap interface which I think makes using it for the explanation confusing

I didn't know that. Worth to mention that fact.
When onFulfilled returns promise (generally "thenable" object) then then works as flatMap, If returned value is not promise  then it works as map.
Links: https://promisesaplus.com/, https://gist.github.com/briancavalier/3296186

We've added the name aliases for chain. I'm not comfortable saying that then is equivalent to map. I think we should add a section on Future/Promise/Task to explain the functor aspects of those types.


Bad link - Features Tab

Full Contact:
This link doesnt exist https://auth0.com/docs/scenarios/fullcontact

fixed


Decorator Creates to many quotes

Hiera Code:
gitlab::gitlab_rails:
  omniauth_enabled: true
  omniauth_allow_single_sign_on: false
  block_auto_created_users: true
  omniauth_providers:
    - name: "github"
      app_id: "9a6161e2354332d5b5f5"
      app_secret: "ec032383e83fb59205a381f3ac9ec98a9a2bb486"
      url: "https://github.com/"
      args:
       scope: "user:email"

The result in /etc/gitlab/gitlab.rb
gitlab_rails['omniauth_providers'] = ["{"name"=>"github", "app_id"=>"9a6161e2354332d5b5f5", "app_secret"=>"ec032383e83fb59205a381f3ac9ec98a9a2bb486", "url"=>"https://github.com/", "args"=>{"scope"=>"user:email"}}"]

The quote in the Array is not needed.

cannot reproduce


multi GPU trainging problem

Hi, when I use your code to pretrain a GIN network using multi GPU,

the source sentence is

could u please give me some suggestions?


Fix for Django 1.7

HttpResponse mimetype was deprecated and replaced with content_type. Fix for issue #22

merge pls


Openvino performance a lot worse than darknet model

[Required] Your device (RaspberryPi3, LaptopPC, or other device name):
LaptopPC
[Required] Your device's CPU architecture (armv7l, x86_64, or other architecture name):
x86_64
[Required] Your OS (Raspbian, Ubuntu1604, or other os name):
Ubuntu1804
[Required] Details of the work you did before the problem occurred:
  python3 convert_weights_pb.py \ --class_names yolov3-tiny-mine.names \ --weights_file weights/yolov3-tiny-mine_final.weights \ --data_format NHWC \ --tiny True \ --output_graph pbmodels/frozen_yolov3-tiny-tw.pb
python3 /opt/intel/openvino_2019.1.094/deployment_tools/model_optimizer/mo_tf.py
--input_model models/tf/frozen_yolov3-tiny-tw.pb
--output_dir models/openvino/FP32
--data_type FP32
--batch 1
--tensorflow_use_custom_operations_config yolo_v3_tiny_changed.json

[Required] Error message:
performance is a lot worse than darknet inference

[Required] Overview of problems and questions:
 I trained the tiny-yolov3 model with darknet and test video perforamnce, it was pretty good. When I conveted it to openvino, the result was pretty bad, it missed several objects. Is it normal? How can I improve?

The v3-tiny will show poor performance in detecting all the objects correctly, but its meant for speed. You will surely get faster results; a higher FPS.

all people meet this bug and nobody can solve it even the intel coporation.

it may decreased 20 MAP


Added tests for errors caught looking up runnable scopes.

👍

ToL could this be a WDL4S unit test? Looks like we just need to make sure that creating a namespace from that WDL returns a Failure instead of a Success?

👍 regarding testing this so we don't regress. I'm still not convinced that Centaur's send-a-workflow-await-a-status is the right way to do it but I'll leave that up to you

This could be a wdl4s unit test, if wdl4s was throwing errors on these WDLs. For now, cromwell+centaur are making sure that we don't leave the workflow in an infinite Running state when tripping over these two known unhandled-by-wdl4s errors.
Filed:

https://github.com/broadinstitute/wdl4s/issues/92
https://github.com/broadinstitute/wdl4s/issues/93


python311Packages.gpsoauth: fix build

Description of changes
fix https://hydra.nixos.org/build/249107029

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 289932 run on x86_64-linux 1

4 packages built:

python311Packages.gpsoauth
python311Packages.gpsoauth.dist
python312Packages.gpsoauth
python312Packages.gpsoauth.dist


Make keys mandatory for entities

nice. Can you add a test to YamlFileValidationTests, with an incorrect yaml (no keys)

@rochar Actually, we already have a test with complex validation of error scenarios, Deserialize_WithNoxYamlSerializer_ThrowsException, so I've increased the error count. Maybe in the future we can add check for particular text error messages


Use parcel vorticity to compute the vorticity tendency

This PR

removes the solenoidal correction in vor2vel
uses the parcel vorticity to calculate the vorticity tendency
removes filtering

Closes #245.

Thanks for approving. I pull this into main. I created a separate branch for the vor2vel version with filtering and solenoidal correction.


Accessing raw elements of CLTensor faster

Is there a faster way to access CLTensor elements? I followed this tutorial: https://arm-software.github.io/ComputeLibrary/v17.04/index.xhtml#S4_6_4_working_with_objects, but it is way to slow for very large tensors. I have float input array with raw data. First, I have to fill the input CLTensor with that data, do the convolution, and afterwards extract the data again from output CLTensor to float output array. Can I just give pointer of float input array with raw data to CLTensor and the same way extract float array from output CLTensor or something similar which is fast as well?

You can't just use an arbitrary pointer, the allocation must be accessible from the GPU side.
What you could do ismap the CLTensor and load the weights directly from your file into the tensor (No need to create a "float input array" first)
Same at the end: you can just map the final buffer and read from the mapped memory, instead of copying to a float array. (Basically both at the beginning and end you should be able to just go from file to CL memory and then from CL memory to file, you shouldn't need to allocate any extra array).

Yes, but what if I have stream of raw data stored in float arrays and that needs to be processed in real time? Loading the data in CLTensor is bottleneck in that case.


applet.interface.jtag_openocd: Document how to improve performance

The jtag_openocd gateware performs all the necessary delays to achieve the configured clock frequency, but by default OpenOCD inserts additional delays on the client side. By using the newly introduced "use_remote_sleep on" option these delays can be disabled; On older versions of OpenOCD, the "adapter speed" can be increased to minimize the delays.

Hm, in that case I may have misunderstood the feature

Hm, in that case I may have misunderstood the feature

It is an extension of the protocol that Glasgow does not yet implement. It's easy to implement though.

I'm going to close this PR since the doc addition is inaccurate and will result in subtle breakage if followed.


API thumbnail is not getting updated when doing an API import --update

Description
When exporting an API with thumbnail image the image is not getting archived. Hence at import using apictl the thumbnail image is not getting reflected in the imported environment.
Steps to Reproduce

Add an image thumbnail to an API through the publisher portal.
Export that API using: apictl export api -n "$API_NAME" -v "$API_VERSION" -e $ENV --latest.
Unzip the file. The thumbnail image is not present anywere.

(step 1)

(step 2 and 3)

Affected Component
APICTL
Version
apim 4.3.0; apictl 4.3.0
Environment Details (with versions)
ubuntu 22
Relevant Log Output
No response
Related Issues
https://github.com/wso2/product-apim-tooling/issues/216
At this issue seems that exporting with thumbnail (.jpg, .png, ...) was working
Suggested Labels
No response

RCA
API thumbnail is not getting exported when the --latest tag is used with the export command. However, this behavior is only noticed when the thumbnail uploading was done post-deployment. Thereby, the deployed API had no knowledge on the thumbnail update that was made to the current API.
In order to resolve this issue, we have replaced the revisioned version with the current API state when it comes to the thumbnail extraction flow. The same applies to API products as well. This PR fixes the issue for both APIs and API Products.


Annotator not annotating some files

Selecting text on some .txt files does not annotate the selected text either on windows or web application

Adding the text for testing purpose
<IP_ADDRESS>.199.txt

Hello @LeoFrom. Thank you for providing the text file, that was helpful. I just tried it on the web version and it seems to be working for me. Could you kindly provide some more information?

What are you using as the text separator?
What are you using as the annotation precision?
If you're okay with doing so, please provide the tags file.

I'd like to get as close as possible to your setup.

Hello, thank you for your reply.

I'm using --- as my text separator
My annotation precision is word level
Here is my tag file : Tags.json

Maybe that can help you : It was fonctionnal at the beginning but it seems that when I play with the text separator sometimes or switch texts it freezes the process of tagging. I'll include two more different texts so you can maybe recreate the bug
<IP_ADDRESS>.55.txt
<IP_ADDRESS>.27.txt

Thanks! I've managed to replicate it now. I'll take a look at why this is happening and try to get back to you soon.

It seems the issue occurs if there are double quotes (") inside the text.
The Treebank Tokenizer we use is a JavaScript port of the one used by the NLTK Python library. The issue was reported (and fixed) by the NLTK team, so it seems we need to update the port.
@tecoholic it would be great if you could help with this one since I haven't looked into your port yet. I could try and give a PR if I can figure out what needs to be changed there.
In the mean time, @LeoFrom if it's an acceptable solution for your use case, you could try replacing all the double quotes with single quotes. I checked locally and it seemed to work alright.

@alvi-khan I will take a look.

@alvi-khan Looking at the code, it looks like those fixes are already in the JS ported version. To confirm, I added the unit tests from the Python version and they are passing as expected. So, I think the issue might be elsewhere and not in the tokenizer.


#21423 | Add underline styling to debug console

 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.:x: danielfrankcom sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hmm, I think there's some kind of bug on master atm?

Thanks for this great PR. Merging in to get user feedback asap


First example in README.md requests importing "log"

Or I got
➜  Desktop go build helloworld.go
# command-line-arguments
./helloworld.go:10: undefined: log in log.Fatalln

After I import "log" , it builds successfully.

Fixed in #59

Thanks!