Misbehavior after updating

The $results = User::select(['id', DB::raw('concat(firstName," ",lastName) as fullName')]) was working, but after a recent update it returns empty string for fullName. I have to change it to $results = User::select(['id','firstName','lastName', DB::raw('concat(firstName," ",lastName) as fullName')]) to work.

Misbehavior after updating

?

User::select(['id', DB::raw('concat(firstName," ",lastName) as fullName')]) returns empty strings for fullName in the last version. Does it work for you?

in the last version

?

@GrahamCampbell  My current version is 5.2.22. I do not know the exact version that the issue appeared afterward. I had  several updates before I found out about the bug.

I do not know the exact version that the issue appeared afterward

Please can you find out then.

The issue  appears in Laravel Framework version 5.2.39 yet. Have it fixed in 5.3?

Closed pending a response. Please provide the exact patch versions this broke between.

5.2.21, I guess.

5.2.21 -> 5.2.22?

Please can you confirm this, and suggest a possible cause?

Maybe I should explain better, these issues are here for the community. They're not support. We expect people to investigate their own issues as far as possible. GitHub issues just allow people to communicate.

I will try to investigate that as soon as possible. I agree with your last comment. However, if there is a bug and the bug reporter for some reason cannot provide more information, I think the issue should be kept open. Closing the issue does not fix that bug! Thanks for your time.

I closed it after 6 months of inactivity.

This query:
User::select(DB::raw('concat(first," ",last) as fullName'))->get();

It works for me on a fresh 5.3 installation, 5.2.21, 5.2.22, and 5.2 latest.
The response contains a model with the following attributes:
#attributes: array:1 [▼
        "fullName" => "Beric Dondarrion"
      ]

Closing this since the issue cannot be replicated as well as lack of activity, please feel free to ping me if you're still facing such issue using the latest 5.2 release or 5.3.

@themsaid I has the attribute but when you apply toArray() the full name will vanish.
try this: dd(User::select([DB::raw('concat(firstName," ",lastName) as fullName')])->limit(2)->get()->toArray());


Request forms and data from customers to add to our database

Request forms used by our customer and set it up in our staging environment.

Data has been acquired, wonder how we will integrate it


Assert error when closing app during lesson 11 : Layers

I have everything working fine with no compile errors, but when I close the app with the 'x' button in the top right corner I will get an Assertion Error and the break will be at the delete mShader; statement in the Layer Destructor. Any clues as to why I might be getting this?

I guess never mind, since you got the same issue just without the assert error and corrected it at the end of the lesson. The correction worked for me also.


[Release-1.27] - MySQL + Kine Conformance Checks Fail

Backport fix for MySQL + Kine Conformance Checks Fail

#10023

Validated using the latest RCs on a single node running sonobuoy with the sig-api-machinery and conformance results. Attention to the fact that the shown failures are expected to fail when tested against a single node.
$ kgn
NAME              STATUS   ROLES                  AGE    VERSION
ip-1-1-1-84       Ready    control-plane,master   102m   v1.27.15-rc3+k3s1

$ sono_results
Plugin: e2e
Status: failed
Total: 7214
Passed: 380
Failed: 2
Skipped: 6832

Failed tests: //these are meant to fail when run against a single node
 [sig-architecture] Conformance Tests should have at least two untainted nodes [Conformance]
 [sig-apps] Daemon set [Serial] should rollback without unnecessary restarts [Conformance]

Plugin: systemd-logs
Status: passed
Total: 1
Passed: 1
Failed: 0
Skipped: 0

Run Details:
API Server version: v1.27.15-rc3+k3s1
Node health: 1/1 (100%)
Pods health: 7/8 (87%)
Details for failed pods:
sonobuoy/sonobuoy-e2e-job-dd21f0420ad84f4c Ready:False: PodFailed:
Errors detected in files:
Errors:
9 podlogs/kube-system/metrics-server-c44988498-bnnl9/logs/metrics-server.txt
3 podlogs/sonobuoy/sonobuoy-e2e-job-dd21f0420ad84f4c/logs/e2e.txt
Warnings:
1 podlogs/sonobuoy/sonobuoy-e2e-job-dd21f0420ad84f4c/logs/e2e.txt
1 podlogs/sonobuoy/sonobuoy/logs/kube-sonobuoy.txt


How can I change text header and text links of left side of html2 version

Hi,
How can I change text header and text links in position 1 and 2 specified in this picture:

Thanks

@NabiKAZ this plugin builds up much of the settings you can used with Swagger Codegen dynamically.
In the html2  generator's "Generator Settings" tab, you can specify a template directory to use modified mustache templates from the swagger codegen project. I haven't tested this with the plugin, but since it's essentially just a pass-through to swagger-codegen, it should just work.
To customize the templates for html2 generation, here are the steps.

Clone or download swagger-codegen

git clone git@github.com:swagger-api/swagger-codegen.git
git checkout v2.2.2
NOTE: above version must match version from plugin (Code -> Swagger -> Codegen Version)

Copy all files from src/main/resources/htmlDocs2 into a separate directory (you'll point to this directory later during generation)
Modify the .mustache files as desired. Don't change the filenames or the generator may not work correctly.
Refer to StaticHtml2Generator.java if you have questions about any variables used in the templates.
When you generate, specify the new directory where you copied the sources in step 2.

Let me know if that helps.

Thanks for your reply.
You can see: https://github.com/swagger-api/swagger-codegen/issues/5680

Ah, I see now. I misunderstood. I thought you wanted to change the appearance rather than the text that gets modified by Swagger Codegen.

I'm going to close this since an appropriate response of customized templates was provided in the linked issue. Let me know if there are additional questions related to this plugin.


New Check: STAT table has Axis Values

Rosalie recently made a pull request for a VF family which didn't contain any STAT Axis Values. By default fontmake will export a STAT table but won't include any meaningful Axis Values. We need to make a check that will fail these fonts.
cc @RosaWagner

Few other possible related STAT checks:
(some may be already implemented)
<- weight axis ->

Regular is correctly linked to bold (when there are both styles)
Regular is elided
Values are correct according to axis registry

<- italic axis ->

There is an ital axis in both upright and italic fonts (when there is an italic font)
Upright : "Roman"
"Roman"=0
"Roman" is elided
"Roman" is linked to 1 ("Italic")
Italic: "Italic"
"Italic"=1

By default fontmake will export a STAT table but won't include any meaningful Axis Values

Is that a better default than not adding a STAT table at all?  I've wondered about this approach in the past too.

This is now urgent @felipesanches :)

@felipesanches Have you started working on this? If you haven't, please assign it to me.

Feel free to got for it! I have way too much on my plate now, so if you see issues in which I reasigned release milestone multiple times in succession, it simply means it is likely beyond my ability to address in time, so it is safe to request ownership of it for yourself if you want to work on them.

thanks, @miguelsousa!

I have a question about this check. In a project I'm working on, I have an fvar location at 5 that is being flagged as "missing Axis Value for 'opsz' value".
However, the STAT table has a rangeMinValue set at 0, and rangeMaxValue set at 16 (and a nominalValue at 11). Wouldn't that indicate that the 5 location is covered by the STAT table? As far as I can tell, the check that @miguelsousa implemented only covers the nominalValue (or value) entry and ignores the min/max range.

reopening this one to track progress. Thanks, @aaronbell & @miguelsousa


Pass down parameters

Parameters defined in a Path object should be passed down to child Paths, but cdk-rest-api-with-spec did not handle it well.

I found this is rather enhancement.

This is a bug because parameters are passed down if we omit defaultMethodOptions in child Paths.

The following lines in aws-cdk-lib overwrites requestParameterSchemas:
    this.defaultMethodOptions = {
      ...props.parent.defaultMethodOptions,
      ...props.defaultMethodOptions,
    };

https://github.com/aws/aws-cdk/blob/0a841611ed7969f0e37424c8d25f7a4d89f30fe0/packages/aws-cdk-lib/aws-apigateway/lib/resource.ts#L474-L477

I think we can merge requestParameterSchemas before passing it to the constructor of Resource, or synthesize defaultMethodOptions.


New features request

Please add in future releases:

Option to set up a hotkey for muting/unmuting program sound;
Parameter that allows the program to start minimized to the system tray at the OS start.

Check if the key is still down or if it has already been released to split the click sound and the key recoil sound.
(it would be necessary to update all the sound packs I think)

split the click sound and the key recoil sound

#146, #49, #21

Sound offset, for me I would like it to offset a bit as I hear a delay from my RL keystrokes to the mechvibes sound


Add a new option to Config to force a specific vcpkg root

Also make Library keep a list of libraries short names to be easily
accessible.

Looks great, thanks!


Making items (drawable) keep their actual sizes

The items (drawables) scale themselves automatically. How to maintain their actual sizes?

The issue is here where I set the bounds:
   Drawable drawable = cacheItem.mDrawable;
                if (drawable != null) {
                    drawable.setBounds(sTempRect);
                    drawable.draw(canvas);
                }


added a proper warning if one or all baseDirs do not exist

trident: gz: getDirectoryContents:openDirStream: does not exist (No such file or directory)

is now
[Info]    Checking base directories...
[Warning] Base directory (-d) "gz" does not exist

Codecov Report
Base: 64.35% // Head: 64.91% // Increases project coverage by +0.56% :tada:

Coverage data is based on head (cf27241) compared to base (3ee14e9).
Patch coverage: 61.53% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #216      +/-   ##
==========================================
+ Coverage   64.35%   64.91%   +0.56%
==========================================
  Files          18       18
  Lines        2233     2289      +56
  Branches      249      253       +4
==========================================
+ Hits         1437     1486      +49
- Misses        547      550       +3
- Partials      249      253       +4

Impacted Files
Coverage Δ

src/Poseidon/Utils.hs
21.34% <ø> (ø)

src/Poseidon/Package.hs
74.61% <54.54%> (-0.93%)
:arrow_down:

src/Poseidon/CLI/Validate.hs
54.54% <100.00%> (+2.16%)
:arrow_up:

src/Poseidon/CLI/Forge.hs
59.83% <0.00%> (+0.15%)
:arrow_up:

src/Poseidon/SecondaryTypes.hs
50.49% <0.00%> (+0.49%)
:arrow_up:

src/Poseidon/CLI/Survey.hs
90.21% <0.00%> (+1.08%)
:arrow_up:

src/Poseidon/EntitiesList.hs
88.80% <0.00%> (+1.45%)
:arrow_up:

