my changes

my changes

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

This is a spam PR that not only fails to add value but actually decreases the quality of effected file.

Hi,
Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We believe it does not follow the recommended guidelines:

Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we have provided examples of the quality standards we encourage. This applies mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs. helping.
Something that's clearly an attempt to simply +1 your PR count for October.

Learn more at https://hacktoberfest.digitalocean.com/details


Remove invalid 'AS' for Oracle Queries

Fixes #2442
Problem
Remove invalid 'AS' word in Table alias for Oracle Queries
Notes
I updated the version of CASSANDRA since 3.11.12 cannot be found
Checklist

[ x] Unit test all changes
[ x] Update README.md if applicable
[ x] Run sbt scalariformFormat test:scalariformFormat to make sure that the source files are formatted

@getquill/maintainers

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

Merged! Thank you @chriseteka!


[BUG]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Enter command '....'
See error
Deployment stage 'Deploy UCS' failed with the exception: Extracting manifest: /var/tmp/install_all/plaintext.ucs Product : BIG-IP Platform: UCS : C109 System: Z101 Version : UCS : <IP_ADDRESS> System: <IP_ADDRESS> Edition : UCS : Point Release 1 System: Point Release 3 Hostname: UCS : old-hostname.mycompany.com System: new-Hostname.mycompany.com The system hostname is changed to old-hostname.mycompany.com Installing --full-- configuration on host old-hostname.mycompany.com Installing configuration... GETDBVAL: Lookup DB provision.restjavad.extramb GETDBVAL: Result >>>>> Unable to find variable: provision.restjavad.extramb <<<<< RU: No PRE; RU false; leave to default to 384 ATTENTION REQUIRED: Your previous configuration files have been archived, as listed below.
Expected behavior
A clear and concise description of what you expected to happen.

Screenshots
If applicable, add screenshots to help explain your problem.
Context

Please specify, app version:

And Platform version of the system you want to migrate.

We probably need to define exceptions and not to print the error logs from Bigip.
Cover the error detail in Journeys log file.

Please follow our SUPPORT readme: https://github.com/f5devcentral/f5-journeys?tab=readme-ov-file#bug-reporting-and-feature-requests and open a new issue using appropriate template providing sufficent data in the template.


W3C Web NFC API draft

Probably api should be rewritten to follow Web NFC API draft. Currently I don't know of any browser implementing it but might make transition easier in long term. Since it would be an breaking change it should land before 1.0 release.
Development has started in Chrome and it's under consideration in Edge. Firefox OS has an NFC Api (also mentioned in #164) but it's not following W3C Web NFC API draft.

Web NFC has been in draft a long time. When it's released I'll see what needs to be done. If you'd like to work on a conversion before let me know and we can collaborate.


Adds GC feature to controller

@steved please let me know what you think about this design at a high-level (defaults, design, etc...). I know there's a FakeClient we can leverage to write unit tests against this so I'm thinking I'd like to use that to exercise and map out the expected behaviour.
Also, I don't believe I need to pass a stop signal to the goroutine I create but let me know if you disagree. It should be terminated just fine when the main routine exits and there's no real in-flight state that could cause a race condition that comes readily to mind.

@steved This is ready to ship as-is I think but I wanted to mention a couple of things.

The tests using the fake client are in-place and exercise everything except the error modes. In order to trigger an error and build test expectations, I'd have to type-embed client.Client into a custom type and use the fake client underneath the hood. Since we don't return an error nor do we panic explicitly, I'm not sure if that's worth it. The cost: these tests are not 100% exhaustive.  Thoughts?
Do you think we should emit a failure event into K8s if an error occurs during GC? With the way this is written, the only way to tell if an error occurs is to check pod logs so we could surface this into the k8s events api as well; doesn't cost much I already have an event recorder embedded into the reconciler.

The tests using the fake client are in-place and exercise everything except the error modes. In order to trigger an error and build test expectations, I'd have to type-embed client.Client into a custom type and use the fake client underneath the hood. Since we don't return an error nor do we panic explicitly, I'm not sure if that's worth it. The cost: these tests are not 100% exhaustive. Thoughts?

Do you think we should emit a failure event into K8s if an error occurs during GC? With the way this is written, the only way to tell if an error occurs is to check pod logs so we could surface this into the k8s events api as well; doesn't cost much I already have an event recorder embedded into the reconciler.

I guess these two points are a little connected, in that if you add support for the k8s event you probably want to test it. I do like the idea of an event, though is it possible it'd spam new events if an object is undeletable? Did you mean an event on the CIB itself?

I guess these two points are a little connected, in that if you add support for the k8s event you probably want to test it. I do like the idea of an event, though is it possible it'd spam new events if an object is undeletable? Did you mean an event on the CIB itself?

I don't think there's a way to have the CIB emit events, only the reconciler. It's true that, if we find ourselves in a GC loop that cannot list nor delete CIBs, then this would emit events continually but I think the spamming would only occur if the interval is set to something low (not the default).
Consider the following:

If RunGC returns an error, how do we handle that? I'm not sure there's anything we can do which is why I don't return one.
Should we panic and kill the process altogether? Probably not since our main concern is process builds.
Should we report the error? Yes, and that's currently done in the logs but how to do we monitor that in a clean way? Log processing is, at best, mildly gross. I was thinking that events, since that can be of type EventTypeWarning, open up a concrete way to monitor for issues (i.e. something can watch events and process warnings).

Your points re: testing are valid. Let me take one more look at this and see if I can easily get that tested. Does this make sense?


Auto activation of environments does not work with multi root workspaces

Not a dup of https://github.com/Microsoft/vscode-python/issues/1476

Create two workspace folders
One with python 3, other with py 2
Ensure you have no files opened in the file tab
Create a new terminal (using + on terminal panel), extension doesn't activate the terminal based on workspace folder for which VSC created a terminal

We need VSC API to provide workspace folder for which the term was created.
Upstream https://github.com/Microsoft/vscode/issues/63052

I have a very similar issue with a multi-root workspace containing two folders. Each of the folders has it's own venv. But for both folders only the venv which is set for the first folder starts in the terminal.
I also found, if I change the positions of the folder path within the <multiroot-workspacename>.code-workspace file, always the venv of the first folder is started.
{
	"folders": [
		{
			"path": "myfolder1"
		},
		{
			"path": "myfolder2"
		}
	]
}

This example would start the venv1 of myfolder1 which is set in it's .vscode/settings.json with "python.pythonPath": "venv1/bin/python", even if I run a file from myfolder2. By the way, the default setting "python.terminal.activateEnvironment": true is active.
If I change my <multiroot-workspacename>.code-workspace file to the following, everything is run with the venv2 of myfolder2 instead.
{
	"folders": [
		{
			"path": "myfolder2"
		},
		{
			"path": "myfolder1"
		}
	]
}

Yeah, I have the same issue whenever I have a workspace with several virtualenv, even if I specify path folders in the settings.json, VSCode will always activate the first one in the terminal. A temporary workaround I found is having only one python/venv per workspace, and using the Project Manager to switch between projects.

Hi @dahelg and @Jordan-Kowal 👋  would you mind trying the latest Insiders version of the extension? https://github.com/microsoft/vscode-python/pull/7534 just got merged (for issue https://github.com/microsoft/vscode-python/issues/6282) and may help with environment activation for multi-root workspaces.
Thanks!

I think this issue maybe fixed. I tried creating the terminal in multiworkspace scenario, and it asks me this before creating the terminal. It then correctly activates the correct environment based on that.

Create a new terminal, extension doesn't activate the terminal based on workspace folder for which VSC created a terminal

Create a terminal by clicking on + on the terminal window (i.e. do not use out command).

That's what I did. But I don't see this issue.

Ok, not sure how it works for you.
This is what i did:

Create workspace folder a, add a virtual environment named helloA
Create workspace folder b, add a virtual environment named helloB
Create python files in both folders and ensure you select helloA as the interpreter for folder a, similarly helloB for b.
Create a terminal for a, then helloA should be activated in terminal
Create a terminal for b, then helloB should be activated in terminal

Yes, I did exactly the same thing. I am using the latest VS Code insiders and extension version. Asking @kimadeline to repro it.

