Snapshotter does not create expected directory prior to extracting layer

Description
When pulling from a repository, layers download successfully, but extraction of layers fails with The system cannot find the path specified. The directory C:\ProgramData\containerd\root\io.containerd.snapshotter.v1.windows\snapshots\1 is expected, but the final leaf .\1 does not exist. Manually creating this subdirectory results in the same error on a subsequent pull, only a leaf of .\2 is expected and not present.
Steps to reproduce the issue

nerdctl.exe pull <repo>:<tag>

Describe the results you received and expected
Received:
time="2024-01-11T20:58:24-08:00" level=fatal msg="failed to extract layer sha256:<digest>: hcsshim::ProcessBaseLayer \\\\?\\C:\\ProgramData\\containerd\\root\\io.containerd.snapshotter.v1.windows\\snapshots\\1: The system cannot find the path specified.: unknown"
Expected:
Successful download and extraction of image layers.
What version of nerdctl are you using?
C:\..\nerdctl.exe version
time="2024-01-12T00:15:47-08:00" level=warning msg="unable to determine buildctl version: exec: \"buildctl\": executable file not found in %PATH%"
time="2024-01-12T00:15:47-08:00" level=warning msg="unable to determine runc version: exec: \"runc\": executable file not found in %PATH%"
Client:
 Version:       v1.7.2
 OS/Arch:       windows/amd64
 Git commit:    e32c4b023bf41e5c8325cfb893a53cefb5fc68ed
 buildctl:
  Version:

Server:
 containerd:
  Version:      v1.6.4
  GitCommit:    212e8b6fa2f44b9c21b2798135fc6fb7c53efc16
 runc:
  Version:

Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
c:\..\nerdctl.exe info
Client:
 Namespace:     default
 Debug Mode:    false

Server:
 Server Version: v1.6.4
 Storage Driver: windows
 Logging Driver: json-file
 Cgroup Driver:
 Cgroup Version:
 Plugins:
  Log: fluentd journald json-file syslog
  Storage: windows-lcow windows
 Security Options:
 Kernel Version:
 Operating System:
 OSType: windows
 Architecture:
 CPUs: 0
 Total Memory: 0B

Might be a containerd issue: https://github.com/containerd/containerd/blob/0a7c192ca95df3dac19b2b18d0f1f94083af9e14/pkg/unpack/unpacker.go#L384

@glerb which image in particular were you trying to pull?

This is an issue for any image.

@glerb (cc. @phhpe)
Thank you for bringing the issue to our attention.
I've been unable to replicate the problem with both version 1.7.2 and the latest version v1.7.6. To ensure that we address this issue, could you please confirm whether you are still experiencing the problem on your end?

Containerd 1.6.4 definitely should have been creating the snapshot directory automatically, so I'm not sure what could possibly have been going on here.
One possibility is that a virus scanner was blocking access to the new directory pending some scan activity, although that sort of this was more-commonly seen when trying to access just-extracted files, rather than new, empty directories. (That would explain why the final part of the error is "unknown" rather than "No such file or directory".)
Either way, this is definitely a containerd responsibility, not nerdctl, and barring a current-version reproduction case, we can probably close this report.


Blog Offline vs Hidden Blog

What's the difference really? There are hidden blogs that appear Online / Hidden but others appear Offline...

I noticed that today too. In general "offline means the blog is down, but yet blog is still active! Occurs with some hidden blog, not all. Did tumblr change something in the way tumblthree access to API? in fact download is no more possible when status is found "offline" by the program.

Now tumblr must be satisfied because they no longer be viewable in tumbex! I think this is their final goal.

It seems to affect some hidden blogs by now, but I'm really afraid tumblr expands this blocking to all flagged blogs! maybe next step?


how is the password stored?

Hi, I'm wondering what processes do you apply to the password once entered by the user. I know it's a mix of the salt in the settings file and that it depends of the user's login, and then bcrypted. But I don't know what's exactly the process.
Any help much appreciated.

The salt in the settings file is to keep older passwords working (version < 0.2).
If a user  has registered or changed their password in 0.3, it's just bycrypt, see https://github.com/sdelements/lets-chat/blob/master/app/models/user.js#L121

Doesn't it depend on the user id or something? I've tried registering several users with same password but different user id and email and the passwords in mongo are diferent.

@JesusTinoco nope, that's how bcrypt is designed.
If you don't mind me asking, what are you trying to do?

@JesusTinoco We follow best practices and salt and hash the password. I'm closing this issue as we have provided you the answer. Feel free to follow-up to clarify your situation.


pyodbc 4.0.9+ segfault with uwsgi

When I'm running an app with uwsgi (<IP_ADDRESS>, 2.0.14) and pyodbc (all versions starting with 4.0.9), I get a segfault like this:
uwsgi(uwsgi_backtrace+0x30) [0x46a800]
uwsgi(uwsgi_segfault+0x21) [0x46aba1]
/lib64/libc.so.6(+0x35250) [0x7f41c0a2e250]
/home/user/env/lib/python3.5/site-packages/pyodbc.cpython-35m-x86_64-linux-gnu.so(PyInit_pyodbc+0x292) [0x7f41b1ebe572]

Environment: CentOS 7 (3.10.0-514.2.2.el7.x86_64), Python 3.5.2.
uwsgi is running with enable-threads = true.
pyodbc versions prior to the 4.0.9 work fine.

Was also getting segfaults in Apache 2.2 + mod_wsgi + pyodbc4.0.11
(gdb) bt
#0  0x00007fabe88b6165 in import_types () at src/pyodbcmodule.cpp:223
#1  PyInit_pyodbc () at src/pyodbcmodule.cpp:1050
#2  0x00007fabefba9989 in _PyImport_LoadDynamicModule (name=0x7fabe8b468b8, path=0x7fabe8ada390, fp=<value optimized out>) at ./Python/importdl.c:88
#3  0x00007fabefba614f in _imp_load_dynamic_impl (module=<value optimized out>, args=<value optimized out>) at Python/import.c:2271
#4  _imp_load_dynamic (module=<value optimized out>, args=<value optimized out>) at Python/import.c:2247
#5  0x00007fabefb900ce in ext_do_call (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:4558
#6  PyEval_EvalFrameEx (f=<value optimized out>, throwflag=<value optimized out>) at Python/ceval.c:2878
#7  0x00007fabefb926b1 in PyEval_EvalCodeEx (_co=<value optimized out>, globals=<value optimized out>, locals=<value optimized out>, args=<value optimized out>, argcount=3, kws=0x7fabfb00cf28, kwcount=0, defs=0x0,
    defcount=0, kwdefs=0x0, closure=0x0) at Python/ceval.c:3588

Crap.  I spent the last hour setting up a CentOS 7 VirtualBox VM and yet it doesn't crash when I import pyodbc.
I can clearly see the code differences between 4.0.8 and 4.0.9 and they are exactly where you are seeing the crash.  But, I can't seem to reproduce it.  (I really don't want to setup an entire Apache instance to do so.).  Does it fail for you when you simply fire up python 3.5 and import pyodbc?

Also, please look at issue #109 - I've had lots of mod_wsgi issues over the years since it compiles in support for Python.  I'm not sure about uwsgi.

I think I know what it is.  Can either of you test with branch fix199?

I'm afraid the version from fix199 doesn't seem to work in my case.
Minimal setup to reproduce the issue:

Create virtualenv, do pip install uwsgi pyodbc.
Create wsgi.py file with:

import pyodbc

Create uwsgi.ini file with:

[uwsgi]
module = wsgi:application
enable-threads = true
master = true
processes = 2

Run the server: uwsgi --ini uwsgi.ini --http-socket <IP_ADDRESS>

@broundee This didn't fail for me.   I'm using CentOS 7 in a VirtualBox VM with Python 3.5.3 from IUS.
All I get is "unable to laod app" since there actually isn't an "app" function.  I assume that's expected.
Anything else you can think of that might make a difference?

@mkleehammer thanks for looking into this. I think I'll try reproducing it on other environments to see what components would be different.

Just checked with clean installations of CentOS 6 & 7: works fine. Must be something specific to our setup, I'll post an update if it'll come clear what's wrong.

I really think the changes I made above are related.  It was a couple of extra decrements in the exact code that failed.  Either that or there was a bit-size mismatch (wrong driver, etc.).
So I'm closing for now, but please do add a comment if you find something and reopen if you find something.

I am still experiencing this problem with 4.0.14. Had to roll back to 4.0.8. Ubuntu 16 + uwsgi latest

I can confirm this with 4.0.14 too, I'm on Ubuntu latest also, switch back to 4.0.8 works.

I encountered the same bug in a Pyramid project on 4.0.15, Python 2.7.5, CentOS 7.2. Downgrading to an older pyodbc version resolved it.
Not sure if it's really a bug in pyodbc but it's very frustrating as the only error you get in Apache logs is "End of script output before headers: " as mod_wsgi crashes.
Per https://github.com/mkleehammer/pyodbc/issues/109 confirmed mod_wsgi was built with the same Python used in the virtualenv.

mkleehammer, could you point me exactly where the code fails? I'll take a look.

@pkokliaguine
pyodbcmodule.cpp 223
uuid_type    = uuid.Detach();

If this is the error I've seen reproduced remotely, commenting out this line makes the error go away.  Wrapper.Detach hardly does anything, though, so I'm trying to reproduce it also so I can see what the compiler is generating.

@litchfield I just setup a new VirtualBox Ubuntu 16.04 and tried broundee's test above with just the wsgi.py and wsgi.ini.  It did not fail - does that simple test fail for you or is there something else that needs to be done?
Ubuntu 16.04
Python 3.5
uwsgi 2.0.14 installed via "pip install uwsgi" (in a Python 3.5 virtualenv)

