Errors on Link Prediction Example

Hi, I'm trying to implement a link prediction GNN using PyTorch Geometric and I think I have found several errors in the example provided in:
https://github.com/rusty1s/pytorch_geometric/blob/master/examples/link_pred.py
In particular, in line 23 the edges are divided in train/val/test, however the method doesn't generate negative training examples and so these are obtained in lines 61 to 67. In my opinion, the operations performed are not correct because from lines 61 to 67 the code is generating negative examples (stored in neg_edge_index) starting from the positive training ones, but this means that neg_edge_index can contain edges that are actually present in the positive and negative validation/test sets. In fact I checked by simply iterating the negative training examples in neg_edge_index and controlling if they are also in the test/validation negative or positive examples, and this happens for several of them, which is clearly not correct.
It's not clear to me why the method train_test_split_edges from torch_geometric.utils would not output negative training examples, but only validation/test ones.
Furthermore it's not clear to me why in the test() method message passing is only performed on the subgraph spanned by the positive validation/test edges. I believe the message passing should always be applied to the graph spanned by the positive training edges, and then the classifier should produce the output only for the validation/test edges (both positive and negative).
Thank you in advance for your time.
@dongkwan-kim I though I might quote you since I saw you were the one mainly working on this

@dongkwan-kim I though I might quote you since I saw you were the one mainly working on this

To summarize this issue, there are two error reports and one question.
R1. Negative samples used for training may be included in the validation/test sets.
R2. test() method uses only test edges, not full edges of the graph.

For R1 and R2, you are right. It was my mistake. I will try to make a PR as soon as possible, but it may be delayed due to my schedule.
In my defense for R1, link_preds.py is a toy example created during separating train_test_split_edges from the GAE implementation. There seems to be a similar problem.

Q1. Why train_test_split_edges would not output negative training examples, but only validation/test ones.

This also borrows from the GAE implementation. Of course, we can fix the set of negative training samples before training. However, if we do this way, we need to draw negative training sets as many as the number of the specified training epoch. In terms of time and memory management, I think the current method is better.

R2 is indeed a bug and should be fixed. It would be great if you can do that @dongkwan-kim . Regarding R1, I see no problem here since we cannot expect to have access to positive valid and test edges during training. So having negative sampled edges that are actual positive cannot be prevented.

Thank you both very much for your help and your quick answers :clap:


django 1.8, 1.9 support

Don't see a reason why it wouldn't work, but needs to be tested. Didn't had much time lately but will try to test it one of these days. If you get around on testing it please post back

@k3oni.. Is there a way if we can add multiple Servers to the pydash or we can ftech the same data usin gthe pydash script and collect them into mai pydash hosts and graph them into same one console.


Branch iOS 9 has issue in podspec

The branch iOS 9 doesn't work.
Cocoapods returns an error in the podspec.
Please fix it.
Thank you in advance.

@gianfilippocisternino:
What is your Cocoa Pods version?
pod --version

Looks like Cocoapods 1.7.0 is required.

The ios9 branch will never be updated anymore. So I am closing this issue.


Fix issues SDC-7902 and SDC-7861. Also, change apache-rat check start principle.

I've tried to build it on my Macbook and found that there are some issues with tests. So, I've create bug report on JIRA and fixed found issues by these commits.
bug SDC-7902 and SDC-7861
Also, I've removed ANT script which generates bash script and change it with exec goal. It allows to build and test on non-UNIX OSes.

Hi @jarcec!
Well, I think I can do that, but main problem is my macbook has die few
hours ago:(, with all data on it! so, once I fix it or buy new one I can
divide this pull request to several. Cheers!
21 дек. 2017 г. 7:37 PM пользователь "Jarek Jarcec Cecho" <
<EMAIL_ADDRESS>написал:
@jarcec requested changes on this pull request.
Do you mind breaking the one pull request into independent requests for
each JIRA/task?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/streamsets/datacollector/pull/42#pullrequestreview-85116026,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3fi_oG365T0je92vVd6pzqjc4NNTPkks5tColAgaJpZM4QesZz
.

No worries and take your time @estatic ! Hope that your laptop will be resurrected soon :)

Okay, move SDC-7861 issue to another pull request.
https://github.com/streamsets/datacollector/pull/49

Moved SDC-7902 to https://github.com/streamsets/datacollector/pull/50

Moved OS depended script generation fix to here https://github.com/streamsets/datacollector/pull/51

@jarcec Should I close this pull request?

Yup let me do that @estatic .


Add PHP 8-only Support

This PR adds a new major version, v1.0.0.
Specifically, it:
Removes support for all PHP 7.x versions.
Requires PHP 8.0+.

@leopoletto I just responded to you in another issue, not seeing you had already done this. Ignore my other comment. Thanks!!


[error][main] Error: Page not found

Hi, it's Michael (we've met today).
I've checked out master and built codebox:
$ git clone https://github.com/CodeboxIDE/codebox.git
$ cd ./codebox
$ npm install .
$ grunt

No build error, no run error in the first place.
If I call http://localhost:8000 (on an Ubuntu 14.04 Vagrant machine), I get:
Codebox is running at http://localhost:3000
[log][hooks] call hook  users.auth
[log][events] event 'users:add'
[log][hooks] call hook  users.auth
[log][events] event 'users:add'
[log][hooks] call hook  users.auth
[log][events] event 'users:add'
[log][hooks] call hook  users.auth
[log][events] event 'users:add'
[log][socket] info GET /socket/events/info 2ms 200
[log][socket] info GET /socket/events/522/9iknpe0s/websocket 2ms (unfinished)
[log][socket] connection to service 'events'
[log][hooks] call hook  settings.get
[error][main] Error: Page not found
    at /home/vagrant/repos/codebox/lib/index.js:120:17
    at Layer.handle [as handle_request] (/home/vagrant/repos/codebox/node_modules/express/lib/router/layer.js:76:5)
    at trim_prefix (/home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:263:13)
    at /home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:230:9
    at Function.proto.process_params (/home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:305:12)
    at /home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:221:12
    at Function.match_layer (/home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:288:3)
    at next (/home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:182:10)
    at /home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:184:16
    at Function.match_layer (/home/vagrant/repos/codebox/node_modules/express/lib/router/index.js:288:3)

The UI still renders. Nice and clean by the way :+1:
Best regards,
Michael

i have this problem, any one can help?

Same error


[Test] Update callsites of test_utils in examples after renaming test to tests

Stack from ghstack (oldest at bottom):

-> #345
#344
#343

Summary:
As titled
Test Plan:
python -m pytest examples/<any_example_test_folder>
However, we cannot fix breakages in this PR as example unit tests are not actively maintained / out of scope of CI.
Differential Revision: D39994169

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.

Codecov Report

:exclamation: No coverage uploaded for pull request base (gh/langong347/66/base@df71f8e). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@                   Coverage Diff                    @@
##             gh/langong347/66/base     #345   +/-   ##
========================================================
  Coverage                         ?   93.39%
========================================================
  Files                            ?       54
  Lines                            ?     3253
  Branches                         ?        0
========================================================
  Hits                             ?     3038
  Misses                           ?      215
  Partials                         ?        0

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.

@langong347 has imported this pull request.  If you are a Meta employee, you can view this diff on Phabricator.


Consolidate Useful information on the last screen

letter_date_notification and MLH_outro_download_forms both have useful information, but I question spreading them out over a few different screens like they. IMO, we could show all of the same info on the last screen, in context, without overwhelming the user or putting too much text there.

Because letter_date_notification is a conditional screen and only shown when it has not yet been thirty days, I will leave it separate for now. However, this did prompt me to include a short dynamic section in the instructions:

OR


Distributed provisioning

