Dynamic balancing broken

It looks like the dynamic balancing is broken.
I can reproduce the error obtained with my huge test case with a smaller test case from the example suite (rev. a8e0877eb4c4):
mus/examples/fluid/benchmark/Channel3D/C3D_Sphere_MultiLevel_LES/D3Q27/CUM17/
I reproduced the error and investigated a bit.
During my investigations I reached a point where I wanted to make sure that the problem is kernel independent.
So I switched to an old and proven one (bgk with d3q19) and got the same problem.
Besides, I reduced the problem size by setting nDiameter = 2 in seeder.lua.
The files are attached: setupFiles.tar.gz
In the following, I briefly summarize what I found out so far with that setup.
The first error I got was:
At line 1753 of file
musubi/build/mus/source/mus_construction_module.f90
Fortran runtime error: Attempting to allocate already allocated variable 'bclevelpointer'

Apparently there is no proper clean-up in the (re-) construction of the mesh, and we run into an attempted re-allocation in  the subroutine build_bcLevelPointer.
A first attempt to fix it with
 if (.not. allocated(bclevelpointer)) then
      allocate(bcLevelPointer(bc_prop%property%nElems))
end if

led to another error. At that point, I switched to debug mode:
At line 1761 of file musubi/build/debug/mus/source/mus_construction_module.f90
Fortran runtime error: Index '10621' of dimension 1 of array 'bclevelpointer' above upper bound of 10620

Deallocation and a new allocation solves that problem.
But then I got another error written in the log file:
Stream and collide
 ERROR! >>> |rho_pre - rho_post| =<PHONE_NUMBER>0000000
 Level =           10

  !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
                Aborting.
  !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

and stemming from
line 379 of file musubi/build/debug/mus/source/compute/mus_compute_d3q19_module.f90
This check is only performed in debug mode.
Besides these things, I also tested different things:

Running it in serial:

The first, simple attempt with  if (.not. allocated(bclevelpointer)) then works and
Gives the same, final error as written above (directly)

No problem at all if:

I deactivate the balancing
I run it with a single level mesh

It seems like something got messed up for multi-level meshes and activated dynamic balancing.
In a next step, I'll look at the error  coming from line 379 of file musubi/build/debug/mus/source/compute/mus_compute_d3q19_module.f90.

Thanks @geriJana when you create a branch for this, could you please also add the inputs to reproduce the failure in there?

Thanks @geriJana when you create a branch for this, could you please also add the inputs to reproduce the failure in there?

I edited the description above and added the files already there.

Update after checking the kernel (mus_advRel_kFluid_rBGK_vStd_lD3Q19) in musubi/build/debug/mus/source/compute/mus_compute_d3q19_module.f90 with respect to
ERROR! >>> |rho_pre - rho_post| =<PHONE_NUMBER>0000000
Before the load balancing, the values are fine.
After load balancing, the error occurs.

omega does not seem to be affected

instate

for some directions extremely high values like<PHONE_NUMBER>8.614670

rho

comes from auxField
extremely high value (e.g. -4.06E+017)

rho_pre

comes from instate
sometimes takes extremely high values like -4.06E+017 (see rho above)

outstate

calculated using instate and rho

rho_post

comes from outstate
sometimes NaN

With the dependencies shown on the already incorrect instate, it is no wonder that the above error occurs (after load balancing).
Next steps include:

[ ] Checking instate to find and fix the bug
[ ] Checking auxField to find and fix the bug

Sounds as if the fields are not properly initialized again.


Profile the compiler for whatever is making it slow right now.

In the past few weeks, there's been a noticeable regression in compile perf.

Closing due to this issue not being directly actionable. We have already removed a significant amount of namespace clones -- perf will be a continual thing we work towards.


Jump too simple

use "z -v", show:
""""
24         /usr/bin
50         /Users/fish/git
56         /Users/fish/work/src/Qingyi/admin
90         /Users/fish/work/src/Qingyi/is
232        /Users/fish/Github/wxgo
""""
when use "z is", I want jump to "/Users/fish/work/src/Qingyi/is" dir, but jump to "/Users/fish/Github/wxgo".
I think just "fish" contain "is".

Cd into /Users/fish/work/src/Qingyi/is directly more often.


Displayed ObjectStore digest is truncated

Observed behavior
The digest reported when adding a file to an object store, or displaying a file, is truncated. It is missing the last four characters.
% nats obj put test testfile
Object information for test > testfile

               Size: 16 KiB
  Modification Time: 08 Mar 24 17:56 +0000
             Chunks: 1
             Digest: SHA-256 3f4f46a69d15e5a371f792f1d9cee802c63f437f98e8962540de81ce1e36

% shasum -a 256 testfile
3f4f46a69d15e5a371f792f1d9cee802c63f437f98e8962540de81ce1e36de77  testfile

Expected behavior
The displayed digest should exactly match that calculated by a tool such as shasum.
Server and client version
Server: 2.10.9
Client: 0.1.3
Host environment
Mac OS 14.2.1.
Steps to reproduce
See observed behavior.

Hey thanks @ebusto I noticed a bunch of other issues with that code so fixed it all in https://github.com/nats-io/natscli/pull/1017 and then realised I also fixed what your PR was fixing by accident.  Sorry, thank you for the contribution in ficxing that but alas already fixed!


Update Composer dependencies (2022-06-08-00-12)

Visual regression test passed!
View the visual regression test report


Googlemap plugins are not working. Please send me latest command line to install googlemap plugin to Ionic 2

Prerequisites

[ ] Did you check the FAQ?
[ ] Did you search open and closed issues to see if your issue has already been reported?

Description
[Description of the bug or feature]
Steps to Reproduce

[First Step]
[Second Step]
[and so on...]

Expected behavior: [What did you expect to happen?]
Actual behavior: [What actually happened?]
Versions
[From typings -v]

This is error
<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed

You haven't filled in any reproduction or description of the issue. See your duplicate issue: https://github.com/typings/typings/issues/814#issuecomment-319431528. This doesn't look related to Typings.


tileseq_mut dies with AttributeError: 'Namespace' object has no attribute 'logger_time'

Adrine came across this error when trying to run tileseqMut. It occurred after the alignment jobs were submitted.
Traceback (most recent call last):
  File "/home/rothlab/adesouza/anaconda3/envs/py37/bin/tileseq_mut", line 62, in <module>
    main(args, v)
  File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 626, in main
    mc.main()
  File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 370, in main
    sam_dir = self._align_sh_()
  File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 201, in _align_sh_
    finished = cluster.parse_jobs_galen(job_list, self._args.logger_time, self._logging.getLogger("track.jobs"))
AttributeError: 'Namespace' object has no attribute 'logger_time'

input parameters:
0.6.940
fastq,/home/rothlab/adesouza/projects/2022vhlassay/fastq
output,/home/rothlab/adesouza/projects/2022vhlassay/
param,/home/rothlab/adesouza/projects/2022vhlassay/VHL_ODD2GFP_postassay22.csv
name,assayODD2postselection
skip_alignment,False
r1,None
r2,None
log_level,info
environment,GALEN
at,8
mt,36
mm,15
c,8
base,33
test,False
rc,False
sr_Override,False
posteriorQC,False
wt_override,False
calibratePhredWT,False
calibratePhredPhix,False
resubmit,False

When trying to run the variant-calling manually afterwards (using skip alignments) a similar error occurred:
 _______ __         ___
