Problem with Importing and Crashing.

worldometer.current_world_population()
Traceback (most recent call last):
File "", line 1, in
NameError: name 'worldometer' is not defined
from worldometer import Worldometer
Traceback (most recent call last):
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/worldometer/core.py", line 234, in _get_html
self.__r.html.render(timeout=self.__timeout)
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/requests_html.py", line 586, in render
self.browser = self.session.browser  # Automatically create a event loop and browser
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/requests_html.py", line 730, in browser
self._browser = self.loop.run_until_complete(super().browser)
File "/nix/store/2vm88xw7513h9pyjyafw32cps51b0ia1-python3-3.8.12/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
return future.result()
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/requests_html.py", line 714, in browser
self._browser = await pyppeteer.launch(ignoreHTTPSErrors=not(self.verify), headless=True, args=self.__browser_args)
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/pyppeteer/launcher.py", line 307, in launch
return await Launcher(options, **kwargs).launch()
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/pyppeteer/launcher.py", line 168, in launch
self.browserWSEndpoint = get_ws_endpoint(self.url)
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/pyppeteer/launcher.py", line 227, in get_ws_endpoint
raise BrowserError('Browser closed unexpectedly:\n')
pyppeteer.errors.BrowserError: Browser closed unexpectedly:

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/worldometer/init.py", line 84, in
from .api import *
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/worldometer/api.py", line 108, in
__w = Worldometer()
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/worldometer/core.py", line 181, in init
self._metrics = self.collect_metrics()
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/worldometer/core.py", line 300, in collect_metrics
html = self._get_html(url=URL)
File "/home/runner/Worldmeter/venv/lib/python3.8/site-packages/worldometer/core.py", line 239, in _get_html
raise Exception(err)
Exception: Browser closed unexpectedly:
This is the error

Did you find a fix to this? Thanks

Hi @Electro18 and @Kabewh,
Thank you for opening this issue, and I apologize for the delay in responding to it.
I've made a significant update to this project to address most of the recurring issues. This update introduces a new API (the old one is deprecated and will be removed in the future), new datasets, and updated documentation.
If you are still interested in this package and the collected data, you can check it out in version >=2.0.0 on PyPI.
If there are any other issues, please open them, and I will review them as soon as possible.


[BUG] Error in inverse-variance method of AutoEnsembleForecaster

Describe the bug
When fitting the AutoEnsembleForecaster forecaster with exogenous data using the inverse-variance method, an error is raised. This is because X is passed to the internal forecasters instead of X_test when a prediction is made (see here).
To Reproduce
from sktime.datasets import load_longley
from sktime.forecasting.arima import ARIMA
from sktime.forecasting.compose import AutoEnsembleForecaster

forecasters = [
    ("forecaster_1", ARIMA()),
    ("forecaster_2", ARIMA()),
]

y, X = load_longley()

ensemble_forecaster = AutoEnsembleForecaster(
    forecasters=forecasters, method="inverse-variance"
)
ensemble_forecaster.fit(y, X)

This code snippet has the following output:
Traceback (most recent call last):
  File "test.py", line 16, in <module>
    ensemble_forecaster.fit(y, X)
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/base/_base.py", line 270, in fit
    self._fit(y=y_inner, X=X_inner, fh=fh)
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/compose/_ensemble.py", line 190, in _fit
    for y_pred_test in self._predict_forecasters(fh_test, X)
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/base/_meta.py", line 72, in _predict_forecasters
    return [forecaster.predict(fh=fh, X=X) for forecaster in self.forecasters_]
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/base/_meta.py", line 72, in <listcomp>
    return [forecaster.predict(fh=fh, X=X) for forecaster in self.forecasters_]
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/base/_base.py", line 325, in predict
    y_pred = self._predict(fh=fh, X=X_inner)
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/base/adapters/_pmdarima.py", line 76, in _predict
    return self._predict_fixed_cutoff(fh_oos, X=X)
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/sktime/forecasting/base/adapters/_pmdarima.py", line 178, in _predict_fixed_cutoff
    result = self._forecaster.predict(
  File "/Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/lib/python3.8/site-packages/pmdarima/arima/arima.py", line 785, in predict
    raise ValueError('X array dims (n_rows) != n_periods')
ValueError: X array dims (n_rows) != n_periods

Expected behavior
The enseble forecaster should run without error and summarize the forecasters based on the inverse error variance of their predictions.
Additional context

Versions

System:
python: 3.8.9 (default, Oct 16 2021, 12:00:06)  [Clang 12.0.0 (clang-1<IP_ADDRESS>)]
executable: /Users/anhoang/Projekte/2022/Daimler/Chronos/chronos/.venv/bin/python
machine: macOS-11.6-x86_64-i386-64bit
Python dependencies:
pip: 22.1.2
setuptools: 63.2.0
sklearn: 1.1.1
sktime: 0.13.0
statsmodels: 0.13.2
numpy: 1.22.4
scipy: 1.8.1
pandas: 1.4.3
matplotlib: 3.5.2
joblib: 1.1.0
numba: 0.56.0
pmdarima: 1.8.5
tsfresh: 0.19.0

Yes, thanks for diagnosing, @AnH0ang!
That looks indeed like the error, thanks for identifying!
Would appreciate a PR with a fix.
Btw, if you (or the person fixing this) wouldn't mind adding the different method parameters as settings in the get_test_params, that would be appreciated - otherwise the tests will not cover the part of the code where the bug was.

adding "good first issue", since it seems like the bug is already diagnosed thanks to @AnH0ang !


Remove instructions on how to build Developer images of Cilium

Referring to this conversation
Signed-off-by: Philipp Gniewosz<EMAIL_ADDRESS>

Closing PR as discussed in https://github.com/cilium/cilium/pull/14425#discussion_r544226135.


Implement hash chain derivation in FMC.

Extra cycles from this commit: 218,055 cycles (0.5ms at 400mhz)
Context from the spec: https://github.com/chipsalliance/Caliptra/pull/171

@bluegate010 this PR implements stable identity, correct?
@nvpachou @Bryankel are you in favor of this feature?
With Prabhu on travel, can @pkwidzin-amd proxy the AMD opinion? Thanks

@bluegate010 this PR implements stable identity, correct?
@nvpachou @Bryankel are you in favor of this feature?
With Prabhu on travel, can @pkwidzin-amd proxy the AMD opinion? Thanks

This PR implements the FMC features required for stable identity. Runtime firmware features would come later.


Windows trouble installing with pip

Having an issue installing parquet-tools on Windows. Keep getting this error. Do I need Python version < 3.7?

@johnhill2424 Thank you for making issue.
But I'm sorry for not giving sure advice about WIndows because of not having windows machine.
I believe that pip installed numpy==1.16.0 but the numpy version does not support 3.9(your python env).
see https://pypi.org/project/numpy/1.16.0/
I thought that you could install parquet-tools if you installed latest numpy. So please run poetry add numpy before installing parquet-tools.
If it will work in your Windows machine, please make PR.


ES6 Rest parameter expansion is broken for invocations with new

Example
http://closure-compiler-debugger.appspot.com/#input0%3Dnew%2520Foo(...arguments)%26input1%26conformanceConfig%26externs%3Dfunction%2520Foo()%2520%257B%257D%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRESERVE_TYPE_ANNOTATIONS%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1
Stuff like
new Foo(...arguments)

gets rewritten to
new Function.prototype.bind.apply(Foo, [].concat($jscomp.arrayFromIterable(arguments)));

where new "applies" to apply.

This may be a code printer bug.

It looks like it's actually two bugs! One in the code printer and one in the transpilation.


typedoc should support tsx

Hello! Thank you for this project!
Now, as i understand, typedoc not supported tsx files, which available in TypeScript 1.6.
Do you have any plans for support jsdocs in tsx?
Its key feature which people needs, because it will allow doc react components which will be reused.
Thank you!

+1 for this. .tsx support would be awesome!

+1

+1

TypeDoc parses TSX without problems but we currently down include them when a folder is specified as source. I've included the *.tsx extension so TypeDoc will pick up and parse those files.
I'm not familiar with React, besides the including *.tsx files, are there any specialities we should show in the documents?

@sebastian-lenz
As i anderstand, your changes
is not in npm. Can you push it to npm for testing? After i'll notify you about my experience with awesome typedoc and tsx.
Thank you!

I'll publish a new release soon, let me check the open issues first.

Thank you, I'll be waiting :)

v0.3.12 is on NPM!

@sebastian-lenz, thank you! It Works for .tsx


CLI for M365 version for script "Export Term Store terms to CSV" closed #359

CLI for M365 version for script "Export Term Store terms to CSV" closes #359

Hey @reshmee011, thank you another awesome sample!! 👏😊 This should be live shortly 😊

Thanks @pkbullock for merging the PR and @Adam-it for creating the specs.


地址失效

https://chrome.google.com/webstore/detail/kpfmckjjpabojdhlncnccfhkfhbmnjfi
地址失效

很抱歉，这个插件 2013 年的时候已经被屏蔽了
源码在这里：https://github.com/gbk/chrome-hosts-manager
替代方案的半成品在这里：https://github.com/gbk/network-mapping
On Sat, Apr 22, 2017 at 12:01 PM, tomtsang<EMAIL_ADDRESS>wrote:

https://chrome.google.com/webstore/detail/kpfmckjjpabojdhlncnccfhkfhbmnjfi
地址失效
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/gbk/chrome-hosts-manager/issues/10, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AA01tZNF_aR7o4uiEMESignC6974CyJqks5ryXuRgaJpZM4NE9we
.


Devonagent pro

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Additionally, when adding a new cask:

[x] Checked there are no open pull requests for the same cask.
[x] Checked there are no closed issues where that cask was already refused.
[x] When naming the cask, followed the token reference.
[x] brew cask install {{cask_file}} worked successfully.
[x] brew cask uninstall {{cask_file}} worked successfully.

@nbsantos Can you make the requested changes?

Closing for lack of reply. If you still wish to include this, please resubmit with the requested fixes.


Bump CAPI infra providers versions

What this PR does / why we need it:

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Checklist:

[ ] squashed commits into logical changes
[ ] includes documentation
[ ] adds unit tests
[ ] adds or updates e2e tests

@furkatgofurov7 Let's bump them together with CAPI and CAPD later when we switch to a newer version of CAPI


NcclCommunicator to expose function for getting the underlying ncclComm_t

I've been looking at your nccl.pyx wrapper for possible use inside RAPIDS cuML.
It would be great if we could initialize the ncclComm_t object in Python using your wrapper and then pass the ncclComm_t object down to C++ for native processing, as necessary.

I think that feature is reasonable. Could you make a PR?
Please be aware that cupy.cuda.nccl module may not be available if user builds CuPy from source (pip install cupy) without installing NCCL. (CuPy wheels like pip install cupy-cuda101 always come with NCCL.)

Certainly, I will make a PR. Thank you.

@cjnolet does this quick patch meet your need (diff'd against #2325)?
diff --git a/cupy/cuda/nccl.pyx b/cupy/cuda/nccl.pyx
index 78baeee..15b2378 100644
--- a/cupy/cuda/nccl.pyx
+++ b/cupy/cuda/nccl.pyx
@@ -341,6 +341,9 @@ cdef class NcclCommunicator:
             ncclCommAbort(self._comm)
             self._comm = <ncclComm_t>0

+    cdef ncclComm_t* get_comm_ptr(self):
+        return &self._comm
+
     def device_id(self):
         cdef int device_id
         status = ncclCommCuDevice(self._comm, &device_id)


Closes: #1

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

Correct Updated validations.py python script.
Fixed the behavior of the validate_user function in validations.py.

correct


Allow opening empty excalidraw file

Fix #1321

This actually also disables invalid-file detection. Seems that it wasn't implemented correctly if it based it on whether elements aren't empty.

Throwing this on line 19 should work:
throw Promise.reject(t("alerts.couldNotLoadInvalidFile"));

Thank you @dwelle great catch! Fixed.

I'm getting the screen of death with this JSON
{
  "labels": {
    "linearElementMulti": "Press Escape or Enter to finish",
    "resize": "You can constraint proportions by holding SHIFT while resizing"
  }
}

This actually also disables invalid-file detection. Seems that it wasn't implemented correctly if it was based it on whether elements aren't empty.
repro:

load an empty file with  json extension, or containing e.g. {}

Yea. It was implemented that way to avoid importing invalid json files :)

Now it display error splash as @lipis pointed. Weird..

Pushed a fix. Apparently we can't reject with an error object in here. Rather than fixing the root cause, I just used what we did before → reject with a string.

Weird, it was working just fine by throwing a regular error. I personally avoid throwing strings and use native errors. In any case, thanks for fixing. :)

