Bob example does not compile

I tried to learn OCaml the hard way, by doing the exercises.
Since I got some error messages when trying to compile my own stuff, I just copied over the example. Trying to compile it yields the following error:
$ make
+ ocamlfind ocamldep -syntax camlp4o -package bin_prot.syntax -package sexplib.syntax,comparelib.syntax,fieldslib.syntax,variantslib.syntax -package core -modules bob.mli > bob.mli.depends
File "bob.mli", line 1, characters 10-13:
Parse error: [semi] expected after [sig_item] (in [interf])
File "bob.mli", line 1:
Error: Error while running external preprocessor
Command line: camlp4 '-I' '/usr/lib/ocaml/camlp4' '-I' '/home/norbert/.opam/system/lib/type_conv' '-I' '/usr/lib/ocaml' '-I' '/usr/lib/ocaml' '-I' '/home/norbert/.opam/system/lib/bin_prot' '-I' '/home/norbert/.opam/system/lib/bin_prot' '-I' '/home/norbert/.opam/system/lib/sexplib' '-I' '/home/norbert/.opam/system/lib/sexplib' '-I' '/home/norbert/.opam/system/lib/comparelib' '-I' '/home/norbert/.opam/system/lib/comparelib' '-I' '/home/norbert/.opam/system/lib/fieldslib' '-I' '/home/norbert/.opam/system/lib/fieldslib' '-I' '/home/norbert/.opam/system/lib/variantslib' '-I' '/home/norbert/.opam/system/lib/variantslib' '-parser' 'o' '-parser' 'op' '-printer' 'p' 'pa_type_conv.cma' 'unix.cma' 'bigarray.cma' 'bin_prot.cma' 'pa_bin_prot.cma' 'sexplib.cma' 'pa_sexp_conv.cma' 'comparelib.cma' 'pa_compare.cma' 'fieldslib.cma' 'pa_fields_conv.cma' 'variantslib.cma' 'pa_variants_conv.cma'  'bob.mli' > /tmp/ocamlpp1ae013

Command exited with code 2.
make: *** [test.native] Fehler 10

My env:
$ ocaml -version
The OCaml toplevel, version 4.02.1

$ opam --version
1.2.2

$ uname -a
Linux norbert-tuxedo 3.13.0-45-generic #74-Ubuntu SMP Tue Jan 13 19:36:28 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

I've tried to reproduce this with 4.02.1 but it compiles normally.
The message is suspicious though. If you look at
https://github.com/exercism/xocaml/blob/master/bob/bob.mli there are no
characters 10-13 on line 1. Do you have the same bob.mli file as is in
the repo?
On 08/12/2015 01:16 PM, Norbert Melzer wrote:

I tried to learn OCaml the hard way, by doing the exercises.
Since I got some error messages when trying to compile my own stuff, I
just copied over the example. Trying to compile it yields the following
error:
|$ make

ocamlfind ocamldep -syntax camlp4o -package bin_prot.syntax -package sexplib.syntax,comparelib.syntax,fieldslib.syntax,variantslib.syntax -package core -modules bob.mli > bob.mli.depends
File "bob.mli", line 1, characters 10-13:
Parse error: [semi] expected after [sig_item] (in [interf])
File "bob.mli", line 1:
Error: Error while running external preprocessor
Command line: camlp4 '-I' '/usr/lib/ocaml/camlp4' '-I' '/home/norbert/.opam/system/lib/type_conv' '-I' '/usr/lib/ocaml' '-I' '/usr/lib/ocaml' '-I' '/home/norbert/.opam/system/lib/bin_prot' '-I' '/home/norbert/.opam/system/lib/bin_prot' '-I' '/home/norbert/.opam/system/lib/sexplib' '-I' '/home/norbert/.opam/system/lib/sexplib' '-I' '/home/norbert/.opam/system/lib/comparelib' '-I' '/home/norbert/.opam/system/lib/comparelib' '-I' '/home/norbert/.opam/system/lib/fieldslib' '-I' '/home/norbert/.opam/system/lib/fieldslib' '-I' '/home/norbert/.opam/system/lib/variantslib' '-I' '/home/norbert/.opam/system/lib/variantslib' '-parser' 'o' '-parser' 'op' '-printer' 'p' 'pa_type_conv.cma' 'unix.cma' 'bigarray.cma' 'bin_prot.cma' 'pa_bin_prot.cma' 'sexplib.cma' 'pa_sexp_conv.cma' 'comparelib.cma' 'pa_compare.cma' 'fieldslib.cma' 'pa_fields_conv.cma' 'variantslib.cma' 'pa_variants_conv.cma'  'bob.mli' > /tmp/ocamlpp1ae013

Command exited with code 2.
make: *** [test.native] Fehler 10
|
My env:
|$ ocaml -version
The OCaml toplevel, version 4.02.1
$ opam --version
1.2.2
$ uname -a
Linux norbert-tuxedo 3.13.0-45-generic #74-Ubuntu SMP Tue Jan 13 19:36:28 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
|
—
Reply to this email directly or view it on GitHub
https://github.com/exercism/xocaml/issues/24.

As I told you, I've tried the example. I took the code from example.ml and pasted it into bob.mli.
So the errormessage does see some error there:
open Core.Std
          ^^^

At leat if I counted right.

Ah wait, into the mli, not into the ml. Read completely past that. The bob.mli is an interface file (think C header). If you copy example.ml to bob.ml and use bob.mli from the repo it should work.
Norbert Melzer<EMAIL_ADDRESS>schreef op 12 augustus 2015 18:59:58 CEST:

As I told you, I've tried the example. I took the code from
example.ml and pasted it into bob.mli.
So the errormessage does see some error there:
open Core.Std
         ^^^

At leat if I counted right.

Reply to this email directly or view it on GitHub:
https://github.com/exercism/xocaml/issues/24#issuecomment-130370104

--
Verzonden van mijn Android telefoon met K-9 Mail.

Now it works after I modified makefile slightly:
I needed to change the invoked command in test.native rule as follows:
test.native: *.ml *.mli
	@corebuild -quiet test.native -pkg oUnit

Perhaps this needs to be done elsewhere?

Just took a look into the current Makefile, the rule there is already like mine altered one. It seems as if I had an old Makefile lurking around, my first encounter of exercism was January or February, so thats about the age of the Makefile.

Hm, yeah. If you delete the file and fetch the exercise again, you should get the updated version.

@NobbZ did @kytrinyx's instructions fix the issue?

I have never tested and since I needed to reinstall my system from scratch in the meantime I won't have the opportunity to do that.
Anyway, I think we can consider this one fixed, so I close the issue.

I'm still seeing this, with an updated Makefile (I deleted the bob directory and re-fetched):
+ ocamlfind ocamldep -syntax camlp4o -package oUnit -package bin_prot.syntax -package sexplib.syntax,comparelib.syntax,fieldslib.syntax,variantslib.syntax -package core -modules test.ml > test.ml.depends
ocamlfind: Package `bin_prot.syntax' not found
Command exited with code 2.
make: *** [test.native] Error 10

bob master % ocaml -version
The OCaml toplevel, version 4.02.3

CAML_LD_LIBRARY_PATH="/Users/jkrmr/.opam/system/lib/stublibs:/usr/local/lib/ocaml/stublibs"
OPAMUTF8MSGS="1"
PERL5LIB="/Users/jkrmr/.opam/system/lib/perl5:"
OCAML_TOPLEVEL_PATH="/Users/jkrmr/.opam/system/lib/toplevel"

Curious, it's saying that the package bin_prot isn't installed which is a dependency of core.
I assume that you've got the core package installed. opam list will show you which packages are installed, it it's not installed do opam install core or perhaps just reinstall core e.g. opam  reinstall core.
Let me know how you go with that.

Hmm no dice. But very curious indeed. I'll poke around with it some more.
bob master % opam list | ag bin_prot
bin_prot             113.24.00  A binary protocol generator
pa_bin_prot          113.00.01  A binary protocol generator
ppx_bin_prot         113.24.00  Generation of bin_prot readers and writers from

bob master % opam reinstall core
The following actions will be performed:
  ↻  recompile core 113.24.00

=-=- Gathering sources =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=  🐫
[core] Archive in cache

=-=- Processing actions -=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=  🐫
⊘  removed   core.113.24.00
Processing  2/2: [core: make]
∗  installed core.113.24.00
Done.

bob master % make clean
rm -rf _build

bob master % make
+ ocamlfind ocamldep -syntax camlp4o -package oUnit -package bin_prot.syntax -package sexplib.syntax,comparelib.syntax,fieldslib.syntax,variantslib.syntax -package core -modules test.ml > test.ml.depends
ocamlfind: Package `bin_prot.syntax' not found
Command exited with code 2.
make: *** [test.native] Error 10

@tmcgilchrist I just got this working.
Steps taken to remedy:

Uninstall ocaml and opam using Homebrew
Following the installation instructions for Real World OCaml, installed opam using homebrew

i. Uninstalled the ocaml homebrew installs as a dependency of opam
ii. Installed ocaml using opam using opam init --comp 4.02.3
3. Completed the remaining steps in the instructions

Interesting. I'm sure I installed open via brew at one point then have
updated the ocaml version as new versions were released.
I'll have a go from scratch this evening. Sounds like we need to be more
specific with our install instructions. Thanks for persevering
On Monday, 29 February 2016, Jake Romer<EMAIL_ADDRESS>wrote:

@tmcgilchrist https://github.com/tmcgilchrist I just got this working.
Steps taken to remedy:

Uninstall ocaml and opam using Homebrew

Following the installation instructions for Real World OCaml
https://github.com/realworldocaml/book/wiki/Installation-Instructions,
installed opam using homebrew
i. Uninstalled the ocaml homebrew installs as a dependency of opam
ii. Installed ocaml using opam using opam init --comp 4.02.3
3. Completed the remaining steps in the instructions
https://github.com/realworldocaml/book/wiki/Installation-Instructions
—
Reply to this email directly or view it on GitHub
https://github.com/exercism/xocaml/issues/24#issuecomment-189963420.


Mirror View

There is a shouldMirror property, but it doesn't seem to be working, because I've set it to "YES" and the histogram isn't mirrored.

Do you mean this property? The implementation of ZLHistogramAudioPlot is rather different from that of EZAudioPlot and it does not make use of shouldMirror.

Yes, that's exactly what I meant. I realized that it doesn't use it, or rather I found out by going through the code. What I'm saying is that something similar should be implemented for ZLHistogramAudioPlot as well.


Expose Tangram's scene object as a non-private element

Current it's possible to get a hold of Tangram's internal scene object via the private Mapzen.js _tangram/_layer attributes.
It would be nice to have a public accessor for this. I feel like this is a conversation I've had with @migurski but I don't remember the outcome, if any.

