Project Administrators need Basic access level to Share Queries

Please clarify that a project administrator needs to have a Basic access level to create a query and save it to the Shared folder. Being just a project administrator is not sufficient.

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

ID: 54179351-62b2-c41b-949b-0652a0f1d354
Version Independent ID: 1ffcd0b3-e67b-6cfe-caa6-814a699627db
Content: Set query permissions - Azure Boards
Content Source: docs/boards/queries/set-query-permissions.md
Product: devops
Technology: devops-agile
GitHub Login: @KathrynEE
Microsoft Alias: kaelli

@salipour105 - Thank you for the feedback. I'm updating the article per your suggestion. Users with Stakeholder access cannot create and save queries in a Shared folder.  You need Basic access or higher.


Write a Scala program to convert weeks to seconds

Description
Write a Scala program to convert weeks to seconds
1 week = 7 days = 168 hours = 10080 minutes = 604800 seconds
Input  : 5
Output : 3024000

How to contribute

Save the solution in program/convertWeeksToSeconds.scala file
Add convertWeeksToSeconds.scala file in convert-weeks-to-seconds folder

!assign


Unitest fails

I get this error when I run tests for my d program that uses dyaml as one of its dependecies.
dub test --compiler=ldc2 results in:
D:YAML Resolver unittest
D:YAML Constructor unittest
std.file.FileException@/build/ldc-I3nwWj/ldc-0.17.1/runtime/phobos/std/file.d(2796): test/data: No such file or directory
----------------
Flags unittest
Program exited with code 1

Hmm this works well with my LDC
LDC - the LLVM D compiler (1.2.0):
  based on DMD v2.072.2 and LLVM 4.0.0
  built with DMD64 D Compiler v2.074.0

Can you still reproduce this?
I am closing this due to inactivity, but please feel free to reopen/ping if it still persists.


Chunking structure of the pyramids

Currently, I think, we are going to impose the chunking structure of the input dataset also into the pyramids.
Do we want to set a default chunk structure, so that we optimize for the map access that we are going to use the data for?
What chunk structure would we like to have on the data?
For the highest pyramid level we could use one chunk for the whole array, but for the lower levels this could be the same as the underlying tile size that we would like to aim for in the highest pyramid level.

We do not infer the chunking from the input but hard-code it to 1024 x 1024, see this line here: https://github.com/JuliaDataCubes/PyramidScheme.jl/blob/df90034d527fa66a97947a0d0790b106ea391bc1/src/PyramidScheme.jl#L137
Of course you are free to choose whatever chunk size you like here.


WIP - Bug 1857246: Attempting to update conditions on Conflict

Implements a workaround for a condition where other controllers updated
the Conditions before our "main" controller did the same.


Add color configuration options

I've set the default colors to white so users updating the plugin will have the same colors.

@bruno- any chance this gets merged in?

Hey, I don't fully understand why we'd want this feature in this plugin? Can't the user just set the color manually before the #{online_status} interpolation?

The user could instead do this:-
set -g @online_icon "#[fg=#73d216]✔"
set -g @offline_icon "#[fg=#fc4138]✘"

I'm happy for you to reject the PR but could we put this in the documentation to make it clear the users how to set custom colours per status?

@srathbone nice, i didn't think of that.
I guess we don't need this then, just an example in readme as you said.


Update architecture diagram picture on home

Now that we have an updated and much better version of the architecture (Code as Data + ML on Code) diagram (see https://github.com/src-d/backlog/issues/1385#issuecomment-483645134), it's time to update it on the landing with any adjustments necessary to make web-friendly.

Hi @marnovo the solution we're crafting has in mind desktop and mobile behavior for enhanced legibility. Having this in mind, we're going to have two images, the first one will be for desktop - horizontal layout - and the second one for mobile - vertical layout. With a css media query, we're going to hide & display each image.
Does it look good?
Desktop

Mobile

/cc @dpordomingo

@ricardobaeta yes that works fine.

Update: PR'd.

Update: Merged.

closing this per objective accomplishment.


[HOTFIX][BUILD] Do not use contains in Option in JdbcRelationProvider

What changes were proposed in this pull request?
This PR proposes the fix the use of contains API which only exists from Scala 2.11.
How was this patch tested?
Manually checked:
scala> val o: Option[Boolean] = None
o: Option[Boolean] = None

scala> o == Some(false)
res17: Boolean = false

scala> val o: Option[Boolean] = Some(true)
o: Option[Boolean] = Some(true)

scala> o == Some(false)
res18: Boolean = false

scala> val o: Option[Boolean] = Some(false)
o: Option[Boolean] = Some(false)

scala> o == Some(false)
res19: Boolean = true

@yhuai @zsxwing Do you mind if I ask to take a look please?

Test build #66519 has started for PR 15393 at commit 83df63b.

Test build #66521 has started for PR 15393 at commit d052ab6.

LGTM pending tests

Actually, I hit this before. Sorry, I did not catch it when I merging the PR. Next time, I will be more careful about it.

I apologize and will definitely be careful as well.

BTW, could you update the PR description, @HyukjinKwon ? Maybe, removing exists part?

@dongjoon-hyun I already did :)

Yep. I knew. What I'm asking is we don't need to explain 'contains' usage here in this PR.
I prefer not to advertise contains for Option[Boolean].
scala> val o: Option[Boolean] = None
o: Option[Boolean] = None

scala> o.contains(false)
res17: Boolean = false

scala> val o: Option[Boolean] = Some(true)
o: Option[Boolean] = Some(true)

scala> o.contains(false)
res18: Boolean = false

scala> val o: Option[Boolean] = Some(false)
o: Option[Boolean] = Some(false)

scala> o.contains(false)
res19: Boolean = true

Test build #66521 has finished for PR 15393 at commit d052ab6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66521/
Test FAILed.

Test build #66530 has started for PR 15393 at commit d052ab6.

test this please

Test build #66519 has finished for PR 15393 at commit 83df63b.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66519/
Test PASSed.

Merged build finished. Test PASSed.

Test build #66530 has finished for PR 15393 at commit d052ab6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66530/
Test FAILed.

retest this please

Test build #66540 has started for PR 15393 at commit d052ab6.

Test build #66540 has finished for PR 15393 at commit d052ab6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/66540/
Test PASSed.

Merged build finished. Test PASSed.

Merging in master.


stereoPan not working react-native >=0.56

Work well
stereoPan work well when I am use with react-native 0.55.4 in separate index.android.js and index.ios.js
Not working
stereoPan not working when I am try with react-native >= 0.56.0 and use only index.js .
I haven't try separate in react-native  >= 0.56.0.
Please help.

@chankroesna Did you make sure you are using MediaPlayer for your 0.56 app? You have to override the default react-native-video link behavior to not use ExoPlayer.

@cobarx How to use MediaPlayer instead of ExoPlayer?
I am facing the same problem either.


x/net/http2: expose which TLS cipher suites are HTTP/2 safe

Launching a web server with the SSL ciphers list taken from the TLS package does not work. I get the following error, that I can't get around programatically:
http2: TLSConfig.CipherSuites index 5 contains an HTTP/2-approved cipher suite (0x1301), but it comes after unapproved cipher suites. With this configuration, clients that don't support previous, approved cipher suites may be given an unapproved one and reject the connection.
What version of Go are you using (go version)?

1.14

Does this issue reproduce with the latest release?
yes (1.15)
What operating system and processor architecture are you using (go env)?
go env Output
set GOHOSTARCH=amd64
set GOHOSTOS=windows

What did you do?
I tried to launch a web server with a custom TLS config, with ciphers taken from tls.CipherSuites(). I do have one of these 2 dead ends:

Ciphers returned by tls.CipherSuites() are not in a suitable order with HTTP/2-approved ciphers first (I guess, they are not intended to be).
http2isBadCipher() in /net/http/h2_bundle.go is not exposed, so I can't sort ciphers accordingly on my own (I don't want to copy that function, because it seems it could get updated over time).

Here is a sample snippet:
		// Prepare list of accepted cipher suites
		var ciphers []uint16
		for _, cipher := range tls.CipherSuites() {
			if cipher.Insecure == false {
				ciphers = append(ciphers, cipher.ID)
			}
		}

		// Create TLS config
		tlsConf :=  &tls.Config{
			MinVersion:         tls.VersionTLS13,
			MaxVersion:         tls.VersionTLS13,
			CipherSuites:       ciphers,
		}

		// Create TLS web server
		server := &http.Server{Addr: listen, Handler: mux, TLSConfig: tlsConf}

		// Start TLS web server
		_ = server.ListenAndServeTLS("cert.crt", "cert.key")

What did you expect to see?
I just couldn't find a way to programatically order the ciphers as required, with the means available.
What did you see instead?
A critical error and program termination.

We might consider adding an HTTP2 bool field to the CipherSuite type, but why are you doing this in the first place? You are effectively setting CipherSuites to the default list, but with worse ordering. Moreover, you are using TLS 1.3, so CipherSuites is ignored because TLS 1.3 suites are not configurable.

That error does need some tweaking though, because the order doesn't matter unless PreferServerCipherSuites is set, and it could also just ignore the suites if MinVersion is VersionTLS13.

(MinVersion should/could be 1.2, my bad)
Could you please point me to the place where the default list of ciphers is selected/generated (and their order is decided), if CipherSuites is not set by me? I could not find it, so I didn't trust it and tried to set it on my own...

https://github.com/golang/go/blob/e6583dc95375c4e266bffab6f8888e8e557b6355/src/crypto/tls/common.go#L1418
It's not documented because it can change and evolve for security and performance reasons.

Hm, okay, I see... many places where things are defined/decided and generally little control, other than hardcoding / maintaining a custom list and always using the latest Golang version to compile, etc...
Somehow I'm starting to doubt whether ciphers hardcoded/compiled into the executable are a good idea... If something changes (e.g. a vulnerability comes up) one needs to have the source code and recompile the program...
I guess this issue can be closed, thank you!

One additional question:
"varDefaultCipherSuites" (i guess you've got a typo in that variable name, but luckily it's private ^^) is the default list in every tls.Config, right? So also third party packages, consuming tls.Config would use those good ciphers (unless explicitly set differently)...?

Hm, okay, I see... many places where things are defined/decided and generally little control, other than hardcoding / maintaining a custom list and always using the latest Golang version to compile, etc...

Most developers want us to make the decisions on specific algorithms and to keep those up to date.

If something changes (e.g. a vulnerability comes up) one needs to have the source code and recompile the program...