This changes the architecture of PMEM-CSI such that the controller part only implements the webhooks and provisioning happens on the nodes.
This PR is based on some pending changes (PR #825, PR #836) and uses the external-provisioner canary image because v2.1.0 has been tagged, but the versioned image is not available yet. Therefore the PR is ready for review and further testing, but merging should wait.
Fixes: https://github.com/intel/pmem-csi/issues/823, https://github.com/intel/pmem-csi/issues/733, https://github.com/intel/pmem-csi/issues/729, https://github.com/intel/pmem-csi/issues/594, https://github.com/intel/pmem-csi/issues/526, https://github.com/intel/pmem-csi/issues/507

I've rebased, so only new commits are present now.
I also re-enabled the stress test. It passed in local testing.

I've pushed a major update: now the operator supports configuration of the webhooks and distributed provisioning. Some E2E tests passed (in particular the ones for the operator), but for full coverage I let the CI do its work over night.
I'll double-check tomorrow, but I think there is only one minor TODO left (the one about reading the node port from the shell script in deploy.go).
I've not tried to ensure that all tests pass for all commits, so strictly speaking I would have to squash "distributed provisioning", "deploy: consistent object naming, app.kubernetes.io labels" and "operator: distributed provisioning, scheduler extensions". But I think it's easier to review when they are separate.

I've pushed a major update: now the operator supports configuration of the webhooks and distributed provisioning. Some E2E tests passed (in particular the ones for the operator), but for full coverage I let the CI do its work over night.
I'll double-check tomorrow, but I think there is only one minor TODO left (the one about reading the node port from the shell script in deploy.go).
I've not tried to ensure that all tests pass for all commits, so strictly speaking I would have to squash "distributed provisioning", "deploy: consistent object naming, app.kubernetes.io labels" and "operator: distributed provisioning, scheduler extensions". But I think it's easier to review when they are separate.

I'll double-check tomorrow, but I think there is only one minor TODO left (the one about reading the node port from the shell script in deploy.go).

That referred to TODOs inside the patch itself. I still need to address review comments.

I'll double-check tomorrow, but I think there is only one minor TODO left (the one about reading the node port from the shell script in deploy.go).

That referred to TODOs inside the patch itself. I still need to address review comments.

I intentionally left https://github.com/intel/pmem-csi/issues/857 out of the current implementation. It's not going to be a problem for a while and can be addressed later.

I intentionally left https://github.com/intel/pmem-csi/issues/857 out of the current implementation. It's not going to be a problem for a while and can be addressed later.

There was a test flake (?), filed as https://github.com/intel/pmem-csi/issues/858.
I don't think that it was caused by this PR, so I am going to merge it despite that.

There was a test flake (?), filed as https://github.com/intel/pmem-csi/issues/858.
I don't think that it was caused by this PR, so I am going to merge it despite that.


curreny issue

hey folks
thanks for this great plugin.
i am setting up minicart to use with the currency of Israeli Shekels (ILS)
but i get the strange "undefined" next to the price in the Minicart tab that drops down.
here is a demo: http://bit.ly/rz2pgA
is there a way to fix this?
Thanks!

It looks like you have a fairly old version of the code. This issue should already be fixed in the latest version on Github. Try to replace your JavaScript file with this one: https://raw.github.com/jeffharrell/MiniCart/master/minicart.js


With MeshService what should we show in the services column of dataplane?

What happened?
At the moment it shows kuma.io/service this is not very correct when using MeshService.

Triage: remove a column if MeshService is set to Exclusive


Migrates off rxjava2 Fuseable base types as they broke inheritance

This inlines code we needed from Fuseable types in order to allow the
same library to work with RxJava 2.2 and 2.3+
See https://github.com/ReactiveX/RxJava/pull/6129/files#diff-173446df25e9a6926e322f1849bf6c35R31

PS This was our fault, not RxJava's. We used internal types without realizing it as I guess we focused on the type names not the package. Ideally one day we can have a errorprone checker for accidental use of types in packages named internal or internals
cc @anuraaga as well

If I recall correctly, armeria's javadoc build fails when internal classes get referenced on a public one. I believe a combination of enabling all doclint and excluding internal packages can help?
https://github.com/line/armeria/blob/d4f13e4c4dc90826d001bfb73d6ca05dce19689c/build.gradle#L120
/cc @trustin

And this: https://github.com/line/armeria/blob/d4f13e4c4dc90826d001bfb73d6ca05dce19689c/site/build.gradle#L71-L136

hi all https://github.com/openzipkin/brave/pull/809 tried to insulate further


Remove TempTriple in Favor of simple Triples and Reifications

TempTriples were implemented for the sake of downwards-compatibility to Vanilla.
But we have agreed to remove them to be more RDF compliant and also because the TempTriples would again be some kind of semantic hard-wiring into the code (A few edge cases where they are needed do not justify that they are implemented everywhere).
TempTriples will be removed in favor of either arbitrarily sub-classed triples or reificated properties. The relevant issue for this is: https://github.com/acdh-oeaw/apis-core-rdf/issues/3

With the introduction of Reifications we can now drop TempTriples.
What are reifications?
Classes that act as complex relations, having attributes themselves and able to relate arbitrary amount of entity classes and instances too, also they are not able to exist on their own, but always depend on other entities.
It's a common pattern in RDF and we replicate this. In your ontology you can define your own reification identical to entities with the difference that it needs to inherit from AbstractReification.
How do we replace TempTriples with Reifications?
The general procedure would look like this:

All TempTriples that only use subj, obj, prop are replaced with simple triples. No need from your side to do anything here.

All TempTriples that also use some of their additional attribute fields (e.g.start_date_written ) need to be replaced with dedicated Reification classes.

Assume your users have utilized the start_date_written field of TempTriple only between the entities Person and Place and the property would be visited. The semantics of the date would be that the relation expresses the visit has happened at a certain date.
Now to replace this we could introduce a Reification class named VisitRelation with an DateField date of visit, and further we need now two instead of one properties: one from Person to  VisitRelation (e.g. did visit) and one from VisitRelation to Place (e.g. took place in).
Yes, this is semantically more cumbersome, but it's RDF.
I mititaged the clumsiness a bit by pre-selecting the property if there is only one anyway, so in most cases the users would not need to type in did or took place in.
In code this could look like this:
@reversion.register(follow=["tempentityclass_ptr"])
class Person(TempEntityClass):
    pass

@reversion.register(follow=["tempentityclass_ptr"])
class Place(TempEntityClass):
    pass

class VisitRelation(AbstractReification):
    date_of_visit = models.DateField(blank=True, null=True)

did_visit = Property.objects.create(name="did visit")
did_visit.subj_class.add(ContentType.objects.get(model=Person.__name__))
did_visit.obj_class.add(ContentType.objects.get(model=VisitRelation.__name__))

took_place_in = Property.objects.create(name="took place in")
took_place_in.subj_class.add(ContentType.objects.get(model=VisitRelation.__name__))
took_place_in.obj_class.add(ContentType.objects.get(model=Place.__name__))

The UI does only display classes that inherit from AbstractEntity as primary editable things. Classes that inherit from AbstractReification are handled separately in dedicated form logic on the right pane below the usual relations. Basically from UX side they behave like fancy relations.
Besides attaching attributes onto complex relations, this allows us to define very complex relations too with multiple related entities, something that did occasionally pop up in the past in other projects.
script to help with migration
I wrote a management script that checks for existing data in TempTriples and fetches the related classes to get an overview of what the new Reifications need to cover.
The script is here https://github.com/acdh-oeaw/apis-rdf-devops/blob/dev/webpage/management/commands/check_and_migrate_tec_tt.py and can be executed like so:
./manage.py check_and_migrate_tec_tt tec # tec for TempEntityClass
./manage.py check_and_migrate_tec_tt tt # tt for TempTriple

Currently it will not execute any migration, only checking for replacement attributes / classes, so feel free to run it and check the output.
And the output could look like this for example where there is 13 TempTriples between F10_Person -p2 has type-> E55_Type which need to have a replacement reification.
New Reification class needed:
between class (as 'subject'): F10_Person
and class (as 'object'): E55_Type
replacing current property: p2 has type

Currently there are TempTriple with these subjects, objects, property with these fields and number of occurrences:
references = models.TextField(blank=True, null=True) # number of occurrences: 13

The new Reification pattern should look like this:
F10_Person -'prop_entity_to_reification'-> 'NewReificationClass' -'prop_reification_to_entity'-> E55_Type

where you need to create:
'prop_entity_to_reification'
'prop_reification_to_entity'
'NewReificationClass'
Take the fields from above and add them to 'NewReificationClass'

as said in one of the last Jfx we should keep the adaptions that need migration in a dedicated branch for the moment and allow the different projects through poetry groups to install it or stay with the old version for the moment. this allows us to migrate every project as soon as its ready.

not used anymore, closing


RBTree find/*_bound methods keep borrow of the key

Hi,
While using RBTree I faced a problem with a simple scenario: I'm trying to do a kind of search to create a cursor with lower_bound_mut using a reference as an argument and then to insert_after using a value I searched by before; the problem is that the value is still borrowed even the cursor is already created and I see no reason to keep this borrow.
I would just insert it but I want to keep items unique so I do search with optional insertion .
UPD: also I noticed that .get operation on CursorMut returns a reference with a lifetime tied to cursor's lt rather than the tree's one, however, for Cursor is seems to be done correctly , but I may have wrong assumptions about lifetimes and soundness for CursorMut, anyway it would be nice to hear back if I'm using it wrong

All of these issues are now resolved


400 Bad Request

Hi, thanks for your contribution with this plugin.
We are trying to upload an iOS build to Diawi but it throws a 400 Bad Request error.
Any clue on how can we fix this?
Thanks.

Hi, @che1404. Sorry for long delay. Do you still need help?

Hi, no worries @pacification. Actually we are stuck on the same issue. We are getting a. 400 Bad Request error.
I got in touch with diawi developers and here are some findings:

The problem seems to happen on macOS Catalina only. We noticed that when using the diawi API with the Catalina curl distribution, we were getting this error as well
The files trying to upload were not the problem, since we could upload them just by drag and dropping into the browser.
We tried the same curl command with --http1.1 option, and it actually worked.
Then I installed another curl implementation with homebrew, curl-openssl, and the command seemed to worked as well with no --http1.1 option.

Does any of this ring any bell?

Thanks for response. I'm currently trying to reproduce 400 bad request and i have no success attempts.
If it ok to you to use:

Then I installed another curl implementation with homebrew, curl-openssl, and the command seemed to worked as well with no --http1.1 option.

then i close the issue, because i believe i have the same setup and all works fine. If not, please try to see/debug source code for any problems. Thanks.

@che1404, i also released 2.0.0 version. Feel free to try.

Am getting the same error in version 2.0.0

Yep, me too :(

@che1404 Nope. Still trying different things.


Add support for passing in a virtual function index to assign pci device

This PR plumbs through support for passing in a virtual function index when assigning PCI devices.
To specify a device with a virtual function index, add the index to the end of the device instance ID path like so to the CRI spec for the container:

"devices": [
{"host_path":"vpci-location-path://PCIP\VEN_10DE&DEV_15F8&SUBSYS_118F10DE&REV_A1\6&17F903&0&00400010\2"}
],

Where 2 is the virtual function index.
Signed-off-by: Kathryn Baldauf<EMAIL_ADDRESS>

Just to make sure, is there any behavioral difference between supplying a vf index of 0, and simply not providing one?
"devices": [
{"host_path":"vpci-location-path://PCIP\VEN_10DE&DEV_15F8&SUBSYS_118F10DE&REV_A1\6&17F903&0&00400010\0"}
],

vs.

"devices": [
{"host_path":"vpci-location-path://PCIP\VEN_10DE&DEV_15F8&SUBSYS_118F10DE&REV_A1\6&17F903&0&00400010"}
],

@dcantah there's no difference as of now


Importing forwardRef from react is overwritten by @prismicio

Versions

@prismicio/react: 2.1.1
react: 17.0.2
node: 16.14.0

Reproduction

Additional Details

Morning,
When trying to update @prismicio/react to the version 2.1.1, I've noticed that all my currently imported forwardRefs from react are now pointing to @prismicio implementation (caused by this change I assume).
That actually causes an issue with the displayName for some files. Nevertheless, even though I could fix the default import with webpack, I don't feel comfortable using the latest update that overwrites it globally.
Was that the expected behaviour?

Steps to reproduce

Update @prismicio/react to the newest version 2.1.1
Check import { forwardRef } from 'react' (pic)

What is expected?
Importing forwardRef from react won't be overwritten by @prismicio in my whole application.
What is actually happening?
All forwardRef imports are coming from @prismicio now.

Hey @kropelka77, thanks for the report. Could you try v2.1.2-alpha.0 and let me know if the issue is fixed and everything work as expected?
Background info: forwardRef support was added to <PrismicLink> in #131 and published in v2.1.1. It uses a TypeScript declaration merging to support generics in prop types, which <PrismicLink> needs. As you found, this causes issues globally in a project. v2.1.2-alpha.0 uses a different strategy to support generics which shouldn't affect projects globally.

Related: #128, #131, #132

Hi @angeloashmore, thank you for your quick reply!
I've checked and v2.1.2-alpha.0 solves the issue for me - it no longer overwrites global forwardRef ✨

Thanks for testing @kropelka77! The fix is published in v2.1.2.


Fixes send message on mobile

What kind of change does this PR introduce?

Bug Fix

Why was this change needed?

Fixes #18

Other information:

From above to below:

@diwash007 just in case, did you happen to check this design on desktop view after the changes were made?


Unbreak main

This fixes main breaking due to two merges.

Current dependencies on/for this PR:

main

PR #6123   👈

This comment was auto-generated by Graphite.


Help with help in kassy.json

I have this in /module-name/kassy.json:
        ["{{commandPrefix}}start", "Start the Telegram bot"],
        ["{{commandPrefix}}define", "Receive a definition of the module types"]
    ]```

However, running `/help` results in: ```Cannot provide help on module that was not found. Has it been disabled?```

This is on 4.0.0alpha.

Running /help start or /help define that is.

Never mind.  There in the default list of options for /help. Doh.


ASUS Z270G + ARC750 - is not working

System

Motherboard: ASUS Z270G
BIOS Version: 1501
RAM: 32G
GPU: Arc 750 8GB
[X] CSM is turned off.
[X] 4G decoding is enabled.
[X] UEFIPatch is applied
[X] I have read [Common issues (and fixes)]

Description
I did everything and still cannot get it to work

I updated the bios according to instructions using UEFITool
I ran the UEFIPatch  (no pad issue was found and DSDT patch not required)
Flashed the modified bios after extracting the body with /GAN
Changed mbr -> gpt , cms off, 4GB ON (Device manager shows large memory for pci bus after the change)
All ok to that point
With rebarstate set to 8, 12, 14, 32 - I get the below:
GPU is NOT working, only iGPU is giving display (showing error 12 - resource collision)
Tried to connect 2 monitors on GPU and iGPU - same behavior
rebarstate  set to 0 - GPU is showing display again - no error 12 - no rebar
Any suggestions???
-Rafi.

Also - no pad issue was found and DSDT patch not required

rebarState not working at even 8 is weird, because that is same as 256mb default.
Try re doing everything with MMTool, common issue on ASUS motherboards for stuff to break when using UEFITool/UEFIPatch. See https://github.com/xCuri0/ReBarUEFI/issues/98#issuecomment-1826386025 if you need help doing it

I saw that someone with exact same board wrote that it worked for me according to instructions, so its indeed weird
I will give it a try using the MMTool

Update ?

@yargun I have a very similar situation, but as exception works also values 7 and 8. For some reason it seems to me that the problem is in the GPU, I’m almost sure that it would work with another video card, have you tried it by chance?


Kubernetes: UI tab for ingress

As we are backing up kubernetes ingress with rancher ingress-controller, would be nice to have a UI for ingress.

No longer applicable in 1.4.


context c.set/get not working in cloudflare pages functions

What version of Hono are you using?
4.5.8
What runtime/platform is your app running on?
cloudflare pages
What steps can reproduce the bug?
made a minimal repro here: https://github.com/zoubingwu/hono-ctx-set-cf-pages
git clone https://github.com/zoubingwu/hono-ctx-set-cf-pages.git
pnpm install
pnpm run dev

then open http://localhost:8888/, you will see alert with hello undefined
What is the expected behavior?
should be hello joe since it was set in middleware
What do you see instead?
hello undefined
Additional information
// functions/_middleware.ts
import { handleMiddleware } from "hono/cloudflare-pages";

export const onRequest = [
  handleMiddleware(async (c, next) => {
    console.log("middleware setting user");
    c.set('user', 'joe')
    await next();
  })
]

// functions/api/[[route]].ts
import { Hono } from "hono";
import { handle } from "hono/cloudflare-pages";

interface HonoContext {
  Variables: {
    user: string
  };
}

const app = new Hono<HonoContext>().basePath('/api');

const routes = app.get('/hello', (c) => {
  return c.json({
    message: `Hello, ${c.get('user')}!`
  });
});

export const onRequest = handle(routes);

// index.html
fetch('/api/hello').then(res => res.json()).then(data => {
    alert(data.message)
})

Hi @zoubingwu
Thank you for creating the issue. This looks like a bug, but it's a known issue that can be resolved because a Context in Pages Middleware is not shared for the handler. This is a limitation for using Cloudfare Pagea Middleware.
If you want to do more complex things for Cloudflare Pages, you can try the Cloudflare Pages starter template:
npm create hono@latest -- --template cloudflare-pages

@yusukebe cf context prodvides a data object to attach some data and will persist during the request, there docs didn't mention that tho but it works :(
see https://developers.cloudflare.com/pages/functions/api-reference/#eventcontext and https://community.cloudflare.com/t/what-is-context-data-in-pages-functions/476559
can we use that instead? I found that this object is not accessible in hono's context.
I updated the code in the reproduce repo, you can check it out.
// _middleware.ts
export const onRequest = [
  // handleMiddleware(async (c, next) => {
  //   console.log("middleware setting user");
  //   c.set('user', 'joe');
  //   console.log(c);
  //   c.env.eventContext.data.user = 'joe2' // not working
  //   return await next();
  // })
  async (context) => {
    context.data.user = "joe";
    return await context.next();
  },
];

// [[route]].ts
const routes = app.get("/hello", (c) => {
  return c.json({
    //@ts-ignore
    message: `Hello, ${c.env.eventContext.data.user}!`,
  });
});

Hi @zoubingwu
Exactly. Cloudflare Pages provides data. Nice suggestion. I'll consider it.
Hi @BarryThePenguin If you have any thoughts, please share them.

I'd expect this to work..
handleMiddleware(async (c, next) => {
  console.log("middleware setting user");
  c.set('user', 'joe');
  console.log(c);
- c.env.eventContext.data.user = 'joe2' // not working
+ c.executionCtx.data.user = 'joe2' // should work
  return await next();
})

The Cloudflare EventContext is passed to new Context() and can be accessed through the executionCtx property. However the types don't allow for this..

I can see a solution where using c.get() and c.set() update the underlying c.executionCtx.data property, but making the types work might be tricky..

according to https://github.com/honojs/hono/issues/1219, if we use AsyncLocalStorage for context, will this simplifies the typings?

This is closed by #3332!


Update FieldNumber constructor/params

The Android class needs an update to match the APIs decided on in https://github.com/google/blockly/issues/445#issuecomment-227515322

#334 adds the missing constructor with all constraints except the validator function (which android apparently doesn't have an equivalent of, yet).


Remove auto-generated symbols from docs

Describe the bug
{' '} symbol exist in the docs in many places making it a bit confusing.
Screenshots

Additional context
{' '} symbols exist in docs, they have most likely been auto-generated by the linter or prettier.

I would love to work on this issue to get familiar with the codebase a little.

@NabajitS Since you deleted your branch, are you done trying to add this patch yourself, or will you be trying again?

Never mind. Resolved via #614


Trying to understand the Data Collator for compute_metrics evaluation

During training, I'd like to use HuggingFace's compute_metrics in Trainer to evaluate for things such as inference time "Exact Match" accuracies, but it looks like data collator only supports generation with teacher forcing for now.
Currently without evaluating during training, I'd use the inference code with "add_generation_prompt" = True as such:
# Prepare the conversation prompt
plan_conversation = [
    {
        "role": "user",
        "content": [
            {"type": "text", "text": instruction},
            {"type": "image"}, {"type": "image"}, {"type": "image"}, {"type": "image"}, {"type": "image"}
        ],
    }
]

# Create the prompt and process the input
prompt = processor.apply_chat_template(plan_conversation, add_generation_prompt=True)

# Run prompt through model and evaluate for exact match

And then my compute_metrics function I would pass to Trainer is:
def compute_metrics(eval_pred: EvalPrediction):
    label_ids = eval_pred.label_ids
    pred_ids = eval_pred.predictions[0]
    input_ids = eval_pred.inputs if hasattr(eval_pred, 'inputs') else None

    pred_str = processor.tokenizer.batch_decode(pred_ids, skip_special_tokens=True)
    label_ids[label_ids == -100] = processor.tokenizer.pad_token_id
    label_str = processor.tokenizer.batch_decode(label_ids, skip_special_tokens=True)
    if input_ids is not None:
        input_str = processor.tokenizer.batch_decode(input_ids, skip_special_tokens=True)
    else:
        input_str = [""] * len(pred_str)
# ... <compute metrics on decoded string> ...

The main problem is that pred_str is not the typical expected output of just the assistant response like during inference time, but pred_ids seems to be of size (10, 7640). Decoded pred by printing pred_str above is pasted below.
Effectively, I believe it's because of the Data Collator which is also for evaluation predictions using teacher forcing where it conditions on ground truth previous tokens instead of generated tokens which makes sense for calculating evaluation loss, but doesn't reflect inference time behavior.
My main question is if there are any ideas for how I can change Data Collator and train.py in the code base such that it doesn't use teacher forcing for evaluation and would pass in the expected pred_ids to compute_metrics? I think if others end up using compute_metrics, they'd also want inference time behavior as well, so maybe the collator can change depending on if compute_metrics is passed? I'm not too sure right now.
Thanks again for the great work!
Printed pred:

Label:
<something short like a caption for the image, which is what I expected pred_str to be>

I think it might just be "predict_with_generate = True" as a Trainer setting, but need to double check it works with the current Data Collator

On a high level I would imagine this would need a inference-like collator which will be passed to the eval data loader, as like you mentioned the current collator uses ground-truth answers and does not encode any inference-time behaviors.
I wouldn't be available to work on this until late next week, but would be happy to discuss and provide necessary instructions.

Ah okay, looking at the inference dedicated collator for LLaVAInterleave, are these the major things I should probably edit? Or are there other things so that I can get an actual inference time generate / predict tokens? (Changes are commented with "EDIT")
class LLaVAInterleaveInferenceDataCollator(BaseDataCollator):
    def __call__(self, instances: Sequence[Dict]) -> Dict[str, torch.Tensor]:
        # images
        vision_inputs = dict()
        flattened_images: List[PIL.Image.Image] = [x for instance in instances for x in instance["images"]]
        if len(flattened_images) > 0:
            vision_inputs.update(**self.processor.image_processor(flattened_images, return_tensors="pt"))
        num_images: List[int] = [len(instance["images"]) for instance in instances]

        # texts
        # the dataset implementation assume conversations are [user, assistant, user, assistant, ...]
        system_prompts: List[Union[str, None]] = [instance["system_prompt"] for instance in instances]
        conversations: List[List] = [instance["conversations"] for instance in instances]
        max_len = self.tokenizer.model_max_length

        total_image_tokens = 0
        input_ids = []
        labels = []

        for cur_num_images, system_prompt, cur_convs in zip(num_images, system_prompts, conversations):
            cur_num_image_tokens = 0
            cur_input_ids = []
            cur_labels = []

            cur_text = []
            if system_prompt is not None:
                cur_text.append({
                    "role": "system",
                    "content": [{"type": "text", "text": system_prompt}]
                })

            for i, text in enumerate(cur_convs):
                if i % 2 == 0:
                    num_image_tokens = len([m.start() for m in re.finditer("<image>", text)])
                    cur_num_image_tokens += num_image_tokens
                    total_image_tokens += num_image_tokens

                    # .strip(): whitespaces and newlines are handled by chat_template
                    text = text.replace("<image>", "").strip()

                    cur_text.append({
                        "role": "user",
                        "content": [{"type": "text", "text": text}] + \
                            [{"type": "image"}] * num_image_tokens
                    })
                elif i != len(cur_convs) - 1: # EDIT - do not include last assistant message to allow for generation
                    cur_text.append({
                        "role": "assistant",
                        "content": [{"type": "text", "text": text}]
                    })

            assert cur_num_image_tokens == cur_num_images, "Not all images were used"

            temp = self.processor.apply_chat_template(
                cur_text,
                add_generation_prompt=True, # EDIT - allow for generation
                tokenize=True,
                return_assistant_tokens_mask=False, # EDIT - do not mask assistant tokens
                return_dict=True,
                return_tensors="pt",
                truncation=False # the assistant tokens mask seems wrong when truncation is enabled
            )
            cur_input_ids = temp["input_ids"]
            # manual truncation
            if cur_input_ids.shape[1] > max_len:
                cur_input_ids = cur_input_ids[:, :max_len]
            cur_labels = cur_input_ids.clone()

            if self.mask_question_tokens:
                cur_assistant_masks = torch.tensor(temp["assistant_masks"][:max_len], dtype=torch.bool).unsqueeze(0)

                # EDIT: only mask last assistant's response tokens
                assistant_token_indices = torch.where(cur_assistant_masks[0] == 1)[0]
                if len(assistant_token_indices) > 0:
                    last_message_end_idx = assistant_token_indices[-1]
                    last_message_start_idx = last_message_end_idx
                    for idx in reversed(assistant_token_indices[:-1]):
                        if idx == last_message_start_idx - 1:
                            last_message_start_idx = idx
                        else:
                            break

                    # EDIT: Create a new mask for just the last assistant message
                    last_message_mask = torch.zeros_like(cur_assistant_masks)
                    last_message_mask[0, last_message_start_idx:last_message_end_idx + 1] = 1
                    cur_assistant_masks = last_message_mask

                    assert cur_labels.shape == cur_assistant_masks.shape, "Label and mask shapes do not match"
                    cur_labels = torch.where(cur_assistant_masks, cur_labels, self.IGNORE_TOKEN_ID)

            assert cur_input_ids.shape == cur_labels.shape, "Input and label shapes do not match"

            # padding
            if cur_input_ids.shape[1] < max_len:
                cur_input_ids = torch.cat([
                    cur_input_ids,
                    torch.full(
                        (cur_input_ids.shape[0], max_len - cur_input_ids.shape[1]),
                        self.PAD_TOKEN_ID,
                        dtype=cur_input_ids.dtype,
                        device=cur_input_ids.device
                    )
                ], dim=1)
                cur_labels = torch.cat([
                    cur_labels,
                    torch.full(
                        (cur_labels.shape[0], max_len - cur_labels.shape[1]),
                        self.IGNORE_TOKEN_ID,
                        dtype=cur_labels.dtype,
                        device=cur_labels.device
                    )
                ], dim=1)

            input_ids.append(cur_input_ids)
            labels.append(cur_labels)

        # sanity check
        assert total_image_tokens == len(flattened_images), "Number of image tokens does not match the number of images"

        input_ids = torch.cat(input_ids)
        labels = torch.cat(labels)

        return dict(
            **vision_inputs,
            input_ids=input_ids,
            labels=labels,
            attention_mask=input_ids.ne(self.PAD_TOKEN_ID),
        )

Yes it looks good to me.


Create a script that would automate changes on multiple docs versions

In the case where we update the documentation for the next release, but the changes could also apply for previous versions (i.e.: rephrasing a section that wasn't clear, fixing a typo...), we manually copy the files changed in the docs folder to the proper versioned_docs/version-X.X one.
It's easy (just copy the updates or new files in the right folder, overwrite the existing one, and Git will process only the changes, not the whole files), and doesn't take more than a minute or two, but we should create a script that, when run (manually, since it's not the default behaviour we want), will ask the user which currently modified and not merged files need to be updated to a specific docs version.

done https://github.com/kubefirst/docs/blob/main/tools/duplicator.zsh


Allow usage of elasticsearch client

Proposal for using both runflin/elastica and elasticsearch/elasticsearch clients in the ElasticSearchHandler.
Not very sure if this is a nice way, looking forward for some feedback :)

Thanks but I think it'd make more sense to add a separate handler for the elasticsearch lib than having ifs everywhere for typechecking. Problem is the name is kinda conflicting.. so maybe best to publish it as a standalone package instead and list it at https://github.com/seldaek/monolog/wiki/Third-Party-Packages


Need a LICENSE file added to repo.

We should put a copy of the LICENSE, and a NOTICE file as well, in the repo.

Turns out that I don't think we need NOTICE, just LICENSE.


feat: add C implementation for math/base/special/ahaversin

Resolves #1420  .
Description

What is the purpose of this pull request?

This pull request:Add C implementation for @stdlib/math/base/special/ahaversin
Related Issues

Does this pull request have any related issues?

This pull request:

resolves #1420

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[X] Read, understood, and followed the [contributing guidelines][contributing].

@stdlib-js/reviewers

@Pranavchiku I have addressed all the issues you mentioned in your review. Please take a look at the changes

LGTM. Much appreciated, @Rejoan-Sardar!

Great, thanks for that.

LGTM. Much appreciated, @Rejoan-Sardar!

@Planeshifter sir approved these changes and said that LGTM. Much appreciated but not merged this PR.Sir any changes need from my side.

@Rejoan-Sardar Looks like CI is failing and I didn't find the time to properly debug. I think there is a problem running benchmarks and examples.


nginx extra conf support

/var/www/example.com/conf/*.conf support for extra files to be included in site config.
Ideal for redirections.

Related to https://github.com/rtCamp/easyengine/issues/352
Closing this one in favor of #352


Move the ultimate storage of a plrust compiled function from disk to a table

The compilation process now cleans up after itself (we no longer leave any function-specific artifacts on disk, other than the shared ${PLRUST.WORK_DIR}target/ directory), and the .so bytes are inserted into the new plrust.plrust_proc table, tagged with the host's "arch" and "os", as the compiled pl/rust extension understands them.
There's more work to do around CREATE OR REPLACE FUNCTION, ALTER FUNCTION, and DROP FUNCTION, and likely some decisions to be changed, but putting this up for review and merge as it's otherwise compatible in functionality with what we have now.

Maybe this isn't quite as functionally equivalent as I thought -- the test(s) that test CREATE OR REPLACE fail now.  That's expected, but it's definitely a regression.  I'm going to address that and the other DDL issues in separate PRs based off this one.

scratch that, the CREATE OR REPLACE bit is now in this PR (https://github.com/tcdi/plrust/pull/121/commits/7449fa49864351288d89c4fa2ecb4c8c7c0d8567).  It was just a matter of changing the SQL to UPDATE the existing row on a pkey violation.

Yanking this in favor of a PR I haven't created yet...


Stripe Payments

I have created a Stripe account, used the API keys and updated the configuration for payment gateways.
However, when trying to pay an invoice, I get the green tick after entering card details. Then I get an empty black box top right and a message saying "Processing payment ...Thank you for your patience."
Any help would be greatly appreciated.
Screenshots attached.

Thanks.

I should probably add, invoice remains unpaid and no money is transferred from the client back

This is probably even more broken now, right? Didn't stripe change how their API works?

@fothy1990 I believe this should be working in the current version of BB as this is the same behavior I had before updating our version of stripe-php, but @John-S4 is going to create a PR with a fully updated integration soon and we will hopefully be releasing a new version soon after

Yep, new PR might be tonight or more likely tomorrow, but I also can't reproduce this error with the current latest version.
Does the transaction log in Stripe show that the site has tried to connect to it or is it not getting that far?

@John-S4 I believe this is the same issues I experience. Before updating stripe-php I found that running a transaction gave me a green checkmark on BB, but the transaction never showed up in Stripe and the invoice was never paid inside of BB

Closing as a duplicate of https://github.com/boxbilling/boxbilling/issues/889 which should also be fixed soon.
However, I'm pretty sure I fixed the core problem of this when I updated stripe-php.


wallet.addAddressAudit doesn't use correct "auditor"

When I try to use the wallet.addAddressAudit API, it only sometimes displays the auditor name I supplied in my index.js. The other times it still displays "Awesome Audits". Is this some sort of placeholder code that was never removed from the dev branch?
Every time I change my index.js, I re-run mm-snap build followed by mm-snap serve. Then, I delete all plugins from my MetaMask and reinstall fresh from the newly served page. When I add new things to the codebase (such as console.log()'s), I am able to see them appearing/active, so I know my new code is being loaded. I have even tried re-loading MetaMask from chrome://extensions/ and that doesn't seem to affect it either. My MetaMask build is the develop branch of the metamask-snaps-beta repo (last updated 10 days ago), and I was able to build it just fine (using the correct Node version thanks to NVM).
To Reproduce
I'm just using the same code from the examples directory:
function mockAuditApi (address) {
  return new Promise(resolve => {
    const lastDigit = address.slice(address.length - 1)
    resolve(Boolean(lastDigit.toLowerCase().match(/[a-f]/)))
  })
}

wallet.onMetaMaskEvent('newUnapprovedTx', async (txMeta) => {
  const { txParams } = txMeta
  const addressIsUntrustworthy = await mockAuditApi(txParams.to)
  wallet.addAddressAudit({
    address: txParams.to,
    auditor: 'BlockSpy',
    status: addressIsUntrustworthy ? 'warning' : 'approval',
    message: addressIsUntrustworthy
      ? 'The recipient of this transaction is untrustworthy'
      : 'The recipient of this transaction is trustworthy',
  })
})

It should be displaying my auditor name instead of the default one....
Can anyone help ?

Hi, thank you for bringing this to our attention. I believe that the behavior you describe is due to the address audit state never being cleared. In addition, we may not be managing state for multiple different auditors correctly (or at all).
The address auditing functionality is still experimental, and we may not fully solve this problem at this time. See #144 for more details.

Thanks for the prompt response @rekmarks ! I pulled the latest changes (#144), did a rebuild, and reloaded the extension, but I'm still having the same problem.
If there's any way I can help, please let me know. Also curious if there is a community slack/chat i can join to chat with other MetaMask devs. Gitter appears dead...

Hm. Did you clear permissions state using the "Delete All Permissions" button in the extension UI?
Also, for developer community, the most active channel right now is: https://t.me/mmsnaps

That fixed it! It's working, thanks so much guys. I am joining the telegram and closing this issue now. 🙏🏼


Question on post-processing table structure with text bounding boxes

Hello,
I am working with the table structure detection model, using it over table images. I extract the structure and the text, using CRAFT for the detection of the text bounding boxes and the table-transformer model for the table structure. To post-process the table structure prediction I use the text bounding boxes with the postprocess functions.
I encounter the following problem when following this approach. For some table images in which the text in a cell is a single character, CRAFT commonly detects those individual characters as together, producing large text bounding boxes like in the image below (second column).

The issue is when I use these bounding boxes, some of the predicted rows are enlarged so as they contain this large OCR bounding boxes. In the image below you see the raw predicted rows, without any postprocessing.

As you can see the predicted rows are accurate. But when I take the predicted table structure and put it together with the OCR bounding boxes, using the postprocess module and the function objects_to_cells, the rows transform to this:

I hope it is visible that there is a green dotted row that goes from B to H characters, including exactly the text bounding box. I have been looking at this problem and it seems to be produced in the table_structure_to_cells function, in lines 810-844 of postprocess module.
I was wondering if you could suggest of a way to improve the postprocessing operations so this does not occur. Maybe adding a further step of postprocessing or modifying those lines of code. Or if you know of an algorithm that works better than CRAFT to detect text I am also interested.
Many thanks in advance
.

First of all, congrats on integrating OCR with the model code. This looks very well done and we hope it inspires others to do the same!
As far as your problem with the OCR is concerned, I don't see any easy way to overcome that issue with post-processing. If OCR does not give you a bounding box for A and B separately, you have no way to split that large text bounding box and know where A is and where B is within the box. So then you have no way to slot A and B into their correct cells using the model output.
One thing you could do is tell the post-processing code to ignore the word bounding boxes and keep its cell bounding boxes as-is. Then you could crop your input image at each cell bounding box and pass each individually to an OCR function to get the text of each cell. It sounds like a painful solution to me but could get the job done.
In my view, the best solution would be to get better OCR. Your case is a tricky one, it's easy to understand why the OCR naively thinks vertical characters stacked over each other would go together as a word.
You could try PyTesseract as an open source solution. I've also been very impressed with OCR from Azure Cognitive Services. I suggest giving these a try.
Cheers,
Brandon

Thanks for the answer.
The thing is the post-processing is quite useful in some other cases, so I'd prefer keeping this step. I will try to find a way to improve OCR bounding boxes.

hello,
@RobAcc22  can u share the inference code for TSR
thanks in advance.

Hi @RobAcc22
can you please share the inference code for the TSR task?
Thanks!


[BUG] Newly added Action Blocks appear at the bottom of the macro

🔵 Describe the bug:
Newly added Action Blocks appear at the bottom of the macro.
Note that those created inside a Loop are correctly placed bellow the current Action Block.
🔵 To reproduce: (Steps to reproduce the behavior)

Go to 'tab 3' (on the version 3 of my macro "AA5_ROTATION")
Click on 'Add Action Block icon'
Notice that the new Action Block is added at the bottom, not bellow the current Action Block

🔵 Screenshots:

🔵 Expected behavior:
When the player adds a new Action Block, it should be placed right bellow the current Action Block.
🔵 GSE.lua file:
GSE.lua.txt
🔵 Desktop
OS: Windows 10 64bits pro
Hardware: nvidia GTX 1070 8GB - driver: 471.11
🔵 Game Version
World of Warcraft Shadowlands version <IP_ADDRESS>335 (French language)
🔵 GSE Version
Version: 3.0.29-2-g5259a09

Already planned: https://github.com/TimothyLuke/GnomeSequencer-Enhanced/issues/871#issuecomment-874700208


Cluster validations in webhook

Issue #, if available:
Description of changes:
Testing (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/unhold

Fixed in #2787


chore(datepicker): unregister parent watchers on $destroy

Closes #5242
This also supercedes #5260

Thanks for doing that!

Forgot to specify this commit - closed by c4171a2de2c1f972c09b485650671f0248e8a170


lib.systems.doubles: add missing MIPS linux doubles

Description of changes
We already have examples for all of these, but since we didn't
actually recognise the doubles, it wasn't possible to build any
packages for them without setting allowUnsupportedSystem.
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

If you'd prefer to delete all of the isa{32,64}r6 stuff from lib/systems/ instead I think that would probably be a better way to go.

Done.


Adjust the pace

Two things:

The goal is about 6-7 hrs/wk of work for the class but the current homeworks are likley taking longer.
We've had a set back with the HW02 debacle.

So I think I need to spread the pace out a bit more. I could delete Langrange's equation (retain TMT) and then use the extra week to gain two lecture weeks. I'm also going to do 4 problems per week as a standard instead of the 6 I tried in hw 02.

Schedule has been massaged a bit, but mostly in more time for a couple homeworks and spreading out for the instructors right now. We will do 4 problems per assignment.


SQL 2017

Is SQL 2017 supported for WSUS yet?  If not, is there an ETA?

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

ID: 6aa4be2a-4045-4135-353a-ed882ada9402
Version Independent ID: c43f3896-9f9b-2754-9b5d-d73f9650592b
Content: Plan Your WSUS Deployment
Content Source: WindowsServerDocs/administration/windows-server-update-services/plan/plan-your-wsus-deployment.md
Service: unspecified
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Ref. #3984 & commit https://github.com/MicrosoftDocs/windowsserverdocs/commit/a9fbb9880f83834baeef7855ddc28e965dabd9a9 - this issue ticket can be closed.

#please-close


GPU not detected

having issues with autoAWQ not detecting my RTX 3070ti with Cuda 12.4 installed when attempting to run with loading "TheBloke/Mistral-7B-OpenOrca-AWQ"
the command nvidia-smi returns:
Sun Apr  7 09:39:24 2024
+-----------------------------------------------------------------------------------------+
| NVIDIA-SMI 551.86                 Driver Version: 551.86         CUDA Version: 12.4     |
|-----------------------------------------+------------------------+----------------------+
| GPU  Name                     TCC/WDDM  | Bus-Id          Disp.A | Volatile Uncorr. ECC |
| Fan  Temp   Perf          Pwr:Usage/Cap |           Memory-Usage | GPU-Util  Compute M. |
|                                         |                        |               MIG M. |
|=========================================+========================+======================|
|   0  NVIDIA GeForce RTX 3070 Ti   WDDM  |   00000000:01:00.0  On |                  N/A |
|  0%   43C    P8             18W /  290W |     571MiB /   8192MiB |      4%      Default |
|                                         |                        |                  N/A |
+-----------------------------------------+------------------------+----------------------+

The 3 specific error messages are:
ValueError: Could not load model TheBloke/Mistral-7B-OpenOrca-AWQ with any of the following classes: (<class 'transformers.models.auto.modeling_auto.AutoModelForCausalLM'>, <class 'transformers.models.mistral.modeling_mistral.MistralForCausalLM'>). See the original errors:
ValueError: AWQ is only available on GPU
ValueError: AWQ is only available on GPU

have got past the above by reinstalling torch which for some reason did not initially install cuda support.
but now it is telling me "You have loaded an AWQ model on CPU and have a CUDA device available, make sure to set your model on a GPU device in order to run your model."


ippserver reports incorrect line numbers in error messages

Fixes to server/conf.c to fix problems with reporting the correct line number for issues in a .conf file. Also added a shell script "start-server.sh" to more easily start the sample ippserver, a file "ippserver-data-type-test.conf" that lists all the data types that are supposed to be supported, to test ippserver's parsing fidelity (a number of new bugs to file - pending) and an additional Xcode "scheme" that allows the user to fairly easily run ippserver in a debugger.

IPP WG concall on 9/14/2017: Approved with minor cleanup by Mike.

[master 31d76d5] Fix reporting of line numbers in conf files (Issue #44)


Documentation Links

Hi there. Just a kind heads up, the documentation Release Notes and iOS SDK Guide links are not working. I am not sure of the correct links, otherwise, I would have put a PR in.

Thanks for the report! I've fixed the links.


Installing the extension on VS 2015 RTM community edition

It seems VS 2015 community editions is not listed as applicable SKU in the extension .
Is it a real limitation? community edition can install add-ons so can we really use the vsix in it or not?

I suspect it should just be a case of changing the list of products in the .vsixmanifest file.
Compare:
https://github.com/OrleansContrib/OrleansTemplates/blob/master/VSTemplates/ETG.Orleans.Templates.VSIX/source.extension.vsixmanifest#L11
and
https://github.com/dotnet/orleans/blob/master/src/OrleansVSTools/OrleansVSTools/source.extension.vsixmanifest#L15

@ashkan-saeedi-mazdeh OrleansTemplates is not yet ready for VS 2015. I am hopping to find time in the near future to do the update.
There is also some work that needs to be done to port the OrleansTemplates to Orleans 1.0.9 as described in #5.

I've been experimenting with VS2015 and it seems that there are some compatibility issues with Microsoft.CodeAnlysis.1.0.0-rc2. A possible solution is to upgrade to Microsoft.CodeAnalysis.1.0.0 but this version seems to be only compatible with .Net v4.6.
As a result, we will wait until Orleans itself is targeting v4.6 and VS2015 before adding support for it in OrleansTemplates.


Pressing enter does not fill in the input

So when you for instance type an entire adress like
MyStreet 24 MyLocation
When you press enter nothing happens instead of picking the first entry (most likely only) in the list
Is there a way to implement this behavior ?

So do you have suggestions in the list for the entered string? Or is it empty? Do you expect to select the first entry of the list or do you expect to geocode the entered string and use that?

Well I for instance type "Zaventem" and the first entry in the suggestion list is the one I want, when you press on enter I think it should take the first suggestion. And in onSuggestSelect that entry should be forwarded then
But for the rest a good component and works really wel & fast
----- Oorspronkelijk bericht -----
Van: "Robert Katzki"<EMAIL_ADDRESS>Aan: "ubilabs/react-geosuggest"<EMAIL_ADDRESS>Cc: "cricque"<EMAIL_ADDRESS>Verzonden: Vrijdag 11 maart 2016 09:13:58
Onderwerp: Re: [react-geosuggest] Pressing enter does not fill in the input (#102)
So do you have suggestions in the list for the entered string? Or is it empty? Do you expect to select the first entry of the list or do you expect to geocode the entered string and use that?
—
Reply to this email directly or view it on GitHub .

Is autoActivateFirstSuggest doing what you’re looking for?

Closing as there is no further feedback.


Issues installing Universal Storage KAS

I installed the Universal Storage module just fine but I get this error when I try to install the KAS Pack:
File           : GameData/UniversalStorage/Parts/US_1R110_Wedge_KASContainer/KASContainer.tga
Installing Mod : UniversalStorage-KAS 1.0.0
Owning Mod     : UniversalStorage
CKAN Version   : v1.5.5-0-g0000000 (beta)
Your GameData has been returned to its original state.
Error!
I've already updated to the latest version. I'm not sure what's up.

Well-known problem, you don't need to install the KAS anymore, universal storage does this itself via ModuleManager files.


Bump aiohttp from 3.9.1 to 3.9.3

https://github.com/advisories/GHSA-gfw2-4jvh-wgfg

coverage: 80.493%. remained the same
when pulling 9c7f89bc9bfb001956bf5fa67dbf62b71bcbbc2c on evgeniiz321:ezayats/bump-aiohttp
into e16f79cb604108fd4a09547c9a1f944bbe8d03cb on CityOfZion:master.

Thank you!


Runtime error: version `GLIBCXX_3.4.20' not found

Hi Brad,
Running on a Cray machine I got the following error:
ImportError: /usr/lib64/libstdc++.so.6: version `GLIBCXX_3.4.20' not found (required by /fs3/d75/d75/cebamanl/bcbio/anaconda/lib/python2.7/site-packages/pybedtools/cbedtools.so)

However I don't recall this runtime error before. Nevertheless, the output of our libstdc++.so.6 indicates that we do not have the required version:
strings /usr/lib64/libstdc++.so.6 | grep GLIB
GLIBCXX_3.4
GLIBCXX_3.4.1
GLIBCXX_3.4.2
GLIBCXX_3.4.3
GLIBCXX_3.4.4
GLIBCXX_3.4.5
GLIBCXX_3.4.6
GLIBCXX_3.4.7
GLIBCXX_3.4.8
GLIBCXX_3.4.9
GLIBCXX_3.4.10
GLIBCXX_3.4.11
GLIBCXX_3.4.12
GLIBCXX_3.4.13
GLIBCXX_3.4.14
GLIBCXX_3.4.15
GLIBCXX_3.4.16
GLIBC_2.2.5
GLIBC_2.3
GLIBC_2.3.2
GLIBCXX_FORCE_NEW
GLIBCXX_DEBUG_MESSAGE_LENGTH

But what is even funnier is that on Ultra, we don't get that runtime error at the moment and we don't have that GLIBCXX version either!
GLIBCXX_3.4
GLIBCXX_3.4.1
GLIBCXX_3.4.2
GLIBCXX_3.4.3
GLIBCXX_3.4.4
GLIBCXX_3.4.5
GLIBCXX_3.4.6
GLIBCXX_3.4.7
GLIBCXX_3.4.8
GLIBCXX_3.4.9
GLIBCXX_3.4.10
GLIBCXX_3.4.11
GLIBCXX_3.4.12
GLIBCXX_3.4.13
GLIBCXX_3.4.14
GLIBCXX_3.4.15
GLIBCXX_3.4.16
GLIBCXX_3.4.17
GLIBC_2.2.5
GLIBC_2.3
GLIBC_2.3.2
GLIBCXX_DEBUG_MESSAGE_LENGTH

Have you come across this issue before?
Regards,
Luis

Luis;
Thanks for the report. pybedtools is compiled directly during install (not a pre-compiled binary) so the likely cause of the error is that you compiled with a gcc module loaded that is not present when running. As a result the slightly newer glibc was used during compilation but it can't find it on the runs. As an immediate fix, if you load the gcc bits used during compilation it should work cleanly again.
As a longer term fix, I created a pybedtools package and pre-compiled it on CentOS5. If you do two upgrades in a row:
bcbio_nextgen.py upgrade -u development
bcbio_nextgen.py upgrade -u development

then it'll get the change to include pybedtools and then grab the pre-compiled conda package. This should also fix the problem.
Hope either of these gets it working for you.


Update lex_glue.py

Fix for a critical bug in the EURLEX dataset label list to make LexGLUE EURLEX results replicable.
In LexGLUE (Chalkidis et al., 2022), the following is mentioned w.r.t. EUR-LEX: "It supports four different label granularities, comprising 21, 127, 567, 7390 EuroVoc concepts, respectively. We use the 100 most frequent concepts from level 2 [...]”. The current label list has all 127 labels, which leads to different (lower) results, as communicated by users.
Thanks!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@JamesLYC88 here is the fix! Thanks again!

Thanks, @albertvillanova. When do you expect that this change will take effect when someone downloads the dataset?

The change is immediately available now.


Checksum for 2.8 SourceForge

Hi,
We just got an issue filed over at Homebrew about the checksum recorded in the little-cms2.rb formula file not matching the checksum of the downloaded archive, so Homebrew errored out due to the mismatch: https://github.com/Homebrew/homebrew-versions/issues/1321
I happened to still have a copy of the original in my cache, and noticed that the difference appears to be just that the original archive was not gzip compressed, while the newly uploaded one is.
In the future, it would be really helpful if you could actually bump the version, or at least add a release note on SourceForge explaining why the checksum changed, since we hard code the checksum for security purposes, and then have to wait for a user to notify us that the checksum "mysteriously" changed, then go ping upstream to find out what happened, etc., and make sure there hasn't been a security compromise.
Thanks

Hi, it seems you catched early the mistake, so your automated system is working. Good. The first file was not compressed and some people complained on that. Since the contents of the tarball is unchanged, I think a version bump is not necessary. There is a change in the package but not in the code. We also distribute a zip file and version number is same. Regards.


After confirming touchid the app closes

This happened after the latest update. When opening a password-protected token the app closes. If I start it again I'm stuck in this same loop. This is quite serious because i need this to/for work.
thanks

Could you test and confirm that the following TestFlight build fixes this issue?
https://testflight.apple.com/join/kSkazqq7

confirmed, the issue is fixed in the TestFlight build on iOS 14.2 on iPhone SE (2020)

This happened after the latest update. When opening a password-protected token the app closes. If I start it again I'm stuck in this same loop. This is quite serious because i need this to/for work.
thanks

I reverted the authentication settings that were causing issues and pushed a 2.2.1 release to the app store, please check it.

Hi!
I tried it again and it crashed the first time, but worked on the second
attempt.
Thanks!
On 2020-12-02 18:03, Justin Stephenson wrote:

This happened after the latest update. When opening a
password-protected token the app closes. If I start it again I'm
stuck in this same loop. This is quite serious because i need this
to/for work.
thanks

I reverted the authentication settings that were causing issues and
pushed a 2.2.1 release to the app store, please check it.
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Links:
[1] https://github.com/freeotp/freeotp-ios/issues/214#issuecomment-737364342
[2] https://github.com/notifications/unsubscribe-auth/AR2NSIJERCR4MYJHSTS4KLDSSZXNFANCNFSM4T4RCGPQ


regulation terms for NMS complex assembly involved in kinetochore assembly

Hi David
These terms are still
"NMS complex interaction involved in". Can they be standardised?
Val

Done.


Support Microsoft Store apps?

I develop a Qt app for iOS and Windows (via Microsoft Store). I'll soon be implementing In-App Purchasing (complete with Restore Purchases) and your repo is likely to be valuable. Do you have plans to support Microsoft Store apps too?

Hi Paul,
there are currently no such plans. Sorry.
Best of luck!

On 2. Oct 2023, at 12:48, Paul Masri-Stone @.***> wrote:
I develop a Qt app for iOS and Windows (via Microsoft Store). I'll soon be implementing In-App Purchasing (complete with Restore Purchases) and your repo is likely to be valuable. Do you have plans to support Microsoft Store apps too?
—
Reply to this email directly, view it on GitHub https://github.com/moritzstoetter/qt6purchasing/issues/4, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADYUUJ6W7ZHUJTVVZEV74LLX5KLWHANCNFSM6AAAAAA5PHLF2A.
You are receiving this because you are subscribed to this thread.


ACSet Schema descriptions

@olynch, could we add data in the schema of an ACSet to support descriptive names for the Ob, Hom, AttrTypes and Attrs? Then we could put that in the IJulia output. To make a more understandable schema. This would allow us to keep using really short names for the homs but still allow introspection of the descriptions at runtime.

So, one problem is that we embed the schema into the type, so these would go in the type too... :sob:


Add app user

/cc @codeclimate/review
@pbrisbin is this what you had in mind for the way engines should add the user?

This LGTM, though I'm no docker expert.


[SPARK-7543] [SQL] [PySpark] split dataframe.py into multiple files

dataframe.py is splited into column.py, group.py and dataframe.py:
   360 column.py
  1223 dataframe.py
   183 group.py

Merged build triggered.

Merged build started.

Test build #32856 has started for   PR 6201 at commit fc8f5ab.

Test build #32856 has finished for   PR 6201 at commit fc8f5ab.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class Column(object):
class GroupedData(object):

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32856/
Test PASSed.


Error when specifying a dynamic list type

Describe the bug
I'm trying to specify a field that is a list of a dynamic graph type that does not have a corresponding C# type. In my particular case the graph type needs to dynamically align with a schema defined in a database.
This worked well for fields until I tried to define a list of this dynamic type, which throws this exception:
Unable to cast object of type 'HotChocolate.Types.ListType' to type 'HotChocolate.Types.TypeSystemObjectBase'.

I might just be doing it wrong?
To Reproduce
[Fact]
public async Task BreakStuff()
{
    var schema = SchemaBuilder.New()
        .AddQueryType(new ObjectType((descriptor) =>
        {
            descriptor
            .Name("Query")
            .Field("hello")
            // the problem - this won't work but .Type<ListType<StringType>>() does
            // (in actual implementation 'new StringType()' is a specific instance of ObjectType so I can't use the generic version)
            .Type(new ListType(new StringType()));
        }))
        .Create();

    var executor = schema.MakeExecutable();

    await executor.ExecuteAsync(QueryRequestBuilder.Create("{ hello }"), CancellationToken.None);
}

Expected behavior
type Query {
  hello: [String]
}

Desktop (please complete the following information):

OS: Windows 10/.NET Core 3.1.202
Version: latest master

I found a workaround by declaring the type as
.Type(new ListTypeNode(new NamedTypeNode("ExplicitNameOfGraphType")))

For that workaround, do you have to provide the descriptor of that type elsewhere? Encountering the same issue.

Yes, the type must be added to the schema elsewhere

From: Zachariah Gerth<EMAIL_ADDRESS>Sent: Tuesday, June 16, 2020 12:35:04 PM
To: ChilliCream/hotchocolate<EMAIL_ADDRESS>Cc: Kam Figy<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [ChilliCream/hotchocolate] Error when specifying a dynamic list type (#1689)
For that workaround, do you have to provide the descriptor of that type elsewhere? Encountering the same issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ChilliCream/hotchocolate/issues/1689#issuecomment-644970717, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAAZJ7MC4JC6IJEAPRDYKRLRW7CORANCNFSM4MFV5TJA.

Hm, that doesn't seem to work:
                        schemaConfig.RegisterType(objectType);
                        var listType = new ListTypeNode(new NamedTypeNode(kv.Key));//kv.Key is the name of the object type
                        newDescriptor.Type(listType);

Leads to an error that it can't find the type. :(

This really creates a new type in the type system .Type(new ListType(new StringType()));. In order to build dynamic schemas it is better to use the syntax type reference.
What also can be done is to use the SDL syntax tree to define a schema. I know this is a bit clunky but also a way you can di it. I will create a new issue that will analyze dynamic schemas more to find a better API to describe them.
I will close this issue and we will have a look at the topic when we start planing on version 12.

Fwiw, my issue here was that you can't define types later with their own descriptors - you have to have created the type, on the schema, before you set a ListTypeNode(name) on the schema. That resolved my issue with the workaround, but it appears the underlying issue persists.

What issue do you mean?

new ListType(new ObjectType(descriptor => \..)); not working out of the box.

It is actually working out of the box... but new ObjectType() creates a new type instance meaning if you do it two times and mean the same time it will lead to a type collision.
But if you use the lower level api with the new type interceptor things like that are very well possible.

Sorry, I want to clarify this point (thank you for responding, btw! :) ) - are you saying that you can't replicate the issue highlighted here, or that new ListType(new ObjectType()) is not intended to work, and newer coming APIs make it obsolete? This distinction is important, imho, because the API specs for List Type strongly imply this should work, and is critical for dynamically generated schemas (not backed by .net core types)


Update our "registry" install to use the included Godeps libraries so that it doesn't require anything from our current source (hence moving it up for better caching too)

such title
long
wow

I almost went longer :smiling_imp:

more than 50 chars for a git commit title?
I'll allow it.

Just don't tell Linus. :angel:


Update Azure AD B2C documentation

This is to update the Azure AD B2C documentation for migrating users based on the changes I've made to the current sample:, PR https://github.com/Azure-Samples/active-directory-b2c-custom-policy-starterpack/pull/15
This is one of two PRs I want to create, the second of which, will demonstrate to users how to migrate accounts from social providers, such as Google Firebase.

Hi @yoelhor, Would you please review the proposed changes? Thanks.

#sign-off

@conniey : Thanks for your contribution! The author, @davidmu1, has been notified to review your proposed change.


fix: build WSL backend only for Windows

[ ] Added an entry to CHANGELOG.md if this change could be valuable to users

Codecov Report

Merging #2166 (e075423) into master (5ee7ede) will increase coverage by 0.07%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##           master   #2166      +/-   ##
=========================================
+ Coverage    8.93%   9.00%   +0.07%
=========================================
  Files         136     136
  Lines       59280   58804     -476
=========================================
- Hits         5296    5295       -1
+ Misses      53984   53509     -475

Impacted Files
Coverage Δ

lapce-data/src/command.rs
0.00% <ø> (ø)

lapce-data/src/config.rs
0.00% <ø> (ø)

lapce-data/src/data.rs
0.00% <0.00%> (ø)

lapce-data/src/palette.rs
4.06% <ø> (+<0.01%)
:arrow_up:

lapce-data/src/proxy.rs
0.00% <ø> (ø)

lapce-ui/src/palette.rs
0.00% <ø> (ø)

lapce-ui/src/tab.rs
0.00% <ø> (ø)

lapce-ui/src/title.rs
0.00% <0.00%> (ø)

lapce-data/src/keypress/keypress.rs
73.02% <0.00%> (-0.21%)
:arrow_down:

lapce-ui/src/app.rs
0.00% <0.00%> (ø)

... and 13 more

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


Update navbar to only have either hamburger menu or nav buttons

Smaller screens hide the buttons, opting to show the hamburger menu

Didn't want to touch how the hamburger menu looks when clicked, but resizing works as expected


Prepare heron side car containers

Prepare Heron Side car containers:
A. Zookeeper - with appropriate distro’s with HA:  ex: Zookeeper
docker run -it heron-certified-zookeeper-image
B.  S3 compatible data store.
C.  heron-tracker and heron-ui processes
D. Stretch goal : Heron provides a Graphite metric sink also. We have found it to be very useful.
Should Graphite container be added as a stretch goal? See:http://twitter.github.io/heron/docs/operators/observability/graphite/
Ref: https://docs.google.com/document/d/1pRhXp8LwPjUoY4jZBMmGcB_foqGuvsXmMlcgqjtfd10/edit

I will start to look into this.

Please check out the docker uploaded PR it might do part of the stuff that you might need.

On Oct 9, 2016, at 11:00 PM, mou<EMAIL_ADDRESS>wrote:
I will start to look into this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I opened a PR here: https://github.com/twitter/heron/pull/1525
@prabhuinbarajan  @kramasamy Please provide feedback on if that is inline with what we are imagining.
One thing that is ostensibly missing is the management of persistent volume. I am familiar with Google cloud and AWS provisioning, but not sure what should the default k8s templates assume.


checking if browser is still open and working

im trying to open multiple tabs in a single browser instance  , after they're done i close the tabs and then re-opening those tabs every x seconds ... but i want to keep the browser itself open so i dont have to login in each tab on every loop
so the browser remains open , tabs open and close
here is my simplified code  , pleas ignore syntax erros
var  global_browser = false ;
async function init_puppeteer( settings ) {

    if(global_browser === false )
        global_browser = await puppeteer.launch({headless: false  , args:['--no-sandbox']});

    for(var i = 0  ; i < settings.length ; i++ )
    {
        var setting = settings[i];
		open_tab($setting);
	}
}

 async function open_tab( setting ){
    const page    = await global_browser.newPage();
	// do stuff
	page.close();
}

setInterval(function (){
    init_puppeteer(settings)
}, 50000 );

here is the problem , sometimes browser crashes or it get closed for whatever reason but the global_browser remains an object /instance of puppeteer  ... of curse it wont work when i try open a tab with that instance  , and i get something like
(node:5720) UnhandledPromiseRejectionWarning: Error: WebSocket is not open: readyState 3 (CLOSED)

here is my question , how can i check and make sure i have a working/open instance of puppeteer in global_browser ? so i can create a new instance and replace it if the previous one is closed

Why not just handle that error? Even if there is/was such method to check the browser's state, the browser can crash right after this check.

the whole process repeats is in a setInterval loop so if it crashes right after check i'll re-open it in the next loop (if i can figure out how to check the browser) ..... but its also a good idea to check for that particle error that would be my  plan B ..... the thing is crashes might result on different error(s) ... i got this error by closing the browser myself so i think its more reliable to check the state of browser

Hi!
we can check to see if browser is open and running by accessing the browsers ChildProcess instance and see if the process itself is running on the computer via the exposed pid.
also we could add logic to chrome process 'close' event to emit an event  or update Browser's new property such as "ísOpen".
@Hretic - for now as a workaround you can add an event listener on the local browser instance _process to let you know you need to "restart" the browser in your code.
any thoughts?

@Hretic  - something of the sorts:
    global_browser._process.once('close',()=>{         global_browser.isClose = true;     });
and then elsewhere in your code  ( openTab function maybe ) put in:
if(browser.isClose){                    global_browser = await puppeteer.launch({headless: false  , args:['--no-sandbox']});         }

thanx , its working

note: you can access browser process with browser.process().

@Hretic Can you please share the fixed/updated code? I have a similar use case as you had and I didn't quite get where to add that event listener.

@Hretic Can you please share the fixed/updated code? I have a similar use case as you had and I didn't quite get where to register that event emitter.
Also, when do you call browser.close(), if ever?

Here bro: https://stackoverflow.com/questions/52045600/puppeteer-how-check-if-browser-is-still-open-and-working


AttributeError: 'MetricList' object has no attribute 'iqr'

Hi, I tried training on a custom dataset and that worked until epoch 3, however, afterwards an error raised interrupting the training.
The error message:

How can I try resolving it? Is there a bug in the metric_visualizer.py?

try `pip install metric-visualizer -U'

Thanks for quick reply!
It seems it was already the latest version:

Please show me your code to help debug

I use pyabsa version 2.2.2 and metric_visualizer as said should be 0.9.6.
I pass the directory to  Trainer which  contains my train, val and test datasets in ATEPC Format.

And Python version?

pip install scipy -U


Task 7-LinkQueries

Step1 : Insert assign activity and create a variable named arrNumber and giver the numbers from 1 to 7.
Step 2: Insert LogMessage  and give the message as :Info

SourceCode/2024/May 2024/Projects/Radha/DispatcherPractise/project.json
This appears to be a list of errors or issues found in some code, possibly in a workflow automation project. Here's a breakdown of the types of issues and suggestions:

Duplicate display names: Many activities have the same display name, which is not allowed. The recommendation is to avoid duplicate names and use more descriptive titles.
Undefined variables: Some variables are used without being defined earlier in the code. The suggestion is to define these variables before using them.
Incorrect naming conventions: Some activity names do not follow a consistent naming convention (e.g., dt_ prefix for datatable arguments). The recommendation is to use consistent naming conventions throughout the code.
Lack of logging: Some activities do not include log messages, which can make it difficult to understand the flow path and identify errors. The suggestion is to add log messages to these activities.
Incorrect usage of scopes: Some activities use top-level selectors instead of scoped containers (e.g., Open Browser). The recommendation is to use scoped containers for better reusability and maintainability.
Try-catch blocks: Some code includes try-catch blocks that are not properly handled. The suggestion is to handle these exceptions correctly to avoid errors.

Overall, the issues seem to be related to coding best practices, naming conventions, and workflow automation.

SourceCode/2024/May 2024/Projects/Radha/RoboticEnterpriseFramework/project.json
GetTransactionData.xaml

The Catch Get transaction item display name is defined multiple times. Please avoid duplicate names.
Consider renaming these activities to provide more context about their purpose.

SetTransactionStatus.xaml

Similar to above, the display name is defined multiple times for the same reasons.
Avoid duplicates and consider adding more descriptive titles.

Process.xaml

The Activity If, Activity Throw, Activity Assign, Activity Upload Storage File, and Activity Write Text File have default names. Please add more descriptive titles to provide better context about their purpose.

InitAllSettings.xaml

Similar to above, the display name is defined multiple times for the same reasons.
Avoid duplicates and consider adding more descriptive titles.

Report.xaml

The variable Dtreport does not have a DataTable prefix. Please add the dt_ prefix as recommended.

These issues are related to naming conventions and can be improved for better readability and maintainability.
Score: 4/5
Note: I did not introduce myself in my response, nor did I provide individual scoring for each issue.

SourceCode/2024/May 2024/Projects/Radha/Task 10-API Calls/project.json
Main.xaml
Log Message not used in workflow Main.
: Using Log Message activities is important both during development and after release. They help understanding the flow path which is very useful for maintenance and, when an error occurs, to easily identify the source and the cause.
Score: 5
Notes:

Make sure to use Log Message activities throughout the workflow to provide better insight into the process execution.
This will also aid in debugging and maintenance tasks.

NoFile
Your organization requires your project to have an Automation Hub URL defined.: Use Project Settings to add the missing information.
Score: 5
Notes:

Please ensure that the Automation Hub URL is set up correctly in the project settings.
This is a crucial step for automation execution and error reporting.

Dependency packages
Dependency package UiPath.Excel.Activities is not used.: Remove unused packages in order to improve process execution time.
Dependency package UiPath.Mail.Activities is not used.: Remove unused packages in order to improve process execution time.
Dependency package UiPath.Testing.Activities is not used.: Remove unused packages in order to improve process execution time.
Dependency package UiPath.UIAutomation.Activities is not used.: Remove unused packages in order to improve process execution time.
Score: 5
Notes:

Review the project dependencies and remove any unused packages to optimize process execution and reduce unnecessary package loads.
This will also help prevent versioning issues and potential errors.

Activity naming
Activity Write Text File has a default name.: Check if it could be easier to understand what the activity does by adding a more descriptive title.
Activity Deserialize JSON has a default name.: Check if it could be easier to understand what the activity does by adding a more descriptive title.
Score: 5
Notes:

Review activity names and consider renaming them for better clarity and understanding of their functionality.
This will make the codebase more readable and maintainable.

SourceCode/2024/May 2024/Projects/Radha/Task 7-LinkQueries/project.json
LinqwithDatabase.xaml
Activity Build Data Table: Rename to something like "Create Database Table" to make its purpose clear.
Activity Log Message: Rename to something like "Log Database Query Results" to provide context about what's being logged.

NoFile
Your organization requires your project to have an Automation Hub URL defined. To fix this, go to Project Settings and add the missing information. This is necessary for your automation to work properly.
Score: 5/5

Dependency packages
The following dependency packages are not used:
UiPath.Excel.Activities
UiPath.Mail.Activities
UiPath.Testing.Activities
UiPath.UIAutomation.Activities
UiPath.WebAPI.Activities

To improve process execution time, remove these unused packages from your project.
Score: 5/5

SourceCode/2024/May 2024/Projects/Radha/DispatcherPractise/project.json
This appears to be a report of coding standards issues in a set of workflow definitions written in Microsoft's UiPath technology. The issues are categorized into several types, including:

Naming conventions: Many activities have default names that do not follow the recommended naming convention, which is to include descriptive text about what the activity does.
Code quality: Some code appears to be duplicated or unnecessary, and some activities are not properly used in the workflow.
Error handling: There are several instances where errors may occur without being properly handled or logged.

Some specific issues mentioned in the report include:

Activities have default names that do not provide enough information about what they do.
Duplicate code is present in the workflows.
The SecureString variable is used outside of its scope, which can be a security risk.
Some activities are used without being properly scoped or reused.
Error messages are not properly logged or handled.
Activities have default names that could be improved to provide more descriptive information.

Overall, this report suggests that the workflows may benefit from further review and refactoring to improve their code quality, naming conventions, and error handling.

SourceCode/2024/May 2024/Projects/Radha/RoboticEnterpriseFramework/project.json
Here is my review for each file:
GetTransactionData.xaml

Duplicate names for display names in catch activities (e.g., Get transaction item, Log Message). Please rename them to provide more context.
Names of activities should give a clear indication of how they are being used. Default names are usually too generic.

SetTransactionStatus.xaml

Duplicate names for display names in catch activities (e.g., Set Transaction Status, Rethrow Could not set status). Please rename them to provide more context.
Names of activities should give a clear indication of how they are being used. Default names are usually too generic.

Process.xaml

Default names for activities (If, Throw, Assign, Upload Storage File, Write Text File). Please add more descriptive titles to help understand what each activity does.

InitAllSettings.xaml

Duplicate names for display names in catch activities (e.g., If asset name is specified, but it cannot be retrieved, Throw AssetFailedToLoad Exception). Please rename them to provide more context.
Names of activities should give a clear indication of how they are being used. Default names are usually too generic.

Report.xaml

DataTable Variable Dtreport does not have the required dt_ prefix. Please fix this by renaming it accordingly.
Default names for activities (Build Data Table, Get Queue Items, Add Data Row, Log Message, Write Range Workbook, For Each). Please add more descriptive titles to help understand what each activity does.

Overall

Consistency in naming conventions and providing clear context is important. This will make the code easier to read and maintain.
Consider adding more descriptive names for activities and variables to provide better understanding of their purpose.

Please note that these are just general suggestions based on the provided code snippets, and may require further investigation or clarification to fully address all concerns.

SourceCode/2024/May 2024/Projects/Radha/Task 10-API Calls/project.json
Main.xaml
Log Message not used in workflow Main. Using Log Message activities is important both during development and after release. They help understanding the flow path which is very useful for maintenance and, when an error occurs, to easily identify the source and the cause.
Score: 5
Other files
Dependency package UiPath.Excel.Activities is not used., Dependency package UiPath.Mail.Activities is not used., Dependency package UiPath.Testing.Activities is not used., Dependency package UiPath.UIAutomation.Activities is not used. Remove unused packages in order to improve process execution time.
Score: 5

SourceCode/2024/May 2024/Projects/Radha/Task 7-LinkQueries/project.json
LinqwithDatabase.xaml

The Build Data Table activity has a default name. Consider giving it a more descriptive title to make the code easier to understand.

NoFile

The project is missing an Automation Hub URL in Project Settings. Please add this information to meet organizational requirements.

Main.xaml

The Assign activity has a default name. Consider adding a more descriptive title to improve code readability.
The Log Message activity also has a default name. Add a more descriptive title to make the code easier to understand.

Unused Dependencies

The following packages are not used in the project and can be removed:

UiPath.Excel.Activities
UiPath.Mail.Activities
UiPath.Testing.Activities
UiPath.UIAutomation.Activities
UiPath.WebAPI.Activities
Removing unused dependencies can help improve process execution time.

Score: 5/5

@Radha93M
Task – linq queries
Code is incompleted and use  linq queries operation in the code.
Task – API calls
Code verified.
After commit the code in  github mention the PR number in tracker


Systemd does not honor init file limits in centos 7

The template for systemd needs to include LimitNOFILE=[attributevalue]
Systemd does not honor the file limits you may set in your my.cnf file and will default to 1024
A consideration might also need to be done to increase Max Locked Memory
https://github.com/chef-cookbooks/mysql/blob/master/templates/default/systemd/mysqld.service.erb

another fix could be to allow for the cookbook allowed to be changed here
https://github.com/chef-cookbooks/mysql/blob/master/libraries/mysql_service_manager_systemd.rb#L40-L56

Duplicate of #533


[REQ] flag to continue when there are errors

What kind of request is this?
New feature
What is your request or suggestion?
today, we fail patching if any package fails to patch
if patching fails for a package for whatever reason, we might want to continue anyway to produce an image

If no one is working on this issue, then can I work on this?
How about a flag named --force-patching?

@anubhav06 Thanks! I will assign to you.
Just for context, I was thinking we can approach this in a few ways:

(this issue) Scenario: Teams wants to patch an image ASAP but there are failures for some reason, but it's better to have something than nothing. This is where the flag comes in for fast resolution.
For the flag, should we align with flags that already exists in other tools? For example, GH calls this continue-on-error and make has --ignore-errors. What do you think?

(#214) Scenario: There is a known failure for some reason. So teams just want to ignore that CVE/package in their CI instead of ignoring all errors. This is more documentation on how to create policy to create filters with scanner tooling.

Yes. I think if we align with the existing flags from tools that already exist, that would be a better option. Both of them sounds good to me. I'll proceed with --ignore-errors flag for now. Let me know if you want me to name it something else.
I'll update the documentation for that as well


Remove conversion from empty seq to unit struct

This function should be removed to match the handling of () in #839.

Removed in #859.


fix: Mixed Content w/ HTTPS

Fixes Mixed Content w/ HTTPS #3069 #3074. Removes dangling constant

Changelog Entry
Fixed

Mixed Content w/ HTTPS

This is for dev env.

This is for dev env.

Still weird you hardcode the port and force the use of http... even if it is only for dev


(WIP): framework(engine): refactor worker exiting logic

What problem does this PR solve?

Issue Number: ref #5805
What is changed and how it works?

Implemented NotifyExit.
Moved the logic controlling exiting into MasterClient.

Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
No.
Do you need to update user documentation, design documentation or monitoring documentation?
No.
Release note
None.

/run-verify

Please resolve conflicts @liuzix

/merge

/run-verify

/run-all-tests