No for exceptions for guest accounts may reduce usability

First of all: having those baseline policies at no cost is brilliant and will definitely help organisations that start into M365.
The user baseline policy is also great, since it balances security with usability (require MFA based on risk, not as default).
BUT: since there are no exceptions to the policies, the user baseline policy also applies to guest accounts, so guests need to register for MFA when accessing a shared document. This reduces usability. Also, for a guest account the risk status comes from the external tenant (of the guest account), which may be a problem if say the account is (for whatever reason) at risk, but a document needs to be shared. Both situations may lead to either turning the baseline policy off (exposing tenant users to a higher risk) or making users use other means to share information, which defeats the idea of the platform... It would be really good to just have the option to except guest accounts.

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

ID: 53ea4e70-2931-8691-e9cc-13c72b514d9c
Version Independent ID: c1aa4a8f-16b2-79ac-edb1-8a87f51a3f7b
Content: Conditional Access baseline protection policies - Azure Active Directory
Content Source: articles/active-directory/conditional-access/concept-baseline-protection.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@Jens-Lorenz Thanks for your feedback! We will investigate and update as appropriate.

hey @Jens-Lorenz thanks for your feedback again, we appreciate your input into this scenario. My suggestion is to file this as product feedback against the uservoice area here : https://feedback.azure.com/forums/169401-azure-active-directory
As the github issues are meant for issues against the docs.
We'll look into this further and get back to you for an update.

Are there anymore questions in regards to this github issue? @Jens-Lorenz if not I will be closing out this github issue by end of day tomorrow.

Thanks. I added a user voice feedback. :)


provider expected: virtualbox

I don't have a lot of vagrant experience so sorry if this is a newbish issue.  When I run vagrant up, with or without specifying '--provider=virtualbox', I still get an error saying the box is for vmware_fusion.  I suspect to fix this I need a different download url.
c:\src\android-vm>  vagrant up

Bringing machine 'default' up with 'virtualbox' provider...
[default] Box 'precise64' was not found. Fetching box from specified URL for
the provider 'virtualbox'. Note that if the URL does not have
a box for this provider, you should interrupt Vagrant now and add
the box yourself. Otherwise Vagrant will attempt to download the
full box prior to discovering this error.
Downloading box from URL: http://files.vagrantup.com/precise64_vmware.box
Extracting box...ate: 11.3M/s, Estimated time remaining: --:--:--)
The box you attempted to add doesn't match the provider you specified.

Provider expected: virtualbox
Provider of box: vmware_fusion

C:\src\android-vm [master]> vagrant up --provider=virtualbox
Bringing machine 'default' up with 'virtualbox' provider...
[default] Box 'precise64' was not found. Fetching box from specified URL for
the provider 'virtualbox'. Note that if the URL does not have
a box for this provider, you should interrupt Vagrant now and add
the box yourself. Otherwise Vagrant will attempt to download the
full box prior to discovering this error.
Downloading box from URL: http://files.vagrantup.com/precise64_vmware.box
Extracting box...ate: 11.3M/s, Estimated time remaining: --:--:--)
The box you attempted to add doesn't match the provider you specified.

Provider expected: virtualbox
Provider of box: vmware_fusion

c:\src\android-vm [master] vagrant up --provider=virtualbox
Bringing machine 'default' up with 'virtualbox' provider...
[default] Box 'precise64' was not found. Fetching box from specified URL for
the provider 'virtualbox'. Note that if the URL does not have
a box for this provider, you should interrupt Vagrant now and add
the box yourself. Otherwise Vagrant will attempt to download the
full box prior to discovering this error.
Downloading box from URL: http://files.vagrantup.com/precise64_vmware.box
Extracting box...ate: 5726k/s, Estimated time remaining: --:--:--)
The box you attempted to add doesn't match the provider you specified.

Provider expected: virtualbox
Provider of box: vmware_fusion

I have provide a bug fix for this in a pull request :
https://github.com/rickfarmer/android-vm/pull/12

Merged the fix provided by elendrim.  Thanks for providing it.


armeria 0.21 does not properly handle Tomcat redirect responses (maybe others too?)

When using armeria 0.21, Tomcat redirect responses don't seem to be returned to the browser. This is with HTTP1 (not https).
The CoyoteResponse has a status of 302 and  MimeHeaderField with key Location and value of a URL. But the browser seems to treat the response returned as corrupt - it doesn't redirect and in developer tools, it shows an error "Provisional headers are shown".

I found that this happens because the redirect Location header is an absolute URL with no port, which doesn't work when the server is not on port 80


Unsound use of Vec::set_len()

This code creates a Vec<u8> and sets its length without initializing it.  This may be undefined behavior.

No, that isn't undefined behavior. One line above we allocate the correct amount. Please read the code before opening such issues.

@bkchr The problem is that the Vec contains uninitialized memory.

Why is that a problem? Everything is overridden below.

It still may technically be undefined behavior, although I am not sure.  if u8 had invalid values it would definitely be undefined behavior.


Installing Robyn

Project Robyn
I re-installed Robyn due to package changes.    Having issues installing.  Here is the error I am getting
The downloaded source packages are in
‘C:\Users\ttodor\AppData\Local\Temp\Rtmp6N9opJ\downloaded_packages’
√  checking for file 'C:\Users\ttodor\AppData\Local\Temp\Rtmp6N9opJ\remotesd005307730b\facebookexperimental-Robyn-d37ec90\R/DESCRIPTION' ...

preparing 'Robyn':
√  checking DESCRIPTION meta-information ...
checking for LF line-endings in source and make files and shell scripts
checking for empty or unneeded directories
looking to see if a 'data/datalist' file should be added
building 'Robyn_3.5.0.tar.gz'

ERROR: dependency 'lares' is not available for package 'Robyn'

removing 'C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Robyn'

Environment & Robyn version
Using Anaconda.
R version:  RBase 3.6.1

Hi @tgtod002
Can you please share your sessionInfo()? Are you using windows?
It's weird because the package is actually available in CRAN. I'd like to better understand this issue.
Do you have Rtools4 installed? Rain-checking.

Here is session info.  I am using windows.  Don't have Rtools4 installed.  Will work on updating R to >4.0.0
R version 3.6.1 (2019-07-05)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)
Matrix products: default
locale:
[1] LC_COLLATE=English_United States.1252  LC_CTYPE=English_United States.1252    LC_MONETARY=English_United States.1252
[4] LC_NUMERIC=C                           LC_TIME=English_United States.1252
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] reticulate_1.24
loaded via a namespace (and not attached):
[1] compiler_3.6.1  Matrix_1.2-17   tools_3.6.1     yaml_2.2.0      Rcpp_1.0.7      remotes_2.4.1   grid_3.6.1      jsonlite_1.7.3  png_0.1-7
[10] lattice_0.20-38
The same, showing the RNGkind, but not the locale :
print(sI, RNG = TRUE, locale = FALSE)
R version 3.6.1 (2019-07-05)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows >= 8 x64 (build 9200)
Matrix products: default
Random number generation:
RNG:     Mersenne-Twister
Normal:  Inversion
Sample:  Rejection
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] reticulate_1.24
loaded via a namespace (and not attached):
[1] compiler_3.6.1  Matrix_1.2-17   tools_3.6.1     yaml_2.2.0      Rcpp_1.0.7      remotes_2.4.1   grid_3.6.1      jsonlite_1.7.3  png_0.1-7
[10] lattice_0.20-38

toLatex(sI, locale = FALSE) # shortest; possibly desirable at end of report
\begin{itemize}\raggedright
\item R version 3.6.1 (2019-07-05), \verb|x86_64-w64-mingw32|
\item Running under: \verb|Windows >= 8 x64 (build 9200)|
\item Matrix products: default
\item Base packages: base, datasets, graphics, grDevices, methods, stats, utils
\item Other packages: reticulate~1.24
\item Loaded via a namespace (and not attached): compiler~3.6.1, grid~3.6.1, jsonlite~1.7.3, lattice~0.20-38, Matrix~1.2-17,
png~0.1-7, Rcpp~1.0.7, remotes~2.4.1, tools~3.6.1, yaml~2.2.0
\end{itemize}

Thanks. Can you also share options("repos")? To check if you're installing packages from CRAN repo.

options("repos")
$repos
CRAN
"https://cran.rstudio.com/"
attr(,"RStudio")
[1] TRUE

Ok, that's how it should be. Checking this issue before upgrading to R4.
What's the output when you run install.packages("lares", repos = "http://cran.us.r-project.org")?

install.packages("lares", repos = "http://cran.us.r-project.org")
also installing the dependency ‘openxlsx’
There is a binary version available but the source version is later:
binary source needs_compilation
openxlsx  4.2.3  4.2.5              TRUE
installing the source packages ‘openxlsx’, ‘lares’
trying URL 'http://cran.us.r-project.org/src/contrib/openxlsx_4.2.5.tar.gz'
Content type 'application/x-gzip' length 1340903 bytes (1.3 MB)
downloaded 1.3 MB
trying URL 'http://cran.us.r-project.org/src/contrib/lares_5.0.5.tar.gz'
Content type 'application/x-gzip' length 1630039 bytes (1.6 MB)
downloaded 1.6 MB

installing source package 'openxlsx' ...
** package 'openxlsx' successfully unpacked and MD5 sums checked
** using staged installation
** libs
g++  -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/include" -DNDEBUG  -I"C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Rcpp/include"   -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/../../Library/mingw-w64/include"     -O2 -Wall  -march=x86-64 -mtune=generic -c RcppExports.cpp -o RcppExports.o
g++  -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/include" -DNDEBUG  -I"C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Rcpp/include"   -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/../../Library/mingw-w64/include"     -O2 -Wall  -march=x86-64 -mtune=generic -c helper_functions.cpp -o helper_functions.o
g++  -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/include" -DNDEBUG  -I"C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Rcpp/include"   -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/../../Library/mingw-w64/include"     -O2 -Wall  -march=x86-64 -mtune=generic -c load_workbook.cpp -o load_workbook.o
load_workbook.cpp: In function 'Rcpp::CharacterVector getChildlessNode(std::string, std::string)':
load_workbook.cpp:925:5: warning: 'auto' changes meaning in C++11; please remove it [-Wc++0x-compat]
auto itr = 0;
^
load_workbook.cpp:925:10: error: 'itr' does not name a type
auto itr = 0;
^
load_workbook.cpp:934:11: error: 'itr' was not declared in this scope
if (itr == 0) begPos = begPos + begTag.length();
^
load_workbook.cpp:945:9: error: 'itr' was not declared in this scope
++itr;
^
make: *** [load_workbook.o] Error 1
ERROR: compilation failed for package 'openxlsx'
removing 'C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/openxlsx'
Warning in install.packages :
installation of package ‘openxlsx’ had non-zero exit status
ERROR: dependency 'openxlsx' is not available for package 'lares'
removing 'C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/lares'
Warning in install.packages :
installation of package ‘lares’ had non-zero exit status

