Support AliasGenerator dataclass for model config

API ex:
from pydantic import AliasGenerator, BaseModel, ConfigDict, Field

class Foo(BaseModel):
    a: int
    b: str

    model_config = ConfigDict(
        alias_generator=AliasGenerator(
            validation_alias=lambda field_name: f'validate_{field_name}',
            serialization_alias=lambda field_name: f'serialize_{field_name}',
        )
    )

Hi @jusexton,
Your interest is greatly appreciated!
I've already made some significant progress on this PR (will put it up shortly). I'd love your feedback on the PR once I get it up, though (should be in the next hour or so 👍). I'll make sure to push up a draft earlier next time to avoid confusion re if it's being worked on or not.
We'd love your support on some of our other feature requests or bug fixes, if you're interested. I've marked some recently with help wanted. If one isn't marked as such and you don't see a PR yet, feel free to ping me and pick it up! 🚀

Awesome, no problem! I didn't expect you to jump on this so quickly. I'll be sure to take a look.


Can modify color and font styling in each Pie at Pie-Chart

Hi.
I just wanted to change each pie's color and set background transparent.
so I modified some codes about things

adjust color on each pie
adjust color on each pie's legend font
alignment -  left padding and adjustment of offset x.

I applied pull request so please check this out.
Thanks.

thanks, I'll review later today

Congrats on the merged, thanks again!


M&Ms dataset

Great work！ but I noticed that the source of the M&Ms dataset is not provided in the paper. Could you please provide the download link for the M&Ms dataset?

thank you for your attention！ The M&Ms dataset is provided in the fourth cited paper, and the link is https://www.ub.edu/mnms/

Great job! I've been reproducing your essay recently, and have been reading the issues you and others have discussed. When reproducing the third dataset, it was a bit off from the metrics in the third dataset of your paper, using the shared dataset, and I wanted to ask if there were any other versions of the M&M's dataset that you were training on at the time? Can you give any advice, because the labeled domain learns high dice values, but the unlabeled domain doesn't seem to be learned, and the results are generally low, do I need to go to any specific place to focus on? Also what is the difference between dice and dc within the metrics that come out of the test results, as the two values are very close. Finally, if there are other versions of the M&M dataset, could you please share the M&M dataset again, thank you very much.

Thank you for your attention to our work. Apart from the publicly available datasets, we did not use any other versions of the M&Ms dataset during training. Regarding the issue where the unlabeled domain seems not to be learned, you can log the quality of the pseudo-labels for the unlabeled samples in each domain in real-time. This would allow you to observe whether there is consistently no learning (including unlabeled samples from the same domain as the labeled ones) or if overfitting occurs in the later stages of training. Dice and DC are the same metric; Dice is defined separately, while DC is calculated using the medpy package. You can simply refer to the DC results.

Thank you for your attention to our work. Apart from the publicly available datasets, we did not use any other versions of the M&Ms dataset during training. Regarding the issue where the unlabeled domain seems not to be learned, you can log the quality of the pseudo-labels for the unlabeled samples in each domain in real-time. This would allow you to observe whether there is consistently no learning (including unlabeled samples from the same domain as the labeled ones) or if overfitting occurs in the later stages of training. Dice and DC are the same metric; Dice is defined separately, while DC is calculated using the medpy package. You can simply refer to the DC results.

Thank you very much for your reply, at the moment it feels like there is a problem with the data enhancement part, but the data enhancement calls a function inside the environment. I would like to ask you the version of python, torch and cuda you were training on at the time


ibtoolzip needs better output when run with -s or --verbose_failures

Right now ibtoolzip doesn't actually show the invocation of ibtool when run with -s or --verbose_failures.
/usr/bin/java -jar bazel-out/host/bin/tools/objc/ibtoolzip_deploy.jar bazel-out/local_darwin-fastbuild/bin/examples/objc/PrenotCalculator/examples/objc/PrenotCalculator/CalculatorViewController.nib.zip CalculatorViewController.nib /Applications/Xcode.app/Contents/Developer/usr/bin/ibtool --minimum-deployment-target 7.0 examples/objc/PrenotCalculator/CalculatorViewController.xib)
is the output, where I would expect to actually see the ibtool invocation.

but that is the ibtool invocation, no?  It says
/usr/bin/java -jar bazel-out/host/bin/tools/objc/ibtoolzip_deploy.jar
that is an invocation of a program written in Java.

agreed, but someone looking at the output trying to debug what is wrong is actually going to want to see  what was passed to ibtool under the hood. Just because bazel is calling ibtoolzip_deploy as an intermediary step is a bazel implementation detail.

bazel -s output the list of bazel command. ibtool is called through the ibtoolzip wrapper. I doubt there is anything more that can be done. Assigning to Daniel for confirming.

The reason this doesn't print the command line is because of the ibtoolzip wrapper around the real ibtool. You can probably work out the commandline used by looking at https://github.com/google/bazel/blob/dfc9b62faa9e512df72bf8c8edca58e62d9d7160/src/tools/xcode-common/java/com/google/devtools/build/xcode/ibtoolzip/IbtoolZip.java#L54
Admittedly this isn't ideal but I also don't have a simple answer on how to fix this as (AFAIK) -s isn't propagated to the tool.

Dave, I saw some changes passing by related to this. Is this fixed or still outstanding?

wasn't this addressed in a56a58b7c63f32ffb7ec08b4d223d6e5bd14befc ?

No, this isn't fixed yet but I'm working on it right now (passing the flags from blaze to the tools).

Is it fixed? Can we close that bug? @Aragos ? @dmaclach ?

closing due to inactivity.


Intermittant "No Panda" issue requireing reboot of C2

Describe the bug
Occasionally, my C2 will say "No Panda" along with a CAN error message.  A reboot is needed to correct the issue, on the C2. Sometimes a restart of the car is needed to clear it's issues too. The C2's panda will remain offline without rebooting the C2, despite what you do to the car.
How to reproduce or log data
Unable to reliably reproduce. Driving data exists however seems cut short.
Issue occurred at the end of the drive on this segment 903210ecc0cb706b|2021-05-25--21-13-22--80
OP was engaged right up until the end of the drive, despite Explorer saying it was disengaged.
Expected behavior
For Panda not to go offline randomly.
Device/Version information (please complete the following information):

Device: Comma2
Dongle ID: 903210ecc0cb706b
Route: 903210ecc0cb706b|2021-05-25--21-13-22--80
Timestamp: 22:34, right at the end of the drive. After the footage ended, about 10 seconds later in that roundabout, OP disengaged, showed the red "CAN Error" message and disengage warning, the car's LKA and ACC went offline, and OP screamed until I rebooted it about 30 seconds later.
Version: 0.8.4
Car make/model: 2019 Golf R Manual

Additional context
Lots of errors logged in the cars ABS, LKA, ECU, EPS, and BCM modules after this this issue. These errors do not occur without the C2 present.

Do you have a SIM card in your device? Does this seem to happen in the same places when driving around?