Aaron and I talked about this, and I showed him that the tangramloaded event passed an event object with a tangramLayer property. It’s not documented at #events, so let’s add this to the docs. Ideally it would link to the Tangram docs explanation of what a Tangram layer is.

related issue : https://github.com/mapzen/mapzen.js/issues/248


Setup modes for Chromatic

📦 Published PR as canary version: 0.0.85--canary.114.997ff4b.0

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

I like it. Can you talk me through the choice to do 2-up views for the pages but 1-up views for the components? I'm not quite understanding that, TBH.

Twofold. Components are supposed to render nicely regardless of their container. They shouldn't really be affected by explicit breakpoints. But that's just from an idealistic view. The other reason is that this setup yields nicer images in Chromatic:

The 2-up really is designed to mimic the addon panel. It even has the appropriate borders and background to make it look like a panel. Throwing individual components on there feels a bit weird.
By the way I imagine we won't use the 2-up layout for fullscreen screens either (since they won't have horizontal/vertical layout variants). I'll rename componentModes and screenModes to better reflect that.


Sign clickEvent json doesn't work properly

When you setup a sign that has clickEvent of action run_command in one of the lines, the command will never run. This has something to do with the command never running and after that the command itself never being executed.
I'm looking into this further.

Found the problem, the command executor does ((CommandSource) sender) to cast the ICommandSender to a SpongeAPI CommandSource, but the sign uses an inner class which isn't mixed in. This should be made into an outer class in later version of 1.8 but for now I will provide a mixin to fix this.

Then it's probably not a SpongeVanilla bug, but actualy a bug in SpongeCommon (because Forge should also have this problem)

Yes, it's an issue with SpongeCommand, this is also only one of the issues, I'm still figuring out why the command fails to be retrieved too.

yeah, this should be fixed in Common. You'll just need to do the same sort of wrapping as in ExecuteAt

Found it I needed to add Subjectable too, I added the inner class as a target of MixinSubject and MixinCommandBlockLogic since these should be treated similarly. That fixed both issues.


Add withJsonBody

Implements withJsonBody as discussed in https://github.com/evancz/elm-http/pull/19
Cleans up docs: indentation, withHeader signature

@ento this is excellent, thank you! I'll merge and publish this tomorrow.

@ento published as 5.2.0, thank you!

Thanks for the review & merge!


Release v1.6.3-up.1

What problem are you facing?
Release UXP v1.6.3-up.1.
How could Upbound help solve your problem?
Release by following https://github.com/upbound/universal-crossplane/blob/main/CONTRIBUTING.md#release-process

[x] Release UXP v1.6.3-up.1
[x] Publish to OLM
[ ] Publish to AWS Marketplace
[ ] Publish to Rancher Marketplace

Release: https://github.com/upbound/universal-crossplane/releases/tag/v1.6.3-up.1

https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/863
https://github.com/k8s-operatorhub/community-operators/pull/817

https://github.com/rancher/partner-charts/pull/355


Resuming experiment results in incorrect test logs

I've been seeing this phenomenon in my experiments, when restarting from already loaded weights. Could be related to #19? Note that logging is fine for training.
Attached are the relevant config files (in a tarball).

configs.tar.gz

The issue here was that you switched load_type from train to train_val. It was originally validating on your training set and then when you loaded it started validating on your validation set. Since the sizes are different for len(val_loader) it plotted at a different point. We are going to update this so that all plots use the same x axis #24


dragCrop: false is not works on latest version

$('.cropper-example-1 > img').cropper({
aspectRatio: 16 / 9,
autoCropArea: 0.65,
strict: false,
guides: false,
highlight: false,
dragCrop: false,
cropBoxMovable: false,
cropBoxResizable: false
});
version 1.0 - works perfect
latest version - not works.
I can still create a new crop.

The old dragCrop option had renamed to dragMode as of v2.0.0.


Initial split into new repo

Argh wait, I didn't realise I needed to update this to latest e-g master - new tinybmp changes just went in!

Ok, this is ready now.


Add a "dismiss" button to ClusterDetails

Probably (but not necessarily) should go to top right. Currently the only way to dismiss is re-clicking the cluster

Hi, I can take this issue!


Can't build in GitHub Action

[ ] https://github.com/gradle/gradle/issues/30236

It's fine after switch to ModDevGradle


chore(blob/cassandra-pod-delete): Adding an ability to generate detailed chaos result

Signed-off-by: shubhamchaudhary<EMAIL_ADDRESS>

chaosresult
Name:         engine-cassandra-pod-delete
Namespace:    shubham
Labels:       name=engine-cassandra-pod-delete
Annotations:  <none>
API Version:  litmuschaos.io/v1alpha1
Kind:         ChaosResult
Metadata:
  Creation Timestamp:  2020-09-28T09:37:46Z
  Generation:          2
  Resource Version:    62065
  Self Link:           /apis/litmuschaos.io/v1alpha1/namespaces/shubham/chaosresults/engine-cassandra-pod-delete
  UID:                 dba83d63-cc72-47da-a952-9b53d8d20e84
Spec:
  Engine:      engine
  Experiment:  cassandra-pod-delete
Status:
  Data:
    PostChaos NodeTool:  Datacenter: datacenter1
=======================
Status=Up/Down
|/ State=Normal/Leaving/Joining/Moving
--  Address    Load       Tokens       Owns (effective)  Host ID                               Rack
UN  <IP_ADDRESS>  145.32 KiB  256          100.0%            609280dd-79e3-4f59-8649-46e1111c82c1  rack1
UN  <IP_ADDRESS>  75.24 KiB  256          100.0%            6b1711a8-d88c-432d-a41c-7db99feb85fa  rack1

    PreChaos NodeTool:  Datacenter: datacenter1
=======================
Status=Up/Down
|/ State=Normal/Leaving/Joining/Moving
--  Address    Load       Tokens       Owns (effective)  Host ID                               Rack
UN  <IP_ADDRESS>  69.99 KiB  256          100.0%            609280dd-79e3-4f59-8649-46e1111c82c1  rack1
UN  <IP_ADDRESS>  70.02 KiB  256          100.0%            6b1711a8-d88c-432d-a41c-7db99feb85fa  rack1

  Experimentstatus:
    Fail Step:  N/A
    Phase:      Completed
    Verdict:    Pass
Events:
  Type    Reason   Age    From                               Message
  ----    ------   ----   ----                               -------
  Normal  Awaited  7m11s  cassandra-pod-delete-z5lpwd-rr8cn  experiment: cassandra-pod-delete, Result: Awaited
  Normal  Pass     4m58s  cassandra-pod-delete-z5lpwd-rr8cn  experiment: cassandra-pod-delete, Result: Pass

There are alternate approaches being discussed around elaborate results. Closing this PR for now.


restructuring

Closes #739
Restructuring of video sections and adding content that fits section title (regardless if video or not).
Signed-off-by: jetzlstorfer<EMAIL_ADDRESS>

Preview: https://deploy-preview-741--keptn.netlify.app/


Redis In-Transit Encryption (TLS) and password support

When using Redis for peer discovery, if Redis has in-transit encryption enabled, Samproxy will fail with an i/o timeout:
time="..." level=error msg="registration failed" err="read tcp <IP_ADDRESS>:44362-><IP_ADDRESS>:6379: i/o timeout" name="http://ip-000-00-000-000.eu-west-1.compute.internal:8081" timeoutSec=10
time="..." level=error msg="failed to register self with peer store" error="read tcp <IP_ADDRESS>:44362-><IP_ADDRESS>:6379: i/o timeout"

This is due to the Redis client used not having the option enabled:
https://github.com/honeycombio/samproxy/blob/7ad0d4372207be9e09d43678d5a7549a76e0ea8b/internal/peer/redis.go#L60-L65
As per Redislabs' "Redis Enterprise and Go" which also uses github.com/garyburd/redigo/redis, this requires specific parameters to be sent when initializing the client:
conn, err := redis.Dial("tcp","<endpoint>:<port>",
    redis.DialPassword("<password>"), // password support would be nice too!
    redis.DialTLSConfig(&tlsConfig),
    redis.DialTLSSkipVerify(true),
    redis.DialUseTLS(true))

Relevant links:

AWS Elasticache for Redis In-Transit Encryption (TLS) docs: https://docs.aws.amazon.com/AmazonElastiCache/latest/red-ug/in-transit-encryption.html
Redislabs' "Redis Enterprise and Go": https://redislabs.com/lp/redis-enterprise-go/

@Vlaaaaaaad thanks for bringing this up! I've added it to our backlog for our current period of work on samproxy so that we can prioritize accordingly

hi! I created a small PR for this with a very basic configuration for TLS and a password here. I tested it running a kubernetes service with multiple instances of refinery using Docker desktop and my redis instance was in Azure, and it was working fine. Unfortunately, I wasn't able to test the AWS redis cluster since it can only be accessed from the VPC and requires a bit more setup to run.
Let me know your thoughts!

