ColorPicker-17636: Fix warning "Failed to get ... mouse position-1,1"

Summary of the Pull Request
The PR fixes issue described in https://github.com/microsoft/PowerToys/issues/17636#issuecomment-1095166440

PR Checklist

[X] Closes: #17636
[X] Communication: I've discussed this with core contributors already. If work hasn't been agreed, this work might be rejected
[X] Tests: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Dev docs: Added/updated
[ ] New binaries: Added on the required places

[ ] JSON for signing for new binaries
[ ] WXS for installer for new binaries and localization folder
[ ] YML for CI pipeline for new test projects
[ ] YML for signed pipeline

[ ] Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments
Root cause analysis
According to conditions, the warning is printed if position cannot be found in any of available monitors.
However, the position in the warning is always [-1,1] and obviously it's invalid.
It's taken directly from MouseInfoProvider right after MouseInfoProvider object creation.
In MouseInfoProvider constructor, _previousMousePosition  is not getting initialized, therefore -1,1 can be returned.
Fix description
Update mouse information in MouseInfoProvider constructor, therefore information provided by the object is valid right after it's creation.

Validation Steps Performed
Check Color Picker logs for warnings

Code looks good, will give a quick test then merge.

Nevermind. Something is broke. This is what I get when trying to call Color Picker on Windows 10 with these changes:

I've found the issue. Going to fix it.

@jaimecbernardo the issue is fixed. Works well on my side now.

@crutkas , While one of the license/cla checks seems to work, the other is just queued without working. Taras has signed it, though. Any action we can do to have the queued license/cla re-calculate?

Think we are going to have to close and make a new PR

@microsoft-github-policy-service agree [company="{your company}"]

@microsoft-github-policy-service agree [company="Janea Systems"]

@microsoft-github-policy-service agree [company="Janea Systems"]

@microsoft-github-policy-service agree company="Janea Systems"


[Performance] Improve show proc statistic #6477

Proposed changes
To speed up show proc '/statistic', this pr refactor the statistic structure to make the logic clear, and use parallel computing.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)
[x] Code refactor (Modify the code structure, format the code, etc...)
[x] Optimization. Including functional usability improvements and performance improvements.
[ ] Dependency. Such as changes related to third-party components.
[ ] Other.

Checklist

[x] I have created an issue on (Fix #6477) and described the bug/feature there in detail
[x] Compiling and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] If these changes need document changes, I have updated the document
[x] Any dependent changes have been merged

There is a problem with the use of parallelStream and it's ForkJoinPool.

There is a problem with the use of parallelStream and it's ForkJoinPool.

avoid to use nested parallelStream, problem solved

Hi @ccoffline , please rebase the master to solve the conflict


or behaving unexpectedly in blade

Laravel Version: 5.3.30
PHP Version: 7.1.1
Database Driver & Version: MySQL driver (MariaDB 10.1.19-win32)

Description:
When using {{ auth()->user()->name or 'No name' }} the output is 1 instead of the user's name.
Same happens with {{ \App\User::first()->name or 'No name' }}..
However, if you send the data from a controller, it works as expected:
$user = auth()->user();
return view('someView')->with('user', $user);

{{ $user->name or 'No name' }}

Steps To Reproduce:
Put {{ auth()->user()->name or 'No name' }} in your blade template.

Looking at the regex it's explicitly meant to be used with variables as the first character must be $, I suggest you use {{ auth()->user()->name ?? 'No name' }} since you're on PHP 7.1 already.

@themsaid
I do usually use ?? but I did not know or was only for variables.
Thank you for your reply and I appreciate the quick response.


💚Move tests to net7 to fix build

Test fixes: cherry-picked from #2644
Add some preprocessor symbols for .NET 7 support

Since creating the PR, I haven't looked at this again. I guess it would be good timing to sync this with the upcoming .NET 8 changes? But I haven't checked if this is tracked anywhere.

Thanks for contributing - if you'd like to continue with this pull request, please rebase against the default branch to pick up our new CI.
From looking at the description only, some parts of this will have been addressed by #2775.

Thanks for the contribution. The TFM updates are covered by the changes in #2799 to add support for .NET 8, and the other changes look like they were covered with #2775.


exploitdb: 2021-06-19 -> 2021-06-23

Motivation for this change
Update to latest upstream release 2021-06-23
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 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

[x] Fits CONTRIBUTING.md.

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

1 package built:

exploitdb

Result of nixpkgs-review pr 127874 at bbe12794 run on x86_64-linux 1

1 package built successfully:

exploitdb

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/security/exploitdb/default.nix:14:3:
   |
14 |   installPhase = ''
   |   ^


Choose multiple pull request template

Hi ! I would like to choose among pull request templates when creating one.
I know for only one template Github integrate automatically but not for multiple templates.
Something like dropdown would be amazing 🤩

Duplicate of #1621


Rs 2022 06 26.1

these files are so nostalgic

second release and somehow I'm still scrambling to review the same damn portion of the game


Print CPUINFO also for s390 processors

I would tend to leave the size unchanged for platforms other than s390x

done.

This pull request is ready to merge

merged.
master: https://github.com/openssl/openssl/commit/b55a0b0ff209f7004c16e1bf69dd557167878fd6


Center rows in column for proper alignment

Hello, I really enjoy using your MagicMirrorModule.
While there are requests to provide some kind of table layout, which could be achieved by simply adding "border: 1px solid white;" to the .col element in the css file, I'm curious on how to align the rows within each column.
When using a border you can see that all columns (up to three) in a row are aligned to the left (except the date) side which leads to irregular distances between each day.
Maybe someone can provide adivice. Maybe this improvement would refer to #28.
regards.

Well, it took me like six hours to figure it out, but in the end this works:
col.setAttribute("align","center");
It has to be added within the MMM-CalendarWeek.js after the var col has been created (line 254).
Maybe somebody wants to add this and make it configurable from within the config.js
regards.

could you elaborate on what you did to get everything centered?


Import Error with strip_on_import

When using "strip_on_import", I get subsequent import errors (Py2.7).
To reproduce, you can use the built-in xml module as an example, but it should be the same for any module:
import strip_hints

strip_hints.strip_on_import("/usr/lib64/python2.7/xml/parsers/__init__.py") # Replace with your location

from xml import parsers

The result:
ImportError: cannot import name parsers

It's not an issue with the actual stripping, but with the "on_import" part.

The import hooks were not handling the dotted imports correctly.  I've pushed a new version with some fixes.  With Python 3 the py3_also flag to strip_on_import must be false.

Nice, thanks for the quick fix @abarker, I gave it a quick test and it seems to work now.
Just for completeness sake, here's an example of how I previously worked around this: https://github.com/PaulSchweizer/ascii-canvas/blob/master/ascii_canvas/init.py#L8

Just a note that stripping on import does not work for Python 3.6, and may or may not work for earlier versions of Python 3.  The code has only been tested for 2.7 and 3.6, but for 3.6 I get 'StripHintsImporter' object has no attribute 'find_spec' errors.
Python 3 import hooks should really use importlib rather than the legacy, deprecated imp module that is currently used for Python 2.  It should also be much simpler to implement, but it hasn't currently been done.

Update: I went ahead and pushed a version to GitHub that uses importlib for Python 3 strip-on-imports.

Thanks @abarker for the great work, I'm using it as intended now, take a look here: https://github.com/PaulSchweizer/ascii-canvas/blob/master/ascii_canvas/init.py


fix: reverse artist is checked value

Description
This PR fixes an issue where the copy is not matching the displayed text for my collection artist. The logic should be: if an artist is private, do not tick the box.

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
#nochangelog
Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

✅ No changelog changes

  Generated by :no_entry_sign: dangerJS against 35eb586838a6a279ed6580118044a5e2acd8959a


CC/CA 65, not 64

As long as you're in there... correct external project reference

Sorry I just pushed and now there's a conflict :)


Changing the structure of atmos terraform ...

Describe the Feature
This is just my opinion, but the structure of atmos is a bit funky when compared to most other CLI tools.
I'm referring to this structure: atmos terraform <command> <component> -s <stack> [options].
The atmos required arguments of component + stack being between command + options has thrown me and others off numerous times now.
Expected Behavior
I'm hoping for a more intuitive structure.
Use Case
N / A
Describe Ideal Solution
My ideal structructure: atmos terraform <command> [options] -c <component> -s <stack>
Alternatives Considered
N/A
Additional Context

If this is of interest to discuss further, I can shed more thoughts on this and try to find examples of new folks that I've introduced to atmos getting hung up on the current structure.

Thanks for the feedback - we'll discuss on our next ARB call when I'm back from vacation.

I agree. A non positional flag for component would make it more intuitive for newcomers and for experienced users. I'd be ok with keeping the positional argument for component and adding a --component and -c flag as an option. Then if either the positional was passed in or the explicit flag was passed in, it should work.
If you want that setup now, you could do this with the custom commands configuration.

My ideal would be
atmos -c <component> -s <stack> terraform <command> [options]

It could be put in a major version change like 2.x, no? Usually the major version is when we break backwards compatibility.
Or it could possibly be enabled with a switch in atmos.yaml?

https://cloudposse.atlassian.net/browse/DEV-905

What if it was instead atmos terraform plan <component> -s <stack> -- arguments?
Adding the -c and -s before the subcommand doesn't make sense, since those are not global options. It's incongruent with just about any command line arguments I can think of.
Using the -- convention is common. Basically, everything after this is passed directly to terraform verbatim.


Cannot export Python Jupyter Notebook due to issue in Python extension

Environment data

VS Code version: 1.52.1
Extension version (available under the Extensions sidebar): v2020.12.424452561
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Python 3.9.1
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: N/A
Relevant/affected Python-related VS Code extensions and their versions: ms-toolsai.jupyter v2020.12.414227025
Value of the python.languageServer setting: Jedi

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
See https://github.com/microsoft/vscode/issues/110934
Actual behaviour
See https://github.com/microsoft/vscode/issues/110934
Steps to reproduce:
See https://github.com/microsoft/vscode/issues/110934
Note that this was deemed by @rebornix to be a vscode-python issue