I do have a SIM card in it, but it doesn't seem to happen in the same place ever. It's only happened a couple times so far.
When I first got my C2, I did have an issue with tower changes causing issues, but that has since been fixed. I used to be able to reproduce it by driving on a specific road, however driving that road doesn't cause this issue.
My C2 DID hard lock a couple times in the winter, but after re-flashing NEOS it seems to have stopped doing that. This is fairly new, only in the last couple weeks has it done it (Maybe since 0.8.4? but it may have happened on 0.8.3 too.  The first time I ignored, and only took note the second time it happened.
I saw this issue: #20872 but it didn't seem similar enough since it doesn't SEEM to be tied to the cell tower switchover like before... BUT they did link to this issue: #19874 which I had before and even commented on, but like I said, was fixed for me by re-flashing NEOS. Maybe it's the same issue and the "fix" was co-incidence, or this is a different expression of the same issue? IDK...
It seemed too different to be related, hence the separate ticket.

it literally just happened again. I was in the hills near my house, we're cell phone reception is generally pretty spotty. Not sure if that's related, but it was a concern that someone had. I'm still in the area, so I'll see if it happens again on my way back.￼￼
I'll post the route information a little bit later today.￼

So, as I mentioned earlier, it happened again today, BUT there were also two other issues that occurred after a reboot shortly after in the same area. Though these presented in a new way that I don't think I've seen happen before. Not sure if they are related or not, but I can go back and test again to see if they happen in that area again.
The first thing that went wrong was the issue I described earlier, CAN connection error, red warning screen, No Panda afterwards, though the panda firmware still showed up in the Developer tab.
The only way I was able to correct this issue was with a reboot of the C2.
This DID occur in an area with poor cell reception, and the C2 was constantly saying that it was gaining and losing cell service. Though the other times it has happened, weren't in areas of poor cell service, and never seemed to be reproducible before. Maybe it will be on this road? I can try again tomorrow.
This occurred at: 903210ecc0cb706b|2021-05-29--17-04-20--30 at approximately 17:35:01

Later that same drive (though after rebooting the C2), I saw it do the two other things I mentioned in: 903210ecc0cb706b|2021-05-29--17-46-50--5 at approximately 15:52:31 I got an error that I wasn't able to read, but I suspect it was another CAN Error. Panda stayed online, but later trying to re-engage showed an error saying something about controls mismatch somewhere after 17:52:56.
Approximately at 17:53:50, I turned the ignition off and right back on again (not giving the car enough time to reset or for modules to go to sleep) and was able to drive again with my C2 without incident for the rest of the time back home.
During the ride back home, after the reboot of the C2 I kept an eye on the "connected" status and cellular signal strength of the C2 and saw it go from red to yellow to white several times, without issue or incident. I also observed the C2 gain and lose cell service, switch back and forth between 2G, 3G and LTE without issue.
I'll probably do this drive again tomorrow, let me know if you want me to try it again, and I can make it a priority.
Hope this helps!

Did a fair bit of testing. Does seem to be SIM card related, so this issue might be one and the same as #20872
I'll post an update tomorrow morning.

So, I managed to find a stretch of road where I could make the issue occur consistently and compared with and without the sim card. which definitely seemed to cause the issue. At one point, the C2 lost the panda entirely, and I had to disconnect power from the C2 entirely for it to come up again.
I did the road 3 times each, up, back down, and up again.
Here are the drives:
903210ecc0cb706b|2021-05-30--22-40-33--27
903210ecc0cb706b|2021-05-30--23-09-52--0
903210ecc0cb706b|2021-05-30--23-14-29--0
Without the SIM I was able to do all 3 drives together, then popped the sim back in for a 4th run, where it died :
903210ecc0cb706b|2021-05-30--23-19-40--0
The drive with the SIM card starts at approximately 23:32:50, and ends at the end of the clip.
The problem manifested in many different ways. From the panda going offline, to the C2 thinking the car was off, to getting a controls mismatch error, or a CAN communication error.

Thanks for looking into this and isolating it to the SIM card/cellular connection. Yes, let's close this and add the info to the existing issue. I'll look at your routes tomorrow. We're actively looking into this issue, and hope to ship a fix soon in 0.8.5.

Cool, will do. I can very consistently reproduce the issue, so let me know if you want me to do any testing.


Export formulae

Dear folks,
first thanks very much for making this open source after you closed down Revolver!
I've always been really curious how it worked, and have finally come round to building it.
Really cool! I do have one question: is it possible to export the data while retaining simple formulae? I currently only see a csv export, but that removes any cell references and replaces it with the calculated values, right? So you cannot backup and then roundtrip a document that way. Is there another option to save data - or even get it from the backend of the server?

Actually, having an restructuredText (or similar) output would already very much suit my use cases...

I guess you just set yourself a coding challenge!

So can you tell me where and the data is stored? Perhaps python-lpod can be used to turn into an OpenDocument spreadsheet ...

I'm not sure I remember to be honest!  Internally the data structure is a dict of objects which have a .value and a .formula object.
I can't remember whether we store it in a database, or on disk as json, or what...  probably in the database, as json, I would guess. if you want to try and extract directly from the db, maybe check the django models and see if you can find what column it's in?
If you wanted to make an actual button and a UI for this, you could have a look at the way the csv exporter works (search the codebase for "csv") and try and make a copy of that, and modify it?


api: Add GET /users/{user_id}/subscription/{stream_id} endpoint.

Fixes #14966.
This new endpoint returns a 'user' dictionary which, as of now,
contains a single key 'is_subscribed' with a boolean value that
represents whether the user with the given 'user_id' is subscribed
to the stream with the given 'stream_id'.

@timabbott Can you have a look ?

@timabbott I've updated the PR.

The backend looks a lot better!  I think we should remove the outer subscription object; i.e. is_subscribed should be in the dictionary we pass to json_success.
Also, please document this endpoint as new via **Changes**, including recording the API feature level change.

Merged as 8c39ddfd28a2eeed25309604c2a127d2abb7eb73, after fixing the following:

Should be /subscriptions/, not the singular, to match other URLs.
Mentioned the feature level in the endpoint's description in addition to the changelog.
Various copy-editing.

Thanks @akashaviator!


CHANGE to type: module - react-component is not a thing

https://github.com/rakannimer/react-google-charts/pull/476


[3.x] update_bitmask_region() error changed to be informative

When executing:
extends TileMap

func _ready():
	set_cell(0, 0, 1)
	update_bitmask_region()

returns, !tile_map.has(p_id)" is true. Returned: SINGLE_TILE
The error returned isn't clear as to what the problem is. Updated the error message to a custom error message for clarity.
Fix #47313

Thanks for opening a pull request!
Please amend the commit to use a human-readable commit message. This way, people reading the Git log can easily figure out what's going on.

Thank you! But the question I have is should this be a tad bit clearer - explaining to the user "why" it cannot assign the tile slot for the cell value.
It can't be assigned because the the tile slot is missing from the tile set. At least that's what I've been led to believe. Am I wrong about the specifics here? The error might also be popping up under different circumstances.
See here: #29715
This error is actually a huge issue that lots of people are running into and they're coming up with all sorts of explanations as to why it's happening.
So the error being vague might actually be appropriate, because it's a vague issue, lol.

@rainlizard So what I get from this is that when setting set_cell(0, 0, 1) where the first two coordinates are the Vector2 position of the tile to be displayed on the screen, and the third coordinate is for the tile which is the actual tile at the index 0, when you input a value say 1, it looks for a tile at that index which it found none, hence the error. So, if you make another tile it gets placed at index 1 and using set_cell(0, 0, 1) should use the tile at index 1.This is my understanding of the error so don't quote me on this ☺.

That's a good idea but you're only fixing the issue for one method out of the 67 methods which can trigger the exact same issue. So I made a thorougher fix with #48586. Thanks for the contribution nevertheless!


Add strings for house= field and disable Taginfo suggestions

Description, Motivation & Context
Currently the house field doesn't have any strings and it shows any value with 100+ uses. This creates a lot of variants for tagging the same thing.
This PR introduces strings for the documented values of house= tag and disables Taginfo's suggestions for the field.
P.S. It doesn't affect the tag editor, I'm not sure if we can influence it.
Related issues
None
Links and data
Relevant OSM Wiki links:
https://wiki.openstreetmap.org/wiki/Key:house
Relevant tag usage stats:
https://taginfo.openstreetmap.org/keys/house#values
Checklist and Test-Documentation Template
Read on to get your PR merged faster…
Follow these steps to test your PR yourself and make it a lot easier and faster for maintainers to check and approve it.
This is how it works:

After you submit your PR, the system will create a preview and comment on your PR:

🍱 You can preview the tagging presets of this pull request here.
If this is your first contribution to this project, the preview will not happen right away but requires a click from one of the project members. We will do this ASAP.

Once the preview is ready, use it to test your changes.

Now copy the snippet below into a new comment and fill out the blanks.

Now your PR is ready to be reviewed.

## Test-Documentation

### Preview links & Sidebar Screenshots

<!-- Use the preview to find examples, select the feature in question and **copy this link here**.
     Find examples of nodes/areas. Find examples with a lot of tags or very few tags. – Whatever helps to test this thoroughly.
     Add relevant **screenshots** of the sidebar of those examples. -->

<!-- FYI: What we will check:
     - Is the [icon](https://github.com/ideditor/schema-builder/blob/main/ICONS.md) well chosen.
     - Are the fields well-structured and have good labels.
     - Do the dropdowns (etc.) work well and show helpful data. -->

### Search

<!-- **Test the search** of your preset and share relevant **screenshots** here.
     - Test the preset name as search terms.
     - Also test the preset terms and aliases as search terms (if present). -->

### Info-`i`

<!-- **Test the info-i** for your fields and preset and share relevant **screenshots** here.
     The info needs to help mappers understand the preset and when to use it.
     [Learn more…](https://github.com/openstreetmap/id-tagging-schema/blob/main/CONTRIBUTING.md#info-i)
 -->

### Wording

- [ ] American English
- [ ] `name`, `aliases` (if present) use Title Case
- [ ] `terms` (if present) use lower case, sorted A-Z
<!-- Learn more in https://github.com/openstreetmap/id-tagging-schema/blob/main/GUIDELINES.md#2-design-the-preset -->

(The screenshot tool doesn't want to capture the dropdown menu)

I looked into houses in https://github.com/openstreetmap/id-tagging-schema/issues/1155 when I tried to understand the different wording in SC vs iD vs Wiki but never finished it.

@Dimitar5555 I think this is a good addition. But we need to make sure we are not missing something relevant and make the situation worse for some cases.
Could you double check that we have all values in this list that StreetComplete uses?
And we should also check document the taginfo most used values so we support any that are used often, like maybe 5k+?

Btw, a great follow up to this would be to create small icons for those values like we have for crossings. Those are supported via icons https://github.com/ideditor/schema-builder?tab=readme-ov-file#icons

And once https://github.com/openstreetmap/iD/issues/10613 is done and we have a better display of the value description, we should IMO add a description for those as well.

Could you double check that we have all values in this list that StreetComplete uses?

Note that SC is using the building=house/detached/semi-detached/... while this field is shown on elements with building=house + house=(empty value).
SC seems to be missing an answer for terraced houses.
https://github.com/streetcomplete/StreetComplete/blob/master/app/src/main/java/de/westnordost/streetcomplete/osm/building/BuildingType.kt#L9-L13
SC is treating building=terraced as deprecated. The documented replacement in SC is terrace but this is not always correct. terraced is for the individual houses while terrace is for the whole row of houses.
https://github.com/streetcomplete/StreetComplete/blob/master/app/src/main/java/de/westnordost/streetcomplete/osm/building/BuildingType.kt#L172-L174
P.S. Whether SC should be using building=*_house or building=house + house=* is a matter of discussion there.

And we should also check document the taginfo most used values so we support any that are used often, like maybe 5k+?

:white_check_mark:

Btw, a great follow up to this would be to create small icons for those values like we have for crossings. Those are supported via icons ideditor/schema-builder#icons

It is a good suggestion in principle but I'm not sure how they could look. detached is easy but semi-detached and terraced could be represented with the same icon (which might be confusing). terraced could have 3 houses next to each other but we will have to change the icon for terrace as well.

And once openstreetmap/iD#10613 is done and we have a better display of the value description, we should IMO add a description for those as well.

I will add it in the coming days (or maybe next year :smile:).

Note that SC is using the building=house/detached/semi-detached/... while this field is shown on elements with building=house + house=(empty value).

Ah, sorry, I did not fully realize, what we are actually looking at here …
So this list is only used for the building=house preset at https://github.com/openstreetmap/id-tagging-schema/blob/main/data/presets/building/house.json#L8
(Side note: There is no other usage of this field https://github.com/search?q=repo%3Aopenstreetmap%2Fid-tagging-schema+\"house\"+path%3A%2F^data\%2Fpresets\%2F%2F&type=code )
FYI: We have three unsearchable presets for specific values of this list in https://github.com/openstreetmap/id-tagging-schema/tree/main/data/presets/building/house which where all added in https://github.com/openstreetmap/id-tagging-schema/pull/921  – those unsearchable presets don't have "fields" themselves so they will reuse the parent fields which in will include the house=* field that we are editing here. This is important because it will allow to change the type of house even after one of those three was selected back to something else.
TBH I am not sure if we actually want those separate, unsearchable presets. The only benefit I can see is that https://github.com/openstreetmap/id-tagging-schema/pull/921/files#diff-cb5181126156c42a37b82bff9e25d5bf13d6c2b4331b6980a24262ceaf8b2c06R2 has a different icon now.

P.S. Whether SC should be using building=*_house or building=house + house=* is a matter of discussion there.

The question for this repo that I see from this is: How can we make those sub-types discoverable when searching or browsing for houses in iD. Right now only https://github.com/openstreetmap/id-tagging-schema/blob/main/data/presets/building/detached.json will be findable but https://github.com/openstreetmap/id-tagging-schema/blob/main/data/presets/building/house/_detached.json will be hidden from the search and browser section.
Again, this is something we don't have to solve now, but we might want to create a follow up issue for this to see if there is a solution for it. Right now it looks like conflicting tagging practices that we cannot really resolve(?)

Test-Documentation
Find test cases: https://overpass-turbo.eu/s/1Wan

🟢  https://pr-1412--ideditor-presets-preview.netlify.app/id/dist/#background=Bing&disable_features=boundaries&id=w1271824908&locale=en&map=20.00/52.20455/13.58388

🟡 one thing we should improve here is to move the "type of house" to the top of the list by changing the order of https://github.com/openstreetmap/id-tagging-schema/blob/main/data/presets/building/house.json#L7-L8

🔴 https://pr-1412--ideditor-presets-preview.netlify.app/id/dist/#background=Bing&disable_features=boundaries&id=w1219479104&locale=en&map=20.00/52.30051/13.78133

I consider this a blocking issue. Right now those unsearchable sub presets remove the "type of house" field from the list of fields. I remember from crossing refactoring that this is some kind auf automatism that I don't think is smart. It means that once I select a "type of house" the UI changes and the only way back is to cmd+z or to start again or edit the raw tags. Instead the "type of house" should always be present at the top to allow to change the preset easily.

same issue https://pr-1412--ideditor-presets-preview.netlify.app/id/dist/#background=Bing&disable_features=boundaries&id=w1324673770&locale=en&map=20.00/52.30858/13.50386

same issue https://pr-1412--ideditor-presets-preview.netlify.app/id/dist/#background=Bing&disable_features=boundaries&id=w1328793108&locale=en&map=20.00/52.40519/13.53411

Preview links & Sidebar Screenshots
Search
Just to document the "two tagging standards" issue described above: This find the building=* but now the house=* that as mapped (manually)

Info-i

Wording

[ ] American English
[ ] name, aliases (if present) use Title Case
[ ] terms (if present) use lower case, sorted A-Z

🟡 one thing we should improve here is to move the "type of house" to the top of the list by changing the order of main/data/presets/building/house.json#L7-L8

Done.

🔴 pr-1412--ideditor-presets-preview.netlify.app/id/dist#background=Bing&disable_features=boundaries&id=w1219479104&locale=en&map=20.00/52.30051/13.78133
* I consider this a blocking issue. Right now those unsearchable sub presets remove the "type of house" field from the list of fields. I remember from crossing refactoring that this is some kind auf automatism that I don't think is smart. It means that once I select a "type of house" the UI changes and the only way back is to cmd+z or to start again or edit the raw tags. Instead the "type of house" should always be present at the top to allow to change the preset easily.

Fixed.

🟡 this line #1412 (files) is written different than the others. Should that be "Row of Terraced Houses"?

Updated accordingly.

residential => MISSING

I skipped this value on purpose. It isn't clear to me what it means. Is it building=residential, building:use=residential or something else? It is also undocumented despite the large usage.


billing for cloud.gov.bc.ca

what is currently being paid
Who in ELT can take over the cost?

According to David Ell, there are no costs.
'There is no cost for domain names on the gov.bc.ca domain, it is a service provided by Networks via their NNR tool.  As part of running the corporate network services they provide DNS for anything that ends in .gov.bc.ca, idir.bcgov and .bcgov.

Cheers,
David
"


Update NPM to new feature Case

Please update npm repository  to feature
Options
ignoreCase: Same as in diffChars.

3.3.1 was just released with everything on master.


Change accept and decline commands for reviewing

Changed accept and decline commands for reviewing

Wait.. you have to change something else

Wait.. you have to change something else

Change what?

And also you're not following on how to contribute actually.
https://github.com/Nicklason/tf2-automatic/blob/master/.github/contributing.md

Basically this:
https://github.com/Nicklason/tf2-automatic/blob/af52e6d376f25082aa2262780eec7e0d34344293/src/classes/Commands.ts#L1510
and this:
https://github.com/Nicklason/tf2-automatic/blob/af52e6d376f25082aa2262780eec7e0d34344293/src/classes/Commands.ts#L1560

Basically this:
https://github.com/Nicklason/tf2-automatic/blob/af52e6d376f25082aa2262780eec7e0d34344293/src/classes/Commands.ts#L1510
and this:
https://github.com/Nicklason/tf2-automatic/blob/af52e6d376f25082aa2262780eec7e0d34344293/src/classes/Commands.ts#L1560

I left it to Nick, as commands work without these changes, and Nick uses these names, and he decides how it is more convenient

No, it won't work unless you change it.

No, it won't work unless you change it.

I already tested it

@idinium96 i don't see why it shouldn't work, all he did was change the name of the commands

🤔 if you said like that, then idk. I thought the one at processMessage should match with the function name.

🤔 if you said like that, then idk. I thought the one at processMessage should match with the function name.

yes, but it will still work

ah.. alright @Nicklason 👌

if you want this merged then you need contributors to approve it

if you want this merged then you need contributors to approve it

Well, what should I do so that they approve?

if you want this merged then you need contributors to approve it

Well, what should I do so that they approve?

make contributors agree with you, then they will approve

I like the command name as it is. !accept/!decline doesn't explain what your accepting/declining.

I like the command name as it is. !accept/!decline doesn't explain what your accepting/declining.

Yeah me too. Say we add a new command for accepting friend requests or whatever. Then there will be "accept" and "acceptfriend". Making the command be just "accept" doesn't tell much about the command.
I'll close the PR for this reason.


js missing in packaging

This is mostly a continuation of #103 -- that issue was closed because the HTML templates are, indeed, in pinax.templates. However, static/js/admin_post_form.js which was also mentioned there, is not in pinax.templates, is included in this git repo, and is missing -- at least by default; we've had to install it manually to prevent errors at the post admin page.
I think it needs to be included.

Confirmed: I have also experienced this

I wonder if the fix is as simple as

relocate admin_post_form.js to static/src/js/, as seen here
update the appconf ADMIN_JS location to "src/js/admin_post_form.js" here

Then publish a new version, instructing upgrading users to npm install or whatever (I'm not sure!) after upgrading. How does that sound @paltman?

I've thought about actually removing it and providing instructions on how to add it with a gist of sample js.

What I really want to see happen for pinax-blog is an authoring interface outside of the Django admin.

Isn't the issue simply that the javascript file is not making it into the pypi package? django.contrib.staticfiles.finders.AppDirectoriesFinder provides a perfectly good mechanism to move JS files into a central location.
I don't understand the mechanism, but my pip installed version of pinax-blog simply doesn't have the static folder.

Fixed in v7.0.4 (just pushed to PyPI)


Better logging of errors on missed exponential backoff

For example, on RequestQueue throttling errors we get in log the following message, which doesn't actually tell anything about what happened. These errors are incorrectly reported by server as 502, which will be fixed in upcoming deploy. But still, we should show more info.
2018-08-11T01:55:13.304Z ApifyError: Server request failed with 9 tries.
2018-08-11T01:55:13.306Z     at Request._request2.default.(anonymous function) [as _callback] (/home/myuser/node_modules/apify-client/build/utils.js:142:35)
2018-08-11T01:55:13.308Z     at Request.self.callback (/home/myuser/node_modules/request/request.js:185:22)
2018-08-11T01:55:13.311Z     at emitTwo (events.js:126:13)
2018-08-11T01:55:13.313Z     at Request.emit (events.js:214:7)
2018-08-11T01:55:13.316Z     at Request.<anonymous> (/home/myuser/node_modules/request/request.js:1157:10)
2018-08-11T01:55:13.318Z     at emitOne (events.js:116:13)
2018-08-11T01:55:13.320Z     at Request.emit (events.js:211:7)
2018-08-11T01:55:13.322Z     at IncomingMessage.<anonymous> (/home/myuser/node_modules/request/request.js:1079:12)
2018-08-11T01:55:13.324Z     at Object.onceWrapper (events.js:313:30)
2018-08-11T01:55:13.326Z     at emitNone (events.js:111:20)
2018-08-11T01:55:13.328Z     at IncomingMessage.emit (events.js:208:7)
2018-08-11T01:55:13.330Z     at endReadableNT (_stream_readable.js:1064:12)
2018-08-11T01:55:13.333Z     at _combinedTickCallback (internal/process/next_tick.js:138:11)
2018-08-11T01:55:13.339Z     at process._tickCallback (internal/process/next_tick.js:180:9)

BTW we should replace promise-chains with async/await wherever possible, which will automagically improve the error messages.

Done.


TSC Meeting (12 Dec 2023) - v3.8.1 patch release progress, v3.9.0 release plans, ST2-K8s security

December 2023 @StackStorm/tsc 1 hour meeting:

Tuesday, 12 December 2023, 09:30 AM US Pacific / 06:30 PM EU CET
See https://github.com/StackStorm/community/issues/33 for the schedule and how to join
TLDR; Jitsi link: https://meet.jit.si/StackStormTSC

Meeting Agenda
v3.8.1 Release Progress (finalizing)

Project - https://github.com/orgs/StackStorm/projects/34
Release Manager: @armab
Release Assistant: @nzlosh
Current state: finishing pre-release testing https://github.com/StackStorm/community/issues/128
To be cut on a week of 11-14 Dec

StackStorm Contributors and Maintainers
StackStorm v3.8.1 patch release wouldn't be possible without our opensource community who listened when we asked for help and stepped in to assist: fixing broken builds, updating dependencies, security, testing. Let's highlight volunteers, contributors and maintainers who were active recently or helped with the v3.8.1 patch release and upcoming v3.9.0.
Starting strong with the community-driven v3.8.1, there's much more work that needs to be done for the upcoming bigger v3.9!
v3.9.0 Release Planning

Project: https://github.com/orgs/StackStorm/projects/31
Target: ???

+1m, +2m, +3m after the patch release?

Release Manager: TSC volunteer Needed!
Release Assistant: TSC volunteer Needed!
Major changes:

Drop py3.6

Drop U18
Migrate py3.6 -> py?? for EL7

Update all the dependencies
Add U22
Upgrade MongoDB to v6.0
SSO/SAML support (PRs by community)
pants builds?
anything else?

Adding automated security scan for stackstorm-k8s
Requested by @ZoeLeah to add security scan like like Snyk to raise awareness around StackStorm K8s/Docker security, its components and builds.

We use Snyk not only for scanning stackstorm-k8s, but for all repositories of StackStorm that we use. Here is some information about Snyk and how you can register open source software for free: https://snyk.io/de/open-source-projects/

If it's not too late, I would like to add another item to the agenda.
SonarCloud: https://www.sonarsource.com/products/sonarcloud/


add proper network graph

[x] provide graph data
[x] render graph
[x] add pop-up modals
[x] fix image data
[x] highlight neighbors on hover
[x] add search box

closed with https://github.com/arthur-schnitzler/schnitzler-orte/commit/71e7f714719c330090d2b449bb14c9089e534fac


How to set index_name for separate environments

Hi,
I'm having some challenges figuring out how to handle multiple indices for different environments and looking for some tips.
As index_name is a part of the configuration when using the Algoliax.Indexer macro the index name needs to be set at compile time. Often variables that can differ between environments are set as environment variables at startup/runtime as one would have a separate index for each environment.
For now I have used if(Mix.env() == :prod, do: :prod_index_name, else: :dev_index_name) which is fine for a while but when we need to introduce a staging environment which would have the same mix env as production.
I hoped there was a way to do this with environment variables but I cant see how as it is required at compile time.
One alternative would be to create one module per environment and use config to select between them but I hoped it would be a better way to handle it.
Does anyone use this library with multiple environments and different index names or have any tips of how to handle this?
Are there any plans to change this or would you consider changing how index_names can be provided to support using environment variables?

Hi @ringvold, sorry for the late reply.
It is already possible by defining a function like this:
defmodule People do
  use Algoliax.Indexer,
    index_name: :runtime_index_name,
    object_id: :reference,
    algolia: [
      attribute_for_faceting: ["age"],
      custom_ranking: ["desc(updated_at)"]
    ]

  defstruct reference: nil, last_name: nil, first_name: nil, age: nil

  def runtime_index_name do
    System.get_env("INDEX_NAME")
  end
end

Thanks!
Yeah, I see it now when looking through the source. Is it documented anywhere? I did not find anything about it last time I checked.

Nope you're right. Sorry about that.
I going to update the doc.

@ringvold fixed in  #94


Broadcast message toOthers throws ApiException in pusher

Reverb Version
1.0.0-beta4
Laravel Version
11.1.0
PHP Version
8.3.4
Description
When i try to send broadcast message via my Event like this
broadcast(new MyEvent())->toOthers()
I receive an Exception
[2024-03-28 16:26:08] local.ERROR: Pusher error: Internal server error.. {"exception":"[object] (Illuminate\\Broadcasting\\BroadcastException(code: 0): Pusher error: Internal server error.. at /home/ss/workspace/chat/vendor/laravel/framework/src/Illuminate/Broadcasting/Broadcasters/PusherBroadcaster.php:164)
[stacktrace]
#0 /home/ss/workspace/chat/vendor/laravel/framework/src/Illuminate/Broadcasting/BroadcastEvent.php(92): Illuminate\\Broadcasting\\Broadcasters\\PusherBroadcaster->broadcast()

In laravel-websockets, which i used before, same method work without errors.
Steps To Reproduce
broadcast(new MyEvent())->toOthers()
receive exception
[2024-03-28 16:26:08] local.ERROR: Pusher error: Internal server error.. {"exception":"[object] (Illuminate\\Broadcasting\\BroadcastException(code: 0): Pusher error: Internal server error.. at /home/ss/workspace/chat/vendor/laravel/framework/src/Illuminate/Broadcasting/Broadcasters/PusherBroadcaster.php:164)
[stacktrace]
#0 /home/ss/workspace/chat/vendor/laravel/framework/src/Illuminate/Broadcasting/BroadcastEvent.php(92): Illuminate\\Broadcasting\\Broadcasters\\PusherBroadcaster->broadcast()

When i add in Reverb inside handleRequest method of Laravel\Reverb\Servers\Reverb\Http\Server
           $this->close($connection, 500, $e->getMessage());
instead of
           $this->close($connection, 500, 'Internal server error.');
Which a give ZERO info about what happening inside this error
I receive some more info
message
:
"Pusher error: Undefined array key \"875833828.561499442\"."

Maybe problem in line
           isset($payload['socket_id']) ? $this->channels->connections()[$payload['socket_id']]->connection() : null
But didn't understand what is wrong. When remove ->toOthers() - all work good
Thank you

https://github.com/laravel/reverb/pull/122
Maybe I'm missing something, but for some reason, my socket_id is not present in the connections when I use toOthers. However, I haven't encountered this issue when working with laravel-websockets. Please don't judge my English harshly, and thank you for the great product.

I'm seeing the exact same issue with Laravel 11.0.7 and Reverb 1.0.0-beta3. Unfortunately the changes in #122 don't appear to have any effect for me.
I assume socket IDs are necessary for toOthers() to work correctly, but I don't see any socket IDs being passed around anywhere (no X-Socket-ID headers, no "socket" field in websocket payloads, etc). Not sure what's responsible for assigning them. In my case, I'm using Reverb with Livewire in case that's relevant.

Hm, I need some more time to debug. It seems that the socket_id was sent to Pusher inside the payload over the API. The problem is that $this->channels->connections() is returning an empty array. When I add a check for null, this probably just muting the error. Will look after work, time to xdebug)

In my branch make some changes, https://github.com/cryptoprof/reverb/tree/fix-issue-121 (can somebody help?), but has some problems with tests. The problem is in EventController probably.
When i send my Event ->toOthers in EventController
$this->channels->connections() contains {"684284594.574173138":{"Laravel\Reverb\Protocols\Pusher\Channels\ChannelConnection":[]}}
$payload['socket_id'] contains "270200814.539673629"
So when we try to call
$this->channels?->connections()[$payload['socket_id']]->connection()
We get an error, in connections there are no this socket_id as a key

In Laravel WebSockets, the TriggerEventController contains a method that broadcasts an event to everyone except a specific socket ID. Here is the code snippet for that  #method:
optional($channel)->broadcastToEveryoneExcept([
    'channel' => $channelName,
    'event' => $request->json()->get('name'),
    'data' => $request->json()->get('data'),
], $request->json()->get('socket_id'));

However, I couldn't find any implementation of the PresenceChannel in the Reverb package that allows us to check for the socket ID in the payload and broadcast to everyone except that socket ID. It seems that this functionality is not available in Reverb.
The payload with the socket_id is only used in the EventsController, if I'm not mistaken. But the problem is that there is simply no connection with that specific socket_id. I tried to debug it with Xdebug, and it seems that connections are formed in the ArrayChannelConnectionManager. I don't understand why that socket_id is not present in the list of connections, and should it be there? Based on the check condition in the EventsController, I assume that it should be. If it shouldn't be there, then the issue can be resolved with my fix.


Upgrade to Angular v14

Is there a plan to upgrade this to support Angular v14 (and higher)?

I'm not actively upgrading the repository to newer Angular versions. However, most of the time upgrading is an easy process. I'll gladly merge any PRs and publish new versions.


attribute is selected => get combination quantity

if attribute is selected on filters, then get combination quantity not quantity, where sa.id_product_attribute = 0

This change is

Can you please create an issue on https://github.com/PrestaShop/PrestaShop/issues/new/choose. I need to get the point of view of Product Managers :)

При выборе различных фильтров, результат фильтрации - не вычитается, а складывается.
When choosing various filters, the filtering result is not subtracted, but added up.

Will be fixed with https://github.com/PrestaShop/ps_facetedsearch/pull/132


[Bug]: NPM error message on windows is not actionable

What happened?
When using Taqueria on Windows in WSL using an older version of NPM, the following output is given to the user:
taq install [@Taqueria-e2e-test-notifications](https://ecadlabs.slack.com/team/U037TPSBKAQ)/plugin-ligo
npm notice created a lockfile as package-lock.json. You should commit this file.
npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=12"} (current: {"node":"10.19.0","npm":"6.14.4"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm WARN notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">=12"} (current: {"node":"10.19.0","npm":"6.14.4"})
npm WARN notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>No description
npm WARN<EMAIL_ADDRESS>No repository field.
+ [<EMAIL_ADDRESS>added 41 packages from 41 contributors and audited 41 packages in 3.472s
5 packages are looking for funding
  run npm fund for details
found 0 vulnerabilities
Plugin installed successfully

What should have happened?
The CLI should detect that an older version of Node is installed and prompt the user to upgrade before proceeding
Steps to reproduce the bug

Install WSL2 on Windows
Ensure that the node --version outputs 10.9 (default for WSL)
Run taq install @taqueria/plugin-ligo
Observe output

Relevant log output
No response
Screenshots and images
Screenshot and images
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Closing: Will be handled by health check


How do you scan the IPs ?

Hi ,
Thanks for the great work  !! could you tell me how to scan the ip addresses ? I can see that you are scanning ports in your Readme .  what if i want to scan ips ?
Thanks

Hi
Thank you. Need to rewrite from scratch this poor and old code using ES6.
@digits21 I don't really understand your question. This scanner scan opened port of one or more IP Addresses, or range. So ... do you want just it give you if the IP reply to ICMP (ping) ?


[vis] big number axes

accommodate long axis labels, don't overlap

show first and last dates on x axis

Notice: this issue has been closed because it has been inactive for 375 days. Feel free to comment and request for this issue to be reopened.


FrameworkTraining/reactjs.org/content/community/courses.md

We would appreciate it if our React course could be added. Thanks, Tom, Framework Training

Success!

@FrameworkTraining the other courses listed are online-only/video - is that an option for this one?

@alexkrolick
Hi Alex, thanks for following up - we're very much advocates of instructor-led face-to-face training - remote attendance is also possible but we really prefer having an expert in the room with the students.


Offline use of PAGE → ALTO conversion

Currently converting from PAGE to ALTO - so one of the primary use cases for me - requires a working network connection and possibly a working HTTP proxy configuration to - apparently - load the ALTO schema. (#29) I also noticed that this conversion also needs to load - at least - xlink.xsd from the network.
There is code in PrimaDla.jar to load from a schema folder (searchForAdditionalSchemas), we should probably explore this first with the aim to pre-install all schemas in such a folder.

I think that code is only used in the unit tests, so we will have to patch prima-page-converter

Personally, I'm not super happy about relying on prima-page-converter for this. It's hard to work with (though that may be largely due to Java...), hard to build (https://github.com/PRImA-Research-Lab/prima-core-libs/issues/10) and one of the dependencies is closed source (https://github.com/PRImA-Research-Lab/prima-page-converter/issues/17).
Did someone explore the option of doing this conversion by other means, i.e. reimplementing it in Python?

Did someone explore the option of doing this conversion by other means, i.e. reimplementing it in Python?

Yes, I am going to invest a bit of time into this, since we need that conversion in many places and it must be fast and robust. I was going to first consider XSLT and if that turned out to be too cumbersome or inefficient, do the conversion in Python.

In addition to @kba's efforts at https://github.com/kba/page-to-alto, I also stumbled upon these XSLT files: https://gitlab.com/readcoop/transkribus/TranskribusCore/-/tree/master/src/main/resources/xslt

In addition to @kba's efforts at https://github.com/kba/page-to-alto, I also stumbled upon these XSLT files: https://gitlab.com/readcoop/transkribus/TranskribusCore/-/tree/master/src/main/resources/xslt

At least PageToAlto.xsl does not contain any mention of ReadingOrder so I think it might me unsuitable.

@mikegerber, did the Pythonic page-to-alto backend solve this for you?

@mikegerber, did the Pythonic page-to-alto backend solve this for you?

Yes!


[SP-703] Feat: Add default_target_url

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


Update QM for new qubits/channels

https://github.com/qiboteam/qibolab/pull/1001

[ ] register QM configs https://github.com/qiboteam/qibolab/pull/995

Looks good to me, I'd just keep on hold until we confirm that it works well on hardware using the corresponding qibolab branches.

Yes, definitely.
I also wanted to test with Qibocal (I have my own branch, based on yours, but for a weird reason related to submodules is now living in a fork https://github.com/alecandido/qibocal/tree/qibolab0.2-qubits - I will bring it back as soon as I'm ready).


Trouble building wheel for windows

While I'm waiting for you to re-add wheels for windows to the releases, I tried my hand at it, but I'm having the below issue
"C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.35.32215\bin\HostX86\x64\link.exe" /nologo /INCREMENTAL:NO /LTCG /DLL /MANIFEST:EMBED,ID=2 /MANIFESTUAC:NO /LIBPATH:C:\Users\mike\Anaconda3\envs\textgen\lib\site-packages\torch\lib /LIBPATH:C:\Users\mike\Anaconda3\envs\textgen\lib\x64 /LIBPATH:C:\Users\mike\Anaconda3\envs\textgen\libs /LIBPATH:C:\Users\mike\Anaconda3\envs\textgen /LIBPATH:C:\Users\mike\Anaconda3\envs\textgen\PCbuild\amd64 "/LIBPATH:C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.35.32215\ATLMFC\lib\x64" "/LIBPATH:C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.35.32215\lib\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\NETFXSDK\4.8\lib\um\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\lib\10.0.22000.0\ucrt\x64" "/LIBPATH:C:\Program Files (x86)\Windows Kits\10\\lib\10.0.22000.0\\um\x64" c10.lib torch.lib torch_cpu.lib torch_python.lib cudart.lib c10_cuda.lib torch_cuda.lib /EXPORT:PyInit_exllama_kernels C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama/cuda_buffers.obj C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama/cuda_func/column_remap.obj C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama/cuda_func/q4_matmul.obj C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama/cuda_func/q4_matrix.obj C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama/exllama_ext.obj /OUT:build\lib.win-amd64-cpython-310\exllama_kernels.cp310-win_amd64.pyd /IMPLIB:C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama\exllama_kernels.cp310-win_amd64.lib    Creating library C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama\exllama_kernels.cp310-win_amd64.lib and object C:\Users\mike\Documents\text-generation-webui\text-generation-webui\repositories\AutoGPTQ\build\temp.win-amd64-cpython-310\Release\autogptq_cuda/exllama\exllama_kernels.cp310-win_amd64.exp q4_matmul.obj : error LNK2001: unresolved external symbol cublasHgemm build\lib.win-amd64-cpython-310\exllama_kernels.cp310-win_amd64.pyd : fatal error LNK1120: 1 unresolved externals error: command 'C:\\Program Files\\Microsoft Visual Studio\\2022\\Community\\VC\\Tools\\MSVC\\14.35.32215\\bin\\HostX86\\x64\\link.exe' failed with exit code 1120
I met all the requisites from https://github.com/turboderp/exllama windows section.

Having the exact same problem

Yep, I have exactly the same issues, I wish @PanQiWei would release the wheels with the exllama kernels https://github.com/PanQiWei/AutoGPTQ/issues/247

This is related to this : https://github.com/turboderp/exllama/issues/33#issuecomment-1578841082

This is related to this : turboderp/exllama#33 (comment)

@PanQiWei your latest release still doesn't have the exllama kernels on windows

Can you please do it so that we won't have to deal with that by ourselves?

I am planning to share mine publicly for my users, so compiling and publishing the wheel should not be to big of a step.
The problem is that I have no idea where to implement the extra_ldflags=['cublas.lib'] fix to get it compatible with Windows.

I am planning to share mine publicly for my users, so compiling and publishing the wheel should not be to big of a step. The problem is that I have no idea where to implement the extra_ldflags=['cublas.lib'] fix to get it compatible with Windows.

It looks like I was able to make this work, with this update to the setup.py:
   cuda_path = os.environ.get("CUDA_PATH")     if os.environ.get("INCLUDE_EXLLAMA_KERNELS", "1") == "1":  # TODO: improve github action to always compile exllama_kernels         extensions.append(             cpp_extension.CUDAExtension(                 "exllama_kernels",                 [                     "autogptq_cuda/exllama/exllama_ext.cpp",                     "autogptq_cuda/exllama/cuda_buffers.cu",                     "autogptq_cuda/exllama/cuda_func/column_remap.cu",                     "autogptq_cuda/exllama/cuda_func/q4_matmul.cu",                     "autogptq_cuda/exllama/cuda_func/q4_matrix.cu"                 ],                 extra_link_args=["-L", f"{cuda_path}/lib/x64/cublas.lib"]             )         )

@hobbyistmaker excellent dude, I was able to make the binaries and exllama kernels are on windows now!

I am planning to share mine publicly for my users, so compiling and publishing the wheel should not be to big of a step. The problem is that I have no idea where to implement the extra_ldflags=['cublas.lib'] fix to get it compatible with Windows.

It looks like I was able to make this work, with this update to the setup.py:
cuda_path = os.environ.get("CUDA_PATH")
if os.environ.get("INCLUDE_EXLLAMA_KERNELS", "1") == "1":  # TODO: improve github action to always compile exllama_kernels
        extensions.append(
            cpp_extension.CUDAExtension(
                "exllama_kernels",
                [
                    "autogptq_cuda/exllama/exllama_ext.cpp",
                    "autogptq_cuda/exllama/cuda_buffers.cu",
                    "autogptq_cuda/exllama/cuda_func/column_remap.cu",
                    "autogptq_cuda/exllama/cuda_func/q4_matmul.cu",
                    "autogptq_cuda/exllama/cuda_func/q4_matrix.cu"
                ],
                extra_link_args=["-L", f"{cuda_path}/lib/x64/cublas.lib"]
            )
)

Thanks for the solution.
My first attempt at this resulted in a exit code 1181, which turned out my CUDA path wasn’t set to the right name. After solving that, the build was successful.

I am planning to share mine publicly for my users, so compiling and publishing the wheel should not be to big of a step. The problem is that I have no idea where to implement the extra_ldflags=['cublas.lib'] fix to get it compatible with Windows.

It looks like I was able to make this work, with this update to the setup.py:
cuda_path = os.environ.get("CUDA_PATH")
if os.environ.get("INCLUDE_EXLLAMA_KERNELS", "1") == "1":  # TODO: improve github action to always compile exllama_kernels
        extensions.append(
            cpp_extension.CUDAExtension(
                "exllama_kernels",
                [
                    "autogptq_cuda/exllama/exllama_ext.cpp",
                    "autogptq_cuda/exllama/cuda_buffers.cu",
                    "autogptq_cuda/exllama/cuda_func/column_remap.cu",
                    "autogptq_cuda/exllama/cuda_func/q4_matmul.cu",
                    "autogptq_cuda/exllama/cuda_func/q4_matrix.cu"
                ],
                extra_link_args=["-L", f"{cuda_path}/lib/x64/cublas.lib"]
            )
)

You made my day, thank you very much for figuring this out and sharing the details!

As promised I have began providing a precompiled build for the KoboldAI users (If you are a Windows KoboldAI user simply use pip uninstall auto-gptq and update to the latest version of United with the updater to get my build).
For the rest of you on Python 3.8 and Cuda 11.8 the link is here : https://github.com/henk717/KoboldAI/releases/download/Snapshot-11-08-23/auto_gptq-0.4.1+cu118-cp38-cp38-win_amd64.whl

Hi everyone, I'll include the patch from @hobbyistmaker it a patch release of AutoGPTQ.

Hi, the fix from @hobbyistmaker was included on main: https://github.com/PanQiWei/AutoGPTQ/blob/7664b8ab3c0460662bce8f92c4a96e6d9ef5f755/setup.py#L144
Please open a new issue if you still face issues with the build.


Remove Delete button from new actions view

Changes
This PR fixes issue #772 by removing the Delete button from the view for new action.
Checklist

[x] have tested my change
[ ] updated relevant documentation
[ ] Remove any/all console.logs I added
[x] eslint is not complaining
[ ] have added myself to the contributors' list in CONTRIBUTING.md
[ ] for feature PRs: PR only includes one feature enhancement.

@chrjorgensen Nice and basic! I love it. Thank you so much for picking this up so quickly.

@worksofliam You're welcome - thank you for giving me credit on Twitter! 😃
I have some work in progress on the code for maintaining actions - see some potential for improvement. So additional commits for this will probably arrive soon.


code linting as a PR qulity check

We should use linters to make sure our code is up to our quality standards.
Failure to meet the standard would disallow merge of the PR.
See discussion about linting rules
Acceptance Criteria:

[ ] relevant errors
[ ] not too many errors
[ ] easy enough to inspect error messages

Tasks/Questions:

[x] should this be integrate with gitlab ci or as a github action? 1st step would be github action
[ ] should disallow pr merge when there are errors
[x] rules for python - see .pylintrc and .flake8 files

for next steps:

[ ] rules for java - do in ingest-core
[ ] rules for javascript/typescript - for ingest-ui
[ ] nice looking error report - 1st step is the text output of the linter

possible tools:

[ ] super linter - will be used for 1st iteration, integrated as a github action
[ ] code climate - seems we used it slightly in the past

@amnonkhen to merge this today


MQTT as Endpoint

Would it be possible to have an MQTT endpoint be an option, instead of Home Assistant or REST?

Moved the issue as we will implement this in WAS.

So I started on an MQTT endpoint for WAS: 63fa17c
If you want to test it you could use this container image: ghcr.io/toverainc/willow-application-server:feature-mqtt_endpoint. As this does not include the UI, you'll have to run it yourself.

Thanks! This is amazing! Great work


Double-verify the merkle root with remerkleable

As discussed offline today, we can import remerkleable to double-verify the roots.
https://pypi.org/project/remerkleable/
Pros

We can be more confident about the deposit message on deposit-cli side. Yay.

Cons

remerkleable required Python 3.8+
The contract will verify it anyway though
It can be double-verify on the launchpad with https://github.com/ChainSafe/ssz (https://github.com/ethereum/eth2.0-deposit/issues/61) too

Considering that it is verified in another two places already, I don't think it is worthwhile making this change.


Batch insert with useBulkCopyForBatchInsert & IDENTITY_INSERT ON

1- Use useBulkCopyForBatchInsert in connection string.
2- Create a table with identity.
3- Enable IDENTITY_INSERT.
4- Add multiple insert statements as batch including the identity column (preparedStatement.addBatch()).
5- Execute the batch (preparedStatement.executeBatch()).
6- The following error occurs:

java.sql.BatchUpdateException: Explicit value must be specified for identity column in table 'tableName' either when IDENTITY_INSERT is set to ON or when a replication user is inserting into a NOT FOR REPLICATION identity column.

The same scenario works well if we remove useBulkCopyForBatchInsert from the connection string.

hi @Eng-Fouad
can you please include the schema and code you used to repro the issue?

closing due to inactivity. Please feel free to request re-opening if you have more info.


Add a PR template to this repo

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem?
No response
Describe the feature you would like
Consider adding a pr template that motivates the contributer to give sufficient information on the work he has done on that pr so anyone could get the proper context on that pr
Describe your proposed solution
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

It would be very useful to add a PR template indeed.
I think it can be inspired by the Cilium PR template, maybe a bit simpler. Things that should be mentioned IMO:

tests (link do dev docs how to run them?)
commit subject, body and sign-off (link to the Cilium dev docs?)
release note (if PR title is not enough)
Add yourself to users (in cilium/cilium repo)

@paularah Any thoughts here?

Indeed it'd be useful. We should be have the USERS.md file in soon, once we've gathered the info we need from users.
@sadath-12 Feel free to open PR if you'd love to take a stab at this.


Is there a way to get a list of dependencies, like require.context in webpack

If I use canvas or webGL or other situations, I want to load resources, how can I get the exact path of these resources?

I'm not sure what you are want.  If you want to load resources, you just use import xxx, why you need get the exact path?
Please use stackoverflow and other plateform to ask question.

For example, if I want to use canvas to render one hundred resources, do I have to import every image and then load it with new Image()? This is too much trouble. In webpack, I only need to write require.context to get the resource list and traverse it once.

This issue was closed, I do not know you can not see

Duplicated of https://github.com/vitejs/vite/issues/77


Implement a config variable that will allow the owners to edit their datasets

For now, we are relying on UNEDITABLE_SCHEMAS, but what if want the owners to change the datasets they own.
For example, it could be: (I don't know atm, which would take precedence)
is_editable = True

if results['schema'] in app.config['UNEDITABLE_SCHEMAS']:
    is_editable = False

if app.config['ALLOW_ADMINS_TO_EDIT'] and current_user in results['owners']:
    is_editable = True

thoughts ?

cc @ttannis @danwom @jinhyukchang
LGTM on my side.

Fine by me as well. I think ALLOW_ADMINS_TO_EDIT could take precedence over UNEDITABLE_SCHEMAS if it's default value is False. Essentially anything that overrides UNEDITABLE_SCHEMAS going forward would just have to be opt-in.

@verdan I understand the need, but please note that anyone can add themselves as owner and edit in this case.

@jinhyukchang not sure if I understood correctly, but only owner will be able to add other owners through Amundsen, right? I can't add myself as owner because the owner field would be uneditable for me as a normal user.

Ah, great point from @jinhyukchang. Right now UNEDITABLE_SCHEMAS only applies to the descriptions, so what Jin mentioned is correct. We now have a use case for not just specifying which schemas are editable, but which pieces of metadata on those schemas are editable as well.

I see. Yes good point. I thought is_editable applies for all the editable fields, like owner, descriptions, tags etc.

@verdan this is just the initial approach that we've taken in order to increase the richness of our metadata by letting anyone with access to the tool edit metadata. In the future we may revisit that decision. All that aside, we're on board with having a more flexible configuration.


fail rendering

Hi, I really like your theme. However,
Building sites … ERROR 2019/01/01 20:33:01 render of "section" failed: "/home/USER/Projects/myblog/themes/hermit/layouts/_default/list.html:21:50": execute of template failed: template: _default/list.html:21:50: executing "main" at <.Site.Params.datefor...>: invalid value; expected string
Total in 91 ms
Error: Error building site: failed to render pages: render of "page" failed: "/home/USER/Projects/myblog/themes/hermit/layouts/posts/single.html:16:54": execute of template failed: template: posts/single.html:16:54: executing "main" at <.Site.Params.datefor...>: invalid value; expected string

Any suggestion? Also, would you like to add MathJax & Highlight.js support? This theme is almost all I need.

Sorry, wrong config file. Still, any plan to add MathJax & Highlight.js support?

About MathJax:
As you can see here https://github.com/Track3/hermit#code-injection, you can add anything to html head or right above the </body>, this should be easy. Docs: https://gohugo.io/content-management/formats/#mathjax-with-hugo . I'll consider add a configurable on/off partial in the future.
About highlight.js:
Hugo uses Chroma: https://gohugo.io/content-management/syntax-highlighting/

Emm, I test everything, and it worked perfect. However, after deploy to github, it seems that everything is broken.

It seems that it's related to cloudflare's CDN, I'll check it in next several day.

This could be Hugo's bug, some times I run into this when use hugo server. You can try delete public and resources, then rebuild your site.

@rep-nop This is because you config file didn't have these dateform set, you probably switch from another theme to this one, so you have to be careful with configurations. You can take a look at exampleSite/config.toml
[params]
  dateform        = "Jan 2, 2006"
  dateformShort   = "Jan 2"
  dateformNum     = "2006-01-02"
  dateformNumTime = "2006-01-02 15:04 -0700"

I have those exact parameters configured and I am still running into this issue.
Building sites … ERROR 2019/02/16 16:13:10 render of "taxonomy" failed: "/home/cameronnemo/docs/src/cameronnemo/io/themes/hermit/layouts/_default/list.html:21:50": execute of template failed: template: _default/list.html:21:50: executing "main" at <.Site.Params.datefor...>: invalid value; expected string
ERROR 2019/02/16 16:13:10 render of "section" failed: "/home/cameronnemo/docs/src/cameronnemo/io/themes/hermit/layouts/_default/list.html:21:50": execute of template failed: template: _default/list.html:21:50: executing "main" at <.Site.Params.datefor...>: invalid value; expected string
ERROR 2019/02/16 16:13:10 render of "taxonomyTerm" failed: "/home/cameronnemo/docs/src/cameronnemo/io/themes/hermit/layouts/_default/list.html:21:50": execute of template failed: template: _default/list.html:21:50: executing "main" at <.Site.Params.datefor...>: invalid value; expected string
Total in 11 ms
Error: Error building site: failed to render pages: render of "page" failed: "/home/cameronnemo/docs/src/cameronnemo/io/themes/hermit/layouts/posts/single.html:22:54": execute of template failed: template: posts/single.html:22:54: executing "main" at <.Site.Params.datefor...>: invalid value; expected string


Refresh token when running cron

Before accessing a repo during a cron job the users token is refreshed like it is done for e.g. create pipeline. User in this context means the creator of the cron job.
fixes #1431
If the scheduled delay between two executions of a cron job is larger than the exiration time of the oauth2 refresh token (provided by the forge) this will fail and the cron job will not be run. But this is the best one can do.

Why doesn't the local 'make lint' find what is found in the CI server?

Codecov Report
Base: 42.33% // Head: 42.29% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (80cc775) compared to base (dda4998).
Patch coverage: 16.66% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1432      +/-   ##
==========================================
- Coverage   42.33%   42.29%   -0.05%
==========================================
  Files         153      153
  Lines       10493    10503      +10
==========================================
  Hits         4442     4442
- Misses       5771     5780       +9
- Partials      280      281       +1

Impacted Files
Coverage Δ

server/cron/cron.go
32.58% <16.66%> (-4.13%)
:arrow_down:

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.


Generalizing to Vectors and Matrices

To generalize to matrices and Vector, the BackProp type must be changed to support T -> U functions, T and U being SomeReal, Matrix32, DMatrix32, Vector32, DVector32 from linalg
This works but is not enough:
import linalg

type
  BackProp[T, U] = proc (gradient: T): U {.noSideEffect.}
    ## Proc for Backward propagation are typed `BackProp` and are (implicit) closures without side-effects
    # To ease search, backward propagation procedures are prefixed with bp_

  Node[T, U] = object
    ## Represent an operation
    ## Stores the gradient transformation for backprop in weights
    ## Stores indices of parent operation in parents
    weights: array[2, BackProp[T, U]]
    parents: array[2, int] #ref indices to parent nodes

  Context*[T, U] = object
    ## Tape / Wengert list. Contains the list of applied operations
    nodes: ref seq[Node[T, U]]

# Templates in Nim are always inlined. They are used for performance reason to save on function calls costs.

proc newContext*[T, U]: Context[T, U] {.noSideEffect.} =
  ## Initialize a context (Tape / Wengert list)
  result.nodes = new seq[Node[T, U]]
  result.nodes[] = @[]

let ctx = newContext[float32, Matrix32[2,2]]

The main issue is that now all proc must transform a scalar to a 2x2 Matrix.
We need at minimum:

Scalar to Vector (backprop of Softmax)
Vector to Matrix (backprop of Flatten)
Matrix NxN to Matrix MxM (Convolutional Layer and MaxPool Layer)

What was tried:
import linalg

type

  Tensor[float32] = float32 or DVector32

  BackProp[T] = proc (gradient: Tensor[T]): Tensor[T] {.noSideEffect.}
    ## Proc for Backward propagation are typed `BackProp` and are (implicit) closures without side-effects
    # To ease search, backward propagation procedures are prefixed with bp_

  Node[T] = object
    ## Represent an operation
    ## Stores the gradient transformation for backprop in weights
    ## Stores indices of parent operation in parents
    weights: array[2, BackProp[T]]
    parents: array[2, int] #ref indices to parent nodes

  Context*[T] = object
    ## Tape / Wengert list. Contains the list of applied operations
    nodes: ref seq[Node[T]]

# Templates in Nim are always inlined. They are used for performance reason to save on function calls costs.

proc newContext*(T: typedesc[SomeReal]): Context[T] {.noSideEffect.} =
  ## Initialize a context (Tape / Wengert list)
  result.nodes = new seq[Node[T]]
  result.nodes[] = @[]

let ctx = newContext(float32)

# Cannot evaluate at compile time: T

import linalg

type

  Tensor = float32 or DVector32

  BackProp = proc (gradient: Tensor): Tensor {.noSideEffect.}
    ## Proc for Backward propagation are typed `BackProp` and are (implicit) closures without side-effects
    # To ease search, backward propagation procedures are prefixed with bp_

  Node = object
    ## Represent an operation
    ## Stores the gradient transformation for backprop in weights
    ## Stores indices of parent operation in parents
    weights: array[2, BackProp]
    parents: array[2, int] #ref indices to parent nodes

  Context* = object
    ## Tape / Wengert list. Contains the list of applied operations
    nodes: ref seq[Node]

# Templates in Nim are always inlined. They are used for performance reason to save on function calls costs.

proc newContext*(T: typedesc[SomeReal]): Context {.noSideEffect.} =
  ## Initialize a context (Tape / Wengert list)
  result.nodes = new seq[Node]
  result.nodes[] = @[]

let ctx = newContext(float32)
# Error: 'BackProp' is not a concrete type.

There is also one variation of the Tensor ... = float32 or DVector32 ... with Cannot instantiate error with this kind of declaration if I remember properly:
BackProp[T: Tensor, U: Tensor] = proc (gradient: T): U {.noSideEffect.}

Forum post: https://forum.nim-lang.org/t/2907
Using this object variant solutions works:
import linalg

type
  ## Unfortunately Nim typechecker won't let us store BackProp[float32, Vector32]
  ## with BackProp[Vector32, Matrix32] in the same list so I have to store each type
  ## of BackProp[T, U] in it's own list
  BackPropKind = enum
    ## List of BackProp[T,U] types
    bpFF,
    bpFV,
    bpVF,
    bpVV,
    bpFM,
    bpVM,
    bpMF,
    bpMV,
    bpMM

  BackProp = ref BackPropObj
  BackPropObj = object
    ## Object storing the procs for Backward propagation are typed `BackProp` and are (implicit) closures without side-effects
    # To ease search, backward propagation procedures are prefixed with bp_
    case kind: BackPropKind
    of bpFF:
        ff: proc (gradient: float32): float32 {.noSideEffect.}
    of bpFV:
        fv: proc (gradient: float32): DVector32 {.noSideEffect.}
    of bpVF:
        vf: proc (gradient: DVector32): float32 {.noSideEffect.}
    of bpVV:
        vv: proc (gradient: DVector32): DVector32 {.noSideEffect.}
    of bpFM:
        fm: proc (gradient: float32): DMatrix32 {.noSideEffect.}
    of bpVM:
        vm: proc (gradient: DVector32): DMatrix32 {.noSideEffect.}
    of bpMF:
        mf: proc (gradient: DMatrix32): float32 {.noSideEffect.}
    of bpMV:
        mv: proc (gradient: DMatrix32): DVector32 {.noSideEffect.}
    of bpMM:
        mm: proc (gradient: DMatrix32): DMatrix32 {.noSideEffect.}

  Node = object
    ## Represent an operation
    ## Stores the gradient transformation for backprop in weights
    ## Stores indices of parent operation in parents
    weights: array[2, BackProp]
    parents: array[2, int] #ref indices to parent nodes

  Context* = object
    ## Tape / Wengert list. Contains the list of applied operations
    nodes: ref seq[Node]

# Templates in Nim are always inlined. They are used for performance reason to save on function calls costs.

proc newContext*: Context {.noSideEffect.} =
  ## Initialize a context (Tape / Wengert list)
  result.nodes = new seq[Node]
  result.nodes[] = @[]

let ctx = newContext()

It's not possible to use the statically typed Matrix and Vector instead of DMatrix and DVector unfortunately, probably because of the Matrix32[M, N: static[int]] signature.
So we lose compiler autodetection for:

square matrices and call of an optimized BLAS procedure
incompatible shapes, for Vector-Matrices and Matrices-Matrices product

Bonus point to de-uglify this type section >_>

Further attempts by @Varriount were unsuccessful, namely:
import linalg

{.experimental.}

# Global templates
template foreach(expression, iter: untyped): untyped =
  for element in mitems(iter):
    expression(element)

type
  BackPropRoot = ref object of RootObj
  BackPropClosure[T, U] = proc (gradient: T): U {.noSideEffect, closure.}

  ## Proc for Backward propagation are typed `BackProp` and are (implicit) closures without side-effects
  ## To ease search, backward propagation procedures are prefixed with bp_
  BackProp[T, U] = ref object of BackPropRoot
    procedure: BackPropClosure[T, U]

  ## Represent an operation
  ## Stores the gradient transformation for backprop in weights
  ## Stores indices of parent operation in parents
  Node = object
    weights: array[2, BackPropRoot]
    parents: array[2, int] #ref indices to parent nodes

  ## Tape / Wengert list. Contains the list of applied operations
  Context* = object
    nodes: ref seq[Node]

# BackProp implementations
method call(target: BackPropRoot, param: int): int {.base.} =
  return param

method call[T, U](target: BackProp[T, U], param: T): U =
  result = target.p(param)

proc newBackProp[T, U](p: BackPropClosure): BackProp =
  new(result)
  result.procedure = p

# Context implementations
proc newContext*(): Context {.noSideEffect.} =
  ## Initialize a context (Tape / Wengert list)
  result.nodes = new seq[Node]

proc main() =
  proc squareInt(gradient: int): int {.noSideEffect, closure.} =
    result = gradient * gradient

  proc squareFloat(gradient: float): float {.noSideEffect, closure.} =
    result = gradient * gradient

  var
    ctx = newContext()
    n = newNode()

And
import linalg

{.experimental.}

# Global templates
template foreach(expression, iter: untyped): untyped =
  for element in mitems(iter):
    expression(element)

type
  BackPropRoot = ref object of RootObj
  BackPropClosure[T, U] = proc (gradient: T): U {.noSideEffect, closure.}

  ## Proc for Backward propagation are typed `BackProp` and are (implicit) closures without side-effects
  ## To ease search, backward propagation procedures are prefixed with bp_
  BackProp[T, U] = ref object of BackPropRoot
    procedure: BackPropClosure[T, U]

  ## Represent an operation
  ## Stores the gradient transformation for backprop in weights
  ## Stores indices of parent operation in parents
  Node = object
    weights: array[2, BackPropRoot]
    parents: array[2, int] #ref indices to parent nodes

  ## Tape / Wengert list. Contains the list of applied operations
  Context* = object
    nodes: ref seq[Node]

# BackProp implementations
method call[T, U](target: BackPropRoot, param: T): U {.base.} =
  # Raise exception here
  echo "Hi"
  return result

method call[T, U](target: BackProp[T, U], param: T): U =
  echo "Ho"
  result = target.procedure(param)

proc newBackProp[T, U](p: BackPropClosure[T, U]): BackProp[T, U] =
  new(result)
  result.procedure = p

# Context implementations
proc newContext*(): Context {.noSideEffect.} =
  ## Initialize a context (Tape / Wengert list)
  result.nodes = new seq[Node]

proc main() =
  proc squareInt(gradient: int): int {.noSideEffect, closure.} =
    result = gradient * gradient

  proc squareFloat(gradient: float): float {.noSideEffect, closure.} =
    result = gradient * gradient

  var
    ctx = newContext()
    n: Node

  n.weights[0] = newBackProp(squareFloat)
  n.weights[1] = newBackProp(squareInt)
  call(n.weights[0], 1)
  call(n.weights[0], 1.0)
  call(n.weights[1], 1)
  call(n.weights[1], 1.0)

  ctx.nodes.add(n)

Given the difficult to use linalg as is and future needs of 3D, 4D, 5D tensors for:

height
width
batch size
RGB color channels for color images
depth for 3D or time for video

It is a must to develop a Nim ndarrays/tensor library to unify vectors and matrices and support at least 5 dimensions with GPU computation. Furthermore type signature between vector, matrices and more should be the same.
Work as started on the Arraymancer project.

Done in Arraymancer/autograd


Change window size to 480x640 as workaround for black screen issue

This fixes #3

It at least launches now and shows something sensible:

Also works on phone now.
But it seems pretty buggy. Rotating the phone or switching between apps causes it to black-screen.


Draft: Attempting iOS url query params

This doesn't fully solve https://github.com/MCBoarder289/shelf-help/issues/22 but makes it a little better.


chore(security): Support Go Vulnerability Management

See https://go.dev/security/vuln/
Signed-off-by: Bo-Yi.Wu<EMAIL_ADDRESS>

make L-G-T-M work

We can probably move this into golangci-lint after https://github.com/golangci/golangci-lint/pull/3199.


fix event validation error from returning dict of counts instead of list of Validation Result

[x] Closes #660  .
[x] I am familiar with the contributing guidelines.
[x] Tests added.
[x] Updates entries to docs/source/api.rst for API changes.
[x] Adds descriptions to appropriate "what's new" file in docs/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Maintainer: Appropriate GitHub Labels and Milestone are assigned to the Pull Request and linked Issue.

Since this is a type issue, I decided to play around with annotating the functions related to filter_resample to get some more real world experience with annotations. It didn't take much to get the type checker to complain, but then I noticed a handful of inconsistencies with the documentation and the scope expanded a little bit.
@alorenzo175 any downside to adding these type annotations? I'm not ready to put in the effort to integrate them into the CI, but I don't think they hurt.
Still need to implement the fix.

As far as I know, the annotations don't hurt the runtime in any meaningful way

@lboeman can you confirm that this is fixing the api issue you ran into?

@lboeman can you confirm that this is fixing the api issue you ran into?

Yep, the event reports are processing completely and the summary data appears to be rendering correctly.


Add support for spacing token

Add spacings theme token for properties such as margin, padding, and more.
Mostly similar to #272 but I've introduced ValueConverter to simplify the transforming logic. So I'd like to get review of @taishinaritomi.

pause this until #272 getting merged

Ready for getting review

@poteboy Sorry to bother you but let me clarify, is there anything blocking this from being merged? I'd like to open next PRs.

@yukukotani
Sorry I forgot about it. I'll merge it right away 🙏


why the Chebyshev polynomials is needed?

When approximate the initial filters kernel gθ(Λ) = diag(θ) with a polynomial

why we can‘t simply combine the eigenvector and eigenvalue matrix together like

because
 *
The paper instead, use Chebyshev polynomials and also said that

I'm quite confused here. I don't know why the Chebyshev is needed?

+1, I was doing GCN paper review and read this paper today, came up with the same question.

I think in the cited wavelet on graph paper (Hammond et al.), they used chebyshev polynomial to approximate a known wavelet function because chebyshev interpolation has certain desired property, in the case where the filters are unknown, the chebyshev polynomials are not necessary, any linear combination of chebyshev polynomials up to Kth degree can be represented by a Kth degree polynomial.

Has anyone tried node classification on a general graph without chebyshev polynomials (e.g. using only the laplacian and its powers)? What was the accuracy compared to the chebyshev method?

See here:
https://www.youtube.com/watch?v=v3jZRkvIOIM.
min 26.

Has anyone tried node classification on a general graph without chebyshev polynomials (i.e. using only the laplacian and its powers)? What was the accuracy compared to the chebyshev method?

I have used the laplacian, and the normalized laplacian to train a convolution NN. The accuracy on the training, testing, and validation set were all above 85%.
The data I used was proxy related data and the model was for a security related to problem. However, doing that is how I found this work, and xaviers work as a whole... so I’ve seen it work, but Xavier Bresson and his team build on those concepts so well....
So, not the best modeling approach, but it is tractable for certain problems.

See here:
https://www.youtube.com/watch?v=v3jZRkvIOIM.
min 26.

Yes, it is stable under coefficient perturbation but otherwise the complexity is the same.

Any polynomial (monomials, Chebyshev, Legendre, etc.) of order K has the same representation power and can be used. We used Chebyshev because we had experience with it and thought it would be easier to optimize as they form an orthogonal basis. There's however not much difference in practice.
Chebyshev polynomials are more orthogonal (under the identity measure, and truly orthogonal with measure 1/sqrt(1-x²)) than monomials, but less than Legendre (truly orthogonal with the identity measure):

In the vertex domain (for a ring/circle graph), Chebyshev and Legendre polynomials are also more orthogonal than monomials:

I have the same question as @zeal-github , and opened a question in stack exchange. After reading this issue, I think the question is clear now. Chebyshev polynomials may have better property with respect to parameter orthogonality, but the computation complexity is the same as directly using laplacian matrix.

Any polynomial (monomials, Chebyshev, Legendre, etc.) of order K has the same representation power and can be used. We used Chebyshev because we had experience with it and thought it would be easier to optimize as they form an orthogonal basis. There's however not much difference in practice.
Chebyshev polynomials are more orthogonal (under the identity measure, and truly orthogonal with measure 1/sqrt(1-x²)) than monomials, but less than Legendre (truly orthogonal with the identity measure):

In the vertex domain (for a ring/circle graph), Chebyshev and Legendre polynomials are also more orthogonal than monomials:

Why you choose Chebyshev Polynomials of first kind instead of Legendre Polynomials?

Mostly for historical reasons. At first we were designing filters (e.g., to solve the diffusion of heat, the propagation of waves, and many others). As Chebyshev polynomials are excellent function approximators, we chose them to approximate those ideal filters we wanted to design. When learning filters, the choice of a polynomial basis doesn't matter for the expressivity of the filters, so we kept Chebyshev as we were familiar with them. We thought it could be easier to optimize, but that doesn't seem to make a difference in practice.

Yes. It seems that SGD works well enough to find the best polynomial, whatever the basis. That might also be because we use low-order polynomials when learning.

Yes. It seems that SGD works well enough to find the best polynomial, whatever the basis. That might also be because we use low-order polynomials when learning.

What is SGD?

Stochastic Gradient Descent.

Mostly for historical reasons. At first we were designing filters (e.g., to solve the diffusion of heat, the propagation of waves, and many others). As Chebyshev polynomials are excellent function approximators, we chose them to approximate those ideal filters we wanted to design. When learning filters, the choice of a polynomial basis doesn't matter for the expressivity of the filters, so we kept Chebyshev as we were familiar with them. We thought it could be easier to optimize, but that doesn't seem to make a difference in practice.

I have another big problem. Does the orthogonal polynomials causes low-pass filter? Or Spectral CNN itself is a low-pass filter GNN?

Polynomial convolutions don't force low-pass filters. They enforce local filters (the locality is controlled by the order of the polynomials). The low-pass limitation is specific to GCN [Kipf et al.] as they merged the 0- and 1-neighborhood in the quest of learning a single parameter per filter.

more orthogonal

What do you mean more orthogonal?

Any polynomial (monomials, Chebyshev, Legendre, etc.) of order K has the same representation power and can be used. We used Chebyshev because we had experience with it and thought it would be easier to optimize as they form an orthogonal basis. There's however not much difference in practice.
Chebyshev polynomials are more orthogonal (under the identity measure, and truly orthogonal with measure 1/sqrt(1-x²)) than monomials, but less than Legendre (truly orthogonal with the identity measure):

In the vertex domain (for a ring/circle graph), Chebyshev and Legendre polynomials are also more orthogonal than monomials:

What do you mean more orthogonal?


No longer needed. Nova docker can now detect if docker image is missing

and pull it itself.

"can now detect" - also for the oldest version where we support docker - cloud5?

No, I think this is gonna be cloud6 only. While we were testing Cloud5 (and even earlier Cloud6), that docker pull call was needed.

@jsuchome looks like you need to update the PR then to only not doing it for iscloudver 6plus..

Ah, right

+1

+1


what versions of the libraries do you use?

do you use tensorflow 1? because using 2 doesn't work.

I don't think these codes work either.


Link reactions to EVE Tycoon

https://evetycoon.com/industryCalculator/46208
depends on #472

Already done


Native soft delete support?

Are there any plans to implement native support for Soft Deletes?
I'm aware there are extensions that are doing this, e.g. https://github.com/yii2tech/ar-softdelete but really should it be necessary to have to do all of this manually? Laravel handles it so elegantly, maybe worth looking at their implementation:
https://laravel.com/docs/5.6/eloquent#soft-deleting

What is wrong with yii2tech/ar-softdelete?


Update cats-core, cats-laws to 2.3.1

Updates

org.typelevel:cats-core
org.typelevel:cats-laws

from 2.0.0 to 2.3.1.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

Applied Migrations

github:typelevel/cats/Cats_v2_2_0?sha=v2.2.0

Documentation:

https://github.com/typelevel/cats/blob/v2.2.0/scalafix/README.md#migration-to-cats-v220

labels: library-update, semver-minor, scalafix-migrations

Superseded by #260.


Fix example function name in the text

This was independently fixed in https://github.com/elm-lang/elm-lang.org/commit/3bfc5f7138a545169f7f94dda3b0af396220bc96, so the pull request here can be closed.

Okay, thanks for the review @jvoigtlaender, and thanks for the fix @jafrog! I'm sorry I did not get to it in the right order :(


./scripts/init.sh

this code can't run because it has erros as the flowing.
running build_ext
building 'C_ROIPooling' extension
gcc -pthread -B /home/anaconda3/envs/vl-bert/compiler_compat -Wl,--sysroot=/ -Wsign-compare -DNDEBUG -g -fwrapv -O3 -Wall -Wstrict-prototypes -fPIC -DWITH_CUDA -I/tmp/pycharm_project_61/common/lib/roi_pooling -I/home/byd/.local/lib/python3.6/site-packages/torch/lib/include -I/home/byd/.local/lib/python3.6/site-packages/torch/lib/include/TH -I/home/byd/.local/lib/python3.6/site-packages/torch/lib/include/THC -I/usr/local/cuda/include -I/home/anaconda3/envs/vl-bert/include/python3.6m -c /tmp/pycharm_project_61/common/lib/roi_pooling/vision.cpp -o build/temp.linux-x86_64-3.6/tmp/pycharm_project_61/common/lib/roi_pooling/vision.o -DTORCH_EXTENSION_NAME=C_ROIPooling -std=c++11
cc1plus: warning: command line option ‘-Wstrict-prototypes’ is valid for C/ObjC but not for C++
In file included from /tmp/pycharm_project_61/common/lib/roi_pooling/ROIAlign.h:4:0,
                 from /tmp/pycharm_project_61/common/lib/roi_pooling/vision.cpp:3:
/tmp/pycharm_project_61/common/lib/roi_pooling/cpu/vision.h:3:29: fatal error: torch/extension.h: 没有那个文件或目录
 #include <torch/extension.h>
                             ^
compilation terminated.
error: command 'gcc' failed with exit status 1

If you have ways, please tell me.

What is the version of your pytorch?

pytorch1.1 0 I just follow your code to create new environment vl-bert. But the apex and. /scripts/init. bash code can't run.  The other steps go well.
 I guess whether the roi component should be compiled in other important procedures that you omitted？
To deal with the problem as the following but it also doen't work.
I install the warpctc pytorch1.0.0 but I don't know if the project needs warpctc-pytorch.
Thanks for your timely reply.
If you have ways, please tell me.
---Original---
From: "Weijie<EMAIL_ADDRESS>Date: Wed, Dec 30, 2020 11:52 AM
To<EMAIL_ADDRESS>Cc: "Author"<author<EMAIL_ADDRESS>Subject: Re: [jackroos/VL-BERT] ./scripts/init.sh (#69)
What is the version of your pytorch?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Dear scholar, I use pip list ,I found torch 0.4.0.   I use conda list, I found pytorch 1.1.0   py3.6_cuda9.0.176_cudnn7.5.1_0  pytorch

I upgrade my torch to 1.1.0 consistent with the version of Pytorch. 1.1.0 and it could run a long way.hah


Intermittent failures

We've recently upgraded from to 0.130.1 from 0.111.0 and are seeing intermittent failures particularly on helmfiles with multiple repositories.
This is the error:
 Building dependency release=istio-collector, chart=charts/istio-collector
 Hang tight while we grab the latest from your chart repositories...
 ...Unable to get an update from the "local" chart repository (http://<IP_ADDRESS>:8879/charts):
 	Get "http://<IP_ADDRESS>:8879/charts/index.yaml": dial tcp <IP_ADDRESS>:8879: connect: connection refused
 ...Successfully got an update from the "aws" chart repository
 ...Successfully got an update from the "stable" chart repository
 Update Complete.
 Saving 1 charts
 Deleting outdated charts

The helmfile.yaml:
---
releases:
- name: istio-collector
  namespace: istio-system
  chart: ./charts/istio-collector
  values:
  - environments/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/shared/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/istio-collector/values.yaml

- name: istio-prometheus
  namespace: istio-system
  chart: ./charts/istio-prometheus
  values:
  - environments/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/shared/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/istio-prometheus/values.yaml

- name: app-prometheus
  namespace: app-metrics
  chart: ./charts/app-prometheus
  values:
  - environments/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/shared/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/app-prometheus/values.yaml

- name: prometheus-alertmanager
  namespace: istio-system
  chart: ./charts/prometheus-alertmanager
  values:
  - environments/{{ env "HELMFILE_ENV" }}/shared/values.yaml
  - environments/{{ env "HELMFILE_ENV" }}/alertmanager/values.yaml
  secrets:
  - environments/{{ env "HELMFILE_ENV" }}/alertmanager/secrets.yaml

- name: prometheus-pushgateway
  namespace: istio-system
  chart: ./charts/prometheus-pushgateway
  values:
  - environments/{{ env "HELMFILE_ENV" }}/shared/values.yaml
  set:
  - name: kubectl.tag
    value: '{{ requiredEnv "GO_DEPENDENCY_LABEL_KUBECTL" }}'

- name: kube-state-metrics
  namespace: istio-system
  chart: ./charts/exporters/kube-state-metrics

- name: helm-platform-node-exporter
  namespace: core-system
  chart: ./charts/exporters/node-exporter

- name: stackdriver
  namespace: core-system
  chart: ./charts/exporters/stackdriver
  values:
  - environments/{{ env "HELMFILE_ENV" }}/shared/values.yaml

- name: helm-prometheus-redis-exporter
  namespace: core-system
  chart: ./charts/exporters/redis-exporter
  values:
  - environments/{{ env "HELMFILE_ENV" }}/redis-exporter/values.yaml
  set:
  - name: image.tag
    value: '{{ requiredEnv "GO_DEPENDENCY_LABEL_REDIS_EXPORTER" }}'

Potentially linked: https://github.com/helm/helm/issues/7526 - although not sure why it's just started happening?

Changed the title to reflect the fact we still see intermittent failures even with the local repo removed

@Stono Thanks for reporting. That's very odd and interesting - Do you actually have a chart repository named local added to your system before running Helmfile, or is it included in e.g. your istio-collector's Chart.yaml dependencies section(helm 3), or requirements.yaml(helm 2)?

Hey @mumoshu,
local is added by default by helm.  Just to rule out some variance - I removed all charts (including local as we don't need it) from helm apart from our remote s3 chart:
❯ helm repo list
NAME	URL
aws 	s3://atuk-prod-helm-charts

Which then gave us a different error:
 STDERR:
   WARNING: Deprecated index file format. Try 'helm repo update'
   Error: no API version specified

 COMBINED OUTPUT:
   Hang tight while we grab the latest from your chart repositories...
   ...Successfully got an update from the "aws" chart repository
   Update Complete.
   WARNING: Deprecated index file format. Try 'helm repo update'
   Error: no API version specified]

So then I noticed we only saw this where we have multiple releases in helmfile, so I tried setting concurrency=1.
We haven't seen any errors since setting concurrency=1.

@Stono Thanks for the info - good to know. Seems like the local repo is added while running helm dep build? Then, it must have something to do with the fact that Helmfile running helm dep build concurrently against all the involved local charts.
Probably we have to disable concurrency there, to run helm dep build sequentially.

I think that sounds like a sensible approach @mumoshu !

@Stono Thanks for confirming! #1532 should fix it.


Using max_staleness_ms > 0 results in a crash

It seems, that some areas of the code was never tested 😱!

defp filter_out_stale(servers, topology, max_staleness_ms) do


Tx fee

fix #124
gas price is fetch from database block table gas price column
gas used is accumulated from transaction outcome and receipt outcome gas_burnt result
tatol gas fee is gas price times gas used

@icerove "Gas Used" is not measured in N

@icerove it's not bytes, it's just Gas

@ilblackdragon what is the unit of gas? really confused if it is not bytes or near N

Gas is the unit. It's an equalizing unit of bandwidth and compute.
See:

Section 4.1 in https://near.ai/economics
https://blockgeeks.com/guides/ethereum-gas/

Also:

"Total gas cost" probably should say "Transaction fee".
And there is also "Attached gas" for FunctionCalls, e.g. what user specified as allowed amount of gas for this function.

For example you can see fields here - https://etherscan.io/tx/0xacdc5837d037c3f2c34b870653338b0bc3673db3e62fa24985cfd261f2ea9fe6 (click see More) , "attached gas" they call "gas limit".
They also have ? around each term to familiarize people with concepts, which a good practice.

@ilblackdragon

And there is also "Attached gas" for FunctionCalls, e.g. what user specified as allowed amount of gas for this function.

what do you want to do with this? make an extra column to put it or add to total gas?

Add an extra column.

left attach gas not added

will fix the test error and then merge

@icerove I wonder if we can turn off componentDidMount triggers when we render with jest as it seems that it does not take into account anything beyond initial render anyway.

will make a new PR to figure out that