[WIP] CXF-8518: Fixing jaxrs.spec.provider.standardnotnull clientJaxbProviderTest

Per specification:
for a zero-length response entities returns a corresponding
Java object that represents zero-length data. In case no zero-length representation
is defined for the Java type, a {@link ProcessingException} wrapping the
underlying {@link NoContentException} is thrown.

For JAXB serialization / desearialization, the empty input stream (empty response) leads to javax.xml.bind.UnmarshalException, for example:
[com.ctc.wstx.exc.WstxEOFException: Unexpected EOF in prolog
 at [row,col {unknown-source}]: [1,0]]
	at com.sun.xml.internal.bind.v2.runtime.unmarshaller.UnmarshallerImpl.handleStreamException(UnmarshallerImpl.java:470)
	at com.sun.xml.internal.bind.v2.runtime.unmarshaller.UnmarshallerImpl.unmarshal0(UnmarshallerImpl.java:402)
	at com.sun.xml.internal.bind.v2.runtime.unmarshaller.UnmarshallerImpl.unmarshal(UnmarshallerImpl.java:379)
	at org.apache.cxf.jaxrs.provider.JAXBElementProvider.readFrom(JAXBElementProvider.java:177)
	at org.apache.cxf.jaxrs.provider.JAXBElementTypedProvider.readFrom(JAXBElementTypedProvider.java:41)
	at org.apache.cxf.jaxrs.provider.JAXBElementTypedProvider.readFrom(JAXBElementTypedProvider.java:34)
	at org.apache.cxf.jaxrs.utils.JAXRSUtils.readFromMessageBodyReader(JAXRSUtils.java:1444)
	at org.apache.cxf.jaxrs.impl.ResponseImpl.doReadEntity(ResponseImpl.java:472)
	at org.apache.cxf.jaxrs.impl.ResponseImpl.readEntity(ResponseImpl.java:415)
	at org.apache.cxf.jaxrs.impl.ResponseImpl.readEntity(ResponseImpl.java:403)

Build is green: https://ci-builds.apache.org/job/CXF/job/CXF-JDK8-PR-Manual/14/

@andymc12 as always, need your expertise, mind please taking a look? thanks a lot!

I like this better than my too-brittle hack :)

https://github.com/dblevins/cxf/commit/32103901420a7811f1b5af0c37e9bf797fac8b81

The one concern I'd have is that inputStream.available() is a guess and could potentially return zero when there is a valid payload coming.  That might happen in scenarios where the server-side could just be taking a while streaming back the payload.
I wonder if we wrapped the input stream instead of returning null and had the wrapper count the bytes read.  We could check the count on an exception and use that to determine our blocking read returned no payload.
It'd be sort of a hybrid of the two approaches.  Thoughts?

@dblevins thanks a lot for looking, that is correct, we do trust inputStream.available() only if it returns non-zero value
            // if available is 0 it does not mean it is empty
            if (is.available() > 0) {
                return false;
            }

Otherwise the assumption is that stream is not empty (and we proceed further with mark / reset of pushback stream). What do you think?

I wonder if we wrapped the input stream instead of returning null and had the wrapper count the bytes read. We could check the count on an exception and use that to determine our blocking read returned no payload.

Thought about that as well but dealing with exceptions and counting the bytes would be more complicated comparing to the simple check that stream is empty.

@reta You're totally right.  I read too quickly and missed that available() is only getting acted upon if the response is a definitive "yes" and there's pretty comprehensive fallback logic in the isEmpty method.  I think you have it covered pretty well.
My +1


glightbox-mkdocs plugin does not work with mkdocs-multirepo-plugin

On its own, I can get the glightbox-mkdocs plugin to work without any issues. My images become 'zoomable' which is the desired effect I am looking for.
However, when I import that same repo that works with glightbox-mkdocs in a standalone manner, the 'zoomable' feature of my images in markdown no longer are 'zoomable'.
I installed the glightbox-mkdocs plugin in both my 'parent' multi-repo repo and the child repo, but that did not help.
Perhaps you could reproduce this and see what the issue might be?

@jdoiro3 I have some further debug info. You can see below that the glightbox script is missing when hosted on Gitlab pages with the mkdocs-multirepo-plugin, but the script is present when hosted on my local workstation via mkdocs serve:
Hosted on Gitlab Pages

Locally Hosted by running mkdocs serve

I had a plugins: section in both my mkdocs.yml and an inherited mkdocs.yml file and they do NOT merge. Moved all plugins to mkdocs.yml file and it worked.


[jvm-packages] Change log level for tracker message.

Close https://github.com/dmlc/xgboost/issues/7962 .

@yangjiandan

Partially revert https://github.com/dmlc/xgboost/pull/7952 .

LGTM


ci: fixed node version and semantic-release

ci: updated semantic-release step
ci: bumped node to v18
build: fixed package name

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


Hotfix/forward header leaking

Same PR as #197, now to a release branch on top of 0.7.0

Note; this is to a patch release branch


Store/expose caught errors before rethrowing

