Hyperdex Alpha new version

Well i downloaded the new Alpha version of the hyperdex, installed it, made a new account, signed in.
It now stuck on the Thank you for helping test hyperdex.
I tried to do random clicks to let this message go, clicked the X mark. nothing happens. Had restarted it twice nothing happens never goes past that message.
Windows 10 pro

HyperDEX 0.1.0-alpha.10
Electron 2.0.4
win32 10.0.10240
Locale: en-US

Sorry about that. We had a slight issue with the latest builds. I've uploaded fixed builds now: https://github.com/atomiclabs/hyperdex/releases/tag/v0.1.0-alpha.10 (You need to download it again).


Does not update cache

With the new version (2.2.0), the cache does not update itself when running gilt overlay on a already overlayed folder. When the target git repo is updated, I need to clear the cache directory (~/.gilt) first in order to have the latest repo. I believe the code only doing a git fetch instead of git fetch and pull.

git pull is for merging worktrees, and modern Gilt brokers in bare repositories for its caches, so trying to run git pull will only throw errors.
I was able to reproduce this, however.
Steps to reproduce
An example Giltfile.yaml:
---
giltDir: ~/.gilt/clone
debug: true
repositories:
  - git: /Users/nisimond/work/blastarea/wtf_gilt
    version: main
    dstDir: /Users/nisimond/Desktop/zomg_gilt

Run the commands:
mkdir -p /Users/nisimond/work/blastarea/wtf_gilt
cd /Users/nisimond/work/blastarea/wtf_gilt
git init .
git commit -m 'initial commit'
cd $GILT_DIR
gilt overlay  # fails
cd /Users/nisimond/work/blastarea/wtf_gilt
date > lol
git add lol
git commit -a -m 'commit #1'
cd $GILT_DIR
gilt overlay

Expected result
The second invocation succeeds
Actual behavior
The cache does not create/move the refs for main, and the second command also fails
Workaround
Tags will get updated, so it is possible to use/move a tag around, if you control the source repository
Miscellany
The git fetch command we are composing is incomplete.  The "correct" command should be:
git fetch --tags --force origin '+refs/heads/*:refs/heads/*'

Since Gilt still uses the git binary in the users' $PATH, it is also subject to the whims of their various .gitconfig files.  So if they have a custom remote name defined, Gilt will clone its caches using whatever is configured.  git clone --origin NAME can be used to override this, and likely should be.
How do invalidate all the existing caches is another story.

Fixed in #201


Terraform v1.0.0 support (w/ fixes and CI testing)

Fixes:

https://github.com/28mm/blast-radius/issues/86
https://github.com/28mm/blast-radius/issues/69
https://github.com/28mm/blast-radius/issues/91

Changelog

Added python-hcl2, support for HCL 2
Added support for Terraform v1.0.0 and backwards compatibility
Added a Terratest-capable feature to be CI-ingrated so its dockerfile build can be  test automatically

Since the maintainer has been AFK for quite some time, I will also create a separate Docker container for those who want to run it without having to deal with the hassle for building and applying the fixes. Hang on. My repository can be found here.

Integrated changes from:

https://github.com/28mm/blast-radius/pull/84

@gruberdev awesome work!
I recommend splitting off (while still giving credit to the original) so you are no longer just a fork of this repository. This way yoru fork is more discoverable, can track issues and has the search feature on github. Otherwise it just appears as one of the many forks (in varying states) of this project. It would be awesome if you updated the Readme so that it mentions your docker image.

@gruberdev awesome work!
I recommend splitting off (while still giving credit to the original) so you are no longer just a fork of this repository. This way yoru fork is more discoverable, can track issues and has the search feature on github. Otherwise it just appears as one of the many forks (in varying states) of this project. It would be awesome if you updated the Readme so that it mentions your docker image.

I will do that eventually, just need to find some spare time on my routine.

@gruberdev awesome work!
I recommend splitting off (while still giving credit to the original) so you are no longer just a fork of this repository. This way yoru fork is more discoverable, can track issues and has the search feature on github. Otherwise it just appears as one of the many forks (in varying states) of this project. It would be awesome if you updated the Readme so that it mentions your docker image.

I will do that eventually, just need to find some spare time on my routine.

@gruberdev did you ever find the spare time?


Big issue with animation's Wobbling

Hi guys, i make animation in sprite sheet and set scale to 0.5 for better performance in the latest version.
but the animation is Wobbling all the time especially for the small size of animated item. I tried PRECISION to high, low and default, all is not working. it happens on iphone 6+, but working well in browser.

In the latest version of PIXI
PIXI.settings.PRECISION_FRAGMENT = 'highp';
solves that problem for me

Also, using https://github.com/pixijs/pixi-picture and adding "sprite.pluginName = 'picture'" helps in some cases.

@tywang2006 closing as I believe the above 2 answers solve your issue. Feel free to reopen if the solutions don't work for you :)


Crash on Windows 11 ARM64 with OpenSSL built in release mode

When OpenSSL is built in release mode for Windows ARM64 (cross-compiling on amd64 host) using MSVC toolchain, the SSL_CTX_new function causes program to crash with "Access violation - code c0000005" error. The crash does not occur when OpenSSL is built in debug mode.
OpenSSL is built using perl Configure no-apps enable-pie no-shared VC-WIN64-ARM command, test program is:
#include <iostream>
#include <openssl/ssl.h>

int main(int argc, char** argv) {
    std::cout << "Entered main\n";
    auto* ctx = SSL_CTX_new(TLS_client_method());
    std::cout << "Created SSL_CTX " << ctx << "\n";
    std::cout << "Exiting main\n";
    return 0;
}