The downloaded source packages are in
‘C:\Users\ttodor\AppData\Local\Temp\RtmpoJ46B8\downloaded_packages’

Ok, it seems openxlsx has an issue there. Can you repeat this process installing openxlsx and share the output?

install.packages("openxlsx", repos = "http://cran.us.r-project.org")
There is a binary version available but the source version is later:
binary source needs_compilation
openxlsx  4.2.3  4.2.5              TRUE
installing the source package ‘openxlsx’
trying URL 'http://cran.us.r-project.org/src/contrib/openxlsx_4.2.5.tar.gz'
Content type 'application/x-gzip' length 1340903 bytes (1.3 MB)
downloaded 1.3 MB

installing source package 'openxlsx' ...
** package 'openxlsx' successfully unpacked and MD5 sums checked
** using staged installation
** libs
g++  -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/include" -DNDEBUG  -I"C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Rcpp/include"   -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/../../Library/mingw-w64/include"     -O2 -Wall  -march=x86-64 -mtune=generic -c RcppExports.cpp -o RcppExports.o
g++  -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/include" -DNDEBUG  -I"C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Rcpp/include"   -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/../../Library/mingw-w64/include"     -O2 -Wall  -march=x86-64 -mtune=generic -c helper_functions.cpp -o helper_functions.o
g++  -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/include" -DNDEBUG  -I"C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/Rcpp/include"   -I"C:/Users/ttodor/ANACON~3/envs/rstudio/lib/R/../../Library/mingw-w64/include"     -O2 -Wall  -march=x86-64 -mtune=generic -c load_workbook.cpp -o load_workbook.o
load_workbook.cpp: In function 'Rcpp::CharacterVector getChildlessNode(std::string, std::string)':
load_workbook.cpp:925:5: warning: 'auto' changes meaning in C++11; please remove it [-Wc++0x-compat]
auto itr = 0;
^
load_workbook.cpp:925:10: error: 'itr' does not name a type
auto itr = 0;
^
load_workbook.cpp:934:11: error: 'itr' was not declared in this scope
if (itr == 0) begPos = begPos + begTag.length();
^
load_workbook.cpp:945:9: error: 'itr' was not declared in this scope
++itr;
^
make: *** [load_workbook.o] Error 1
ERROR: compilation failed for package 'openxlsx'
removing 'C:/Users/ttodor/Anaconda31/envs/rstudio/lib/R/library/openxlsx'
Warning in install.packages :
installation of package ‘openxlsx’ had non-zero exit status

The downloaded source packages are in
‘C:\Users\ttodor\AppData\Local\Temp\RtmpoJ46B8\downloaded_packages’

Ok, I suggest you google this and try different alternatives. I'm 99% sure that if you upgrade your R version it'll get fixed; but for the sake of understanding how to fix this for older R versions, can you please try this or this or any other solution you find out and let us know if/what worked? Thanks :)

So, should I install Rtools4?

Worked.  Upgraded R to 4.1.2.   New version of Robyn installed.

That's great. Did you install Rtools or no need? Only upgraded R? Thanks for letting us know

I just upgraded R.
It looks like since I have R for Windows, I don't need RTools.
See below from RTools download site:
By default, R for Windows installs the precompiled “binary packages” from CRAN, for which you do not need Rtools._


Don't call setDimensions when widget has been disposed and BuildContext is null

When resuming the app from an external link on iOS only, didChangeMetrics is called which registers a postFrameCallback. Between the didChangeMetrics call and the frame completing the state is disposed causing an error to be thrown about the buildcontext being null. This allows setDimensions and setState to only be called if there is still a valid context meaning the widget is still in the tree. This fixes issue #29

Thank you for your help.


Copy FIDO2 V2 webapp.

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

Refers https://github.com/wso2/identity-apps/pull/218


⚠ Bofrost.es has degraded performance

In 8544613, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1520 ms

Resolved: Bofrost.es performance has improved in 4fca0e3 after 6 minutes.


Handle existing scheme in host

This PR aims to fix https://github.com/weaviate/typescript-client/issues/87 and adds a simple check before creating the baseUrl
const version = '/v1';
const baseUri = config.host.startsWith(`${config.scheme}://`)
    ? `${config.host}${version}`
    : `${config.scheme}://${config.host}${version}`;

This allows users to use a host URL with a scheme if it matches the specified scheme.
The PR also adds two unit tests to verify that it constructs the correct URLs.

I agree,
As far as I can see, the scheme parameter is only being used to build the baseURL for both the HTTPClient and GQLClient. As you suggested, If the scheme is already present in the host, we can make specifying the scheme optional.
I would also move the whole scheme/host processing upwards, I'll look into it!

Made the scheme parameter optional and removed throw error when it doesn't exist
Moved schema and host checks to Connection, adding following unit tests:

Host contains scheme and it's the same as specified (Use host only)
Host contains scheme but it's not the same as specified (throw error)
Host contains scheme and scheme was not specified (Use host only)
Host does not contain scheme and no scheme was specified (throw error)
Host does not contain scheme but a scheme was specified (default)

Both the HttpClient and GQLClient are now using params.host parameter + adding their prefixes


Integrate CycloneDX SBOM API into build.sh

Signed-off-by: Andrew Leonard<EMAIL_ADDRESS>

Jenkins built SBOM:
16:16:52  CycloneDX SBOM:
16:16:52  {
16:16:52    "bomFormat" : "CycloneDX",
16:16:52    "specVersion" : "1.3",
16:16:52    "version" : 1,
16:16:52    "metadata" : {
16:16:52      "timestamp" : "2022-03-08T16:16:46Z",
16:16:52      "component" : {
16:16:52        "name" : "Temurin",
16:16:52        "type" : "application"
16:16:52      },
16:16:52      "manufacture" : {
16:16:52        "name" : "Eclipse Foundation",
16:16:52        "url" : [
16:16:52          "https://www.eclipse.org/"
16:16:52        ]
16:16:52      },
16:16:52      "properties" : [
16:16:52        {
16:16:52          "name" : "OS_KERNEL",
16:16:52          "value" : "Linux"
16:16:52        },
16:16:52        {
16:16:52          "name" : "OS_ARCHITECTURE",
16:16:52          "value" : "X86_64"
16:16:52        },
16:16:52        {
16:16:52          "name" : "VARIANT",
16:16:52          "value" : "Temurin"
16:16:52        }
16:16:52      ]
16:16:52    },
16:16:52    "components" : [
16:16:52      {
16:16:52        "author" : "Vendor: Adoptium",
16:16:52        "group" : "Eclipse Temurin",
16:16:52        "name" : "Temurin",
16:16:52        "version" : "19+9",
16:16:52        "type" : "application"
16:16:52      },
16:16:52      {
16:16:52        "name" : "JDK",
16:16:52        "description" : "Temurin JDK Component",
16:16:52        "properties" : [
16:16:52          {
16:16:52            "name" : "scmRef",
16:16:52            "value" : "jdk-19+1_adopt-2871-g4092ff6cb37"
16:16:52          },
16:16:52          {
16:16:52            "name" : "buildRef",
16:16:52            "value" : "https://github.com/andrew-m-leonard/openjdk-build/commit/2ac0106"
16:16:52          },
16:16:52          {
16:16:52            "name" : "full_version_output",
16:16:52            "value" : "openjdk version \"19\" 2022-09-20 OpenJDK Runtime Environment Temurin-19+9 (build 19+9) OpenJDK 64-Bit Server VM Temurin-19+9 (build 19+9, mixed mode, sharing)"
16:16:52          },
16:16:52          {
16:16:52            "name" : "makejdk_any_platform_args",
16:16:52            "value" : "--clean-git-repo --jdk-boot-dir /home/jenkins/workspace/build-scripts/jobs/jdk/jdk-linux-x64-temurin/jdk-19 --configure-args --disable-warnings-as-errors --with-source-date=updated --with-build-user=adoptium --enable-ccache --disable-ccache --with-source-date=version --with-version-build=9 --with-vendor-version-string=Temurin-19+9 --without-version-pre --without-version-opt --target-file-name OpenJDK-jdk_x64_linux_hotspot_2022-03-08-16-09.tar.gz --create-sbom -b repro -r https://github.com/andrew-m-leonard/jdk-1 --skip-freetype --use-jep319-certs --create-debug-image --build-variant temurin jdk"
16:16:52          },
16:16:52          {
16:16:52            "name" : "make_command_args",
16:16:52            "value" : "make product-images test-image static-libs-image"
16:16:52          }
16:16:52        ]
16:16:52      },
16:16:52      {
16:16:52        "name" : "ALSA",
16:16:52        "description" : "dependency_version_alsa",
16:16:52        "properties" : [
16:16:52          {
16:16:52            "name" : "url",
16:16:52            "value" : "https://ftp.osuosl.org/pub/blfs/conglomeration/alsa-lib/alsa-lib-1.1.6.tar.bz2"
16:16:52          }
16:16:52        ]
16:16:52      }
16:16:52    ]
16:16:52  }

LGTM, although I'm wondering if we should look at splitting out the SBOM functions into a separate script under common that's sourced to keep it all in one tidy place at some point in the future?

This will be the plan yes, as it's going to get a lot more complex I suspect with more detailed info.


Support for newer version

This adds support for a new version of SIF-files that have XML in the end.
However I removed the spectrum support, since my files didn't have that and I couldn't test it. Perhaps you can see if you can merge it and test it with spectrum support on your files.
Otherwise, just close this :)
/Yzmo

This adds support for a new version of SIF-files that have XML in the end.
However I removed the spectrum support, since my files didn't have that and I couldn't test it. Perhaps you can see if you can merge it and test it with spectrum support on your files.
Otherwise, just close this :)
/Yzmo

Hi mryzmo,
Sorry I no longer have access to the sif files. I can either transfer the project ownership to you or direct to you repo in the readme file. Please let me know which one works better for you.
Best,


ninja jobs explode when nesting.

The default on my machine is 6 jobs. I have nesting going on, and don't have control over all the nested scripts. I was looking for a environment variable to change the default to 1 but all those issues have been closed. Just 2 nests thats 66, I have 3 nests that's 66*6 processes doing high intensity work. This makes my system unusable and slows down overall the entire build.
One idea is to use file locking or inteprocess communication to sync total number of ninja jobs and have a minimum of 1 job per nest to prevent deadlocking.
Just a simple environment variable works too.
Thank you.

This would be addressed by https://github.com/ninja-build/ninja/issues/1139

Yeah it looks like it would. That issue has been open for a while. if doing the jobserver approach will take much time I think it would be best to introduce a environment variable for the short term.


MILESTONE 1: Create animals table

In this Milestone, I:

Created a database named vet_clinic.
Created a table named animals
Inserted animals data into the table
Wrote queries for the various quests

Below are the result for the queries

Thanks for the approval 👍


Repo sync for protected CLA branch

The pull request is created from master637528318854650048 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Angular Project documentation usin CompoDoc

Todos:

Install compodoc
Add scripts to execute compodoc at the package.json
Add a link to the compodoc index.html
Test it

Ref 1 - Theory
Ref 2 Practice

This issue is done !! just run  following command npm run compodoc


Paper and code

Sparse MultiHead Attention (https://arxiv.org/abs/1904.10509) - it is in deepspeedsparseselfattention.py or attention.py

I think both work. They just have a different API. The attention.py file has a torch-like interface, but the deepspeedsparseselfattention.py was contributed by Microsoft for compatibility with the DeepSpeed software. I suspect the latter will be deprecated when it gets merged in the deepspeed repo. Hope this answers your question!

ty!


how to get the right reference_ligand info

I am trying to run it in colab notebook, using the new protein pdb file that was input. how do I enter the correct ligand information in the reference_ligand column? The ligand x information from the pdb file, when entered, does s not recognize it?

The format should be <chain_id>:<residue_idx> (e.g. A:330), which specifies the chain and residue number the ligand can be found at in the PDB file.
Also, the 'reference_ligand' text box is an interactive cell. You don't need to run it again after changing its value (in case it caused any issues).

thanks for your explanation.


Only show first layer of my pytorch .pth model

I want to check my full model include all the sub modules insider my model.
But neutron only shows the first layer of my model, it is meaning less for me because it does not provide the informaition I need.
Here is my model.
critic.zip

Duplicate of #338


transit gateway, transit gateway attachments (including vpn attachments) don't show on diagram

Is it expected that transit gateways and VPNs terminated to transit gateways do not show in the network diagram?

The network visualization functionality of this project is no longer being maintained. As such I'm closing this issue.


Issue with k8s.io/docs/setup/release/building-from-source/

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

ok


Handled few negative test scenarios in wdmp-c and added unit tests

Current coverage is 98.67% (diff: 97.77%)

Merging #6 into master will decrease coverage by 0.40%

@@             master         #6   diff @@
==========================================
  Files             1          1
  Lines           216        226    +10
  Methods           0          0
  Messages          0          0
  Branches         36         41     +5
==========================================
+ Hits            214        223     +9
- Misses            2          3     +1
  Partials          0          0

Powered by Codecov. Last update 8757626...11b1d15


set soak test timeout

Current run time-outs while the must-gather runs
https://prow.ci.openshift.org/view/gs/test-platform-results/logs/periodic-ci-openshift-knative-serverless-operator-main-417-test-soak-aws-417-c/1860922812198293504
Setting a timeout: 4h0m0s just like the mesh tests  do, will set the overall timeout to test+1h=5h, which should be enough, hopefully.

/lgtm


Feature/xinnanli/preferred email

Critical Changes
Changes
Replaced picklist value labels for Preferred Email and Preferred Phone with custom labels to facilitate language translations.
Issues Closed
Fixes #400

@lianeyee
Could you review the release note once you get time? Thank you in advance! :)

Thanks, @xinnanlisf! I edited slightly to capitalize the field names and mimic the wording from the other custom label PR you worked on.

@xinnanlisf the changes are looking great. Should the release notes be updated to match the recent changes?
FYI @lianeyee ?

@xinnanlisf What's an appropriate way to describe the final implementation? Something like this? "Enhanced picklist value labels for Preferred Email and Preferred Phone to use labels from Translation Workbench if available."

@lianeyee
You might mention using dynamic email labels.
@davidjray What do you think?

How should we describe "dynamic"? What are the potential places that we're getting the labels from?

I think the message should be something like 'Modified "Preferred Email" validation to allow translation of HEDA email field labels and Preferred Email field picklist options via the translation workbench.' - Since that is basically the goal.

A-ha. Thanks for clarifying that more flexible validation is the essence of the change, @davidjray. I suggest "Enhanced validation of Preferred Email picklist values to support translated email field labels from Translation Workbench."

@davidjray
I think it is working now. Basically, I let it check if translated picklist value exist. If exists, then replace the pick list value api names with labels. If not, then it will still go through the old process.
For example, there is a translation for the university: Universidad. The new method catches that the Universidad exists thus will use university email.
Let me know if this makes sense for you.

@chrismcmahon and @sreesureka here's what I did to test this:
Testing dynamic Email field references:

Check the "Disable Preferred Email enforcement:" on the Accounts and Contacts tab of the HEDA Settings
Edit a contact to add a value to a HEDA provided Email field (Work, University or Alternate), then add a separate value to the Standard Email field. Make sure to leave "Preferred Email" blank in this step.
Then I activated "End-User Languages" in the Language Settings and then changed my Users default language to "Español".
Back in the Accounts and Contacts tab of the HEDA Settings  I unchecked "Disable Preferred Email enforcement:" and saved the settings. Then I ran the Preferred Email Data Cleanup by clicking the "Run Clean Up" button.
Back on the Contact edited in step 2 I was able to verify the translated label of the Email field was added to the Preferred Email picklist.

Testing the ability to translate Email field Labels and Picklist values

Using the translation workbench, I overrode the label of a HEDA email field such as "Work" email.
Then translated the label of the matching Preferred Email picklist value to match the value added in the previous step.
Open a contact and add a value to the email field whose label was translated, and select the corresponding value in Preferred Email that was also just added via translation.

Hopefully, that helps speed the testing along!


Don't use alpha versions in version specifier

It could potentially break package management tools such as pdm.

Since the current alpha contains tomllib now, I guess the specifier can be omitted.
Nonetheless, this strictly seems to be a bug in pdm ( https://github.com/pdm-project/pdm/blob/871e29828a645d5a92141afe9774bb1312c1c6fe/pdm/models/versions.py#L31 ); e.g. pip does not have any issues with the specifier.

But black uses https://github.com/psf/black/pull/2987


retina tiles jumbled on maps api export

When I set my custom base map to CartoDB's retina tiles and then export an image, the tiles are jumbled, or every other tile is shown, or the tiles are overlaying each other.

(updated.. image disappeared for some reason)

HTTP layers don't support retina yet: https://github.com/CartoDB/Windshaft/issues/243
Probably time to prioritise that @saleiva?

@javisantana 's call here... I'd say no except it helps  with medusa v3

they are independent things.

retina can wait for the moment, we don't support in any part of our stack

so?

As per @javisantana I'm gonna drop this one from the milestone and create a new issue in windshaft.
We will work on this when we decide to invest time in full retina support.

@rochoa has this issue been resolved yet? My exported images are still being jumbled:

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


fix(credential-eip712): update plugin schema

Build is failing: https://github.com/uport-project/veramo/runs/6725302730?check_suite_focus=true#step:15:21


Override old default values in update_defaults

[x] Closes #10155
[x] Tests added / passed
[x] Passes pre-commit run --all-files

Thanks @jrbourbeau. I'll see if I can port this to the donfig package. It does look like donfig has missed a couple updates/changes to the dask version so I'll need to back track and find the previous PRs.


Different color marker icon

What is represented by this different color?

Please use search:
https://github.com/tananaev/traccar/issues/2100#issuecomment-231784949


Share nix store with devenv devcontainer?

I was trying out the the .devcontainer.json interop, and am wondering if sharing the host nix store is possible. It would save some bandwidth/disk space (and, most important, time!), and using the devenv devcontainer as base for all my projects would be cool.

Does devcontainer support mounting? If so, yes :)

Yep, devcontainers should support mounting.
I've made some progress wrt mounting the store:
$ docker run  --entrypoint bash --mount='source=/nix,target=/nix,readonly,type=bind' -e 'NIX_REMOTE=daemon' --rm -ti ghcr.io/cachix/devenv:latest

Seems to work properly, but not clear to me if I'm risking breaking things. Didn't yet plug that into the .devcontainer.json, will be trying that once I have time to tinker again.


Fixed issue #377.

Closing as a fix has been put into develop to ensure values are always strings 👍


Allow option for developmental releases in PEP 440

PEP 440 describes a .devX suffix that describes developmental releases leading up to a full release.
In my projects, I'd like to use this nomenclature, since pip will choose not to install .dev releases unless the --pre option is included. We push development versions on CI builds, which vcversioner currently tags with a .postX suffix. These version do get installed without the --pre tag which can be undesirable for development builds. PEP 440 even states that "the use of post-releases to publish maintenance releases containing actual bug fixes is strongly discouraged. In general, it is better to use a longer release number and increment the final component for each maintenance release."
I have a fork of vcversioner that contains an option use_dev_not_post, and am looking for feedback on whether such a change would be considered for inclusion into the upstream project. I'm not attached to the option name either, if there is a better one available.

I actually used to have it to .dev instead of .post, but the problem is
that 1.0.dev1 < 1.0 < 1.0.post1. You want commits made after a
release to order after the release, not before the release.

Indeed. The way I have things setup right now is that if you have something tagged "1.0" and commit once after that, vcversioner returns "1.1.dev1". It even works for situations like alpha builds where one commit after "1.0.2a1" becomes "1.0.2a2.dev1".
This really helped with our CI builds stomping all over actual "releases", by taking advantage of that --pre tag.
The biggest drawback I see is that it enforces a PEP-440-ish version tag to begin with, if the option is enabled.


httpx: 1.0.8 -> 1.0.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/projectdiscovery/httpx/releases.
meta.description for httpx is: "Fast and multi-purpose HTTP toolkit"
meta.homepage for httpx is: "https://github.com/projectdiscovery/httpx"
meta.changelog for httpx is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/mbilnmbfpy7cnzka6mkc43kriflxhmb9-httpx-1.0.9/bin/httpx passed the binary check.

1 of 1 passed binary check by having a zero exit code.

1 of 1 passed binary check by having the new version present in output.

found 1.0.9 with grep in /nix/store/mbilnmbfpy7cnzka6mkc43kriflxhmb9-httpx-1.0.9

found 1.0.9 in filename of file in /nix/store/mbilnmbfpy7cnzka6mkc43kriflxhmb9-httpx-1.0.9

directory tree listing: https://gist.github.com/194357959d8e0b43fe5a010d86d4a039

du listing: https://gist.github.com/67133883dfd1db44676106978b56459e

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

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
httpx

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

httpx

Maintainer pings
cc @fabaff @kalbasit for testing.

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

1 package built:

httpx


Using $patch within a Pinia plugin does not show event in devtools timeline

Reproduction
https://github.com/Matt-Deacalion/pinia-devtools-bug
Steps to reproduce the behavior

git clone git@github.com:Matt-Deacalion/pinia-devtools-bug.git
cd pinia-devtools-bug
npm install && npm run dev
go to: http://localhost:3000/
open vue devtools, see pinia timeline