If a vulnerability comes up developers are expected to obtained a patched version and recompile, like any other vulnerability.

I guess this issue can be closed, thank you!

I am going to keep it open to expose a HTTP2 bool in CipherSuite. Thank you!

"varDefaultCipherSuites" (i guess you've got a typo in that variable name, but luckily it's private ^^) is the default list in every tls.Config, right? So also third party packages, consuming tls.Config would use those good ciphers (unless explicitly set differently)...?

Correct! What's the typo?

If a vulnerability comes up developers are expected to obtained a patched version and recompile, like any other vulnerability.

Well, not if cipher suites aren't part of the application, but instead outsourced to OpenSSL. Then OpenSSL (ideally) maintains a good and secure cipher set. And application adminsitrators can update OpenSSL and change the applied ciphers via application configuration... no need to re-compile.

Correct! What's the typo?

Why is "var" part of the variable name?
var (
	once                        sync.Once
	varDefaultCipherSuites      []uint16
	varDefaultCipherSuitesTLS13 []uint16
)

Well, not if cipher suites aren't part of the application, but instead outsourced to OpenSSL. Then OpenSSL (ideally) maintains a good and secure cipher set. And application adminsitrators can update OpenSSL and change the applied ciphers via application configuration... no need to re-compile.

I was referring to any vulnerability in Go, which compiles to static binaries.

Why is "var" part of the variable name?

Not pretty, but it's a somewhat common pattern to hide the global var of a sync.Once based function (which is the symbol name without "var").

Any new updates to when you would expose a HTTP2 bool in CipherSuite? Appreciate any timelines.

@rolandshoemaker, can you mail a CL adding an HTTP2 bool to the CIpherSuite struct?

Sorry to sound like am bikeshedding but I just added a review comment to the CL, and asked perhaps for UsableWithHTTP2 instead of just HTTP2, because that field is a read only property, and not a toggle to enable HTTP/2, hence let’s try to make it more descriptive.

After talking through this with the security team we're going to take a different path than was initially suggested here. Since this metadata is not a property of cipher suites themselves, but rather a property of the HTTP/2 protocol (by which I mean certain cipher suites don't support HTTP/2, HTTP/2 supports certain cipher suites) we're going to opt to expose this in x/net/http2 instead.
x/net/http2 already has a private function, isBadCipher, which provides this functionality. Current proposal is just to make this function public. Given we've already hit the freeze, and this extends the API surface for a package which is vendored into the standard library, this is likely to get pushed to 1.17.

Thank you @rolandshoemaker for relaying back the new direction. Punting accordingly to Go1.17. Happy holidays!

Given we've already hit the freeze, and this extends the API surface for a package which is vendored into the standard library, this is likely to get pushed to 1.17.

Did this make it into Go 1.17?
Can an example of usage be provided?

This change has not been committed to any version.
CC @rolandshoemaker


should graphhopper out put some map data statistics on properties file?

Like the boundingbox, the node number, edge number, number of restrictions.
number of shortcuts.

The logging statement is not sufficient? You mean as some kind of checksum or for what you would need that.

Maybe related to #285 ?

@karussell the logging statement is good. But it need to start grasshopper. :)


New Component: "MultiRow" (or something)

The idea behind this component is to facilitate a grid of items where the content is unknown, but needs to be arranged in a grid design.
Why
With float, you have to do some trickery to insert a helper div (usually called "row") every time you want the components to break onto a new line. This often leads to functions that are not very robust (for example, you want 4 items in each row, instead of 3) and can lead to other issues (for example, one item has content that is significantly longer than the others)
What
A component that uses CSS Grid can facilitate both of the designs enclosed. The scope of the component is to
a) provide equal columns
b) provide infinite rows
c) provide a flex / float fallback for IE11
Example UI Patterns:

@endymion1818 thanks for creating the issue. The current grid solution relies on flexbox layout rather than float, and the <Row /> component is there to determine the layout of the columns and compensate for the columns spacing on the sides. Regarding your concerns, you can add as many elements in a row as you want (you can pick any amount of columns you want through <Row />'s cols prop so you're not limited to the default 12). If one of the items is longer than the others you can always do <Col align="stretch" /> which is gonna stretch all the columns to the longest one in a row. Also, the layouts posted above can be achieved with the current grid solution.


Posting Tag Table may not necessarily exist, causes failure

Lever integration is enabled and table posting_tag has ✔

However table does not exist in target snowflake database - likely because we do not use posting tags?
Causes error:
Database Error in model stg_lever__posting_tag_tmp (models/tmp/stg_lever__posting_tag_tmp.sql)
  002003 (42S02): SQL compilation error:
  Object 'FIVETRAN_ETL.LEVER.POSTING_TAG' does not exist or not authorized.
  compiled SQL at target/run/lever_source/models/tmp/stg_lever__posting_tag_tmp.sql

Possible solutions:

Create the table myself in snowflake with proper columns (as described here in schema.yml) - this isn't a code fix, so I'd close this ticket
Open a PR against this package to enable manually toggling this off ?
Open a PR against this package to smartly check if this table exists and do something (create temp empty table?) if the table does not exist

I am happy to do these, though I am on vacation for the next two weeks

Hey @nelsonauner, thanks for raising this issue!  When you have this table configured to sync, but you don't have the table in your warehouse, it's due to not using the functionality in the source.  Do you know if you use posting tags in Lever?
As a quick-fix, you could test creating a posting tag in Lever to generate this table.
As a longer-term solution, we'll want to add functionality to manually turn this functionality off.  We can do this on our team over the next few sprints, or we're happy to review a PR if you want to do this.
cc: @fivetran-jamie

Thank you for the helpful comments! I've opened a PR and am eager for feedback. I will also start work on the model/transform package as well.

Thanks. I also didn't version up, hopefully you do that on your end somehow?


Feature Request: Gray protected branches if not enabled by repo flag.

It would be nice to gray out all Protected Branches entries if the repository setting "Enable protected branches" is unchecked. This is especially true in the case of adding a protected branch and forgetting to check the "Enable protected branches" box. Having a visual cue would be very helpful. Even a box saying that "The protected branches below are not enabled. Please enable Protected Branches in the repository settings above."
Alternatively, adding an Enable flag for each branch name and removing the repository-wide flag might be a better long-term option. The enable flag could just add a comment marker on the line in the gitolite.conf file.

Alternatively, adding an Enable flag for each branch name and removing the repository-wide flag might be a better long-term option.

Why not... For now I'm thinking about creating a relation table to manage users rather than a serialized field...

Why not... For now I'm thinking about creating a relation table to manage users rather than a serialized field...

It will be for next release.


Resolving build issues with powershell and M1 Apple Chip capability

Patched the dockerfile to comment out the installation of powershell and update the README file for troubleshooting several of the build issues related to M1 Apple Chip.
https://github.com/ossf/alpha-omega/issues/70 - Migrate the Toolshed powershell scripts to bash
https://github.com/ossf/alpha-omega/issues/81 - MacBook bug: Unable to locate package powershell
PR https://github.com/ossf/alpha-omega/pull/96 updated the branch. This push will update the dockerfile and readme

Looks great -- I don't think the .idea/ files are a problem (there are some .vscode ones too), but if you wanted to remove them, feel free.


Scoping for custom query in subsetMaf

Enjoying maftools. Excellent package, thank you!
I have an issue that I think may have more to do with the defaults in eval/parse than subsetMaf, but it is in using subsetMaf that I noticed this, and it is a bit inconvenient.
Seems that there is something odd about the behavior of the environment that a custom query is evaluated in. Using subsetMaf in the global environment, there is no problem, but from within a function it can't "see" an object that should be available to it, passed through a function.
MWE
library(maftools)
laml.maf <- system.file("extdata", "tcga_laml.maf.gz", package = "maftools")
laml <- read.maf(maf = laml.maf)

# In global env this works fine
classes = 'Splice_Site'
subsetMaf(maf = laml, query = "Variant_Classification %in% classes")

# But within my own function, I get an error...
myFun <- function(maf, myClasses){
	subsetted_maf <- subsetMaf(maf, query = "Variant_Classification %in% myClasses")
	return(subsetted_maf)
}

myFun(laml, classes)

Here's the output
> myFun(laml, classes)
Error in eval(stub[[3L]], x, enclos) : object 'myClasses' not found

Session info
> sessionInfo()
R version 4.0.2 (2020-06-22)
Platform: x86_64-conda_cos6-linux-gnu (64-bit)
Running under: Ubuntu 16.04.6 LTS

Matrix products: default
BLAS/LAPACK: /home/cait/miniconda3/envs/mtordash/lib/libmkl_rt.so

locale:
 [1] LC_CTYPE=en_CA.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_CA.UTF-8        LC_COLLATE=en_CA.UTF-8
 [5] LC_MONETARY=en_CA.UTF-8    LC_MESSAGES=en_CA.UTF-8
 [7] LC_PAPER=en_CA.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_CA.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] maftools_2.4.05

loaded via a namespace (and not attached):
 [1] compiler_4.0.2     Matrix_1.2-18      RColorBrewer_1.1-2 survival_3.2-3
 [5] R.methodsS3_1.8.0  splines_4.0.2      grid_4.0.2         data.table_1.13.0
 [9] R.utils_2.9.2      R.oo_1.23.0        lattice_0.20-41

Hi,
Thanks for using maftools and I am glad that you find it useful.
Regarding the issue I can not think of any workarounds for it. It seems to be an issue with cascading environments and scope of variables. I will read more and see if I can find any solution.

I am closing this issue for now since @ShixiangWang answer seems to work. Please feel free to reopen if necessary.


Add the ability to set a custom buffer capacity for console communication over websocket

Question:
why limit output size in streamFromTCPConnToWebsocketConn  in 1024 byte?
this cause containers web console crash when i use some command like 'pm2 list'
I will be very grateful if you can raise the limit
Thanks!

Hi @wangkai1994
It was added as a sane default. I could update it to another value, although I'm not sure what would help in your particular case?
What kind of error do you have when using this command in a container console?

@deviantony container console crash when i send command. I think it is over 1024 byte in one line.When i resize container console(.../docker/exec/{}/resize?h=100&w=89), container console is normal.

waiting for you update in next version~!

Ok, let me push an image with an upgraded limit for you to test.

@deviantony get. testing

@deviantony you can test this command:  echo '你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好我的朋友你好你好我的朋友你好我的朋友你好我的朋友'
i guess (character parse to byte) >> byte limit(1024)  in one line cause crash