/_ __(_) /__ ___ ___ ___ _/ _ \_______
 / / / / / -_|_-</ -_) _ `/ ___/ __/ _ \
/_/ /_/_/\__/___/\__/\_, /_/ /_/ \___/
           /_/    v1.2.2
2023-01-05_16:15:01 INFO: Running tileseqMave version 1.2.2
2023-01-05_16:15:01 WARNING: Tile 1 includes start codon!!
2023-01-05_16:15:01 WARNING: Region 1 includes start codon!!
2023-01-05_16:15:01 INFO: Writing output to /home/rothlab/adesouza/projects/2022vhlassay/VHL_ODD2GFP_postassay22.json
2023-01-05_16:15:01 INFO: Conversion successful!
TileSeqMut release 0.6.940
parameter sheet in csv format, converting to json
2023-01-05 16:15:01,816 - main.log: INFO Sam files are read from /home/rothlab/adesouza/projects/2022vhlassay/assayODD2postselection_2023-01-05-14-23-01/assayODD2postselection_2023-01-05-16-15-00_mut_count
2023-01-05 16:15:01,816 - main.log: INFO Mutation count sh files are made in /home/rothlab/adesouza/projects/2022vhlassay/assayODD2postselection_2023-01-05-14-23-01/assayODD2postselection_2023-01-05-16-15-00_mut_count/GALEN_jobs
2023-01-05 16:15:01,821 - main.log: INFO Sample wtR1tile1
2023-01-05 16:15:01,821 - main.log: INFO Read1: /home/rothlab/adesouza/projects/2022vhlassay/assayODD2postselection_2023-01-05-14-23-01/sam_files/wtR1tile1_S10_R1_001.sam
2023-01-05 16:15:01,821 - main.log: INFO Read2: /home/rothlab/adesouza/projects/2022vhlassay/assayODD2postselection_2023-01-05-14-23-01/sam_files/wtR1tile1_S10_R2_001.sam
Traceback (most recent call last):
 File "/home/rothlab/adesouza/anaconda3/envs/py37/bin/tileseq_mut", line 62, in <module>
  main(args, v)
 File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 626, in main
  mc.main()
 File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 460, in main
  finished = self._makejobs(sh_output, sam_dir, self._sample_names)
 File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 294, in _makejobs
  job_id = self._submit_mutcount_jobs(sample, sh_output)
 File "/home/rothlab/adesouza/.local/lib/python3.7/site-packages/TileSeqMut/main.py", line 315, in _submit_mutcount_jobs
  cmd = f"tileseq_mut -n {self._args.name} -r1 {self._r1} -r2 {self._r2} -o {self._output} -p" \
AttributeError: 'Namespace' object has no attribute 'logger_time'

Fixed. Was caused by two conflicting versions of tileseqMut being installed simultaneously.


Energy drain while logged out

Describe the bug
Power from Energy storage seems to drain even when logged out and the Refined storage controller is connected. The chunk they are in is NOT chunk loaded while the player is logged out (OPAC is used to chunk load but only while the player is logged in, the forceload ticket is removed on logout). It doesn't appear to happen when i disconnect the controller. The generators to make the RF obviously aren't active when the chunk isn't loaded/active, and they don't show energy drain while logged off. Their buffers are full. But on logon, the longer i was logged out, the more empty the Energy Storage tanks are, then fuel has to be wasted in order to refill them. The RS controller does not use power when not ticking.
To Reproduce
Connect generator to Flux point
Connect flux plug to RS controller
Add Flux energy storage and let it fill.
leave the chunk allowing it to unload or log off (this is a client > server setup), when logging back in, the energy in the tank is reduced.
disconnect/disable the RS controller and log off, the energy stored will be the same when logging back in.
Expected behavior
Power levels to stay the same during log off without drain.

https://drive.google.com/file/d/1PrewU_stk0sXJos12G8XHtGa73b8Q_-n/view?usp=drive_link

I also tested this in a chunk that's never been chunk loaded before. and after leaving the area, it still shows an energy drain. i set it to 1RF/t for testing.

I'm going to try and do some testing with no other mods and see what happens.


Image not shown when trying to share your created art to Twitter

If you try to share your art to Tweet from the link, you only get the text and not the created art like this:

This problem is because image is stored as buffer in the database and it is not possible to add image from buffer to tweet. Also, this tweet text must have the link of your artwork, so I'm gonna open another issue for that. Thank you so much!


Added MarketRaker circulating supply to MinSwap

Adding code to show MarketRaker coin circulating supply

please update index.ts

Updated index.ts accordingly appreciate it. :)

run npm run format to pass formatting checks
the output is { circulating: '10614253999994.799', total: '10614254000000' }, which is ~10 trillion+ circulating but only ~10m total supply.

Reviewed and done tests, thanks for the guidance.


Platform Output Support

Platform Output Support (POS) is the third component in the back-end data and infrastructure generation pipeline. POS is an automatic and unified place to perform a release of OT Platform to the public. The other two components are Platform Input Support (PIS) and the ETL. POS will be responsible for:

infrastructure task:

Fill in ElasticSearch
Fill in ClickHouse
Create the images

publishing datasets in different services

Copy the input into the release bucket. Eg. from ot-snapshot to data-releases-prod
Generate BigQuery
Copy the output into the release bucket.
Copy the release into FTP (rclone vs gsutil sync)
Script for the association dumps

technical chores:

combined datasets metadata
site maps
move old release data to cold storage

data validation:

check the number of ES documents should match the number of BQ entries.

Potential user interface for command line program:
resources = elasticsearch | clickhouse | bigquery
storage = ftp | bucket
environment = dev | prod

>> pos create [resources]

>> pos validate [resources]

>> pos clean # remove all

>> pos metadata site-maps # used for google crawler on front end

>> pos copy <storage> <storage>

>> pos deploy <env>

Overview of workflow (Background)

PIS writes to
ETL performs data manipulations
POS generates infrastructure (via Terraform), moves data to where it needs to be (ftp, BigQuery, etc), releases/promotes candidates.

List of things that might be involved with POS:
*) Copy the input into the release bucket. Eg. from ot-snapshot to data-releases-prod
*) Copy the release into FTP

Possible technologies:

Terraform
SBT
Google Cloud Functions
Python / Scala CLI
Travis
Ammonite scripts

Combination of Terraform for infrastructure components (images) and SBT project.

combined datasets metadata is a file with all JSONLines from JSON and parquet from ETL into a new one. This file is required for the frontend web app to generate the download page. @mbdebian @cmalangone a chat with FE to know a bit more about all that needs to be bundled or not
gsutil -m cat gs://open-targets-data-releases/21.04/output/etl/json/metadata/**/part\*

File currently requires both json and parquet:
gsutil cat 'gs://open-targets-data-releases/21.04/output/etl/*/metadata/**/*.json'

combined datasets metadata is a file with all JSONLines from JSON and parquet from ETL into a new one

@cmalangone can we close this? Seems the work is done here?


role "zabbix" does not exist

Hi tiredofit,
I think I've found a bug with the most recent postgres images.
Summary
Postgres zabbix agent is reporting that it cannot connect to the postgres DB due to a non-existent role "zabbix".
Connection failed: failed to connect to `host=localhost user=zabbix database=user=zbx_monitor`: server error (FATAL: role "zabbix" does not exist (SQLSTATE 28000)).

Replication

Set up stock zabbix stack here with needed setup
Import postgres zabbix template.
Start up a stock postgres:15 image using example compose. (just changed network external: true)

The issue is not present in the following image when tested with the same steps: tiredofit/postgres:15-3.1.1
Potential Causes

The most recent commit changes how the zabbix agent is installed. It looks like the "make" command was melded with the "mkdir" command here
I looked at the monitoring set up, but I wasn't able to see any recent changes or issues here.

Please let me know if I can perform any additional testing or provide any more info.
Screenshots
Click to expand

It looks like you fixed it before I could submit. Thanks

Not sure if this is an issue anymore? Is there not a Docker image built?

The last postgres docker image appears to be 21 days ago. I deleted the old db directory to refresh, re-pulled 15-3.2.2. The issue is still present in my test environment.

That's unfortunately when the commit was done - Jun 19..
Will try to trigger a rebuild..

Both have been rebuilt: latest 15-3.2.2

Thank you for triggering a rebuild. It's still not working for me. I'm doing a side-by-side comparison of the working vs non-working.
I'm still looking, but I found a Zabbix agent version mis-match in the 3.2.2:


Bug: OpenApi Plugin missing some boxed types

Should I file a issue?

Great work on this and all of the other PRs and issues @28Smiles !
It looks like java.lang.Double::class.java was already included, did you add it by mistake?
I will look at the rest of the PRs and issues tomorrow, but I think @TobiasWalle and/or @sealedtx should give their opnion, as they are way more familiar with the OpenAPI plugin than I am.

Thanks.
Yes I did this by mistake, fixed.

Hi @tipsy, @28Smiles. I am not sure, but it looks like we should also add those missed types here?
https://github.com/tipsy/javalin/blob/b2319def3256bfaa8858fb011a56225b303bd11f/src/main/java/io/javalin/plugin/openapi/dsl/DocumentedContent.kt#L112-L131

sounds right

I also added double, since it was missing completely

Great, thank you @28Smiles !

Thank you @28Smiles !

This has been released as part of 3.8.0 now: https://javalin.io/news/2020/03/27/javalin-3.8.0-released.html !


Reflect up to 200 fields by default in c++17 implementation via struc…

…tured bindings

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
407

Relevant Lines:
407

💛  - Coveralls


Update Issue - 66720750

default description
URL tested: http://ci.bsstag.com/welcome
Open URL on Browserstack

Property
Value

Browser
Chrome 38.0

Operating System
Windows XP

Resolution
1024x612

Screenshot Attached
Screenshot URL

Click here to reproduce the issue on Browserstack

Updated Screenshot Attached


[Bug] Class Feature Expertise/Proficiencies ALL checked after import or re-import

After importing a character, clicking the Features tab, then the edit button on the class name, then the details tab, there is a button to choose class proficiencies that has EVERY proficiency checked. It should either check the ones the player has selected in dndbeyond or else leave them all empty.

Resolved in v0.2.10


Opening a *.scopesession with CSV data results in partial display

Opening the attached *.scopesession results in the presentation below. I'm not 100% certain if the decode is cached from before, or if it's genuine. Note, there appears to be (almost exactly?) 128ns of waveform at the left, and "a pixel or two" at the right.
The only way I've found to rectify this, is to re-open each CSV file, as depicted below.
data.zip

Possibly related, I've also just managed to get multi-channel CSV working (it doesn't like more than on header row at all...)
It seems that some samples before t=0 are missing.

Investigating. Your screenshots certainly don't seem right.
The render looks fine for me, though. What OS are you using?

This is what I get opening your file (after fixing paths, import filters use absolute paths now which have to be fixed when the session is moved to another computer).

It looks fine with no issues.
As far as decode output being cached, we don't cache any filter outputs in files. Only the configuration and the input waveforms are saved. So if the decode output is right, the data is correct internally and this is purely a display bug.
Also, what GPU and video driver version are you running? Is this native or in a VM?

What OS are you using?

Windows 10, 64-bit (22H2, build 19045.5011)

what GPU and video driver version are you running?

Nvidia RTX3080, v<IP_ADDRESS>94

native or in a VM?

Running native, built in a VM (also Win 10, but I don't imagine that's a factor).

import filters use absolute paths

Oops, I'll sanitize the archive and re-supply (I'll edit the first post).
I'll also place the files locally instead of on a network share, and see if that makes a difference.

There's no real change when opening from local disk, but a different channel is fully rendered...

I've attached the original CSV (but modified to only have a single header row), this has data back to -1.2μs.
scope_20241111_165051.csv

Hmm, weeeird. Nothing is jumping out at me as an obvious cause.
If you close and reopen a bunch of times, do you get the same or different results? This smells like a race condition to some extent (trying to render data that's not fully loaded or something) but I'm not yet sure how that would happen from looking at the code.
Any other Windows users, maybe @bvernoux or @fredzo , want to look at this test case and see if you can reproduce? Also would be very interested if anyone can make it happen on non-Windows or not.

This bit in extended debug logs is interesting:
[CSVImportFilter::OnFileNameChanged]         Initial parsing completed, 9997 lines, 4 columns, 4 names, 9999 timestamps
[ImportFilter::TryNormalizeTimebase]         Min sample interval:     1000 ps
[ImportFilter::TryNormalizeTimebase]         Average sample interval: 879.763 ps
[ImportFilter::TryNormalizeTimebase]         Max sample interval:     18446.74407 s
[ImportFilter::TryNormalizeTimebase]         Stdev of intervals:      12.01120 ns

It thinks there are two samples 18000 seconds apart. If timestamps are getting corrupted somehow, that would absolutely explain the bad rendering since the shaders include a lot of optimizations assuming that sample timestamps are strictly monotonic (they don't have to be uniform, but sample X+1 must have a timestamp strictly greater than that of sample X).

Ok so it looks like we have a bug where sample 0 in the CSV import filter has a timestamp of 0, then sample 1 has a valid timestamp.
This results in sample 0 having a negative duration, then a bunch of non-monotonic timestamps, then it recovers once the timebase hits zero. Still working on root-causing and fixing it, but this explains at least some of your problem.

This should fix at least part of the issue. Can you update to 06e7130fdef8e40f1ce69e98332057feab05b559 (make sure to recursively pull submodules) and see if things work better now?

Not sure if you expected it to, but that appears to have resolved both issues! (data before t=0 and multi-channel open on Windows)
Many thanks for such a quick fix!

... to follow up, the missing data before t=0 still occurs if there are multiple header rows... not sure if that's perhaps something you'd like to detect / handle / notify the user on.

I'll file a separate issue for better handling of this in the CSV import filter.


Fix icon in copy button

Description & motivation 💭
Removes unnecessary stroke attribute from Icon on copy button in the FilterOrCopyButtons Holocene component.
Screenshots (if applicable) 📸

Before
After

Design Considerations 🎨
Testing 🧪
How was this tested 👻

[x] Manual testing
[ ] E2E tests added
[ ] Unit tests added

Steps for others to test: 🚶🏽♂🚶🏽♀
Checklists
Draft Checklist
Merge Checklist
Issue(s) closed
Docs
Any docs updates needed?

Does this work for dark mode?

@Alex-Tideman I don't think it's supported in dark mode yet, but looks like @KATIETOLER was working on that here.


fixed array check

causes problems on GWT usage

Sorry, but this is valid JavaScript. If a third-party tool cannot support valid JavaScript, the tool is broken.

"1" instanceof String also valid JavaScript but returns false

So you're saying roots instanceof Array returns false if roots is an Array?

yes, https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/instanceof

I'm reading the "Not an instanceof" section at the bottom.
To test if an object is not an instanceof a specific constructor, you can do
if (!(mycar instanceof Car)) {
// Do something, like:
// mycar = new Car(mycar)
}
Doesn't that agree with our original code?

I call execute from native method
@com.mxgraph.gwt.client.util.WrapperUtils::unwrap(Lcom/mxgraph/gwt/client/IJavaScriptWrapper;)(this).execute(parentJS, rootsJS)
if i check rootsJS instanceof Array in console on this line it returns true, but in your library roots instanceof Array already false and no any steps between.
from link:
instanceof and multiple context (e.g. frames or windows)
Different scopes have different execution environments. This means that they have different built-ins (different global object, different constructors, etc.). This may result in unexpected results. For instance, [] instanceof window.frames[0].Array will return false, because Array.prototype !== window.frames[0].Array and arrays inherit from the former.
This may not make sense at first, but for scripts dealing with multiple frames or windows, and passing objects from one context to another via functions, this will be a valid and strong issue. For instance, you can securely check if a given object is, in fact, an Array using Array.isArray(myObj)

The code is valid JavaScript and performs correctly in native JavaScript. The bug is in GWT, you haven't proved otherwise.

isArray is also an ES5 method, our baseline is ES3. We can't change this until we move to ES5.

it's a pity (
you can try:

It's not related to GWT, but it's also not the code we're using. It's all very well building an example that fails, but show me how our line of code fails in a JavaScript, then at least we can class it as a bug.

em..
this line shows it:
alert(window.frames[0].frameElement.contentWindow.a instanceof Array);
here is false
if send roots which i received from other frame your check will fail
for ES3 you can check it like
Object.prototype.toString.call(roots) == '[object Array]'

But why would you reasonably run the layout in another iFrame? This is really bad practice at a coding level, I just don't see why anyone would write such code in a normal situation.

ajax can be done with iframes, GWT uses iframes(maybe they don't know about good practice :) )
not sure what is problem to check it in more right way
but ok - no means no

Well, if our baseline were ES5, it's not a big change. But the argument seems to be IE 6-8 vs GWT support. Given we've always supported IE 6-8, it doesn't seem fair to drop that suddenly for GWT.
I'm afraid you're the only person I've heard mention GWT for about 7 years, I really thought the project was dead.


Use pretty log encoding

Q
A

Bug fix?
no

New feature?
no

API breaks?
no

Deprecations?
no

Related tickets
-

License
Apache 2.0

What's in this PR?

The currently used log encoding is not really suitable for easy reading. I suggest to configure the encoding object provided by zap.
Existing logs:
{"level":"info","ts":1603284010.713752,"logger":"controller-runtime.metrics","msg":"metrics server is starting to listen","addr":":8080"}
{"level":"info","ts":1603284010.714328,"logger":"controller-runtime.webhook","msg":"registering webhook","path":"/validate"}
{"level":"info","ts":1603284010.714356,"logger":"setup","msg":"starting manager"}
{"level":"info","ts":1603284010.714595,"logger":"controller-runtime.manager","msg":"starting metrics server","path":"/metrics"}

New logs:
2020-10-21T14:16:38.154+0200	INFO	controller-runtime.metrics	metrics server is starting to listen	{"addr": ":8080"}
2020-10-21T14:16:38.155+0200	INFO	controller-runtime.webhook	registering webhook	{"path": "/validate"}
2020-10-21T14:16:38.155+0200	INFO	setup	starting manager
2020-10-21T14:16:38.155+0200	INFO	controller-runtime.manager	starting metrics server	{"path": "/metrics"}

Why?

It helps everyone to understand the logs faster.
Checklist

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

While this log is more human readable, there is a reason it's only used in the development configuration of zap. Writing the logs in json format creates a more digestible format with clearly defined fields for logging ingestion, e.g. LogStash/ElasticSearch. Also the development config will disable log sampling as well.
The least I would do is make this configurable by the user with the default being the existing production config.
You can also find more info here: https://github.com/uber-go/zap/blob/master/config.go

While this log is more human readable, there is a reason it's only used in the development configuration of zap. Writing the logs in json format creates a more digestible format with clearly defined fields for logging ingestion, e.g. LogStash/ElasticSearch. Also the development config will disable log sampling as well.
The least I would do is make this configurable by the user with the default being the existing production config.
You can also find more info here: https://github.com/uber-go/zap/blob/master/config.go

I see. Thanks for the comment @hi-im-aren
I kept the existing configuration and used the command line options to configure the log. I also deprecated the -verbose option as we have direct control over the logging object.
Current options:
...
  -verbose
    	(deprecated) Enable verbose logging
  -zap-devel
    	Development Mode defaults(encoder=consoleEncoder,logLevel=Debug,stackTraceLevel=Warn). Production Mode defaults(encoder=jsonEncoder,logLevel=Info,stackTraceLevel=Error)
  -zap-encoder value
    	Zap log encoding ('json' or 'console')
  -zap-log-level value
    	Zap Level to configure the verbosity of logging. Can be one of 'debug', 'info', 'error', or any integer value > 0 which corresponds to custom debug levels of increasing verbosity
  -zap-stacktrace-level value
    	Zap Level at and above which stacktraces are captured (one of 'info', 'error').

What do you think about the new implementation?

Force pushed new implementation.
I use two flags (--verbose and --development) for configuring the logging. The --verbose is an existing config.
I set the log level based on --verbose (DEBUG), and build the logger based on the --development flag. The logger object is build as the zap would have built it with one exception: the timestamp format is set to the ISO one.
For testing purposes I added a debug log entry right after configuring the logger (not included in the PR). The logs looks like this with different options specified:
$ go run ./main.go
{"level":"info","ts":"2020-10-22T12:02:35.051+0200","logger":"controller-runtime.metrics","msg":"metrics server is starting to listen","addr":":8080"}
{"level":"info","ts":"2020-10-22T12:02:35.052+0200","logger":"controller-runtime.webhook","msg":"registering webhook","path":"/validate"}
...
$ go run ./main.go --development
2020-10-22T12:02:44.075+0200	INFO	controller-runtime.metrics	metrics server is starting to listen	{"addr": ":8080"}
2020-10-22T12:02:44.076+0200	INFO	controller-runtime.webhook	registering webhook	{"path": "/validate"}
...
$ go run ./main.go --development --verbose
2020-10-22T12:02:52.335+0200	DEBUG	setup	test debug log
2020-10-22T12:02:52.593+0200	INFO	controller-runtime.metrics	metrics server is starting to listen	{"addr": ":8080"}
2020-10-22T12:02:52.593+0200	INFO	controller-runtime.webhook	registering webhook	{"path": "/validate"}
...
$ go run ./main.go --verbose
{"level":"debug","ts":"2020-10-22T12:03:01.624+0200","logger":"setup","msg":"test debug log"}
{"level":"info","ts":"2020-10-22T12:03:01.883+0200","logger":"controller-runtime.metrics","msg":"metrics server is starting to listen","addr":":8080"}
{"level":"info","ts":"2020-10-22T12:03:01.883+0200","logger":"controller-runtime.webhook","msg":"registering webhook","path":"/validate"}

Let me know what you think.

Add default value for the new flag to https://github.com/banzaicloud/kafka-operator/blob/master/charts/kafka-operator/values.yaml
Also bind the new flag to the operator deployment: https://github.com/banzaicloud/kafka-operator/blob/master/charts/kafka-operator/templates/operator-deployment.yaml#L80

Sorry I missed that. Fixed in latest commit.


move event types to separate class

fixes #341

Coverage decreased (-0.1%) to 78.81% when pulling f55a33085f5681d2f46709725c6666b8b462b92d on event-class-341 into fdb875ed9e18651bb171c03de26a97242f38c9f6 on master.


update standard and use its fix flag

This can help us avoid linting gotchas like the one in #90

Looks like this may be a bug in standard: https://github.com/standard/standard/issues/1063

Thanks for following up, @feross. Giving 11 a try.


Option to switch location of summary and method/path

One key principle in information design is that you want to tell someone what something does before telling them how to do it. For example, "To add a customer, send this request." People approach documentation with their current task in mind, such as "I want to add a customer." Only someone who is already very familiar with an API can go to the documentation already knowing what method and path they are looking for.
Swagger UI just lists methods and paths, grouped by tags, but there isn't a single word of explanation anywhere until you click on something. Not good information design.
RapiDoc does better. The "path" view includes tag descriptions and method summaries. However, the summaries are off on the right, so the layout is "how before what," not "what before how."
Would it be possible to have an attribute that would cause the method summaries to appear on the left and the method/path on the right? Or maybe not show the method and path at all until the user clicks the summary. I know that must sound like heresy to a programmer, but it's actually common and respected in the world of API documentation. For example, Stripe documentation, PayPal API reference, Microsoft Azure API Reference.
Sorry this was so long -- just trying to describe and justify what I'm asking for.

path summaries and descriptions are optional in a spec, plus they can be written down in a markdown syntax, therefore we dont use it as the primary text.
Having said that, most of the spec-renderers focus on one  type of rendering, either geared towards programmers (SwaggerUI) or towards read friendly doc (Redoc)
When you make a renderer specifically for reading, then you expect certain fields to exist even though it is not mandatory in the spec, else it appears broken, this is also the case with ReDoc. Try providing a spec with empty summaries or extra long summaries with markdown syntax, it becomes difficult to read/navigate around
In the current state RapiDoc is more a programmers tools (our goal was to accommodate most API specs not just the ones written specifically with documentation in mind).
but I am working on a feature where RapiDoc can also be used more like a reader, which is what you are after I guess. this will be provided by an option allowing multiple rendering styles.
This is not just about shifting the path and summaries from right to left but many other aspects are being considered to make it read friendly.
we are working towards a 2 column, fully expanded docs without the need of collapse/expand sections. more or less like in these lines https://npr.api-docs.io.
I am happy to take visual design feedback if you have some or want to contribute in some of ur ideas.
Unfortunately, I do not have any timeline on it for now, but i am actively working on it

I am very close to finish up designing/developing -  RapiDoc - Read Mode . wanted to share few screenshots of whats coming up
Read - Mode

Read - Mode with markdown content

in version 6.0.0 there is a render-style = 'read' option, which is more appropriate to what you are looking for.
The priority here is given to summary and descriptions rather than the paths. The view style is also more tuned for reading
Here is a demo
Feedbacks are welcome

I'm sorry that my attention had to go to other projects for a while and I just now found your comments on this. I just tried render-style="read" and I'm loving it! I appreciate how you've built two different render styles for two different uses. Nice.
I'm thrilled to see the navigation in the left column. I know that you aren't a big fan of accordion sections, but you might want to consider making it an option to expand/collapse each heading there. The API that I'm working on right now only has four tags, and each of those has 4-6 requests, so only about 20 requests grouped under 4 headings and I already have to scroll a little. Your swagger pet store demo is about the limit in size for being able to see the navigation without scrolling. For any API with a large number of endpoints/requests, it would be nice to be able to collapse the headers.
If clicking the headers in the navigation column doesn't expand/collapse, perhaps it could link to that heading in the main column. Right now I can click a method summary to navigate right to that method but I can't navigate right to the start of the section.
I'll spend more time with this over the next few days and send more comments if I have any. I really like it, though!
Steve

I've used the Chrome inspector to look at elements in the page source, but I can't tell from that what format I need to use in my OAS file to create a link within the page. I know how to create a link in markdown, but what anchor name do I use to link to a  section (tag) heading, and what anchor name to link to a specific request? It looks like RapiDoc doesn't use the operationIds defined in the file, but creates its own IDs for requests.
Thanks again,
Steve

I just noticed that longer descriptions in response models don't wrap when the column is too narrow to display the full text. Also true for request models.

is it possible for you to share a minimal spec where you see the description cut off ?
I am reopening this issue to address cut off description

we dont get much notification on closed issues, so please open up a new issues for us to address it quicker.

If clicking the headers in the navigation column doesn't expand/collapse, perhaps it could link to that heading in the main column. Right now I can click a method summary to navigate right to that method but I can't navigate right to the start of the section.

so are you looking to have those tags too act as links to the start of section ?
we do have plans to make them links, but isn't the first operation/method under the tag takes you closer to the start of the section. unless you have lot of markdown written at the tag level. I am trying understand the useless here

Clicking the first operation under a tag scrolls the page so that operation is right at the top, which is nice, but it means that any Markdown description of the tag is off the top of the page and can't be seen. I use the tags to group operations by resource type, such as customers, orders, etc. I use the tag description to provide information about each resource type. If that scrolls off the page, you're looking at a request such as "Get all customers" but you don't know everything you need to know about customers. (That example is too simple because most people understand customers, but I have other resource types that aren't as obvious.)

I don't see an Expand Details link in the top right corner, so I couldn't try clicking on that. I added schema-description-expanded = true and that caused long descriptions to wrap (which is what I want). I then set it to false, and the descriptions are cut off. I'll just leave it set to true.

I don't see an Expand Details link in the top right corner, so I couldn't try clicking on that.

I can see it on the screen-shot you attached. The orange colored text top-right

Oh, I was looking in the top right corner of the page, not the top right corner of the model box. I see that link now. :-)


Unable to access individual container(s) (or names) from DockerComposeContainer

After starting containers with DockerComposeContainer it would be nice to be able to access containers or have access to container names. The problem is the names are changed when testContainers starts the containers so they are unknown and different each execution.
I want to be able to stop and start individual containers during testing. If I could access the containers or the container names so I could use Citrus (or another tool) to stop and start.

Stopping and starting will cause the ports to be re-assigned and likely to break your setup.
Consider using GenericContainers and/or the ToxiProxy module for chaos testing


Adding some of the code to a features as plugin for core?

Hey
What about adding some of the features to a core plugin?
https://make.wordpress.org/core/2015/01/13/feature-plugins-in-4-2-and-beyond/
Or adding some of the code/take part in the Customizer project going on for core right now.
Have a great day!

Hey there!
That decision is up to the WordPress Core development team, not us.
IF the core team decides they want to focus on a feature for a future release then a new feature plugin is created.
I can't submit my plugin as a feature core suggestion as I understand it.

Anyone can suggest their plugin as a feature core plugin.
The bottom line is that the Core group will discuss various feature plugins in a development chat (probably tonight). Some might be continued as core plugins, others might just be added as a trac ticket, others they might say that this will not fit into core but can be a separate plugin. So you are right in the end it is up to the Core development team, but it starts out as a suggestion from whomever made it. There have been very few Core plugins for a while now and the thread link I gave you it is nice to see a bunch of projects suggested.
There is a Weekly Developer Meeting - Wednesdays @ 21:00 UTC. Where various projects will likely be discussed. One can suggest a plugin for core anytime.

I'm afraid that I'll have to decline.
I love using Kirki for my own projects, and it does simplify a lot of things!
It's a lot easier to create new settings etc with it than using the default WordPress methods.
However, it's still important that users know how to do things the WordPress way.
If something like this were to be merged in-core then devs would no longer deal with the default API and they wouldn't know what to do to do the stuff they want.
Kirki extends the WordPress Customizer API, it does not (nor would I want it to) replace it.
Besides that however, I don't think a lot of people would want it in-core.
Perhaps we could suggest adding a couple more controls to WP-Core, but other than that no, I don't think so.
WordPress core should be minimalistic and not bloated with 3rd-party stuff like this.
Installing a plugin if you want to take advantage of certain features is not a bad thing... it's the joy of WordPress's modularity!


Titanic dataset has been rescaled?

It seems that the features in the Titanic dataset have been rescaled and become continuous. I don't think this is what we want.

There indeed some issues with this dataset:

The target (survived) has been transformed to -1.0 and 1.0 values, and should be categorical.
The class feature should be categorical instead of continuous.
The sex feature should be categorical instead of continuous.
The age feature has been rescaled and normalised. Moreover, it seems that age and class columns were swapped.

It might be best to replace the dataset.  There is a dataset under a CC0 (public domain) license here: https://www.kaggle.com/datasets/aliaamiri/titanic-passengers-and-crew-complete-list. Working from this dataset I propose the following:

Drop the name and ticketno columns.
Most of the missing data in sibsp and parch columns can be inferred to be 0. These are mostly missing for crew.

To discuss:

How to handle remaining missing values? Do we encode these as unknown or leave NA in the file? Or do we drop such instances completely?

Thank you @alexzwanenburg for following up on this issue. 💯  🚀
I like your proposed replacement dataset and suggested changes. Regarding missing values, in the python and R package, we have a dropna argument (bool) and so I think we should leave NAs in the file and let the user decides.

Most of the missing data in sibsp and parch columns can be inferred to be 0. These are mostly missing for crew.

If we do encode missing data in these two columns as 0, we want to make sure that we explain so in the metadata.

Addressed in #165


crate doesn't exist?

I run  npm i rust-to-npm-cli -g but it doesn't seem to work...
Instead it fails with this error:
Installing and compiling rust-to-npm-cli 0.4.10  ...

Error: Command failed: cargo install rust-to-npm-cli --vers 0.4.10
    Updating crates.io index
error: could not find `rust-to-npm-cli` in registry `crates-io` with version `=0.4.10`

    at ChildProcess.exithandler (child_process.js:383:12)
    at ChildProcess.emit (events.js:400:28)
    at maybeClose (internal/child_process.js:1088:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:296:5) {
  killed: false,
  code: 101,
  signal: null,
  cmd: 'cargo install rust-to-npm-cli --vers 0.4.10 '
}

it seems to be looking for a crate calle

d "rust-to-npm-cli" which doens't exist at all on crates.io

@JimLynchCodes thank you, published now https://crates.io/crates/rust-to-npm-cli. We moved the crate from https://crates.io/crates/rust-to-npm/0.4.9 which prior was the CLI and split the rust crate with the CLI.


JSDoc

The Readme documentation is great but we should also JSDoc the code.

I can grab this today :)


Bump content models for CSG

This commit bumps govuk_content_models to 28.5.0 to allow Specialist Publisher to publish countryside stewardship grants.
Also bumping gds-sso to 10.0.0 as this is required by govuk_content_models 28.5.0

To follow up on a conversation in Slack – I'm pretty sure this is unnecessary. I need to dig a bit more to understand why publishing countryside stewardship grant documents is failing


feat: update curriculum links on unit listing pages

Description
Music year: 1953

Updated "Curriculum download" link on unit listing pages
Updated Curriculum Previous Downloads page to accept subject and key stage in URL params
Fixed timer issues in tests
Refactored Header Listing Curriculum Download Button component for simplicity

How to test

Go to https://deploy-preview-2284--oak-web-application.netlify.thenational.academy/teachers/programmes/citizenship-secondary-ks3-l/units
Click on "Curriculum Download"
You should navigate to the "Previously released curricula" page
The tab "KS3" should be active
The option for "Citizenship" should be selected for download

Checklist

[x] Added or updated tests where appropriate
[x] Manually tested across browsers / devices
[x] Considered impact on accessibility
[x] Design sign-off
[x] Approved by product owner
[x] Does this PR update a package with a breaking change

Simple but effective solution, meets the stretch goals - nice work!
Tested on Chrome and iOS Safari with a range of different subjects (different key stages, different subjects, different levels of the teachers journey).

CUR-232

@lioneljingles Feedback from @Monika-Koziol is that the icon next to the link should be changed to a right-facing arrow, like we did for the Legacy downloads link in the subject-phase picker.

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


Problem calling macro outside of named!

Prerequisites
Here are a few things you should provide to help me understand the issue:

Rust version : 1.55.0
nom version : 6.1.2
nom compilation features used: None

Test case
use nom::do_parse;

#[derive(Debug, PartialEq)]
pub struct Struct {
    one: u8,
    two: u16,
    three: u32,
}

fn main() {
    let input = &[0, 0, 0, 0, 0, 0, 0];
    let output = do_parse!(
        input,
        one: nom::number::complete::u8 >>
        two: nom::number::complete::u16 >>
        three: nom::number::complete::u32 >>
        (Struct {
            one,
            two,
            three,
        })
    );
    let s: &[u8] = &[];
    assert_eq!(output.unwrap(), (s, Struct { one: 0, two: 0, three: 0}));
}

Result:
error: no rules expected the token `input`
  --> main.rs:12:18
   |
12 |       let output = do_parse!(
   |  __________________^
13 | |         input,
14 | |         one: nom::number::complete::u8 >>
15 | |         two: nom::number::complete::u16 >>
...  |
21 | |         })
22 | |     );
   | |_____^ no rules expected this token in macro call
   |
   = note: this error originates in the macro `do_parse` (in Nightly builds, run with -Z macro-backtrace for more info)

error: aborting due to previous error

error: could not compile `test`

To learn more, run the command again with --verbose.

I'm a little confused particularly because input should be an expression so I'm kind of at a loss as to why this is happening after looking at the do_parse! macro source code.

hmm I don't know why it's doing that. But why aren't you using functions? You coud write it that way:
let (input,one) = nom::number::complete::u8(input).unwrap();
let (input, two) = nom::number::complete::u16(input).unwrap();
let (input, three) = nom::number::complete::u32(input).unwrap();
let output = (input, Struct {
            one,
            two,
            three,
        });

or smaller:
let (input, (one, two, three)) = tuple((nom::number::complete::u8, nom::number::complete::u16, nom::number::complete::u32))(input).unwrap();


Added array key casing helpers

This PR allows array key cases to be converted.
This is useful when working with JSON camel case conventions, whilst maintaining PHP array key conventions.
return response()->json(camel_array_keys([
    'key_one' => 'value',
    'key_two' => 'value',
]));

{
  "keyOne": "value",
  "keyTwo": "value",
}

And how is keys treated if they already exists in the array? This needs to be documented and written a test for so it doesn't change in the future.
$data = [
    'my_key' => 1,
    'myKey' => 2,
];

No plans on adding.


Order the entries in the "Miners" table

My miner IDs have the form "-" (e.g. "miner1-4").  It would be convenient to be able to scan through the table to verify that all ethminers on a given host are performing as expected.  Currently, the entries are unordered.  If they were ordered by ID or IP, it would be easier to check the status of a mining server.

Will do, it's planned.

Added simple sorting by ID.

Looks good, thanks!


fixed typo for 9XXX

updated error.rst with more user actionable messages and solutions.

Summary
added more user actionable errors and solutions to fix the issue.
Details and comments

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Whitespaces handling for fileOutputHandler.

File output handler have problem with whitespaces handling. When there are spaces in file name it dont't work at all. It can be solved with this simple fix.

Thanks @matuslittva. Merged.


Unable to start debugging

When running this extension I am able to use the watch functionality, but the debugger never attaches. I noticed the extension was throwing the below error:
Command failed: ps -o pid, ppid, command error: improper format list Usage: ps [options] Try 'ps --help <simple|list|output|threads|misc|all>' or 'ps --help <s|l|o|t|m|a>' for additional help text. For more details see ps(1).
The problem appears to be the space inbetween the pid and ppid argument.
I am running Manjaro 21.2.5
ps --version returns ps from procps-ng 3.3.17

@murugaratham Created a PR that will fix it


[TF-TRT] Disable TF32 for TF-TRT unittests that needs it

TF32 computation may require in some cases a higher threshold to pass some unittests. Instead of implementing some complex logic, the standard procedure for these few cases is to test in FP32 instead of TF32 and not modifying the threshold.
This PR corrects some TF-TRT unittests with this logic

@bixia1 for review

@DEKHTIARJonathan Can you please check @bixia1's comments and keep us posted ? Thank you!

@DEKHTIARJonathan Any update on this PR? Please. Thank you!

@DEKHTIARJonathan Any update on this PR? Please. Thank you!

@DEKHTIARJonathan Any update on this PR? Please. Thank you!

@bixia1 comments fixed, good for review

I am waiting for the author to address my comments.

@DEKHTIARJonathan Any update on this PR? Please. Thank you!

@DEKHTIARJonathan Any update on this PR? Please. Thank you!

@DEKHTIARJonathan Any update on this PR? Please. Thank you!

@bixia1 could you review again ? All conflicts have been solved


Hyperspace technology change

OGame made this change:

Hyperspace technology adds now 2% of cargo capacity to all ships per level. It starts from level 1.

So here are the necessary changes.

Codecov Report

Merging #24 into develop will not change coverage.
The diff coverage is 66.66%.

@@           Coverage Diff            @@
##           develop      #24   +/-   ##
========================================
  Coverage    42.95%   42.95%
========================================
  Files           94       94
  Lines         6169     6169
========================================
  Hits          2650     2650
  Misses        3468     3468
  Partials        51       51

Impacted Files
Coverage Δ

ogame.go
10.97% <ø> (ø)
:arrow_up:

fleetBuilder.go
0% <0%> (ø)
:arrow_up:

baseShip.go
100% <100%> (ø)
:arrow_up:

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 204391f...96af7a1. Read the comment docs.


About page basic content

Resolve #7
This is only basic content to fill this page with some informations. Future plan is to support markdown rendering and add FAQ, Contribution Guide, maybe some setting user can adjust.

It's good time to update the bootstrap script to include erlangpl-ui build. It should help you with testing. I hope to fix it soon.

Should I take care of bootstrap script?

Sure.

I'm going to add erlang-ui as erlangpl submodule.

I've tested it with latest version of erlangpl. Going to merge and release 0.1 version.
/cc @michalslaski


Dockerfile no longer produces a working image

Expected Behavior

Following the 'Build the docker image locally' instructions in the readme should result in a working gimme-aws-creds command

Current Behavior

~ $ gimme-aws-creds --config
Traceback (most recent call last):
  File "/usr/local/bin/gimme-aws-creds", line 4, in <module>
    __import__('pkg_resources').run_script('gimme-aws-creds==2.4.3', 'gimme-aws-creds')
  File "/usr/local/lib/python3.8/site-packages/pkg_resources/__init__.py", line 651, in run_script
    self.require(requires)[0].run_script(script_name, ns)
  File "/usr/local/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1455, in run_script
    exec(script_code, namespace, namespace)
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds-2.4.3-py3.8.egg/EGG-INFO/scripts/gimme-aws-creds", line 13, in <module>
  File "<frozen zipimport>", line 259, in load_module
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds-2.4.3-py3.8.egg/gimme_aws_creds/main.py", line 32, in <module>
  File "<frozen zipimport>", line 259, in load_module
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds-2.4.3-py3.8.egg/gimme_aws_creds/okta.py", line 33, in <module>
  File "<frozen zipimport>", line 259, in load_module
  File "/usr/local/lib/python3.8/site-packages/gimme_aws_creds-2.4.3-py3.8.egg/gimme_aws_creds/u2f.py", line 19, in <module>
  File "/usr/local/lib/python3.8/site-packages/fido2-0.9.2-py3.8.egg/fido2/ctap1.py", line 32, in <module>
    from .cose import ES256
  File "/usr/local/lib/python3.8/site-packages/fido2-0.9.2-py3.8.egg/fido2/cose.py", line 32, in <module>
    from cryptography.hazmat.primitives import hashes, serialization
  File "/usr/local/lib/python3.8/site-packages/cryptography-35.0.0-py3.8-linux-x86_64.egg/cryptography/hazmat/primitives/serialization/__init__.py", line 15, in <module>
    from cryptography.hazmat.primitives.serialization.base import (
  File "/usr/local/lib/python3.8/site-packages/cryptography-35.0.0-py3.8-linux-x86_64.egg/cryptography/hazmat/primitives/serialization/base.py", line 11, in <module>
    from cryptography.hazmat.primitives.asymmetric.types import (
  File "/usr/local/lib/python3.8/site-packages/cryptography-35.0.0-py3.8-linux-x86_64.egg/cryptography/hazmat/primitives/asymmetric/types.py", line 7, in <module>
    from cryptography.hazmat.primitives.asymmetric import (
  File "/usr/local/lib/python3.8/site-packages/cryptography-35.0.0-py3.8-linux-x86_64.egg/cryptography/hazmat/primitives/asymmetric/dsa.py", line 12, in <module>
    from cryptography.hazmat.primitives.asymmetric import (
  File "/usr/local/lib/python3.8/site-packages/cryptography-35.0.0-py3.8-linux-x86_64.egg/cryptography/hazmat/primitives/asymmetric/utils.py", line 6, in <module>
    from cryptography.hazmat.bindings._rust import asn1
ImportError: Error loading shared library libgcc_s.so.1: No such file or directory (needed by /usr/local/lib/python3.8/site-packages/cryptography-35.0.0-py3.8-linux-x86_64.egg/cryptography/hazmat/bindings/_rust.abi3.so)

Steps to Reproduce (for bugs)
docker build -t gimme-aws-creds .

set up alias as described in README
attempt to run command
observe ImportError

Your Environment

App Version used: latest checkout of master
Environment name and version:  Docker Desktop 3.5.1
Operating System and version: Mac OS Mojave

@bwynsm Any chance we can get https://github.com/Nike-Inc/gimme-aws-creds/pull/323 merged? The fix works for me.


New version: RecursiveArrayTools v3.21.0

Registering package: RecursiveArrayTools
Repository: https://github.com/SciML/RecursiveArrayTools.jl
Created by: @ChrisRackauckas
Version: v3.21.0
Commit: 75d9322b55fc31b397f5ce3de21fb112ddb558cc
Reviewed by: @ChrisRackauckas
Reference: https://github.com/SciML/RecursiveArrayTools.jl/issues/66#issuecomment-2139016736
Description: Tools for easily handling objects like arrays of arrays and deeper nestings in scientific machine learning (SciML) and other applications

block


Get file path is slowly

https://github.com/CaiJingLong/flutter_photo_manager/blob/50986fa8ae0fc85bbea26549ed2875f84539ac03/ios/Classes/core/PMManager.m#L612
Why do I need to transfer the image to the sandbox when reading the image? This makes it very slow to obtain the path of all the pictures in the album. My current requirement is to only obtain the path and not the complete file. May I ask whether You can directly add the original path of the picture when initializing the album list to use

I'm very sorry, I have already figured out that flutter does not support this problem caused by the heic format. I decided to use the scheme of displaying thumbnails and transferring original images to solve my problem.


How to convert custom scalar to List/BuiltList?

I can't seem to figure out how to convert a custom scalar to a List or BuiltList of a local model.
type_overrides:
  DateTime:
    name: DateTime
  MyModels:
    name: BuiltList<MyModel>
    # import: 'package:name/path/to/model.dart'

Ive implemented a MyModelsSerializer which returns BuiltList<MyModel>. Ive also tried importing the model file that. defines MyModel (a freezed class)
It doesn't seem to resolve the types properly so build_runner keeps throwing `Make field [name] have non-dynamic type. Is there a naming convention I'm missing for the config?