Expected behavior
Using $patch within a Pinia plugin would show up in Vue devtools within the Pinia timeline.
Actual behavior
No $patch event is shown.
Additional information
Vue devtools: 6.0.0-beta.21
Firefox: 96
Chromium: 97.0.4692.71

This is intentional: subscriptions are activated after plugins.
Maybe we could add a short note about this at https://pinia.vuejs.org/core-concepts/plugins.html#adding-new-state


unstyle img could not be delete

http://codepen.io/Kiwka/pen/YNYgWa
press key delete key could not delete the img, only text deleted, if content create by convertFromHTML.
Is there a way to create atomic block from html

I think the question about using the delete key may be the same question in #915


Wallet api types package

Purpose
Provide types for apps interacting with the Concordium wallet, without having to include code that is already injected into the app by the wallet.
The package added is meant to be published.
Changes

Add package with types for the wallet API
Use types from added package in the other wallet packages where applicable.
Cleaned up references, so only packages meant to be published are workspace dependencies, i.e. linked into the root node modules folder.

Checklist

[x] My code follows the style of this project.
[x] The code compiles without warnings.
[x] I have performed a self-review of the changes.
[x] I have documented my code, in particular the intent of the
hard-to-understand areas.

Should we add the types package as a dependency to the piggybank example?

Yes, exactly my thoughts 🙂


Firestore Export to BigQuery add map as JSON string to schema

This feature request is for extension: Export Collections to BigQuery
What feature would you like to see?
In our document, we have a map which has fields whose values are ids, so we can't add these map fields to the schema. Therefore we'd like to add the map as a JSON string to the schema. I've tried specifying the type as a string but the values come through to BigQuery as null. I've also tried specifying the type as array but I get a JSON error. I tried leaving the "name" field empty to see if it would pick up the name of the field but I get a JSON error there too.
Here's a screenshot of the document:

Any help on how to get this issue fixed would be great, thanks!

@jackforesightmobile thanks for raising this issue. Would you be able to provide a bit more detail on what you're expecting to see added as a feature to the extension, I'm a bit confused to what is being asked here 🙈 sorry.

Hey @Salakar I guess I'm not sure whether it's something I'm doing wrong, or whether it's a new feature, so I wasn't sure whether to add a feature request or a bug. I'll try to clarify some more:
So the default way the whole document comes into BigQuery is as a JSON string. I'm not sure if there's something I'm doing wrong, but I want that questionStats map in my document to come through to BigQuery as a JSON string, while the rest of the fields come through as number, string etc. It's not possible to map the fields inside that questionStats map because the keys of the maps inside questionStats are never the same. Therefore I wanted that whole questionStats map to come through as a JSON string under the field "questionStats" in BigQuery. I guess in the schema there could be a type "json" or something that would output everything in that field as a JSON string.
What I'm not sure about is whether there's already a way to do this that I'm missing, or whether it's a new feature that needs to be added?

Hey @jackforesightmobile, it is not possible to control property types inserted from your Firestore document using the big query extension. The document is stringified and inserted into the data column which expects a string.
Have you looked into creating your own views using the gen-schema-view script?

Hey @russellwheatley so yeah I'm using that gen-schema-view script. For the document I posted this is my schema:

I'm wondering if there's a way I can make just the questionStats field stringified, while all the other fields are in the schema. I would use the "map" type in the schema, but since the keys are always changing, I can't use that type as far as I can tell

Ok, I think I understand your requirements. There is no way of making a property a json type, or stringifying a map in your case. This would have to be a feature request for the gen-schema-view script to create a new json type that may or may not be outside the scope of this extension. I'm sure the Firebase Extension will review this feature and get back to you. Thanks!

I have a similar scenario where I have maps where the key is unknown. Is there any supported way to specify something along the lines of
{       "name": "users",       "type": "map",       "fields": [         {           "name": "*",           "type": "string",         }       ]     }

Hey @alancinacio, wildcard queries are not supported I'm afraid, you have to explicitly name the key in the schema.

@russellwheatley do you think this is the same as #293?

@i14h  No, not the same. This request is to have a type that takes the value from Firestore, stringifies and inserts into BigQuery. This would be useful if you have a map where the keys are unknown before hand.
The current behaviour of a map requires you to describe the property names. Any property names not described, will be ignored by the gen-schema-view script and not inserted into BigQuery.
#293 is a request for the ability to describe the values contained within arrays taken from Firestore. (i.e. an array of numbers), and insert them as numbers (I guess non-numbers would be ignored). Although the feature request is more specifically asking for the ability to describe the shape of maps contained within the arrays which would be a lot more tricky.
The current behaviour takes every value from the array, stringifies and inserts it into BigQuery.
I think maybe @fredzqm or @IanWyszynski input would be appreciated on this one.

Initial PR added at https://github.com/firebase/extensions/pull/518

Closing now that #518 is merged

Closing now that #518 is merged


How does xESMF treat missing values?

I have a use case where I am regridding from/to grids that have missing values in them (water mask over land). I'm curious how xESMF is treating these grid cells. I am noticing a large difference in how the grids come out between the bilinear and conservative regridding methods. For example:
bilinear:

conservative:

I should note, this regridding operation is a coarse-graining operation so there are up to 64 1/8th deg source grid cells going into each 1 deg dest grid cell.

It seems that ESMF has options to specify src_mask_values and dst_mask_values but I don't see these in the xESMF source code at the moment.
The ESMF docs show a usage that looks something like:
regrid = ESMF.Regrid(srcfield, dstfield,
                     regrid_method=ESMF.RegridMethod.NEAREST_DTOS,
                     unmapped_action=ESMF.UnmappedAction.ERROR,
                     src_mask_values=mask_values)

refs: - http://www.earthsystemmodeling.org/esmf_releases/non_public/ESMF_7_0_0/esmpy_doc/html/api.html#masking

http://www.earthsystemmodeling.org/esmf_releases/non_public/ESMF_7_0_0/esmpy_doc/html/examples.html

In ESMPy, the mask is configured on the grid and "selected" in the regrid call. You'll need to merge any data variable masks with any spatial masks and set that on the grid (see this code for an example). You can then select the value used by ESMF to construct the mask in the regrid call. The *mask_values argument is used to allow changing masks based on the grid mask values...kind of specialized. As far as I know, you can use whatever integer values you want for the mask.
When masking, also pay attention to the zero_region argument as it affects the handling of missing values in the output field. That is if you are using ESMF's sparse matrix multiply. Set create_rh=False otherwise!

Xarray uses np.nan as its missing value flag. I'm thinking ESMF is not handling this as expected in some cases (conservative regridding). Perhaps we just need to make some changes in xESMF to make ESMpy aware of the mask (or use a different fill value).

@jhamman The different behaviors you get from bilinear and conservative algorithms is actually expected. The handling of nan is explained in this comment. Let me cite myself:

xESMF has no idea about land/sea mask. It just maps nan in the input data to nan in the output data. Say a destination point receives contributions from 4 source points, as long as 1 out of the 4 source points is nan, the total contributions will add up to nan (because any_number + nan = nan). If you don't want the contributions from the other 3 points to be corrupted bynan, you may set the 4th point to 0 instead of nan

When coarse-graining, conservative regridding will average over a large number of small cells (8*8=64 in your case), while bilinear regridding will only average over the nearest 4 cells. That's why you see a lot of nan in the conservative case (especially in the middle-west ), but few nan in the bilinear case. I assume that there are some nans  in the middle-west in the original data; they are "ignored" by bilinear algorithm but are caught by conservative algorithm.
I am not sure if an additional option for masking is needed. If I understand correctly, ESMPy's masking capability can be equivalently done by masking the input/output data itself, before or after the regridding computation.

@JiaweiZhuang and @bekozi - thank you both for sticking with this issue. I am starting to understand the situation a bit better.

In xarray, we can extract the "mask" from any DataArray using the notnull method.
I understand that the current API does not do this. Moreover, the Regridder only looks at the coordinates when developing the weights (E.g. xe.Regridder(ds_in, ds_out, ...)) and not the data variables.
One option that could be considered is a check during the weights generation step to see if either dataset has a mask variable. There may also be more intuitive way of determining the masks for the source/destination grids. The key point is that ESMpy allows for the specification of masks for both of these grids.

As a bottom line, I think it is incorrect to let xarray's mask nans propagate through regridding operations. Most of xarray's operators are nan skipping so it seems logical to me, that xESMF would take a similar approach.
I also want to be clear, I'm not asking anyone to do any work here. I'm mostly trying to understand the challenges in this area. If we can settle on a path forward, I wouldn't be surprised if I would contribute to this development.

Most of xarray's operators are nan skipping so it seems logical to me, that xESMF would take a similar approach.

It is true that xarray.DataArray.mean skips nan by default. But what does it mean by "skipping nans" in regridding operations? If one needs to treat nan as 0, simply convert nan to 0 before regridding.
Or do you mean some sort of "re-normalizing"? Say, if the weights for a destination point are [0.25, 0.25, nan, nan], you would like the weights to be converted to [0.5, 0.5, 0, 0]? I don't think ESMPy's masking functionality has such an advanced feature, but this should be easy to do by directly modifying the weight matrix, similar to this comment.

Or do you mean some sort of "re-normalizing"? Say, if the weights for a destination point are [0.25, 0.25, nan, nan], you would like the weights to be converted to [0.5, 0.5, 0, 0]?

Something like that. I image that is how ESMpy uses the masks internally. Perhaps @bekozi can elaborate thought.

There's quite a bit going on this discussion, so I apologize for glossing over anything...
@JiaweiZhuang is correct regarding the way masks are handled for the conservative case. Masks will drop cells if only one masked cell is mapped to a destination cell.
I don't have any experience working with nan values in ESMF opting for masks instead. If the decision is made to stick with nan over masks, we should chat with @oehmke more about the implications. Like mentioned above, the biggest concern is faulty normalization.
If configured properly, the mask on the source grid will lead to an appropriate normalization for weight factors with masked entries excluded in the final weight matrix. The destination mask will simply exclude entries in the weight matrix for its associated index. Dynamic masking (changing mask with time/level) was implemented recently in ESMF, but I think that is beyond the scope of this discussion.
Without fully understanding xarray internals, I would recommend the grid mask be constructed from the first time/level/whatever dimension of the data variable(s) unless provided by the user. Using boolean masks are a good idea as they can be quickly be converted to binary. I have seen masks on spatial coordinates too, so maybe it's best to incorporate that at the start. It may make sense to incorporate the idea of a "spatial mask" at a very high level in xarray that xESMF could reference for cases such as this. How to automatically construct the mask is tricky. Some climate simulations have the first timesteps masked entirely from cold starts. :cry:

If one needs to treat nan as 0, simply convert nan to 0 before regridding.