I'm on latest insider and extension as well, & it doesn't work for me.
Please ensure you close all files when opening a terminal (there's no mention of having files opened here).

Oh right, I had files opened in the tab. I closed all the tabs and I am able to repro this. Sorry for the confusion.

@DonJayamanne  @karrtikr
I am not sure if this is a separate issue, but there sure are some similarity: I have one VSC workspace with multiple folders. There is no settings.json or .vscode folder(s), just a single .code-workspace file with all settings, launch and other workspace settings. Here is my folder structure:

folder1/:

<package 1>/
wks/:

dev.code-workspace
venv/

folder2/:

<package 2>/

folder3/:

<package 3>/
...

I want to use venv from folder1 in all folders/packages added to dev.code-workspace, so I set workspace-wide python interpretter setting to python in venv. As it currently works, venv is detected, selected and activated properly for folder1, but not for others. Others report missing python interpretter path setting and therefore also doesn't work.
Am I doing something wrong or is this a bug?

@DonJayamanne  @karrtikr
I am not sure if this is a separate issue, but there sure are some similarity: I have one VSC workspace with multiple folders. There is no settings.json or .vscode folder(s), just a single .code-workspace file with all settings, launch and other workspace settings. Here is my folder structure:

folder1/:

<package 1>/
wks/:

dev.code-workspace
venv/

folder2/:

<package 2>/

folder3/:

<package 3>/
...

I want to use venv from folder1 in all folders/packages added to dev.code-workspace, so I set workspace-wide python interpretter setting to python in venv. As it currently works, venv is detected, selected and activated properly for folder1, but not for others. Others report missing python interpretter path setting and therefore also doesn't work.
Am I doing something wrong or is this a bug?

ping <someone>
I am wondering if there is any clarification on this subject? I am really starting to wonder I am doing something wrong since no other is reporting/thumbing up this issue.


reload problem in production env.

hi,
I'm working on the https://github.com/greglobinski/gatsby-starter-personal-blog project for v2. I have published the project to netlify. There's something wrong with that. There is no problem when the page is first loaded. However, after entering a post, such as "component - src-pages-index.js" source file is not loaded when refreshed..
link : https://admiring-agnesi-af7a2e.netlify.com/
First load:

After loads:

Yup, it's a known problem #64 but I'm not longer try to solve it, I decided to rewrite the codebase from scratch using different stack #89
So, sorry but I can't help.


Integrating mailserver to an existing gsuite

Hi currently my office is using gsuite for their email, i'm planning to use this docker image for sending email from my system since gmail smtp is very unreliable. But i'm having trouble with sending or receiving the email. I am very noob on dev ops thingy, so i don't really understand where to start.
Steps i've done so far :

Created A record on my zone editor ( cPanel )

Added MX Record

All the container already running successfully
Certificate is also issued and mounted ( checked with tlscheck )

But every time i tried to send and email it never been delivered, also when i tried to send an email to the address i generated from the setup.sh -Z email add gmail says that the email address is not exists.
Here is my docker compose :
version: '3.8'

services:
  mail:
    image: tvial/docker-mailserver:latest
    hostname: ${HOSTNAME}
    domainname: ${DOMAINNAME}
    container_name: ${CONTAINER_NAME}
    ports:
      - "25:25"
      - "143:143"
      - "587:587"
      - "993:993"
    volumes:
      - maildata:/var/mail
      - mailstate:/var/mail-state
      - maillogs:/var/log/mail
      - ./config/:/tmp/docker-mailserver/${SELINUX_LABEL}
      - ./letsencrypt/etc:/etc/letsencrypt
    env_file:
      - mailserver.env
    cap_add:
      - NET_ADMIN
      - SYS_PTRACE
    restart: always

volumes:
  maildata:
  mailstate:
  maillogs:

Please kindly help me to figure out.
Thank you

i generated from the setup.sh -Z email add gmail says that the email address is not exists.

Let me guess here: You don't seem to use SELinux. Why are you not reading the README carefully:
# without SELinux
docker-compose up -d mail

./setup.sh email add <user@domain> [<password>]
./setup.sh alias add postmaster@<domain> <user@domain>
./setup.sh config dkim

# with SELinux
############################## WITH SELINUX !!! #########################################
docker-compose up -d mail

./setup.sh -Z email add <user@domain> [<password>]
./setup.sh -Z alias add postmaster@<domain> <user@domain>
./setup.sh -Z config dkim

Is there something unclear about this?

i generated from the setup.sh -Z email add gmail says that the email address is not exists.

Let me guess here: You don't seem to use SELinux. Why are you not reading the README carefully:
# without SELinux
docker-compose up -d mail

./setup.sh email add <user@domain> [<password>]
./setup.sh alias add postmaster@<domain> <user@domain>
./setup.sh config dkim

# with SELinux
############################## WITH SELINUX !!! #########################################
docker-compose up -d mail

./setup.sh -Z email add <user@domain> [<password>]
./setup.sh -Z alias add postmaster@<domain> <user@domain>
./setup.sh -Z config dkim

Is there something unclear about this?

i generated from the setup.sh -Z email add gmail says that the email address is not exists.

Let me guess here: You don't seem to use SELinux. Why are you not reading the README carefully:
# without SELinux
docker-compose up -d mail

./setup.sh email add <user@domain> [<password>]
./setup.sh alias add postmaster@<domain> <user@domain>
./setup.sh config dkim

# with SELinux
############################## WITH SELINUX !!! #########################################
docker-compose up -d mail

./setup.sh -Z email add <user@domain> [<password>]
./setup.sh -Z alias add postmaster@<domain> <user@domain>
./setup.sh -Z config dkim

Is there something unclear about this?

I've enabled all the README and create 3 different mail account but nothing are working.

i generated from the setup.sh -Z email add gmail says that the email address is not exists.

Let me guess here: You don't seem to use SELinux. Why are you not reading the README carefully:
# without SELinux
docker-compose up -d mail

./setup.sh email add <user@domain> [<password>]
./setup.sh alias add postmaster@<domain> <user@domain>
./setup.sh config dkim

# with SELinux
############################## WITH SELINUX !!! #########################################
docker-compose up -d mail

./setup.sh -Z email add <user@domain> [<password>]
./setup.sh -Z alias add postmaster@<domain> <user@domain>
./setup.sh -Z config dkim

Is there something unclear about this?

I've enabled all the README and create 3 different mail account but nothing are working.

Are you using SELinux?

Are you using SELinux?

Please read the README carefully. If you're not using SELinux, use
./setup.sh email add <user@domain> [<password>]
./setup.sh alias add postmaster@<domain> <user@domain>
./setup.sh config dkim

without -Z. Then try again and please use Google as a search engine and not this issue tracker.

Please read the README carefully. If you're not using SELinux, use
./setup.sh email add <user@domain> [<password>]
./setup.sh alias add postmaster@<domain> <user@domain>
./setup.sh config dkim

without -Z. Then try again and please use Google as a search engine and not this issue tracker.


chore: fix linting issues

address some linting issue

Lesson to self, make sure your CI pipeline passes before hitting merge :laughing:


feat: allow traversal plugins to be passed through makeAnalysis

for when a plugin needs to manipulate or traverse the AST

Thanks!

The next beta is tomorrow, your changes will come through


SCRUM-1855: represent a range of identifiers

Two classes representing AGR identifier and a consecutive range of identifiers. These will be used in MaTI and future MaTI API consumers (A and Blue teams)

@abecerra - I imagine this is the way we will assign AGRKB identifiers to an object? Do you think we could add a bit to the description to point that out?

Thank you @sierra-moxon, I've updated the description

Also, would it make sense to have a pointer to a named controlled vocabulary with the subdomain_codes already in use? This might be helpful for example, if the BlueTeam wants a domain for a new AGRKB type, they would now which CV in the ATeam interface to go looking for (eg. CV.name = 'AGRKB_domain_vocabulary' or something).

At the moment all we have, behind the VPN, is:
https://alpha-mati.alliancegenome.org/api/subdomain
or
https://beta-mati.alliancegenome.org/api/subdomain
or
https://mati.alliancegenome.org/api/subdomain
returning the existing codes and subdomains (only one for now)
As we are closing to start minting IDs in the curation app, your suggestion can be discussed, also with the Blue team.


Enable matching curly braces inside of quotation marks in an f-string

Typically VS Code supplies a matching pair of parentheses, braces, and brackets when you type an opening character. But when you are inside of quotation marks, VS Code apparently gives up on this behavior, assuming that you are writing free text and not code. So you don't get a closing brace automatically; you have to type it yourself. This is sub-optimal in Python when writing f-strings. In Python, you will often need to write matching braces inside of quotation marks, like so:
print(f"Let's talk about {some_variable}.")

When you type the opening brace inside the quotation marks, VS Code does not supply the closing brace—again, because VS Code apparently assumes you aren't writing code. I would like VS Code to be able to detect when we are inside an f-string and, if we are, to automatically supply matching braces.

Thank you for the suggestion! We have marked this issue as "needs decision" to make sure we have a conversation about your idea. We plan to leave this feature request open for at least a month to see how many 👍 votes the opening comment gets to help us make our decision.

One more idea related to this topic: it would be great to add a feature for r-string (i.e., raw string, usually used for regex) that matches square brackets and parenthesis.

Any update on this issue ?

Have there been any updates on this issue?

We have marked this issue as "needs PR" which means we would like to see it resolved, but we have no ETA on when that would be (but if you would like to try submitting a PR that would be great!).

is there any update on this issue?

I would definitely be up for trying to implement this, but I have no idea where the relevant code is :P

I would really like to see this issue as well.

I think I might have found a workaround for this.  Change the Auto Closing Brackets setting to always.

You can enable this setting (in settings.json) only for Python by specifying a language scope:
"[python]": {
    "editor.autoClosingBrackets": "always",

   ...
},

Amazing!

The Pylance language server now offers the python.analysis.autoFormatStrings setting automatically inserts the f prefix as well as the closing curly braces when you add {

https://github.com/microsoft/vscode-python/assets/45497113/2c39c951-b906-48b8-abe4-1c3c08a8500e


Define new pattern set

Hi Poucotm,
Firstly, such an awesome package!
I wonder can I define a new pattern set, for example:
"info_pattern": [
[ "\[INFO\]", "\n" ],
],
and then choose the color for this type of pattern set?
Also, in some big log file, the "Bookmarks" and "Go To Position" make sublime very lag. Can I have an option to turn off these feature for big log file?
Thanks very much!
Giap

Hi Giap,
I've added info pattern and bookmark enable options.
Please, check v0.7.0 with its README
Best Regards,
Kris

Hi Giap,
I've added info pattern and bookmark enable options.
Please, check v0.7.0 with its README
Best Regards,
Kris

Hi Poucotm,
The info pattern seem not work in my case. Here is my user setting:

And here is my log file after highlighted:

You can see that the INFO line is not highlighted.
Also, although that the bookmark is set to false, when I tell the Log Highlight to highlight my text, it still running finding something (in this case, I just want it highlight the text for me, and not anything more)
Can you check for me, please?
Thank,
Giap

Hi, Giap,
For INFO pattern, it looks working well with your settings. I don't know
the reason. Did you try to re-run sublime-text ?
[image: 본문 이미지 1]
The process which you have seen something is searching base directory of
relative path link. Do you want to turn off that also ?
If then, you can not go to link for relative path when you double-click it.
Best Regards,
Kris.
2017-01-06 13:52 GMT+09:00 giaplv57<EMAIL_ADDRESS>
Btw, I generated custom custom syntax & theme already
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/poucotm/Log-Highlight/issues/2#issuecomment-270830975,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASUglU-JI-7OYoEg_SinAp-aM2V-zUpiks5rPchygaJpZM4LXWoP
.


Fixed exceptions in tests where no connection exists

The latest changes introduced a check on the connections id, which is not available in unit-tests when running the subscription using johanbrook:publication-collector

Coverage decreased (-0.8%) to 77.941% when pulling 7093c0eef2fca494c637af2f1aa3dc3112421ed6 on SimonSimCity:tests-bug into 2aaa1be191b286709fb3fecbf48121f34da53e80 on nlhuykhang:master.

hey @SimonSimCity , I would like to suggest you take liberty of this PR and also release a new version to NPM. What do you think?


feat: query datasets from SQL Lab

SUMMARY

Introduce the dataset macro, allowing users to query physical and virtual datasets in SQL Lab. This is useful if you've defined computed columns and metrics on your datasets, and want to reuse the definition in adhoc SQL Lab queries.
Because currently dataset names are not unique the macro uses the dataset ID:
SELECT * FROM {{ dataset(42) }} LIMIT 10

If users want to select the metric definitions as well, in addition to the columns, they can pass an additional keyword argument:
SELECT * FROM {{ dataset(42, include_metrics=True) }} LIMIT 10

Since metrics are aggregations, the resulting SQL expression will be grouped by all non-metric columns. Users can also specify a subset of columns to group by instead:
SELECT * FROM {{ dataset(42, include_metrics=True, groupby=["ds", "category"]) }} LIMIT 10

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

Added unit tests.
ADDITIONAL INFORMATION

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

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

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

Any progress in this?
I wonder to what extent this functionality requires the correct database/schema of the dataset to be selected in order to make it work.
Especially when two datasets are to be joined this way, they need to refer to the same database.
Is this being checked?
Dreaming this functionality a bit further, I could imagine having a dataset browser included in the schema browser, which presents all datasets of the selected database, so they can be selected from like the existing tables/views of the DB.

This issue should not be closed by the stale bot, in my eyes. The suggested functionality is just too promising.

I agree, @rumbin, let me do some more work on this.

Want, want want....take my money!


6.2.0.Final

Hi Kevin,
This Pull Request contains 460 files changed, included many of my own commits, so I can't merge this PR into blessed. Please send a PR with only your changes so I can review them.


Setting additionalProperties to false does not have any effect

Problem statement
I want to reject unknown fields provided in request body. According to the FAQ, it seems to be possible by simply adding additionalProperties: false to my definition.
However, this does not have any effect.
Is this a regression or just a misunderstanding?
Swagger specification
swagger: '2.0'
info:
  title: Example
  version: '0.1'
consumes:
- application/json
produces:
- application/json
paths:
  /test:
    post:
      operationId: test
      parameters:
      - name: body
        in: body
        schema:
          type: object
          properties:
            foo:
              type: boolean
          additionalProperties: false
      responses:
        200:

Steps to reproduce
The following request should raise a 400 error, but does not:
curl -H 'Content-Type: application/json' -d '{"foo": true, "bar": false}' "http://<IP_ADDRESS>:$PORT/test"

Environment
swagger version: v0.19.0
go version: go1.12.7 linux/amd64
OS: Linux

https://goswagger.io/use/models/schemas.html#known-limitations-with-go-swagger-models

Thanks for the pointer, I didn't notice this doc before.
So at this point, I guess my best bet is to set additionalProperties to true, so that I can check that the map[string]interface{} is empty in my own code. However, it remains annoying because the Swagger doc will basically say the reverse of what it should.
Any idea for a better workaround?

No, no other idea.
As explained in the link above, the trade-off in our implementation is currently to ignore additional properties when set to false, so this does not bear any validation semantics.
This is the same for additionalItems.
That being said, I am not against some opt-in behavior, e.g. --strict-additional-properties that would enforce validation (i.e. unwanted extra properties make the message invalid). You can add this kind of additional check in the custom unmarshaller template for schemas with additionalProperties.

@fredbi let me know if #2040 fits what you had in mind. At least, it perfectly solves my issue.
Thanks a lot for your help!

For future references, two alternatives are available:

using --strict-additional-properties as implemented in #2040 (currently unsure whether it will be merged)
using a custom runtime.Consumer as pointed out by @casualjim here


Accents for Cyrillic may not be placed correctly

From a tweet: https://twitter.com/achkasov/status/751839938950393857
I tried the three sequences I saw in the screenshots, я́, о́, and ы́, which looked goods in lowercase, but not good in uppercase. Also, there's probably many more common Cyrillic sequences. @moyogo may have a list somewhere, but at least look at CLDR exemplars in both lowercase and uppercase.

Rendering of all the CLDR exemplars (I think) with latest Roboto:

The following glyphs are missing anchor data in Roboto:
БДКЛПФЦШЩЪЬЂЅІЈЉЊЋЏҐҔҚҜҢҤҲҶҸӉ
бъьђљњћџґҕқҝңҥүұҳҷҹӊ

That is, they don't inherit anchors from components or have anchors manually set. Some seem to be obviously similar to glyphs with anchors; it would probably be easy to carry over anchors for those.
Ќ seems to render fine because Harfbuzz automatically combines it into a precomposed character. Some of the lowercase renderings look fine without anchors simply because the default mark position works (but certainly this isn't the case for the wider or taller glyphs).

Oh, I forgot that I'd made the following local change while testing:
diff --git a/scripts/lib/fontbuild/generateGlyph.py b/scripts/lib/fontbuild/generateGlyph.py
index ad0d66c..52f19ed 100644
--- a/scripts/lib/fontbuild/generateGlyph.py
+++ b/scripts/lib/fontbuild/generateGlyph.py
@@ -34,8 +34,7 @@ def parseComposite(composite):

 def copyMarkAnchors(f, g, srcname, width):
     unicode_range = set(
-        range(0x0030, 0x02B0) + range(0x1E00, 0x1EFF) +
-        [0x430, 0x435, 0x440, 0x441, 0x445, 0x455, 0x456, 0x471])
+        range(0x0030, 0x02B0) + range(0x1E00, 0x1EFF) + range(0x0410, 0x500))

     for anchor in f[srcname].anchors:
         if "top_dd" == anchor.name:

So the fonts in the latest release don't actually produce the results I posted (they have more missing anchors). But I think this is a safe first step to take, leaving the problem of missing anchors in sources.

Other marks besides acute seem to behave as expected. The mark is displaced on all the same glyphs, and additionally К. Renderings for U+0300 through U+030F attached.

These were generated with the following script:
for hexval in $(seq '0x300' '0x30f'); do
    hexstr="$(printf '%x' "${hexval}")"
    markstr="$(hb-unicode-encode "${hexstr}")"
    s=''
    for c in А Б В Г Д Е Ж З И Й К Л М Н О П Р С Т У Ф Х Ц Ч Ш Щ Ъ Ы Ь Э Ю Я Ё Ђ Ѓ Є Ѕ І Ї Ј Љ Њ Ћ Ќ Ў Џ Ґ Ғ Ҕ Қ Ҝ Ң Ҥ Ү Ұ Ҳ Ҷ Ҹ Һ Ӊ Ә Ӣ Ө Ӯ; do
        s="${s}${c}${markstr}"
    done
    s="$s\n"
    for c in а б в г д е ж з и й к л м н о п р с т у ф х ц ч ш щ ъ ы ь э ю я ё ђ ѓ є ѕ і ї ј љ њ ћ ќ ў џ ґ ғ ҕ қ ҝ ң ҥ ү ұ ҳ ҷ ҹ һ ӊ ә ӣ ө ӯ; do
        s="${s}${c}${markstr}"
    done
    hb-view --font-size=28 Roboto-Regular.ttf "$(echo -e ${s})" > "${hexstr}.png"
done

I also noted U+02BC in the Cyrillic exemplars. Those look fine to me:

@alexeiva please could you review this in https://github.com/google/roboto/releases/download/v2.134/roboto-unhinted.zip and https://github.com/google/roboto/releases/download/v2.134/roboto-hinted.zip

From @jamesgk test, glyphs БДКЛПФЦШЩЪЬЂЉЊЋЏҐҔҚҜҢҤҲҶҸӉ бъьђљњћџґҕқҝңҥүұҳҷҹӊ do not have top accents in the UFO source. Although these tested combinations are superfluous.
To fix the mentioned issue on twitter here is a list of vowels that require combined acute to mark stressed syllables:
а́е́ё́и́о́у́ы́є́э́і́ї́ю́я́ү́ұ́ӑ́ӓ́ӗ́ә́ӛ́ӣ́ӥ́ӧ́ө́ӫ́ӭ́ӯ́ӱ́ӳ́ӹ́
А́Е́Ё́И́О́У́Ы́Є́Э́І́Ї́Ю́Я́Ү́Ұ́Ӑ́Ӓ́Ӗ́Ә́Ӛ́Ӣ́Ӥ́Ӧ́Ө́Ӫ́Ӭ́Ӯ́Ӱ́Ӳ́Ӹ́
Render from Impallari's font-testing-page by manual input

А́Е́І́Ү́Ұ́О́Ә́Ө́ have problems:

Ә U+04D8 the top anchor is placed too low at y=1566, but  should be y=1600, as in glyph A
Cyrillic glyphs АЕІҮҰОӨ do not exist, but are created via components. Top anchor in Latin A does exist, but when used as a Cyrillic А(U+0410) seems to not cooperate withacutecomb in my test. But works on @jamesgk's test.

My suggestion is manually adding АЕІҮҰОӨ as tangible glyphs in the UFO sources, and placing top accents in these glyphs.

Ah yea, sorry, as I mentioned before the fonts in the latest release are missing more anchors than the ones I was testing with in this thread. I was going to add another release soon but got caught up with other stuff.

Thank you Alexei! :)

FYI I pushed a new release, which should have more anchors.

Hi guys! Have a strange behavior of Roboto in Chrome and Opera when using the font from google fonts. When using U+0301 with 'a' all symbols before and including 0301 itself fall back to sans-serif in case of font-family: "Roboto", sans-serif. Would appreciate any ideas.


Adding new API for Counting Number of Token from Input Text

Updated git ignore
Remove unused import statement
Add count token from text api

@ramitsurana Thanks for the contribution. Dividing the number of characters by 4 only offers an estimate for the number of tokens, and for certain kinds of text (code, symbols, non-English languages, etc) it may not be very accurate at all. If we were to add a token counter to this project, We'd likely want to provide the real count of tokens instead, which is non-trivial – we'd either need to add in something similar to this (but more lightweight) or call a backend endpoint that offers this functionality.

@schnerd Sure, Thanks for your feedback and sharing the useful insights. I think we could do it by creating a endpoint at the backend and calling it for this command,so in case in future we want to use this functionality for other commands ( like for file create command for comparison between the tokens available in the account vs the tokens that would be consumed by the file that we want to process) we could use it more easily.

I'm going to close this PR for now since we don't currently support a tokenizer in our API.  Feel free to open an issue and we can track progress there


Docstrings for GAT modules

When you declare a new GAT with @theory, the produced module should have a good docstring.

Do we want to capture any structured metadata about the GAT in this issue? For example a link to a paper or something? We can use this to provide a database of GATs in the docs.

Current behavior
help?> GATlab.StdTheories.ThCMonoid
  No docstring or readme file found for module GATlab.Stdlib.StdTheories.ThCMonoid.

  Exported names
  ≡≡≡≡≡≡≡≡≡≡≡≡≡≡

  default, e, ⋅

that docstring should include:
julia> GATlab.StdTheories.ThCMonoid.Meta.theory
GAT(ThCMonoid):
  default::TYPE ⊣ []
  (x ⋅ y)::default ⊣ [x::default, y::default]
  ((x ⋅ y) ⋅ z == x ⋅ (y ⋅ z)) ⊣ [x::default, y::default, z::default]
  e()::default ⊣ []
  (e() ⋅ x == x) ⊣ [x::default]
  (x ⋅ e() == x) ⊣ [x::default]
  (a ⋅ b == b ⋅ a) ⊣ [a::default, b::default]


Cannot set width of the SidebarMenu when expanded.

<SidebarMenu.Collapse getScrollValue={200}>, this dose not work either passing string or number as indicated by the api

In order to adjust the width of the sidebar according to your preference, it is necessary to modify the target width in your stylesheet for the CSS class ".sidebar-menu-collapse.show." There is an example - https://codesandbox.io/s/react-bootstrap-sidebar-menu-ck23cz?file=/src/App.jsx

Then how would you use the API as described in the documentation?
The documentation is very limited, and your example does not respond to my question. You modified the default value in the $sidebar-menu-width-expanded: 500px !default; SCSS file while this file is in the library and should not be modified by users. Moreover, in the CSS file, I could not find any CSS definition to sit the width of the expanded sidebar. Even if you modify the default value in the SCSS file, it will create a transition issue, please check with sitting it to 200px instead of 500.

For avoiding transition issues, please set the target value as "getScrollValue={200}" and apply the same size to the class "sidebar-menu-collapse.show: {width: '200px'}".

Feel free to open issue


Fix/create collection

PR Type

[x] Bugfix
[ ] Feature
[x] Refactoring

Context

[x] Closes #6959

[ ] Requires deployment <snek/rubick/worker>

Before submitting pull request, please make sure:

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

Optional

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

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

[x] Fill up your DOT address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot 📸

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

Copilot Summary

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

🤖 Generated by Copilot at 769abca

ShoppingCartButton
Simpler, no more emit -
Autumn leaves falling

So the essence of this PR is that you moved useTransaction and associated watches outside of the submit function. correct?
Your copilot summary has nothing to do with the PR content. please copy a fresh template for your PRs. These summaries are actually helpful in order to understand the gist of the PR

So the essence of this PR is that you moved useTransaction and associated watches outside of the submit function. correct?

Your copilot summary has nothing to do with the PR content. please copy a fresh template for your PRs. These summaries are actually helpful in order to understand the gist of the PR

That is correct. It is to avoid having registration of multiple watchers.
Sorry about the copilot summary. I mistakenly included the summary from the previous PR.


fix: install common TestWorkflowTemplates in the context of testkube-api

Pull request description

to make it working for multi-namespace agents, which don't have testkube-operator enabled (testkube-operator.enabled=false)

Checklist (choose whats happened)

[ ] breaking change! (describe)
[ ] tested locally
[ ] tested on cluster
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

This seems to be a breaking change:
Warning  InstallFailed     2m21s  helm-controller  Helm install failed for release testkube/testkube with chart<EMAIL_ADDRESS>failed post-install: unable to build kubernetes object for deleting hook testkube/charts/testkube-api/templates │
│ /testworkflows/builtin-templates/artillery.yaml: resource mapping not found for name: "official--artillery--beta" namespace: "" from "": no matches for kind "TestWorkflowTemplate" in version "testworkflows.testkube.io/v1"                       │
│ ensure CRDs are installed first

apiVersion: helm.toolkit.fluxcd.io/v2beta2
kind: HelmRelease
metadata:
  name: testkube
spec:
  interval: 5m
  install:
    crds: CreateReplace
  upgrade:
    crds: CreateReplace
  chart:
    spec:
      chart: testkube
      version: 1.17.27
      sourceRef:
        kind: HelmRepository
        name: testkube
  # https://github.com/kubeshop/helm-charts/blob/develop/charts/testkube/values.yaml
  values:
    testkube-api:
      uiIngress:
        enabled: true
        hosts:
          - testkube.${cluster_domain}
        path: /v1 # This is required for the ingress to work per https://github.com/kubeshop/helm-charts/issues/428#issuecomment-1873973628
    testkube-dashboard:
      ingress:
        enabled: true
        hosts:
          - testkube.${cluster_domain}
      apiServerEndpoint: testkube.${cluster_domain}${cluster_ingress_port:=}

We are using Flux GitOps, which is configured to always use the latest CRDs. This fails during install and not during an upgrade, as we run our workloads in CI before we apply any changes. So I would imagine this is an issue for most.
Do you have any mitigation/fix?

Thanks @devantler for the report, it shouldn't be a breaking change. @tkonieczny, could you check?

Could it be that the testkube-operator was not bumped?
https://github.com/kubeshop/helm-charts/commit/cf990c28c128f9f662cbb072a10e896908d186e8

@rangoo94 I just checked rechecked the installation, and everything seems fine (chart: testkube-1.17.27). I will check other cases

@tkonieczny did you replicate this? It is quite consistent here, and I can hear from another person in my org that they are experiencing the same error. They are also using Flux.


Issue #17019: Reword commands in Run/Debug button in editor title

Renamed commands in Run//Debug button in editor title to Run Python File and Debug Python File
Closes #17019

Hi @arsentieva, thank you for your contribution.
I changed the base of this PR to be the OSD branch, for the same reason as https://github.com/microsoft/vscode-python/pull/17587#issuecomment-932504749.

Thank you for your contribution @arsentieva!


Ability to drag files between tabs in the same window

Is your feature request related to a problem? Please describe.
Currently it's impossible to drag a file from one tab to another in the same window.
Describe the solution you'd like
Make inactive tabs into valid drop targets, and also activate the target tab after hovering over it for a certain period of time (similar to how you can hover a file over an opened application on the taskbar and the window will activate).

@duke7553 Can you confirm that this has been implemented for a couple of versions already?

@yaichenbaum I've actually had issues with this as of late. It seems as though you can only do it if you drag the file in from the right side of the tab.

Version <IP_ADDRESS>. Not sure if this is exactly what OP is talking about, but if so then maybe this should be a bug instead? Otherwise I can raise a new ticket.

@generalguy41 Please open a new issue and if it turns out they are the same, we can merge them then.

@yaichenbaum will do.
@asuetin Can you confirm that this is the same issue you're having?

@generalguy41 Yes, seems like the same issue. I did manage to make the Move prompt appear but as soon as I let go of the LMB, the app crashed, so I though it was an extension of the original bug. I can't reproduce the crash in 0.15.0 though, just the weird prompt behavior.


Inject a mechanism that allows you to serve uploads via proxy.

Adds two new constants: UPLOADS that is the URL prefix for each uploaded file (default to /wp-content/uploads) and UPLOADS_HTTP_PROXY that represents a host from which the uploads are served (default to <IP_ADDRESS>).
Reference https://github.com/presslabs/wordpress-integration/issues/11
Tests

git checkout uploads-http-proxy
docker build -t presslabs/php-runtime .
docker network create rclone
docker run -p 8080:8080 --network=rclone --name=rclone quay.io/presslabs/rclone serve http . --addr <IP_ADDRESS>:8080
docker run -p 8081:80 --network=rclone -e UPLOADS=/etc -e UPLOADS_HTTP_PROXY=rclone:8080 presslabs/php-runtime
curl http://localhost:8080/etc/rclone.conf should be the same with curl http://localhost:8081/etc/rclone.conf
curl -SI http://localhost:8081/etc/ should throw 403
curl -SI http://localhost:8081/etc should display php info

Will be moved to https://github.com/presslabs/wordpress-integration/blob/master/docker/templates/nginx-vhost-conf.d/50-gcs-media.conf


Categorization on /slaves page is confusing and hard to read

Recommendation: Put “Decommissioned” Agents in the “Inactive” section instead of the “Decommissioning” section. It might also be helpful to highlight agents that are in transitional states.

Update to the slaves page released in 0.13


Featured images

What about featured images?

There isn't specific support for them yet; I'd welcome a patch. It would need to replicate things like wp_get_attachment_url as well as the necessary database stuff.

So I found this in a Google search, and thought that I'd go and see what can be done (shouldn't be that hard right?)
HOLY SHIT. How do Wordpress developers work with this thing??!
Anyway, let me see what happens first: I get something out and send in a patch, or I give up and just use XMLRPC :P