This is what WinDbg shows on crash:
(1940.d00): Access violation - code c0000005 (first chance)
First chance exceptions are reported before any exception handling.
This exception may be expected and handled.
arm64_crash!ssl_cipher_collect_ciphers+0x34:
00007ff7`02d75fbc a903dff6 stp         x22,x23,[sp,#0x38]

Stack trace is:
WARNING: Stack pointer is outside the normal stack bounds. Stack unwinding can be inaccurate.
 # Child-SP          RetAddr               Call Site
00 ffffda56`12eff3a0 00000000`00000000     arm64_crash!ssl_cipher_collect_ciphers+0x34 [F:\projects\openssl-3.4.0\ssl\ssl_ciph.c @ 696]

OpenSSL version is 3.4.0, MSVC version is "Microsoft (R) C/C++ Optimizing Compiler Version 19.42.34435 for ARM64".

When OpenSSL is built using clang-cl (VC-CLANG-WIN64-CLANGASM-ARM) then it doesn't crash.

Im actually getting the same crash with VC-CLANG-WIN64-CLANGASM-ARM but called from code compiled with VC

Branch origin/openssl-3.2 seems unaffected by this. FWIW

Branch origin/openssl-3.2 seems unaffected by this. FWIW

Notably, I forgot to update the submodules when I checked out 3.2 if thats helpful.

Please ignore both of my comments. I see now. Its the debug build that is not affected. Both ARM configs were affected for me.

Defining -DNO_INTERLOCKEDOR64 in the CLFAGS works around it for me for anybody that lands here looking for a solution until the maintainers can get something landed. It kind of looks like this is the only intrinsic that hasn't been replaced?

please see https://github.com/openssl/openssl/issues/26258#issuecomment-2562487830
It appears that clang doesn't fully support MSVC intrinsics:
https://clang.llvm.org/docs/MSVCCompatibility.html
Adding NO_INTERLOCKEDOR64 as a define in threads_win.c when building with clang seems like the right solution to me


Refactor test mocking

Continuing discussion from #798
@mikemorris @dereklieu Take a look and let me know what you think.  Summary of the refactor:

File structure in test/mocks that's intended to mirror the app directory.
test/mocks/helpers module exposes a distate singleton (implementation), which holds on to dependency injection state that allows for coordinating mock objects as we discussed.  Only has to be referenced explicitly in the mock code itself (i.e., test code need not be aware of it).  Usage:

mocks/mock/mocksomething.js
var distate = require('../helpers').distate;
module.exports = distate.register(myFunctionThatMakesAMockObject);

use in a test
var mockSomething = require('../../mocks/mocksomething');
/* ... describe() blah blah ... */
beforeEach(function() { mockSomething.reset(); } // `distate.register` decorates the function with a reset(), which clears the distate.
it('does whatever', function() { var mockedObject = mockSomething(); })

@mikemorris  @dereklieu  You guys okay with merging this?  I'd like to add a couple more tests based on this refactor..

:+1: to merge

@mikemorris Thanks and welcome back :)


”Alțina” is listed in the drop-down menu

Description:
"Alțina" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Alțina" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Alțina, SIBIU".


[BUG] Typing an attribute to float in an init makes slearn "clone" method fail on compiled version.

Hello Everyone,
I hope i'm the right place for this bug,  as I have very little experience with cython.
While trying to apply cython on my code, I encountered a problem with sklearn "clone" method. This seems to be due to some variable "disapearing" in the __init__ method when a variable is typed as float. I haven't found any similar bug reported elsewhere, so I made a minimal code example so that everyone can reproduce the bug.
To Reproduce
I used the basic tutorial from https://cython.readthedocs.io/en/latest/src/tutorial/cython_tutorial.html to generate an example that reproduces the bug.
The following compilation produces unexpected results at runtime on 0.29.24 :
helloworld.pyx

from sklearn.base import clone, BaseEstimator, TransformerMixin
import numpy as np

class MyTransform(BaseEstimator, TransformerMixin):
    def __init__(self, var_float: float = 0.01):
        self.var_float = var_float

    def transform(self, X: np.ndarray):
        return X

    def fit(self, X):
        return self

    def _more_tags(self):
        return {"no_validation": True, "stateless": True}

    def __sklearn_is_fitted__(self):
        """Return True since Functors are stateless."""
        return True

X = np.random.randn(100, 10)
my_transform = MyTransform()
cloned_transform = clone(my_transform)
new_X = my_transform.fit_transform(X)

print("Hello World")

setup.py (unchanged)
from setuptools import setup
from Cython.Build import cythonize

setup(ext_modules=cythonize("helloworld.pyx"))

Expected behavior
Like in the basic tutorial, this should print "Hello World" when calling import helloworld
Instead, i get RuntimeError: Cannot clone object MyTransform(), as the constructor either does not set or modifies parameter var_float
The python code works fine when not compiled through cython
Important Additional information !
If var_float is no longer typed as float, then the resulting compiled code works just fine
Environment :

OS: Windows, Linux
Python version : 3.8.11
Cython version : 0.29.24

It's a bug in sklearn - https://github.com/scikit-learn/scikit-learn/blob/74016abcdffe67cdc58fc326c58fb5033371d96b/sklearn/base.py#L91 - they do an "is not" check. Since typing the variable as float involves unboxing the Python value to a C float this check can never pass.
If you don't intend for Cython to use your annotations then turn the annotation_typing directive off.


Fix crash with the PMMO plugin

Fix a crash regarding the small parts table or tesla synthesizer and the PMMO plugin. Fixes #3 on 1.19.

 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.


chore: update axoproject and use changelog api

fixes #262

depends on https://github.com/axodotdev/axoproject/pull/37

DO NOT MERGE UNTIL AXOPROJECT RELEASE

Mark as draft so we block the ability to merge it before that?

axoproject 0.4.7 is out now.


Music & Sound Updates

Added new music to game
Updated sound and music volume levels & allowed for full mute
Removed master volume slider and changed to sound & music

Closing PR.


Update demo to SF 2.7

Hi guys!,
How about updating SF to 2.7 in sake of consistency between docs ? . BR

You are right and we should do that quickly. There is a pending PR in #72.


define 'matching'

In the section 'Using Profiles in Statements' it's unclear what the adverb "matching" means.
The process of matching is explained somewhat here: https://github.com/DataInteroperability/xapi-profiles/blob/master/xapi-profiles-spec.md#statement-templates
But

It would be good to have a separate section explaining about matching and the concept of "determining properties" and
It would be good to link to that section whenever we talk about matching.

Reviewing the current text I'm going to close this one, I think things read pretty well currently


[Perf-tests: arktos]object daemonset-priorityclass-0 creation error

What happened:
when run perf-tests - load testing, test failed with error:
E1015 14:02:06.830967   14966 clusterloader.go:217]   Errors: [namespace 0270w4-testns object daemonset-priorityclass-0 creation error: the server could not find the requested resource

What you expected to happen:
daemonset-priorityclass created successfully
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Arktos version (use kubectl version):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Network plugin and version (if this is a network-related bug):
Others:

This is 100% repro in perf-test load testing, even 100 hollow-nodes can repro. you can find detailed logs in any load testing result. here is one example: under GCP project: workload-controller-manager on sonyadev4:
/home/sonyali/logs/perf-test/gce-100/arktos/1204-daily-100-1a0w1e


fix: env vars

✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog

Fixed some spelling/grammar.
Added some links to documentation where appropriate.
Comments in ".env.example" should not go past the 80 character line limit, which makes things easier to read for people with disabilities, people who work with two files side by side, etc.
The env template should provide quotation marks, which prevents end-users from shooting themselves in the foot when they add a new environment variable with a special character.

Further questions:

Should SERVERVAR be changed to SERVER_VAR and NEXT_PUBLIC_CLIENTVAR changed to NEXT_PUBLIC_CLIENT_VAR?

can you respond to this?

sorry - nah it's fine imo


Fixing Misspellings Across Documentation

Summary
Fix spacing or spelling across repository.

Address same typos seen in #59


will not find eGPU

when I open Gswitch I get the following message "Error! failed to find multiple GPUs"  I have a Sonnet Breakaway puck rx 5500 xt. my GPU monitor app show that both cards are working

Getting the same error message.. tried disconnecting external monitor/thunderbolt connections, but to no avail.
Can anyone help?
*MacBook Pro (Retina, 15-inch, Mid 2015)

I'm also having the same issue! I have a MacBook Pro 13-inch 2020


Unable to use the plugin when upgrading from 4.2.3 to 4.2.4

The commit "3d46ac13134d2b0b4e41b245906f71cf4cfd6584" that replaced httpc with gun library makes the auth-http plugin unusable.
Same config working with 4.2.3 is no more working with 4.2.4 (and up). I always get a timeout error. It seems like something in the configuration file is broken.
Any idea?
Thanks
Denis

It could be this?
https://docs.emqx.io/en/broker/v4.3/changes/upgrade-4.3.html#important-config-changes
I am currently seeing some similar issues but i know that my authentication endpoint is taking a long time to respond

It could be this? https://docs.emqx.io/en/broker/v4.3/changes/upgrade-4.3.html#important-config-changes
I am currently seeing some similar issues but i know that my authentication endpoint is taking a long time to respond

Is the plugin supporting these changes?
I've update the EMQX_AUTH__HTTP__ACL_REQ to EMQX_AUTH__HTTP__ACL_REQ__URL but it doens't change anything inside the container


Update quantum chess code

Re-add quantum chess commit #278 which was inadvertently squashed during import.
Fixes: #17

LGTM


It does not allow almost any requests. Even the fonts

CRS version - v3.3.4:
Paranoia level - 1:
ModSecurity version 3:
Web Server - nginx/1.22.0:
Operating System - Ubuntu 22.10:

Why it does not pass a simple query like
curl --location '<IP_ADDRESS>/api/'
--header 'Content-Type: application/json'
--header 'Accept: application/json'
--data '{"any":"param","i":"post"}'
ModSecurity: Warning. Matched "Operator `Rx' with parameter `^\d+$' against variable `REQUEST_HEADERS:Content-Length' (Value: `26' ) [file "/etc/nginx/modsec/coreruleset/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "128"] [id "920160"] [rev ""] [msg "Content-Length HTTP header is not numeric"] [data "26"] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/210/272"] [hostname "<IP_ADDRESS>"] [uri "/api/"] [unique_id "168018524010.980622"] [ref "v256,2"]
ModSecurity: Warning. Matched "Operator `Rx' with parameter `^[\w/.+-]+(?:\s?;\s?(?:action|boundary|charset|type|start(?:-info)?)\s?=\s?['\"\w.()+,/:=?<>@-]+)*$' against variable `REQUEST_HEADERS:Content-Type' (Value: `application/json' ) [file "/etc/nginx/modsec/coreruleset/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "916"] [id "920470"] [rev ""] [msg "Illegal Content-Type header"] [data "application/json"] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/255/153"] [tag "PCI/12.1"] [hostname "<IP_ADDRESS>"] [uri "/api/"] [unique_id "168018524010.980622"] [ref "v34,16t:lowercase"]
ModSecurity: Warning. Matched "Operator `Rx' with parameter `^(?:(?:\*|[^\"(),\/:;<=>?![\x5c\]{}]+)\/(?:\*|[^\"(),\/:;<=>?![\x5c\]{}]+))(?:\s*+;\s*+(?:(?:charset\s*+=\s*+(?:\"?(?:iso-8859-15?|windows-1252|utf-8)\b\"?))|(?:(?:c(?:h(?:a(?:r(?:s(?:e[^t\"(),\/:;<=> (714 characters omitted)' against variable `REQUEST_HEADERS:Accept' (Value: `application/json' ) [file "/etc/nginx/modsec/coreruleset/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "1147"] [id "920600"] [rev ""] [msg "Illegal Accept header: charset parameter"] [data "application/json"] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [hostname "<IP_ADDRESS>"] [uri "/api/"] [unique_id "168018524010.980622"] [ref "v59,16t:lowercase"]
ModSecurity: Warning. Matched "Operator `Rx' with parameter `^(?i:(?:[a-z]{3,10}\s+(?:\w{3,7}?://[\w\-\./]*(?::\d+)?)?/[^?#]*(?:\?[^#\s]*)?(?:#[\S]*)?|connect (?:\d{1,3}\.){3}\d{1,3}\.?(?::\d+)?|options \*)\s+[\w\./]+|get /[^?#]*(?:\?[^#\s]*)?(?:#[\S]*)?)$' against variable `REQUEST_LINE' (Value: `POST /api/ HTTP/1.1' ) [file "/etc/nginx/modsec/coreruleset/rules/REQUEST-920-PROTOCOL-ENFORCEMENT.conf"] [line "48"] [id "920100"] [rev ""] [msg "Invalid HTTP Request Line"] [data "POST /api/ HTTP/1.1"] [severity "4"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-protocol"] [tag "paranoia-level/1"] [tag "OWASP_CRS"] [tag "capec/1000/210/272"] [hostname "<IP_ADDRESS>"] [uri "/api/"] [unique_id "168018524010.980622"] [ref "v0,19"]
ModSecurity: Access denied with code 403 (phase 2). Matched "Operator `Ge' with parameter `15' against variable `TX:ANOMALY_SCORE' (Value: `18' ) [file "/etc/nginx/modsec/coreruleset/rules/REQUEST-949-BLOCKING-EVALUATION.conf"] [line "81"] [id "949110"] [rev ""] [msg "Inbound Anomaly Score Exceeded (Total Score: 18)"] [data ""] [severity "2"] [ver "OWASP_CRS/3.3.4"] [maturity "0"] [accuracy "0"] [tag "application-multi"] [tag "language-multi"] [tag "platform-multi"] [tag "attack-generic"] [hostname "<IP_ADDRESS>"] [uri "/api/"] [unique_id "168018524010.980622"] [ref ""]

Hi @blackmold9, probably a known problem, see this: https://github.com/coreruleset/coreruleset/issues/3117

+1 to @azurit's comment.
Where are you building nginx from, @blackmold9?

hi, @azurit's, from https://nginx.org/download

After building nginx from https://modsecurity.digitalwave.hu/, everything functions correctly


"Submit when ready" message after successful attempt shouldn't be shown for draft/retired kata

On a draft Beta kata, you can 'Train Again', but the button to 'Submit' never appears after a successful attempt, although it instructs the user to submit as it would regularly do. Either it's not possible to submit, in case 'Train Again' should not appear in the first place, or it should be possible to submit.

but as a matter of facts, you can train again. You just cannot finish the training, sort of.

Yes, I can train again, but not submit, although it claims I can. Something is wrong here.

One of the two is true: either you can train and submit or you can't train and therefore not submit
Or maybe: if you can train, but can't submit, the website should not report that you can
but I think that's weaker and not to be preferred.

Yes, I can train again, but not submit, although it claims I can. Something is wrong here.

might be worth to change the title to 'submit button shouldn't show up after an attempt on drafts", no?

Yes, I can train again, but not submit, although it claims I can. Something is wrong here.

might be worth to change the title to 'submit button shouldn't show up after an attempt on drafts", no? (or did I misunderstand the intent again?)

I'm not the judge of that. I can see both behaviors can be consistently defended, and I don't have a real preference in this.
If it's the intention to not let someone submit a second solution, than prevent the 'train again' button. If not, allow a second submission.

Draft kata has never allowed submitting final. It's just that the recent beta process change to send katas to draft causes this to happen a lot more frequently.

If that's true, the 'Train Again' button should be removed.

"Train Again" button won't be removed because you can train again, and it's also necessary when reviewing draft kata.
The solution is not discarded. The unsubmitted solution should be there when you come back later, and you can submit when the kata is published.

although it instructs the user to submit as it would regularly do.