@broundee I've now tried your test in a brand new CentOS 7 VirtualBox image.  (I added two print statements, one before and one after to print the python version.)
*** Starting uWSGI 2.0.14 (64bit) on [Fri Mar 24 18:23:32 2017] ***
compiled with version: 4.8.5 20150623 (Red Hat 4.8.5-11) on 24 March 2017 18:21:32
os: Linux-3.10.0-514.10.2.el7.x86_64 #1 SMP Fri Mar 3 00:04:05 UTC 2017
nodename: new-host-5
machine: x86_64
clock source: unix
detected number of CPU cores: 1
current working directory: /home/mkleehammer/uwsgitest
detected binary path: /home/mkleehammer/uwsgitest/venv35/bin/uwsgi
!!! no internal routing support, rebuild with pcre support !!!
your processes number limit is 3850
your memory page size is 4096 bytes
detected max file descriptor number: 1024
lock engine: pthread robust mutexes
thunder lock: disabled (you can enable it with --thunder-lock)
uwsgi socket 0 bound to UNIX address <IP_ADDRESS> fd 3
Python version: 3.5.3 (default, Jan 17 2017, 14:34:36)  [GCC 4.8.5 20150623 (Red Hat 4.8.5-11)]
Python main interpreter initialized at 0x14ca460
python threads support enabled
your server socket listen backlog is limited to 100 connections
your mercy for graceful operations on workers is 60 seconds
mapped 218280 bytes (213 KB) for 2 cores
*** Operational MODE: preforking ***
before
after: 4.0.15
unable to load app 0 (mountpoint='') (callable not found or import error)
*** no app loaded. going in full dynamic mode ***
*** uWSGI is running in multiple interpreter mode ***
spawned uWSGI master process (pid: 4261)
spawned uWSGI worker 1 (pid: 4263, cores: 1)
spawned uWSGI worker 2 (pid: 4264, cores: 1)

@litchfield @broundee Did both of you install uwsgi using pip or are you using a version installed with yum or apt?  Versions installed with pip should match the Python version but perhaps there is a chance that the distro provided are compiled against their default python version (2.7?).
I've tried using mod_wsgi also (which is more likely related to compile issues) but there are even more variables.
Any more ideas?

Now I've tested mod_wsgi on Ubuntu 16.04 with Python 3.5 with no failure.
I installed mod_wsgi via pip instead of the system mod_wsgi.  For those with a mod_wsgi crash - how did you install?  The same version of Python must be used to build mod_wsgi and all extensions imported into it.
There are definitely problems that need to be fixed, which I've started on the subinterpreters branch, but I cannot reproduce a crash.  If anyone can reproduce the crash, can you try the new branch?  And provide any steps necessary?

I've just checked 4.0.16 and it works fine in my case.
@mkleehammer, thanks for fixing this!

Quick check with 4.0.16 seems to be ok. Thank you!


I would like take up the task of improving documentation

Currently, most of the documentation is textual for some core modules without sample code and it's output. Do you guys think it's a good idea to have documentation with some sample code for core package? If yes, I can raise series of pull request to improve on those lines?
Also, thinking of providing a repl so a user can play in place without any installations?

Hey @anilreddykatta,
GitHub issues here are for reporting bugs. Ideas for how to improve something broader like documentation are best discussed on the elm-dev mailing list or the #elm-dev channel in slack


finalize FFMP rule implementation

Check that FFMP rule implementation in pywr is complete. e.g., a couple of rules currently pegged to aggregate storage need to use individual storages. Also need to figure out plan for salt-front based rules, as well as non-NYC reservoirs.

Commit 5/17: Added NJ diversion upstream of Trenton, and added the NJ diversion max flow as additional flow needed to be released by NYC to meet Trenton target within volume balancing formula in linked 10/4 commit in comment above.
Closing this Issue for now as overly broad. We have implemented the major pieces of FFMP. Remaining pieces (eg thermal releases or USACE resops) can be addressed in more focused Issues in future.


Collect and grade this revision does not work in Chrome

The collect and grade this revision button in the Submissions Repository browser does not seem to work in Chrome. Works fine in Safari for me. Probably related to the "Are you sure" javascript prompt.

Can't reproduce

Tried this on several machines and can't reproduce it.

Random thought: if the user sets it so a site stops showing alert/prompt popups, it's possible that it's just not shown so nothing will ever happen.

Good point, but I'm not sure if we can do anything about it.


Fix link in NashornMigrationGuide.md

Just signed the OCA.

Hi @hongshaoyang
thanks for your contribution. Once the OCA is official (you are listed on https://www.oracle.com/technetwork/community/oca-486395.html) I can merge this. Have you signed the OCA under this exact name "Shao Yang Hong"? Note that it might take a few days or weeks until you are listed there.
As this is a smaller change without code contribution, I could also merge it under my name, but I don't think it is that critical and don't want to take the contribution from you, so I'll rather wait until the OCA is resolved.
Thanks,
Christian

Have you signed the OCA under this exact name "Shao Yang Hong"?

Urgh unfortunately in the OCA I wrote my name as "Hong Shao Yang", how nitpick-y is Oracle about this?

That's fine - we'll just reference your comment above when updating the OCA database.


add type hints to tasks.py

Generate type hints database from types used during runtime:
pip install monkeytype
monkeytype run nox -s tests-3.6

See modules that can have types applied:
monkeytype list-modules

Apply types
monkeytype apply <module>

This exposed a circular dependency between session.py and manifest.py which was resolved in this PR. It ended up causing many LOC to be changed, but the change was cut+pasting code.
There are still several modules that haven't had types added yet since this PR got quite large.
Note that there are no type errors, unit tests did not change, and test coverage is still 100%.

honestly we can just disable flake8's import order checking.
On Mon, Aug 26, 2019 at 10:48 AM Chad Smith<EMAIL_ADDRESS>wrote:

Sounds good, I will do that in a different PR and then revisit this one.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/theacodes/nox/pull/241?email_source=notifications&email_token=AAB5I47EIL5YGSNWNXMBM43QGQJOLA5CNFSM4IPJUDLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5FDT2Y#issuecomment-524958187,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB5I42QFDFJOZD664IDKZLQGQJOLANCNFSM4IPJUDLA
.

Merged master into this PR. Now the diff is much smaller and only affects types in tasks.py.

Rad, thanks!


Refresh icon getting cut around round edges when stroke width is increased.

Hi,
Trying to edit style on px-icon 'px-vis:refresh' by modifying its width, but the icon is getting cut around the edges.
Below image shows how it is getting cut at top and left when icreased stroke-with to 2.5px:

We even tried changing stroke-width (stroke-width: 3px) on demo page, similar error was seen.
Please let us know how to handle this.

There is no way to handle it; it is just a problem if you increase the stroke-width too much and each icon will be different. Each icon is optimized for its size and changing properties may work, or clipping may occur


AIRFLOW-2787 Allow is_backfill to handle NULL DagRun.run_id

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2787

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

is_backfill will fail with a AttributeError: 'NoneType' object has no attribute 'startswith' error when called on a DagRun with NULL run_id (allowed by model definition). This PR introduces a None check to prevent the crash.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

extension to test_is_backfill to check the NULL run_id case

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Test failure is unrelated.

LGTM

Do you have a pass travis CI link for your pr in your local repo?

I have not seen a full set of passing tests yet but it is different CI targets that fail each time. The tests seem to be failing due to some sort of problem with kerberos setup before anything even runs.

thanks @gwax . Could you rebase the pr with master? There was an issue with the master which fails the CI. It just get reverted.

feel free to ping me once you rebase with master and we are good to go.

@feng-tao rebased

thanks @gwax . LGTM.


Add The Leffell School

Please :)

@TechnologyEli First stage of verification passed, official domain is leffellschool.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@TechnologyEli Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Alternative keyboard shortcuts for macOS

Follow up of https://github.com/godotengine/godot/pull/14892.
Adds several built-in alternative shortcuts for Mac (very useful on MacBooks).
They're added additionally and do not replace Godot's default shortcuts (which means a Windows Godotian on macOS could still use the Windows shortcuts, keyboard provided):

If a user changes a shortcut to non-default, the alternative shortcut will also go away (the latter will thus be also free to be set to another user-defined action).
The custom handling happens purely inside ShortCut, the rest of Godot should not see this replacement handling happening.
Hard coded alternative shortcuts for Mac:
Play: ⌘ B
Play Scene: ⌘ R
Play Custom Scene: ⌘ ⇧ R
Pause Scene:  ^ ⌘ Y
Stop: ⌘ .
Find Next: ⌘ G
Find Previous: ⌘ ⇧ G

please these shortcuts for macbooks
delete: backspace
delete with dialog: shift  + backspace

I would replace them, not add alternatives
On Dec 25, 2017 12:15 PM, "Razah"<EMAIL_ADDRESS>wrote:

please these shortcuts for macbooks
delete: backspace
delete with dialog: shift + backspace
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/15033#issuecomment-353875234,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2_z9-wylKUDy7OCe0DiOn4-3VToxks5tD7wigaJpZM4RMNvb
.

All basic macOS shortcuts are in place now; no longer as alternatives but as replacements.
@icnikerazah I've added Command + Backspace as a general replacement for Delete; I'm not sure about the dialog part - having a dialog or not for some deletion operations should probably be a different PR, as it needs to be changed in different parts of the code defining different deletion shortcuts.

@poke1024 thanks


[simulation] remove use of internal definitions/headers in radio platform

This commit updates the simulation/radio.c and removes the uses of
core internal macro VerifyOfExit() and replaces them with
otEXPECT_ACTION(). This ensures that radio platform layer does not
include the internal OT header like "common/code_utils.hpp".

Related PRs:

https://github.com/openthread/ot-nrf528xx/pull/90

This looks good to me. But I am also wondering if it's good to have some utilities accessible by both Core and platform.

I agree it would be useful. I think platform/util modules can be used for this. It is a group of common utility module that can be used by all platform implementation (and we can use and share core definitions in the implementation of the utility functions (e.g., we do this with util/mac_frame.hpp)). We can also consider exposing some common header files and definitions.
One thing to have in mind is that for now we should assume platform can be in C (same as our OT public/private APIs which are in C), so we cannot expose any C++ related definitions from any of these utility headers.


add mysql support

add mysql  and mongodb support as write database

Hi @aliaqa256, we'll internally discussed and concluded that we'll add mysql support, ship it this quarter. About the mongodb we'll definitely add it but not seeming in near future.

Hey @EgeAytin can I pick this up ?

@tolgaOzen can I pick this up?

any update for mysql integration?

Im also interested on mysql integration. Any updates?

yeah, i also want to work on mongodb integration


how i can create apk in local?

hi . i want to build apk in local .
please help me

Hi @vahidalvandi, the Quick Start Guide should have all you need to build an APK locally.

thank you .
Is this project (bubblrwrap) supported by Google?

It is built by Googlers, with help from people from the community, but not an not an officially supported Google product