src/Poseidon/CLI/Summarise.hs
67.74% <0.00%> (+1.61%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update to more current version of Natsort

comictagger uses an extremely old version of natsort, and if the user uses any other software that uses natsort it will cause conflicts. In addition, comictagger only seems to use it here, so it make sense to update to a more current version:
https://github.com/comictagger/comictagger/blob/93f5061c8f023ed838bc968f1c06f1ed4c3275fb/comicapi/comicarchive.py#L818
And to update it should only require modifying to something like this (depending if you wanted to drop the corresponding key function):
files = natsorted(files, alg=ns.IGNORECASE)

This has now been merged


Add more Settings:

auto-logout  options:   3 minutes, 5 minutes, 10 minutes, 20 minutes, 30 minutes, never (disabled)
Use MWC MQS  listener
Use  Keybase listener.

Also, for settings seems like we will need another page, what we have is crowded.
@condensed-io , I will add a button 'More' that will open next setting page.
What do you think about that? Do you have better alternative?
Note:  ticket can be started after Cold Wallet code will be in the master.  We don't want merge conflicts, especially at the resources level.

Solution with Tab and multiple pages is available at the branch   feature/expand-settings

@sarawut11
Could you please confirm, whether the above fix has been merged  to the Master branch?
If yes, I'll verify the same.
Thanks,
Vinayaga.

Yes, it was merged into the master. I did that. Nightly build should have it
On Mon, Apr 20, 2020 at 9:25 AM vinayaga07<EMAIL_ADDRESS>wrote:

@sarawut11 https://github.com/sarawut11
Could you please confirm, whether the above fix has been merged to the
Master branch?
If yes, I'll verify the same.
Thanks,
Vinayaga.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mwcproject/mwc-qt-wallet/issues/158#issuecomment-616663483,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKDOACCUCTGOEA36VZT7OVTRNRZQDANCNFSM4JXWHGRQ
.

Verified with the latest build and observed the following settings,

auto-logout options: 3 minutes, 5 minutes, 10 minutes, 20 minutes, 30 minutes, never (disabled)
mwc MQS listener
keybase

Hence closing this ticket.


fix: round microsecond when specify scale of microsecond

Proposed changes
select cast('0001-01-02 00:00:00.514514' as datetime(3));
should handle it to be: '0001-01-02 00:00:00.515000'

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall


Containers' architecture changed to 32bit somehow

I just updated my ubuntu15.10 server to "16.04.1" and I realized that 2 of the 3 containers' architecture changed to 32bit:
~ # uname -a
Linux atlassian 4.2.0-18-generic #22-Ubuntu SMP Fri Nov 6 18:25:50 UTC 2015 i686 i686 i686 GNU/Linux
~ # file /bin/ls
/bin/ls: ELF 64-bit LSB  executable, x86-64 ....[stripped]

~ # lxc list --fast
+------------+---------+--------------+------------+----------+------------+
|    NAME    |  STATE  | ARCHITECTURE | CREATED AT | PROFILES |    TYPE    |
+------------+---------+--------------+------------+----------+------------+
| atlassian  | RUNNING | i686         |            | default  | PERSISTENT |
+------------+---------+--------------+------------+----------+------------+
| huysuzadam | RUNNING | i686         |            | default  | PERSISTENT |
+------------+---------+--------------+------------+----------+------------+
| phpci      | RUNNING | x86_64       |            | default  | PERSISTENT |
+------------+---------+--------------+------------+----------+------------+

~ # sqlite3 /var/lib/lxd/lxd.db
sqlite> select * from containers;
id          name        architecture  type        ephemeral   creation_date  stateful
----------  ----------  ------------  ----------  ----------  -------------  ----------
1           atlassian   1             0           0           0              0
2           huysuzadam  1             0           0           0              0
3           phpci       2             0           0           0              0

I'm not sure if this is caused by the distro updater because it seems like the updated took a backup of the configuration file (/var/lib/lxd/lxd.db) and the arch parameter is also 32bit in the backup too. I'm pretty sure that the container was working before since I use it frequently... Maybe it was changed while doing apt-get upgrade or smt.
I still couldn't find a way to fix the containers, how do I change the architecture of an existing container to 64bit? Would it work if I update the lxd.db manually?

Updating lxd.db did it. It looks like old lxc was pretty old probably alpha or beta version. You may delete this issue if you like.


Contribution to Hugging Face 🤗 cookbook: Add two tutorials on Setfit and using HF Inference Endpoints

From Argilla, we would like to contribute with two new tutorials.

Fine-tune a SetFit model using the ArgillaTrainer: to learn how to use the ArgillaTrainer to fine-tune a SetFit model on a FeedbackDataset made of categorical questions like LabelQuestion and MultiLabelQuestion.
Create a legal preference dataset: to learn how to use the Notus model on Inference Endpoints to create a legal preference dataset based on RAG instructions from the European AI Act

Hi @sdiazlor ! Both suggestions sound good to me, I look forward to reviewing your notebooks! For the first one, please add some context on use cases, and reasoning for the choices made.


Code Snippet Error in Ep 213

in 44:01, 'letter' is defined in Destination enum.
but after that, this case is referred as 'text'.
and in  44:31 snippet has error, I think.
  for destination in self.path {
    switch destination {
    case let .counter(model):
      sum += model.count
    case .text:
      break // this line must be continue.
    }
  }

addition

in 48:42, delegate closures are implemented just empty closure. but in 52:16, it says we will get instant notification. Should it be 'unimplemted'?

addition2
in 52:53, Defined case is text. But in 54:36, the letter case is used.

Fixed, thanks! Will go out with the next deploy.


GCC Countries Summary not Updated

For the GCC countries of Bahrain, Kuwait, Saudi Arabia, UAE, and Oman, the data indicates data available to 2021-07-24, but the summary estimate shows a week behind of 2101-07-15..
Many thanks.

Thanks for flagging. Related #164.

This has been fixed.

Noted and thanks.
Sent from Yahoo Mail for iPhone
On Friday, July 30, 2021, 3:25 PM, Sebastian Funk @.***> wrote:
This has been fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


python27Packages.fire: 0.3.0 -> 0.3.1

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/python2.7-fire/versions.
meta.description for python27Packages.fire is: "A library for automatically generating command line interfaces"
meta.homepage for python27Packages.fire is: ""
Updates performed:

Version update

Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

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

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

found 0.3.1 with grep in /nix/store/ny1nppi35yly9c019kqfjvkznnq2c6kx-python2.7-fire-0.3.1

directory tree listing: https://gist.github.com/9c61af7c514ea938362c20c6f5c1f3a9

du listing: https://gist.github.com/60ab866c89cb863f0fa845092527064d

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

12 total rebuild path(s)
3 package rebuild(s)
3 x86_64-linux rebuild(s)
3 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
3 aarch64-linux rebuild(s)
First fifty rebuilds by attrpath
python27Packages.fire
python37Packages.fire
python38Packages.fire

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ny1nppi35yly9c019kqfjvkznnq2c6kx-python2.7-fire-0.3.1 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ny1nppi35yly9c019kqfjvkznnq2c6kx-python2.7-fire-0.3.1
ls -la /nix/store/ny1nppi35yly9c019kqfjvkznnq2c6kx-python2.7-fire-0.3.1/bin

cc @leenaars for testing.

Result of nixpkgs-review pr 84914 1

1 package failed to build:
- python38Packages.fire

2 packages built:
- python27Packages.fire
- python37Packages.fire


Fix --exclusions

Fixed --exclusions  in CLI
The string argument is split on comma then evaluted as RegExp

Published to NPM as 0.5.0


ASP.NET Core 2.2

Saw the blog post but wiring it all up in asp.net core 2.2 seems to result in UnAuthorized response.
Debugging seems to hit the TestAuthHandler etc but still get back 401.
Did something change in 2.2?
Thanks

You need to set the authenticationType on the ClaimsIdentity ctor


Store FoundationDB clusterfile content in Zookeeper

FDB clusterfile holds information needed to open connection to FDB database.
This branch adds functionality to easily set this information to zookeeper.
Probably this solution requires the smallest amount of code line changes and coding time.
I believe it's the only strong side of this PR.
It seems that AppController options is the only way we have for setting deployment properties. And I don't think it's perfect for cases like fdb-clusterfile-content.

http://ocd.appscale.net:8080/job/Daily Build/6574/

http://ocd.appscale.net:8080/job/Daily Build/6578/

http://ocd.appscale.net:8080/job/Daily Build/6610/


so many old methods error

using latest xcode8.2, swift3

Same issue here.
We may need to convert it from Swift2.3 to Swift3.
I found a test.swft in the project which indicates the author tried to convert it but not finished.

Try pod 'XLPagerTabStrip', '~> 6.0' instead of 5.0 (which is deprecated in the readme).


Don't add the original property to a mapping unless one exists (#519)

Fixed an error when generating source maps where original.line and/or original.column were not given.
Don't add the original property to a mapping unless one exists. This is needed for mappings to pass the _validateMapping check in https://github.com/mozilla/source-map/blob/master/lib/source-map-generator.js#L264
Fixes #516
[x] CHANGELOG.md has been updated

Thanks to @Lange for the fix.

Thanks, @aomarks


CLOUDSTACK-8628: kvm: Disable Fencing when no NFS storage pools are p…

…resent
On NFS we write a heartbeat, but without those we can not safely
fence off a host.
If we fence without knowing about a heartbeat we can cause a split-brain situation.
Signed-off-by: Wido den Hollander<EMAIL_ADDRESS>

Hi @wido
Could you please add an extra test to cover the "pools.size() == 0)"?
Cheers,
Wilder

@wido any news on this PR?

Working on a additional commit which brings in more logging and a unit test.

Travis failed due to NPE:
Tests in error:
KVMFencerTest.testWithFailingFence:135 » NullPointer
KVMFencerTest.testWithSingleHost:75 » NullPointer
KVMFencerTest.testWithSingleHostDown:88 » NullPointer
KVMFencerTest.testWithTimeoutingFence:158 » NullPointer

Indeed, I noticed that to late. Fixing the tests.

The tests have been fixed, Mocking issue in the test.

LGTM, how about the alert @bhaisaab asked for?

@DaanHoogland It is in the KVMFencer in the MGMT server. The Agent can't send an alert, so I had to do this in the mgmt server.

@wido thanks for the clarification. as said LGTM


I have added Compatibility to Extended Jamo A.

Hello. Now I have also added Compatibility to Hangeul Jamo extension block B.

Hello. Now I have also debugged many bugs in the program.

LGTM


Make org headings more varried in color

It feels weird to have fun, colorful code, then going into my org file and have it look almost monotone.

I have setup in my config and forgot to merge it lol.
I have now merged it into main so if you download the latest theme it should have it :D


resources to pull from

Pull in resources from:

Galaxy: https://toolshed.g2.bx.psu.edu/repository/browse_datatypes
IGV: https://www.broadinstitute.org/software/igv/FileFormats
UCSC: https://genome.ucsc.edu/FAQ/FAQformat.html

Files read/written by:

ReadSeq: http://www.ebi.ac.uk/Tools/sfc/readseq/
EMBOSS seqret http://www.ebi.ac.uk/Tools/sfc/emboss_seqret/


Add sort transform

This PR aims to add the sort transform mentioned in issue 74
This transform returns the table with rows sorted by values of a specific column in ascending or descending order,

@ClaroHenrique that is looking great already. Can you please add the transform to the docs page in the sample place after the Levels transform?

Codecov Report

Merging #87 (6d7ca2e) into master (639f212) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #87      +/-   ##
==========================================
+ Coverage   93.09%   93.25%   +0.15%
==========================================
  Files          23       24       +1
  Lines         565      578      +13
==========================================
+ Hits          526      539      +13
  Misses         39       39

Impacted Files
Coverage Δ

src/transforms.jl
97.82% <ø> (ø)

src/transforms/sort.jl
100.00% <100.00%> (ø)

src/transforms/quantile.jl
93.33% <0.00%> (-1.12%)
:arrow_down:

src/transforms/scale.jl
86.66% <0.00%> (-0.84%)
:arrow_down:

src/distributions.jl
90.90% <0.00%> (-0.40%)
:arrow_down:

src/transforms/eigenanalysis.jl
79.66% <0.00%> (+0.71%)
:arrow_up:

Continue to review full report at Codecov.

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

@ClaroHenrique I was thinking if we could generalize this PR a bit and instead of just providing the rev keyword argument, actually forward all keyword arguments to the internal sortperm call?
sortperm(v; alg::Algorithm=DEFAULT_UNSTABLE, lt=isless, by=identity, rev::Bool=false, order::Ordering=Forward)

That would be a much richer set of options for end users. Please let me know if you have questions about how to forward keywords generically. You can simply type foo(; kwargs...) = bar(; kwargs) if I am not mistaken.

Another possible feature that we can consider here is not only sort by a single column, but by a list of columns. In that case whenever a user specifies a ColSpec, we retrieve all columns and treat them as a collection of named tuples to be sorted lexicographically. For example, Sort([:a,:b])would sort a vector of the form [(a=1,b=2), (a=2,b=1), ...].

@ClaroHenrique I was thinking if we could generalize this PR a bit and instead of just providing the rev keyword argument, actually forward all keyword arguments to the internal sortperm call?
sortperm(v; alg::Algorithm=DEFAULT_UNSTABLE, lt=isless, by=identity, rev::Bool=false, order::Ordering=Forward)

That would be a much richer set of options for end users. Please let me know if you have questions about how to forward keywords generically. You can simply type foo(; kwargs...) = bar(; kwargs) if I am not mistaken.

Like this:
struct Sort{T} <: Stateless
  col::Symbol
  kwargs::T
end

Sort(col::Symbol; kwargs...) = Sort(col, value(kwargs))

Ok @eliascarv, thanks.
Are going to work on the multi column sorting also?


Does this package support ES256?

I work at a big company with an OIDC IDP that only supports the ES256 ID token signing algorithm, so no RS256. I found that in the docs  it says that the server "should" support RS256. What is the consequence for me? Can I still use this library?

The docs seems to be outdated. "Servers SHOULD support RS256" should be removed. Since this library removed the implicit flow and only supports code/PKCE, the tokens are only decoded and no longer checked against the signature. I therefor to not see why you can not use it. So give it a try. Which IdP do you intent to use?

"Servers SHOULD support RS256" should be removed

This text is the official one from https://openid.net/specs/openid-connect-discovery-1_0.html#ProviderMetadata


[BUG]:corrupt PDFs from agent

How are you running AnythingLLM?
All versions
What happened?
If I request that the agent create a PDF (locally or hosted) the PDF when downloaded will not open ...  Adobe reader reports it as damaged or corrupt.   Tried on multiple systems.   I don't get any errors.
Are there known steps to reproduce?
Happens with every creation and export

Further info...What appears to be happening is that the chat text is exported as a PDF file but it is just a plain text file.   Export as txt instruction works as it sends the same data but gives it a .txt filename vs. pdf.   I used the syntax from the DOCs to ensure I was instructing it correctly    @agent can save this information as a PDF on my desktop folder?

If I request that the agent create a PDF (locally or hosted) the PDF when downloaded will not open ... Adobe reader reports it as damaged or corrupt. Tried on multiple systems. I don't get any errors.

This is not a skill that we even offer. The only agent skill built-in we offer is text files. You cannot just change the extensions of a text file into .pdf and have that work - they are two totally different file types.
If you specifically require PDF generation you will need to make a skill for that with custom agent skills. The LLM will simply hallucinate the PDF generation using the text doc skill and tell you it works - which is what is occurring here.
https://docs.anythingllm.com/agent/custom/introduction


MAINT: Set version for patch release

It has been almost a year since our last release. This is just a bugfix release so I think 0.9.1 is appropriate, but we could do 0.10.0 instead if people want.

We don't have a release branch here, so I'm not sure why main is set to 0.10.dev0..

Done https://pypi.org/project/pyvistaqt/0.9.1/


add error check on default check for RKE templates

Proposed changes
There is an old back end bug where the revisions are not getting cleaned up correctly and if we hit this case we can't load the UI because the revision id doesn't check first if it exists. With this check we should be safe from hitting this case even though we should really never hit it when the back end issue is resolved.

Types of changes

Linked Issues

Further comments

This PR is for https://github.com/rancher/rancher/issues/24339


Tim Short

'Im interested in contributing to your GitHub repository and would like to add Tim short with proper comments in the java short folder  to enhance its functionality. Please let me know if you would be open to accepting this addition. I believe it will benefit the community. Thank you for your consideration.
@Kumar-laxmi please assign this issue to me.

Do you mean Tim Sort?

yes sir sorry for the typo


MongoServer lifecycle

This is half question, half bug report.
I believe there might be a significant bug in MongoRepository's management of the lifecycle of MongoServer. I believe MongoServer should be singleton/static, and the same instance should be used every time. Otherwise there's some overhead in connecting to the database with each instantiation of the repository.
Here in Util.cs:
private static MongoDatabase GetDatabaseFromUrl(MongoUrl url)
{
var client = new MongoClient(url);
var server = client.GetServer(); // <---- should not be returning a new MongoServer instance!
return server.GetDatabase(url.DatabaseName); // WriteConcern defaulted to Acknowledged
}
See also http://stackoverflow.com/questions/10241641/what-is-the-right-way-to-manage-mongodb-connections-in-asp-net-mvc
I picked this up when looking at performance of queries - was seeing 200ms+ overhead with each new MongoRepository.
Apologies if I have misunderstood the code.

See MongoClient Connection Pooling. What goes there essentially also goes for MongoRepository since MongoRepository is merely a layer on top of the C# Driver.

The driver will, for the most part, use the same connection settings. [...] Hence, the best practice will be to only instantiate 1 MongoClient at app startup and use that. Put it in an IoC container, create a static variable somewhere, etc... Essentially, make it a singleton. Is there any reason you are constructing a new client for every request?

I'm just instantiating MongoRepository, with a connection string argument. MongoRepository is calling Util's GetDatabaseFromUrl() on my behalf. Callstack is:
MongoRepository(connString)
Util.GetCollectionFromConnectionString(connectionString)
Util.GetCollectionFromConnectionString(connectionString, GetCollectionName())
Util.GetDatabaseFromUrl(new MongoUrl(connectionString))
.GetCollection(collectionName)
from where it does this:
var client = new MongoClient(url);
var server = client.GetServer();
return server.GetDatabase(url.DatabaseName); // WriteConcern defaulted to Acknowledged

I understand; but why not keep the repository alive (e.g. keep a reference around instead of instantiating a new one every time)?

Thanks Rob. I did structure things that way for a while. I would prefer to see the dependency on a long lived MongoServer/MongoDatabase object, or a long lived MongoRepository instance made very explicit. The examples need to show that pattern of use - bear in mind many people don't use DI containers, and even if they do, they may inject a MongoRepository instance per request without thinking about it.
I managed to figure this out myself, but I know there's a lot of Mongo / .NET noobs who will fall into the trap of simply new'ing up a MongoRepository every time they need it. For your typical business application they may never notice the latency they have incurred.

I am having trouble reproducing your findings; I have created a very simple test-project that

Creates a new repository
Adds a document/entity

In a loop:
using MongoRepository;
using System;
using System.Collections.Generic;
using System.Diagnostics;
using System.Linq;

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            var results = new List<TimeSpan>();

            for (int i = 0; i < 1000; i++)
            {
                var s = Stopwatch.StartNew();
                var repo = new MongoRepository<Customer>();
                repo.Add(new Customer { Name = Guid.NewGuid().ToString("N") });
                results.Add(s.Elapsed);
            }

            Trace.WriteLine(string.Format("Average: {0}", TimeSpan.FromTicks((long)results.Average(r => r.Ticks))));
            Trace.WriteLine(string.Format("Total  : {0}", TimeSpan.FromTicks((long)results.Sum(r => r.Ticks))));
        }
    }

    class Customer : Entity
    {
        public string Name { get; set; }
    }
}