If your schema defined a MyModel, you'd just need to define an override and a custom serializer for MyModel.
If a graphql operation returns a list of MyModel, ferry is smart enough to automatically deserialize it into a BuiltList<MyModel>.


[Bug] basePath unable to use

I have done a react-static (7.6.2) web page that works under root path (that is /) and everything works as expected.
Last thing that I needed to do is to make application start from a folder /dir1/dir2.
From documentation is look quite simple just to change/set configuration property basePath.
Unfortunately this didn't make it. I spend few hours checking documentation, source code, issues and I'm unable to make this work.
What I need is to change application to work from / to /dir1/dir2.
When I change basePath then a lot of is working as expected:

head link css/js are from required path,
useBasepath() return valid string,
router works.
But when I try to (hover a link, click on it) go to subpage it ends with error:

main.7ec4ee31.js:1 GET http://localhost/templates/__react_static_root__/src/pages/takie-same.9ae1d448.js net::ERR_ABORTED 404 (Not Found)
main.7ec4ee31.js:1 Uncaught (in promise) ChunkLoadError: Loading chunk 30 failed.

It tries to load templates from root directory and not from basePath+'/templates'
I tried to change:

siteRoot - doesn't change requests for templates at root,
assetsPath - something starts to work, but routes are invalid; when used with basePath some values are /dir1/dir2/dir1/dir2,
publicPath
but I was unable to make it work as expected.