it works

it is still not work ,i don't know why it is 1024？

I don't know but I can assure you that it was updated.

Are you using any reverse proxy in front of Portainer?

it is not work when i resize terminal,it crash because bytes length no enough.
so, can you reopen and fix it ?

@wangkai1994
Having another look through this... can you reproduce this problem when using the Docker CLI ?

@deviantony
thank you for the response.
no, using with Docker CLI is totally ok,and i test this for a long time.
i think it's not a good way to set a static value
or can you push a test image for set buffer as 20480?

I'm going to re-open this issue.
Our approach will be to provide a way to override the default value in the Portainer settings. A new input with "Console buffer size" as well as a descriptive text should be added in the settings and it will be used for each console session started from this Portainer instance.

@deviantony nice!

@deviantony let's check out this , i think it is the source of the bug, not the buffer capacity

@wangkai1994 I'm pretty sure that the same problem happens when you try to use vim inside a console.
Could be related to https://github.com/portainer/portainer/issues/322

@deviantony  so how to slove it ? it is important for debug

I investigated that a while ago and as stated in https://github.com/portainer/portainer/issues/322#issuecomment-330837029 I was not able to pinpoint what is causing the problem.
There are some workarounds regarding setting up the locales inside your containers that you can try though to see if that helps solve the problem here (and to determine if this is related to #322 or not).

i think it's not about docker , it's bug in portainer websocket implement, because it's work in docker exec

@deviantony i think you should convert Text messages to UTF-8 before send. Do you think so ?

@deviantony https://github.com/gorilla/websocket/issues/168

func streamFromTCPConnToWebsocketConn(websocketConn *websocket.Conn, br *bufio.Reader, errorChan chan error) {
	for {
		out := make([]byte, 2048)
                //Utf-8 is composed of 1-6 bytes and may be truncated by br.Read(). It will not be parsed by websocket when it is truncated. Is my understanding correct?
		_, err := br.Read(out)
		if err != nil {
			errorChan <- err
			break
		}

		err = websocketConn.WriteMessage(websocket.TextMessage, out)
		if err != nil {
			errorChan <- err
			break
		}
	}
}

@deviantony Utf-8 is composed of 1-6 bytes and may be truncated by br.Read(). It will not be parsed by websocket when it is truncated. Is my understanding correct?

@deviantony https://github.com/portainer/portainer/pull/2834

@wangkai1994 nice thanks ! Have you tested it? We'll try this soon.

@wangkai1994 nice thanks ! Have you tested it? We'll try this soon.
@deviantony
i write some simpletest unit and pass,but i don't have portainer dev env.
It's an important feature for us beaceuse we use portainer in our company prod.
Waiting for your merge and test ~!


Create full example about relayer + gas station

Add examples that show how developers can leverage both the gas contract and the relayer server to relay cross-chain transactions

@ravigautham @anthony-near is the multichain-gas-relayer service ready for us to start working on this ticket?

no. I will let you know when it is. Should be ready by Wednesday at the latest

awesome! tysm for the speedy reply!

Instructions on how to generate an example are here: https://github.com/near/multichain-relayer-server/blob/feature/MVP_ethers/integration_tests/README.md. I will continue testing tomorrow.

@bucanero ready for development here :)


adding nassma 2020 summer school (Istanbul)

Thanks for the contribution :)

Thank you. Great list
Le sam. 22 févr. 2020 à 21:26, Shashank Shekhar<EMAIL_ADDRESS>a écrit :

Merged #35 https://github.com/sshkhr/awesome-mlss/pull/35 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sshkhr/awesome-mlss/pull/35?email_source=notifications&email_token=ADXG3M44U5QRIFC3LOO7473REGC7DA5CNFSM4KYWHO32YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOW2IRHWI#event-3062961113,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXG3M72QORTZT4TU566OG3REGC7DANCNFSM4KYWHO3Q
.


Fix: content comparison

:tada: This PR is included in version 1.10.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Buffer overflow while using hash agg

Describe the bug
We meet a core dump when running the sql with hash agg. In this sql, the agg key is constant and one of the selected columns has Chinese value in it. After deep in it, we found that there is a buffer overflow problem when calling GetOrInsert.
Exception:
Capacity error: array cannot contain more than<PHONE_NUMBER> bytes, have<PHONE_NUMBER>

To Reproduce
The data of chinese_col needs to be all different.
select chinese_col, col2, col3, sum(col4) from table group by 1,2,3,4

Expected behavior
No exception or coredump thrown.

Using LargeStringHashMap instead of StringHashMap, which actually uses LargeBinaryBuilder to append string values, works with me for this problem.


Change simulator UI help text to refer to directory

When using the mode of the simulator in which you provide an existing index.json file, the help text said to provide the absolute path to the index.json file. However if you do so, the string index.json is then appended to this leading to errors such as:
RuntimeError: No `remote` provided, but local file /home/scott/scratch/dummy-mds-dataset/index.json/index.json does not exist either

This PR changes the help text to instruct the user to provide the path to the directory containing the index.json file, instead of to the file itself.
Before:

After:

@srstevenson ah thanks, nice catch! lgtm


(#10251) Creating RC tarballs should be handled by rake.

Previously generating tarballs for rc's required a lot of manual
intervention.  This fix corrects the versioning scheme so that
everything is handled automatically via the rake task.

This has been updated in the new commit.


Add ESP32 GENERIC build

ok I suck at github apparently but this pull request should include adding generic.sh to the esp32 build scripts which calls build_esp32 "ESP32_GENERIC"

@GanerCodes I've re-named generic in the latest commit, so this is going to fail. What do you want to compile for?

ESP32_GENERIC is the one I'm trying to compile for, I ran it in a docker container and it worked
https://gist.github.com/sekcompsci/2bf39e715d5fe47579fa184fa819f421

ESP32_GENERIC is the one I'm trying to compile for, I ran it in a docker container and it worked https://gist.github.com/sekcompsci/2bf39e715d5fe47579fa184fa819f421

This is OK, but please, fix the copyright at the top.

You've got to set generic.h to be executable.
chmod a+x generic.h

Merged. Thanks for contributing!


Load Bundle Services

Load the services defined in the services.yml configuration file from inside the bundle's DI extension class. This prevents errors during installation (service aimeos_context not defined) and on pages such as /list (service aimeos_page not defined).
If this omission was meant to be like this, could you update the documentation to specify how services were intended to be loaded :)

Hi Zander
The services.yml file should be loaded by the imports section of the config.yml file, shouldn't it?
imports:
    - { resource<EMAIL_ADDRESS>}

Do you have any pointers why it doesn't work in your case and you need to load it manually?
The functional tests for the checkout process fail now. Could you have a look why they do this?

That's probably a better idea, unfortunately the file Resources/config/config.yml doesn't contain any import sections in master or the latest version.
I'll have a look at why things are failing tomorrow, the dependency errors from ScrutinizerCI look a little strange :\

Did you added the scripts to the composer.json file:
    "scripts": {
        "post-install-cmd": [
            "Aimeos\\ShopBundle\\Composer\\ScriptHandler::installBundle",
            "Aimeos\\ShopBundle\\Composer\\ScriptHandler::setupDatabase",
            ...
        ],
        "post-update-cmd": [
            "Aimeos\\ShopBundle\\Composer\\ScriptHandler::installBundle",
            "Aimeos\\ShopBundle\\Composer\\ScriptHandler::setupDatabase",
            ...
        ]
    }

The Aimeos\\ShopBundle\\Composer\\ScriptHandler::installBundle script should add the necessary lines to your applications config.yml file automatically after executing composer update
Yes, the ScrutinizerCI error is strange and we've already talked to Johannes about this. He doesn't know either and suggested to add the composer.lock file as well which in fact fixed the problem. Unfortunately, the travis-ci tests will fail afterwards due to dependency problems.

I'll try again from a fresh installation of symfony/symfony-standard:2.6.7 and see why mine didn't work as expected.
However, it may also be a good idea to work with the service container through the bundle extension instead of Composer scripts in case someone decides to disable shop functionality (the bundle) at a point in the future.
I'll look into it further :)
I'll consider this PR obsolete for now.

A compiler pass might be a good idea to update the application config (without editing user files) on every cache update (either cache:clear or aimeos:cache).
And an interactive input asking for automatic update of the routing similar to the generate:bundle command.
Just a thought :)

Actually, it's indeed the responsibility of the bundle's extension class to load all the service definitions of that bundle. So the change proposed by @zanderbaldwin looks sensible to me.

@xabbuh This shouldn't have any effects for the tests, shouldn't it? What about the bundle specific config? Should this also loaded in the bundle's extension class? This would seem natural to me in this case.
@zanderbaldwin The composer script for installing the bundle was intended to make installation as easy as possible for new users, so it adds all necessary configuration to the application files. We could add a cache:clear as well to the documentation if that helps to prevent errors.
Alternatively, we could describe what has to be done to install the bundle manually. Perhaps it's a better idea for those who want to have full control instead of making installation more complicated?

@aimeos Which bundle specific config do you mean?

@xabbuh We need some default config for configuring the components: https://github.com/aimeos/aimeos-symfony2/blob/master/Resources/config/config.yml
This is different to normal bundles as we didn't want to reimplement all configuration options (ca. 1400) of the underlying library in the Symfony bundle and add default values for them.

@aimeos Then it should be loaded in the extension class too. However, I suggest to use the semantic configuration for these options even if that means a lot of work for you. But the benefit actually is that your users will be able to configure the bundle with more confidence as the configuration processor is then able to provide sensible error messages in case of misconfigurations.

@xabbuh Tried to change it like you've suggested but loading the config.yml in https://github.com/aimeos/aimeos-symfony2/blob/master/DependencyInjection/AimeosShopExtension.php fails:
$loader = new Loader\YamlFileLoader( $container, new FileLocator( dirname( __DIR__ ) . '/Resources/config' ) );
$loader->load( 'config.yml' );
$loader->load( 'services.yml' );

With only the services.yml it works fine. The error is
Symfony\Component\DependencyInjection\Exception\InvalidArgumentException: There is no extension able to load the configuration for "aimeos_shop" (in /aimeos-symfony2/Resources/config/config.yml). Looked for namespace "aimeos_shop", found none