I have been looking for a project for a long time. I hope it will last


Can there be a way to have pmodload look for modules somewhere other than ZDOTDIR/HOME?

It would be nice to be able to be able to store modules in places other than ~/.zprezto/, or better yet to store the entire .zprezto folder somewhere else, and maybe even give it a different name.
Right now the tree must look like this:
~
|-- .zshrc
|-- .zpreztorc
|-- .zprezto
    |-- modules
        |-- helper
        |-- git

It would be nice if I could configure Prezto to use this instead:
~
|-- .zshrc
|-- .zpreztorc

/usr/local
|-- prezto
    |-- modules
        |-- helper
        |-- git

or even
~
|-- .zshrc
|-- .zpreztorc
|-- .antigen
    |-- https-COLON--SLASH--SLASH-github.com-SLASH-sorin-ionescu-SLASH-prezto.git
        |-- modules
            |-- helper
            |-- git

I think this would be a good feature for a few reasons:

You could store all of the machinery inside /usr/local/prezto or something like that, leaving the home directory solely for configuration files.
It would make it easier for users to write their own modules without forking the entire Prezto project.
It would allow users of programs like Antigen to isolate sections of Prezto. Right now Antigen is getting pmodload to work from inside its own .antigen directory by permanently changing ZDOTDIR. See zsh-users/antigen/issues/137

Duplicate of #761, #271.


Crash with IC2, Base Minerals, Thermal Expansion, AE2 and Actually Additions

This crash was first spotted by one of our testers who also does some testing for Team CoFH. The difficulty in reliably reproducing it was recently solved when work was doing to find that the following list of mods - on Forge <IP_ADDRESS>4 or <IP_ADDRESS>1:
Actually Additions r104 or r105
Advanced Rocketry 1.1.1
Base Metals 2.5.0 beta1.55+
Base Minerals 2.5.0 beta1.10
CodeChicken Core <IP_ADDRESS>
CodeChicken Lib <IP_ADDRESS>
CoFHCore <IP_ADDRESS>
IC2 2.6.173-ex110
LibVulpes 0.2.0
MMD OreSpawn <IP_ADDRESS>
Thermal Expansion <IP_ADDRESS>
Thermal Foundation <IP_ADDRESS>
When starting a Server using the above selection, an error will trigger in Thermal Expansion where an item - from a support library inside Base Metals - is given two different wrappers and these wrappers hash differently, causing the GNU Trove code to crash.
Removing any mod from the listed set resolves the issue, as does changing Advanced Rocketry to the stable version (1.0.13 as of the last check of Curse). The fact that changing the version of Advanced Rocketry fixes the issue leads me to believe something in the code of 1.1.1 is broken badly. King Lemming has stated that this error is caused by a mod adding or changing stuff in the OreDictionary during the Post Init phase, but it might have a different cause.

Would it be at all possible to provide logs or a crash report so I can get a better sense of what is going on?
Also the only changes between 1.1.1 and 1.0.13 in the post init phase is the addition of a gravity handler being registered, the gravity controller machine being registered with the holoprojector, and loading of asteroid configuration.   So I doubt it's an issue with something during AR's post init phase.

None of them were much help - that mod-list was found through building up the inclusions one at a time. What I found truly odd was I tested that against the "stable" of Advanced Rocketry and got no crash, but with the Alpha the crash happens.
I've alerted you to this because none of us on the BaseMetals dev team can figure out what we could change to make this a non-issue and because it seems odd that it triggers only with the specific, specified version of Advanced Rocketry.

Can you just listen to z and give us a crash report? thx.

I tried building a test enviroment, however I cannot even find a build of Base Metals 2.5.0, nor even find any mods by the name of Base Minerals.  Without a copy of all of the mods and without any kind of information as to the nature of the error there's I really have to go off of.

Without a copy of the crash report there is nothing zmaster could do. Without a link to base metals 2.5.0 beta there is nothing zmaster could do. Does this happen with only libvulpus and advanced rocketry?
Sent from my iPhone

On Mar 14, 2017, at 8:24 PM, zmaster587<EMAIL_ADDRESS>wrote:
I tried building a test enviroment, however I cannot even find a build of Base Metals 2.5.0, nor even find any mods by the name of Base Minerals. Without a copy of all of the mods and without any kind of information as to the nature of the error there's I really have to go off of.
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@dmodoomsirius He did mention it is an error stemming from thermal expansion and that "Removing any mod from the listed set resolves the issue", so I doubt it.

@dmodoomsirius As I'm a dev for BaseMetals and ModernMetals I understand entirely. The problem is that the crash log makes no sense and KingLemming swears that the cause is a mod playing in the Ore Dictionary during Post Init. However... the trigger is actually some strange cross-mod interaction within the list of mods I've given.
@zmaster587 I wish I could be more help, but I can't figure out why its occurring. You can find the entirety of the 2.5 series of BaseMetals and related development files at http://maven.mcmoddev.com/ in 'com.mcmoddev'
On a related note, I think the issue might be that Applied Energistics 2 actually does muck about in the Ore Dictionary in post and something about that list of mods causes things to slide about just enough to trigger the error. What really, really puzzles me is that you can change BaseMetals settings and the bug goes away and the error indicates a BaseMetals item as being the cause, but the error cannot be replicated until, as noted, the above list of mods is used.
Anyway... I need to get back to working on the TODO list for BaseMetals, rewriting "Lootable Bodies" to make it easier to maintain and try to get stuff for my personal projects rolling. Any insights you've got into whats happening or tools you know of that could help localize the actual cause would be appreciated. A cross-mod timing bug like this needs solved, if just for the players.

Again give us a crash report otherwise nobody is going to be looking into this. We need info.... We need the crash reports. If your a mod dev you should know how important crash reports are....

I'll try again to build the test environment later.  Although I am also obliged to mention that I have very little time to work on stuff lately and it is rather inconvient for me to have to spend 15 minutes setting up a test bench to generate a crash dump you already have.  I also feel I should remind you that you are reporting this as a bug to me here, not kinglemming.
I understand that the crash report may seem confusing, but I must impress upon you that the info you've given me is equivalent to me coming to you and saying "base metals is crashing due to a hashing error somewhere, cpw says it might be related to registering a block in post init, I am withholding the crash dump as it is confusing.  Oh and it only happens with an exact set of [insert 10 mods here]".
Be that as it may, I am hoping to get an hour of time between work and sleep (6 hours is good) tonight when I will try to test this.
Also I would suggest to test your theory you can put a print statement in the ore registration event.  Then just look for ore regustation in the logs after post init.

All you have really told us is its broken.. Ok well if we had reports we can figure out a lead to see why its broken. Otherwise telling us basicly its broken and to fix it wont get us anywhere without a crash report.

Sorry, let me put it this way:  The crash report doesn't even hint at what is actually happening. I'm not even sure that the one means we found of reliably reproducing it with a minimal set of mods is the only method.
I filed this to alert the dev team here that this was occurring so they'd know if someone came to them reporting similar - I have a very bad feeling that this is one of those bugs that would take every dev team from all indicated mods working together to actually find the cause - and after the interactions I've had with King Lemming I cannot see that happening.
The biggest reason I brought it to your attention was that it does not occur for me - or anyone that has tested this - with your current stable, only the alpha release. This could, very likely, be exactly because the alpha has the added code which is causing the timing to hit perfectly - I really do not know and have not been able to figure it out. (and I've been working on this since first reported to me by the tester who discovered the bug a week ago... I've almost given up, because it makes no sense! 'Turn off this bit of config in this mod and the bug disappears', 'take out this mod and the bug disappears', 'change the version of this mod and the bug disappears' - it's gotten to the point that I'm sick of looking at code trying to figure out why it is occurring)

Unless there's mutlithreaded loading going on there shouldn't be any timing involved.  However it might be possible that the mod load order has changed (I'm not familar with the exactly algorithm forge uses to determine load order so it's a bit of a pot shot).
Anyway, I just got off work and I'll try sticking a test bench together to see if I can peice it together.

The crash occurs at the "serverAboutToStart" phase, so there's no reason adding an oredict entry in post init would do this.  I have also confirmed that at least one of the culprits is "dustNickel".  There appears to be two items, one that is registered to the ore dictionary and one that is not.  Both objects however are type "com.mcmoddev.lib.item.ItemMetalPowder".
Essentially it's saying there are two instances of itemstack (ItemMetalPowder , meta:0), one has the forge ore id of 691, the other does not have an oreID.
Procedure:
From the stack trace generated on launch:
Object 1: ... hashCode= 691 ... m:0 i:com.mcmoddev.lib.item.ItemMetalPowder
Object 2: ... hashCode= 348454912 ... m:0 i:com.mcmoddev.lib.item.ItemMetalPowder
Also given:
"public int hashCode()
{
return this.oreID != -1 ? this.oreID : super.hashCode();
}"  --from ComparableItemStack in COFH core
we can determine if an oredict entry exists for the item then the hash is the oredict number.
Also given the fact super.hasCode is "return this.metadata & 0xFFFF | getId() << 16;" we can deterimine the itemID which I've determined to be 5317 by taking 348454912 = 0x14C50000 >> 16 => 14C5 = 5317 (will likely differ on your world due to forge's item id mapping)
Because the hashcode is the oreID in the comparableItem stack class means I can do "logger.fatal("########################" + OreDictionary.getOreName(691) + "##############################");" to print out the item name.  "########################dustNickel##############################"
The code:
"Item item = Item.getItemById(5317);
logger.fatal(item.getUnlocalizedName());"
yields
"item.basemetals.nickel_powder"
That's about as far as I've got right now

Yep. You've gotten a touch farther than I had, but the results you've found are about what I expected.

Thos can probably be much more easily solved by injecting a trace printout
in the method responsible for adding items to the registration list for the
furnace.  From there we can know exactly where the addition is made

Next time i dont care how confusing the crash report is give it to us because otherwise its a waste of time trying to collect info you already have.

We have busy lives and cant afford to spend the time debugging to generate a crash report that already exists in this case it helped us

@zerotheliger I think that's been said enough times, please chill

Something else odd I just found is that nickel powder, the itemstack registered for the furnace recipes, does not appear to have an ore dictionary entry associated with it, however previous dumps indicate there also exists an item stack with the same item registered in the ore dictionary, so that does indicate two versions are floating around.