Logs

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

None

@oaustegard I am able to reproduce.
Could you please provide the output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

@oaustegard I am able to reproduce.
Could you please provide the output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

> C:\Python39\python.exe c:\Users\austegard.oskar\.vscode\extensions\ms-toolsai.jupyter-2020.12.414227025\pythonFiles\pyvsc-run-isolated.py jupyter nbconvert --version
Error 2021-01-15 19:35:45: Export failed [r [Error]: Traceback (most recent call last):
  File "c:\Users\austegard.oskar\.vscode\extensions\ms-toolsai.jupyter-2020.12.414227025\pythonFiles\pyvsc-run-isolated.py", line 26, in <module>
    runpy.run_module(module, run_name="__main__", alter_sys=True)
  File "C:\Python39\lib\runpy.py", line 210, in run_module
    return _run_module_code(code, init_globals, run_name, mod_spec)
  File "C:\Python39\lib\runpy.py", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "C:\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\austegard.oskar\AppData\Roaming\Python\Python39\site-packages\jupyter.py", line 4, in <module>
    main()
  File "C:\Users\austegard.oskar\AppData\Roaming\Python\Python39\site-packages\jupyter_core\command.py", line 285, in main
    command = _jupyter_abspath(subcommand)
  File "C:\Users\austegard.oskar\AppData\Roaming\Python\Python39\site-packages\jupyter_core\command.py", line 124, in _jupyter_abspath
    raise Exception(
Exception: Jupyter command `jupyter-nbconvert` not found.

	at ChildProcess.<anonymous> (c:\Users\austegard.oskar\.vscode\extensions\ms-toolsai.jupyter-2020.12.414227025\out\client\extension.js:9:101000)
	at Object.onceWrapper (events.js:313:26)
	at ChildProcess.emit (events.js:223:5)
	at maybeClose (internal/child_process.js:1021:16)
	at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)]
	```

> C:\Python39\python.exe c:\Users\austegard.oskar\.vscode\extensions\ms-toolsai.jupyter-2020.12.414227025\pythonFiles\pyvsc-run-isolated.py jupyter nbconvert --version
Error 2021-01-15 19:35:45: Export failed [r [Error]: Traceback (most recent call last):
  File "c:\Users\austegard.oskar\.vscode\extensions\ms-toolsai.jupyter-2020.12.414227025\pythonFiles\pyvsc-run-isolated.py", line 26, in <module>
    runpy.run_module(module, run_name="__main__", alter_sys=True)
  File "C:\Python39\lib\runpy.py", line 210, in run_module
    return _run_module_code(code, init_globals, run_name, mod_spec)
  File "C:\Python39\lib\runpy.py", line 97, in _run_module_code
    _run_code(code, mod_globals, init_globals,
  File "C:\Python39\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\austegard.oskar\AppData\Roaming\Python\Python39\site-packages\jupyter.py", line 4, in <module>
    main()
  File "C:\Users\austegard.oskar\AppData\Roaming\Python\Python39\site-packages\jupyter_core\command.py", line 285, in main
    command = _jupyter_abspath(subcommand)
  File "C:\Users\austegard.oskar\AppData\Roaming\Python\Python39\site-packages\jupyter_core\command.py", line 124, in _jupyter_abspath
    raise Exception(
Exception: Jupyter command `jupyter-nbconvert` not found.

	at ChildProcess.<anonymous> (c:\Users\austegard.oskar\.vscode\extensions\ms-toolsai.jupyter-2020.12.414227025\out\client\extension.js:9:101000)
	at Object.onceWrapper (events.js:313:26)
	at ChildProcess.emit (events.js:223:5)
	at maybeClose (internal/child_process.js:1021:16)
	at Process.ChildProcess._handle.onexit (internal/child_process.js:283:5)]
	```

Likely a dupe of #4347

Likely a dupe of #4347


Fix opensaml bundle not resolving issue in the CAS authenticator

Description
With the commons-codec upgrade https://github.com/wso2/product-is/issues/8042
The opensaml bundle used in CAS authenticator is not getting resolved at the server startup.
This needs to be fixed.

upgrade the codec version. Didn't observe any issue.
https://github.com/wso2-extensions/identity-inbound-auth-cas/


NavLinks with parentheses in the path do not set active className

The active link for NavLinks are not created for paths that have parentheses.  It seems that the underlying regex that is generated by path-to-regex does not escape the parentheses in the generated regex and instead interprets it as a match group.  Notice in the example how the active classname is not set for the about component.
Version
4.1.1
Test Case
https://codesandbox.io/s/n55VljYk7
Steps to reproduce
Create a NavLink component whose path has parentheses, ie /some/path (1).
Expected Behavior
When the component is routed to and rendering the active classname should be added.
Actual Behavior
The classname is missing.
I wasn't sure if this was expected behavior in the underlying path-to-regex, so I figured it'd be best to ask here first.

Oops, didn't save the Test Case:
https://codesandbox.io/s/qqov6y0w3q

If you want to submit a PR for this, I think that you would need to modify the value passed to path by escaping any of path-to-regexp's "special" characters.

Thanks for the info.  I did a poor job searching path-to-regexp's issue list.  I'll try my hand at tackling this issue.  Should be easy enough.


Feature request Retrieve header/cookies

As a user i want the ability to develop apps that require extra info outside of just JSON such as cookies or headers.
Currently the only way of grabbing the response object itself to grab headers/cookies/ResponseCode from that response is by changing the 'Accept' header to be something other than JSON (was done to allow for xml code at time). But this usecase of requiring access to headers/cookies can still apply to queries using a "Accept": "application/json" etc

theres nothing stopping you from using just "fetch" instead of "args.fetch".  args.fetch just makes "fetch" a little easier to use with relative to absolute URL etc.   so you should be able to achieve what you are after by just using "fetch" directly.

sounds fine to me


How does SAM solve the problem of catastrophi forgetting problem?

How does SAM solve the long-term learning problem? That is, what methods he would take to overcome catastrophic forgetting.

same question


Fee approximation question

Hey,
I'm trying to understand the fee approximation algorithm here:
https://github.com/ixofoundation/ixo-blockchain/blob/master/x/ixo/internal/types/auth.go#L99-L121
I need this ported to Javascript and the client side. I'm not asking for code :), but some details regarding the steps would be helpful for me for porting, especially since I'm no blockchain expert.
Also please inform me if this is not the right place to ask such a question.

Hi,
The key function there is the EnrichWithGas function, which is a standard Cosmos SDK function. You can read a (very) short description about it here:
<EMAIL_ADDRESS>Basically what it does is that it simulates the processing of the transaction to figure out how expensive it will be (i.e. how much gas it will consume). I do not recommend that you implement this on the client side since there's a lot happening in the background and you are better off requesting the gas consumption from the blockchain. This is available via the POST endpoints for each module: https://cosmos.network/rpc/v0.38.3
For ixo transactions, similar endpoints are available but are not documented! There is also a /txs/sign_data endpoint that one can use but even though it is convenient, I suggest avoiding it if possible.
After EnrichWithGas is used, the function just uses the txBldr (now containing gas consumption information from EnrichWithGas) to calculate the actual fee (in IXO tokens) using the formula fee = gas * gas prices where gas prices is approximated using an expected value of 0.025uixo.
For example if:

Transaction will consume 100,000 gas (note: no currencies here)
Validator will (approximately) charge 0.025uixo for each unit of gas
Total fee will be 100,000 x 0.025uixo = 2500uixo

Thank you @migueldingli1997 , your detailed response is really appreciated! I'm closing the issue, will follow up if necessary.


Failed to execute command "localsend_app"

Describe the bug
I downloaded the *.deb file
sudo dpkg -i LocalSend-1.14.0-linux-x86-64.deb
Selecting previously unselected package localsend.
(Reading database ... 115964 files and directories currently installed.)
Preparing to unpack LocalSend-1.14.0-linux-x86-64.deb ...
Unpacking localsend (1.14.0+45) ...
dpkg: dependency problems prevent configuration of localsend:
localsend depends on gir1.2-appindicator3-0.1 | gir1.2-ayatanaappindicator3-0.1; however:
Package gir1.2-appindicator3-0.1 is not installed.
Package gir1.2-ayatanaappindicator3-0.1 is not installed.
dpkg: error processing package localsend (--install):
dependency problems - leaving unconfigured
Processing triggers for desktop-file-utils (0.26-1) ...
Processing triggers for mailcap (3.70+nmu1) ...
Processing triggers for hicolor-icon-theme (0.17-2) ...
Errors were encountered while processing:
localsend
neo@debian:~/Downloads/localsend$
popup error says:
Failed to execute command "localsend_app"
Failed to execute child process "localsend_app" (No such file or directory)
I am running  the following:
Debian GNU/Linux 12 (bookworm)
64bit
xfce version: 4.18
GTK version: 3.24.38
kernel version: 6.1.0-22-amd64
To reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

Expected behavior
Should install
Screenshots

Desktop (please complete the following information)
No response
Smartphone (please complete the following information)
No response
Additional context
No response

sudo apt install gir1.2-appindicator3-0.1

Have you tried using this?

sudo apt install gir1.2-appindicator3-0.1

Have you tried using this?