The namespace "aimeos_shop" is defined in the Configuration.php (https://github.com/aimeos/aimeos-symfony2/blob/master/DependencyInjection/Configuration.php) if I remember correctly. Is this a dependency problem?

I've tried a lot to load the content of the config.yml file too including the possibility to use semantic configuration and defaults at least for the top level configuration but all ways had severe disadvantages. At the moment I believe that there's no viable way to get an elegant solution for this problem.
Nevertheless, loading the service.yml works fine so I will merge your PR now and try to get the tests on travis-ci running again.


Adjust page number for contact names business card field

Currently, the service is returning a null for page number for "ContactNames" field in business card recognition API's. This is a workaround for setting that on the client-side until the service fixes it in v2.1preview.2

/azp un java - formrecognizer - ci


Disable default-features on serde

Serde enables by default its std features, we do not want that.

Seems we're having the same problem as we did with tss-esapi - compiler too old for the log crate. Do we want to bump MSRV?

You could remove the change to ci.sh and rebase this onto main now.

Updated. Hopefully it'll pass CI 🤞


convert info to BlasInt in Cholesky(::LowerTriangular)

Fixes #143 by manually converting the system Int to LinearAlgebra.BlasInt which may not be the same on some system BLAS (like arch linux).

the real fix for this is upstream at julialang/julia#29732 but this will patch things over until the next release.

Thanks.


Divide by zero

In cpBodySetMass, cpBody.c, there is a division by zero when it is called from cpSpaceInit. Line 261 body->m_inv = 1.0f/mass The 'mass' has value 0.0f when called from cpSpaceInit in cpSpace.c at line 173, cpBody *staticBody = cpBodyInit(&space->_staticBody, 0.0f, 0.0f)
In C(99) the behaviour of division by zero is undefined.

C/C++ fpu exceptions are disabled by default, this issue was caught when working on a Delphi/Object Pascal translation of the header files.
I fixed it by explicitly setting it to infinity, instead of dividing by zero.
I like to run my code with NaN exceptions enabled.
You can use this:
bram@ubuntu64:~/src/Chipmunk2D/src$ git diff cpBody.c
diff --git a/src/cpBody.c b/src/cpBody.c
index a8e0797..8ad2bc9 100644
--- a/src/cpBody.c
+++ b/src/cpBody.c
@@ -258,7 +258,7 @@ cpBodySetMass(cpBody *body, cpFloat mass)

        cpBodyActivate(body);
        body->m = mass;
-       body->m_inv = 1.0f/mass;
+       body->m_inv = mass == 0.0f ? INFINITY : 1.0f/mass;
        cpAssertSaneBody(body);
 }

@@ -275,7 +275,7 @@ cpBodySetMoment(cpBody *body, cpFloat moment)

        cpBodyActivate(body);
        body->i = moment;
-       body->i_inv = 1.0f/moment;
+       body->i_inv = moment == 0.0f ? INFINITY : 1.0f/moment;
        cpAssertSaneBody(body);
 }

I created pull request for this fix, at:
https://github.com/slembcke/Chipmunk2D/pull/163


Revert "Create env/consolev2"

Reverts pinterest/teletraan#609

Commit #609 works as expected so no need to revert cc @yongwen


Bug: AWS account details failed to retrieve in higher TF version

Motivation
I am mostly opening this to spare some time when we decide to upgrade the TF version on Claudie.
Current Behaviour
I ran terraformer locally because I wanted to use a debugger for development. I had a TF version v1.7.2 and bumped into an error when initializing the TF project. This error also occurs on TF version v1.7.4 (currently latest).
jakub-test-xckgd7f       │ Error: Retrieving AWS account details: AWS account ID not previously found and failed retrieving via all available methods.
jakub-test-xckgd7f       │
jakub-test-xckgd7f       │ See https://www.terraform.io/docs/providers/aws/index.html#skip_requesting_account_id for workaround and implications.
jakub-test-xckgd7f       │ Errors: 2 errors occurred:
jakub-test-xckgd7f       │      * retrieving caller identity from STS: operation error STS: GetCallerIdentity, https response error StatusCode: 0, RequestID: , request send failed, Post "https://sts.local.amazonaws.com/": dial tcp: lookup sts.local.amazonaws.com on <IP_ADDRESS>:53: no such host
jakub-test-xckgd7f       │      * retrieving account information via iam:ListRoles: operation error IAM: ListRoles, https response error StatusCode: 403, RequestID: cdcacc3c-822a-430c-bedc-292aa99fb085, api error InvalidClientTokenId: The security token included in the request is invalid.

After downgrading TF to the v1.5.7 version everything worked fine.
Potential fix - to resolve this error in higher versions of TF. One needs to add skip_requesting_account_id  = true in the backend.tf for the backend  "s3" configuration (see).
Expected Behaviour
TF project successfully initialized.
Steps To Reproduce

Run Claudie locally with TF version v1.7.2
Try to build a cluster

We intend to migrate to OpenTofu for TF 1.5+. Let's keep this issue open to see if the bug reappears with OpenTofu 1.7.

I tried to replicate this bug on the current OpenTofu latest version (v1.6.2) and the bug didn't appear.

We are closing this one, because it isn't an issue in OpenTofu.


upgrade omniauth-saml

Summary
Security patch noise reduction for omniauth-saml to 1.10.1 which now requires ruby-saml >= 1.7 which fixed CVE-2017-11430, even though we already had ruby-saml 1.9 in our Gemfile.lock, so we were already patched up.
/domain @Betterment/test_track_core
/no-platform

Also hit CVE-2015-9284. This is a pretty tricky one because omniauth gem decided not to accept a fix (optionally) relying on rails CSRF protection in order to mitigate. They suggested creating an omniauth-rails gem which never got released. Subsequently Cookpad released a different gem that patches things up similar to the original omniauth proposal.
So now we're good if we added multi-SAML-provider support. I'm not sure how to get github alerts to be quiet up about our vulnerable version of omniauth now that we've mitigated. But lets get this in first.

<< domain LGTM


Cleanup, refactoring, and removal of duplicate code

This PR looks a lot larger/scarier than it actually is. I've broken it down below!
Adapter::get, post, put, patch and delete all had non-optional parameters for $headers and $query or $body. These have been made optional. I also re-ordered the parameters as $headers was never used while $query/$body were heavily used. I also condensed and removed some duplicate calls so that every call to Adapter::get/post sends that call to Adapter::request. It could use a magic method to do this but it might make it more difficult to test.
Adapter::get/post/put/patch/delete all had non-optional parameters for $headers and $query or $body. This has been standardized (reordered some) to array $data = [] and array $headers = [] as $headers is rarely ever used (if at all). Both parameters are optional now. This allows us to remove tons of unnecessary empty array parameters all over method calls and tests!
Adapter::get/post/put/patch/delete all had nearly-duplicate code. Each of those methods have been updated to call a shared Adapter::request() which accepts a method parameter first and then the standard string $uri, array $data = [], array $headers = [].
I've also removed the unneeded object casting for arrays that are sent to the API. In the case of things like organization => (object) ['id' => $orgId] it just adds unnecessary fluff when Guzzle json_encodes it, it all turns out to be the same json whether it's an object or array. Removing it allows us to remove tons of (object) casts all over the project.
All in all, this PR has a net of 100 deletions (173 additions / 277 deletions) which cleans things up a lot. Test suite is all passing and I've (actually) tested the majority of each endpoint through the API. There's no breaking changes unless someone has extended the class or calls methods on the adapter/client directly.
Further down the road, I'd like to have the adapter's methods return an object decoded from the json. Instead of having $body = json_decode($data->getBody()); in every API class method, it'd be handled within the adapter, removing tons of duplicate code. The error handling is already checked in the adapter, so it seems this would be a good place to handle it too.

I'm not a Cloudflare employee, but this PR looks good for me. I've added a small inline comment.


Copy-paste mistake in emacs instructions

At the end of the readme, (global-set-key "\C-c a" 'delete-trailing-whitespace) should probably be (global-set-key "\C-c a" 'line-up-words) instead.

Oops, indeed. Fixed in master


[BUG]: Default stroke color doesn't work

I have black (the top left color) as the default stroke color.
Whenever I add a new field the stroke color is as shown in the screenshot below.
The default font color (white in my case) works fine as can be seen in the screenshot.

@EricClaeys A few things

Currently the stroke colour against the field will not reflect the default colour, this is by design and intended to reflect the fact that the field is using the default. After the field has been saved it will change to the default.
In the screenshot above there is no 'Stroke Size' so nothing will appear

Cheers
Alex

I request a change to the design based on an inconsistency which confuses me:

The "Colour" field shows the default colour, if one is set, otherwise the gray/black boxes.  This makes perfect sense.
The "Stroke Colour" field ALWAYS shows the gray/black boxes, regardless of whether or not a default Stroke Color is set.

This is inconsistent.  I suggest having BOTH fields show the default color if one is set, otherwise the gray/black boxes.
After some more testing I noticed the default stroke color does in fact work, but the "Stroke Colour" drop down doesn't show the default until AFTER a stroke size it set AND the field is saved.

Fix added - In testing will be committed with other changes


redesign search

Hey @anna-kuzii,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 236822e0-f64e-11e9-9890-1bf57bc9f734

Hey @anna-kuzii,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 69142f40-f65e-11e9-9890-1bf57bc9f734


Add support to add data attribute to autocomplete

Add a data attribute to the autocomplete input field so that the value can be used when we pass data list to autocomplete plugin.
E.g.
$('input.autocomplete').autocomplete({
    data: {
      "Apple": 1,
      "Microsoft": 2,
      "Google": 3
    },
    data_attr : 'data-id'
  });

I'd rather have a callback when a value is selected.
const data = {
  "Apple": 1,
  "Microsoft": 2,
  "Google": 3
};
$('input.autocomplete').autocomplete({
  data: data,
  select: function (target, selection) {
    var val = data[selection];
    $(input).data('id', val);
  }
});

@yanickrochon Looks a good idea, let me find some time and come up with a better solution. Thanks

Closed due inactivity, feel free to reopen it if it is still necessary.