This SQL helped me:
 SELECT
        p1.*,
        wm2.meta_value
    FROM
        wp_posts p1
    LEFT JOIN
        wp_postmeta wm1
        ON (
            wm1.post_id = p1.id
            AND wm1.meta_value IS NOT NULL
            AND wm1.meta_key = "_thumbnail_id"
        )
    LEFT JOIN
        wp_postmeta wm2
        ON (
            wm1.meta_value = wm2.post_id
            AND wm2.meta_key = "_wp_attached_file"
            AND wm2.meta_value IS NOT NULL
        )
    WHERE
        p1.post_status="publish"
        AND p1.post_type="post"
    ORDER BY
        p1.post_date DESC

Hahaha thanks! :)

It's my Post model
class Post < WPDB::Post
    dataset_module do
      def default_scope
        filter(post_type: 'post', post_status: 'publish').reverse_order(:post_date)
      end

      def published
        filter(post_status: 'publish')
      end

      def with_featured
        left_outer_join(:main_postmeta, {
          :post_id => :main_posts__id,
          :mpm__meta_key => '_thumbnail_id'
        }, :table_alias => :mpm).
        left_outer_join(:main_postmeta, {
          :post_id => :mpm__meta_value,
          :mpm2__meta_key => '_wp_attachment_metadata'
        },  :table_alias => :mpm2).select_append(:mpm2__meta_value___metadata)
      end
    end

    set_dataset(self.default_scope)

    ALLOWED_COLUMNS = [:ID, :post_author, :post_date, :post_title, :post_excerpt, :post_name, :comment_count]

    def thumbnail size = nil
      if self[:metadata].present?
        metadata = PhpSerialization.load(self[:metadata])
        if size
          date = /(\d+\/\d+\/)/.match(metadata['file']).try(:[], 1)
          image = date + metadata['sizes'][size]['file']
        else
          image = metadata['file']
        end
      end
      image = image || get_default_image(size)
      URI.join(uploads_url, image).to_s
    end

    def get_default_image size = nil
      case size
      when 'thumb'
        "2014/12/sidebar-300x110.jpg"
      else
        "2014/12/sidebar.jpg"
      end
    end
  end