I was told NOT to install that way.
I discovered that the instructions on how to install this deb file would not work for me, I don't know about others.
I have several ways to install listed below (yes I am a new Linux user so I assume you already know this):
How to install .deb package ?
LocalSend-1.14.0-linux-x86-64.deb
Verify installation is successful by checking the applications menu
(1) sudo apt install ./.deb , 	handles dependecies
(2) sudo dpkg -i ./.deb , 	does not handle dependecies, low level tool
(3) use Gdebi GUI package-file, sudo apt install gdebi , 3rd party installer ?
sudo gdebi-gtk ./.deb
OR right click and install with gdebi application
(4) use Gdebt CLI, sudo gdebi ./.deb
(5) from Thunar file manager: double left clicking installation to install *.deb package
I choose (1).
It worked fine and I can send/receive files from others but I received an error which I listed below.
Can you please explain what this error is ?
neo@debian:~/Downloads/localsend$ sudo apt install ./LocalSend-1.14.0-linux-x86-64.deb
[sudo] password for neo:
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Note, selecting 'localsend' instead of './LocalSend-1.14.0-linux-x86-64.deb'
The following NEW packages will be installed:
localsend
0 upgraded, 1 newly installed, 0 to remove and 1 not upgraded.
Need to get 0 B/9,354 kB of archives.
After this operation, 31.3 MB of additional disk space will be used.
Get:1 /home/neo/Downloads/localsend/LocalSend-1.14.0-linux-x86-64.deb localsend amd64 1.14.0+45 [9,354 kB]
Selecting previously unselected package localsend.
(Reading database ... 119051 files and directories currently installed.)
Preparing to unpack .../LocalSend-1.14.0-linux-x86-64.deb ...
Unpacking localsend (1.14.0+45) ...
Setting up localsend (1.14.0+45) ...
Installed Localsend successfully
Processing triggers for desktop-file-utils (0.26-1) ...
Processing triggers for hicolor-icon-theme (0.17-2) ...
Processing triggers for mailcap (3.70+nmu1) ...
N: Download is performed unsandboxed as root as file '/home/neo/Downloads/localsend/LocalSend-1.14.0-linux-x86-64.deb' couldn't be accessed by user '_apt'. - pkgAcquire::Run (13: Permission denied)
neo@debian:~/Downloads/localsend$
Thanks.

It's nothing important. You can ignore it since Localsend was successfully installed.


HN changed their layout again

Top and New stories doesn't load.
Best stories and Ask will load the titles but no comments.

I've started updating the JSON Configuration file to reflect the new layout, and posts seem to work, but comments need more work. I'm at work right now so I don't have a lot of time to mess with this right now. The easiest way to get this working would be to make changes to a local copy of hn.json and my News/YC app to make sure it works.

Should be fixed now.

@bennyguitar ah confirmed its working now! Thanks again :) Also loving this new json configuration stuff :) Also I still owe u a copy of my mac app, where can i send you a redeem code?

You can send one to<EMAIL_ADDRESS>and thanks for using my library!

@bennyguitar oh plz thanks for building it :). Just sent you the code to redeem the app from the Mac App Store :)


Implement GraphQL + Apollo/Relay for the Node + React codebases

Originally brought up by @kentcdodds here; further discussion here.
Opening this issue to get a discussion going!

I can help with this (eventually)

Awesome! Wanna fork the starter kit and post the link here so that others can watch/help?

I don't have time right now, but I will see if I can connect you with people in the Apollo community who have more time than I do

I've been talking a bit with mxstbr here.  He might be willing to help with it.  He was mentioned in the tweet that you linked to above

I think this boilerplate https://github.com/entria/graphql-dataloader-boilerplate can be used as a start point
I have this one for React Native + React Navigation + Relay https://github.com/sibelius/ReactNavigationRelay.
It is almost migrated to Relay Modern as well: https://github.com/sibelius/ReactNavigationRelay/pull/5

This is a great idea! Unfortunately I don't have time to work on this myself but would love to help if I can.

@sibelius awesome! Would you be interested in taking lead on GraphQL/Relay?
@stubailo no worries — would be ❤ if you could give feedback on the final code once it's ready!

@EricSimons would definitely encourage use of Apollo over Relay.  It seems to be getting more traction these days

We can do both, Apollo, Relay and Relay Modern.
I can help with any questions for sure.
I don't have time to work on this myself

@leebyron do you know of anyone in the GraphQL community who might be interested in this?

@ericSimons I can help with the GraphQL implementation in the server side.

@thebergamo awesome! I just spun up a new issue for it (https://github.com/gothinkster/realworld/issues/46) — can you fork the starter kit and post it in the issue? I'll update the status afterwards 👍

I can chip away at a Relay Modern implementation, @EricSimons, @thebergamo.

@EricSimons done!

@steveluscher awesome! Just spun up a new issue for it — can you fork the starter kit and post it in the issue? I'll update the status afterwards 👍
Also, I created a gitter room so that all of the folks working on graphql/apollo/relay can chat and coordinate (since they rely on each other) cc/ @thebergamo

It looks like this issue has been split into multiple other issues. Do we still need this one or can we close it?

@EricSimons I can handle Apollo if you give me a few weeks haha

@thebigredgeek all help will be very welcomed! until the end of week we will have a version of the server side graphql implemented with Apollo

@thebergamo @thebigredgeek should I create a new issue for the Apollo part of this (right now we have GraphQL and Relay issues)?

I'm planning using Apollo as a graphql server. Probably isn't necessary create a new issue.

@thebergamo apollo has graphql-server, but their main focus is apollo-client which is the most popular client-side graphql implementation.

@EricSimons same issue is probably fine

@thebergamo and anyone else here: If you want, you can join the #realworld channel in the Apollo Slack to help coordinate an example, both server and client!

@stubailo I'm in! Thanks for the option!

Is there any progress happening on the apollo/graphql example?
I could definitely help, i did put something together last year, but i reckon an example here would make a lot more impact.
https://github.com/ap-o/intro

I have done a GraphQL implementation with React + Apollo front-end and Ruby on Rails back-end.
Front-end - https://github.com/dostu/react-apollo-realworld-example-app
Back-end - https://github.com/dostu/rails-graphql-realworld-example-app
Demo - http://realworld-react-apollo.herokuapp.com/
Would be great to see other implementations, and mix and match them like we can with the REST API. For that, we should first decide on a schema. So far I have come up with this: https://github.com/dostu/rails-graphql-realworld-example-app/blob/master/GRAPHQL_API_SPEC.md. Would love to hear any opinions about it.

Found https://github.com/thebergamo/realworld-graphql by @thebergamo which is pretty nice, just wish it was in express instead.

I am working on a java version GraphQL implementation recently after releasing GraphQL ecosystem is quite sophisticated recently. The code base is also https://github.com/gothinkster/spring-boot-realworld-example-app, the same place of the REST implementation, which means it reuse lots of common codes (such as database access, input validation).
The GraphQL schema is firstly copy from https://github.com/thebergamo/realworld-graphql. But I changed some details about pagination, comments:
https://github.com/gothinkster/spring-boot-realworld-example-app/blob/master/src/main/resources/schema/schema.graphqls
The GraphQL code is using dgs-framework just release recently by Netflix.

I have a Relay Real World example here https://github.com/sibelius/relay-workshop
it also has 12 practical exercises to help learning Relay hooks
it is a tiny social network https://react-europe-relay-workshop.now.sh/

Hello, we moved the addition of new implementations to this website : https://codebase.show/projects/realworld
Please submit your implementation there.


FATAL: Error, the fuse library (libfuse.so.x from fuse2, not fuse3) must be installed

Based on my run using the linux runner there is an mismatch in the provided libs.
See this run.

@jimkring the needs for libfuse was a thing when Nuitka was still using a special package on Linux, but this is no longer true, so there is no need to check this. I will create a PR if I figure out how to do it. :)

Ah wait, this is using ancient Nuitka 0.9 and Nuitka is the one asking for it. That is of couse unsupported.


Merge debug configs always when generating

This is to fix the issue where the launch.json is only updated for projects that are open when editing the runtime path is set.

Added launchConfigKeysToUpdate which defines what parameters of launch config can be updated by new config.
Clone old config and then merge new config into clone. If the old config and new merged config are not the same then update.
Added option to not show dialogs

@stevapple what are your thoughts?

I learnt that there's no inout parameter in TS... Have you verified that this will work?

I learnt that there's no inout parameter in TS... Have you verified that this will work?

Yes it is verified. There might not be any inout parameters in TS, but every object is a reference object, so if you pass an object into a function and update it inside that function the reference outside the function also gets those changes. That is why I have to do the deep copy before calling updateConfigWithNewKeys.


fix(ara): Set default probe path to /healthcheck/

ARA has built-in healthcheck under /healthcheck/ path.
It is better suited for probes, especially in the case of also setting ARA_READ_LOGIN_REQUIRED to True. Previously used /api/v1/ will return 401 in that case, failing the check, even though app is running.

lgtm. thx


KeyError: 'tools'

View details in Rollbar: https://rollbar.com/Zooniverse/Zooniverse_Aggregation/items/49/
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/rq/worker.py", line 568, in perform_job
    rv = job.perform()
  File "/usr/local/lib/python2.7/dist-packages/rq/job.py", line 495, in perform
    self._result = self.func(*self.args, **self.kwargs)
  File "/app/engine/jobs.py", line 10, in aggregate
    with AggregationAPI(project_id, environment=environment) as project:
  File "/app/engine/aggregation_api.py", line 224, in __init__
    self.__setup__()
  File "/app/engine/aggregation_api.py", line 312, in __setup__
    self.workflows,self.versions,self.instructions,self.updated_at_timestamps = self.__get_workflow_details__()
  File "/app/engine/aggregation_api.py", line 888, in __get_workflow_details__
    for tool_index,tool in enumerate(task["tools"]):
KeyError: 'tools'

this was caused by the retina project and a drawing task. An example drawing task from Whales as Individuals has the keys [u'tools', u'instruction', u'type', u'help', u'next']. I use tool to figure out what marking tools the user is using for the task. Note that for WasI, for this task we have type ="drawing".
For Retina, I only have the keys [u'instruction', u'type', u'help', u'next'] with type = "crop". I don't know how much of Retina is based on Panoptes and how much is a custom build but for now I think this is more that the aggregation does support the custom tools in Retina rather than a bug that would affect other projects.
@camallen @aliburchard - can you tell more about the Retina tool. I have some vague memory of Ali saying that the retina team would take care of the aggregation themselves.

