_serialize_sentence() missing 1 required positional argument: 'cfg'

when running predict.py
是否使用范例[y/n]，退出请输入: exit .... y
Traceback (most recent call last):
File "predict.py", line 154, in
main()
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/main.py", line 32, in decorated_main
_run_hydra(
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/_internal/utils.py", line 346, in _run_hydra
run_and_report(
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/_internal/utils.py", line 201, in run_and_report
raise ex
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/_internal/utils.py", line 198, in run_and_report
return func()
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/_internal/utils.py", line 347, in
lambda: hydra.run(
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/_internal/hydra.py", line 107, in run
return run_job(
File "/home/hmp/anaconda3/envs/python38/lib/python3.8/site-packages/hydra/core/utils.py", line 127, in run_job
ret.return_value = task_function(task_cfg)
File "predict.py", line 89, in main
data, rels = _preprocess_data(data, cfg)
File "predict.py", line 29, in _preprocess_data
_serialize_sentence(data, serial)
TypeError: _serialize_sentence() missing 1 required positional argument: 'cfg'

Thanks for using deepke. You can git pull the latest version to solve this issue.

Thanks for using deepke. You can git pull the latest code to solve this issue.

您好，我已经更新了最新代码，这个问题还是存在


Packit doesn't support %autorelease

See for example: https://src.fedoraproject.org/rpms/man-pages-l10n/pull-request/9#request_diff
%autorelease docs: https://docs.pagure.org/Fedora-Infra.rpmautospec/autorelease.html

You can support it with update_release=false

Yes, but that's a workaround, packit should be able to handle it automatically.

Could you elaborate? Do we want it to set update_release=false automatically when %autorelease is detected, to change the format or what in particular?
For my usecase I am setting update_release=false globally (copr_build.branch=main, propose_downstream, etc.), but I turn it on for copr_build.trigger=pull_request for testing versions.

Yes, I would expect it to set it automatically when it detects the %autorelease because I don't see any reason why not to.
I would rather have it optional that the user can set the update_release=true manually because it would be just a small number of users that will set it. The majority will expect to have it false automatically.

So to summarize:

On copr_build and other upstream jobs: respect update_release
On propose_downstream and pull_from_upstream: check if the Release contains %autorelease

If true: do not update the field
If false: reset to 1

I think this is perfectly reasonable, but it should be documented properly, and if the user overwrites the update_release and related options, respect that choice over this default (e.g. they have a specific release_suffix they want to use there)

Hm, I checked the code and it seems that update_release is ignored in propose_downstream and pull_from_upstream jobs, packit always sets Release to 1. @LecrisUT - are you sure that is actually has an effect?

Oh thanks for checking, I did not check it because I don't have a working project to freely check that without borking upstream tags. It would seem rather grave otherwise.

The thing is, you may want to have (almost always) a different value of update_release for upstream and for dist-git. That only concerns propose_downstream where there is a common packit config, but I think in general update_release shouldn't affect dist-git jobs.

I am thinking if they have some metadata like 1%{?dist} that they would want to set it to.

Since we already have release_suffix, it would be confusing to use something else just for dist-git.

Ok, let us discuss this within the team.

TODO:

[ ] Update the docs to make it clear that update_release and release_suffix have no effect on propose_downstream and pull_from_upstream
[ ] Make propose_downstream and pull_from_upstream not to touch Release if %autorelease is used


Build participate/join us page

This will be a page that provides information to the Interns, Volunteers and Collaborators on how they can help us, be associated with us. We might have FAQs also on the page.
Shall share the content soon.


Section Title Has a Mispeled Word: Create a zone-redunant database

The section title "Create a zone-redunant database" misspells redundant.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e4eb2267-d11f-e3db-66cb-163544155849
Version Independent ID: 52b05396-bacb-6dd5-9b65-ae56f0ec55b3
Content: Create a zone-redundant Hyperscale database - Azure SQL Database
Content Source: azure-sql/database/hyperscale-create-zone-redundant-database.md
Service: sql-database
Sub-service: service-overview
GitHub Login: @dimitri-furman
Microsoft Alias: dfurman

@emu7800 thanks for pointing this out. The fix should be live tomorrow.

@emu7800
Thanks for your feedback! We will investigate and update as appropriate.

The fix is now live.
#please-close

Awesome! Thanks!!


does not convert simple numbers in English sentences

Hi,
I am using python 3.7.
text_to_num.alpha2digit("she is one foot taller than me.",  "en")
returns 'she is one foot taller than me.', so 'one' is not converted.
if I change it slightly to:
text_to_num.alpha2digit("she is two feet taller than me.",  "en")
then it returns 'she is 2 feet taller than me.'
So, a simple conversion does not happen in this case and I don't know why?
thanks!

one is quite special, as it is also a pronoun:

"The first one"
"Another one"
"No one"
…

We cannot easily identify between a number and a pronoun when one is isolated, so we only convert it when it is part of a sequence, that is, when another number precedes of follows it.
I'm wondering: maybe we should do that for all single digit numbers, shouldn't we?

I would advise against having all small numbers left in text form, let the user decide how he wants the conversion done. Perhaps via a flag, eg. CMS, could follow the guidelines of the Chicago Manual of Style, https://getitwriteonline.com/articles/using-numbers/. Its absence would have all numbers converted to digits.

@rtxm commented on May 7, 2020, 1:12 PM GMT+4:30:

one is quite special, as it is also a pronoun:

"The first one"
"Another one"
"No one"
…

We cannot easily identify between a number and a pronoun when one is isolated, so we only convert it when it is part of a sequence, that is, when another number precedes of follows it.
I'm wondering: maybe we should do that for all single digit numbers, shouldn't we?

That's even more "buggy" than this current predicament with one.

See #42


Commit of work for final sprint for Round 1 paper submission.

I don't think we need to test this at all, but it is good to have for
posterity.

TODO: Add license headers to this

I didn't add license headers, but I am merging for now because I have a few PRs stacked on top of this and I want a single commit of this for the sake of posterity.


Cards hover enhacement

Title and Issue number

Title :
Issue No. : #1009
Code Stack : HTML,CSS
Close #1009

Video (mandatory)

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md
[x] I have gone through the  contributing.md file before contributing

Additional context (Mandatory )
Are you contributing under any Open-source programme?

[x] I'm a GSSOC'24 contributor
[ ] I'm a SSOC'24 contributor

Can you assign this issue to me. #1092 #1091


Fix code snippet

Fixes #45115

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

#sign-off


UVC camera support

Is there any plan to support UVC camera input, rather than only front/rear?
An alternative could be the possibility to use a video stream as source, but I see it's only supported in the native SDK.
Thanks

Do you mean custom video capture?

No, I mean the possibility to use an external (UVC) camera for video calls or streaming.
As of now we can only switch between front and rear camera

you said it only supported in the native SDK.
could you pls tell me how it support?

I meant that Agora supports the possibility to use a video stream as a source, but only in the native SDK, as you also stated here: #182
Will this be supported on react native side?
Apart from this, the main question was: will the use of an external UVC camera be supported?

react-native SDK just a bridge of native SDK, you can add it in the native layer by yourself.


Updated the base image and python package versions.

Note: The ipywidgets update was removed because the docker-stacks base image includes the current version.

Seems to work, maybe add a comment in the README.md indicating that users should use Shift + RightClick when copy/pasting string from JupyterLab for authentication. The rest seems to work just fine.

For what OS does Shift + RightClick work? (for OSX, it is Command-C and Command-V)

For Win10, I could not use Ctrl+C / Ctrl+V with JupyterLab, prints ^V, also right-click does not work, standard browser menu is overridden, the only way I could paste was to use Ctrl+Shift+RightClick:

... actually, Ctrl+Shift+V works as well, but then Ctrl+Shift+C does something very different

Here is the log file for build on Win10. I get at the following lines marked red.
jupyter-scipy-ee.log
...
debconf: delaying package configuration, since apt-utils is not installed
...
The directory '/home/jovyan/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
The directory '/home/jovyan/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
...


loadImageFromURL not working with filter blur shape

Describe the bug
Blur shape not working in new image from loadImageFromURL
To Reproduce
Steps to reproduce the behavior:

Filter blur shape image origin
Load new file
Filter blur shape image origin

Expected behavior
Filter blur shape working in image for step 2
Screenshots
attach file

@latmerry
Thank you for the information.
I will resolve the bug where the previous image appears in the blur shape.

@latmerry
Thank you for the information.
I will resolve the bug where the previous image appears in the blur shape.

Thank you.


Layer resize arbitrarily when you try to create a symbol

when you apply a paddy plugin to a group of layers and you want to convert one layer of that group into a symbol, that a layer resize arbitrarily...

Thanks for this @danielmariluz.
I've made some changes in v1.0.3, which I'll release soon. Those changes seem to fix it. See below:

@danielmariluz I have just released version 1.0.3 which seems to fix this. I'll close this issue for now. If you find the issue still persists, then please re-open it. 👍


"Error converting value "True" to type 'System.Collections.Generic.ICollection`1[System.String]"

I'm getting the following error when I attempt to load my JSON file with what I believe is a valid JSON Schema:

Error converting value "True" to type 'System.Collections.Generic.ICollection`1[System.String]'. Path 'properties.Name.required', line 5, position 22.

I have attached the JSON Schema file, and a sample JSON file that contains the data for your reference.
My system is a Windows 10 PC (Insider Fast Ring, 14376) and v1.4.5944.34935 of your Visual JSON Editor.
HPOneView.300_CmdletHelp.json.txt
Test.json.txt

I think this is not a draft v4 schema...

Try removing all required attributes.. in draft v4 required contains a list of the required properties..

Not sure I completely understand.  Are you suggesting that I remove the "required" property/attribute in the schema file I manually created?

Okay, so I did a bit more research, and I believe I created a JSON v4 Schema.  I usedhttp://jsonschema.net/#/ to create the schema.  Here is the updated schema:
HPOneView.300_CmdletHelp.json.txt

And when I use the schema file, I now get a different error:
Unable to cast object of type 'Newtonsoft.Json.Linq.JArray' to type 'Newtonsoft.Json.Linq.JObject'.

The attribute items is used to describe a tuple type (I.e. it must be an array of schemas. I think in your case you need the item property which describes the array item type...

I used http://jsonschema.net/#/ to create the JSON v4 Schema file.  The JSON object is quite simple. It is an Array of an Object that is very specific.  The Object contains a String attribute called Name, then an Object called Contents.  Contents contains nested objects and arrays.
Is there any way to debug your tool to find out where exactly it is generating the error?

You can clone or download the source code and run it with debugger attached.. can you provide a sample json and json schema v4?


Workflow definitions that include a workflow.zip file fail parsing in CromwellWESAdapter

Describe the Bug
Running a multi-file workflow that includes a workflow.zip file as part of its definition produces the following error with agc workflow run:
2022-12-06T17:47:00-08:00 ✘   error="unable to run workflow: 500 Internal Server Error"
Error: an error occurred invoking 'workflow run'

Steps to Reproduce

Create a multi-file WDL workflow definition
Add a zip file called workflow.zip to the definition folder
Add the workflow to agc-project.yaml
Start a Cromwell context
Run the workflow

Relevant Logs
Context adapter log contains the following:
Tue, 06 Dec 2022 17:19:28 -0800 [ERROR] 2022-12-07T01:19:28.581Z        091259f7-8c63-4b27-9362-72f6f91e9125    Exception on /ga4gh/wes/v1/runs [POST]
Traceback (most recent call last):
  File "/var/task/amazon_genomics/wes/adapters/CromwellWESAdapter.py", line 514, in get_workflow_from_s3
    props = parse_workflow_zip_file(file, workflow_type)
  File "/var/task/amazon_genomics/wes/adapters/CromwellWESAdapter.py", line 555, in parse_workflow_zip_file
    zip.extractall(wd)
  File "/var/lang/lib/python3.9/zipfile.py", line 1642, in extractall
    self._extract_member(zipinfo, path, pwd)
  File "/var/lang/lib/python3.9/zipfile.py", line 1697, in _extract_member
    shutil.copyfileobj(source, target)
  File "/var/lang/lib/python3.9/shutil.py", line 205, in copyfileobj
    buf = fsrc_read(length)
  File "/var/lang/lib/python3.9/zipfile.py", line 924, in read
    data = self._read1(n)
  File "/var/lang/lib/python3.9/zipfile.py", line 992, in _read1
    data += self._read2(n - len(data))
  File "/var/lang/lib/python3.9/zipfile.py", line 1027, in _read2
    raise EOFError
EOFError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/var/task/amazon_genomics/wes/adapters/CromwellWESAdapter.py", line 239, in run_workflow
    props = get_workflow_from_s3(workflow_url, tmpdir, workflow_type)
  File "/var/task/amazon_genomics/wes/adapters/CromwellWESAdapter.py", line 516, in get_workflow_from_s3
    raise RuntimeError(f"{s3_uri} is not a valid workflow.zip file: {e}")
RuntimeError: s3://agc-111122223333-us-west-2/project/orca/userid/pwymingJKP3z/context/cromwellCtx/workflow/broad_gtex/workflow.zip is not a valid workflow.zip file:

Expected Behavior
Workflow definitions that are accompanied by extra modules bundled as a zip file should run regardless of what the module bundle zip is named.
Actual Behavior
Screenshots
Additional Context
Proposed fix:
The workflow definition bundle needs to be extracted to a distinct folder. The following line:
https://github.com/aws/amazon-genomics-cli/blob/c2e830c243453f3b1c133dad505b333b8e49bab4/packages/wes_adapter/amazon_genomics/wes/adapters/CromwellWESAdapter.py#L555
should be replaced with something like:
zip.extractall(path='path/to/tmpdir')

where path/to/tmpdir is different than wd which is currently set to the parent folder of the downloaded workflow definition bundle.
Operating System: macOS
AGC Version: 1.5.2
Was AGC setup with a custom bucket: No
Was AGC setup with a custom VPC: No

Doing the following should be a sufficient fix:
# rest of code ...
wd = path.join(path.dirname(file), 'workflow')
with zipfile.ZipFile(file) as zip:
    # rest of code ...


Awesome Flutter

Awesome-Flutter is the main curated resources list for Flutter
Would you like to join ?
Thank you.
https://github.com/Solido/awesome-flutter

Sure, what do we need?

You can PR so your name is added to the contributors list and I'll merge it

@Solido PR sent!


Create Notional Methodology

Notional Finance methodology.

see https://github.com/messari/subgraphs/pull/656


addEvent method does not return Event object

Hello,
The title sums it up.
Check out the eventObject variable after creating an event by clicking on a date: fiddle
I also tested with fullcalendar, and fullcalendars method actually returns the Event object.
Best regards.

Made some minor changes and created a pull request.

Thank you very much for your contribution. I will review and accept your pull request soon.

This is fixed in v3.4.0.


Revert "OCPBUGS-13116: Implementation of filtering by channel for OCI catalog"

Reverts openshift/oc-mirror#627
This is a test revert to verify a breakage on the master. If confirmed, I will update instruction for further steps.

/payload ci 4.14 blocking

/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-aws-ovn-upgrade 5

/payload-job periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-gcp-ovn-rt-upgrade

/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-upgrade-from-stable-4.13-e2e-azure-sdn-upgrade 3

/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade 3

/payload-aggregate periodic-ci-openshift-release-master-ci-4.14-e2e-azure-ovn-upgrade 2

It turned out that is not the cause for the regression


python311Packages.dvc-studio-client: 0.9.0 -> 0.9.2, dvc: 2.56.0 -> 2.57.2

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

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

2 packages failed to build:

python311Packages.dvclive
python311Packages.dvclive.dist

10 packages built:

dvc
dvc-with-remotes
dvc-with-remotes.dist
dvc.dist
python310Packages.dvc-studio-client
python310Packages.dvc-studio-client.dist
python310Packages.dvclive
python310Packages.dvclive.dist
python311Packages.dvc-studio-client
python311Packages.dvc-studio-client.dist


fix(app): Fix entering error recovery while door is open

Closes RABR-608

Overview
Currently, if you open the robot door while on the error recovery splash screen, the "launch recovery" button is not disabled, and the door CTA appears as soon as the user clicks "launch recovery". The pipette does not home (which isn't great), but for the most part, a user can complete recovery. However, the robot position is lost, which has downstream affects when the user either retries a failed command or resumes the protocol run, typically resulting in a MUST_HOME_ERROR.
To solve for the pipettes not homing and this downstream error, we should just block the users from entering ER until the door is closed. After testing a few different approaches, showing a warning toast seems best.
1edd67dfb7fb0b4e7e41f8d14b4b1a5cfb42e5e5 - The fix
9261cc9cc96d5bdf1c3e0e3bde07dce407124c6d - In the low-fi days of ER designs, we had a "run paused" screen instead of the fancy splash screen. Calling that view RunPausedSplash doesn't make much sense anymore.
Desktop

ODD

Test Plan and Hands on Testing

Verified the new functionality works as intended.
Verified that the old in-flow functionality works as intended...it actually doesn't fully for different reasons, but @TamarZanzouri is putting up a PR soon!

Changelog

You can no longer click an error recovery splash page button and proceed if the door is open.

Risk assessment
low

Okay, sounds good to me. Wish we could make the buttons visually disabled but I think this is fine.

Here you go:


COPY FROM silently fails if timestamps cannot be converted

CrateDB version:  2.0.7 (however I didn't see any changes to COPY FROM in the newer changelogs)
JVM version:
java version "1.8.0_144"
Java(TM) SE Runtime Environment (build 1.8.0_144-b01)
Java HotSpot(TM) 64-Bit Server VM (build 25.144-b01, mixed mode)
OS version / environment description:
Ubuntu 16.04.2 LTS
Problem description:
When importing a bulk dataset using COPY FROM, I had created a schema beforehand with timestamp entries. It turned out that the data I was importing did not have dates in ISO-8601 format. The COPY FROM command just ends with COPY OK (2.629 sec) and no other data on why nothing was imported from the file (even though Crate was obviously reading the data).
The crate logs are mostly empty and don't say anything useful.
It would be great to have some indication that X rows couldn't be imported because of a data type mismatch, or something.

Steps to reproduce:

create table test ( t timestamp )
Create a json file like { t: '2017-07-24 00:15:26.649303' }
Try to import that file into crate using COPY FROM
See COPY OK but nothing has been imported.

hi @SneakyMax, we are returning the number of affected rows, which is the only possibility we have within the sql standard. what we could do and think of is logging the stats more explicitly.
copy dates from '/Users/johannes/sandbox/tests/copy_from/dates.txt';
COPY OK, 6 rows affected  (0.022 sec)

As the copy from is intended for fast, highly optimized data transfer, there are almost no checks done. To achieve what you want to you need to compare the number of lines of the files and the number of affected rows.

Yeah, I understand there's not much room here. I spent a whole day struggling to figure out why my data wasn't importing though. Only once I created a separate table without a schema did the import work, then I found out about the timestamp mismatch. The COPY OK was just super misleading :/

we will improve the documentation and think about being more verbose in the logs.

We've added a  RETURN SUMMARY clause to COPY FROM which will be available in the upcoming 3.1 which will cause COPY FROM to return more detailed information about what was going on. That should help debug such issues so I'm closing this for now.


Encryption chapter: Add curiosity about tractability in brute force attacks section

A good topic to cover when talking about guessing keys. Can link to the tractability chapter for keen students.
perhaps you can do this Tim? You're good at writing about this kind of thing :-)

@timbellnz I noticed a bit later in the RSA section that we have an existing box about intractability, although it uses the factoring problem. It is also a little bit long. What I'm thinking is that it'd be awesome if we introduced tractability in the caesar cipher section (when we talk about key lengths), and then expand on it a bit in the RSA section. So effectively two curiosity boxes on tractability. Thoughts?
Text in guide
curiosity}
Easy vs Difficult problems in Computer Science
If you were asked to multiply the following two big prime numbers, you might find it a bit tiring to do by hand (although it is definitely doable!), but could get an answer in milliseconds by putting it in the big numbers calculator! (included just below this box).
97394932817749829874327374574392098938789384897239489848732984239898983986969870902045828438234520989483483889389687489677903899

34983724732345498523673948934032028984850938689489896586772739002430884920489508348988329829389860884285043580020020020348508591

If on the other hand you were asked what two prime numbers were multiplied to get the following big number, you’d have a lot more trouble!  (If you do find the answer, let us know! We’d be very interested to hear about it!)
3944604857329435839271430640488525351249090163937027434471421629606310815805347209533599007494460218504338388671352356418243687636083829002413783556850951365164889819793107893590524915235738706932817035504589460835204107542076784924507795112716034134062407

So why is it that despite these two problems being similar, one of them is “easy” and the other one is “hard”? Well, it comes down to the algorithms we have to solve each of the problems.
You have probably done long multiplication in school by making one line for each digit in the second number and then adding all the rows together. We can analyse the speed of this algorithm, much like we did in the algorithms chapter for sorting and searching.
Assuming that each number has the same number of digits, which we will call n (“Number of digits”), we need to write n rows. For each of those n rows, we will need to do around n multiplications. That gives us n*n little multiplications. We need to add the n rows together at the end as well, but that doesn’t take long so lets ignore that part. We have determined that the number of small multiplications needed to multiply two big numbers is approximately the square of the number of digits. So for two numbers with 1000 digits, that’s 1,000,000 little multiplication operations. A computer can do that in less than a second! If you know about Big-O notation, this is a O(n^2) algorithm, where n is the number of digits! Note that some slightly better algorithms have been designed, but this estimate is good enough for our purposes.
For the second problem, we’d need an algorithm that could find the two numbers that were multiplied together. You might initially say, why can’t we just reverse the multiplication? The reverse of multiplication is division, so can’t we just divide to get the two numbers?
It’s a good idea, but it won’t work. For division we need to know the big number, and one of the small numbers we want to divide into it, and that will give us the other small number. But in this case, we only know the big number. So it isn’t a straightforward long division problem at all!
It turns out that there is no known good algorithm to solve the problem. One way is to just try every number that is less than the number (well, we only need to go up to the square root, but that doesn’t help much!) There is still billions of billions of billions of numbers we need to check. Even a computer that could check 1 billion possibilities a second isn’t going to help us much with this! If you know about Big-O notation, this is an O(10^n) algorithm, where n is the number of digits -- even small numbers of digits are just too much to deal with!
There are slightly better solutions, but none of them shave off enough time to actually be useful for problems of the size of the one above!
The chapter on Complexity and Tractability looks at more computer science problems which are surprisingly challenging to solve. If you found this stuff interesting, do read about Complexity and Tractability when you are finished here!
{curiosity end}

Note that I'm also putting the details of RSA (factors and primes) into a curiosity box, so that the main text focuses primarily on public key system concepts, and the curiosity boxes are an extra for those interested in the mathy details (I'm going to do that particular bit, as I already have the text for it).

Written a curiosity box (for the Caesar section), and adapted the text above for a curiosity in the RSA section.
commit 1e4842784259d3379a944a4e04628493b0b0ca4d


Ending, reundant config for sweb wasnt removed, so i just give up.

final comit ( I hope), removed old main for gui temp.
Bye.

lgtm


IBMa June 🧁

The IBM Accessibility team has product launch in June and needs the following 13 components cleared for AVT1, 2, and 3
Legend:
❌ open issue(s)
✔ no open issues
🔵 IBMa reviewed

accordion ✔
breadcrumb ❌
button ❌
checkbox ❌
content switcher ❌
form ✔
inline loading ❌
link ❌
list ✔
modal ❌
tag ❌
toggle ❌
tooltip ❌

☮ 👋 @joshblack @dakahn i am wondering if i can help with this? Really trying to find out something I could help with!! Let me know!

Oh totally! Our epics are a little opaque right now -- but we're migrating our workflow over to Github Projects which would be much easier for you to track and jump on an issue. I'll make a note to ping you when we have that up 😸

@paschalidi We're tracking our work on this Github Projects Milestone 👍

@dakahn such cool feature! thanks mate!


Calls to Equals() on a generic type parameter generates invalid IL.

bool C<T>(T t) where T : struct { var l = t; return l.Equals(l); }

should generate something like:
IL_0000: ldarg.0
IL_0001: stloc.0
IL_0002: ldloca.s 0
IL_0004: ldloc.0
IL_0005: box !!T
IL_000a: constrained. !!T
IL_0010: callvirt instance bool [System.Runtime]System.Object::Equals(object)
IL_0015: ret

but generates:
IL_0000: ldarg.0
IL_0001: stloc.0
IL_0002: ldloca.s 0
         constrained. !!T // Non expected.
IL_0004: ldloc.0
IL_0005: box !!T
        // Missing constrained
IL_0010: callvirt instance bool [System.Runtime]System.Object::Equals(object)
IL_0015: ret

The generated code looks ok for calls to other methods defined in ValueType (like ToString())

The problem happens because the constrained opcode needs to precede the actual Callvirt  instruction and Cecilifier is emitting it just after loading the address of a variable typed as a generic type parameter;

Search for the issue id and re-enable disabled tests


Update box-connector.adoc

Fixed per: Update Box Connector with technical changes #338
closing as content was updated to #338.


Get shape properties?

Is it possible to get shape properties like stroke and fill?

Any direction on this would be great...

What exactly do you mean by shape properties? Are you referring to the vector points?

When I create a shape with the Rectangle Tool and select the layer with that shape, the toolbar at the top of the window gives me access to shape properties like fill, stroke, stroke width, stroke type, width, height, etc. I'd like to know if its possible to access these properties and if so how would I do it. Thanks.

Not yet for most of it, but it won't take long to add.
To get the width/height you can do:
node = psd.tree().childrenAtPath('path/to/layer')[0]
node.get('width')
node.get('height')

Hmm, I'm more interested in the fill and stroke properties. Is this something you're working on adding? Or is there a way for me to help implement this?

I haven't started implementing them yet, but they are all layer info blocks that I just need to port from PSD.rb. For example: gradient fill, solid color, and vector stroke/vector stroke content.
It shouldn't take more than 20 minutes. I'll see if I can find time to add them today.

Thanks, man! I appreciate your work on this. It's an awesome project.

Hi guys,
Thanks for the great work.
I was wondering if the package in npm does support this merged features.
I didn't find the way to get shape properties like background color or border size.
I tried this way:
var PSD = require('psd');
var psd = PSD.fromFile(__dirname + '/psd/test21.psd');
psd.parse();
var data = psd.tree().descendants()[0].descendants()[0].get('solidColor');

I tried a lot of times but I still got a Buffer and an empty RGB Object
 data: {},
     b: 0,
     g: 0,
     r: 0 },

I can do tricky system to get the main color from the buffer but it would be cool if I could extract color directly.
Also, how can I get border size properties?
Best.


Make sure you have SELECT permission on that table or view.

I am sure it's just a user error, but I can't figure out what I am doing wrong. This would be the perfect script.
I want to use it for a table I have (TRAVEL.dbo.AIRLINES) for example.
I execute (I am logged in as sa):
EXEC sp_generate_merge @schema = 'TRAVEL', @table_name ='dbo.AIRLINES'
Then I get this error.
Msg 50000, Level 16, State 1, Procedure sp_generate_merge, Line 216 [Batch Start Line 0]
User table or view not found.
Make sure you have SELECT permission on that table or view.

I would appreciate any help.

How about this
EXEC [Travel].[dbo].sp_generate_merge @schema = 'dbo', @table_name ='AIRLINES'

Notice the difference in @schema and @table_name params.

So cool, thank you for the quick reply, that worked great!!


WIP: docs(tutorial): update and publish step 3

DO NOT MERGE
We'll do a timed merge to control when step 3 is published.

Libraries overview + graphQL query writing
Loading, error, and normal states
Necessary components (Skeleton, DataTable, Pagination)
Common DataTable miss-steps


Learn button points to last lesson visited not most advanced lesson completed

Visit the map page
Click on an older, already completed lesson
Visit anywhere else in the site that isn't a lesson
Click the learn button

You are taken to the old lesson not your most recent progress.

This is by design. Whatever challenge you navigate to will become your most recent challenge.


Error: only one instance of babel-polyfill is allowed

Version of EOSJS
20.0.0-beta2
Describe the bug
When a project that already use babel-polyfill, include this library, an exception occur:
Uncaught Error: only one instance of babel-polyfill is allowed
To Reproduce
check here the code to reproduce:
https://codesandbox.io/s/q33k7vmk14
import '@babel/polyfill';
import { JsSignatureProvider } from 'eosjs';

as far as I know, 3rd party libraries, such this one, should not include babel-polyfill, check the answer here for more info:
https://stackoverflow.com/questions/34962902/should-javascript-library-which-uses-es6-and-es7-features-require-babel-polyfill

I'm also running into this. It's cost me a lot of time at this point.

Fixed in PR #600
It turns out that @babel/polyfill was being included by @babel/cli, which the project was not using, so I just removed that dependency, which removed the @babel/polyfill dep.  This is by far the preferred solution, given ~"libraries shouldn't be changing global state"
https://github.com/babel/babel/issues/4019#issuecomment-245409644
Thanks to @or2008 !!
The codesandbox.io you setup was super useful in quickly reproducing the issue and testing a couple things.
There are lots of people solving this on the client side as well, given how often this problem arises.  See these example solutions.
https://github.com/NYPL/nypl-dgx-react-header/pull/7
https://github.com/sprucelabsai/workspace.sprucebot-skills-kit/pull/114/files
https://github.com/codex-team/editor.js/pull/424/files


MB-5700 Make phone validation more consistent

Description
When entering phone numbers in the customer on boarding flow, phone number validation was inconsistent. The Backup Contact phone number input does not allow the user to type dashes, and automatically adds them in. The Receiving Agent phone number input allowed dashes and didn't automatically add them, but if you use dashes you encounter a field length validation error. This PR makes Agent phone number validation match Customer/Backup Contact validation.
Setup
Add any steps or code to run in this section to help others prepare to run your code:
make server_run
make client_run

Go through customer onboarding flow and enter phone numbers (dashes should automatically be added and only numerical inputs will be recognized)
Create a shipment and enter a receiving agent phone number (dashes are now automatically added and only numerical inputs will be recognized)

Code Review Verification Steps

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] User facing changes have been reviewed by design.
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-5700

  Generated by :no_entry_sign: dangerJS against 9e6ec79f941c70b37aed6f313c5e84e68b2cbc24

this task was passed off and merged in https://github.com/transcom/mymove/pull/7167


include motivation and stability in README

@shapr please weigh in on this README, and let us know if the Motivation section makes sense.
@andrewthad please review as well.

I've added some changes

ping @shapr

Thanks for the readme! I am very appreciative!


[hunspell] build failure autopoint: archive.dir.tar.xz: No such file or directory

Host Environment

OS: Windows
Compiler: MinGW

To Reproduce
Steps to reproduce the behavior:
./vcpkg install hunspell
Failure logs
-- Using msys root at D:/a/Mudlet/Mudlet/3rdparty/vcpkg/downloads/tools/msys2/aa5af7b2aa7e90e8
  -- Generating configure for x64-mingw-dynamic
  [LogCollection][Start]File:'D:\a\Mudlet\Mudlet\3rdparty\vcpkg\buildtrees\hunspell\autoconf-x64-mingw-dynamic-err.log':
  autoreconf: export WARNINGS=
  autoreconf: Entering directory '.'
  autoreconf: running: autopoint --force
  /c/Program Files/Git/usr/bin/autopoint: line 485: /usr/share/gettext/archive.dir.tar.xz: No such file or directory
  tar.exe: gettext-0.19.8: Not found in archive
  tar.exe: Error exit delayed from previous errors.
  autopoint: *** infrastructure files for version 0.19.8 not found; this is autopoint from GNU gettext-tools <IP_ADDRESS>
  autopoint: *** Stop.
  autoreconf: error: autopoint failed with exit status: 1

  [LogCollection][End]File:'D:\a\Mudlet\Mudlet\3rdparty\vcpkg\buildtrees\hunspell\autoconf-x64-mingw-dynamic-err.log'.
  CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:127 (message):
      Command failed: D:/a/Mudlet/Mudlet/3rdparty/vcpkg/downloads/tools/msys2/803521c5733336c9/usr/bin/bash.exe --noprofile --norc --debug -c "autoreconf -vfi"
      Working Directory: D:/a/Mudlet/Mudlet/3rdparty/vcpkg/buildtrees/hunspell/src/v1.7.0-9251c91a4b.clean/
      Error code: 1
      See logs for more information:
        D:\a\Mudlet\Mudlet\3rdparty\vcpkg\buildtrees\hunspell\autoconf-x64-mingw-dynamic-err.log

  Call Stack (most recent call first):
    scripts/cmake/vcpkg_configure_make.cmake:604 (vcpkg_execute_required_process)
    ports/hunspell/portfile.cmake:64 (vcpkg_configure_make)
    scripts/ports.cmake:142 (include)

/c/Program Files/Git/usr/bin/autopoint

Does git for Windows come with (an incomplete) autopoint <IP_ADDRESS>? :eyes:

I'm building in GitHub actions, if it helps - here's the list of software: https://github.com/actions/virtual-environments/blob/main/images/win/Windows2019-Readme.md

Is https://www.gnu.org/software/gnulib/manual/html_node/gettextize-and-autopoint.html relevant?

Is https://www.gnu.org/software/gnulib/manual/html_node/gettextize-and-autopoint.html relevant?

At least it indicates that env AUTOPOINT might be used to enforce vcpkg's installation of autopoint.

@dg0yt would you have an idea on how to fix this?

@dg0yt, would you have an idea?

I think the hints were already given. It may be necessary to work with the AUTOPOINT variable when building hunspell.

I think the hints were already given. It may be necessary to work with the AUTOPOINT variable when building hunspell.

@vadi2 , Can you try it locally? We don't have the environment to reproduce your problem

Confirmed fixed, thank you!


Support injecting into containers in linux

Can check if the target process has a different /proc/<pid>/task/<pid>/ns/mnt. If it does, create the shared library in /proc/pid/root instead in the current filesystem. Python's tempfile module doesn't support searching for a tempdir in a different filesystem, and it caches it for the whole process once it's found. So maybe we can create a subprocess, nsenter the target filesystem, and then continue normally. Alternatively, just copy the logic in tempfile or assume that /tmp is writable.

Hi, I think this sounds really useful.
I think there may also need to be a change to the pyinjector.inject(...) API. Currently it checks that the shared library exists before injecting.
A couple of ideas came to mind already but neither are too satisfying:
pyinjector could relativize the path when it sees that it starts with /proc/<pid>/root
pyinjector.inject(123, '/proc/123/root/path/to/lib.so')

# ->

injector.inject(b'/path/to/lib.so')

Or it could double check /proc/<pid>/root when the path is not initially found
pyinjector.inject(123, '/path/to/lib.so')

# ->

if ... and not os.path.isfile('/proc/123/root/path/to/lib.so'):
   raise ...

Hmmm how about just always using /proc/pid/root both for the shared library tmpdir in hypno and for the existence check in pyinjector?

Ahh, you mean NamedTemporaryFile(..., dir=f'/proc/{pid}/root') instead of NamedTemporaryFile(..., dir=f'/proc/{pid}/root/tmp') ? I think it's quite common to mount / as readonly. I think it's less common to not have /tmp mounted read/write. But maybe I've misunderstood.
...
I was thinking over this some more this morning and I think I've perhaps come up with a clearer API for pyinjector
def inject(pid: int, library_path: AnyStr, uninject: bool = False,
           process_root: AnyStr = "") -> int:
           ...

which would be called, in those examples, like so
    injector.inject(b'/path/to/lib.so', process_root=b'/proc/123/root')

This has the advantage that it extends generally to any kind of chroot jail and that pyinjector doesn't have to bake in any knowledge about Linux's /proc filesystem.
The caller is responsible for making it clear at what path they expect the target process to see the library.

In hypno I just meant that it's possible to always look for a tempdir in the target process' fs using /proc/pid/root, regardless if it's the same or not as /.
In pyinjector, I like your suggestion, but I think that it'd also be useful to always check for existence of the shared library in the target process' fs (again using /proc/pid/root) by default. The additional argument you suggested could still be used to override this default behavior.


Add additional filter for Pull Request builds

Is your feature request related to a problem? Please describe.
We are using the gitactionboard as part of our Daily Operations tasks. As someone who is responsible for the status of all of our pipelines, it would be great to add an additional filter for all Pull Request builds. Right now it's possible to filter green builds and show the red ones only. Unfortunately tools like dependabot are leading to some additional builds and this is getting confusing for the person who is doing the operations for the team.
Describe the solution you'd like
I would like to get an additional filter so the daily operations guy can get a quick overview of the status of all applications on the master branch.

Hi @bennetelli, it's a nice idea to have more filters but instead of having just for master branch it'll nice to add possibility to filter any branch.

Hi @bennetelli, it's a nice idea to have more filters but instead of having just for master branch it'll nice to add possibility to filter any branch (from UI).

yea, I am fine with it as well.

any updates on this issue? :)

Hi @bennetelli, currently I am working in a different feature/improvement. Do you want to create a PR for this feature


train accuracy above 100%

When I am running funetune_youtube_las.py, finetune_insults_chain-thaw.py, finetune_semeval_class-avg_f1.py files, I am getting training accuracy above 100%.
Why is this happening? How can I solve?

I have the same problem, Do you figure out why?

I have the same problem. Do you know why?


Mistake

I ran the installed into the live version by accident, i removed all the game files and am now reinstalling will it be okay?

Yes, we are only copying 3 files in the game directory. So if you removed everything and reinstalled you'll be fine


Options for renaming filenames using product_title, creating directories based on bundle subtitles, additional allowed_chars

I find the downloaded filenames to sometimes be really esoteric (especially on the 3D printer bundles, but even the Book bundles can be difficult at times), copying all the files into sub-directories based on product_title really helps, but sometimes when I'm pulling PDFs onto my phone it's really helpful to use product_title for the filenames as well. So I added an option -f/--human_filenames that will use the product_title for the filenames.
Also I liked to group my downloaded bundles (well mostly the Book bundles) by bundle type, or bundle sub-title, so that "Humble Book Bundle: Fun with STEM by Adams Media" becomes "Humble Book Bundle/Fun with STEM by Adams Media" instead of "Humble Book Bundle - Fun with STEM by Adams Media". So I added an option -s/--sub-title that will control that.
The list of allowed characters seemed a little short, especially with with say the C# book bundles, so I added in some more allowed characters.
Added option to split bundle title into title and sub-title and create directories as such, ex Humble Book Bundle: Fun with STEM by Adams Media => Humble Book Bundle/Fun with STEM by Adams Media instead of Humble Book Bundle - Fun with STEM by Adams Media
Added additional allowed_chars , # and &

Gah... that rsplit in bundle_title_parts = order['product']['human_name'].rsplit(': ',1) should be a split

I will take a deeper look into the changes this weekend. But I know what you mean about the filenames, its not ideal, but I had to do it that way because some items (games, music, etc) will have the same bundle name and file extensions, and the only way to download all the files is to use the original filename.
For example, a game may be named Awesome Game and the file names are awesome_game_mac.zip & awesome_game_windows.zip. So if the bundle title is used, the files will save over each other into Awesome Game.zip
I do like the idea of having the option, but I would want to make sure it can work for all the data.
I see 2 options for this (would love to hear others too):

Be lazy, and just have the user accept that by using this option that some files may get overwritten (and save that to the cache file so its not asking every time)
2 Make it robust by checking if by using the bundle name it will not overwrite other files in the bundle, if it will, then fallback to using the original file name.

I think your Option 1 would be non-ideal, as with your example of different platforms, the user would have no control over which platform version they end up with (and may not even KNOW which version they got until they try to use it) - though if assuming alphabetical order, they'd basically always get the Windows version.
I'd propose something along the lines of "if using the bundle title and multiple files exist, download all of those files and put them all into a NEW zip, with the bundle title as the name of the zip"


Add thumbnail in the video directory

Description

This PR ensures thumbnails are downloaded inside the video directory.
Motivation and Context

This makes it consistent and easy to find.

For https://github.com/iiab/calibre-web/issues/161
How has this been tested?

by running
lb tubeadd test.db https://www.youtube.com/watch?v=WcLlpWmEpQ8
then
lb dl test.db --video https://www.youtube.com/watch?v=WcLlpWmEpQ8 --write-thumbnail --subs

Tested on Ubuntu 24.04 with xklb <IP_ADDRESS>

Screenshots (if appropriate):
root@box:/home/ubuntu# lb tubeadd test.db https://www.youtube.com/watch?v=WcLlpWmEpQ8
Importing playlist-less media https://www.youtube.com/watch?v=WcLlpWmEpQ8

root@box:/home/ubuntu# lb dl test.db --video https://www.youtube.com/watch?v=WcLlpWmEpQ8 --write-thumbnail --subs
root@box:/home/ubuntu# tree Youtube/
Youtube/
└── TED-Ed
    ├── WcLlpWmEpQ8.webp
    └── Why dont we cover the desert with solar panels Dan Kwartler_4.74M_[WcLlpWmEpQ8].webm

1 directory, 2 files

Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Question: Is this PR a companion needed alongside... ?

https://github.com/chapmanjacobd/library/commit/bb97e2401db3477bb76790233e941fb8f60b041e (commit merged 5h ago!)

No, I have yet to test it.

Just tested with and without https://github.com/chapmanjacobd/library/commit/bb97e2401db3477bb76790233e941fb8f60b041e. I cannot spot a difference.

it's probably because you already had https://github.com/chapmanjacobd/library/commit/bb97e2401db3477bb76790233e941fb8f60b041e before you wrote 86e33a2231bdf6dcbc70cd9d22b5ae7d51e74808
This PR is modifying the behavior of lb tubeadd --extra and not lb dl
This is why it's better to start with a failing test so that you can know if you are changing the right function 😁 https://en.wikipedia.org/wiki/Falsifiability

I was not clear by saying I spot no difference, I meant your commit (https://github.com/chapmanjacobd/library/commit/bb97e2401db3477bb76790233e941fb8f60b041e) doesn't affect PR#38 behavior. Isn't --subs using the same --extra mechanism?
Anyway, I'd like to add a proper test to support this claim. Is test_lb_fs a good hint to do this https://github.com/chapmanjacobd/library/blob/6a99a76efb2fed38cef3290f70ba3bb76c518a8a/tests/createdb/test_tube_add.py#L31?

You are right. I modified tubeadd --extra and your commit https://github.com/chapmanjacobd/library/commit/bb97e2401db3477bb76790233e941fb8f60b041e fixed it. I'll close this PR.

Is test_lb_fs a good hint to do this?

To test --write-thumbnail you'd need to go through the full download process. I think this is a good example:
https://github.com/chapmanjacobd/library/blob/6a99a76efb2fed38cef3290f70ba3bb76c518a8a/tests/mediadb/test_download.py
More tests like these can be committed to the repo but because these tests use the network I'd probably add @skip("network") before merging. However, I think it is good to have at least one full download test. I'd accept a PR that changes this test to use write-thumbnail instead of -no, remove the TODO: notice and assert that the thumbnail file exists

Thanks. Looks easy when I have a good example to start with. I'll try.

https://github.com/chapmanjacobd/library/blob/6a99a76efb2fed38cef3290f70ba3bb76c518a8a/tests/mediadb/test_download.py

@chapmanjacobd can you clarify how often tests like the above are run?
(e.g. is this run automatically by GitHub Actions?)

good to have at least one full download test. I'd accept a PR that changes this test to use write-thumbnail instead of -no, remove the TODO: notice and assert that the thumbnail file exists

👍

is this run automatically by GitHub Actions

Yes. They are run before each release. If any fail then the release is not published.
Linux, Mac OS, and Windows are tested


LIP-25

Renamed branch to feat/lip-25 and created another PR https://github.com/lidofinance/lido-improvement-proposals/pull/59


Basic devcontainer starting point for bicep

The Devcontainer installs:

Azure cli
Powershell
Bicep
Curl and Make

There is a simple readme with commands to run to get people deploying with the minimum amount of effort.

I'm not seeing bicep installed:

I'm not seeing it in the /user/local/bin:

I'm not seeing bicep installed

Oh woops not sure how I've managed to mess that up. Let me have a play tomorrow.

@alex-frankel I've had a play and I can't reproduce the issue you're seeing. I've applied for the codespaces beta so I can test, not sure when/if I'll get accepted.
One query I had was did the codespace checkout root then move to checkout this branch? I'm wondering if the Dockerfile and DevContainer definition weren't built as they're not yet merged into the repo.
To test out this theory I've pushed this branch to the main on my fork, if you launch the codespace from there does it work as expected?

That didn't work either unfortunately. I can try pinging someone on the codespaces team tomorrow. I just tried with the VS Code remote extension and it worked. Though there is an issue when the container is first run, dotnet core fails to install. I don't think that is something you can solve. Reopening the remote session works.

@lawrencegripper I had 2 problems running this locally:

The powershell install command doesn't currently work - I think because the new powershell version has a dependency on .net5. I recommend updating the docker image to mcr.microsoft.com/dotnet/sdk:5.0 (this fixed it for me, and note that we're also going to update Bicep to use .net5 very soon).
I believe because of an issue in the way Bicep is acquiring dotnet, we use the system-installed version over the locally-downloaded one. I'm publishing a new release (v0.2.14) to address that.

With those two changes, I have it working locally.

@anthony-c-martin Ah interesting, I didn't see the powershell install failures when I put this together and the update requiring dotnet5 explains why.
To avoid this problem in the future I've pinned the powershell version to 7.1.0 as an ARG in the dockerfile.

@lawrencegripper, great! Feel free to push and I can sign off and merge :)

oh man 🤦♂ Pushed to the wrong branch!


Generic is not materialized in macro-generated companion object of nested case class

Reproduction: Define a macro annotation. It adds shapeless.Generic[A] to the companion object of A being a class or trait (this should compile if the class is a case class or the trait is a sealed trait with case-class children). If the companion object doesn't exist the annotation creates it. The annotation can annotate not only a class/trait but a companion object itself (anyway Generic is materialized inside the object).
@compileTimeOnly("enable macro annotations")
class generateGeneric extends StaticAnnotation {
  def macroTransform(annottees: Any*): Any = macro GenerateGenericMacroImpl.macroTransformImpl
}

object GenerateGenericMacroImpl {
  def macroTransformImpl(c: blackbox.Context)(annottees: c.Tree*): c.Tree = {
    import c.universe._

    def modifyObject(obj: Tree): Tree = obj match {
      case q"$mods object $tname extends { ..$earlydefns } with ..$parents { $self => ..$body }" =>
        q"""$mods object $tname extends { ..$earlydefns } with ..$parents { $self =>
        ..$body
        _root_.shapeless.Generic[${tname.toTypeName}]
      }"""
      case _ => sys.error("impossible")
    }

    def modify(cls: Tree, obj: Tree): Tree = q"..${Seq(cls, modifyObject(obj))}"

    annottees match {
      case (cls: ClassDef) :: (obj: ModuleDef) :: Nil => modify(cls, obj)
      case (cls: ClassDef) :: Nil => modify(cls, q"object ${cls.name.toTermName}")
      // this works for the companion object of a sealed trait or top-level case class but not nested case class
      case (obj: ModuleDef) :: Nil => modifyObject(obj)
      case _ => c.abort(c.enclosingPosition, "@generateGeneric can annotate only traits, classes, and objects")
    }
  }
}

Everything compiles if the annotation annotates:

a sealed trait (top-level or nested) or its companion object (existing or not),
a top-level case class or its companion object (existing or not),
a nested case class.

This doesn't compile if the annotation annotates:

the companion object of a nested case class.

object App {
  case class A(i: Int, s: String)

  @generateGeneric
  object A

  //implicit error;
  //!I gen: shapeless.Generic[App.A]
  //  @generateGeneric

  //scalac: object A extends scala.AnyRef {
  //  def <init>() = {
  //    super.<init>();
  //    ()
  //  };
  //  _root_.shapeless.Generic[A]
  //}

  //scalac: Generic.instance[App.A, Int :: String :: shapeless.HNil](<empty> match {
  //  case (x$macro$3 @ _) => ::(x$macro$3.i, ::(x$macro$3.s, HNil)).asInstanceOf[Int :: String :: shapeless.HNil]
  //}, <empty> match {
  //  case ::((i$macro$1 @ _), ::((s$macro$2 @ _), HNil)) => App.this.A(i$macro$1, s$macro$2)
  //})
}

If we replace _root_.shapeless.Generic[${tname.toTypeName}] with explicitly resolved _root_.shapeless.Generic[${tname.toTypeName}](_root_.shapeless.Generic.materialize) then the error is
//App.A.type does not take parameters
//  @generateGeneric

https://scastie.scala-lang.org/DmytroMitin/im8OhAEySOqOiS0d7tOxvQ/15 (toolbox is not a part of reproduction, it's just for Scastie because it doesn't support multiple files/subprojects)
It seems that Generic materialized in the macro-generated companion object of a nested case class (when the object is annotated) is typechecked when apply method doesn't exist yet.
A possible fix is https://github.com/milessabin/shapeless/pull/1286
This impacts how Circe materializes semi-automatic codecs (when implicits are macro-annotation generated).
Discovered in https://stackoverflow.com/questions/74364545/type-parameter-for-implicit-valued-method-in-scala-circe

Interesting... I wonder if we could force the type checking of apply somehow. I will take a look maybe this weekend.


Add new meteorological forcing data type, sunshine duration (h).

In our research we have not short wave or cloud cover data but we have sunshine duration data with a so-so quality during 1950-2014. And we find that potential evapotranspiration data estimated in Pearl River Basin, China is rather inaccurate (it increased significantly, but other exam showed that it should be decreased obviously, while sunshine duration decreased during those years). So we manage to add sunshine duration into our model.

@Sibada -
Thank you for issuing a pull request.  However, I don't think we will be able to accept this feature. The MTCLIM meteorologic forcing pre-processor has been depreciated in VIC 5 (#288)  which we will be releasing shortly.  In the future, we hope you will continue to make contributions to VIC.  Next time please inquire with us ahead of time to check on their applicability.


Include info to automatically generate manifest

Fixes #29.
So, I just got tired of having my CI fail to pass the build every time we added or updated our Gemfile, because it takes a while for the tests to run, and by the time I learn of the failure, I've already moved on to something else, and I need to go back and fix the manifest.
We don't really care about versions of the gems -- only the licenses -- I didn't see why I needed to verify the gem versions in the manifest matched.
Are there any pitfalls to doing this?
Or, on the other side of the issue: what benefits does having the manifest bring us in general, and more specifically, what benefit do we get from checking the gem versions?
It seems like as long as the versions match the expectations I set up in my Gemfile, which Bundler takes care of, then I feel good about it, so I'm not sure what is the purpose of having Papers check it.

That sounds reasonable. Does 463a655 do a good job of that?

Works for me. Thanks!


html meme redirects

Quick switch on-off html page that checks if user is still there

https://www.youtube.com/watch?v=ZXsQAXx_ao0


Missing doc generation from pre-loaded controller

I have a case where some controllers are loaded before Swagger::Docs::Geneator resulting in use of fake swagger_docs method and no doc generated. I can't postpone Controller declaration because I need to reopen the class to add methods (and swagger_api call).
I'm encounter this case because I'm using rails_engine_decorators. But since controllers are load earlier than Swagger::Docs::Geneator there the problem.

Hey! I'm not familiar with rails_engine_decorators and I'm not really close enough to your code to try anything out. The only thing I can think of is that you find the most appropriate location and timing in your app to somehow call swagger-docs. If you find a way that is something we can reuse then definitely submit a PR and we'll get that merged in. Thanks!


leaps: 0.5.1 -> 0.9.0

Motivation for this change
Significant update of the leaps software.
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build leaps

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘leaps-0.9.0’ in /private/var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/development/tools/leaps/default.nix:19 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/ciij20ryc9va31rslp2hmbp754ya9qfz-leaps-0.9.0-bin/bin/leaps
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/ciij20ryc9va31rslp2hmbp754ya9qfz-leaps-0.9.0-bin/bin
patching script interpreter paths in /nix/store/ciij20ryc9va31rslp2hmbp754ya9qfz-leaps-0.9.0-bin
checking for references to /tmp/nix-build-leaps-0.9.0.drv-0 in /nix/store/ciij20ryc9va31rslp2hmbp754ya9qfz-leaps-0.9.0-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/57x90f79i3l7x4lgbrc8vqwgc87aa9jv-leaps-0.9.0
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/57x90f79i3l7x4lgbrc8vqwgc87aa9jv-leaps-0.9.0
checking for references to /tmp/nix-build-leaps-0.9.0.drv-0 in /nix/store/57x90f79i3l7x4lgbrc8vqwgc87aa9jv-leaps-0.9.0...
/nix/store/ciij20ryc9va31rslp2hmbp754ya9qfz-leaps-0.9.0-bin

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/gvgvnkzc13chcwn1wf87b9lkv99n5i6k-leaps-0.9.0-bin/bin/leaps
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/gvgvnkzc13chcwn1wf87b9lkv99n5i6k-leaps-0.9.0-bin/bin
patching script interpreter paths in /nix/store/gvgvnkzc13chcwn1wf87b9lkv99n5i6k-leaps-0.9.0-bin
checking for references to /build in /nix/store/gvgvnkzc13chcwn1wf87b9lkv99n5i6k-leaps-0.9.0-bin...
shrinking RPATHs of ELF executables and libraries in /nix/store/0rbr5z0c979llbnrmh03qiiqafij7g7h-leaps-0.9.0
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
patching script interpreter paths in /nix/store/0rbr5z0c979llbnrmh03qiiqafij7g7h-leaps-0.9.0
checking for references to /build in /nix/store/0rbr5z0c979llbnrmh03qiiqafij7g7h-leaps-0.9.0...
/nix/store/gvgvnkzc13chcwn1wf87b9lkv99n5i6k-leaps-0.9.0-bin


move bot error channel config to env file

What issue is this solving?

Closes #283
Description
Moves setting of the channel for bot errors to the .env file.
Any helpful knowledge/context for the reviewer?
Set the channel in the .env file with the variable BOT_ERROR_CHANNEL_ID. You can add the line below to the .env file:
BOT_ERROR_CHANNEL_ID=ID_FOR_audit-logs

Is a re-seeding of the database necessary? No
Any new dependencies to install? No
Any special requirements to test? No

(e.g., admin perms, alt account, etc.)

Please make sure you've attempted to meet the following coding standards

[X] Code has been tested and does not produce errors
[X] Code is readable and formatted
[X] There isn't any unnecessary commented-out code

Not sure if this is a viable option. This is what I tried earlier and had issues with the bot being in multiple servers sharing the same named channel. It posted the errors in the wrong one.
See
https://discord.com/channels/810930683732033597/810977155130458145/947623394990497883
And
https://discord.com/channels/810930683732033597/810977155130458145/947629493760172113

I modified this to use the error channel ID which now has to be set in the .env file. That should work across servers. Solution found in this discord.js bot: https://github.com/Simpleboy353/REAPER-2.0


What's the meaning of the first parameter of this code?

What's the meaning of the first parameter of this code? You only need an account and password to connect to the matt server. Why add it? Or what's the point? I don't understand. Please let me know
 if (client.connect("arduinoClient", "testuser", "testpass")) {     client.publish("outTopic","hello world");     client.subscribe("inTopic");   }
The case code appears in the 'mqtt_auth. Ino' file

The first parameter is the MQTT Client ID. This is used to unique identify the client to the broker. Every client that connects to a broker must have a unique client id.
See https://www.hivemq.com/blog/mqtt-essentials-part-3-client-broker-connection-establishment/#clientid for more information.


[ZEPPELIN-4238] Fixed gson-2.2.jar file not dependency in shellInterpreter pom

What is this PR for?
shell interpreter pom file need dependency commons-lang-2.5.jar and gson-2.2.jar
What type of PR is it?
[Bug Fix]
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4238

How should this be tested?
CI Pass
Screenshots (if appropriate)
Questions:

Does the licenses files need update?
Is there breaking changes for older versions?
Does this needs documentation?

@zjffdu , @felixcheung  Please help me review code, Thanks!

@liuxunorg Could you describe why shell interpreter needs gson-2.2.jar and what is the impact of missing gson-2.2

Ok, I updated PR comment https://github.com/apache/zeppelin/pull/3403#issue-297074981.

LGTM


material components documentation examples (V 12.2.5) are failing to run on Stackblitz

When we choose option to edit example on stackblitz, new tab gets open with example but it fails to run and prompt as below error
Steps:

Go to https://material.angular.io/components/sidenav/overview (Version 12.2.5)
Open any component example and edit it on stackblitz
example fails to run and prompts error as below:
Error in ~/src/main.ts ngcc failed to run on<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

Related to stackblitz/core#1364.
Workaround: set enableIvy: false in tsconfig.json.


docker-api v1.32 dropped support for Ruby 1.9

No longer possible to install this with Ruby 1.9, as https://github.com/swipely/docker-api/releases/tag/v1.32.0 dropped support for 1.9.  Setting <1.32 in the synapse.gemspec appears to resolve this.

@ianbamforth can you just install docker-api manually before you install synapse?
e.g.
gem install docker-api -v ...
gem install synapse


variableId in viz responses also includes entityId

think the plan here is to pass the column header to R as we currently do, but to not pass the entity id. and instead let plot.data parse the var and entity ids from the column name and return them independently. need to decide i guess if were ignoring units and scale when those come along, as theyll also be part of the column header.

this should be resolved w plot.data v1.0.0


krustlet does not support Kubernetes v1.22

krustlet 1.0.0-alpha.1 and lower use certificates.k8s.io/v1beta1 which is deprecated in Kubernetes v1.19+ and unavailable in v1.22+. A pull request must be created for krustlet to use certificates.k8s.io/v1 instead. This was already done and tested in our fork (see https://github.com/stackabletech/krustlet/commit/ab5b00ac3997d2dc3b257b18bf17e5d264f41a5e).
The Kubernetes version feature should not be specified anyway in libraries like kubelet and krator (see https://docs.rs/k8s-openapi/0.13.0/k8s_openapi/index.html#crate-features). Perhaps it can be removed.

Pull request for krustlet with the migration to certificates.k8s.io/v1 created (krustlet/krustlet#671).
Removing the Kubernetes version feature from k8s-openapi will be addressed in kubelet with krustlet/krustlet#612. For krator this was already done.

The pull request was merged.


Compile maybe?

Any chance you could compile it and upload on here geosn0w? I have xcode but am on windows right now and really don't want to switch over if I don't have to. Sorry lol im lazy

It is compiled in the IPA BUILD folder.


Issues with Lists in Slate Transform

Describe the bug
The slate transform seems to swallow lists in some cases. Here is a sample test.
list-slate.zip
update Markdown to Slate seems to lose any structure except for the first paragraph:
bash-3.2$ cat list.md
1. item one
2. item two
   - sublist
   - sublist

bash-3.2$ markus parse --sample list.md --slate --output data.json
13:16:40 - info: Creating file: data.json
13:16:40 - info:
{
  "object": "value",
  "document": {
    "object": "document",
    "data": {
    },
    "nodes": [
      {
        "object": "block",
        "type": "ol_list",
        "data": {
          "tight": "true",
          "start": "1",
          "delimiter": "period"
        },
        "nodes": [
          {
            "object": "block",
            "type": "list_item",
            "data": {
            },
            "nodes": [
              {
                "object": "text",
                "text": "item one",
                "marks": []
              }
            ]
          },
          {
            "object": "block",
            "type": "list_item",
            "data": {
            },
            "nodes": [
              {
                "object": "text",
                "text": "item two",
                "marks": []
              }
            ]
          }
        ]
      }
    ]
  }
}

Which means normalization returns only the top level list:
bash-3.2$ markus draft --data data.json --slate
13:18:14 - info:
1. item one
2. item two
bash-3.2$ markus normalize --sample list.md --slate
13:18:31 - info:
1. item one
2. item two

This is true of anything which isn't pure text from a single paragraph (breaks, nested lists, multiple chunks of text, etc.)

Proposed fix in #123
Here is a test with nested lists:
bash-3.2$ cat test/data/nested-list.md
Prolog
1. item one
2. item two
   - sublist
   - sublist

Epilog
bash-3.2$ ../markdown-cli/index.js normalize --sample test/data/nested-list.md --slate
16:45:40 - info:
Prolog
1. item one
2. item two
   - sublist
   - sublist

Epilog

Fixed in #123


You need hands to lay in bed which doesn't make any sense

Description

Play entity with buckle but no hands
Try to lie in bed
You don't have hands

funny thing about this is you can still unbuckle, and it works on other players too

I could not replicate unbuckling. The cuffable comp seems to resolve that one. Beds are still un-layable though.


Fix release date in RPM spec files

Describe the bug
We should change the 4.9.1 release day in the RPM spec files from Fri to Thu to avoid packaging errors.

This was fixed here:

https://github.com/wazuh/wazuh-indexer/pull/338


[release-v1.12] Bump to new 3.8.x Quarkus.io

Let's see if we want this... or not...
Therefore putting it on
/hold

/retest

/retest

/unhold

We will postpone the bump to 1.34+


Refused to load the script 'https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.8.0/highlight.min.js' because it violates the following Content Security Policy directive: "script-src 'self'". Note that 'script-src-elem' was not explicitly set, so 'script-src' is used as a fallback.

Describe the issue

Hello library developer. I am encountering some minor errors when using your library within my Chrome extension application. I think you are aware of these errors and can fix them quickly. Thank you."
Procedure version
^4.17.3
Reproduction link
No response

Can you use this library or download and save it locally for your own library? I look forward to your response.
https://github.com/highlightjs/highlight.js
npm install highlight.js

You can refer to this example
https://imzbf.github.io/md-editor-v3/en-US/demo#🙍🏻♂ Import All Library

I want to tell you: 'YOU ARE AWESOME'. I have fixed my code, and it is now working without any errors. Thank you very much. ❤


feat: Update usage of get-env in gh-actions (deprecation)

Summary
Update gh-actions workflow to use the new env file style instead of get-env
Related issues
#511
Changes these areas

[X] Bugfixes
[ ] Feature behavior
[ ] Command line interface
[ ] Configuration options
[ ] Internal architecture
[ ] Snapshot data layout on disk

@pirate tests are passing after the change, so it must have worked.


Modifies odo app commands to work outside a directory

fixes #1646 (only the app part)

odo app list
odo app describe app
odo app delete app -f

all those commands are working fine for me

[odo]  •  Waiting for project to come up  ...
Deleting project:
Running odo with args: [project delete  -f]
[odo]  ✗  The project  does not exist. Please check the list of projects using `odo project list`

/retest

  	github.com/openshift/odo/pkg/url/labels	1.011s	coverage: 100.0% of statements
--- FAIL: TestRemoveDuplicate (0.00s)
    --- FAIL: TestRemoveDuplicate/Case_2_-_Remove_duplicates,_none_in_array (0.00s)
        util_test.go:1069: expected [bar foo], got [foo bar]

/retest
fix in #1786

Please rebase with latest changes, travis is failing in project creation stuff.

Done

[odo] I0604 13:27:28.348462   10889 cmdutils.go:26] Error:
[odo] Unable to connect to OpenShift cluster, is it down?
[odo]  ✗  Unable to connect to OpenShift cluster, is it down?
Deleting project:
Running odo with args: [project delete  -f]
[odo]  ✗  Unable to connect to OpenShift cluster, is it down?

/retest

Unable to connect to OpenShift cluster, is it down?
[odo]  ✗  Unable to connect to OpenShift cluster, is it down?
Deleting project:
Running odo with args: [project delete  -f]
[odo]  ✗  Unable to connect to OpenShift cluster, is it down?

/retest

@amitkrout Done

Looks good to me with respect to test. Can someone review the core implementation cc_ @kadel
/approve

@cdrage Done

@girishramnani @kadel @cdrage I have removed the os.exit() call. Ready for review again.

@cdrage Fixed

 ✗  Applying configuration
[odo] I0610 06:39:34.776776    7005 cmdutils.go:26] Error:
[odo] unable to update DeploymentConfig image for cmp-git component: unable to update the current DeploymentConfig cmp-git-testing: unable to start deployment: unable to instantiate Deployment for cmp-git-testing: replicationcontrollers "cmp-git-testing-2" not found
[odo]  ✗  Failed to update config to component deployed

/retest

[odo] I0610 09:32:42.921827   18489 odo.go:70] Could not get the latest release information in time. Never mind, exiting gracefully :)
Running odo with args: [component create nodejs --project inevklodqq cmp-git --ref test]
[odo]  ✗  The --ref flag is only valid for --git flag
Deleting project: inevklodqq
Running odo with args: [project delete inevklodqq -f]
[odo]  ✗  Unable to connect to OpenShift cluster, is it down?