Thanks again. I'm more familiar with ActiveRecord, so I'm learning a lot about how Sequel does scopes :)
@robmiller how do you want the attachment_url implemented? I'm thinking of a method called #attachment_url that returns the url of the attachment (thanks to @adisos code).
Also, I suggest common scopes e.g. published or post (for excluding revisions) also be implemented, or at least a canonical way to implement them added to the documentation.

By the way, I had to rewrite these models in activerecord, so I'm not using this gem now, but it helped me a lot.

Off the top of my head, I'd expect it to work like:
post = WPDB::Post[42]
post.featured_image
# => a WPDB::Post object that points to the attachment
post.featured_image.attachment_url
# => the URL of this specific attachment

At this point, I'm thinking that we should probably do single table inheritance on the posts table so that what's returned from featured_image is a WPDB::Attachment (which extends WPDB::Post) and so that the attachment_url (or, I guess, just url) method is defined on WPDB::Attachment rather than WPDB::Post.
Implementing scopes is an excellent idea — I'll open a separate issue for that. I must confess, I've always done this with class methods on models and only learned about dataset_module from adisos's above post, so I've learned something too!

Sounds great, my only issue with chaining like that is that featured_image might return a nil and so we have a lot of boilerplate code to make sure it isn't.
It's not like the scopes where it returns the self-modified object, so we're almost sure it won't return nil.
Or maybe we can return a WPDB::Null object that will return an empty string for any method called on it? Or get rid of the extra method call on featured_image and maybe just name the method featured_image_url
For WPDB::Attachment maybe we can implement it differently than STI. I'm not very experienced at Sequel, but we have something called polymorphic associations in ActiveRecord (see http://www.gotealeaf.com/blog/understanding-polymorphic-associations-in-rails)
My worry is that with STI we'll end up with a table that has a lot of fields that are nil because they're not used.