I can change this to something explaining you can't submit draft/retired.

I can change this to something explaining you can't submit draft/retired.

Alright, thanks for clarifying the situation.

Point in case: https://www.codewars.com/kata/5a2b703dc5e2845c0900005a/discuss#6324604f2341c9004b0c18e4

Deployed


Update GoJS version (2.1.19)

Coverage decreased (-0.5%) to 93.651% when pulling 222d581e374a47a4a12ac59a95441f35bda7b2a1 on nicolaserny:gojs-udpate into a7ff198ae8fac033d4cd3dfadd940c249e4e2378 on xcomponent:master.


Account for bash being in location other than /bin for postgresql checks

looks fine

Anything else needed for this to be merged? Keep having to manually edit things to keep up with other nad changes.


500 error

The API is currently throwing a 500 at https://botw-compendium.herokuapp.com/api/v2 endpoint

This error seems to have been caused by downtime on the database I'm dependent on. This problem started less than six hours ago according the API tests. I'm working on a temporary fix right now.
Thanks for pointing this out @VagrantApe!

FYI Rockset's uptime monitor is here.

Thanks for the quick response.

@VagrantApe all systems back up :)


Extend the language specification with imports from other files

Although the language manual discusses imports, it does not say anything about importing modules from other files. Of course, we can pretend that files do not exist, similar to TLA+. Alternatively, we can have a more explicit importing strategy, like in TypeScript and other PLs:
import { foo } from "commonDefs"
import { bar } from "../parentDefs"

cc @bugarela, @shonfeder, @thpani, @Kukovec

This is implemented. This has to be documented as tracked in #832


Update README.md

@hectorsector This is a good start. One thing you can do to make your PR even better is to include one of the keywords followed by the issue number: closes #1. This does a few things:

It lets your team know the pull request is addressing a specific issue.
It creates a permanent cross link between the two discussions
It automatically closes the issue with the pull request is merged.

Add a Reviewer
Before you merge, you should ask someone to review the changes you made. You can do this via the Reviewers section. Go ahead and add GitHubTeacher as a reviewer now.


Course Description page responsive

Is your feature request related to a problem? Please describe.
I want to make course description page responsive for different screen sizes.
Describe the solution you'd like

Looks great @poojadas07 :)


Feedback for “List of commands”