My output is:
Average: 00:00:00.0004114
Total  : 00:00:00.4114397

There's much to say about this test-case (as it doesn't take "warmup" into account, has a abysmal way of measuring etc. etc.) but as you can see my results are very different than from what you are seeing.
Are you sure you're not doing very expensive/slow DNS lookups? Or connecting to a high-latency host? Maybe something other network related?


Incorrect Number Shown - Max Daily Measurement vs Avg Rainfall Chart

Reporting on behalf of @joaoporto
Issue: The max. daily measurement is 65mm but in the chart we see measures of up to 130mm
See mismatch between statistics here

@diegopajarito - I remember that the calculations for the chart became quite complex in the end. Are you able to confirm what the correct statistics are here so that I have a 'ground truth' before attempting a fix?
I need rainfall measurements for Santa Catarina - from June to September 2023. This will help me to understand which of the statistics is incorrect.


port_mapping.name, port_definition.[name|lables], health_check.port a…

…nd volume.external are added

I try to resolve this test case, but the test is posting a json definition with container.volumes.volume.external = {} when there is no external in the definition that causes marathon api to produce error. The code is running in our environment.

The test that @cihangirbesiktas mentioned is now fixed but we have another problem:
* marathon_app.app-create-example: Marathon API error: Object is not valid (path: '/upgradeStrategy/maximumOverCapacity' errors: got 0.3, expected 0.0; path: '/container/volumes(1)' errors: Feature external_volumes is not enabled. Enable with --enable_features external_volumes); path: '/container/portMappings(1)/name' errors: must fully match regular expression '^[a-z0-9-]+$')