Well, we needn't be worried about ending up with a table with any sort of fields, because we're bound by the database structure of WordPress: ruby-wpdb's motive is to let you access the schema of WordPress without modifying it in any way.
Single-table inheritance makes sense in this case because that's what the data is from the perspective of WordPress: lots of things go into the posts table that aren't posts per se. Some of them are very similar to posts, like pages (which have the post_type "page"); some of them are very different, like attachments (which have the post_type "attachment").
We'd just be modelling this existing structure by using STI, ensuring that an Attachment object is returned for posts of type attachment.


Update README.rst

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 26.471%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
72

Relevant Lines:
272

💛  - Coveralls


Parquet support

I tried running datatc for saving in parquet and noticed that it requires either:

pyarrow
fastparquet

But they are not in the datatc package requirements.

I made that decision intentionally, because that’s how I saw it was done in pandas. By default, pandas doesn’t install pyarrow/fastparquet. When you run pd.read_parquet for the first time, it gives an error message instructing you to install one of them. I assume this is to reduce the memory footprint of the pandas package install.
I forget exactly, but I’m assuming I didn’t make datatc give a nice error message instructing to install pyarrow/fastparquet…? Would doing that satisfy your issue?

On Nov 25, 2020, at 15:26, Matteo<EMAIL_ADDRESS>wrote:
I tried running datatc for saving in parquet and noticed that it requires either:
pyarrow
fastparquet
But they are not in the datatc package requirements.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/uzh-dqbm-cmi/data-traffic-control/issues/12, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYLV7TZQPYYE3UGLB7A3DDSRUHZ5ANCNFSM4UCO7AEQ.

Maybe you can have a section in the docs for the supported file formats... and necessary packages for using them with datatc  think would be more helpful..
I was looking in the docs and didn't find it, then I crawled through the code to see which file types are supported...

That is an excellent point. I added this task to my backlog.

On Nov 25, 2020, at 16:51, Matteo<EMAIL_ADDRESS>wrote:
Maybe you can have a section in the docs for the supported file formats... and necessary packages for using them with datatc think would be more helpful..
I was looking in the docs and didn't find it, then I crawled through the code to see which file types are supported...
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/uzh-dqbm-cmi/data-traffic-control/issues/12#issuecomment-733790427, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYLV7TEAYROUGETNWOAXETSRURYZANCNFSM4UCO7AEQ.


removed keyboard configChanges flag

Removing Keyboard flags from Activity configChanges for now, as this will cause errors for the upcoming broker release until a major release of MSAL can be completed.

Probably want to merge these directly into the release branch


[ParseableInterface] Pass down the module name and import source loc

Use the name for the cached module, so that we don't end up with a zillion "x86_64-XXXXXXXX.swiftmodule" files in the cache when we're working with architecture-specific swiftmodules.

Diagnose if the expected name is different from the name specified in the swiftinterface.

Emit all diagnostics at the location of the import, instead of without any location at all.

Also sneak in a separate commit to allow @objc even without import Foundation in swiftinterfaces.

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - c805993db9191f3a5104a2040ba30e9e2b75384e

Mac full test
@swift-ci Please test Linux

@swift-ci Please smoke test macOS

Build failed
Swift Test Linux Platform
Git Sha - c805993db9191f3a5104a2040ba30e9e2b75384e


New sample  ace-covid-indicator

Q
A

Bug fix?
no

New feature?
no

New sample?
yes?

What's in this Pull Request?
New sample ace-covid-indicator, calling external API and custom UI

@joaojmendes you rock! Thank you and we will get this processed asap! 🧡💜 I'll chat with you offline about a demo! 🤩

@joaojmendes Thank you for the sample. Sorry it took so long. I was on holiday. I have approved it. @PopWarner will reach out and see if you would like to do a demo at some point in the future.


Usability issue with submission's repeatable fields

Describe the bug
The new approach used for repeatable fields during submission generated a usability problem.
Indeed is not clear for the user that to really add a field when it is repeatable, he need to click on the "plus" button. Thereby in a situation like the following screenshot the user doesn't really fill the repeatable fields and after a save data are lost :

To Reproduce
Steps to reproduce the behavior:

start a new submission
fill only the first field for a repeateble metadata
after saving the submission, data are deleted

Expected behavior
In my opinion the behavior of repeatable fields should be restored to as it was previously, i.e. without the need to click on the "plus" button to add only one field, but using the "plus" button only to add additional fields

I agree with the expected behavior & can reproduce this usability issue on the demo site.  This is loosely related to #835, which pointed out that the auto-save is a bit too aggressive & also may remove values from these repeatable fields if you haven't pressed "+" before auto-save is triggered.
Therefore, I've tentatively scheduled this for beta 5 alongside #835 .

I agree the way it works now is confusing and needs to be improved
But I'd like to provide some background on why it turned out this way:

To keep UI clutter down, we wanted to avoid having hints, and buttons such as lookup and search, on every value of a repeatable field, only on the top value
For that reason we didn't want to make the top row draggable, as it is strange if you drag the top row down, to lose a bunch of buttons, and have the new top row sprout those buttons. So the idea was to have the top row be an input field only, and have saved values in a list below. Similar to how dspace 6 does it.
The top row is also a query field now. You can type in a query and press lookup to search for entities. #751 adds similar functionality for authority control searches. If the value of the top row was auto-saved, and the form happened to auto-save before you pressed lookup or selected an authority value, your query would end up being stored as a metadata value. Thats why we decided it would only save the entered value if the user explicitly took action (clicking the plus button)

Thanks @artlowel for the summary behind this decision. However, I do think we need to think further on the best behavior here, as with this current behavior and #835 (aggressive auto-save), data loss is unfortunately frequent.
I think we need to step back and look at the behavior we want to achieve, and how to make the interaction more user friendly -- currently, the need to press the "+" is not obvious, and the auto-save makes it frustrating if you forget.

Note: I've found this behavior of the "+" button also causes this bug: #884
Simply put, if you enter in a value without clicking "+", the submission form thinks you've still entered that value. So, for example, it'll put a green checkmark next to the section even if you forget to click "+" next to your entered author name.  This results in the "Deposit" button being enabled before it should be.  Clicking on "Deposit" throws an error & locks up the form as described in #884

To solve this issue we propose to automatically add an edited repeatable field when the field loses the focus. In this way we can avoid the main issue and at the same time we can maintain useful sorting functionality and keeping UI clutter down as noted by Art.
This work can be done within estimation given for https://github.com/DSpace/dspace-angular/issues/835

Won't the field lose focus if you type a query and click the lookup or search buttons? We need a way to prevent search queries typed in that box from being stored as metadata

As discussed today, for now we are going with the "Add" button approach -- change the "+" button to say "Add" and still require it to be pressed before a value is saved.  This behavior is identical to DSpace 6.x XMLUI behavior for repeated fields, and therefore is good enough for 7.0.  It could be enhanced in a later 7.x release as needed.
The estimate for this ticket is set to 0 hr as it is included in the estimate on #835 & will be solved by that ticket.  See this comment for details.

We expect this bug will be fixed by the (in progress) fix for #971

We expect this bug will be fixed by the (in progress) fix for #971

I've verified this is fixed by #1025


Fix the "frozen column" test case

vaadin-components-gwt/src/main/webapp/vaadin-grid/test/grid-editing-columns.html, line 269 [r1] (raw file):
Should we also check the actual data cells or is it enough to just check the headers?

Comments from the review on Reviewable.io

vaadin-components-gwt/src/main/webapp/vaadin-grid/test/grid-editing-columns.html, line 269 [r1] (raw file):
It's up to the core dev team to test that imo. We should merely test that grid.frozenColumns = 2 api in v-grid actually freezes the right amount of columns so I would say this is more than enough.

Comments from the review on Reviewable.io

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks broke.

Comments from the review on Reviewable.io


'BasePermissionMetaclass' object is not iterable

Keep getting this error when trying to do basic logic outside of authentication or authorization.  Py3.7 and Django 2.1

add this to the settings.py file
REST_FRAMEWORK = {
...
'DEFAULT_PERMISSION_CLASSES': ( 'rest_framework.permissions.IsAdminUser', ),
...
}
write "  permission_classes "  in a pair square bracket [ ] not in ()
permission_classes = [permissions.IsAuthenticatedOrReadOnly]

Even i was getting this issue and observed that i missed to add comma in last,since i am using tuple ()
to fix this issue either we can user like this permission_classes = [permissions.IsAuthenticatedOrReadOnly] or
permission_classes = (permissions.IsAuthenticatedOrReadOnly,)


Support Feibit Inc co.  FB56-ZCW11HG1.2

This adds support for Feibit Inc co.  FB56-ZCW11HG1.2 RGBW 11w LED Downlight
It seems to be that same as the Smart Home Pty. device that is already added, and shares a Zigbee Model ID. From what I've seen online, there are many versions of this device, I'm guessing they may just be white-label devices.
Z2M showed my light as unsupported until I added this entry to devices.js.
There was no need for me to add it to the homeassistant.js as it seems entries there are purely based on modelId, and my light was added to home assistant just on the devices.js change.

Closing, it seems this wasn't needed - Looks like I was suffering from another issue that seems to have cropped up recently where new devices are showing up as unsupported in the log but are if they pairing and enrolling successfully.


Alpha (confidence interval) not used in plotting functions

The alpha parameter in the plotting functions iplot and coefplot is not used
https://github.com/s3alfisc/pyfixest/blob/af27baf87f8b2d78f17c11e55be43cb07a480c34/pyfixest/report/visualize.py#L241
See https://github.com/s3alfisc/pyfixest/pull/395#issuecomment-2080184678 for an idea on how to fix this.

Hi @s3alfisc , i would like to work on this!

Awesome, I've assigned this to you @rafimikail! =)