/retest

/lgtm


Lured Pokemons

How do I show lured pokemons?

I was wondering the same thing,  Does anyone know if this can be done?

It's not implemented yet I guess

Duplicate of #1222


Title: Dragon's World Game

Is your feature request related to a problem? Please describe.
I want to add a dragon world game.
Describe the solution you'd like.
Dragon’s World is a game in which one dragon tries to save itself from the other dragon by jumping over the dragon which comes in its way. The score is updated when one dragon saves himself from the other dragon.
Describe alternatives you've considered.
It contains various features like:

Player score
It will contain the obstacle from which the dragon has to save itself.
It will contain the dragon which has to be saved from the obstacle i.e. the other dragon.

Add any other context or screenshots about the feature request here.

@swapnilsparsh kindly assign me this.

@Explorer003 Thank you for providing your insight but it is not required as of now


Ignore all auxiliary LaTeX files.

These files are generated by the LaTeX compiler and serve as a cache between compilations. They should not be version controlled.
More information about these file types: http://en.wikibooks.org/wiki/LaTeX/Basics#Ancillary_files
The LaTeX homepage: http://www.latex-project.org/

Thanks, but this is what TeX.gitignore is for.

Oh, my mistake!


add helper fn for task codes

fixes issue #412

work in progress. i will add another helper for channel mappings, too