The instance of marathon that we are using doesn't have --enable_features external_volumes @nicgrayson any ideas on how to fix this?

Thanks @calvernaz

The failing test has wrong expectations but I'm not sure where to fix it. The expected value is wrong.

--- FAIL: TestAccMarathonApp_basic (5.76s)
testing.go:268: Step 0 error: Error applying: 1 error(s) occurred:
	* marathon_app.app-create-example: 1 error(s) occurred:

	* marathon_app.app-create-example: Marathon API error: Object is not valid (path: '/upgradeStrategy/maximumOverCapacity' errors: got 0.3, expected 0.0)

I did not change upgradeStrategy but getting this error, it seems the test env is not set up properly, I could not find out the problem but it would be better if it is fixed.

@cihangirbesiktas I believe this happens because you've external volume attached. In such case, upgrade strategy has to be set to 0, because Marathon cannot start an additional task (since volume can be mounted to only one task).

@kamsz but when I set the maximum over capacity to 0.0, I got the following error:
--- FAIL: TestAccMarathonApp_basic (5.05s)
testing.go:268: Step 0 error: Error applying: 1 error(s) occurred:
            * marathon_app.app-create-example: 1 error(s) occurred:

            * marathon_app.app-create-example: Marathon API error: Object is not valid (path: '/upgradeStrategy/maximumOverCapacity' errors: got 1.0, expected 0.0)

maybe @nicgrayson could give us a hand here?

I found out that schema.ResourceData.GetOk function returns false when the value of a key is zero, but we need the zero value in /upgradeStrategy/maximumOverCapacity. There is another function schema.ResourceData.GetOkExists which returns true regardless of zero value, but it is in the latest repo of github.com/hashicorp/terraform/helper/schema. Could @nicgrayson update the repos?


Create an interactive Dice Roller app: Android Basics with Compose

URL of codelab https://developer.android.com/codelabs/basic-android-kotlin-compose-build-a-dice-roller-app#3
In which task and step of the codelab can this issue be found? Add a Modifier
Describe the problem The @Preview is shown above the fun DiceWithButtonAndImage() and has a parameter. In the solution file, it is positioned above fun DiceRollApp(). Confusing.)
Steps to reproduce?

Go to...
Click on...
See error...

Versions
Android Studio version:
API version of the emulator:
Additional information
Include screenshots if they would be useful in clarifying the problem.

Hello @MikeDeignan,
Thank you for reaching out to us and bringing this inconsistency to our attention. The code snippet is fixed:


How to make the progress bar animate like a pie chart instead of it filling up the border?

Is there any way that I can make the progress bar looks like this?

yup, I think you can achieve it with the combination of this technique, which allows you to overlay text on top of the progressbar, and modifying strokeWidth and styles to make it look like a pie chart.
I don't have time at the moment, but I can create a codesandbox demo later. In the meantime, you can see if you can combine those to get what you want.

Here's a demo showing how it's done: https://codesandbox.io/s/qxz12ym92w

Going to close this, but feel free to ask if you've got followup questions!


Inverted colors when rendering RGB Raster

Example of RGB raster rendering in COG Explorer:

Example of rendering with default parameters via TIFFImageryProvider:

P.S. Setting Chrome to 3.0 makes the look better, but still far from true colors.

Thank you for your issue, I'll fix it in this week.

Fixed yet in v2.2.0, now add multi band tiff will first try rgb mode.

There are still black areas present currently. If it's necessary to remove these black areas, I will add an optional exclusion parameter.

Thank you!
You are really doing a great job that is very valuable for the community. As Cesium is moving very slow in everything excepting 3D Tiles you are the hero we need!

Thank you for your kind words. I have only done some insignificant work with the strong support of Cesium, and I hope that it can be helpful to everyone.


Broke. Link

[Enter feedback here]
Broken link
https://github.com/Azure/cognitive-search-vector

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

ID: baadfe89-fbfa-58bc-dcb0-99c2b3bfc60b
Version Independent ID: 1b2548d0-56fc-333d-dac0-c35e3e99cb5a
Content: Quickstart integrated vectorization - Azure AI Search
Content Source: articles/search/search-get-started-portal-import-vectors.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@vikramhn, thanks for reporting it!  I'm updating the docs now, but to unblock you, here's the link to the repo:  https://github.com/Azure/cognitive-search-vector-pr/

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

@vikramhn This link is fixed now, sorry for the bad link.
#please-close


run example

I try to run the demo HoodieClientExample,I find the "hoodie.keep.min.commits" > "hoodie.cleaner.commits.retained",so the program runs fail,I don't know whether the logic is correct
// Create the write client to write some records in
HoodieWriteConfig cfg = HoodieWriteConfig.newBuilder().withPath(tablePath)
.withSchema(HoodieTestDataGenerator.TRIP_EXAMPLE_SCHEMA).withParallelism(2, 2).forTable(tableName)
.withIndexConfig(HoodieIndexConfig.newBuilder().withIndexType(IndexType.BLOOM).build())
.withCompactionConfig(HoodieCompactionConfig.newBuilder().archiveCommitsWith(2, 3).build()).build();
Exception in thread "main" java.lang.IllegalArgumentException: Increase hoodie.keep.min.commits=2 to be greater than hoodie.cleaner.commits.retained=10. Otherwise, there is risk of incremental pull missing data from few instants.
at com.google.common.base.Preconditions.checkArgument(Preconditions.java:93)
at org.apache.hudi.config.HoodieCompactionConfig$Builder.build(HoodieCompactionConfig.java:296)
at HoodieClientExample.run(HoodieClientExample.java:98)
at HoodieClientExample.main(HoodieClientExample.java:72)
65941 [Thread-1] INFO  org.apache.spark.SparkContext  - Invoking stop() from shutdown hook