So.. this is really nasty.  I have discovered that a majority of basemetal's items have furnace itemstack recipe entries that do not have ore dictionary matching, but the new ItemStack(getItem,1,0) does.
This can occur if you're using metadata to store multiple types per item, but given the names like nickel_powder I assume this is not the case.  This could very easily result in two versions of the item floating around, one pulled the the furnace recipe handler (Thermal expansion does do this) and a version of dustNickel pulled from the ore dictionary registered to the same item (different item count/meta/NBT?).

I think I just found your problem.  If you register an item as a furnace input rather than an item stack it is registered as such:
    public void addSmelting(Item input, ItemStack stack, float experience)
    {
        this.addSmeltingRecipe(new ItemStack(input, 1, 32767), stack, experience);
    }

So if you have your oredictionary entry as ItemStack(Item_Nickel_powder, 1, 0) or equivilently ItemStack(Item_Nickel_powder), you suddenly can have two variations floating around if a furnace loads oredict dust -> ingot AND explicitly registered furnace recipes.

hrm... changed all smelting registrations to not use the raw item - and thanks for this hint, if it works I'm going to credit you, @zmaster587 and your whole team for the fix. It's the least I can do.
(five minutes later)
And... no change.

would you be able to toss me a build of the mod with the recipes changed for me to test with?

Not a problem. I'll attach it here...
BaseMetals-1.10.2-2.5.0-beta1.jar.zip
Just remove the '.zip' - I stuck that on to get around GH's restrictions when I remembered that Jar's are already zip-files.

Alright, will do

I tested it and it still crashed and outputted the same dump for the recipes as I had linked earlier.  I also opened up the file to check to make sure the code had been changed and found that it had not.
com.mcmoddev.lib.init.Recipes.java:195 still has the line "GameRegistry.addSmelting(material.powder, new ItemStack(material.ingot, 1), oreSmeltXP);" in BaseMetals-1.10.2-2.5.0.beta1.jar.  Which is where I think this problem is coming from.

huh... I changed the code to not have that... let me make sure the "./gradlew clean" actually goes through.

Okay, with the recipe fix (thanks for that hint) and using a variation on your furnace recipe dump (I have come to habitually loop a maps entrySet() instead of its keySet() - YMMV) things are now showing as I'd expect them to in the ore dictionary.
[17:14:11] [Server thread/FATAL] [basemetals/basemetals]: 1xitem.basemetals.nickel_powder@0 => 1xitem.basemetals.nickel_ingot@0
[17:14:11] [Server thread/FATAL] [basemetals/basemetals]: Ore 1:  dustNickel
[17:14:11] [Server thread/FATAL] [basemetals/basemetals]: Ore 2:  dustNickel

However, the issue persists. I have no more ideas now than I did, but had hoped this change would work. Perhaps I should spread the recipe change further...

Alright. I have no more ideas - I've traced the BaseMetals code, I've dumped the furnace recipe list, I've dumped the entire ore dictionary... About all I can think is that there is something strange going on in the CoFHCore "Comparable" wrapper class.

Mind tossing me an updated build to work with?  Maybe I can find something?

Most up-to-date is now on the Maven - this one includes the changes to the recipe code you suggested. http://maven.mcmoddev.com/  (in com.mcmoddev)

Doesn't appear the change had any effect on the error.. same two hashes and items
Looks like Thermal expansion loads default recipes (which includes dustNickel and would add the first recipe) first thing is post init.
I kinda stumped on this one...

Yeah - I think something is going on in a background thread and the
specific timing of various threads (and the determinism of Java's
threading) is causing this. But what, exactly, is going on is beyond me.
The only thing I can think of would be to try moving the TE integration of
BaseMetals from running during preInit to running during Init - which,
thankfully, is not that hard.
On Wed, Mar 22, 2017 at 12:25 AM, zmaster587<EMAIL_ADDRESS>wrote:

Doesn't appear the change had any effect on the error.. same two hashes
and items
Looks like Thermal expansion loads default recipes (which includes
dustNickel and would add the first recipe) first thing is post init.
I kinda stumped on this one...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zmaster587/AdvancedRocketry/issues/456#issuecomment-288295236,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAS_okasQllrMCbJRfnWEZqidleMPQYJks5roKKogaJpZM4MbGip
.

I'm fairly sure it's not a threadin problem.  Minecraft only uses one
thread to load (can be verified with a debugger).  Also generally problems
involving threading timing occur seemingly at random and you can launch the
program with the same settings and sometimes it'll break sometimes it won't.
That being said I think you're probably correct about it being a load order
issue, though I think it's due to mod load order determined by forge.

You could try moving it, if it fixes it then that's just more information
about the nature of the problem and may lead to nailing down an exact cause

Fairly sure it's not an AR issue, so closing

Have managed to (accidentally) find a related crash where the item causing the trigger is from Vanilla. This now appears to be, wholly, a timing bug of some sort in the TE code. Not an AR - or any other possibly indicated mod - issue at all.


Always make the window surface opaque

This makes the examples and tests render as they do on other platforms,
with a black background instead of a transparent or semi-transparent
one.
Too bad, it made some examples look way better, but consistency is a good thing. Also waiting on #157 to provide the API to re-enable transparency.

Hmm, I see I have more to learn about the new merge options. Sorry about losing your commit description.

@elmindreda I don't think the patch is correct. What wl_surface_set_opaque_region() does is to notify what part of the surface is opaque. GLFW has no clue about the content of the frames it passes along, but as far as I can see (at least when using mesa) the buffers are still ARGB8888, not XRGB8888.
Only if we always glClear() with black, or request an XRGB framebuffer configuration would it be Ok to set the opaque region to the whole surface.

@jadahl Noted. Will look into it.

jadahl, what XRGB8888 means is that the buffer has eight bits of padding where an ARGB8888 would have its alpha channel.  The representation is the same, only the way the compositor composites (or scanout) the buffer will change.

It's probably often harmless yes, but for example a compositor that may still just use the full buffer to do a complete composition including alpha blending while only using the opaque region to determine whether a surface that would be hidden should be painted or not the compositor may end up not painting/replying to frame callbacks that surface while it still is actually visible.


Response regex requires '' to be escaped