oauth_body_hash

What are your thoughts on supporting oauth_body_hash? I'd be happy to write the code for it and submit a pull request. I just want to get some feedback first.

As far as I'm understanding you are going to need additional flag parameter for the sign method to indicate whether body hash should be used or not. Then you need to create that hash and add the oauth_body_hash parameter when creating the signature. Is that it?

Yeah I think so. The new sign method could be
function sign (signMethod, httpMethod, base_uri, params, consumer_secret, token_secret, body) with body as the last parameter to preserve backwards compatibility.
The mere presence of the body will act as a truthy value to go ahead and create the hash.

Actually now when I think about it probably the hash generation should not be part of this module, because this module is concerned only with generating the signature, not with generating the actual OAuth parameters. See how all of the OAuth parameters are being created prior calling this method here https://github.com/request/request/blob/master/lib/oauth.js
So if we follow the logic from that example oauth_body_hash parameter should be created there instead.

Oh you know what, you're totally right.
Do you think the oauth options object in request should have a new key body_hash as a boolean, and just have it be false implicitly?
The user can just do oauth = { body_hash: true } to tell https://github.com/request/request/blob/master/lib/oauth.js to build the oauth_body_hash parameter.

That sounds good. Then after the initial loop there you can have
if (oa.oauth_body_hash) {
  oa.oauth_body_hash = // generate
} else {
  delete oa.oauth_body_hash // that's just in case it was false or something other than undefined
}