retina is a panoptes front end project using an experimental tool. best to ask @brian-c for details on how the tool works.

Ah, sorry: A crop task produces an annotation with a value map of the x, y, width, and height of the area the user cropped from the natural width and height of the subject image.
After a crop task, the view of the subject is cropped to those bounds, but maintaining the natural aspect ratio of the image (for now).

Still not sure what I'm supposed to be doing with results from the crop tool - suspect that this will involve a talk with Ali and the Retina people. Will have to wait until I get back from vacation.

@ggdhines we'll want to incorporate the crop tool into aggregation for future projects that might use it. The retinas team turned out to not be doing the aggregation we anticipated (aggregating polygons, and then calculating polygon ratio), so it might actually be a good test case to build the aggregation on.
But yes, can and should wait until you get back from holiday.


[FLINK-31319][connectors/kafka] Kafka new source partitionDiscoveryIntervalMs=0 cause bounded source can not quit

What is the purpose of the change
Fix error kafka partitionDiscoveryIntervalMs checking for noMoreNewSplits.
Brief change log
Correct partitionDiscoveryIntervalMs checking for noMoreNewSplits with <=0
Verifying this change
Add KafkaEnumeratorTest
#testRunWithDiscoverPartitionsOnceWithZeroMsToCheckNoMoreSplit,
#testRunWithDiscoverPartitionsToCheckNoMoreSplitOnce
#testRunWithPeriodicPartitionDiscoveryToCheckNoMoreSplitOnce
new cases and refine existed cases.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature?  no
If yes, how is the feature documented? docs

@flinkbot run azure

@PatrickRen hi, qingsheng. PTAL.

Thanks for the patch @chucheng92 ! LGTM.
As Kafka connector will be migrated from Flink main repo to an individual repo since 1.18, could you close this PR and create a new one in the flink-connector-kafka repo? Also it'll be nice to back port this patch to 1.16 and 1.17 in Flink repo.

OK. thanks QingSheng.

Thanks for the patch @chucheng92 ! LGTM.
As Kafka connector will be migrated from Flink main repo to an individual repo since 1.18, could you close this PR and create a new one in the flink-connector-kafka repo? Also it'll be nice to back port this patch to 1.16 and 1.17 in Flink repo.

OK. thanks QingSheng.

PR-8(external-repo)
BP-1.16
BP-1.17

@PatrickRen Hi, QingSheng. Can you help to check it?


Roll Flutter Engine from 9891efea6966 to 3125a760ad61 (1 revision)