Update re: adding SSH key to the ssh-agent for Mac users (Issue #23159)

Added a note to step 3 of Adding your SSH key to the ssh-agent for Mac users, for whom the system may continue to ask for SSH passphrase on every git push.

Why:
Following the directions given under Adding your SSH key to the ssh-agent can result in Mac systems continuing to ask for SSH passphrase on every git push. The suggested edit resolves this problem, if imperfectly.
Closes: https://github.com/github/docs/issues/23159

What's being changed (if available, include any code snippets, screenshots, or gifs):
Adding an extra line to the note for Mac users on Step 3 of Adding your SSH key to the ssh-agent, instructing that ssh-add --apple-use-keychain ~/.ssh/id_ed25519 can be added to .zshrc or .zshrc to resolve the problem of being repeatedly asked for SSH passphrase.

Check off the following:

[x] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

@oC-n Thanks so much for opening a PR! I'll get this triaged for review :zap:


added support for trusted connections through odbc

I know this isn't the official repository, but zendframework/zf1 has been EOL'ed by now

Sorry, this is supposed to be a mirror of the official zend framework 1.x, just split into separate packages. I know the official repo is no longer maintained, but I'm not going to add features here. You're welcome to fork it though!


Git submodule parsing error

Issue Type: Bug
Steps to Reproduce:

Clone the repository https://github.com/ItachiSan/example_repo_with_submodules
Change the .gitmodules file, so that instead ofurl = https://github.com/ItachiSan/example_repo

the line isurl  = https://github.com/ItachiSan/example_repo

which has 2 spaces after url instead of 1.
Reload the window.

Expected:
Nothing changes.
What happens:
The submodule is now not detected.
Workaround:
Keeping the .gitmodules file in the initial format.
Possible resolution:
Improve the parser for the .gitmodules file.
System informations:
VS Code version: Code 1.34.0 (a622c65b2c713c890fcf4fbf07cf34049d5fe758, 2019-05-15T21:59:37.030Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.85GB (17.77GB free)

Process Argv

Screen Reader
no

VM
0%

Well, that was fast 😜 😄
Thanks @joaomoreno 😄


Fix/backend install

Mainly for the notes on installing a python environment:
pip install . installs the backend package as defined by the pyproject.toml, and checks for its dependencies and installs them if needed
pip install -e . does an editable install, so changes in the source python files are directly reflected in the environment without needed to reinstall (still need to restart the uvicorn server)
pip install -e .[dev] includes the "dev" group of optional dependencies.

Actually, I now checked again with pip install . and the dependencies also got installed, which is slightly surprising because they didn't before. I'm thinking maybe it was originally a problem with using some older version of pip that didn't support installing from pyproject.toml (even though I thought I'd checked for the right version).
In any case, the installation now works correctly when following the readme, which is most important!


Date Time Issue In Batch Request

Describe the bug I'm encountering an issue with the SAP Cloud SDK when generating a batch request in Node.js. The error occurs specifically when trying to handle a Date field (orderDate) during batch processing. The error message is:
Error during batch request: TypeError: momentInstance.unix is not a function
The issue seems to arise from the use of date handling and serialization, as shown in the error trace:
serializeFromMoment (/home/vcap/app/node_modules/@sap-cloud-sdk/odata-v2/dist/de-serializers/converters.js:62:38)
To Reproduce Steps to reproduce the behavior:
Set up a Node.js project using the SAP Cloud SDK for OData V2.
Create a batch request with a Date field formatted using a custom function like formatToTimestampDate.
Send the request to an OData service using the SAP Cloud SDK.
Observe the error when processing the orderDate field.
Expected behavior The orderDate should be correctly serialized and processed without throwing the momentInstance.unix error, and the batch request should execute successfully.
Screenshots N/A
Used Versions:
Node.js version: v16.13.0
npm version: 8.1.0
SAP Cloud SDK version: 1.59.0
Code Examples The code snippet below is where the error is triggered. The error occurs when setting the orderDate field in the batch request.
const formatToTimestampDate = (date) => {
const dateObj = new Date(date);
const timestamp = dateObj.getTime();
return \/Date(${timestamp})\/;
};
const createOrderRequestBuilder = zcssc_C_Order_HeaderApi.requestBuilder().create(
zcssc_C_Order_HeaderApi.entityBuilder().fromJson({
orderDate: formatToTimestampDate(req.data.createdAt),
currency: req.data.currency,
customerName: req.data.customerName,
grossValue: req.data.grossValue.toString(),
guid: req.data.guid,
itemsAddedOnline: req.data.itemsAddedOnline,
orderType: req.data.orderType,
orderTypeDescr: req.data.orderTypeDescription,
salesOrg: req.data.salesOrg,
plant: req.data.storeId
})
);
Log file Here’s the relevant part of the error stack trace:
Error during batch request: TypeError: momentInstance.unix is not a function
at serializeFromMoment (/home/vcap/app/node_modules/@sap-cloud-sdk/odata-v2/dist/de-serializers/converters.js:62:38)
at /home/vcap/app/node_modules/@sap-cloud-sdk/odata-common/dist/de-serializers/default-de-serializers.js:103:20
at /home/vcap/app/node_modules/@sap-cloud-sdk/odata-common/dist/de-serializers/de-serializers.js:28:28
Impact / Priority
Affected development phase: Development
Impact: Blocked
Timeline: Go-Live is in 1 weeks.
Additional context The issue appears to stem from how the SDK handles the Date field (orderDate). I’ve attempted various methods to format the date, including using the /Date()/ format as shown in the code, but it consistently leads to this issue.

this is complete code
const { zgwCsscSrv } = require('../services/ZGW_CSSC_SRV');
const { changeset, batch } = require('../services/ZGW_CSSC_SRV/BatchRequest');
const formatToTimestampDate = (date) => {
const dateObj = new Date(date);
const timestamp = dateObj.getTime();
return \/Date(${timestamp})\/;
};
module.exports = async (req) => {
try {
console.log("Starting batch request process...");
   // const service = await cds.connect.to('ZGW_CSSC_SRV');
    const { zcssc_C_Order_HeaderApi, zcssc_I_Order_ItemApi, zcssc_I_Pay_SplitApi } = zgwCsscSrv();

    // Create the header request
    const createOrderRequestBuilder = zcssc_C_Order_HeaderApi.requestBuilder().create(
        zcssc_C_Order_HeaderApi.entityBuilder().fromJson({
            orderDate: formatToTimestampDate(req.data.createdAt),
            currency: req.data.currency,
            customerName: req.data.customerName,
            grossValue: req.data.grossValue.toString(),
            guid: req.data.guid,
            itemsAddedOnline: req.data.itemsAddedOnline,
            orderType: req.data.orderType,
            orderTypeDescr: req.data.orderTypeDescription,
            salesOrg: req.data.salesOrg,
            plant: req.data.storeId
        })
    );

    // Create the order item requests dynamically
    const createOrderItemRequests = req.data.orderItems.map((item, i) =>
        zcssc_I_Order_ItemApi.requestBuilder().create(
            zcssc_I_Order_ItemApi.entityBuilder().fromJson({
                material: item.articleNumber,
                discountValue: item.discountValue,
                forced: item.forced,
                giftIndicator: item.isGift,
                itemId: (i + 1).toString(),
                netValue: item.netValue,
                saId: item.salesAssociateId,
                saName: item.salesAssociateName,
                saNumber: item.salesAssociateNumber
            })
        )
    );

    // Create the payment split requests dynamically
    const createPaymentSplitRequests = (req.data.paymentSplits || []).map(payment =>
        zcssc_I_Pay_SplitApi.requestBuilder().create(
            zcssc_I_Pay_SplitApi.entityBuilder().fromJson({
                amount: payment.amount,
                paymentCode: payment.paymentCode,
                paymentDesc: payment.paymentDescription,
                saleType: payment.saleType
            })
        )
    );

    // Combine the header, order items, and payment splits into a changeset
    const batchRequest = batch(
        changeset(createOrderRequestBuilder, ...createOrderItemRequests, ...createPaymentSplitRequests)
    );

    const token = req.headers.authorization?.split(' ')[1];
    const batchResponse = await batchRequest.execute({ destinationName: 'DRGUP_2', jwt: token });

    console.log("Batch request executed successfully:", batchResponse);
    return batchResponse;

} catch (error) {
    console.error("Error during batch request:", error);
    return req.error({
        code: 'BATCH_EXCEPTION',
        message: error.message || 'An error occurred while processing the batch request.',
        status: 400
    });
}

};

Hey @rpathak-max, I think you are using the wrong assumptions on how fromJson works. The docs are not very exact on that, but I think what you want is rather deserialization. Otherwise you would have to pass an instance of moment instead.
Let me know if that helps.

Hi @marikaner, than you for your reply. can you please guide me with any docs and any sample how can i pass instance of moment because i have tried with moment library but still facing that error. may be you can share some example of it if its possible.

Sure:
const moment = require('moment');

const createOrderRequestBuilder = zcssc_C_Order_HeaderApi.requestBuilder().create(
    zcssc_C_Order_HeaderApi.entityBuilder().fromJson({
        orderDate: moment(),
        // ...
    })
);

Is this what you tried?

No i have to pass the order date which i mentioned as request payload so first i passed that date to the moment and then assigned it i a variable then passed it to the orderDate can you please share the example of that like passing the parameter from postman and convert it into the instance. this is the format i am passing from postman "/Date(1603843200000)/"

As I pointed out before, you have two options:

Use our deserializer for this. I think this is the preferred option in your case.
Parse the date according to your needs: https://momentjs.com/docs/#/parsing/string-format/ and https://momentjs.com/docs/#/parsing/date/ might be helpful. Please note that the native JS Date object does not have timezone information so it can get lost.

Ok let me try this solution then i will let you know thank you for your support

Hi @rpathak-max , were you able to resolve your issue using @marikaner 's suggestion?

Hello @deekshas8,
I apologize for the delayed response, and I wanted to express my sincere gratitude for your support. I’m pleased to inform you that the issue has been resolved, thanks to your invaluable assistance. I greatly appreciate your patience and guidance throughout this process.
Thank you once again for your support.


[mass Obsolete] children of idiopathic disease

http://purl.obolibrary.org/obo/MONDO_0700007
idiopathic disease
http://purl.obolibrary.org/obo/MONDO_0008347
idiopathic and/or familial pulmonary arterial hypertension

http://purl.obolibrary.org/obo/MONDO_0700007
idiopathic disease
http://purl.obolibrary.org/obo/MONDO_0016352
idiopathic inherited hypercalciuria

@nicolevasilevsky I don't see idiopathic inherited hypercalciuria in the PR for https://github.com/monarch-initiative/mondo/pull/6703

good catch. :) the script threw up an error and it was because that term was already obsoleted, hence I did not obsolete it on the linked PR.


stages/sfdisk: support changing GPT partition attribute bits