Hmm wait, someone may want to generate it on their own, so you'll actually need a little bit of typechecking there. If it's string then use it, it it's boolean and true, generate it, otherwise delete it.

When you say

someone may want to generate it on their own

it is just their own custom payload to pass to the generate function here oa.oauth_body_hash = // generate right? As opposed to request automatically grabbing the payload and passing it through the new generate function?
If so, I think that sounds like a good idea.

Well the value for the oa.oauth_body_hash parameter should be a hash string right?
If that's the case, someone may want to pass their own hash string there and in fact there is a possibility that someone is already doing that.
The boolean option is intended for users that want to have the hash string, but don't want to generate it on their own.

Gotcha, that makes a lot more sense. I guess it's time to code! I'll try to send a pull-request sometime today, unless you beat me to it haha

:+1: just make sure you have the proper tests too :)


Allow poison ~> 2.0

Otherwise it breaks if any of your dependencies is using poison 2.0 or 2.1
I ran the tests with 2.1 and it passes.

Thank you!


How to enable all loging ABOVE a certain level.

How could I enable logging above a given level, such as  INFO, WARN & DEBUG ?
When level warn is active with set DEBUG=WARN  then all warn and info messages but no debug messages shall be shown.
This would be a manual workaround:
var info= require('debug')('level.XXX')
var warn = require('debug')('level.XX')
var debug = require('debug')('level.X')