No sure what is the problem and how to make it work.

I have also attached build log with logged out parameter passed to getRoutes
build.txt
In header values are correct:

But when trying to go to a subpage or start from a subpage:

then it tries to load from root directory, but values i header are valid.

I just encountered something similar after upgrading from 7.5.x to 7.6.2. This might be related. My staging environment uses a basePath and stopped working.
The BASE_PATH/templates/styles.1e947e5d.js, BASE_PATH/main.6ceefc00.js and BASE_PATH/templates/vendors~main.4f7bf9a9.js are fetched correctly using the base path, but then there are calls to what looks like pages:

templates/vendors~__react_static_root__/src/pages/404~__react_static_root__/src/pages/agb~__react_static_root_~e8299391.a84e1847.js
templates/vendors~__react_static_root__/src/pages/index.6ceefc00.js
templates/__react_static_root__/src/pages/404~__react_static_root__/src/pages/agb~__react_static_root__/src/pa~75aeaec7.6c0e096c.js
templates/__react_static_root__/src/pages/index.b6cc7ca9.js

none of which are using the set basePath and thus fail.
My build for the staging env is done with --staging
Hope this helps to point in the right direction.

I'm having an issue with the default app, I am using basePath to xxx, the nav bar links when you hover have the correct value that includes the basePath.   However, when you click the link it instead takes you to the root.  For example when hovering the about link is https://abc.com/basePath/about, but when you click the link it tries to take you to https://abc.com/about, no network activity and the page is blank