https://github.com/flutter/engine/compare/9891efea6966...3125a760ad61
2024-08-22<EMAIL_ADDRESS>Roll Skia from a9af6f466355 to 0429e894d3f6 (1 revision) (flutter/engine#54707)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jimgraham@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Updating doesn't change the start command

When the server updates to another version, the .jar will be named differently, but the start command in the service file wont be changed to reflect this.
It would also be good to cleanup the old .jar when updating, and have warnings when the user tries to switch from one server software to another. Examples below. I don't think disallowing these switches is good either because there are some legitimate scenarios where it could work perfectly fine.
Switching from vanilla --> paper/purpur is fully supported. And purpur is a drop-in replacement for paper so switching from paper --> purpur is fully supported. But switching from fabric --> anything else (except manual) is unsupported because many mods change the world. Purpur --> paper works, but you will lose any purpur-specific configuration. And of course the script doesn't know what the manual server .jar is, so it's up to the user to make sure the switch is compatible.
Downgrading minecraft versions doesn't work unless you want to reset your world, or more depending on how far you go back.
Went on a little rant, but it's still kinda related lol.

Thanks for your detailed comment! You're absolutely right about the .jar name changes after updates. I'll look into automating the start command in the service file. Cleaning up old .jars during updates is a great idea too; I'll work on that.
Switching server software should definitely come with warnings based on scenarios like vanilla to paper/purpur being supported but fabric to anything else being tricky due to mod changes.
And yes, downgrading Minecraft versions can be a headache, so a warning about potential world resets is smart. Your rant is definitely related and appreciated! LOL It's feedback like this that helps improve the tools and scripts for everyone. I'll work on integrating these ideas into the next update. Thanks again for your insights!
Also if you see anything else, let me know or see if you can come up with a improvement.


[occm] octavia: support fallback to TCP health monitor

/kind feature
The OVN provider load-balancer in OpenStack is getting in shape ... and in many ways a nice option when used in k8s, e.g. in front of an ingress controller:

It's lightweight, fast to create, light on resource consumption (compared to amphorae)
It preserves the client IPs without resorting to hacks like the PROXY protocol

Unfortunately, the health monitors for OVN LBs don't support HTTP mode, only TCP (and UDP-CONNECT). This may be resolved in OVN/octavia/neutron some day, but it does not appear trivial to me.
It would be nice if OCCM would allow using a TCP health-monitor instead of HTTP.
Environment:
OCCM-1.26.2
OpenStack Zed
OVN-23.03

Discussion:

HTTP health-monitor is better than TCP health monitor, as there is a small chance that services that are stuck still accept TCP connections without really responding to any HTTP.
TCP health monitor is much better than no monitor at all (as it still detects most issues with the backend service, like e.g. the node having gone away, the network connection being disturbed, the process/container having died).

Implementation options:
(1) This could be made configurable with yet another custom annotation.
(2) Or we just fall back in case there is no HTTP health-monitor support (which is currently hardcoded if OVN is being used as LB provider).
(3) We do nothing an wait for OVN/Octavia to deliver HTTP health monitors (at which point in time we remove the hardcoded knowledge about HTTP health-mon non-support)

PoC patch for option (2) is here:
https://github.com/SovereignCloudStack/issues/issues/298#issuecomment-1529832566
It has been tested successfully.
If this finds the support of the OCCM experts, I'm happy to submit it via a PR here.
If there is preference to make this configurable via yet-another annotation, that can be done as well, of course.
Guidance appreciated.

seems reasonable to me but I can't call myself expert in this LB area...
so @dulek please comment here in case you have any additional comments

Should I fork it into my namespace and create a PR from there to contribute this?
Or do you have a mechanism to grant write access so I can commit to a branch and do the PR from there?
(Sorry for these questions -- while I have contributed to many OSS projects, I don't know all the details on the process in the CNCF world yet.)

Should I fork it into my namespace and create a PR from there to contribute this?

this is the right approach.. https://github.com/kubernetes/cloud-provider-openstack/blob/master/docs/developers-guide.md might be helpful?


Identity: Page keeps reloading if opening the unfinished project immediately

Describe the bug
Identity: Page keeps reloading if opening the unfinished project immediately
Steps to reproduce the behavior
1.Navigate to https://arcade.makecode.com/beta#
2.Sign in Microsoft account
3.Open a tutorial and don't finish it
4.Ruturn to the homepage
5.Open this unfinished project
Actual behavior
Page keeps reloading if opening the unfinished project immediately

Additional context
1.OS: Windows(rs6)
2.arcade version: 1.4.6
3.Microsoft MakeCode version: 6.9.7

Dupe of: https://github.com/microsoft/pxt-arcade/issues/3058

Doesn't repro for me any more.


Listing tile

Ref.: https://github.com/plone/volto/issues/737

Listing tile will also have collection powers. See also #287

Updated with new sidebar, now waiting for the querystring widget. See #801

Updated to latest master. It would be awesome to keep working on this block.
@tisto @sneridagh Is it ok to go on with this, or do you have something different in mind?
AFAIK, this needs a check on keyboard behavior while going through tiles in edit mode, and all the sidebar querystring widgets.

@pnicolli delivered what I promised. Now it's all yours for polishing, etc... the CSS might want some love for sure.

@pnicolli I started to use this convention of naming components in a Block. It turned out very well, and allow good composability/extensibility. I would like to use it in all blocks. So it would be like:

View - entrypoint for the block, same as before, try to be superslim
Edit - same as above
Body - The body of the UI unit, trying to be as autocontained as possible. eg. if it's an image, it should be able to know how to upload an image by itself. For the grids this is very important. In the case of this Block, I still think it could be improved... I will think about it.
Data - the parameterization of the Block, what goes inside the sidebar
Sidebar - the content of the sidebar, this is important in case you want to reuse Data components from other blocks, eg. images.

What do you think?
/cc @tisto @jackahl @robdayz @robgietema @rodfersou

@pnicolli just pushed more improvements.
I've also thought that a setting for parameterizing the block in case you want to show the image or not would be suitable too.

@sneridagh Great news! Will look into it asap.
I like the naming convention after the first look, will also look deeper into this with other Blocks.
Especially for this block, it is probably very useful because we will need several listing blocks with different styles and this would help a lot.

@pnicolli @giuliaghisini is this ready to merge? If green I will merge then.

@sneridagh Last time I checked, only tests needed some work. Will take a look at failing tests asap so we can eventually merge this.

@sneridagh Looked into it. There was a bug in tests due to another merged fix (sidebar default tab), and this is fixed.
Actually, at this time, the relative path criterion isn't working, so its test is correctly failing. I tried both with the provided plone buildout as-is ('api' folder) and with the same buildout but activating the latest master for plone.rest, plone.restapi and kitconcept.voltodemo.
It needs some debugging to understand where the problem is (querystring widget or restapi), I will work more on it in the next 2 days and will post another comment here. (see #1059)

Found the problem with relative path and fixed it. The querystring endpoint doesn't allow calling it on a context, and there was no way of calculating the correct context for the relative path. At this time, I fixed it in the querystring action by manipulating the query in the relative path case.


refactor(dashboard): split the rendering decisions from the data decisions

Closes #653
Summary

Downstream consumers that want to build their own dashboard with custom card types can't reuse our Dashboard component as it only supports a fixed set of card types.

Change List (commits, features, bugs, etc)

refactor(Card): move the isExpanded decision into the Card component so downstream teams can share the isExpanded behavior
refactor(Dashboard): split the rendering into a new component (DashboardGrid) so downstream teams can render their own card types.
refactor(Dashboard): Remove the ability to pass custom row heights, gutters, dimensions to the component to standardize our paddings across the whole IoT Dashboard organization
-refactor(CardRenderer): move the isExpanded styles out of the CardRenderer component so downstream teams can share the isExpanded behavior.

Acceptance Test (how to verify the PR)

You can verify that I didn't break anything by running the existing Dashboard and card stories

@stuckless you can do either though the second is better as it will support the isExpanded bit automatically.  I'll add a DashboardGrid story to serve as the README

@scottdickerson that sounds good.  The sooner we can get this mergedt he better, so that I can start on the maximo integration pieces.  thx.

:tada: This PR is included in version 2.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fixed the loading issue of nuget packages

fixed the loading issue of nuget packages
remove unused code

@geffzhang Is your comit still working? I cannot verify changes to Tencent.

my commit has done @seriouz


Add F-Droid and Google Play Badges with Links

Good PR, but I don't want to merge it now. I am going to do some improvements for F-Droid version

Similar PR already exist:
https://github.com/flipperdevices/Flipper-Android-App/pull/450
But thank you for contribution!


Dockerfile: apt: do not install recommended packages

This is quite beneficial on the image size (40%+ image shrink). Do you think we currently rely on any package being present because of a recommendation instead of it being explicitly installed?

This is quite beneficial on the image size (40%+ image shrink).
great

Do you think we currently rely on any package being present because of a recommendation instead of it being explicitly installed?
I hope not. In any case we can explicitly add them


Clarification on use with SPAs

Thanks very much for all the effort that's gone into this package - it's a great resource!
I have a question regarding the use of this package in an SPA context.  I'm specifically looking for some clarification on the requirements listed out in #305.  In that PR @darrynten mentions that a number of headers need to be appended to requests made by the SPA:
X-Shop-Domain - the domain of the target store
X-Shop-Signature - the signature (a.k.a. hmac)
X-Shop-Time - the timestamp used in the creation of the signature
X-Shop-Code - the code used in the creation of the signature
Like @StefanNeuser, I'm building my apps with InertiaJS.  If you're not familiar with that library then the only important thing to note is that my app largely functions like an SPA in that the majority of calls from the front end are ajax calls made using axios under the hood.
I know how to modify the instance of axios that InertiaJS uses to make calls to add an interceptor to add these additional headers to requests BUT I'm really not clear where some of the values are meant to come from.  My javascript is aware of the shop domain so I could add that header and obviously time would be easy enough but how would my JS send the code and signature headers back.
Sorry if I'm missing something super obvious (quite possible!) - any clarification on this would be really helpful.  Thanks!

Sorry, I have not dabbled in SPA for the embedded apps yet. I'm not sure I could provide the answers you need. I can do some digging though, but I would say @darrynten would have a good answer.

Hey @ohmybrew - thanks for the reply and no problem.
@darrynten - would you be able to shed some light on the above by any chance?  Any insight greatly appreciated!

@fourstacks I've started building a SPA, so will check to see what I did to handle the requests.
It would still be good to hear @darrynten thoughts on it as he probably has a better implementation anyway!

@Kyon147 that would be great - would be very interested to hear how you've integrated with this package!

@ohmybrew - is there a resolution to this question?  Appreciate you probably want to close inactive issues to keep things tidy but I'd still really appreciate any tips or guidance on how those headers are supposed to be used in an SPA context.  The fact they are in the package at all suggests that someone knows!

Hey @fourstacks
Sorry for the massive delay to replying, I've only just picked up the SPA project I am working on again and wanted to give some bits a test before replying.
What issue are you facing? With the package, I am able to make calls using Axios without any additional issues. I'm using VueJS but I have a method which is tied to a button that fires this code.
Vue component file, in the methods
 axios.post('add', this.data)
                    .then(res => {
                        console.log(res);
                    }).catch(err => {
                        console.log(err);
                })

web.php
Route::post('add', 'Controller@add_data')->name('add data test');
This is sending and receiving data back without any issue or additional headers, as long as the csrf token is in your header so requests can be made via ajax you should be okay.
<meta name="csrf-token" content="{{ csrf_token() }}">

I haven't had to change the headers or treat my workflow differently with my SPA (React using Polaris components). All of my Shopify requests are happening through this package via PHP but through their respective resource controllers which extend this package's auth middleware.

Hi @Kyon147 and @ptcampbell
Thanks for the replies - much appreciated.  I'm not really facing an issue per se, but I'm curious as to why those specific headers are checked in the package.  If you are embedded your application then you have to be careful not to bleed session info in situations where a store owner is logged into two different stores in two browsers.  Those headers are designed to ensure that the shop domain is always checked to make sure that things are scoped appropriately but I was curious as to how the 'code' header would be set.
To be honest I think we can probably wrap this thread up as I think we're all taking slightly different SPA approaches from the sound of things.  Also we've decided to move away from the embedded approach since I first wrote this post so a lot of these issues are not really relevant in my situation right now so I'll probably not have anything useful to add for now.
Thanks again for the responses!

@fourstacks do you mind if I ask you why you are moving away from embedded? I am thinking of doing the same thing.

I thought about moving away from embedded but I think the user experience is better when it's in the Shopify admin.
Some apps work not embedded when they need a more bespoke dashboard like Privy or Loyalty Lion.

Hey @ptcampbell, so we got a reasonable way with embedded but ultimately the app we're building just has a bit too much going on to cram into the limited real estate that you get when you go embedded.  There's not really a great solution for subnavigation in Polaris yet for example.  We've also found the dev experience with embedded apps to be a bit clunky and hard to debug.
We'll be watching embedded closely over the next year or so and would like to move over to it again if some of those issues are sorted but for now we're liking the increased canvas that we have to play with having chosen a non-embedded Polaris app instead

@fourstacks thanks for sharing. I think I'm in the same boat. However as I looked more into it I discovered that the embedded ResourcePicker is such a high value item for me and the trade-off might just be six of one, half a dozen of the other. But the debugging is a huge pain too.


Replace promises with async/await

Async/await syntax is fully supported by Node 7.6, as well as TypeScript.
There are several reasons to use this syntax:
<EMAIL_ADDRESS>The work required here would be:

Remove the Q and Bluebird libraries
Change functions returning Promise<T> to instead use the async keyword
Use the await keyword instead of chaining promises

Q and Bluebird are now removed, we can tackle the remaining Promise functions to async/await incrementally.


Got InvalidResourceType for any netappfiles command

az netappfiles account list -g rg-netapp
Command group 'netappfiles' is in preview. It may be changed/removed in a future release.
Azure Error: InvalidResourceType
Message: The resource type could not be found in the namespace 'Microsoft.NetApp' for api version '2019-06-01'.
az -v
azure-cli                         2.0.74
command-modules-nspkg              2.0.3
core                              2.0.74
nspkg                              3.0.4
telemetry                          1.0.3

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

ID: 93030c78-4181-ea69-4c2a-ca245723d7f1
Version Independent ID: b354ae2d-55b5-ff37-fe26-23499d5c837b
Content: az netappfiles
Content Source: src/azure-cli/azure/cli/command_modules/netappfiles/_help.py
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

Please ignore it. I need to change subscription to the one with netapp enabled first.


incompatible filtersize comparison should always return match instead of no match

https://github.com/teragrep/blf_02/blob/5671f1f4da530a1565d462adb9ac2a51bd0ccd76/lib_mysqludf_bloom.c#L85
should be 1

https://github.com/teragrep/blf_02/blob/5671f1f4da530a1565d462adb9ac2a51bd0ccd76/lib_mysqludf_bloom.c#L76
same here

false positives are a feature of bloommatching so match should be safer than no match in case of a failure in matching.


Render Box Issue

I cant put ListView.builder() widget inside Row or Column

@MazinHashim
I think you can
import 'package:flutter/material.dart';

void main() => runApp(
      MaterialApp(
        home: Material(
          child: Column(
            children: [
              Expanded(
                child: ListView.builder(
                  itemBuilder: (context, index) => const FlutterLogo(size: 80),
                ),
              ),
            ],
          ),
        ),
      ),
    );

@MazinHashim
I think you can
import 'package:flutter/material.dart';

void main() => runApp(
      MaterialApp(
        home: Material(
          child: Column(
            children: [
              Expanded(
                child: ListView.builder(
                  itemBuilder: (context, index) => const FlutterLogo(size: 80),
                ),
              ),
            ],
          ),
        ),
      ),
    );

@MazinHashim
See above solution and check if it solves your issue. Also this issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you

@MazinHashim
See above solution and check if it solves your issue. Also this issue doesn't seem to describe a bug or a feature request. Please see https://flutter.dev/community for resources,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


Client should raise exceptions, not print error messages

Do it plz

fixed in https://github.com/reduct-storage/reduct-py/commit/d01848e7e8a5f8a0f82ac1a377ac87c910dace63


doc: updated dependencies for first time build

Signed-off-by: Catalin Hofnar<EMAIL_ADDRESS>

Looks good to me, thanks!


new X-at-point whichs allow all non-blank char

This commit changes symbol-at-point as you suuggested to support all non-blank char as keyword, including < of course.
related Issue: https://github.com/Kungsgeten/yankpad/issues/67
First time to do this on github, if I did anything inappropriate, just tell me ;)

I tried this patch and it works quite fine. Currently I'm using yasnippet snippets alongside with yankpad because yankpad does not support keywords with non-ascii chars while yasnippet supports it. Any reason for not including this @Kungsgeten?

@isamert I intended to try this patch but then forgot about it. The submitted code needs some minor tweaks in order to conform with the Emacs Lisp code standards, but otherwise there's no reason not to include it (if it works). I've recently moved to a new apartment and will be a first time father in about a month, so unfortunately my Emacs packages have a lower priority at the moment.

I see, thanks! Just wanted to give you a friendly nudge.
I'm aware that it needs minor tweaks (and some edge cases might need handling, I saw a minor issue while using it). Maybe @yuchen-lea can clear things up (and I can give some feedback), or I can also just clear things up to ease up your work. No rush tho, I patched in my local and using it. Seems like you are not going to do much with this package for a while, so there is no maintenance burden on my side.
Oh, by the way, congratulations on being father!

Seems to work fine, though I will refactor a bit and fix an issue I noticed (not related to the PR).


jobs: make space for BLM's Xeno counter resource box

The #right-side-icons are being pulled back behind the resource box as BLM is being treated as having 0 boxes on the right side.
Before:

After:

Thank you for the bug report plus fix! Super appreciated.


users : updated

image for the showcasing my website , pr for adding the website already done

Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Hi,
Please use a single PR to add your site + image, not 2.
Also please use a stable domain name that will likely keep working in 1 year, not a deploy preview or domain like https://gifted-murdock-801453.netlify.app


Fix: Issues generated in regression testing for 0.4 release.

Description
This PR fixes the frame overlay button not being visible and disabling the cookie deletion button when no cookie is selected.
Relevant Technical Choices

Testing Instructions

Additional Information:

Screenshot/Screencast

Partially addresses #339

@amovar18 The delete all cookie button is still not disabled.


[BUG] Transpiled files don't work in Safari

Bug Report Quick Checklist

[x] I am on the latest version of Snowpack & all plugins.
[x] I use package manager yarn 2
[x] I run Snowpack on OS macOS
[x] I run Snowpack on Node.js v12+ => N/A

Describe the bug
The files from snowpack dev doesn't work in Safari, triggering this error in the Safari/WebKit console (from Parser.cpp in WebKit):
SyntaxError: Unexpected identifier 'as'. Expected 'from' before exported module name.

I'm guessing it's from an export line that doesn't include a 'from', such as:
% grep "^export .* as " .node_modules/.cache/**/*.js | grep -v from
./node_modules/.cache/snowpack/build/esrever@0.2.0/esrever.js:export { esrever as __moduleExports, reverse, version };
./node_modules/.cache/snowpack/build/immer@7.0.15/immer.js:export { on as Immer, vn as applyPatches, K as castDraft, $ as castImmutable, pn as createDraft, D as current, J as enableAllPlugins, T as enableES5, C as enableMapSet, F as enablePatches, ln as finishDraft, L as immerable, t as isDraft, r as isDraftable, H as nothing, e as original, an as produce, fn as produceWithPatches, cn as setAutoFreeze, sn as setUseProxies };

Perhaps some extra transpilation needs to happen for Safari by default?  Potentially related to https://github.com/snowpackjs/snowpack/discussions/2354 as I tried to set target in tsconfig.json, but it was ignored.
Expected behavior
I was expecting Safari to work out-of-the-box.

Have you tried setting the target in the snowpack.config.js optimize section? ie:
optimize: {
  target: 'es2107'
}

Have you tried setting the target in the snowpack.config.js optimize section? ie:
optimize: {
  target: 'es2107'
}

Yeah, I just tried and getting the same error.


HVP for everyone

Core

Define gradient_and_hessian_vector_product for every second order backend (except symbolic ones), mutation-compatible or not
Better supports_mutation and support_hessian utilities, checking correct result (that way FiniteDiff over FiniteDiff fails)

Tests

Add second-order tests for more backends, especially SecondOrder
Add correctness, stability and allocation tests for gradient_and_hessian_vector_product
Better scenario display in test set

Extensions

Enzyme: Make sure to convert dx everywhere before building Duplicated(x, dx)

Enzyme is crapping all over the test logs: https://github.com/EnzymeAD/Enzyme.jl/issues/1354


[Feature] Ability to reference files from assets directory from CSS/Templates OR use relative links without rewriting

Bug Report or Feature Request (mark with an x)

[ ] bug report -> please search issues before submitting
[X] feature request

Versions.
Future version
Description of problem being solved
Now that CSS path rewriting works with component CSS there is no way to make plain old CSS relative links. This effectively means that in order to reference assets/ files (or files in other configured directories to be copied to dist/ on build) a root relative path must be used. This works well for standard deployments where the app will be deployed to a single base URL which is known at compile time as angular-cli will rewrite URLs.
Some deployments will either deploy to a base URL unknown at compile time or will deploy to multiple base URLs. For example J2EE applications will have a "context root" which is configurable by the server administrator. Applications built for such environments should use relative URLs only in order to work in this environment.
Desired functionality.
It is desirable that a CSS/LESS/SASS file be able to refer to assets from angular-cli.json's apps.assets (i.e. assets which will be copied to dist/ ) and have those be represented as relative links (which the browser will load from the base href at runtime).
Current workarounds
All of the below assume that the asset is in the assets folder and that it will be referenced by both HTML and CSS (which is what I currently do).

Include all assets in the component, even if they are repeated. Simple to do, harder to maintain.
Use paths relative to the source location (e.g. ../assets/file.png). This will cause webpack to make a second copy of the file in the root of dist/ meaning that dist/assets/file.png and dist/file.png will both exist (and browsers will need to load the same content twice).
Use root relative path. Make separate builds for different context roots with different deploy url for each target so that angular rewrites the paths. This will require N builds (one for each target) which can get tedious. It also doesn't solve the problem for unknown root at compile time.
Use a root relative path with a fake path (e.g<EMAIL_ADDRESS>and use postprocessing on the dist/ directory to replace these with relative paths. This adds an extra step to the build and defeats the objective of having angular-cli be a one stop build tool for angular.
Don't use assets in CSS. Find a way to do this in HTML instead. Obviously this isn't going to work for all cases and is somewhat onerous.

Potential solutions (somewhat in order of preference)

Webpack resolver which is aware of the angular-cli.json apps.assets and resolves resources to relative paths accordingly.
Resolver alias which, when used in a path, causes the resolver to transform it to a relative path and do resolution based on the expected dist/ structure. For example relative:assets/file.png or @relative/assets/file.png
Same as the previous, but without resolving resources (may result in broken links)
Flag to globally turn off CSS path resolution (developers will need to know what the path will be of their assets and code accordingly).

This is related to the closed issue https://github.com/angular/angular-cli/issues/4858

I'm having a hard time understanding the exact conditions where our current options are insufficient.
Can you give me a concrete example? Please include all relevant paths.

OK to boil it down to the most basic concrete example with a single image (we will probably have more than one image in this situation):
Angular app
I have an application which is basically a search engine for locations our clients will be intersted in. A picture of a map pin is used in multiple places.
I will use %pinurl% to refer to a picture of the map pin at 'assets/icons/map-pin.svg' (in the output). The image is located in src/assets/map-pin.svg

src/components/search-results/app-search-results.less - list-style-image: url('%pinurl%'); // This adds the map pin next to each search result.
src/components/location-details/app-location-details.html - As an img with src='%pinurl%') // Added by our designer as a bit of fluff to fill some white space.
src/components/location-details/app-location-details.ts - %pinurl% added as a graphic on an ESRI maps Graphics layer. // This is used to display the search result on the map.

Hosting Environment
The application is hosted in two different sites for two different sets of clients (there will be some non-trivial differences in what is presented to each).

https://www.website.com/merchants/finder
https://www.website.com/customers/finder

A JSP  in each storefront will load the web application and set the base URL to /%contextroot%/finder where %contextroot% is 'merchants' or 'customers' depending on which storefront the user is accessing.
The problem
The image is being accessed from multiple locations in the application, some of which don't support resource resolving (AFAIK URIs in HTML templates and TS files cannot be resolved, correct?) and some of which have mandatory resource resolving for relative paths.

For TS/HTML templates the only option is to specify the assets folder as an "asset" in .angular-cli.json resulting in the file being copied to dist/assets/map-pin.svg. It may then be referred to using 'assets/map-pin.svg', allowing the browser to resolve it using base href.
For CSS files the only option is to refer to an image path on the file-system, resulting in a copy of the image included in dist/map-pin.%hash%.svg and a rewritten CSS referring to "map-pin.%hash%.svg" or to use a root relative URL.

Using file system path results in two copies of the file.
Using root relative URLs is undesirable as it will make changing the context root difficult in the future (we intend to host multiple angular applications in a similar manner, all of them would need to be updated) and because it doubles (or triples, etc. if we add more context roots) build time.

Writing all of that out has made me realise that potentially this could also be resolved by allowing resources to be resolved from HTML templates and TS files. This would allow consistent referencing of resources from HTML, TS and CSS removing the file duplication.
Preferably this would go along with adding the ability to specify such paths relative to .angular-cli.jsons'  apps.root, probably using some kind of prefix. For example my src/tsconfig.json defines a "paths" entry with "@me/" mapped to "/" so I don't need to use relative paths in my import declarations, making refactoring easier. Being able to use something like this consistently across CSS, HTML and TS would be helpful for maintainability.
As such the problem can be restated as

Resource resolving is inconsistent across the three main types of source file (CSS, HTML, TS) meaning that in some cases resources must be duplicated.
Using relative paths to refer to other resources is brittle. Can we have a mechanism to resolve paths relative to the src/ folder that works for the three main types of source file (CSS, HTML and TS).

It is true that resources are resolved differently across HTML, CSS and TS. CSS fully resolves and fingerprints resources, while TS and HTML don't. The HTML bit is being tracked in https://github.com/angular/angular-cli/issues/3415.
I don't think there's any way to truly solve this problem at the moment though.
Regarding relative/absolute paths... that's also a though cookie, because it's really not about src/ but rather where the built app is deployed.
Keeping the issue open for discussion.

We have a similar problem here. We've converted all links of form /assets/file.png to ../../assets/file.png to solve this for now. I've tried --base-href . and --deploy-url ./, but this didn't help; /assets/file.png still ends up as /assets/file.png (which doesn't exist), while ../../assets/file.png ends up as ./file.xxxxx.png (as expected with fingerprinting).
Maybe the best solution to this problem would be that /assets/file.png should end up as ./assets/file.png when --deploy-url ./ is specified?
(NB: when using --deploy-url ., i.e. without a final slash, ../../assets/file.png ends up as .file.xxxxx.png, which doesn't work. Is that a bug or intended behavior?)

@ffdybuster what was the ng command that you ran? I updated my links to use ../../ but when i ran ng build --prod --no-aot --deploy-url . the images in the assets folder do not have any fingerprint. Only images referenced in CSS files have fingerprinting.

@thienedits: I ended up with ng build -prod --environment prod --base-href ./ --deploy-url
I forgot to mention that I was only looking at images in SCSS, since we don't have images in templates (yet).

@filipesilva
This is something that was fixed on webback years ago with alias paths. What makes it impossible to have alias paths relative to (compile time) app root with angular-cli, as that was the commonplace with webpack aliases?
Having to use fully relative paths (../../../../../../../assets/img/xy.svg) is a MAJOR PITA. Also paths absolute to web server root (/assets/img/xy.svg) do not work on many virtual test server environments.
Funnily enough, when using absolute paths + run time modified base url in index.html we get assets referred from less files working IFF they are from a lazy loaded module, but NOT when they are from non-lazy-loaded module. That is when app is run in "www.somehost.com/non-root-url/#/" of course.

@LyraelRayne this is out of our scope for now. Please, take a look at ngx-build-plus. I believe it may help you to achieve this. If I'm missing something, and there are a lot of folks who'd want to see this feature in the CLI, please open a new issue.


Call to undefined method Phinx\Util\Util::globAll()

Running phinx migrate gives me this:
Fatal error: Call to undefined method Phinx\Util\Util::globAll() in /home/involvme/.composer/vendor/robmorgan/phinx/src/Phinx/Console/Command/AbstractCommand.php on line 103
This is potentially because I have phinx 0.6.5 installed as a local dependency and 0.8.0 as global, so the global executable somehow autoloads the local files.

you may use ./vendor/bin/phinx inside your relative path of your project @okneloper

Thanks @daison12006013, I've solved that for my particular case, but the issue overall is still a bug, what do you think?

Yeah it is still a bug since it is a command, there should be some checking in 0.6.x to check the version for compatibility. to limit only to the same version area 0.6.*, as well for future versions.

Are you able to provide a patch with your suggested changes as PR?


Option::zip

I sure would like to have the method Option::zip
fn zip<U>(self, Option<U>) -> Option<(T, U)>

Currently I have to do a.into_iter().zip(b).next() which is just a tiny bit verbose.

a.and_then(|x| b.map(|y| (x, y))) also works, although isn't much less verbose.

The desired behavior here is to return Some iff self and other are Some?
The signature looks like it should take self by value rather than by ref.
I think this is a pretty good idea since it's so simple, but I am not sure what the use-case is.

alternatively there could be an impl<T, U> Into<Option<(T, U)>> for (Option<T>, Option<U>). But that might be a bit too much logic for the into operation.

Option::zip(a, b)

vs
(a, b).into()

I think that second approach is better, but it type of (a, b).into() is not inferrable right now.

:-1: for me. It seems like a niche use case. Also, having an operation named zip on Iterator that acts slightly different seems unfortunate.

FWIW using #243 extended with Option compatibility, it would also be possible to express this as try { (a?, b?) }.

This is now implemented, although unstable.
Closing in favor of tracking issue: https://github.com/rust-lang/rust/issues/70086

Interesting idea. That’s basically a specialization of applicative functors. Indeed, you use a tuple constructor, but you could do much more with an applicative functor. It’s a pity we don’t have applicative functor notation in Rust. :)
As a base of comparison, you do it this way in Haskell:
-- all these are identical
x = liftA2 (,) firstMaybe secondMaybe

x = (,) <"> firstMaybe <*> secondMaybe

x = do
  a <- firstMaybe
  b <- secondMaybe
  pure (a, b)

@phaazon note that we have special-cased do (I think?...) for some types, e.g.:
let x = try { (first?, second?) };

The try{} block itself is unstable (https://github.com/rust-lang/rust/issues/31436) but the question mark operator in functions is stable.


OpenMP Issue

When I tried to train the model it shows:
Creating out...
Found vocab_size = 371 (inside tokens_v1_all/meta.pkl)
Initializing a new model from scratch...
number of parameters: 85.24M
Compiling the model (takes a ~minute)...
OMP: Error #15: Initializing libomp.dylib, but found libiomp5.dylib already initialized.
OMP: Hint This means that multiple copies of the OpenMP runtime have been linked into the program. That is dangerous, since it can degrade performance or cause incorrect results. The best thing to do is to ensure that only a single OpenMP runtime is linked into the process, e.g. by avoiding static linking of the OpenMP runtime in any library. As an unsafe, unsupported, undocumented workaround you can set the environment variable KMP_DUPLICATE_LIB_OK=TRUE to allow the program to continue to execute, but that may cause crashes or silently produce incorrect results. For more information, please see http://openmp.llvm.org/
zsh: abort      python bin/train.py dataset=tokens_v1_all

Hi, this error is due to there being different, conflicting OpenMP libraries in your Python environment. This sounds like an environment-specific issue, and not directly related to the code itself.
Sometimes this happens if dependencies are installed from different sources (e.g. some from wheels, some from conda, etc.). If you haven't already done so, I would recommend creating a virtual Python environment:
python -m venv crystallm_venv
source crystallm_venv/bin/activate
poetry install

There's also this issue in the PyTorch repo, which may provide some guidance.

Hi, thanks, I set export KMP_DUPLICATE_LIB_OK=TRUE to bypass the error. Can I ask, How long does it take to train a simple model, I wanted to train a model based on 50 crystals, but it seems like it is taking forever

Training a model normally requires a GPU with sufficient memory, like an NVIDIA A10G or A100. It isn't practical to train on something like a laptop, as training will be extremely slow. If you're trying to train on a laptop, that may be why it's taking so long. Recall that these models have millions of parameters, even if the training set size is very small. You can adjust the number of parameters in the model by setting the number of layers and embedding size. However, even a model with 25 million parameters (what we call a small model) is going to be slow without a GPU. Inference (i.e. sampling from the model), on the other hand, may be practical on a laptop.

Hi, thanks for the clarification. Although I reduced the number of parameters to 0.02M, it never finishes compiling the model. It gets stuck right here, although I have waited for ~5 hours

Well,  it has complied, but I do not see anything in the output, like checkpoint and optimized model.

When I used model's checkpoints to generate crystal, it just gave me some text, I didn't see any CIF files. Can you please explain how I can get the CIF file? Also, I don't see any prompt.txt files when the training is completed.
python bin/sample.py
out_dir=out/my_model
start=FILE:out/prompt.txt
num_samples=2
top_k=10
max_new_tokens=3000
device=cuda

Hi, thanks for bringing this up. I have added more information to the README related to prompting the model and generating CIF files. I have also added a new script to help create prompt files (bin/make_prompt_file.py), and enhanced the bin/sample.py script so that generated CIF files can optionally be saved locally.
The strange-looking sequence of characters in your output was generated by the model you trained. However, I'm not sure how long you trained the model, and your model also has a relatively small number of parameters (0.01M). I therefore suspect that your model may not have sufficiently learned the data. Going forward, I will use the pre-trained small model (with 25M parameters) for my examples, which runs fine on my macbook for sampling.
First, download the model. From the root of the project:
python bin/download.py crystallm_v1_small.tar.gz

Unpack the file:
tar xvf crystallm_v1_small.tar.gz

This will results in a folder named crystallm_v1_small
Next, make a prompt file. We'll generate  a structure for NaCl with Z=2 and let the model determine the space group:
python bin/make_prompt_file.py Na2Cl2 my_prompt.txt

Finally, sample from the model:
python bin/sample.py \
out_dir=crystallm_v1_small \
start=FILE:my_prompt.txt \
num_samples=1 \
top_k=10 \
max_new_tokens=3000 \
device=cpu

This gives me the output:
Using configuration:
out_dir: crystallm_v1_small
start: FILE:my_prompt.txt
num_samples: 1
max_new_tokens: 3000
temperature: 0.8
top_k: 10
seed: 1337
device: cpu
dtype: bfloat16
compile: false
target: console

number of parameters: 25.36M
data_Na2Cl2
loop_
_atom_type_symbol
_atom_type_electronegativity
_atom_type_radius
_atom_type_ionic_radius
Na 0.9300 1.8000 1.1600
Cl 3.1600 1.0000 0.7800
_symmetry_space_group_name_H-M P4/nmm
_cell_length_a 4.9038
_cell_length_b 4.9038
_cell_length_c 3.9603
_cell_angle_alpha 90.0000
_cell_angle_beta 90.0000
_cell_angle_gamma 90.0000
_symmetry_Int_Tables_number 129
_chemical_formula_structural NaCl
_chemical_formula_sum 'Na2 Cl2'
_cell_volume 95.2964
_cell_formula_units_Z 2
loop_
_symmetry_equiv_pos_site_id
_symmetry_equiv_pos_as_xyz
1 'x, y, z'
loop_
_atom_site_type_symbol
_atom_site_label
_atom_site_symmetry_multiplicity
_atom_site_fract_x
_atom_site_fract_y
_atom_site_fract_z
_atom_site_occupancy
Na Na0 2 0.0000 0.0000 0.0000 1
Cl Cl1 2 0.0000 0.5000 0.9817 1

---------------

You can also have the generated content be saved locally to a file by including the target=file argument:
python bin/sample.py \
out_dir=crystallm_v1_small \
start=FILE:my_prompt.txt \
num_samples=1 \
top_k=10 \
max_new_tokens=3000 \
target=file \
device=cpu

A file called sample_1.cif would be created, in this case.
Note that these generated CIF files must be post-processed. Create a directory called my_raw_cifs and place any generated CIF files in there. Then:
python bin/postprocess.py my_raw_cifs my_processed_cifs

The post-processed CIF files will be in a new directory called my_processed_cifs.

Hi, thanks for the detailed explanation. I think the  sample.py script does not take target arguments; I also looked into the script. Can you please clarify?

Oh, I see, The sample.py script has been updated to accept the argument.

Is it possible to put multiple composition in the same prompt file?
python bin/make_prompt_file.py Na2Cl2 my_prompt.txt
Can I write python bin/make_prompt_file.py Na2Cl2 CdTe AlCl3 my_prompt.txt

No, a prompt .txt file is expected to contain only a single prompt. You will need to make separate prompt files for each composition. Running that command with multiple compositions should result in an error.

I do have another question. When it generates new CIF files, it only changes the lattice parameter and coordinate of each atom; are these all I'd expect? For example, if I say I want a CdTe-based crystal with a certain space group, it will give me some identical crystal files. We could have alternatively applied strain and compression to the parent CdTe to make these crystals too.  Therefore, what is the point of training these generative models? Apologies if I am not understanding the key points of generative models. Can you please explain the benefits of these models in a bigger picture?

Sorry but I don't understand what else you're expecting to see. These models generate a structure for you when all you know is the cell composition (and possibly space group). If you prompt the model with the same cell composition and space group, it will likely give you a very similar structure each time. I'm not sure what you mean by "parent CdTe", as there is no concept of a parent structure in this context. These models generate structures based on given compositions, independent of any "parent" structure.
Have a look at our manuscript, located at https://arxiv.org/abs/2307.04340, if you'd like to know more about what the point is of these generative models.
Thanks for your questions, but this discussion is no longer related to the original issue. Please create a new Issue if you have additional questions related to the code in this repo, thanks!


fix error

I have not checked it, but I believe this should fix Issue #105 .

I think this is just another journal which also has the same name as Nature.
It comes from the web of science sources https://images.webofknowledge.com/WOK48B5/help/WOS/N_abrvjt.html
Maybe you can change the order?
Overriding JabRef built in list: In the preferences, you can add another CSV, they then should take precedence over the others.
The inbuilt ones are from this repo and are usually synced before a release. But we can always trigger a manual sync for the development version.

The CSV files I add don't seem to take precedence over the built-in.

maybe you need to restart JabRef after adding this and saving the preferences

Thank you!

On second thought, I will reopen this pull request for the reasons described in Issue #105.

Yes I agree with you. I think the real nature is more important than this other obscure Magazin


[Agentless Extension] Rename \web directory to \extension and legacy \extension to \legacyExtension

closing as it was filed under the wrong location


multi domain provider support

Describe the bug
I have several domains from several domain providers.
Please help me.
I have a config file like this but it wasn't work.
{
  "provider": "Cloudflare",
  "login_token": "XXX",
  "domains": [{
      "domain_name": "XXX",
      "sub_domains": ["XXX","XXX","XXX","XXX"]
    }
  ],
  "resolver": "<IP_ADDRESS>",
  "ip_url": "https://api-ipv4.ip.sb/ip",
  "ip_type": "IPv4",
  "interval": 300,
  "socks5_proxy": ""
}
{
  "provider": "Google",
  "email": "XXX",
  "password": "XXX",
  "domains": [{
      "domain_name": "XXX",
      "sub_domains": ["XXX","XXX"]
    },
  ],
  "resolver": "<IP_ADDRESS>",
  "ip_url": "https://api-ipv4.ip.sb/ip",
  "ip_type": "IPv4",
  "interval": 300,
  "socks5_proxy": ""
}

The multi-domain provider is not supported yet.
So the temporary solution is to define 2 configuration files for each provider and run GoDNS with 2 independent instances...


ActiveAdmin and CanCanCan :create authorization

I created some rules and one of them is the following:
can :create, Ticket, { author_id: user.id }

This should allow the currently authenticated user to create tickets only when author_id is equal the user's id.
From rails console I can test that my rule works fine:
my_user.can? :create, Ticket.new(author: my_user)      # returns true
my_user.can? :create, Ticket.new(author: another_user) # returns false

Considering that I'm using ActiveAdmin, I now need to use my authorization rules with it by using its ActiveAdmin::AuthorizationAdapter.
One problem I am facing is that whenever I create a "New Ticket" I get access denied.
I doubled checked what condition is failing and it seems that AA asks for the following:
my_user.can? :create, Ticket.new # which returns false!

When I believe it should ask for the following instead:
my_user.can? :create, Ticket # which returns true!

Ticket.new has all parameters set to nil:
<Ticket author_id: nil, ..., created_at: nil, updated_at: nil>

that is why my CanCanCan hash condition is failing (author_id = nil is not valid, it should be user_id instead).
Is there a a possibile fix for this? Maybe I'm setting my CanCanCan rule in the wrong way?ActiveAdmin is also offering a CanCanCan adapter out of the box so I'm wondering how this could have been overlooked at.

I think this issue is related to ActiveAdmin more than cancancan, sorry.


How to convert the original weights?

Suppose I have a model with the original weights in .pth format.
How do I convert it so that model = BaseModel.load("path/to/dir") works properly?
Right now it says: AssertionError: We were not able to find the xturing.json file in this directory <...>.
Even if I do convert it to HF format and gain .bin parts, the xturing.json file will still be missing.

Hi @ch3rn0v,
For loading HF models, you can use below code sample:
from xturing.models.base import BaseModel

model = BaseModel.create("cerebras",weights_path="/path/to/a/directory")

At the moment, loading from HF models only works with normal models. For LoRA models, there is a small bug for loading HF models. We will fix it soon.

I've installed 0.0.9 release. It still doesn't work.
I tried model = BaseModel.load(...), model = BaseModel.load_from_local(...).
The error remains the same:
AssertionError: We were not able to find the xturing.json file in this directory.
The library is only useful to me if it allows for running fine-tuning programmatically. Bonus points if I can test it locally and then pay you to use your cloud computing for upscaling (like 65b model and hundreds of thousands of examples). However right now it just doesn't work. Please tell me what I'm doing wrong. Thank you.

Hi @ch3rn0v
You can not use load method in this case. Please try create method:
from xturing.models.base import BaseModel

model = BaseModel.create("llama",weights_path="/path/to/a/directory")

or
model = BaseModel.create("llama_lora",weights_path="/path/to/a/directory")
Let us know if it works with you.

Thank you for prompt response! Is my understanding correct that .create method downloads the raw weights from some source into weights_path dir? I'd rather not follow this route as I've already spent time downloading these.

Yes it will download and load the default model first then overwrite your model secondly.

Got it, thank you. It's not something I aim for, so I'll keep this as a backup plan.

Thank you for your try. We will keep improving xTuring and will improve load/create methods to support more type of models. Let us know if you need any further support.


Adding unique identifier & support for data.world

Creating a unique identifier for each person, see https://github.com/open-austin/harris-county-bookings/issues/13 for context
Adding support for pushing to data.world datasets. Done through streaming and files are appended so there aren't 365 per year, just one per year. Closes https://github.com/open-austin/harris-county-bookings/issues/16
Brought the requirements up-to-date

Sample of the data in data.world:

Looks great to me! I'm still familiarizing myself with this project so I'll give @colbywhite or @mateoclarke a chance to look at the pull request.


nixos/v2ray: add an option for specifying v2ray package used in the systemd service

Motivation for this change
To use a v2ray package with overridden GeoIP data in the systemd service.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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

[x] Fits CONTRIBUTING.md.

@servalcatty

@ofborg test v2ray


Docs: clarify the usage of string_view

Given the unclear situation in Boost regarding the string_view, I think the docs should say that the library uses alias boost::urls::string_view, that this alias:

Is a custom implementation of std::string_view's interface, which works in C++11.
Is convertible to/from std::string_view
Is an implementation detail of Boost.URL.
That the namespace and the name of the aliased class may change without notice.

See: https://github.com/CPPAlliance/url/issues/190#issuecomment-1148006983


Load Predefined configuration button missing

I am trying to load a saved configuration into another browser. In version 2.12.3, I do not see the button for "Load predefined configuration or rest configurations to defaults." This is using Brave Browser with the shields off and Microsoft Edge.

Hi. Thanks for reaching out.
Where did you save the configuration and with what version?
I suggest you look at the Help on Configuration Management. This will give you background on handling KIP config.
Let me know if you need more support. You can also join the KIP Discord channel where other users can assist.

I am using SignalK 2.8.3 and KIP 2.12.3. I authenticate KIP with a token in signalk and it is in the Admin scope. When saving the config, I choose the Global area and give it a name. I cannot see or retrieve this in another browser from a separate machine or even the same machine.

Both users need to be admin to r/w to global. You can also use the same user and it will load the configuration automatically


Final Project Plan (ML + Algorithm)

Individual: My plan is to incorporate an algorithm into my machine learning project. Specifically, I want to include an anomaly detection algorithm that can identify potential outliers within the cancer dataset. The potential anomalies will be displayed after the ML portion, once the probabilities of a clump being benign or malignant have been calculated.
w/ Remy: If I have trouble with the plan above, I will work with Remy on a search function on the database to uncover certain users.

Individual: I will implement a text-matching search function for an expanded college database, from which I will only show matching entries in the frontend. I want to match results using similarity (possibly using an ML?) in order to add robustness against misspellings/inaccurate results. I will show the computational complexity of this program.


Add page for react-native-gesture-handler component

On top of https://github.com/microsoft/react-native-gallery/pull/86, will rebase if needed after that is merged.

Rebased, should be good to go.