This just started happening recently (probably in the same update as #69). If I have a response regex such as the following: "INT(.|\n)*"
What actually gets imported to API Gateway is: "INT(.|)*"
In order for it to get properly imported, I have to escape the backslash, i.e. "INT(.|\\n)*"
Is this by design, or is this a bug?

Swagger/OpenAPI import is now generally available in the API Gateway REST API, the AWS CLI and all AWS SDKs. You can also import and export Swagger definitions using the API Gateway console. This release addresses many of the open issues and feedback in this repository.
I would encourage you to migrate your workflow to the standard AWS tools. aws-apigateway-importer will receive minimal support from the API Gateway team going forward.
Any feedback or issues with the new importer should be directed to the official API Gateway forums.
Thanks,
Ryan


python310Packages.bleak-retry-connector: 1.10.0 -> 1.10.1

Description of changes
https://github.com/Bluetooth-Devices/bleak-retry-connector/releases/tag/v1.10.1

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

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

16 packages built:

home-assistant-component-tests.bluetooth
home-assistant-component-tests.homekit_controller
home-assistant-component-tests.switchbot
home-assistant-component-tests.xiaomi_ble
python310Packages.aiohomekit
python310Packages.bleak-retry-connector
python310Packages.pymicrobot
python310Packages.pyswitchbot
python310Packages.xiaomi-ble
python310Packages.yalexs-ble
python39Packages.aiohomekit
python39Packages.bleak-retry-connector
python39Packages.pymicrobot
python39Packages.pyswitchbot
python39Packages.xiaomi-ble
python39Packages.yalexs-ble


.DS_Store causing error

I get this error when running generateJsonSchema2Pojo task on a folder
due to the temp file created by OS X

* What went wrong:
Execution failed for task ':json2pojo:generateJsonSchema2Pojo'.
> Unrecognised URI, can't resolve this: file:/Users/xxx/android/schema/.DS_Store

@ir2pid , this issue was resolved in the Maven plugin, but the changes were not applied to the Gradle plugin, since it was not clear how Gradle plugins typically specify file filtering.  Any interest in creating a PR?

I previously hit this but didn't come up with a solution (I know way too little about gradle!). Workaround was delete the .DS_STORE file from the command line and avoid visiting the folder inside finder.
CI systems should deal fine, it's only locally on my machine I sometimes run into this.

In my java code I check for the file suffix before trying to generate classes for it.
List<String> ignoreList;
ignoreList = new ArrayList<String>();
ignoreList.add(".DS_Store");

private static boolean isValidFile(File file) {
		for (String ignoredSuffix : ignoreList) {
			if (file.getAbsolutePath().endsWith(ignoredSuffix)) {
				return false;
			}
		}
		return true;
	}


Got CLIInternalError when logging in to Azure with command "az login"

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az login
Errors:
local variable 'token_entry' referenced before assignment
Traceback (most recent call last):
Local\Temp\pip-unpacked-wheel-s90ugolk\knack\cli.py, ln 215, in invoke
azure\cli\core\commands\__init__.py, ln 654, in execute
azure\cli\core\commands\__init__.py, ln 717, in _run_jobs_serially
azure\cli\core\commands\__init__.py, ln 710, in _run_job
AppData\Local\Temp\pip-unpacked-wheel-7f66x3i5\six.py, ln 703, in reraise
azure\cli\core\commands\__init__.py, ln 688, in _run_job
azure\cli\core\commands\__init__.py, ln 325, in __call__
pip-unpacked-wheel-_1bqfhco\azure\cli\core\__init__.py, ln 784, in default_command_handler
azure\cli\command_modules\profile\custom.py, ln 158, in login
pip-unpacked-wheel-_1bqfhco\azure\cli\core\_profile.py, ln 195, in find_subscriptions_on_login
pip-unpacked-wheel-_1bqfhco\azure\cli\core\_profile.py, ln 875, in find_through_authorization_code_flow
UnboundLocalError: local variable 'token_entry' referenced before assignment

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az login

Expected Behavior
Environment Summary
Windows-10-10.0.16299-SP0
Python 3.6.8
Installer: MSI

azure-cli 2.15.0

Additional Context

az login

Duplicate of #15961

This is a known issue and we are working on a fix. To get over it, please use a valid pair of username + password.


feat: epoch 2.5 testnet updates

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


Suggested tracking date

Describe your suggested feature
Some tracking services such as Anilist tracks the date when you started reading a manga and when you finished it. On Neko, when you click the "started reading date", it gives you a suggested date which I think is based on your history.
This feature is very convenient as you don't have to know the date and manually type it every time you starts reading a manga. It also remembers the date when you starts reading a manga so you wouldn't have to worry about forgetting to track the "started reading date" on the day that you started reading a manga.

Other details
In TachiJ2K, when you already started reading a manga, Anilist would automatically track the date instead of suggesting it.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.9.3.
[X] I will fill out all of the requested information in this form.

Personally not interested in this, I don't use tracking. Open a issue with main Tachi and SY will get it if Main Tachi does.


Exercise 3 - Manage Sensitive Information Types, Task 1, Step 8

Module: 01
Lab/Demo: 01
Exercise: 03
Task: 01
Step: 08
Description of issue
Step should say "On the Requirements for matching page, select (+) Create Pattern." instead of "On the Requirements for matching page, select (+) Add an element."
Repro steps:

Hi,
Thanks for pointing this out.  The M365 tenant that ships with the course just had a refresh the first week of May.  These steps changed as you noted with that refreshed tenant.  The steps have been adjusted as you suggest in Exercise 1.

closing issue now that it's resolved in the repo.

Sorry I'm late responding. Thank you!


Create release?

Time for a 0.0.2 tag?  @aartaka

Yeah, could be :D

Done in 4d067ad9cad65ef9e3f8c67be713d75e5a5f9bf5!


Make DB Backend Aligned with ICAT Backend

Description:
A previous issue (#205) changed the database backend to use camelCase, where SNAKE_CASE was previously used. This was a good step to make the two backends aligned with one another, but they are not fully aligned. The two should be be fully interchangeable, allowing them to be used on DataGateway without having to change the frontend each time the API backend is changed.
The differences between the backends remain with foreign keys and related entities. DB backend uses foreign keys (due to its direct connection to DB tables) whereas ICAT backend does not. On DataGateway, I tried to keep foreign keys in, but I encountered naming collisions. For example, I tried to make a related entity called facility and also make the foreign key called facility but (obviously) the frontend doesn't like that because two attributes can't have the same name in one class. This was not an issue within the API due to where the names (of related entities specifically I think?) are defined.
I think to make the DB backend aligned with the ICAT backend (in that order), the foreign keys need to be removed. Or at least some modification to the foreign keys so that stuff is done in the background (i.e. they don't get returned to the user and they aren't required when updating/creating data in a request).
In theory, if the two backends are fully aligned, the tests which use a Flask test client could be merged into one. Currently, there are two sets of these tests - one for DB backend, another for ICAT backend. The single set of tests can then be parameterised to get both backends running on it). It was my hope to merge these when I camelCased the DB backend, but I couldn't get the full alignment so I chose to keep the separate set of tests.
I'm not sure how all this can be done at the moment, we might need to have a wider discussion to make some form of plan.
Acceptance criteria:

[ ] Both backends should run and pass the frontend tests without any frontend code modification between backend changes
[ ] IF fully aligned, the tests using Flask test client should be merged
[ ] Functionality changes of the ICAT backend should be avoided where possible, to reduce any further changes needed to the frontend

Adding the wontfix label to this issue - there's no plans to work on this issue for the time being and because the DB backend won't be used in production, this is a low priority, potentially high effort issue.

DB backend to be removed as per #489


std::null_ptr breaking my clang build

Hi,
I've been using CppUTest for a while, and have several test suites.  I use both Clang and GCC.  Recently, I updated to master from this repository and my Clang tests now don't compile.  I get a "fatal error: Unknown type name 'nullptr_t'; did you mean 'std::nullptr_t'?" on line 235 of SimpleString.h.  Modifying the header to use std::nullptr_t fixes the issue.
It seems like this is a change that was introduced only a few weeks ago.  Are there any pointers for how to go about fixing my build so I can work off of master CppUTest again?  I'm using the default clang toolchain installed via apt on Ubuntu Xenial.
Thank You!

Hmmm... that is interesting. We fixed the usage of nullptr_t so that is also works better with other compilers. CLang is included in the CI build, so I'm surprised that it breaks for you. My guess is that it is then wrong in an older (or newer?) clang version. That doesn't help you a lot though.
It probably can be fixed though we'll end up using a hack for this. An easier fix would be to add "using std;" before the #include. Would that be good enough or do you think this is common enough to warrant a more hack-ish fix?

Mingw64 also gives the same error. The issue is mentioned in #1232 but then kinda overlooked due to other issue. The fix in #1234 fixes it for ghs and similar compiler but actually breaks it for other compilers. If we revert these changes it will break green-hills compiler and few others which don't put nullptr in std namespace in their library implementation.
This is a messy issue as some libraries define nullptr_t in stddef.h and then include it inside cstddef within the namespace std, others do not. Some like arm or mingw implementation define it in c++config.h and not at all in stddef.h. I am sure if we remove the !defined __arm__ check for this function it will break due to same reasons in arm as it is in mingw. I haven't check the Clang library but I am guessing they have same issue.
Technically, C standards doesn't mention nullptr_t so I am not sure why these compilers have implementation in C library. This is a C++11 concept and should be in the C++ library like cstddef. But we have to work with what we have.
Now, I can't think of a good way to solve this, I haven't looked much on how StringFrom(const nullptr_t value) is used but a couple of solutions can be either to get rid of this function (which @basvodde mentions in issue 1232) or to put it under something like #ifdef USE_NULLPTR_T check. Just some ideas....

Hi Bas,
Thanks for your reply.  I did see that you run a clang build as part of your CI, I agree it's very curious that it doesn't work for me even though those builds pass.  So far, I have only been able to try with clang-6 and clang-8, and it looks like your CI server is using clang-7.  I'm going to try to get clang-7 installed so I can attempt with that soon, but since the issue exists for both the previous and following versions I don't hold out much hope.
I wasn't able to get your "using std; before the # include" recommendation to work, but I am much more familiar with C than CPP.  Can you be more explicit about what you're proposing?  Do I add using std; before the CppUTest includes in my test files, or in the system header file?

I like the USE_NULLPTR_T but it won't solve the problem as the problem relates to whether it is in std namespace or not.
I guess we'll need to revert back to add the namespace std:: there and change to include both the stddef and the cstddef in StandardC. Perhaps that would also resolve the other problem. Feel free to try out some pull requests.

I am encountering the same issue trying to build the IAR platform using IAR EWARM 8.10.1.

Is replacing (std::)nullptr_t with direct decltype(nullptr) an option?

Not sure, but the std:: is back for now. Let's see what problems it'll cause :)

This is solved by #1264 and #1261, isn't it?


Better arrow (U+2192) in Repository Structure title

The string "- >" that should represent the relation between repository and a project inside it looks awkward with the font currently used in the worbench. I have replaced it with a unicode arrow.
Before:

After:

Am I wrong being concerned with the inclusion of the new arrow literal
rather than using /u2192 (sorry I forget the syntax)? Any foreseeable
problems with someone editing the file in the future with a non-unicode
editor (IDK vim for example?) Perhaps I'm too stuck on ASCII7 ;-)
On 4 Nov 2015 18:12, "Jiří Locker"<EMAIL_ADDRESS>wrote:

The string "- >" that should represent the relation between repository and
a project inside it looks awkward with the font currently used in the
worbench. I have replaced it with a unicode arrow.
Before:
[image: red hat jboss brms business central 2015-11-04 18-59-33]
https://cloud.githubusercontent.com/assets/673386/10947258/4ffde2f2-8327-11e5-856d-e86eec08c123.png
After:
[image: red hat jboss brms business central 2015-11-04 19-01-05]
https://cloud.githubusercontent.com/assets/673386/10947264/56be9eec-8327-11e5-8aae-5f416ce2b8fa.png
You can view, comment on, or merge this pull request online at:
https://github.com/droolsjbpm/guvnor/pull/179
Commit Summary

Better arrow (U+2192) in Repository Structure title

File Changes

M
guvnor-asset-mgmt/guvnor-asset-mgmt-client/src/main/java/org/guvnor/asset/management/client/editors/repository/structure/RepositoryStructurePresenter.java
https://github.com/droolsjbpm/guvnor/pull/179/files#diff-0 (4)

Patch Links:

https://github.com/droolsjbpm/guvnor/pull/179.patch
https://github.com/droolsjbpm/guvnor/pull/179.diff

—
Reply to this email directly or view it on GitHub
https://github.com/droolsjbpm/guvnor/pull/179.

+1 to merge (even vim supports unicode)

Yes, I made the change using Vim :-) I agree it is safer to encode Unicode characters using the \uXXXX syntax. It can prevent an unwanted change with some kind of old-school/buggy editor but such a mistake could also be caught during the code review.
Just let me know if you want me to changed that.

@yurloc I'm torn to be honest. Personally I prefer \uXXXX but it makes no logical reason in this modern (generally Unicode proof world); plus it seems really pedantic asking for it to be changed. So..

+1 to merge.

Thanks!
master: ff4e2e0aaf2bb041b50059e0879b1487ca6674ea


After composer update of Yii2, this error occurs

If the debug mode is enabled, then on the subsequent request, this error is shown in Firefox as well as in Chrome.
GET http://localhost/yii2/ net::ERR_CONNECTION_RESET

The performance boost updates after Yii2 Version 2.0.10 does create more child processes.
If you are using Apache2 you can enlarge your allowed threads: https://httpd.apache.org/docs/2.4/mod/worker.html
<IfModule mpm_winnt_module>
    ThreadStackSize 2000
</IfModule>

or downgrade to Yii2 2.0.10

Hi @linslin .
Could you tell me more about this performance boost updates from 2.0.10?
Once in a while I get this ERR_CONNECTION_RESET in production environment, but i thought it was a problem in the Chrome browser or even my Internet provider.
I'm curious about it now.


Fix copy_shape() for group of shapes

Fix VisioFile.copy_shape() to save changes to page after copying a shape.
Fix VisioFile.set_page_max_id() to consider groups inside group.

Add VisioFile.get_shape_max_id() as helper function.
Use Element.find() to simplify code.
Add tests for grouped shapes.

Apologies for the delay, this PR merged and tested - looks good.
Element.find()/findall() is definitely the way to go. Note I added a couple of new tests in latest commit to test finding elements using xpath as ElementTree is new for me.