As for me there is no problem.
It seems that I had older react-static on build server and removing node-modules did the trick.
I'm not closing this issue, because of two other comments above.

It seems that bug exists.
If I run react-static build then problem occurs.
If I run react-static build --debug then everything works like expected.
This is why it worked on my build server, because I had this additional option.
Tried this about 3 times and always same result.

How did you solve it? I have the same problem

How did you solve it? I have the same problem
If I run react-static build --debug then everything works like expected.
Adding --debug solved this for me.


TS-4107: proxy.process.ssl.total_success_handshake_count_in has wrong record type

proxy.process.ssl.total_success_handshake_count_in should be the
metrics generated in the core SSL code, and the old compatible name
proxy.process.ssl.total_success_handshake_count should be generated
as a custom metric. The custom metric needs to be registered in
RecordsConfig.cc but it should be of type RECT_PROCESS not RECT_NODE.

ping @zwoop @shinrich

Looks good to me.  It wasn't clear to me how to do the generated metric when I was originally fixing up the names, so I no doubt messed it up.


Add dep package management

The Go team recently added dep as a runner up for the Official package management tool, be nice to add it.
https://github.com/golang/dep

PR #196 already in progress

Done and in master, but not yet released on Heroku. Until it's released (a few weeks) it's usable by pinning the buildpack to the master branch vs. the released heroku/go version.
Thanks!