Hi @rafimikail, very nice talking to you earlier! As discussed, here's one way to tackle the issue:

add a function argument alpha to Feols.tidy().
if alpha is not None, re-compute the confidence interval by reimplementing logic in self.get_inference()
update the CI level names in the series
pass the alpha argument to adjust iplot() and coefplot() (e.g. here for iplot)
adjust the CI levels in _coefplot (it always states 95%)
add some API tests here or here

And that's it! =)

Hi @s3alfisc , Thanks for wrapping it up, i will give a try!


[AAP-12935] Cleanup jobs from RQ when disable pending activations

This PR contains:

removing jobs from RQ when disables the pending activations;
the pending activations have correct pending state;
for OCP, fixing activation status to show stopped when disable it;
for OCP, fixing activation unexpected behaviors when restarting it;

Solves:
AAP-11762
AAP-12935
AAP-12986
AAP-13109
AAP-13113
AAP-13416
AAP-13782
AAP-14479

FYI: Recommended practices about labels in k8s: https://kubernetes.io/docs/concepts/overview/working-with-objects/common-labels/


Issue with destructor of BlockOperator

Hi,
I noticed a small bug in the destructor of BlockOperator at
https://github.com/mfem/mfem/blob/ea5c4664864cb84c0a453f527d6befec05a8b8a9/linalg/blockoperator.cpp#L133-L145
where it seems that iRow and jCol have been mixed up. I did not find any mention of it so I opened a new issue.
Best regards,
Andreas

Thanks for reporting this, Andreas! I'll add a fix for this directly to #3384.

Addressed in #3384.


Arrow form of generic identity function produces Flow error

Flow, a static type checker for JavaScript, version 0.38.0
Perhaps I'm just having one of those days and can't see the woods for the trees.
Nevertheless why does this compile:
function identity<T>(arg: T): T {
  return arg;
}
const id: <T>(arg: T) => T = identity;

but this doesn't?
const identity = <T>(arg: T) => arg;

const id: <T>(arg: T) => T = identity;

Produces:
inconsistent use of library definitions
14:     const identity = <T>(arg: T) => arg;
                                        ^^^ T. This type is incompatible with
16:     const id: <T>(arg: T) => T = identity;
                                     ^^^^^^^^ some incompatible instantiation of `T`

Hmm, writing const identity = <T>(arg: T): T => arg works. I'm not sure why leaving off the explicit return type (you've got <T>(arg: T) => arg) breaks though.
Interestingly, flow suggest seems to break with the <T>(arg: T) => arg version; it just won't give me any suggestions.

Indeed, you should annotate return type explicitly.

I have a similar problem with this. In flow v0.75.0, the following patterns work and they all poduce the same type signature: <T> (T) => T
// @flow

// function with implicit return type
function identity1<T>(x: T) {
  return x;
}

// function with explicit return type
function identity2<T>(x: T): T {
  return x;
}

// function expression with implicit return type
const identity3 = function<T>(x: T) {
  return x;
};

// function expression with explicit return type
const identity4 = function<T>(x: T): T {
  return x;
};

// arrow expression with implicit return type
const identity5 = <T>(x: T) => x;

// arrow expression with explicit return type
const identity6 = <T>(x: T): T => x;

console.log(identity1(123), identity1('123'));
console.log(identity2(123), identity2('123'));
console.log(identity3(123), identity3('123'));
console.log(identity4(123), identity4('123'));
console.log(identity5(123), identity5('123'));
console.log(identity6(123), identity6('123'));

But if I try to extract the type signature and use it as a standalone type for the function, it fails (see here)
type ID = <T>(T) => T;
const identity: ID = x => x;

37: const identity: ID = x => x;
                              ^ Cannot assign function to `identity` because `T` [1] is incompatible with `T` [2] in the return value.
References:
36: type ID = <T>(T) => T;
                  ^ [1]
36: type ID = <T>(T) => T;
                        ^ [2]

Does anyone know what's going on here?

I have a similar problem with this. In flow v0.75.0, the following patterns work and they all poduce the same type signature: <T> (T) => T (see here)
// @flow

// function with implicit return type
function identity1<T>(x: T) {
  return x;
}

// function with explicit return type
function identity2<T>(x: T): T {
  return x;
}

// function expression with implicit return type
const identity3 = function<T>(x: T) {
  return x;
};

// function expression with explicit return type
const identity4 = function<T>(x: T): T {
  return x;
};

// arrow expression with implicit return type
const identity5 = <T>(x: T) => x;

// arrow expression with explicit return type
const identity6 = <T>(x: T): T => x;

console.log(identity1(123), identity1('123'));
console.log(identity2(123), identity2('123'));
console.log(identity3(123), identity3('123'));
console.log(identity4(123), identity4('123'));
console.log(identity5(123), identity5('123'));
console.log(identity6(123), identity6('123'));

But if I try to extract the type signature and use it as a standalone type for the function, it fails (see here)
type ID = <T>(T) => T;
const identity: ID = x => x;

2: const identity: ID = x => x;
                             ^ Cannot assign function to `identity` because `T` [1] is incompatible with `T` [2] in the return value.
References:
1: type ID = <T>(T) => T;
                 ^ [1]
1: type ID = <T>(T) => T;
                       ^ [2]

However, I can make it work if I add generic type T to function definition (see here)
// @flow

type ID = <T>(T) => T;
const identity: ID = <T>(x) => x;

Can anyone explain what's going on here?


Create a field to add a replay code to a map when created

📝 Changed routes:

…]/scrim/[scrimId]/edit
…/[scrimId]/map/[mapId]

(view full size)
(view full size)

/[team]/scrim/[scrimId]

(view full size)

Commit dedb7851e963e09111712dcc848b570e15d1ded6 (https://parsertime-71oxtv8kz-luxdotdev.vercel.app).


Add TLS verification configuration

Hey @ernestre,
I'm using an internal certificate along with a TPM instance and want to disable the TLS verification step. Therefor I've created a small change. Let me know what you think. :)

@ernestre Are you still maintaining this project?

@ernestre Are you still maintaining this project?

Hey, yeah. My bad for the delay, I'll take a look and test it some day this week 👌

I've added a few fixes and additional stuff for local testing, but thank you very much for contributing!


International Leave this site options

Overview
We have a leave this site button to keep our users safe by allowing them to quickly navigate away if they feel unsafe using our site.
Currently, our leave this site button directs everyone to  https://www.google.co.uk/search?tbm=isch&sa=1&ei=esSCW4LPH4WugQaZsbqoDw&q=cute+baby+animal+memes&oq=cute+baby+animal+memes&gs_l=
However this link is in english. We would like to have international option.
Action Items

[ ] create a new key and value pair in the /bloom-frontend/messages/shared/en.json and for the rest of the languages.
[ ] In the same way you fetch the inner text of the button, fetch the url from the messages file in /bloom-frontend/components/layout/LeaveSiteButton.tsx
[ ] For now just copy and paste the english url and the team will chose different urls at a later date.

Resources/Instructions

@eleanorreem I would like to work on this issue. Please assign me this issue to work on.


Fix dangling module comments

Summary
This PR fixes an issue where a comment only module caused an instability.
Test Plan
Added test

Current dependencies on/for this PR:

main

PR #7456   👈

This comment was auto-generated by Graphite.


[CDAP-19970] Fixing null pointer error in prepareRun

https://cdap.atlassian.net/browse/CDAP-19970

LGTM, Please squash commits before merge.


Service no longer allows client connections

Hi,
I still seem to get into this state:

The service is running, and it's localhost on the standard 19134 port, but it doesn't connect.  The service log contents are shown above.  I used the client to test delete an old backup, closed the client cleanly, and then tried to connect again and it's timing out.
I have to restart the service to get the client to connect again.  It only happens on some client activities (I think either the pack manager or the backup manager).  Most of the time I can bring up the client, perform a one-off backup without stopping the world, and then close the client, and then do it again later with no issues.  So its probably some unfinished transaction between the client and service that is causing the error, probably related to the pack manager.

I tried the latest release.  Steps to recreate the problem:
1 start service
2 start client
3 connect client to server
4 click the X close in the upper to right of the window without clicking disconnect first
5 restart client
6 fails to connect to client
likely the service is holding the 19134 port open and does not realize the client is gone, and the new client is unable to connect
workaround: either leave the client running or disconnect before closing client
previously even if you disconnected before closing, it would not connect.  So the changes you made did improve things, but did not completely resolve the issue.

Hello again! I will dive into bad disconnects a bit deeper then. I started focusing a bit on proper service/server shutdown and restarts, as well as refactoring things to be a bit more testable. Currently everything is manually tested, but I am starting to build tests to automate these lengthy steps. It wasn't all too possible beforehand, and just was ignored. Soon testing steps like this to prevent rolling back issues will be a reality.

This issue was fixed, but I accidentally uncovered a related issue, and will open it separately.


After clicked the item "Generate Android Selectors" and nothing worked for me.

My  Gradle:
dependencies {
classpath 'com.android.tools.build:gradle:3.0.1'
    // NOTE: Do not place your application dependencies here; they belong
    // in the individual module build.gradle files
}

because it can't work on the AS3.0


interface definitions with nomenclature

It would be of great use if interfaces where arguments are ambiguous, for example Observer and Filter, named their arguments according to control theory typical nomenclature.
This is current interface definition
// Filter is a dynamical system filter.
type Filter interface {
	// Predict estimates the next internal state of the system
	Predict(mat.Vector, mat.Vector) (Estimate, error)
	// Update updates the system state based on external measurement
	Update(mat.Vector, mat.Vector, mat.Vector) (Estimate, error)
}

// Propagator propagates internal state of the system to the next step
type Propagator interface {
	// Propagate propagates internal state of the system to the next step
	Propagate(mat.Vector, mat.Vector, mat.Vector) (mat.Vector, error)
}

// Observer observes external state (output) of the system
type Observer interface {
	// Observe observes external state of the system
	Observe(mat.Vector, mat.Vector, mat.Vector) (mat.Vector, error)
}

Problems with this definition include:

All arguments are mat.Vector. What convention is being used?
Comments do not help either. Overall lackluster documentation

Possible solutions
Below is a mix of adding a little bit of documentation + naming the variables.
type Filter interface {
	// Predict estimates the next internal state of the system. x is state, u are inputs
	Predict(x,u mat.Vector) (filter.Estimate, error)
	// Update updates the system state based on external measurement. x is state, u are inputs, z is the measurement
	Update(x, u, z mat.Vector) (filter.Estimate, error)
}

If you give me the all-clear I can try fixing it tonight and PRing. This and #7

Happy to accept a PR for this. This sounds like a great idea.