and then for example set DEBUG=level.XX*   But that seems cumbesome. Is there a better way?

Debug isn't a logging library, it's meant to conditionally turn on debug logs. There is no concept of log levels. Instead, break up your namespaces into different components and enable the ones you care about: my_app:http_server, my_app:redis_connection, etc.


Artist details card design

New version of the artist details page. Yes, the navigation (and the rest) is still ugly, but functional. The image gallery looks quite nice already I think. But from now on it should be mostly styling as the functions are good for now.
https://trello.com/c/C65jtjIY

 Revision of artist detail page


CHET-298: Update default Jira version to 8.9.1

Although still functional, I noticed when digging around in Jira, that the default version we install - 8.5.3 - complains about not fully supporting the DB versions recently added to the templates, see below:

I've re-tested all of these DB versions (9, 10, 11) on Postgres and Aurora for version 8.8.1 of Jira and we get a supported result, see below.
Although testing was done with 8.8.1 this PR is recommending we upgrade the default version to 8.9.1 which is currently the latest version of Jira: https://confluence.atlassian.com/jirasoftware/jira-software-release-notes-776821069.html and which will obviously support the same DB versions as 8.8.1

Typically we try to keep the quickstart on the latest enterprise release/LTS version. I think that we should continue this, and update to 8.5.4 because the default configuration (jira 8.5.4 & postgres 9.6) is a supported configuration. With the caveat the the DB version parameter description says postgres 11 is only supported from Jira X