DAOhaus app

Name/Description
DAOhaus
Type

New addition

Compatible Networks
- Mainnet
- Rinkeby
- xDai
- Kovan
- Polygon

Code for review
https://github.com/HausDAO/daohaus-app
IPFS hash/App URL
https://dapp.daohaus.club/#/

This is updated @mikheevm

Hey, I tested the app, and it works nicely. it has been added to the list of safe apps :) thank you for the contribution, and welcome on board!


Link to documentation is broken

Link to documentation is broken
http://doc.octris.org/org.octris.ncurses/
404 Not Found
nginx/1.4.6 (Ubuntu)

Thanks for your interest in this ... yep, the API documentation is currently offline for all my libraries for quite a while now. Reason is that i switched the documentation tool and did not finish setting it up on the server. I think i'll have to fix this finally ... i think i'll have some time for this at the weekend.

The source documentation is available again:
http://octris.org/doc/octris/ncurses/html/
It's not perfect, yet, but better than nothing i think ...
If you are interested in other projects, i've put up some more documentation at: http://octris.org/doc/
Documentation will get updated nightly (CET).


siad loading transaction pool

Hello,
I launched sia daemon using siad. its connect successfully but when I call consensus API always getting synced: false.  I searched on this all search results saying that to get synced: true you have to download its full blockchain.  I downloaded consensus.db file from https://consensus.siahub.info/ and paste this file in consensus folder and trying to launch SIA daemon its taking too much time to load transaction pool like in below image

I am not getting why this is taking lots of time please help me on this.

"download the blockchain" doesn't mean you have to download a specific file from a site. If you just wait, siad will download the blockchain from your peers automatically. Downloading consensus.db directly is just faster.
When you downloaded consensus.db, you got a copy of the blockchain. But the other siad "modules" (transaction pool, wallet, renter, etc.) still need to process that blockchain. So your first load will take a while because the modules are processing 100k blocks all at once.
After it finishes loading the first time, subsequent loads will be much faster.

@lukechampine thanks for replying.


How to get the name of the downloaded file from popup ?

I want to implement the following scenario:

Click on the file download button > get a native browser download popup with filename, path, etc.