No, you're right. I should have just fixed the one more case we pass error as string. Pushed a fix to go back to error objects to not subvert expectations :).

🚢


Using a Template Resource in an Accumulator using with_run_context :root gets the cookbook_name wrong sometimes

Seems we're picking up the cookbook_name from the root run_context sometimes and looking for the template source in the calling cookbook, not the cookbook_name the resource was in.
Workaround looks something like this (from the ssh_known_hosts cookbook):
  r = with_run_context :root do
    # XXX: remove log resource once delayed_actions lands in compat_resource
    find_resource(:log, 'force delayed notification') do
      notifies :create, 'template[update ssh known hosts file]', :delayed
    end
    find_resource(:template, 'update ssh known hosts file') do
      source "ssh_known_hosts.erb"
      path node['ssh_known_hosts']['file']
      owner new_resource.owner
      group new_resource.group
      mode new_resource.mode
      action :nothing
      backup false
      variables(entries: [])
    end
  end

  # messing with the run_context appears to cause issues with the cookbook_name
  r.cookbook_name = "ssh_known_hosts"

Whats the eta on this, I landed on this after a lot of googling. I have it like this in a custom resource  cookbook I'm writing.
action :create do
  r = with_run_context :root do
    edit_resource(:template, 'update config file' ) do
      source "#{process}.erb"
      path cfg_file
      mode '0644'
      variables( here: )
    end
  end
r.cookbook_name = "new_custom_resource"
end

Errors out saying undefined method cookbook_name=' for ["calling_cookbook"]:Array
If I comment r.cookbook_name section, the same problem where it cannot find the template files in the calling cookbook.

@awolde Put self.cookbook_name = "new_custom_resource" inside the edit_resource block.

putting in cookbook = 'new_custom_resource' inside the edit_resource block worked.

wonder if this isn't another scoping issue like in #5438


[Typo/Mistranslation] 02-load-data-with-power-query-in-power-bi-desktop

Query table names need to stay in English.
Current:
매출
대상
To be:
Sales
Targets

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Discussed with the PM - update made to the older HB locfiles and this will seed the TM when handedback


add separator for raw format config

https://github.com/streamnative/pulsar-io-cloud-storage/issues/147

@nlu90 the separator is to solve the byte collision issue with less interference in the output. If we use a List or Object to wrap all the records, it will need the user to deserialize the structure first, then retrieve the records, which is not like a raw or bytes format.

@freeznet Just curious to learn, whether it's possible that the selected separator will appear in the raw or bytes content, in which case, it's a collision also?

@nlu90 yes, this is just the collision issue I talked about above. Before this PR, the separator is been set as \n (0x10), and it is super easy to get the collision issue  So we encourage the user to set a long separator to avoid the collision.

To totally avoid the collision problem, is it possible to first write the length of the record bytes and then write the actual bytes? It has a scheme lenght_byte:padding_bytes | lenght_byte:padding_bytes | lenght_byte:padding_bytes|

@nlu90 yes, nice solution, that will help and solve the collision issue, but should we introduce it as a new format type? since it changed the bytes format a lot.


Python package being built is 0.8 not latest

Hi, I built the python package through docker & everything worked fine. However the version on this package is 0.8, not 0.8.9. Why is that the case? When I pip install mediapipe on my non aarch machine I will get the latest package 0.8.9, but with building my self I get 0.8.
I noticed in setup.py has version which is 0.8 & has not changed over the various version.

The setup.py we release is for customizing graphs and testing locally. We probably should put "local" or "dev" as the version tag to make this point clear. If you are looking for an official version, please go to https://pypi.org/project/mediapipe.


 But when you navigate away client-side then navigate back to the page containing the embedded checkout element, you get the error IntegrationError: You cannot have multiple Embedded Checkout objects.
Proposed Solution
This error seems to be an issue with the lifecycle management of the underlying Stripe Embedded Checkout element. The element does provide an interface to destroy it. We can wait for the wrapper element to be destroyed then manually destroy the Stripe Element. This seems to resolve the issue when I tested it locally.
This solution was brought up by github user @conorw.

Hi @unruha!
Looks good.
Can you add a semantic release commit message?
The commit message should contain fix:, that way the release will happen automatically

@joshnuss Thanks for taking a look! I've amended the commit with the semantic commit message.

Thanks @unruha! Much appreciated!


feat: add dropout for ascend

在dicp中为昇腾芯片增加 dropout 前向和反向算子

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


⚠ Secret Website 3 has degraded performance

In 90b9f12, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2017 ms

Resolved: Secret Website 3 performance has improved in a6eb389.


Content: make a list of activities and points of interest

DoD:

[x] There is a preliminary list of procrastination activities (and their short descriptions) that would be suggested to users.

Put it in the wiki.


bSendPacket not working as intended?

seems fakelag or anything to do with it doesn't work? I haven't changed anything yet it just doesn't work however if I use smth like
uintptr_t* framePtr;
__asm mov framePtr, ebp;
*(bool*)(*framePtr - 0x1C) = G::bSendPacket;

it works fine

Well just use this method then. Anyways how are you trying to change packet state with original method

how did u used it

how did u used it

i was just trying with the fakelag that is already in the source as well as a send packet indicator to see whether or not i was chocking or not

could you check after this commit? should works as intended

@HDTRPlayzZ-Git could you check after this commit? should works as intended

ill check now but instead im just getting it via proxy

thanks for your reply


