#Rank issue

when I run tools/data/custom_2d_skeleton.py file, I get the following error:
Traceback (most recent call last):
File "./tools/data/custom_2d_skeleton.py", line 172, in
main()
File "./tools/data/custom_2d_skeleton.py", line 119, in main
init_dist('pytorch', backend='nccl')
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/mmcv/runner/dist_utils.py", line 40, in init_dist
_init_dist_pytorch(backend, **kwargs)
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/mmcv/runner/dist_utils.py", line 51, in _init_dist_pytorch
rank = int(os.environ['RANK'])
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/os.py", line 681, in getitem
raise KeyError(key) from None
KeyError: 'RANK'
Dependencies:
python 3.7
pytorch 1.11
cudatollkit 11.3
mmcv-full 1.5.0
mmdet  2.24.0
-Use single GPU
Can anyone give me some idea to solve this? Thanks!

Hello, please check this example on how to use the provided script: https://github.com/kennymckormick/pyskl/blob/main/examples/extract_diving48_skeleton/diving48_example.ipynb

I ran the above mentioned link.
/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/launch.py:186: FutureWarning: The module torch.distributed.launch is deprecated
and will be removed in future. Use torchrun.
Note that --use_env is set by default in torchrun.
If your script expects --local_rank argument to be set, please
change it to read from os.environ['LOCAL_RANK'] instead. See
https://pytorch.org/docs/stable/distributed.html#launch-utility for
further instructions
FutureWarning,
load checkpoint from http path: https://download.openmmlab.com/mmdetection/v2.0/faster_rcnn/faster_rcnn_r50_fpn_1x_coco-person/faster_rcnn_r50_fpn_1x_coco-person_20201216_175929-d022e227.pth
load checkpoint from http path: https://download.openmmlab.com/mmpose/top_down/hrnet/hrnet_w32_coco_256x192-c78dce93_20200708.pth
0%|                                                 | 0/16997 [00:00<?, ?it/s]/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/mmdet/datasets/utils.py:70: UserWarning: "ImageToTensor" pipeline is replaced by "DefaultFormatBundle" for batch inference. It is recommended to manually replace it in the test data pipeline in your config file.
'data pipeline in your config file.', UserWarning)
0%|                                                 | 0/16997 [00:16<?, ?it/s]
Traceback (most recent call last):
File "/home/ubuntu/Documents/aiguard/pyskl/tools/data/custom_2d_skeleton.py", line 172, in
main()
File "/home/ubuntu/Documents/aiguard/pyskl/tools/data/custom_2d_skeleton.py", line 145, in main
pose_results = pose_inference(pose_model, frames, det_results)
File "/home/ubuntu/Documents/aiguard/pyskl/tools/data/custom_2d_skeleton.py", line 68, in pose_inference
pose = inference_top_down_pose_model(model, f, d, format='xyxy')[0]
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/mmcv/utils/misc.py", line 340, in new_func
output = old_func(*args, **kwargs)
File "/home/ubuntu/Documents/aiguard/mmpose/mmpose/apis/inference.py", line 387, in inference_top_down_pose_model
use_multi_frames=use_multi_frames)
File "/home/ubuntu/Documents/aiguard/mmpose/mmpose/apis/inference.py", line 247, in _inference_single_pose_model
data = test_pipeline(data)
File "/home/ubuntu/Documents/aiguard/mmpose/mmpose/datasets/pipelines/shared_transform.py", line 107, in call
data = t(data)
File "/home/ubuntu/Documents/aiguard/mmpose/mmpose/datasets/pipelines/top_down_transform.py", line 289, in call
c = results['center']
KeyError: 'center'
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 7369) of binary: /home/ubuntu/miniconda3/envs/aiguard1/bin/python
Traceback (most recent call last):
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/launch.py", line 193, in
main()
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/launch.py", line 189, in main
launch(args)
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/launch.py", line 174, in launch
run(args)
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/run.py", line 718, in run
)(*cmd_args)
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/launcher/api.py", line 131, in call
return launch_agent(self._config, self._entrypoint, list(args))
File "/home/ubuntu/miniconda3/envs/aiguard1/lib/python3.7/site-packages/torch/distributed/launcher/api.py", line 247, in launch_agent
failures=result.failures,
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
/home/ubuntu/Documents/aiguard/pyskl/tools/data/custom_2d_skeleton.py FAILED
Can anyone give me some idea to solve this? Thanks!


Bumped upstream version to v0.9.9

Closing because there is already #582. Everything must be updated in a single PR.


Add newline under Module Loaders heading

The list wasn't actually being displayed as a list on the site, due to the lack of a blank line above the first item:

Thanks!


The pop-up message is blocked when upload file success at the interface of create a custom mirror

In GitLab by @Robin-Mao on Sep 21, 2017, 18:11

In GitLab by @liyanq528 on Nov 8, 2019, 07:37
fixed by pull request #422

In GitLab by @sokril on Nov 8, 2019, 08:11
closed

In GitLab by @sokril on Nov 8, 2019, 08:11
assigned to @liyanq528


update_fastlane

Hey,
Is there a difference between update_fastlane and gem update fastlane ?
Cheers,
Romain

Hey,
update_fastlane is an action you can add to your Fastfile to automatically update fastlane if required, while gem update fastlane is executed by you manually.
Does that make sense?

Sure but my question is more : is the result of both command identical?

On 26 Aug 2015, at 23:54, Felix Krause<EMAIL_ADDRESS>wrote:
Hey,
update_fastlane is an action you can add to your Fastfile to automatically update fastlane if required, while gem update fastlane is executed by you manually.
Does that make sense?
—
Reply to this email directly or view it on GitHub.

It should be, yes.

Thanks
On Thu, Aug 27, 2015 at 10:04 AM, Felix Krause<EMAIL_ADDRESS>wrote:

It should be, yes.
—
Reply to this email directly or view it on GitHub
https://github.com/KrauseFx/fastlane/issues/545#issuecomment-135333854.

:+1:


How do you render when the variable could be an single object or an array of objects?

Hi,
Here is the issue i am having. Lets say we have an API return a data object which could be either
this
{
    to : {  name : 'John'  }
}

or this
{
    to : [
             {  name : 'John'  },
             {  name : 'Jenny'  },
             {  name : 'Markl'  }
          ]
}