ElementTree is new for me too, to be honest.
I had already done some work looking to changing the codebase to find()/findall(), gotta pick it up again where I left it off.


[Market Page] - Refresh needed for the Transfer to Trade button to disappear

When I connect my wallet I need to refresh for the "Transfer to Trade" button to disappear.

Fixed


Dialog height broken

Seems like these days the dialog heights are broken on most browsers (Chrome, FireFox, Opera, Edge (Chromium)).

Thank you for reporting this. It's now fixed in a new version.


Github Release

It would be amazing if you would tag your releases as Github-releases too. It would simplify the act of packaging your library in Void Linux.

Could I get an ETA when this can be done?

I can start looking at it on the weekend.

Done, let me know if there are any issues.

It'll be at least the weekend before I can do releases for the sub-projects though.


[2.6] v2 charts are not properly scheduled on Windows nodes in RKE2 custom clusters

rancher/dashboard issue: https://github.com/rancher/dashboard/issues/3974
Rancher Server Setup

Rancher version: v2.6-head

Versions
Component 	Version
Rancher 	v2.6-be960478b7d8f83eecd3d33fbd8bdd2504ed409f-head
Helm 	v2.16.8-rancher1
Machine 	v0.15.0-rancher68

Installation option (Docker install/Helm Chart): docker
Proxy/Cert Details: self-signed

Information about the Cluster

Kubernetes version:  v1.21.3-rc7+rke2r2
Cluster Type (Local/Downstream): downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Custom RKE2

Describe the bug
When deploying v2 monitoring and logging apps, the Windows workloads are not being scheduled out of the box.
To Reproduce
Provision custom cluster with 1x linux all role or 1x etcd/cp and 1x worker
Enable v2 Logging and/or v2 monitoring apps with default helm chart configuration.
Result
The linux logging workload deploys as expected. There are no windows logging (fluentd) workloads deployed.
Expected Result
Windows workload (rancher-logging-win-agent-fluentbit) for v2 logging should be scheduled on Windows nodes.
Windows workload (rancher-monitoring-windows-exporter) for v2 monitoring should be scheduled on Windows nodes.
Additional context
Rancher and/or the UI need logic to detect that a windows node is present in a cluster and therefore set the global.cattle.windows.enabled value to true in the cluster.
Both the v2 monitoring and v2 logging charts rely on .Values.global.cattle.windows.enabled to be present and set to true.
https://github.com/rancher/charts/blob/dev-v2.6/charts/rancher-logging/rancher-logging/100.0.0%2Bup3.12.0/templates/_helpers.tpl#L68-L71
For v2 logging, the value was not present in the chart values yaml after a successful deployment of v2 logging.
For v2 monitoring, the value was set to false (below) in the chart values yaml after a successful deployment of v2 monitoring.
global:
  cattle:
    systemDefaultRegistry: ''
    windows:
      enabled: false

Workaround
Editing the values yaml for both v2 monitoring and v2 logging and setting global.cattle.windows.enabled to true allow the workloads to be scheduled.
global:
  cattle:
    windows:
      enabled: true

If that workaround works, then the fix would only be on the dashboard side, that is how that flag gets set. The backend is responsible for making sure it works once the flag is set.

Re-purposing this issue to add a provider for rke2-windows: https://github.com/rancher/kubernetes-provider-detector/tree/master/providers

We are going to use this one to track being able to detect RKE2 Windows in the https://github.com/rancher/kubernetes-provider-detector.

the rke2-windows provider has been added: https://github.com/rancher/kubernetes-provider-detector/commit/f2c737b763e2e94f73faa74cb8b88ff4edeadcdc
PR for the rancher/rancher version bump for kubernetes-provider-detector and adding rke2-windows (and harvester) providers: https://github.com/rancher/rancher/pull/34428


Add support for multiple scan (multiple barcode at once)

I saw that there was a question regarding continuous scanning. And got me wondering if it is possible to scan multiple barcodes at once.
So not just continuously scanning without closing the camera but capturing all the visible barcodes in the image in one go.

With the current implementation this is not working really well. The currently used libs on Android and iOS will only return the first detected code (if you center to another code, this should be de the first detected and therefore it is kind of possible), the "multiple option" is disable since the results are not consistent at all.
We are working on a MLKit rewrite which should make this possible and should provide better results, but I guess the initial version will keep the "one code at a time" behaviour. But it should be possible to read up to 10 codes in a more consistent way than the actual implementation. The new version will then provide a stream of all the detected codes and the logic which one to use aso. has to be handled by de developer :)

@thegnuu Any advances on the ml-kit implementation?

@smachi Let me check that, someone else is working on the Android part at the moment :)
@HunSpeedi there is currently a branch, but only the iOS version is working. There are still small things to do, but in general it should work.

@HunSpeedi I just received a PR for the Android implementation. I will do my best to check and merge this ASAP so you have a version to play around with. I hope I can finish it by the end of this week :)

@HunSpeedi @smachi I just release v5.0.0-beta.1, the initial pre-release which uses ml-kit! There is still some work to be done, but I should deliver quite good results already :)

@thegnuu Thank you!! I hope I will have some time in the near future to try it!!


change the way React.createElement is called

this is a change for paf31/purescript-thermite#27
when you return an array of JSX elements, it passes
an array as children in createElement, so this works:

  { this.props.map(function(el) {
    return {el};
  }}

but when you structure your tree like this:

  one
  two

it calls createElement(..., one, two).
this is important because elements passed in the 2nd way
don't require unique keys and reacts fills them in automatically.
when passing elements the first way you always have to give unique
keys yourself

What happens now if you use nested arrays, or a mixed structure?

React will render [[a]] exactly the same as [a]. Their validators for uniqueness of keys don't look deeper than one array though, so if a someone uses passArrayImpl twice they won't get proper warnings in development mode of React.

But what if array is used alongside other unwrapped elements? I guess what I'm asking is - can we separate your two changes? Is array needed to fix createElementImpl?

If array is used alongside other unwrapped elements then its behavior is the same as in JSX.
array is not needed to fix createElementImpl, but if you take only this bit:
-          return React.createElement(element, props, children);
+          return React.createElement.apply(this, [element, props].concat(children));

Then users won't get a warning from React if they render dynamic arrays. React will think it was hardcoded in JSX and assign ids manually (which I imagine will negatively affect diffing).


Using React 15.3.x: rsg-alt works fine, but react-simpletabs breaks NPM

So I'm using<EMAIL_ADDRESS>in my parent project, and rsg-alt is working without a problem as a dependency, but NPM is choking on one of the peerDependencies specified in an rsg-alt dependency. It's called react-simpletabs:
"react-simpletabs": "github:matthewgertner/react-simpletabs",

react-simpletabs' package.json has this line:
  "peerDependencies": {
    "react": "0.12.x - 1.x"
  },

Since that version of react isn't what I'm installing, and it's also not in the react-simpletabs dependencies, it doesn't install it, and I'm getting a non-fatal NPM error when installing (and a fatal NPM error when doing other tasks such as shrinkwrapping).
Do you have any solutions immediately come to mind? I have a feeling react-simpletabs will work with<EMAIL_ADDRESS>so perhaps I could just fork react-simpletabs and remove the peer dependency, test it, and update rsg-alt to use the forked one. Or, since it's only two smallish files, maybe we could pull it into rsg-alt, if tabs are a vital part of rsg-alt's experience. Or, I could just fork rsg-alt for myself and leave out react-simpletabs. Lot of options.
Any suggestions how to best proceed? This will be a pretty big problem once more and more people use<EMAIL_ADDRESS>I'm guessing.

So the use of this custom repo was the result of https://github.com/pedronauck/react-simpletabs/issues/26
I think it's safe at this point to fork that repo and publish an alternative version with the proper react updates
I have a feeling it should work as well

Sounds good, I'll work on that this week.


small tweaks

update version info for new release
fix arduino.h to Arduino.h (for case-sensitive systems)
add info about filename has to end with MP3 or WAV

@codingABI please accept and add a new release v1.0.2

yeah i know its a wildcard :-)
and i think your correction is very good!
i have added it.
and also did a quick experiment on the first topic.
with g_audio.playFileByName("/BELL*MP3");
and these songs available:

test1

BELL.mp3
BELL0.mp3
BELL0.mp3  was played.

test2

BELL.mp3
BELL1.mp3
BELL.mp3  was played.

I will update the PDF docu and make a new realase with you changes in Github soon

1.0.2 is released now. I had to change some linebreaks from your comments because doxygen, that generates the PDF docu, makes i little bit trouble

In your "quick experiment on the first Topic" the order in which the first file matches seems not to be in alphabetic order.
Perhaps it is "copy order" (=First audio file copied to the drive will be found first, second copied will be found next and so on)? The DFR0534 uses "copy order" already in the function playFileByNumber, playPrevious, playNext….
Could you try to delete all files on the DFR0534 and copy BELL3.mp3 first to the DFR0534 and than copy as second file BELL.mp3 and then BELL1.mp3?
Which file is played with g_audio.playFileByName("/BELL*MP3") ?
If it is BELL1.mp3, is seems that the DFR0534 searches the files when using wildcards in function playFileByName also in "copy order" (and I would add these information to the code comments)
Thanks for you input

i will du the tests.

i just created some small test-files
https://github.com/s-light/DFR0534/tree/playFileByName-test_order/assets/test_sounds
and copied them one after each other in this order to the board:

120bpm 16bars 96kbps.mp3 32sec - 375,6k - as bell3-mp3
240bpm 16bars 96kbps.mp3 16sec -  188,3k - as bell.mp3
300bpm 4bars 96kbps.mp3 3s - 38,3k - as bell1.mp3

and copied together this test-sketch
https://github.com/s-light/DFR0534/blob/playFileByName-test_order/examples/playFileByName_order_test/playFileByName_order_test.ino
the output is
23:00:27.862 -> Ready drives: FLASH
23:00:27.927 -> Current playing drive: FLASH
23:00:27.958 -> Total files: 3
23:00:27.958 -> Total files in directory: 3
23:00:27.958 -> First file: 1
23:00:27.958 -> play /BELL*MP3
23:00:28.119 -> fileNr: 1 name: BELL3   MP3 status: Playing
23:00:28.408 -> fileNr: 1 name: BELL3   MP3 status: Playing
23:00:28.698 -> fileNr: 1 name: BELL3   MP3 status: Playing
....
23:00:59.384 -> fileNr: 1 name: BELL3   MP3 status: Playing
23:00:59.705 -> fileNr: 1 name: BELL3   MP3 status: Playing
23:00:59.995 -> fileNr: -- name:  status: Unknown
23:01:00.548 -> fileNr: 1 name: BELL3   MP3 status: Stopped
23:01:00.843 -> fileNr: -- name: BELL3   MP3 status: Stopped
23:01:01.139 -> fileNr: 1 name: BELL3   MP3 status: Stopped
23:01:01.433 -> fileNr: 1 name: BELL3   MP3 status: Stopped