[fix](csv reader) fix csv parser incorrect if enclosing line_delimiter (#38347)

Csv reader parse data incorrect when data enclosing line_delimiter, for example, line_delimiter is \n and enclose is ', data as follows:
'aaaaaaaaaaaa
bbbb'

it will be parsed as two columns: 'aaaaaaaaaaaa and bbbb', rather than one column
'aaaaaaaaaaaa
bbbb'

The reason why this happened is csv reader will not reset result when not match enclose in this output_buf_read, causing incorrect truncation was made.

run buildall


[Bug]: EfficientAD - Very slow training and bad predictions

Describe the bug
Don't know if it's actually a 'bug', but i'm getting weird results on efficientAD. (I have applied PR#1143 to be able to train on my larger custom dataset.)
Firstly, the training takes ages. Although i'm not using the best GPU in town, a RTX 3050ti, other models train a lot faster. One major aspect to this might be that a batch size of one has to be used. It took about 2 days to train for about 60.000 iterations.
Are you getting similarly slow training speeds?
Second, the actual learning curves are very bad. I guessed the model would perform quite well on my custom data because another student-teacher-based model, reverse distillation with OCBE, performed very well on it.
Unfortunately, it's very time intensive, with such long training times, to test different hyperparameters to see if it's just some badly chosen parameters (although i used the default ones for the important parameters).
Addtionally, the inference times aren't fast too as stated. about 1.5s/it.
I would be grateful if someone could take a look at my configs to find any misconfigurations or possible issues leading to that behaviour. Below i will share more relevant infos and graphs about the training process.
Dataset
Folder
Model
Other (please specify in the field below)
Steps to reproduce the behavior
Model: EfficientAD
not possible to reproduce because of missing custom training data (?)
OS information
OS information:
OS: win 10
Python version: 3.10.11
Anomalib version: 0.6.0
PyTorch version:
CUDA/cuDNN version: 11.6
GPU models and configuration: rtx3050ti
Any other relevant information: custom data
Expected behavior
Getting better pixel- (and image-)level auroc scores.
Screenshots

Pip/GitHub
GitHub
What version/branch did you use?
0.6.0 + pr #1143
Configuration YAML
dataset:
  abnormal_dir: bad
  center_crop: null
  extensions: null
  format: folder
  image_size: 256  # mandatory
  mask: ground_truth
  name: vespa
  normal_dir: good
  normal_test_dir: null
  normalization: imagenet
  num_workers: 8
  path: ./../data/images/data_structure_anomalib
  split_ratio: 0.2
  task: classification
  test_batch_size: 16
  test_split_mode: from_dir
  test_split_ratio: 0.2
  tiling:
    apply: false
    random_tile_count: 16
    remove_border_count: 0
    stride: null
    tile_size: null
    use_random_tiling: false
  train_batch_size: 16
  transform_config:
    eval: c:\Users\benedict\dev\VespaUseCase\machine_learning\src\d01_config\transform_config_1.yaml
    train: null
  val_split_mode: from_test
  val_split_ratio: 0.5
logging:
  log_graph: false
  logger: []
metrics:
  image:
  - F1Score
  - AUROC
  pixel:
  - F1Score
  - AUROC
  threshold:
    manual_image: null
    manual_pixel: null
    method: adaptive
model:
  lr: 0.0001
  model_size: small
  name: efficientad
  normalization_method: min_max
  padding: false
  teacher_out_channels: 384
  weight_decay: 1.0e-05
optimization:
  export_mode: onnx
project:
  path: ./results
  seed: 42
trainer:
  accelerator: auto
  accumulate_grad_batches: 1
  auto_lr_find: false
  auto_scale_batch_size: false
  benchmark: false
  check_val_every_n_epoch: 1
  default_root_dir: null
  detect_anomaly: false
  deterministic: false
  devices: 1
  enable_checkpointing: true
  enable_model_summary: true
  enable_progress_bar: true
  fast_dev_run: false
  gradient_clip_algorithm: norm
  gradient_clip_val: 0
  limit_predict_batches: 1.0
  limit_test_batches: 1.0
  limit_train_batches: 1.0
  limit_val_batches: 1.0
  log_every_n_steps: 50
  max_epochs: 200
  max_steps: -1
  max_time: null
  min_epochs: null
  min_steps: null
  move_metrics_to_cpu: false
  multiple_trainloader_mode: max_size_cycle
  num_nodes: 1
  num_sanity_val_steps: 0
  overfit_batches: 0.0
  plugins: null
  precision: 32
  profiler: null
  reload_dataloaders_every_n_epochs: 0
  replace_sampler_ddp: true
  strategy: null
  sync_batchnorm: false
  track_grad_norm: -1
  val_check_interval: 1.0
visualization:
  image_save_path: null
  log_images: false
  mode: full
  save_images: true
  show_images: false

Transform Config YAML
__version__: 1.3.1
transform:
  __class_fullname__: Compose
  additional_targets: {}
  bbox_params: null
  is_check_shapes: true
  keypoint_params: null
  p: 1.0
  transforms:
  - __class_fullname__: Resize
    always_apply: true
    height: 256
    interpolation: 1
    p: 1
    width: 256
  - __class_fullname__: RandomBrightnessContrast
    always_apply: false
    brightness_by_max: true
    brightness_limit:
    - -0.3
    - 0.3
    contrast_limit:
    - -0.3
    - 0.3
    p: 0.5
  - __class_fullname__: RandomSunFlare
    always_apply: false
    angle_lower: 0
    angle_upper: 1
    flare_roi:
    - 0.3
    - 0.4
    - 0.7
    - 0.6
    num_flare_circles_lower: 1
    num_flare_circles_upper: 2
    p: 0.3
    src_color:
    - 255
    - 255
    - 255
    src_radius: 100
  - __class_fullname__: Rotate
    always_apply: false
    border_mode: 4
    crop_border: false
    interpolation: 1
    limit:
    - -10
    - 10
    mask_value: null
    p: 0.5
    rotate_method: largest_box
    value: null
  - __class_fullname__: Affine
    always_apply: false
    cval: 0
    cval_mask: 0
    fit_output: false
    interpolation: 1
    keep_ratio: false
    mask_interpolation: 0
    mode: 0
    p: 0.3
    rotate:
    - 0.0
    - 0.0
    rotate_method: largest_box
    scale:
      x:
      - 1.0
      - 1.0
      y:
      - 1.0
      - 1.0
    shear:
      x: &id001
      - -5
      - 5
      y: *id001
    translate_percent: null
    translate_px:
      x:
      - 0
      - 0
      y:
      - 0
      - 0
  - __class_fullname__: ToTensorV2
    always_apply: true
    p: 1.0
    transpose_mask: false

Logs
No logs available

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Edit:
Added tranform config below config

You apply transformation during evaluation but not during training, is this intended?
You mention that other methods work better, can you explain this in more detail using the respective metrics?
Also you should not apply normalization to efficientad, this is done internally by the model already. If you provide transforms for eval and not for training, the default transforms for training will be used (that include normalization unless you set normalization: none)
Generally you should not change the default configy values if you are not exactly sure what you're doing I guess.

You apply transformation during evaluation but not during training, is this intended?
You mention that other methods work better, can you explain this in more detail using the respective metrics?
Also you should not apply normalization to efficientad, this is done internally by the model already. If you provide transforms for eval and not for training, the default transforms for training will be used (that include normalization unless you set normalization: none)
Generally you should not change the default configy values if you are not exactly sure what you're doing I guess. That also applies to the batch size that you set to train_batch_size: 16

I'm sorry, i didn't provide the correct transform config.. actually i applied transforms only during training, and normalization was turned off. Batch size was 1, too.
I guess the architecture is just not well suited for the domain.
Because of time constraints i cannot continue right now with working on tuning this model, so no pressure to get this right, but thank you anways!
I'm gonna close the issue and re-open it if necessary.


New art assets added

Added art assets to create the boar.
I read that wild boars IRL are docile animals however they can become aggressive when threated or their territory is trespassed upon. So my idea is that the boar is a neutral enemy until the player gets too close or attacks it.

The boar comes with two eye variants, one effect, and three material drops.
Closed eyes can be used to simulate blinking or resting and KO eyes can be used when boar is slain or to simulate dizziness.
The enraged effect can be used when the the boar is aggroed.

Boar leather, tusk, and hair can be used to craft various equipment.

Additionally I've added one food drop and consumable for pig. When pig is slain it will drop raw pork that can then be cooked and become cooked pork that can be eaten to replenish health, stamina, or whatever else is decided.

Merged, thx for your contribution!


OpenWrt 23.05.4 is working

Describe the issue
I can't install software in 18.06.1 with this error
root@(none):/# opkg update
Downloading http://downloads.openwrt.org/releases/18.06.1/targets/x86/legacy/packages/Packages.gz
Failed to establish connection
*** Failed to download the package list from http://downloads.openwrt.org/releases/18.06.1/targets/x86/legacy/packages/Packages.gz

Downloading http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/base/Packages.gz
Failed to establish connection
*** Failed to download the package list from http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/base/Packages.gz

Downloading http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/luci/Packages.gz
Failed to establish connection
*** Failed to download the package list from http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/luci/Packages.gz

Downloading http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/packages/Packages.gz
Failed to establish connection
*** Failed to download the package list from http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/packages/Packages.gz

Downloading http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/routing/Packages.gz
Failed to establish connection
*** Failed to download the package list from http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/routing/Packages.gz

Downloading http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/telephony/Packages.gz
Failed to establish connection
*** Failed to download the package list from http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/telephony/Packages.gz

Collected errors:
 * opkg_download: Failed to download http://downloads.openwrt.org/releases/18.06.1/targets/x86/legacy/packages/Packages.gz, wget returned 4.
 * opkg_download: Check your network settings and connectivity.

 * opkg_download: Failed to download http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/base/Packages.gz, wget returned 4.
 * opkg_download: Check your network settings and connectivity.

 * opkg_download: Failed to download http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/luci/Packages.gz, wget returned 4.
 * opkg_download: Check your network settings and connectivity.

 * opkg_download: Failed to download http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/packages/Packages.gz, wget returned 4.
 * opkg_download: Check your network settings and connectivity.

 * opkg_download: Failed to download http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/routing/Packages.gz, wget returned 4.
 * opkg_download: Check your network settings and connectivity.

 * opkg_download: Failed to download http://downloads.openwrt.org/releases/18.06.1/packages/i386_pentium/telephony/Packages.gz, wget returned 4.
 * opkg_download: Check your network settings and connectivity.

How did I get network working in 23.05.4?
root@OpenWrt:/# uci show network
network.loopback=interface
network.loopback.device='lo'
network.loopback.proto='static'
network.loopback.ipaddr='<IP_ADDRESS>'
network.loopback.netmask='<IP_ADDRESS>'
network.globals=globals
network.globals.ula_prefix='fd2a:ab97:4ae0::/48'
network.lan=interface
network.lan.device='br-lan'
network.lan.proto='static'
network.lan.ipaddr='<IP_ADDRESS>'
network.lan.netmask='<IP_ADDRESS>'
network.lan.ip6assign='60'
network.wan=interface
network.wan.device='eth0'
network.wan.proto='dhcp'

But there is an issue with lan, I can't connect to luci from other system. I tried to set network.lan.ipaddr to <IP_ADDRESS>, but br-lan isn't active.

I did it!

root@OpenWrt:/# uci show network
network.loopback=interface
network.loopback.device='lo'
network.loopback.proto='static'
network.loopback.ipaddr='<IP_ADDRESS>'
network.loopback.netmask='<IP_ADDRESS>'
network.globals=globals
network.globals.ula_prefix='fd2a:ab97:4ae0::/48'
network.lan=interface
network.lan.device='eth0'
network.lan.proto='static'
network.lan.ipaddr='<IP_ADDRESS>'
network.lan.netmask='<IP_ADDRESS>'
network.lan.ip6assign='60'
network.wan=interface
network.wan.device='eth0'
network.wan.proto='dhcp'

This helped me.
Also don't forget adding port 80 to address bar :)

Cool!


Support Azure Service Bus

Implement an IBroker to integrate with Azure Service Bus.

any idea when this will be started?

It's not planned yet. It's not a technology I use in my day-to-day work, and I'm already struggling to find the time to work on the next major release, continuing to evolve the core, Kafka, and MQTT.
Can you maybe contribute it?

I can give it a try


Sync speed

Hi,
Firstly, thanks for this project. I have been using it a few months and I am really happy with it. I have an issue though that I couldn't fix yet. I use it to make a backup from my local pc to onedrive. The bidirectional functionality is handy because sometimes I modify things in onedrive (with a tablet) and I want them to be copied to the pc when I do the backup.
Everything works great but the process is very slow (I have 34k files and 58gb). The thing is that copying the files from A to B and from B to A is fast, but then, the program makes a second file refresh, and the one in B (onedrive), takes a long time (like 20 minutes). What I don't understand is why the first file refresh at the beginning of the process is fast (it takes 2 minutes more or less), and this second one is so slow. Is there anyway to configure the program to speed this up?
Thanks,
Pablo

Hi.
I am glad the program is useful to you. I also use it with OneDrive (115k files, 344.72 gb) so I can probably be more helpful in tuning.
Your question

a second file refresh, and the one in B (onedrive), takes a long time

This is almost certainly OneDrive throttling. There is no reason one is fast and the other is not from the syncrclone perspective.  It is the identical call.
OneDrive is annoying about this and I've noticed it too. It doesn't have listr functionality (to use --fast-list) so rclone has to iterate over every directory.
 That helps since nobody else will be using OneDrive on it but you. This is part of configuring your rclone remote.
What are your flags. I have the following set: ['--onedrive-list-chunk' , '5000' , '--checkers' , '15']. That works pretty well  for me.

Future Enhancements
At the end of the sync, you need to know what the remotes look like hence the second file listing. I already optimize it to not re-list if nothing changed on one side but any change will cause a full re-list.
The thing is, I theoretically could eliminate this by using the initial listings and propagate the changes from one to the other. I am not opposed to some-day considering that but it introduces a lot of edge cases (including ones I don't even know how to test).
For example, if you use hashes for move tracking and but not sync, you now could not have a compatible hash anymore and it'll be really hard to fix it. And if you reuse_hashes they won't ever get fixed (e.g. SFTP remote to Dropbox). Or if you use inodes (local only and I may deprecate that anyway), then you don't have them. Finally, I've had bugs with OneDrive where the file listings is wrong. This may actually fix that but I'd rather know something is fishy.
None of these are insurmountable but I worry about adding so many edge cases. And again, that will take a lot of time. I had some a few weekends ago and added the faster actions (OneDrive is super slow with some of that) but again, that doesn't help the re-list.
Future Alternatives
Rclone itself is going to come with a bi-directional sync command in the next major release. I am curious if they figure out how to speed it up or not. I will say though, I like my algorithm more (of course, I am biased). I think it is more robust and has fewer edge cases. Essentially, theirs uses the old file listings to propagate changes whereas mine using the previous state only to resolve conflicts. It is much simpler and resilient to "bad" (or missing) previous list without accidentally deleting everything, But alas, I do not know golang, nor have the time to learn and implement it. So they are going forward with the other algorithm.
I also think that since bi-directional sync is inherently  stateful (while mirroring is not), you need to be careful about how you track history and ensure you sync the whole directory every time. Hence my choice in a config-file approach vs a full CLI.
I am curious how they will handle cases like your original sync being A: to B: but then later doing A:subdir to B:subdir. I guess we will see.

I hope this helps.

Thank you so much. After updating to rclone 1.56 and using your flags I have solved the problem. The whole process takes now a couple of minutes (plus the file transfers that may be needed). Thank you!

Hi. I meant to update this a while ago.
I ended up doing that future enhancement. If you update syncrclone, update the config (see tips), then set
avoid_relist = True

and it will avoid the second listing.
There are some minor caveats but it works great for the most part.

That looks great!


Fix Pickling Issue with SWA and AdvancedProfiler

What does this PR do?

Fixes #6522
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃
I am not sure, why the AdvancedProfiler cannot be pickled. There seems to be an issue with a local function. But I was not able to find any. So this doesn't fix that but bypasses it!

Hello @justusschock! Thanks for opening this PR.

In the file pytorch_lightning/callbacks/stochastic_weight_avg.py:

Line 140:1: W293 blank line contains whitespace

Do see the Hitchhiker's guide to code style

@justusschock Could we add a test for this, of Pickling not having an Issue?

@kaushikb11 I'll try to come up with one, but not sure, how good it will be since this is something non-trivial to test

Hey @justusschock,
I would say the bug should be resolved on the profiler side, not the callback side.
Best,
T.C

The issue will be partly resolved by https://github.com/PyTorchLightning/pytorch-lightning/pull/6621
However, we still have the problem of not being able to pickle cProfiler.Profile
Can reproduce it with this test in test_profilers.py
def test_advanced_profiler_cprofile_deepcopy(tmpdir):
    """Checks for pickle issue reported in #6522"""
    model = BoringModel()
    trainer = Trainer(
        default_root_dir=tmpdir,
        fast_dev_run=True,
        profiler="advanced",
        stochastic_weight_avg=True,
    )
    trainer.fit(model)


microk8s 1.27 error accessing EKS 1.26?

Summary

$ k version
Error processing line 1 of /usr/lib/python3/dist-packages/google_auth-1.5.1-nspkg.pth:

Fatal Python error: init_import_site: Failed to import the site module
Python runtime state: initialized
Traceback (most recent call last):
  File "<frozen site>", line 192, in addpackage
  File "<string>", line 1, in <module>
  File "<frozen importlib.util>", line 2, in <module>
ModuleNotFoundError: No module named 'importlib._abc'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "<frozen importlib._bootstrap>", line 982, in exec_module
  File "<frozen site>", line 635, in <module>
  File "<frozen site>", line 622, in main
  File "<frozen site>", line 403, in addsitepackages
  File "<frozen site>", line 232, in addsitedir
  File "<frozen site>", line 202, in addpackage
  File "/snap/microk8s/5250/usr/lib/python3.8/traceback.py", line 5, in <module>
    import linecache
  File "/snap/microk8s/5250/usr/lib/python3.8/linecache.py", line 11, in <module>
    import tokenize
  File "/snap/microk8s/5250/usr/lib/python3.8/tokenize.py", line 32, in <module>
    import re
  File "/snap/microk8s/5250/usr/lib/python3.8/re.py", line 125, in <module>
    import sre_compile
  File "/snap/microk8s/5250/usr/lib/python3.8/sre_compile.py", line 17, in <module>
    assert _sre.MAGIC == MAGIC, "SRE module mismatch"
           ^^^^^^^^^^^^^^^^^^^
AssertionError: SRE module mismatch
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.1", GitCommit:"4c9411232e10168d7b050c49a1b59f6df9d7ea4b", GitTreeState:"clean", BuildDate:"2023-04-25T19:00:00Z", GoVersion:"go1.20.3", Compiler:"gc", Platform:"linux/amd64"}
Kustomize Version: v5.0.1
Unable to connect to the server: getting credentials: exec: executable aws failed with exit code 1

inspection-report-20230502_170548.tar.gz
What Should Happen Instead?

Reproduction Steps

...
...

Introspection Report

Can you suggest a fix?

Are you interested in contributing with a fix?

Upgrade awscli to the latest version


Setup reports wrong requirements

The Installer reports false information, Hyper-V disabled - No but Hyper-V is deactivated. Same goes for EPT, EPT is support but it reports it isn't aviable

Paste a screenshot of the installer screen with these statements. Paste also your cpu model and host OS.

Thanks for your report. You may follow the wiki page to set up the system environment first and try to install HAXM then. Thanks.

Paste a screenshot of the installer screen with these statements. Paste also your cpu model and host OS.

Intel Core i7-9700 supports VMX and EPT. Disable Hyper-V as described in the mentioned wiki page. Hopefully this will solve the issue.

It worked! Thanks!


Data not updating after put transformation

I'm having an issue getting the data to be added to the database with the actual transformed data.
The following code demonstrations the issue I'm having:
'use strict';

var _ = require('lodash');
var dat = require('dat');
var through = require('through2');

var products = [
  'PROD-001',
  // 'PROD-002',
  'PROD-003',
  // 'PROD-004',
  // 'PROD-005',
  // 'PROD-006',
  'PROD-007',
  'PROD-008',
];

var transforms = [
  function () {
    return through.obj(function features (product, enc, next) {
      console.log('features', product);
      product.features = (product.features || []).concat([
        { color: 'red',   sizes: ['S', 'M', 'L', 'XL', 'XXL'] },
        { color: 'blue',  sizes: ['S', 'M', 'L', 'XL'] },
        { color: 'green', sizes: ['M', 'L'] }
      ]);
      product.transformed = (product.transformed || []).concat(['doowb-features']);
      // this.push(product);
      next(null, product);
    });
  },
  function () {
    return through.obj(function PROD1 (product, enc, next) {
      console.log('PROD1', product);
      if (product.id === 'PROD-001') {
        product.cost *= 0.80;
        product.transformed = (product.transformed || []).concat(['doowb-PROD1']);
      }
      // this.push(product);
      next(null, product);
    });
  },
  function () {
    return through.obj(function PROD8 (product, enc, next) {
      console.log('PROD8', product);
      if (product.id === 'PROD-008') {
        product.cost *= 0.50;
        product.transformed = (product.transformed || []).concat(['doowb-PROD8']);
      }
      // this.push(product);
      next(null, product);
    });
  }
];

var userDB = dat('./data/products/doowb', function (err) {
  if (err) {
    throw new Error(err);
  }

  var relatedDB = dat('./data/products/jon', { transformations: { put: transforms } }, function (err) {
    if (err) {
      throw new Error(err);
    }

    // get the products from userDB and pass them through the transforms
    var readOpts = {
      start: 'PROD-001',
      end: 'PROD-008'
    };

    var userStream = userDB.createReadStream(readOpts);
    userStream.pipe(through.obj(function (product, enc, callback) {
      var stream = this;
      if (!product || _.contains(products, product.id) === false) {
        this.push(product);
        return callback();
      }
      console.log('transforming product', product);

      relatedDB.put(product.id, product, {force: true}, function (err, updated) {
        if (err) {
          throw new Error(err);
        }
        console.log('Product ', updated.id, 'updated in related database');
        console.log(updated);
        stream.push(product);
        callback();
      });

    }));
  });

});

I have 2 different databases. The first userDB is using createReadStream to get a subset of rows, then put those rows into the second relatedDB.
When I add the transformations as options on userDB like {transformations: { get: transformations}} it works fine because the data is changed after the get, then passed into the relatedDB with .put.
When I run it with transformations as options on relatedDB like {transformations: { put: transformations}}, it runs through the transformations and I see the output of the modified data, but the updated record doesn't contain the changes.
I run dat cat from the command line to check and none of the data gets updated correctly.
From looking at the source code, it looks like transformations should be running, but I don't see it happening.
Thanks for any help.

Hey @doowb, we have re-written dat and have merged the new changes into master. Would you like to give dat another spin and see if you can find any bugs? Check out the
cli api
whitepaper overview draft
and the javascript api
high-five I'll be watching dat issues and available in #dat on freenode if you want to talk to a human


Implement linear regression for [ArithmeticType]

It could be very helpful for testing larger sets of data to notice trends.
More specifically, when testing error in the Timeline, it could be nice to ensure that there isn't an accumulated drift.
See example implementation.

Closed by 862d8785faaa0b83651fd08ee356be84d4c562b7.


Expression browser states two time series returned for 0/ 1000*10^6

To reproduce, simply run 0/ 1000*10^6 in the expression browser.
It will state Total time series: 2 when it only displays scalar 0

It's wrong for all scalars, as it's counting the value and the timestamp as 2 in the response.
@Conorbro


Fixes iobroker-community-adapters/ioBroker.firetv#56

This should fix issue caused by the scope change of ar from the last commits

LGTM

See my comment, not for me ... in fact the old cb also needs to stay for "just" error cases, else users are stuck

See my comment. I'm not sure if error cases where handled correctly before, but of course we could add that now

updated to also call callback on error. Alternatively one could let ar earlier instead of using const

ggf noch ein leerzeichen dazu ... sonst LGTM


keccak256 tests failing

Maybe this is a known temporary issue in keccak256 assembly procedure since the tests were ignored but tests keccak256_2_to_1_hash and keccak256_bit_interleaving are failing.
Error Message:
---- stdlib::crypto::keccak256::keccak256_2_to_1_hash stdout ----
thread 'stdlib::crypto::keccak256::keccak256_2_to_1_hash' panicked at 'Failed to compile test source.: assembly error at 709: dangling instructions after module end at std::crypto::hashes::keccak256', miden/tests/integration/helpers.rs:134:14

---- stdlib::crypto::keccak256::keccak256_bit_interleaving stdout ----
thread 'stdlib::crypto::keccak256::keccak256_bit_interleaving' panicked at 'Failed to compile test source.: assembly error at 709: dangling instructions after module end at std::crypto::hashes::keccak256', miden/tests/integration/helpers.rs:134:14

Closed by #227


Rename module.fileName to module.filename to match Node.js behavior

https://github.com/svaarala/duktape/pull/639#issuecomment-201111627
https://nodejs.org/api/modules.html#modules_module_filename

I don't know what the current behavior is, but I'd probably also expect module.filename to be undefined if modSearch doesn't explicitly set it (since in that case Duktape doesn't know what the filename is, or if it even originated from a file at all).  Tracebacks of course should default to the module ID in that case (as they do now).

By the way, unrelated but does Duktape currently support module.loaded?  That seems like a useful thing to have that would be trivial to implement.

Short answer: I've been trying to avoid feature creep ;)
Long answer:
For now I've tried to add a minimum of custom fields beyond CommonJS based on concrete needs arising so that footprint and feature creep is avoided for very low memory targets. Unlike several other components it's possible (and quite straightforward even) to replace the entire module loading mechanism with a custom one - so every feature should defend its place, being weighed against the option of using a replacement loader.
The current plan has been for the built-in module loading mechanism to be as barebones and abstracted as possible, but that a separate "deluxe" module loader would also be included in the distributable. The deluxe loader wouldn't port to all environments but would include, for example, a native module DLL loader for all major platforms. It could then also provide a much larger set of module properties to match the behavior of e.g. the Node.js loader.
In fact, I'd actually prefer removing the built-in module loader entirely and only ship a few optional loaders in the distributable because not all environments need module loading. But because ES6 has a module mechanism, Duktape does need some module mechanics to be built in.
The first step in this process would be to figure out what features can be removed from the current loader (keeping ES6 in mind), and then implement the first versions of those optional loader(s), at least the deluxe loader with DLL support which is often requested.

Adding to the above, I think .fileName (or .filename if renamed) and .name defend their place for being included by default, because they concretely affect at least tracebacks and debugger integration. Especially tracebacks are quite important for developers so it seems useful to include them despite the (relatively small) footprint cost.

Yeah, I know you're trying to keep this as minimal as possible. :)  I mentioned module.loaded specifically because it seemed like it would be practically free to implement, both in implementation cost as well as footprint.  You're just setting a property to false when loading the module and flipping it to true afterward, so it shouldn't amount to more than a few bytes of additional code (in the single digits, most likely).
That said, while it would be possible to replace the built-in require() with a polyfill which set module.loaded itself, with the current architecture (single entry point, modSearch), the entire loader would have to be replaced wholesale in user code, which isn't really ideal (you lose automatic module ID resolution, among other things) and results in a LOT more code than if Duktape just sets the flag itself.
This I see is another strength of the planned changes for Duktape 2.0, with split modResolve/modLoad.  That will make it easier to plug in pre-loading logic such as this without having to reinvent the wheel.

I mentioned module.loaded specifically because it seemed like it would be practically free to implement, both in implementation cost as well as footprint. You're just setting a property to false when loading the module and flipping it to true afterward, so it shouldn't amount to more than a few bytes of additional code (in the single digits, most likely).

The cost is very small but probably around 60-80 bytes on a 32-bit platform for one module: two calls, a new built-in string, and RAM overhead of storing the loaded flag for each module which is 13 bytes * module count. Almost nothing is single digits (9 bytes), that's barely enough to make one function call. 60-80 bytes taken alone is around 5-7 fewer user strings (for example) for a low memory target: very small, but of course the creep adds up over time.

That said, while it would be possible to replace the built-in require() with a polyfill which set module.loaded itself, with the current architecture (single entry point, modSearch), the entire loader would have to be replaced wholesale in user code, which isn't really ideal (you lose automatic module ID resolution, among other things) and results in a LOT more code than if Duktape just sets the flag itself.

Yeah, but the idea would be to include a deluxe replacement directly in the Duktape distributable. The user wouldn't implement it from scratch naturally. I don't mind that implementation providing a lot of features because you can easily replace it with a barebones one on a low memory target.

This I see is another strength of the planned changes for Duktape 2.0, with split modResolve/modLoad. That will make it easier to plug in pre-loading logic such as this without having to reinvent the wheel.

Depending on how that goes, it may be the case that Duktape only provides a require() function which calls into user code to do both resolution and loading. As a result, user code will need to implement the ID resolution (which you mentioned above as a drawback :).
Duktape could also ship with a default resolver, but because not all environments need module loading or a proper resolver even when using modules, so my gut feeling is that while one or two full featured loaders should be included with Duktape, the hardcoded built-in feature should just be the loader framework without an implementation.

That's true.  Being primarily a C programmer, I was estimating the footprint cost in terms of:
module->loaded = false;
/* load module */
module->loaded = true;

Which of course is pretty much free.  But I forgot that the Ecmascript environment necessarily adds a lot of overhead to that.  60-80 bytes is of course rather significant when you're fighting for kilobytes.

By the way, the reason user code needing to do module ID resolution is a pain is because string parsing isn't really one of C's strong suits.  If require() is implemented in JS that's less of an issue since you can use regular expressions etc. to make things easier, but that also requires file bindings so it's usually more convenient to implement the module loading directly in C.

Commenting a bit more on what could be included by default, assuming a split resolve/load model:

Duktape provides nothing. User code must provide a require() to get CommonJS and ES6 modules working. Distributable includes a default portable require() and a deluxe require() with DLL support.
Duktape provides a skeleton require() which relies on user resolve/load functions but neither is provided automatically. Distributable includes a resolver and a few loaders.
Duktape provides a skeleton require() and a default resolver but no loader. Distributable includes a few loaders and user can override the resolver for non-CommonJS behavior (like understanding file paths).
Duktape provides a require(), a default resolver, and a default loader which works on mainstream platforms (perhaps including DLL support); config options are needed for non-mainstream platforms to disable the loader and use a custom one. Upside is that no additional files need to be included in a project.

Assuming there's nothing magic about require() right now I'd be leaning towards the first option: it's best for portability and footprint, and because you need to implement a modLoad() and possibly a modResolve() anyway (for non-CommonJS semantics which are nevertheless quite common), why not simply provide the resolver and loader(s) in the distributable?
Another upside of that would be that the resolve/load logic (i.e. what split is used) wouldn't concern Duktape at all: it would only assume the user provides a require() with much simpler guarantees and what goes on after that is up to the user. That's what DukLuv actually already does: it implements the resolve/load on top of Duktape 1.x by replacing require(). If Duktape came with a default loader, it'd be pretty trivial to use, and you could choose from a portable loader and a DLL-supporting deluxe loader.
Yet another reason which makes me lean this way (at least now) is that users would like Duktape to ship with a loader that supports DLLs for Linux, OS X, and Windows, so that it'll be possible to easily write native loadable modules. That loader can't realistically be inside the Duktape core because it'd be a portability issue - so it will be external but possibly shipped with Duktape. But if that's the default loader for most people, and it's already external, why include anything in the Duktape core?

I have similar feelings about the logging framework by the way: it was probably a mistake to include it by default. It'd be better to define the API, and ship the default implementation as a separate source file because many enviroments don't use the logging framework or may replace big parts of it. So while it's good to have a shared API you, it'd probably be best if the implementation was always external (with a default implementation included in the distributable).
While this is not ideal for embedding Duktape, instead of a single source file you have maybe 2-3, the advantage is that these optional modules don't have the same footprint/feature pain as all features in the core parts.

I thought ES6 didn't have require() at all but instead used import/export keywords?  i.e. it's built into the language and therefore is magic.

Yeah, that's I meant above is that the key question is how require() and ES6 modules work together - and what primitive is the smallest possible interface to make both work. Basically having a require+ which handled both while making minimal assumptions about anything else (such as how ID resolution and loading is structured) would be ideal.

So while Duktape needs to deal with CommonJS require() calls and ES6 import/export, it could call into user code as early as possible. Offloading complicated resolution logic there shouldn't be a big issue considering (1) a default implementation would be included, and (2) you need to be pretty involved anyway to get something like DLL loading to work as is. So why not push the resolution/loading algorithm also into user code?
In concrete terms, including a default loader with DLL support and an ordinary filesystem search path would probably make it easier to get into using modules compared to the current situation of explaining modSearch() and coming up with one :) So this is quite likely more important than what the division of work is between require, modResolve, modLoad, etc.