How do i render this on a template? If i use something like this.
To:  {{#each to}} {{name}} , {{/each}}

I get the error,
'to' is used like an array by the template, but is a scalar value or object in your model.
What am i missing?  Couldn't really find this in the documentation

Ok, have created a fix which essentially extends each syntax to accept the scenario above.
Let me know what you guys think?

@justshrey so in affect, you want to change the behaviour of "each", because your own code switches between types?
You should edit your model to always pass an array - even if there is only a single item

@soreng  yes. The point is that is not necessarily my code. Its a data which i get from an endpoint which i would prefer not to encapsulate.
What i am trying to understand is what exactly is the benefit of throwing an error?
In a lot of use cases there a distinct advantage in treating a scalar value as a single item in an array since we have explicitly stated  the intent to handle one or many by the use of each within the template.  It eliminates the need for conditionals in a template and avoid un-necessary encapsulation of any JSON that comes in.

What i am trying to understand is what exactly is the benefit of throwing an error?
it is to tell you that you are trying to iterate a list, but providing a scalar value

I really don't see a problem in this, since this is how "each" works in every language. It would not be very intuitive to change that behaviour.
I have worked with a api that switched types, and it took forever to figure out a solution. All the serialization code had to have two input / output types, and then manually convert to a list with a single element.

Okay,
I understand when it comes to any programming language having an array is expected behavior. However, this is supposed to be a logic-less templating system, ergo the comparisons would not be the same..
Having the each autobox a single scalar value to an array, would in this case be expected behaviour from the perspective on an end user. For that matter, if you look at the original mustache demo, http://mustache.github.io/#demo .
If we change the default data to this
{
  "header": "Colors",
  "items": {"name": "red", "first": true, "url": "#Red"},
  "empty": false
}

i.e setting the items property to an object instead of an array, it still renders correctly as expected.
I really think that this should be added to the library since it help it become more congruent to the expected mustache template behaviour.
In case i am missing the point, could you help me understand this with an example where the boxing of a scalar value into a single item array would be an issue? Specifically, in this context i.e the logic-less template system.


Error: Cannot find module 'js-zimbra' constant problem

Hello,
While i'm using this module, and while i'm instaishiating the module, once i run my code i keep on getting the Error: Cannot find module 'js-zimbra' error, i don't know where the problem is comming from, any suggestions ?

Can you paste your code
Saludos.

Patricio Bruna
CEO - IT Linux
(+56-2) 2902 3001
(+56-9) 8899 6618
www.itlinux.cl / blog.itlinux.cl
Correo enviado a través de
ZBox: Correo y Colaboración en la Nube

El 20-06-2016, a las 18:27, Houssem Yahiaoui<EMAIL_ADDRESS>escribió:
Hello,
While i'm using this module, and while i'm instaishiating the module, once i run my code i keep on getting the Error: Cannot find module 'js-zimbra' error, i don't know where the problem is comming from, any suggestions ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I finally got the problem, i will pull request it, there's some major naming troubles in the package.
Houssem.

Looking forward to your PR.


make tests that cehck stderr more robust

motivation: broken tests when warnings are present in stderr
changes: use XCTAssertMatch and check for error instead of straight comparison

@swift-ci please smoke test


watson edit on the currently running frame fails

Version 2.0.0
I updated my watson version from 1.10 to 2.0.
In 1.10 I used to be able to call "watson edit" without any argument to edit the currently running frame. In 2.0 I get an error:
> watson start foo +mytag
Starting project foo [mytag] at 15:56
> watson edit
Error: Error converting date: Cannot parse argument of type None.

I suspect that this might be a bug in 2.0

Yes it is! But the fix has not been released yet. My bad, sorry :grimacing:
I'll make a new release this week.
Duplicate of #417

No problem, its not urgent, at least for me.
I'm sorry I missed the fact that a ticket for this already existed.

Just checking in @jmaupetit, it doesn't look like a release has been cut for this bug fix. Any ETA on when it'll go out and be available via Homebrew?


Update developer docs to reflect unbundling of python.

Resolves #142.
This does update to reflect unbundling, but mostly removes libxslt / lxml specific guidance that is really nothing to do with GEE. The versions described are ancient, and I don't think the documentation needs to be directive on how to put together the dependencies.

@asmitaw Incorporated changes per review comments.

This has an approval, so I'll go ahead and merge


Linkis1.0 EngineConn ProcessEngineConnLaunch port problem

Split the service and divide the ecm into multiple loads. The engine is distributed to different nodes and the port is uncontrollable. This will cause the security firewall to be unmanageable and eventually fail to execute the task.

component=EngineConnPlugin/Plugins; type=Improvement


Create a "high-interaction" honeypot server

The simple_http server is currently a "low interaction" honeypot. Meaning that an attacker isn't likely to spend time (i.e. interactions) exploring the server, giving away valuable information in the process. (It's really probably most valuable for detecting port scans at the moment.)
However it (or a new server) could be written to simulate a realistic server, one that engages the attacker for longer.
There are lots of OSS examples of these so it probably makes sense to start by researching those.

Ideally avoiding pulling in too many (or zero) 3rd party Pypi packages would be nice re: maintenance and supply chain security

Also probably want to avoid giving away full blown shell/OS access (to avoid becoming part of an attacker's intrusion strategy).
Making otherwise realistic fake application servers or similar devices that are explicitly bounded in what can be explored is probably the way to go


[BUG] check_env_specs shouldn't set torch.manual_seed by default.

Describe the bug
Current default behaviour of check_env_specs() is to set the manual key to 0.
I think this is dangerous as people might run this function after setting their own seed (at the beginning of main(), not realizing it's overwriting what they set before.
Reason and Possible fixes
Set default argument of seed in check_env_specs to None.
Checklist

[x] I have checked that there is no similar issue in the repo (required)
[x] I have read the documentation (required)
[ ] I have provided a minimal working example to reproduce the bug (required)

Yep I agree it isn't ideal.
Note that the intended use of this function is to be run out of your training loop.
I wish PyTorch had some more refined way of locally set the seed but as of now it isn't the case.
I can have a look at not seeding things by default

If you want to have a look at #1872 it should be ready

Looks great, thank you!


qpdf: 11.2.0 -> 11.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/qpdf/qpdf/releases.
meta.description for qpdf is: A C++ library and set of programs that inspect and manipulate the structure of PDF files
meta.homepage for qpdf is: https://qpdf.sourceforge.io/
meta.changelog for qpdf is: https://github.com/qpdf/qpdf/blob/v11.3.0/ChangeLog
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 11.3.0 with grep in /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0
found 11.3.0 in filename of file in /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0

Rebuild report (if merged into master) (click to expand)

44 total rebuild path(s)

44 package rebuild(s)

First fifty rebuilds by attrpath
buildbot-full
cups-filters
foomatic-db
foomatic-db-ppds
foomatic-db-ppds-withNonfreeDb
glaxnimate
img2pdf
manga-cli
mirage-im
ocrmypdf
paperless-ngx
pdfarranger
pdfmixtool
pdfslicer
python310Packages.Nikola
python310Packages.buildbot-full
python310Packages.buildbot-plugins.badges
python310Packages.cairocffi
python310Packages.cairosvg
python310Packages.graphite-web
python310Packages.img2pdf
python310Packages.ocrmypdf
python310Packages.pikepdf
python310Packages.pygal
python310Packages.textnets
python310Packages.wavedrom
python311Packages.Nikola
python311Packages.buildbot-full
python311Packages.buildbot-plugins.badges
python311Packages.cairocffi
python311Packages.cairosvg
python311Packages.graphite-web
python311Packages.img2pdf
python311Packages.ocrmypdf
python311Packages.pikepdf
python311Packages.pygal
python311Packages.wavedrom
qpdf
qtile
skytemple
splix
streamdeck-ui
system-config-printer
xsser

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A qpdf https://github.com/r-ryantm/nixpkgs/archive/9f747c87f8570f4713f9adb5d1c539e9b25cfb83.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0
ls -la /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

10 packages failed to build:

cups-filters
foomatic-db
foomatic-db-ppds
foomatic-db-ppds-withNonfreeDb
pdfmixtool
python311Packages.buildbot-full
python311Packages.graphite-web
python311Packages.qtile-extras
splix
system-config-printer

33 packages built:

buildbot-full (python310Packages.buildbot-full)
glaxnimate
img2pdf (python310Packages.img2pdf)
manga-cli
mirage-im
ocrmypdf (python310Packages.ocrmypdf)
paperless-ngx
pdfarranger
pdfslicer
python310Packages.Nikola
python310Packages.buildbot-plugins.badges
python310Packages.cairocffi
python310Packages.cairosvg
python310Packages.graphite-web
python310Packages.pikepdf
python310Packages.pygal
python310Packages.qtile-extras
python310Packages.textnets
python310Packages.wavedrom
python311Packages.Nikola
python311Packages.buildbot-plugins.badges
python311Packages.cairocffi
python311Packages.cairosvg
python311Packages.img2pdf
python311Packages.ocrmypdf
python311Packages.pikepdf
python311Packages.pygal
python311Packages.wavedrom
qpdf
qtile
skytemple
streamdeck-ui
xsser

Maintainer pings
cc @abbradar for testing.

I got this compilation error while building the PR:
error: builder for '/nix/store/80hxh6z7s6yrc2ckdhh4i3ryzi8jn7yr-pdfmixtool-1.1.drv' failed with exit code 2;
       last 10 log lines:
       >       |                           ^~~~~~~~~~~
       > /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0/include/qpdf/QPDFWriter.hh:559:30: error: 'std::string_view' has not been declared
       >   559 |     void writeStringQDF(std::string_view str);
       >       |                              ^~~~~~~~~~~
       > /nix/store/aiaq7wyr90hba0fdy6zcm22s6zp37cjg-qpdf-11.3.0/include/qpdf/QPDFWriter.hh:560:32: error: 'std::string_view' has not been declared
       >   560 |     void writeStringNoQDF(std::string_view str);
       >       |                                ^~~~~~~~~~~
       > make[2]: *** [CMakeFiles/pdfmixtool.dir/build.make:319: CMakeFiles/pdfmixtool.dir/src/mainwindow.cpp.o] Error 1
       > make[1]: *** [CMakeFiles/Makefile2:84: CMakeFiles/pdfmixtool.dir/all] Error 2
       > make: *** [Makefile:136: all] Error 2
       For full logs, run 'nix log /nix/store/80hxh6z7s6yrc2ckdhh4i3ryzi8jn7yr-pdfmixtool-1.1.drv'.

Result of nixpkgs-review pr 220175 run on x86_64-linux 1

10 packages failed to build:

cups-filters
foomatic-db
foomatic-db-ppds
foomatic-db-ppds-withNonfreeDb
pdfmixtool
python311Packages.buildbot-full
python311Packages.graphite-web
python311Packages.qtile-extras
splix
system-config-printer

33 packages built:

buildbot-full (python310Packages.buildbot-full)
glaxnimate
img2pdf (python310Packages.img2pdf)
manga-cli
mirage-im
ocrmypdf (python310Packages.ocrmypdf)
paperless-ngx
pdfarranger
pdfslicer
python310Packages.Nikola
python310Packages.buildbot-plugins.badges
python310Packages.cairocffi
python310Packages.cairosvg
python310Packages.graphite-web
python310Packages.pikepdf
python310Packages.pygal
python310Packages.qtile-extras
python310Packages.textnets
python310Packages.wavedrom
python311Packages.Nikola
python311Packages.buildbot-plugins.badges
python311Packages.cairocffi
python311Packages.cairosvg
python311Packages.img2pdf
python311Packages.ocrmypdf
python311Packages.pikepdf
python311Packages.pygal
python311Packages.wavedrom
qpdf
qtile
skytemple
streamdeck-ui
xsser

The latest pdfmixtool on its master branch has a fix: https://gitlab.com/scarpetta/pdfmixtool/-/commit/bd5f78c3a4d977d9b0c74302ce2521c737189b43

Opened https://github.com/NixOS/nixpkgs/pull/220356 to fix pdfmixtool ahead of this merge.

Can we please not merge things that are known to break builds? Especially when it comes to major packages like basically every printer driver.

Yes sorry for approval


Add eCloud v2 functionality into SDK

What is the feature?
We need to update SDK to support eCloud v2 resources
What benefits will this change bring?
.NET SDK support for eCloud V2
Additional context
Include the following resources:

[x] Availability Zones
[x] DHCP
[x] Firewall Rules
[x] Floating IPs
[x] Instances
[x] Networks
[x] Routers
[x] Regions
[x] VPCs
[x] VPNs

I missed off load balancer resources from that list.  I'll add in those resources now

[ ] Load balancers


TGRSIRunInfo not saved to analysis file

The custom TGRSIRunInfo used to construct an analysis tree does not appear to be saved to the analysis tree.
Intentional?

Should be writing. Don't know what would have happened to change that.

On Aug 28, 2015, at 7:18 PM, SmithJK<EMAIL_ADDRESS>wrote:
The custom TGRSIRunInfo used to construct an analysis tree does not appear to be saved to the analysis tree.
Intentional?
—
Reply to this email directly or view it on GitHub.

Dug a bit more into this. It is saving fine to file, but if you open the analysis file, the stored TGRSIRunInfo (name of the instance is the same as the class name?) isn't loaded into the current session. There is the TGRSIRunInfo::SetInfoFromFile function. I tried
GRSI [1] TGRSIRunInfo* temp
GRSI [2] TGRSIRunInfo::SetInfoFromFile(temp)

and
GRSI [0] TGRSIRunInfo::SetInfoFromFile(TGRSIRunInfo)

which both failed, but
GRSI [1] TGRSIRunInfo

worked.
Question:
*What is the proper syntax for using this function (both in the interpreter and in a script)?
Suggestion:
*Let's be consistent and change SetInfoFromFile to ReadInfoFromFile.

Using TGRSIRunInfo->Print() does not work for you? It is not a TNamed, so it’s class name is what is written to file. I think TGRSIRunInfo->Get()->Whateveryouwant() should work as well. Is this what you have been trying?

On Sep 1, 2015, at 4:16 PM, SmithJK<EMAIL_ADDRESS>wrote:
Dug a bit more into this. It is saving fine to file, but if you open the analysis file, the stored TGRSIRunInfo (name of the instance is the same as the class name?) isn't loaded into the current session. There is the TGRSIRunInfo::SetInfoFromFile function. I tried
GRSI [1] TGRSIRunInfo* temp
GRSI [2] TGRSIRunInfo::SetInfoFromFile(temp)
and
GRSI [0] TGRSIRunInfo::SetInfoFromFile(TGRSIRunInfo)
which both failed, but
GRSI [1] TGRSIRunInfo
worked.
Question:
*What is the proper syntax for using this function (both in the interpreter and in a script)?
Suggestion:
*Let's be consistent and change SetInfoFromFile to ReadInfoFromFile.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/463#issuecomment-136847617.

I was trying to use TGRSIRunInfo::AddBackWindow.
I was using a call to griffin_hits.GetAddbackMultiplicity() in a Scan and the results were different after I read in a RunInfo file than immediately after I'd opened the file.

I don’t believe the RunInfo is loaded from the file automatically. You are likely getting the default add back window with that call as it calls Get() which creates a new one if one doesn’t already exist in memory. If you try TGRSIRunInfo->AddBackWindow() it should work. I wouldn’t be shocked if you also tried, TGRSIRunInfo; TGRSIRunInfo::AddBackWindow(); just because the first one might load the one in file to memory. Let me know if that fixes things. Maybe we can come up with a way to avoid the accidental default usage. Automatic reading sometimes comes with it’s own problems, but should also be doable if it isn’t being done already.

On Sep 1, 2015, at 6:08 PM, SmithJK<EMAIL_ADDRESS>wrote:
I was trying to use TGRSIRunInfo::AddBackWindow.
I was using a call to griffin_hits.GetAddbackMultiplicity() in a Scan and the results were different after I read in a RunInfo file than immediately after I'd opened the file.
—
Reply to this email directly or view it on GitHub https://github.com/GRIFFINCollaboration/GRSISort/issues/463#issuecomment-136878058.

Yes, running
GRSI [0] TGRSIRunInfo
GRSI [1] TGRSIRunInfo::AddBackWindow()

does print out the correct addback window.
Loading the run info automatically would help prevent accidental usage, similar to how TChannel reads in automatically. I think these two should be as similar as possible in terms of how we access them will also help users. This is why I also think that changing "SetInfoFromFile" should be "ReadInfoFromFile" - read is the verb used in TChannel and in the TGRSIRunInfo::ReadInfoFile() command.

Works for me. Change what you need to and I'll check and make sure there aren't any conflicts.

On Sep 1, 2015, at 7:08 PM, SmithJK<EMAIL_ADDRESS>wrote:
Yes, running
GRSI [0] TGRSIRunInfo
GRSI [1] TGRSIRunInfo::AddBackWindow()
does print out the correct addback window.
Loading the run info automatically would help prevent accidental usage, similar to how TChannel reads in automatically. I think these two should be as similar as possible in terms of how we access them will also help users. This is why I also think that changing "SetInfoFromFile" should be "ReadInfoFromFile" - read is the verb used in TChannel and in the TGRSIRunInfo::ReadInfoFile() command.
—
Reply to this email directly or view it on GitHub.

Pull request #468.
I added a line to GRSIint.cxx to read in the file. I'm not sure about backward compatability here - what happens in the TGRSIRunInfo->Get() function if there is no run information saved in the file?
Also, after I made this change, saw a note from @pcbend in TGRSIRunInfo.h suggesting a slightly different way to do this. This seems to work though.

I'll update it to ignore the info if it doesn't exist.

Also looking at this now, I think I prefered the name Set, but maybe SetRunInfo(TGRSIRunInfo*) would be better. I'll change it to that. You'll see what I mean when I push.

Is this fixed in my last update?

Yes, this seems to work in the last update.


fix: typo distinct until changed russian

@GregorBiswanger , yep, you can merge, it's my error, i wrote  "diverUntilChanged" instead distinctUntilChanged

@HotspurHN , zemlyak, please, if you will have free time, look at the correctness of the translation.


Adding a CMORiser for CMAP data

Description

Closes CMORiser for CMAP 1979-2018 Data in ESMValCore
Link to documentation

Checklist
It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

[ ] 🛠 This pull request has a descriptive title
[ ] 🛠 Code is written according to the code quality guidelines
[ ] 🛠 Documentation is available
[ ] 🛠 Tests run successfully
[ ] 🛠 The list of authors is up to date
[ ] 🛠 Any changed dependencies have been added or removed correctly
[ ] 🛠 All checks below this pull request were successful

New or updated data reformatting script

[ ] 🛠 Documentation is available
[ ] 🛠 The dataset has been added to the CMOR check recipe
[ ] 🛠 The dataset has been added to the shared data pools of DKRZ and Jasmin by the @ESMValGroup/OBS-maintainers team
[ ] 🧪 Numbers and units of the data look physically meaningful

To help with the number of pull requests:

🙏 We kindly ask you to review two other open pull requests in this repository

Hi @flicj191, I think this one needs a closer look. The CMORiser needs to be added to the tests. Can you help @max-anu when you have a chance?

Values seem sensible:

As far as I can tell, remaining errors are either not issues with the submitted code, or are expected (e.g. unused variables in function definition).

Hi @flicj191, I think this one needs a closer look. The CMORiser needs to be added to the tests. Can you help @max-anu when you have a chance?

@rbeucher, which tests do you mean?

I thought there was a more extensive list of tests in tests/integration/test_cmorizer.py. Forget about that...

I can't see @max-anu in the list of authors: https://github.com/ESMValGroup/ESMValTool/blob/main/CITATION.cff
What about you @flicj191 ? Are you there?

I can't see @max-anu in the list of authors: https://github.com/ESMValGroup/ESMValTool/blob/main/CITATION.cff What about you @flicj191 ? Are you there?

I added myself as an author here: https://github.com/ESMValGroup/ESMValTool/pull/3765


Update library.html template

fix #153 Bad font rendering

Thanks for your contribution @PaoloFalomo, the website will be updated with in 15 mins!
cc @pvnr0082t

Glad to help the open source world 🌐 🎆

@PaoloFalomo Welcome to join, if you are interested in CDNJS, we have lots of issue there :)

@PeterDaveHello oh! Great! I'll see where i can help. I'm expert on js,html,css,php

@PaoloFalomo Awesome, thanks a lot!


Fix jest-haste-map on vagrant.

This is a similar fix to what @spicyj did in https://github.com/spicyj/jest/commit/cff14103b247fcf539b5d5434b07fb2a55819d6d a long time ago. I wasn't aware of why we had this but it seems obvious to me that we shouldn't create processes if there are less than one worker available. Not really sure how that could happen but it seems that on vagrant/docker installs the number of cores is 0.
See #980.

@facebook-github-bot import

Thanks for importing. If you are an FB employee go to Phabricator to review.

Any chance this is the same fix needed to make coverage and runInBand work together?

Probably not, looks like a different bug unfortunately :(

We really need to start using flow + add more integration tests.

I think the proper fix should be around here (assuming that options.maxWorkers should represent the number of cores)
https://github.com/facebook/jest/blob/master/packages/jest-cli/src/jest.js#L41-L42
In the highlighted lines the options.maxWorkers is set to 0 for when cpus equals 1

I can confirm this PR fixes my issue.


Failing to parse a xml.gz sitemap

Our sitemapParser fails downloading this sitemap
When trying to download it with the sitemapTester tool, it returns a 403 http code - which I don't understand, as it should be a 200 ok as my browser succeeds in downloading this xml.gz file (as well as I succeed it downloading it successfully [200 OK] using UniRest Http client).
When I download it using UniRest fetcher then trying to parse it using our parser, using this method: parseSiteMap(String contentType, byte[] content, URL url)
Or the method without the contentType, in both cases it fails with the following exception:
crawlercommons.sitemaps.UnknownFormatException: Failed to detect MediaType of sitemap 'https://docu.gsa-online.de/sitemap.xml.gz' 	at crawlercommons.sitemaps.SiteMapParser.parseSiteMap(SiteMapParser.java:269)

This is strictly speaking not a parser issue but a common problem when fetching content (as any crawler/spider does): the remote server blocks certain user agents, it's reproducible by
% wget -U 'Java/1.8.0_222' -d https://docu.gsa-online.de/sitemap.xml.gz
...
---response begin---
HTTP/1.1 403 Bad Behavior
...

Error 403

We're sorry, but we could not fulfill your request for
/sitemap.xml.gz on this server.

You do not have permission to access this server.

...

The recursive parser methods are provided for convenience. We can hardly transform them into real crawler methods which would force us to make them polite (follow robots.txt exclusions, guarantee delays between successive fetches), make the user agent string configurable, etc.
I cannot reproduce the issue when the sitemap.xml.gz is downloaded by another tool and then parsed. What was passed as "contentType" - application/x-gzip or similar? The MIME detector definitely works on the binary content (passed as byte[]).

This issue is one of the reasons I'm not a big fan of putting "tools" like the SitemapTester into the core of crawler-commons. Instead, I think we should have a separate tools sub-project that depends on crawler-commons core, and can be used for this type of functionality.

Hi @kkrugler, agreed about the SiteMapTester (the separation is addressed in #197). But what about the methods in SiteMapParser which take nothing more than a URL (no byte[] content)? Deprecate? Or just document the fact that users lack the control over fetching the sitemap content? We may instead/additionally provide methods which take a fetch function, e.g. parseSiteMap(URL onlineSitemapUrl, Function<URL, byte[]> fetcher).

I prefer leaving our own fetching (although flawed), but adding Javadoc to that method disclaimering we are not a fetching library, thus the fetch faults are not necessarily the parsing faults, as well as wrapping the fetching line with a try-catch segment logging out a short exaplanation that the sitemap failed the fetch not necessarily the parse.
I do think there are/will be many cases where people will need a fast sitemap parsing and they won't want to delve too deeply into their own fetcher etc, they will look for a quick and dirty sitemap fetch.

@sebastian-nagel - I like the idea of deprecating the "built-in fetcher" API, and adding an API that takes a Function. We could also include a simple example in JavaDoc of how to do the quick & dirty approach, as in:
new Function<URL, byte[]>() {
    byte[] apply(URL url) {
        try {
            return IOUtils.toByteArray(url);
        catch (Exception e) {
            // TODO - Log the exception
            return null;
        }
}


Some WebPs don't play on IOS18

guard let testString = String(data: testData, encoding: .ascii) else {
return false
}
iOS18 May have more stringent validation of how strings and data are encoded. In this case, non-pure ASCII characters or abnormally encoded data may cause the conversion to fail.
Some Webp files failed to convert to testString.

This issue appears to have been resolved in https://github.com/yeatse/KingfisherWebP/pull/81. The patch has been published as version 1.6.1. Feel free to share any feedback on this version.


⚠ Bofrost.com has degraded performance

In 01a1dd0, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 1990 ms

Resolved: Bofrost.com performance has improved in 5c47a33 after 9 minutes.


Bump testcontainers-selenium to newest version

Existing testcontainers-selenium v14.2 has issues on Docker for Mac:
https://github.com/testcontainers/testcontainers-java/issues/2395

Completed with https://github.com/Roche/spock-geb-quickstart/pull/10


Improve the optimization of If with one branch of type Nothing.

When one of the branches of an If is of type Nothing, we know it cannot return normally. We can therefore rewrite the If in statement position, and extract the other branch out of the If. This is better because the other branch can stay fully virtual as a PreTransform, and need not be forced to a PreTransGenTree.

Review by @gzm0
The test suite diff can be seen here: https://gist.github.com/sjrd/5a7321f3ea8e7f993128daa04694f7bc

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/3220/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/4184/
Test PASSed.

LGTM


Post date should correspond to when first published

Posts are dated according to date created, but they should be dated according to when they're first published. This causes problems when maintainers want to queue up posts for later publishing or work on drafts within prose itself - and there's no straightforward way to change the date of a post after it's been created.

I can note that yes, Jekyll skips future posts by default. It is noted here http://jekyllrb.com/docs/configuration/


[AIR][AMD-AIE] Remove unused variable

-- This commit trivially removes an unused variable in
AIRDependencyScheduleOpt.cpp.
Signed-off-by: Abhishek Varma<EMAIL_ADDRESS>The unused variable leads to build failure in IREE(iree-amd-aie) - hence raising this PR.

Ping @erwei-xilinx @fifield .


Add TraceID(ctx context.Context)

Related issues
#25

One suggestion, the documentation should update for this.

One suggestion, the documentation should update for this.

provided


Implement AI

Description
Implement AI on LivingEntityMock
Checklist
The following items should be checked before the pull request can be merged.

[x] Unit tests added.
[x] Code follows existing code format.

Info on creating a pull request

Make sure that unit tests are added which test the relevant changes.
Make sure that the changes follow the existing code format.

For maintainer
When a PR is approved, the maintainer should label this PR with release/*.

If the PR fixes a bug in MockBukkit, update the patch version. (if the current version is 0.4.1, the new version should be 0.4.2)
If the PR adds a new feature to MockBukkit, update minor version. (if the current version is 0.4.1, the new version should be 0.5.0)

Note that a PR that fixes an UnimplementedOperationException should be considered a new version and not a bugfix.

This is already implemented in #328. Not sure if it'd be better to merge this now or wait until then since 328 might take a while.

This is already implemented in #328. Not sure if it'd be better to merge this now or wait until then since 328 might take a while.

I would probably wait till that is merged. So I will close this PR now


Fix Timestamp subsecond handling.

Closes #933.
I added system tests to Firestore and Spanner which write a timestamp, reads it back, writes the read value and reads it back again, and compares the three values to prove that data is persisted back correctly. I also updated Firestore to make sure it handled timestamps in a consistent manner throughout the client. The firestore change is purely internal and does not impact users.

We had an issue around microseconds too:
https://github.com/census-instrumentation/opencensus-php/issues/159
Can you make sure it's working even you set the Locale to something like fr_FR.UTF-8?

@tmatsuo it doesn't seem to be a problem when run locally. I've added additional tests using setLocale.


[presentation service] List aggregate number of downloads on presentation page

On the main presentation sharing service page (https://osf.io/presentations), we list the names of conferences and number of submissions for each.

On each individual conference site (osf.io/view/spsp2015, for example), we list all talks and include number of downloads of each.

It would be great for marketing purposes to also list the aggregate number of downloads of all the posters/talks for a particular conference on osf.io/presentations, to show the impact the work is having.

Talk with @lyndsysimon. This will be included in the new piwik update.


alias flock?

Is this type of lock multi-process? Sorry but I don't get it. Can I use it for inter-process communication like flock module?
A real world example would be useful. Thanks

This locks can be shared between multiple processes (using storages: https://github.com/indutny/deadbolt/blob/master/lib/deadbolt/storage/memory.js ) , but as you can see I implemented only memory storage so far, so for now you can just experiment with it in a single process.
You can't use it for communication. Basically, deadbolt works with MakeWeak, so it's releasing locks automatically when your program has no reference to Lock instance.


Does auto size text support Android X ?

Question
Does auto size text support Android X ?
Version

Flutter version: [e.g. 1.7.8]
auto_size_text version: [e.g. 1.2.1]

Since auto_size_text has no platform dependencies, it does not care whether you use AndroidX or not.


Bugfix for dagster asset selection

What are the relevant tickets?
Tobias made the following changes to refactor how we are creating the AssetSelection for the elt pipeline.
Instead of using the "ol_warehouse_raw" group and collecting all the downstream assets:
selection=AssetSelection.groups("ol_warehouse_raw").downstream(),
We are now using "dbt_assets" loaded from the manifest.json to figure out what assets are upstream,
selection=AssetSelection.assets(*dbt_assets).upstream(),
Description (What does it do?)
Both the error handling suggestion and a few examples online suggest adding required_multi_asset_neighbors() to select any remaining assets to resolve this error. This will make sure that all the required keys for the given airbyte sync are included in that selection.
How can this be tested?
Deploy to QA
Check for error when loading code location in Dagster

I jumped the gun and pushed this exact change to main this morning before I saw that you had opened this PR. Good work :)


Division of device parameters into sections.

To facilitate device setup.

The location of the device in a separate tab, separate form.


ndt_matching: Query transform in between baselink and primary lidar through tf tree

If the query to tf tree fails, the code falls back to query ros param tf_baselink2primarylidar for the transform.
Only update ndt_mapping and ndt_matching since they are the only two nodes we are using.

See original MR for more details: https://gitlab.com/astuff/autoware.ai/core_perception/-/merge_requests/29

@icolwell-as can you have a look at the CI failures please? The patch failed to build.

I couldn't reproduce the test failures locally, but hopefully this commit fixes it.


verify_block not invoked when using environment variables

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Issue Description
Similar to #11017 I can confirm that values set via environment variables are not validated by the verify_block. This allows invalid values to get into the run of an action where the validation should be assumed.
Sample code
https://github.com/jaleksynas/VerifyBlockBroken
Command executed
fastlane test
fastlane test_env

Difficult from #11017 to retrace if this once worked differently or if there were just related changes that might have fixed (or not fixed) the issue back then.
This is where it seems to get the values from the option or ENV var alternatively:
https://github.com/fastlane/fastlane/blob/4ad2e58e41b11c8197f517d902bd0fbdb90f8e71/fastlane_core/lib/fastlane_core/configuration/configuration.rb#L219-L227
The verification seems to happen here:
https://github.com/fastlane/fastlane/blob/4ad2e58e41b11c8197f517d902bd0fbdb90f8e71/fastlane_core/lib/fastlane_core/configuration/config_item.rb#L207-L228
Which the first one connects to via set (valid? is wrapped in verify!):
https://github.com/fastlane/fastlane/blob/4ad2e58e41b11c8197f517d902bd0fbdb90f8e71/fastlane_core/lib/fastlane_core/configuration/configuration.rb#L265-L277
I don't really understand where the readinf of the var and the verification come together - but maybe that is the problem.

@janpio That is the problem!  Setting the value in other ways verify!'s  correctly.  It looks like the value from the env is read during fetch (but without the verification).
Adding && option.verify!(ENV[option.env_name]) on line 221 of configuration.rb does the trick.

That is a great first step!
Is this maybe where all the "normal" parameters are validated?
https://github.com/fastlane/fastlane/blob/4ad2e58e41b11c8197f517d902bd0fbdb90f8e71/fastlane_core/lib/fastlane_core/configuration/configuration.rb#L80-L89
I don't really understand the code flow here...

I believe that block is handling command-line arguments only, therefore values are each of the --key value pairs.

This is merged and released in 2.119.0 💪 Thanks for making the fix!


Jupyter notebook linting for UCL/stokes-nc-ill-posed

Relates to https://github.com/UCL-ARC/python-tooling/issues/49

Closed #2 as not a priority right now.


all: Use Go1.6 vendoring

This change continues to use Godep, but moves dependencies to the /vendor dir. Go 1.6 will now be required to develop.

Ahh, Go 1.6-style vendoring with Godep appears to be broken when there are dependencies on other packages that use Godep rewrite vendoring. I'm going to retract this for now.


Docs: Ensure published version uses the yard extension

We can generate YARD API docs, but when others do it, do they use our addons? Be sure that the standard invocation generates the right thing at rubydocs sites.

Update: No, the rubydocs sites do not take the addons in this project. They work, but they're non-standard. Publishing somewhere - like Github Pages, if that's a thing, would make this not a problem.
Especially the Config stuff is nifty to have generated.

Should we instead generate a README section on the Config options?

@olleolleolle we can try and see how well it works out.

@michaelklishin I should test-run the YARD thing and see what output I can get my hands on. Should be pastable.

Closing this, in favor of discussing further in #258.


feat: Add Coordinates endpoint

Close issue #164
Description

Added Coordinates endpoint location/co-ordinates/random/{qty}  and location/co-ordinates/country/{code}/{qty} to generate random coordinates.

Added Centroids for countries to generate coordinates within a specified country.

Hi @mdfarhaan great PR, just one minor change and then its ready to be pulled in.


Minimal effort  GAT prediction in docking workflow.

Adds a very basic attempt at GAT prediction in the current docking workflow.  Thought I would do an MVP first and separate from an in-progress a rewrite of the docking script.

Codecov Report

Merging #203 (9613d1a) into main (1a901aa) will increase coverage by 0.53%.
The diff coverage is 51.35%.

This MVP of adding the GAT prediction to the docking workflow is probably ready for a review @kaminow @JenkeScheen

@kaminow hopefully I have addressed your review. :)


Parallels Display Adapter DXVK driver

Hello.  I am a M1 MacBook owner who wants to run FFXIV.  As I understand it, the present Mac version is a CodeWeavers CrossOver wrapper around Wine.
IIRC, the pipeline is DX <-> DXVK <-> Vulkan <-> MoltenVK <-> Metal.  As an alternative, I installed Windows 11 ARM on Parallels 17 and the VM itself runs at bare metal speed (but no 3D acceleration).
The only problem is that the VM uses a Parallels Display Adapter to translate DirectX to OpenGL.  It reminded me of DXVK.
If this VM driver was just a wrapper around DXVK instead, it would be like Wine, but more stable/less buggy.  Maybe it would also be more performant.  But I am not a low-level coder.
@doitsujin is this something you can do on a contract basis, or anyone else with the necessary knowledge if he is not available?

I installed Windows 11 ARM on Parallels 17 and the VM itself runs at bare metal speed

No it does not. Parallels has a virtual GPU for Windows that works by passing the graphics calls to the host OS (Mac OS) and translating them to Metal.
It dies not support Vulkan at all, so needless to say, DXVK won't work on that.
As you correctly pointed out, Crossover is shipping a forked version of DXVK that works on Mac OS, so I suggest you use that.

As you correctly pointed out, Crossover is shipping a forked version of DXVK that works on Mac OS, so I suggest you use that.

Would virgrlrenderer with Vulkan solve the VM problem?

We're not interested in supporting Mac OS directly, and DXVK is not designed to be used as a Windows display driver since it reimplements the whole D3D11 runtime rather than the DDI.
That being said, I don't really see how running the Windows version of the game through a VM and then translating D3D11->Vulkan->Metal would be any better than running the exact same Windows version through Wine while translating D3D11->Vulkan->Metal. If performance is low, this is likely due to the combination of wrappers which are all running on top of x86 emulation, but none of this would change with a VM.

@K0bin The game runs at 25-30fps inside the VM which is pretty shocking, I did not expect it to even run.  I guess this has something to do with the unified video memory for the M1 GPU's.

I guess this has something to do with the unified video memory for the M1 GPU's

No, this has nothing to do with unified memory.

Ok, to make sure I understand this right, it is a direct D3D <-> Metal translation?

Parallels GPU driver is most likely a DDI (Direct3D device driver interface) to Metal translation layer.

We're not interested in supporting Mac OS directly, and DXVK is not designed to be used as a Windows display driver since it reimplements the whole D3D11 runtime rather than the DDI.
That being said, I don't really see how running the Windows version of the game through a VM and then translating D3D11->Vulkan->Metal would be any better than running the exact same Windows version through Wine while translating D3D11->Vulkan->Metal. If performance is low, this is likely due to the combination of wrappers which are all running on top of x86 emulation, but none of this would change with a VM.

I heard that the Apple M1 chip is able to directly run some x86 instructions with Rosetta 2, while Windows WOW64 is supposedly already slower because of having to work with 32-bit programs and having to work with less consistent development libraries and hardware.  On top of that, the VM does not run any x86 instructions directly.
Is that what you were referring to?


add missing translation.json

Changes / Comments

add missing root translation.json to be picked up by Crowdin

Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.


Conditional dependency using target_feature="neon" does not work

I've wanted to use huonw/simd in a Cargo.toml like this:
[target.'cfg(any(target_feature = "sse2", target_feature = "neon"))'.dependencies]
simd = {git = "https://github.com/huonw/simd"}

When I cross compile to ARM with the additional RUSTFLAGS=' -C target-feature=+neon' environment variable setting, the setting is not detected, and I get the "missing package: simd" kind of errors. Here's the related issue in Servo: servo/servo#10900.
It does work fine during a normal, x86_64 build (where sse2 is on by default by the way).

What version of the compiler are you using? Also, target_feature is currently a nightly-only feature, so this won't work on stable/beta

I'm working on Servo, which currently uses rustc 1.10.0-nightly (23ccaddaa 2016-04-23)

Aha! I think I know what the bug is gonna be.

@mmatyas I remembered having a brief look at this crate a few months ago on ARM, and NEON didn't just work plus I was getting a similar error (IIRC related to some import problem) so unless you actually saw it working natively, your positive x86_64 experience might be misleading.


Integrity error "(1048, "Column 'user_id' cannot be null")"

What i am trying to achieve is that i want to extend the profile model further to either teacher or student. in signup for i added a choice field where user select whether he is teacher or student. below is my model structure.

class Profile(UserenaLanguageBaseProfile):
     """ Default profile """
     GENDER_CHOICES = (
             (1, _('Male')),
             (2, _('Female')),
         )

     user = models.OneToOneField(User,
                                unique=True,
                                verbose_name=_('user'),
                                related_name='profile')

    gender = models.PositiveSmallIntegerField(_('gender'),
                                              choices=GENDER_CHOICES,
                                              blank=True,
                                              null=True)

class Teacher(Profile):
    profile = models.OneToOneField(Profile,
                                unique=True,
                                verbose_name=_('profile'),
                                related_name='teacher')

    home_address =  models.CharField(_('home_address'), max_length=255, blank=True)
    home_phone =  models.CharField(_('home_phone'), max_length=30, blank=True)
    cell_phone =  models.CharField(_('cell_phone'), max_length=30, blank=True)
    experience =  models.IntegerField(default = 0)
    summary =  models.TextField(_('summary'), max_length=500, blank=True)

class Student(Profile):
    profile = models.OneToOneField(Profile,
                                unique=True,
                                verbose_name=_('profile'),
                                related_name='student')

    grade = models.CharField(_('grade'), max_length=50, blank=True)

I am overriding the signup save method as:

def save(self):
        new_user = super(SignupFormExtra, self).save()
        new_user.first_name = self.cleaned_data['first_name']
        new_user.last_name = self.cleaned_data['last_name']
        new_user.save()

        if self.cleaned_data['teacher_or_student'] == 'teacher':
            teacher = Teacher(profile = new_user.get_profile())
            teacher.save()
        elif self.cleaned_data['teacher_or_student'] == 'student':
            student = Student(profile = new_user.get_profile())
            student.save()
        return new_user

When teacher.save() or student.save() method is called it raises an integrity error that "(1048, "Column 'user_id' cannot be null")" but i am not creating a new user instance here i am trying to assign the newly created profile_id to teacher or student model. I am doing in the wrong way?? what should i do?

This is not an issue with userena, for others, the answer has been given on Stackoverflow.

hlo i am  great man


Invalid dev version identifiers in setup.py

There is a bunch of invalid version matchers in setup.py.  PEP 440 states:

The canonical public version identifiers MUST comply with the following scheme:
[N!]N(.N)*[{a|b|rc}N][.postN][.devN]

So you are missing a dot and a number in every version identifier that contains the string dev.
It is also considered bad practice to have an upper bound on package versions and installers like pip do not typically consider development versions in any case (unless explicitly told to).
See: https://github.com/googleapis/google-api-python-client/issues/2151
See: https://github.com/googleapis/google-api-python-client/issues/2151

Closing as this repository doesn't contain setup.py


Watcher for mocked global property doesn't fire

Version
1.0.0-beta.31
Reproduction link
https://github.com/Marcelinka/vue-test-utils-watcher-bug
Steps to reproduce

Clone repository
In console run command yarn install
In console run command yarn test

What is expected?
Test will be passed
What is actually happening?
Test failed

I have global property for size of screen from package vue-mq. And I have watcher for this global property to add or remove event listener on scroll. When I try to test that addEventListener or removeEventListener are fired, I see that problem.
I try to set it by wrapper.vm.$mq = 'sm',  I try to use setData, I try to wait for nextTick (or flushPromises), it doesn't work.

The $mq mock does no create a reactive data property (and it's not supposed to), so the watcher can't watch it. So this behaviour is expected.
Since you left out the actual implementation of $mq in your reproduction I can't recommend other approaches.

This won't work since watch only works on reactive properties, and what you are doing with mocks is setting a static value (not part of Vue's reactive system).

This package counts width of screen and give me a breakpoint, but in unit-tests no screen and no width. That will be complicated to change it's value with functionality of package. And I need to test only watcher, but not this package. I don't really understand what you propose. Can you give me example, please?

Besides, why exactly mock property can't be reactive? I think it'll be useful for cases, when we use it in computed properties or watch for it.

Maybe we can do special method, like setMocks, that can force rerendering?

I was able to make this work. You can make something reactive by calling Vue.observable. Here is the code:
import Vue from 'vue'
import TestComponent from "./TestComponent.vue";
import { shallowMount } from "@vue/test-utils";

it('Test firing watcher for mocked global property', async () => {
  const mock = jest.fn()
  global.window = Object.create(window);
  Object.defineProperty(window, 'removeEventListener', {
    value: mock
  });

  const wrapper = shallowMount(TestComponent, {
    mocks: {
      $mq: Vue.observable({ val: 'md' })
    }
  })

  wrapper.vm.$mq.val = 'lg'
  await wrapper.vm.$nextTick()
  expect(window.removeEventListener).toHaveBeenCalled()
});

<template></template>

<script>
export default {
  watch: {
    $mq: {
      handler() {
        console.log('Watching')
        window.removeEventListener('scroll', this.resizeHeader);
      },
      deep: true
    }
  }
}
</script>

I had to add deep: true since $mq is an object, not a primitive value - I'm not sure if this is helpful to you.
The test deals with a lot of browser specific things that are not really applicable in the test environment (which uses jsdom, a headless browser, so it doesn't handle things like resizing) so I'm not sure how valuable writing a unit test for this feature is. For now, this is the best I could do. Hopefully you find this useful.
I don't know how much sense to make mocks reactive, since you can just use Vue.observable. There might be a use case where someone does not want their mock to be reactive.


simplify subnet_size()

is_ipaddr() returns the type of IP (4 or 6) if valid anyway.  Use this to simplify the function

Merged to master for 2.4, thanks @stilez


inline import duplication

If I require('sub1.css') and require('sub2.css') which both @import url("./base.css") then I'll get rules in bass.css applied twice in my page, which may cause some problem.
Any plan to solve this?

I'm still seeing duplicates (both with ExtractTextPlugin generated files, and the vanilla less loader which includes multiple style tags)
Is additional configuration required other than the following?
    loader: ExtractTextPlugin.extract('style-loader', 'css-loader!less-loader')

<EMAIL_ADDRESS><EMAIL_ADDRESS>

When you have multiple require() for different Less-files, it's the expected behavior to add multiple style tags.

@jhnns But I am importing the same .scss file, just in different React components. So why can't webpack identify that these are the same .scss files and only inject a

While the original issue here is related to sass/less/stylus, a fix/hack/patch for this is to use the optimize-css-assets-webpack-plugin in conjunction with extract-text-webpack-plugin to produce, like this: https://gist.github.com/phun-ky/766e5ec9f75eac61c945273a951f0c0b.
If you want to use this in dev, you will have to use a plugin like write-file-webpack-plugin to force webpack to write the file to disk in dev.


Create architecture document

Add documentation for how the various projects involved with java operators fit together. For initial implementation, this could just start in the "getting started" guide and be moved into its own section over time.

fabric
java-operator-sdk
java-operator-plugins
quarkus

@csviri

@laxmikantbpandhare

Created this doc, with the diagram, hope it helps:
https://javaoperatorsdk.io/docs/architecture-and-internals
will close this issue for now.


[CloudInit][HA-Setup] consider allowing external datastore to be specified with cloud-init

Is your feature request related to a problem? Please describe.
When trying to setup K3OS as HA with three Master Nodes, the dqlite is still experimental.
So I would like to use an external datastore beeing implemented in cloud-init
Describe the solution you'd like
I would like to see those implemented into cloud-init:
K3S_DATASTORE_ENDPOINT
--datastore-endpoint
K3S_DATASTORE_CAFILE
--datastore-cafile
K3S_DATASTORE_CERTFILE
--datastore-certfile
K3S_DATASTORE_KEYFILE
--datastore-keyfile

So that I can for example do the following:
[...]
k3os:
  datastore:
    cafile:
    cert:
    key:
[...]

Parameters can be found here
Describe alternatives you've considered
Using the CLI is still fine
Additional context
Since dqlite is experientel it would be nice to specify external datastore in the bootstrap process.

@Moep90 what you’re looking for is https://github.com/rancher/k3os#k3osk3s_args

@Moep90 what you’re looking for is https://github.com/rancher/k3os#k3osk3s_args

I couldn't find "datastore" there. Is it possible to just add it like the following:
k3os:
  datastore:
    endpoint: mysql://username:password@tcp(hostname:3306)/database-name
    cafile: my.ca.file.cert
    certfile: my.cert.file.cert
    keyfile: my.key.file.cert
 data_sources:
  - aws
  - cdrom
  modules:
  - kvm
  - nvme
  sysctl:
    kernel.printk: 4 4 1 7
    kernel.kptr_restrict: 1
  dns_nameservers:
  - <IP_ADDRESS>
  - <IP_ADDRESS>
  ntp_servers:
  - 0.us.pool.ntp.org
  - 1.us.pool.ntp.org
  wifi:
  - name: home
    passphrase: mypassword
  - name: nothome
    passphrase: somethingelse
  password: rancher

k3os:
  k3s_args:
    - server
    - "--datastore-endpoint=<datastore-endpoint-url>"
    - "--datastore-cafile=<cafile>"
    - "<all the other datastore options>"
    - "..."

I'm pretty sure this is what @dweomer meant @Moep90.

k3os:
  k3s_args:
    - server
    - "--datastore-endpoint=<datastore-endpoint-url>"
    - "--datastore-cafile=<cafile>"
    - "<all the other datastore options>"
    - "..."

I'm pretty sure this is what @dweomer meant @Moep90.

Yes, exactly.

Also, please consider https://github.com/rancher/k3os#k3osenvironment for k3s arguments that can alternatively be passed as environment variables. So, instead of:
k3os:
  k3s_args:
    - server
    - "--datastore-endpoint=<endpoint-url>"
    - "--datastore-cafile=<cafile>"
    - "--datastore-certfile=<certfile>"
    - "--datastore-keyfile=<keyfile>"
    - "..."

you would have:
k3os
  environment:
    K3S_DATASTORE_ENDPOINT: "<endpoint-url>"
    K3S_DATASTORE_CAFILE: "<cafile>"
    K3S_DATASTORE_CERTFILE: "<certfile>"
    K3S_DATASTORE_KEYFILE: "<keyfile>"

I tend to favor envvars, when available, because they let me unclutter the service command-line.

@dweomer / @zimme  thanks.
Thanks for your patience.
We already set up PoC clusters with our company so we're looking fwd for further development :-).
Happy first Advent!


Prevent class constructor renaming

When I use one-to-many relationship, my parent class gets renamed. For example if parent class has name Parent, this.constructor.name returns Parent1.
Is there a way to prevent such behaviour? It's a kind of unexpected thing. I haven't checked the source code yet, but I believe Parent1 can be stored in some WeakMap, instead of renaming original constructor name.

This is not done by the ORM, you probably have some bundler that does it.


Title: Request for Unraid Docker Template for Bolt.diy

Description:
Bolt.diy is an incredible project that has great potential to be widely adopted by the Unraid community. Many Unraid users would benefit from having a dedicated Docker template available in the Unraid Community Applications repository.
To facilitate this, I kindly request the creation of an Unraid Docker template for Bolt.diy. This would make it easier for users to deploy and manage Bolt.diy directly through the Unraid interface, fostering its adoption among the community.
References:

Unraid Community Apps: https://unraid.net/community/apps

Example Template: [Anything LLM XML](https://raw.githubusercontent.com/nwithan8/unraid_templates/master/templates/anything_llm.xml)

Thank you for considering this request, and please let me know if I can assist in providing additional information to support this feature!

Would be convinient!

That can easily be created as soon as there is an official Docker image published


new feature: Export ALL Excel worksheets to CSV file at once

Currently Import-Excel has the option to read data from specific worksheet. It would be useful if there is an option to get the data from all worksheets at once (may be using -ALL switch or ALL parameter name??). One scenario I have is exporting all worksheets to CSV file (Here is my current implementation ... https://gist.github.com/DaniSQL/3c6f0baab62c83256a8c89a1013beb6c )
If this functionality existed in Import-Excel, then it would have be possible to pipe it to Export-Csv and no need to write the above code linked in the gist(looping through each worksheet and then piping).
Or may be we it is better to consider a separate command to export ALL excel worksheets to CSV files in a single folder? Thanks!

Thanks for setting this up


Add "Cargo: New" command

Add support for "Cargo: New" command.
Usage:

Open workspace folder
Execute cargo new command (project or library)

Issues #169

@KalitaAlexey Thanks for review!

Hi @Gordon-F,
I haven't actually tested your solution.
I have some doubts.
I am going to check it later.
Regards,
Alexey

Hi @Gordon-F,
I have added your changes to my repository: https://github.com/KalitaAlexey/vscode-rust
Regards,
Alexey

@KalitaAlexey Thanks!  How about move your fork to organization ? And find second maintainer.

@Gordon-F,
I have thought about it. Can we have a chat in gitter chat instead of chatting here?
https://gitter.im/vscode-rust/Lobby


Add ability to use {filename} in aliases

By using {filename} in an alias, you can use the name of the currently
edited file in the resulting import statement. @lencioni suggested
looking at vim-projectionist 1 for inspiration. Which I did, but I
decided to not use much of it. The main reason being that projecting is
a different usecase, where you have configuration that apply on a large
set of files. We want to apply a configuration on a single alias.
To keep it simple, I only added the filename (without file extension) to
begin with. I can imagine usecases where you need more than just that
though (e.g. a relative path within the project, or a relative path from
a lookup_path). We can extend this functionality when those usecases
pop up.
[Fixes #76]

:+1:


Tool to assess effectiveness of Corona-Warn-App?

It would be nice if the developers team could provide a small tool (perl script or similar should probably do) to transparently download published diagnostics keys from the CDN. This would allow the public (including journalists) to make some independent assessment of the effectiveness of the Corona-Warn-App by comparing the number of published diagnostics keys (as actually retrieved from the CDN and directly corresponding to the number of people reporting their positive test result through the app) to the total number of newly-infected cases in Germany reported e.g. by RKI or other reliable sources (maybe taking into account a certain reporting delay).
In my opinion such a ratio would be much more relevant than any App Store download figures which will likely be published once the App has been launched.
Unfortunately your documentation still seems to be a bit fuzzy about the details of the download process/protocol for the diagnostics keys from the CDN.

Thank you for opening this issue.  The project team will review and comment back soon.

This should go a long way, once it is figured out where and how the key bundles are actually accessible.

Google also explains how to work with this data: https://github.com/google/exposure-notifications-server/blob/master/examples/export/README.md

We currently don't have anything planned in that direction. But it should be rather easy for the community to build something like that tailored to the respective use case - the source code is open and the architecture is well documented.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Something that is missing in the other systems currently running is a way to know early on which TEKs are injected that are test TEKs.
Something to consider when you roll out the system.

Here's a Python script that reads Diagnosis Keys .zip files ("Exposure Key export files" as specified here):
https://github.com/mh-/diagnosis-keys

Thanks. That is what I have been looking for. Now has the official download URL for the data already been published? I understand that the newly uploaded diagnosis keys will be published once a day in a single Zip file; is that correct?
Will the Zip file also contain fake/dummy/test entries? That would of course dilute the statistics.
When a person publishes his or her data or day X, at first 14 diagnosis keys will be uploaded (and ultimately published?) on that day (corresponding to days X-14 through X-1), plus a fifteenth one corresponding to day X will be uploaded on day X+1, correct?
Is a ZIP file for day X created in the backend and then provided for download at some time in the early hours of day X+1? (all dates and times referring to UT)
Did I get this right? Is there any documentation explaining the procedure?

Dear maintainers, given that there are valid questions raised by the community, it would be helpful if you could reopen the issue and answer them quickly.
Yes, the source code is open, but the documentation is not complete (of course it can't be this quickly). That's why we need insider help.
Please respond in particular to the "test key" question raised by @pdehaye
Also, do we know what the official server is up, what is the URL?
Also, I don't like how @SebastianWolf-SAP says things are "rather easy" when there are valid unanswered questions.
I opened issue #258 not knowing about this issue - this discussion here is much more valuable. But since this one got closed, I will keep #258 open for now to show that there are questions to be answered.


How to distribute finished game to one directory with CI?

Is there an easy way to automate copying shared libs to a binary location for all three platforms (win, osx, linux)? For example for Travis? The problem is that when i use cargo build --release, it puts shared libs in paths like build/tcod-sys-395905244d0762cd/out, and i don't know what commands i should use to create a release ZIP for all platforms.

Sorry, I'm not sure :-(. There's OUT_DIR which is accessible in Cargo build scripts, but I don't know how to get one for a dependency (which is what you want here).
Here's what I used to do in my game when I was using tcod:
https://gitlab.com/tryjumping/dose-response/blob/911e420c13e63cf903f6ddfd5b0e31cfc2f8d82f/ci/before_deploy.sh#L23-29
It's not super pretty, but it worked. This was before any of the CIs (that I knew) had Windows, Linux and Mac support so I had a separate scripts for each platform.
Note that at least on Linux (and I'm pretty sure Mac too) just copying the DLL next to the binary does not work. You need to either set the right environment variables (e.g. LD_LIBRARY_PATH) or configure your binary to look in the right place.

This is very important information, because the task is very popular in home gamedev, but the information is impossible to find. Thank you, you really helped. I will try to finish my CI config.
Maybe some of this information should be added to the README?

Yeah absolutely. Would you be open to putting up a pull request at some point?
I'd prefer to have it in the readme (or a separate file linked from there if it's too big) -- that way it's always there if you clone the repo.

Yeah absolutely. Would you be open to putting up a pull request at some point?

Yeah, I'd like to, but not soon. I need to resolve this issue in my playground project first.

I'd prefer to have it in the readme (or a separate file linked from there if it's too big) -- that way it's always there if you clone the repo.

OK 👍


[SPARK-31372][SQL] Check the validity of built-in function aliases

What changes were proposed in this pull request?
Although SPARK-30184 Implement a helper method for aliasing functions, developers always forget to using this improvement.
We need to add more powerful guarantees so that aliases outputed by built-in functions are correct.
This PR found four issue and resolved it.

ln
negative
div
position
bit_count

Why are the changes needed?
Ensure that the output alias is correct
Does this PR introduce any user-facing change?
'No'.
How was this patch tested?
Jenkins test.

Test build #120972 has started for PR 28154 at commit 74d919a.

Merged build finished. Test PASSed.

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

Test build #120974 has started for PR 28154 at commit 1331c8d.

Merged build finished. Test PASSed.

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

Test build #120974 has finished for PR 28154 at commit 1331c8d.

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

Merged build finished. Test FAILed.

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

Test build #120972 has finished for PR 28154 at commit 74d919a.

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

Merged build finished. Test FAILed.

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

Because the expression rewrites the SQL function in a variety of ways, there is no way to automatically identify whether the alias of an expression can be displayed correctly in the schema.
I will close this PR.


fix(region): host search by any ip

这个 PR 实现什么功能/修复什么问题:
修正：host通过ip搜索ip地址，access_ip或ipmi_ip等字段

是否需要 backport 到之前的 release 分支:

release/3.7
release/3.6

/cc @zexi @yousong
/area region

Codecov Report

Merging #10205 (b459bc7) into master (3587659) will increase coverage by 1.91%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #10205       +/-   ##
===========================================
+ Coverage    5.93%    7.84%    +1.91%
===========================================
  Files         860      110      -750
  Lines      133693    13364   -120329
===========================================
- Hits         7939     1049     -6890
+ Misses     125048    12195   -112853
+ Partials      706      120      -586

Flag
Coverage Δ

aFlag
7.84% <ø> (+1.91%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/compute/models/policy_definition.go

pkg/compute/guestdrivers/base.go

pkg/mcclient/options/dbinstance_database.go

pkg/baremetal/tasks/create.go

pkg/compute/models/managedresource.go

pkg/multicloud/aws/region.go

pkg/multicloud/aws/dbinstance.go

pkg/cloudcommon/db/quotas/handler.go

pkg/keystone/models/default_admin.go

pkg/multicloud/ucloud/region.go

... and 738 more

Continue to review full report at Codecov.

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


Hycomv2 3 First source code push

In order to make sure that the right things get in I will push the new version in chunks. First is the source code.

@Jipingnersc Thank you.  I will update it

@Jipingnersc I have updated some issues here and there. It is running a longer test now, thus it compiles and run.
The issue that you found was that I moved the name of the folder from 3.1 to 2.3 and forgot to change it in the script.
Note that I changed a few values back from yours (if I remember correct). I think that we should make them variables and add them to the name list if they are to be tuned. The parameters are
ice_ruf in ice_constant.F90
cos(w) and sin(w) in ice_dyn_shared.F90
Pstar in ice_mechred.F90
The values that you added are currently commented out.

@Jipingnersc I have updated some issues here and there. It is running a longer test now, thus it compiles and run. The issue that you found was that I moved the name of the folder from 3.1 to 2.3 and forgot to change it in the script.

Thanks Till for your response. I will try it again and feedback to you later.

Hi Till,
I am trying to run a short model run after model compiling.
Could you share blkdat.input used in that case?

@Jipingnersc I have added a new experiment folder exp_02.3. The files needed should be in there

forcing.surtmp.a I dont have one. It is only read if one of the following flags are set:
surtmp: Did you set lwflag = 2 or sstlfg = 2 or icmflg = 2  ticegr=0.0?
I dont need it.
This is the standard restart naming. It should be fairly easy to change. Someone at NERSC has probably at some point found it annoying that one have to link the restart_out to restart_in

Thank Til. The new run with hycom2.3 has been done on Betzy, the related result will be compared with the current version.
Now the main differences are highlighted on the salinity near sea surface. So based on the initial checking into the code, I found one bug in thermf.F90 of  hycom2.3 at Line 1939 as follows:
if     (abs(sssdif).gt.abs(sssrmx(i,j))) then  !  large sss anomaly
if     (sssrmx(i,j).lt.0.0) then
sssdif = 0.0    !  turn off relaxation except under ice
elseif (sssdif .lt. 0.0) then
sssdif = -sssrmx(i,j)   ! limit relaxation
else
sssdif = -sssrmx(i,j)   ! limit relaxation   (It should be sssdif=sssrmx(i,j))
endif
endif
In addition, the relevant salinity flux differences are presented in thermf.F90 also:
Line 92-93:                       salflx(i,j) =             sflice(i,j) +  & !ocn/ice frac. in coupler
sssflx(i,j)   !relax  evrywhere
But in the develop branch of hycom2.2.98, at Line 73-90:
if (sss_underice==.false.) then
if (covice(i,j)>=0.15) then
sssflx(i,j)=0.0                            !skip relaxation bellow sea ice
else
sssflx(i,j)=(1.0-covice(i,j))*sssflx(i,j)  !skip relaxation bellow sea ice
endif
endif
if     (rivers(i,j,1).eq.0.0) then  !usual case
salflx(i,j) =
&                      (1.0-covice(i,j))*salflx(i,j) +
&                                        sflice(i,j) + !cell average
&                                        rivflx(i,j) + !rivers everywhere
&                                        sssflx(i,j)   !relax everywhere
else !a river point
salflx(i,j) =                 salflx(i,j) + !might be a river
&                                        sflice(i,j) + !cell average
&                                        rivflx(i,j) + !rivers evrywhere
&                                        sssflx(i,j)   !relax evrywhere
endif !rivers
So, should these differences be implemented in HYCOM2.3 or during the later development upgrade? Let us discuss this at the group meeting.

I fixed the bug and rewrote the NERSC_HYCOM_CICE part to a more compact version without if statements. It should be the same but that should be tested.


Rendering of CartPoleVectorEnv is broken

Describe the bug
Hi,
the rendering of CartPoleVectorEnv is broken in two places:

In line 548, self.clock is assigned instead of self.clocks
In line 561, self.state is used instead of state

Because of these two bugs, CartPoleVectorEnv.render() throws exceptions. I will create a PR in a moment.
Best,
Tim
Code example
No response
System info
No response
Additional context
No response
Checklist

[X] I have checked that there is no similar issue in the repo

Just noticed that I overlooked #872, which addresses this issue already but still needs a pull request. Apologies for the duplication! Should I close this issue in favor of #872?

Thanks, for the issue. My proposal was to actually not contain "human" rendering in the custom vector environment as this is difficult to maintain.
Rather, we should have a wrappers.vector.HumanRendering for which all "human" vector rendering is computed through.
This still requires rgb array rendering in the vector environment but this feels like a simpler solution for most vector environments.
Thoughts @TimSchneider42 and maybe @Kallinteris-Andreas  or @RedTachyon

I like the idea of solving human rendering with a wrapper. At least in this case, I expect it to be easier to implement and maintain, since the rgb_array image simply gets rendered by pygame.

Agreed. I have been thinking about how to implement the vector human rendering but I haven't come to a satisfactory general solution.
If you have any idea, I would love to hear them

I think for most environments we could simply render in "rgb_array" mode and then have the wrapper you proposed render all of them side by side in a single window with pygame. For more sophisticated environments, like this one, though, there should still be an option for custom rendering solutions, like just showing the simulator interface without having to render RGB images.
In general, I think that human rendering should be detached from RGB rendering in the API. I just wrote a small proposal about that in #1010.

Can this be closed now?

Yes, it is fixed


feat: add modal component

feat: add modal component
fix up styles

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#240  👈
#233
#231
#229
#228
#227
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @connorlindsey and the rest of your teammates on  Graphite


react on a supply from a sequence

The problem
I asked this question on StackOverflow:
https://stackoverflow.com/questions/55089250/react-on-a-supply-from-a-sequence
about the different behavior of Supply.interval and (some infinite sequence).Supply.
Suggestions
Since the answer kindly provided by @raiph is "just a guess," I think it would be useful to flag this behavior in the docs. From my naïve point of view, both supplies deliver an infinite number of values, so it's not clear to me why they behave differently. (Is it a "trap?")
I can't provide a solution, because I think that this needs a deeper investigation on Rakudo internals.

Well, one of them is infinite but "lazy". Let's see what can be done.


Add link to demo figma file

[x] Add link to demo figma file
[x] Improve demo figma file

Fixed in v5


Improve unbounded channels

I've added benchmarks and it turns out that comm channels are sometimes faster than stdlib channels. Unfortunately I messed up the commit message because I tried to use markdown syntax. Here is what it's supposed to look like:
# Where comm is faster than stdlib:

mpsc unbounded async_comm        45177
mpsc unbounded async_stdlib      51012

spsc bounded   async_comm        16160
spsc bounded   async_stdlib      19679

spsc bounded   sync_large_comm   31155
spsc bounded   sync_large_stdlib 67057

spsc bounded   sync_mid_comm     52558
spsc bounded   sync_mid_stdlib   68266

spsc bounded   sync_small_comm   424309
spsc bounded   sync_small_stdlib 544187

spsc one_space comm              7077
spsc one_space stdlib            8626

spsc unbounded async_comm        26718
spsc unbounded async_stdlib      30650

# Where comm is slower than stdlib

spsc unbounded sync_comm         36019
spsc unbounded sync_stdlib       29139

mpsc unbounded sync_comm         60856
mpsc unbounded sync_stdlib       44723

# Possible reasons

- Something in the sleep/wakeup handling: Maybe. But maybe not because
  our sync stuff is faster in the bounded channels.
- Node caching: The stdlib reuses old nodes instead of allocating new
  ones.

This is already looking quite good except for the unbounded+sync versions. Note that this doesn't touch Select and that the mpsc tests only test with two producers.

Cool! Maybe you should try to merge with the standard library? There seem to be mostly advantages over the current implementation.

I don't believe channels should be in the stdlib.


Feature: Add proptest

Note: I'm putting this one up as an early draft prior to sinking more time into it. If y'all like it, great! I'll keep going. If not, I'll be happy to throw away this PR & branch.
I've previously mentioned being interested in adding property-based testing using proptest to do the heavy lifting for us. So far, this PR adds:

two new dev dependencies, viz.

proptest to do the testing,
and paste to do some macro naming tomfoolery

one new testing macro called roundtrip_proptest! to assert that parse_thing(lex(thing.to_string)) == thing
two examples of using roundtrip_proptest! to start, one for parse_qubit and one for parse_variable_qubit

The roundtrip_proptest! macro takes a name, a proptest generator of valid data, and a parsing function:
roundtrip_proptest!(thing, any<data>().prop_map(Thing::stuff), parse_thing);

and expands that into
mod thing_roundtrip {
    use super::*;
    use proptest::prelude::*;
    proptest! {
        #[test]
        fn roundtrip(datum in any<data>().prop_map(Thing::stuff)) {
            let string = datum.to_string();
            let lexed = lex(&string);
            let parsed = parse_thing(&lexed);
            prop_assert!(parsed.is_ok());
            prop_assert_eq!(datum, parsed.unwrap().1);
            }
        }
}

As an example, I found something that wasn't clear to me from the docstrings when running these tests. The docs for parse_qubit say that a qubit designator can be

an integer (1), variable (%q1), or identifier (q1).

So I wrote my test:
roundtrip_proptest!(
    qubit,
    prop_oneof![
        any::<u64>().prop_map(Qubit::Fixed),
        "%[a-z][a-z0-9]+".prop_map(Qubit::Variable)
    ],
    parse_qubit
);

and it failed!
<SNIP>
thread 'parser::common::tests::qubit_roundtrip::roundtrip' panicked at 'Test failed: assertion failed: `(left == right)`
  left: `Variable("%a0")`,
 right: `Variable("a0")` at quil-rust/src/parser/common.rs:255; minimal failing input: datum = Variable("%a0")
<SNIP>

I didn't know that the lexer removes the percent symbol. Changing the generator for that case to "[a-z][a-z0-9]+".prop_map(Qubit::Variable) makes the test pass.

Neato.
The docstr for parse_qubit is wrong. A qubit is an identifier (e.g. q1) or an integer (e.g. 1). The %-prefix is strictly for circuit parameters.

In the latest two commits (6b8f3dd mainly, 75a28b3 is a short cargo fmt fix), I've simplified & renamed the roundtrip testing macro and added more tests. The format is now
parser_roundtrip!(parse_thing, generate_thing());

where parse_thing is a predefined parsing function you want to test, and generate_thing is a generator defined as in
fn generate_thing() -> impl Strategy<Value = Thing> {
  todo!()
}

See the bottom of src/parser/common.rs for examples. One can also put native proptest generators directly in that spot, but I opted for writing functions for legibility.
In running these, some test failures arose, either highlighting bugs in the code or (much more likely) my own ignorance.

parse_arithmetic_operand fails to correctly (?) parse the string version of ArithmeticOperand::LiteralReal(0.0), instead returning ArithmeticOperand::LiteralInteger(0). Should we allow this to occur, in which case I should fix my test?
parse_frame_identifier fell to an even more interesting trap: it looks like the underlying parse_qubit fails if
A. a FrameIdentifier has a Qubit::Variable as one of its qubits, and
B. that qubit's name is the string version of a poorly behaved float, like "inf" or "nan"

This can be surfaced by a simpler unit test:
 #[test]
fn bad_variable_qubits() {
    let datum = Qubit::Variable("inf".to_string());
    let string = datum.to_string();
    let lexed = lex(&string);
    let parsed = parse_qubit(&lexed);
    assert_eq!(datum, parsed.unwrap().1);
}

fails with
Error(ExpectedToken { actual: Float(inf), expected: "$ expected_variant" } (at Float(inf)))

What do y'all think, dear reviewers? In 1 above, should parsing return an integer when the data started out as a real? In 2, should I tweak the generator for variable qubits to filter out "inf" and "nan"?

I'm not totally sold on this type of testing... but it's already caught some bugs so maybe it is valuable. The current "bug" is that 0.0 is printed as 0 which is interpreted as a LiteralInteger -- does that constitute a bug? Wherever we expect a LiteralReal I think it's probably safe to allow a LiteralInteger too, and in this case nothing is lost.

@notmgsk totally understandable! I recognize I’m a fanboi of property testing (repeatable randomized generation plus automatic test case shrinking/minimization! 🤩), and I’ll happily hush up about it if the addition here seems more trouble than it’s worth.
As for the literal integer/real-ness of 0.0, bowing to subject matter expertise and adjusting the generation is absolutely an option. As is completely nuking this PR of course!

This is cool, and I think it's something that would be great to have in the library! Adds new coverage with little code. As you note though, round-tripping might find a lot of formatting quirks rather than parsing bugs. I'd say that shouldn't bar this from inclusion right now, if the tests are green when it's merged.


[WiP]: shell: Update MQTT shell backend

Update MQTT shell backend

Closing for now pending interest from users/customers


Separate creation of training data and eval data and start eval_state skeleton

This PR

[x] Separates the creation of training data and evaluation data
[x] Creates eval_state.py, an analogous script to forecast_state.py, to be used for scoring with new data.

To do:

[ ] Verify setup_job.py and forecast_state.py still work.
[ ] Verify post-processing R code works with new file name: data.csv -> eval_data.tsv
[ ] Create new setup_job-like script that runs eval_state.py

For later:

[ ] add scoring to new setup_job-like script that runs eval_state.py

This was incorporated into https://github.com/CDCgov/pyrenew-hew/pull/102


send init command to terminal

I am opening the teminal in a iframe
From the parent page JScript I can access the xterm-  textarea.
I tried to chnage the value but the command does not get send.
I would like to send an initial command to the shell right after I have the prompt.
Which term JAVA script function I can use to do that on the HTML element terminal-*.
Thank you

-a, --url-arg           Allow client to send command line arguments in URL (eg: http://localhost:7681?arg=foo&arg=bar)

run ttyd with the --url-arg option and a shell script as start command, then you can receive the url params via command line arguments in the script.

Thank you. I figure it out form the source code mean while.


In fact, there is no such file of src/platform/OpenThread/BUILD.gn. Maybe we can add the source set later but not in this PR.

          In fact, there is no such file of `src/platform/OpenThread/BUILD.gn`. Maybe we can add the source set later but not in this PR.

Originally posted by @wqx6 in https://github.com/project-chip/connectedhomeip/pull/33872#discussion_r1675381200

@wqx6 PTAL

I cannot add a source_set for OpenThreadBorderRouter delegate in src/platform/OpenThread/BUILD.gn. The sources use the headers such as platform/ThreadStackManager.h which are in the static_library ${chip_root}/src/platform:platform. If we make the thread_br source_set depend on the static_library ${chip_root}/src/platform:platform. There will be a dependency cycle error. I guess that is the reason why we didn't create BUILD.gn file in src/platform/OpenThread before.
ERROR Dependency cycle:
  //third_party/connectedhomeip/src/platform:platform ->
  //third_party/connectedhomeip/src/platform/ESP32:ESP32 ->
  //third_party/connectedhomeip/src/platform/OpenThread:border-router-delegate ->
  //third_party/connectedhomeip/src/platform:platform


Browser stalls during development saves

Minimal reproduction of the bug with instructions:
Expected behavior:
Other information:
I would be willing to submit a PR to fix this issue:
[ ] Yes (Assistance is provided if you need help submitting a pull request)
[ ] No

Thank you for the starter - very good project for learning Angular techniques.
While experimenting with my own components I occasionally omit a bug or a typo and that gets the entire application into stall.  In browser debugger the count of errors goes to thousands and the browser stalls. The only way to get out of it is to kill the process. I am assuming this may have something to do with the Redux library. Is there a way to bypass it for development or instruct to simply stop on first error?

Hi, do you got a reproduction of this? This shouldn't be happening... 🤔

Sorry, for the dealy. I tried now some errors and compiling on the original 'starter' project and I do not seam to be able to reproduce the behavior there. However, in my project based on the 'starter' where I have made some additions (not too many), I am experiencing the described issues. I tried to isolate it, but my experience with Angular and TS is still rather of a "semi-educated novice" and I could not get to the bottom of it. If someone is eager to dig around , I'll be happy to upload my project somewhere.

I think I figured out the way to reproduce the issue, so I am re-opening this one for your input. Thank you.
Please see:
 https://github.com/ngrx/ngrx.github.io/issues/17

Project went through major update to Angular 8, please open new issue if the behavior persists. ( Old issue cleanup)


how to add lang and access page ?

How to add language and how to access pages that have been created in 'admin/page'?

you can see this video on 5dmat-web
how to work with language
https://5dmat-web.com/video/832

thanks

how about access pages that have been created ?


Payload size

Hi, any way to increase message payload size > 8k?

Hello, I am not sure to be honest. In any case, this project is meant to be a proof of concept, and there are lots more considerations to be made before putting this system in production.


Idle timeout / automatic ping/pong

as option to context

how are we supposed to do ping/pong now? My chrome ws connection got disconnected every 10-20 seconds.

It will disconnect as per idleTimeout if you don't send or receive anything, so you either have to make the client send an empty message or the like, or make the server send something.
This issue discussed the autoPing feature to have the server automatically ping clients in time for idleTimeout - if you want me to work on that you can have the people at aax.com pay for the hours, you can reach me on email.


SSD1306 shield does not have obvious support for devices with I2C address of 0x3D

This is a continuation of the side-discussion from Issue #42881. The SSD1306 device supports I2C addresses of both 0x3C and 0x3D, but the current shield only supports 0x3C. This can be confusing for boards that default to 0x3D, like the Adafruit 128x64 board. Although this board can be configured for I2C address of 0x3C, this requires soldering across some jumper pads.
Some quotes from the discussion on #42881:
From jfischer-no:

Documentation for generic SSD1306 shield can be updated to mention that the default address is 0x3c.

From nandojve:

+1
I was wondering if could be valid add an specific overlay like adafruit_ssd1306_128x64.overlay which uses 0x3D address with proper documentation. This could complement @jfischer-no suggestion.

I think the original issue is pretty clear, but let me know if there are questions. I still have the Adafruit 128x64 board. The project I was thinking of using it for is a decent way down my list of ideas, but it would be nice to use Zephyr if I eventually get to it :smile:.


Remove blueprint_dir update in integration tests

The blueprint_name is not used anywhere now that deployment_name is used for the directory location, this PR removes the update and any other references to the variable.

[x] Is unit test coverage still above 80%?
[x] Have you updated all applicable documentation?
[x] Have you followed the guidelines in our Contributing document?

I will run integration tests before submitting, this is lower priority than other PRs, so I'm going to wait until they are passing or merged in.

I have assigned #583 to @heyealex which is somewhat likely to cause an easily resolved conflict with this PR.

I have assigned #583 to @heyealex which is somewhat likely to cause an easily resolved conflict with this PR.

Merge yours first, this is lower priority, and I can handle the conflict here.

I had to dismiss the prior approval to handle the conflict