Put all the API calls actions in a single action

Put all the API calls actions in a single action to populate the AppoModalComponent React form component.
Call "owners" and "doctors" inside a promise.

Instead of call appo, owners, doctors and pets through four actions, set al of them inside one single action.
Link react with the reducers.


Expose genesis_config and genesis_records RPC API

We have recently added (nearprotocol/nearcore#2109) new RPC API methods: genesis_config and genesis_records (with EXPERIMENTAL_ prefix).
The API documentation is currently defined in the PR description mentioned above.

Hi @frol . Is this issue still open ?

Nope, these methods are already implemented:
https://github.com/near/near-api-js/blob/5c2ea7b41e9be97394ccf15ef7926c592904fa82/src/providers/json-rpc-provider.ts#L201-L220
(well, genesis_records endpoint was removed from the JSON RPC implementation on nearcore side)


I can create the file, but the application stops right after creating the file.

Where should I put the code <?php moonland\phpexcel\Excel::export([ ..... ]); ?> ? The code was successfully executed, the file was generated but the the application stop there, it doesn't execute the codes below the above statement.

I have the same issue.


Cutting module

Resolves #101
This PR establishes a cutting module in the circuit_cutting package. It assumes cutqc has already been packaged in accordance with #138 , and it assumes the base QPD classes are available as in #137 .
This PR was created so we could discuss the arrangement of the packages, modules, and CI while we finalize the implementation internally.

[ ] Merge #137
[ ] Merge #138

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1086 of 1206 (90.05%)  changed or added relevant lines in 20 files are covered.
844 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-26.3%) to 57.587%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/mip_model.py
0
2
0.0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_verification.py
0
3
0.0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting.py
0
6
0.0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_evaluation.py
0
8
0.0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_post_processing.py
0
10
0.0%

circuit_knitting_toolbox/circuit_cutting/cutqc/mip_model.py
176
191
92.15%

circuit_knitting_toolbox/circuit_cutting/cutqc/wire_cutting_evaluation.py
123
144
85.42%

circuit_knitting_toolbox/circuit_cutting/cutqc/wire_cutting_post_processing.py
164
188
87.23%

circuit_knitting_toolbox/circuit_cutting/cutqc/wire_cutting.py
321
352
91.19%

Files with Coverage Reduction
New Missed Lines
%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/init.py
6
0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_verification.py
54
0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_evaluation.py
123
0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting_post_processing.py
164
0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/mip_model.py
176
0%

circuit_knitting_toolbox/circuit_cutting/wire_cutting/wire_cutting.py
321
0%

Totals

Change from base Build<PHONE_NUMBER>:
-26.3%

Covered Lines:
1685

Relevant Lines:
2926

💛  - Coveralls


Updated README with Postgres/PostGIS geo example

Added Postgres/PostGIS example for importing simple point table as geom_point - using the MySQL example here or others on the web will cause big problems when using spatial point type in Elasticsearch
(Source file for example attached - uses geonames data which is open source)
geo.sql.zip

Thanks!

No problem, happy to help!

On Sep 24, 2016, at 7:41 PM, Jörg Prante<EMAIL_ADDRESS>wrote:
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
, or mute the thread
.


dpkg installation error when retz user already exists

When I tried to install retz-server with dpkg ( retz-server_0.0.29_amd64.deb ) on retz user, I recieved installation error with "failed to create retz user" and deb package system of my environment has become inconsistent - cannot re-install and uninstall any packages.

Plus: Retz installer (RPM and DEB) overwrites retz.properties with new one; while it must preserve old config file.

1c33276e6b20348ef3d


Change package name

The more involved part will be updating all the type references and import statements to have the new package. Plus making sure the files actually get moved to their new home, something which should work but I don't know if we've exercised

The more involved part will be updating all the type references and import statements to have the new package. Plus making sure the files actually get moved to their new home, something which should work but I don't know if we've exercised

@sambsnyd That totally makes sense. What about to rename it to “SimplyChangePackageName” for the start since I think it’s needed for some cases where we need just to change package name. It’d be a good start. And then create complete version of it with the current name: «ChangePackageName». Although I am not sure is this a good approach in terms of design.
Anyway I am going to work on it, just probably need more information about how it’s better to implement. So any suggestions are welcome.

Hey @Guseyn, sorry I forgot to respond to this for so long.
andThen()ing a ChangeType should handle most of updating type references and imports to the new package name.
As for physically moving the files... let me check. RefactorVisitor.generate() can be used to create new files, but I'm not actually sure if we have a mechanism for deleting/moving files yet.

Returning null from visitCompilationUnit should result in the file being deleted.
So what I suggest is: When you visit the compilation unit to be moved to a new package, store it off to the side and return null.
In generate() return the compilation unit you stored off to the side. Be sure to set the appropriate package and corresponding sourcePath on the modified compilation unit.
That should do the trick. Let me know how it works out.

I'm going to close this since there hasn't been any activity in a while and we've made major, incompatible changes to the core APIs. Would be happy to accept a 7.x compatible version.

I'm going to close this since there hasn't been any activity in a while and we've made major, incompatible changes to the core APIs. Would be happy to accept a 7.x compatible version.


fix ist8310: pass I2C address to I2CSPIDriver

Fixes https://github.com/PX4/Firmware/issues/14526

Thanks @bkueng


change response status to maintain backwards compatibility to ownClou…

…d 10
When executing a propfind to a resource owned by another user the service would respond with a HTTP 403.
In ownCloud 10 the response was HTTP 207. This change sets the response code to HTTP 207 to stay backwards compatible.

@ishank011 @labkode Can you review?


Silence NonInteractiveExampleWarning in test_unary

The .example() call leads to a number of warnings in the test suite:
/.../array-api-tests/array_api_tests/test_special_cases.py:1253: NonInteractiveExampleWarning: The `.example()` method is good for exploring strategies, but should only be used interactively.  We recommend using `@given` for tests - it performs better, saves and replays failures to avoid flakiness, and reports minimal examples. (strategy: from_dtype(float64, min_value=0.0, max_value=13_407_807_929_942_595_611_008_317_640_802_934_282_464_207_265_959_107_021_466_054_756_094_376_573_581_619_054_852_612_241_927_956_455_650_759_996_398_703_782_412_109_633_178_814_162_067_569_513_603_268_608, exclude_min=True))
    in_value = case.cond_from_dtype(xp.float64).example()

I don't see an easy way for @given to be used here, but silencing the warning will make it more friendly to downstream CI jobs.

These special case tests are doing some strange things with hypothesis. It's not clear to me if there aren't other downsides to using example here, or even if we can be sure that just drawing one example is going to definitely do what we want. IMO we should either draw multiple examples normally, like we used to, or just hard-code a specific example, rather than just assuming that a single hypothesis draw is going to work.
At any rate, we do need to clear all the annoying warnings that are currently printed. I will add a comment to the code here and merge this.


Added free programming book

What does this PR do?
Add Resource(s) | Remove Resource(s) | Add info | Improve Repo
For resources
Description
Why is this valuable (or not)
How do we know it's really free?
For book lists, is it a book?
Checklist:

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

The English part of the description is hilarious!
The Payhip site requires an access note. The English version is (email address *requested*, not required) See CONTRIBUTING.md


Adding changes to iOS theme

Dave,
Per conversation on Twitter, here is my modification to the iOS CSS theme.
Thanks,
Kevin

Sweet, thanks! I'll pull it in and check it out this weekend. Look for a them directory reorg soon.


Managed identity to get metrics from Azure

Will be used in a kubernetes helm chart

Work in progress, I will verify that I have enough access that I need to perform all my queries.

Partly solves: https://github.com/XenitAB/terraform-modules/issues/286

I can see metrics in prometheus, this is how they look like.
We could change this to a podmonitor so I will do that tomorrow and fix the changelog.md
cluster_autoscaler_cluster_safe_to_autoscale{aggregation="average", container="azure-metrics-exporter", endpoint="http", instance="<IP_ADDRESS>:8080", job="azure-metrics-azure-metrics-exporter", namespace="az-metrics", pod="azure-metrics-azure-metrics-exporter-594754789b-xbv4x", resourceGroup="rg-1", resourceID="/subscriptions/000000000000000-11111-222222/resourceGroups/rg-1/providers/Microsoft.ContainerService/managedClusters/aks1", resourceName="aks1", service="azure-metrics-azure-metrics-exporter", subscriptionID="000000000000000-11111-222222", timespan="PT1M", unit="Count"}

The question is if we should try to remove some of these labels.

Woop woop! Epic job @NissesSenap 🙌🚀🎉🎉🎉🎉


whisk CLI updates action with a nonexistent file

When I tried to update an existing action with an nonexistent file, the update succeeds. I think it should give me an error showing that the file doesn't exist.

Confirmed - this is related to an ambiguity in the positional argument in the CLI on an update.
Thanks for reporting this.

I'll take this one.  Since the Go CLI and tests are not yet full incorporated right now, I'll add a test and fix the python CLI as well.

Fixed via PR #862


GCV node needs update

In the current implementation of the GCV node the parameters kappa and omega are not being updated.
Update rules lack mean-field approximation which makes it impossible to connect the output of the GCV node to a pointmass variable.

#14

We have both structured and VMP rules for GCV now, but I will keep this issue for the time being. Maybe we will want to add something else.


Add PDF print export

HTM-81
HTM-375

The "sqlserver:wegdeel" layer seems to be missing from the image/pdf depending on the zoomlevel of the map, this is with window.devicePixelRatio of 1.25 (which is an unzoomed window on my system)
Perhaps it's an idea to warn users about this if they have a divergent DPI?

This is indeed a consequence of the 'map scale' differing because the 'DPI' of the web map is different from the DPI of the print map image. Usually the DPI for printing would be higher, so for most situations the WMS server would chose a style that would render more details than are visible on the web map (as a consequence these would be harder to read, smaller labels/lines etc.). But that is arguably less worse than having less details visible on a low DPI which is currently hardcoded in this PR.
I'm going to check out making a new ol.Map instance with a pixelRatio setting, maybe that can solve multiple issues including this one at once.


When using option3 to build an image, is it not possible to use curl directly?

I used the image built with option 3. Currently, the service can start normally, but there is no response when called using the method below.

Is it because the triton version is too low?

Do you use 23.10 docker image as base docker image? The default of release branch is 23.08. You can try the latest main branch.


Add conditional links in cookie banner

What is the context of this PR?
This adds a dynamically generated cookie url displayed on cookie banner that is based on the current survey type . It also makes the display of cookie banner conditional, depending on the cookie session existence in runner.
How to review
New tests added similar to the tests of cookie settings url var in templates helpers tests. Some tests updated to reflect the changes we made to visibility of cookie banner.
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Conditional links: Cookie banner (2 SP)

 Conditional links: Cookie banner (2 SP)