Yet another thing to add to the Duktape 2.x to-do list I guess. :)

I'm actually pretty much ready to start work on Duktape 2.x once 1.5 is done :) 1.5.x maintenance will then go on for a long while, and I think it's quite OK as a baseline until 2.0.

I agree, AppRequest/AppNotify will go a long way in making the debugger extensible for me so I don't need to ask for more debugger features in 1.x. :)

The Node.js semantics for .filename are:

The fully resolved filename to the module.

This is not exactly the same as Duktape .fileName whose semantics are:

Filename which should be used for the internal module wrapper function and appears in tracebacks, etc.

In particular, the .fileName doesn't have to be an absolute path, and may be "censored" (for example: [app]/foo/index.js).
In any case it's probably best to rename the Duktape key. If the semantic differences are an issue going forward, we can add an explicit .moduleWrapperFilename for the module wrapper specific semantics (and if missing, default to .filename which is a good default).

I think the semantics not matching is fine.  Node.js differs in other ways too, for example require() module resolution is file-based and so on.  Since Duktape can run even on systems without a file system, it seems okay if module.filename is, well, not a filename. :)

As an example where this difference would matter: suppose you wanted to add extra properties to the module table so that it'd be closer to Node.js behavior. In doing so, you'd like to add .filename with Node.js semantics, i.e. provide the full file path. However, you wouldn't be able to do that without affecting how the module looks like in tracebacks. Ideally these two issues wouldn't need to be tied.