Description
Store/expose caught errors before rethrowing custom wagmi-specific errors.
Motivation: I wanted to dive deeper into why Rainbow was throwing a "switch chain error" when calling switch chain and realized I didn't have access to the originally thrown error, which might have more hints as to what's going on. (See #403)
I initially extended ~all of the custom wagmi errors, but found only a subset were used in try/catch blocks, so ended up only wrapping those. There are a couple places in mocks where we pass an undefined argument, which is a little gross, but the argument type safety seemed nice to have in all other spots.
Additional Information

[x] I read the contributing docs (if this is your first contribution)

Your ENS/address:
frolic.eth

For my specific use case, this was an issue switching to Rinkeby on Rainbow for iOS. The underlying error is Chain currently not supported. I wonder if wagmi should detect/throw SwitchChainNotSupportedError instead?

Great idea! (Amazing this is #404.) Traveling and will take a look soon.

🙏  much nicer, thanks for making this pretty


[ new ] grow

This PR adds grow for MArrays, provided by @stefan-hoeck, which grows an MArray by the given number of elements.

@stefan-hoeck
Not sure if there is a more efficient way to implement this, maybe using unsafeCreate?  I was having trouble getting it to compile though.

@stefan-hoeck
Not sure if there is a more efficient way to implement this, maybe using unsafeCreate? I was having trouble getting it to compile with unsafeCreate unfortunately.

I've addressed this via f1c0a836743d272ea96671953ab3a57393754d89.

@stefan-hoeck
I've converted this to draft, the implementations of grow here are very slow.  Would love if you had any ideas of how to implement this efficiently (maybe via a %foreign)?  Not sure.

I should be able to find some time to look into this and your PR for containers tomorrow. Sorry, it's a busy time.

@stefan-hoeck
I've converted this to draft, the implementations of grow here are very slow.  Would love if you had any ideas of how to implement this efficiently (maybe via a %foreign)? Or maybe a grow on a MArray would be best performance-wise (have no idea how to implement this though).

@stefan-hoeck
Actually, I think for maximum performance here, we'd definitely want grow on a MArray, again just have no idea how this would be implemented.

Here's how to safely grow a mutable array:
parameters {0 rs : Resources}
           {n : Nat}
           (r : MArray n a)
           {auto 0 p : Res r rs}

  writeMArray :
       (k : Nat)
    -> {auto 0 lte : LTE k n}
    -> (tgt : MArray (m+n) a)
    -> F1' (tgt::rs)
  writeMArray 0     tgt t = () # t
  writeMArray (S k) tgt t =
    let v # t := getNat r k t
        _ # t := setNat tgt k {lt = ltAddLeft lte} v t
     in writeMArray k tgt t

  ||| Allocates a new mutable array and adds the elements from `r`
  ||| at its beginning.
  export
  mgrow : (m : Nat) -> (deflt : a) -> (1 t : T1 rs) -> A1 rs (MArray (m+n) a)
  mgrow m deflt t =
    let A tgt t := newMArray (m+n) deflt t
        _ #   t := writeMArray n tgt t
     in A tgt t

||| Utility for growing and replacing a single mutable array.
mgrow1 :
     {n : Nat}
  -> (r : MArray n a)
  -> (m : Nat)
  -> (deflt : a)
  -> (1 t : T1 [r])
  -> A1 [] (MArray (m+n) a)
mgrow1 r m dflt t =
  let A tgt t := mgrow r m dflt t
      _ #   t := release r t
   in A tgt t

Growing an immutable indexed array could be implemented in a similar fashion by allocating a new mutable array, writing its elements, and freezing it at the end. Growing an existential array (Array) should be implemented in terms of the indexed version.
The above requires the following proof to be added to Data.Array.Index:
export
0 ltAddLeft : LT k n -> LT k (m+n)
ltAddLeft {m = 0}   lt = lt
ltAddLeft {m = S x} lt = lteSuccRight $ ltAddLeft lt

For performance reasons, we might want to add array copying as a primitive via the FFI. This should go to another PR, however. Also, after some discussions on discord, I'll look into making R1 in idris2-ref1 a proper dependent type. Not sure how well that will work out but it might make resource juggling a bit more straight forward.

@stefan-hoeck
Awesome, thank you for spending time to help with this!
I have added the grow implementations you've provided for MArray via ab669e3729696524c1688f9517f60470bd6b5037.
I've also removed the slow grow implementations I wrote for Array/IArray in ab669e3729696524c1688f9517f60470bd6b5037.

Could you verify that my version shows decent performance? I haven't tested it.

Could you verify that my version shows decent performance? I haven't tested it.

Yes I'll re-write the benchmark with it and report back here.

@stefan-hoeck
I think the builds are now failing for some reason:
[ build ] Data.Array.Mutable:221:9--221:10
[ build ]  217 |         -> (deflt : a)
[ build ]  218 |         -> (1 t : T1 rs)
[ build ]  219 |         -> A1 rs (MArray (m+n) a)
[ build ]  220 |   mgrow m deflt t =
[ build ]  221 |     let A tgt t := newMArray (m+n) deflt t
[ build ]                ^
[ build ]
[ build ] Error: While processing right hand side of mgrow1. Undefined name A.
[ build ]
[ build ] Data.Array.Mutable:235:7--235:8
[ build ]  231 |   -> (deflt : a)
[ build ]  232 |   -> (1 t : T1 [r])
[ build ]  233 |   -> A1 [] (MArray (m+n) a)
[ build ]  234 | mgrow1 r m dflt t =
[ build ]  235 |   let A tgt t := mgrow r m dflt t
[ build ]              ^

Maybe its because of the latest PR?

@stefan-hoeck
Also seeing this now trying to build locally:
[ build ] 3/8: Building Data.Array.Mutable (src/Data/Array/Mutable.idr)
[ build ] Error: While processing right hand side of mgrow. When unifying:
[ build ]     a (implicitly bound at Data.Array.Mutable:221:9--221:10)
[ build ] and:
[ build ]     MArray (?m + n) a
[ build ] Mismatch between: a (implicitly bound at Data.Array.Mutable:221:9--221:10) and MArray' RPure (?m + n) a.
[ build ]
[ build ] Data.Array.Mutable:222:34--222:37
[ build ]  218 |         -> (1 t : T1 rs)
[ build ]  219 |         -> A1 rs (MArray (m+n) a)
[ build ]  220 |   mgrow m deflt t =
[ build ]  221 |     let A tgt t := newMArray (m+n) deflt t
[ build ]  222 |         _ #   t := writeMArray n tgt t
[ build ]                                         ^^^
[ build ]

Yes, sorry, there has been a breaking chang in idris2-ref1. It should be enough to replace A x t with x # t for all x everywhere in the code I posted above. Data type A1 and its constructor no longer exist.

Seems that this now gives:
[ build ] Error: While processing right hand side of mgrow. Ambiguous elaboration. Possible results:
[ build ]     ?postpone [no locals in scope]
[ build ]     ?postpone [no locals in scope]
[ build ]     ?postpone [no locals in scope]
[ build ]
[ build ] Data.Array.Mutable:221:13--221:14
[ build ]  217 |         -> (deflt : a)
[ build ]  218 |         -> (1 t : T1 rs)
[ build ]  219 |         -> A1 rs (MArray (m+n) a)
[ build ]  220 |   mgrow m deflt t =
[ build ]  221 |     let tgt # t := newMArray (m+n) deflt t
[ build ]                    ^
[ build ]
[ build ] Error: While processing right hand side of mgrow1. Sorry, I can't find any elaboration which works. All errors:
[ build ] If Data.Linear.Token.(#): When unifying:
[ build ]     R1 ?rs ?a
[ build ] and:
[ build ]     A1 ?rs (MArray' RPure (plus ?_ ?_) ?_)
[ build ] Mismatch between: R1 ?rs ?a and A1 ?rs (MArray' RPure (plus ?_ ?_) ?_).
[ build ]
[ build ] Data.Array.Mutable:235:7--235:14
[ build ]  231 |   -> (deflt : a)
[ build ]  232 |   -> (1 t : T1 [r])
[ build ]  233 |   -> A1 [] (MArray (m+n) a)
[ build ]  234 | mgrow1 r m dflt t =
[ build ]  235 |   let tgt # t := mgrow r m dflt t
[ build ]              ^^^^^^^
[ build ]
[ build ] If Builtin.(#): When unifying:
[ build ]     LPair ?a ?b
[ build ] and:
[ build ]     A1 ?rs (MArray' RPure (plus ?_ ?_) ?_)
[ build ] Mismatch between: LPair ?a ?b and A1 ?rs (MArray' RPure (plus ?_ ?_) ?_).
[ build ]
[ build ] Data.Array.Mutable:235:7--235:14
[ build ]  231 |   -> (deflt : a)
[ build ]  232 |   -> (1 t : T1 [r])
[ build ]  233 |   -> A1 [] (MArray (m+n) a)
[ build ]  234 | mgrow1 r m dflt t =
[ build ]  235 |   let tgt # t := mgrow r m dflt t
[ build ]              ^^^^^^^
[ build ]
[ build ] If Builtin.DPair.(#): When unifying:
[ build ]     Res ?a ?t
[ build ] and:
[ build ]     A1 ?rs (MArray' RPure (plus ?_ ?_) ?_)
[ build ] Mismatch between: Res ?a ?t and A1 ?rs (MArray' RPure (plus ?_ ?_) ?_).
[ build ]
[ build ] Data.Array.Mutable:235:7--235:14
[ build ]  231 |   -> (deflt : a)
[ build ]  232 |   -> (1 t : T1 [r])
[ build ]  233 |   -> A1 [] (MArray (m+n) a)
[ build ]  234 | mgrow1 r m dflt t =
[ build ]  235 |   let tgt # t := mgrow r m dflt t
[ build ]              ^^^^^^^
[ build ]

Maybe I have to replace A1 with something else as well?

You need to be on the latest pack collection for this to work. Use pack switch latest to update the pack collection.


Logging on slurm stopped working

🐛 Bug
Logging and checkpoint saving stopped working for me when I run experiments via slurm system.
I am using log keys in return functions: training_epoch_end/validation_epoch_end.
Version 0.7.6 works.
To Reproduce
Steps to reproduce the behaviour:

Define Tensorboard logger
Run training using slurm system sbatch ...
No logs.

Code sample

Expected behaviour

Environment

PyTorch 1.4.0:
PyTorch-lightning 0.8.1,
Linux,
Python 3.7.6,
CUDA/cuDNN 10.1, 7.6.5,

Hi, I think I'm having the same problem, running locally logs work correctly (I'm sending to comet), but when I run on a cluster through slurm using sbatch or srun, the experiments in comet are created, but none of the logging works.

I think this might be due to an issue due to how the rank id is set, I'm not totally sure, but it could have occurred here: https://github.com/PyTorchLightning/pytorch-lightning/pull/2231
I guess it's due to a malfunction with rank_zero_only, sucht that the gated code is never executed.
See also comment in https://github.com/PyTorchLightning/pytorch-lightning/issues/2278#issuecomment-646997797

If you want a quick fix, just remove this line. (Dirty solution)

Fixed by #2339


Vanilla Snapshots Encountering Fatal Errors "No Such File or Directory"

I'm trying to spin up a container with the latest snapshot jar, but for some reason it refuses to launch for me. I had a snapshot up and running previously, but ever since the pre-release of 1.16, none of the snapshots (including older ones like 20w19a) are working.

Here are the logs that are output when the server crashes:

[init] Running as uid=1000 gid=1000 with /data as 'drwxrwxrwx 1 1000 1000 4096 Jun 5 11:08 /data'
[init] Resolved version given SNAPSHOT into 1.16-pre2
[init] Resolving type given VANILLA
[init] server.properties already created, skipping
[init] Setting/adding ops
[init] log4j2.xml already created, skipping
[init] Checking for JSON files.
[init] Setting initial memory to 1G and max to 4G
[init] Starting the Minecraft server...
[11:08:59] [main/ERROR]: Unknown biome, defaulting to plains
[11:09:01] [main/INFO]: Environment: authHost='https://authserver.mojang.com', accountsHost='https://api.mojang.com', sessionHost='https://sessionserver.mojang.com', name='PROD'
Exception in thread "main" [11:09:01] [main/INFO]: [STDERR]: java.lang.Error: java.io.IOException: No such file or directory
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.FileKey.create(FileKey.java:46)
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.SharedFileLockTable.(FileLockTable.java:120)
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.FileLockTable.newSharedFileLockTable(FileLockTable.java:47)
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.FileChannelImpl.fileLockTable(FileChannelImpl.java:1039)
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.FileChannelImpl.tryLock(FileChannelImpl.java:1106)
[11:09:01] [main/INFO]: [STDERR]: at java.nio.channels.FileChannel.tryLock(FileChannel.java:1155)
[11:09:01] [main/INFO]: [STDERR]: at ado.a(SourceFile:42)
[11:09:01] [main/INFO]: [STDERR]: at daa$a.(SourceFile:280)
[11:09:01] [main/INFO]: [STDERR]: at daa.c(SourceFile:268)
[11:09:01] [main/INFO]: [STDERR]: at net.minecraft.server.Main.main(SourceFile:118)
[11:09:01] [main/INFO]: [STDERR]: Caused by: java.io.IOException: No such file or directory
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.FileKey.init(Native Method)
[11:09:01] [main/INFO]: [STDERR]: at sun.nio.ch.FileKey.create(FileKey.java:44)
[11:09:01] [main/INFO]: [STDERR]: ... 9 more
2020-06-05T11:09:01.719-0400 INFO mc-server-runner Done

Also, here are the contents of my docker-compose file:
minecraft-server:
    ports:
      - "25567:25565"

    environment:
      EULA: "TRUE"
      TZ: "America/Indiana/Indianapolis"
      TYPE: "VANILLA"
      VERSION: "SNAPSHOT"
      DIFFICULTY: "normal"
      SPAWN_PROTECTION: 0
      OPS: "AverageCakeSlice"
      INIT_MEMORY: "1G"
      MAX_MEMORY: "4G"

    volumes:
      - "C:/Users/Bread/_containers/gameboise/_data:/data"

    image: itzg/minecraft-server

    container_name: gameboise

    tty: true
    stdin_open: true
    restart: always

Unfortunately the exception it threw was pretty vague since it didn't seem to state what file wasn't found. FWIW, I was able to start a fresh snapshot container:
https://gist.github.com/itzg/bcd2315a1d658c0af14b5dc04a76146b#file-gistfile1-txt
I'll try later with an existing container / data directory that I have running on a previous snapshot.

Huh, I was able to execute the command that you just did, and was able to do it with the detach flag. It must be something in my compose file or on my local machine. Thanks.

Same problem occurred with me, same log data.  Seems to be related to attaching data directory to local system (-v  and using WSL2 as the docker engine.
My machine with docker using WSL2 ran fine when removing only the "-v " argument and another one running Docker still using Hyper-V engine runs fine with the "-v ".  Hope this helps.

@seannik sorry for not providing this info on the issue, but this is caused by the upstream issue https://bugs.mojang.com/browse/MC-177336

no worries

FYI, the upstream issue is now fixed in 1.16-pre3


Encounter value error

My working environment employs tensorflow 2.0.x
The problem which I encountered is as follows:
ValueError: Input 0 of node import/Pad was passed half from sub:0 incompatible with expected float.
I haven't tried it on tf1, may be it's the version-related problem, but i still wanna use it on tf2.
Could you help me solving it?
Thanks,

Hi, I found this problem was arose by the type float16, atfer re-convert to float32.
Problem solved.
Thanks,


Generate request spec by default when generating scaffolds

Similar problem like in #2056. Upon using rails generate scaffold it would be good if controller tests would not get generated and instead request specs are generated.

Now that #2222 is merged do you fancy a go at this as well @klyonrad ?

Yes, that's why I created this issue 😉

Hey @JonRowe do you happen to have an idea why the Scaffold Generator creates the empty request spec?
I think my previous PR had this effect
See

rails generate scaffold product title:string body:text published:boolean -p

....
      invoke    rspec
      create      spec/controllers/products_controller_spec.rb
      create      spec/views/products/edit.html.erb_spec.rb
      create      spec/views/products/index.html.erb_spec.rb
      create      spec/views/products/new.html.erb_spec.rb
      create      spec/views/products/show.html.erb_spec.rb
      create      spec/routing/products_routing_spec.rb
      invoke      rspec
      create        spec/requests/products_spec.rb

...

I suspect that it caused because the scaffolds generator executes the controller Generator and the Controller Generator has the default option for the request spec from my previous PR

So I investigated, and I'm pretty certain this happens even without your changes...

In fact its hook_for that triggers it

So I investigated, and I'm pretty certain this happens even without your changes...

@JonRowe Are you sure or did I maybe make things confusing? I didn't mean the PR that is linked to this PR, but rather #2222 is the culprit. Should I open a separate issue to avoid confusion?
My theory is that the added class_option :request_specs,    type: :boolean, default: true, desc: "Generate request specs" trickles over while the scaffold generator executes controller generator. This theory comes from the fact that scaffold generator acts weird even when I use the current stable version (3.8.3) and add the --request_specs option through the command line usage, for example rails generate scaffold product title:string body:text published:boolean -p --request_specs.

In fact its hook_for that triggers it

It was not abvious to me why the hook_for was added there. I went into the git history in the file and landed at the historic initial commits where josé valim (❤ ) is mentioned f06aa507280f8f9e6316c9bd5f72e640269a49b1

@JonRowe Are you sure or did I maybe make things confusing? I didn't mean the PR that is linked to this PR, but rather #2222 is the culprit. Should I open a separate issue to avoid confusion?

I'm sure, I reverted #2222, and it still happened. What happens is hook_for is a trigger for activating other generators, and it runs the equivalent of rails generate integration product which is currently implemented as a request spec.

Fixed in #2288


feature-wip support explode and explode_outer table function

Proposed changes
Issue Number: close #7570
Problem Summary:
explode(ArrayColumn) desc:

Create a row for each element in the array column.

explode_outer(ArrayColumn) desc:

Create a row for each element in the array column. Unlike explode, if the array is null or empty, it returns null.

Usage example:

create a table with array column, and insert some data;
open enable_lateral_view and enable_vectorized_engine;

set enable_lateral_view = true;
set enable_vectorized_engine=true;

use explode_outer

> select * from array_test;
+------+------+--------+
| k1   | k2   | k3     |
+------+------+--------+
|    3 | NULL | NULL   |
|    1 |    2 | [1, 2] |
|    2 |    3 | NULL   |
|    4 | NULL | []     |
+------+------+--------+

> select k1,explode_column from array_test LATERAL VIEW explode_outer(k3) TempExplodeView as explode_column;
+------+----------------+
| k1   | explode_column |
+------+----------------+
|    1 |              1 |
|    1 |              2 |
|    2 |           NULL |
|    4 |           NULL |
|    3 |           NULL |
+------+----------------+

explode usage example. explode return empty rows while the ARRAY is null or empty

> select k1,explode_column from array_test LATERAL VIEW explode(k3) TempExplodeView as explode_column;
+------+----------------+
| k1   | explode_column |
+------+----------------+
|    1 |              1 |
|    1 |              2 |
+------+----------------+

Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM

rebase upstream/master to avoid conflict.

LGTM


MM Black Screen with 'Electron' window showing

Have you tried all steps under Troublehsooting?
Yes
Describe the bug
Installed DynamicWeather onto my MM and did an NPM install in the module folder for good measure. Added module config into my config file. Then a restart of the mm.sh caused a blank black screen with the title 'ELECTRON'
Configuration
		{ 			module: 'MMM-DynamicWeather', 			position: 'fullscreen_above', 			config: { // See https://github.com/scottcl88/MMM-DynamicWeather for more information. 				  api_key: "*******************************", 				  locationID: "8224783", 				  effects: [ 				   { 					month: "2", 					day: "14", 					year: "2022", 					images: ["heart1.png", "heart2.png"], 					direction: "up" 				   } 				  ], 				 } 		},
Expected behavior
Expected module to load
Screenshots

Additional context
Raspbian Stretch with MM current version as at today

just do the npm install request , commit the package.json and and push it.. then users won't have this problem

I just pushed a fix for this, it should work now. Thanks all for the help, closing.

also need to update the readme as users need to do the npm install now in the module folder after the git clone


Various fuzzing updates

See commit list (you can probably skip the last one...).

FTR, the GitGuardian report above is a false positive (the cert/key pair is just used for fuzzing and is self-signed).


Fix comment

Codecov Report

Merging #295 (e5ab058) into master (1774a7a) will increase coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #295      +/-   ##
============================================
+ Coverage     63.29%   63.33%   +0.04%
  Complexity      957      957
============================================
  Files           169      168       -1
  Lines          3152     3150       -2
  Branches        325      325
============================================
  Hits           1995     1995
+ Misses         1037     1035       -2
  Partials        120      120

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/seedu/dictionote/commons/core/Config.java
68.18% <ø> (ø)
7.00 <0.00> (ø)

...in/java/seedu/dictionote/commons/core/Version.java
97.50% <ø> (ø)
22.00 <0.00> (ø)

...te/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)
1.00 <0.00> (ø)

...in/java/seedu/dictionote/commons/util/AppUtil.java
88.88% <ø> (ø)
5.00 <0.00> (ø)

.../seedu/dictionote/commons/util/CollectionUtil.java
87.50% <ø> (ø)
5.00 <0.00> (ø)

...n/java/seedu/dictionote/commons/util/FileUtil.java
77.27% <ø> (ø)
9.00 <0.00> (ø)

...n/java/seedu/dictionote/commons/util/JsonUtil.java
94.11% <ø> (ø)
8.00 <0.00> (ø)

...java/seedu/dictionote/commons/util/StringUtil.java
95.65% <ø> (ø)
8.00 <0.00> (ø)

...u/dictionote/logic/commands/AddContentCommand.java
41.66% <ø> (ø)
2.00 <0.00> (ø)

...ictionote/logic/commands/AddDefinitionCommand.java
41.66% <ø> (ø)
2.00 <0.00> (ø)

... and 42 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1774a7a...e5ab058. Read the comment docs.


types: ElementFinder isn't compatible with browser.actions()

This line doesn't compile:
browser.actions().doubleClick(element(by.css('.row_0'))).perform();

Error:
Argument of type 'ElementFinder' is not assignable to parameter of type 'string | WebElement | undefined'.
       Type 'ElementFinder' is not assignable to type 'WebElement'.
         Types of property 'findElement' are incompatible.
           Type '(subLocator: Locator) => Promise<any>' is not assignable to type '(locator: Function | By) => WebElementPromise'.
             Type 'Promise<any>' is not assignable to type 'WebElementPromise'.
               Property 'getDriver' is missing in type 'Promise<any>'.

Seems like the core problem is that ElementFinder isn't a subtype of WebElement (from selenium- webdriver).

browser.actions().doubleClick(element(by.css('.row_0')).getWebElement()).perform();

We are exploring this; however, there is a way to get around it via the existing API

What do you think @juliemr?

For #3726 to fix this, we need to modify jasminewd a bit.

As an alternative, see #3732.

#3732 has been merged


A special character in a file name causes a <FILE_ENTRY> to be omitted from DB.xml without notifying user.

Hello, hope all is well.  I was testing PsFCIV on some real-world examples, My Documents containing 16,535 Files, 1,050 Folders.  All seemed OK at first, "Total files processed" and "Total new added files" both indicated 16,535, and the Verbose output also printed that many lines.
But when I passed DB.xml through a program to count <FILE_ENTRY> tags, 16,534 Files came back.  I traced the issue to one file name containing something that seems like a hyphen but is not.  Attached is a smaller data set containing the problem file name.  As a matter of fact, the github form isn't even displaying the character as I try to paste it, so check the attached .zip
PS C:\test\data_set> Start-PsFCIV -Path .\ -Include * -XML "C:\test\DB.xml" -HashAlgorithm MD5 -Recurse -Verbose
VERBOSE: Perform file 'C:\test\data_set\Barn Owl.txt' checking.
VERBOSE: Perform file 'C:\test\data_set\Barn_Owl.txt' checking.
VERBOSE: Perform file 'C:\test\data_set\Barn_Owl.txt' checking.
Total files processed      : 3
Total new added files      : 3
Total good files           : 0
Total bad files            : 0
Total unknown status files : 0
Total missing files        : 0
Total locked files         : 0
Total   : 3
New     : 3
Ok      : 0
Bad     : 0
Missed  : 0
Locked  : 0
Unknown : 0
Deleted : 0
Notice only Two <FILE_ENTRY>'s are actually written to DB.xml
<FCIV> <FILE_ENTRY> <name>Barn Owl.txt</name> <Size>12</Size> <TimeStamp>01/18/2022 07:49:28</TimeStamp> <MD5>HIkS05fEaGZwazcMovokbQ==</MD5> </FILE_ENTRY> <FILE_ENTRY> <name>Barn_Owl.txt</name> <Size>16</Size> <TimeStamp>01/18/2022 07:49:41</TimeStamp> <MD5>NYX+pTjx8Ee6bzjBC+hyLQ==</MD5> </FILE_ENTRY> </FCIV>
Thanks again for making this program available, it is still very useful even with current issues.
TEST.zip

To speed up the lookup process and avoid potential duplicate entries I'm using a hash set to store entries. This requires that every entry's name is unique within a collection and duplicates are not inserted (although counted in stats as inserted and it is the result of my lazyness).
The actual issue is in how file names are compared. I'm using this one https://docs.microsoft.com/en-us/dotnet/api/system.stringcomparer.invariantcultureignorecase when checking if such file is already processed. In your case, this comparer thinks that last two file names in your attachment are identical, thus second file although has different name, comparison thinks they are identical. You can test it yourself against your data set:
$list = dir .\DATA_SET
[string]::equals($list[1].Name, $list[2].Name,"InvariantCultureIgnoreCase") # will return $true

It seems I have to replace comparison option from invariant culture to ordinal. Though, cultures are hard to deal with and I didn't test such scenarios.

I think I've fixed this and fix works against your data set. Wouldn't you mind to grab this library and test it in your environment: https://github.com/PKISolutions/PsFCIV/tree/master/PsFCIV/lib. Replace the file stored in "lib" folder in module installation directory.

Great!  The change worked for my data set.
test run to create DB.xml
PS C:\temp\TESTING_ZONE\My Documents Backup 01-13-2022> Start-PsFCIV -Path .\ -XML "C:\temp\TESTING_ZONE\DB.xml" -HashAlgorithm MD5 -Recurse -Verbose

Total files processed      : 16535
Total new added files      : 16535
Total good files           : 0
Total bad files            : 0
Total unknown status files : 0
Total missing files        : 0
Total locked files         : 0
Total   : 16535
New     : 16535
Ok      : 0
Bad     : 0
Missed  : 0
Locked  : 0
Unknown : 0
Deleted : 0
Now double checking <FILE_ENTRY> tag count:
C:\temp\TESTING_ZONE>java StringCount "<FILE_ENTRY>" ".\DB.xml"
<FILE_ENTRY> was found 16535 times.
I also checked the name element containing the international character, and it is preserved OK in the DB.xml
Finally another run in checking mode:
PS C:\temp\TESTING_ZONE\My Documents Backup 01-13-2022> Start-PsFCIV -Path .\ -XML "C:\temp\TESTING_ZONE\DB.xml" -HashAlgorithm MD5 -Recurse -Verbose

Total files processed      : 16535
Total good files           : 16535
Total bad files            : 0
Total unknown status files : 0
Total missing files        : 0
Total locked files         : 0
Total   : 16535
New     : 0
Ok      : 16535
Bad     : 0
Missed  : 0
Locked  : 0
Unknown : 0
Deleted : 0


Clicking on any compose button results in exceptions

Clicking on the 'picture' icon results in:
null is not an object (evaluating
'_this.state.noMore')

Clicking on other buttons result in future exceptions of this variety when continuing to use the app:
null is not an object (evaluating
'internalInstance.getHostNode')

Fixed these exceptions.
The options are not yet ready to be used, so also hid them.


Refactoring - unifying service loading mechanism in Main. Always use Module.loadService() instead of ServiceLoader.load()

Thanks for review @dmlloyd !


Add filter option to search for contributor overlap

Overview
Adds a new option to the facility search API and and associated UI that allows optionally filtering facilities by contributor using an "AND" search rather than an "OR" search.
Connects #921
Demo
NOTE the help text was revised after this initial screencast was recorded.

Notes
While adding support for this to the filter_by_query_params method we also replaced the previous filter by contributor code with a statement that computes the "IN" query on the server side. The large SQL statements generated by passing thousands of IDs in a literal "IN" statement were so large that they made analyzing query performance from SQL slow query logs difficult to impossible.
The tooltip text is a placeholder and will be replaced with a submission from Katie.
Testing Instructions

Run ./scripts/resetdb and ./scripts/server
Search for "regina" and select the facility in China. Remember 2 of the three listed contributors.
Reset the search, return to the "Search" tab, and enter the 2 contributors from the previous step in the "Filter by Contributor" box. Verify that the new checkbox only appears after the second item is added to the list.
Run the search and verify that there are multiple facility matches.
Return to the "Search" tab and check the checkbox. Run the search again and verify that only a single facility is now returned.
Refresh the page, verify that the search is restored, return to the "Search" tab and verify that the checkbox is checked.
Remove one of the facilities from the  "Filter by Contributor" box and verify that the checkbox is hidden and the combine_contributors querystring argument is no longer present in the URL.
Browse http://localhost:8081/api/docs/#!/facilities/facilities_list and verify that the combine_contributors argument is documented.

Checklist

[x] fixup! commits have been squashed
[x] CI passes after rebase
[x] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

I force pushed an amended commit that fixes the JS unit tests that failed the CI build

I pushed a fixup commit that adds the full help text I received from @katieashaw

@jwalgran It looks from this screen shot that the text in bold at the top of the document I shared is missing. Could you include it please?

Thank you for catching that, Katie. I added the title in another fixup commit.

Taking a look at this now.

One interesting behavior I'm seeing is that when there's a single match, the sidebar jumps into the detail view, but the map remains in the general view:

This may be previous behavior unrelated to this card though, in which case we can ignore it.

Loading the result when there is a single match is an existing behavior. You can see an example on staging by searching for contributor "Aldi [Public List]" and Country "Portugal"

Chiming in here slightly belatedly having checked a point from this exchange with @nataliegrillon in reference to the point made by @rajadain (copied below). We would prefer for the map view to zoom in these instances. Is that possible @jwalgran ? I know we've had a similar exchange about the zoom function on specific facilities in the past [see issue "The zoom on the OAR map is behaving in an inconsistent manner #757"], so it might be worth revisiting as a group.

One interesting behavior I'm seeing is that when there's a single match, the sidebar jumps into the detail view, but the map remains in the general view:

This may be previous behavior unrelated to this card though, in which case we can ignore it.

We discussed this on a weekly check-in call and agreed that the solution proposed #406 is a good one.


OpenSSH documentation incorrectly categorizes StrictModes as not supported

OpenSSH Server Configuration Documentation incorrectly categorizes StrictModes as a not supported setting.
While testing OpenSSH configurations on a Window Server 2019 machine, I noticed that the StrictMode setting in C:\ProgramData\ssh\sshd_config did have an effect on SSH authentication. When the setting is set to on and I attempt to authenticate with the OpenSSH Server using public key authentication it returns a User Authentication Error, but when the setting is set to off the authentication works successfully.
This indicates that the StrictMode setting does indeed check for local folder permissions on Windows and is in fact supported.

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

ID: c8ad5c9a-efb1-d994-0e3c-53c90096a245
Version Independent ID: 5cb0ea97-4c3d-ffdf-b69c-8a717f9f9f3f
Content: OpenSSH Server Configuration for Windows
Content Source: WindowsServerDocs/administration/OpenSSH/OpenSSH_Server_Configuration.md
Product: windows-server
Technology: administration
GitHub Login: @maertendMSFT
Microsoft Alias: damaerte

Hi @alvinzhengq ,
Thanks for your dedication to our documentation. I have shared your feedback with the team and have confirmed that although StrictModes may work it is not currently supported and remains in the currently published list. You can of course raise an issue in the engineering repository for this functionality.
As this does not require an update to the documentation, we are closing this pull request for now.
Thanks,
Robin
#please-close


update ug

Codecov Report

Merging #174 (64d2b8b) into master (ed74710) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #174   +/-   ##
=========================================
  Coverage     64.92%   64.92%
  Complexity      586      586
=========================================
  Files           101      101
  Lines          1973     1973
  Branches        230      230
=========================================
  Hits           1281     1281
  Misses          609      609
  Partials         83       83

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed74710...64d2b8b. Read the comment docs.


Create UI for adding 979 fields

Staff will use a UI in airflow to upload a file of instance UUID's that they want to add 979 fields to. The upload form should present the user with a list of available bookplate fund metadata to add to records in FOLIO. This is what that form looks like in libsys-webforms:

The upload form will trigger the add 979 field DAG. We have a pattern for this interaction with the boundwiths form. We should also keep track of the user who submitted the form so that they can be emailed when the add 979 field DAG is complete. The email should also go to sul-unicorn-devs list and<EMAIL_ADDRESS>(confirm Tech Services still uses this list).

When there is no fund_name for the bookplate, in the form we previously used, we would use "ZZ_NO_FUND" as the fund name, see screenshot:

We should maybe do something similar. We used "ZZ_" because then those would sort at the bottom. Maybe it doesn't matter with whatever data table we use to present the information. The column for "Fund name" could be blank for these since I think users might search by the title anyways (and not ZZ_NO) in order to find the bookplate for the gift funds, etc.

When the form is submitted, an email of the submission should go to sul-unicorn-devs, sul_bulk_bookplates list and whatever email was entered in the form.


Maybe it'd be better to change the legend in Figure 7 from 'country' to 'country/region'

Hi, thanks for sharing the great work and demo. It is truly amazing.
I noticed Figure 7 in the manuscript. Apparently, there are disputes in some political stuff, and I suggest maybe it'd be better to change the legend and the caption in Figure 7 from 'country' to 'country/region'.

Excellent work should not involve political issues.

can i work on this>

Can't agree more.

I very much agree, I hope developers can pay attention to this issue.

@ericmintun @HannaMao Sorry for bothering, but I think this issue is indeed serious and indisputable. Please pay attention to it. 🙏

I wholeheartedly agree and hope that this error can be corrected.


[Weighted Routing] Allow Search with preference parameter in weighed in nodes

Signed-off-by: Anshu Agarwal<EMAIL_ADDRESS>
Description
This PR adds support to allow searches with preference parameter when strict weighted routing is enabled.
Issues Resolved
https://github.com/opensearch-project/OpenSearch/issues/6297
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Does this effectively revert #5874? Is there a top level issue somewhere the fully describes the behavior of weighted routing that can help me understand this?

Does this effectively revert #5874? Is there a top level issue somewhere the fully describes the behavior of weighted routing that can help me understand this?

No the previous change was strict, this change relaxes the checks to allow the search preference parameter if preference isn't hard(like ONLY_NODES) when those nodes are weighed away.

Might need exhaustive tests for this

@Bukhtawar added tests for all the preference parameter

Codecov Report

Merging #6296 (648f068) into main (49d9a90) will decrease coverage by 0.19%.
The diff coverage is 9.30%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #6296      +/-   ##
============================================
- Coverage     70.78%   70.59%   -0.19%
+ Complexity    59062    58951     -111
============================================
  Files          4800     4800
  Lines        282420   282496      +76
  Branches      40718    40735      +17
============================================
- Hits         199908   199442     -466
- Misses        66126    66658     +532
- Partials      16386    16396      +10

Impacted Files
Coverage Δ

...g/opensearch/cluster/routing/OperationRouting.java
71.17% <0.00%> (-11.59%)
:arrow_down:

...search/cluster/routing/IndexShardRoutingTable.java
78.39% <13.79%> (-10.30%)
:arrow_down:

...search/indices/recovery/RecoveryTargetHandler.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...adonly/AddIndexBlockClusterStateUpdateRequest.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...a/org/opensearch/client/cluster/SniffModeInfo.java
0.00% <0.00%> (-58.83%)
:arrow_down:

...readonly/TransportVerifyShardIndexBlockAction.java
9.75% <0.00%> (-58.54%)
:arrow_down:

...cluster/coordination/PendingClusterStateStats.java
20.00% <0.00%> (-48.00%)
:arrow_down:

...indices/readonly/TransportAddIndexBlockAction.java
20.68% <0.00%> (-37.94%)
:arrow_down:

...cluster/coordination/PublishClusterStateStats.java
33.33% <0.00%> (-37.51%)
:arrow_down:

.../indices/readonly/AddIndexBlockRequestBuilder.java
0.00% <0.00%> (-33.34%)
:arrow_down:

... and 460 more

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


Linker issue: symbol(s) not found for architecture x86_64

Hello!
For specific libraries this error occurs when using xcode-test on Bitrise CLI @Xcode11. It's a hit or miss with different libs. Libs that caused this error are:

Google Maps SDK via CocoaPods
Google Maps SDK via manual install
https://github.com/zxingify/zxingify-objc

Error message:
❌  ld: symbol(s) not found for architecture x86_64
❌  clang: error: linker command failed with exit code 1 (use -v to see invocation)

When building locally everything works as expected.
When connecting remotely to the CI machine, opening xcode from there and building/testing/archiving the version everything works as expected.

The only place where this issue happens is the actual step xcode-test via Bitrise CLI. I've checked Catalina as well as Mojave for Xcode 11. Same issue. When I remove the libs, the issue's instantly gone.
Is this something that you can address? If it can be build locally I see no reason for it to not work on the Bitrise CLI.

Hi @vol24pl!
Thank you so much for the detailed report. Can you please share a build URL and enable Support Access for us so we can investigate this?

Hi :)
Google Maps manually integrated:
https://app.bitrise.io/build/7ec963172e6e3040#?tab=log
Google Maps integrated through the Cocoapods:
https://app.bitrise.io/build/d64336c1e291dac1#?tab=log
Support should be enabled

Any updates?

I'm waiting too.

Hi @vol24pl @bmarcall
Thank you for your patience and I'm sorry for how long this has taken until now.
This has been escalated to our L3 support team for further investigation.
We will follow-up as soon as I've heard back from them.
Kind regards,
Luna

Hi @vol24pl @bmarcall,
I talked to L3 support. Here is what was said: 😸
The only thing they could notice is that you have no configuration specified in his build. More detail:
Xcode works 2 way:

when you start the IDE and running builds/tests within IDE
using command line tool (xcodebuild) to run builds/tests without the IDE itself.

When user uses method 1, then xcode gets the configuration from the scheme only

When the user uses method 2, xcodebuild gets the configuration from the configuration settings

in both cases configuration can be overridden by specifying configuration in (xcode-archive, xcode-test, etc... steps.
They checked the build log and there is no configuration specified. Please try setting configuration in the step or in your project (whatever you prefer), and re-run the build.
The reason I think it might fixes the issue is because project component linking and build parameters can depend on configuration.
Please let us know if this fixes the problem for you.
Kind regards and stay safe,
Luna

As Cocoapods generates the xcworkspace and I need to change by hand:
FROM: "Use Realese for command-line builds"
TO: "Use MyConfig for command-line builds"
Do you know how I can preconfigure that in Podfile? This would help me a lot.
Also can you tell me how can I choose config in xcode-test step?

Im having this error as well. :(

I'm having this issue and i do have Release configuration specified :/

For me the issued seemed to be that my bitrise build step was set to Always latest, which is a legacy deprecated setting. Setting it specifically to 1.x.x resolved this issue.

Hey @kylebrowning thanks for contribuiting, could you provide a llitle bit more of information about where that setting would be? Thanks!

@jeffveleze In the workflow builder, where you choose the Xcode test step. Theres a version you choose on the right hand side.


Implement RelationalAlgebraFrozenSet difference in pandas

Difference of RelationalAlgebraFrozenSet is currently done by relying on python sets' difference, which seems to work fine in most cases, but does not work when using difference on dee sets.
Example
>>> from neurolang.utils.relational_algebra_set.pandas import RelationalAlgebraFrozenSet
>>> dee = RelationalAlgebraFrozenSet.dee()
>>> dee - dee
Empty DataFrame
Columns: []
Index: [0]
>>> len(dee - dee)
1

Expected output should be {}.
This is causing problems later on in the chase algorithm when sets that should be empty are not.
Difference does work as expected for NamedRelationalAlgebraFrozenSet:
>>> from neurolang.utils.relational_algebra_set.pandas import NamedRelationalAlgebraFrozenSet
>>> dee = NamedRelationalAlgebraFrozenSet.dee()
>>> dee - dee
{}

Good catch! Can you make a PR to solve it please?

Yes

I think I went too fast in closing this issue. This can bring bad performance problems if the set difference is needed. As the title is very generic, I think it's best to keep this open until we implement the full pandas version of the operation. However merging quickly PR #608 was good to ensure that when we went from using mutable sets to frozen sets, meaning -= to - the system is still working.


Migrate to Java 11 while keeping Java 8 binary compatibility.

Move sources to Java 11 and keep binaries compatible with Java 8.
Don't create a module yet.

This won't work as compatibility works only one way. Older Jars run with newer JREs. Other way around will not work.
Instead, for the first, a multi release jar which will work with Java 8 and Java 11 is the better option.

Moved to Java11 with 18a7d542b12ea5c842a833fd7de3234c8cd3edd8, not modularized yet.


README: add "Pooling" section

Added a link to phantom-pool to README, hope that's ok. See https://github.com/amir20/phantomjs-node/issues/590

LGTM


[Not beginner friendly] Umbrella issue for debugging issue with Netlify preview builds

⚠ Please note that this is not a beginner-friendly issue ⚠

What is the issue?

The Netlify staging build generation is failing for some pull requests due to timeouts.

What is the error message?

12:31:26 AM: $ git submodule update --init --recursive --depth 1 && make deploy-preview
12:31:26 AM: GOMAXPROCS=1 hugo --cleanDestinationDir --enableGitInfo -buildFuture --environment preview -b https://deploy-preview-43702--kubernetes-io-main-staging.netlify.app
12:31:27 AM: Start building sites
12:31:27 AM: hugo v0.111.3-5d4eb5154e1fed125ca8e9b5a0315c4180dab192+extended linux/amd64 BuildDate=2023-03-12T11:40:50Z VendorInfo=gohugoio
1:03:04 AM: Failed during stage "building site": Command did not finish within the time limit
1:03:04 AM: Execution timed out after 31m40.000962935s
1:03:04 AM: Error running command: Command did not finish within the time limit
1:03:04 AM: Failing build: Failed to build site
1:03:04 AM: Finished processing build request in 32m7.209s
10:32:58 AM: [ERROR] Deploy logs are currently unavailable. We are working on resolving the issue.

Since when has this issue occurred?

This issue has been recurring since Feb 2022.

Are there records for any previous investigation conducted?

#36741 and #38834 were PRs raised by @tengqm for debugging the issue previously.

What is the expected outcome?

Root cause analysis of the issue followed by application of the fix.

What are some must-haves?

Knowledge of Hugo (particularly the docsy theme)
Knowledge of Netlify
Ability to work collaboratively
Should be a Kubernetes GitHub org member

Will you sponsor membership for this work?

No. This requires familiarity with the Kubernetes project and demonstration of consistent contribution.
cc: @natalisucks @reylejano @sftim @tengqm @onlydole @kbhawkey @seokho-son @a-mccarthy

/triage accepted
/priority important-longterm

/sig docs

I’d love to help, if someone can get me up to speed / show me the ropes
i’m not familiar with Netlify, but I can go figure it out
I do however have an org membership and a hugo blog. (not familiar with docsy)

This issue is not beginner friendly, and for that reason a contributor working on it needs to be mostly a self-starter.

I didn’t mean it in terms of skilling up @sftim .
I just meant it  in terms  of having  a running conversation with folk to my arms around it.

Ack. will follow these steps. thank you.

Update as of 2024-01-15:
Have gotten the site to work locally.
and am getting an idea of how api-generation fits in.
Am currently getting my arms around the Docsy theme, and will ask for help in a day or two. (cc: @tengqm, @divya-mohan0209 )

Got the hang of Docsy basics. Ready now to learn and understand how k/website is staged. (with temp access if needed.)
@divya-mohan0209, @tengqm would appreciate further help and guidance.

@jasonbraganza : Thank you so much for reaching out! Since you've gotten the hang of the docsy theme & are a GitHub org member, I think I am comfortable giving you temp privileged access after you've read Netlify documentation regarding staging & rendering of content.
@kubernetes/sig-docs-leads: Please could you advise if you're okay with @jasonbraganza having temp privileged access (i.e. collaborator) after reading through the docs suggested above?

if it helps any with trust, i am also the current New Membership Coordinator, helping new folks become org memebers
and wrt Netlify docs, I’ll give it a run through asap and update here once done.
Thank you, @divya-mohan0209 for patiently guiding (herding? 😂) me.

@divya-mohan0209 I'm fine with temporary collaborator access, thanks for flagging


epo.syncDirectory command

Hi @Kreloc
Have you looked further to add function to synchorinze Domains and AD with the system tree using epo.syncDirectory command?
What I'm interested in about this funtion is to sync OUs let's say is named "Managed".
These OUs belongs to different sites and all of these sites belongs to a root OU called "Sites".
One example how the strucutre looks like:
Sites --> NEWYORK --> Computers --> Managed
Under the site specific OU, there are subOUs called Computers that contains "Legacy", "Managed" and "Other" OUs.
Sites --> NEWYORK --> Computers --> Legacy / Managed / Other
How I sync today:
I first need to sync the root OU called "Sites".
Then in the exclusions, I need to find all OUs called "Legacy" and Other", that will mean I'll only sync "Managed" OUs.
As you can see, this is a manual process and I want to automate this that the AD sync will only look for OUs called "Managed" and only sync them and not adding any exclusions.

Hi @Kreloc ,
I'm just curious, have you start looking into this?

Any updates? :) @Kreloc

Sorry, I ended up putting this module off to the side for a while and haven't come back to it. The  last time I looked into this, that command, epo.SyncDirectory, doesn't not actually change that information and the only way to do it is thru the McAfee epo web console.


unsure if its on my end or the build problem

/usr/bin/ld: warning: CMakeFiles/Osiris.dir/LinuxEndpoints.s.o: missing .note.GNU-stack section implies executable stack
/usr/bin/ld: NOTE: This behaviour is deprecated and will be removed in a future version of the linker
/bin/sh: 1: Syntax error: "(" unexpected
gmake[2]: *** [Source/CMakeFiles/Osiris.dir/build.make:105: Source/libOsiris.so] Error 2
gmake[2]: *** Deleting file 'Source/libOsiris.so'
gmake[1]: *** [CMakeFiles/Makefile2:98: Source/CMakeFiles/Osiris.dir/all] Error 2
gmake: *** [Makefile:91: all] Error 2

Do you have strip installed? What is the output of which strip command?


OBJ-279 change links on webpage

Add change links to the check you answers web page.

Updated PR.

Are you referring to the CHANGE_ANSWER_KEY flag?


Discord.js: When ever I turn my computer to sleep it, it crashes.

When my computer goes to sleep, it crashes with the log of
throw err; // Unhandled 'error' event
^
Error [ERR_UNHANDLED_ERROR]: Unhandled error. ([object Object])
at CommandoClient.emit (events.js:171:17)
at WebSocketConnection.onError (C:\Users\Charps\Mythical Bot\node_modules\discord.js\src\client\websocket\WebSocketConnection.js:374:17)
at WebSocket.onError (C:\Users\Charps\Mythical Bot\node_modules\ws\lib\event-target.js:128:16)
at WebSocket.emit (events.js:182:13)
at _receiver.cleanup (C:\Users\Charps\Mythical Bot\node_modules\ws\lib\websocket.js:211:14)
at Receiver.cleanup (C:\Users\Charps\Mythical Bot\node_modules\ws\lib\receiver.js:535:15)
at WebSocket.finalize (C:\Users\Charps\Mythical Bot\node_modules\ws\lib\websocket.js:206:20)
at TLSSocket.emit (events.js:182:13)
at emitErrorNT (internal/streams/destroy.js:82:8)
at emitErrorAndCloseNT (internal/streams/destroy.js:50:3)
Can you please help?

The bot will not work when your computer goes asleep. You should prevent your pc from autosleeping.


PowerShell command working in basic use case

I ran out of brain juice on the NuGet package stuff... But that shouldn't be too hard. On a weekday ;)

I read your blog post, looks pretty sweet!

Who - yeah. BTW - do we need all those dlls? The NuGet package is not 9.3 MB which feels quite heavy.
I especially thought of the IKVM.*.dll? I didn't dare remove anything since I didn't know what's needed and used.
Do you?

I would have to go take a peek through the other assemblies to see what is needed specifically.  I know there are a lot of the IKVM assemblies but this is because I prefer the NuGet package over storing binaries.  I want to use ILMerge (see https://github.com/x97mdr/pickles/issues/20) to package single assemblies for all of the runners and this should help reduce the package size a lot.

oh ... in your blog post you talk about using a .nuspec file ... where is it at?

There - new pull request with small changes added


Add isRecord() and isNotRecord() Class assertions

Check List:

Fixes NA
Unit tests : YES
Javadoc with a code example (on API only) : YES
PR meets the contributing guidelines

Add isRecord() and isNotRecord() Class assertions.
As record only landed in Java 16 I'm having to use reflection to check whether the class is a record or not. I'm also only able to add unit tests for the negative cases, because I can't compile code that uses record.
If this is accepted I thought I'd also open a PR that adds a hasRecordComponents assertion.

Thanks @ljrmorgan, this one is better targeted to assertj 4.0 which will use java 17

A Java 17 baseline would also remove all the reflection code, which would be great. Is there a 4.x branch or anything that I could rebase onto?

Not yet but as I mentioned I still think we could ship this in 3.x and change the implementation in 4.x.
@joel-costigliola how do you see it?

I can live with that

Thanks for your first contribution, @ljrmorgan!


Error: LINE 64

At line 64 its url not source

Which file are you referencing?

upload-photo-user-timeline.html
Line62


Following font is jagged.

Probably rendering bad on a non-retina screen

Same for Store, Home, and Followers.

What resolution are you using?
I tried my monitor at 1440 x 900 and 800 x 600, but didn't have any jaggedness.
What OS is this on? You may not have font smoothing activated.

3440 x 1440 ubuntu.

That would be considered retina resolution, so it's not a resolution issue.
Do you have font smoothing turned on?

I'm labeling this as a question until we can determine if it's an app issue or a systems settings issue.

I'm still thinking about this one. Haven't come up with a better solution yet, but I agree that it can be improved.


Text Component not showing when use numberOfLines and text dir changed

Hello
Text Component not showing when i use numberOfLines prop for it and text content direction changed
if i have below code and change "TextContent" instantly by change app language (i18next) and if the direction of the language be different, Text Not Shown
different between direction like : from english (ltr) to arabic (rtl) or Conversely
also this issue not appear by re-render or new render and after re-render (or new render) it's work fine. it just happened when in current screen, the text content direction changed.
<Text numberOfLines={1} ellipsizeMode={'tail'} style={{color: '#ffffff'}}>TextContent</Text>
React Native version:
react: 16.13.1 => 16.13.1
react-native: 0.63.2 => 0.63.2

I'm seeing other issues logged against numberOfLines. Related to #30437?

I'm seeing other issues logged against numberOfLines. Related to #30437?


add example hash string using bcrypt algorithm

add example to create hash string in bcrypt
also add example to compare whether the hashed string is one of plain string hash version

👍


unsupported error on open driver

Hi,
I'm trying to learn automation in go, I'm a relatively newbie at programming so might be missing something obvious.
I'm trying to run the "generic_driver" example against a vMX device I have in GNS3. The device is reachable from my Windows PC, I'm using go version 1.21 and VScode as my IDE.
When I run the example (after updating the hostname) all I get is;
failed to open driver; error: unsupported
I'm not sure if I've not installed something I needed to? The code compiles and runs ok so I thought I'd installed everything I needed to but it might not be the case.
Thanks

Use standard transport type here or don’t use windows. Scrapli wraps /bin/ssh with the default transport.
Please use discussions or ask on discord for non bug/feature related things.


Don't name the output of the const block

Don't name the output of the const block in which traits are implemented. This doesn't play well with Rust RFC 3373, and isn't necessary for correctness.
Fixes #46

@yaahc can you take a look at this? As noted in https://github.com/yaahc/displaydoc/issues/46#issuecomment-1964170940 recent nightly Rust versions are seeing a bunch of warnings from code generated by displaydoc.

@yaahc can we get some feedback on this issue?


There is an extra '/' before the blob name when upload one blob to a folder which name is the whitespace

Storage Explorer Version: 1.22.0-dev
Build Number: 20211013.3
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04
Architecture: ia32\x64
How Found: Ad-hoc testing
Regression From: Previous release(1.21.2)
Steps to Reproduce

Expand one non-ADLS Gen2 storage account -> Blob Containers.
Create a blob container -> Click 'New Folder' -> Type a whitespace as the folder name -> Click 'OK'.
Upload one blob to the folder.
Check there is no extra '/' before the blob name.

Expected Experience
There is no extra '/' before the blob name.

Actual Experience
There is an extra '/' before the blob name.

Additional Context
Here is the screenshot for the two activities logs:

@JasonYeMSFT
This issue reproduces on Windows & Linux.
This issue doesn't reproduce on Mac.

@v-xianya The activity log message is tweaked to better reflect the destination directory so this is by design.


Next release?

Sorry to bother you with an issue, but I was wondering if there were plans for a new release? It seems that the master branch has quite a few new things while the latest release e.g. doesn't support macOS yet :)

There's barely any actual support for macOS :) but sure. Released 0.1.3.

Awesome, thank you :)


"Evaluation of this constant expression throws an exception" should say what the exception is

When the analyzer says "Evaluation of this constant expression throws an exception" it can be really hard to figure out what the problem is without actually running the code and looking at the exception.

@Hixie this is a Dup of 36526, I think...

Yup.
Duplicate of #36526


Fix crasher and banner size on ipad

Fixes https://github.com/artsy/eigen/issues/1921 & https://github.com/artsy/eigen/issues/1912

          1 Warning

:warning:
The following files do not have tests: lib/components/home/artist_rails/artist_card.js, lib/components/home/artwork_rails/artwork_rail_header.js & lib/components/home/hero_units.js

  Generated by :no_entry_sign: danger


BCM94352z issue

Hi
First, Thank you for your efi.
Based on your efi.
I choose the same motherboard as you.
I used BCM94352z for my wifi/bt but not works.
May I know which wifi/bt card you used?
Thank you again.

I haven't replaced my card at the moment but this is something I will do in future, if you have Telegram there are a lot of helpful people in there which may be able to help you out 👍

I haven't replaced my card at the moment but this is something I will do in future, if you have Telegram there are a lot of helpful people in there which may be able to help you out 👍


Add missing fields in Account.

See https://github.com/tootsuite/documentation/blob/master/Using-the-API/API.md#account
Also fixes a spurious pagination-related error.

Coverage decreased (-0.2%) to 92.632% when pulling b83071c1d6fe6df91044d4da16680f0f335b4c90 on BenLubar-PR:master into 2ccbcfe14d7ae9129fa84fbf37d030b046abbfdf on mattn:master.

Thank you


Allow Shoot to be created when Seed .spec.backup=nil

How to categorize this PR?

/area quality
/kind bug
/priority critical
What this PR does / why we need it:
Allow Shoot to be created when Seed .spec.backup=nil.
Which issue(s) this PR fixes:
Fixes #2366
Special notes for your reviewer:
Release note:

NONE

/assign @swilen-iwanow @vpnachev

@swilen-iwanow Only code owners are allowed to change labels or milestones. Authors may only add certain white-listed labels as part of the description such as kind/*, priority/*, area/*, component/*, platform/*.

/approve
/lgtm


RAM usage scales when more GPUs are used

I notice that when my training process uses more GPUs, the RAM usage also increases, almost linearly w.r.t. the gpu number. Specifically, the 2-gpu training uses around 85GB and the 4-gpu training uses around 170GB, my dataset is 82GB and I am doing in_memory=True. Is this behavior expected?
One difference I find between how my training process and the ffcv-imagenet way of doing the training is that I am starting my process through something like python -m torch.distributed.launch --nproc_per_node=4 to start my script, while the ffcv-imagenet training script does something like ch.multiprocessing.spawn(cls._exec_wrapper, nprocs=world_size, join=True) within the script while the script is run just like a normal python script, is this the reason?

Hello @chengxuz .
This is a known bug that has been fixed. It will be part of the next release but you can already use it by using the branch v0.0.4

Hi @GuillaumeLeclerc ,
All my tests were actually performed in branch v0.0.4 (as I used that to fix the GPU memory bug). So the fix does not seem to work for me.

Thanks for your answer. Are you using our imagenet example or did you write your own? Two things:

Try with v0.0.4 and our imagenet example, that's what I tested
If it works there and not in yours my suspicion is that you are missing ch.cuda.set_device in your code. FFCV uses the current device for the pipeline regardless of where you copy the tensor as you might want to prepare the data with a GPU and use another for training.

Keep us updated!

There must be some misunderstanding here. I think you were talking about an earlier bug I reported in here.
But here I am talking about the computer memory (RAM, am I using the wrong term?) not GPU memory. I don't understand why you mention the cuda set_device here, it's in my mind not related to the RAM usage at all. It should be related to how you allocate the memory to get the dataset in (the in_memory choice).

Oh sorry. I Indeed misunderstood. I don't know what you mean by in memory but the behavior is expected with os_cache=False. Indeed each process (one per GPU caches the data it needs).
If using os_cache=False. Make sure you use quasi random ordering as otherwise you will have a copy of the dataset in each process memory.
if you use os_cache=True. Do you divide your batch size when you use multiple GPUs. The memory used by FFCV is proportional to the amount of data being prepared. If you use two GPU with a batch size of 128 it will use twice the amount of ram than a single GPU with batch size=128

Closing due to inactivity


Dropdown breaks with Chrome's built-in translator / Google Translate API

Bug Report
When using Chrome's built-in feature or the Google Translate API to translate forms everything works fine except Dropdowns. The option selected by the user doesn't match what is shown in the dropdown field.
Steps

Add the Google Translate script to a form (easier to reproduce, otherwise you'll have to write the form page in another language)
Select a different language in the Google Translate dropdown (ex: Spanish)
Select an option from the Dropdown field

Expected Result
The text shown in the dropdown field should be the one just selected (or at least the original text without translation corresponding to the option selected)
Actual Result
The input value changes to the correct one, but the text shown in  doesn't change.
Version
2.1.3
Testcase
https://codesandbox.io/s/dropwdown-google-translate-q9pghu?file=/example.js

As a sidenote, Multiple Selections work fine, which is interesting.


chore: remove conflicting eslint indent rule and whitespce in README.md

What does this PR do
Address conflicting eslint and prettier indent rules.
Removing indent rules within .eslintrc in favour of prettier.

prettier format automatically address indenting issue following the .editorconfig specifications.

Description of Task to be completed
How should this be manually tested
Break an indentation and run npm run fix
What are the relevant issues this PR belongs to
#
Any background context you want to add
Screenshots

@BolajiAyodeji hopefully fixes very existing issue.

Great work chief!


Unable to filter multiple name?

filter n/Bernice n/Alex don't work
filter n/Bernice Alex works.
So which one.. See user guide on filter command example :)

Thanks. The changes is not reflected in the latest .jar file, but it is already been addressed.


Allow to pull charts from OCI registry

Would someone write some pointers for how to implement this?

I just tried to install an OCI chart and it works just fine if you put the full URI (oci://...) into the spec.chart field of a HelmChart CR. (No need to specify thespec.repo)

I just tried to install an OCI chart and it works just fine if you put the full URI (oci://...) into the spec.chart field of a HelmChart CR. (No need to specify thespec.repo)

I think this works only for registries using HTTPS. When using k3d managed registries, these are HTTP, so the install errors out with failed to do request: Head "https://k3d-registry.localhost:5000/v2/mychart/manifests/0.1.0": http: server gave HTTP response to HTTPS client for me

I ran into this issue when trying to leverage a private OCI registry. It seems that public OCI registries work by excluding the repo field and fully-qualifying the chart. However, private registries do not and the authSecret support added here only works when repo is provided, which doesn't work with oci:// registries. When installing locally, I login using docker login and then helm pull oci:// works fine as it seems to inherit the credentials. Given that it is fairly common to create a docker registry secret in the cluster so that kubelet can pull the images from the oci:// registry, I found that if you can mount that same registry secret in /home/klipper-helm/docker then it works fine. I would be glad to push a PR if there is interest but honestly not all that familiar with go, so it might take a minute.

@smbecker hmm, so there is no convention for a repo index when using an OCI registry as the chart store?

@smbecker hmm, so there is no facility for hosting a repo index when using an OCI registry as the chart store?

Helm only supports a subset of commands when targeting OCI registries. You can explicitly login but it has been my experience that when using OCI-based registries for Helm charts, then the containers are likely on the same registry as well. In that case, it is easier to just use the inherited docker authentication to login for Helm as well.


Mime type of yanked content?

Hi :wave:
First of all, thanks for this plugin. I have a weird issue which causes yanked content to show up on clipboard history but not able to paste it (xclip reports its targets are empty).
Related ticket on greenclip side: https://github.com/erebe/greenclip/issues/116
Am I doing something wrong, or is there a tunable for the plugin? :eyes:

Hello!
The plugin is pretty simple: when you select some text and call :OSCYank, the only thing the plugin does is to encode the text in base64 and emit the result with the OSC52 sequence code in front of it (it basically does echo <OSC52;your-text-in-base64>
Then it's your terminal that does the processing: when it sees any string with OSC52 in front of it, it decodes the string that follows and put it in your system clipboard. That's why the plugin work through SSH. So from the point of view of the plugin, there's no MIME type to handle or any other stuff.
So I think it's an issue with your terminal. What terminal are you using?

Thanks for the explanation, It really sounds simple, I'll close this one. I'm using wezterm and currently walking between projects to find out the missing link :)


Weapons with a double quote in the name will only use one of two "usages" regardless of which is pressed.

Part of the round/ammo description for shotguns in the length of cartridge (2.5", 2.75" or 3"). This is listed at the end of the weapon name in HighTech. As such, the weapon listings in GCS (and I believe GCA) include that in the name. However, when I created attack lines for both a slug buckshot, only the buckshot was used for making attacks. The damage amounts, damage types, rate of fire and recoil are different for the two different uses. Attaching a sample character sheet. Changing the double quote character to two single quote character preserves how the weapon name looks, but allows the two different usages to be triggered appropriately. The second version of the attached sheet has been modified in this fashion
nels.zip
.

Fixed with: https://github.com/crnormand/gurps/pull/1380


AWS SecretManager Metadata secretPushFormat: string is ignored

Describe the bug
The secretPushFormat: string in the PushSecret specification is being ignored, and the secret is still being stored in AWS Secrets Manager in binary format. This issue occurs even though the secret is explicitly configured to be pushed as a string.
To Reproduce
Steps to reproduce the behavior:
We have simple secret in key:value format
KEYCLOAK_CLIENT_SECRET: xyz
apiVersion: external-secrets.io/v1beta1
kind: SecretStore
metadata:
  name: streams-secrets-store
  namespace: platform-keycloak
spec:
  provider:
    aws:
      service: SecretsManager
      region: eu-west-1
      auth:
        jwt:
          serviceAccountRef:
            name: streams-dge-external-secrets-sa
---
apiVersion: external-secrets.io/v1alpha1
kind: PushSecret
metadata:
  name: dge-argonath-keycloak-push-secret
  namespace: platform-keycloak
spec:
  secretStoreRefs:
    - name: streams-secrets-store
      kind: SecretStore
  selector:
    secret:
      name: dge-argonath-keycloak-secrets
  data:
    - match:
        secretKey: KEYCLOAK_CLIENT_SECRET
        remoteRef:
          remoteKey: streams/dge/keycloak/credentialss
      metadata:
        secretPushFormat: string

provide the Kubernetes and ESO version
Kubernetes version:  v1.29.7-eks-2f46c53
(ESO) version: 0.9.12

Expected behavior
The expected result is that the secret should be stored in plaintext (key-value format) in AWS Secrets Manager.
Screenshots

Ah, I think this has been updated, but poorly communication. :/ The way the metadata has to be provided now is like an object.
apiVersion: external-secrets.io/v1alpha1
kind: PushSecret
metadata:
  name: pushsecret-example # Customisable
  namespace: default # Same of the SecretStores
spec:
  deletionPolicy: Delete # the provider' secret will be deleted if the PushSecret is deleted
  refreshInterval: 10s # Refresh interval for which push secret will reconcile
  secretStoreRefs: # A list of secret stores to push secrets to
    - name: aws-parameterstore
      kind: SecretStore
  selector:
    secret:
      name: pokedex-credentials # Source Kubernetes secret to be pushed
  data:
    - match:
        remoteRef:
          remoteKey: my-first-parameter # Remote reference (where the secret is going to be pushed)
      metadata:
        parameterStoreType: "SecureString"

Or rather String in your case I guess.

Unfortunately It did not help for both "String" and "string"
apiVersion: external-secrets.io/v1alpha1
kind: PushSecret
metadata:
  name: dge-argonath-keycloak-push-secret
  namespace: platform-keycloak
spec:
  secretStoreRefs:
    - name: streams-secrets-store
      kind: SecretStore
  selector:
    secret:
      name: dge-argonath-keycloak-secrets
  data:
    - match:
        secretKey: KEYCLOAK_CLIENT_SECRET
        remoteRef:
          remoteKey: streams/dge/keycloak/credentials
      metadata:
        secretPushFormat: "string"

The key is parameterStoreType now.

But this is for AWS ParameterStore pushSecret not for AWS SecretsManager. According to the doc https://external-secrets.io/latest/provider/aws-secrets-manager/#additional-metadata-for-pushsecret it should be secretPushFormat so just as it is yaml file which I provided.

Ahhh so sorry. Let me take a look.

Okay, yeah, so these are the keys and values:
// Declares metadata information for pushing secrets to AWS Secret Store.
const (
	SecretPushFormatKey    = "secretPushFormat"
	SecretPushFormatString = "string"
	SecretPushFormatBinary = "binary"
)

Let's see what's going on there/

Hmm, probably its because current ESO version installed on our clusters is 0.9.12 and looks like that SecretPushFormat paramater has not been added there yet. Looking into docs, section related to that params has been added in 0.9.14 version.

Ahh, that's a good thought checking that. :D

Can confirm that upgrading ESO to latest version (0.10.3) helped.

Phew, good. :D Thanks very much for confirming.


Legends are overlapping with x axis lables in small devices.

Need to check if this is a problem with the angular library or a problem with the c3js library. Have an easy way to check it? Maybe with one of the examples on the c3js site?


Test alternate greenness indices

The camera was down for the early part of 2020, but otherwise it seems like the understory count of # of pixels with gcc > 0.4 doesn't really lead the overstory gcc_90 in the deciduous broadleaf (DB) region of interest.  Will test different thresholds next between 0.3 and 0.4.

Here is a comparison of different gcc thresholds. 0.35 and 0.36 look especially promising for detecting the lead between understory and canopy. Should I to other NEON forest sites to confirm?


edtr.io: do not display empty text plugins

Empty text plugins are displayed as "empty space". IMHO this is not a way to use for spacing.

Moved to backlog document


Bottle for<EMAIL_ADDRESS>uses old version of libavif

I installed<EMAIL_ADDRESS>which resulted in a bottle install of PHP and the libavif dependency:
==> Installing<EMAIL_ADDRESS>dependency: libavif
[...]
==> Pouring libavif--1.0.1.arm64_ventura.bottle.1.tar.gz
🍺  /opt/homebrew/Cellar/libavif/1.0.1: 19 files, 536KB
==> Installing<EMAIL_ADDRESS>==> Pouring<EMAIL_ADDRESS>
During the post-install steps, errors already appeared:
dyld[33162]: Library not loaded: /opt/homebrew/opt/libavif/lib/libavif.15.dylib

Apart from post-install steps the installation was successful, but when running PHP the following error occurs:
$ php --version
dyld[35306]: Library not loaded: /opt/homebrew/opt/libavif/lib/libavif.15.dylib
  Referenced from: <87C1A268-34E4-396F-8BBC-D5591064E333> /opt/homebrew/Cellar/gd/2.3.3_5/lib/libgd.3.dylib
  Reason: tried: '/opt/homebrew/opt/libavif/lib/libavif.15.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/opt/homebrew/opt/libavif/lib/libavif.15.dylib' (no such file), '/opt/homebrew/opt/libavif/lib/libavif.15.dylib' (no such file), '/usr/local/lib/libavif.15.dylib' (no such file), '/usr/lib/libavif.15.dylib' (no such file, not in dyld cache), '/opt/homebrew/Cellar/libavif/1.0.1/lib/libavif.15.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/opt/homebrew/Cellar/libavif/1.0.1/lib/libavif.15.dylib' (no such file), '/opt/homebrew/Cellar/libavif/1.0.1/lib/libavif.15.dylib' (no such file), '/usr/local/lib/libavif.15.dylib' (no such file), '/usr/lib/libavif.15.dylib' (no such file, not in dyld cache)
zsh: abort      php --version

A quick look reveals that libavif 1.0.1 is installed (as a dependency of PHP 8.3), but the library is libavif.16.dylib, while the bottle seems to have been built against libavif.15.dylib:
$ ls -l /opt/homebrew/opt/libavif/lib
total 296
drwxr-xr-x   7 alcaeus  admin     224 Sep 14 09:41 .
drwxr-xr-x  11 alcaeus  admin     352 Sep 14 09:41 ..
drwxr-xr-x   3 alcaeus  admin      96 Aug 29 23:58 cmake
-rw-r--r--   1 alcaeus  admin  149504 Sep 14 09:41 libavif.16.0.1.dylib
lrwxr-xr-x   1 alcaeus  admin      20 Aug 29 23:58 libavif.16.dylib -> libavif.16.0.1.dylib
lrwxr-xr-x   1 alcaeus  admin      16 Aug 29 23:58 libavif.dylib -> libavif.16.dylib
drwxr-xr-x   3 alcaeus  admin      96 Sep 14 09:41 pkgconfig

After reinstalling from source, everything works as expected:
$ brew reinstall -s<EMAIL_ADDRESS>[...]
==> Summary
🍺<EMAIL_ADDRESS>517 files, 98MB, built in 5 minutes 35 seconds

$ php --version
PHP 8.3.0-dev (cli) (built: Sep 14 2023 07:51:56) (NTS DEBUG)
Copyright (c) The PHP Group
Zend Engine v4.3.0-dev, Copyright (c) Zend Technologies
    with Zend OPcache v8.3.0-dev, Copyright (c), by Zend Technologies

Additional context
I noted that the bottle downloaded for PHP 8.3 was one for Monterey (and potentially newer): Pouring<EMAIL_ADDRESS>In my case, I'm running this on MacOS Ventura 13.5.2. Curiously enough, a second Mac running MacOS Sonoma 14.0 doesn't have this issue, despite also installing from the same bottle:
$ brew reinstall<EMAIL_ADDRESS>Warning: You are using macOS 14.
We do not provide support for this pre-release version.
[...]
==> Pouring<EMAIL_ADDRESS>[...]
==> Summary
🍺<EMAIL_ADDRESS>523 files, 99.4MB

$ php --version
PHP 8.3.0-dev (cli) (built: Sep 13 2023 00:22:56) (NTS DEBUG)
Copyright (c) The PHP Group
Zend Engine v4.3.0-dev, Copyright (c) Zend Technologies
    with Zend OPcache v8.3.0-dev, Copyright (c), by Zend Technologies

@alcaeus
Try upgrading gd
brew upgrade gd


fixes calculations and dashboard for Golang

Fixes aggregations of results when file has no classes.

Codecov Report
Attention: Patch coverage is 50.00000% with 19 lines in your changes missing coverage. Please review.

Project coverage is 53.49%. Comparing base (ad9e9a5) to head (a8fc80b).

Files with missing lines
Patch %
Lines

src/Analyzer/Aggregator.go
58.06%
12 Missing and 1 partial :warning:

src/Engine/Golang/GolangRunner.go
0.00%
2 Missing :warning:

src/Engine/Php/PhpRunner.go
0.00%
2 Missing :warning:

src/Engine/Python/PythonRunner.go
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
+ Coverage   53.25%   53.49%   +0.23%
==========================================
  Files          42       42
  Lines        4792     4827      +35
==========================================
+ Hits         2552     2582      +30
- Misses       2009     2016       +7
+ Partials      231      229       -2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Only modifier keys get registered on MacOS

When running this code on mac (with both accessibility and input monitoring enabled for terminal and Vscode, where I am running the program):
    let hook = Hook::new().unwrap();
    hook.register(
        Hotkey {
            key_code: KeyCode::KeyP,
            modifiers: Modifiers::empty(),
        },
        || println!("pressed key"),
    )
    .unwrap();

No keys get registered. I found that putting Modifier keys in the key_code variable does work, and I can combine e.g. RightAlt + Modifiers::META. Any non-modifier key does not get registered.

I'm assuming this is #659. Can you try with the master branch?

Downloaded the livesplit-hotkey crate from master and replaced the dependency in my code, and now no events get sent at all. Tried with regular keys, as well as modifier keys, no events seem to go through.

Updated to MacOS 14.3.1 and it started working. Thanks for help.


Mpc prototype

MPC code using pytorch. Only works having code on both machines. Currently supports addition, subtraction, negation, multiplication, matrix multiplication and an approximation of sigmoid.
Should be refactored into pysyft asap.
demo currently in lib folder (probably needs a refactor)
has method stubs for use with grid

Started resolving conflicts earlier, got distracted by other work then got lost and broke things 🎉
Will take a more detailed look later tonight or tomorrow!

Also, does this nullify #178?


increase coverage enforcement from 50% to 80%

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 88.784%

Files with Coverage Reduction
New Missed Lines
%

qiskit_research/utils/pulse_scaling.py
3
82.64%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
2454

Relevant Lines:
2764

💛  - Coveralls


Improve transaction signing experience for "allow" mode

Not having to pass all post conditions would be useful for devs who want to test, and don't quite care about post conditions just yet. They should be able to sign transactions in 'allow' mode somehow, but for users it should be forced to 'deny' mode. Maybe an advanced toggle somewhere.

Not sure I follow. The user wants to swap x for some amount of y (plus or minus some slippage). Aren't those your post conditions?

unless there is some metadata available, swapr does not know what the token names to put conditions on are.  The only thing it knows about the tokens are their contract principal (.e.g ST31HHVBKYCYQQJ5AQ25ZHA6W2A548ZADDQ6S16GP.plaid-token-v0a, ST31HHVBKYCYQQJ5AQ25ZHA6W2A548ZADDQ6S16GP.stx-token-v0a, and ST31HHVBKYCYQQJ5AQ25ZHA6W2A548ZADDQ6S16GP.tokensoft-token-v0a, so ft-token is plaid in the first case, not an ft-token in the second case, and tokensoft-token in the third case).  And going the static analysis route is NOT a rabbit hole I want to get into...
so, the plan is that if some metadata is available, swapr will use that, or use Allow if it is not available.  Yet an other addition to sip-010, or some addendum to write

@jasperjansz would you be able to work on a design for displaying this information in the wallet? (what mode the current transaction is in, and an option to switch)

See related discussion with my suggestions here for "allow" mode:

Clearly indicate this mode with some sort of warning saying "There are no guarantees this contract will do anything it's promising!". With post conditions, this could go in place of the "...or the transaction will abort" guidance in "deny" mode, or as a standalone message if no post conditions are passed.
Soften the framing of post conditions. E.g. instead of "You'll send" for each condition, "You may send".

https://github.com/blockstack/stacks-wallet-web/discussions/1359

Let's tackle the problem here with "allow" mode and no conditions provided better as well per https://github.com/blockstack/stacks-wallet-web/issues/1120

Here's our latest proposal for contract calls in allow mode: https://www.figma.com/file/ebj00CNeRhipdzBgxPmgnS/🔎-Transactions?node-id=15%3A521
Currently, the web wallet does not differentiate between deny and allow mode. While we'll still enable allow mode, this will change the design of allow mode in the following ways:

In allow mode, we'll show a warning to the user that any of their tokens could be transferred if they confirm
In allow mode, even tokens not covered by any post-conditions can be transferred, voiding the post-conditions' security guarantees. That's why we choose not to display any post-conditions that were passed.
In allow mode, the primary call to action is to go back (which will close the indow), though users could still confirm and make the contract call.

Deny mode (left) vs. Allow mode (right)

Moving this to be bottom of the dev backlog, but please chime in if you have feedback!

I would just note that this change is urgent given that we currently do show post conditions while allow mode is enabled, which could lead users into believing incorrectly that the transaction is protected by deny mode. Currently the developer can claim any number of false things about the transfers involved.


Pan gesture handle onchange callback only support on version 2.2.0+

The onchange callback cannot be found on version 2.1.0
Need to update the package.json to require react-native-gesture-handler at v2.2.0+

btw this is a great project!

Done in 45bd181ba77daef7881b94d881aa1e888c37d973

@alexandrius  Not a bug, and kind of unrelated, but why not support<EMAIL_ADDRESS>If I force dependencies will this be an issue?


Add X-Stream header

When streaming is true.

At present - stream=true is passed, but the documentation states an X-Stream header with true needs to be sent.


Use a real datastore for ProfileCardDataStore in tests.

It helps us conduct more high-fidelity tests.
https://github.com/DroidKaigi/conference-app-2024/pull/458/files#r1719188027

I would like to work on this issue. ✋

Thanks! Assgined


DataAttributes + Origins = Cursed stats

Having more recent versions of Data Attributes, and using Origins addons like Extra Origins causes these origins to have a positive stat instead of negative without any use of DataAttributes for this individual mod/addon.
ie; Pick Inchling origin, you are supposed to have reduced hearts by 6, instead, you gain 6 hearts. Same with attack, range, etc etc. Everything that Extra Origins is reducing in its datapack, Data  Attributes is turning to positive.

I'll look into this; thank you for bringing it to my attention.

Hi there,
I've done some testing, and I actually can't replicate this. Using the mods listed below, I've selected Inchling origin and I have my hearts reduced by 6 like it's supposed to.

Mod
Version

Fabric API
0.46.1+1.18

Data Attributes
1.1.5

Origins
1.3.1

Pehkui
3.1.0

Extra Origins
1.18.7

Mod Menu
3.0.0

Could you provide a list of the mods/datapacks that you've used when this has happened? Otherwise I'll be forced to close this issue.
Thanks.

It some how fixed itself between updates at least for my modpack AQM2. Another person discovered data attributes was the cause for their pack. I'll link the issue later, it's on the extra origins repo


error with SHT75

I am trying to read temperature and humidity from SHT75 sensor and I have installed sht-sensor library
I connected the sensor as below
VDD-> 3.3
GND-> PI GND
SCK-> GPIO 21
DATA-> GPIO 17
my raspberry run with raspbian and I wrote the following code
from sht_sensor import Sht
sht = Sht(21, 17)
print 'Temperature', sht.read_t()
print 'Relative Humidity', sht.read_rh()
I got the following error
Temperature
Traceback (most recent call last):
File "sht_sens.py", line 3, in
print 'Temperature', sht.read_t()
File "/usr/local/lib/python2.7/dist-packages/sht_sensor/sensor.py", line 257, in read_t
t_raw = self._get_meas_result(self.cmd.t)
File "/usr/local/lib/python2.7/dist-packages/sht_sensor/sensor.py", line 197, in _get_meas_result
self._send(cmd)
File "/usr/local/lib/python2.7/dist-packages/sht_sensor/sensor.py", line 157, in _send
raise ShtCommFailure('Command ACK failed on step-1')
sht_sensor.sensor.ShtCommFailure: Command ACK failed on step-1
so kindly advice what I have to do to solve this problem

I'd suggest checking whether sensor is connected to specified pins and/or working at all.
Basically, command gets sent on a wire and at the very first step where there should be response (acknowledgement) from the sensor, there is none.
This would happen if specified pins would be not connected to anything for example, but might also be some issue with the sensor if everything is connected correctly and it just doesn't respond, which is usually less likely.
I'd suggest paying close attention to pin numbers btw, as 21 and 17 in your example seem like an odd choice, given that they're located at the opposite sides of the header, maybe you want 17 and 27, which are next to each other?
Or guess maybe you've picked these numbers from the readme, where I think I've tested the thing on Beagleboard, which has these ones close-by, unlike RPi.
Note that there should be a pull-up resistor connected with this sensor on the data pin, and this module doesn't tinker with internal RPi resistors (which can only be enabled via RPi-specific registers).
I think Pi-Sht1x module (see Links section in the readme), which uses RPi.GPIO (and /dev/mem interface) might actually set internal pull-up resistor, so if that is desirable, might want to use that one instead.
I'll close the issue as I don't think there's any indication of anything going wrong in the code, but feel free to ask/post stuff here anyway, might be able to answer.

Also note that Raspberry Pi 1 doesn't even have GPIO 21, but I guess you mean RPi2 or RPi3 (newer boards) above.


Document byKeyValue in changelog.

I haven't yet checked, does the generated changelog look ok? (Personally I always mess up closing the parens in the wrong place before I get it right (: )

That's where vim's jump-to-matching-parenthesis feature comes in very handy. :-)
You can see a preview here: http://eusebeia.dyndns.org/~hsteoh/tmp/web/changelog


About consolidation of HttpRequest method.

Hello, I have one suggestion. HttpRequest.scala in net package has two method that get and post.But written content same mostly. In there, I suggest that method name send in method param. In addition,If you want to POST request only then,  setRequestMethod("POST").
image
if(mehod.equal("POST")){
    urlconnection.setRequestMethod("POST");
}

Thank you for sending the issue.
I understood your Suggestion.
But I don't understood any asset that I can get with this.
I think that the nature of GET and POST is different in HTTP.
Is it a problem to integrate things with different properties? (show other HTTP method)
Please tell me the clear asset by this. (Other than shortening the code)
P.S.
The sample you presented is wrong.
At first "mehod" -> "method", "equal" -> "equals". It's a simple misspelling.
Next, It isn't a big mistake.
You can also use setRequestMethod when using the GET method.
Therefore, it can be modified as follows
//if( method.equals("POST") )
    urlconnection.setRequestMethod(method); //method expected GET or POST

Thank you for replying me. I understood why you divide method. ;)


generic functions

Should Flow report an error for the last line below because the arguments are not the same type?
// @flow
function printPair<T>(p1: T, p2: T): void {
  console.log(`${String(p1)} and ${String(p2)}`);
}
printPair('one', 2);

No, it should. Flow infers that T is string | number

How do I tell it that I want both arguments to be the same type?

You can't. I understand what you mean, but it's even hard to describe it formally. In your example arguments ARE of the same type, technically.

imho something like this should bind T to string, but flow is complaining:
function printPair<T>(p1: T, p2: T): void {
  console.log(`${String(p1)} and ${String(p2)}`);
}
type PairPrinter = typeof printPair;

(printPair: PairPrinter<string>)('one', 2);

https://flow.org/try/#0GYVwdgxgLglg9mABABwE4zFACgQxqgHgBUA+ACmQEYAuRIgGhQCZaiBKWgNzhgBNEA3gChEiCAgDOcADYBTAHTS4AczIADACQCAylHRhVVNgF9EOMPy279hpibVsA3EONCoAT2SzEufFn1QsqiIALyIHl5wwCgBvqjOQhSxeKi0cf4YgYQSehjKJGxkAOQIskWMdo5AA

Polymorphic functions and generic function types are not this same:
type A = <T>(T) => void; // polymorphic function
type B<T> = (T) => void; // generic type

oh, you're right 😺 , then this will do the trick:
function printPair<T>(p1: T, p2: T): void {
  console.log(`${String(p1)} and ${String(p2)}`);
}
type PairPrinter<T> = (T, T) => void;

(printPair: PairPrinter<string>)('one', 2);

6: (printPair: PairPrinter<string>)('one', 2);
                                           ^ number. This type is incompatible with the expected param type of
6: (printPair: PairPrinter<string>)('one', 2);
                           ^ string

https://flow.org/try/#0GYVwdgxgLglg9mABABwE4zFACgQxqgHgBUA+ACmQEYAuRIgGhQCZaiBKWgNzhgBNEA3gChEiCAgDOcADYBTAHTS4AczIADACQCAylHRhVVNgF9EOMPy279hpibVsA3EONCoAT2SzEufFn1QsoSkiAC8iGQMdGxhJIjcfM5CFAG+qLRp-hiBhBJ6GMokbGQA5AiyJYx2jkA