This will generally work, but may lead to different masked data and values when compared with a regrid + mask.

I don't think ESMPy's masking functionality has such an advanced feature, but this should be easy to do by directly modifying the weight matrix, similar to this comment.

Nothing like this that I'm aware of!

If configured properly, the mask on the source grid will lead to an appropriate normalization for weight factors with masked entries excluded in the final weight matrix. The destination mask will simply exclude entries in the weight matrix for its associated index. Dynamic masking (changing mask with time/level) was implemented recently in ESMF, but I think that is beyond the scope of this discussion.

Thanks. This is what I was looking for. I'm going to experiment a bit with adding a mask variable to the grid datasets ds_in, ds_out and passing that to ESMpy.

I don't have any experience working with nan values in ESMF opting for masks instead. If the decision is made to stick with nan over masks, we should chat with @oehmke more about the implications. Like mentioned above, the biggest concern is faulty normalization.

This would also be good to sort out. xarray's nans are just sentinel values. I'm beginning to think there is no reason they should be included in any gridding operation.

This would also be good to sort out. xarray's nans are just sentinel values.

I'm genuinely curious as well. @oehmke is on vacation for a bit. I'll check in with him on this when he gets back.

the mask on the source grid will lead to an appropriate normalization for weight factors

I did some tests with ESMPy masking, but could not get any "weight normalization" by using ESMPy mask. The effect of masking seems to be equivalent to setting input data to zero before regridding.
For full details please see this gist. I put some key results here.
Here're my test grids:

Several cells in the input data are set to nan:

Without any masking, nan in the input data will propagate to the output data:

With ESMPy masking, the output data get very low values (those near the mask):

Such an result is equivalent to setting nan to 0 before regridding. The resulting values should have been much higher (closer to surrounding cells) if there were any normalization.
I believe @jhamman wants this sort of conversion:
[0.25, 0.25, nan, nan] -> [0.5, 0.5, 0, 0]
But ESMPy masking seems to be doing:
[0.25, 0.25, nan, nan] -> [0.25, 0.25, 0, 0]
Did I understand correctly? If so, such a "masking" can be equivalent done by one line of pre-processing code (data[np.isnan(data)]=0.0) and doesn't need to be added to xESMF.

xarray's nans are just sentinel values. I'm beginning to think there is no reason they should be included in any gridding operation.

It is true that xarray.DataArray.mean skips nan by default. But implementing what you described needs some extra code outside of ESMPy. Also, what if all weights for a destination cell are all nan?

Just realized that ESMPy masking does make a difference for nearest neighbor algorithm. My above comment still hold for bilinear and conservative methods.
nearest neighbor + setting nan to 0:

nearest neighbor + masking nan in ESMPy:

The masked cell are skipped during nearest-neighbor-search, so the values in second-nearest cells are used.
Also, the problem of "all weights are nan" does not exist in nearest-neighbor method. If a destination cell is over the ocean (say only land has valid values), a cell along the coast should be found as the nearest cell. I do see the usefulness of this case.
Is it possible to avoid the addition 'mask' variable in https://github.com/JiaweiZhuang/xESMF/pull/23? Say, just set some elements in the lat and lon variables in the grid object ds_in and ds_out to nan, to indicate masks? Am I missing anything🤔?

@JiaweiZhuang Nice work on the thorough testing. In the interest of not leading us down the wrong path, I'll work through this with @oehmke when he gets back and present those results. We need to expand the masking docs as well to be more explicit!

@JiaweiZhuang - thanks for doing some extra leg work. I agree we need to hear from the ESMF crew how best to proceed. In the end, I'll be happy if we can fully describe the ESMF behavior, even if #23 isn't merged.

@JiaweiZhuang I've been investigating ESMF masking and have something to run by you. I am using a very simple case for bilinear regridding. There is a 4x4 source grid (squares) with a 3x3 destination grid (star circles) - you'll see these points with the single masked source element in the picture at the end of this post. The coordinates are regional spherical similar to your test grids.
The weight file generated with ESMF looks like:
netcdf weights {
dimensions:
	n_s = 20 ;
variables:
	double S(n_s) ;
	int col(n_s) ;
	int row(n_s) ;
data:

 S = 0.254167701295419, 0.254167701295418, 0.245832298704582,
    0.245832298704581, 0.255115371771049, 0.255115371771049,
    0.244884628228951, 0.244884628228951, 0.255447214127049,
    0.255447214127049, 0.244552785872951, 0.244552785872951,
    0.25544721412705, 0.255447214127049, 0.244552785872951, 0.24455278587295,
    0.25544721412705, 0.255447214127049, 0.244552785872951, 0.24455278587295 ;

 col = 3, 4, 7, 8, 7, 8, 11, 12, 9, 10, 13, 14, 10, 11, 14, 15, 11, 12, 15, 16 ;

 row = 3, 3, 3, 3, 6, 6, 6, 6, 7, 7, 7, 7, 8, 8, 8, 8, 9, 9, 9, 9 ;
}

You'll notice destination indices 1, 2, 4, 5 are missing in the row variable. So, ESMF drops the destination elements that are not fully mapped by four "corners" on the source. I was wrong about the normalization. Do you see similar behavior with xESMF and nan values? Dropping the partially mapped destination elements make sense from a POLA perspective...

@bekozi Thanks for the example. This agrees with my above results that masking a cell is mathematically equivalent to "setting the input data in that cell to zero" (in bilinear case).
One possible advantage of this masking approach (as opposed to zeroing-out the input data) is that it can save some computation by avoiding adding and multiplying on 0s. However, a disadvantage is that two separate regridders need to be built for unmasked and masked input data, even though the underlying grid is the unchanged.
The nearest-neighbor method is probably the only case where masking is useful (cannot be replicated by pre&post processing tricks). In your case, I expect that indices 1, 2, 4, 5 will be preserved in nearest-neighbor search. Note that this is only for "source to destination"; masking for "destination to source" should behave pretty much the same as for bilinear&conservative&patch, where some input cells are simply thrown away.

I was wrong about the normalization. Do you see similar behavior with xESMF and nan values?

The weight generation in xESMF exactly relies on ESMPy so the behavior is exactly the same.

We've been mostly talking about masking on source cells. On the other hand, masking destination cells seems pretty pointless because it is equivalent to zeroing-out destination cells after regridding.
However, destination masking should have some real effect on the "destination to source" nearest-neighbor method, where a masked destination cell should be not able to receive contribution -- according to the principle of 'nearest_d2s' that "a source point must be mapped to a destination point", a cell near the masked cell will receive the contribution instead. This can not be replicated by simple pre&post-processing (just like source masking for 'nearest_s2d')
I frankly don't know the use case for 'nearest_d2s'. It might make sense for Grid<->LocStream conversion where you may want to aggregate/spread the data along the LocStream trajectory. For Grid<->Grid conversion, this algorithm is a bit weird...

Overall, I tend to accept @jhamman's proposal of adding a mask variable. I do see the (somewhat limited) usefulness of ESMPy masking, especially for nearest-neighbor methods. However, there needs to be a very clear documentation on the effect of masking, for different regridding algorithms.

Hi, @JiaweiZhuang. If you don't implement masking, then nan values should be left in the field data to ensure the sparse matrix multiplication will create nan values appropriately in the destination field. This assumes your SMM operation containing a nan will return a nan. Setting nan to zero implies data is defined.
You will also need to be explicit that "masking" is handled on the field end during SMM by xESMF and degenerate grid coordinates are not supported. For the bilinear test case with the masked element, the weight file is portable as the source grid defines what is masked. Without the mask, the weight file would imply all field data built on that grid is defined at every location. That may be just fine (and preferable) depending on how the weight file is used, but it is something to consider when advising xESMF users on reusing weights.
Basically, it comes down to whether you want ESMF or xESMF to determine what is mapped/unmapped following a regridding operation and also if a grid or grid+field is required.

degenerate grid coordinates are not supported

My bad - I need to be more specific. Degenerate coordinates support through nans will probably work by setting ignore_degenerate=True. This flag is best reserved for well understood cases and should not be generally be used as it can hide errors you want to see like user code generating bad coordinates. Known coordinate issues could not be masked a priori. This is an edge case to be sure, but it is good to be aware of.

Thanks both of you for keeping this conversation going. I've been busy working on other things but recently ran into how CDO treats this case. For conservative area remapping, there are two environment variable options that must get passed to the SCRIP library:
ENVIRONMENT
    CDO_REMAP_NORM
        This variable is used to choose the normalization of the conservative interpolation.
        By default CDO_REMAP_NORM is set to 'fracarea'. 'fracarea' uses the sum of the
        non-masked source cell intersected areas to normalize each target cell field value.
        This results in a reasonable flux value but the flux is not locally conserved.
        The option 'destarea' uses the total target cell area to normalize each target cell
        field value. Local flux conservation is ensured, but unreasonable flux values may result.
    REMAP_AREA_MIN
        This variable is used to set the minimum destination area fraction. The default
        of this variable is 0.0.

This seems to cover my personal use case (fracarea) and provide an option for more detailed handling.

This variable is used to choose the normalization of the conservative interpolation.

This is very similar to (probably the same as) #17.
But this normalization would not help if a destination cell has no overlap with unmasked source cells (i.e. all overlapping source cells are masked). You can still get some nan, say over the ocean or large lakes.

Hi @JiaweiZhuang, you are partially correct about what an ESMF considers a degenerate cell. Instead of two points collapsing to leave a triangle, for ESMF a degenerate cell is one where enough points collapse to leave a cell either a line or a point. Regarding the earlier question about fracarea, in ESMF we don't divide if the destination fraction is 0.0, so we don't get nans. (I think that was what you were asking (?))