The GPT attribute field is a 64-bit field that contains two subfields. The higher field is interpreted only in the context of the partition ID, while the lower field is common to all partition IDs. sfdisk supports to change this fields, and also a osbuild user might be interested on change these bits. This patchset adds support to change these bits in the sfdisk stage.
The meaning of these bits (from Wikipedia) :

0 Platform required (required by the computer to function properly, OEM partition for example, disk partitioning utilities must preserve the partition as is)
1 EFI firmware should ignore the content of the partition and not try to read from it
2 Legacy BIOS bootable (equivalent to active flag (typically bit 7 set) at offset +0h in partition entries of the MBR partition table)
3–47 Reserved for future use
48–63 Defined and used by the individual partition type

Pushed a new branch with the requested changes. Using an array of integers and removed the debug message introduced by mistake.

Restarted the test.


Drop ipython install requirement?

Why do we have ipython as an install requirement for Persephone in setup.py?
https://github.com/persephone-tools/persephone/blob/11c68545e358125d6d523aa5ae39ee65f10a04c7/setup.py#L20-L21

This is the sort of thing I would package in say a Docker image because there's definitely some utility in having it there for the tutorial. I just am aware of trying to make a minimal amount of requirements for the main library.

Agreed. Having it as a dependency in the docker image only and not the main tool makes sense. It would add one more step in the "native" install tutorial though.


Nested languages

When using nested syntaxes, like javascript or markdown, the syntax highlighter is still action as if the code is slim. Here is an example:

I had a look at the syntax definitions and can see that there are filters defined for all the nested languages. I did not find any references to them in the main syntax file however.
I know very little about how the syntax highlighter definitions work, but I'm willing to learn if you could point me to some resources on the subject and maybe mentor me occasionally. I'd be happy to fix this myself, if I could ;)

@d-Pixie Been a while, but I believe the filters are now highlighting correctly. Please comment if you're still seeing this and I'll re-open.


osg Hugo de Groot

@Kubister11 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@Kubister11 You might miss my previous comment. Please react.

@Kubister11 I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


Add Redis probe to Skylight

Context
Skylight is not tracking the performance of redis. We can update this by explicitly adding redis in the skylight probes.
Docs
https://www.skylight.io/support/advanced-setup#probes
Changes proposed in this pull request
Skylight show the Redis commands in the Event Sequence.
Guidance to review

Link to Trello card
Trello Ticket

 [Apply] Add Redis probe to Skylight config


Error in doing ros2 launch isaac_ros_apriltag isaac_ros_apriltag_usb_cam.launch.py

I want to run apriltag by USB camera and I'm entirely followed the guidance in https://nvidia-isaac-ros.github.io/. But in the last step ,when I running   ros2 launch isaac_ros_apriltag isaac_ros_apriltag_usb_cam.launch.py, there will be errors :
[component_container_mt-1] 2023-12-08 15:28:28.409 ERROR /workspaces/isaac_ros-dev/src/isaac_ros_image_pipeline/isaac_ros_image_proc/gxf/tensorops/extensions/tensorops/components/TensorStream.cpp@104: tensor stream creation failed.
[component_container_mt-1] 2023-12-08 15:28:28.411 ERROR gxf/std/entity_warden.cpp@406: Failed to initialize component 00052 (stream)
[component_container_mt-1] 2023-12-08 15:28:28.412 ERROR gxf/core/runtime.cpp@685: Could not initialize entity 'QAOFSAUKML_resource' (E51): GXF_FAILURE
[component_container_mt-1] 2023-12-08 15:28:28.414 ERROR gxf/std/program.cpp@236: Failed to activate entity 00051 named QAOFSAUKML_resource: GXF_FAILURE
[component_container_mt-1] 2023-12-08 15:28:28.415 ERROR gxf/std/program.cpp@238: Deactivating...
[component_container_mt-1] 2023-12-08 15:28:28.420 ERROR gxf/core/runtime.cpp@1384: Graph activation failed with error: GXF_FAILURE
and at last
[ERROR] [1702020508.439103213] [rectify]: [NitrosNode] runGraphAsync Error: GXF_FAILURE
[component_container_mt-1] terminate called after throwing an instance of 'std::runtime_error'
[component_container_mt-1]   what():  [NitrosNode] runGraphAsync Error: GXF_FAILURE
[ERROR] [component_container_mt-1]: process has died [pid 49742, exit code -6, cmd '/opt/ros/humble/lib/rclcpp_components/component_container_mt --ros-args -r __node:=apriltag_container -r __ns:=/'].
I can never start it.

Could you provide some more information about your platform?

x86 or Jetson
GPU model / Jetson model
Software version (CUDA version, JetPack version)
USB camera model name/version

Were you able to run the AprilTag Quickstart prior to trying the USB Camera demo? If not, could you please run the quickstart first? This will help us understand if you're experiencing a problem with AprilTag as a whole or with the USB Camera component specifically.


SWUpdate warnings

$ swupdate -i iot2050-image-swu-example-iot2050-debian-iot2050.swu
Swupdate v2021.11.0

Licensed under GPLv2. See source distribution for detailed copyright notices.

[INFO ] : SWUPDATE started :  Software Update started !
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd0 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd1 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd2 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd3 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd4 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd6 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd7 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd8 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd9 - maybe not a NAND or raw device
[ERROR] : SWUPDATE failed [0] ERROR : cannot attach mtd10 - maybe not a NAND or raw device

@gylstorffq, can we get rid of these error messages and the underlying probing of mtdX?

I will look into disabling the MTD driver of SWUpdate.


Add AWS Lambda example

JAVA-4474
Note that there is only one example rather than a separate one for MONGODB-AWS authentication.  My argument for it is that the AWS credentials should properly be obtained by the driver via either environment variables or, perhaps, EC2 or ECS endpoints, and can be used just by specifying a MONGODB_URI environment variable like:
mongodb+srv://<host>/?authMechanism=MONGODB-AWS

I wonder if we should also have an example of fetching those AWS credentials since it would help fit in the existing page structure. Eventually, we'll pull in all the examples and put them in tabs [here|https://www.mongodb.com/docs/atlas/manage-connections-aws-lambda/].

The credentials are fetched internally to the driver via reading environment variables or accessing the EC2 endpoint (I think... I haven't actually tried it).  There is no code to show in the application for that, except perhaps showing the template for the connection string indicating the authMechanism of MONGODB-AWS.

Perhaps ExampleAWSLambdaHandler

The Java "standard" is to use lowercase for acronyms, so I went with ExampleAwsLambdaHandler


Translater not working since the update of August 16th

Hi,
The translater is not working since the new update. Is there a way to make it work ?
Tanks a lot,
Mejanos.

you can delete dinput8.dll and play. but some of the options will not be in English translated.

you can join the blue protocol translate discord server to find the current workarounds.

After the last update, the client patch is broken. To make the game start working, delete dinput8.dll
A new version dinput8.dll may be coming soon.
The update for the English server patch can be found in Discord

After the last update, the client patch is broken. To make the game start working, delete dinput8.dll A new version dinput8.dll may be coming soon.
The update for the English server patch can be found in Discord

May I know what is the Discord channel for this English patch sever?


clone in express route not exiting.. (for large repo)

Below route in node.js prints clone completed, but not sending the response back to calling clients. I tested the same code in node command line and it worked.  Also noticed that while the clone is in progess ,it repeatedly calls this route over and over.. not sure why that is happening also.
this is the server output when this route runs-
clone start
cloning..
start
clone start  <-- (**note the second recursive call here)
clone completed
(at this point browser is still waiting for response)
code
router.get('/CloneRepository', function (req, res, next) {
console.log('clone start');
var cloneOptions = {};
var cloneComplete = false;
fs.exists(repoPath, function (exists) {
    if (!exists) {
        console.log('cloning..');

        // clone into tmp folder
        var url = 'https://stash.xxx.com/scm/stp/salesforce.git'

        var clone = nodeGit.Clone.clone,
            userpassPlaintextNew = nodeGit.Cred.userpassPlaintextNew,
            user = 'XXX',
            pass = 'XXX',
            path = repoPath,
            options = {
                remoteCallbacks: {
                    credentials: function () {
                        return userpassPlaintextNew(user, pass);
                    }
                }
            };

        function url() {
            var url = 'https://';
            url += encodeURIComponent(user);
            url += ':';
            url += encodeURIComponent(pass);
            url +=<EMAIL_ADDRESS>            return url;
        }
        var surl = url();
        console.log('start');
        clone(surl, path, options)
            .then(function (repo) {
                console.log('clone completed');
                res.json('{success}');
            });

    }

});

});

I temporarily fixed this issue by moving the method out of router and putting it in a helper class.

I don't know how this would be calling the route again from inside itself, Without a bit more code (at least the basics of your route) there's not much any of us can do about this.
A few things seem out of scope, but I'm assuming they come from elsewhere in the route.
Since you said you've been able to get it to work when factored out into a helper, I'm going to mark this closed. If moving it back is a priority for you, try to provide a more complete code example (I realize privacy can be an issue here, so perhaps try stripping it down and pointing at a large public repo) and we can reopen this for you and take a closer look :+1:


Chat scrollbar is not responsive when pressed at the right edge of the viewport

Bug Report
Description
Bug Summary:
When the mouse cursor is positioned at the very right edge of the viewport, pressing the left mouse button does not impact the scrollbar at all.
You can't operate it normally without "pixel-hunting" the exact bounding box (every time), which is frustrating (also since it's especially thin).
Steps to Reproduce:

Position cursor at the very right edge of the browser's viewport.
Press the left mouse button.

Expected Behavior:
Should impact the scrollbar.
Actual Behavior:
Doesn't impact the scrollbar.
Environment

Open WebUI Version: v0.3.10

Operating System: Windows 11

Browser (if applicable): Brave 1.67.134 Chromium: 126.0.6478.186

Reproduction Details
Confirmation:

[x] I have read and followed all the instructions provided in the README.md.
[x] I am on the latest version of both Open WebUI and Ollama.

Installation Method
docker run -d -p 3000:8080 -e OPENAI_API_KEY=your_secret_key -v open-webui:/app/backend/data --name open-webui --restart always ghcr.io/open-webui/open-webui:main
Additional Information
This issue is has become a user experience plague recently. It's usually caused either by:

Using a non-native scrollbar (usually JavaScript based) on the body or another element
Using a scrollbar that's bounded on an inner element, like a DIV, instead of using the native BODY element's scrollbar, which should not create this problem (given there is no right margin). However, it seems like this doesn't always occur due to the inner element's scrollbar - for example see https://pi.ai/threads.