No update icon

I've just noticed that I'm way behind on updates. There are no "update" icons showing in the Webmin interface. (For both Theme and Webmin) Not sure what to do to correct this.  I've checked using both Chrome and Firefox.
Nigel Aves.

Hello,
This is expected, as Virtualmin repos haven't received Webmin update yet.

Illia,
That's a tad confusing when you have one icon that says there is an update!
:)
Nigel.
On Wed, Jul 20, 2022 at 10:00 AM Ilia @.***> wrote:

Hello,
This is expected, as Virtualmin repos haven't received Webmin update yet.
—
Reply to this email directly, view it on GitHub
https://github.com/webmin/authentic-theme/issues/1607#issuecomment-1190464872,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIBBPGXNITBTBXWYI377ODDVVAPCLANCNFSM54EAP6OQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

Any update on this?


Adds materialized views to list of relations in GET / [#242]

Still does not implement the OPTIONS for materialized views, but I figured that could go on another PR.
A nice side effect of using the pg_catalog is:
Planning time: 0.721 ms
Execution time: 0.655 ms

Whereas the previous implementation using information_schema in the same system was:
Planning time: 2.063 ms
Execution time: 1.245 ms

Love it, you're a coding machine. :factory:
Can you throw a line in the changelog about it too?


Certain value types serializing as binary?

So I'm just trying this out for the first time and I noticed that types like Guid and DateTimeOffset were getting serialized as binary blobs and/or byte arrays. Is that normal? I would have thought they would have called ToString() before storing or something.

You can use an attribute (for example: a BsonDateTimeOptions attribute) to change the serializiation to your needs. This is not something MongoRepository provides but provided by the underlying mongo-csharp-driver. You can find documentation here.

"I would, however, advice to use the 'binary blob' unless you have a very specific and good reason to want to serialize to string."
Not sure how I'd query on binary...
I'll look into these other things. Thanks!

Not sure how I'd query on binary...

If you store an object:
class MyObject : Entity {
    public Guid SomeProperty { get; set; }
}

var g = Guid.NewGuid();
var x = new MyObject { SomeProperty = g; }

// Save object
myrepo.Update(x);

You can simply query it like this:
myrepo.Where(o => o.SomeProperty == g);

(Code written on the fly; may contain some (syntax)errors).
If you want to query it from other platforms you can query it with something like:
{
    "SomeProperty": new BinData(3, "MSlQ8XxCwE6I9tCqLtXy9g==")
}

Where MSIQ....y9g== is the Base64 representation of the GUID; this may differ for other datatypes like datetime which would be something like:
{
    "SomeProperty": ISODate("2016-10-20T15:23:11Z")
}

But unless you write your queries 'manually' like in the above format MongoRepository (actually the underlying driver) should take care of it.


POI Updates

Thread for additional POIs that need to be added.

Big Buns - sells craft beer and hosts craft beer events (DT)


TE-2635 More logging on test failures, v0.8.0

Logging improvements as noted in the CHANGELOG.  Hopefully this will help diagnose some of the recent flaky test failures in edx-platform.

@jmbowman This is PR is in prod?

Yes, it's already fully gone through the release process: https://github.com/edx/edx-platform/pull/18617 .  I forgot to upgrade it in edx-e2e-tests also, though; just opened a PR for that: https://github.com/edx/edx-e2e-tests/pull/300


Dehypenisation is removing hypens in chemical formulas in certain cases

The input PDF is like this:

the data arrives correctly to grobid:

The issue seems to be line 1233 of TEiFormatter.java, when calling the dehypenisation:
String clusterContent = LayoutTokensUtil.normalizeDehyphenizeText(cluster.concatTokens());

here the result text:
[...] low cost rare-earth element) based (Nd 1x Ce x ) 2 Fe 14y Co y B nanostructured magnets. [...]
I'm working on this issue, I will add details here while I'm moving forward, later I will send a PR.
Update:
First issue is here, where we assume that text in the same line has the exact same Y coordinate:
if (breakLine == 0) {
            // check if there is a break-line using coordinates, if not, no dehypenisation
            if (j < tokens.size() && tokens.get(j).getY() == coordinateY) {
                return false;
            }
        }