@jhamman Just a quick note: the "re-normalization" you need can be enabled by applying masks and setting norm_type=ESMF.NormType.FRACAREA in ESMPy (#17). Only using masks without the new normalization option just means setting nan to 0.
One use case for this is to regrid emission data that is only defined over the land. With NormType.FRACAREA and masking, cells along the coast can get a realistic magnitude, instead of getting nan (default behavior) or getting low values (only masking, averaged with empty emission over the sea):

source code
This only works for conservative algorithm. There is no "re-normalization" for bilinear algorithm, where masking literally means zeroing nans.

Thanks @JiaweiZhuang - this is exactly what I was going for.

@JiaweiZhuang - circling back here. What do you think the next steps should be to resolve this issue. Your example above is compelling to the point I would like to see some explicit support for this use case in xESMF.

@jhamman It is in the masking branch right now:
pip install --upgrade git+https://github.com/JiaweiZhuang/xESMF.git@masking

regridder = xe.Regridder(grid_in, grid_out, method = 'conservative_normed')

where grid_in contains a 'mask' variable.

@JiaweiZhuang, just to clarify mask should be a coordinate as datasets are currently not handled?

@NicWayand mask is part of input/output grid (which are dataset), not input data fields. Masking is performed when building the regridder (modify the regridding weights), not when applying to data.

Ah yes, thank you.

Thanks for this package. It would be great to have @jhamman pull requests merged into master. Please let me know if there is anything I can do for help.

For a more recent discussion on this topic, see https://github.com/pangeo-data/xESMF/issues/256

Closing in favor of the issue above.


lxd/migrate/container: Merges duplicate multi sync logic and adds comments

Signed-off-by: Thomas Parrott<EMAIL_ADDRESS>

Testsuite passed


Formio and vue js in js format not typeScript !!

Is it possible to publish this version in javaScript format !?
And also when the stable version will be released!?

@amirmf If you install the package via npm, you will notice that the files are no longer in Typescript and are just pure Javascript.  Hope this helps!

CompTrain is correct. You are not required to use Typescript if you install it from npm/yarn. Stable version is now released.


cargo-bloat: 0.6.2 -> 0.6.3

Motivation for this change
Update cargo-bloat to the latest release.
https://github.com/RazrFalcon/cargo-bloat/releases/tag/v0.6.3
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build cargo-bloat


Fixes login. Closes #6334

Fixes login. Closes #6334

Ready to merge 🚀

Merged manually. Thank you for your awesome work 👍
You Rock 🤩


chore: add comparator for diff tester

Summary

Test Plan

Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

!canary


[dv,c] Ensure strdup() is defined when compiling tcp_server.c

Given the filename, this code is technically supposed to be C. It uses strdup(), which isn't necessarily defined until C23(!). With standard libraries like glibc, it can be requested by defining a magic STDC_WANT_LIB_EXT2 preprocessor variable, so we should probably do that.
With this change,
gcc -std=c99 -c hw/dv/dpi/common/tcp_server/tcp_server.c

runs to completion.

Rupert, I assume you are trying to get verilator to work, since this would have caused a compilation failure. I also tried at HEAD, and tests pass.
//sw/device/tests:uart_smoketest_sim_verilator                           PASSED in 45.3s
This is strange, unless tcp_server.c is not compiled in all verilator tests? Are there some particular tests that trigger the compilation failure you are addressing in this PR?

This isn't related to Verilator. It's following up on issue #12395 which (reasonably) expects tcp_server.c to be C (rather than C++). The reasonable paths I can see are:

Merging this change
Changing our required C version to C23 once that gets released sometime next year.
Renaming the code to something like tcp_server.cc and making it explicitly C++.

As you might expect, I like option 1 the most :-)


Auto complete support

This should be pretty self-explanatory. I'd like to be able to type :tho, hit tab and have it auto-complete to :thonk:.

It's something im considering in 1.13 but im not sure a reliable way of doing it with 1.12, i'll look into it but dont expect it.


[TimePickerDialog] fix #5211 invalid call to onDismiss

[X] PR has tests / docs demo, and is linted.
[X] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[X] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

https://github.com/callemall/material-ui/issues/5211

@Nihaux That looks good to me. We should make sure the TimePicker API is more consistent with the other components 👍 .


CIF-1305 - Support Grouped Products

Adds css for grouped products (see screenshot).
Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[x] All unit tests pass on CircleCi.
[x] I ran all tests locally and they pass.

Is "companion" PR of https://github.com/adobe/aem-core-cif-components/pull/232

Verified


Adding onPanDrag compatibility for iOS

Does any other open PR do the same thing?

No, not any other PR does this.
What issue is this PR fixing?
Currently, onPanDrag does not work in iOS, this PR adds this behavior.
How did you test this PR?

Which platform (eg. Android/iOS) & Maps API (eg. Google/Apple) does this change affect, if any?
IOS + Google Map

Did you test this on a real device, or in a simulator?
Both

Are there any platforms you were not able to test?
No.

@jherencia thanks for the PR., few questions:

is the event working on ios and apple maps?
is the event working on android?
do we need to adapt the docs in any way?

@jherencia

The documentation currently does not mention anything about this issue in iOS, it says that you need to use scrollEnabled, but it is not right.

Could you update the docs? And what's wrong? Maybe its for Apple maps ?

Hi,
we need this option for production, so when can you merge it and release a new version, contains this changes??

Too bad it's not in the recent release :(

I too am blocked by this. Was taken aback it didn't work too as from what I read, I though I just needed to disable scrollEnabled but that's not working for me, so assume this PR is required.
Would really appreciate a committer review this PR.

As a follow-up, slightly tangentally, where isn't there an onPanDragEnd event like for markers? Even with this PR, I'm struggling to understand how to detect the end of a drag.
I'm guessing I'm going to have to schedule timeout and keep track of the timestamp of last onPanDrag event and once the elapsed time crosses a threshold, assume end of drag. Seems far from ideal as user could keep the finger pressed but just paused dragging.
Be much better to get notified when the user lifts their finger. 🤔

Please merge this

merged into master


pull request for e2e test at 12/19/2016 05:26:20

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update get-started.md

Add dev up step to instructions.

I've missed this post-merge, but its worth noting that dev up is only needed for Shopifolk, and that dev doesn't exist externally. Obviously most contribution is in-house and I think its fine to optimise for that, but wouldn't want to leave someone running the CLI locally externally -- e.g. to debug something -- confused.

Agree, I’ll back this out when I’m back in the office.
On Mon, Jul 1, 2024 at 4:06 AM Shaun Stanworth @.***>
wrote:

I've missed this post-merge, but its worth noting that dev up is only
needed for Shopifolk, and that dev doesn't exist externally. Obviously
most contribution is in-house and I think its fine to optimise for that,
but wouldn't want to leave someone running the CLI locally externally --
e.g. to debug something -- confused.
—
Reply to this email directly, view it on GitHub
https://github.com/Shopify/cli/pull/4136#issuecomment-2199856796, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AW35HVGU5N36HICGZXIAY6TZKEZ4RAVCNFSM6AAAAABKCAKQDOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJZHA2TMNZZGY
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I don't know why I thought I was reviewing the internal get-started doc 🤦
thanks @shauns


Add state for geckodriver

Looking good spyd3r!  Thanks!  Checkout the comment on the beef state, otherwise everything else looks great.


[bazel] Add support for the CW340

TODO, this is a work in progrss. I was able to make it work on fred after fixing the tty devices permissions and running
bazel test --test_output=streamed --cache_test_results=no //sw/device/tests:example_test_from_flash_cw340_rom_with_fake_keys --test_arg=--uarts=/dev/ttyUSB2,/dev/ttyACM0

This seems to significantly increase the number of targets to build due to the CW340 binaries being different from the CW310. This probably explains the timeout. Maybe, as @engdoreis suggests, it would be better to treat the CW310 and CW340 as identical for now.

It seems that aliasing the two did not work as expected. My next try will be to keep distinct binaries but not build CW340 targets by default. I will then only selectively enable cw340 on some tests.


Router + Fluxxor

Run router in js with flux params
var flux = new Fluxxor.Flux(...);
Router.run(routes, function(Handler) {  React.render( <Handler flux={flux} />, document.getElementById("app") );});
Question:
How pass flux in view helper?
Thanks in advance

For now, it's not possible. But I'm working on custom rendering to enable both client and server side rendering using custom code.

Clear. Thanks for the quick response


Network errors - cannot do anything?

since 4 or 5 months ago, I have been getting the error message "network error" on any action I try to make on my 1wallet (stake, claim, even create a new wallet).
Also, I really need help to undelegate my stake from 1wallet. I staked on staking.harmony.one and whenever I am trying to undelegate via staking.harmony.one,, it is asking to use either Ledger Nano or Harmony Browser extension. However, I delegated using 1wallet that uses Google Aithenticator and there is no such option.. if I am trying to press "Undelegate" on 1wallet interface,  I am getting a message: "Cannot find pre-computed proofs for this wallet. Storage might be corrupted. Please restore the wallet from Google authenticator".
Another issue is that my 1wallet will expire in 11 days, and I cannot restore it because I am getting a message: "Local state verification failed" and "Network error". It means that my staked ONE will be staked forever because I will lose access to the wallet.
Could somebody please help me as I am getting really desperate and I feel like I am out of options as there is no seed phrase here with 1wallet as it is relying on Google Authenticator and therefore I cannot use another wallet interface to try undelegate.
P.s. I have ONE in my wallet for gas.

Can you try again? We had a HTTPS certificate issue. Sorry about that.

now, the 1wallet is not accessible at all at this address : https://1wallet.crazy.one/
did it change ?

It should work now

Please reopen if the issue continues


Remove platform log middleware import

:tada: This PR is included in version 8.0.0-alpha.8 :tada:
The release is available on:

v8.0.0-alpha.8
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 8.0.0-beta.1 :tada:
The release is available on:

v8.0.0-beta.1
GitHub release

Your semantic-release bot :package::rocket:

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

v8.0.0
GitHub release

Your semantic-release bot :package::rocket:


SA-MP server refuses to stop with normal interrupt signal

There's an issue when asyncio code is being used with the plugin and when ^C as interrupt signal (SIGINT) is sent.
Server log will show the following:

As you mentioned @habecker - the sub-processes doesn't stop. I consider this a bug, and a fix would be nice :)
The side effects are that game panels hosting SA-MP servers will need to issue force-kill signals in order to stop the server.

@dennorske (is it working?)

I will test this evening, haven't checked in a while so I don't remember. Keeping you posted.

Confirmed fixed :)


parser: Test fixtures?

Fluent syntax is complex and needs a lot of tests to be sure it follows the specs.
As parts of their testing setup, the other fluent implementations uses Fixtures which are Fluent Resources as files and their corresponding AST as JSON files. It allows us to test a set of features easily.
We might need to consider implementing a way to transform our AST as a JSON file to make possible the uses of Fixtures.
Where fixtures exists?

fluent-rs
fluent-kotlin
fluent-js
python-fluent

TODO List;

Parsing errors :

[x] Junk nightmare and error recovery
[ ] Multiline literal.
[ ] Missing line end after variant list.

Validation errors:

[ ] Function identifier can not use lowercase characters.
[ ] Positional arg must not follow keyword args.
[ ] Named arguments must be unique.
[ ] Messages cannot be parameterized.
[ ] Equivalent to a MessageReference callee.
[ ] Term attributes may not be used in Placeables.
[ ] Variables cannot be parameterized.
[ ] Term values may not be used as selectors.
[ ] Term attributes may not be used for interpolation.
[ ] Message attributes may not be used as selectors.
[ ] Message values may not be used as selectors.
[ ] Function references are invalid outside of call expression.
[ ] CallExpressions on Terms are similar to TermReferences.
[ ] Nested expressions are not valid selectors.
[ ] Select expressions are not valid selectors.


Fix/remove icrc test token from chain fusion