@c-f-cooper : Which version of hudi are you using ? This has been fixed in master ( https://github.com/apache/incubator-hudi/blob/master/hudi-client/src/test/java/HoodieClientExample.java#L98)

0.5

could you try master or 0.5.2?

@c-f-cooper : You can use any of 0.5.1/0.5.2 or master to see the fix.


Bug: Nilable type leaks out of closure scope

foo = "foo"
-> { foo.not_nil!.upcase }.call
typeof(foo) # => String | Nil
foo = nil

The type of foo should only be String at the position of typeof. It can't be Nil here because the nil assignment is only the next expression.
Inside the closure it is obviously String | Nil because when the proc is executed, it might be either in the main scope.
Possibly related to #6040, #6037

Not a bug. When a variable is closured, the compiler can't track when the closure will be called, so immediately after that happens, all references to the variable will have all types assigned to them. You can't even restrict them with is_a?, because in between the closure or outside code might be called.
Well, maybe in this case the compiler could figure it out, because the variable is clearly not reassigned in the closure. However, the algorithm is already quite complex, and I have no idea how to improve this (and I doubt anyone else will touch the compiler's code). So we can close this as wontfix (or "needs superhero").


IOS Build Failed (Xcode >= 11.3)

Hi, I have a problem with a IOS when I try to build my application after updating the Xcode to the version 11.3 and now 11.4. It's related to DKImagePickerController, bunch of Swift compiler errors.
For example:
'pngData()' has been renamed to 'UIImagePNGRepresentation(_:)'
...
'jpegData(compressionQuality:)' has been renamed to 'UIImageJPEGRepresentation(_:_:)'
...
etc

Hi @Stanteq Please check the Swift Language Version of DKImagePickerController and make sure it's Swift 5:


Flutter not working after switching to Flutter channel master

Steps to Reproduce

... I was using Flutter v1.19 in the beta channel before, as I tried to use to Layout Explorer in the Flutter DevTools. It required me to switch to Flutter channel master.
... I gave a command to switch to flutter channel master, and the installation went fine, I even checked if my devices were connected, and they were!
... When I try to run my App, nothing happened.

Logs

"flutter devices"
bash: flutter: command not found
"flutter --version"
bash: flutter: command not found

Hi @chankalun
Can you please run from your terminal
flutter doctor -v

and provide the full logs.
Thank you

Hi, I have sorted this with setting my path in zsh :)


[dataset] Persuasion for Good

Dataset name: Persuasion for Good
Description: A large metadata-rich collection of online conversations between two people where one participant ("persuader") is trying to convince the other ("persuadee") to donate to a charity. This dataset contains 1017 dialogues, each with several conversational turns. Additionally, every user has a personality vector associated with them generated through psycological surveys.
Distributed together with: "Persuasion for Good: Towards a Personalized Persuasive Dialogue System for Social Good" https://arxiv.org/pdf/1906.06725.pdf. Xuewei Wang et al. ACL 2019.
Dataset details:
Original data: https://gitlab.com/ucdavisnlp/persuasionforgood/tree/master/data

User-level information: Users in this dataset are Amazon Turks. For each user, the authors provide a 23-dimension characteristic vector based on a pre-task psychological survey. Each of these personality characteristics takes on a value from 1 to 6; in addition, demographic info such as age, sex, race, education, marital status, employment, income, religion, and ideology are also included as user-level metadata.

Utterance-level information:
For each utterance, we provide:
id: index of the utterance
user: the user who authored the utterance
root: index of the conversation root of the utterance
reply_to: index of the utterance to which this utterance replies to (None if the utterance is the first utterance in a conversation.)
text: textual content of the utterance
er_label_1: first persuasion strategy used by the persuader
ee_label_1: first persuasion strategy used by the persuadee
er_label_2: second persuasion strategy used by the persuader
ee_label_2: second persuasion strategy used by the persuadee
neg : The negative sentiment of the utterance
neu : The neutral sentiment of the utterance
pos : The positive sentiment of the utterance
Metadata for utterances include:
dialogue_idx: index of the dialogue from which this utterance occurs
role: whether the utterance is spoken by the persuader ("0") or persuadee ("1")
num_turns: The particular turn in which this utterance occurs in the dialogue

Conversational-level information:
Conversations are indexed by the id of the dialogue. Conversation metadata now includes donation amount, num of turns, intended donation amount (only available for 300 annotated conversations), and the persuader/persuadee user ids.

Licensing information: The data for this project was obtained from Xuewei Wang et al. ACL 2019https://gitlab.com/ucdavisnlp/persuasionforgood.
Related Publication:
https://arxiv.org/abs/1906.06725
Contact Information:
Please email any questions to<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>(Frank Li, Grace Deng, Di Ni).
Link:
p4g_corpus.zip

Hi,
thanks for your work on this! More comments coming, but for now, could you point me to a repository you worked with in converting this dataset, and/or the code you used to perform the conversion?

Hi Justine,
Our repository is here :
https://github.com/junfrankli/Cornell-Conversational-Analysis-Toolkit/tree/master/datasets/persuasion_corpus
and  dataconversion file is:
https://github.com/junfrankli/Cornell-Conversational-Analysis-Toolkit/blob/master/datasets/persuasion_corpus/DataConversion.ipynb
On Wed, Oct 16, 2019 at 10:31 AM Justine Zhang<EMAIL_ADDRESS>wrote:

Hi,
thanks for your work on this! More comments coming, but for now, could you
point me to a repository you worked with in converting this dataset, and/or
the code you used to perform the conversion?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CornellNLP/Cornell-Conversational-Analysis-Toolkit/issues/31?email_source=notifications&email_token=AGVCBBSHYDG26Y7GKVAFF6LQO4QTTA5CNFSM4I73QUU2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBMWSOQ#issuecomment-542730554,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGVCBBQCWWJFK43THHOE7MDQO4QTTANCNFSM4I73QUUQ
.

Okay, got it! I see that the user-level metadata description is more expanded -- thanks! My other comments are still standing, I think -- let me know what you think.

A few more things:

It would be great if you could clean up the data conversion file to only encapsulate the data conversion steps you've taken (and as a quick pointer, calling pd.read_csv to read in all the csv files should fix some of the datatype problems, regarding ints being stored as strings)
Try calling convo.add_meta to add conversation-level meta instead of assigning to conversation.meta -- this might help fix the missing conversation-level metadata.
I also noticed that in the paper, both the persuader and persuadee can offer donations, but it seems that you only have donation values for one person.

Hi Team,
Since we did not hear from you and there seems to have been several major issues with your corpus and documentation,* Justine went ahead and addressed them herself.  If you are curious, you can go ahead and test the corpus on the convokit main website and let us know if you see any issues.  I am going to close this task now.
*(some of the issues that Justine mentioned were brought up before in our discussions)

@all-contributors please add @dn273 for data

@all-contributors please add @gdeng96 for data

@all-contributors please add @junfrankli for data


feat: http client tracing

Add an option to trace Http call with resilient http client.
This originates from an (Work in progress) MR for kratos: https://github.com/ory/kratos/pull/2154
Checklist

[x] I have read the contributing guidelines
and signed the CLA.
[ ] I have referenced an issue containing the design document if my change
introduces a new feature.
[ ] I have read the security policy.
[x] I confirm that this pull request does not address a security
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got green light (please contact
<EMAIL_ADDRESS>from the maintainers to push
the changes.
[x] I have added tests that prove my fix is effective or that my feature
works.
[ ] I have added necessary documentation within the code base (if
appropriate).

Further comments

Oh. Wait I found a nil access. Will  fix.


Heroic and GRIP: Combat Racing: game not found due to name mismatch

Ludusavi version
v0.13.1
Operating system
Linux (Steam Deck)
Installation method
Standalone
Description
Related to #94

The game is listed as "GRIP: Combat Racing" in the ludusavi manifest.yaml.
heroic .config/heroic/gog_store/library.json names it as "title": "GRIP"
during heroic::detect_gog_games it is properly found and memorized as "GRIP"
ludusavi backup does not recognize it since it compares the name from the manifest with the name from the library.json which do not match

I expect quite some games to have this issue, for example "X-Morph: Defense Complete Edition" (gog_store/library.json) versus "X-Morph: Defense" (manifest.yaml) seems problematic as well.
Logs
No response

Another option might be to get data from GOG DB.  They provide access to their complete data (moreinfo) here: https://www.gogdb.org/backups_v3/
With some work one can build aliases in the manifest from that:

for GRIP their product.json includes the game id (aka appName in HEROIC_CONFIG/gog_store/library.json):<PHONE_NUMBER>
this id can be used in https://pcgamingwiki.com/api/gog.php?page=1342466874 to get to the PCGaming Wiki page
and from there one can retrieve the proper game title also found in the  ludusavi manifest.

X-Morph is detected by the heroic code as "X-Morph: Defense Complete Edition" but normalize reduces it to "x morph".  Thus no backup.

I'm open to the GOG DB option or even a combination of both. Just need to play around with it and see how it goes.
Since GOG DB names might conflict with PCGW article titles, I would probably store it like this and just build a mapping of alias -> official on the client side:
GRIP: Combat Racing:
  aliases:
    - Grip

Yes, thats what I had in mind as well.  In heroic::memorize_game a check could work like this:

title matches?
alias matches?
normalized matches?
Log an error

In fact it can be queried from the pcg wiki cargo API:
{{#cargo_query:
tables=Infobox_game
|fields=Infobox_game.GOGcom_ID, Infobox_game._pageName
|where=GOGcom_ID holds "1342466874"
}}

Result:
[
    {
        "GOGcom ID": [
<PHONE_NUMBER>,
<PHONE_NUMBER>
        ],
        "_pageName": "GRIP: Combat Racing"
    }
]

Or to put it as HTML request: https://www.pcgamingwiki.com/w/api.php?action=cargoquery&tables=Infobox_game&fields=Infobox_game._pageName=Page,Infobox_game.GOGcom_ID&where=Infobox_game.GOGcom_ID%20holds%20%221342466874%22&format=json
results in
{"cargoquery":[{"title":{"Page":"GRIP: Combat Racing","GOGcom ID":"1342466874,1926778718"}}]}

1926778718 is the GOG product ID for the GRIP soundtrack

Completed in #152.

PS: detects everything on the deck now, too!  Including X-Morph: Defense.


30-50% performance degradation after #126

Before https://github.com/intridea/grape-entity/pull/126:
JRuby 1.7.19
 jruby -G -S bench/serializing.rb
                   user     system      total        real
serializing    1.450000   0.020000   1.470000 (  1.254000)

2.2.2
                   user     system      total        real
serializing    1.750000   0.010000   1.760000 (  1.771028)

Now:
JRuby 1.7.19
                  user     system      total        real
serializing   2.270000   0.020000   2.290000 (  1.818000)

2.2.2
                  user     system      total        real
serializing   2.440000   0.020000   2.460000 (  2.500005)

I've got even worse results in some other scenarios.
And I think code becomes much more complicated after it.

Before we consider reverting anything, lets ping @u2.

@etehtsea Do you have a serializing.rb that someone can run with?

@dblock https://github.com/intridea/grape-entity/blob/master/bench/serializing.rb :)

I will dive in it later.Do you have some suggestion about #126 @etehtsea .And is there some good gem for performance analyzing.

I don't feel that #56 is a serious issue to fix. At least not for a such price. Especially if suggested workaround works https://github.com/intridea/grape-entity/issues/52#issuecomment-49027130
About profiling tools:

mri - https://github.com/tmm1/stackprof
jruby - https://github.com/jruby/jruby/wiki/Profiling-jruby or jvisualvm/jprofiler

Yes, I agree with you.So reverting it?
@dblock

Go ahead and PR reverting that, but I encourage you to spend a bit of time in a profiler because often it's something small and fixable.

OK.Anyway I will do a profiler.

Reverted via https://github.com/intridea/grape-entity/pull/130.


It looks like the shape VM.Standard.E2.1.Micro isn't available in your region.

It looks like the shape VM.Standard.E2.1.Micro isn't available in your region.
You can check it by:
Go to Home > Governance > Limits, Quotas and Usage to see where your VM.Standard.E2.1.Micro shapes are available.

_Originally posted by @r0b2g1t in https://github.com/r0b2g1t/k3s-cluster-on-oracle-cloud-infrastructure/issues/2#issuecomment-1083209846_

For others, its available, but not in the availability domain set by the script.
Changing the AD, allowed the instance to be created.

yes, you have to check which AD provides the shape

Don't like open issues then :)
Wondered if it worth noting that for other people perhaps, so they don't waste time like I have.


Weather Forecast

Project title
(Write project title over here)
Project Description
(Describe the project over here)
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[x] Html
[x] CSS
[x] JavaScript
[] ReactJS

Fill the project title and description


Error deploying Azure Kubernetes Service template

Error while creating build definition: The pipeline is not valid. A task is missing. The pipeline references a task called 'a8515ec8-7254-4ffd-912c-86772e2b5962'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version , job 'Phase_1', step ''.) A task is missing. The pipeline references a task called 'a8515ec8-7254-4ffd-912c-86772e2b5962'. This usually indicates the task isn't installed, and you may be able to install it from the Marketplace: https://marketplace.visualstudio.com. (Task version 3.*, job 'Phase_1', step ''.)

This looks like an issue with Kubernetes extension, which needs to be installed and enabled before provisioning the project.
Please try to install the extension manually from the marketplace and provision the project.
Thanks!


Robotjs error with pkg

Expected Behavior

running pkg . should create an. exe of node/express application.
Current Behavior

Pkg . creates faulty .exe, debug shows the following error with robotjs
[debug] prebuild-install
failed[C:\Users\Documents\node_modules\robotjs\build\Release\robotjs.node
]:
Error: spawnSync
C:\npm\node_modules\pkg\node_modules.bin\prebuild-install
ENOENT
Possible Solution

Steps to Reproduce (for bugs)

1.create a nodejs/express server app with an end point.
2.add robot js typeString("helloworld") and keyTap(Enter)
3.setup assets for pkg in package.json as error this stackoverflow question.
4.run pkg . --debug to see the build, create .exe and run in terminal to see call error on robotjs
Context
I want to create an. exe file of the project I have built.

I have created an express server that I can run pkg . on and create a stable .exe for my application. As soon as I introduce my call to robot js, I can run my application via the terminal using 'npm start' script with the exact behaviour that I expect from robotjs, but if I run pkg . with the calls to robotjs included the .exe fails to run with the errors shown above and in the stackoverflow question link.
Your Environment

RobotJS version:0.6.0
Node.js version:14
npm version:8.1.0
Operating System: windows 10

+1

+1

I solved in this way:
Require this after requiring robotjs
require("./node_modules/robotjs/build/Release/robotjs.node")
I hope I have been helpful

@VitoFarella thank you so much! This worked perfectly :-)