so your assumption that it is the copy order is correct.
i thin  it searches the index table just in order (and that is the copy order) ....
i have tweaked the description a little bit to reflect the findings.


missing error parameter limits the way you can use the api

Before I start . Thank you for the great library this was just the api that I was looking for.
The errors are currently handled by a global error hander:
Characteristic.prototype.read = function(completeFn) {
adapter.readCharacteristic(this, executeFn(completeFn), raiseError("read characteristic error"));
};
This results into problems when you want to write code which does do the a second read after the first read was completed successfully or with an error. It would be easier to write this kind of code if all the routines also had an optional error handler. Something like this
Characteristic.prototype.read = function(completeFn,errorFn) {
adapter.readCharacteristic(this, executeFn(completeFn), function (e) {
if (errorFn) ErrorFn(e)
raiseError("read characteristic error")
});
};
Writing it this way makes all routines still backwards compatible.
An even nicer solution would be to get rid of all the completed and error functions by using ES6 promis using a poly fill. But this may be much work.I will also be happy with an optional error parameter But  even a promis can be done in a backwards compatible way because the functions do not yet return values.

Thanks for the feedback. I intend to re-implement this library following the web-bluetooth specification once I find some time to put some effort in. That specification uses promises, so should be a nicer interface to work with.
Have a look at the web-bluetooth branch for progress so far.

New solution looks very good to me.  Following the official standard is a very good idea.  Can't wait till it is ready.

I've decided to keep both specifications, so will change this to an enhancement to add individual error callback functions to the classic API.

Thanks i will check it out I also made a small bug fix in commit
https://github.com/tijmenvangulik/bleat/commit/ec7fe444d9b1cd25f3ae39eb76b8ac8e50ec3760
When bleat was initialized at a later moment the init function looses the this because the internal function is called without bind. The bug fix was committed together with other changes so you need a manual merge
Tijmen

Op 26 feb. 2016 om 17:28 heeft Rob Moran<EMAIL_ADDRESS>het volgende geschreven:
I've decided to keep both specifications, so will change this to an enhancement to add individual error callback functions to the classic API.
—
Reply to this email directly or view it on GitHub.

Can you open this as a new bug?


Added Skeleton-Loader

What is the change?
Nothing has changed rather added a new loader called Skeleton-Loader. It is very popular kind of loader used in many modern websites like facebook, etc.
Related issue?
closes: #124
How was it tested?
It was tested using the VSCode Live Server.
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelineswhile contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[x] Have you made corresponding changes to the documentation?
[x] Have you tested the code before submission?
[x] Have you formatted your code ? (You can use any html,css beautifier)
[x] My changes generates no new warnings.
[x] I'm Hacktoberfest'21 contributor.
[x] I have commented my code, particularly in hard-to-understand areas.

GIF/video of the working of component you created

:

Added component name in the general readme by creating another PR #137. I had done it different because here it counters some issues.


Support re-using GenericWebHostBuilder and GenericWebHostService

Problem statement
I'm trying to help on this issue: https://github.com/microsoft/service-fabric-aspnetcore/issues/48
in addition to private integration of ASP.NET Core 3.0 + kestrel hosting within Service Fabric. Consider a basic kestrel web server, exposing http and https endpoints, within a SF node.
We'd like to add support for using a generic host within Service Fabric ASP.NET Core 3.0 apps. The advantages of using a generic host are:

Use a single configuration instance, DI container, and web pipeline across multiple SF endpoints (eg 1 endpoint for http, 1 endpoint for https, perhaps another for gRPC) in a host
WebHostBuilder is deprecated, so we want to avoid investing in that if possible