Motivation
The Chain Fusion network was mistakenly showing the ICRC test tokens together with all the mainnet tokens.
We filter those tokes based on they Ledger Canister ID.
Changes

Create utils to distinguish Test Ledger Canisters.
Include filter in store networkTokens to filter the ICRC tokens by environment, based on the property ledgerCanisterId.

Tests
Before:

After:

@peterpeterparker before I submit for review, let me know what do you think of this solution.
I thought of another way to go to, that is defining an optional prop overrideNetworkEnv for Token object and use that:
// pseudo code

export type Token = {
	id: TokenId;
	network: Network;
	standard: TokenStandard;
	category: TokenCategory;
	overrideNetworkEnv?: NetworkEnvironment;  // <-- new prop
} & TokenMetadata;

[...]

export const mapIcrcToken = ({
	metadata,
	icrcCustomTokens,
	ledgerCanisterId,
	...rest
}: IcrcLoadData): IcTokenWithoutIdExtended | undefined => {
	[...]
	return {
		network: ICP_NETWORK,
		[...]
		ledgerCanisterId,
		...(isIcrcTestLedgerCanister(ledgerCanisterId) && { overrideNetworkEnv: 'testnet' }),  // <-- here we set it for ICRC
		...metadataToken,
		...rest
	};
};

Closed in favor of PR #1429


Update vpc resource names inline with issue #89.

Are we going to add the moved block in so that it does not break existing uses?
https://developer.hashicorp.com/terraform/language/modules/develop/refactoring#moved-block-syntax
https://developer.hashicorp.com/terraform/language/modules/develop/refactoring#removing-moved-blocks

Are we going to add the moved block in so that it does not break existing uses? https://developer.hashicorp.com/terraform/language/modules/develop/refactoring#moved-block-syntax https://developer.hashicorp.com/terraform/language/modules/develop/refactoring#removing-moved-blocks

I believe this is going to be in another PR. This PR just changes the resource name.

The agreed approach was to bump up a major version to not have to deal with the moved block, as far as I'm aware.

This is my PR so I can't approve it, but I have checked and this looks good to me 👍


#[repr(align(...))] structures cause an out of bounds memory access

I'm not sure if I should report this issue here or in rust-lang/rust or even somewhere else.
Consider the following code:
#[repr(align(64))]
#[derive(Copy, Clone, Default, Debug)]
pub struct CacheBucket([CacheEntry; 3]);

#[repr(C)]
#[derive(Copy, Clone, Default, Debug)]
pub struct CacheEntry {
    pub alpha: bool
}

fn main() {
    let buckets = 4194304;
    let xd = vec![CacheBucket::default(); buckets];
    println!("{:?}", xd);
}

compiled to wasm using rustup run stable cargo build --target=wasm32-wasi with rustc 1.42.0 (b8cedc004 2020-03-09)
Then run the wasm binary using wasmtime:
$ cargo run --release -- run oob.wasm
    Finished release [optimized] target(s) in 0.09s
     Running `target/release/wasmtime run ../chess/oob/target/wasm32-wasi/debug/oob.wasm`
Error: failed to run main module `../chess/oob/target/wasm32-wasi/debug/oob.wasm`

Caused by:
    0: failed to invoke `_start`
    1: wasm trap: out of bounds memory access, source location: @56bf
       wasm backtrace:
         0: <unknown>!core::ptr::write::h5550856a4b897d6d
         1: <unknown>!alloc::vec::Vec<T>::extend_with::h42807d753bccf81d
         2: <unknown>!<T as alloc::vec::SpecFromElem>::from_elem::h3a932571fd38db0f
         3: <unknown>!alloc::vec::from_elem::ha55e830b1b839b10
         4: <unknown>!oob::main::h5b4df0d7d93a566d
         5: <unknown>!std::rt::lang_start::{{closure}}::h4b37f8cbafc64616
         6: <unknown>!std::sys_common::backtrace::__rust_begin_short_backtrace::h09ce0967d7b1e05d
         7: <unknown>!std::panicking::try::do_call::h9f56360d08e2b60c
         8: <unknown>!__rust_maybe_catch_panic
         9: <unknown>!std::rt::lang_start_internal::h1c0f367ef14e29b3
         10: <unknown>!std::rt::lang_start::h81110b05d9ba24fa
         11: <unknown>!__original_main
         12: <unknown>!_start

The code runs correctly if buckets is small (e.g. 4) or if #[repr(align(64))] is removed.
wasmtime master, 78772cf5e1d019d060d2a5427aa7058604f924e5

How much memory is the wasm module allowed to use? The version with #[repr(align(64))] would need 256MiB for buckets alone, while the version without would need just 16MiB.

How should I check it? For reference, the original structure looks like this:
pub struct CacheEntry {
    pub alpha: bool,
    pub beta: bool,
    pub pv_node: bool,
    pub depth: i8,
    pub plies_played: u16,
    pub score: i16,
    pub upper_hash: u32,
    pub lower_hash: u32,
    pub mv: u16,
    pub static_evaluation: i16,
}

Switching to this defintion of CacheEntry doesn't change anything.