Take the filename from the popup and do some checks (don't need to download the file, this is an excess step, only need the filename)

How to get file name from popup ? Or is there another way to do this ?

@AndreyPermyakov have you tried with https://pptr.dev/#?product=Puppeteer&version=v1.20.0&show=api-class-filechooser ? normally this will used to upload contents, but maybe with this code
const [fileChooser] = await Promise.all([
  page.waitForFileChooser(),
  page.click('#upload-file-button'), // some button that triggers file selection
]);

you can debug the fileChooser constant and see if there are the file name you are searching for ...

You could watch for a response after clicking the download button and try to parse the filename from the headers - something like:
page.on('response', response => {
  //check for "Content-Disposition"
  const disposition = response.headers()['content-disposition'];

  if (disposition && disposition.indexOf('attachment') !== -1) {
    var filenameRegex = /filename[^;=\n]*=((['"]).*?\2|[^;\n]*)/;
    var matches = filenameRegex.exec(disposition);
    if (matches != null && matches[1]) {
      filename = matches[1].replace(/['"]/g, '');
    }
  }
});

@edelauna On some resources no content-disposition header is present, but a file download dialog is triggered: https://httpbin.org/drip


feat: expose all error responses

OverloadedError was being flattened to an ApiError, which meant my retry logic couldn't detect it. I've broken it out to another enum case to address this.
I've also added a String message to RateLimitExceeded (for consistency / traceability) and introduced an UnknownError for ApiErrorType::Other and ApiErrorType::NotFoundError (as those are separate kinds of failures to API misuse, and NotFoundError implies the wrong endpoint is being used, which the user can't really fix themselves)

Looks good, thanks!


save show(VIS) to .jpeg

I did get this working awhile ago, where I was able to write show(VIS) directly to an image file(.jpeg) . but I lost my source code.
I rerember trying with PIL and CV2 but dont rerember which one is the one that did end up working.
for example by default the examples simply display the output once finished, but does not save it to a file.
#-------
vis = render.render_vis(model, objective, param_f=param_f, thresholds=[512], verbose=False, print_objectives=[content_obj, style_obj])[-1]
show(vis)
#------
I want to modify the above to also save vis to physical img. But the array format is tricky

You want to use lucid.misc.io.save


Crash when destructuring an object

Hi !
During my React development I found out that using ES6 Object destructuring causes ESDoc to crash with the following:
TypeError: Cannot read property 'charAt' of undefined
    at Object.<anonymous> (XXX/node_modules/esdoc/out/src/Publisher/Builder/DocResolver.js:121:39)
    at XXX/node_modules/taffydb/taffy.js:743:17
    at each (XXX/node_modules/taffydb/taffy.js:126:17)
    at Object.<anonymous> (XXX/node_modules/taffydb/taffy.js:740:7)
    at Object.API.(anonymous function) [as update] (XXX/node_modules/taffydb/taffy.js:166:18)
    at DocResolver._resolveAccess (XXX/node_modules/esdoc/out/src/Publisher/Builder/DocResolver.js:119:20)
    at DocResolver.resolve (XXX/node_modules/esdoc/out/src/Publisher/Builder/DocResolver.js:51:12)
    at CoverageBuilder.DocBuilder (XXX/node_modules/esdoc/out/src/Publisher/Builder/DocBuilder.js:57:42)
    at new CoverageBuilder (XXX/node_modules/esdoc/out/src/Publisher/Builder/CoverageBuilder.js:35:81)
    at publish (XXX/node_modules/esdoc/out/src/Publisher/publish.js:164:5)

The crash is caused by the following syntax:
const obj = { 'first' : 'Jane', 'last' : 'Doe' };
const { 'first' : f, 'last' : l } = obj;
// f = 'Jane'; l = 'Doe'

There's a simple workaround:
const obj = { 'first' : 'Jane', 'last' : 'Doe' };
const f = obj.first;
const l = obj.last;
// f = 'Jane'; l = 'Doe'

but this might get messy and unconvenien when the number of attributes "renamed" using this syntax increases.

The forthcoming v1.0 of ESDoc will support Babylon for the AST generation / parsing. You might give my current fork of esdoc a shot as it uses Babylon and should support object destructuring / not crash. If it does though let me know and I'll take a look.

Sweet :) thanks for your answer :)

By chance did my fork (Babylon AST parsing) help / stop ESDoc from crashing? If not I'll look into things as the v1.0 release of ESDoc should / will cover this use case. Welcome to ESDoc BTW! If the fork solves the problem consider closing this issue as the v1.0 mainline release of ESDoc which is switching to Babylon should also solve the issue, but is something to be tracked if not.

Hi :) sorry, I maintain a few different projects and I had to switch on another one before having the chance to try your fork :-\ However, I'll try it in the next few hours so as to answer you :)

No worries.. I'd be glad to try and help out; getting things going with the fork is a bit of a workaround right now, but does help in certain use cases. ESDoc is pretty powerful and the forthcoming v1.0 version should provide significantly better performance with modern syntax.

@Manuel-Manoury Please check latest ESDoc v0.5.
I re-open this issue if is problem.


Command based multi voice generation does not take effect

python ./f5_tts/infer/infer_cli.py -c ./f5_tts/infer/examples/multi/story.toml
After generation, only the tones defined by main cannot be effective for town and country tones. How to solve this problem？

how is the command link output, e.g. provide a screenshot
how is the generated sample
what is the toml and txt file content you use
need more info to figure out

(share_venv) E:\F5-TTS\src>python ./f5_tts/infer/infer_cli.py -c ./f5_tts/infer/examples/multi/story.toml
--Load vocos from local path /models/charactr-vocos-mel-24khz
--Using F5-TTS...
--vocab :  E:\F5-TTS\src\f5_tts\infer\examples\vocab.txt
--tokenizer :  custom
--model :  \models\F5TTS_Base\model_1200000.safetensors
--Whisper-v3-turbo 从audio中提取参考词...
E:\share_venv\lib\site-packages\transformers\models\whisper\generation_whisper.py:496: FutureWarning: The input name inputs is deprecated. Please make sure to use input_features instead.
warnings.warn(
You have passed task=transcribe, but also have set forced_decoder_ids to [[1, None], [2, 50360]] which creates a conflict. forced_decoder_ids will be ignored in favor of task=transcribe.
Passing a tuple of past_key_values is deprecated and will be removed in Transformers v4.43.0. You should pass an instance of EncoderDecoderCache instead, e.g. past_key_values=EncoderDecoderCache.from_legacy_cache(past_key_values).
The attention mask is not set and cannot be inferred from input because pad token is same as eos token. As a consequence, you may observe unexpected behavior. Please pass your input's attention_mask to obtain reliable results.
Finished transcription
ref_t:Six spoons of fresh snow peas, five thick slabs of blue cheese and maybe a snack for her brother Bob.
gen_text 0 A Town Mouse and a Country Mouse were acquaintances, and the Country Mouse one day invited his friend to come and see him at his home in the fields. The Town Mouse came, and they sat down to a dinner of barleycorns and roots,
gen_text 1 the latter of which had a distinctly earthy flavour. The fare was not much to the taste of the guest, and presently he broke out with [town] “My poor dear friend, you live here no better than the ants. Now, you should just see how I fare!
gen_text 2 My larder is a regular horn of plenty.  The Country Mouse had never seen anything like it, and sat down to enjoy the luxuries his friend provided: but before they had well begun,
gen_text 4 the door of the larder opened and someone came in. The two Mice scampered off and hid themselves in a narrow and exceedingly uncomfortable hole. Presently, when all was quiet, they ventured out again; but someone else came in, and off they scuttled again.
gen_text 5 This was too much for the visitor. [country] “Goodbye,” [main] said he, [country] “I’m off. You live in the lap of luxury, I can see, but you are surrounded by dangers; whereas at home I can enjoy my simple dinner of roots and corn in peace.”
--Generating audio in 6 batches...
0%|                                                                                | 0/6 [00:00<?, ?it/s]Building prefix dict from the default dictionary ...
Loading model from cache R:\Temp\jieba.cache
Loading model cost 0.971 seconds.
Prefix dict has been built successfully.
E:\share_venv\lib\site-packages\torch\functional.py:650: UserWarning: ComplexHalf support is experimental and many operators don't support it yet. (Triggered internally at ..\aten\src\ATen\EmptyTensor.cpp:32.)
return _VF.stft(input, n_fft, hop_length, win_length, window,  # type: ignore[attr-defined]
100%|████████████████████████████████████████████████████████████████████████████████████| 6/6 [00:20<00:00,  3.37s/it]
tests\infer_cli_out.wav

story.toml
F5-TTS | E2-TTS
model = "F5-TTS"
ref_audio = "E:/F5-TTS/src/f5_tts/infer/examples/multi/main.flac"
If an empty "", transcribes the reference audio automatically.
ref_text = ""
gen_text = ""
File with text to generate. Ignores the text above.
gen_file = "E:/F5-TTS/src/f5_tts/infer/examples/multi/story.txt"
remove_silence = true
output_dir = "tests"
[voices.town]
ref_audio = "E:/F5-TTS/src/f5_tts/infer/examples/multi/town.flac"
ref_text = ""
[voices.country]
ref_audio = "E:/F5-TTS/src/f5_tts/infer/examples/multi/country.flac"
ref_text = ""

@newstargo seems an issue that you haven't proper install the pkg
need to follow readme, e.g. it seems you want to do the generation locally, so (after you have set up conda env and proper install torch stuffs) run:
git clone https://github.com/SWivid/F5-TTS.git
cd F5-TTS
pip install -e .

then
f5-tts_infer-cli -c src/f5_tts/infer/examples/multi/story.toml

will close this issue, feel free to open if further questions


Improve callbacks while zooming

Describe the bug
There are two main problems:

The callback for onCropComplete is being called repeatedly while zooming - so they are being simultaneously called
While zooming, the onZoomChange is being called a lot of times (maybe it's possible to throttle it)

To Reproduce
Steps to reproduce the behavior:

Add logs in the onCropComplete and onZoomChange callbacks
Zoom in

Expected behavior

onCropComplete will be called only once - after you finish zooming (the same behavior as happens with onCropChange)
onZoomChange will be called less frequently

Here's a sample of the logs while zooming in:

Thanks for your help! This package is super useful!

Sorry for missing this issue.
I understand your point. The problem with the zoom is that it's harder to detect that the zoom gesture is complete compared to other gestures. This is due to the wheel event not having a start/end specific event.
I guess we could add some internal logic to only fire the onCropComplete when the wheel event hasn't been received for a while. Basically debouncing it.


Using --data with a multiselect based choice

Describe the problem
Follow on from: https://github.com/orgs/copier-org/discussions/1473
Template
fruit:
    type: str
    help: "Fruit?"
    choices:
        - apple
        - pear
        - banana
    multiselect: true

To Reproduce
$ copier copy --data 'fruit=apple' ../copier-issue .
...
ValueError: Invalid choice

Logs
No response
Expected behavior
Primary:
--data is able to find the 'apple' from the 'fruit' multiselect based choice, and select it.
Secondary:
To support multiple options perhaps multiple --data entries are required akin to:
--data 'fruit=apple' --data 'fruit=banana'

or comma separated
--data 'fruit=apple, banana'

or an array?
--data 'fruit=[apple, banana]'

Screenshots/screencasts/logs
No response
Operating system
macOS
Operating system distribution and version
Sonoma 14.2.1
Copier version
copier 9.1.0
Python version
Python 3.9.6
Installation method
pip+pypi
Additional context
No response

My 2 cts (as the author of the multiselect feature):
Given https://github.com/copier-org/copier/blob/master/copier/cli.py#L162-L178 is already a list, and current behavior is "last value overrides previous", I would avoid the multiple flags approach as it would mean different behavior and different parsing strategies.
Between --data 'fruit=apple,banana' and --data 'fruit=[apple, banana]', I would chose the first one based on this:

I think comma separated list are more common as parameter and potentially simpler to parse
this is very personal, but I don't like CLI expecting JSON-like data: most of the time it's a lot of noise for nothing (accepting JSON but expecting only a subset which most of the time is easier to write in basic cli style), and often double escaping required
if it's not JSON, it means it needs a custom parser and so more complexity and maintenance than splitting on commas

I think comma-separated values won't work in general, e.g. type: yaml or type: json might contain commas which should not be used to split multiple answer values in a multiselect question. 🤔

Have you actually tried the array way? Something like:
copier copy --data 'fruit=[apple]' ../copier-issue .

@yajo I've tried several ways, also with an array, none works. The problem is that the answer passed via CLI is a string, which gets parsed by
https://github.com/copier-org/copier/blob/f92c1fe0c952c9ab7f839ecd3d8dc27752ac71aa/copier/main.py#L438
which for multiselect questions does not work as expected because a list is expected:
https://github.com/copier-org/copier/blob/f92c1fe0c952c9ab7f839ecd3d8dc27752ac71aa/copier/user_data.py#L439-L440
I'm not sure what the best solution to this problem is though. Should be support string answers for all questions, which get parsed according to the question type?

I think the we just need a specific parser for multiselect questions. It could be a yaml that asserts the value becomes a list and casts its elements to the type of the question.

I just stumbled into this error. I also tried all the combinations of passing date mentioned above before deciding to open an issue, an then found this existing one :smiling_face_with_tear:
Is there really a need for allowing , in (multiselect) choice values?
I would like to vote for the simple comma separated format. If needed '\,' could be used on the commandline to allow ,.
Not being able to specify all data on the commandline is a real showstopper for automation.

I have the same problem. I would prefer any solution, may it be comma-separated or JSON.
Until then it is: modify .copier-answers.yml, commit and update :(

@johannesloibl rather than editing the .coper-answers.yml file directly - we have had more success by using the --data-file argument (--data-file fruit.yml for example) that we then author (programatically) to include the following:
---
fruit:
- apple
- banana


Dependencies are not packaged for the lambda which has Individual packaging (per lambda, not the top level).

Are you certain it's a bug?

[X] Yes, I'm using the latest plugin release

Is there an existing issue for this?

[X] I have searched existing issues, it hasn't been reported yet

Issue description
When individual packaging is defined per lambda, dependencies are not packaged for that lambda.
I.e.
We have a such function in serverless.yml:
  MyIndividualFunction:
    module: src.my_module
    handler: my_file.my_handler
    package:
      individually: true

and such project structure:
src/
-- my_module/
---- my_file.py
---- requirements.txt

And libraries listed in requirements.txt are not packaged to the final MyIndividualFunction.zip
Service configuration (serverless.yml) content
N/A

Command name and used flags
N/A
Command output
N/A

Environment information
Framework Core: 1.83.0
Plugin: 3.8.4
SDK: 2.3.2
Components: 2.34.9

"serverless-python-requirements": "5.1.0"

@pgrzesik Thanks for responding. You are correct.
But I have updated the framework and the plugin and have the same result, unfortunately
Current:
Framework Core: 3.25.0 (local) 3.25.0 (global)
Plugin: 6.2.2
SDK: 4.3.2
{...
"serverless-python-requirements": "^6.0.0"
...}

@amediantsev I am not sure if you still have this issue.
What worked for me is removing serverless-python-requirements cache.
I found the directory by running this:
SLS_DEBUG=* sls package --verbose

Line specifying the cache dir starts with Using static cache of requirements found at
Hope that helps

@amediantsev did you get to solve this thing?
I am using the latest version and the packed zip includes the whole project (instead the module only) and does not include any of the requirements that exist in the req file inside the module defined.
any solution for this? where does it says it is supported?

@roy-firman I worked around it:
I installed serverless-plugin-scripts
Then added:
custom:
  scripts:
    hooks:
      "package:setupProviderConfiguration": |
        bash -c "pushd sub-dir && poetry export --without-hashes -f requirements.txt -o requirements.txt --with-credentials && popd"

  pythonRequirements:
    usePoetry: false
    ...
    ...

Hope it helps

I have the same issue trying to deploy inside a Bitbucket pipeline. Also, I'm using serverless compose to deploy two services:

the first containing a single lambda function
the second containing 3 lambda functions

The packaging for the second fails for 1 single lambda but randomly: sometimes it works, sometimes don't.
I'm using the most recent versions and nodejs 20.

Hey @jacksoncassimiro 👋 Sorry you've ran into trouble. I've tried to reproduce this bug but I was not successful in my attempts. Would you have the ability to share a reproducible example by any chance? Does it only fail when working Compose?
If anyone else could provide a concrete repro example, I would be grateful 🙇

@jacksoncassimiro  I was facing the same issue. You need to add individually: true at the top level package: key in serverless.yaml file. The steps I followed are:
service: my-service-${self:custom.version}

custom:
  version: ${env:VERSION}
  pythonRequirements:
    dockerizePip: true
    noDeploy:
      - moto # This is already available in aws lambda so no need to deploy.
      - boto3 # This is already available in aws lambda so no need to deploy.

package:
  individually: true # This is important to package requirements individually with all lambdas.
  patterns:
    - "!.venv/**"
    - "!.venv*/**"
    - "!package*.json"
    - "!.vscode"
    - "!Docker/**"
    - "tests/**"
    - "!buckets/**"
    - "!node_modules/**"
    - "!openapi.yml"
    - "!models.yaml"
    - "!serverless.doc.yaml"
    - "!serverless-*.yaml"

Its working fine for me.


Update readme with latest inspections

@sksamuel Update the readme to bring it in sync with the latest inspections

:+1:


Remove signup flow

Motivation and context

How has this been tested?

Checklist

[ ] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below)

# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

you haven't touched package.json but package-lock.json changed. how's that?


fix(pulse-plugin): changed metric to MB for RAM chart (#1565)

Pull Request Test Coverage Report for Build 1695

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 53.534%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/app/pulse/charts/pulse-cpu-ram-chart/pulse-cpu-ram-chart.component.ts
0
1
0.0%

Totals

Change from base Build 1693:
0.0%

Covered Lines:
9417

Relevant Lines:
15612

💛  - Coveralls


Can create Stack - Can't connect to the container

Server side (Synology)/direct docker socket connection
Portainer SE: 2.9.2, used as GUI
works as expected for more than 20 containers within the Server side hosts (no troubles there for many years)
Client side(TrueNAS Scale)
OS: Debian 5.10.70 kernel
Docker: 20.10.9
Docker API: 1.41
OS/Arch: linux/amd64
is ACTIVE and is running as expected (CLI: # sudo service docker status)
Portainer CE Environment setup for the Clien side:
Directly connect to the Docker API
port 2376
Public and Environment IP is IP of the TrueNAS host
TLS hardened by the Client cert
works as expected ... is UP (Portainer GUI)
docler status  (client side):
6127 /usr/bin/dockerd --tlsverify --tlscacert=/path to the cert/ca.pem --tlscert=/path to the cert/server-cert.pem --tlskey=/path to the cert/server-key.pem -H fd:// -H tcp://<IP_ADDRESS>:2376
Portainer Agent (client side):
Installed in the client side
running on 9001 as expected
Binded both Mounts paths:

/run/docker.sock : /var/run/docker.sock
/path-to-the-volumes : /var/lib/docker/volumes
works and is HEALTHY (Portainer GUI)

What I can:

I can create New Stack successfully, e.g. MySQL container from Portainer GUI
This container is Healthy and running
I can click to the container's Logs -and all si working as expected (MySQL is running)
sudo service docker status  (client side):
6444 /usr/bin/docker-proxy -proto tcp -host-ip <IP_ADDRESS> -host-port 9010 -container-ip <IP_ADDRESS> -container-port 3306
docker ps
744d3bfa3fb3   mysql/mysql-server:5.7   "/entrypoint.sh mysq…"   8 hours ago         Up 2 hours (healthy)   33060/tcp, <IP_ADDRESS>:9010->3306/tcp, :::9010->3306/tcp   001_MySQL_TrueNAS

The Problem:
4. I can click to the container's Console. I can run /bin/bash as root ... but the connection stuck every time
also I can't connect to the host IP:port defined for the container - 9010:3306 (MySQL Workbench used for the test)

tcpdump seen from the client and the container port 3306, when I'm using the Console from the Portainer:
... nothing
tcpdump seen from the client and the exposed port 9010, when MySQL workbench tried to connect to the db:
# tcpdump port 9010
tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on enp0s3, link-type EN10MB (Ethernet), snapshot length 262144 bytes
18:09:05.206982 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [S], seq 696605945, win 62720, options [mss 8960,nop,wscale 8,nop,nop,sackOK], length 0
18:09:05.207008 IP TRUENASHOST:9010 > MYSQLWORKBENCH-PC: Flags [S.], seq<PHONE_NUMBER>, ack 696605946, win 64240, options [mss 1460,nop,nop,sackOK,nop,wscale 7], length 0
18:09:05.207667 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [.], ack 1, win 8212, length 0
18:09:20.303382 IP TRUENASHOST:9010 > MYSQLWORKBENCH-PC: Flags [.], ack 1, win 502, length 0
18:09:20.303687 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [.], ack 1, win 8212, length 0
18:09:35.410858 IP TRUENASHOST:9010 > MYSQLWORKBENCH-PC: Flags [.], ack 1, win 502, length 0
18:09:35.411185 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [.], ack 1, win 8212, length 0
18:09:50.512091 IP TRUENASHOST:9010 > MYSQLWORKBENCH-PC: Flags [.], ack 1, win 502, length 0
18:09:50.512427 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [.], ack 1, win 8212, length 0
18:10:05.217592 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [F.], seq 1, ack 1, win 8212, length 0
18:10:05.224119 IP TRUENASHOST:9010 > MYSQLWORKBENCH-PC: Flags [.], ack 2, win 502, length 0
18:10:20.462979 IP TRUENASHOST:9010 > MYSQLWORKBENCH-PC: Flags [.], ack 2, win 502, length 0
18:10:20.463387 IP MYSQLWORKBENCH-PC > TRUENASHOST:9010: Flags [.], ack 1, win 8212, length 0
This is my draft of connections:

I found the reason by myself
There was a problem when I connect into Portainer UI (browser) by:

'IP ADDRESS:PORT' in my LAN
'hostname.localdomainname' in my LAN

Both ways are applicable to run Portainer CE in the browser, as expected.
All Portainer services are running as expected within the GUI for both ways.
Except for the single one: Run Console for (each) Container, when I use the second version
then the HTTP. handler has trouble in portainer/libhttp

because I found this error in Portainer logs (every time when I use the second way of connection into Portainer UI):
level=info msg="2021/12/03 09:20:15 http error: An error occured during websocket exec operation (err=websocket: the client is not using the websocket protocol: 'upgrade' token not found in 'Connection' header) (code=500)"
level=info msg="2021/12/03 09:20:15 http: superfluous response.WriteHeader call from github.com/portainer/libhttp/error.writeErrorResponse (error.go:37)"

Solution: I can use just the first way to connect to the Portainer UI by the browser when I need WebSocket for Shell usage within containers.
Your point?

Can I ask how did you managed to install the Portainer client on the TrueNAS Scale Docker host?
Upon trying to download the image and install the agent I get the following error:
Status: Downloaded newer image for portainer/agent:2.17.1
docker: Error response from daemon: invalid volume specification: '/var/lib/docker/volumes:/var/lib/docker/volumes': '/var/lib/docker/volumes' 'path' not allowed to be mounted.


Configuration file ignores specified paths in lieu of its own

The documentation for the configuration file feature appears quite lacking, but it seems that when a configuration file is specified it is expected to provide its own Finder instance, and rules that specify which paths to search.
This doesn't make sense because the command line also accepts arguments that specify which paths/files to scan. Why should the configuration file be allowed, much less required, to override the paths specified by at the command line?
Furthermore, in the case that the configuration author wishes to just pass through the paths specified on the command line, it does not seem easy to do this. PHP-CS-Fixer makes no effort to provide the passed paths to the configuration so the best alternative is to re-parse the command line using the $_SERVER super global, which is very awkward.
This ticket should be considered satisfied when one or more of the following solutions are implemented.

Allow configurations to opt-out of specifying a Finder instance and search the paths passed on the command line instead.
Provide a mechanism for configurations to easily access the passed paths to seed such a Finder instance, assuming one is still required.

configuration file is specified it is expected to provide its own Finder instance

possible, not required

Why should the configuration file be allowed, much less required, to override the paths specified by at the command line?

not true, CLI arg may override value from config file. config file provides default values, CLI just a one-run values.

Furthermore, in the case that the configuration author wishes to just pass through the paths specified on the command line, it does not seem easy to do this.

use path from config or CLI arg, no need to pass CLI arg to config instance.

Allow configurations to opt-out of specifying a Finder instance and search the paths passed on the command line instead

Either use path from CLI or from config. dont mix them. You could use intersection of config/CLI using intersection mode.

Provide a mechanism for configurations to easily access the passed paths to seed such a Finder instance, assuming one is still required.

config file should not be aware about CLI args.

Allow configurations to opt-out of specifying a Finder instance and search the paths passed on the command line instead.

This would be a BC break that does not add a lot of value. As --path-mode can be set to override (is by default) the files returned by the configuration are not used; the location passed by the command line is used.

Provide a mechanism for configurations to easily access the passed paths to seed such a Finder instance, assuming one is still required.

I'm not in fav. of this design change. A configuration should be independent of values passed by the command line, because the code can be used in a diff. context. Resolving the configuration based on diff. sources; configuration file, cli options etc. is the responsibility of the ConfigurationResolver, not of the configuration classes.


Benchmarks

@smothers here it is, back from the dead. Happy Halloween! 💀
If it looks good: I can merge it in if I'm online or if you want to do the manual merge feel free.

I think if we do merge it in we should shut off the benchmarking action until we're ready.

@rubinovitz I agree. I'd like to be able to use the checks for CD so we should turn off checks we don't expect to pass

Should I just remove the workflow for now before we merge?
On Mon, Nov 2 2020 at 10:15 AM, Chris Smothers <<EMAIL_ADDRESS>> wrote:

@rubinovitz ( https://github.com/rubinovitz ) I agree. I'd like to be able
to use the checks for CD so we should turn off checks we don't expect to
pass
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/homebaseio/homebase-react/pull/3#issuecomment-720640617
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AANZMF7NIYGMPWNPDS6A4WDSN3ZMVANCNFSM4TFQG5GA
).

@rubinovitz given we're not ready to turn the benchmarks on yet what do you think about?

pulling the packaging code into a separate PR
significantly relaxing the timings on the benchmarks so they pass and can be used for CI

Or: pulling the JS testing code into a separate PR with some simplified passing tests just to kick off CI and revisiting benchmarks later

Yeah there’s some of this that are resolving conflicts that emerged when a bunch was merged in while this branch was broken. I can go back and go through it
On Fri, Nov 6 2020 at 7:00 AM, Chris Smothers <<EMAIL_ADDRESS>> wrote:

@smothers commented on this pull request.
In package.json (
https://github.com/homebaseio/homebase-react/pull/3#discussion_r518806411
) :

 "shadow-cljs": "2.11.4"

},

"dependencies": {}

"dependencies": {},

"bugs": {
"url":
"https://github.com/homebaseio/homebase-react/issues"

and this
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (
https://github.com/homebaseio/homebase-react/pull/3#pullrequestreview-525231512
) , or unsubscribe (
https://github.com/notifications/unsubscribe-auth/AANZMFYWBYIB2HG73MYNLTTSOQFRZANCNFSM4TFQG5GA
).

@smothers The packaging code is a result of me having to manually merge a bunch of code and accidentally modifying the packing code. I'll go back and remove the packing code changes.

@rubinovitz given we're not ready to turn the benchmarks on yet what do you think about?

pulling the packaging code into a separate PR

significantly relaxing the timings on the benchmarks so they pass and can be used for CI

Or: pulling the JS testing code into a separate PR with some simplified passing tests just to kick off CI and revisiting benchmarks later

#1 is just going to occur when I remove the packaging changes the arose during the messy merge, #2 is fine

@rubinovitz reduce number of components to 100