The current ASP.NET Core hosting model within SF is mostly dictated by this project:
https://github.com/microsoft/service-fabric-aspnetcore
Which is documented here:
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reliable-services-communication-aspnetcore
If you follow the documentation literally (see https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reliable-services-communication-aspnetcore#kestrel-in-reliable-services), you end up with separate kestrel pipelines, DI containers, etc per endpoint. It's possible to work around this, but it's not trivial and many developers struggle with it, per Stack Overflow.
In order to implement a better, generic host-based hosting model for Service Fabric, it would be great to be able to re-use much of GenericWebHostBuilder and GenericWebHostService without copying + maintaining all the classes. Also note that ISupportsStartup and ISupportsUseDefaultServiceProvider are internal, so they can't be implemented.
Describe the solution you'd like
I'd like to see a separation + make public of the portions of GenericHost that should be re-used in scenarios like this.
If it's helpful I can provide an implementation of a ServiceFabric host builder, with details on what the sticking points are.

@davidfowl

I'd need to understand the proposed solution before we expose anything.

Sure - I provided a more detailed analysis of the issues in https://github.com/microsoft/service-fabric-aspnetcore/issues/48#issuecomment-509346009  - but here's the crux of the problem as it relates to ASP.NET Core code. Sorry if my initial request wasn't clear enough:
Looking at GenericHostWebHostBuilderExtensions:
        public static IHostBuilder ConfigureWebHost(this IHostBuilder builder, Action<IWebHostBuilder> configure)
        {
            var webhostBuilder = new GenericWebHostBuilder(builder);
            configure(webhostBuilder);
            builder.ConfigureServices((context, services) => services.AddHostedService<GenericWebHostService>());
            return builder;
        }

If I call this method today, it sets up the IWebHostBuilder and adds the WebHost IHostedService. But what if I want or need to run the webhost within a separate IHostedService? The service fabric hosting use-case requires a different IHostedService which starts and stops the web host.
Should it be possible to use the GenericWebHostBuilder functionality without registering the GenericWebHostService as an IHostedService? The 2 classes appear separate at least conceptually, and they both provide useful and standard functionality that probably shouldn't be duplicated. I would like to be able to call ConfigureWebHost(Action<IWebHostBuilder>) or a similar method and not get the IHostedService registered; and separately re-use the GenericWebHostService within a custom IHostedService.
It may be possible to do what I want by creating a wrapper IHostBuilder with a local IServiceCollection and pass that to ConfigureWebHost(), then pull the IHostedService registration out and put it in a separate IServiceProvider that is only used to create the GenericWebHostService within my custom IHostedService. But this seems a bit hacky and likely to break with changes, which is why I opened this issue.

But what if I want or need to run the webhost within a separate IHostedService? The service fabric hosting use-case requires a different IHostedService which starts and stops the web host.

Please expand on this, why does it have to be another IHostedService? What is that service adding doing differently?

You can find more info in https://github.com/microsoft/service-fabric-aspnetcore/issues/48, and the diagram on this page should help conceptually:
https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-reliable-services-communication-aspnetcore
As the diagram shows, service fabric activates SF service objects, which create webhosts.
Currently, service-fabric-aspnetcore creates 1 WebHost per SF service type, and the WebHost lives within the SF service object, as explained in my comment. The code for this is here: AspNetCoreCommunicationListener.cs line 115.
For this model to work within a generic host there would have to be a new IHostedService (eg a ServiceFabricHostedService) that manages SF registration and contains the SF service objects. For the new IHostedService to create and start/stop the WebHost(s) correctly, the setup code needs to be able to create an IWebHostBuilder without registering the GenericWebHostService to run directly within the IHost.
Ideally, a new version of the service-fabric-aspnetcore libs would support disassociating the SF service lifecycle from the webhost lifecycle. In other words, a webhost pipeline and DI container could be shared by multiple SF services (currently not supported), or each SF service could have its own webhost pipeline and DI container (the only case that's supported today).
Hope that helps...

If it was creating multiple web hosts' before, it should create multiple generic hosts now, not multiple IHostedService instances in one host. Yes we were able to re-platform WebHostBuilder onto HostBuilder, but you still can't have more than one IServer instance per Host, there would be too many conflicting services.

I'm not surprised/makes sense that you can't have more than one IServer instance per IHost, though technically that's about the shared DI container in the IHost, right? With a separate IServiceProvider one could create a new webhost in the IHostedService. There would still be a benefit from putting that within a generic host, eg shared environment, configuration, and lifecycle management.
For this scenario, I suppose it could work to create a new IHostBuilder + IHost within the IHostedService that needs its own WebHost, and still use the top level IHostBuilder + IHost for global configuration and lifecycle management. That wouldn't require any changes to GenericWebHostBuilder. Let me look into that.


The problem with using Aos and Remodal

Hello. I will describe my problem.
I have a "Callback" button on my site, when I click on it, using the Remodal plugin, a modal window with fields for filling is called. I connected aos.js to  so that when loading the site, there would always be a smooth appearance animation. The fact is that when you connect aos.js and call the modal window, the page of the site is scrolled exactly (or roughly) to the middle. Without an aos.js connection, the modal window, when called, does not scroll down the page, but starts exactly at the position of the page on which it was called. This problem becomes even worse if the page of the site is very short - the modal window is called somewhere at the bottom of the page and half of the modal window is chopped off. In the mobile version of the site the same problem - when you call the modal window, the page scrolls down. I tried to remove the call button and the "body" of the modal window for the  tag - this does not help.
What else can I give you to help you solve this problem? I will give you some screenshots.

Have you tried adding data-aos attribute on other element? Perhaps styles that are initially being applied to this element causes remodal script to miscalculate something?


deviceProperties.Get and Set

Hello what does the query look like for GetAutoplayRoomUUID? And how for SetAutoplayRoomUUID?
For GetAutoplayRoomUUID it works like this:
device.deviceProperties.GetAutoplayRoomUUID ([('InstanceID', 0)])
But I can not change the value with set. Can someone give me a fitting example?

That's strange - it works for me. What I did was:
device.deviceProperties.SetAutoplayRoomUUID(RoomUUID=device.uid)

Thanks the lower one works.


Timestamp report

A bit more robust implementation, but still lacking some tests. Work in progress...

Thanks for the PR - it looks good - just left some comments for small things (for code refactors it's up to you if you want to leave it like it is).


New version: OSGeo.QGIS_LTR version 3.34.13 (3.34.13-1)

Created by 🥟 Dumplings in workflow run #9887.

Log
Updated: 3.34.12-1 → 3.34.13-1
The input object cannot be bound to any parameters for the command either because the command does not take pipeline input or the input and its properties do not match any of the parameters that take pipeline input.
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

This package installs and launches normally in a Windows 10 VM.

This package installs and launches normally in a Windows 10 VM.


fix(compiler-sfc):   fix 'export    default' with extra spaces  transformation

fix: https://github.com/vuejs/vue-next/issues/4371

this pr is actually still WIP


Providing different results of FLOPS calculation when using tf.compat.v1.profiler.profile on Mac M1 chip and on NVIDIA 1650

Please make sure that this is an issue related to performance of TensorFlow.
As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:performance_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
When use the following function for FLOPS calculation on mac M1
`def get_flops(model: Union[Model, Sequential], batch_size: Optional[int] = None) -> int:
"""
Calculate FLOPS for tf.keras.Model or tf.keras.Sequential .
Ignore operations used in only training mode such as Initialization.
Use tf.profiler of tensorflow v1 api.
"""
if not isinstance(model, (Sequential, Model)):
raise KeyError(
"model arguments must be tf.keras.Model or tf.keras.Sequential instanse"
)
if batch_size is None:
    batch_size = 1

inputs = [tf.TensorSpec([batch_size] + inp.shape[1:], inp.dtype) for inp in model.inputs]

# print("@@@@@@@@@@@@@",inputs)
real_model = tf.function(model).get_concrete_function(inputs)
# frozen_func, _ = convert_variables_to_constants_v2_as_graph(real_model)

# Calculate FLOPS with tf.profiler
run_meta = tf.compat.v1.RunMetadata()
opts = tf.compat.v1.profiler.ProfileOptionBuilder.float_operation()
flops = tf.compat.v1.profiler.profile(
    graph=real_model.graph, run_meta=run_meta, cmd="scope", options=opts
)

return flops.total_float_ops`

give wrong FLOPS as output: 2632160
Describe the expected behavior
While the same code gives the correct calculation when tried on NVIDIA enabled machine. VGGNET-16 FLOPS:<PHONE_NUMBER>
Expecting the same result to be calculated when done on Mac M1 machine.
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

Hi @shujaat81 ! Could you please attach the above code as a Colab gist? Thank you!

Hi @mohantym Please find the link for colab
https://colab.research.google.com/drive/1Hw3HuAa2vOEM6VxwnorV5pdToqx3mlJB?usp=sharing

@shujaat81 ! Could you please confirm your Tensorflow version too?

@mohantym Iam using 2.4.0-rc0 version

Hi @Saduf2019 ! Could you please look at this issue? Attaching gist for reference.


refactor(AttributeInput): to compositionApi

PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Related with #4750
[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality

Optional

[ ] I've tested it at </ksm/collection>
[ ] I've tested PR on mobile
[ ] I've written unit tests 🧪
[ ] I've found edge cases

Did your issue had any of the "$" label on it?

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

[ ] My fix has changed something on UI; a screenshot is best to understand changes for others.

Copilot Summary

🤖 Generated by Copilot at 769abca
Refactored the code for opening the shopping cart modal from the navbar. Moved the logic and configuration from ShoppingCartModalConfig.ts and ShoppingCartButton.vue to Navbar.vue to improve code organization and performance.

🤖 Generated by Copilot at 769abca

ShoppingCartButton
Simpler, no more emit -
Autumn leaves falling

yolo
pay 50 usd

@yangwao Thanks for the payout. But i believe there was a system error

It didn't push through

@stephenjason89 yes, fixing it
pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 4.49 USD/DOT ~ 11.136 $DOT
🧗 16UcV9V6nVvPYdHz98ymUKmNLkzjCEU5sbKJMi7hxYyTHjzR
🔗 0xab6ca9e67f22a3fec4c5f79e4f60d3b2c64a3fe2c192410bc0c631699f5e42a5
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


Make the experience better for actions to show success or failure

What are you thinking for this one? I was thinking that ideally there would be a notification drawer that would keep track of all the actions' results in a session.

I guess that's what Portal does. :) But yes, it may be the right model.


Reintroduced creating mysql process socket in extracted dir instead of /tmp

This is a regression related to https://github.com/wix/wix-embedded-mysql/commit/c7a74304b134d5420786713f01822b1734b755e0

@viliusl perhaps i'm missing something but it looks like the PR title is exactly the opposite of what the pr is doing. title - create on extracted dir, while you are creating the socket on temp dir. what am i missing ?

Impl is correct, pr title not:) i just wanted to do it in 2 mins so ci
would finish before i come home.
On May 19, 2016 00:47, "Noam Almog"<EMAIL_ADDRESS>wrote:

@viliusl https://github.com/viliusl perhaps i'm missing something but
it looks like the PR title is exactly the opposite of what the pr is doing.
title - create on extracted dir, while you are creating the socket on temp
dir. what am i missing ?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/wix/wix-embedded-mysql/pull/47#issuecomment-220168263

superseded by pull req #56


Using Known Cell Types from scRNA seq data?

Hello,
Based on the biorxiv paper, it is suggested that Baysor can use known known cluster definitions from scRNA seq data. I dont see any option to integrate scRNA seq data in the docs though. Is this no longer a possible implementation?

Hi,
This kind of functionality is hard to wrap in a CLI. Atm, it's only available from Julia code. I'm working on wrapping Baysor calls to a Python package, but it goes slowly.
So far, Baysor can support arbitrary molecule clustering if you do it with some other package, save it to cluster column and then set --n-clusters=1. Then, in the logs you'll see something like Using the following additional information about molecules: [:confidence, :cluster], which indicates that it uses provided clusters.


fix ListView renderSeparator

Thanks for submitting a pull request! Please provide enough information so that others can review your pull request:
(You can skip this if you're fixing a typo or adding an app to the Showcase.)
Explain the motivation for making this change. What existing problem does the pull request solve?
Example: When "Adding a function to do X", explain why it is necessary to have a way to do X.
Test plan (required)
Demonstrate the code is solid. Example: The exact commands you ran and their output, screenshots / videos if the pull request changes UI.
Make sure tests pass on both Travis and Circle CI.
Code formatting
Look around. Match the style of the rest of the codebase. See also the simple style guide.
ListView separator should not render after last row at any sections.

I believe this is a matter of choice for the developer and some may want the separator to render. It should be up to them whether they display it or not.
The current implementation is how it works on UIKit.

@facebook-github-bot expected

@pglotov would you mind taking a look at this pull request? It's been a while since the last commit was reviewed.

@facebook-github-bot expected

This is expected behavior so I'm going to close this PR.


Windows defender marks exe as Trojan

Windows defender marks the following as trojans: Trojan:Win32/Spursint.F!cl
:: GW2 Addon Manager.exe
:: UOAOM Updater.exe

Submitted to Microsoft, but there's nothing more I can do.


Enables Use of input_to_graph_point() by CurvesAsSubmobjects Instances

Overview: What does this pull request change?
Related to #3150
Fixed the issue where input_to_graph_point() was unusable with curves created by CurvesAsSubmobjects since the latter did not have 'points' as an attribute.
Motivation and Explanation: Why and how do your changes improve the library?
Links to added or changed documentation pages
Further Information and Comments

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Any idea why those automatic tests are failing?
This is my first PR to the community, thank you for your patience and any input is appreciated.


feat(sitemap): add support for static urls (gridsome#390)

Thanks :) Should pathPrefix be added to the URLs in staticUrls too?

Yes, that's probably best :+1: I'll do some edits.

Adding pathPrefix to every url is a little trickier than I initially imagined. There's multuple nested url properties in for example img and video. How about just letting sitemap.js handle the pathPrefix by just adding it to hostname?
    const sitemap = require('sitemap').createSitemap({
      hostname: config.siteUrl + pathPrefix,
      cacheTime: options.cacheTime,
      urls: [...generatedUrls, ...staticUrls]
    })

Yes, that is probably a better approach. You might need to ensure there are no trailing slashes in siteUrl and pathPrefix before they are joined.

Did some edits and this removes trailing slashes from siteUrl if there are one or more. Do you think it's necessary to also remove from pathPrefix? The example in the docs shows pathPrefix without the slash.

The pathPrefix is normalized by Gridsome so it should be okay. I haven't been able to test the changes yet, but the code looks good to me :)


refactor: command dependence use spf13/cobra instead

What this PR does:
refactor command dependence
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Codecov Report

Merging #234 (15708c4) into develop (71a2d60) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #234      +/-   ##
===========================================
- Coverage    40.58%   40.50%   -0.08%
===========================================
  Files           39       39
  Lines         2267     2264       -3
===========================================
- Hits           920      917       -3
  Misses        1255     1255
  Partials        92       92

Impacted Files
Coverage Δ

pkg/logger/logger.go
94.44% <100.00%> (-0.30%)
:arrow_down:

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 71a2d60...15708c4. Read the comment docs.

readme needed to be updated


add optional serde support

#40

Can you add add some tests?

@jrraymond I wonder if it's necessary to add tests, because this totally relies on the automatic implementation of serde. Since FixedBitSet only contains a Vec<u32> and a usize, it should be okay as long as the crate serde has been properly tested.

I think it's worth it to add a few tests because they are relatively easy to add.
The tests in serde verify that Vec<u32> and usize serialize/deserialize correctly. A test would verify a FixedBitSet X serializes to bytes Y.  This test has value because if the layout of FixedBitSet changes, for example the block size changes to u64, then FixedBitSet X might serialize to bytes Y'. This might be acceptable, but a test would make such an API change obvious.

No problem.


New version: BelledonneCommunications.Linphone version 5.0.18

[ ] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.
Resolves: #113716

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/113717&drop=dogfoodAlpha

@microsoft-github-policy-service agree


Setup Github actions to deploy and publish this project

We want proper CI/CD on this project. Therefore we want to deploy a showcase runtime of this project but also publish it as an npm module for people to check out.
It's probably easiest to use Github Actions for this.

[ ] tests
[ ] chromatic
[ ] npm publish