Plugin not apply  indent_style, indent_size to *.java files

I need to format the *.java code with indent_style, indent_size configured in the .editorconfig but I don't see any changes when executing the phase mvn editorconfig:format.
My plugin in the pom.xml is this
			<plugin>
				<groupId>org.ec4j.maven</groupId>
				<artifactId>editorconfig-maven-plugin</artifactId>
				<version>0.0.10</version>
				<executions>
					<execution>
						<id>check</id>
						<phase>verify</phase>
						<goals>
							<goal>check</goal>
						</goals>
					</execution>
				</executions>
				<configuration>
					<includes>
						<include>**/*.java</include>
					</includes>
				</configuration>
			</plugin>

Java files won't get indented, because there is currently no Linter for those - see https://github.com/ec4j/editorconfig-maven-plugin#how-it-works
Indenting structured languages is not possible without parsing the files. I work on a Java Linter based on Eclipse JDT formatter in https://github.com/ec4j/editorconfig-java-domain and it will take a while till it is ready for the end users.
Closing because the reported behavior is expected at this time.


How to find all channels bound to a specific eventloop？

Netty does not provide you with such information out of the box, but you can easily keep that registry by yourself if you add a ChannelHandler that implements channelRegistered() and channelUnregistered().

get it! Thanks


feature: Initialize kn admin plugin

Hi All,
This kn-admin plugin is designed to help administrators or operators better manage a Knative platform installation with kn CLI.
This is an initial commit. More details in the proposal: https://docs.google.com/document/d/1oD5P4WQZs39GIr_4ezcURgRCuUx8dXrcCi3N7GKmp5c/edit#
Thanks!

@rhuss
Don't worry, take your time:-)

@zhanggbj sorry, I couldn't make it before my PTO (and I'm heading now off to the Alpes with little connectivity). I will be back the first week in March and get your PRs merged then is on the top of my priority list.
Apologies, and have a nice week !

/ok-to-test

Hi, @zhanggbj, so all the tests are failing :(
I’ll leave some comments as well. But please fix this ^^^ first. Thanks.

@maximilien Thank you for you review!
I've address your most of your comments and left one comment (add version in next PR), would you please help to take a review, thx!
I check the test failure, looks like it doesn't relate do plugin code, it cannot find the proper test file to execute. For e.g.
+ ./test/presubmit-tests.sh --integration-tests
./test/presubmit-tests.sh: line 37: ./test/../vendor/knative.dev/test-infra/scripts/presubmit-tests.sh: No such file or directory
./test/presubmit-tests.sh: line 40: main: command not found
+ EXIT_VALUE=127
+ set +o xtrace

Hi, @zhanggbj I think you need to add the test files in your PR. Doing that too for my PR #4.
I guess which ever gets merged first will have these files set and we can resolve conflicts in subsequent PRs.
Please do that and get all green and I will do a final pass. Thanks.

/retest

The following users are mentioned in OWNERS file(s) but are not members of the knative org.
Once all users have been added as members of the org, you can trigger verification by writing /verify-owners in a comment.

zhanggbj

OWNERS

@zhanggbj I asked the Knative admins to add you to the GitHub orga

Hi @rhuss @maximilien,
I've restructured kn-admin to align with the hello plugin, including folders/vendors/Reame/LICENSE/etc. would you please help to take a review? Thanks!
This is an initial PR for kn-admin and I still have a lot of TODOs in my list. I'll raise new PRs for review later as this already got a little bit hard to review:-)

Fill up unit test and e2e test for kn-admin
New features (detailes in the docs folder)

@zhanggbj btw, I would be happy to already commit the PR and then we can continue to work on it later.

@rhuss
I've addressed all your comments and would you please help to take a look? Thanks!
And this is an initial PR, i think there're still a lot of todos for new subcommand and for existed domain and registry CRUD actions, and we can discuss later after this PR is merged and easy for collaboration.


Add Commit message to PR

Add commit message variable to pull template for PR in merge

Good catch. I have tested it, LGTM.
And it has been confirm by  parnic (https://github.com/go-gitea/gitea/issues/20634#issuecomment-1202877581)

This change looks good, thanks!
Once this is fixed, the approvers still don't get listed like they used to, so that will need a separate change. The template looks like it's trying to generate that list, but it's not coming through for some reason.

Sorry for bothering. After some more thoughts, I think as a bug fix, it's better to keep the same behavior as 1.16.
In 1.16, only the squash merge has the commit messages.
https://github.com/go-gitea/gitea/blob/release/v1.16/templates/repo/issue/view_content/pull.tmpl#L407
So I made a new change to this PR: 'mergeMessageFieldText': {{.GetCommitMessages}} + defaultMergeMessage, for 'name': 'squash',.

bot


Improvements to screen reader report from graph

Some comments, to discuss improvements to default behaviour, or surfacing multiple options
Currently the live-region reports "Estimated action: "{action}""
Questions/suggestions:

Is this repeated based on a timer, or just repeats once it's finished?
Consider if it should report based on transitions, so it's more time sensitive? Obviously with some kind of debounce/hysteresis to avoid too-rapid change.
Consider whether the report "certainty for {action}" should also give continuous updates on a single class? I can see situations where this would be helpful.
How is it expected to navigate to the graph for updates? In my experience playing with NVDA:
(a) the updates happened automatically when I started testing the model
(b) I then lost focus and struggled to get back there
(c) I now (with visual help) can get there via arrow key navigation in NVDA
Have I missed an expected way to get there?
Consider adding a region to allow quick navigation

Add:

Consider removing "Estimated action" after first announcement for less verbosity / less latency on new information, e.g. "estimated action wave... shake ... shake ... nothing ... shake"

The estimated action and certainty values update every 250ms. The estimated action live-region is announced when it changes from one action to another, e.g., still to shake. What actually happens in practice is a transition from still, to unknown and then to shake. Therefore, the announcement can get out of sync quite quickly. We can look at debouncing the announcement.
I'm worried about removing "estimated action" after the first announcement. If you don't move your micro:bit for a while, then accidentally knock it, it's going to say "shake" without any further context.
We'd need to think about how adding certainty for {action} is going to work together with what we already have. So I don't think we should rush to add this.
I haven't managed to lose focus. It will always read out the estimated action when it changes, so I'd like to know if this can be reproduced if you can get it into the same state?

We discussed the possibility of only focussed regions reading out either the estimated action or the confidence level of the focussed action row. It might be nice for the user to opt-in to these annoucements in this way rather than having estimated action always announced.


Fix audit errors

Summary of the changes
Fix audit errors in each package.

Are there supposed to be 5 commits on this?

Are there supposed to be 5 commits on this?

Yeah we need each audit fix to be in the correct scope so all 5 are needed in this case


terragrunt graph adds header, invalidates dot format

Just a cosmetics thing, if you run terragrunt graph | dot -Tpng > mygraph.png
you will get an error like so Error: <stdin>: syntax error in line 1 near '[' because terragrunt adds the two lines [terragrunt] 2016/08/30 11:41:34 Remote state is already configured for backend s3 [terragrunt] 2016/08/30 11:41:34 Running command: terraform graph
It's easy to just run terraform graph instead, but I wanted to give a heads up

Ah, good point. I suppose there are two options: either a) add a --silent flag that suppresses terragrunt logging or b) detect if we're being piped into another process, at least for certain commands like graph, and automatically suppress logging in those cases.

I was thinking of just disable logging if the command was graph, but maybe figuring out if we are piping the command is the nicer way to solve it.
A flag like --silent would be confusing I think, as it isn't clear if we silence terragrunt only or if it is a flag passed to terraform.

One solution would be to log to stderr instead of stdout. This would ensure "transparent" wrapping around terraform.
As a personal preference, I would also opt for being quiet by default.

This should be fixed by #94.


Housekeeeping

make it easier to use the intended node version
cleaned up the README
added a .editorconfig to help us keep consistent formatting.

Add .editoconfig

^ Other than this typo this LGTM! :dog:


Experiments tree: show middle state icon and "N out of something" for checkpoints

It's not visible where exactly that checkpoint is selected. Circle in the experiment should have some different form + we need show some text.

@maxagin Is there a Figma spec for this middle-state?

@wolmir this is what we have discussed yesterday, so we do not have the specs yet, first, we will have to agree on the concept.

More arguments: The life cycle for checkboxes and checkpoints is short, for stars it's long, but they won't be produced massively.

Let's test it with the visuals. Thanks

Ah, got it! Thanks!

Updated:

Sorry, @maxagin  - it looks like a bug to me tbh + experiments will be "jumping" right to left and back, also not good.

experiments will be "jumping" right to left

Everything is staying exactly in the same position as is.

Everything is staying exactly in the same position as is.

yep, you are right. It means that this picture was playing even more tricks with my mind than I thought :)
btw, it also means that if do something (deselect an experiment - quite lightweight action) it might have to disappear from the table

Yes, if you remove Checkbox, Star or Checkpoint from the nested and folded experiments, the row will disappear. However, currently, does not it feels the same when you fold the experiment - when all the checkpoints are disappearing? Do not you think that the goal is to show selected objects, and removing them does not play a significant role in the future interaction?

Please also consider the implications of https://github.com/iterative/vscode-dvc/issues/1269

Folding and unfolding exp that have nested starred checkpoint
V1
https://user-images.githubusercontent.com/98249521/176787255-f0158307-2aeb-48b2-8200-74355ed15ba5.mov
V2
https://user-images.githubusercontent.com/98249521/176787266-a96efc53-a5f5-4956-9c88-40eb0783bf9c.mov

Another option
https://user-images.githubusercontent.com/98249521/176788566-aaf72de9-1099-4d5c-9daf-90e50d9ab99f.mov

Something like this for the first iteration?
https://user-images.githubusercontent.com/1231848/177645211-416a8202-8f7f-4b41-ab0d-4925fd7f8469.mov

This approach visually feels familiar, but conceptually it's not good I think. However, I am not 100% sure we will have some different concepts. From my latest research, I was not able to find any product that shows nested info/marks at the parent level, meaning any ideas will be too original for this case.
Thank you @wolmir  good stuff !

Just for the record: I think we do not need to inform the user about hidden states.

What are hidden states @maxagin ?

Sorry, @wolmir I confused this ticket with one I have planned to create.

@wolmir , what is left here? I think it's all done. No? Thanks!


Help on Error

An option to display the help command if there is an error or another command throws

Done in 55cb2b6b26ca51b03490c84bbae4aabb64aa6026. Should be as easy as:
public class MyHelpCommand: DefaultHelpCommand {
    let executeOnCommandFailure = true
}

CLI.helpCommand = MyHelpCommand()


Kubernetes Ingress using self signed certificated instead of cert manager certificate

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
My Kong APIGW uses a self-signed certificate instead of using the cert-manager generated certificate.
From the documentation I use below annotations
    annotations = {
      "kubernetes.io/ingress.class"                           = "kong"
      "kubernetes.io/tls-acme"                                = "true"
      "certmanager.k8s.io/cluster-issuer"                     = "le-clusterissuer"

    }

I have pushed the cert certificate by using the kong admin API but it still using self-signed certificate.
The ingress controller is attached to a classic load balancer in AWS.
Expected Behavior
By using those annotations, kong gateway should use valid SSL certificate.

Steps To Reproduce
No response
Kong Ingress Controller version
-------------------------------------------------------------------------------
Kong Ingress controller
  Release:    1.1.1
  Build:      ba4eea9
  Repository: git@github.com:Kong/kubernetes-ingress-controller.git
  Go:         go1.15.3
-------------------------------------------------------------------------------

Kubernetes version
1.19

Anything else?
No response

Can you show the entire Ingress? Has cert-manager successfully provisioned the certificate and added a matching tls: block to the Ingress spec? The controller won't do anything with the cert-manager annotations itself; it can only parse the standard Ingress TLS configuration block. cert-manager should add that though.


Specificity issue when components share the same style

Repro: https://codesandbox.io/s/identical-styles-specificity-issue-forked-qijm7?file=/src/App.tsx

Upon looking at this more, I wasn't accurate labeling it as specificity, the core issue here is the duplicate class definitions.
This is becoming a more critical issue in our app, not because we're re-using styles among multiple components, but because this is also happening when we're lazy loading. When a section loads async, it adds only the stitches classes needed for the component instances contained within it, which results in an uneven duplication of classes and some funky behavior.
Ex:
App 1 includes a "" with "size: normal" and "style: primary" variants
App 2 includes a "" with "size: normal" and "style: secondary" variants
App 1 loads first, then App 2, in different sections of the page so both are visible at once. Stitches CSS ends up with class definitions like
.<Button>

.<Button-size-normal>

.<Button-style-primary>

.<Button>

.<Button size-normal>

.<Button style-secondary>

So the "primary" button has the base  styles applied after primary variant, which messes up the display

@peduarte So the sandbox I originally posted is no longer reproducible on v0.1.0-canary.20, because of https://github.com/modulz/stitches/pull/485/files, but that hasn't solved the root issue.
Here's another sandbox which demonstrates the problem more clearly https://codesandbox.io/live/1d1t7h0?file=/src/App.js
Notice that the border of <Button2> changes from blue to red on click (because a new, duplicate class is inserted after <Button2>'s class).
https://github.com/modulz/stitches/pull/485/files fixes the first issue because it's avoiding a write to the style tag, which includes the duplicate class definitions, but the root cause (that when update is called the unitedCss includes duplicates) is not addressed. So as soon as a "new" rule is added and update is called, the duplicates will be written.
You can see a downside of the approach in 485 play out in the sandbox I linked here if you change the order of components.
In this case,<Button3> styles are added first, they aren't unique so in insertMethod the write is skipped. Then, Wrapper is added, now we have a unique new style, and the write with the duplicate happens.
      { showThing &&
<Button3>button3</Button3><Wrapper>NewComponent</Wrapper>
}

If you reverse the order, the issue is fixed 😄 ! Now <Wrapper> styles are added, write happens, then <Button> is added, write is skipped as there's nothing new. But we've just postponed the bug until a new style is added.
      { showThing &&
<Wrapper>NewComponent</Wrapper><Button3>button3</Button3>
}

Actually, we overcomplicated the sandbox there, it's not even necessary to hold back the render so it happens asynchronously. It simply comes down to rendering order https://codesandbox.io/s/duplicate-classes-9y0fx?file=/src/App.js

@acconway thanks for that

Any news here or a workaround? We're being plagued by a similar bug. We have two apps on the same page (both using a stitches DS) loading both completely breaks the style tag output

@hennessyevan not sure I follow. a minimal reproducible example would help. sounds like in your case the insertion method API might help.

Thanks @peduarte. The issue I was happening actually isn't with stitches at all but with our webpack setup. That was a frustrating revelation. Sorry for polluting this issue.

Validated: https://codesandbox.io/s/identical-styles-specificity-issue-forked-uvd54?file=/src/App.tsx

This fix has now been released 🚀
Release post: https://stitches.dev/blog/stitches-0.2.0
Thank you


Peer 2 Peer sample is failing

Peer to peer sample from HIP's git repository(https://github.com/ROCm-Developer-Tools/HIP/tree/develop/samples/2_Cookbook/8_peer2peer) is failing on my server with this log: Peer2PeerLog.txt.
My Server Specification:

Motherboard
Cpu
10 x Instinct MI50 GPUs
PCI Peer-to-Peer enabled in BIOS
IOMMU disabled, in BIOS and grub

Here is a little background on how I reached this point:
I stumbled upon the Memory access fault by GPU node-1 on address "(nil)." Reason: Page not present or supervisor privilege error while executing the training sample code of PaLM-RLHF-Pytorch and that led me to https://github.com/RadeonOpenCompute/ROCm/issues/1339 which eventually convinced me to run the peer to peer sample.
Here is the output of dmesg, I can see page faults, dmesg output.txt.
My thoughts are going towards the driver which might be the issue, but I am not sure. I have ROCm 5.4.1 installed with all usecases.
I do appreciate if you could help me on this.

I can confirm that this issue still exists as of rocm 5.7 and that it is specific to gfx906. gfx900, gfx908, gfx1030 are unaffected

Hi @ramin-raeisi, please check latest ROCm6.0.2 to see if your issue has been fixed.  If fixed, please close the ticket.  Thanks.

the issue appears fixed by updated kernel not rocm but yes, fixed

Closing the ticket as issue is fixed.


Specify spaces rather than tabs

Not sure how critical this is given the site's stack/publish process, but I figured I'd ask the question: Should the style guide choose between spaces and tabs in the YAML files? My guess is spaces offer more uniformity and granularity needed for this project.

Good add. Thanks.


implement getRequestContext utility

This PR Introduces a new getRequestContext utility that allows developer to get access not only
to their Cloudflare env but also to their cf and ctx objects
The utility can only be used in server-only code (meaning that it is incompatible with
Pages-router components).
Usage example:
// app/api/hello/route.js

import { getRequestContext } from '@cloudflare/next-on-pages';

export const runtime = 'edge';

export async function GET(request) {
	const {
		env,
		cf,
		ctx: { waitUntil },
	} = getRequestContext();
	// ...
}

Demo/Example app showcasing the usage of the getRequestContext utility:
       https://github.com/dario-piotrowicz/next-on-pages-get-request-context-test-demo

How to use this inside App Router page.tsx file?

@paramaggarwal you can import the utility from @cloudflare/next-on-pages and call it from your component
If should work fine as long as the component is using the edge runtime
You can see an example of that here: get-request-context/page.jsx

Thanks for the example @dario-piotrowicz. I tried the same and am still not able to get it to work. Here are some learnings from my investigation:

The setup with wrangler types, updating env.d.ts to use it with interface CloudflareEnv extends Env {} is done
When I create a new app with create-cloudflare the next.config.mjs gets await setupDevPlatform() and not just setupDevPlatform() as seen here.
This was causing my npx next dev command to hang and the Next.js running console log was not showing up. Then I removed the await as per the other places in the docs and this let my Next app run, but my getRequestContext failed with error failed to retrieve context from code here
Then I added logs locally to the flow and can see that this API call is not returning and an error is also not being thrown for me in this line of code

I've still not managed to fix my issue, but I think my local wrangler setup is not in sync with the remote cloudflare worker/pages instance because I had renamed the remote one and I think that broke something. So this is an issue on my side.
But this highlights two fixes:

Recommend developers to keep the await in await setupDevPlatform() so that if there are any errors they can be thrown on app startup itself.
The await fetch(${DEV_REGISTRY_HOST}/workers) seems to be hanging indefinitely for me - not sure what is the issue there.

Hi @paramaggarwal thanks for the message 🙂
Let me reply bit by bit...

EDIT: I was able to resolve it by forcing the local worker registry to restart. (reference)

I see, glad to hear that things are better 😄 , is it all good now then? or do you need further assitance? 🙂

When I create a new app with create-cloudflare the next.config.mjs gets await setupDevPlatform() and not just setupDevPlatform() as seen here.

The await that create cloudflare adds is correct, I can see that I forgot to add that in our next-dev readme, that's my bad, I'll look into adding it (although things should also work without the await)

This was causing my npx next dev command to hang and the Next.js running console log was not showing up. Then I removed the await as per the other places in the docs and this let my Next app run, but my getRequestContext failed with error failed to retrieve context from code here

This makes it sound to me like indeed the setupDevPlatform was facing some issues, and by remove the await you effectively simply made the issue being ignored still without the dev platform getting set up (hence why getRequestContext would fail).

I've still not managed to fix my issue, but I think my local wrangler setup is not in sync with the remote cloudflare worker/pages instance because I had renamed the remote one and I think that broke something. So this is an issue on my side.

next-dev as well as wrangler pages dev don't have anything to do with the remote Cloudflare worker/pages instance, so having it out of sync with your local setup should not really be an issue at all

Recommend developers to keep the await in await setupDevPlatform() so that if there are any errors they can be thrown on app startup itself.

👍

The await fetch(${DEV_REGISTRY_HOST}/workers) seems to be hanging indefinitely for me - not sure what is the
issue there.

Seems like a wrangler issue, anyways one that you've solved right? 🙂

All good now, thanks a lot for the proactive help. 👍

One small idea. I can foresee a situation where someone did not add the setupDevPlatform() line in nextjs config but is then calling the getRequestContext() method. So we could update the failure log to suggest adding setupDevPlatform() to the nextjs config.

That's a good idea, I can add that, thanks 🙂👍


enhancement(cli): Adds help info to cli tool

This PR attempts to quickly provide some help information as requested in #21.
Below a screenshot of the help on Windows 10

A screenshot of the help on Ubuntu

Thanks, @soapdog, this is tremendously useful! A few notes:

I'm resolving the TaskCluster build issue over in #23.
The path argument is intended to be specified without the --path flag, the way you've documented it in the synopsis as <path or URL>. Thus I wouldn't include it as an option in the Run options and Package options sections. (The module we're using to parse command line flags will allow it to be specified via --path as well, but I expect to change that in the future, so I don't want to document that behavior.)
The --jsdebugger and --wait-for-jsdebugger flags only apply to run, not to package. They're also intended to be used by the developers of qbrt itself (i.e. you and me 😉) rather than the developers of apps that use qbrt. (The latter can use Runtime.openDevTools() to open a DevTools window for the app, as demonstrated by the example and shell apps.) I wonder if there's a way to identify them as "internal" options. What would you recommend?

@mykmelez,
Just after sending the PR and shots, I noticed the mistake on the shots and sent a new commit fixing it but forgot to send any shot representing the actual version which looks like:

As for the options, I believe I can add descriptions to them. If that is the case, then I can write that they are optional and be more descriptive of how and when each option should be used. How about that? 😸

@soapdog Are you still interested in finishing this up? It's almost ready! I think it just needs descriptions for the --jsdebugger and --wait-for-jsdebugger options as well as removal of the --path option.
Here are my recommendations for the descriptions:
For --jsdebugger: Open the runtime toolbox, which is primarily useful for debugging the runtime itself.
And for --wait-for-jsdebugger: Pause the runtime at startup until the runtime toolbox connects.

@mykmelez I am interested, sorry for the delay, will finish them in some hours when I arrive home.

@mykmelez I think it is good now. 😸

@mykmelez I think it is good now. 😸

Looks great, thanks @soapdog! I fixed a few style nits (now that master is checking those via eslint) and pushed this to master!


chore: run unit tests for Firefox on MacOS

I noticed that the unit tests for Firefox are only run on Linux, but not on MacOS. To get out of experimental state it would be good to have these running as well.

@jschfflr any idea why the job for Mac isn't running?

The failures for linux-main-checks seem to be unrelated to my changes.


Some minor clean up

Hello @walterhiggins. First of all, thank you very much for ScriptCraft!! FYI we used it as part of https://www.youtube.com/watch?v=83I8DzWjnNQ. I've started looking into if we could package what's underlying that video up suitable for simple use and included out-of-the-box into ScriptCraft - would you be interested in such a contribution? To "test the water" so to say, here is a trivial PR with just some minor code formatting type clean-up - no real functional enhancements here, yet. If I can find the time, then hopefully more to come where this came from... incl. e.g. https://github.com/vorburger/ScriptCraft/commit/66aa7c665cc7d33ff168fa4ecd7fa85bf3582f0d - OK for you?

Hey @walterhiggins tx for your interest re. how we used #Blockly with ScriptCraft! See https://github.com/walterhiggins/ScriptCraft/compare/master...vorburger:BlocklyCraft. Also watch https://www.youtube.com/watch?v=cat5f-Hy16k. Interested in a PR? ;-)


 Can you assign this to me ?

I have been trying to look over this issue . But Can I ask for some code reference to existing prometheus metrices which might be relevant to this issue ?

@h3llix something like this?
https://github.com/cilium/cilium/blob/6231257f7fac37c8b3b599508ba234025be3c0f6/pkg/metrics/metrics.go#L1095-L1103

@aanm I have created a draft pr . Can you review it ? Sorry took me a while to figure out as I recently started learning about prometheus.

@aanm Is this still considered a good-first-issue? Seems like a lot more work now according to https://github.com/cilium/cilium/pull/16448#issuecomment-859183629.

yeah, it's just a matter of adding a LRU map and make sure that is part of the sysdump


Make a new mob for dragon larva and wither minions

Wither Minion: A smaller wither skeleton, much like a skeleton but with black skin and maybe sightly smaller.
Dragon Larva: ???

Not sure about that


Feature/web view failure listener

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@googlebot I consent

@just-kip i updated pr, can you accept license?

@googlebot I consent.

@googlebot I fixed

@Ivansap  I have not idea why CLA marks unsigned. In https://github.com/flutter/plugins/pull/1788 CLA is succeed

@googlebot I fixed

Thanks for the contribution! We apologize for the long delay in triaging this PR. We’re in the process of overhauling our PR triage system to respond much more quickly, as well as working through the backlog.
It looks like in the meantime this was implemented in https://github.com/flutter/plugins/pull/2639, so I'm going to close this as obsolete. Apologies for the duplicated effort.


[Question] How to determine controller in visit(Page page) ?

I have started 2 controller, how do i determine for which controller did i get call on visit(Page page) function.

Have you tried using getThread().getName()) ? You can put this in a System.out.println or logger.debug and it will return a string Crawler 1 .

This will always give crawler 1.
I don't think we have option to name a controller.

I put the getThread().getName() call in the visit() method of the BasicCrawler class and I ran it with 75 crawlers and it worked.

I just crawled wikipedia.org with 3 crawlers just now and I have copied the output lines with the visit(Page, page) statements:
12:41:14 INFO  [Crawler 1] - [WebCrawler]- URL: https://en.wikipedia.org/wiki/Main_Page
12:41:14 DEBUG [Crawler 1] - [WebCrawler]- Domain: 'wikipedia.org'
12:41:14 DEBUG [Crawler 1] - [WebCrawler]- Sub-domain: 'en'
12:41:14 DEBUG [Crawler 1] - [WebCrawler]- Path: '/wiki/Main_Page'
12:41:14 DEBUG [Crawler 1] - [WebCrawler]- Parent page: null
12:41:14 DEBUG [Crawler 1] - [WebCrawler]- Anchor text: null
12:41:14 DEBUG [Crawler 1] - [WebCrawler]- In the visit(Page page method of:Crawler 1
12:41:14 INFO  [Crawler 2] - [WebCrawler]- URL: https://en.wikipedia.org/wiki/David_Sugarbaker
12:41:14 DEBUG [Crawler 2] - [WebCrawler]- Domain: 'wikipedia.org'
12:41:14 DEBUG [Crawler 2] - [WebCrawler]- Sub-domain: 'en'
12:41:14 DEBUG [Crawler 2] - [WebCrawler]- Path: '/wiki/David_Sugarbaker'
12:41:14 DEBUG [Crawler 2] - [WebCrawler]- Parent page: https://en.wikipedia.org/wiki/Main_Page
12:41:14 DEBUG [Crawler 2] - [WebCrawler]- Anchor text: David Sugarbaker
12:41:14 DEBUG [Crawler 2] - [WebCrawler]- In the visit(Page page method of:Crawler 2
12:41:17 INFO  [Crawler 3] - [WebCrawler]- URL: https://en.wikipedia.org/wiki/Beer_festival
12:41:17 DEBUG [Crawler 3] - [WebCrawler]- Domain: 'wikipedia.org'
12:41:17 DEBUG [Crawler 3] - [WebCrawler]- Sub-domain: 'en'
12:41:17 DEBUG [Crawler 3] - [WebCrawler]- Path: '/wiki/Beer_festival'
12:41:17 DEBUG [Crawler 3] - [WebCrawler]- Parent page: https://en.wikipedia.org/wiki/Main_Page
12:41:17 DEBUG [Crawler 3] - [WebCrawler]- Anchor text: beer festivals
12:41:17 DEBUG [Crawler 3] - [WebCrawler]- In the visit(Page page method of:Crawler 3

I remembered that if your crawl is too shallow then it can end quickly with only the first thread grabbing the first link and the other threads without any links to crawl. Try crawling a large site with a depth of 5 with three crawlers.

May I ask why you are creating multiple controllers, as opposed to a single controller with multiple crawlers?

As most of us do, i'm using just one controller to create multiple crawlers with each crawler having its own thread.

sorry, the question was for @rockyhalo

@rockyhalo, no problem.

@pgalbraith I have a different use case.
Suppose i start a crawler with 2 seed  url and 2 crawlers. Now if i need to start another crawler with 3 seed url and 2 crawlers what do i do ? Either i have to use a controller or wait for existing crawling to stop.
If now i make a new controller they will have count again as crawler 1 and crawler 2.
So my main aim here is to identify first 2 seed url as separate identity and second 3 seed url as separate identity.
I may be wrong or there might be a better way but i fell this is an issue as we should know which controller response is coming in visit page.

@rockyhalo It looks to me like you can start your controller asynchronously and then continue to call controller.addSeed(String pageUrl) to add new seed URLs as you discover them after the controller is already started.  But that's still a single controller using a fixed number of threads.
If you really need to have separate controllers with possibly differing number of threads then there are also some simple ways that you can have your crawler identify it's "owning" controller.
You could extend CrawlController and add an identifier field.  Then in your crawler class use getMyController() and cast it to to your extended class type to obtain the identifier.
You could also make use of the WebCrawlerFactory and have the factory inject a controller identifier of some sort into each WebCrawler as they are instantiated.

@sahlipoto If I understand correctly you're also suggesting extending CrawlController and adding a toString() method in the extension?  This works too, and has the benefit that the crawler wouldn't need to cast the controller when getting a reference to it.

@pgalbraith, no need to extend the controller but just add the myController.toString() statement to the crawler file in the visit(Page page) method. I got an output something like this:

In the visit() method of the SavePageWebCrawler:
Crawler 1
called by the controller:
edu.uci.ics.crawler4j.crawler.CrawlController@57bc24d0

Yes that works if the hash code is enough for the OP's needs.

@rockyhalo, what do you think?

@shalipoto This looks like it will work but there is no access to "myController" object in visit(Page page) as the call is coming in new thread.

@rockyhalo, if you are happy with the answers, can you close this issue please?


Error

This error in any version python 3.8+ and others

SOLVED UP
MORE Multi Threading requests not working

it's Frozen !


Updated last stable release in readme file

Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

Merging directly since it's super simple :wink:


[PG16] Added integer conversion in toBoolean functions

Implemented explicit integer conversion to boolean in both toBoolean and toBooleanList functions. Also made corresponding changes to regression tests.
This PR makes the toBoolean() and toBooleanList() functions convert integers to boolean, as occurs in Neo4j.

@WendelLana Don't forget to add in the not-so-optional, optional extended description -


[WI-46754] Add ext-uv stubs

Resolves: https://youtrack.jetbrains.com/issue/WI-46754

For future, UV extension requires PHP 7.0 but you made stubs PHP 7.1 only compatible. I've fixed that otherwise, people will get error code navigating to the stubs.
Please also note that resource is not a primitive type (removed this as well).

Will keep that in mind fix next time 👍 !