On second thought it's probably more important that they don't end up in an unsupported DB so I think this is ideal.

On second thought it's probably more important that they don't end up in an unsupported DB so I think this is ideal.

Speaking with @ddomingorh we think leaving the default Jira version as is, is a better option. We want to stay on an LTS release by default as things brings obvious benefits. The suggested change is to update the description on the DBVersion dropdown to provide a warning that version selected needs to be supported by the Jira version supplied.

@dylanrathbone how about:

The database engine version to use; we'll install a suitable minor version for your chosen engine. Make sure that the Jira version you're installing supports the database engine you chose. Check https://confluence.atlassian.com/x/bqr1Nw to verify this.

@dylanrathbone how about updating the DBEngineVersion description to:

The database engine version to use; we'll install a suitable minor version for your chosen engine. Make sure that the Jira version you're installing supports the database engine you chose. Check https://confluence.atlassian.com/x/bqr1Nw to verify this.

Slight tweak...?

The database engine version to use; we'll install a suitable minor version for your chosen engine. Make sure that the Jira version you're installing supports the database engine selected. Check https://confluence.atlassian.com/x/bqr1Nw to verify this.


End date for All Day Events

When an All Day event ends on the same day it starts, the widget should not show the end date as it just duplicates the event header.

Sorry this duplicate of #236