I can confirm this seems to work fine in AWS using ElastiCache too!
When the next release is cut (https://github.com/honeycombio/refinery/pull/201 was already merged) I think this issue can be closed.

Closing as #201 was merged and part of the v0.15.0 release.


Perico

B

@Perico22 please create an issue according to the template, thanks. this issue will close.


devguides: wallet-rpc transfer amount with fee included missing

https://github.com/monero-project/monero/commit/b4eb3f18340e04583d66324f33a97947666016b5

can be moved to: https://github.com/monero-project/monero-docs


custom-http-provider

Hi @ronzeidman
1-2 hours ago i submitted an issue about extending JwtAuth. If you check the code it is an older version. To be more specific 16days old (was using my fork with this mod: https://github.com/ronzeidman/ng2-ui-auth/commit/2bde6aa1f153c129774112eb07fb8204fa850127)
Anyways, finally i was able to extend JwtHttp successfully. And happily used it, but then i realized if i call any of the methods from AuthService it will internally use JwtHttp again. I needed a way to tell ng2-ui-auth to use my custom Http provider.
This is the solution.
You can also check out ng2-translate's solution for this. They also provide an abstract class. That way u can do smth like class CustomHttp implements JwtHttp.

Nice, I'll add it along with issue #65 and the master fixes in next week's release.

This one introduces breaking changes tho. Renamed getWithConfig to forRoot. Is that ok?
Maybe we could put getWithConfig back like this:
static getWithConfig() {
  console.warn('getConfig is deprecated, use forRoot instead. <link to changelog / docs>')
}

I'll try to minimize the breaking change (the master branch is not "stable") I think I'll make them aliases, maybe I'll use a deprecation notice, I haven't decided yet.
Anyway like I always try to do, If this ends up breaking I'll bump the version and write it in the changelog.


[AutoPR network/resource-manager] Removed May folder because this API version is not published

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3326

(message created by the CI based on PR content)

This PR has been merged into https://github.com/Azure/azure-sdk-for-python/pull/2376


README update for new build

Added a clarification that adding text before or after retest this please in the same comment will trigger the build, after trying both.

@Pisush thanks!


binary file source - windows blocking read with timeout

Need a blocking read with timeout implementation. May have to use the windows file APIs instead: ReadFile and SetCommTimeouts...

I'm pretty sure this issue would be rendered obsolete when this one is addressed: https://github.com/pothosware/PothosBlocks/issues/1


Remove references to broadcast joins and large_table_shard_count

Broadcast joins
When we talk about joins and broadcasted tables in our docs, we refer to the dated citus.large_table_shard_count config setting. Citus uses the new reference table API and doesn't rely on this setting anymore.
https://docs.citusdata.com/en/v6.0/dist_tables/querying.html#joins
We should update the following paragraph to reflect this change:
"To determine the best join strategy, Citus treats large and small tables differently while executing JOINs. The distributed tables are classified as large and small on the basis of the configuration entry citus.large_table_shard_count (default value: 4). The tables whose shard count exceeds this value are considered as large while the others small. In practice, the fact tables are generally the large tables while the dimension tables are the small tables."
Colocated joins
We have the following Note in the colocated joins section:
"In order to benefit most from colocated joins, you should hash distribute your tables on the join key and use the same number of shards for both tables.  ..."
I had two questions on this note. First, we changed our table creation APIs in v6.0 and we could probably reframe this sentence to the following: "In order to benefit most from colocated joins, you should hash distribute your tables on the join key, where the join keys have the same data types."
Second, this note communicate information that's very similar to the paragraph before it. Do we need to mention it as a Note?

Two other questions.

With the introduction of reference tables, Citus no longer broadcasts shards to all machines to execute a join. Do we want to use a different name for "broadcast joins"?
We have a note in our docs about broadcast joins erroring out with explain. Do we still have this issue with reference tables?

I'm noting #229 as a related issue.


Update install.rst

'bootstrapform' was missing in the list of installed apps

The install.rst has been fixed in some other commit, it seems:
https://github.com/django-helpdesk/django-helpdesk/blob/master/docs/install.rst
This PR can be closed.

Thanks for the effort nonetheless everyone!


i Login to the web application on the Pi: http://(wired Ethernet IP of the node):8080/

when i login to the web,i don't know how to do next step. the web shows that this website is Apache2 Debian Default page ,including four main parts: It Works , Configuration Overview , Document Roots and
Reporting Problems. How could i access the Admin account using the 'admin' username and 'changeme' password

Hi, sounds like some of the setup wasn't run, either completely or correctly.
I had to try it twice before it worked, for me.
You also, might want to go to the Google Plus Community, seems to be a bit more active than it is here.
https://plus.google.com/u/0/communities/109339235992496491942


[BUG]: graph html not printing dashed lines for given input nodes

Expected Behaviour
When printing a graph where inputs are given, the edges from these nodes are full lines, when should be dashed lines
How to reproduce

see closed PR https://github.com/flypipe/flypipe/pull/118


Plugin for Arirang TV?

http://www.arirang.co.kr/player/onair_tv.asp?sys_lang=Eng
Would be great if there would be a plugin for this service.

meanwhile
you can do this
livestreamer "hds://http://worldlive.arirang.co.kr/arirang/arirangtvworld.mp4.f4m" best -o arirang.flv

Thanks a lot for the tip. :)


Starburst connector impersonation based on another field than email field

Hello 🙂
We opened an issue in metabase github but maybe it's more here that we can find some help !
Here is our problem:
Is your feature request related to a problem? Please describe.
We are benchmarking Metabase for our next BI tool. We are making some test trying to use Metabase with Starburst Trino connector. As authentication system, we use LDAP. Actually, if we want to impersonate queries, Metabase use by default user mail field filled by LDAP when new user sign-on.
We take a look inside Metabase code and found that function that confirm that email user field is used by default (here):
(defn impersonate-user
  [conn]
  (if
    (clojure.string/includes? (.getProperty (.getClientInfo conn) "ClientInfo" "") "impersonate:true")
    (let [email (get (deref api/*current-user*) :email)]
      (.setSessionUser (.unwrap conn TrinoConnection) email))
    nil))

Describe the solution you'd like
We think that it could be really nice to choose which field we want to use between user fields (first name, last name and email).
Describe alternatives you've considered
We don't really have alternative as our Trino authentication is based on another field than email. For the moment, we are not going to use impersonation which is a problem for us because we can't manage data access very well using Trino user rules..
How important is this feature to you?
To be honest, this feature is really important for us because without this feature, we can't go futher using Metabase in production while we think that it's a great tool for our needs.
Additional context
Metabase version: v0.49.3
Starburst driver version: 4.1.0
Trino version: 439
Thanks for your help 🙂

Hello @maxpoulain
Thanks for reaching out and using Trino :)
I'm afraid email is the only reasonable field in this case to be used for user impersonation.
Proposed fields (names) are not unique.
What field  actually you use in Trino?
Why not switch to email?

Hello @leniartek
Thanks for your answer !
Actually it seems that we use uid field which is firstname.lastname.
Regarding the switch to email, I'm not an expert so I don't really know why uid is used historically but I can ask to our infrastructure team.


Add Wiki Page on how to update and test code snippets for documentation

Currently, while the tools exist, there is no step-by-step process explaining a procedure for generating sample code on docs.flutter.dev (see ListTile sample code example in the blue boxes).
I think it could be worthwhile to add instructions on the Flutter wiki on how to make the changes, then test them locally so that they pass the pre-submit analyzer tests. This would help Flutter developers who want to add sample code do so.

Addressed with https://github.com/flutter/flutter/pull/34073

Added a link in the wiki sidebar to the snippet generation README


fix: Deduplicate deps while amending

To address https://github.com/bazel-contrib/rules_jvm_external/pull/1251#issuecomment-2407255281

Does this change make sense? Can someone help review it? Thanks.

(responding in this PR because this came first, but #1269 also appears to solve the same thing)
cc @shs96c who wrote the parent PR
Can you add comments like in #1269, and regression build tests by adding the triggering artifacts in https://github.com/bazel-contrib/rules_jvm_external/issues/1267 to https://github.com/bazel-contrib/rules_jvm_external/blob/1202665f8e233f43325396af209f4eecbc1587f6/MODULE.bazel#L501 and https://github.com/bazel-contrib/rules_jvm_external/blob/1202665f8e233f43325396af209f4eecbc1587f6/tests/unit/build_tests/BUILD#L7?

I added a few more commits, but please feel free to take #1269 instead and cherry pick useful commits from this PR.

Shall we move forward with this? I'm fine either this PR or #1269. Thanks.

Thanks, sorry for the delay.


Can't mock with jest.mock

Guidelines for posting a new issue

Please replicate your issue with this CodeSandBox and provide a link to it along with the issue description

I'm trying to mock react-slick to unit test my component with jest but the real react-slick is called
jest.mock('react-slick', () => jest.fn(() => null));
How do I mock react-slick?

This is not an issue with this library.
Try asking this question on stackoverflow


2.0.2 copies partials when building.

I just updated from 2.0.1 to 2.0.2, and now all my subfolders and partials are copied to the output folder when I build.
Here's the task I'm currently running which provides different results between 2.0.1 and 2.0.2:
var gulp = require('gulp');
var sass = require('gulp-sass');
var postcss = require('gulp-postcss');
var autoprefixer = require('autoprefixer-core');
var mqpacker = require('css-mqpacker');
var csswring = require('csswring');
var handleErrors = require('../util/handleErrors');

gulp.task('sass:production', function() {
  var processors = [
    autoprefixer({browsers: ['last 2 version']}),
    mqpacker,
    csswring
  ];

  return gulp.src('app/assets/styles/**/*.{sass,scss}')
    .pipe(sass({
      errLogToConsole: true,
      includePaths: ['bower_components']
    }))
    .on('error', handleErrors)
    .pipe(postcss(processors))
    .pipe(gulp.dest('build/assets/styles'));
});

Did something change or am I doing something wrong?
Thanks for all your hard work!
Edit: This is only have affecting empty files. I tried running it in a boilerplate of mine where I've scaffolded out a folder structure with some empty files. If I revert the changes from https://github.com/dlmanning/gulp-sass/commit/7f6605e433a32ecc01ed84b0024d7e6fcf7a2ed3 it works like before.

I'm also having this exact problem, which I made reference to in https://github.com/dlmanning/gulp-sass/issues/305
The other odd thing: Partials and subfolders are copied to the output folder as scss files, not css files.

1

This is… weird. We didn't change anything that should have affected this (in fact, I tried reverting back the change we made with partials and it's still happening). I'm digging in

Are all of the partials that are being moved over empty? Or do they have content?
I've got a few tests running and it seems as if only empty partials are being moved over for me. I think it has to do with https://github.com/dlmanning/gulp-sass/blob/master/index.js#L28-L30, where we need to have it work the same way of our partial ignore, or move our partial detection code above that.
@dlmanning @Keats thoughts?

This is also one of those nifty things we can't effectively automatically test for because we can't check in empty files :\

Moving partial detection code https://github.com/dlmanning/gulp-sass/blob/master/index.js#L31-L33 above https://github.com/dlmanning/gulp-sass/blob/master/index.js#L28-L30, seemed to fix the issue.

@edtorba Can you confirm that the only partials being moved over for you are empty ones?

@Snugug Yes, only empty ones.

Just released 2.0.3 that resolves this.


Some unicode characters tripping encode_keycode_char

ValueError: encode_keycode_char: Key length != 1
An error occurred: encode_keycode_char: Key length != 1
Listener stopped!
Warning: validate_keystrokes: Lengths not equal.
Input string found extra char at index 11: ¨ <-
11 Keystrokes validated: False
Log not created.
Log had trouble saving!

Narrowed it down:
An error occurred: WEIRD KEY: ¨¥, len: 2
Turns out that in the pynput library, a KeyCode.char value can hold a string like "¨¥" when I type these characters very quickly together or simultaneously.
Replicated by:
cd KeyMaster
python

from client.configurate import Config
config = Config()
config.listen_script()

I then mashed unicode characters, resulting in a 2-character string equivalent to: "¨" + "¥" causing a ValueError.


How to specify all columns as sortable using pre-built function Reactable.Sort.Currency?

It seems like the API allows for individual columns to be marked as sortable using the pre-built function,
ex:
sortable={[
{
column: 'ColName',
sortFunction: Reactable.Sort.Currency
}
but is there a a way to specify all columns, assuming I don't know what the name of the column will be ahead of time? It would be even better if I could specify the sort algorithm based on the index of the column, so say column at index 0 gets the default sort algo, whereas index > 0 gets Currency. My table has over 150 columns
ex:
sortable={[
{
columns: " * " ,
sortFunction: (columnIndex > 0) ? Reactable.Sort.Currency : Reactable.Sort.Default
}
etcetc...
Would this be possible?
Thanks!

nevermind.. i figured it out. I love the way this table exposes stuff . Very developer friendly.
I set the sortable parameter using this function:
getSortParams(column){
let arr = [];

for (let i = 0; i < column.length; i++){
      let obj = {};
      obj['sortFunction'] = Reactable.Sort.Currency;
      obj['column'] = column[i];
      arr.push(obj);
}
  return arr;

}


Hiero: yOffset is not 0 when it should be?

Issue details
When exporting a BMFont with Hiero, the tallest characters don't have a yOffset of 0. Please let me know if this is expected as I can't figure out why they should have an offset?
Reproduction steps/code

Open Hiero
Select Java rendering
Select NEHE character set
Select Verdana
Export
In the .fnt file, only characters with id 10 and 32 will have a yOffset of 0

Version of LibGDX and/or relevant dependencies
Hiero v5

Why do you expect the yOffset to be zero? Why does that matter?

I'm trying to keep my gui label class as tight a fit as possible whilst keeping enough space for any possible character, however when applying the offset the text has a larger gap than expected above it.
I should note that I'm creating the font with padding 8 on all sides and -16 x and y offset to compensate, but even when leaving those as default there is large yoffset.
Am I missing something?

Font rendering and metrics can be pretty wild. I'm not sure it makes sense to expect yOffset to be zero, it's really up to whatever provided the metrics (Java rendering in your repro). You could adjust the glyphs after loading the BitmapFont. I do this in some of my apps.


The v0.8.3 Tags is pointing to the v0.8.2 commit

The tag v0.8.3 is identical to the v0.8.2
commit 382977a79c8a57f21c686b7b32a364165c668f87 (HEAD, tag: v0.8.3, tag: v0.8.2, origin/release-v0.8)
Author: Barni S<EMAIL_ADDRESS>Date:   Mon Apr 13 14:34:46 2020 -0400

    bump to 0.8.2 to in release-0.8 branch

/assign @nan-yu

The tag v0.8.3 was updated to the latest commit. However, there is an issue in the release script: https://github.com/kubernetes-sigs/application/blob/master/Makefile#L328
Currently, it is set to release from a release-v0.8 branch. However, the latest changes were not cherry-picked to the release-v0.8 branch. I ended up releasing from the master branch without the script.
We need to further enhance the release script when we support multiple major versions in the future.

The release script enhancement work is tracked in issue: https://github.com/kubernetes-sigs/application/issues/196


frontmatterContent field is not working on production server

Using the next options in api.staticman.com doesn't work as expected
format: frontmatter
transforms:
  mdfield: "frontmatterContent"

It generates a field where the mdfield is included in the yaml block instead of the content area of the markdown file.
Additional random behaviors:
Extra fields added:
layout: verslag
excerpt_separator: <!--more-->

If the field is optional and not filled out the content area of the markdown file is generated as undefined
In dev.staticman.com
Only when the field is empty it generates undefined in the content area, the other option works correctly

There is no mdfield property, it's contentField. Could you try with that?

Hi @eduardoboucas ,
Correct me if I'm doing something wrong,
In the allowedFields: property I added content
allowedFields: ["title", "content", "name", "email", "twitter"]
Then in format I use the value frontmatter to generate a markdown file
Finally, in transforms I took the field added in allowedFields and add the value "frontmatterContent" so, that field Is going to be used as the body of the post/file
the result something like:
allowedFields: ["title", "content", "name", "email", "twitter"]
format: frontmatter
transforms:
  email: md5
  content: frontmatterContent

Is that correct? or where do the contentField needs to be used?
thanks!

Sorry, @usrix. My previous comment made absolutely no sense.
That looks correct. Are you getting any errors at all?
I'll test this myself over the weekend.

@eduardoboucas
Not really, it seems like it just gets ignored.
The content field went to the yaml block, and the body as undefined, like when you leave the field empty

@eduardoboucas The same config on dev.staticman.com is working fine,
Just one issue there, when the frontmatter field is not filled out, it generates and undefined content, I think that can be handled by a Jekyll validation.

I really don't get how it's working on the dev instance but not on the live one, since they're both exactly the same at the moment. 😢

Just one issue there, when the frontmatter field is not filled out, it generates and undefined content, I think that can be handled by a Jekyll validation.

This sounds like something we should handle in Staticman. I'll look into it.

@usrix If you're still using this, could you please test the live instance again and let me know if the issue persists?

Yes, it persists @eduardoboucas

Hello @eduardoboucas It seems like it persists, when my last try the frontmatter field is attached to a yaml var named excerpt

Now it's working !!! :clap:  cc. @eduardoboucas

I facing the same issue. You can see my staticman.yml file here and an example file generated by staticman in PR here.

I have just re-deployed the live instance and did some testing and it seems to be working fine now. Can you please confirm?
Thanks

sorry for the laaaaate reply, but this bug seems hard to squash, the issue just returned (as reported on Oct 11 by abhin4v), I want to help, let me know where to start, haven't code for a while, but I can at least check

Having the same issue.
staticman.yml
format: "frontmatter"
extension: "md"

transforms:
  message: "frontmatterContent"

Transform has no affect, message remains in YML front matter, and undefined added to the body.
Example
---yaml
_id: 975cfb20-6055-11e8-bf3d-35969255318e
_parent: /markup/markup-syntax-highlighting/
name: Foobar Baz
email: ade7352bc2b46f667fb73c91fe19ee9c
url: 'http://foobar.com'
message: This is a comment with some **Markdown**.
date: '2018-05-25T19:55:36.596Z'
timestamp:<PHONE_NUMBER>
tags:
  - comment-subscription
type: comment
---
undefined

Posting to https://dev.staticman.net/v2/entry/ instead of the production endpoint works as expected.
Also possibly related to #176, using the DEV endpoint gives a much cleaner data file. When I use api.staticman.net there's all kinds of fields I haven't set like timestamp and tags.


Wrong pandoc URL for v2.2.2

Correct link:
https://github.com/jgm/pandoc/releases/download/2.2.2/pandoc-2.2.2-1-windows-x86_64.zip
Error prompted:
Installing 'pandoc' (2.2.2) [64bit]
Richiesta annullata: Chiusura imprevista della connessione..
URL https://github.com/jgm/pandoc/releases/download/2.2.2/pandoc-2.2.2-windows-x86_64.zip is not valid

Fixed in #1053


[FEATURE REQ] How to handle different certificates(Thumbprint) for different environments.

Please describe the feature.
Hello.
Our site registers and uses different certificates for each environment.
for example,
Certificate settings can be set through Client cert item in Backend Gateway credentials of Azure portal - APIM - APIs - API common/operation.
If you check their policy.xml file, the Thumbprint value of the Certificate is entered in the '<authentication-certifiacte thumbprint=>' item. It is the same when extracted with APIOPs.
However, they have different values for different environments like below.
DEV APIM : openapi-dev.sampel.com (Thumbprint : 4147891~)
UAT APIM : openapi-uat.sampel.com (Thumbprint : 6D35F1C~)
PRD APIM : openapi.sampel.com       (Thumbprint : 9E99A48~)
Please let me know something feature or how to handle different certificates(Thumbprint) for different environments.
Thanks

Hi @junoya-yun
I would suggest using named values in this case. You can create a named value for your thumbprint, override the value in each environment, and reference the named value in your policy instead of hardcoding the thumbprint.

Thanks for providing the solution. I will do that.


(IMPORT FAILED) Crystools

Describe the bug
(IMPORT FAILED) Crystools
To Reproduce
workflow-relight-better-than-magnific---relighting-v5-better-ux-nSqO2P2ZmDQGwohEbgl3-risunobushi-openart.ai.json
Expected behavior
Can't run the workflow properly. (IMPORT FAILED) Crystools
Screenshots

Error in console:
-Crystools
0.0 seconds (IMPORT FAILED): C:\Users\b\Desktop\ar\st\AI\ComfyUI_windows_portable_nvidia\ComfyUI_windows_portable\ComfyUI\custom_nodes\ComfyUI-Crystools
ComfyUI Revision: 2652 [0c7c98a9] *DETACHED | Released on '2024-09-05'
ComfyUI-Manager (V2.51.1)
** Platform: Windows
** Python version: 3.11.9 (tags/v3.11.9:de54cf5, Apr  2 2024, 10:12:12) [MSC v.1938 64 bit (AMD64)]
Total VRAM 12288 MB, total RAM 65377 MB
pytorch version: 2.4.1+cu124
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 3080 Ti : cudaMallocAsync
Using pytorch cross attention

I need more info from your logs, please, paste the on the startup here

I need more info ¯_(ツ)_/¯


Fix AWS console generation bug where role was incorrectly double assumed

There is a bug in the console generation for AWS accounts where OrganizationAccountAccessRole was assumed once by the inital user, then again by itself (OrganizationAccountAccessRole assumed OrganizationAccountAccessRole). This has silently worked due to AWS allowing implicit trust of a policy assuming itself. However, this behavior is changing and this no longer works with newer accounts. See https://aws.amazon.com/blogs/security/announcing-an-update-to-iam-role-trust-policy-behavior/ for more details.
This is a stop gap to get this command to work, so please excuse the messiness of the code. All of k8sclusterresourcefactory will be refactored soon, so this will be made much cleaner in an upcoming PR.

/lgtm


Restores eslint rules

Good job team, no rules violated in between the eslint refactoring and now! 🙇♀ 🙏 👏
These rules were commented out to satisfy an older version of eslint.
Now that dependencies have been updated they can be restored.
@chriskacerguis
closes #228

@AllenBW doesn't this need to go in the Euwe branch, not master?

@chriskacerguis I thought euwe was a ⛔ for eslint and friends? (defer to you)

It is, but you are doing a PR to the "master" branch.  You need to do it against the Euwe branch 😄

mmm ewue doesn't suffer from what this pr cures, https://github.com/ManageIQ/manageiq-ui-self_service/blob/euwe/.eslintrc.json

@chriskacerguis actually this reveals a larger concern, so we include eslint config, but no dependencies in package.json furthermore, this pr reverses what I would expect to reflect the status of eslint config for the lover version of node

@AllenBW ok, so the question I have right now is...we are using ESLint now NOT JShint, correct?

@chriskacerguis Correct, On master there is no jshint. On euwe it was my understanding we would continue to use jshint, (because its still in place) but what do i know :P

So, I'm thinking we need to delete the jshint file on MASTER, and update the one in Ewue.

transposed? jshint isn't on master eslint is and neeeeds to be updated

Ok, got my jshint / eslint confused.


[Bug]: USB Inrush uses fixed path for x86 only, no x64.

Joulescope serial number
N/A
UI version
0.9.11
What OS are you seeing the problem on?
Windows 11
What happened?
The Joulescope UI does not start the USBET tool if the x64 version is installed.  It only works if the x86 version is installed.
What was expected?
The Joulescope UI should be able to use either x86 or x64 USBET version.
How to reproduce?
Download the latest USBET20.
Install USBET20 2.20.00 x86.  Try launching "USB Inrush" in Joulescope UI.  Works.  Uninstall.
Install USBET20 2.20.00 x64.   Try launching "USB Inrush" in Joulescope UI.  Does not work.
Extra information
The code contains this hard-coded path.

Fixed in 0.10.x


Update Overview Section and Fix Image Visibility in Performance Metrics [Fixed #196]

closes #196
Description
This pull request addresses two improvements:

Updated the Overview Section: Enhanced the "Overview" section to provide more clarity about the project's purpose, data source, and objectives. This update makes it easier for users to understand the goal and scope of the project. The new content includes a detailed explanation of the project’s focus on predicting stock prices for the State Bank of India (SBI) using machine learning regression algorithms.

Fixed Image Visibility in Performance Metrics: Corrected the broken image links in the "Performance Metrics" section. The images for Root Mean Square Error (RMSE), Mean Absolute Error (MAE), and Mean Absolute Percentage Error (MAPE) are now visible, displaying the performance metrics for the 10 regression algorithms.

This PR enhances the readability and presentation of the documentation, providing a better user experience.
Screenshots of the changes have been included

I made the following changes. Please accept my PR and merge it. Thank you for your valuable time

I Synced and updated my repo

Thank You. Happy Diwali🎉💥


Miss component will not render after going back to / in the ambiguous matches example

Version
4.0.0
Test Case
https://react-router.now.sh/ambiguous-matches
Steps to reproduce

Click on Kim
Go to / (either by going back or manually typing in the url bar)
Click on Kim

Expected Behavior
Kim should render again
Actual Behavior
The Miss component does not render.
noMatchesInContext on the Miss component state is false.
If you click on About Us or Company it seems to reset and start working again. I can reproduce this on our app so it is not just the example. Interestingly, if a hot reload occurs and the app is rendered again the page shows up.

As I mentioned in my duplicate issue, I think the <Miss/> component does not render, because the <MatchProvider/> is not notifying it's subscribers, preventing <Miss/> from updating state.noMatchesInContext to true.
I believe this line is the culprit: https://github.com/ReactTraining/react-router/blob/a603eaae85d8b9abf157669137406657567475de/modules/MatchProvider.js#L80
Since <MatchProvider/> is never unmounted, this.hasMatches is false. This works for the first time around, because this.hasMatches is initiated as null.
I didn't submit a pull request, because I'm unsure what the desired approach is here. May I remove the optimization, or is there a better way of handling this case?

Nil strikes again!
I don't think it's a severe perf loss. I say submit a PR (also set it to allow contributors here to edit it) and we can discuss based on real code instead of theoreticals 👍

@timdorr Done!

Can someone publish this latest change on npm?

Apparently not.

You can buy my time if you'd like, I charge $10k per week.

It's alpha software, pull from github, npm run build, then npm link. Solve your own problems.

Sorry @ryanflorence. didn't mean to take you for granted


Problem installing dask[complete]

What happened:
Installing dask with the 'complete' extra doesn't seem to work and gives a warning.

$ pip install -U dask[complete]

Requirement already up-to-date: dask[complete] in ./env/lib/python3.6/site-packages (2.17.1)
WARNING: dask 2.17.1 does not provide the extra 'complete'
Requirement already satisfied, skipping upgrade: pyyaml in ./env/lib/python3.6/site-packages (from dask[complete]) (5.1.2)

What you expected to happen:
I expect it to install with the 'complete' extras.
Anything else we need to know?:
Environment:

Dask version: 2.17.1
Python version: 3.6.9
Operating System: WSL
Install method (conda, pip, source): pip

One of our docker images is failing the build process for the same reason. We had to force it to install dask[complete]==2.17.0 for now.

Thanks for reporting @jackieleng @DVerzal! This was fixed by https://github.com/dask/dask/pull/6257. We'll push a new 2.17.2 release with the fix shortly

dask==2.17.2 is on PyPI now and should be on conda-forge and docker hub within an hour or two

Closing as you can now pip install  "dask[complete]" again. Thanks all for reporting


Access packages installed in Ubuntu in Windows

I'm sorry if I'm opening an already existing issue. I tried to find something similar but I failed.
I installed node on Ubuntu using nvm and it works fine there. However, I have VSCode installed on Windows. I also have a few node packages installed globally like eslint, prettier, etc.
I'm trying to hook up eslint installed in Ubuntu with VSCode but it doesn't seem to be working. I tried updating the eslint nodepath but that doesn't work. Is this even possible? Should I just install node on Windows?

@skippednote See: https://msdn.microsoft.com/en-us/commandline/wsl/interop?f=255&MSPPError=-2147217396

Keep an eye on the useWSL flag and VSCode#25033 too (but it's early).

Really want this to work. I have my whole setup going in wsl and use VSCode. But I can't get Linting on my Node.js code.
Has anyone figured this out yet?


[FR]: Support deleting/cleaning the default project

Proposal summary
It's quite common for default project to turn into dump if you were not specifying project names properly during your work and experiments.
Would be nice to be able to delete all this data. Currently it's not possible to do via the UI because there is only button "Delete" on the projects page and it's disabled for "Default Project".

This is in progress


weighted with NaN weight returns unexpected results

chance.weighted(['a','b','c','d'],[NaN, 100, 1, 1]) always returns 'd'.
fixed by throwing an error if any weight is NaN

Changes Unknown when pulling bb37e53c5ef1e7dda72c1a1f2eb555cdfc6f851d on adamoonact:master into ** on chancejs:master**.


Inconsistency with naming repos/packages

What?
The repo is called "pusher-platform-nodejs" and package name on NPM is "pusher-platform". On feeds we are using repo name like: "pusher-feeds-node" and then package is "pusher-feeds-server". Then is easy to have client JS library repo called "feeds-client-js" and NPM package "pusher-feeds-client" which is a logical way to do. So it would be good to just unified this repo/package with our pattern.
Suggested improvements
I would just rename

pusher-platform-nodejs => pusher-platform-node

pusher-platform => pusher-platform-server

CC @pusher/sigsdk

I think the artefact should have the same name as the repo, for practicality. pusher-platform-server-node maybe?

Tbh we did have a naming document somewhere - I'll try to find it.

Well that is the question.. Because on NPM when you say "server" means automatically Node.js. The same when you say "client" means browser. So for example we would have to call our client SDK on NPM "pusher-feeds-client-js" which doesn't makes much sense since all on NPM is JS.


[ Hold ]Show url of http trigger with authLevel ANONYMOUS after deployment

Show url of http trigger with authLevel ANONYMOUS after deployment, for issue #571
Known issue:
Maven will show warning when execute function maven goal.
[WARNING] The POM for com.microsoft.azure.appservice.v2016_08_01:azure-mgmt-appservice:jar:1.0.0-beta-1 is invalid, transitive dependencies (if any) will not be available, enable debug logging for more details

Does this cover only anonymous functions or other authentication levels as well? Just wanted to check since the original issue targeted both scenarios, later being the more widespread amongst users.

Does this cover only anonymous functions or other authentication levels as well? Just wanted to check since the original issue targeted both scenarios, later being the more widespread amongst users.

@asavaritayal  For now it only contains anonymous HTTP-trigger functions, as other functions need auth with its function key. It's possible to get the keys as mgmt sdk provide related methods.but there may have safety issue to print keys directly.


BAAAS-305 - fix Decision.byCustomerAndIdOrName named query typo

@r00ta Is there a test that this has fixed something?

@r00ta Is there a test that demonstrates this has fixed something and that we don't break it in the future?

@robpblake done :+1:


remove generated yaml files (#3183)

the circle failure seems to be #3125 / unrelated to this pr so merged


Desktop Session

I've got the flake working and I can launch niri manually, but I was wondering if there's a magic trick to get niri as an option with gdm or sddm?

Nevermind. Session wasn't added via home-manager module, but works great when used as nixos module.

As far as i'm aware, it's not possible to add from the home module.  It can only see system-wide files, and as such the sessions is can autodetect must be system-wide ones.


Add Japanese translation of "Advanced Concepts" pages

This pull request adds Japanese translation versions of "Advanced Concepts" pages.
Category (place an x in each of the [ ])

[ ] slack_bolt.App and/or its core components
[ ] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[x] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

I've confirmed the outlook on my laptop. Let me merge this.


docs: lib.autoExternal

Summary
docs: lib.autoExternal
Related Links

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

What if adding a link to lib.rsbuild.dev/guide/advanced/third-party-deps?

updated


Blueprint pages: Add contact us link

Description
Added a contact us CTA at the bottom of the blueprint pages
Related Issue(s)
https://github.com/FlowFuse/customer/issues/225
Checklist

[x] I have read the contribution guidelines
[x] I have considered the performance impact of these changes
[ ] Suitable unit/system level tests have been added and they pass
[ ] Documentation has been updated

These pages need design work, but I'll address that in a different PR


UI part done for posting jobs

✨ Codu Pull Request 💻
Fixes #1082
Pull Request details

Only UI part done for Job posting page

Any Breaking changes

None

Associated Screenshots

Just for uniformity and it being hard to explain, I added a PR with updates of the layout of how it all hangs together responsively (so hopefully the two screens look uniformed): https://github.com/codu-code/codu/pull/1109

Would you feature flag this with a flag named jobs. Here is an example of how it works: https://github.com/codu-code/codu/tree/develop/app/(app)/feature-flag-example

Yeah sure will do afterwards updating the UI according to settings page

@NiallJoeMaher I added the flag. Let me know if things are ok or anything to refactor.

@NiallJoeMaher @John-Paul-Larkin Let me know if things are fine now

🌮 You can start working on the validation and I'll tag it as Hacktoberfest. Thanks for this.

On it 💪


No module named 'tensorflow.contrib'

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: jupyter  notebooks
TensorFlow installed from (source or binary): Github
TensorFlow version: 2.1.0
Python version:3.6
Installed using virtualenv? pip? conda?:conda
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:Intel HD

Describe the problem
11 sys.path.append("C:/Users/xx/.conda/envs/tensorflow_env/Lib/site-packages/tensorflow/models")
12 import cv2
---> 13 from nets import vgg
14 from preprocessing import vgg_preprocessing
15 from mlxtend.preprocessing import shuffle_arrays_unison
~.conda\envs\tensorflow_env\lib\site-packages\slim-0.1-py3.7.egg\nets\vgg.py in
43
44 import tensorflow as tf
---> 45 # from tensorflow.contrib import slim as contrib_slim
46
47 slim = contrib_slim
ModuleNotFoundError: No module named 'tensorflow.contrib'
Provide the exact sequence of commands / steps that you executed before running into the problem
import os
import sys
import numpy as np
import pandas as pd
import matplotlib.pyplot as plt
import tensorflow as tf
from skimage.transform import resize
from tqdm import tqdm
from sklearn.model_selection import train_test_split
%matplotlib inline
sys.path.append("C:/Users/Firoz/.conda/envs/tensorflow_env/Lib/site-packages/tensorflow/models")
import cv2
from nets import vgg
from preprocessing import vgg_preprocessing
from mlxtend.preprocessing import shuffle_arrays_unison
checkpoints_dir = 'C:/Users/Firoz/checkpoints'
import warnings
warnings.filterwarnings("ignore", message="numpy.dtype size changed")
I have gone through these steps

https://github.com/tensorflow/tensorflow/issues/30794
https://github.com/tensorflow/tensorflow/issues/31350

without downgrading my tensorflow,  so Assisstance ??

Contrib does not exist in 2.0 or later. If you need contrib you have to use 1.15, although we recommend upgrading code to the contrib alternatives (since they are being actively maintained by an owner, unlike contrib which doesn't have an owner to fix issues)

When I do "sudo -H pip2 install tensorflow==1.15" I get
ERROR: Package 'mock' requires a different Python: 2.7.18 not in '>=3.6'
When I do "sudo -H pip3 install tensorflow==1.15" I get
ERROR: Could not find a version that satisfies the requirement tensorflow==1.15 (from versions: 2.2.0rc1, 2.2.0rc2, 2.2.0rc3, 2.2.0rc4, 2.2.0).
The fact that  tensorflow.contrib was removed from python3 tensorflow is causing quite pain. Any help?

Before TF 2.2 you need python 3.5, 3.6 or 3.7. Not Python 2 or python 3.8

trizen -S python37 --noconfirm
sudo -H python3.7 -m pip install tensorflow==1.15.3
sudo -H python3.7 -m pip install tf_slim
sed -i "s/from tensorflow.contrib import slim/import tf_slim as slim/g" file.py
sudo -H python3.7 -m pip install opencv-python

Contrib does not exist in 2.0 or later. If you need contrib you have to use 1.15, although we recommend upgrading code to the contrib alternatives (since they are being actively maintained by an owner, unlike contrib which doesn't have an owner to fix issues)

Thank you, it was really helpful @mihaimaruseac


Strava.com

@adguard-bot commented on Wed Sep 12 2018
Issue URL (Incorrect Blocking)
https://www.Strava.com
Comment

Screenshots

Screenshot 1

System configuration

Information
value

Platform:
And 7.0

AdGuard version:
2.12.219

Browser:
Strava application

AdGuard mode:
VPN

Filtering quality:
Simplified

HTTPS filtering:
enabled

DNS requests blocking:
enabled

Filters:
EasyList Italy,AdGuard Mobile Ads,Liste FR,AdGuard Annoyances,AdGuard French,AdGuard Base,(Obsolete) Anti-Adblock Killer

@BlazDT commented on Thu Sep 13 2018
Please try to disable HTTPS filtering.
Can't sign up to get the app - please check @AdamWr.

@AdamWr commented on Thu Sep 13 2018
It happens even with all filters disabled, so it looks like a bug.
https://play.google.com/store/apps/details?id=com.strava

Can't reproduce on the latest nightly


Handle maxDataNodes and maxDataPoints

When attempting to copy log data from a server with larger values of maxDataNodes or maxDataPoints, the copy operation will fail due to attempting to send more data than the target server allows. Handle this by reworking all workers that handle log data to take maxDataNodes or maxDataPoints into consideration by splitting Add/Update queries into smaller chunks if they are larger than what the target server allows.

Highpriority if it creates problems for our current set of servers/datasets.

Nodes is numbers of rows, points is number of cells. If that answers your questions.

What exactly maxDataNodes and maxDataPoints represent in the context of WitsmlLog objects? E.g. are the data nodes equal to WitsmlLog and the data points equal to WitsmlLogCurveInfo?

Ok, so maxDataPoints are number of curves/mnemonics in a log. In the case we looked at, where the number was 7000000, that means 700 curves. (Multiply nodes with number of curves, and you'll receive maxDataPoints.)

test environment needed


Add version numbers to the install client docs

Feedback from customer -- would be nice to see the version of everything on the Install clients docs.

Completed as part of #5398


文件详情页

Coverage increased (+0.03%) to 98.569% when pulling 79b8719e60650a040e101bce531190ff119027a7 on Amaya-wyw:master into 5afc01c04b7f3184fed56d6d284542e141a43cb8 on rugao-web-fullstack:master.


logout not cleaning up document in _users

superlogin.logout, superlogin.logoutUser, and other forms of logout still leave documents in _users (attempted with both superlogin and superlogin-client)

I think I have the same issue. There are a number of duplicate entries for sl-users in _users that are not cleaned up but I'm not sure if they are supposed to be?

There are quite a few reasons this can occur.  One due to this issue I documented #186, then there is the case of logout on expired sessions if you use superlogin-client issue 71.  Of course if a user doesn't logout that would also leave sessions active, once the session expires you can periodically call removeExpiredKeys() on the server through an interval on your server.js or as a separate cron job.


Container crashes, how to find out more

We see sporadically crashing containers in our environments. Sometimes it takes a couple of hours, sometimes a couple of days, sometimes it doesn't happen and I struggle how to find out more. In the logs we see this

In this case it is a container from an image that was generated using artifactUrl=https://bcartifacts.azureedge.net/onprem/17.4.21491.21531/de, but we have seen it on other versions as well. This particular crash happened while no one was interacting with the system (1am)
Any ideas what we can do to find out what causes this problem?

Is it always stack overflows?
Is the container doing something special at this time?
Was the host doing something special at this time?

Is it always stack overflows?
So far yes

Is the container doing something special at this time?
Not that I know of

Was the host doing something special at this time?
Again, not that i know of

We tried increasing the memory allocation and are investigating further

So you don't know whether this is during container generation, app compilations, publishing - or is this just all over the place?


Code for generating benchmark

Adds a notebook to test different models against the representative test sets in the repository.

@gabrieltseng I am mainly looking for a second look at the Presto benchmarks at the bottom of the notebook, just to make sure everything is setup right.

The presto benchmarks look good to me :)
One note for the encodings - for the Taro mapping, results were better when we concatenated each channel group's embeddings, instead of taking a mean (link, but the code would look slightly different now because of the SRTM updates).
If we are confident that all the channel groups will be present then this might make sense to experiment with (but a drawback is that it makes the encodings quite a bit larger).

Thanks @gabrieltseng, that suggestion definitely seems like the best next step!


Clock

Hello, just wanted to let you know. From 12:00 Pm the clock shows 00:0523. First two characters stay 00 until 01:00

Thank you so much! I will fix that right away :)


Nearby Enemies have -9% to Chaos Resistance is misidentified

On a helm with both +25% to Chaos Resistance and Nearby Enemies have -9% to Chaos Resistance these values are getting wrapped into a single % total to Chaos Resistance pseudo stat.  For that helm the filter value in the evaluate window can't be increased beyond their combined "total" of 16%.
Same issue on a helm that has the nearby -9% chaos mod without +% chaos resistance on it.  Doesn't seem to be a problem for helms with the other elemental -9% delve mods (eg. helm with +lightning resist and nearby enemies have -9% lightning resist works fine).
Sample item:
Rarity: Rare
Woe Veil
Siege Helmet
--------
Quality: +20% (augmented)
Armour: 332 (augmented)
--------
Requirements:
Level: 69
Str: 101
Int: 134
--------
Sockets: B-R-R-B
--------
Item Level: 85
--------
Scourge Arrow creates an additional spore pod at Maximum Stages
--------
+95 to maximum Life
+25% to Chaos Resistance
Regenerate 1% of Life per second
Nearby Enemies have -9% to Chaos Resistance
20% increased Armour (crafted)
+16 to maximum Life (crafted)

Thanks for reporting. I'll look into it. Seems like a mismatching problem of the "Nearby Enemies have -9% to Chaos Resistance" stat.

0.5.16 (2020-02-13)

fix nearby enemies counted as chaos resistance (#213)


Chat - agent hover alignment issues

Disable wrapping for the agent name + text ellipsis in case it is too long

Pushed a fix, although it's not quite right. If we can't make the hover wider, I'm not exactly sure how this text should wrap. Probably it should be on a single wrapping line rather than this flexbox thing that wraps like a table.

Hi @roblourens thanks for the ping. I have traced the appearance of the 500 in the hover and it comes from here https://github.com/microsoft/vscode/issues/67076 in a PR merged 5 years ago. We can increase the maximum width of the hover to 750 px perhaps?

Fine with me!

I'll leave this issue open though because it will still be a problem for other extensions. @daviddossett do you have an opinion on how that text should wrap? I'm thinking it would just wrap as one line like
long extension
name | publisher name

where right now, the two parts wrap independently like a table

👆 that proposal seems fine to me


Download link

Your win64 download link is down.

Working fine on both website and GitHub page https://github.com/alex-free/gshax-tool/releases/download/v2.0.1/gshax-tool-v2.0.1-windows_x86_64.zip

Only works on my phone (safari?) Not working on firefox, use the release link for now: https://github.com/alex-free/gshax-tool/releases/tag/v2.0.1

Fixed thanks!


🌱 Fix pivoting feature tests in release 1.3

fix setting INIT_WITH_KUBERNETES_VERSION in release 1.3

/test-centos-e2e-feature-release-1-3

/test-centos-e2e-feature-release-1-3

/test-centos-e2e-feature-release-1-3

/test-ubuntu-integration-release-1-3

/test-ubuntu-integration-release-1-3

/test-e2e-1-25-1-26-upgrade-release-1-3

/test-e2e-1-25-1-26-upgrade-release-1-3

/test-e2e-1-25-1-26-upgrade-release-1-3

This PR also needs to run clusterctl upgrade test, as it changes  INIT_WITH_KUBERNETES_VERSION variable which is used in clusterctl upgrade test.

/lgtm


Remove NIO 2.9.0 build warnings.

Motivation:
We were one of the few first-party users of
writeWithUnsafeMutableWritableBytes, and so we hit some build warnings.
We should use the nice interface to request the number of bytes we
actually want.
Modifications:

Remove build warnings.
Correctly ask to read a whole SSL record instead of a fragment of one.

Result:
Better code

Looks like a spurious failure: the tests all ran and succeeded.

@swift-nio-bot test this please


How can I get named columns in rows?

Description
Is there is a way to get named rows in result?
query = """
    SELECT name
    FROM game
    WHERE id = :id
"""
parameters = {
    "id": game_id,
}
async with self.db_connection.execute(query, parameters) as cursor:
    result = await cursor.fetchone()

    if not result:
        return None

    game_name = result["game_name"]

print(result['game_name'])
TypeError: tuple indices must be integers or slices, not str

Right now I can only get retrieved values only by column index :\
    game_name = result[0]

Details

OS: OS X
Python version: 3.6
aiosqlite version: 0.17.0
Can you repro on 'main' branch? Yes
Can you repro in a clean virtualenv? Yes

Same way as you would with the stdlib sqlite3 module: set the .row_factory attribute on the connection object to aiosqlite.Row or sqlite3.Row.  See https://docs.python.org/3/library/sqlite3.html#sqlite3.Connection.row_factory for details.

Thank you for quick detailed response!


Show briefs as open instead of live in csv

Part of this story on Pivotal.
Briefs were being listed in the csv with their status as whatever their
status in the database is set at. To be more consistent with the rest of
the digitalmarketplace, if a brief's status is live it should be listed
as open in the csv.

Coverage remained the same at 89.07% when pulling ca90eb4a942ca300d7b30c480ae69fffcd304ca7 on change_live_to_open_on_buyers_csv into 546ef83c4b743f042dcf6809260c74d42eddfafc on master.

:+1:


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on<PHONE_NUMBER> for 46f9ecf41dddcb5ea5833e6d5d41a9a1491d97a2
https://gitlab.com/MatrixAI/open-source/js-rpc/-/pipelines/1033126805

Pipeline Succeeded on<PHONE_NUMBER> for 46f9ecf41dddcb5ea5833e6d5d41a9a1491d97a2
https://gitlab.com/MatrixAI/open-source/js-rpc/-/pipelines/1033126805


Production build isn't tree shaking buildable libraries

Current Behavior
I have produced a reproduction repo. The repo contains one application and one library. In the library are two components: HelloWorld and GoodByeWorld. HelloWorld is used by app.vue but GoodByeWorld isn't used by anything in the application. As a result, I would have expected GoodByeWorld to be removed from the bundle by webpack's tree shaking.
The current behaviour is that the app.js created by the production build still contains GoodByeWorld.vue. Here is the image from Bundle Analyzer:

And when I bring app.js into the editor and search for GoodByeWorld, I find:
...
,_=[],j=n["a"].extend({name:"GoodByeWorld"})
...

This indicates that GoodByeWorld is indeed in the bundle.
Expected Behavior
I expect GoodByeWorld to be removed from the bundle because it isn't referenced anywhere.
I have been converting a fairly large project over to NX and when I was finished I realized that tree shaking wasn't happening and my bundles were much bigger and contained much different content than I had expected.
Steps to Reproduce
Here is a github reproduction:
https://github.com/gregveres/nx-plus-vue-tree-shaking
clone this repo, and then run the  production build with:
nx build main --prod --mode="production"
This will launch bundle analyzer immediately. Just look at the app.js bundle (uncheck others).
Failure Logs
There are no failure logs
Environment
I am bulding a Vue 2 application

Plugin name and version<EMAIL_ADDRESS>
06-16 18:22:37 11> nx report

 >  NX   Report complete - copy this into the issue template

   Node : 16.13.2
   OS   : win32 x64
   npm  : 8.1.2

   nx : 14.3.6
   @nrwl/angular : Not Found
   @nrwl/cypress : 14.3.6
   @nrwl/detox : Not Found
   @nrwl/devkit : 14.3.6
   @nrwl/eslint-plugin-nx : 14.3.6
   @nrwl/express : Not Found
   @nrwl/jest : 14.3.6
   @nrwl/js : Not Found
   @nrwl/linter : 14.3.6
   @nrwl/nest : Not Found
   @nrwl/next : Not Found
   @nrwl/node : Not Found
   @nrwl/nx-cloud : Not Found
   @nrwl/nx-plugin : Not Found
   @nrwl/react : Not Found
   @nrwl/react-native : Not Found
   @nrwl/schematics : Not Found
   @nrwl/storybook : Not Found
   @nrwl/web : Not Found
   @nrwl/workspace : 14.3.6
   typescript : 4.7.3
   ---------------------------------------
   Community plugins:
         @nx-plus/vue: 14.0.0

Is there a way to extract the configuration for webpack?

I figured out what is missing.
I had to add "sideEffects": false to all of my library's package.json files.
Once I did that webpack started to work as expected.
Here is the issue  on the Nx main project site that pointed me in the right direction. Apparently this is a common issue. Webpack 5 is supposed to be better at doing the right thing automatically, but you can achieve it with webpack 4 with the use of the sideEffects: false entry.
I dropped over 200KB from my main app.js bundle size and 2MB from the total of my javascript files.
I am not sure if the plugin should be adding "sideEffects": false to the library's package.json file when it is created. I can see the argument that sideEffects: false might be promising something that might not be true for some libraries. But I always operate under the premiss that defaults should work for 90% of the use cases and I think that sideEffects: false definitely falls into that category.

This can also be changed from a bug to a feature request.

I think I have found the proper solution as mentioned here..
For the libraries that contain vue files, I have changed sideEffects: false to sideEffects: ["*.vue"].
This tells webpack that vue files still have side effects and this causes webpack to properly include the .css files that correspond to the vue files.
I believe this new version of the sideEffect can be used for libs that don't have .vue files without problem. So, to amend my question above, should the libs have this sideEffect: ["*.vue"] by default?

I've run into a very similar problem with nx + Angular in the past. Just to clarify, are you building the libs and then consuming them in your app or relying on Typescript aliases. From your repro it looks like the former, I'd be curious about tree-shaking when it comes to just relying on the aliases.
I think it makes sense to add the sideEffect to the generated packge.json for buildable libs if you're interested in contributing.

I am using buildable libs. I suspect that with non buildable line you would write the code to just reach into the lib directory and pull out the file you need. This would be more like not using no and I tree shaking would work automatically.
I will look at a PR.
I see you guys have added a vite plug-in. Are you moving to vue 3 and vite from vue 2 and webpack?

On Jun 17, 2022, at 1:29 PM, Zachary Williams @.***> wrote:

I've run into a very similar problem with nx + Angular in the past. Just to clarify, are you building the libs and then consuming them in your app or relying on Typescript aliases. From your repro it looks like the former, I'd be curious about tree-shaking when it comes to just relying on the aliases.
I think it makes sense to add the sideEffect to the generated packge.json for buildable libs if you're interested in contributing.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

@gregveres as long as vue-cli is popular I plan on supporting it. I added the vite plugin cause it was fun to do

I have the same problem


Ethan: Blacklist 10play.live

Ethan requests the blacklist of the website 10play\.live. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

10play\.live has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Machavity in Charcoal HQ


Add Cache Yarn dependencies

https://github.com/c-hive/gha-remove-artifacts

Looks good, fixed the title casing. Thanks!


[BUG] Integration test is flaky

Describe the bug
can_stop_detached_host seems to be flaky, failing ~50% of the time with nextest
Example in GHA
https://github.com/wasmCloud/wash/actions/runs/4441870676/jobs/7797463153
I've reproduced this locally 3x in a row while running make test-integration. Not sure I can get it to pass locally at all

The following tests consistently fail for me on main including can_stop_detached_host .
--- TRY 3 STDERR:        wash-cli::integration_up integration_up_can_start_wasmcloud_and_actor ---

{
  "error": "Could not start NATS server, a process is already listening on <IP_ADDRESS>:5893",
  "success": false
}
thread 'integration_up_can_start_wasmcloud_and_actor' panicked at 'assertion failed: status.success()', tests/integration_up.rs:28:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

------------
     Summary [  46.890s] 29 tests run: 26 passed (1 flaky), 3 failed, 0 skipped
   FLAKY 2/3 [   7.906s] wash-cli::integration_inspect integration_inspect_cached
  TRY 3 FAIL [   1.101s] wash-cli::integration_up can_stop_detached_host
  TRY 3 FAIL [   0.665s] wash-cli::integration_up doesnt_kill_unowned_nats
  TRY 3 FAIL [   0.040s] wash-cli::integration_up integration_up_can_start_wasmcloud_and_actor

Fixed by #520

Resolved by #520

Resolved by #520


instance predicate returns false for objects

Raised elsewhere by @euoia:

I have found a case where check.instance doesn't agree with instanceof. In Chrome and Safari:
document.getElementById('el') instanceof HTMLElement
> true

But due to how the instance function in check-types.js checks that prototype is a function (see https://github.com/philbooth/check-types.js/blob/master/src/check-types.js#L151), the following returns false in Safari and true in Chrome:
check.assert.instance(document.getElementById('el'), HTMLElement);

Due to this behaviour in Safari:
typeof HTMLElement
> "object"

And this in Chrome:
typeof HTMLElement
> "function"

So although this seems due to a browser implementation difference, it is at least surprising that check.instance doesn't agree with instanceof. Is it necessary for the instance function to check that prototype is a function?

Yeah, whatever the reason for doing that was, I don't remember it now. If there even was a reason. :confused:
Will fix it.

Oh wait, I do know why it's there.
If you use a non-function as the rhs to instanceof, it throws a TypeError. At least it does for me in Firefox, Chrome and node.js. What happens when you do null instanceof {} in your Safari console?

It throws a TypeError:
null instanceof {}
> TypeError: Object is not a valid argument for 'instanceof' (evaluating 'null instanceof {}')

Thanks. I'm minded to keep the current behaviour at the moment, or perhaps a modified version of it.
I made a conscious decision a few versions back that the predicates should never throw, they must always be relied on to return true or false. Before then, the like predicate had a condition in which it threw for instance, which made calling code more complex than it needed to be. So I'd definitely like to adhere to the 'never throw from a predicate' principle.
But maybe it is possible to special-case the instance predicate so that it handles your example sanely without causing errors to be thrown when random objects get passed to it. I haven't thought about how to do that, so feel free to make suggestions if you have any.
In the meantime, you could work around it using assert-as-a-function:
check.assert(document.getElementById('el') instanceof HTMLElement);

What do you think?

Oh, I didn't realise you can do that. That's useful, thanks.
I completely agree that predicates should never throw. It seems like for now special casing instance might be the best solution.
Thanks again for taking the time to respond to this.

Decided to fix this just by catching the error. I'm mystified why that didn't occur to me before tbh. :confused:
Anyway, published on npm as 4.0.0, with a new unit test added for this case.


fix: Fixes a server client boundary error

Summary | Résumé
Fixes a prod error where NextJS tried to server render some client code.  Here is the prod error for reference: "ReferenceError: sessionStorage is not defined"

@thiessenp-cds can you add test instructions ?

@thiessenp-cds can you add test instructions ?

This is a difficult one to test. The error only happens sometimes and manually throwing an error etc. won't really help since it's a hydration related. For this fix, I think all we can do is test that it doesn't break anything. (still works)
@bryan-robitaille can you think of any way to test this?

:)
Can just test by visiting https://localhost:3000/en/auth/mfa/resend direct


ISPN-9047 Enable view bundling in JGroups configurations

https://issues.jboss.org/browse/ISPN-9047

Integrated, thanks.


Looking forward to code😄

Nice work!

Nice work!

Thanks for your recognition. Code and models are available.


No logs appear in logLevel: error | v0.8.2

Karpenter: v0.8.2

Kubernetes: v1.21.5
Hi guys, I upgraded to v0.8.2 and I cannot see any logs when I change logLevel to error. Logs will be seen only in debug.
The installation has been done through terraform with changing log level from there:
  set {
    name  = "logLevel"
    value = var.log_level
  }

I can see in config-logging that level was changed to error:
apiVersion: v1
data:
  zap-logger-config: |
    {
      "level": "error",
      "development": false,
      "disableStacktrace": true,
      "disableCaller": true,
      "sampling": {
        "initial": 100,
        "thereafter": 100
      },
      "outputPaths": ["stdout"[],
      "errorOutputPaths": ["stderr"[],
      "encoding": "console",
      "encoderConfig": {
        "timeKey": "time",
        "levelKey": "level",
        "nameKey": "logger",
        "callerKey": "caller",
        "messageKey": "message",
        "stacktraceKey": "stacktrace",
        "levelEncoder": "capital",
        "timeEncoder": "iso8601"
      }
    }
kind: ConfigMap
metadata:
  annotations:
    meta.helm.sh/release-name: karpenter
    meta.helm.sh/release-namespace: karpenter
  creationTimestamp: "2022-04-20T13:03:57Z"
  labels:
    app.kubernetes.io/instance: karpenter
    app.kubernetes.io/managed-by: Helm
    app.kubernetes.io/name: karpenter
    app.kubernetes.io/version: 0.8.2
    helm.sh/chart: karpenter-0.8.2
  name: config-logging
  namespace: karpenter
  resourceVersion: "482477052"
  uid: 67150e97-6f12-45bf-b697-5368799c00fe

When following the logs after applying the logLevel change, no logs will appear at all.
I would see logs again only if I changed logLevel back to debug.

Hey @devopsjnr ,
This is actually by design assuming Karpenter is functioning correctly.  At ERROR log level you will not see logs emitted with the INFO or DEBUG log level.  You should only see logs which are at severity of ERROR or greater.
I'm curious if the INFO level is a better fit for what you're expecting to see in logs.  Or can you share what you were expecting to see in the logs at ERROR log level?

@dewjam is correct. To ensure logs would show up if there are errors you can cause a fake error, e.g. you can rename one of the many values in the provisioner to a fake one.

when I choose logLevel to error, I expect to log in all levels including ERROR logs.
Currently if I set logLevel to error, no logs appear at all, nothing. although I caused errors deliberately.
I saw now that if I leave logLevel as debug, I see all kinds of logs - including ERROR logs. Why is that?

Generally, log levels only show logs at that level and "worse". Here's a table that might make this clearer:
const (
	// DebugLevel logs are typically voluminous, and are usually disabled in
	// production.
	DebugLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>	// InfoLevel is the default logging priority.
	InfoLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>	// WarnLevel logs are more important than Info, but don't need individual
	// human review.
	WarnLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>	// ErrorLevel logs are high-priority. If an application is running smoothly,
	// it shouldn't generate any error-level logs.
	ErrorLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>	// DPanicLevel logs are particularly important errors. In development the
	// logger panics after writing the message.
	DPanicLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>	// PanicLevel logs a message, then panics.
	PanicLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>	// FatalLevel logs a message, then calls os.Exit(1).
	FatalLevel = [zapcore](https://pkg.go.dev/go.uber.org/zap<EMAIL_ADDRESS>)

https://pkg.go.dev/go.uber.org/zap#pkg-constants
So when you set the logger to ERROR, you'll only see things logged at ERROR, PANIC, and FATAL.  If DEBUG logs are showing you too much, you probably want to set your logger at INFO. INFO will show you everything at INFO, WARN, ERROR, PANIC, and FATAL.

@bwagner5  Got it, thanks a lot!!


Errors in the example for Built-in Scrapers

The example under https://github.com/yields/ant#built-in-scrapers has a few minor errors in it.
A corrected version would be:
package main

import (
	"context"
	"os"

	"github.com/yields/ant"
)

func main() {
	// Describe how a quote should be extracted.
	type Quote struct {
		Text string   `css:".text"`
		By   string   `css:".author"`
		Tags []string `css:".tag"`
	}

	// A page may have many quotes.
	type Page struct {
		Quotes []Quote `css:".quote"`
	}

	// Where we want to fetch quotes from.
	const host = "quotes.toscrape.com"

	// Initialize the engine with a built-in scraper
	// that receives a type and extract data into an io.Writer.
	eng, err := ant.NewEngine(ant.EngineConfig{
		Scraper: ant.JSON(os.Stdout, Page{}),
		Matcher: ant.MatchHostname(host),
	})
	if err != nil {
		panic(err)
	}

	// Block until there are no more URLs to scrape.
	if err := eng.Run(context.Background(), "http://"+host); err != nil {
		panic(err)
	}
}

One suggestion I have is to include examples in your README using something like https://github.com/campoy/embedmd which would make it easier to spot when examples are out of date for some reason.

I like it! thanks for the suggestion, fixed in https://github.com/yields/ant/commit/619106f7cc62d1985bb1df4493dd96d5256aee52


UnboundLocalError: cannot access local variable 'ss' where it is not associated with a value

immediately after  python -m main
tried api keys with and without quotes
all packages installed fine pip install -r req...
using BYBIT feed and exchange
i did not add quotes or removed quotes from param files
what are your exact packages versions?
what python do u use?
Traceback (most recent call last):
  File "/home/ubuntu/bybit-smm/frameworks/sharedstate.py", line 238, in load_parameters
    self.process_parameters(params, reload)
  File "/home/ubuntu/bybit-smm/smm/sharedstate.py", line 35, in process_parameters
    raise e
  File "/home/ubuntu/bybit-smm/smm/sharedstate.py", line 21, in process_parameters
    self.load_exchange(settings["exchange"])
  File "/home/ubuntu/bybit-smm/frameworks/sharedstate.py", line 155, in load_exchange
    self.exchange = Bybit(self.api_key, self.api_secret)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/bybit-smm/frameworks/exchange/bybit/exchange.py", line 18, in __init__
    formats=BybitFormats(),
            ^^^^^^^^^^^^^^
  File "/home/ubuntu/bybit-smm/frameworks/exchange/bybit/formats.py", line 17, in __init__
    convert_side=BybitSideConverter(),
                 ^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/bybit-smm/frameworks/exchange/bybit/types.py", line 11, in __init__
    super().__init__(BUY="Buy", SELL="Sell")
  File "/home/ubuntu/bybit-smm/frameworks/exchange/base/constants.py", line 99, in __init__
    super().__init__(str_to_int={f"{BUY}": Side.BUY, f"{SELL}": Side.SELL})
TypeError: StrNumConverter.__init__() got an unexpected keyword argument 'str_to_int'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/ubuntu/bybit-smm/smm/main.py", line 21, in main
    ss = SmmSharedState(debug=True)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ubuntu/bybit-smm/smm/sharedstate.py", line 11, in __init__
    super().__init__(debug)
  File "/home/ubuntu/bybit-smm/frameworks/sharedstate.py", line 83, in __init__
    self.load_parameters()
  File "/home/ubuntu/bybit-smm/frameworks/sharedstate.py", line 241, in load_parameters
    raise Exception(f"Error loading parameters: {e}")
Exception: Error loading parameters: StrNumConverter.__init__() got an unexpected keyword argument 'str_to_int'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/ubuntu/bybit-smm/smm/main.py", line 35, in main
    await ss.logging.critical(f"Unexpected exception occurred: {e}")
          ^^
UnboundLocalError: cannot access local variable 'ss' where it is not associated with a value

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/ubuntu/bybit-smm/smm/main.py", line 47, in <module>
    asyncio.run(main())
  File "/usr/lib/python3.12/asyncio/runners.py", line 194, in run
    return runner.run(main)
           ^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/asyncio/runners.py", line 118, in run
    return self._loop.run_until_complete(task)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "uvloop/loop.pyx", line 1517, in uvloop.loop.Loop.run_until_complete
  File "/home/ubuntu/bybit-smm/smm/main.py", line 39, in main
    await ss.logging.critical("Starting shutdown sequence...")
          ^^
UnboundLocalError: cannot access local variable 'ss' where it is not associated with a value

tons of stuff still not finished in the alpha version! many core functionalities are still being pushed, but most of it is finished up locally. ill get a working version for bybit/binance up and running by next week if you want to have another go then :)


Text + Suggestions: Rendering Buttons (Rich Cards)

Text + Suggestions (buttons) currently don’t create buttons in neither the chat plugin nor messaging UI.
As an example, it would be a good idea render a button below “some text” here:

this is currently not supported by the live chat plugin and it's unclear if this is a request to support the feature (it reads more like a bug report... so I'm just trying to clarify)

if I'm not mistaken, @steffh and I discussed it as a feature similar to what facebook calls "quick replies" - so plain text with a button below it... I think it fits more as a request to support the feature; that was a mistake on my end!

this seems to be a duplicate to this ticket https://github.com/airyhq/airy/issues/862 and be also part of that ticket: https://github.com/airyhq/airy/issues/1072
i think we can close it as this is already being worked on.

it will get rendered in both since the work is done in the lib

@steffh how can it be a duplicate if we're talking about different sources? The ability to render structured data is source dependent so, say you want to render "suggestions from google" like we did in #862 for the chat plugin as a source (basically using their payloads in our own source), we have to implement the feature.
I believe the source of confusion here is around the fact we add rendering capabilities to core via a TypeScript library. While that's true, it doesn't solve the problem that each source must have their own definitions/ability to render specific payloads. The library solves a different problem (which is only organizational so to speak), it's because we'd have to duplicate the code of the rendering for the chat plugin both in the plugin itself and in the UI.
So if this is the problem, then this is not a duplicate nor a bug report. It's "multiple features" requests in one ticket (which is totally understandable)

@lucapette okay, got it. if that's the case, we maybe just need to change the title of this ticket here because it currently doesn't say "chat plugin" as the source it is related to.
i just was under the impression we use the same code, because we follow the google standard in the payload, but as i can see you seem to duplicate that and treat it per source even if you copy code, but that's most likely a smart choice as sources can indeed take different directions.

Suggestion Requirements

A row of suggested replies that appear as a row of suggestion chips following the associated message. Maximum 13 suggestions, with a maximum of 25 characters each.

The chips only display when the associated message is the most recent message within the conversation (including both system user and contact messages). The contact can tap a suggested reply to send the text reply to the system user.

The chips will disappear when one has been clicked, so it is an ephemeral element of a conversation.

These suggestions (buttons) should be displayed horizontally below the plain text that prompts them.

Note: Chips are compact elements that represent an input, attribute, or action. Chips allow users to enter information, make selections, filter content, or trigger actions.

Sorry, but from reading this I am still not clear where the data is coming from that will be displayed here.

Since this is in the chatplugin source we define the data we want to send, like we do with the RichCards. We can follow the same payload structure as the RichCards, and adapt it.