FWIW: Lucet gets a HeapOutOfBounds fault running this module (as I compiled it locally with 1.40, we're still stuck on snapshot 0).

Looks like this was a bug in the Rust standard library, fixed here.
As a local workaround while we wait for that to propagate you can depend directly on dlmalloc and use that as a global allocator:
# in Cargo.toml
[dependencies]
dlmalloc = { version ="0.1", features = ['global'] }

#[global_allocator]
static A: dlmalloc::GlobalDlmalloc = dlmalloc::GlobalDlmalloc;

#[repr(align(64))]
#[derive(Copy, Clone, Default, Debug)]
pub struct CacheBucket([CacheEntry; 3]);

#[repr(C)]
#[derive(Copy, Clone, Default, Debug)]
pub struct CacheEntry {
    pub alpha: bool,
}

fn main() {
    let buckets = 4194304;
    let xd = vec![CacheBucket::default(); buckets];
    let string = format!("{:?}", xd);
    println!("{}", string.len());
}

This works for me on nightly, so I guess we can close it.


v0.9.0 seems to break tox

Not exactly sure what is going on, but it seems like the environment activation is going haywire. A snippet of my output with -vvv.
... [snipped] ...
py38 start: run-test-pre
py38 run-test-pre: PYTHONHASHSEED='4208903247'
py38 finish: run-test-pre  after 0.00 seconds
py38 start: run-test
py38 run-test: commands[0] | pytest --cov=raster --cov-report=xml --cov-report=term-missing tests/
setting PATH=[snipped]/.tox/py38/bin:[snipped]:/home/broot/.local/bin
[684100] [snipped]$ [snipped]/.tox/py38/bin/pytest --cov=raster --cov-report=xml --cov-report=term-missing tests/
/tmp/tmp1g07361n: line 1: export: `.': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `[snipped]/.tox/py38/etc/conda/activate.d/gdal-activate.sh': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `.': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `[snipped]/.tox/py38/etc/conda/activate.d/geotiff-activate.sh': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `.': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `[snipped]/.tox/py38/etc/conda/activate.d/pdal-activate.sh': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `.': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `[snipped]/.tox/py38/etc/conda/activate.d/pdal-python-activate.sh': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `.': not a valid identifier
/tmp/tmp1g07361n: line 1: export: `[snipped]/.tox/py38/etc/conda/activate.d/proj4-activate.sh': not a valid identifier
... [snipped] ...

Downgrading to v0.8.4 makes everything work as it has before.

Could you please provide a reproducer?

I can try. I am actually not terribly familiar with tox or tox-conda. I only noticed this because the CI/CD of our project broke today, and I didn't set any of this up. I guess you would want a tox.ini and a conda environment file, right?

Yes let's start with your tox.ini and the output of conda info --all.

Well, I made a minimized version of that:
(base) bash-4.4$ cat tox.ini
[tox]
envlist = py38

[testenv]
conda_spec = env.txt
commands = echo foobar
(base) bash-4.4$ cat env.txt
numpy
gdal
fiona
(base) bash-4.4$ cat setup.py
from setuptools import (
    find_packages,
    setup
)

setup(
    name='foo',
    description='Foo to the Bar',
    package_dir={'foo': 'foo'},
    packages=find_packages(where='foo', exclude='.'),
    # Dependencies
    install_requires=[
        'numpy',
        'fiona'
    ],
    # pytest setup
    setup_requires=['pytest-runner'],
    tests_require=['pytest', 'pytest-cov']
)
(base) bash-4.4$ cat foo/__init__.py
import fiona
import numpy as np

This was done in a fairly clean conda environment (the base one, I pip-installed tox and tox-conda today in it).
(base) bash-4.4$ tox
GLOB sdist-make: /rd22/scratch/broot/Programs/example/setup.py
py38 create: /rd22/scratch/broot/Programs/example/.tox/py38
py38 installcondadeps: --file=/rd22/scratch/broot/Programs/example/env.txt
py38 inst: /rd22/scratch/broot/Programs/example/.tox/.tmp/package/1/foo-0.0.0.zip
py38 installed: attrs @ file:///tmp/build/80754af9/attrs_1620827162558/work,certifi==2021.10.8,click @ file:///home/linux1/recipes/ci/click_1610990599742/work,click-plugins @ file:///Users/ktietz/demo/mc3/conda-bld/click-plugins_1630665883852/work,cligj==0.7.2,Fiona==1.8.13.post1,foo @ file:///rd22/scratch/broot/Programs/example/.tox/.tmp/package/1/foo-0.0.0.zip,GDAL==3.0.2,mkl-fft==1.3.1,mkl-random @ file:///tmp/build/80754af9/mkl_random_1626186064646/work,mkl-service==2.4.0,munch @ file:///Users/ktietz/demo/mc3/conda-bld/munch_1630566382504/work,numpy @ file:///tmp/build/80754af9/numpy_and_numpy_base_1634095647912/work,Shapely @ file:///tmp/build/80754af9/shapely_1632813486044/work,six @ file:///tmp/build/80754af9/six_1623709665295/work
py38 run-test-pre: PYTHONHASHSEED='3078875635'
py38 run-test: commands[0] | echo foobar
WARNING: test command found but not installed in testenv
  cmd: /usr/bin/echo
  env: /rd22/scratch/broot/Programs/example/.tox/py38
Maybe you forgot to specify a dependency? See also the allowlist_externals envconfig setting.

DEPRECATION WARNING: this will be an error in tox 4 and above!
/tmp/tmpuu_9e2cq: line 1: export: `.': not a valid identifier
/tmp/tmpuu_9e2cq: line 1: export: `/rd22/scratch/broot/Programs/example/.tox/py38/etc/conda/activate.d/gdal-activate.sh': not a valid identifier
/tmp/tmpuu_9e2cq: line 1: export: `.': not a valid identifier
/tmp/tmpuu_9e2cq: line 1: export: `/rd22/scratch/broot/Programs/example/.tox/py38/etc/conda/activate.d/geotiff-activate.sh': not a valid identifier
/tmp/tmpuu_9e2cq: line 1: export: `.': not a valid identifier
/tmp/tmpuu_9e2cq: line 1: export: `/rd22/scratch/broot/Programs/example/.tox/py38/etc/conda/activate.d/proj4-activate.sh': not a valid identifier
foobar
__________________________________________________ summary ___________________________________________________
  py38: commands succeeded
  congratulations :)

(Yes, it succeeded, only because the command was an "echo" command instead of a usual pytest. In my actual tests, a bunch of the fiona and gdal calls are failing because the package activation scripts weren't sourced, as noted in the error output above.

This is weird... how is stuff like this in my log?
click-plugins @ file:///Users/ktietz/demo/mc3/conda-bld/click-plugins_1630665883852/work,

I am on a linux system, not a mac.

In case it is relevant,
(base) bash-4.4$ conda --version
conda 4.11.0

conda info --all is relevant.

(base) bash-4.4$ conda info --all

     active environment : base
    active env location : /home/broot/scratch/miniconda3
            shell level : 1
       user config file : /home/broot/.condarc
 populated config files : /home/broot/.condarc
          conda version : 4.11.0
    conda-build version : not installed
         python version : 3.7.10.final.0
       virtual packages : __linux=4.18.0=0
                          __glibc=2.28=0
                          __unix=0=0
                          __archspec=1=x86_64
       base environment : /home/broot/scratch/miniconda3  (writable)
      conda av data dir : /home/broot/scratch/miniconda3/etc/conda
  conda av metadata url : None
           channel URLs : https://repo.anaconda.com/pkgs/main/linux-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/linux-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : /home/broot/scratch/miniconda3/pkgs
                          /home/broot/.conda/pkgs
       envs directories : /home/broot/scratch/miniconda3/envs
                          /home/broot/.conda/envs
               platform : linux-64
             user-agent : conda/4.11.0 requests/2.26.0 CPython/3.7.10 Linux/4.18.0-348.2.1.el8_5.x86_64 centos/8.5.2111 glibc/2.28
                UID:GID : [snipped]
             netrc file : /home/broot/.netrc
           offline mode : False

# conda environments:
#
base                  *  /home/broot/scratch/miniconda3
... [snipped ] ...
                         /rd22/scratch/broot/Programs/example/.tox/py38

sys.version: 3.7.10 | packaged by conda-forge | (defa...
sys.prefix: /home/broot/scratch/miniconda3
sys.executable: /home/broot/scratch/miniconda3/bin/python
conda location: /home/broot/scratch/miniconda3/lib/python3.7/site-packages/conda
conda-build: None
conda-env: /home/broot/scratch/miniconda3/bin/conda-env
user site dirs:

CIO_TEST: <not set>
CONDA_DEFAULT_ENV: base
CONDA_EXE: /home/broot/scratch/miniconda3/bin/conda
CONDA_PREFIX: /home/broot/scratch/miniconda3
CONDA_PROMPT_MODIFIER: (base)
CONDA_PYTHON_EXE: /home/broot/scratch/miniconda3/bin/python
CONDA_ROOT: /home/broot/scratch/miniconda3
CONDA_SHLVL: 1
CURL_CA_BUNDLE: <not set>
PATH: /home/broot/scratch/miniconda3/bin:/home/broot/scratch/miniconda3/bin:/home/broot/scratch/miniconda3/condabin:[snipped]:/home/broot/.local/bin
PYTHONSTARTUP: /home/broot/.pythonrc
REQUESTS_CA_BUNDLE: <not set>
SSL_CERT_FILE: <not set>

Snipped out possibly sensitive information such as my other environments, user id, and parts of my PATH.

Thank you, I am able to reproduce it and a fix is coming.

Thank you for your prompt response on this!


chore(EMS-3969): align test descriptions

Introduction :pencil2:
Some tests had the following descriptions:
describe('when xyz returns success as true', () => { ... });
describe('when xyz returns success as false', () => { ... });

it('should display ...')

Whereas the majority use the following descriptions:
describe('when xyz returns success=true', () => { ... });
describe('when xyz returns success=false', () => { ... });

it('should render ...')

Resolution :heavy_check_mark:

Align test descriptions.

Miscellaneous :heavy_plus_sign:

Remove some unnecessary comments.

@Zainzzkk Updated various descriptions.


Quantization failed

https://github.com/intel/neural-compressor/tree/master/examples/onnxrt/nlp/huggingface_model/text_generation/llama/quantization/weight_only
bash run_quant.sh --input_model=./Meta-Llama-3.1-8B  --output_model=./Meta-Llama-3.1-8B_AWQ                    --batch_size=1                 --dataset=NeelNanda/pile-10k                  --tokenizer=meta-llama/Meta-Llama-3.1-8B                 --algorithm=AWQ
/usr/local/lib/python3.10/dist-packages/diffusers/models/vq_model.py:20: FutureWarning: `VQEncoderOutput` is deprecated and will be removed in version 0.31. Importing `VQEncoderOutput` from `diffusers.models.vq_model` is deprecated and this will be removed in a future version. Please use `from diffusers.models.autoencoders.vq_model import VQEncoderOutput`, instead.
 deprecate("VQEncoderOutput", "0.31", deprecation_message)
/usr/local/lib/python3.10/dist-packages/diffusers/models/vq_model.py:25: FutureWarning: `VQModel` is deprecated and will be removed in version 0.31. Importing `VQModel` from `diffusers.models.vq_model` is deprecated and this will be removed in a future version. Please use `from diffusers.models.autoencoders.vq_model import VQModel`, instead.
 deprecate("VQModel", "0.31", deprecation_message)
The tokenizer class you load from this checkpoint is not the same type as the class this function is called from. It may result in unexpected tokenization.
The tokenizer class you load from this checkpoint is 'PreTrainedTokenizerFast'.
The class this function is called from is 'LlamaTokenizer'.
You are using the default legacy behaviour of the <class 'transformers.models.llama.tokenization_llama.LlamaTokenizer'>. This is expected, and simply means that the `legacy` (previous) behavior will be used so nothing changes for you. If you want to use the new behaviour, set `legacy=False`. This should only be set if you understand what it means, and thoroughly read the reason why this was added as explained in https://github.com/huggingface/transformers/pull/24565 - if you loaded a llama tokenizer from a GGUF file you can ignore this message
Traceback (most recent call last):
 File "/root/neural-compressor/examples/onnxrt/nlp/huggingface_model/text_generation/llama/quantization/weight_only/main.py", line 118, in <module>
   tokenizer = LlamaTokenizer.from_pretrained(args.tokenizer)
 File "/usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py", line 2271, in from_pretrained
   return cls._from_pretrained(
 File "/usr/local/lib/python3.10/dist-packages/transformers/tokenization_utils_base.py", line 2505, in _from_pretrained
   tokenizer = cls(*init_inputs, **init_kwargs)
 File "/usr/local/lib/python3.10/dist-packages/transformers/models/llama/tokenization_llama.py", line 171, in __init__
   self.sp_model = self.get_spm_processor(kwargs.pop("from_slow", False))
 File "/usr/local/lib/python3.10/dist-packages/transformers/models/llama/tokenization_llama.py", line 198, in get_spm_processor
   tokenizer.Load(self.vocab_file)
 File "/usr/local/lib/python3.10/dist-packages/sentencepiece/__init__.py", line 905, in Load
   return self.LoadFromFile(model_file)
 File "/usr/local/lib/python3.10/dist-packages/sentencepiece/__init__.py", line 310, in LoadFromFile
   return _sentencepiece.SentencePieceProcessor_LoadFromFile(self, arg)
TypeError: not a string

Hi @endomorphosis , which version of transformers are you using? Could you please try the latest version 4.44.0 for transformers?


Deposit update

As a user, I would like to know if a pool I'm about deposit to is in deficit or surplus and how it may affect my holdings.
High level
Adding an indication for users about the pool status and explaining what it means and how it affects them.
Mockups
Web - https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=15566%3A234470
Mobile - https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=16094%3A227793
changes
we are adding 4 new elements to the deposit screen.
they have different behavior based on the pool deficit condition (which will be explained below)
the sections are:

vault balance

https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=15566%3A234470 (notice different font color based on positive/negative pool value)
Use the same calculation we currently have on the withdrawal flow

expected calculation:
{current pool balance} / {total staked balance}
example:
(80 / 100) - 1 = -0.2 -> to % multiple by 100 -> -20%
(110 / 100) - 1 = +0.1 -> to % multiple by 100 -> +10%

(I) icon and a tooltip explaining the meaning of vault balance.
Text -  "The % of tokens available"

disclaimer text

Surplus - "This pool is currently NOT in deficit. This may change over time. Should this pool be in deficit when you’re ready to withdraw, your deposit will accrue the pool deficit at that time."
Deficit -
Empty state - "This pool is in deficit. If an immediate withdrawal were initiated, the claimable amount will be --TKK/ETH. Its value and deficit amount can change over time."
Input  - "This pool is in deficit. If an immediate withdrawal were initiated, the claimable amount will be xxxxxxxx TKK/ETH. Its value and deficit amount can change over time."
*** For calculation, use the same function we are using when showing the user he withdraws the amount in deficit

checkbox

https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=15824%3A244285
Add warning text and checkbox of BNT distribution is disabled as we have now in withdraw flow
CTA button is enabled only if the user checks the BNT warning

faq

https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=15824%3A244253
By default, FAQ are close
There are two FAQs
Use the text from Figma
"Learn more" - link TBD

Auto-filling of the deposit amount

Removing the 25/50/75/100% deposit from the available balance.
Add "MAX" button next to the available amount
** Clicking the button will auto-fill the deposit amount with the max available balance

Deposit options
1. Deposit while the vault is in surplus

Pool has more tokens than its commitment
https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=15824%3A244266

Vault balance - Use the calculation above and mark it in blue
Disclaimer text - Use the surplus option
Add the checkbox
Add FAQ section

2. Deposit while the vault is in deficit

Pool has fewer tokens than its commitment
https://www.figma.com/file/fwADI9wqDrRAdlMX8EddCw/Bancor-v3?node-id=15824%3A244267

Vault balance - Use the calculation above and mark it in red
Disclaimer text - Use the deficit  option
Add the checkbox
Add FAQ section

3. Deposit BNT

BNT pool cannot be in deficit or surplus
Do not add the option above, everything stays the same as of now.

Blocked by: #617


how can i displaying pan image start with its compass point?

hi please let me know if i am moving to detail page, pan image should be display that corner side of its compass field. is there any way to manage newViewer setting ?
Thanks
Sanny Soni

I have no idea what you're asking.