Here the Y coordinate is as follows:

Nd:  406.746
1: 410.506
-: 410.506
x: 410.538

Finally it ends at line 284 where the coordinates of - and x are compared and the Y of x is > Y of -:
                //I check if the coordinates have changed, this means there is a newline
                if (tokens.get(j).getY() > coordinateY) {
                    return forward;
                }

so the tricky part is to estimate correctly what is the amount of Y coordinate that is reasonable for a new line.
One quick solution could be to do not de-hypenise if the sequence is all subscript or all superscript

Another case, in the same paper:

we have 14-y at the end of line.

Right now the dehypenisation is activated only when a "dash" is encountered. I've add a condition that requires the dash to be neither superscript or subdscript


Web server file structure on startup #48

From https://github.com/NeRF-or-Nothing/vidtonerf/issues/48
alexandorr55 commented https://github.com/NeRF-or-Nothing/vidtonerf/issues/48#issue-1450457373
Right now the webserver does not initialize it's file structure resulting in errors if the user does not manually define the folder structure. This should be handled automatically on start up.

Should be handled by V3. go-web-server verifies its file structure on startup now. sfm-worker and nerf-worker probably should receive the same treatment


towncrier fixups for 0.58.0rc2

Run towncrier build to remove the snippets from upcoming_changes. Copy old CHANGE_LOG into sphinx docs from root of repo for conservation. Fixup the release-notes.rst file to include toctrees for both the new and shiny towncrier notes and the conserved notes. Finally, replace towncrier output with the existing fixups for 0.58.0rc1 that were made on the release0.58 branch.

This makes it such that, from now on Towncrier generates release notes and places them in releases. The old CHANGE_LOG file is preserved. Also, the Towncrier generated notes now have a table of contents at the start:

superseded by https://github.com/numba/numba/pull/9173


JSON-LD Tests should use local jsonld loader

We don't want tests to depend on internet connectivity. The w3c credential service already depend on the local jsonld loader, but it seems the new jsonld credential format service don't.

@NB-MikeRichardson is this something you could pick up?

Sure. I can do that.

Should we just add this to the list of items in issue #1089

Actually I think just leave this issue as it is: I'll do this one first and get a PR in for this issue only.

If I run the w3cCredentialService tests (in modules/vc/tests/W3cCredentialService.test.ts) with my internet connection turned off I get the jsonld.invalidUrl error so I am not sure what I am trying to achieve with this issue.

It looks like the custom loader has a call to jsonld.frame for did: URLs :-
    if (url.startsWith('did:')) {
      result = await jsonld.frame(result, {
        '@context': result['@context'],
        '@embed': '@never',
        id: url,
      })
    }