Notes
A few similar websites or services that are notorious for this issue:

Claude desktop web UI (recently introduced! Used to be fine!)
HuggingChat
Poe.com
Google Gemini desktop web UI
Discord!
Many others

A few chatbot UIs that are not suffering from this issue! (so much more usable!)

ChatGPT
Perplexity.ai
Pi.ai

Update:
After further trial and error, it seems that this issue occurs with some zoom levels but not with others.
For example a zoom level of 110% causes it, but 100% or 125% it does not happen.
I also tested other websites with the problem, but for a lot of them, like Claude.ai and HuggingChat, Poe.com the issue happened regardless of zoom level.
On Discord, changing the zoom level to something other than 125% fixed the issue.
Seems like it's a combination of a UI issue and possibly a Chromium scrollbar issue with different zoom levels (maybe related to rounding of viewport or element dimensions).


Error when searching for text after having loaded a "tag" page

In the browser, load a "tag" URL, http://bestof.js.org/#/tags/test for example.
Enter any character in the search text field => an error occurs:
Uncaught TypeError: Cannot read property 'id' of undefined

The error does not occur when loading the application from the root URL and navigating using the links (the bug was difficult to find!)

In TagFilterPage.jsx container, shouldComponentUpdate has been fixed:
  shouldComponentUpdate(nextProps) {
    return nextProps.tag && nextProps.tag.id !== this.props.tag.id;
  },

Fix included in release 0.2.6


Real-time ticker of transferred runes

Weighted by number of transfers.

As discussed previously, eligible tickers would need to be unpremined only, as to not directly contribute to the visibility of some runes that might be considered for profit.  It would be cool if not only did the ticker show eligible real-time rune xfers and mints and etchings, but also if it ran a list of upcoming free/fair-mint rune launches that are scheduled at a future block height.  I would think such a feature would need to include the option to disable the ticker, as well.
Would this ticker just be across the home page or would it persist as a header/footer across all ordinals.com pages?


az account get-access-token doesnt seem to work in multi-tenant scenarios correctly

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

If you log in to az cli with multiple tenants, and then run az account get-access-token with --tenant-id, it fails to retrieve the tenant. However, if you use --subscription with a subscription in that tenant, it works.
To Reproduce

az login (into tenant A)
az login --device-code (into tenant B)
az account get-access-token --scope "https://vault.azure.net/.default" --tenant
az account get-access-token --scope "https://vault.azure.net/.default" --tenant
az account get-access-token --scope "https://vault.azure.net/.default" --subscription
Here, the first  get-access-token call succeeds, the second fails, and the third succeeds.
Expected behavior

All three get-access-token calls succeed
Environment summary

az cli 2.33.0 running on WSL2
Additional context

@jiasli for awareness

Could you give more information on how the second command fails? Any error, output?

az account get-access-token --scope "https://vault.azure.net/.default" --tenant "<redacted tenant A>"
AADSTS50020: User account '{EmailHidden}' from identity provider 'https://sts.windows.net/<redacted tenant B>/' does not exist in tenant 'Microsoft' and cannot access the application '04b07795-8ddb-461a-bbee-02f9e1bf7b46'(Microsoft Azure CLI) in that tenant. The account needs to be added as an external user in the tenant first. Sign out and sign in again with a different Azure Active Directory user account.
Trace ID: 4a296d31-eab2-4719-a474-4c4f852b0100
Correlation ID: 8a2d8304-11b7-42df-ba85-4491756a9780
Timestamp: 2022-02-08 22:03:22Z
To re-authenticate, please run:
az login --scope https://vault.azure.net/.default

Also, i just noticed that my original post didnt escape the angle brackets correctly- updated in place for clarity

@yonzhan @jiasli Do you have any update here? This issue is still requiring us to use a custom AzureCliCredential wrapper that handles this behavior correctly as the one shipped with the Azure.Identity package does not handle this correctly.

I think I figured out the problem. You log into 2 tenants with 2 different users. After that, you can get expected token with --subscription, but not with --tenant.
There will be no error when the current user belongs to both tenants:
> az account get-access-token --scope "https://vault.azure.net/.default" --tenant 72f988bf-86f1-41af-91ab-2d7cd011db47
{
  "accessToken": <Decoded: "tid": "72f988bf-86f1-41af-91ab-2d7cd011db47">,
  "expiresOn": "2022-12-07 11:36:29.000000",
  "tenant": "72f988bf-86f1-41af-91ab-2d7cd011db47",
  "tokenType": "Bearer"
}
> az account get-access-token --scope "https://vault.azure.net/.default" --tenant 54826b22-38d6-4fb2-bad9-b7b93a3e9c5a
{
  "accessToken": <Decoded: "tid": "54826b22-38d6-4fb2-bad9-b7b93a3e9c5a">,
  "expiresOn": "2022-12-07 11:17:30.000000",
  "tenant": "54826b22-38d6-4fb2-bad9-b7b93a3e9c5a",
  "tokenType": "Bearer"
}

The tenant ID provided with --tenant matches the tenant ID of the access token.
I got the same error if the current user doesn't exist in the provided --tenant:
> az account get-access-token --scope "https://vault.azure.net/.default" --tenant 72f988bf-86f1-41af-91ab-2d7cd011db47
AADSTS50020: User account '{EmailHidden}' from identity provider 'https://sts.windows.net/54826b22-38d6-4fb2-bad9-b7b93a3e9c5a/' does not exist in tenant 'Microsoft' and cannot access the application '04b07795-8ddb-461a-bbee-02f9e1bf7b46'(Microsoft Azure CLI) in that tenant. The account needs to be added as an external user in the tenant first. Sign out and sign in again with a different Azure Active Directory user account.
Trace ID: 988ded24-207a-4a5d-9d98-680f78610100
Correlation ID: e33352c9-d0e4-4b0b-876b-9bcedbb9da27
Timestamp: 2022-12-07 02:27:18Z
Interactive authentication is needed. Please run:
az login --scope https://vault.azure.net/.default

This is because we disabled tenanted passthrough due to security:

https://github.com/Azure/azure-cli/issues/23255

The reason why subscription ID can be used to select user account is because subscription ID is the primary key of the list from az account list, but tenant ID is not. Tenant ID can lead to ambiguity. Imagine 2 entries for different users contain the same tenant ID, which user identity should we use?

[subA], tenantA, userA
[subB], tenantA, userB

So the behavior can be summaries as

If --subscription is provided, the user identity matching that --subscription is used
If --tenant is provided, the current user identity is used

This is how the user identity is selected (by --subscription or the current account):
https://github.com/Azure/azure-cli/blob/060b414d8ac5e4d8f4fc17cc2e23c2154c762226/src/azure-cli-core/azure/cli/core/_profile.py#L362
--tenant is then applied to the selected user identity:
https://github.com/Azure/azure-cli/blob/060b414d8ac5e4d8f4fc17cc2e23c2154c762226/src/azure-cli-core/azure/cli/core/_profile.py#L382

This topic was also previously discussed in https://github.com/Azure/azure-devops-cli-extension/pull/1226


Add path for LED identification

We can use the same one as the beaglebone.
Change-type: patch
Changelog-entry: Add path for LED indentification.
Signed-By: Lisandro Pérez Meyer<EMAIL_ADDRESS>

Can one of the admins verify this patch?

Hey @perezmeyer, thanks for the PR. Just a nitpick, all other boards override beaglebone except for the AI64, but the led path is the same for all. Why not remove the "beaglebone" override so it applies to all of them?

@acostach agreed, it's better that way.

lgtm


Add Xamarin.mac support, bump to SQLitePCLRaw 1.1.1

This pr's adds support to install the package on Xamarin.Mac unified application.
Also bumps support on SQLitePCLRaw since on 1.1.1 they are also now supporting Mac OSX.
Fixes #483

Do you have any plans to include support for Xamarin.Mac unified application ?

Hi! Thank you for working on this! I ended up changing your code a bit and so will close this PR.
I'm working on this in #593


Content-MD5 mismatch seems to be ignored

I'm working with api-auth-1.0.3 along with rails 2.3 on JRuby.
On debugging I've found that when
Header = "Content-MD5: random-sequence-of-chars-unrelated-to-body"
and that same value is used in the canonical string for the HMAC signature
@request.md5_mismatch? returns false

Thanks for reporting this issue! Please note that v1.0.3 is over 2 years old. It is possible that there was a bug with the way md5 calculations were handled in that version. Please updated to v1.5.0 or v2.0.0 to get the latest bug and security fixes. Please let us know if the issue still exists in the latest released versions.


Adblock passed with avast online security chrome extension

https://chrome.google.com/webstore/detail/avast-online-security/gomekmidlodglbbmalcneegieacbdmki

Hello, I don't understand the problem.
Thank you to give me a little more information.

When using that extension, Ads is blocked without detect adblock.

I installed the extension of Avast and I go on http://sitexw.fr/fuckadblock/. But it is detecting AdBlock.
Can you give me information on your config (OS, browser, ...), sample urls or even a sample code (on jsfiddle.net).
Thank you.

It block ad-network's address.

So, if it is real ad, user can block the addresss without detection.

Ok, I'll watch that.
But I think that to detect this, it will make an HTTP request. This is a similar subject for the detection method:
https://github.com/sitexw/FuckAdBlock/issues/18


Use Zewo/COpenSSL

/cc @tannernelson @loganwright

\o/

Linking ./.build/release/SocksCoreExampleTCPServer
Linking ./.build/release/SocksCoreExampleTCPClient
/app/.swiftenv/versions/DEVELOPMENT-SNAPSHOT-2016-08-23-a/usr/lib/swift/linux/x86_64/glibc.modulemap:412:10: error: header '///usr/include/uuid/uuid.h' not found
  header "///usr/include/uuid/uuid.h"
         ^
<unknown>:0: error: could not build Objective-C module 'CUUID'
Compile Swift Module 'TLS' (7 sources)
Linking ./.build/release/SocksExampleUDPServer
Linking ./.build/release/SocksExampleTCPClient
Linking ./.build/release/SocksExampleUDPClient
<module-includes>:13:10: note: in file included from <module-includes>:13:
#include "/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl3.h"
         ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl3.h:125:11: note: in file included from /tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl3.h:125:
# include "ssl.h"
          ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl.h:1699:11: note: in file included from /tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl.h:1699:
# include "dtls1.h"     /* Datagram TLS */
          ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/dtls1.h:257:5: error: unknown type name 'SSL3_BUFFER'
    SSL3_BUFFER rbuf;
    ^