@neilromig Let's make #236 more concrete


IllumioSaaS solution repackaged with modified logo link

Required items, please complete
Change(s):

Modified logo link and repackaged solution

Reason for Change(s):

When we selected 'go live', got the following message during certification
"The logo URL should point to file in repository and master branch.
Please raise a Pull Request in GitHub and push updated offer for certification once the Pull Request is approved/merged. "

Version Updated:

No

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

To the reviewer: Could you please check if any other links need to be modified as well before we go live?


Ship minimal monthly payments to sample of MDN users

User story
As a user supporting MDN with payments, I'd like to schedule a monthly payment, so I don't have to remember to pay every month.
Acceptance criteria

Monthly payment banner is shown in production to:

[x] Beta users (target Thursday, November 15th)
[x] Logged-in users (target Friday, November 16th)
[x] 1% of anonymous users (target Monday, November 19th)

[x] One-time payment option is still available
[x] Interaction is measured by Google Analytics, to allow measuring where users drop out of the pipeline

References

MDN Recurring Payments Sprints (Potato's private Trello board)
bug 1490727 - Payments feature tracker

This shipped on schedule, with a big list of follow-on tweaks. Let's track those in a new user story / epic for the next sprint, but keep the bug open for now.


Not working on Windows

Doesn't work anymore in Windows and Ruby 2.2. due to issues with coolline https://github.com/Mon-Ouie/coolline/issues/19

Even with config.coolline = false?

@jjgh Did you try the recommendation in the README?

Closing due lack of feedback.
If the error persists or someone is willing to take a stab just comment over here please.


Menu looks distorted in libreoffice

Hi FreeRDP developers,
there seems to be a little problem with the drawing operation libreoffice uses to draw it´s menu, I´ll attach some screenshots so you can see for yourself.

Okay it looks like i can´t attach files, so here is a link to a sample:
http://dl.dropbox.com/u/21920446/libreoffice.png

His server is ws2008r2, I believe it was running at -a 16. -sw gdi doesnt not seem to help, nor -a32.
(ignore my terminal window in bottom right)


Add API for explicitly starting/stopping runtimes and spawning futures on a specific runtime

Currently runtimes in runtime are fully implicit and run in the background without any control over their lifetime, what futures get run on them or what (and how many threads) they use.
It would be useful to have API for all of the above. There are a couple of use-cases I can think of, some of which I'm doing with tokio currently in one way or another.

Usage of runtimes in library crates without interfering with any other futures code other library crates or the application might use. This would potentially also go with specific per-thread configuration inside the library crate, for e.g. setting thread priorities of the runtime in a way that is meaningful for what this specific library is doing. See also https://github.com/rustasync/runtime/issues/8
Similar to the above, but an extension: plugins. For plugins you might want to use a runtime internally, but at some point you might want to be able to unload the plugin again. Doing so requires to be able to shut down the runtime at a specific point and to ensure that none of the code of the plugin is running anymore.
Error isolation. While this is probably done even better with separate processes, being able to compartmentalize the application into different parts that don't implicitly share any memory with each other could be useful, also for debuggability.
Integration with other runtimes, like the GLib one (see also https://github.com/rustasync/runtime/issues/21#issuecomment-492608914). There you can (and generally do in bigger applications) create and run multiple runtimes (GMainContext / GMainLoop). There's the global default one, which is used also for the GTK+ toolkit to do all the UI things, but if you have other parts of your application doing async operations it often makes sense to create a separate one that runs on another thread (they're always single-threaded) to not interfere with the UI event loop in any way.

I realize that this goes a bit against the goal of making everything as simple as possible to use, but maybe such an API for starting/stopping/configuring runtimes could be made available in addition to the simple, do-it-all-implicitly API.

I wonder if we follow #43, we could in addition also add a builder to set varying options. Not quite sure what that API would look like, but I think it might allow us to tick the right boxes (:

#43 is mostly about being able to call async functions in a blocking fashion, or not? While that might internally spawn a runtime (or maybe just run things on an existing runtime and block on a condition variable?), this seems not that closely related to being able to spawn and shut down your own runtimes in one way or another.
For that you probably don't necessarily want to block directly on a future but have some more API for a) creating a runtime (with whatever settings), b) actually starting it (which for a single threaded runtime would block the current thread, otherwise probably not?), and c) for shutting it down (and being able to wait for it to be shut down). I realize that you model a) and b) together with such a block_on() API, and have c) simply be a way to finish the given future (e.g. via a oneshot channel), but the mixing of the aspects of block_on and "start/create a new runtime" seems a bit unclean to me.
But maybe you're thinking of a nice API for doing all that together at once that I don't see right now, so maybe it's actually fine :)

@sdroege thanks for clarifying! -- I think I might have intially missed some of the constraints you laid out, so this has been very helpful!


[WIP] Improve realm item adapter with collection changes

@FabianTerhorst as the realm extension is no longer (at least for now) part. I'll close this


fix: use params in get_all_users(GET) endpoint

Please describe the purpose of this pull request.
Ensure that the GET request does not expect a body and correctly handles the parameters as part of the query string
How to test
poetry run pytest -s tests
Have you tested this PR?
NA
Related issues or PRs
NA
Is your PR over 500 lines of code?
No
Additional context
NA

hey @sarahwooders, hope all is well. what do you think about this PR?


Tiles on the quick settings panel are disabled

Upon installing DevQS and adding the 'dev', 'adb', and 'wireless adb' tiles in my phone's quick settings panel, the three tiles are disabled even after granting the 'WRITE_SECURE_SETTINGS'.
By the way, I'm using Android 12.

Is the device unlocked? The tiles are disabled on the lock screen.

Even if it's unlocked, the tiles are still disabled. I also tried turning off the 'developers option' but it was still the same.

Do you mean it still says "Unavailable" even after unlocking the device?

What launcher / home screen app are you using? Have you tried other launchers / home screen apps?

It doesn't say 'Unavailable', it's just disabled.

It doesn't say 'Unavailable', it is just disabled for some reason. I even changed to my default launcher.

Do you by any chance use a device from one of the manufacturers listed in dontkillmyapp.com? Have you tried allowing auto-start / turning off "optimizations" for the app?

My device doesn't belong to the list of phone manufacturers indicated on the site but I tried enabling it in the auto-start, and it worked!!!
Thank you so much for the help and sorry for the trouble...


Add BufferTextWriter and SequenceTextReader

These remove the need for AsStream() and StreamReader/StreamWriter to read and write text from ReadOnlySequence and IBufferWriter and dramatically reduce GC pressure that otherwise apply when using such adapters.
BufferTextWriter adapts IBufferWriter<byte> to be access as a TextWriter.
SequenceTextReader adapts ReadOnlySequence<byte> to be access as a TextReader.

@davidfowl, does corefx include such types already?

No these types don’t exist

Do you see general applicability sufficient to warrant a PR to corefx? Should I file an issue to discuss?