This code (deep inside the digital credentials library is resolving a context by going across the internet).
I tested by bypassing this code and hard-wiring the result and it works.
Not sure what is wrong with the above: I shall see if i can find out.

There is an issue (Issue #1111 ) which documents the problem with the loader

This issue can now be closed


Add releases to CI jobs

Right now, releases are done locally (and are a bit manual). Try to reduce manual intervention and move to CI like our other libs.

From @andrewxhill , this looks promising: https://github.com/azu/lerna-monorepo-github-actions-release


Unable to Access Wave-ML Docs

Wave SDK Version, OS
Current Wave documentation - v0.14
Actual behavior
Click the sub-module and get a 404: https://wave.h2o.ai/docs/api/h2o_wave_ml/index/ml
Expected behavior
Click the sub-module and end up on docs page: https://wave.h2o.ai/docs/api/h2o_wave_ml/ml/
Steps To Reproduce

Start in the Wave-ML section of the 0.14 release blog post: https://wave.h2o.ai/blog/release-0.14#wave-ml
Click Wave-ML

3a. Click the first link h2o_wave_ml to be redirected to the page you're already on
3b. Click h2o_wave_ml.ml submodule to get a 404 https://wave.h2o.ai/docs/api/h2o_wave_ml/index/ml

@mtanco thanks for catching this.
The error is not Wave ML related only. Try this, go here: https://wave.h2o.ai/docs/api/index and click on random submodule. You should get the same error. However, If you go through the left menu, you should get the page without error (Package h2o_wave -> h2o_wave.core).
The problem is that the direct link, after landing, is changed to contain the / i.e:
https://wave.h2o.ai/docs/api/index

is changed to
https://wave.h2o.ai/docs/api/index/

That is enough to mix up relative links submodules are pointing to.
IMO the issue should be resolved on the webserver site. There is no such misbehavior when docusaurus run locally. I don't know the details of how the page is served now but I guess there is some auto-slash option enabled.
Is it still served from github?
cc @lo5 @mturoci

We're on S3 now. See and https://github.com/facebook/docusaurus/issues/2394 and https://github.com/facebook/docusaurus/issues/3372


added usage of subtests for rotational cipher

One of #2098

CI failed here: https://github.com/exercism/go/runs/8042103759?check_suite_focus=true
Added a dummy change to run CI again and it helped


Uptime page not work!

http://stats.pingdom.com/htbkuegtgycv is broken for a while, @thomasdavis would you please take a look at it? thanks.

What conclusion did you come to @ryankirkman ?

Fixed: https://github.com/cdnjs/new-website/commit/b42afad542d73709155570a57c728c85785317d9

thanks.


Update UI to use current API

Grasshopper returns data for a couple of underlying geocoding services now, UI application needs to be updated

Closing, we're using the services now. Will probably need a new issue created after some of the geocoder API refactoring.


[Bug]: 404 error at URL: /txns/dzcq5-vDNvbUpSsbJ-3L7cgThRf6mhHPLJOPO3XZnaU

What happened?
when trying to register the hotspot within the helium network does the transaction it throws me a ded hash because the transaction is canceled and it says not find a transaction with that hash.
URL
https://explorer.helium.com/txns/dzcq5-vDNvbUpSsbJ-3L7cgThRf6mhHPLJOPO3XZnaU
What browsers are you seeing the problem on?
Chrome
Do you see the issue on Mobile or Desktop?
Mobile
Relevant log output
404 — Transaction Not Found

Double check the transaction hash

Transaction hash didn't match any transactions

working now. probably clicked on the transaction too quick and ETL did not see it yet.


Unable to login with MicrosoftAccount

node:3103) UnhandledPromiseRejectionWarning: MicrosoftAccountLogin: Unable to login with MicrosoftAccount.
  caused by GetLiveToken: Unable to get the Live token for Skype
  caused by LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error may be caused by a change in the Microsoft login workflow. HTML page: "<!-- Copyright (C) Microsoft Corporation. All rights reserved. --><!DOCTYPE html><!-- ServerInfo: BY1PPF45C32AC58 2<IP_ADDRESS>.32.58 Live1 Unknown LocVer:0 --><!-- PreprocessInfo: azbldrun:AzBuildCU-Ha11, 2019-12-11T17:27:10.0518177-08:00 - Version: 16,0,28426,6 --><!-- RequestLCID: 1033, Market:EN-US, PrefCountry: US, LangLCID: 1033, LangISO: EN --><html dir=\"ltr\" lang=\"EN-US\"><head><meta http-equiv=\"Content-Type\" content=\"text/html; charset=utf-8\"/><meta http-equiv=\"X-U

Have you found a solution?

Hello @devk1d
Did you found any solution for the below error when we are using Skype-HTTP in our node js application -
UnhandledPromiseRejectionWarning: MicrosoftAccountLogin: Unable to login with MicrosoftAccount.
caused by GetLiveToken: Unable to get the Live token for Skype caused by LiveTokenNotFound: Unable to find the Live token. This token is normally found in the HTML response as the value of the element with the id "t". This error
maybe caused by a change in the Microsoft login workflow.


PR #32708 Test issue - Ignore it  issue reported by

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Break down in numbered steps
Expected Result:
Describe what you think should've happened
Actual Result:
Describe what actually happened
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Reproducible in staging?:
Reproducible in production?:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Slack conversation:
View all open jobs on GitHub

Triggered auto assignment to @stephanieelliott (Internal) because issue was reported by a contributor