The introspection example seems to be incorrect:
(https://orm.drizzle.team/kit-docs/commands#introspect--pull)

export default {
  schema: "./src/schema/*",
  out: "./drizzle",
  connectionString: "postgresql://postgres:password@host:port/db",
} satisfies Config;

Error:
    if (typeof options.dbCredentials.connectionString === "undefined" && (typeof options.dbCredentials.host === "undefined" || typeof options.dbCredentials.database === "undefined")) {
                                     ^

TypeError: Cannot read properties of undefined (reading 'connectionString')

This worked:

export default {
  schema: "./src/schema/*",
  out: "./drizzle",
  dbCredentials: {
    connectionString: "postgresql://postgres:password@host:port/db",
},
} satisfies Config;```

The error you've seen should be fixed. It's a bug
The docs you've sent are for cli params and not config. For config please check: https://orm.drizzle.team/kit-docs/config-reference#dbcredentials

What is the error that is being fixed, and are the docs for introspect/pull fixed?
I still see
import type { Config } from "drizzle-kit";

export default {
  schema: "./src/schema/*",
  out: "./drizzle",
  connectionString: "postgresql://postgres:password@host:port/db",
} satisfies Config;

and
import type { Config } from "drizzle-kit";

export default {
  schema: "./src/schema/*",
  out: "./drizzle",
  user: "postgres",
  password: "password",
  host: "<IP_ADDRESS>",
  port: 5432,
  db: "db",
} satisfies Config;

as the example code.

I think you're right. I was actually a bit confused by the response of @AndriiSherman as the cli docs provide the problematic config.ts example. Maybe we are missing something here.

Both docs references were fixed


multiple NICs not showing up in a multi-homed virtualbox cpi deployment

I have a configuration for a multi-homed VM using a VirtualBox CPI, and although the NICs show up in bosh instances, when I ssh into the VM, I only see one NIC. I am not sure if this is an actual issue or just something silly I am doing. Any and all help would be appreciated. Thank you.
versions
stemcell: bosh-warden-boshlite-ubuntu-trusty-go_agent  3586.23*
bosh: 265.2.0
Cloud config
networks:
- name: network1
  type: manual
  subnets:
  - range: <IP_ADDRESS>/24
    gateway: <IP_ADDRESS>
    dns:  ["<IP_ADDRESS>"]
    az: default
- name: network2
  type: manual
  subnets:
  - range: <IP_ADDRESS>/24
    gateway: <IP_ADDRESS>
    dns:  ["<IP_ADDRESS>"]
    az: default

Manifest
instance_groups:
 ...
  azs:
  - default
  stemcell: default
  networks:
  - name: network1
    default: [dns, gateway]
  - name: network2

instances output

Instance
Process State
AZ
IPs

vmname/uuid
running
default
<IP_ADDRESS>

<IP_ADDRESS>

ip a output
Once I ssh, I only see one NIC, even though the bosh instances output says the VM has two IP addresses:
$ ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
336: we0crq5cpbmh-1@if337: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default qlen 1
    link/ether ce:98:fd:51:b9:3e brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 scope global we0crq5cpbmh-1
       valid_lft forever preferred_lft forever

director ip output
If I ssh into the director VM, I can see a lot of NICs without any IPs. Maybe this also has to do with the issue?
bosh/0:~$ ip a
1: lo: <LOOPBACK,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group default qlen 1
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/24 scope global lo:1
       valid_lft forever preferred_lft forever
2: eth0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
    link/ether 02:71:cd:85:96:e8 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eth0
       valid_lft forever preferred_lft forever
3: eth1: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc pfifo_fast state UP group default qlen 1000
    link/ether 02:23:6d:2b:d1:54 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eth1
       valid_lft forever preferred_lft forever
319: wbrdg-0af40400: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue state UP group default
    link/ether 5e:80:06:be:eb:31 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/24 scope global wbrdg-0af40400
       valid_lft forever preferred_lft forever
325: we0crq5cpbmb-0@if324: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue master wbrdg-0af40400 state UP group default qlen 1
    link/ether da:a5:1e:01:2b:3a brd ff:ff:ff:ff:ff:ff
329: we0crq5cpbmd-0@if328: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue master wbrdg-0af40400 state UP group default qlen 1
    link/ether 6a:90:4a:aa:67:c8 brd ff:ff:ff:ff:ff:ff
331: we0crq5cpbme-0@if330: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue master wbrdg-0af40400 state UP group default qlen 1
    link/ether f6:4e:17:c4:2c:45 brd ff:ff:ff:ff:ff:ff
333: we0crq5cpbmf-0@if332: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue master wbrdg-0af40400 state UP group default qlen 1
    link/ether b6:25:03:77:7f:8d brd ff:ff:ff:ff:ff:ff
335: we0crq5cpbmg-0@if334: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue master wbrdg-0af40400 state UP group default qlen 1
    link/ether 8a:33:3b:f5:14:fd brd ff:ff:ff:ff:ff:ff
337: we0crq5cpbmh-0@if336: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc noqueue master wbrdg-0af40400 state UP group default qlen 1
    link/ether 32:e0:a9:df:ca:de brd ff:ff:ff:ff:ff:ff

I was able to reproduce it on BOSH v268.2.x. Closing and re-opening to have a story created in our tracker.

Does VirtualBox show the VM as having multiple NICs as being attached? Sounds like the NIC doesn't show up physically in the VM which suggests it may be a bug in bosh-virtualbox-cpi-release?


ENH: use Pydantic for input validation on SobolEngine

POC showing how Pydantic could be used for input validation
cc @mdhaber
(Not adding it to the deps for now as to keep this simple.)

On the topic of automatic testing, I've found https://github.com/litestar-org/polyfactory/tree/main
I am not sure though. Seems like it still makes sense to use our usual testing strategy with pytest.raises(..., match=...).

I just had a quick look at what sklearn is doing (https://github.com/scikit-learn/scikit-learn/blob/main/sklearn/utils/_param_validation.py). I am not sure I like it as it really sounds like a rewrite of Pydantic (or attrs) in the end.

Would you mind trying it out, though? I think we knew that they do the same job, so they would be similar, but there seems to be some advantages.

Yep on it 👍


CDPS-916: Health history

Codecov Report
Attention: Patch coverage is 94.28571% with 4 lines in your changes missing coverage. Please review.

Project coverage is 85.67%. Comparing base (6757701) to head (0b68f80).
Report is 1 commits behind head on main.

Files
Patch %
Lines

...digital/hmpps/prisonperson/jpa/WithFieldHistory.kt
91.17%
1 Missing and 2 partials :warning:

...v/justice/digital/hmpps/prisonperson/jpa/Health.kt
94.73%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main      #71      +/-   ##
============================================
+ Coverage     85.50%   85.67%   +0.16%
- Complexity      462      471       +9
============================================
  Files            76       77       +1
  Lines          1373     1396      +23
  Branches        118      118
============================================
+ Hits           1174     1196      +22
  Misses          126      126
- Partials         73       74       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Tile formatting

Hi,
thank you for this extension, very helpful!
I'm facing an issue when editing JSON code in Tile view. When I enable the extension the JSON code is reset to default view.
I’ve tested this behavior on both Microsoft Edge (101.0.1210.53) and Google Chrome (102.0.5005.6) on different SharePoint website. Same with the VS extension.
Thank you for your support
Mark

Hi,
yeah, I was able to reproduce it, will work on a fix.

The fix is published and is currently going to the extension stores. Usually, it takes a few days until it will be published after the verification process. Thank you for reporting the issue!

it works like a charm on Google Chrome. Actually on Edge as well installing the chrome extension... 😉
Thank you

Just started using the extension and I love it! (Great work!!) But, I'm running into this same problem with Tile view formats. Using v0.1.2 of the extension and seeing the same thing in both Edge and Chrome. If I enable the extension and make a change or save the format, the view reverts to the default Tile/Gallery view. The custom JSON does remain in the editor until a refresh is performed, but has no affect on the actual view.
After refreshing the page, the view is still the default Tile view and when I select "Format custom view", I only see the default "Card Designer"/"Edit card" option from SP Online.
Please let me know if there's any other info I can provide to help identify what might be going on. Thanks!!

Hi,
yeah... seems something was changed again, will take a look when I have time.

Hi ,
have you had a chance to take a look at the issue? Running into the same problem with Version 0.1.2.
Thanks!
Florian

Hi, yeah, I took a look sometime ago, but haven't found the fix yet. Unfortunately, the trick, which worked previously, doesn't work right now.

After a few more attempts, I found a fix for the issue. It's already available for Chrome version 0.1.3 and will be soon available for Edge (it's under review, normally they publish in 24hrs).


[DocDB] MongoDB connection string should be displayed after creating a MongoDB instance

After running az documentdb create --kind MongoDB, the following info is displayed:
DatabaseAccountOfferType    DocumentEndpoint                          Kind     Location    Name     ProvisioningState    ResourceGroup
--------------------------  ----------------------------------------  -------  ----------  -------  -------------------  ---------------
Standard                    https://<NAME>.documents.azure.com:443/  MongoDB  West US     <NAME>  Succeeded            <GROUP>

It would be ideal if this info presented the MongoDB connection string, which is what would be immediately valuable for the user, as opposed to the *.documents.azure.com URL.

Aside from displaying this value immediately altering creating a MongoDB/DocDB instance, I'm not even sure whether you can acquire the MongoDB connection string via the Azure CLI right now. The list command doesn't seem to display it (even when looking at the JSON output), and I can't get the show command to work, so I don't know if it's available there.
Being able to grab this value is critical for completing the basic E2E of managing DocDB/MongoDB instances with the CLI.

@dmakwana Any updates here? Now that the MongoDB support is GA, it would be great to be able to provision an instance from the CLI and retrieve the right connection string, without needing to resort to using the portal.

Addressed in the PR #2580

This was addressed by adding the list-connection-strings command. As explained in #2661 we are not going to provide the keys in the show command to remain consistent.


Invisible Offline Players after server start or restart

Versions

Minecraft 1.18
EasyAuth 2.0.5
Fabric API 0.43.1

Describe the bug
Offline players are invisible on first join after server start.
To Reproduce
Steps to reproduce the behavior:

Start Server
Join server with Premium Account
Join server with Offline Account and /login psswd
Premium player can't see offline player (invisible)
Disconnect offline player
Reconnect offline player
Premium player now can see offline player

Happens with both players being offline too
Expected behavior
Players should be able to see each other on first join
Additional context
This three images are in the same place and first connection of every player after server start

Please check your hideUnauthenticatedPLayersFromPlayerList setting.
While in 1.17 it works fine, in 1.18 something changeв and that setting pretty unstable right now.

Yeah, it seems that solved it.

I will temporarily disable that feature


[IDE] Skip visiting constructor references when the decl is unknown

Skip visiting constructor references when the decl ends up being a nullptr. Only happens if the expression in CtorRefs ends up not being a DeclRefExpr after stripping off the (Identity|Try|Apply|AutoClosure)Expr's. I'm not sure when that would be, so for now I've left in an assert and skip the reference for release.
Resolves rdar://67412430

@swift-ci Please smoke test


remote execution from jdownload container

Hi,
is there a way to call the amc script from an other container?
When I was running downloader natively on my system, I was running a script after the extraction was finished.
This is how I use the event trigger by jd.
[{"eventTrigger":"ON_ARCHIVE_EXTRACTED", "enabled":true, "name":"FileBot", "script":"var amcFile = \"/volume1/video/filebot.sh\";var path = archive.getFolder(); callAsync(function() {}, amcFile, path);", "eventTriggerSettings":{}, "id":}]
And this is the scirpt I normally run.
#!/bin/sh
##FileBot benötigt diese Variablen
##/bin/mkdir /volume1/video/Downloads/videos/testordner
LOG="/volume1/amc.log"
EXC="/volume1/amc.txt"
SERIEN="/volume1/video/Serien/{n}/{s}/{n}.{s00e00} - {t} ({y}) {vf}"
FILME="/volume1/video/Filme/{n} ({y}) {source}({vf}@{vc}) [{rating}]/{n} ({y}) {source}({vf}@{vc}) [{rating}]"
##UNSORTIERT="/path/{file.structurePathTail}" ### Wenn Filebot mal etwas nicht zuordnen kann, dann kannst du es in einen seperaten Ordner verschieben lassen, dann bleibt es im Downloadordner übersichtlich.
##FileBot ausführen
/var/packages/filebot/target/filebot.sh -script fn:amc --lang de --log-file $LOG --action move "$1" --def "seriesFormat=$SERIEN" "movieFormat=$FILME" "animeFormat=$ANIME" "unsortedFormat=$UNSORTIERT" -non-strict --def unsorted=y "artwork=y" --def clean=y
## Aufräumen, --def root löscht den Unterordner mit
/var/packages/filebot/target/filebot.sh -script fn:cleaner "$1" --def root=y

This container supports automatic AMC invocation using a watch folder: https://github.com/jlesage/docker-filebot#automated-media-center-amc
Is this what you are looking for ?

Sorry for my late answers.
no that isn’t what I’m looking for.
As I was using the nativ installation, the script waits until the extraction is finished.
Afterwards the script calls filebot and gives the folder name to the script any only this folder is executed.
At the moment it is only possible to call the script to execute not to give the folder to run in.
pherhaps this an feature for a further release (node version)?
Cheers Robert

So you want to use the filebot binary inside the container like the native one you had ?  And you want filebot binary to be called from JDownloader running in another container ?  This doesn't seem possible.  However, you should be able to reproduce the same behaviour by using the watch folder functionality.

Closing this issue.  Please re-open if needed.


Enter text, delete, type more, all text reappears (macos)

Describe the bug
Enter some text, do not select a chip from the suggestions overlay, delete the text, type more text, now all text that was entered appears again.
Expected behavior
Would expect the text that was deleted (backspaced) to disappear forever and not come back.
Library version, Flutter version, platform
The "unreleased" main version of the library as of today, Flutter 3.7.7, on the macos platform. Also happens on Android if using the hardware keyboard instead of the emulator keyboard.

The entire setup of the TextInputClient and TextInputConnection is very fragile. Somehow on the browser the backspace works as expected and keeps the editing value updated, while on macos it does not. Fixed in commit 742398be


[Misc] - Additions from my own notes

I've added some of the things that I've been curating in similar list of my own.
This makes sense, afaik there are a lot of curated resources out there focus on Bitcoin in general, on users of Bitcoin, etc. but nothing specific for Bitcoin Core development. At least that was my experience and that's why I've initially started curating my own list of resources.
Also, just a personal opinion: I think that adding "users" to the scope of this curation widens too much its scope. Not sure what the end-goal is, but judging from your initial resources it seems that the target audience is Bitcoin Core contributors.

Thanks!


[RESOLVED] Interface eth0(): High error rate (>2 for 5m)

Problem started at 10:30:29 on 2023.06.06
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Operational data: errors in: 4, errors out: 0
Original problem ID: 23422
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23483&eventid=23422

Problem has been resolved in 18m 0s at 10:48:29 on 2023.06.06
Problem name: Interface eth0(): High error rate (>2 for 5m)
Host: S100-Primario
Severity: Warning
Original problem ID: 23422


add initial scheduler config

What type of PR is this?
/kind feature

What this PR does / why we need it:
Add initial scheduler configuration from file path taken from environment variable.
Which issue(s) this PR fixes:

Fixes #37
Special notes for your reviewer:
I've confirmed that it also worked when I passed configuration file path.
I'd like to learn more so correcting my every little mistake is welcome!
/label tide/merge-method-squash

/assign

/ok-to-test

And could you sign cla?
https://github.com/kubernetes-sigs/kube-scheduler-simulator/pull/80#issuecomment-1012050774

/test pull-kube-scheduler-simulator-run-tests

a

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please follow instructions at https://git.k8s.io/community/CLA.md#the-contributor-license-agreement to sign the CLA.
It may take a couple minutes for the CLA signature to be fully registered; after that, please reply here with a new comment and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address.  Check your existing CLA data and verify that your email is set on your git commits.
If you signed the CLA as a corporation, please sign in with your organization's credentials at https://identity.linuxfoundation.org/projects/cncf to be authorized.
If you have done the above and are still having issues with the CLA being reported as unsigned, please log a ticket with the Linux Foundation Helpdesk: https://support.linuxfoundation.org/
Should you encounter any issues with the Linux Foundation Helpdesk, send a message to the backup e-mail support address at<EMAIL_ADDRESS>

new comment

I signed it

I signed it.

/ok-to-test
/test pull-kube-scheduler-simulator-backend-unit-test

/test pull-kube-scheduler-simulator-backend-lint

/test pull-kube-scheduler-simulator-backend-lint
/test pull-kube-scheduler-simulator-backend-unit-test

/hold

/retitle support initial scheduler configuration option via an environment variable

/unhold


Matrix - draft yaml structure

Follow ups (not part of initial structure):

resource_detectors->attributes -> add type and description
metrics -add attributes
improve metadata bye requirements.

alternative version for settings, you can use code instead if env:
settings:
  - env: SPLUNK_TRACE_RESPONSE_HEADER_ENABLED
    code: alternatively something here
    description: Activated by default. Adds server trace information to HTTP response headers. The default value is `true`.
    default: 'true'
    type: boolean
    category: general

alternative way to define metrics by reference to upstream sem-conv:
      - metrics:
          - id: metric.http.server.request.duration
            sem-conv-href: hredhttps://github.com/open-telemetry/semantic-conventions/blob/v1.22.0/model/metrics/http.yaml

Merged in https://github.com/signalfx/splunk-otel-dotnet/pull/352


Add start_at: option to sequences

When using sequences I occasionally find myself wanting to start the sequence from a value other than zero. The solution I came up with was to change sequence/1 to work like this:
sequence("cameron", start_at: 1)
sequence({:name, & "cameron" <> &1}, start_at: 100)
It introduces a breaking change removing sequence/2 and using a tuple to pass a formatter instead. This seemed like the most idiomatic way while implementing it.
Pull request tested and ready if this would be a useful addition?

Thanks for bringing this up @camcaine. I usually think of using sequences when I want to make sure I don't run into uniqueness validations in my tests, but I haven't run into a scenario where I want to control the starting point.
Do you have a use case that could help me understand when you need to control the starting point?

We have a case where 0 is a default / fallback value with a special meaning, so it's a problem to have sequences start at 0.

It's been a while, but I recently looped back to this, and I realize the change is quite simple. If you're still interested in this, I opened PR for this work, and I'd love to hear your thoughts. https://github.com/thoughtbot/ex_machina/pull/414

It's everything I hoped to see :)


Convert primary keys from UUID to Bigint

Vampire-squid uses UUIDs as primary keys. This choice was made to avoid key clashes during database replication in MBARI's particular environment (2 ship databases, 1 shore database). UUID's as primary keys have the following short comings:

They are relatively large (128 bits) vs bitints (64 bits) so joins are a bit clunky
By default, if you use the schema generated by JPA, UUIDs are stored as CHAR(36). When you get into millions of records this results in terrible performance. SQL standards don't provide a standard way to store UUIDs. On SQL Server, we use the uniqueidentifier type. Which give vastly better performance without any code changes to vampire-squid.
Primary keys are usually the clustering key. This means when you insert a record the data is physically shuffled on disk so that they are sequentially ordered. MBARI uses a funky custom UUID to avoid page shuffling but this is not a cross-database solution.

SQL Server Replication supports using ints as primary keys and managing the key clashes. So the goal of this issue is to keep UUID's as surrogate keys, but use an id field that is a bigint for primary clustering key and joins.

The issue_1 branch contains this replacement. At MBARI I used the m3_migrate_media.py script to move all video information from our production database, which uses UUIDs (unigueidentifier) as PKs, to a derby database that uses bigint as PKs. That same script can be used to migrate media from the production database to a new one that uses bigint pks. Neil is out for a few weeks. When he returns, I'll work with him to migrate the database.

Important! The m3_migrate_media.py script did not preserve the uuids. Alter the script to preserve those before running this in production.

Closing this issue. No plans to migrate uuids to big ints anymore.


Restrict '+ Dataset' button to admins

SUMMARY

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

I'm ok with this change, but it is very much a Band Aid. It will prevent users from falling into this trap, but the issue also affects Admins - I was able to reproduce it last time a user reported this. So I think this still warrants more investigation...


ENYO-3319 : Delegate focusType property to onSpotFocused

Issue
:focusType info is not delagate until onSpotlightFocused handler
Fix
:delegate focusType info from onSpotlightFocus to onSpotlightFocused

Closing PR as Jira ticket is closed.


Can't define variables in function

It looks like if you define variables in a function, then they can't be referenced outside the function.  This makes code like the following not work:
function Integer[] cumsum(Integer[] xs) {
         acc = 0
         for (ix in 1:3) {
             acc = acc + xs[ix]
             ys[ix] = acc
         }
         return ys
}

While it would be ideal if such things would work, it might be OK to just document what you can and cannot expect from functions.

This is sort of odd, because I get the same error if I save rb-command.Rev in a regular text editor and try running it with the eb executable at the cmd line. But if I run cell-by-cell in a Jupyter notebook, I get the expected output. But then if I run it line-by-line in the revbayes exectuteable, I still get the Variable ys[1] does not exist. I'm not sure what's going on here.

This is sort of odd, because I get the same error if I save rb-command.Rev in a regular text editor and try running it with the eb executable at the cmd line. But if I run cell-by-cell in a Jupyter notebook, I get the expected output. But then if I run it line-by-line in the revbayes exectuteable, I still get the Variable ys[1] does not exist. I'm not sure what's going on here.


Migrate primary branch name from master => main

Will need to update GitHub actions, primarily.

@AlexeiShmidman do you have any concerns with me doing this? I will leave master in place for a while, but will move the Actions to use main.


fix: fix ALICLOUD_ACCESS_KEY_SECRET and support ALICLOUD_STS_TOKEN

Codecov Report

Merging #69 (f8e62f5) into master (29b0eb0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #69   +/-   ##
=======================================
  Coverage   95.22%   95.22%
=======================================
  Files           7        7
  Lines         649      649
=======================================
  Hits          618      618
  Misses         31       31

Impacted Files
Coverage Δ

lib/config.js
87.80% <ø> (ø)

Continue to review full report at Codecov.

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


sharees API don't return shareType 4 if user is guest (user role guest)

Describe the bug
shareType is 0 but should be 4 if the user is a guest or has the guest role, as spoken with @butonic

@hurradieweltgehtunter let me clarify the expectations:
The "guest" user is a user type property which cannot be set currently from the frontend. So we should think of renaming the role "guest" to "Contributor" to avoid naming clashes and wrong expectation.

I highly agree with the Contributor role and we should make this change ASAP before we introduce real guest users

Wait, why Contributor?
Looking at share roles (instead of share types), we already have a link sharing role Contributor for public links. So that would be a name clash and probably confusing.
Apart from that we already made the experience that Contributor and Editor link sharing roles are a frequent source of confusion, because nobody understands from the title what the difference is. I would suspect, that having a Contributor share type (now talking about share types again) causes similar lack of intuitive understanding.

Agreed.
The issue is the naming of the "Guest"  Role.
Suggesttions?

As discussed we don't need this role in the project and therefore it can be omitted

Clarified in a call:

the "Guest" role and the "guest" user flag are semantically connected, but the single source of truth is the ldap guest user flag. It should not be derived from the Guest role.
whether or not a user is a guest is defined by how the user comes into the system (managed in LDAP by sysadmin vs. invited via sharing)
a guest user doesn't have a user role. Therefore we will delete the Guest user role in our set of default roles!

@micbar has the guest role removed? if yes, feel free to close the issue

fixed by https://github.com/owncloud/ocis/pull/6456


Invalid literal for int: error while solving MPS files with longer column names

I'm facing an error when running linopy on MPS files where column names have more than one letter in them (e.g. col01 as opposed to x01). Here is an example MPS file:
NAME        sample_mip
ROWS
 N  obj
 G  c1
 L  c2
 E  c3
COLUMNS
    col1        obj       5
    col1        c1        2
    col1        c2        4
    col1        c3        1
    MARK0000  'MARKER'                 'INTORG'
    col2        obj       3
    col2        c1        3
    col2        c2        2
    col2        c3        1
    col3        obj       7
    col3        c1        4
    col3        c2        3
    col3        c3        1
    MARK0001  'MARKER'                 'INTEND'
RHS
    RHS_V     c1        12
    RHS_V     c2        15
    RHS_V     c3        6
BOUNDS
 UP BOUND     col1        4
 UI BOUND     col2        3
 UI BOUND     col3        5
ENDATA

I run this using the python script
import sys
from pathlib import Path
from linopy import solvers

s = solvers.Highs()
result = s.solve_problem(problem_fn=Path(sys.argv[1]), solution_fn=Path("/tmp/sid-linopy-out.sol"))
print(result)
print(result.solution.objective)

Which results in the following error (a similar error appears when using GLPK/Gurobi solvers):
Traceback (most recent call last):
  File ".../linopy-test.py", line 6, in <module>
    result = s.solve_problem(problem_fn=Path(sys.argv[1]), solution_fn=Path("/tmp/sid-linopy-out.sol"))
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/linopy/solvers.py", line 299, in solve_problem
    return self.solve_problem_from_file(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/linopy/solvers.py", line 775, in solve_problem_from_file
    return self._solve(
           ^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/linopy/solvers.py", line 867, in _solve
    solution = self.safe_get_solution(status=status, func=get_solver_solution)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/linopy/solvers.py", line 224, in safe_get_solution
    return func()
           ^^^^^^
  File ".../lib/python3.12/site-packages/linopy/solvers.py", line 860, in get_solver_solution
    ).pipe(set_int_index)
      ^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/pandas/core/generic.py", line 6231, in pipe
    return common.pipe(self, func, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/pandas/core/common.py", line 502, in pipe
    return func(obj, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/linopy/solvers.py", line 138, in set_int_index
    series.index = series.index.str[1:].astype(int)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/pandas/core/indexes/base.py", line 1097, in astype
    new_values = astype_array(values, dtype=dtype, copy=copy)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/pandas/core/dtypes/astype.py", line 182, in astype_array
    values = _astype_nansafe(values, dtype, copy=copy)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File ".../lib/python3.12/site-packages/pandas/core/dtypes/astype.py", line 133, in _astype_nansafe
    return arr.astype(dtype, copy=True)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'ol1'

I first noticed this error when running on the following JuMP-HiGHS benchmark: Sienna_modified_RTS_GMLC_DA_sys_NetDC_Horizon12_Day332
Could it be that linopy is expecting all column names to look like \w\d+? @FabianHofmann 🙏

hey sid, you are exactly right! perhaps we need a pre-check on long the prefix is? we could use something like:
def count_initial_letters(word):
    count = 0
    for char in word:
        if char.isalpha():
            count += 1
        else:
            break
    return count

Run that once, just before https://github.com/PyPSA/linopy/blob/e5cc814959864f6ed35521ae4cd93bdf974d476b/linopy/solvers.py#L141
I am proposing a change right now

Thanks, Fabian, for the super quick response and fix! Btw, does the MPS format guarantee that all columns are named with the same prefix followed by sequential integers?

@siddharth-krishna I cannot say for sure, I would just say that another format would not make sense. But if we should extend the pre-processing, let me know. If the fix works and you are fine with it, we can merge it it.

Btw, does the MPS format guarantee that all columns are named with the same prefix followed by sequential integers?

It most surely does not!!!
See

https://lpsolve.sourceforge.net/5.5/mps-format.htm
https://www.ibm.com/docs/en/cofz/12.9.0?topic=formats-working-mps-files

Pretty much any string is allowed. There are two MPS formats, free and fixed. In the free format, anything except ' ' and *  are allowed. In the fixed format, names are based on the column of each row, and names are theoretically white-space sensitive, except that most optimizers will probably complain.
You cannot infer the integer index of a column or row based on the name.
Rows are provided in the order of the ROWS section, and columns are ordered (most commonly) based on their order in the COLUMS section, with the complication that some variables may appear in other sections like BOUNDS without appearing in COLUMNS.


Update LICENSE

Summary

Fixes
Depends on
Test plan

Screenshots

Checklist

[ ] My code follows the code style of this project and passes make format.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] My change has adequate unit test coverage.

The changes in the LICENSE file look good. Thank you, @kushal34712, for updating the license information.


Add ability to import libraries on top of the translations file in an es6 way

In order to use rails with webpack and the i18n-js npm package all together, import I18n from 'i18n-js'; should be added to the resultant translations.js file.
This PR allows an user to add the import programatically just adding the option in the i18n-js.yml file.
Thanks @PikachuEXE for your package, I hope you find this option useful 😄

I currently use https://github.com/usabilityhub/rails-erb-loader:
// i18n-js.js.erb or i18n-js-ts.erb
<%# encoding: UTF-8 %>

// Optional dependency watching
// Tell webpack to watch these files
// https://github.com/usabilityhub/rails-erb-loader#dependencies
/* rails-erb-loader-dependencies ./../config/locales/ */

import I18n from "i18n-js"

I18n.translations = <%= I18n::JS.filtered_translations.to_json %>;

export default I18n

My erb loader file:
module.exports = {
  test:     /\.erb$/,
  enforce:  "pre",
  exclude:  /node_modules/,

  use: [{
    loader:   "rails-erb-loader",
    options:  {
      runner:     (/^win/.test(process.platform) ? "ruby " : "") + "bin/rails runner",
      env:        {
        ...process.env,
        DISABLE_SPRING: 1,
        RAILS_CONFIG_EAGER_LOAD_ENABLED_IN_DEVELOPMENT: "false",
      },
    },
  }],
}

I really like your solution @PikachuEXE , but you need an extra package and change the options in webpacker, what do you think about the solution I offer in this PR? The new option in the yml file is completely optional.

LGTM

Still pending change from @mcamara though

@PikachuEXE done :)

Travis tests have failed
Hey @mcamara,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
Ruby: 2.5
View build log

     rake

Using bundler 1.17.3
Using thor 0.20.3
Using json 2.2.0
Using docile 1.3.2
Using simplecov-html 0.10.2
Using tins 1.21.1
Using gem-release 2.0.3
Using i18n 0.7.0
Using diff-lcs 1.3
Using appraisal 2.2.0
Using simplecov 0.16.1
Using term-ansicolor 1.7.1
Using i18n-js 3.3.0 from source at `../`
Fetching rspec-support 3.9.0
Using coveralls 0.8.23
Installing rspec-support 3.9.0
Fetching rspec-core 3.9.0
Fetching rspec-expectations 3.9.0
Installing rspec-expectations 3.9.0
Installing rspec-core 3.9.0
Fetching rspec-mocks 3.9.0
Installing rspec-mocks 3.9.0
Fetching rspec 3.9.0
Installing rspec 3.9.0
Bundle complete! 7 Gemfile dependencies, 20 gems now installed.
Bundled gems are installed into `./gemfiles/vendor/bundle`
travis_time:end:021db642:start=1570611323856195776,finish=1570611326931619234,duration=3075423458,event=install
travis_fold:end:install.bundler

travis_time:start:10bb8aec
$ bundle exec rake
/home/travis/.rvm/rubies/ruby-2.5.5/bin/ruby -I/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/gems/rspec-core-3.9.0/lib:/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/gems/rspec-support-3.9.0/lib /home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/gems/rspec-core-3.9.0/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb
[Coveralls] Set up the SimpleCov formatter.
[Coveralls] Using SimpleCov's 'rails' settings.
.........................................................................F............................

Failures:

  1) I18n::JS namespace, prefix and pretty_print options exports with defined locale as fallback when enabled
     Failure/Error: erb_result_from_yaml_file = ERB.new(File.read(config_file_path)).result

     Errno::ENOENT:
       No such file or directory @ rb_sysopen - /home/travis/build/fnando/i18n-js/spec/fixtures/js_file_with_namespace_import_and_pretty_print.yml
     # ./lib/i18n/js.rb:112:in `read'
     # ./lib/i18n/js.rb:112:in `config'
     # ./lib/i18n/js.rb:247:in `export_i18n_js_dir_path'
     # ./lib/i18n/js.rb:235:in `export_i18n_js'
     # ./lib/i18n/js.rb:43:in `export'
     # ./spec/ruby/i18n/js_spec.rb:362:in `block (3 levels) in <top (required)>'

Finished in 0.17968 seconds (files took 0.22494 seconds to load)
102 examples, 1 failure

Failed examples:

rspec ./spec/ruby/i18n/js_spec.rb:361 # I18n::JS namespace, prefix and pretty_print options exports with defined locale as fallback when enabled

[Coveralls] Submitting to https://coveralls.io/api/v1
[Coveralls] Job #616.24
[Coveralls] https://coveralls.io/jobs/54195942
Coverage is at 67.98%.
Coverage report sent to Coveralls.
/home/travis/.rvm/rubies/ruby-2.5.5/bin/ruby -I/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/gems/rspec-core-3.9.0/lib:/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/gems/rspec-support-3.9.0/lib /home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/gems/rspec-core-3.9.0/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb failed
travis_time:end:10bb8aec:start=1570611326936774567,finish=1570611329403220626,duration=2466446059,event=
The command "bundle exec rake" exited with 1.
travis_fold:start:cache.2
store build cache
$ bundle clean
Removing rspec-expectations (3.8.4)
Removing rspec (3.8.0)
Removing rspec-mocks (3.8.1)
Removing rspec-support (3.8.2)
Removing rspec-core (3.8.2)
travis_time:start:08443fe2
travis_time:end:08443fe2:start=1570611329638933680,finish=1570611329642578021,duration=3644341,event=cache
travis_time:start:2ffe98cc
changes detected (content changed, file is created, or file is deleted):\n/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-3.8.0.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-3.9.0.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-core-3.8.2.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-core-3.9.0.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-expectations-3.8.4.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-expectations-3.9.0.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-mocks-3.8.1.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-mocks-3.9.0.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-support-3.8.2.gem
/home/travis/build/fnando/i18n-js/gemfiles/vendor/bundle/ruby/2.5.0/cache/rspec-support-3.9.0.gem
/home/travis/build/fnando/i18n-js/ge\n...
changes detected, packing new archive
uploading cache-linux-xenial-e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855--rvm-2.5--gemfile-gemfiles/i18n_0_7.gemfile.tgz
cache uploaded
travis_time:end:2ffe98cc:start=1570611329647236389,finish=1570611331467281638,duration=1820045249,event=cache
travis_fold:end:cache.2

Done. Your build exited with 1.

TravisBuddy Request Identifier: 638af320-ea73-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 67b502e0-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 6a66dc70-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 89671c20-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 916e50f0-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 923ceb90-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 98175550-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 990c8ac0-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: c0a73e90-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: c18df510-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: c6ad4c30-ea75-11e9-90a0-7556777797a6

Hey @mcamara,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: d96f8b30-ea75-11e9-90a0-7556777797a6

Thanks
Releasing next week if no other urgent pending changes

Released in 3.4.0


unexpected results with argmax

Seems that argmax is incorrect in some cases when pulling vectors out of a matrix. For example:
scala> val m = DenseMatrix.zeros[Double](3,3)
scala> m(2,::) := DenseVector(1.0,2.0,3.0).t
scala> argmax(m(2, ::).t)
res2: Int = 0
scala> max(m(2, ::).t)
res7: Double = 3.0

But it works with a normal DenseVector, or if you change to an array:
scala> argmax(DenseVector(1.0,2.0,3.0))
res8: Int = 2
scala> argmax(m(2, ::).t.toArray)
res11: Int = 2

This is with breeze 0.10

(fixed in master)


[BUG] Building wheel with ext_modules on Windows 10 with VC throws linker error

setuptools version
63.2.0
Python version
3.10
OS
Windows 10 Pro 64-bit
Additional environment information
Ran from within an x64 Native Tools Command Prompt for VS 2022 terminal; using VS 2022 pro.
Venv setup:
(venv) C:\Users\me\source\setuptools-test>pip freeze
build==1.0.3
colorama==0.4.6
packaging==23.2
pyproject_hooks==1.0.0
tomli==2.0.1

Description
I have a basic package as follows:
root/
├── setup.py
└── src
    └── foo
        └── bar.c

Content of setup.py:
from setuptools import Extension, setup, find_packages
setup(
    name="foo",
    version="0.0",
    packages=find_packages(where="src"),
    ext_modules=[
        Extension(
            name="foo.bar",
            sources=["src/foo/bar.c"],
        ),
    ],
)

Content of bar.c:
int foobar(int x)
{
    return x*2;
}

Now, the first time I run the command to build a wheel python -m build --wheel I get a linker error LINK : error LNK2001: unresolved external symbol PyInit_bar (full output below), but if I immediately rerun this command the command succeeds (Successfully built foo-0.0-cp310-cp310-win_amd64.whl) but on pip installing the resulting wheel it throws an import error that seems to point to the heart of the issue
(venv) C:\Users\me\source\setuptools-test>python -c "import foo.bar"
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ImportError: DLL load failed while importing bar: %1 is not a valid Win32 application.

It seems as though, despite the fact that I'm running completely within the x64 Native Tools Command Prompt and using a 64-bit python distro (confirmed via this method: python -c "import sys;print(\"%x\" % sys.maxsize, sys.maxsize > 2**32)" -->
7fffffffffffffff True), the python package is expecting the compiled extension to be 32-bit?
The issue I'm seeing looks to be almost identical to this SO question:, however I am indeed running from within a 64-bit VS environment against a confirmed 64-bit version of python.
(Note I also tried using the python setup.py bdist_wheel command in case it was an issue with the build wrapper specifically, but I get the same results as well as a deprecation warning about using setup.py)
Any help/suggestions would be greatly appreciated if I'm overlooking something simple.
Expected behavior
I would expect that running python -m build --wheel with 64-bit python from a 64-bit VS terminal will compile the extension as 64-bit and that the installed wheel will then expect a 64-bit DLL for the module.
How to Reproduce

Set up the directory structure as above
Open an x64 Native Tools Command Prompt for VS 2022 terminal
Create a virtual environment that has build installed
Run python -m build --wheel <--first time throws linker error
Run python -m build --wheel <--second time apparently succeeds
Pip install the wheel foo-0.0-cp310-cp310-win_amd64.whl
Run command python -c "import foo.bar" <-- throws import error %1 is not a valid Win32 application

Note that I am seeing the exact same linker error when running this same process on github actions' default windows-latest runner.
Output

Output from first python -m build --wheel command:
(venv) C:\Users\me\source\setuptools-test>python -m build --wheel
* Creating venv isolated environment...
* Installing packages in isolated environment... (setuptools >= 40.8.0, wheel)
* Getting build dependencies for wheel...
running egg_info
creating foo.egg-info
writing foo.egg-info\PKG-INFO
writing dependency_links to foo.egg-info\dependency_links.txt
writing top-level names to foo.egg-info\top_level.txt
writing manifest file 'foo.egg-info\SOURCES.txt'
reading manifest file 'foo.egg-info\SOURCES.txt'
writing manifest file 'foo.egg-info\SOURCES.txt'
* Installing packages in isolated environment... (wheel)
* Building wheel...
running bdist_wheel
running build
running build_ext
building 'foo.bar' extension
creating build
creating build\temp.win-amd64-cpython-310
creating build\temp.win-amd64-cpython-310\Release
creating build\temp.win-amd64-cpython-310\Release\src
creating build\temp.win-amd64-cpython-310\Release\src\foo
"C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\bin\HostX86\x64\cl.exe" /c /nologo /O2 /W3 /GL /DNDEBUG /MD -IC:\Users\me\AppData\Local\Temp\build-env-d_11j2_6\include -IC:\Users\me\AppData\Local\Programs\Python\Python310\include -IC:\Users\me\AppData\Local\Programs\Python\Python310\Include "-IC:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\include" "-IC:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\ATLMFC\include" "-IC:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Auxiliary\VS\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\um" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\shared" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\winrt" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\cppwinrt" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.7.2\include\um" "-IC:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\include" "-IC:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\ATLMFC\include" "-IC:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Auxiliary\VS\include" "-IC:\Program Files (x86)\Windows Kits\10\include\10.0.22621.0\ucrt" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\um" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\shared" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\winrt" "-IC:\Program Files (x86)\Windows Kits\10\\include\10.0.22621.0\\cppwinrt" "-IC:\Program Files (x86)\Windows Kits\NETFXSDK\4.7.2\include\um" /Tcsrc/foo/bar.c /Fobuild\temp.win-amd64-cpython-310\Release\src/foo/bar.obj
bar.c
creating C:\Users\me\source\setuptools-test\build\lib.win-amd64-cpython-310
creating C:\Users\me\source\setuptools-test\build\lib.win-amd64-cpython-310\foo
"C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\bin\HostX86\x64\link.exe" /nologo /INCREMENTAL:NO /LTCG /DLL /MANIFEST:EMBED,ID=2 /MANIFESTUAC:NO /LIBPATH:C:\Users\me\AppData\Local\Temp\build-env-d_11j2_6\libs /LIBPATH:C:\Users\me\AppData\Local\Programs\Python\Python310\libs /LIBPATH:C:\Users\me\AppData\Local\Programs\Python\Python310 /LIBPATH:C:\Users\me\AppData\Local\Temp\build-env-d_11j2_6\PCbuild\amd64 "/LIBPATH:C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\ATLMFC\lib\x64" "/LIBPATH:C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\lib\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.7.2\lib\um\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.22621.0\ucrt\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\\lib\10.0.22621.0\\um\x64" "/LIBPATH:C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\ATLMFC\lib\x64" "/LIBPATH:C:\Program Files\Microsoft Visual Studio\2022\Professional\VC\Tools\MSVC\14.37.32822\lib\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.7.2\lib\um\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.22621.0\ucrt\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\\lib\10.0.22621.0\\um\x64" /EXPORT:PyInit_bar build\temp.win-amd64-cpython-310\Release\src/foo/bar.obj /OUT:build\lib.win-amd64-cpython-310\foo\bar.cp310-win_amd64.pyd /IMPLIB:build\temp.win-amd64-cpython-310\Release\src/foo\bar.cp310-win_amd64.lib
LINK : error LNK2001: unresolved external symbol PyInit_bar
build\temp.win-amd64-cpython-310\Release\src/foo\bar.cp310-win_amd64.lib : fatal error LNK1120: 1 unresolved externals
error: command 'C:\\Program Files\\Microsoft Visual Studio\\2022\\Professional\\VC\\Tools\\MSVC\\14.37.32822\\bin\\HostX86\\x64\\link.exe' failed with exit code 1120

ERROR Backend subprocess exited when trying to invoke build_wheel

Output from second python -m build --wheel command:
(venv) C:\Users\me\source\setuptools-test>python -m build --wheel
* Creating venv isolated environment...
* Installing packages in isolated environment... (setuptools >= 40.8.0, wheel)
* Getting build dependencies for wheel...
running egg_info
writing foo.egg-info\PKG-INFO
writing dependency_links to foo.egg-info\dependency_links.txt
writing top-level names to foo.egg-info\top_level.txt
reading manifest file 'foo.egg-info\SOURCES.txt'
writing manifest file 'foo.egg-info\SOURCES.txt'
* Installing packages in isolated environment... (wheel)
* Building wheel...
running bdist_wheel
running build
running build_ext
installing to build\bdist.win-amd64\wheel
running install
running install_lib
creating build\bdist.win-amd64
creating build\bdist.win-amd64\wheel
creating build\bdist.win-amd64\wheel\foo
copying build\lib.win-amd64-cpython-310\foo\bar.cp310-win_amd64.pyd -> build\bdist.win-amd64\wheel\.\foo
running install_egg_info
running egg_info
writing foo.egg-info\PKG-INFO
writing dependency_links to foo.egg-info\dependency_links.txt
writing top-level names to foo.egg-info\top_level.txt
reading manifest file 'foo.egg-info\SOURCES.txt'
writing manifest file 'foo.egg-info\SOURCES.txt'
Copying foo.egg-info to build\bdist.win-amd64\wheel\.\foo-0.0-py3.10.egg-info
running install_scripts
C:\Users\me\AppData\Local\Temp\build-env-qhg0holo\lib\site-packages\wheel\bdist_wheel.py:108: RuntimeWarning: Config variable 'Py_DEBUG' is unset, Python ABI tag may be incorrect
  if get_flag("Py_DEBUG", hasattr(sys, "gettotalrefcount"), warn=(impl == "cp")):
creating build\bdist.win-amd64\wheel\foo-0.0.dist-info\WHEEL
creating 'C:\Users\me\source\setuptools-test\dist\.tmp-qy_o0f31\foo-0.0-cp310-cp310-win_amd64.whl' and adding 'build\bdist.win-amd64\wheel' to it
adding 'foo/bar.cp310-win_amd64.pyd'
adding 'foo-0.0.dist-info/METADATA'
adding 'foo-0.0.dist-info/WHEEL'
adding 'foo-0.0.dist-info/top_level.txt'
adding 'foo-0.0.dist-info/RECORD'
removing build\bdist.win-amd64\wheel
Successfully built foo-0.0-cp310-cp310-win_amd64.whl

Hi @charelstoncrabb, thanks for reporting the issue. Please see my comments below:

Regarding the difference between the first run and second run that probably is explained by some caching and/or maybe the linker producing a file even if it fails. Probably that could be improved if we have a contributor willing to tackle it, but I think we can agree this is not the core of the error. It should be easy to remove the cached files with git clean -f -d, as long as all the changes that matter have been properly committed into git.

In the error message ImportError: DLL load failed while importing bar: %1 is not a valid Win32 application., the term "Win32" does not necessarily imply that you are using a 32-bit architecture for the compilation... Microsoft has this confusing nomenclature in which "Win32" is actually the name of a Windows API that is used for both 32-bit and 64-bit environments. See https://learn.microsoft.com/en-us/windows/win32/api/ and https://learn.microsoft.com/en-us/windows/win32/apiindex/windows-api-list.

The error message LINK : error LNK2001: unresolved external symbol PyInit_bar is more significative for debugging. And this happens because Python expects some specific contents inside of the .c file. For example, Python expects you to define PyInit_bar if you want to be able to import bar. Since your .c file is very simplified and does not contain such definitions, the error that you are seeing is normal. See Python docs on extensions: https://docs.python.org/3/extending/building.html.

Could you please confirm that after changing bar.c to have the appropriate C code as expected by Python the compilation works and the module can be imported correctly?

Hi @abravalheri , thank you very much for your detailed response!

Can confirmed that indeed using the --wheel flag keeps the build folder on disk, and without it it fails every time 👍
One of a loong list of reasons I cringe every time I have to deal with Windows. The links are helplful, thank you
This is it. I overlooked the fact that PyInit_ has to be implemented.

Thank you very much for your help


awscli: 1.19.39 -> 1.19.40

Motivation for this change
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 118082 at 5931d2e2 run on aarch64-linux 1

10 packages marked as broken and skipped:

awsebcli
mlflow-server
python38Packages.cartopy
python38Packages.flower
python38Packages.geopandas
python38Packages.osmnx
python39Packages.cartopy
python39Packages.flower
python39Packages.geopandas
python39Packages.osmnx

2 packages failed to build:

python38Packages.certbot-dns-route53
python39Packages.certbot-dns-route53

10 packages skipped due to time constraints:

mapproxy
python38Packages.fiona
python38Packages.labelbox
python38Packages.rasterio
python38Packages.tensorboardx
python39Packages.fiona
python39Packages.labelbox
python39Packages.rasterio
python39Packages.tensorboardx
tts

110 packages built successfully:

aws-adfs (python38Packages.aws-adfs)
aws-google-auth
aws-mfa
aws-sam-cli
awscli
awscli2
awslogs
barman
checkov
cloud-custodian
credstash (python38Packages.credstash)
deja-dup
duplicity
duply
dvc-with-remotes
elasticsearch-curator
git-remote-codecommit
home-assistant
iceshelf
lexicon
luigi
nixops
python38Packages.aiobotocore
python38Packages.aws-sam-translator
python38Packages.aws-xray-sdk
python38Packages.boto3
python38Packages.botocore
python38Packages.celery
python38Packages.cfn-lint
python38Packages.chalice
python38Packages.django-anymail
python38Packages.django-raster
python38Packages.djmail
python38Packages.ec2instanceconnectcli
python38Packages.flowlogs_reader
python38Packages.fs-s3fs
python38Packages.gensim
python38Packages.gradient
python38Packages.hass-nabucasa
python38Packages.kombu
python38Packages.mapbox
python38Packages.moto
python38Packages.pycognito
python38Packages.pynamodb
python38Packages.s3fs
python38Packages.s3transfer
python38Packages.sagemaker
python38Packages.sapi-python-client
python38Packages.scrapy
python38Packages.scrapy-deltafetch
python38Packages.scrapy-fake-useragent
python38Packages.scrapy-splash
python38Packages.serverlessrepo
python38Packages.slack-sdk
python38Packages.slicedimage
python38Packages.smart_open
python38Packages.snowflake-connector-python
python38Packages.snowflake-sqlalchemy
python38Packages.warrant
python39Packages.aiobotocore
python39Packages.aws-adfs
python39Packages.aws-sam-translator
python39Packages.aws-xray-sdk
python39Packages.boto3
python39Packages.botocore
python39Packages.celery
python39Packages.cfn-lint
python39Packages.chalice
python39Packages.credstash
python39Packages.django-anymail
python39Packages.django-raster
python39Packages.djmail
python39Packages.ec2instanceconnectcli
python39Packages.flowlogs_reader
python39Packages.fs-s3fs
python39Packages.gradient
python39Packages.hass-nabucasa
python39Packages.kombu
python39Packages.mapbox
python39Packages.moto
python39Packages.pycognito
python39Packages.pynamodb
python39Packages.s3fs
python39Packages.s3transfer
python39Packages.sagemaker
python39Packages.sapi-python-client
python39Packages.scrapy
python39Packages.scrapy-deltafetch
python39Packages.scrapy-fake-useragent
python39Packages.scrapy-splash
python39Packages.serverlessrepo
python39Packages.slack-sdk
python39Packages.slicedimage
python39Packages.snowflake-connector-python
python39Packages.snowflake-sqlalchemy
python39Packages.warrant
s3bro
s4cmd
snakemake
sourcehut.buildsrht
sourcehut.dispatchsrht
sourcehut.gitsrht
sourcehut.hgsrht
sourcehut.hubsrht
sourcehut.listssrht
sourcehut.mansrht
sourcehut.metasrht
sourcehut.pastesrht
sourcehut.todosrht
streamlit

2 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/boto3/default.nix:40:3:
   |
40 |   meta = {
   |   ^

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/botocore/default.nix:43:3:
   |
43 |   meta = with lib; {
   |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement.

N.b.: certbot failure is fixed by https://github.com/NixOS/nixpkgs/pull/117734, which could use another review from any interested parties.


Recognize image/jpg as MimeType::Jpeg

With this patch, when MIME type is image/jpg, Lofty will recognize it as MimeType::Jpeg too, instead of falling back to Unknown.
I tested this patch with Amberol (Which uses MimeType:: enums to decide whether to show album arts) and now it displays JPG album arts just fine.
In my opinion, I don't see a reason for creating a MimeType::Jpg enum, since actually JPG and JPEG are the same thing but with a different extension.

Thanks!


Fix auto-translations in editor

This PR enables auto-translation of Windows and Controls inside the editor, if they are not part of the edited scene.
Fixes #50650
Technically TTR() can now be retired in favor of TTRC(), probably.
I use get_viewport() != get_tree()->get_root() to determine whether a node is part of the scene tree, but this might yield some false-positives. A more reliable condition is
if (Engine::get_singleton()->is_editor_hint() && get_tree()->get_edited_scene_root() &&
		(get_tree()->get_edited_scene_root()->get_viewport() == get_viewport() || get_tree()->get_edited_scene_root()->get_viewport()->is_ancestor_of(get_viewport())))

and for Window
if (Engine::get_singleton()->is_editor_hint() && get_tree()->get_edited_scene_root() &&
		(get_tree()->get_edited_scene_root()->get_viewport() == get_parent_viewport() || get_tree()->get_edited_scene_root()->get_viewport()->is_ancestor_of(get_parent_viewport()))) {

but not sure if it's necessary 🤔

I use get_viewport() != get_tree()->get_root() to determine whether a node is part of the scene tree, but this might yield some false-positives

Right. Wouldn't it also trigger in projects whenever you have multiple Window nodes or controls inside of windows, dialogs, or popups? I think a TOOLS_ENABLED injection should be okay here. We already have it in Controls at least. See get_parent_anchorable_rect, which relies on something similar.

No this PR affects only the editor. False-positives is about nodes that are part of the editor, but get detected as scene nodes and don't not translated. It can happen if some node is not part of the main viewport.

Ah, right, you guard with Engine::get_singleton()->is_editor_hint(). Well, since we already do TOOLS_ENABLED checks, we could also exclude this code at compile time. And then you would still reduce the chance of error if you check for the edited root's viewport, no?

Ok added TOOLS_ENABLED check.
Also I switched to the complex condition, because buttons in dialogs are not under main viewport 🤦♂ My only concern is the is_ancestor_of() check, but it's 100% accurate now at least.

My only concern is the is_ancestor_of() check, but it's 100% accurate now at least.

Similar conditions are already used in several other nodes ;)
https://github.com/godotengine/godot/blob/ce90d77a1857e9cee4b7571258d6e917bf4ba268/scene/main/timer.cpp#L38
https://github.com/godotengine/godot/blob/ce90d77a1857e9cee4b7571258d6e917bf4ba268/scene/gui/menu_bar.cpp#L252
I think we can turn this logic into a method of Node.

I added is_part_of_edited_scene() method and used it where applicable.

Thanks!


Question: Does black have a special role in generated tasks?

Looking at the original tasks and those generated by re-arc, it seems that in ARC the background colour is usually black and that there's an implicit quality of black that it represents empty space. This doesn't seem to be the case in re-arc. Is there an easy way to create that constraint?

The idea was to have no bias against or for certain symbols, unless of course a given task logic demands it, so: no, at least not in a straight-forward manner at generation time.
Oftentimes, the background symbol is sampled from a set of eligible colors. I typically used the variable name bgc for storing the symbol that corresponds to the background color. bgc = choice( occurs 205 times in generators.py. Hence, modifying all those to bgc = 0, where 0 is mapped to black, would probably work for a majority of those tasks, but does not seem like a robust / exhaustive solution.
What I would suggest instead is some post-generation trickery. What may work and would be efficient in terms of generating not too many throw-away examples would be to do a color permutation s.t. the most common (or whichever other notion defining "background color" suitable for your use case) color maps to black and then re-running the verifier on that example and only keeping it if the verifier still works. But that is probably also not super robust in and of itself.
I think the easiest way to achieve having only examples with a black background would be to simply filter out all generated examples which do not have a black background color. Note however that this may not work for all tasks, since a task may have a fixed other usage for black which happens to therefore never be the background color (or a task may have a fixed other color for the background, for whatever reason), in which case you'd end up with an empty set of examples. Thus you'd have to reject some tasks (ideally as soon as you - based on the generated examples - find the probability of black possibly being a background color low enough, if you care about being efficient).
If you care about exhaustiveness of tasks and don't care too much about retaining the exact logic of a given task, you could add color permutation to the rejection sampling idea: Whenever you see an empty set of examples for a given task after filtering for black background color, pick another background color for which the set is non-empty and simply "redefine" the task s.t. you apply the same color permutation with the picked color mapping to black on all examples in your set of filtered examples. Note that doing the same color permutation on all examples of a given task by definition (as the symbols don't bear meaning beyond being distinct from oneanother) gives you a (possibly new) valid task. But also note that even with that extension to the simple filtering approach you'd probably be left with some nonsensical results (e.g. there are some tasks where there just is no notion of background color in the first place).
Hope that helps.

For the original ARC dataset 0 is definitely predominantly the most common symbol. However I view this as more an artifact similar to the fact that grids of size 10 by 10 are also overrepresented. But ARC itself does not define further meaning of those 10 unique symbols, and surely does not require mapping 0 to black when visualizing grids.


rg.load takes too long because of the vector field

Is your feature request related to a problem? Please describe.
rg.load takes too long because of the vector field, even when I don't need it.
Describe the solution you'd like
Allow passing a param or through the query
use the elasticsearch _source query field to exclude vector when not needed.
"_source": {
        "exclude": ["vector"]
    }

Describe alternatives you've considered
wait
Additional context
Low priority

https://github.com/argilla-io/argilla/issues/2261

Hi @cceyda, thanks for picking this up! I think we need to review the rg.log/rg.load performance in general, given that a lot of users have had some issues with this for larger records.
 At least, I expect to be able to fix some of these load issues by implementing this https://github.com/argilla-io/argilla/issues/2341

Excluding vectors would be great. Technically vectors will not be required in some cases when training a model.
The option vectors=None in rg.load could be used to exclude the vector of the record? @davidberenstein1957

thanks @cceyda and @orasik , this is a needed and requested feat. @frascuchon we discussed this a long time ago (get a projection of only some fields when using rg.load) Is there a relatively easy solution to this?

I agree with the idea about the projection but IMO they are 2 separate things. Perhaps we can also think about quantization of the vectors to reduce the size a bit. Ideally, I also see this as a broader topic where we potentially can choose which attributes to load/update for a list of records.  On 24 Feb 2023, at 21:40, Daniel Vila Suero @.***> wrote:
thanks @cceyda and @orasik , this is a needed and requested feat. @frascuchon we discussed this a long time ago (get a projection of only some fields when using rg.load) Is there a relatively easy solution to this?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@davidberenstein1957 @frascuchon I found a bug when I tried to hard-code a list of fields to test.
This line https://github.com/argilla-io/argilla/blob/c35d63a5323fd360f1ef79456e6816d3470b3151/src/argilla/client/client.py#L707
projection={"*"}

is leading to this code for selecting which fields to include
https://github.com/argilla-io/argilla/blob/c35d63a5323fd360f1ef79456e6816d3470b3151/src/argilla/client/apis/datasets.py#L185
"fields": list(projection) if projection else ["id"]

If you make "fields": ["id"]
You will receive this error:
records = [sdk_record_class.parse_obj(r).to_client() for r in records]
  File "pydantic/main.py", line 527, in pydantic.main.BaseModel.parse_obj
  File "pydantic/main.py", line 342, in pydantic.main.BaseModel.__init__
pydantic.error_wrappers.ValidationError: 1 validation error for TextClassificationRecord

Not sure what is the validation behind parse_obj that is causing this error.

Found the issue.
inputs is a required field in the validation. If I change the line
https://github.com/argilla-io/argilla/blob/c35d63a5323fd360f1ef79456e6816d3470b3151/src/argilla/client/apis/datasets.py#L185
to:
"fields": ["id", "inputs"]

it works fine.
I will raise a PR soon

PR https://github.com/argilla-io/argilla/pull/2417

@frascuchon I think we should add some w.r.t. which fields we update during a rg.log afterwards to avoid overwriting records without vector. Can this be done easily?


HangingProtocol.viewportSettings not working...

Before Creating an issue

Are you running the latest version? - Yes, v3-stable
Are you reporting to the correct repository? - Yes
Did you search existing issues? - Yes, no related issue or issues

Bug Report
Describe the Bug
I want to achieve the effect as shown in the picture:

so I changed the code in platform/core/src/services/HangingProtocolService/ProtocolEngine.js to test if viewportSettings can work, saddly it not, pelease help, thks.
What steps can we follow to reproduce the bug?

change the code in platform/core/src/services/HangingProtocolService/ProtocolEngine.js, see /// MARK /// park below
run the Viewers

const defaultProtocol = {
  id: 'defaultProtocol',
  locked: true,
  hasUpdatedPriorsInformation: false,
  name: 'Default',
  createdDate: '2021-02-23T19:22:08.894Z',
  modifiedDate: '2021-02-23T19:22:08.894Z',
  availableTo: {},
  editableBy: {},
  protocolMatchingRules: [],
  stages: [
    {
      id: 'nwzau7jDkEkL8djfr',
      name: 'oneByOne',
      /// MARK - this is working
      viewportStructure: {
        type: 'grid',
        properties: {
          rows: 1,
          columns: 1,
        },
      },
     ///
      viewports: [
        {
          /// MARK - followed the instructions in docs, not working
          viewportSettings: [
            {
              options: {
                voi: {
                  windowWidth: 400,
                  windowCenter: 40,
                },
              },
              commandName: '',
              type: 'viewport',
            },
          ],
          ///
          imageMatchingRules: [],
          seriesMatchingRules: [],
          studyMatchingRules: [],
        },
      ],
      createdDate: '2021-02-23T19:22:08.894Z',
    },
  ],
  numberOfPriorsReferenced: -1,
};

ViewportSettings are not for placing the displaySets on the viewports. You need to find the correct displayset and putting it in the order you want it to be displayed in viewports: [

You can take a look at our HP docs here https://docs.ohif.org/platform/extensions/modules/hpModule