RuntimeError: Given groups=1, weight of size [128, 3, 3, 3], expected input[1, 4, 256, 256] to have 3 channels, but got 4 channels instead

The following values were not passed to accelerate launch and had defaults used instead:
--num_processes was set to a value of 1
--num_machines was set to a value of 1
--mixed_precision was set to a value of 'no'
--dynamo_backend was set to a value of 'no'
To avoid this warning pass in values for each of the problematic parameters or run accelerate config.
[{'target': '', 'positive': 'best quality,masterpiece,ultra high res,photorealistic,highly detailed,fine texture,raw photo,photorealism,dslr,ultra realistic 8k,hd', 'unconditional': 'nsfw,logo,text,blurry,greasy skin,skin reflection,mutated hands and fingers,poorly drawn face,extra limb,missing limb,disconnected limbs,malformed hands,ugly,semi-realistic,cloned face', 'neutral': '', 'action': 'enhance', 'guidance_scale': 1, 'resolution': 768, 'dynamic_resolution': True, 'batch_size': 1}]
[{'target': 'girl ', 'positive': 'girl best quality,masterpiece,ultra high res,photorealistic,highly detailed,fine texture,raw photo,photorealism,dslr,ultra realistic 8k,hd', 'unconditional': 'girl nsfw,logo,text,blurry,greasy skin,skin reflection,mutated hands and fingers,poorly drawn face,extra limb,missing limb,disconnected limbs,malformed hands,ugly,semi-realistic,cloned face', 'neutral': 'girl ', 'action': 'enhance', 'guidance_scale': 1, 'resolution': 768, 'dynamic_resolution': True, 'batch_size': 1}]
1 1
['modify', 'before'] [1, -1]
C:\sd\sliders-for-windows\venv\lib\site-packages\diffusers\loaders.py:1722: FutureWarning: The function from_ckpt is deprecated in favor of from_single_file and will be removed in diffusers v.0.21. Please make sure to use StableDiffusionPipeline.from_single_file(...) instead.
deprecate("from_ckpt", "0.21.0", deprecation_message, standard_warn=False)
text_config_dict is provided which will be used to initialize CLIPTextConfig. The value text_config["id2label"] will be overriden.
C:\sd\sliders-for-windows\venv\lib\site-packages\transformers\models\clip\feature_extraction_clip.py:28: FutureWarning: The class CLIPFeatureExtractor is deprecated and will be removed in version 5 of Transformers. Please use CLIPImageProcessor instead.
warnings.warn(
create LoRA for U-Net: 214 modules.
Prompts
target='girl ' positive='girl best quality,masterpiece,ultra high res,photorealistic,highly detailed,fine texture,raw photo,photorealism,dslr,ultra realistic 8k,hd' unconditional='girl nsfw,logo,text,blurry,greasy skin,skin reflection,mutated hands and fingers,poorly drawn face,extra limb,missing limb,disconnected limbs,malformed hands,ugly,semi-realistic,cloned face' neutral='girl ' action='enhance' guidance_scale=1.0 resolution=768 dynamic_resolution=True batch_size=1 dynamic_crops=False
Module:
Parameter: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q.lora_down.weight, Requires Grad: True
Parameter: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q.lora_up.weight, Requires Grad: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q
Parameter: lora_down.weight, Requires Grad: True
Parameter: lora_up.weight, Requires Grad: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q.lora_down
Parameter: weight, Requires Grad: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q.lora_up
Parameter: weight, Requires Grad: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_k
Parameter: lora_down.weight, Requires Grad: True
Parameter: lora_up.weight, Requires Grad: True
Module: , Training Mode: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q, Training Mode: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q.lora_down, Training Mode: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_q.lora_up, Training Mode: True
Module: lora_unet_down_blocks_0_attentions_0_transformer_blocks_0_attn1_to_k, Training Mode: True
target='girl ' positive='girl best quality,masterpiece,ultra high res,photorealistic,highly detailed,fine texture,raw photo,photorealism,dslr,ultra realistic 8k,hd' unconditional='girl nsfw,logo,text,blurry,greasy skin,skin reflection,mutated hands and fingers,poorly drawn face,extra limb,missing limb,disconnected limbs,malformed hands,ugly,semi-realistic,cloned face' neutral='girl ' action='enhance' guidance_scale=1.0 resolution=768 dynamic_resolution=True batch_size=1 dynamic_crops=False
girl
girl best quality,masterpiece,ultra high res,photorealistic,highly detailed,fine texture,raw photo,photorealism,dslr,ultra realistic 8k,hd
girl
girl nsfw,logo,text,blurry,greasy skin,skin reflection,mutated hands and fingers,poorly drawn face,extra limb,missing limb,disconnected limbs,malformed hands,ugly,semi-realistic,cloned face
0%|                                                                                | 0/1000 [00:00<?, ?it/s]
Traceback (most recent call last):
File "C:\sd\sliders-for-windows\trainscripts\imagesliders\train_lora-scale.py", line 545, in
main(args)
File "C:\sd\sliders-for-windows\trainscripts\imagesliders\train_lora-scale.py", line 456, in main
train(
File "C:\sd\sliders-for-windows\trainscripts\imagesliders\train_lora-scale.py", line 260, in train
denoised_latents_low, low_noise = train_util.get_noisy_image(
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\utils_contextlib.py", line 115, in decorate_context
return func(*args, **kwargs)
File "C:\sd\sliders-for-windows\trainscripts\imagesliders\train_util.py", line 221, in get_noisy_image
init_latents = vae.encode(image).latent_dist.sample(None)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\diffusers\utils\accelerate_utils.py", line 46, in wrapper
return method(self, *args, **kwargs)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\diffusers\models\autoencoder_kl.py", line 242, in encode
h = self.encoder(x)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\diffusers\models\vae.py", line 111, in forward
sample = self.conv_in(sample)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\nn\modules\module.py", line 1518, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\nn\modules\module.py", line 1527, in _call_impl
return forward_call(*args, **kwargs)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\nn\modules\conv.py", line 460, in forward
return self._conv_forward(input, self.weight, self.bias)
File "C:\sd\sliders-for-windows\venv\lib\site-packages\torch\nn\modules\conv.py", line 456, in _conv_forward
return F.conv2d(input, weight, bias, self.stride,
RuntimeError: Given groups=1, weight of size [128, 3, 3, 3], expected input[1, 4, 256, 256] to have 3 channels, but got 4 channels instead
How do I solve this problem, my python version is 3.10.11，

Just convert the training images to jpg, this will solve this issue.

只需将训练图像转换为jpg，即可解决此问题。

Changed to jpg according to your way and worked successfully, thank you


chore: upgrade to yarn 3

Description
Updates the package manager from Yarn 1 to Yarn 3
Type of Change

[ ] Bug Fix
[ ] New Feature
[ ] Breaking Change
[ ] Refactor
[ ] Documentation
[x] Other (maintenance)

Looks good, please just triple check that the yarn cache doesnt end up as part of the published package but otherwise 👍


operator project-quay (3.9.0-nightly.20230720)

Add 3.9.0-nightly.20230720 to community operators

/merge possible

/merge possible


Automated cherry pick of #4258: Streamline EKS BPF enablement

Cherry pick of #4258 on release-v3.17.
#4258: Streamline EKS BPF enablement

Deploy preview for calico ready!
Built with commit 03da7812d359d8ddc5cd725e4f271da7c0f3d101
https://deploy-preview-4272--calico.netlify.app


Bug 2308304: csi: stop deleting csi-operator resources

to keep it simple we'll not delete the csi-operator resources we'll only document this for users who want to switch to csi-drivers. Anyways by next release we'll only support csi-operator only.
Signed-off-by: subhamkrai<EMAIL_ADDRESS>(cherry picked from commit a16ad75c929cce19ab29cda2fabea80fd61eb40f) (cherry picked from commit 3eaf33b0777cffb9dc140338fca0faa45a155760)

Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/approve
/lgtm


Option to store http responses to file

An option to download all discovered files would be great. I know I can write a simple function to download all files via curl/wget etc, but having the option via katana itself would be ideal.
Based on - https://github.com/projectdiscovery/httpx/pull/848
   -sr, -store-response              store http response to output directory
   -srd, -store-response-dir string  store http response to custom directory

@ehsandeep What are your thoughts on expanding this to allow for storing the HTTP request as well? Perhaps they are stored as a pair, rather than the same file?
The current PR seems to drop the request body, which is a fixable issue on it's own, but I'd consider to be a bug.

The raw HTTP request is something that could be used by other tools, so long as it's "well-formed".

@wdahlenburg thanks for pointing this and good point on the request, I will create a follow-up ticket as this can be extended to get more controls like response, request + response, response body and response header.

@wdahlenburg thanks for pointing this and good point on the request, I will create a follow-up ticket as this can be extended to get more controls like response, request + response, response body and response header.

I wrote that comment before I messed with proxify, but is there a good reason for the output formats to be different between katana and proxify?


Needs Clarification for modifying Client.exe for multiClient

From your wiki:
" Modified client.exe shortcut. Right click client.exe in the Mabinogi directory, send shortcut to desktop and add code:1622 ver:143 logip:<IP_ADDRESS> logport:11000 chatip:<IP_ADDRESS> chatport:8002 setting:"file://data/features.xml=Regular, USA" at the end of the "Target" line. "
Do I add:
code:1622 ver:143 logip:<IP_ADDRESS> logport:11000 chatip:<IP_ADDRESS> chatport:8002 setting:"file://data/features.xml=Regular, USA"
that into the Target box, when I right click on properties on Client.exe shortcut located on the desktop?
It says the end of the Target line, so does that mean after the Target directory?
Final result:
C:\Nexon\Library\mabinogi\appdata\Client.exe\code:1622 ver:143 logip:<IP_ADDRESS> logport:11000 chatip:<IP_ADDRESS> chatport:8002 setting:"file://data/features.xml=Regular, USA"

Should be C:\Nexon\Library\mabinogi\appdata\Client.exe code:1622 ver:143 logip:<IP_ADDRESS> logport:11000 chatip:<IP_ADDRESS> chatport:8002 setting:"file://data/features.xml=Regular, USA"


update openapi spec for healthcheck

The healthcheck api did not match documentation, I will open a separate PR for the docs repository.
NOTE: I could not get the generation of the openapi.yml working and therefore made this small change manually. php artisan openapi did not apply the change I've done.
Changes

updated the openapi.yml with the correct API path
updated the openapi annotation on the healthcheck with the correct path

Issues

as posted on discord

Thank you for the PR!