Triggered auto assignment to @jboniface (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Disregard @jboniface I was testing to see if AutoAssignerTriage added the Daily label, it does


Deprecations: Common Context

DataPropagationProvider - clearData should use new method (no info)

Tasks:

[x] remove the deprecated code
[x] make sure the deprecation is correctly documented in 2.x (since, forRemoval on @Deprecated annotation, use @deprecated javadoc tag to link to the correct new implementation, or describe what to do if a simple replacement does not exist)
[x] validate documentation and examples are correct with regard to the deprecated element

PR #4388
PR #4389


Patient Assessment : SAT and SBT delivery category name change

Hi All,
We have changed 2 mcide category names in Patient Assessment table.
sat_delivery -> sat_delivery_pass_fail
sbt_delivery -> sbt_delivery_pass_fail
You can ref: https://github.com/clif-consortium/CLIF/blob/main/mCIDE/clif_patient_assessment_categories.csv
Thank you!

Changed is added to tracker sheet


Version placeholder in macOS commands

I think now is clearer and should avoid any future misunderstanding - thanks simran


Automated generation of images

Auto generate images non-interactive, as snc*, based on either tagged release or master.

[x] generate image => https://github.com/praveenkumar/snc/blob/master/createdisk.sh
[ ] use branch/tag indicator
[ ] collect and store metadata #34
[ ] publish artifacts

The "image generation" item should be addressed by this pull request https://github.com/praveenkumar/snc/pull/7. Unless we want to move from a shell script to go code integerated in the crc repo, in which case this is not done :)

No go code for this. Just traceability, ... and perhaps this repo has to
move to the code-ready namespace?
On Fri, Apr 19, 2019 at 12:04 AM Christophe Fergeau <
<EMAIL_ADDRESS>wrote:

The "image generation" item should be addressed by this pull request
praveenkumar/snc#7 https://github.com/praveenkumar/snc/pull/7. Unless
we want to move from a shell script to go code integerated in the crc repo,
in which case this is not done :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/code-ready/crc/issues/53#issuecomment-484574615, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAOZVRKPJZK2CEDIBVPI3PRCLZ3ANCNFSM4HF6DLAA
.

--
Gerard Braad | http://gbraad.nl
[ Doing Open Source Matters ]

Also, the image would need to be flat... no directory structure.
On Fri, Apr 19, 2019 at 12:23 AM Gerard Braad<EMAIL_ADDRESS>wrote:

No go code for this. Just traceability, ... and perhaps this repo has to
move to the code-ready namespace?
On Fri, Apr 19, 2019 at 12:04 AM Christophe Fergeau <
<EMAIL_ADDRESS>wrote:

The "image generation" item should be addressed by this pull request
praveenkumar/snc#7 https://github.com/praveenkumar/snc/pull/7. Unless
we want to move from a shell script to go code integerated in the crc repo,
in which case this is not done :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/code-ready/crc/issues/53#issuecomment-484574615, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAAAOZVRKPJZK2CEDIBVPI3PRCLZ3ANCNFSM4HF6DLAA
.

--
Gerard Braad | http://gbraad.nl
[ Doing Open Source Matters ]

--
Gerard Braad | http://gbraad.nl
[ Doing Open Source Matters ]

Also, the image would need to be flat... no directory structure.

It seems there are 2 somehow conflicting goals here. What is currently produced is a tarball with a crc_libvirt.sh script to test the image. It's assumed the user will download this, unpack it and run the script. In such scenarios, I'd expect a directory to be created.
When you suggest a flat structure, I think what you have in mind is a use case where the 'crc' go binary is used, and it would fetch a disk bundle, and know how to use it.
Maybe for now we'll need to ship 2 separate artifacts? One flat image, and one with crc_libvirt.sh?

The script is NOT the final product we are delivering. It is a means
to test the image.
Sure, I could get the code to flatten the image...  but the idea of
the snc repo is to generate the images for use with CRC.
On Fri, Apr 19, 2019 at 11:05 PM Christophe Fergeau
<EMAIL_ADDRESS>wrote:

Also, the image would need to be flat... no directory structure.
It seems there are 2 somehow conflicting goals here. What is currently produced is a tarball with a crc_libvirt.sh script to test the image. It's assumed the user will download this, unpack it and run the script. In such scenarios, I'd expect a directory to be created.
When you suggest a flat structure, I think what you have in mind is a use case where the 'crc' go binary is used, and it would fetch a disk bundle, and know how to use it.
Maybe for now we'll need to ship 2 separate artifacts? One flat image, and one with crc_libvirt.sh?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

--
Gerard Braad | http://gbraad.nl
[ Doing Open Source Matters ]


net: doc: Add missing defgroups to network header files

Some of the networking header files in include/net/ directory were
missing @defgroup doxygen directives.
There was also duplicate @defgroup directives which are now changed
to @addtogroup directives.
Jira: ZEP-2308
Signed-off-by: Jukka Rissanen<EMAIL_ADDRESS>

New version uploaded, this one adds new entries to doc/api/networking.rst file.

The first commit is sent also separately as #887

@dbkinder Shippable gives lot of errors from doxygen, to me these look like errors that are in doxygen itself. Anyway, I am lost how to solve these issues so please advise what to do with them.

Submitted new version which adds some exceptions in .known-issues/doc/networking.conf file in order to satisfy shippable.


refactor(usage-reporter): split into classes

UsageReporter was a sizeable class that became hard to extend. We want to extend the behaviour of error reporting in the Thread, which means we need to manage the Thread separate from the rest of the class.
This PR breaks UsageReporter into components, removes the need to pass an arbitrary options hash to each class, and makes the reporter composable by extracting knowledge of other classes from the reporter itself.
In doing this work I've improved test coverage of the usage reporter and how it sequences events between threads.
This PR is best reviewed commit by commit due to its size.

Also, THANK YOU for making these changes! I think it already makes this gem more readable and easier to understand. This will be in a much better shape now.

👋 are you looking for feedback on this PR yet?  noticed it was still in draft

@rperryng apologies for leaving this in draft so long. I stepped away from this to work on #34, which revealed to me that the benchmark and integration tests don't work well. Perhaps we can talk about this in an issue (happy to make one).
I think it would be better for me to break this PR into a few smaller ones that can be reviewed individually and then released as one version.


chore: simpler filter segmentation

Simplify filter type detection.

@mmerrill3 can you take a look?

I tried this code for a combination of repositoryMatch and branchMatch filters and it seems it doesn't work as expected. Only a default branch gets discovered, but not others.

@oleg-glushak sounds like I need to write some more tests...

@crenshaw-dev : Can u move this to argocd and close here?


Better example in Read Me for "Connect to MongoDB from another Docker container"

At https://hub.docker.com/_/mongo/, under "Connect to MongoDB from another Docker container", the example given is if you want to run two MongoDB containers side-by-side, but you want to connect them so you only have to start the Mongo server in one of them. This makes for some very confusing language, with mongo-this and mongo-that spread over two lines. It's impossible to unpack host name from link name from tag name from service name across two run commands.
Would it be possible to replace this example with a more useful one please? The perfect example would be this: Mongo is running one container, your web server is running in another, and the web server needs to access Mongo.
Many thanks.

$ docker network create some-network
$ docker run -d --name some-mongo --network some-network mongo:X.Y
$ docker run -d --name some-website --network some-network -e SOME_WEBSITE_MONGO=mongodb://some-mongo some-website-image

(Quite a bit more verbose and IMO not really any more clear.)
There's also a stack.yml example that I think is somewhat more clear: https://hub.docker.com/_/mongo#-via-docker-stack-deploy-or-docker-compose

Thanks. This worked for me:
$ docker network create skynet
$ docker run -d --name mongo --network skynet mongo:3.4.19
$ docker run -it --net skynet -e MONGO_HOST=mongo samsao/mongo-shell:3.4

Adjusted via https://github.com/docker-library/docs/pull/1441 :+1:


AP approaching zero when training with custom dataset from CVAT

Issue Description
I created a custom dataset using CVAT and exported it in the COCO 1.0 format, which provided a JSON annotation file containing only bounding boxes, without segmentation. This has led to a few issues:

Using a mock dataset with segmentation removed causes the Average Precision (AP) to drop nearly to zero.

When using my custom dataset, the results are even worse, as the code cannot detect any valid bounding boxes.

My question is: why are bounding boxes extracted from segmentation data rather than directly using them from the annotation file? Also, how should I go about addressing this issue?

The dataset_utils.py code seems to assume that the bbox element in the annotation file represents a bounding box in segmentation style, defined by eight points: (x1, y1, x2, y1, x2, y2, x1, y2).
if "segmentation" in anno:
    seg_list = [item for sublist in anno["segmentation"] for item in sublist]
elif "bbox" in anno:
    seg_list = anno["bbox"]

In contrast, the COCO format represents bounding boxes as [x, y, width, height]

The dataset_utils.py code seems to assume that the bbox element in the annotation file represents a bounding box in segmentation style, defined by eight points: (x1, y1, x2, y1, x2, y2, x1, y2).
if "segmentation" in anno:
    seg_list = [item for sublist in anno["segmentation"] for item in sublist]
elif "bbox" in anno:
    seg_list = anno["bbox"]

In contrast, the COCO format represents bounding boxes as [x, y, width, height]

Actually, this issue exists because I initially used segmentation to extract bounding boxes and never tried using the bounding boxes directly—this is indeed a bug.
I can fix it in the next update, or if anyone is able to make a PR, that would be great!

Yes I'll create a pull request 👍

I create a pull request (https://github.com/WongKinYiu/YOLO/pull/112)

#116 should fix the problem


Size of download icon on course outline screen increased a little

https://openedx.atlassian.net/browse/MA-1274
Looks like this now:

@aleffert @bguertin @1zaman plz review

:+1:

:+1:


Deprecate .NET 6

With one month to ago before .NET 6 is officially end-of-life, the .NET 6 branch will be no longer be maintained. The .NET 6 README will be updated accordingly.

Updated README and locked branch.


[DRAFT] Replace pubsub client with libcloudforensics client

The following PR replaces the GCP PubSub client to instead be called with libcloudforensics client. It does so by updating the setup_publisher and setup_subscription to instead create the pubsub service through an API call/libcloud forensics

This PR can be closed in favour of https://github.com/google/turbinia/pull/1052

Closing this in favor of https://github.com/google/turbinia/pull/1052


unexpected 'require' (T_REQUIRE), expecting identifier (T_STRING)

With PHP 5.6 i got this error message.
Parse error: syntax error, unexpected 'require' (T_REQUIRE), expecting identifier (T_STRING) in ComposerAPI.php on line 203
Unfortunately, I have no idea how to fix this.
Regards,
Bjoern

It seems, that your version of PHP takes the method name "require" as a reserved keyword. Unfortunately, I do not have a PHP 5.6 server to test on, but on PHP 7+ it does not cause any problems. If needed, you can try to change the name of the method to anything you like. Nothing depends on that method, so you are free to change it's name.

Thank you for the feedback


How can I incorporate textual inversion into the UniControl? Will unicontrol support textual inversion?

Hello, I'm aiming to incorporate textual inversion into your impressive project. To start, my initial objective is to extract the stable diffusion weights from the unicontrol_v1.1.ckpt model and then load these weights to the Hugging Face diffusers pipeline for training textual inversion. Subsequently, I'll need to introduce the requisite code to enable the model to employ the trained embedding.
Does this approach seem viable? If it does, could you guide me on the precise steps to successfully extract the stable diffusion weights from the unicontrol model?

Yes. I believe that stable diffusion can be extracted from unicontrol_v1.1.ckpt. Please feel free to let me know anything I can help.


Remove clipboard suggestion once it's added

Clipboard suggestion should no longer be suggested once it's added, unless a new URL is copied into the clipboard.
TODO

[x] Save clipboard item to clipboard manager (or somewhere else)
[x] Add flag to clipboard manager to indicate that user used clipboard suggestion
[x] If user adds from clipboard, set flag to true
[x] If URL from clipboard is different from saved value, then replace the saved value with new clipboard URL and set flag to false

Fixed in #37, no flag needed.


Apple News Icon in the widget is incorrect

Right now it's two squares where in the real icon it's a scroll. Also the "N" in the logo doesn't match correctly.
I suggest we add a page with the high res icons for the new apps, which we can then reuse across the entire UI.

Aye, definitely! We'll need some help getting all the icons in.

I had some trouble getting the branches and pull requests going, but will work try it again next week.

Tried replicating it better:


How to shutdown server?

Hi @poutyface, after you start the server it should give you instructions on how to stop:
$ model-explorer
Loading extensions...
Loaded 8 extensions:
 - TFLite adapter (Flatbuffer)
 - TFLite adapter (MLIR)
 - TF adapter (MLIR)
 - TF adapter (direct)
 - GraphDef adapter
 - Pytorch adapter (exported program)
 - MLIR adapter
 - JSON adapter

Starting Model Explorer server at:
http://localhost:8080

Press Ctrl+C to stop.

Does Ctrl+C work for you? Let us know if this works.

hmm  Ctrl+C don’t work on ubuntu22.04
$ model-explorer —host <IP_ADDRESS> —port 8080

Seeing the same thing on Ubuntu 22.04 on WSL. Was only able to close the server using pkill pid.

Hi @poutyface and @KevinKaChunLee, I have not been able to reproduce with Ubuntu or Debian. I have not been able to try WSL but that could be a good clue. Do you guys usually use Ctrl+C to stop processes within your workflows normally? Is there any other environmental factor which can explain the different reaction? Such as..
Python version? Conda/venv? Is there any other output? Any other options supplied to the command? Any additional information will be helpful. Thanks.

Hi @pkgoogle , I am running python 3.12.2 in a virtual env. Ctrl + C usually works to cancel other processes.
It's strange we don't get any of the following prints after -JSON adaptor. Perhaps something is hanging preventing stdin to be captured from the keyboard. The server appears to be running fine, so I imagine it to occur between starting the server and printing out "Starting Model Explorer" print line.
Pressing Ctrl+C just shows ^C in the terminal which shows the input is being captured by the terminal, but its just not being registered. Ctrl + Z still works in suspending the program.

The issue seems related to running the server through a proxy. The request never gets the 200 response status code so the code is stuck in this while loop.
https://github.com/google-ai-edge/model-explorer/blob/f36d8394f8eba7504faa8dd251abd2ed8c0a1111/src/server/package/src/model_explorer/server.py#L279-L287

Hi @KevinKaChunLee, I'm feeling this is likely a WSL networking issue related to what you said above... currently trying to find an environment where I can replicate this.