Pycodestyle-whitespace-rules

Add support for E2 and maybe E3 pycodestyle rules.

My approach here is to typically try and stay as closely as I can to the existing pycodestyle implementation -- you can see that check here. So, typically, I'd say take that logic, and add it to check_lines.rs (the line-based checker, rather than the token- or AST-based checker).
The hiccup is that pycodestyle is running that check over "logical" lines rather than "physical" lines. (I'm not sure whether you're familiar with the distinction (I wasn't until recently), but a logical line is Python statement, while a physical line is an actual line-of-code in the editor. We don't have support for generating logical lines in check_lines.rs right now.
I could see a few paths forward here:

Add support for generating logical lines by porting that logic from pycodestyle. (Not totally trivial but maybe a good exercise if you're interested, and useful beyond this specific rule since it would make it much easier to port arbitrary pycodestyle rules.)
In check_ast.rs#visit_stmt, you could do something like this to extract the logical line based on the statement locations (we already know where statements start and end since we created an AST):

let text = checker.locator.slice_source_code_range(&Range::from_located(stmt));

Then, use run regular expressions against the text as in pycodestyle.

Sorry for no response. I was working on a solution using method 2 you described. I believe I have a working solution but have three problems.

How to "activate" the rule. As in in check_ast.rs we frequently use the pattern if self.settings.enabled.contains(&CheckCode::XXX) but how do I activate this code? If I place the logic outside that block it runs and is successful but not within.
The logic I have for logical_line inside #visit_stmt function results in the message having the correct row for the error but defaults to the column 1 always and I am trying to find a way to return a new range while keeping the standard Range::from_located(stmt) pattern.
Finally the regex pattern they use in pycodestyle that you shared (I verified it is the one they still currently use on main as well) uses a backward look-around ((?!=)) which Rust-Regex does not support. I removed this and the code seems to function properly but it may be for capturing an edge-case my fixture file does not include.

I can check this code in if you would like to look at it.
I am also not opposed to attempting to implement the logical_line functionality myself. I believe it is found here.

Ok cool!
I've made a little bit of progress on the logical line stuff, but I need to benchmark it to know if it's way too slow.
Happy to take a look at what you have thus far, it may end up being better. The main thing I was unsure about with Approach 2 (after writing that comment) is that statements are nested, so taking the source code for a single statement isn't actually equivalent to a logical line, unless I'm misunderstanding.
For example, if you have...
def f():
  x = 1

And you use the approach I described above, the first string would be the entire function, and the second would be x = 1. But maybe there's a way around this...
To answer your questions:

If you've added a new check code, you should run cargo dev generate-check-code-prefix to update the autogenerated mapping. Then you can run cargo run foo.py --select E201 or whatever the code is. Does that work?
We have some helpers that might be useful for this, helpers::to_absolute might do what you want!

https://github.com/charliermarsh/ruff/pull/1130 implements the logical line building, and it seems to work in testing a variety of cases against pycodestyle. But it's way too slow, like more than doubles Ruff's runtime on the CPython benchmark. All the time is spent building the logical lines.

Alright I figured I would just push my code and we can go from there. I focused on implementation so the location may be wrong but I basically just pull out the logical line you suggested and then implement the regex search. You can see from returned data (first screenshot) that we are capturing the errors but always with column: 1. The print statements verify we can get the correct column using stmt.location.column() + found + 1 (second screenshot).

I think that to get the right range, we'd want to do something like this:
self.add_check(Check::new(
    CheckKind::WhiteSpaceAfter(character),
    Range {
        location: Location::new(stmt.location.row(), stmt.location.column() + found),
        end_location: Location::new(stmt.location.row(), stmt.location.column() + found + 1)
    }
));

(Not exactly right, doesn't handle newlines and multiple whitespaces, but the idea would be to use Range { ... } rather than Range::from_located(...).)

I think this approach is going to be somewhat limiting though, because it's not the case that each statement corresponds to one logical line.
For example, if we have a nested function:
def f():
  def g():
    pass

Then with the current approach, we'd first look at this string:
"""
def f():
  def g():
    pass
"""

And then, when we recursed, we'd look at this string:
"""
  def g():
    pass
"""

Pycodestyle also has some rules whereby it removes comments and removes string contents (which I assume get in the way of the regular expressions), so we'd need to tokenize and find a way to handle those too.
In short, I think we do need to create logical lines separately, or come up with a whole new strategy for this that's very different from Pycodestyle's strategy (e.g., iterate over the token stream, look for Lpar and Lbrace tokens, and see if they have whitespace after them).

Closing for now to keep the PR list up-to-date, can always reopen.


[ENH] Kernel conditional discrepancy test (conditional k-sample test) to enable Psi-FCI algorithm

First part of #57 and step towards #56
Changes proposed in this pull request:

Implements the kernel CD test and relevant unit-tests
Refactors the documentation API into relevant sections

The main files are the dodiscover/cd/* files and the unit test.
Before submitting

[ ] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[ ] I've updated or added any relevant docstrings following the syntax described in the
Writing docstrings section of the CONTRIBUTING docs.
[ ] If this PR fixes a bug, I've added a test that will fail without my fix.
[ ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

After submitting

[ ] All GitHub Actions jobs for my pull request have passed.

Codecov Report

Merging #81 (23a579a) into main (e94e038) will decrease coverage by 0.32%.
The diff coverage is 80.13%.

@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
- Coverage   82.13%   81.81%   -0.33%
==========================================
  Files          20       23       +3
  Lines        1304     1424     +120
  Branches      229      242      +13
==========================================
+ Hits         1071     1165      +94
- Misses        152      171      +19
- Partials       81       88       +7

Impacted Files
Coverage Δ

dodiscover/cd/base.py
53.33% <53.33%> (ø)

dodiscover/ci/utils.py
75.51% <75.51%> (ø)

dodiscover/cd/kernel_test.py
87.01% <87.01%> (ø)

dodiscover/ci/kernel_test.py
91.12% <100.00%> (+0.78%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How to use dynamic request parameter when attacking/test a service every time?

Question
We want to use different input parameter whenever invoking a new request to a URL with parameters in the url through vegeta, how to implement that?

for example in the following test:
echo "GET http://localhost/?p1=123&p2=abc" | vegeta attack -duration=5s | tee results.bin | vegeta report
if we need use different values for p1 & p2 for every test request, how to do to achieve that?

Got the following error when using jq, what's jq? Is anything in the context?
jq -ncM 'while(true; .+1) | {method: "POST", url: "http://:6060", body: {id: .} | @base64 }'
jq: Unknown option -ncM
Use jq --help for help with command-line options,
or see the jq documentation at http://stedolan.github.com/jq

Have addressed the problem, it's due to the wrong version of jq

Another question:
For any GET and POST request, if we want it has dynamic parameter, the document said that we can use the option "-target", but what could the content of target file be? Are there any example? thx

how to read parameters from a file and then randomly fetch one line as parameter in jq?

jq -ncM 'while(true; .+1)
in the above while loop, hot to set the initial value for the loop when there is certain business requirement related to it?

I lost track of what your current problem is. Can you reformulate and summarise so I can help?

@tsenart Thanks a lot for your kind concern!
I have addressed all the mentioned questions. For the last one, we can initiate a value instead of 1 in the following way:
jq -ncM '18 | while(. < 28; . + 1)|{method: "POST", url: "http://:6060", body: {id: .} | @json }'
Now my new question is: if we need to read dynamic parameters from a static file and then generate one target (url) for each line, how to use jq to implement it?
Would you be so kind as to provide one or several examples? Thank you very much! @tsenart

When you get to those use cases, I recommend you write your own script / program that generates targets.
This old blog post goes into detail on how to do that: https://web.archive.org/web/20180304073213/https://thisdata.com/blog/load-testing-api-interfaces-with-go-and-vegeta/
At that time, JSON target format didn't exist yet, so these days you should use that because it's much easier to generate.

Sadly found that both cannot be opened:
I cannot successfully open  https://web.archive.org/web/20180304073213/
Not Found  ---- https://thisdata.com/blog/load-testing-api-interfaces-with-go-and-vegeta/
The requested URL /blog/load-testing-api-interfaces-with-go-and-vegeta/ was not found on this server.
Apache/2.2.15 (CentOS) Server at blog Port 80

The URL is exactly what I wrote above, not two separate URLs: https://web.archive.org/web/20180304073213/https://thisdata.com/blog/load-testing-api-interfaces-with-go-and-vegeta/


Backup Load Problem

Trying to load a backup for a section of a world and it caused it to bug out and mess up.
Not sure what caused it.
I'm running the 1.12.2 pack, but the pack author recommended I make a post here and ask.
Here is the git link to that question with screenshots and more information:
https://github.com/xJon/The-1.12.2-Pack/issues/547#issuecomment-705787708

Unfortunately, I don't know how to fix this bug. I think it happens when you add or remove mods and the block ID lists shift around, but I have been unable to reproduce it it so I can can't really tell. I'm leaving this issue open because it really needs to be fixed, but I don't expect to be able to fix it anytime soon.


Evaluating switching to Capsule ProxySetting rather than using RoleBindingReflector

Starting the discussion from https://github.com/clastix/capsule-proxy/issues/266#issuecomment-1456226221 thanks to @MaxFedotov:

I remember why I added this feature. The idea was the following:

Tenant admin create a role and rolebinding in one of his namespaces for user Bob. This role has a limited set of permissions (it is not cluster-admin)
Bob must be able to see all namespaces, where he has access when running kubectl get ns

This reflector caches all rolebindings and is used for getting a list of namespaces to be shown for each of the users.
And as reflectors are asynchronous by default, this can add a delay between kubectl create ns and kubectl get ns
Now capsule-proxy adds new ProxySetting CRD, which can be used to add permissions for both, tenant admins and normal users. But removing this reflector would require tenant admins to create a separate ProxySetting CRD for each of the users rolebindings in all namespaces.

My thoughts: we could create a specific addon that replicates the ProxySetting resources for each ServiceAccounts that has GET permission on its Namespace, in order to let them list Namespace as would happen for a regular user.
Not sure if this would be the duty of capsule-proxy here, but I'd like to get feedback from end-users and adopters.

From one side I like the idea of moving to ProxySetting, as it can be seen as a natural capsule way.
But on the other side, I don't like the idea of breaking the current design (and being not backward-incompatible), which can be already used by the community (and yes, it is already used by us :) ) and will require them to perform additional activities during and update and afterward.


don't include binary name and version when printing help of any of the binaries

Is your refactoring request related to a problem? Please describe.

When printing the help text of a command (e.g. c8y-firmware-plugin --help), the binary name and version is displayed on the first line of the help text.
Having the information within the help text makes parsing the output more complicated for automation purposes (e.g. generating automatic do documents from the command output as the version changes more frequently than the help contents).
Below shows an example of the version information which is printed on the first line.
c8y-firmware-plugin 0.11.0-145-ge59e17be

Here is the full example:
Command
c8y-firmware-plugin --help

Output
c8y-firmware-plugin 0.11.0-145-ge59e17be
Thin-edge device firmware management for Cumulocity

USAGE:
    c8y-firmware-plugin [OPTIONS]

OPTIONS:
        --config-dir <CONFIG_DIR>
            [default: /etc/tedge]

        --debug
            Turn-on the debug log level.

            If off only reports ERROR, WARN, and INFO If on also reports DEBUG and TRACE

    -h, --help
            Print help information

    -i, --init
            Create required directories

    -V, --version
            Print version information

Describe the solution you'd like

Don't display the binary name and version on the first line of the help/usage text for any of the project's release binaries (e.g. tedge, tedge-agent, tedge-mapper, c8y-*-plugin, tedge-apt-plugin etc.

In comparison with other tooling, it is more common to just display the version via --version (or something similar), rather than printing it out when using --help.

curl --help
docker --help
grep --help

All of those commands have a dedicated version flag should the user want to know the version of the command they are using.
Note

I realise this is a bit of a bike-shed issue, however if no-one really cares it will at least make documentation automation slightly easier. Also the requested behaviour has already been accepted as the default for clap-rs PR4160

Describe alternatives you've considered

Additional context

Mostly a matter to upgrade to clap ="4.10" with some issues with deprecated methods:
error[E0599]: no method named `parse` found for struct `Arg` in the current scope
  --> crates/core/tedge/src/cli/mqtt/cli.rs:37:29
   |
37 |         #[clap(short, long, parse(try_from_str = parse_qos), default_value = "0")]
   |                             ^^^^^ method not found in `Arg`
``

As a side effect of bumping clap version, --help no longer show the version, and this for all the tedge-xxx executables.

QA has thoroughly checked the feature and here are the results:

[ ] Test for ticket exists in the test suite.
[x] QA has tested the feature and it meets the required specifications.

Checked for:

c8y-firmware-plugin
tedge
tedge-agent
tedge-mapper
tedge-apt-plugin
c8y-configuration-plugin
c8y-log-plugin
c8y-remote-access-plugin


Getting item size in custom layout

New issue checklist

[x] I have reviewed the README and docuxmentation
[x] I have searched existing issues and this is not a duplicate

How can I get value of - (CGSize)sizeForItemAtIndex:(NSInteger)index; method in my custom UICollectionViewLayout?

I have a similar issue where my custom UICollectionViewLayout does the size calculation to adapt to the screen (list vs grid), something I'm not sure I want to leave to the ListSectionController and where collectionContext doesn't have enough info anyway.

I am trying to do Pinterest layout Any Ideas are welcome

@jassimkm You can get item size from the adapter with layout delegate. Like this:
extension ViewController: CustomCollectionViewLayoutDelegate {
    func sizeForItem(at indexPath: IndexPath) -> CGSize {
        return adapter.sizeForItem(at: indexPath)
    }
}


cp target is not a directory

Hi team - docker-compose fails:

Why there is something like this in entrypoint??

Hey @zrks,Take a look at etc/volumes/default.yml (https://github.com/Accenture/adop-docker-compose/blob/master/etc/volumes/local/default.yml#L4). This is when the directory is being created.
Because you are running the main docker-compose file, the volumes compose one is not executed and the directory is not created. The copy commands throws the error because of this.
I would recommend using the adop cli instead. This will also fix the warning messages you are getting because it will source the variables from env.config.sh.
Also feel free to take a look at https://github.com/Accenture/adop-nginx to see how the adop-nginx image has been implemented.

Hi - yes already inspected that - thanks for reply!
We can close this issue then.


youtube: Playback on other websites has been disabled by the video owner.

In trying to fetch a video off youtube, I encountered the following error.  I'm wondering if there's a way around this.  youtube-center seems to be able to but it can't download DASH videos.  NSFW warning on that video link by the way.
$ youtube-dl -v https://www.youtube.com/watch?v=dxpRkMxTMAE
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'https://www.youtube.com/watch?v=dxpRkMxTMAE']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2<IP_ADDRESS>
[debug] Python version 2.7.10 -<EMAIL_ADDRESS>[debug] exe versions: ffmpeg N-76643-g5324109, ffprobe N-76643-g5324109, rtmpdump 2.4
[debug] Proxy map: {}
[youtube] dxpRkMxTMAE: Downloading webpage
[youtube] dxpRkMxTMAE: Downloading embed webpage
[youtube] dxpRkMxTMAE: Refetching age-gated info webpage
ERROR: dxpRkMxTMAE: YouTube said: Watch this video on YouTube.
Playback on other websites has been disabled by the video owner.
Traceback (most recent call last):
  File "/usr/lib64/python2.7/site-packages/youtube_dl/YoutubeDL.py", line 663, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/common.py", line 290, in extract
    return self._real_extract(url)
  File "/usr/lib64/python2.7/site-packages/youtube_dl/extractor/youtube.py", line 1186, in _real_extract
    expected=True, video_id=video_id)
ExtractorError: dxpRkMxTMAE: YouTube said: Watch this video on YouTube.
Playback on other websites has been disabled by the video owner.

Pass login and password with -u and -p or pass authenticated cookies with --cookies.


Use camp::Platform

Needs fix from #333

@davidbeckingsale, I approved #333 .  I will wait for it to be brought in and for things to compile here.

@mcfadden8 - this now has the iterator fix.


Cellular automata books list has moved

What does this PR do?
Improve Repo
For resources
Description
The original link to the cellular automata books list leads to a not found page. Going to the root page (https://uncomp.uwe.ac.uk/genaro) shows that the site has moved. The same exact list (and the rest of the site it seems) can be found at the new location.
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

I was initially wondering about the new location but it seems that the original site had the following copyright on the bottom of the page even before the move:

(© ESCOM IPN MEX, ICUC UWE UK, 2005-2019)

Looks like the content was simply moved to the ESCOM IPN MEX site, which is a polytechnic in Mexico if I'm not mistaken.


Database crash

Information

Minecraft Version: 1.16.1
Fabric API Version: lastest
OS: Linux Mint
Other installed mods:
Database: mariadb mysql

Describe the issue
Server crash
Crash/Error logs
https://hastebin.com/esevayosub.rb
https://hastebin.com/wometiduqe.rb
db error
Expected behavior
Claims too work.
Too essentials don't crash game

crash-2020-07-26_15.59.51-server.txt
crash-2020-07-26_15.41.48-server.txt
Aditionaly text log versions.

First error means you didn't set up PostgreSQL correctly, the second one is a known bug in claims

You were telling to change from mysql on fabric discord

new one: from clear mysql mariadb usege
Crash on hastebin and in .txt
https://hastebin.com/vuyejuhifa.rb
crash-2020-07-26_17.49.54-server.txt
Aditional:
Everything is fine until I punch block.

mayby I should use datapack for claims
for now...

I know this isn't best place but what tutorial about fabric dev would you suggest for start of fabric dev ? I think about making own mod/plugin - for server only for plot security mayby upgrading source code to something better or making something own more flexsible like using item on 2 sides of plot and securing it


Documentation out of date for Multiple select

Reproduction Link
Fiddle
Steps to reproduce
Copy the code from Multiple Select and compare the results to the embedded app.
Expected behaviour
The resulting app resembles the app within the documentation.
Actual behaviour
In the documentation, as you select options, they don't appear while the dropdown is open. They also don't appear when it is closed as a template is evidently configured.
However in the fiddle, they appear as you select them, and are then hidden once the dropdown closes. A note in the documentation suggests a template exists for this:
<template slot="tag" slot-scope="props"><Your code></template>

However without a value, it is ignored.
What is the optimal way to prevent options from being displayed when the dropdown is open?

The Fiddle uses an outdated version of the component (v2.1.0). The current version is 2.1.4.
See: https://jsfiddle.net/7ox2tnfq/

Hi, thanks for the quick response.
However, the behavior is unchanged (even after bumping the css to the same current version).
The issue is the document presents a component configuration which is desirable when selecting several options, however the code does not produce that result even with a current dist.

Yep, I ran into this same issue, as there was no discernible flag to disable the "tags" as a whole (instead, only flag to disable "new" user-entry tags).
Took me way too long to figure it out, but you just need to supply an empty slot for "tag", but at least give it one element:
<template slot="tag"></template>


Add the type of watch or blacklist (e.g., keyword, website, number) to the title of the PR

Resolves #12205. See explanation there.

Tested this here: https://chat.stackexchange.com/rooms/154419/conversation/testing-pr12206
Looks good to me.


Bug: Importing GCS characters without advantages does not work

Version information:

I get the following messages when attempting to import a character who has no advantages:

It seems some traits are imported, like the attributes and name, but skills and equipment and the resulting weapons are not:

I'm attaching two different characters I'm having this issue with. Please also check the code which handles other traits this way, maybe a similar error happens when the character has no disadvantages or skills.
characters.zip

Hi @Exxar, thanks for the message. The issue occurs because when the character does not have any traits the import process is interrupted and the final sheet has incomplete data.
This is the sheet after the fix:

Please let me know if this fix the issue.

Yep that looks good now! Thanks!


Add support for model 1 with a length of 67 to support legacy implementations

Originally, model 1 did not have a pad register at the end so the length is 67. The library should support model 1 with a length of 67 even though it does not conform to the model definition length of 68. This is the one and only condition of this type and allows support for legacy devices.

The length in the 'L' point is actually 65 for a total length of 67.

Fixed in commit 4405d92786733f1fed118be9c468eaaf97ca8b1d.


Make xgboost an extra dependency

This has been done in order to allow for lightgbm-ray to be installed without xgboost. The new install command for xgboost-ray will be pip install "xgboost-ray[default]". If xgboost is not installed, train and predict methods will throw an ImportError.

(oops, can we merge)?

Needs to pass CI now ;)


Update HA docs

real IP -> your VIP

Coverage decreased (-0.07%) to 64.281% when pulling de91098fc5cdcb0248e1cb4744d208d4d810610d on yixingjia:docsissue into dce1896d92722bb7d5d5f27d683b64bc80555e28 on vmware:master.


Add Composer-Velocita plugin

Velocita is a caching reverse proxy for Composer repositories and package distribution locations, such as Packagist or GitHub.

Speeds up downloads for package metadata and dist files
Serves cached files even if the source location is unreachable
Can be used as a shared cache by multiple developers, CI, etc.
No changes are required to your project's files!

The Composer-Velocita plugin instructs Composer to use a Velocita Proxy instance.

Thank you for contributing! ❤


Feature request: local language server

Maybe this is already implemented and I did not find.
My point is, that I dont want to move my code into the internet,
not even for being checked by a language server.
The only way to use a language server (which I love)
is to use a loal one.
Is there a way to install?

All editors run the language server locally - why do you think anything
else is occuring?
On Sun, Aug 18, 2024, 06:12 Ernst Reissner @.***> wrote:

Maybe this is already implemented and I did not find.
My point is, that I dont want to move my code into the internet,
not even for being checked by a language server.
The only way to use a language server (which I love)
is to use a loal one.
Is there a way to install?
—
Reply to this email directly, view it on GitHub
https://github.com/bscan/PerlNavigator/issues/137, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFURPKW4RFXFGJU2G6YVYBDZSAGKBAVCNFSM6AAAAABMV6UNH6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ3TCNZWGU3TANY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @Reissner, yes you are likely already running it entirely locally, especially based on the issue in #136 where it showed Perl warnings. When running normally, Perl Navigator does not require a web connection and does not make any web requests for telemetry, functionality, or required assets.
The one exception is when running in web-mode (press . in a browser when viewing any Github repo). The Perl Navigator works in this web based version of vscode, but is limited in functionality. Because we can't run Perl in a browser, it does not have syntax checking, perlcritic, or perltidy. However, as you can see in the image, you can still get sticky-scroll, outline view, improved syntax highlighting, and some go-to definition and hover. This method of using the Perl Navigator is uncommon, although I'd love to know if anyone is using it this way. Standard installations via vscode and npm are entirely local.


Matches behavior

The following expression returns false:
define MatchesFalse: Matches('1,2three', '\w+')
Why?

This is a Java thing. '' is a special character in Java land and must be escaped.
Try:
define MatchesFalse: Matches('1,2three', '\w+')

@brynrhodes : was my explanation satisfactory or is there some further action required to resolve this issue?

Still doesn't match, even though online regex matching testers indicate it should:
define MatchesTest: Matches('1,2three', '\w+')


Unable to use SSL with MySQL back-end

Setting MySQLOrchestratorUseMutualTLS=false currently prevents any SSL connection to the MySQL backend server, as it is used to guard the call to SetupMySQLOrchetsratorTLS. Further, SetupMySQLOrchestratorTLS does not, itself, guard on this same flag and always attempts to add the client key pair.
As the documentation suggests this flag may be used to enable encryption while not requiring client authentication, this is unexpected behaviour. We have a working fix for this problem by additionally evaluating whether or not MySQLOrchestratorSSLCAFile is blank, but I'd like to make sure this is not an intentional decision before opening a pull request.

I am not aware this was an intentional decision. All TLS code was added by way of contributions, so I cannot speak on behalf of the contributors.
I'm not sure I understand the nature of the problem:

currently prevents any SSL connection to the MySQL backend server

Did you mean any orchestrator SSL connection to the MySQL backend server, or that this somehow affects other clients? (which would be super unexpected)
I'm happy to receive a PR.

Did you mean any orchestrator SSL connection to the MySQL backend server, or that this somehow affects other clients? (which would be super unexpected)

This is specifically connections to the MySQL backend server, yes. In particular, SSL can be used iff a client key pair is provided.

This is specifically connections to the MySQL backend server, yes. In particular, SSL can be used iff a client key pair is provided.

Sorry, I'll ask again since this does not answer my question. Did you mean that because of orchestrator's configuration, other clients which are unrelated to orchestrator are unable to connect to the same backend server?

No, this only about Orchestrator. Other clients should be out of scope here. When Orchestrator is configured with MySQLOrchestratorUseMutualTLS=false, Orchestrator is unable to access its MySQL backend specified in MySQLOrchestratorHost using SSL. Sorry for the confusion.


remove distinct from search

A large enterprise installation reported that adding distinct to the main hapi-fhir search function had serious performance consequences for them.  Commented out with comment for now.
Also improved migration logging.

Large enterprise installation tested and confirmed this fix resolved the issue for them.

Fixing this breaks a test for #1300 - Seeing if I can find another way to fix that


Can I develop models in Julia and solve them in Python?

I want to export my julia optimization models as files supported by scip solver so I can use them in other frameworks such as pyscipopt in python. Is this possible?

I guess you could export them in standard file formats such as lp, mps or cip (which is specific to SCIP). These can be loaded by solvers, including via PySCIPOpt. Note that you'll have to identify variables by name.

By Julia optimization model, I'll assume this is a JuMP model, you may want to look into the following function:
https://jump.dev/JuMP.jl/stable/reference/models/#JuMP.write_to_file

@rschwarz @matbesancon Thank you, it works! and yes I am using JuMP, and saving with *.mps format. However, the pyscipopt objective value was the exact negative of what was in my original model! Could it be because mps format treats all models as minimization problems? (my original problem was Max and it is fixed with Min)
I also wonder if I could do the same with nonlinear models because I was not able to use JuMP.write_to_file for a model with non-linear constraints?

thanks @rezabayani that's unexpected. Can you re-read the MPS in JuMP to see if the MPS itself was transformed or if pyscipopt is inverting it upon reading?
From the JuMP MPS writer, it seems writing out both min and max are supported

@matbesancon I found an issue where if the mps file contains the objective sense, pyscipop returns error. It was fixed when I manually deleted that line from the mps file.

@matbesancon I am still facing an issue where pscipopt can not load 'mps' files unless I manually delete the ObjSesne line in the saved file. Do you think there is a remedy to this besides manually editing these files?


[LayoutReader] Training loss is low but inference performs terrible

Describe
Model I am using (UniLM, MiniLM, LayoutLM ...): LayoutReader
Hi @zlwang-cs
I am using layoutreader to predict layout-only data like this, which the order is from right to left, top to bottom.

However, I encountered some problems and hope you can share some insights.

When I tried to train the model, an error occurs at line 671, said that the two summed tensor dimensions are not aligned, and when I remove the self.bias, I can train normally
https://github.com/microsoft/unilm/blob/cd2eb8ade8b6e475aefa9b769ced2eefc4245a3e/layoutreader/s2s_ft/modeling.py#L669-L673
The training process is normal, and the initial loss is around 30 and can drop to 0.01. However, when I test with thetest set (shuffle rate=1.0), the results can be very poor. A lot of boxes are missing and the ARD is around 20.1. When I pre-sort the inputs in the test set with rules (simply orderd by x coordinate), and feed them into the network. But the network just predicts the exact same reading order as the inputs. What's more, when I tested with the training set, the results were also very poor

Hi, thanks for your interest in our paper. I'd love to help you to fix the issues.
For the first question, I am not sure what the problem is without any more detailed information. I would recommend you to use stop points or other debugging tools to see what the shape of tensors looks like in this step.
For the second question, I guess the problem may be from the reading order of your dataset. You can see that the data you use is quite different from the original settings in our paper. The pre-trained weight is based on the left-to-right and top-to-bottom reading order so such a pre-training setting may be an obstacle in your experiment. Considering this, I don't think the poor performance is surprising. If you would like to continue solving this reading order setting, maybe you need to collect enough data to pre-train the model again or resort to other approaches. Another possible way is to turn the image 90 degrees counterclockwise so that it will be similar to the common reading order settings.

Hi @zlwang-cs
Tks for the prompt reply. I'll try to debug this and see if I can locate the problem. Rotate the image 90 degrees counterclockwise seems to be quite reasonable and I'll try this too.
There is one thing that still puzzles me. You mentioned that LayoutReader loads pre-trained weights during training. Is this pre-trained weight based on word-level or textline-level? It seems that I need textline-level pre-trained weight here

Hi @Mountchicken, the pre-trained model is based on the word-level. Unfortunately, I cannot help you with the textline-level pre-training.

Hi @zlwang-cs
Thanks for the reply. BTW, how to load the pre-trained weights and finetune it on my own dataset? I downloaded layoutreader-base-readingbank.zip from this link and got config.json, pytorch_model.bin after unpacking it.
 Please refer to some related documents and I am sure you can find the right answer. And I see you are using the run_seq2seq.py which is for training, but the weight you downloaded is actually for decoding. I guess that is the reason why you are confused.


Move FPS API into runner

Need to move FPS counter from GameCreatorRunner to GameRunner so it can be used in builds.

This has been fixed.


'import: command not found' errors

Hey. I'm looking for a bit of help. My setup is:

Running on Ubuntu server
Running CLI as root
Created requirements.txt in /opt/udm
Ran the command pip3 install -r requirements.txt
Have enabled SSH and set password on UDM Pro (not from the Network application but from initial login area)
Running 1.10.0 as the Unifi OS
Running 6.2.26 as the Unifi Network app
Created get_unifi_reservations.py in opt/udm
Amended the script variables for user name/password

However when I run ./get_unifi_reservations.py -b https://<IP_ADDRESS>:443 -p <pass> -u <user> -su root -sp <ssh_password> -sa <IP_ADDRESS> -f -vvv or bash get_unifi_reservations.py -b https://<IP_ADDRESS>:443 -p <pass> -u <user> -su root -sp <ssh_password> -sa <IP_ADDRESS> -f -vvv I get the following output:
get_unifi_reservations.py: line 3: import: command not found
get_unifi_reservations.py: line 4: import: command not found
get_unifi_reservations.py: line 5: import: command not found
from: can't read /var/mail/argparse
get_unifi_reservations.py: line 9: import: command not found
get_unifi_reservations.py: line 10: import: command not found
from: can't read /var/mail/scp
get_unifi_reservations.py: line 14: import: command not found
get_unifi_reservations.py: line 15: syntax error near unexpected token `urllib3.exceptions.InsecureRequestWarning'
get_unifi_reservations.py: line 15: `urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)'

Any ideas on how to get this working? I also get the same message when running ./get_unifi_reservations.py -h or bash get_unifi_reservations.py -h
Thanks in advance

This looks like an issue with your python interpreter installation. Can you run the following commands and let me know the output?
which python3

python3 --version

Also, which OS are you running this on? Linux, OSX, or Windows?

@cdchris12 when I run which python3 I get the following:
/usr/bin/python3

When running python3 --version I get:
Python 3.8.10

This is on Ubuntu 20.04.3 LTS (Linux)

@cdchris12 the python3 command worked


[change] Added parser for Wireguard #100

Closes #100

it seems to me those links are simply ignored right now which is not correct.

Yes because if it is not up, ignoring it will remove it from diff of old and new wireguard parse and will show it as down in topology graph.
In openwisp-network-topology, I tested it by creating topology using the fetch strategy and after 5 minutes without updating the url I ran the fetch strategy again and it showed as down. I have added a diff test for the same.


Help with ping

The AP-ESP1-STA is in Automesh mode.
I can control my Home Automation with my iPhone connected to AP-ESP1 or AP-AirPort.
My Home Router is set to give the iPhone <IP_ADDRESS> address.
The ESP32 can ping <IP_ADDRESS> to find my iPhone and know when I'm home.
The ESP1 has given the iPhone <IP_ADDRESS> and I have saved the DHCP lease.
But the ESP32 is unable to ping <IP_ADDRESS>
Please can you help me with the settings I can change on ESP1 so I can
ping iPhone <IP_ADDRESS>
Thank you.
Im using an ESP8266 USB dongle. It has an sm LED connected to GPIO 4
On your next update, Is it possible that when you disable HW Factory Reset
with "set hw_reset 255" that no voltage is on GPIO 4

Thank you for your reply.
Im now using Version V2.2.2
I can confirm that setting portmap add TCP 80 <IP_ADDRESS> 80
that I can reach my iPhone, With the iPhone running a small web server.
But that's not really what I want.
My ESP32 is running my own Home Automation Arduino sketch with a ping library.
I can ping my iPhone and get a reply when it has a <IP_ADDRESS> Address. When it
is dark the lights will turn on but I have to be inside my house. I want to extend
my WiFi by using your WiFi Repeater code and turn the lights on when Im outside.
Im unable get a ping reply from my iPhone when it is connected to the ESP8266 with
an Address of <IP_ADDRESS>
I have been searching the internet to find as much information as I can so I
would not ask stupid questions. I understand that ping does not use TCP UDP
or port numbers It uses ICMP.
In your README documentation you talk about ping in Static Routes.
I have read this many times trying to understand how it works and if would
would work in my case.
I would appreciate any more help you could give me.
I would like to thank you for changing the behaviour of the HW-reset.
I would also want to apologise for my mistake. The LED on my ESP8266 USB dongle
is connected to GPIO 14 and not GPIO 4 When the dongle is connected the GPIO14
LED is not very bright but it is still on.
Thank you.

You cannot use a portmap for ICMP, that's right. If you really want to use ping (i.e. ICMP), static routing is probably an option to consider. However, this requires, that the ESP32 knows, that it can reach the 10.24.x.x network behind the <IP_ADDRESS> router. I.e. it needs a static route to it. Don't think you have a static route option on the ESP32...
As the iPhone behind the esp_router has the router as default gateway the route back should work without additional route entry.

Thank you for your help.
I will look into static routing, Maybe put another ESP8266 repeater before the ESP32


⚠ Bofrost.be has degraded performance

In e624efb, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1133 ms

Resolved: Bofrost.be performance has improved in 3e313f4 after 30 minutes.


MakeCode: button pressed register has incorrect valued on pressed

This is using the simulator.
According to the button spec, pressure === 0xffff when the button is fully pressed, but the following code prints 100 when the button is pressed
forever(() => {
    if (modules.button1.pressure() !== 0)
        basic.showNumber(modules.button1.pressure())
    else
        basic.showString("U")
})

There is always going to be a threshold involved, specially with capacitive buttons.

From: Tom Ball @.>
Sent: Sunday, September 19, 2021 2:48:54 PM
To: microsoft/jacdac @.>
Cc: Peli de Halleux @.>; Assign @.>
Subject: [microsoft/jacdac] MakeCode: button pressed register has incorrect valued on pressed (#644)
According to the button spec, pressure === 0xffff when the button is fully pressed, but the following code prints 100 when the button is pressed
forever(() => {
if (modules.button1.pressure() !== 0)
basic.showNumber(modules.button1.pressure())
else
basic.showString("U")
})
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fjacdac%2Fissues%2F644&data=04|01|jhalleux%40microsoft.com|e8f6245296374b580e7b08d97bb7470e|72f988bf86f141af91ab2d7cd011db47|1|0|637676849363747949|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=c4OCmniDXXbTU5XHd3xIQNiJVJ8UeHgT7DrfT8p5y0Y%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA73QKMHAMXMCRPFKO3F7LLUCZLENANCNFSM5EKYIDZQ&data=04|01|jhalleux%40microsoft.com|e8f6245296374b580e7b08d97bb7470e|72f988bf86f141af91ab2d7cd011db47|1|0|637676849363757942|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Srm8VHXkK4zYgUIPqzOHKYMZSkN%2FzOmdfMGJk%2FI49pU%3D&reserved=0.

Peli - I am just following the spec.

do we have capacitive buttons for the hackathon?

No capacitive buttons, though all buttons should implement the pressure register.
MakeCode probably does some wrapping:
https://github.com/microsoft/pxt-jacdac/blob/master/button/client.g.ts#L26

From what I can tell, the docs page massages pressure into a range between 0 and 1.

I think this is a question around how we surface values to the user. I think the current choice of putting values in a range of 0 - 100 is quite nice for novices. What is not good is if there are inconsistencies with simulator, the spec, and MakeCode. Where do we tell users to look for documentation help on MakeCode APIs? Figuring out the range by trial and error is not great.

Related: https://github.com/microsoft/jacdac/issues/641

The specification comment is incorrect. The pressure register unit is /, it's a scaled short so the range is 0...1. Moreover every physical button implementation may come with a different set of threshold to decided whether it is pressed or not. I've updated the comment to keep the fact that 0 is open.

@tballmsft if relevant to hackathon, add label to the issue.

We are using 0..100 instead of 0...1 ranges. The story is mostly told by the makecode skeleton code generator.

Peli, we need to get the button client cleaned up. In particular, we really need the boolean pressed implemented as pressure is low-level and not needed for the hackathon (no capacitive buttons).
Currently, with the simulator, we never see the value 100 for fully pressed:
basic.forever(() => {
    if (modules.button1.pressure() === 100)
        basic.showNumber(100)
    else if (modules.button1.pressure() >= 99)
        basic.showNumber(99)
})
``


Authorize attribute example

Could you please provide an AuthorizeAttribute example? Once the authentication is done calling the Web Authenticator in the Xamarin Forms App, what is the proper way to send an http request to the backend server? Should the access token be added as the Authorization Header (Bearer)?

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

ID: 1231a1e1-3734-9021-2cc7-eaf99478afca
Version Independent ID: 01e5fdbe-0b15-4dbe-f8a0-ed40f8593302
Content: Xamarin.Essentials: Web Authenticator - Xamarin
Content Source: docs/essentials/web-authenticator.md
Product: xamarin
Technology: xamarin-essentials
GitHub Login: @Redth
Microsoft Alias: jodick

You should be able to add it as the bearertoken and then setup this on your backend https://stackoverflow.com/questions/25764732/retrieve-bearer-token-in-asp-net-webapi

@n6ither did this help?

@jamesmontemagno sorry the late response but yes sir, it did the trick. Thank you!


Birth and death support

The Simulation interface does not support birth and death processes. Does it make sense to add this to dworp.simulation - something like BirthDeathSimulation with hooks that developers implement to define the birth and death processes?

Yes, makes sense to me.
Vlad
On Wed, May 9, 2018 at 2:31 PM, Cash Costello<EMAIL_ADDRESS>wrote:

The Simulation interface does not support birth and death processes. Does
it make sense to add this to dworp.simulation - something like
BirthDeathSimulation with hooks that developers implement to define the
birth and death processes?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ACI-ESP/dworp/issues/17, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqBnNETgTRrtftcnTA5eYuBjkkXlXZRks5twzX4gaJpZM4T4yjo
.


Invalid parameter -Title used instead of -Identity in Example 3

The command shown in Example 3 contains an invalid parameter -Title.
The correct parameter is -Identity.

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

ID: eb2d3c28-638c-262d-b851-de589cedf3c0
Version Independent ID: 3054989c-a9fb-74aa-9121-fdb24b1a3f65
Content: Remove-PnPList (SharePointPnPPowerShell)
Content Source: sharepoint/sharepoint-ps/sharepoint-pnp/Remove-PnPList.md
GitHub Login: @Techwriter40
Microsoft Alias: kirks

Hello @pramodghuge
how are you?
Even though the PnP documentation articles are on this repository, I need to kindly redirect you to open PnP cmdlets issues here, in that repository, where a member of the PnP community will be able to correct the article with your feedback.
The documentation for PnP cmdlets is then sync from that repo into this one.
(So, Pull Requests created against the documentation on this repo, if merged, will be rewritten by that process)
Thank you!


✨ Create utility to delete instance by id in the hub

A proposal to fix the violation of DRY:

https://github.com/laminlabs/lamindb-setup/issues/690


D3D11 rendering stops working if multiple windows are resized at once

In my app I have two windows - one for editing and one for game view. If they are resized both in the same frame, one of them do not display anything anymore, until its framebuffer is recreated again.
This can be easily reproduced in example 22
Steps to reproduce:

add following code to example 22
static int ii = 0;
bgfx::reset(1280+ii, 720);
++ii;

add it here:
if (NULL != win.m_nwh)
{
	fbh[viewId] = bgfx::createFrameBuffer(win.m_nwh, win.m_width, win.m_height);
}

start the app

press c to create a new window

observe that the new window does not display anything

Note: Win7, D3D11
Note 2: if there is bgfx::frame(); between reset and createFrameBuffer, everything works fine

I found an easier way to reproduce this:

start example 22
press c to create new windows
resize the original window
observe that cubes in the new window stop rotating

Cool, I think I know what is it... I will fix it soon.

It should be fixed now.

Great, it works. Thank you.


{AzureIoT} fixes Azure/azure-cli#26305 Removed az_iotcentral word

Removed az_iotcentral command
Issue reference : https://github.com/Azure/azure-cli/issues/26305

@SwathiDhanwada-MSFT, This update is made to an autogenerated article and will be overwritten.  It is my understanding that the source code for IoT is in the Azure/azure-iot-cli-extension repo, but the Azure CLI engineering team will route your 26305 issue appropriately.


Allow conditional expressions to compare against an array

Hi,
Not sure if this is a enhancement request or an issue.
We have a bicep module that creates a vnet and subnets based on values in an object.
We are trying to use a conditional expression which compares a name against multiple values which are assigned to an array and if the name matches a name which is found in the array the resource (a subnet in this instance) will be created with that name, otherwise it will use the parameters from the object and add a prefix defined in the module.
for example if the subnet name is GatewaySubnet it will look in the array and see GatewaySubnet is there and it will not add the prefix.
I've included the module below.
param location string
param vNetPrefix string
param subnetPrefix string
param vNetAddressSpace string
param subnetAddressSpaces object
var servicesubnets = [
   'GatewaySubnet'
   'AzureFirewallSubnet'
   'AzureFirewallManagementSubnet'
]

resource virtualNetwork 'Microsoft.Network/virtualNetworks@2021-03-01' = {
  name: '${vNetPrefix}-01'
  location: location
  properties: {
    addressSpace: {
      addressPrefixes: [
        vNetAddressSpace
      ]
    }
    subnets: [for item in items(subnetAddressSpaces): {
          name: '${item.value.name}' ==  '${item.value.name}' ? '${servicesubnets}' : '${subnetPrefix}-${item.value.name}'
          properties: {
            addressPrefix: item.value.addressPrefix
        }
      }]
  }
}

The error when trying to run this is below.
ERROR: {'code': 'InvalidTemplate', 'target': '/subscriptions/sub-id/resourceGroups/rg-id/providers/Microsoft.Resources/deployments/vnetDeploy', 'message': "Deployment template language expression evaluation failed: 'Unable to parse language expression ''GatewaySubnet','AzureFirewallSubnet','AzureFirewallManagementSubnet'': expected token 'EndOfData' and actual 'Comma'.'. Please see https://aka.ms/arm-template-expressions for usage details.", 'additionalInfo': [{'type': 'TemplateViolation', 'info': {'lineNumber': 1, 'linePosition': 1627, 'path': 'properties.template.resources[0].properties'}}]}

Thank you

Couple of things going on here:

I cleaned up this line because there was some unnecessary string interpolation:

          name: item.value.name == item.value.name ? servicesubnets : '${subnetPrefix}-${item.value.name}'

At which point, we throw a warning, which points us closer to the problem (discussed in point 3 below):

That comparison you are doing will always be true since you are asking if item.value.name is equal to itself. That does not look to be your intention.
the second argument is an array, but it should be a string. The way this expression is written, you are trying to set the value of *.properties.subnets[*].name to the value of [ 'GatewaySubnet',  'AzureFirewallSubnet', 'AzureFirewallManagementSubnet'], but presumably you only want to set it to one of those values (i.e. servicesubnets[0])

We are trying to use a conditional expression which compares a name against multiple values which are assigned to an array

If this is what you are trying to do, I think you are looking for some of the functionality described in #2065, which we don't have right now. Maybe @brwilkinson has some creative ways to solve this.
First @scottbrien01, I'd recommend taking another look at the intent of this code and do some refactoring, but either way you will likely be limited.

@scottbrien01 can you please provide default parameter values for all of your parameters.

Hi Alex/Ben,
Just to explain how we got here, initially our code had the below line
name: '${item.value.name}' ==  'GatewaySubnet' ? 'GatewaySubnet' : '${subnetPrefix}-${item.value.name}'
We wanted to expand on that and if the subnet name included other Azure service specific subnet names e.g AzureFirewallSubnet  to name the subnet that else to add the subnet prefix to the name defined in the parameter.
Below is the main bicep file with parameters.
targetScope = 'subscription'
// Define naming convention parameters
param location string = 'Australia East'
param customerPrefix string = 'sb'
param environmentPrefix string = 'dev'
param appPrefix string = 'hub'
var locationSearch = {
'Australia East': {
prefix: 'auea'
}
'Australia Southeast': {
prefix: 'ause'
}
'Australia Central': {
prefix: 'auce'
}
'Australia Central 2': {
prefix: 'auc2'
}
}
var locationPrefix = locationSearch[location].prefix
var rgHubName = toLower('rg-${customerPrefix}-${locationPrefix}-${appPrefix}-${environmentPrefix}')
var vNetPrefix = toLower('vnet-${customerPrefix}-${locationPrefix}-${appPrefix}-${environmentPrefix}')
var subnetPrefix = toLower('snet-${customerPrefix}-${locationPrefix}-${appPrefix}-${environmentPrefix}')
// Define Network parameters
param vNetConfig object = {
vnetaddressspace: '<IP_ADDRESS>/20'
subnets: {
subnet1: {
name: '01'
addressPrefix: '<IP_ADDRESS>/24'
}
subnet2: {
name: '02'
addressPrefix: '<IP_ADDRESS>/24'
}
AzureFirewallSubnet: {
name: 'AzureFirewallSubnet'
addressPrefix: '<IP_ADDRESS>/26'
}
AzureFirewallManagementSubnet: {
name: 'AzureFirewallManagementSubnet'
addressPrefix: '<IP_ADDRESS>/26'
}
GatewaySubnet: {
name: 'GatewaySubnet'
addressPrefix: '<IP_ADDRESS>/26'
}
}
}
// Hub Resource Group
module rghubnetwork './modules/management/modResourceGroup.bicep' = {
name: 'rgHubNetworkDeploy'
params: {
location: location
rgName: rgHubName
}
}
// Hub Virtual Network
module hubvnet './modules/network/modVirtualNetwork.bicep' = {
name: 'vnetDeploy'
scope: resourceGroup(rgHubName)
params: {
location: location
vNetPrefix: vNetPrefix
vNetAddressSpace: vNetConfig.vnetaddressspace
subnetAddressSpaces: vNetConfig.subnets
subnetPrefix: subnetPrefix
}
dependsOn: [
rghubnetwork
]
}

I think in this scenario I would opt for a simple replace() with the prefix, using a replacement string e.g. {prefix}
param vNetConfig object = {
  vnetaddressspace: '<IP_ADDRESS>/20'
  subnets: {
    subnet1: {
      name: '{prefix}01'
      addressPrefix: '<IP_ADDRESS>/24'
    }
    subnet2: {
      name: '{prefix}02'
      addressPrefix: '<IP_ADDRESS>/24'
    }
    AzureFirewallSubnet: {
      name: 'AzureFirewallSubnet'
      addressPrefix: '<IP_ADDRESS>/26'
    }
    AzureFirewallManagementSubnet: {
      name: 'AzureFirewallManagementSubnet'
      addressPrefix: '<IP_ADDRESS>/26'
    }
    GatewaySubnet: {
      name: 'GatewaySubnet'
      addressPrefix: '<IP_ADDRESS>/26'
    }
  }
}

output loop array = [for (item, index) in items(vNetConfig.subnets) : {
  subnetName: replace(item.value.name,'{prefix}', appPrefix)
}]

Outputs :
Value
===============
[
{"subnetName":"AzureFirewallManagementSubnet"},
{"subnetName":"AzureFirewallSubnet"},
{"subnetName":"GatewaySubnet"},
{"subnetName":"hub01"},
{"subnetName":"hub02"}
]

I am not convinced on using a keymap for the subnets, however I haven't thought about it too much, so you may have a reason for it, I prefer an array.
https://github.com/brwilkinson/AzureDeploymentFramework/blob/main/ADF/tenants/AOA/ACU1.T5.parameters.json#L343
        "SubnetInfo": [
          {
            "name": "snMT01",
            "prefix": "0/27",
            "NSG": 1,
            "FlowLogEnabled": 1,
            "FlowAnalyticsEnabled": 1,
            "delegations": "Microsoft.Web/serverfarms"
          },
          {
            "name": "snFE01",
            "prefix": "32/27",
            "NSG": 1,
            "FlowLogEnabled": 1,
            "FlowAnalyticsEnabled": 1
          }

here are some other considerations...
generate the prefix table:
https://github.com/brwilkinson/AzureDeploymentFramework/blob/main/ADF/1-prereqs/00-Get-AzPrefixTable.ps1

This accounts for every Azure region.

This requires 3 custom naming convention overrides, however you may not even be using those regions.

  "AUS1": {
    "displayname": "UK South",
    "location": "uksouth",
    "PREFIX": "AUS1",
    "pairedRegion": "ukwest"
  }

generate the location table, including secondary/partner regions.
https://github.com/brwilkinson/AzureDeploymentFramework/blob/main/ADF/1-prereqs/00-Get-AzRegionTable.ps1
"eastus": {
    "displayname": "East US",
    "location": "eastus",
    "PREFIX": "AEU1",
    "pairedRegion": "westus"
  }

They generate these metadata files:
https://github.com/brwilkinson/AzureDeploymentFramework/blob/main/ADF/bicep/global/prefix.json
https://github.com/brwilkinson/AzureDeploymentFramework/blob/main/ADF/bicep/global/region.json
These can then be leveraged within bicep via loadtextcontent() and convert json() function.
// region to prefix
var regionLookup = json(loadTextContent('./global/region.json'))
var primaryPrefix = regionLookup[Global.PrimaryLocation].prefix

// prefix to location
var locationlookup = json(loadTextContent('./global/prefix.json'))
var location = locationlookup[Prefix].location

also maybe check the naming standards here for more info the the prefix usage etc.
https://brwilkinson.github.io/AzureDeploymentFramework/

That worked pretty well for us, thanks for that Ben and thanks for the additional tips!
Apologies for taking a while to get back.


Migrate Python API methods to MVVM

Codecov Report

Merging #1404 into master will decrease coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1404      +/-   ##
============================================
- Coverage     86.58%   86.52%   -0.07%
- Complexity     3974     3980       +6
============================================
  Files           387      387
  Lines         16001    16044      +43
  Branches       1778     1785       +7
============================================
+ Hits          13855    13882      +27
- Misses         1669     1685      +16
  Partials        477      477

Impacted Files
Coverage Δ
Complexity Δ

...api/codegen/viewmodel/OptionalArrayMethodView.java
100% <ø> (ø)
9 <0> (ø)
:arrow_down:

...ransformer/py/PythonApiMethodParamTransformer.java
98.64% <100%> (+0.92%)
24 <9> (+8)
:arrow_up:

...api/codegen/transformer/py/PythonSurfaceNamer.java
94.79% <100%> (+0.16%)
44 <4> (+4)
:arrow_up:

...n/transformer/DynamicLangApiMethodTransformer.java
99.27% <100%> (+0.05%)
30 <3> (+3)
:arrow_up:

...main/java/com/google/api/codegen/GapicContext.java
43.9% <0%> (-19.52%)
8% <0%> (-4%)

...com/google/api/codegen/py/PythonContextCommon.java
70% <0%> (-10%)
5% <0%> (-1%)

...com/google/api/codegen/py/PythonImportHandler.java
84.28% <0%> (-2.15%)
24% <0%> (-2%)

...m/google/api/codegen/config/GapicMethodConfig.java
72.22% <0%> (-1.39%)
43% <0%> (-1%)

.../com/google/api/codegen/py/PythonGapicContext.java
66.51% <0%> (-0.47%)
44% <0%> (-1%)

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 b387e49...c80e2c8. Read the comment docs.

I will take a look this afternoon.


Add "[Workshop] Bioimage data management and analysis with OMERO"

https://zenodo.org/records/11350689

done in #482


Stable release

Is this lib stable enough to have a public stable release?

Yeah, probably about time. Will tag now.


Implement domain-adaptive pretraining

@tmills I'm not sure where to put DAPT.md--feel free to make this an example in examples or something as we briefly discussed at one point.
Also, when reviewing this PR, could you go through the steps there on your end and make sure it all works?


Look up CallSets by sample ID

Allowing search by sample ID, and reworking some comments.

OK, I have brought this up to speed with master and pared it down to just the CallSet change: you can now look up CallSets by sample ID.

Nice!  Thank you for this @adamnovak :)

Can you give a quick example of the use case please @adamnovak ? How does this interact semantically with the other fields in the request? Should this be unioned with null so it can be omitted?

The use case is to look up CallSets by sample ID; if you went over to the metadata API and found a sample you care about, you will need to be able to get genotype information for it.
I meant to have semantics where an empty array doesn't apply the filter; I forgot to specify that in the comment and will fix that momentarily.

OK, it now describes the proper syntax for not using this field.

Sorry, now it actually actually describes the proper syntax for not using the field; I had amended my commit without adding the file.

Thanks @adamnovak, +1.

@adamnovak can we close given https://github.com/ga4gh/schemas/pull/636?


security meta

We fixed the problems by updating the nginx conf https://github.com/101labs-tech/near-academy/pull/27


Jack backend to master

this broke things before presentation, had to revert. consider revisiting/bug-fixing


Thank you! [Not issue]

Thanks for this wonderful repo! Helped me solve some issues I was having with my S3.

Yes, it was a very long ride!


Refactor XML parser

Start pulling out the XML node handlers into objects (strategy pattern).

Refactor complete, but not quite the way I want it going forward.  It does provide a decent platform for future extended options though.


Missing line break after Howdy Boy interaction

Pressure Pier
Water south, passage north. You smell food to the west, and the land sinks a bit to the east. Ahead north, things open up further behind the Saver Screen.
The Howdy Boy stands here, looking disinterested now he's greeted you.
"Oh, hey! Still here? I'm moving ahead in life!" you say to the Howdy Boy, who runs off in embarrassment.
>

Missing a line break before the prompt.

Also right after that with the jerks:

"Oh! Hey! You all talking about something interesting? I won't disturb you. Which way is the BM?" They look shocked you...USED HIS INITIALS. They point north. "I KNOW," you boom. They scatter.
>

Also right after that:

You give a pre-emptive "Oh, I KNOW," before Turk and Dutch can say any more. "But you're doing a pretty good job. I mean, almost as good as I could if I weren't destined for better things."
>

Ah--got these. I had the proper line break in 1 ending but not the other 3. This got swept up in another transcript I had.
Also I cleared up that the endings were existential failures and the player may want to try again.


Error installing extension manually

the extension has an error when installing manually. i can ignore and it works temporarily, but when i restart my session, it's simply removed

Fixed #9


Refactoring adding init

Maybe this PR is not ready to be merge, but I've done a general refactoring moving

imageView
titleLabel
subtitleLabel
layoutSubviews

to PKHUDSquareBaseView and with a new init with optional values is possible to create all the possible combination of HUD (for example PKHUDSuccessView with title, only subtitle of both).

Some classes (PKHUDTitleView, PKHUDSubtitleView, PKHUDStatusView, PKHUDStatusProgressView) became obsolete but I think that only PKHUDStatusProgressView should be removed for compatibility reason.

Very cool work. This was on my plate for far too long and now you've done it. Very cool. Before I merge this, could you do me one favor though and create a CONTRIBUTORS file in the root of the directory, and add the name + email (if available) of all who have contributed to this repo?

I've declared as a WIP because some files can be deleted and I want discuss it with you.
Do you agree with me to remove only PKHUDStatusProgressView?

So I actually like being fairly aggressive about getting things forward even if that means breaking something along the way. It's only gonna hurt more down the road than it would if we did it right now.
Obviously we'll need a version bump to 3.0 since we're breaking API contracts.
But keeping too much stuff around in such a simple component only increases surface area unnecessarily and makes things complex than they need to be for new users.
So since we'll already be breaking compatibility by removing PKHUDStatusProgressView, we might as well go ahead an remove all the others that are redundant in favor of the convenience initializers.

PKHUDStatusProgressView has been introduced recently and never been added to Cocapods for that reason I suggest the removal, it's not necessary.
If you want to bump a new version breaking the compatibility we can also add the Determinate Progress Bar as mentioned here (#51, by the way I'm still waiting for an answer) and develop other improvements.

I've removed the not necessary class.  Do you think is necessary the file CONTRIBUTORS? All the information are available at this link CONTRIBUTORS

@pkluz please can you answer to my questions? There are a lot of things to do and there are other PR like #69 (it seems a very good PR) that could case conflicts with this.
Thanks

@PGLongo
It will definitely cause conflicts. I'll adapt the wrapper once this PR is merged.

Ok @Flar49. By the way great job with your PR!

Thanks ;)

@Flar49 All right let's stick to the GitHub contributors only. Regarding the spinner I'm not sure I want to pull in a dependency - it is pretty however... Let me get back to you on that - will merge this PR over the weekend though.

Ok, great for the merge.
Discussing about dependency, I've some doubts because I don't like to reinvent the wheel and for that reason I'm using a lot of Pods but that kind of view should be easy to implement so we can also avoid the dependency.
I will make some tries in the weekend.

@pkluz  congratulations for the 100th commit and the great merge work you made today!

@PGLongo Thanks, and sorry again for the immense delay... I blame work ;-)

Don't worry. I can understand

Working on getting #69 updated + merged. Gonna bump spec after. Prolly bump to 3.0 as I've been breaking things along the way. And the introduction of a secondary - simpler - api might require it.

Sure, also for me bumping 3.0 is the best solution.


Windows 11 10.0.22000.2124 not supported

termsrv.zip

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/836ee5ca5d87c58f61083b087c0883c0a1ff0737

sebaxakerhtc/rdpwrap.ini@836ee5c

Worked very well. 😄  Thank you!
-Gamma


Add listener functionality to receive emails

Purpose
Receive emails as a trigger from the email receive operation of an external POP3/IMAP4 server.
Fixes #22844
Check List

[x] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[x] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[x] Increased Test Coverage
[ ] Added necessary documentation

[x] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #22843 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #22843   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

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 a1eb83b...37525b9. Read the comment docs.


[Bug] Using Base64 encode in header not working after update

Insomnia Version: Version 6.4.1
Operating System: Ubuntu 18.04

Details
Insomnia was updated today (2019-04-25) at some point automatically, and now that I try to use my existing request, including a header with the function base64 encode, I get an error, and the header value is empty when posting:

The variable basic_auth is in my environment, and is like so:
"basic_auth": "{{ client_id  }}:{{ client_secret  }}"
I double checked that the client_id and client_secret are correctly displayed in the variables.
So this seems to be a bug in the base64 encoding functionality.

Ah, it looks like there was a breaking change in the last release for this tag (sorry about that). You'll need to switch the Kind selection to "Normal" and move your basic_auth variable into the Value parameter.

I tried doing the above and it still couldn't resolve the error. I had to remove and re-add every encoding tag that I had.

Thanks for letting me know. This breaking API change definitely should not have ended up in production but it's a bit late now to fix it, unfortunately.


Complete PMP configuration CSRs and address CSRs are not supported for 64 PMP entries

Arch-tests do not have the support of following CSRs in encoding.h file.

pmpcfgx where x is [4,15]
pmpaddrx where x is [16,63]

Little correction: The file present by default in riscv/riscv-opcodes is not complete but the most up-to-date version (even latest than what is in riscv-isa-sim/riscv) of the file can be generated by running make in riscv/riscv-opcodes.

I've filed an issue to try to ensure that the version in the repository is always the latest.
The question is how the framework pulls it in, or should the framework always regenerate it as you did
In either case it doesn't need to be in the repo at all.

According to Andrew's comment on your issue (here) for now, we need to regenerate it every time. How about writing a makefile that clone riscv/riscv-opcodesriscv/riscv-opcodes and build encoding.h locally to place in riscv-arch-test/riscv-test-suite/env. What do you say

We could either

clone it as you said

add a CI script to the riscv-opcodes repo to produce it as a
release and simply copy it from as part of make

3.copy it from the spike directory.

Short term, your solution is best.
Longer term: I'd like to see the CI script solution so others can use the
same approach, if that is possible
On Thu, Jun 30, 2022 at 8:30 AM Wadood @.***> wrote:

According to Andrew's comment on your issue (here
https://github.com/riscv/riscv-opcodes/issues/132#issuecomment-1170642498)
for now, we need to regenerate it every time. How about writing a makefile
that clone riscv/riscv-opcodes https://github.com/riscv/riscv-opcodes
riscv/riscv-opcodes https://github.com/riscv/riscv-opcodes and build
encoding.h locally to place in riscv-arch-test/riscv-test-suite/env
https://github.com/riscv-non-isa/riscv-arch-test/tree/main/riscv-test-suite/env.
What do you say
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/263#issuecomment-1171364395,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJVWF3XLN643QU4AAITVRW4RXANCNFSM52GJRZCQ
.
You are receiving this because you commented.Message ID:
@.***>

Hi @allenjbaum, I have implemented a long-term solution by writing a CI script. Kindly review PR #268


Improvement/tooltip

Describe pull-request
Describe what the pull-request is about
Added offsetskidding and offsetDisatnce prop to tooltip component, to control the placement of tooltip
Solving issue
Add which issue this pull-request solves by adding # plus the number of the issue (for example #123)
Fixes: #
How to test
Add description how to test if possible

Go to storybook > component > tooltip
Test the new attribute

Looks good!


Uploading files using FIPS endpoints is broken

The following code, inspired from https://docs.min.io/docs/golang-client-quickstart-guide.html will fail because it is using a FIPS endpoint and the GetRegionFromUrl method doesn't handle FIPS endpoints.
package main

import (
	"context"
	"log"
	"os"

	"github.com/minio/minio-go/v7"
	"github.com/minio/minio-go/v7/pkg/credentials"
)

func main() {
	ctx := context.Background()

	endpoint := "s3-fips.us-east-1.amazonaws.com"
	useSSL := true

	accessKeyID := os.Getenv("AWS_ACCESS_KEY_ID")
	secretAccessKey := os.Getenv("AWS_SECRET_ACCESS_KEY")
	bucketName := os.Getenv("AWS_BUCKET_NAME")

	// Initialize minio client object.
	minioClient, err := minio.New(endpoint, &minio.Options{
		Creds:  credentials.NewStaticV4(accessKeyID, secretAccessKey, ""),
		Secure: useSSL,
	})
	if err != nil {
		log.Fatalln(err)
	}

	objectName := "test_file.zip"
	filePath := "/tmp/test_file.zip"
	contentType := "application/zip"

	// Upload the zip file with FPutObject
	info, err := minioClient.FPutObject(ctx, bucketName, objectName, filePath, minio.PutObjectOptions{ContentType: contentType})
	if err != nil {
		log.Fatalln(err)
	}

	log.Printf("Successfully uploaded %s of size %d\n", objectName, info.Size)
}

$ go run FileUploader.go
2022/06/10 17:02:30 The authorization header is malformed; the region 'fips.us-east-1' is wrong; expecting 'us-east-1'
exit status 1

https://github.com/minio/minio-go/pull/1663 is my WIP attempt to fix it.


Undefined Variable issue in fcm push 1.1.0

There are two variables anyFail & anySuccess  which not declared but used in lib/fcm.js (line 73-83). I found it is fixed in latest master branch but we are not able to get it via npm install since valid install targets are 1.1.0, 1.0.7, 1.0.6, 1.0.5, 1.0.3, 1.0.2, 1.0.1, 1.0.0
How can we install fcm-push 1.1.2 via npm install? Please find the fcm-push code below
                `anyFail = ((JSON.parse(data)).failure > 0);

                    if (anyFail) {
                        error = data.substring(0).trim();
                    }

                    anySuccess = ((JSON.parse(data)).success > 0);

                    if (anySuccess) {
                        id = data.substring(0).trim();
                    }`

Can't, publisher (@nandarustam) should publish it to registry as it is

Hi folks, just published 1.1.2.
Very much thanks Mas @ans-4175 for being such a huge help to this fcm module.
Me so sorry for being less helpful, will try my best 🙏

Thank you @nandarustam!!!


The controller specs don't run the callbacks setup in JSONAPI::ActsAsResourceController

I have the following controller spec
describe "'POST' create" do
  before do
    @data = body_params[:data]
  end

  subject do
    -> { post :create, data: @data }
  end

  it 'creates the article' do
    is_expected.to change(Article, :count).by(1)
  end

  it 'does not respond with error' do
    subject.call
    expect(data['errors']).to eql(nil)
  end
end

which passes the test. However if I request to the same call with the same body, I get an error from the #setup_request from JSONAPI::ActsAsResourceController module. Does this mean on the controller spec didn't run the callback? How to set the spec to make sure it's running the same way on an actual request call?

@apraditya are you still having this problem?
Let me suggest using request specs instead of controller specs.
Especially for testing API endpoints.
There are talks around community to remove controller tests from Rails
completely.
https://github.com/rails/rails/issues/18950#issuecomment-77924771


FFmpeg编译失败

描述(Description)
git clone https://github.com/ossrs/srs
cd srs/trunk
git pull
./configure --with-nginx --with-ffmpeg && make
build x264
Found no assembler
Minimum version is nasm-2.13
If you really want to compile without asm, configure with --disable-asm.
build x264 failed
build ffmpeg-4.1 failed, ret=1
环境(Environment)
uname -a
Linux localhost.localdomain 2.6.32-754.el6.x86_64 #1 SMP Tue Jun 19 21:26:04 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
cat /etc/redhat-release
CentOS release 6.10 (Final)

是需要手工安装x264吗？

源码安装最新版的ffmpeg后编译srs成功了
wget http://www.ffmpeg.org/releases/ffmpeg-4.2.tar.gz
tar -zxvf ffmpeg-4.2.tar.gz
cd ffmpeg-4.2
./configure
make
make install

See https://github.com/ossrs/srs/issues/1523


Fix endpoint hanging

Signed-off-by: Romy<EMAIL_ADDRESS>Explain the changes

Promise.all() hangs due to unhandled rejection, replaced it with 3 different awaits.

Issues: Fixed #xxx / Gap #xxx

Fixed #6856

Testing Instructions:

@romayalon I'm curious where did an unhandled rejection cause this issue with Promise.all, and why does awaiting one-by-one fix it?

@guymguym I updated the PR, the older code fixed just one flow of this issue.


Init add system-library charts

issues: https://github.com/rancher/rancher/issues/15702
(+) prepare for rancher monitoring

this pr caused test_project_catalog_creation and test_cluster_catalog_templates_access to fail

https://github.com/rancher/rancher/pull/16777 will merge the commits in this PR, so close it. FYI, @cjellick


Unable to fetch secrets

Since this morning all my workflow has been failing with the following error, Any support would be super helpful. They were working fine till last week and on a side note, I was able to access secrets via curl to the OP_CONNECT_HOST endpoint
Run 1password/load-secrets-action@v1
/usr/bin/sh -c /home/runner/work/_actions/1password/load-secrets-action/v1/entrypoint.sh
Authenticated with CONNECT
Archive:  op.zip
extracting: /usr/local/bin/op.sig
inflating: /usr/local/bin/op
Populating variable: XXX
Error:  2022/12/19 14:11:34 could not read secret op://XXX/XXX/password: could not retrieve vaults: Get "XXX/v1/vaults": unsupported protocol scheme ""
Error: The process '/usr/bin/sh' failed with exit code 1

I was wondering if anyone could help :)  Here below is the snippet of the action workflow file
all our pipelines stopped working and it would be great to get some hints to solve this issue. Apologies in advance for the rush
      - name: Set environment variables
        run: |
          echo "ONE_PASSWORD_GITHUB_ACCESS_TOKEN=${{ env.ENVIRONMENT }}_ONE_PASSWORD_GITHUB_ACCESS_TOKEN" >> $GITHUB_ENV
          echo "OP_CONNECT_HOST=${{ env.ENVIRONMENT }}_OP_CONNECT_HOST" >> $GITHUB_ENV

      - name: Load secret
        id: load_secrets
        uses: 1password/load-secrets-action@v1
        env:
          OP_CONNECT_HOST: 'one-p-LoadB-x-x.x.com'
          OP_CONNECT_TOKEN: ${{ secrets[env.ONE_PASSWORD_GITHUB_ACCESS_TOKEN] }}
          GITHUB_ACTION_USER_ACCESS_KEY: op://Engineering_Staging/GITHUB_ACTION_USER_ACCESS_KEY/password
          GITHUB_ACTION_USER_KEY_ID: op://Engineering_Staging/GITHUB_ACTION_USER_KEY_ID/password
          GITHUB_ACTION_USER_ARN: op://Engineering_Staging/GITHUB_ACTION_USER_ARN/password
          POSTGRES_HOST: op://Engineering_Staging/POSTGRES_HOST/password
          POSTGRES_PORT: op://Engineering_Staging/POSTGRES_PORT/password
          POSTGRES_USER: op://Engineering_Staging/POSTGRES_USER/password
          POSTGRES_DB: op://Engineering_Staging/POSTGRES_DB/password
          POSTGRES_PASSWORD: op://Engineering_Staging/POSTGRES_PASSWORD/password
          S3_BUCKET_STACKS: op://Engineering_Staging/S3_BUCKET_STACKS/password

We are facing the similar issue. But don't see any error except a warning
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

So after the action executed. We are missing variables in the deployed app.

In my case, it fails when attempting to fetch the secrets. I tried to replicate the same on OP CLI and was also seeing the same behavior ( https://developer.1password.com/docs/ci-cd/github-actions)
output from OP CLI
op read op://X/x/password
[ERROR] 2022/12/20 10:19:39 could not read secret op://x/x/password: could not retrieve vaults: Get "one-p-x.com/v1/vaults": unsupported protocol scheme ""

We managed to solve this issue by adding the following code:
with:
  # Export loaded secrets as environment variables
  export-env: true

No idea why it stopped working this week.

Hey @mathis-lab
I tried your suggestion but still it's the same error for us, unfortunately :( could you see if I have export-env set rightly?

It seem that it is 2 differeents issues. Our issue was the same than @gihan9a.

@florisvdg
I was wondering if you could have a look at this problem and suggest a solution for this problem am facing. That would help us resume our CICD pipelines
I posted on the 1password community forum and the support channel hasn't got any solution yet.
Regards
Rakesh

Hey all!
I'm really sorry to hear that the latest version of the GitHub action doesn't work as expected! I've updated the v1 tag to point to the latest version of the action (currently v1.1.2), which it looks like it brought up bugs that we were not aware of.
Let me try to reply to the concerns raised here so far.

Error: 2022/12/19 14:11:34 could not read secret op://XXX/XXX/password: could not retrieve vaults: Get "XXX/v1/vaults": unsupported protocol scheme ""

This is caused by the fact that the host provided doesn't have the protocol prefix (i.e. HTTP:// or HTTPS://). Looking back at the changes done at the action, it may have slipped that curl command works without the prefix as well (today I learned about that). Therefore, adding the suitable prefix for your use case should do the trick for now. In the meantime, I'll investigate and see if we can adjust the action so that your previous use case works again, as well as highlight in our documentation the format that OP_CONNECT_HOST expects.

We managed to solve this issue by adding the following code:
  # Export loaded secrets as environment variables
  export-env: true

No idea why it stopped working this week.

It looks like the latest changes seem breaking since the default of that value is false. I will come back with a quick fix for that and change the default to true so that the current workflows don't fail.
Again, apologies for the sudden unfortunate events with your pipelines and thank you for raising them with us. 😃

I've just opened a PR that brings the suggested changes to the action: #25

Hey all,
We've just released version v1.2.0 that contains the changes to ensure that v1.x.x releases stay backwards compatible with earlier v1 versions.
Also, the v1 tag now points to that version, so now the broken pipelines should be fixed.
I will close this issue for now, but feel free to reopen if other unexpected behavior rises.


Wrongly "is not assignable to type" for Object

TypeScript Version:  3.7.2

Search Terms:
Code
// A *self-contained* demonstration of the problem follows...
// Test this by running `tsc` on the command-line, rather than through another build tool such as Gulp, Webpack, etc.
type a = 'a'
type ab = a | 'b'
interface b {b: ab};

type a = 'a'
type ab = a | 'b'
interface b {b: ab};

function ca(d: a) {}
function c(d: ab) {
    if(d === 'b') {}
    else ca(d);
}

function da(d: {b: a}) {}
function d(d: b) {
    if(d.b === 'b') {}
    else da(d);
}

Expected behavior:
It should compile.
Actual behavior:
TS2345: Argument of type 'b' is not assignable to parameter of type '{ b: "a"; }'.
Types of property 'b' are incompatible.
Type 'ab' is not assignable to type '"a"'.
Type '"b"' is not assignable to type '"a"'.
Playground Link:
Not yet.
Related Issues:
Not yet.

Duplicate #10065


Grub doesn't load encrypted disk if the drives are not empty during installation

Similar to #3 but only occurs if the drives contain an encrypted volume that is closed before the installation.
This probably means that erasing the disks isn't implemented properly since the encrypted volume isn't opened and thus not listed in lsblk.

Edit because the error also occurs while the crypt volume is opened during install.

The original description is wrong! This is exactly the same issue as #3 but probably for another reason.
It might have to do with /dev/md0 being displayed as /dev/md127 in archiso


Respond to requests with Accept header set to application/json with JSON

Responding with JSON currently requires the /format/json URL parameter, but if the Accept request header is application/json, the controller should respond with JSON even without the flag (provided JSON is enabled for that controller).

As implemented, this fix only accommodates headers that exactly equal application/json, but the Accept header can have multiple weighted values.
Use the same technique implemented for Accept-Encoding (see line ~600 in server.js) to sort through multiple Accept values, order them based on priority, and use the first available format that matches.
for ( var i = 0; i < acceptEncoding.length; i++ ) {
  acceptEncoding[i] = acceptEncoding[i].trim().split(';')
  acceptEncoding[i][1] = acceptEncoding[i][1] ? +querystring.parse(acceptEncoding[i][1]).q : '1'
}

for ( i = 0; i < acceptEncoding.length; i++ ) {
  if ( acceptEncoding[i][1] > weight ) {
    encoding.unshift([acceptEncoding[i][0], acceptEncoding[i][1]])
    weight = acceptEncoding[i][1]
  } else {
    encoding.push([acceptEncoding[i][0], acceptEncoding[i][1]])
  }
}

for ( i = 0; i < encoding.length; i++ ) {
  // Use the appropriate encoding if it's supported
  if ( encoding[i][1] && ( encoding[i][0] === 'gzip' || encoding[i][0] === 'deflate' || encoding[i][0] === 'identity' ) ) {
    params.request.client.encoding = encoding[i][0]
    break
  }
}


Nametag Problem

Its been really annoying that everyone can see each other's names even when a admin is in OB or no-clip people can see their names. I been trying everything to remove this problem but nothing works. This can be used as meta and im trying to prevent that. Image below:

I don't understand.

A player's name shows above his head to those who recognise him, otherwise the physical description will show. MPF names show without having to be recognised initially.


Added name validation and cloud-init update to overview card

Fixes: https://github.com/oVirt/ovirt-web-ui/issues/803
When name is invalid, save button is disabled:

If name is valid, appear checkbox and user can choose to update cloud-init hostname or not:

@bond95 - I updated the CSS to remove the unused switch css classes.  Add a unit test for the hostname helper function and make sure it works/fails for a reasonable set of names.

ci build please

Everything works fine and PR can be merged.
I have a few small nitpicks:

if a new VM name is not a valid host name, we could maybe tell the user why it is not valid, or what criteria there are for a valid host name.
criteria label for a valid VM name doesn't tell us how long can it be.

@bond95 - Needs a rebase and then looks like it's all set.


show users list when user is joined

fixes #2 Shows users lists when user is joined in the channel. @aviaryan

I had tried this fix before and it didn't work for me (the layout gets disturbed on late addition of box, screenshot attached).
Sadly, it looks to be a platform-specific issue. @chjj Would appreciate your thoughts.


No tests matched when we run on specific test

I noticed the behavior of parser to discover the position of test only supports a string in describe(string). It will not work if we have the real class name like this example below:
describe(Calculator, () => {    // <---- will not match
  it("sum a and b", () => {     // <---- run the nearest test
    expect(new Calculator().sum(1, 2)).toEqual(3);
  });
});

describe("Calculator", () => {    // <---- works
  it("sum a and b", () => {       // <---- run the nearest test
    expect(new Calculator().sum(1, 2)).toEqual(3);
  });
});

It seems if the test or suite name isn't a singly or doubly quoted string, test detection does not work for that test/suite. Template literals and class/function references cause this issue.
I've also noticed this same issue in neotest-vitest - which seems to be based on neotest-jest.


vs-bicep sanity testing: The main.bicep file does not show an error after creating a bicepconfig.json file in the same folder with contents

OS Version/SKU: Win10 22H2/Enterprise
VS Version/SKU: 17.12.0 Preview 2.0 [35213.224.main]
Bicep Build: 34197
Repro Steps:

Create a new main.bicep file with these contents:

resource a 'Astronomer.Astro/organizations@2023-08-01-preview' = {
	name: 'name'
	location: 'westus'
}

Verify that no errors.
Don't close the main.bicep file.
Create a new bicepconfig.json file in the same folder with these contents:

{
	"analyzers": {
		"core": {
			"verbose": false,
			"enabled": true,
			"rules": {
				"use-recent-api-versions": {
					"level": "error"
				}
			}
		}
	}
}

View -> Error List.
Check whether there is an error in "Error List".

Expect:
There is an error in "Error List".

Actual:
There is no error in "Error List".

@stephaniezyen Could you please try it again, but this time, create a project first (shouldn't matter what kind, e.g. c# project), and create the main.bicep file inside the project's folders.
It seems to work for me in that scenario.  Thanks!

@StephenWeatherford  Verified on both Console App project and Azure Cloud Service (extend support) project that this issue still reproduces.

RECOMMENDATION: won't fix
WORK-AROUND: Close/reopen the main.bicep file

Won't fix


Deployment script for contract should include verification script

@xBalbinus verification through filfox verification api?

Yep, that should be fine.
On Tue, Sep 10, 2024 at 9:37 AM Parva Jain @.***> wrote:

@xBalbinus https://github.com/xBalbinus verification through filfox
verification api?
—
Reply to this email directly, view it on GitHub
https://github.com/FIL-Builders/fil-frame/issues/7#issuecomment-2341448050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASH4FPR5KF5P6NL4ZM2B5YTZV4N6FAVCNFSM6AAAAABN7CZDYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNBRGQ2DQMBVGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

I think I had finished this already during the hack, but feel free to add on.


Release new versions of the existing data packs

[ ] afk

[ ] Test in min
[ ] Test in max

[ ] afk-sleep

[ ] Test in min
[ ] Test in max

[ ] afk-message

[ ] Test in min
[ ] Test in max

Well, that was fast. Guess I'll start prepping the releases!

I think I want to create a new release of No Free Deaths that doesn't support old versions, too.


fromElement selfClosingTags

Try pasting this code in the page HTML, enable fromElement: true and reset the canvas.
<mjml>
  <mj-body>
    <mj-container>
      <mj-section>
        <mj-column>
          <mj-social facebook-content="Share" twitter-content="Tweet" google-content="+1" instagram-content="Share" linkedin-content="Share" pinterest-content="Pin it"/>
          <mj-text>Test</mj-text>
        </mj-column>
      </mj-section>
    </mj-container>
  </mj-body>
</mjml>

The MJML code remains the same but the HTML doesn't include the Text component.
A workaround is to close the mj-social tag like so:
...
          <mj-social facebook-content="Share" twitter-content="Tweet" google-content="+1" instagram-content="Share" linkedin-content="Share" pinterest-content="Pin it"></mj-social>
          <mj-text>Test</mj-text>
...

Then the Text component is visible when resetting the canvas.

closing this as we already have an issue for self-closing tags: https://github.com/artf/grapesjs-mjml/issues/149


Doc - Describe mock_call using Rustdoc comments (three slashes: ///)

In an effort of having fully documented code, we need our mocking process to be documented.
Please proceed to comment above the pub fn mock_call declaration in an equivalent way as pub fn compile is documented.

I take


Pruning-Archival service refactoring and unexpected behaviour fix

This is issue is for improving and eventually fixing Pruning-Archival service written in go.
This thread can be used for continuous discussion and progress of the changes required for this service to work properly.
Flow diagram for the existing service can be found here

need to understand flaws and gaps in current flow and compare with expected behavior and improve
update the service with updated behaviour

Current Behaviour:
Does not seem to prune & archive DAG project chains consistently.
Expected Behaviour:
Should periodically prune DAG chains and archive and maintaine valid state.
This behaviour is already detected and thus needs to be fixed.
Do you want to work on this issue?
Yes (WIP)

Added code analysis report for production code with linting changes

Changes related to pruning-archival service are merged along with dag-verifier service. PR #10


refactor noms to use kingpin instead of hand-rolled flags parser

Initial work on #2157:
Summary
This PR switches the CLI to use kingpin, which has a number of advantages, namely:

arbitrary depth contextual commands and help. This will remove weird hacks like noms ds --delete <ds>, which should be noms ds delete <ds>, and allow for creating "suites" of CLI tools, like noms blob [command].
don't hide basic usage docs under thing help commands, show it at the root and provide details when thing --help'd
no need to check args.length anymore, can just mark args required
everything is named, so stuff is a bit easier to parse
non-defined args are nil when not provided by the user now that they are pointers. Could simplify things like using -1 as a proxy for nil
easier CLI doc customization
free shell completion and manpage generation

in general, it's also nice to have a lib handle deciding how to represent things like optional params and short flags, so it'll be consistent.
I opened this PR for initial eyes, but I'll at minimum break out the dataset command on this PR, and possibly start adding commands from #2157 (e.g. blob) on here depending on the temperature of the room.
TODO

[ ] @<file> stuff for file paths?
[ ] Restore or intentionally remove profiling
[ ] Fix any broken tests
[ ] Simplify verbosity and remove old flags libs

examples
output from noms:
usage: noms [<flags>] <command> [<args> ...]

Noms is a tool for showing off Noms data.

Flags:
  --help  Show context-sensitive help (also try --help-long and --help-man).

Commands:
  help [<command>...]
    Show help.

  commit [<flags>] <database> [<absolute-path>]
    Commits a specified value as head of the dataset

    If absolute-path is not provided, then it is read from stdin. See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the dataset and absolute-path arguments.

  config
    Prints the active configuration if a .nomsconfig file is present

  diff [<flags>] <object1> <object2>
    Shows the difference between two objects

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the object arguments.

  ds [<flags>] [<database>]
    Noms dataset management

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the database argument.

  log [<flags>] <path-spec>
    Displays the history of a path

    See Spelling Values at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the <path-spec> parameter.

  merge [<flags>] <database> <left-dataset-name> <right-dataset-name> <output-dataset-name>
    Merges and commits the head values of two named datasets

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the database argument.

    You must provide a working database and the names of two Datasets you want to merge. The values at the heads of these Datasets will be merged, put into a new Commit object, and set as the Head of the third provided Dataset name.

  root [<flags>] <database>
    Get or set the current root hash of the entire database

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the database argument.

  serve [<flags>] <database>
    Serves a Noms database over HTTP

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the database argument.

  show [<flags>] <object>
    Shows a serialization of a Noms object

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the object argument.

  sync [<flags>] <source-object> <dest-dataset>
    Moves datasets between or within databases

    See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the object and dataset arguments.

  version
    Print the noms version

output of noms merge --help
usage: noms merge [<flags>] <database> <left-dataset-name> <right-dataset-name> <output-dataset-name>

Merges and commits the head values of two named datasets

See Spelling Objects at https://github.com/attic-labs/noms/blob/master/doc/spelling.md for details on the database argument.

You must provide a working database and the names of two Datasets you want to merge. The values at the heads of these Datasets will be merged, put into a new Commit object, and set as the Head of the third provided Dataset name.

Flags:
      --help      Show context-sensitive help (also try --help-long and --help-man).
      --policy=n  conflict resolution policy for merging. Defaults to 'n', which means no resolution strategy will be applied. Supported values are 'l' (left), 'r' (right) and 'p' (prompt). 'prompt' will bring up a simple command-line prompt allowing you to resolve conflicts
                  by choosing between 'l' or 'r' on a case-by-case basis.
  -v, --verbose   show more
  -q, --quiet     show less

Args:
  <database>             a noms database path
  <left-dataset-name>    a dataset
  <right-dataset-name>   a dataset
  <output-dataset-name>  a dataset

Wow, fast work. One comment on the example output -- can it wrap the lines?

I think it might wrap based on terminal width? These are all literals so can easily manually wrap them. I think a lot of these longer messages could be made shorter by documenting more rather than linking, or by moving links to a more global place, there's quite a lot of redundancy.

This looks great. I didn't look closely at the code but based on the PR summary this is clearly an improvement.

Closing in favor of #3621


Small lie in the documentation of first

The documentation for more_itertools.first claims:

It is marginally shorter than next(iter(...)) but saves you an entire try/except when you want to provide a fallback value.

This is completely false.
The next function has always had the second argument to be used exactly as is done by first. (See the What's new in Python 2.6 document: https://docs.python.org/3.6/whatsnew/2.6.html#other-language-changes) so calling first doesn't save any try/except if you know how to use next properly.
I'd remove that final sentence and, instead, add the fact that first is simply a better name for that operation.

This was also reported in #31, and was fixed in 64ee6cc4b38349f68cb7465b3e6045c8bcf0e47b. It will be fixed in the docs for the next release.