<module-includes>:13:10: note: in file included from <module-includes>:13:
#include "/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl3.h"
         ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl3.h:125:11: note: in file included from /tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl3.h:125:
# include "ssl.h"
          ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl.h:1699:11: note: in file included from /tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/ssl.h:1699:
# include "dtls1.h"     /* Datagram TLS */
          ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/COpenSSL-0.8.0/Sources/include/dtls1.h:258:5: error: unknown type name 'SSL3_RECORD'
    SSL3_RECORD rrec;
    ^
/tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/Packages/TLS-0.0.0/Sources/TLS/Method.swift:1:8: error: could not build Objective-C module 'COpenSSL'
import COpenSSL
       ^
<unknown>:0: error: build had 2 command failures
swift-build: error: exit(1): /app/.swiftenv/versions/DEVELOPMENT-SNAPSHOT-2016-08-23-a/usr/bin/swift-build-tool -f /tmp/build_2b50065adcedfc4426b3906cf42b9699/vapor-engine-8e63410/.build/release.yaml
 !     Push rejected, failed to compile Swift app.
 !     Push failed```

Looks like this only works on `08-18`


shap for stacked LSTM

Hi, thanks again for this tool.
Are there any issues with stacked LSTMs, or should it just work out of the box?

GradientExplainer and DeepExplainer are both approximations, and I have not benchmarked them on stacked LSTMs, so keep that in mind. But that being said GradientExplainer will for sure work, and DeepExplainer should work as long as all the operations are supported (typical LSTM ones are).

@slundberg , when our model is a sequence model, the "order" of feature attributions matters, right?
If yes, we need to do something similar to what it's done in game theory: evaluate the contribution for all orders?

Well the ordering from Shapley values is designed to measure the importance of a member joining a group. For example if I join the group after you am I less valuable than if I were to join before you? So for sequences going into ML models we don't want to change the order of the sequence, but just mask different subsets (where those subsets are defined by an ordering of how we grow the size of the subsets).
The tricky part about sequence models is that each "feature" in the sequence is very dependent on what came before and after it. So "hiding" a word might make for a strange sentence or sequence if we don't account for the feature dependence. I have future ideas on how to handle that, but none of the current deep learning explainers handle feature dependence explicitly. I have been working on a benchmark for local interpretability methods (in the code right now), and once that gets published I plan to post results here publicly, so that might help better gauge how well we are doing for LSTMs.

@slundberg a benchmark for local interpretability methods? feature importances or does it include things like anchors and local decision tree surrogates? Benchmarking interpretability methods is also something I'm into (can't say more for now). Better not go into offtopic.


Dispatch information while loading the user

This PR is intended to correct the behavior described in #76 .

Hi @rmkpatchaa, I also needed to fix this in my own code. I think you should use the existing USER_NOT_FOUND constant instead of creating the new  LOADING_USER_END because if the code gets to proposed line 26 in loadUser, this really means that the oidc-client-js returned successfully and did not find a user.

Hi @Tankanow I've created the new LOADING_USER_END constant because it could be dispatched in many other cases (like a try, catch, finally -> LOADING_USER_END).
At this time we have only one case but we give also the possibility to someone to listen on the LOADING_USER_END action in his redux app.

Note that the USER_NOT_FOUND constant exists in the reducers already, but is missing from constants/index.js. The reducers can handle USER_NOT_FOUND the same way they handle USER_FOUND.

@rmkpatchaa you're proposed code does not dispatch LOADING_USER_END on USER_FOUND! I think USER_FOUND is a LOADING_USER_END event. To me, "loading user end" means either USER_FOUND, USER_LOADING_ERROR, or USER_NOT_FOUND.

Hi @Tankanow this makes sense. I've update the code to use the user_not_found action instead.

@rmkpatchaa, thank you for submitting this patch! I appreciate that you were open to my comments. Looking forward to getting this code in my app once it's merged!
Cheers!

@Tankanow you're welcome. The purpose was the same. Not the implementation. And I did it quickly to correct the behavior I had and did not take the time to think about other possibilities.
I hope it will be merged soon.

I have merged this manually, but without the LOADING_USER_END action. I hope this helps. Thank you for the PR.

Hi @maxmantz I think it should do the job. I'll test it later on my project.
Cheers.

@Tankanow  @maxmantz @rmkpatchaa - thanks!  Confirmed that in 3.0.0-beta.14 that the issue is resolved.  Also confirmed resolved: https://github.com/maxmantz/redux-oidc/pull/69.


contributor code of conduct

In an effort to establish a good and healthy environment for collaboration, I'd like to suggest that the project adopt the well-written Contributor code of conduct available at http://contributor-covenant.org and reproduced below. In the best of circumstances, we would just say that we agree with the message and we post the file into the root directory. In non-ideal circumstances, we have a document to point to which outlines a policy for handling unfortunate situations.
Feedback on this issue is encouraged!

Contributor Code of Conduct
As contributors and maintainers of this project, and in the interest of
fostering an open and welcoming community, we pledge to respect all people who
contribute through reporting issues, posting feature requests, updating
documentation, submitting pull requests or patches, and other activities.
We are committed to making participation in this project a harassment-free
experience for everyone, regardless of level of experience, gender, gender
identity and expression, sexual orientation, disability, personal appearance,
body size, race, ethnicity, age, religion, or nationality.
Examples of unacceptable behavior by participants include:

The use of sexualized language or imagery
Personal attacks
Trolling or insulting/derogatory comments
Public or private harassment
Publishing other's private information, such as physical or electronic
addresses, without explicit permission
Other unethical or unprofessional conduct

Project maintainers have the right and responsibility to remove, edit, or
reject comments, commits, code, wiki edits, issues, and other contributions
that are not aligned to this Code of Conduct, or to ban temporarily or
permanently any contributor for other behaviors that they deem inappropriate,
threatening, offensive, or harmful.
By adopting this Code of Conduct, project maintainers commit themselves to
fairly and consistently applying these principles to every aspect of managing
this project. Project maintainers who do not follow or enforce the Code of
Conduct may be permanently removed from the project team.
This Code of Conduct applies both within project spaces and in public spaces
when an individual is representing the project or its community.
Instances of abusive, harassing, or otherwise unacceptable behavior may be
reported by contacting a project maintainer at [INSERT EMAIL ADDRESS]. All
complaints will be reviewed and investigated and will result in a response that
is deemed necessary and appropriate to the circumstances. Maintainers are
obligated to maintain confidentiality with regard to the reporter of an
incident.
This Code of Conduct is adapted from the Contributor Covenant,
version 1.3.0, available at
http://contributor-covenant.org/version/1/3/0/

Thanks @samhatchett for putting this together.
I'm for this version but also like the guidlines version we have on the forum. Maybe we should have a one version for both places?

Looks good to me. Thanks Sam.

It''s good, and I also agree with @eladsal to have the same one for the forum as well as for Github

Code of conduct? Hahaha. I can't believe people take those things seriously.  Keep that nonsense out of engineering.
We don't need a code of conduct. FOSS is the first culture, in the history of mankind, that's truly international, post-monetary, and discrimination on the basis of race, sex, age, etc. is impossible. Welcome to the Internet. It unites the human race. And hackers invented it.
So we already know what we're doing. We did nothing wrong. So no one really takes the entryists who tell us otherwise seriously. I'm sure a small subset of engineers are dim enough to be fooled by the political agitators. But they shouldn't be the ones driving policy.

agitators? https://github.com/jart?tab=activity


Integration of plugin and VIC

Vsphere integrated Containers (VIC) have a different (from regulatr Docker-in-a-vm case) model of Engine/VM/Containers deployment. In VIC, one VM runs a docker engine and that handles creating containers (VM == container) on request. The container VMs can run on any node in a vSphere cluster. Also, the volume attach is done by the VIC engine creating the correct VM reconfigure spec, and running this when the container VM is forked.
So in order to use named VMDK volumes in VIC there need to be an integration with this mechanism.
This is a lower priority item, as VIC already auto-creates VMDK volumes for -v /path:/container_path option, so basic support of  external VMDK volumes is already there
Also, this may be better done by VIC team. Still, we need to decide when / who does the integration, and execute. Adding  tracking item

Based on discussion with VIC experts, there is no immediate integration required. Closing for now.


Minor bundle regression: Un-treeshakable "FsMigrations" import from knex

Issue
Version 0.2.0 added support for migrations! In that update, a new index-imported top-level file, src/fs-migration-source.ts, was added, which imports the following from knex:
import {FsMigrations} from 'knex/lib/migrations/migrate/sources/fs-migrations.js'

Unfortunately the knex package does not declare itself as "sideEffects": false the way kysely-knex does. As such, this import is not treeshaken away when not needed - even when KyselyFsMigrationSource is treeshaken away.
This issue manifests itself further downstream: Packages that import kysely-knex will end up bundling that particular knex file, which itself includes various imports from path, os etc. (potentially problematic in certain runtimes) and further bloats bundle outputs.
Repro
I can spin up a proper repro if requested, but basic, minimal example looks like this:
// src/my-file.ts
import {KyselyKnexDialect} from 'kysely-knex'

export function foo() {
  new KyselyKnexDialect({} as any)
}

Worst case this file, when bundled, will already include the entire fs-migrations.js file knex.
Best case, when declaring knex as external dependency, the dist output may look something like this (exact output depends on the bundler, target, etc.):
// dist/my-file.js
import { FsMigrations } from "knex/lib/migrations/migrate/sources/fs-migrations.js";
var KyselyKnexConnection = class {
  #connection;
  #knex;
  // ...
};

function foo() {
  new KyselyKnexDialect({});
}
export {
  foo
};

However, the next package that utilizes this, without declaring knex (or specifically knex/lib/migrations/migrate/sources/fs-migrations.js) as external, will in turn bundle said knex file into its dist output.

There may be better ways to handle this, but two potential solutions off the top of my head:

A: somehow mark knex/lib/migrations/migrate/sources/fs-migrations.js as tree-shakable? Unsure if this can be done in this package without requiring upstream changes to the knex package.
B: perhaps move the migrations feature out of the top-level index exports and into a separate subpath export? This would bypass the issue, but also represent a breaking change: Migration features would then need to be imported e.g. via import {KyselyFsMigrationSource} from 'kysely-knex/migrate'.

Great catch!
I'm leaning towards B, kysely-knex/migrations.

would a PR for option B be welcome? code change should be minimal and I can likely get to it later this week