SecRemoteRules - ModSecurity: Error parsing actions: Unknown action: \

both versions 2.9.0 & 2.9.1
[Mon Jun 27 16:12:05.018321 2016] [:notice] [pid 17018] ModSecurity for Apache/2.9.1 (http://www.modsecurity.org/) configured.
Trying load apache - modsecurity.conf:
SecRemoteRules 576d00bb44b4c576d00bb44bc3 https://rules.malware.expert/download.php?rules=generic
Then get always this error:
ModSecurity: Found another rule with the same id
Remote rules is 1 line, id not matter which try use:
SecRule REQUEST_HEADERS:User-Agent "<?" 		"id:400115,t:lowercase,deny,status:500,log,msg:'Malware expert - user-agent: php execute'"

Hi @jwillberg ,
That is a know bug within Apache. See here: https://bz.apache.org/bugzilla/show_bug.cgi?id=55910
You may want to update your apache.


No log file generated with code and config set up via Visual Studio for Mac

Hi,
I'm using Visual Studio for Mac. After getting NLog installed via NuGet and NLog.config configured, I can't get my code (see https://github.com/robertchong/NLogVS4MacDemo) to get log file generated. What am I missing?
Type (choose one): Question
NLog version: 4.4.11
Platform: Mono <IP_ADDRESS>
Current NLog config
See https://github.com/robertchong/NLogVS4MacDemo/blob/master/NLog.config

Is it a case sensitive file system?  Then is should be "nlog.config" (lower case)
Otherwise check the internal log.

Yes it is. Renamed to nlog.config. Still no luck. I did specify internalLogFile="/tmp/nlog-internal.log" in nlog.config but there's no internal log in /tmp.

@robertchong Think you have to setup the InternalLogger by code, since the NLog.config is not used. Maybe include the NLog.config in your app.config:
https://gist.github.com/Chrisso/1703644

I included App.config and no log. I then created a new branch1 branch with the App.config, checked it out into my Visual Studio 2017 running on Windows 10, updated NLog.config by changing   internalLogFile to "C:\Temp\nlog-internal.log" and internalLogLevel to "Trace" but still can't see app.log or nlog-internal.log. What am I missing? I would appreciate if you can shed any light on this. Thanks.

Update: I just ported the same code into my Visual Studio 2015 running on Windows 7 machine and it works fine. Without including the NLog.config to my App.config, I can see app.log as well as c:\temp\nlog-internal.log generated.

Update: Using the same Visual Studio 2017, I recreated the NLog demo code from scratch. Without adding NLog.config to App.config, I can get app.log and nlog-internal.log generated. Looks like there are things needed to be sorted out on the VS4Mac side.

Is this resolved or do we need to do something?

I assume your question has been answered, if not, let us know!


chore(tests): bump

bumped tests from https://github.com/blockfrost/blockfrost-tests/pull/128
bumped openapi to include fixes for cip68 and json encoding of /labels/label/n


Wrong layout for tessellation evaluation shader

The header of my tessellation evaluation shader looks like this:
#version 450
layout(triangles, equal_spacing, ccw) in;

However, in the output the 'triangles' is omitted from layout line.
#version 330
#ifdef GL_ARB_shading_language_420pack
#extension GL_ARB_shading_language_420pack : require
#endif
#extension GL_ARB_tessellation_shader : require
layout(ccw, equal_spacing) in;

This throws error at runtime:
GLSL linker error: ERROR: Tessellation evaluation shader must specify triangles, quads or isolines.

After editing the output and explicitly specifying 'triangles', everything appears to work perfectly.
layout(triangles, ccw, equal_spacing) in;

Any idea on the possible cause? Or is this implementation specific and 'triangles' should be taken as default?
I am using SPIRV-Cross through krafix, macos 10.11.6, Intel Iris GPU.

For some reason, triangle mode was just missing. Fixed, and added regression tests.


dropdown search gets stuck open

Console error:
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'x')
    at index-d47627e0.js:18:15458
    at Array.forEach (<anonymous>)
    at O_.e.$$.update (index-d47627e0.js:18:15416)
    at Bf (index-d47627e0.js:4:1832)
    at Fu (index-d47627e0.js:4:1547)

Fixed bug where if you select all items it crashes.


problem def GO:0097466 - ubiquitin-dependent glycoprotein ERAD pathway

GO:0097466 - ubiquitin-dependent glycoprotein ERAD pathway

Definition
Parents

An ERAD pathway whereby endoplasmic reticulum (ER)-resident glycoproteins are targeted for degradation. Includes differential processing of the glycoprotein sugar chains, retrotranslocation to the cytosol and degradation by the ubiquitin-proteasome pathway. A glycoprotein is a compound in which a carbohydrate component is covalently bound to a protein component.

2 things

I don't think this only applies to resident ER proteins does it? maybe it does...
Shouldn't this say

An ERAD pathway whereby terminally misfolded endoplasmic reticulum (ER)-resident glycoproteins
is a child of
response to glycoprotein
should that be misfolded glycoprotein?

@ValWood
How about this for a new definition:
An ERAD pathway whereby terminally misfolded glycoproteins are targeted for degradation.  In this pathway, terminally misfolded glycoproteins undergo removal, or trimming, of sugar chains, retrotranslocation from the endoplasmic reticulum to the cytosol, and degradation by the ubiquitin-proteasome pathway.  A glycoprotein is a compound in which a carbohydrate component is covalently bound to a protein component.
I can also reate a new 'response to misfolded glycoprotein' term as a parent for GO:0097466.
How important, though, do you think the 'terminally misfolded' aspect is?  Should the new term instead be named 'response to terminally misfolded glycoprotein'?
For the gene products that are currently annotated to this term, it seems the 'terminally misfolded' part is critical, i.e. these gene products are only active once there's no longer any potential for re-entry of the glycoprotein into the productive folding pathway,

Completely coincidentally we got a curation session related to this today.
The author would like a new term
endoplasmic reticulum quality control of glycoprotein folding
The endoplasmic reticulum quality control of glycoprotein folding is a process that ensures that only corrected folded proteins stay in the ER and pursue their way in the secretory pathway (for a review see D'Alessio et al 2010 Semin Cell Dev Biol. 21, 491-9) . This process is related with N-glycosylation and N-glycan processing in the ER and involves a number of genes including calnexin, UGGT, glucosidase  and glucosidase II, among others.
The ERAD pathway (Endoplasmic reticulum glycoprotein associated degradation) is a process that occurs when the previous mechanism has failed: irreversibly misfolded glycoproteins are retrotranslocated to the cytosol and degraded in proteasomes. So "endoplasmic reticulum quality control of glycoprotein folding" could include ERAD but not the other way around
So....it seems we have conflated endoplasmic reticulum quality control of glycoprotein folding
and ERAD!

Hi @ValWood --
For the new term request, what do you think of adding:
GO: protein folding in endoplasmic reticulum
-GO:new glycoprotein folding in endoplasmic reticulum
and annotating gene products like calnexin, UGGT, and the glucosidases to that new term.
The mannosidase and gene products acting downstream of that, such as the EDEMs, would then be annotated to the ERAD term.
Here's another attempt at refining the ERAD definition:
An ERAD pathway whereby terminally misfolded glycoproteins are targeted for degradation. In this pathway, terminally misfolded glycoproteins undergo demannosylation, retrotranslocation from the endoplasmic reticulum to the cytosol, and degradation by the ubiquitin-proteasome pathway. A glycoprotein is a compound in which a carbohydrate component is covalently bound to a protein component.

I think we should capture that this is a "quality control" pathway.
The pathway is particularly sensing whether the proteins are folded correctly?
and refolding if necessary.
"The endoplasmic reticulum quality control of glycoprotein folding is a process that ensures that only corrected folded proteins stay in the ER"
Can we use the term name the author suggested
"endoplasmic reticulum quality control of glycoprotein folding"
makes this clearer.
v

ER glycoprotein folding and quality control
Glycoprotein folding, detection and refolding of  misfolded glycoprotein
(the folding and the quality control are not really separable  processes because they happen cyclically until the folding is correct, or the glycoprotein terminally misfolded)
See
https://www.ncbi.nlm.nih.gov/books/NBK1958/

@ValWood
Okay, so I will add the term with the following parentage:
GO:0034975 protein folding in endoplasmic reticulum
-GO:new endoplasmic reticulum quality control of glycoprotein folding
For a definition, can we use this variation on your earlier post:
"A process that ensures that only corrected folded proteins stay in the ER and pursue their way in the secretory pathway . This process is related with N-glycosylation and N-glycan processing in the ER and involves a number of genes including calnexin, UGGT, glucosidase and glucosidase II, among others."
One reference could be this review: D'Alessio et al 2010 Semin Cell Dev Biol. 21, 491-9
What is the PMID for the paper that your user is curating?  I can add that as a reference, as well.
I will also update the ERAD definition.
Note that we also have this term in the ontology:
GO:0006515  protein quality control for misfolded or incompletely synthesized proteins
Definition: The chemical reactions and pathways resulting in the breakdown of misfolded or attenuated proteins. Source: GOC:jl
In this case, it seems that 'protein quality control' is strictly referring to the breakdown of misfolded proteins, which is consistent with the parentage for this term that includes 'proteolysis'.
Perhaps GO:0006515 should be renamed to more accurately reflect its definition.
I think we are now again touching on some of the issues in #13717 .......

sounds good.
other paper is PMID:27587357

Re
GO:0006515 protein quality control for misfolded or incompletely synthesized proteins
Definition: The chemical reactions and pathways resulting in the breakdown of misfolded or attenuated proteins. Source: GOC:jl
Aren't all of the descendant terms protein quality control pathways though?
The part which needed splitting out was the "quality control of folding" which  precedes identification clearance of terminally misfolded (ERAD). These are all still protein quality control mechanisms though?.
Note that this term only has 27 direct annotation and some of these seem a little odd. I would make this term "not for direct annotation", to clean these up....

I just spotted
https://www.ebi.ac.uk/QuickGO/term/GO:0044322
might be useful here.

Hi Kimberley,
Would it be possible to push these fixes  through fairly soon? I just had an expert in this area  send to our helpdesk that there were no terms for quality control of glycoprotein folding:
I think these suggestions will fix this issue and  I really want to get this one curated
https://github.com/pombase/website/issues/1127
Val

see
https://github.com/geneontology/go-ontology/issues/20914


[ADP-3290] Specify delegation history data structure in Agda

This pull request specifies the behavior of the Delegations data structure in Agda. We also remove the previous specification in Lean.
Issue number
ADP-3290

Paolo: shouldn’t we change the properties together with this ?
Heinrich: You’re right in that the code in the module Cardano/Wallet/Delegation/Properties.hs should be a more-or-less straightforward translation of the spec. 🤔 I will adapt it.


Assert not null listeners are being passed to Dexter

We should be checking that so that we don't throw NPEs

@Serchinastico could you elaborate a bit more? What you had in mind when you created this issue? I have a patch that protect ourselves from a null PermissionRequestErrorListener like this:
diff --git a/dexter/src/main/java/com/karumi/dexter/Dexter.java b/dexter/src/main/java/com/karumi/dexter/Dexter.java
index 42ff146..70cfeb7 100644
--- a/dexter/src/main/java/com/karumi/dexter/Dexter.java
+++ b/dexter/src/main/java/com/karumi/dexter/Dexter.java
@@ -89,7 +89,9 @@ public final class Dexter
   }

   @Override public DexterBuilder withErrorListener(PermissionRequestErrorListener errorListener) {
-    this.errorListener = errorListener;
+    if (errorListener != null) {
+      this.errorListener = errorListener;
+    }
     return this;
   }

This will protect us from a null assignation for the error listener, but I'm not sure if that is what this issue aimed to describe.

Yes, that's what (I think) I had in mind, don't forget we have several other listeners, not only the error one: https://github.com/Karumi/Dexter/blob/master/dexter/src/main/java/com/karumi/dexter/Dexter.java#L60
It might be a good idea to report back to the user when they try to do something like this, maybe adding another possible value to https://github.com/Karumi/Dexter/blob/master/dexter/src/main/java/com/karumi/dexter/listener/DexterError.java and sending it back to the error listener (if not null 😅 ).
Anyways, just protecting us from NPEs is good enough


New version: DirSoft.ARPro version <IP_ADDRESS>3

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

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

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

Microsoft Reviewers: Open in CodeFlow

Since this package uses a vanity URL, the old version should be removed.


Drawer navigator re-render on goBack()

This issue has been reported multiple times but i will try to do it well with the issue template .
I've got two screens in my DrawerNavigator. HomePage screen with a map in it, and ProfilePage with a simple button activating the navigation.goBack() function.
Current Behavior
On the app startup, the HomePage is displayed. Open the Drawer and select the ProfilePage. Click on the goBack Button. Then the ProfilePage is unmounted and the HomePage is completely remounted. Easely testable with:
componentWillMount() {
   console.log("Basic Test");
}

Expected Behavior
Same steps than above but the HomePage is not completely remounted and the componentWillMount() does not display anything.
Your Environment
Using Expo sdk 20.0.0, Galaxy S7 but the bug is here is every other tested phone.

software
version

react-navigation
^1.0.0-beta.11

Expo
20.0.0

npm
5.0.3

can reproduce

Pinging OP @poptocrack since this issue has not been active for a while, and it's related to an old version of the lib.
Please let me know if you want this to remain open (does it still happen in the latest version?); if I get no answer in the next 7 days I will close it.

Hi,
I'm using react-navigation ^1.0.0-beta.21, and the problem is still here.

Actually just realized it's a duplicate of https://github.com/react-community/react-navigation/issues/2277 - gonna close this in favour of that to keep the redundancy low


Hci Bridge - unpack_from requires a buffer of at least 3 bytes

Hi,
Objective:
Establish connection between external Bluetooth device and the emulator.

Issue:
After launching emulator from CMD I get the following error in running instance of hci_bridge CMD:
struct.error: unpack_from requires a buffer of at least 3 bytes for unpacking 2 bytes at offset 1 (actual buffer size is 2)

Stacktrace:
	ERROR:bumble.transport.usb:←[31m!!! Exception in sink.on_packet: unpack_from requires a buffer of at least 3 bytes for unpacking 2 bytes at offset 1 (actual buffer size is 2)←[0m
Traceback (most recent call last):
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\transport\usb.py", line 290, in dequeue
	self.sink.on_packet(packet)
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\bridge.py", line 40, in on_packet
	hci_packet = HCI_Packet.from_bytes(packet)
				 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 2037, in from_bytes
	return HCI_Event.from_bytes(packet)
		   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 4859, in from_bytes
	return cls.from_parameters(parameters)  # type: ignore
		   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 5869, in from_parameters
	return_parameters = cls.parse_return_parameters(self.return_parameters)
						^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 2154, in parse_return_parameters
	return_parameters = HCI_Object.from_bytes(
						^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 1679, in from_bytes
	return cls(fields, **cls.dict_from_bytes(data, offset, fields))
						 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 1578, in dict_from_bytes
	field_value, field_size = HCI_Object.parse_field(data, offset, field_type)
							  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\xx\AppData\Roaming\Python\Python312\site-packages\bumble\hci.py", line 1534, in parse_field
	return (struct.unpack_from('<H', data, offset)[0], 2)
			^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
struct.error: unpack_from requires a buffer of at least 3 bytes for unpacking 2 bytes at offset 1 (actual buffer size is 2)

Software and Hardware:
Windows 10
Pixel 8 API 35
Android Studio Koala | 2024.1.2
Bumble 0.0.198
Generic Unbranded Bluetooth USB Dongle advertised as 5.0 (Showing as USB CSR8510 A10)
            ←[30;43mID 0A12:0001←[0m
            ←[34m  Bumble Transport Names:←[0m ←[36musb:1←[0m or ←[36musb:0A12:0001←[0m
            ←[32m  Bus/Device:            ←[0m 001/014
            ←[32m  Class:                 ←[0m Wireless Controller
            ←[32m  Subclass/Protocol:     ←[0m 1/1 [Bluetooth]
            ←[32m  Product:               ←[0m CSR8510 A10
              Configuration 1
                  Interface: 0 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x83: INTERRUPT IN
                    Endpoint 0x01: BULK OUT
                    Endpoint 0x81: BULK IN
                  Interface: 1/0 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x02: ISOCHRONOUS OUT
                    Endpoint 0x82: ISOCHRONOUS IN
                  Interface: 1/1 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x02: ISOCHRONOUS OUT
                    Endpoint 0x82: ISOCHRONOUS IN
                  Interface: 1/2 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x02: ISOCHRONOUS OUT
                    Endpoint 0x82: ISOCHRONOUS IN
                  Interface: 1/3 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x02: ISOCHRONOUS OUT
                    Endpoint 0x82: ISOCHRONOUS IN
                  Interface: 1/4 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x02: ISOCHRONOUS OUT
                    Endpoint 0x82: ISOCHRONOUS IN
                  Interface: 1/5 (Wireless Controller, 1/1 [Bluetooth])
                    Endpoint 0x02: ISOCHRONOUS OUT
                    Endpoint 0x82: ISOCHRONOUS IN

Steps to reproduce:
Command 01:
	python apps/hci_bridge.py android-netsim:_:8877,mode=controller usb:1

Result:
	>>> connecting to HCI...
	>>> connected
	>>> connecting to HCI...
	>>> connected

Command 02:
	emulator -packet-streamer-endpoint localhost:8877 -avd Pixel_8_API_35

Result (the emulator will launch however in hci_bridge instance I will get the following):
	struct.error: unpack_from requires a buffer of at least 3 bytes for unpacking 2 bytes at offset 1 (actual buffer size is 2)

Notes:
Within the emulator the Bluetooth will remain off even if I try to toggle it on.

Issue is completely resolved by using Bumble compatible USB Bluetooth dongle.


FluentQueryBuilder won't replace values (vb.net)

Found issues for From method as follow:
`
    Dim q As FluentQueryBuilder = DbConnection.FluentQueryBuilder()
    Dim ProductId As Integer = 1
    Dim TableName As String = "products"

    With q
        .Select($"ProductId")
        .Select($"ProductName")
        .From($"{TableName}")
        .Where($"ProductId = {ProductId}")
        .OrderBy($"ProductName")
    End With

    Debug.WriteLine(q.Sql)

    Dim p As List(Of Product) = q.Query(Of Product)()

`
Result:
SELECT ProductId, ProductName
FROM @p0
WHERE ProductId = @p1
ORDER BY ProductName
If I set fixed string values then everything is working well.
`
    Dim q As FluentQueryBuilder = DbConnection.FluentQueryBuilder()

    With q
        .Select($"ProductId")
        .Select($"ProductName")
        .From($"Products")
        .Where($"ProductId = 1")
        .OrderBy($"ProductName")
    End With

    Debug.WriteLine(q.Sql)

    Dim p As List(Of Product) = q.Query(Of Product)()

`
Result:
SELECT ProductId, ProductName
FROM Products
WHERE ProductId = 1
ORDER BY ProductName
Also If I set fixed string value of table name and use where part as dynamic value then also is working.
`
    Dim q As FluentQueryBuilder = DbConnection.FluentQueryBuilder()
    Dim ProductId As String = 1

    With q
        .Select($"ProductId")
        .Select($"ProductName")
        .From($"Products")
        .Where($"ProductId = {ProductId}")
        .OrderBy($"ProductName")
    End With

    Debug.WriteLine(q.Sql)

    Dim p As List(Of Product) = q.Query(Of Product)()

`
Result:
SELECT ProductId, ProductName
FROM Products
WHERE ProductId = @p0
ORDER BY ProductName
Guess only issue is From method if you want to use it for table soruce.

Use the raw modifier, like this:
        .From($"{TableName:raw}")

Thanks mate!
I followed documentation and didn't noticed that as any option :)
It would be a nice to include this kind of example as well in documenation for other people.
Anyway, shell I use :raw sufix and for other methods or this is only for From() method?

Only for things that you do NOT want to be parameterized.
Anything that you pass with this modifier should be safe (you probably don't want to allow user-input here)

Of course. Idea  was for internal code part and code resuse to keep it 'dry'.


Support for custom SSL configuration

This looks like a very promising library. I would like to use it for communicating with our company private Buildbot using its REST API. The problem is that our Buildbot instance is proxied behind Apache, which is configured to do client certificate authentication (all employees have their own client certificate).
So I would need a way to configure the QSslConfiguration that qtrest uses, so that in my application, I can let the user select his/her .p12 certificate bundle to be used for authentication.

Hello. At now you may to use it by defaultSslConfiguration. Is it normal?

That would work if I only had to communicate with one Buildbot instance. But the tools I'm planning to make should have the ability to communicate with several instances at the same time. For that, I would need to be able to specify the QSslConfiguration to use for each connection separately.

You need hook function for doing some manipulations with QNetworkRequest before send this request to server, right? E.g. - set ssl configuration for request?

Yes I think that should work (haven't written any code yet, just exploring options).
How about a virtual protected function in APIBase like
QNetworkRequest APIBase::createRequest(const QUrl &url) const {
    return QNetworkRequest(url);
}

and change the get, post, put, deleteResource, head, options and patch functions to do
    QNetworkRequest request = createRequest(url);

instead of
    QNetworkRequest request = QNetworkRequest(url);

?
Then I could override createRequest in MyAPI with e.g:
QNetworkRequest MyAPI::createRequest(const QUrl &url) const {
    QNetworkRequest request = QNetworkRequest(url);
    request.setSslConfiguration(...);
    return request;
}

Yes, i think it's good solution. May you create pull request? Now I have no way to add the code =(


rebase feature/data-entry with dev

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.5 out of 6 committers have signed the CLA.:white_check_mark: aniruddha-k:white_check_mark: boonbaluri:white_check_mark: agnelpraveen:white_check_mark: ajayKrishnankm:white_check_mark: shalini-peeka:x: AluriAluri seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Added Concatenation of Array solution

Hey there thanks for the contribution. Would be great if you could star the repo so that it will be helpful for future contributors also. Thanks!

Sure, no problem.
On Tue, Oct 17, 2023, 8:53 AM Ankit @.***> wrote:

Hey there thanks for the contribution. Would be great if you could star
the repo so that it will be helpful for future contributors also. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/ankit0313/Leetcode-Problems/pull/39#issuecomment-1765602366,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYPOJ65TUUY5WA7G7APQKP3X7X235AVCNFSM6AAAAAA6CNWRXKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVGYYDEMZWGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Use cache for "latest" queries

We hit a scalability problem with latest queries for telemetry. @prattmic improved the db query substantially, which will help us survive. We can use the now consistent cache to store the result of "latest" queries, like latest telemetry for each user.

@prattmic Instead of modifying the telemetry interface, what do you think about editing the team interface? It already queries for all active team telemetry using a timestamp filter on all telemetry.
https://github.com/auvsi-suas/interop/blob/master/server/auvsi_suas/models/access_log.py#L60

Do you mean using that DB query to determine if the team is active, plus returning it to the caller, thus saving an extra query?
That will help, but I'm not sure it will be enough.

I didn't fully explain :(
I'm thinking of having a function users_active which returns a map from user to latest, where latest is None if the user is not active and the last log otherwise. This would cause 2N queries (per team one to determine active, one to determine telemetry) to become 1 with some python-level filtering.


Preheat models with Pod initialization

Support to preheat models when pod is in initialization, and block the initContainer until preheat completed.

/kind feature
/lgtm
/approve


Added for negative horizontal-, vertical offsett and scale

By setting the min value for horizontal_offset and vertical_offset, the offset can also be used in negative directions. This is usefull if you need an offset in the other direction. (For example because your toolhead/laser is on the left from the carriages head.) The negative gcode scaling factor allows for a decrease in size.

Good catch. Thank you!


Re-training SSD MobileNet on host platform rather than on Jetson platforms

Hi,
There are instructions given in https://github.com/dusty-nv/jetson-inference/blob/master/docs/pytorch-ssd.md for retraining MobileNet with custom dataset on Jetson platforms. What I like to train is quite big and I want to make use of the GTX 2070 / Ubuntu 16.04 host platform to speed up the process. Is there any instruction to train for Jetson on a computer?
Thank you very much in advance.

Hi @mozcelikors , it is the same process, I train it on my Ubuntu laptop with GeForce 1070 card.
You can clone the submodule directly to your PC:   https://github.com/dusty-nv/pytorch-ssd
Then make sure you have CUDA and PyTorch installed to your PC, according to the instructions from PyTorch.org.  You also need to install the prerequisite Python packages for pytorch-ssd.

Hi @mozcelikors , it is the same process, I train it on my Ubuntu laptop with GeForce 1070 card.
You can clone the submodule directly to your PC:   https://github.com/dusty-nv/pytorch-ssd
Then make sure you have CUDA and PyTorch installed to your PC, according to the instructions from PyTorch.org.  You also need to install the prerequisite Python packages for pytorch-ssd.

Thank you

Thank you


Fix autocompletion for clears command

When using /clears the autocompletion for the World field returns the following response:
Could not send Discord autocompletions: HTTP 400 Bad Request, {"message": "Invalid Form Body", "code": 50035, "errors": {"data": {"choices": {"_errors": [{"code": "BASE_TYPE_MAX_LENGTH", "message": "Must be 25 or fewer in length."}]}}}}
This should be fixed to show the autocompletion options.

seems like the issue solved itself. closing.


Jigen failure

Codecov Report

Merging #130 (e2c176a) into master (e675c8c) will increase coverage by 0.73%.
The diff coverage is 97.77%.

@@            Coverage Diff             @@
##           master     #130      +/-   ##
==========================================
+ Coverage   94.38%   95.11%   +0.73%
==========================================
  Files         108      112       +4
  Lines        3348     3667     +319
==========================================
+ Hits         3160     3488     +328
+ Misses        188      179       -9

Flag
Coverage Δ

unittests
95.11% <97.77%> (+0.73%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

domainlab/utils/hyperparameter_sampling.py
95.74% <95.74%> (ø)

domainlab/exp_protocol/run_experiment.py
97.72% <97.72%> (ø)

domainlab/compos/exp/exp_main.py
96.66% <100.00%> (ø)

domainlab/compos/exp/exp_utils.py
99.10% <100.00%> (+0.05%)
:arrow_up:

domainlab/utils/generate_benchmark_plots.py
100.00% <100.00%> (ø)

domainlab/compos/exp/exp_cuda_seed.py
100.00% <0.00%> (ø)

domainlab/tasks/a_task.py
90.32% <0.00%> (+2.15%)
:arrow_up:

domainlab/compos/vae/zoo_vae_builders_classif.py
90.90% <0.00%> (+6.81%)
:arrow_up:

domainlab/compos/builder_nn_alex.py
100.00% <0.00%> (+7.69%)
:arrow_up:

...omainlab/compos/vae/compos/encoder_xyd_parallel.py
90.00% <0.00%> (+10.00%)
:arrow_up:

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Car-la-F @schoersch , shall we delete this pull request?
I have fixed jigen already, which means we do not allow other than 3*3 tile division there and we should disable sample that hyper-parameter.
Best,
Xudong

We agree


Extensions manager keeps loading

When I click extensions manager, it keeps loading. All tabs are like this.
Although if I drag any zip file to the drag here point, it installs fine.
It started with version 1.5, updated to 1.6 and it's still the same.
I'm on mac osx 10.7.5

Hi  @nabtron,
sounds like a Firewall problem. Make sure that the Node executable in Brackets is allowed to connect outside of Brackets.
The steps are as follows:

Choose System Preferences from the Apple menu.
Click Security or Security & Privacy.
Click the Firewall tab.
Unlock the pane by clicking the lock in the lower-left corner and enter the administrator username and -password.
Click Advanced to customize the firewall configuration.
Click the Firewall Options button
Click the Add Application (+) button.
Select the app you want to allow incoming connection privileges for.
Click Add.
Add the node executable in Brackets folder (should be something like ~/Applications/Brackets/node)
Click OK.


Logging of roles in DB (Wounds+Deaths) + fix weapon bug @mattjons0n

This PR includes;

Fix for the weapons being shown as roles in DBLog_Deaths (Thanks to @mattjons0n (Jons0n#2156))
Logging of roles in DBLog_Wounds & DBLog_Deaths

Servers that already have a running DB please run those commands in your MySQL server after updating your squadJS to this version;
-- Wounds
ALTER TABLE `DBLog_Wounds`
ADD COLUMN `role` VARCHAR(255) NULL DEFAULT NULL AFTER `weapon`;

-- Deaths
ALTER TABLE `DBLog_Deaths`
ADD COLUMN `role` VARCHAR(255) NULL DEFAULT NULL AFTER `weapon`;

I believe I did broke this pr...
Will close this and re-open when I'm ready with this.


onTouchEvent to handle onClickListener

Hi.
How is onTouchEvent supposed to work? I can't find the correct way to handle onClick events. I've tried setOnClickListener and setOnTouchListener without luck. And I can't cast FloatingActionButton to Button because it gives an error.
Can you provide an example? Thanks!

Hi samuelmgalan,

Try this one to handle onTouchEvent call:

@Override
	public boolean onTouchEvent(MotionEvent event) {
		super.onTouchEvent(event);
	        // Your code goes here
        return true; // if handled, otherwise false
	}

So you just need to call super.onTouchEvent(event) and return true if your button click is handled, otherwise false
2. FloatingActionButton extends directly the android.view.View class, so it can't be casted to android.widget.Button class

Here is how it is done in sample:
Declaration inside XML:
<com.software.shell.fab.ActionButton
			android:id="@+id/fab_activity_action_button"
			style="@style/fab_action_button_style"
			android:onClick="onActionButtonClick"
			fab:type="DEFAULT"
	        />

and then onActionButtonClick method in code:
public void onActionButtonClick(View v) {
		if (hideAndShowOnClickCheckBox.isChecked()) {
			actionButton.hide();
			new Handler().postDelayed(new Runnable() {
				@Override
				public void run() {
					actionButton.show();
				}
			}, ACTION_BUTTON_SHOW_DELAY_MS);
		}
	}

I will provide the full demo sample a bit later today

Thanks for your examples.
My problem is that onTouchEvent() cannot be overriden inside a Fragment, but that's another problem.
Thanks again!

What about setting the click listener to this button? Did you manage to make it work?

It doesn't work. Never gets called and I think it should be, but i have no idea why not.

Could you please send me the code here:
<EMAIL_ADDRESS>I'll try to help

great
From: samuelmgalan<EMAIL_ADDRESS>Sent: Monday, February 16, 2015 2:32 PM
To: shell-software/fab
Cc: Shell Software
Subject: Re: [fab] onTouchEvent to handle onClickListener (#5)
Closed #5 https://github.com/shell-software/fab/issues/5 .
—
Reply to this email directly or view it on GitHub https://github.com/shell-software/fab/issues/5#event-236286527 .


fix exit codes in extract command

The extract command is inconsistent in its error handling and exit codes.
FIX IT!

@robhudson This should be an easy fix, I think. Want to grab this one, too?


Ability to add track to a specific position

Should be able to add tracks to a specific position, e.g. /play {query} [position]

Might not be needed after #239 /move command which is can be used universally.

Closing, can move with /move in #239 now.


[ignition-math6] Bump to version 6.14.0

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

I checked upstream and found that the latest version is 7.2.0, we should update to the latest version.
Note: I will be converting your PR to draft status. The above suggested changes are only recommendations. If you are willing to adopt them, please click "ready for review" after making the modifications. If you do not wish to make any changes, please click "ready for review" directly. That way, I can be aware that you've responded since you can't modify the tags.

For ignition-math6 the latest version is 6.14.0. Version 7 has breaking changes from versions 6, so updating the math6 port to a version 7 release will break dependend packages.


fix(demoData):The URL was pointing to a private AWS account

Fix to point to a radical imaging AWS account for longer term usage.
@sedghi - this is just moving things away from my own AWS account to the radical one.  Data available is a superset.

Not sure if you can see this but the CircleCI checks are failing when building the docs:
[INFO] [en] Creating an optimized production build...
../core/examples/programaticPanZoom/index.ts:49:26 - error TS2339: Property 'getPan' does not exist on type 'IVolumeViewport'.

49     const pan = viewport.getPan();
                            ~~~~~~

../core/examples/programaticPanZoom/index.ts:51:14 - error TS2339: Property 'setPan' does not exist on type 'IVolumeViewport'.

51     viewport.setPan([pan[0] + 5, pan[1]]);
                ~~~~~~

../core/examples/programaticPanZoom/index.ts:67:27 - error TS2339: Property 'getZoom' does not exist on type 'IVolumeViewport'.

67     const zoom = viewport.getZoom();
                             ~~~~~~~

../core/examples/programaticPanZoom/index.ts:69:14 - error TS2339: Property 'setZoom' does not exist on type 'IVolumeViewport'.

69     viewport.setZoom(zoom * 1.05);
                ~~~~~~~

../core/examples/programaticPanZoom/index.ts:85:14 - error TS2339: Property 'setZoom' does not exist on type 'IVolumeViewport'.

85     viewport.setZoom(1);
                ~~~~~~~

../core/examples/programaticPanZoom/index.ts:86:14 - error TS2339: Property 'setPan' does not exist on type 'IVolumeViewport'.

86     viewport.setPan([0, 0]);
                ~~~~~~

../core/examples/programaticPanZoom/index.ts:103:14 - error TS2339: Property 'setZoom' does not exist on type 'IVolumeViewport'.

103     viewport.setZoom(viewport.getZoom(), true);
                 ~~~~~~~

../core/examples/programaticPanZoom/index.ts:103:31 - error TS2339: Property 'getZoom' does not exist on type 'IVolumeViewport'.

103     viewport.setZoom(viewport.getZoom(), true);
                                  ~~~~~~~

../core/examples/programaticPanZoom/index.ts:156:12 - error TS2339: Property 'setZoom' does not exist on type 'IStackViewport'.

156   viewport.setZoom(0.8);
               ~~~~~~~

../core/examples/programaticPanZoom/index.ts:157:12 - error TS2339: Property 'setPan' does not exist on type 'IStackViewport'.

157   viewport.setPan([-128, 0]);
               ~~~~~~

../core/examples/programaticPanZoom/index.ts:159:12 - error TS2339: Property 'setPan' does not exist on type 'IStackViewport'.

159   viewport.setPan([-128, 0]);


Multitenancy

Hi Castore team !
Thank you for your great work on the library.
Is your feature request related to a problem? Please describe.
I am looking for a multitenancy event sourcing library.
How would you approach this in Castore ?
Describe the solution you'd like
I am not an expert of event sourcing but I would like to add to all events a tenantId property and modify the dynamodb event-storage-adapter in order to partition data by tenants.
Describe alternatives you've considered
One alternative is to create everything from scratch but I'd rather use your library.
Thank you very much for your help !

Hi @mat9819 and thanks for your interest !
I think that's an interesting feature so I'll add it to the roadmap. We could create a TenantizedEventStore that follows the EventStore interface while adding a tenantId argument for each method. And maybe modify the event storage adapter to expose a listTenants method ?
That's a lot of work and you're the only person having this need. We're a small team so we won't prioritize it for the next 6 months I think (we have more frequently asked subjects like snapshots and read models).
Two solutions that could do the trick right now:
1: Keep one event store and prefix your aggregate ids by a tenant id:
await eventStore.getAggregate(`${tenantId}#{aggregateId}`)

It will work but will degrade the performance of the listAggregateIds method for one tenant as you would have to list all the aggregate ids and filter manually the ones that begin by your tenantId. It should an acceptable tradeoff as listAggregateIds is mainly there for maintenance / migration jobs.
2: Suffix your event store by a tenant id
const tenantEventStore = new EventStore({
  eventStoreId: `POKEMONS#${tenantId}`
  ...
})

This solves the problem of the listAggregateIds but means that:

You need a custom method outside of Castore to save/retrieve all your tenantIds
In your code, you will construct an event store class each time you want to interact with one of your tenant. So it will impact your backend performances.

I would rather go with the first solution.
Hope it helped ! Thanks for the issue !

@mat9819  there's actually a 3rd solution which is to create a new event store called tenant and join your data with message groups. See this page for an example: https://castore-dev.github.io/castore/docs/event-sourcing/joining-data/

Closing this as there's no plan to implement this in the near future.


Refractor props for blending

Refractors the props as we talked about in our VR check in and adds classes for some object types

Adds useColorMap prop
Moves slices and spacing as props of VolumeViewer, not each individual model
Renames channel as blending and makes it a prop of VolumeViewer, not each individual model
Adds prop validation for position, rotation, scale, slices, and spacing
Adds custom validation for sliders
Adds classes folder
Adds Point class - transferFunction  is an array of Points
Adds ColorMap class, default color maps are static properties
Adds Blending enum as a class of static properties
Adds Model classes
Each individual model is now merged with the default properties (Model.Default) in the Model constructor
Moves custom validation for models into VolumeViewer.propTypes, written for Model class and all individual properties
Adds a <hr /> between <TabPanel  /> and <ClipControls in Controls.jsx
Updates README accordingly

This PR includes the changes to sliders so merging should wait until #79 is merged. There's still some validation that needs to be done based on useTransferFunction and useColorMap but I'll work on that in another PR.

closes issue #80
closes issue #75
closes issue #75
closes issue #73

An updated description based on the re-refractor:
Refractors the props as we talked about in our VR check in and adds classes for some object types

Adds useColorMap prop
Moves slices and spacing as props of VolumeViewer, not each individual model
Renames channel as blending and makes it a prop of VolumeViewer, not each individual model
Adds a custom vec3 string validation function for position, rotation, scale, and spacing
Adds a custom int validation function for slices
Adds a custom validation function for an individual slider
Adds a custom validation function for an individual model
Example color maps are now exported as COLOR_MAPS
Adds Blending enum object
DEFAULT_MODEL and DEFAULT_SLIDERS exported as objects
Adds a  between  and <ClipControls in Controls.jsx
Updates README accordingly

There's still some validation that needs to be done based on useTransferFunction and useColorMap but I'll work on that in another PR. PR closes all the same issues as before.

Added range -> dataRange as a comment in #95, will do that refactor together with the changes to the controls props


section IConfiguration parameter: useless call to Bind

There is a code sample in the section "IConfiguration parameter":
namedConfigurationSection.Bind(new LibraryOptions
{
    // Default library options are overridden
    // by bound configuration values.
});

The method Bind assigns values from namedConfigurationSection to the instance of newly created object of type LibraryOptions.
But the object is not used anywhere. So the call to Bind is useless.
What is the purpose of this code?

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

ID: 03494bb6-433a-5385-b664-7b830c37f9ce
Version Independent ID: d0e9398d-50c6-01b8-a242-4edc8b27df31
Content: Options pattern guidance for .NET library authors
Content Source: docs/core/extensions/options-library-authors.md
Product: dotnet-fundamentals
GitHub Login: @IEvangelist
Microsoft Alias: dapine

Hi 👋 @fantaclaus
Thank you so much for posting this issue. We really appreciate it.
The new of the LibraryOptions instance is used to bind to the named section of the IConfiguration section. When instantiated it may contain default values, the section from the config overwrites anything that was configured. When consumers ask for an IOptions<LibraryOptions> they'll get the instance that was bound with the calling .Bind function and all of the corresponding configured values. Does that make sense?

It doesn't work this way. I've checked it out.
Bind just copies data from namedConfigurationSection to the object instance. It doesn't really bind anything (I have no clue why it is named so). You can see it in the source code.
If you want to achieve the described behavior you need this code:
services.Configure<LibraryOptions>(namedConfigurationSection);

Ah, after looking into this in more detail - and testing out your claims, I stand corrected. You're right, I'm updating this. Thank you


Small bug with double "$" before variables and documentation inside menu

Hello!
I think there are 2 small problems after the last update (maybe related to https://github.com/phpactor/phpactor/issues/1775).

When completing the variable name a new "$" is added to the variable (please note the double "$" before $myCollection)
The completion menu contains the documentation and so it becomes quite difficult to read (at the end of the video you can se the first and other methods completion

I made a GIF to explain both problems:

Thanks again.

hm, strange.
for the first see https://phpactor.readthedocs.io/en/master/lsp/vim.html#two-dollars-on-variables although that would not have been caused by that PR - and AFAIK I don't need this in my config.
For the second ... that looks very odd -- are you sure it's not custom config on your end?

testing with CoC - I get neither the $$ or the other issue:

Hello! The first problem was solved by following your guide on phpactor:
autocmd FileType php set iskeyword+=$
The second one seems a problem only with GVim or Neovim-qt (so the GUIs), it doesn't show up using the terminal.

hm, does it happen also with other language servers?

As far as I can tell only for phpactor: JS is fine, CSS is fine, HTML is fine and PHP when using yaegassy/coc-intelephense is fine.
Please tell me if I can be of any help!
Thanks!


Will msal_auth automatically refresh tokens?

I see that there's an expiration time for the access token that's returned and I'm currently storing that token locally using flutter_secure_storage (see here). I was wondering if this library will automatically refresh the token / behave as it seems to do for the msal library for javascript. What should that process look like with this package?
Relevant links related to refreshing tokens and how it behaves for msal-react

Does the msal-react library automatically refreshes the token? #6039
Token Renewal

@paulhwangfti This plugin does not automatically refresh the token, instead, you have a method acquireTokenSilent that needs to be called when the existing token is expired.

@nayanAubie I see. It's my first time working with auth tokens and so I'm still learning but thank you for that information. 👍


N_TILINGS with getQ and getQb

With the last for loop in getQ and getQb of agent.cpp, i is from N_TILINGS to 3N_TILINGS, I think it should be from 2N_TILINGS to 3*N_TILINGS.
Is it a bug or I just miss something?
Thanks. @tspooner

Hey,
You're absolutely right. I'm not sure why, but this seems to be yet another discrepancies between my private development repo and this public one - as in your issue from before (#9).
Rather than do a small patch like last time, I will go through all the code this weekend and make sure everything is in sync. Thanks for bringing this to my attention and sorry for the hassle.
Regards,
Tom

Thanks, I tried multithreaded training, but I failed, the data in agent just crashed.
Is this multithreading feature workable? @tspooner

Honestly, I stopped using multi-threaded training quite some time before the main results of the paper were found. It doesn't surprise me much that it is broken.
I realise that's not ideal but you will probably have to do some modification yourself. This would include using mutexes to ensure that write-access is unique at any one time etc.

I noted on-policy R learning seems perform better than the other counterparts in the paper, so is OnlineRLearn the on-policy R learning?
Which learning method do you recommend with market making?
@tspooner
Thanks

Yeah, OnlineRLearn is the on-policy R-learning algorithm that was introduced by Sutton. It's the equivalent of Q-learning for continuing tasks - i.e. it solves for a different objective: the expected average return as opposed to the expected discounted return.
In general, there is no "right" algorithm for market making. It really depends on what type of solutions you want and what assumptions you want to make about the setting; one can formulate market making as an episodic task (i.e. day-to-day trading), or as one with no terminal time. These yield different results, but it's not clear if one is necessarily better than the other. We certainly found in our experiments that it performed well, but that could also be said for Expected SARSA.
It also depends on whether you intend to use a discretised action-space or not...
Given all this, I would strongly suggest you start with Q-learning and SARSA and branch out from there. Until you try it for yourself and what the results are, where the limitations of on- vs off-policy methods lie etc, it's hard to gain proper insight. This makes for more effective solution development, in my experience.


Launcher marks releses as installed which are actually not installed

I assume this is related to the way we compare versions based on the GameIdentifier, and all the preview releases from the "new" Jenkins end up with the same identifier taken from the display version =/ Guess we'll need to do some custom mangling with version ids again...

Related:

https://github.com/MovingBlocks/Terasology/pull/4231
https://github.com/MovingBlocks/Terasology/issues/3888

Fixed by #631


fix: catch-all route segment always return an array of values

What kind of change does this PR introduce?
Bug fix
What is the current behaviour?
#110
What is the new behaviour?
Catch all routes always return an array of params.
Does this PR introduce a breaking change?
No.
Other information:
Please check if the PR fulfills these requirements:

[X] Tests for the changes have been added
[ ] Docs have been added / updated

Coverage remained the same at 100.0% when pulling 86766d03fb98f4dd070aa736ad2a2a33f718f752 on fix-catch-all-route-segments-value into 9379a53bcbcbc160cec1de87f3206105db3928f5 on master.

Coverage remained the same at 100.0% when pulling 86766d03fb98f4dd070aa736ad2a2a33f718f752 on fix-catch-all-route-segments-value into 9379a53bcbcbc160cec1de87f3206105db3928f5 on master.


dep: Update graceful-fs dependency to the latest

graceful-fs used to monkey-patch node's core fs module.
This has been fixed in the version 4.
This might also fix https://github.com/nodejs/node/pull/2714#issuecomment-138092313

LGTM

LGTM if it works

@thefourtheye Do you have a commit bit or should I land it for you?

@bnoordhuis I had to update my email in the commit log and its done now. I don't have commit access to this repo, so you can land this.

Thanks Sakthipriyan, landed in 1ea7ed0.  Let me know if you plan to work on node-gyp more and I'll give you your commit bit.

@bnoordhuis Sure. I ll start by monitoring the PRs and issues. Thanks :-)


check before appending confirm container

fixes #741

Coverage remained the same at 87.67% when pulling d0908a4f862cf79e0b5f6c15bfcc13899b20448d on webdevan:patch-1 into 1f1d629c4708a1e06422dcdbba5acbe0e1f7804e on chmln:master.

I'm going to make a new PR


refactor: refactored typing on Checkbox

Thank you for your PR. Your branch seems to be protected and I cannot modify it. Part of the code needs to be moved to PR #220 .


merge from arlac77/template-github,arlac77/template-kronos-module

package.json

chore(package): add ^3.7.1 remove ^3.7.0 (devDependencies.ava)

:tada: This PR is included in version 8.3.24 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fail to run ./make.sh

c_type’?
tstate->exc_type = local_type;
^~~~~~~~
curexc_type
pycocotools/_mask.c:14327:13: error: ‘PyThreadState {aka struct _ts’ has no member named ‘exc_value’; did you mean ‘curexc_value’?
tstate->exc_value = local_value;
^~~~~~~~~
curexc_value
pycocotools/_mask.c:14328:13: error: ‘PyThreadState {aka struct _ts’ has no member named ‘exc_traceback’; did you mean ‘curexc_tracebac’?
tstate->exc_traceback = local_tb;
^~~~~~~~~~~~~
curexc_traceback
error: command 'gcc' failed with exit status 1

same error...

same error...

change all the 'exc' stuff into 'curexc', it will be alright.

In case anybody hasn't found a solution yet, forcing the build worked for me.
python build.py build_ext --inplace --force


Make blufi(nimble) customizable broadcast (IDFGH-7378)

BluFi which using the Nimble host cannot customize the BLE GAP broadcast, because the esp_blufi_gap_event function called by the esp_blufi_adv_start function has a static identifier, and the esp_blufi_gap_event function uses the specified function to re-broadcast when the connection fails or disconnects. So I tried removing the static flag and passing the broadcast function pointer through the arg parameter. These changes worked fine in my project.

Thanks for your contribution.

 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.Pok.Liu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Hi @rahult-github
Yes, I plan to replace the esp_blufi_adv_start function as I want to configure my own GAP broadcast data such as appearance and manufacturer_data .
As you can see in the ESP-IDF documentation The BluFi Flow:

Set the ESP32 into GATT Server mode and then it will send broadcasts with specific advertising data. You can customize this broadcast as needed, which is not a part of the BluFi Profile.

In Bluedroid powered BluFi component:
void esp_blufi_adv_start(void)
{
    esp_ble_gap_set_device_name(BLUFI_DEVICE_NAME);
    esp_ble_gap_config_adv_data(&blufi_adv_data);
}

I can implement my own esp_blufi_adv_start function to easily configure my GAP broadcast data by passing in a custom blufi_adv_data via the esp_ble_gap_config_adv_data function. Bluedroid implements the features described in the documentation.
In Nimble powered BluFi component:
void esp_blufi_adv_start(void)
{
    ...
    rc = ble_gap_adv_start(own_addr_type, NULL, BLE_HS_FOREVER,
                           &adv_params, esp_blufi_gap_event, NULL);
    ...
}

When I tried to implement my own esp_blufi_adv_start function, I found that esp_blufi_gap_event has a static flag which prevents me from calling it in my code.
Other than that, I have no idea to implement my own esp_blufi_gap_event function. But the esp_blufi_gap_event function calls esp_blufi_adv_start hard-coded when the BLE connection fails or disconnects, which should call back the user-defined esp_blufi_adv_start function. So I use the last parameter to pass the callback function of re-broadcast.

Hi @pokliu ,
I agree to your explanation. Can you please raise the PR on the master branch ( instead of release/v4.4 specific branch ?) Will include this change at all branches in that case.
Thanks,
Rahul

Hi @rahult-github
I have clicked the edit button on this PR page and Choose a base branch to master, at this time the page prompts merge 982 commits into master branch, I think this is not a normal operation.
I don't know much about this kind of operation, but I want these changes to be reflected on the release/v4.4 branch (currently using this branch), can you give me some hints how to the PR on the master branch.
Should I base my code changes on the master branch?
Thanks,
Pok

Hi @pokliu

I don't know much about this kind of operation, but I want these changes to be reflected on the release/v4.4 branch (currently using this branch),

So basically, we have master branch as the main development branch on which this change should go and then we will internally commit on all needed branches.

Should I base my code changes on the master branch?

Yes please .
Thanks,
Rahul

HI @pokliu ,
The PR you raised has multiple commits. Can you squash the commits to have a single commit ?  I can do it manually but it will override your contribution details , which i want to avoid.
Thanks,
Rahul

Hi @rahult-github ,
I closed the previous PR and raised a new PR #8977
Thanks,
Pok


Super Reduced String Solution issue #4

If anyone having any doubt can ask.

This PR will be counted in Hactoberfest or not ?
As of now it is showing Not-Accepted.

This PR will be counted in Hactoberfest or not ?
As of now it is showing Not-Accepted.

You have to wait for sometime for the not accepted to change to waiting. Don't worry unless it shows something like "PR on hacktoberfest-excluded repository". Rest, afaik there's no such issue yet, it will be included after 7 days


Install fails on MacOS Mojave

I have just updated my MacOS to Mojave 10.14.4 and I'm finding that zendesk_apps_tools will no longer run. Have tried to re-install/update but get errors when doing so.
partial error log here:
configure: error: in `/Users/alexfrench/.rvm/gems/ruby-2.6.0/gems/nokogiri-1.8.5/ext/nokogiri/tmp/x86_64-apple-darwin17.7.0/ports/libxml2/2.9.8/libxml2-2.9.8':
configure: error: C compiler cannot create executables.
I updated xcode command line tools. No change.
I drilled down a bit and I believe that the root of the problem may be the version of nokogirl (current version 10.3.1, version being installed by zendesk_apps_tools 1.8.5). 10.3.1 builds on Mojave. 1.8.5 does not (for me).
I'm not familiar with Ruby so would be great if someone could confirm this and offer a solution. Update to zendesk_apps_tools needed to support latest MacOS?
Thanks.

Hi @alexfrench, thanks for informing us of the issue. We've heard similar feedback about zendesk_apps_tools not working properly with Ruby 2.6.* and while this is something we'd like to patch in the long term, in the meantime can you try using Ruby 2.4.0 instead and see if it resolves the issue for you?

A belated answer to this. I managed to get things working again using Ruby 2.4.3.
For those not familiar with the Ruby environment, this link was very helpful - I used RVM to set the version of Ruby.
https://stackify.com/rvm-how-to-get-started-and-manage-your-ruby-installations


feat(SetTheory/Surreal/Basic): add some lemmas for Surreal.mk

add some lemmas for Surreal.mk

Split out of #14498
This contains all the lemmas in #14278 plus a few more. I was unable to reopen that one (I think I messed something up by doing a force push), so I am creating a new one.

bors merge


fix: Change UpdateUserMetadata to proper type

Type of change

[x] 🐛 Bug fix (non-breaking change which fixes an issue)
[ ] 🌟 New feature (non-breaking change which adds functionality)
[ ] 🔨 Breaking change (fix or feature that would cause existing functionality)
[ ] 📖 Docs change / refactoring / dependency upgrade to change)

Description
Fixes issue with incorrect type on UpdateUserMetadata which is currently set to interface{} to json.RawMessage which is what is needed to make the request work however anything currently passed into it such as a map[string]string is currently valid.
Related Issue (optional)
https://github.com/clerkinc/clerk-sdk-go/issues/109

@dgocoder Thank you so much for the PR. :heart:
How would you feel about changing the metadata fields in the UpdateUser struct so that they are typed as *json.RawMessage? Similar to how it is for creating a user.

@gkats  No issue in doing so however it seems across the board there is discrepancies. For instance in Org it is not a pointer and instead has omit empty on the json value.
https://github.com/clerkinc/clerk-sdk-go/blob/efd909d8d336ce7af469660779bf5f38dc706581/clerk/organizations.go#L32
I'm happy to change it but I followed pattern suggested in the issue which you referenced the org which did not use a pointer.

however it seems across the board there are discrepancies

@dgocoder You're right, apologies for this. Let's not increase the scope of this PR. We'll tackle all discrepancies one at a time.
:raised_hands: Thanks again for your contribution!


static server attempting to server JS as JSON

would nice to add namespace for JSON API such that static server can serve resources and don't conflict with REST services.

Sure, do you have a scenario that conflicts?
It will help to add the feature.

Example, index.html attempting to server /static/main.js. Workaround is to wrap json-server with your own and use app.use(namespace, jsonRouter(db))

Closing this issue since it's now possible to use the --router option to add a namespace/prefix.


windows package cache root switch does only work with "{root}" not this.root

The package cache root switch does not work with this.root in an envvar under windows.
Only if the the var is being set with "{root}/whatnot" the root switch to the local payload works.
The payload copy does work, only when resolving it wont use the local data even with REZ_READ_PACKAGE_CACHE=1
Changing the package from this.root to "{root}" will make it work.

Thanks, I'm gonna make it priority to fix this.

I tried to find the root (ha, what a semantic coincidence) of this, but with no success, still getting into the code base and reading.
Maybe it is the typical backslash windows exception... this.root was always used in conjunction with os.path.join ....
Would love to be able to make a PR but I am just not up to speed with REZ.

https://github.com/nerdvegas/rez/pull/1046/files
Is this related ?
Could be the issue itself.

Here is an example package to reproduce:
name = 'foo'
version = '1.0.0'
uuid = "8a004223-992e-47c8-8b0b-f6719ada4c15"

def commands():
    env.PKG_ROOT_STRING.append('{root}')
    env.PKG_ROOT_PYTHON.append(this.root)

With payload caching enable run rez env foo once to cache the package. In this first resolve the paths both point to the repository. Run again, this triggers payload caching to be used. Looking at the env var, the ROOT_STRING var points to the local payload cache, the ROOT_PYTHON var points to the package repository still.
Tested with 2.8x and pwsh

Ah, thanks. Now I understand the issue and able to reproduce.

So it looks like this line in ResolvedContext
https://github.com/nerdvegas/rez/blob/3f66e9ca1a0db227af608757e8dced08e88bd461/src/rez/resolved_context.py#L2007
is binding a installed variant, not a cached one.
Tested with this change can correct the result.
# Line 2007 in rez/resolved_context.py
pkg_bindings_["variant"].root = pkg_roots.get(pkg.name, pkg.root)
executor.bind('this', pkg_bindings_["variant"])

But that shouldn't be the right way to fix it, thoughts ? I think we should let the variant know it's been cached.


call depth can be > 1024 in javascript VM

Apparently it is 1025, see https://github.com/ethereumjs/ethereumjs-vm/pull/164.

Can this be closed?


Can't send file with same name and different content.

Hi
Amazing package! Using it daily, however, in my application, I'm always sending files with the same name, just changing the content, which requires a webpage refresh every time, because for some reason it doesn't let you send a file with the same name as the previous one but with a different content. In my case its .txt files with a few duzen characters.
You this get solder would be amazing, meanwhile, i'm looking for a way to make node-red automatically refresh the dashboard after each send.
Thank you.
Cheers, Luís Pereira.

Hello,
I tried briefly and cannot replicate the problem (tested with Firefox on Windows).
I am able to:

Click again on the "play" button and the same file is sent again;
Modify the file, click the "play" button and the updated file is sent;
Delete the file, replace with a file of the same name and different content; click the "play" button the new file is sent,

My guess is that some of it might be browser/platform specific. Could you please update to the latest version of this module and test some more?


ConVar ns_is_modded_server

Bug
Hello to whoever might be looking at this, hope you're doing well! :D
I updated Northstar to the newest v1.24.4 & everything worked perfectly fine as it always has, until when trying to connect to the Northstar servers, I started encountering this problem.
I checked & rechecked all installed mods, all were up to date & after some digging, I found that "ModSettings" by EladNLG was deprecated. Thinking this might fix my problem, I uninstalled it. Still nothing.
I've scoured all of Northstar GitHub, Wiki & Discord as much as I could for the past 2 hours trying to find a fix & the "most relevant(?)" info I could find was an indication that in release v1.19.6 the current problem I'm facing was fixed. So I installed v1.19.6 & still nothing.
Validated game files via Steam - still nothing. Used both -northstar & -northstardll in the launch properties of Steam - still nothing. Opened Nortstar via the manual launcher, Viper & VTOL launchers - still nothing.
I'm unsure whether it's something I haven't checked which could be causing this problem to persist? But if there's any further info which I can provide like logs, a video recording etc. please just indicate & I'll be happy to assist :}
I truly love the Northstar Mod & appreciate the hard work that you all put into it!
Steps to reproduce

Update to v1.24.4
Launch through any launcher &/or launch properties which opens Northstar normally
Once in the main menu & regardless of the server location, when clicking on "Launch Northstar", a menu with ConVar ns_is_modded_server pops up as the guilty party.

Specifications

Northstar version: v1.24.4
Platform: Steam

Thank you!
Just found the problem, it was an older mod "iTz Nao - Remove Locked Multiplayer button" which I now surmise was added as a permanent fix to Northstar.
I appreciate your assistance!

Great to hear you could resolve it :D


Failed to load resource http://localhost:3000/ng2-facebook-sdk

Browser gets stuck at loadiing.
Developer tools console shows the following
http://localhost:3000/ng2-facebook-sdk Failed to load resource: the server responded with a status of 404 (Not Found)
I have followed the steps in the readme. I am just running the basic example.
Thanks in advance for your help!

I'm guessing you're using  systemjs? Please see https://github.com/zyramedia/ng2-facebook-sdk/issues/25#issuecomment-263220566.

Thanks ihadeed, that was what I needed. I also want to use facebook login via pop up.Do you have an example I can refer to?
I tried using the UI method but that just takes me to a complete Facebook login in a pop up. I would like to just see the username and password frame in pop up instead.
FBDialog() {
this.fb.ui(
{
method: 'auth.login',
})
.then(
(response) => {
if (!response) {
console.log('dialog canceled');
return;
}

@pickAUserName3 see https://github.com/zyramedia/ng2-facebook-sdk#login


Official example in official doc throws error for "Layering State Template Files"

Hi thanks for the wonderful lib! I need to use the "Layering State Template Files" feature but it does not seem to work.
I followed step-by-step for https://github.com/roboll/helmfile/blob/master/docs/writing-helmfile.md#layering-state-template-files . Then it has the following error:
helmfile sync
in ./helmfile.yaml: error during mydefaults.yaml.gotmpl.part.0 parsing: template: stringTemplate:3:25: executing "stringTemplate" at <.Values.kubeContext>: map has no entry for key "kubeContext"

Full things:
(base) ➜  simple_example git:(master) ✗ ls
helmfile.yaml          mydefaults.yaml.gotmpl myenv.yaml             test.env.yaml
(base) ➜  simple_example git:(master) ✗ cat helmfile.yaml
# Part 1: Reused Environment Values
bases:
  - myenv.yaml
---
# Part 2: Reused Defaults
bases:
  - mydefaults.yaml.gotmpl
---
# Part 3: Dynamic Releases
releases:
  - name: test1
    chart: mychart-{{ .Values.myname }}
    values:
      replicaCount: 1
      image:
        repository: "nginx"
        tag: "latest"
(base) ➜  simple_example git:(master) ✗ cat myenv.yaml
environments:
  test:
    values:
      - test.env.yaml%
(base) ➜  simple_example git:(master) ✗ cat test.env.yaml
kubeContext: test
wait: false
cvOnly: false
myname: "dog"%
(base) ➜  simple_example git:(master) ✗ cat mydefaults.yaml.gotmpl
helmDefaults:
  tillerNamespace: kube-system
  kubeContext: {{ .Values.kubeContext }}
  verify: false
  {{ if .Values.wait }}
  wait: true
  {{ else }}
  wait: false
  {{ end }}
  timeout: 600
  recreatePods: false
  force: true%
(base) ➜  simple_example git:(master) ✗ helmfile sync
in ./helmfile.yaml: error during mydefaults.yaml.gotmpl.part.0 parsing: template: stringTemplate:3:25: executing "stringTemplate" at <.Values.kubeContext>: map has no entry for key "kubeContext"

You seem to be defining kubeContext value only for the test environment, while you're running helmfile with the default environment, that explains the issue for me.
Could you run helmfile -e test sync instead?

Thanks!


stdlib: use the correct va_list type on Windows

Windows uses char * for the va_list type even on x86_64.  Restore
the correct selection of __VaListBuilder for Windows x86_64.  This
partially fixes variadic functions.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test and merge

@swift-ci please test Linux platform

Build failed
Swift Test Linux Platform
Git Sha - 75e31751ba62eb54892f7277dac02b3412bf964a

@swift-ci please test Linux platform


Root Component showing error

Root Component Showing Error
Type '{ children: Element; }' has no properties in common with type 'IntrinsicAttributes & IProps'

It was pushed 👍🏼
#10

thanks

Getting another issue at launch Time, screen attached .

but if removed Root from above code then code will start working but alert will not be visible

@CodingByJerez

Strange it passed all the tests and for me it works.
You thought about clearing the caches?

i got problem , its  https://reactnative.dev/docs/usecolorscheme, in my project some components support class type and usecolorscheme is functional Hook that's why its crashing @CodingByJerez

0.2.1
Normally it should work with this patch tell me.

no, @CodingByJerez  if you replace useColorScheme with Appearance.getColorScheme() then only it work for both class and functional component

useColorScheme is not placed in a class, It's only because you are using an old version of react native below version 0.64


Coverage badge broken

https://coveralls.io/github/fastify/fluent-schema?branch=master doesn't return the coverage.
@mcollina it seems to me this is related to how fastify is handled by coveralls.io but I'm not sure

I can't really add/configure this. coveralls is totally broken for me as I have too many repos.

http://docs.coveralls.io/troubleshooting

I can’t add repos from an organization of which I’m a member.
If your membership of an organization is not publicized (your username isn’t listed on the >organization’s Github page), Coveralls can’t see it either, so you’ll need to upgrade your OAuth scope >to private here:
https://coveralls.io/refresh?private=true

it could be that the case?  or someone else from fastify can help on that. No rush I think it's nice to have


Support sample value updates, modification, and/or accumulation

Proposal
In complex systems it can be necessary to update or amend metric values after-the-fact. This can happen if a metric is emitted by multiple systems that may not report simultaneously, and due to implementation, the user wants the last one to "win". Or, a user wants an initial estimate value recorded ASAP, and then a slow cron job later updates the sample with a more accurate value. Or, a user may want a metric to be a summation of multiple reports of the same metric. These patterns are similar to Out Of Order, for which support was added recently. As a companion feature to OOO, it would be helpful if Prometheus could support metric updating.
We may also consider support for multiple types of updates: replacement is one option, but other metrics may want to accumulate by sum.
We should also consider how this is exposed in the API. Is it a new exposition format grammar, or would users prefer a new API endpoint similar to delete_series?
Would extra care be needed in the case of histograms?
Sketching out a possible approach via scraping/remote write, updates could be accumulated in the WAL/WBL and resolved at compaction time. Updates would probably not be viewable until compaction happens. Subsequent OOO updates would also not be visible until the OOO samples are compacted.

From other discussions, I understand that this issue has been raised before and it's something Prometheus chose not to support because it's out of scope for a monitoring db. In my experience, Prometheus is gaining use in business intelligence contexts that are slightly outside its original purview; contexts where it actually works quite well even if it's not a perfect fit. I think given the wider adoption it's seeing, it's worth revisiting the question.

Or, a user wants an initial estimate value recorded ASAP, and then a slow cron job later updates the sample with a more accurate value.

What blocks users to create a new series, with different interval, for more accurate slower value?


feat: Remove user part of http clone url

Some configuration of Bitbucket Datacenter seems to return username as part of the clone URL. This needs to be removed for .netrc to work properly.
Closes #3443

I assume you tested?

Yes, it is tested :-)


Irrelevant results

When doing a call with a url, I get the exact song. Which makes sense. But when doing a call with a search query, for example: 50 cent good die young. It will return an array with 20 results. I would expect the first result to be the best match and with such a specific query, to be the correct one. But instead, it returns '50 cent in da club'. And somewhere as 14th song in the array, is the correct song.
Am I doing something wrong?

the search results are provided by YT, not by us, but we have optimised the query encoding, feel free to retest it and reopen, if the result is still irrelevant, please provide an example


Please modify parsing for latest sg_edges_list format

Really, you don't need csv.reader() at all. Just use:
rows = line.strip().split()
That will split on all whitespace, including tabs, and it will drop leading and trailing whitespace.
For reference: PacificBiosciences/FALCON-integrate#45

Thanks for the heads-up, will fix this ASAP.

Sorry for the delay - this is now fixed for parsing latest sg_edges_list format


Refactor pastel-console tests to rely

Summary: Moves pastel console to rely tests
Differential Revision: D13476581

This is so good!


universal user cache following XDG_CACHE_HOME posix

universal user cache following XDG_CACHE_HOME posix compliance

There's a cleaner way to write this as two lines, this is hard to read

Merged this: https://github.com/tinygrad/tinygrad/pull/2235/files
Thanks for pushing for compliance with this


Complete branching steps for Rancher 2.9 for Fleet

Tasks

[x] change the rancher-version annotation to catalog.cattle.io/rancher-version: '>= 2.9.0-0 < 2.10.0-0' while adding charts to dev-v2.9 branch. We do this so we can add patches to a particular Rancher minor version, in this case it would be the Rancher 2.9 line.
[x] must have a new chart inside the rancher/charts repository that has a version 104.0.0+upX.Y.Z instead of 103.x.x+upX.Y.Z
[x] testing

All checkmarks set, done in "Fleet" and "Platform Team" - closing


Add newtypes for Monoid instances

We can support both *> (which is the default) and <*, so do
that.

Clean up some unnecessary unsafeCoerce and avoid some unnecessary
eta expansion.

This PR started out when this package was buried in dependent-sum. I dunno how it all got lost...

Hmm... looks like the lower bound's gone up and I can cut some CPP....

Oh, not quite that, no...

There's more happening than just Add newtypes for Monoid instances in that commit.
There is canonical Applicative f => Monoid (Some f) instance already. Use either Backwards or Dual to reverse it. Looks like I arrived to the same conclusion as in https://github.com/obsidiansystems/dependent-sum/pull/37#issuecomment-604164111

Do you want me to reopen with just the other stuff then?

@treeowl I didn't look closely what else is there. If you think it should go in, I'll happy to take a look in the separate PR.


Draft: switch to another rollup snowpack plugin

Fixes: #10
I was able to open the Datepicker-Demo with this locally without any problems. Seems to be working from what i can see.

If you run a build, as if you were publishing to npm - does the component still work?
The demo locally doesn't bundle at all, which is why swapping out the plugin makes no difference :)

If you run a build, as if you were publishing to npm - does the component still work?
The demo locally doesn't bundle at all, which is why swapping out the plugin makes no difference :)

Ahh I get it, but then i have literally no idea which options i should provide to the rollup config.

Ahh I get it, but then i have literally no idea which options i should provide to the rollup config.

I've just made the rollup plugin public - seems easier!
Give it a go and see if it works now :)

I've just made the rollup plugin public - seems easier!
Give it a go and see if it works now :)

Works fine now. Thanks!

Works fine now. Thanks!


Add support for "private" API methods

While technically all API methods are "public" (i.e. their endpoints are visible to all callers), it may be beneficial to designate some as "private". This would not necessarily make the methods inaccessible, but either hide them from "public" API documentation or flag them as "private" in the generated documentation.
This designation would primarily be a hint to callers that the signatures of annotated methods may change within a given major version, whereas non-annotated "public" method signatures would not change.
Terminology options might include "private", "internal", "experimental", etc.

A new Internal annotation has been added for designating non-public methods.


facioacoustic ganglion development terms

A student from UCL (Mathew Tata) is annotating PMID:18356247, and needs some new terms to describe development of the acoustico-facial VII-VIII ganglion complex (UBERON_0012175). (The sibling trigeminal ganglion development terms are already in place).
I'd suggest the primary names to be:
facioacoustic ganglion complex development
facioacoustic ganglion complex morphogenesis
facioacoustic ganglion complex structural organisation
under 'cranial ganglion development ; GO:0061550' etc. With all the necessary intersections to the Uberon term.
For the definitions:
The facioacoustic complex comprises the vestibuloacoustic ganglion in its proximal part and the geniculate ganglion in its distal part (PMID:18356247).
synonyms:
acoustico-facial VII-VIII ganglion complex development/morphogenesis/structural organisation (UBERON_0012175)
facioacoustic ganglia development/morphogenesis/structural organisation (PMID:18356247)
I can add these in myself- just making a note here to keep track, and in case there's any comments.
thanks.
Reported by: rebeccafoulger
Original Ticket: geneontology/ontology-requests/10928

facioacoustic ganglion morphogenesis ; GO:1904836
Decided the structural organisation term isn't needed currently, so closing this ticket.


Add dialog to send bitcoin from wallet

There should be a button in the wallet balance tab to open the "send bitcoin" dialog.

https://github.com/yzernik/squeaknode/pull/663

https://github.com/yzernik/squeaknode/pull/663


Implement betweenness centrality algorithm

 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.zaixing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Feature/Apply Prettier to Ensure Consistent Styling Across Files

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
The files are not formated and are not following prettier standard.
Use Case
Not much but open-source repos standard.
Benefits
Helps in maintaining clean and formated code.
Add Screenshots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I have starred the repository

Can you assign me this issue?

@Aditya-PS-05 Assigned, Thank you.


[Feature] RFC6712 Certificate Management Protocol (CMP) Proxy support for cert-manager webhook.

Is your feature request related to a problem? Please describe.

RFC6712 Certificate Management Protocol (CMP) Proxy in cert-manager.

If you have an internal CA which is not allowed to create a SubIssuer Cert for the CA IssuerType of cert-manager. You need a simple way of proxying those requests through cert-manager to the external CA. Most of them inside of companies support CMP over HTTPS with different options of CSRs.
Describe the solution you'd like
Hints if this is possible with the current cert-manager or it needs to be implemented.
Describe alternatives you've considered
Implementation as an external Issuer. Then I need support for the development of such an adapter.
Is their any preferred library available or do you know a project which already supports this.
Additional context
Many big companies have internal CA's for their Server Infrastructure and don't want to rely on  external CA's for privacy issues They also have a PKI. Most of them support CMP over HTTPs or SCEP if its a Microsoft CA.
Environment details (if applicable):

Kubernetes latest stable
Local Deployment
cert-manager latest
helm

/kind feature

+1

Describe alternatives you've considered
Implementation as an external Issuer. Then I need support for the development of such an adapter.
Is their any preferred library available or do you know a project which already supports this.

This is the preferred way - if you can implement a controller as per the design document here: https://github.com/jetstack/cert-manager/blob/master/design/20190708.certificate-request-crd.md you should be able to 'plug in' to the rest of the cert-manager ecosystem neatly (i.e. utilising Certificate resources to get auto-renewal, and annotating Ingress resources to get 'automatic TLS').
You can see an example of one of these, built with kubebuilder here: https://github.com/smallstep/step-issuer/pull/3.
Note that I have linked to a pull request, as they are in the process of a few updates in order to support the latest cert-manager releases - this issuer was implemented at a point where our 'CertificateRequest' doc was not finalised, and there's been some nasty but necessary breaking changes since then. We don't intend to make any more breaking changes here however, so you should be good to begin building against the v1alpha2 CertificateResource resource type (like in that PR!).

This isn't something that we're going to be staffing ourselves at this point, so would be a good candidate for someone in the community to look into 😄
/priority backlog
/help

Also, whilst we talk about 'external' issuers here, it's worth noting that we are not against/adverse to the idea of an issuer that is developed externally moving into the project, albeit as a potentially optional component.
We've not yet worked out the best UX for 'external issuer management' within cert-manager, nor have we worked out what we want to do in this area, so for now it involves users installing the additional controllers/component themselves.
I can see this being a useful extension, so it may be a good candidate for us to explore how we could move it into the wider cert-manager project over time and what requirements there are to make installation as easy as possible for end users 😄

@ralienpp , I would also like to contribute!

@munnerz, does the API of cert-manager have provisions for revoking certificates?

This is a status update about the attempt to implement the feature. With @sathyanarays' help I was able to put together a minimal prototyping environment that would invoke the CMP-related logic. However, I was unable to implement a complete solution in the time I had allocated for this, because of struggles with the ASN1 library of Golang.
I describe the challenges I've encountered, perhaps someone can provide assistance, or keep this in mind when attempting it again in the future.
Go's ASN1 library is limited in its capabilities. When reasoning about this in the beginning, I wrongly assumed that if it is in the standard library - it must be a comprehensive implementation. However, that is not the case, in my opinion the documentation has many gaps. It might be enough for an experienced Go programmer, but if you're new to this scene - the few examples are not easy to follow and much is left as an exercise for the reader. One has to look for snippets in other open source projects to get ideas about how a given ASN1 feature can be expressed in Go's encoding/asn1 notation.
So, I failed to implement even a simple CMP request; the protocol has a lot of functionality - implementing it entirely would not be feasible unless someone who knows the ins and outs of asn1 in Go can assist.
Therefore I was thinking of a plan B - wrapping an existing client in a way that makes it usable from Go. For example, there is a full-featured CMP client in OpenSSL, which can be invoked by calling the openssl executable with specific command line arguments. There is also a C library that exposes a CMP API. In other words, the idea is not to re-implement CMP, but to leverage an existing implementation.
I would like to know whether such an approach would be acceptable for being merged into cert-manager. And if not, what would be a good way to decouple this logic into an external plugin, such that others can use it without having to maintain their own fork of cert-manager.
To summarize, my questions are:

Would the project be open for accepting an implementation which is not pure Go, but a wrapper around something else?
If yes, would (a) starting an OpenSSL process per request be accepted? What about (b) wrapping a C library?
If no, can this functionality be refactored into a separate plugin? It would not be a part of cert-manager per se, but could be installed separately by users who need it, without having to maintain their own fork of cert-manager.

For example, I feel rather skeptical about the idea of claiming CMP support with an implementation that spawns a new OpenSSL process for each request. However, in some environments this might be an acceptable approach (e.g. if one is more concerned with full support of all the protocol features, and is less concerned with the Go purity of the software).

Did you decide which CMP message types you want to support? Looking into LwCMP RFC I'd say:

IR
CR
KUR
RR (if cert-manager at all supports the concept of revocations)

This would reduce the complexity drastically.
One could even boil it down to support only P10CR. This wouldn't be according to the RFC but could by sufficient for this task.

I attempted to start with P10CR, and subsequently add others. But as described above, I didn't manage to get that far.


Union of array types does not work with slice() method

https://flow.org/try/#0FAMwrgdgxgLglgewgAnBAFAQwE7YFzICCumAngDwDOM2cEA5gHzIA+RJFEYAtgEYCm2RgEoCXPoOQBvYMmRQk1ZDAAWcSgBEE-SgDkEMAOoJsAa2QBeZDmwA6SgBs4UfugC0ARmEBuWcmz8MGDYKAAMvgC+wEA flow seems to be determining that arr.slice(-1) is necessarily an array of strings, which is not the case.

I agree that something is not right. It works if you explicitly cast union to $ReadOnlyArray:
function fun(arr: Array<string> | Array<number>): number {
  const thisDoesNotWork = (arr: $ReadOnlyArray<string|number>).slice(-1);
  return 0;
}

Isn't this expected behavior because of variance? Any value you pass in would only be a subtype of the input type. And because slice returns a T from Array<T>, it has to pick a T, which would mean the value is being cast to one vase or the other rather than treated as the original union type.

This is not the expected behavior. Array.slice only works if you cast the union type immediately before slice: https://flow.org/try/#0C4TwDgpgBAqgdgSwPZwCrmgXigQQE54CGIAPAM7B4JwDmAfFAD64HElwCuAtgEYR50A3AChhAMw5wAxsGRwoAEyQQyAOSTAA6kjwBrABSECALhZFSFKrQbN859tz4CAlKc69+UAN7CoUKSgUUMAAFghkACLKahraelDYRngAdGQANghSEPoAtACMziJ+eBDAHHjyAAwiAL6iEtKyKFCEaWRIUSrqWjoGSabwcuiQrlDuTt6+-oHAwWFkOG0d0d1xuglQhiawiCjDEM6pGVm5BUVQJWUVUNXCdeKSMnJQAO69ZABi1Nn9ZmyW1HoTD+pHG-DoozBeEmfgCcCCoXCnTIaw2Wzwpjs-0ogJsIIcHhcR0y2XyhSml3KVVqwiAA
All of these functions are doing the same thing. Why do the first two not work, yet the last does.

This now works.


feat(unconfirmed): Created v5 Fulcrum /unconfirmed endpoints

PR related to this trello task --> https://trello.com/c/75nZyYS8

:tada: This PR is included in version 1.40.0 :tada:
Your semantic-release bot :package::rocket:


Split out the Term Graph Loaders

Is your feature request related to a problem? Please describe.
Split out the Term Graph Loaders
Describe the solution you'd like
Split out the Term Graph Loaders
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Acceptance Criteria

GIVEN [a precondition]
AND [another precondition]
WHEN [test step]
AND [test step]
THEN [verification step]
AND [verification step]

Additional context
Add any other context or screenshots about the feature request here.

PR https://github.com/IBM/FHIR/pull/2813

The project has been split out and everything compiles.  #2798 will add additional checking to ensure the feature works as expected.


AttributeError: 'NoneType' object has no attribute 'Str'

after apispec 0.38.0 version apispec/apispec/ext/marshmallow/swagger.py has moved，replace with openapi.py and common.py. so in file flasgger/marshmallow_apispec.py can not import schema2jsonschema, schema2parameters from apispec.ext.marshmallow.swagger ,
but i change  to import from apispec.ext.marshmallow.openapi also get error  ImportError: cannot import name schema2jsonschema
when i install apispec with 0.38.0 version  (pip install marshmallow apispec==0.38.0)  it's ok.
so I think it‘s time to update document “pip install marshmallow apispec” to “pip install marshmallow apispec==0.38.0”  or upgrade origin code to match apispec latest verson .
however,I'm so. I don't know how to do it .  I hope somebody can fix it when he(or she) have time.

oh,I see master branch has changed ,waitting for new tag^_^

@suifengpiao14 looks like you found this already fixed on master, duplicates fix #205 .  Thanks for the report.


Suggestion: Gradually replace AI generated images

Currently the site uses AI generated images in it's definition popups and "Advanced Sections" buttons, but doesn't have any page detailing if these are ethically sourced or what their training data was:

The ethics behind generative AI is still under heavy debate, and a lot of people might view the site in a negative light if they spot AI generated images. People who are against generative AI are also likely to want safer AI, so keeping these images can be detrimental to the cause.
My suggestion is to gradually replace the AI generated images with ones contributed by artists. Preferably also doing steps to ensure it's easy to see what images still haven't been replaced, and making it easy for artists to contribute an image since not everyone knows how to use GitHub.
This of course excludes images like the ones on "What is interpretability and what approaches are there?" that are used as examples.

This is an interesting perspective. My (relatively uninformed) view was that the use of these images was debated if it was either 1) Emulating the style of a specific artist or 2) Replacing the work of an existing paid artist.
Given that it seems unlikely to me that we would pay artists to produce images for e.g. the glossary, the ethical constraints do not seem to apply as much to us. Still, if the perception is "AI-generated images -> bad", then it might be worth taking into consideration and switching to a paid bank of images.
At the moment, images for the glossary and icons for the sections are produced by prompting DALL-E, which I don't expect to have the best standards in terms of fair use of artists's work. If there is a platform that generates images that are more ethically sourced, I suggest we move to it even if the quality is lower. Unfortunately, if the people visiting the site cannot see the difference, this might not help the perception.

This is an interesting perspective. My (relatively uninformed) view was that the use of these images was debated if it was either 1) Emulating the style of a specific artist or 2) Replacing the work of an existing paid artist.

There's definitely variation in how people feel about it. I think within artist circles it's a lot more frowned upon than in other places, which makes sense since they're affected by it the most. But there's definitely people who are fine with it there too, or are fine with it if it doesn't emulate/replace work like you say.

Given that it seems unlikely to me that we would pay artists to produce images for e.g. the glossary, the ethical constraints do not seem to apply as much to us. Still, if the perception is "AI-generated images -> bad", then it might be worth taking into consideration and switching to a paid bank of images.

Obviously very difficult to tell how people will perceive it in advance, but I figure having a discussion about it is good. I personally don't think switching to a paid bank of images would be necessary. The rest of the website is open source both in content and code, so doing the same for images could work too.

At the moment, images for the glossary and icons for the sections are produced by prompting DALL-E, which I don't expect to have the best standards in terms of fair use of artists's work. If there is a platform that generates images that are more ethically sourced, I suggest we move to it even if the quality is lower. Unfortunately, if the people visiting the site cannot see the difference, this might not help the perception.

I'm unaware of any platforms that are more ethically sourced unfortunately. It might be smart to have a small text saying "Image generated with DALL-E" somewhere to not seem like it's being hidden? Not sure if that would change people's perception of it though.

I'm closing this because:

We have no sign at the moment suggesting users are not using the site because of AI images.
I agree with Murphant's reasoning about this particular usage of AI art not being much of an ethics violation.
This seems unlikely, but anyone bouncing off the site due to our use of AI art, I do not think they would read much of our content, as I am doubtful that the content team would argue that AI art is unethical to the degree artists claim it is.

Now, the AI icons for the advanced section are ugly. I would be very glad if ITER decides to replace it with something better. OR if we see that users prefer that we don't use any AI art, then I think we should remove it from the site.


send notification on action failure

send a Slack notification if Github Actions fails
Does this PR need a docs update or release note?

[ ] :no_entry: No

Type of change

[ ] :robot: Supportability/Tests

Issue(s)
Test Plan

[ ] :muscle: Manual

I've triggered a manual run on this branch at https://github.com/alcionai/corso/actions/runs/4729929001 just to see how this works.


[Feature][Helm] Enable sidecar configuration in Helm chart

Why are these changes needed?
This PR exposed an interface in KubeRay helm charts to enable users to configure sidecar containers (See the logging documentation and https://github.com/ray-project/kuberay/issues/353 for more details).
Explanation of the interface:

 .Values.fluentBitConfigMaps
fluentBitConfigMaps:
  - name: fluentbit-config
    fluentBitConf: |
      [INPUT]
          Name tail
          Path /tmp/ray/session_latest/logs/*
          Tag ray
          Path_Key true
          Refresh_Interval 5
      [OUTPUT]
          Name stdout
          Match *

.Values.fluentBitConfigMaps[0].name: Name of the ConfigMap.
.Values.fluentBitConfigMaps[0].fluentBitConf: Used to configure fluentBit. Note that this configuration assumes logs are mounted on /tmp/ray.

 .Values.{head, worker, additionalWorkerGroups.smallGroup}.fluentBitConfigMap
fluentBitConfigMap:
  enabled: true
  logVolumeMount:
    mountPath: /tmp/ray
    name: log-volume
  configMapName: fluentbit-config

enabled: Whether enable sidecar feature on the head, worker, and smallGroup pods. This configuration is optional, and the default value is false.
logVolumeMount.mountPath: The mount path of the log volume should be synchronized with .Values.fluentBitConfigMaps.fluentBitConf
logVolumeMount.name: This field should be synchronized with .Values.{head, worker, additionalWorkerGroups.smallGroup}.volumes.
configMapName: The config map name should exist in .Values.fluentBitConfigMaps.

Related issue number
Closes #511

Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

Sorry, I was a bit unclear in the issue description --
I was suggesting adding support for arbitrary lists of sidecar containers, as in
https://github.com/helm/charts/blob/master/stable/prometheus/values.yaml#L565
https://github.com/helm/charts/blob/master/stable/prometheus/templates/deployments/server.yaml#L149-L151
This would enable users to specify sidecar containers for log processing using any tool (not just fluentbit).
There are other uses for sidecar containers too (e.g. custom networking setups).