unsupported type "invalid type" error when using Linux binary

Problem statement
I'm getting a unsupported type "invalid type" when generating a spec using the v0.25.0 linux binary. I don't get this error when using the darwin binary. It's also fixed if I remove any references to a schema outside of the package.
Swagger specification
// SyncStatusResult represents the status of a request
//
// swagger:model SyncStatusResult
type SyncStatusResult struct {
	// Enum: queued,success,failure
	SyncStatus ss.SyncStatus `json:"sync_status"`
	Status string `json:"status"`
}

Steps to reproduce

download v0.25.0 linux binary and darwin binary
run swagger.sh generate spec -m -o swagger.json with above code and linux binary and get unsupported type "invalid type" error
remove reference to ss package in above code and don't get errors
run with darwin binary with above code and get no errors

Environment
swagger version: v0.25.0
go version: 1.14.4
OS: linux

To investigate this it would be great to have a more complete reproduction case.
Have you tried this in a docker container/clean env?

I apologize for leaving out some details. I'm running this in a ubuntu:20.04 docker container and the go version is 1.14.4.

Any updates here? I am also facing the same issue with the alpine image.

/prj # cat /etc/*-release
3.12.0
NAME="Alpine Linux"
ID=alpine
VERSION_ID=3.12.0
PRETTY_NAME="Alpine Linux v3.12"
HOME_URL="https://alpinelinux.org/"
BUG_REPORT_URL="https://bugs.alpinelinux.org/"
/prj # uname
Linux
/prj # swagger  --log-output=/tmp/swagger.log generate spec -o  ./app-resources/swagger.json -w .
unsupported type "invalid type"
/dm # swagger version
version: v0.25.0
commit: f032690aab0634d97e2861a708d8fd9365ba77d2
/prj #

@ShriprasadM could you please include some valid go source to be able to reproduce the case?

Somehow, for this sample, I was able to reproduce it on macOS. But, actually, I am facing an issue with the Linux library.

Download sample.zip
cd ~/Downloads/sample

My Swagger command version   (MacOs)
version: v0.25.0
commit: f032690aab0634d97e2861a708d8fd9365ba77d2

swagger generate spec . (Run this command)
unsupported type "invalid type"

how does that project compile? The project in sample isn't valid go the imports can't work

I have not focused on project compilation it may not work. I was preferring to look for catching the swagger command error

does your actual application compile?

Yes. it compiles

if you run with the env var DEBUG=1, does it tell you which type is wrong? Can you share the type?
it would be very helpful to see both the content the file that has ss.SyncStatus and the import definition used in the file that has:
// SyncStatusResult represents the status of a request
//
// swagger:model SyncStatusResult
type SyncStatusResult struct {
	// Enum: queued,success,failure
	SyncStatus ss.SyncStatus `json:"sync_status"`
	Status string `json:"status"`
}

as well as how their paths are related on the filesystem? (one is a module or a sub/sibling/parent/unrelated)

This codebase is shared by @ahobbs-cb . Let me try to share my working version of code

2020/12/08 13:40:15 building parameters for: details
2020/12/08 13:40:15 build from type VehicleDetails: *types.Named
2020/12/08 13:40:15 field Vehicle: invalid type(*types.Basic) [""] ==> in: body
2020/12/08 13:40:15 build from field Vehicle: *types.Basic
unsupported type "invalid type"

sample.zip
Please use this zip file
Extract it at /tmp
Run following commands

export GOPATH=go env | grep GOPATH | awk -F "=" '{print $2}'|xargs:/tmp/sample
echo $GOPATH
cd sample/
go run b.go

I am using linux (go1.15.3).
From your screenshot above, the generate spec fails on Mac AND linux with the same error.
My hunch on how this works is that to make generate spec work, the code needs to build correctly.

This works:
GO111MODULE=off go run b.go
{{1234}}

and so does the spec generation:
GO111MODULE=off swagger generate spec ./...
{
  "swagger": "2.0",
  "paths": {
    "/postit": {
      "post": {
        "tags": [
          "tag1"
        ],
        "operationId": "details",
        "parameters": [
          {
            "name": "Vehicle",
            "in": "body",
            "schema": {
              "$ref": "#/definitions/Vehicle"
            }
          }
        ]
      }
    }
  },
  "definitions": {
    "Vehicle": {
      "type": "object",
      "properties": {
        "vehicleno": {
          "type": "string",
          "x-go-name": "No"
        }
      },
      "x-go-package": "a"
    }
  }
}

This fails:
 GO111MODULE=on go run b.go
b.go:4:2: package a is not in GOROOT (/usr/local/go/src/a)

This works:
b.go:
import (
	"fmt"
	"sample/src/a"
)

go mod init sample

 GO111MODULE=on go run b.go
{{1234}}

I can see, if the package is no under the src directory above issue is occurring. See the following screenshot. Execute same commands on Linux and macOS respectively

This is what I have observed

export GOPATH=`go env | grep GOPATH | awk -F "=" '{print $2}'|xargs`:/tmp/sample

If you execute this step, go run will work

When can we expect the fix for Linux build?

@fredbi any updates on this

@ShriprasadM as I've shown above, running the generate command with the proper environment set up just worked.
I failed to see any evidence of how this works differently on macOS (the screenshots above essentially show the same failure).
There might be some uncovered edge case in the way we resolve GOPATH with multiple values. However, this can be worked around rather easily.
If you feel strongly about it, you can contribute a PR. What we do for packages identification just relies on the standard library here https://github.com/go-swagger/go-swagger/blob/c36f768fdb217769eb0242a9e48ca9966a40a402/codescan/application.go#L14


Improve the performance of sbgemm_tcopy on neoversen2

Based the implementation of https://github.com/xianyi/OpenBLAS/pull/3793 by @imzhuhl , I use neon to improve the performance of sbgemm_tcopy in this PR.
I benchmarked cblas_sbgemm (m=n=k=1000) on neoversen2(2.75GHz), and the performance improved from 134.52468 GFLOPS to 143.59852 GFLOPS.

I found that sbgemm_ncopy_8_neoversen2 and sbgemm_tcopy_4_neoversen2 are missing, which will lead to get the wrong results from calling cblas_sbgemm(CblasColMajor, CblasTrans, CblasTrans...). To fix this bug, I implemented these two functions.

Based the implementation of #3793 by @imzhuhl , I use neon to improve the performance of sbgemm_tcopy in this PR.
I benchmarked cblas_sbgemm (m=n=k=1000) on neoversen2(2.75GHz), and the performance improved from 134.52468 GFLOPS to 143.59852 GFLOPS.
Great, I confirmed that the performance has been improved.

I found that sbgemm_ncopy_8_neoversen2 and sbgemm_tcopy_4_neoversen2 are missing, which will lead to wrong results from calling cblas_sbgemm(CblasColMajor, CblasTrans, CblasTrans...). To fix this bug, I implement these two functions and add a new commit.
We should consider the case of Trans. Thank you for the correction.
But there may be a small mistake in sbgemm_ncopy_8_neoversen2.c:

The unroll number is 8 now and We may need to modify the code.

The unroll number is 8 now and we may need to modify the code.
Fixed, thanks for your reminder!

Oops, sorry for almost forgetting to merge this


Error When Setting a Tag on a Load Balancer

Related command
az network lb update
Describe the bug
When setting a tag that is an 'int', e.g. az network lb update -g ... -n ... --set 'tags.label=1544' we get the following error:

invalid argument: Expect <class 'str'>, got 1544 (<class 'int'>

To Reproduce
Run the command above ensuring that the tag value is an integer value.
Expected behavior
The 'int' value should be set as the tag value
Environment summary
Windows-10-10.0.19045-SP0
Python 3.10.8
Installer: MSI

azure-cli 2.43.0

Additional context

This has been tested on azure-cli 2.36.0 and it works.
We have also tested 2.43.0 on Ubuntu and get the same error.
We have tested using the 2.43.0 Docker image and get the same error.
We can set tags as long as they aren't integers (e.g. az network lb update -g ... -n ... --set 'tags.label=a1')
We can set int tags using other sub-commands (e.g. az vm update -g ... -n ... --set 'tags.label=1')

@necusjz for awareness

Just done some additional tests using the Docker image, this issue first appears in 2.42.0 (2.41.0 is fine)

@andrewpleasants-bjss-nhs We can take az network lb update -n {} -g {} --set tags.label=1544 --force-string as a workaround, and we'll expose --tags in Azure CLI 2.44.0 (https://github.com/Azure/azure-cli/pull/25035, around 01/10/2023).
After that, we can use:
az network lb update -n {} -g {} --tags label=1544


issue in rendering

in ur game a scene is being rendered above the other scene wich may be annoying i think we can fix this easily ill see if i can fix it if someone is already fixing it please comment so we dont have any conflicts in the code

i dont think this is rly important until now

What do you mean by rendered above the other scene? during the main menu or battle or just how the game rendered in general?

like for example when in the menu i press this occurs and also i cant see the start button

i think this is happening because your drawing the buttons before the details screen wich triggers this issue this on smaller screens

thats wat i meant if i increase my res a bit i can see half of the start button but its underthe details pane

oh ok, that look really weird. yeah, it is very likely the issue with a smaller screen. I will see what I can do

i think its pretty easy just draw the buttons above the details pane

oh wait i jsut realized thats not the issue of drawing its the issue of the screen size

to fix this one of the approaches is making the details pane smaller

depending on the screen size

yeah, I did that for the most sprite in the menu already, guess I forgot to add that to the details pane.

what is your screen resolution, if you don't mind telling?

yes its not a big problem btw, congrats ur game is rly good but tbh i cant find out how to play it correctly i think u have to finish the enclopyedia screen huuh i cant spell it correctly

yes wiat
its about 1024 by 600

wait ill tell u exact numbers

its 1920x1080 but i set the game screen to 1024x600 cuz none of the default resolutions are working for me

weird, when you say not working. The game simply can't start at a higher resolution? or just you can't change it.

no no no it works but its higher than my screen

Well, that quite an unexpected problem. As it works fine for me on all option. I will see what I can do about this.
And yeah I will work on the manual/tutorial at some point. Mostly working on getting the main function to work right now. I will probably begin adding on that in version 0.7.

maybe adding fullscreen option will help a bit

yes all options work with u cuz i think ur using a high res screen but my screen is a bit low res

ok, I try update a quick fix for this but no idea how it will really help. really tough working on the compatibility issue with only a laptop in hand :l

At least the button will always show up above the detail pane now I think. even if the UI scale looks ugly. I wonder why it look so weird on your screen though. my laptop also uses 1920 x1080 screen resolution (but it has 3840 x 2160 recommended). I will try look for other people to check this issue to see what the real issue is.

i think its the issue with sprite sizes

the best fix in my opinion is changing the sprite size depending on the screen size

i mean window size*

I already did that though. But I use the number from the configuration file instead of pygame. I am thinking that maybe the real issue is where your pygame automatically scale the window based on your screen so it no longer matches the scale number option.

yes i think thats a good idea

u can implemnt it easily in fact i wnated to do that and puch a pull request but i cant rly find where the window setup code rly is

I am on it. but if you want to have a look, it would be in main.py at line 36 (that get config resolution number) and line 135 (where the display is setup)

My guess is the pygame.RESIZABLE flag in pygame.display.set_mode at line 136 that cause this issue but I can't really confirm it. Unless you can try delete this part and test, I will need to try to test on the virtual machine and see if it related.

thats not related at all in fact when u resize it just puts the content of the window in the corner and everything stays the same size but we can do is using the screen info library check the users screen size and depending on that we set the default resolution that suits that screen size

so wait will u work on it or do i cuz i dont think u will enjoy any conflicts

It will be hard for me to check if I really fix or not on my end. So if you can fix it then yes please. and thanks a lot.

np ok ill work on it will take a few minutes anyways
testing it is easy just go to ur os settings and try different screen resolutions and see if it works

I have tried changing to various in the os setting from 3840 x 2160 to 1280 x 720. And can't seem to replicate the issue shown on your screenshot that why I am not really sure what I tried really work or not. The UI always scale properly on my end somehow.

its because of the window size ur not handling resizing sprites properly i guess try chaging ur window size to 1024 by 600

height 600 width 1024

yeah, I tried that on the game config and get a similar result as yours now. at least my quick fix make the button show up with that resolution at least but my scale equation probably cause this problem.

at least the battle ui still look fine with 1024 x 600 (phew), ok I will fix this problem. no worry :)

oh ok i fixed the problem of the initial screen size but i have a problem look

basically i made instead of statically settign the screen size for the game it automatically sets one depending on the screensize
but im getting this error huh

did you delete this line in main.py? :    Soundvolume = float(config['DEFAULT']['SoundVolume'])

it used to be like this:
ScreenHeight = int(config['DEFAULT']['ScreenHeight'])
ScreenWidth = int(config['DEFAULT']['ScreenWidth'])
FULLSCREEN = int(config['DEFAULT']['Fullscreen'])
Soundvolume = float(config['DEFAULT']['SoundVolume'])
Profilename = str(config['DEFAULT']['playername'])

yeah when I comment that line out I get your error.

no no no infact look

mhmm ok this is weird.  can you put up pull request? I will have a full look.

ok

OOH LOL heres the error
theres an error occuring in my code that i added basically llook

my code is producing float numbers

ill fix it with floor division //

ah lol ok :P

xd, im actually having fun helping you in this

haha well glad to hear :) and thanks. Hopefully, you will like the game when it gets full 1.0 release. still a long way from finish.

yes

i fixed it ill send a pull request so u can merge the code

sent it 👍🏻

u can try it just delete the configuration.ini and run main.py

This manage to fix even the small resolution problem?

its perfect
it basically when u first run the game sets the resulotion of ur window to ur screen res

it fixed my issue too

ah, that's superb. Thanks again.

np

lets close this issue now

ok gotta add dependency for screeninfo as well, I will do that

i did that in requirements.txt


Reporting success rate of unbounded attacks is meaningless

Three of the attacks presented (EAD, CW2, and BLB) are unbounded attacks: rather than finding the “worst-case” (i.e., highest loss) example within some distortion bound, they seek to find the closest input subject to the constraint that it is misclassified. Unbounded attacks should always reach 100% “success” eventually, if only by actually changing an image from one class into an image from the other class; the correct and meaningful metric to report for unbounded attacks is the distortion required.

Three of the attacks presented (EAD, CW2, and BLB) are unbounded attacks: rather than finding the “worst-case” (i.e., highest loss) example within some distortion bound, they seek to find the closest input subject to the constraint that it is misclassified. Unbounded attacks should always reach 100% “success” eventually, if only by actually changing an image from one class into an image from the other class; the correct and meaningful metric to report for unbounded attacks is the distortion required.

We definitely agree with you that only reporting the success rate of attacks is not as useful. Actually, it is the initial motivation for us to evaluate other performance metrics of attacks such as imperceptibility, robustness, and computation cost in our paper. And also, we reported the L0, L2, and L\inf distortion performance of EAD, CW2, and BLB in Table III.

So definitely it's good that you do report it somewhere, but nevertheless it's not meaningful to talk the success rate of unbounded attacks. Again, you may want to read https://arxiv.org/abs/1902.06705.


Fix atomics

This pull request fixes few conformance tests

Forgot to commit the last fix

I also started pre-checkin tests

This looks OK to me.

Ran some tests locally. I will monitor the merge job.


Missing configuration when trying to use SSR and not using solid-start

Just a heads up if anyone is trying to add this to just a Vite TS app without using Solid-Start and getting this error when trying to use Vite SSR.
Error: Cannot find module '~/lib/utils' imported from '[project path]/src/components/ui/button.tsx'
Had to manually add some changes, here:
/* vite.config.ts */
+ import { resolve } from 'node:path';

export default defineConfig({
  plugins: [solid({ ssr: true })],
  + resolve: {
  +  alias: [{ find: '~', replacement: resolve(__dirname, './src') }],
  + },
});

/* tsconfig.json */
  "paths": {
      "~/*": [
        "./src/*"
      ]
    },
  +  "baseUrl": "./",

Thanks for the Feedback.
Even though you already closed the issue I'll have a look at it since the fix could be similar to the one for astro.

Thanks for the quick reply. I'm still tinkering with it. I might end up using Solid Start anyway just because the TypeScript the way I have it set up is giving me linting errors. So I'm trying to figure out what I'm missing.
Unrelated thing but a quick request: when using the CLI to download components, they default to line endings CRLF (/r/n Windows line endings). Would be nice to have the option to chose Unix line endings instead: LF (/n). I'm on a Mac, so I have to go through every file each time in VSCode and manually change it. It's not a big deal since this is just a personal preference, but would be nice if it were an option.
You're doing great work, thank you!

Thanks for the input.
I switched from windows to unix linefeeds since windows can handle both anyway.
I bumped the version to v0.3 but since the changes are in the online registry it's already active even without an update to the cli.

I am running into the same issue, despite having added the suggested changes to vite.config.ts and tsconfig.json. These fixes seem to work for npm run dev but not for npm run buid. The problem is probably down to this issue with Vite.


Create StackUsingLL.cpp

The solution to implementing stack using a linked list. Please review it.
Regards,
Siddharth

LGTM :+1:


Mark items with shared retention agreements with a CGD of "Committed" when accessioned into SCSB

User story
Items marked for retention commitment in ReCAP will have the proper Collection Group Designation (CGD) of 'Committed' assigned in SCSB after accession.
Acceptance criteria

[ ] For an item that returns the values in the API response described in the notes, the CGD will have to be 'Committed' instead of 'Shared'.

Notes

The API call that retrieves the item information (/almaws/v1/bibs/{mms_id}/holdings/{holding_id}/items/{item_pid}) has
"committed_to_retain": {     **"value": "true"**,     "desc": "Yes"   },
in the item_data section, AND in this section:
"retention_reason": {     **"value": "ReCAPItalianImprints",**        "desc": "ReCAP Collaborative Collection Development Italian Imprints"   },   "retention_note": "" }

For now, this is the only shared retention program we are participating in, but there will be others ("retention_reason" "value").

Concrete example
In the sandbox, see https://api-na.hosted.exlibrisgroup.com/almaws/v1/bibs/99126831126106421/holdings/22961480120006421/items/23961480110006421
The barcode is<PHONE_NUMBER>7524.
Re-index required?

[ ] Full re-index
[ ] Partial re-index
[x] No re-indexing required

Implementation notes, if any
Records are anticipated to be loaded in March 2023.

The updated CGDs will be included in the response of the bibdata barcode controller that SCSB's accession processing calls.

https://github.com/pulibrary/bibdata/blob/main/app/controllers/barcode_controller.rb#L14

The desired end result is that:

The SCSB goes to https://bibdata.princeton.edu/barcode/[BARCODE_OF_SCSB_ITEM]/scsb
If the alma endpoint @mzelesky documented has "committed_to_retain": { "value": "true", "desc": "Yes" }, bibdata will return a MARC record that includes an 876$xCommitted, rather than 876$xShared.
If the alma endpoint doesn't have this committed_to_retain info, then keep the current behavior.

I updated the description with a ReCAP record to allow for better testing.

3. committed_to_retain

@mzelesky and I reviewed again the ticket and we should also consider the retention_reason with value "ReCAPItalianImprints". We expect to have more values in the future. Some values will not result in a "Committed" CGD.
When a new value comes @mzelesky will create a ticket.

The desired end result is that:

The SCSB goes to https://bibdata.princeton.edu/barcode/[BARCODE_OF_SCSB_ITEM]/scsb
If the alma endpoint @mzelesky documented has "committed_to_retain": { "value": "true", "desc": "Yes" }, bibdata will return a MARC record that includes an 876$xCommitted, rather than 876$xShared.
If the alma endpoint doesn't have this committed_to_retain info, then keep the current behavior.

@mzelesky and I reviewed again the ticket and we should also consider the retention_reason with value "ReCAPItalianImprints". We expect to have more values in the future. Some values will not result in a "Committed" CGD.
When a new value comes @mzelesky will create a ticket.


Is it possible to get all the devices in one request?

Hey guys.
Nice API!
I needed all the devices, either in a json file or doing only one request. It's for a school project lol
Is that possible?
Thanks!

Sorry it is not possible. you can get maximum of 100 devices per request.
Thanks

Hi @shakee93 ,
Very good job on the API. I have a similar question: If the API limits the result to 100 devices per request, then how to get the next 100? I am trying to get all samsung devices.

Sorry currently its a no.. i am working on the second version of the api. ill try to add this to the list.
Thanks

Thanks @shakee93  for considering this in the next version.
Any estimates on the timeline of the next api?

I have already started working on the second version. but unfortunately i cannot guarantee a timeline as i am maintaining this with my full time job. ill try to publish it as soon as possible.
Thank you


The Sample Size Atoms (stsz) are incorrectly parsed

The sample size atom structure differs slightly from the others, as it has an extra 4 bytes after the version and flags to store the sample size in case its the same for all the samples.
If not, the the sample size entry is set to 0 and the sample size table is used instead. Said table contains an an entry for each sample in the media's data stream.
Source
Because of this error, the parser returns as if the sample size table has no entries.

I think @jhiesey recently fixed a lot of outstanding issues. Can you try 2.0.0 and see if this is still broken?

Its fixed now. Great work @jhiesey!


PowerToys Run still being triggered by ALT+SPACE when disabled.

Environment
Windows build number: Microsoft Windows [Version 10.0.18363.900]
PowerToys version: v0.19.0
PowerToy module for which you are reporting the bug (if applicable): PowerToys Run

Steps to reproduce

Disable PowerToys Run.
Press ALT+SPACE.

Expected behavior
Nothing happens because PowerToys Run isn't enabled.
Actual behavior
The PowerToys Run pop up appears.

@dtslubbersen
are you running PowerToys as administrator?
In that case this is a duplicate of https://github.com/microsoft/PowerToys/issues/4569
As a workaround restart PowerToys.
The fix will be available in 0.20.
Thanks.

I'm running PowerToys (preview) v0.25.0 and ALT+SPACE is "eaten" whether running PowerToys as Admin or not, and whether PowerToys Run is disabled or not. If PowerToys Run is enabled then I the search box pops up, if it is disabled nothing happens (no character passed to app). Changing the shortcut key to CTRL+ALT+SPACE does not help (ALT+SPACE is still "eaten").
I want to pass ALT+SPACE through to emacs as it is used as a command therein.


ESM Support

Summary
Hi Everoyne!
Creating this issue cause at jpg.store we were trying to use  Blaze (which internally uses cardano-js-sdk)and at the same time using the sdk directly for a lot of datum building and other cardano relate stuff and noticed several issues while using the library in ESM mode.
Talking with people at butane, they had issues with ESM too, so it seems its not an problem only for us.
@pocesar ( our js wizard) did an analysis of the codebase and even try fixing it with no luck. Here there is a rough summary of the findings he got and some output he gave us.

If you try to use this library in a "type": "module" (ESM) codebase it will fail because the generated libraries in dist/esm folder aren't valid ESM modules.
We tried to fix the ESM build step, but it's nearly impossible with code changes, since some files are mixing CJS and ESM dependencies, using import { } from 'dep' when it's a CJS and vice-versa. ESM modules doesn't have a require global function like CJS (the same goes to all CJS specific globals, like require.resolve,  __dirname, __filename).
Some dependencies got a mismatching .d.ts types for ESM and CJS, that makes the build break as well.
Package manager vendor specific hacks, like workspace:~ makes it even harder to make changes.

@pocesar and also @micahkendall and @ezepze from butane recommended actually using a build step through something like esbuild, rollup, tsup instead of relying on correctness from the tsc compiler.
Think this is a major issue to try to address as ESM is starting to get more and more adoption and CJS is starting to almost be unproductive to create new codebases with given a lot of the newer libraries are starting to ship with esm support only.
Steps to reproduce the bug
https://github.com/joacohoyos/cardano-sdk-esm-issue
Here there is a repo which you already start getting issue with if you try to run it.
It start with libsodium issues, but even if you fix those you then start getting different issues with different pieces of the library.
According to @pocesar another  quick way to see the issue is doing a  --target ESNext --module ESNext --moduleResolution Bundler inside each workspace module.
Actual Result
Getting multiple issues when using with ESM
Expected Result
Library works both with CJS and ESM environemntes
SDK version
^0.35.2
Environment type

[X] Node.js
[ ] Browser
[ ] Browser Extension
[ ] Other

Environment details
Node: 20.9.0

Hi, we're planning to replace tsc with a bundler and include some transpiled dependencies. In a meantime, consumer apps in Node.js have to either use cjs or a bundler

@joacohoyos please try with version 0.35.4

Seems to be working. Thanks so much for the work done


Control panel with active admin tasks is not present

Control panel with UI support for executing administrative tasks, is not provided in the actual version.

Sorry, is this about the "look&feel," ie. there's no single place to have all these. Or is it rather about the functionality hidden under one (or multiple) of the tabs?


is function calling supported?

I have tested this model with same system prompt of GLM4. This model only emits parameters, but not the function name. For example, here, get_weather is not generated:
You  > what's the weather like in beijng?
A.I. >
{"city_name": "beijing"}

I have tested this model with same system prompt of GLM4. This model only emits parameters, but not the function name. For example, here, get_weather is not generated:
You  > what's the weather like in beijng?
A.I. >
{"city_name": "beijing"}

Hi, we uploaded a function calling demo, you may check it.
demo

Got it. I forgot to have a check on tokenization_chatglm.py.
I managed to do a full demo here:
https://github.com/foldl/chatllm.cpp/blob/master/docs/tool_calling.md#codegeex4


Add possibility to receive custom data from datatables

Add possibility to receive custom data from datatables, usefull when you want to filter by column on server-side

Thank you for your PR, it has been released in the new version 2.1.4. I only added a unittest for the custom data processing.

Nice!
Thank you for your bundle.
El El dom, 11 jun 2017 a las 9:47, Artem Rodygin<EMAIL_ADDRESS>escribió:

Thank you for your PR, it has been released in the new version 2.1.4. I
only added a unittest for the custom data processing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/webinarium/DataTablesBundle/pull/2#issuecomment-307612574,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABoJ5rxqB6-tOi9siYcgrJ9W3DjxboyYks5sC5uagaJpZM4Nz4GR
.


vagrant: build and install vagrant-libvirt plugin

Motivation for this change
I was having trouble installing vagrant-libvirt with vagrant plugin install vagrant-libvirt, and I think it's cleaner to just let Nix manage everything.
#26202
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@calbrecht could you have a look at this?

sure, i will

@calbrecht I think you might have accidentally put the wrong hash. 0gzfmcywp1da8nzfqsql2zqi648mfnx6qwkig3cv36n9m0yy676y seems to be the hash for netrc, as seen here: https://github.com/NixOS/nixpkgs/blob/92a047a6c4d46a222e9c323ea85882d0a7a13af8/pkgs/servers/monitoring/sensu/gemset.nix#L198-L205

Thanks!


reterraforged-0.0.4 on server side error [1.19.4]

When I try to start my server (there is only one mod and it is reterraforged) I get an error that the mod is trying to load the UI, screenshot below:

I tried adding a condition that loads the UI only on the client side, but that didn't do anything.

Also when I load the client part I get warning:

What could have gone wrong?
Could you please check it out from your end?
Thank you!

you have to move registerPresetEditors to a separate class

ill leave this open until in case you still run into any issues

ill leave this open until in case you still run into any issues

Thanks!!!
Yes, I tried moving the initialization of the UI part to a separate class, everything works fine, the server doesn't crash, but I don't understand why reterraforged is not applied when generating the server world.
Can you please tell me how I can customize the level-type in the server.properties file?

theres a guide for setting it up on servers here https://discord.gg/petc6SF6P2 under the #wiki-faq channel


vencord: 1.6.9 -> 1.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vencord is: Vencord web extension
meta.homepage for vencord is: https://github.com/Vendicated/Vencord
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.6.9 with grep in /nix/store/5pbskqsf982wxqfkj6lwjxbwm5k7m628-vencord-1.6.9
found 1.6.9 in filename of file in /nix/store/5pbskqsf982wxqfkj6lwjxbwm5k7m628-vencord-1.6.9

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
vencord
vencord-web-extension
vesktop
webcord-vencord

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/5pbskqsf982wxqfkj6lwjxbwm5k7m628-vencord-1.6.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/b75e627a0235e2e11022b3615a14330f52073ef9#vencord

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/5pbskqsf982wxqfkj6lwjxbwm5k7m628-vencord-1.6.9
ls -la /nix/store/5pbskqsf982wxqfkj6lwjxbwm5k7m628-vencord-1.6.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

vencord
vencord-web-extension
vesktop
webcord-vencord

Maintainer pings
cc @FlafyDev @fwam @NotAShelf @Scrumplex for testing.

I don't think our update script works well here as we effectively regenerate the lockfile

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for vencord is: Vencord web extension
meta.homepage for vencord is: https://github.com/Vendicated/Vencord
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.7.0 with grep in /nix/store/bhwh34pwi60vx2brz47fwhsflrfskrc2-vencord-1.7.0
found 1.7.0 in filename of file in /nix/store/bhwh34pwi60vx2brz47fwhsflrfskrc2-vencord-1.7.0

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

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
vencord
vencord-web-extension
vesktop
webcord-vencord

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bhwh34pwi60vx2brz47fwhsflrfskrc2-vencord-1.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/a35685e157f589dd77fecc2b6fd4ad13a4309bb2#vencord

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bhwh34pwi60vx2brz47fwhsflrfskrc2-vencord-1.7.0
ls -la /nix/store/bhwh34pwi60vx2brz47fwhsflrfskrc2-vencord-1.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

4 packages built:

vencord
vencord-web-extension
vesktop
webcord-vencord

Maintainer pings
cc @FlafyDev @fwam @NotAShelf @Scrumplex for testing.


⚠ Bofrost.de has degraded performance

In c7821da, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1463 ms

Resolved: Bofrost.de performance has improved in 896eb44 after 2 hours, 42 minutes.


First checkpoint of Dereks edits

@jmvera255 You can merge this as a checkpoint before other people start testing.


FN for ChangeNow Funding Bot

0x1cdb7ebb6b18d2587ad5471ba4aa65663f672ad5586d18299b3dd43ea9bd587f
0x71f2cb90c63c89900abdf882e3406cb1114a0e8fa6cbca9331a8da36c73a0554

fixed in https://github.com/forta-network/starter-kits/pull/437


allow option and target specification for //tools:lint

Allow arguments to be specified for the @rules_clang_tidy aspect, as
well as target specification.
If no arguments are passed to //tools:lint, linting applies to all
targets in the workspace using a wildcard label. If any argument is
passed, even a non-label, the lint targets must be explicitly
specified.
Change-Id: Id0f236dd9929deccd495d674121daf64f1223859

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 65.96%. Comparing base (74e3c9f) to head (06ca4a3).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage   65.96%   65.96%
=======================================
  Files          20       20
  Lines         238      238
  Branches        5        5
=======================================
  Hits          157      157
  Misses         81       81

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bump allowable maximum versions for sami & php-parser

What

Add ~3 to the allowed versions for nikic/php-parser (necessary)
Add ~4 to the allowed versions list for sami/sami (nice-to-have)

Why
Because using this library in a project that also requires sami ~4 is currently impossible, as sami ~4 requires php-parser ~3.

Travis no longer supports HHVM tests on the default 'precise' test platform. All other CI tests pass correctly.

Master branch is updated to build on 'trusty', but PHPUnit 6.2.2 does not support the version of HHVM on that platform (reports as PHP-5.6.99-hhvm), so tests are unable to run at all for HHVM (see the HHVM build log).
All CI tests that are executable on Travis pass correctly.

I just had a quick look into travis docs for hhvm. I give it a quick try, which didn't work ;). Though I have more libs to update. Will do this in early july, update them all and also release new version for this.

Awesome, thanks :-).

fixed in 661fa282410d850c615e5f265a81663d29b38c10

version 0.5 is out.

Thanks! :-)


DBZ-1146 Handle toastable columns for snapshotted tables

https://issues.jboss.org/browse/DBZ-1146

@gunnarmorling Unfortunately this is a little bit tricky again - for postgres the value will be in schema for wal2json not

@gunnarmorling The fix itself is only at PostgresSchema - the rest is only improvement in terms of defensive programming. The problem was that the logic worked for tables that where create in streaming phase but not in snapshot phase. The snapshot phase did not gathered the list of toastable columns but streaming phase did. Tha change in PostgreSchema ensures that the list of toastable columns is collected even in snapshot phase.

@jpechane, ok, so I feel something is still not quite right here. I observed this:

When setting SchemaRefreshMode to columns_diff_exclude_unchanged_toast, the unchanged toastable column will be part of the after structure (set to null); i.e. this can be added to the test as it is and it will still pass (tested with both DecoderBufs and wal2json):
new SchemaAndValueField("text", SchemaBuilder.OPTIONAL_STRING_SCHEMA, null)

When leaving SchemaRefreshMode to its default, the test will fail for wal2json but not DecoderBufs (when adding that expected SAVF above to the test)

The invocations of PostgresSchema#refresh(PostgresConnection connection, TableId tableId, boolean refreshToastableColumns) look odd; for the boolean we pass the value of taskContext.config().skipRefreshSchemaOnMissingToastableData(). I.e. skipping the refresh will actually cause the refresh; this looks wrong, at the very least that's super-confusing.

Btw. this issue in wal2json looks interesting: https://github.com/eulerto/wal2json/issues/98. This would definitely be helpful to us.

@jpechane Test failure seems intermittent?


Ghostly Essence can be used to dye sign texts

Description (Required)

The Ghostly Essence is actually white dye, it can be used to dye sign texts.
Steps to reproduce the Issue (Required)

Give Ghostly Essence.
Place sign with text.
Right click to sign with Ghostly Essence

Expected behavior (Required)

Ghostly Essence can't used to dye sign texts.
Server Log / Error Report

None
Environment (Required)

Tuinity git-Tuinity-"c303521" (MC: 1.16.5)
Slimefun DEV - 861 (git 73b5ef8b)
Metrics-Module #21
Java 11
Installed Addons: (3)
InfinityExpansion vDEV - 98 (git 4cf7e322)
FoxyMachines vDEV - 42 (git ae8d8aba)
DynaTech vDEV - 88 (git c06f0e25)

Being able to dye sign text with ghost essence is intended behavior


Chat Filter improvements

Incorrigible Alehound announcement is still missing?? I thought we'd got the code for that one...
Maybe classify lockpicks as rare items? They're worth more money than general unids, and anyone who cares about Wisdom is probably also opening chests.

Life of the Party

_____ drops a Destructive was Glaive.
other item spells too I guess

The mission could not be entered. Please try again later.

Could it be possible to have certain messages appear in a different colour? For example, when I get an ecto drop, I'd love for "Glob of Ectoplasm" (or the entire message) to be more visible, so I actually notice the message.

Maybe classify lockpicks as rare items? They're worth more money than general unids, and anyone who cares about Wisdom is probably also opening chests.

2 and a half years later, can I tempt anyone to care about this?

____ has started a fireworks display.

Legendary Survivor... do we have a list of titles we need to find the packet for?

Any titles that I havent got here https://github.com/3vcloud/guildwars-chat-app/blob/7b072766f2627cce8cfc782658583be7de3c5618/src/js/GuildWars_StringDecoder.js#L116 but theres no easy way for you to find the code yourself for these


Add bundle outputs to claim

https://github.com/deislabs/cnab-spec/pull/217 added an outputs section to the claim spec.
This ticket is for Porter to add the corresponding outputs section to the the claims it generates after each action.

Depends on https://github.com/deislabs/cnab-go/pull/66 (and updating our cnab-go dep once merged/released)

What we have been doing is to not let waiting for a cnab-go change to hold up the Porter work. So when you need a change in cnab-go, submit the PR, then immediately do the work you need in Porter relying on your PR's branch and update Gopkg.toml to use your fork. Then once the cnab-go PR is merged, remove the alternate source/branch. If we need multiple PRs merged into one, we can discuss and work on a coordinated strategy.
Otherwise this seriously slows down our work on Porter, and also prevents us from adequately verifying our PRs to cnab-go. Often we don't realize that we need more in those PR until we use them in Porter. 😀


[releng] Mention kubernetes/websites everywhere in the same way

Related to https://kubernetes.slack.com/archives/C0191RDKHU1/p1680549820387879

@kubernetes/release-engineering : PTAL

QQ, Does https://github.com/kubernetes/website/pull/40493 supersede this PR?

Nope, afaik, it doesn't supersede the current PR. This PR was suggested as a branch-out since the script was required to be amended. The explicit changes to release.md will be discarded on PR # 40493.

@xmudrii done


Type conversion in C++

Description
Giving explanation of topic using examples and code for better understanding.
Domain
C/CPP
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

I have only 3 open issue including this one. can i work?

/assign

@Harsh-Chaudhary7 can u pls add hactober label on this one


Stop checking encoding names

Comparing the names is much less efficient than
comparing the instance directly.
It may also change in the future: https://bugs.ruby-lang.org/issues/18576

Thanks for opening all these PRs for this potential issue.


Add broad server-side user input sanitization/validation

Closes https://github.com/ethyca/security-issues/issues/3
Closes https://github.com/ethyca/security-issues/issues/4
First step here is to identify all of the places where user input might be. The following resources will be helpful:

Endpoint Audit Doc
localhost:8080/docs when running the webserver locally
OWASP Cheat Sheet

Code Changes

[ ] create a SafeInputStr custom data type to be used in place of str wherever user input is expected

Steps to Confirm

[ ] list any manual steps for reviewers to confirm the changes

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md

Description Of Changes
Write some things here about the changes and any potential caveats

@RobertKeyser @daveqnet would you mind giving  me some examples here of clearly dangerous/invalid user input so I can use them for test cases? I trust y'all here as the experts 😄 thank you!

@RobertKeyser @daveqnet would you mind giving me some examples here of clearly dangerous/invalid user input so I can use them for test cases? I trust y'all here as the experts 😄 thank you!

Hmmm, it's a tricky one, as what constitutes dangerous input depends on what the input is being used for. Does the guidance here help: https://owasp.org/www-project-web-security-testing-guide/latest/4-Web_Application_Security_Testing/07-Input_Validation_Testing/ ?
I'll also try to provide you a few examples of some real injection vulnerabilities from non-fides penetration tests to help.

If you're just looking for a list of our API endpoints that accept strings in the request body, I'd assume the majority do, too many to just list here.
Most of the ops schemas are here:
https://github.com/ethyca/fides/tree/main/src/fides/api/ops/schemas
If you're just trying to make a list to work from, you might write a script that goes through the contents of http://localhost:8080/openapi.json, for each request looks for the requestBody key and gets the schema from there - lots of recursion.

based on the feedback here, it sounds like the safest way to go about things here is to update every string to use InputStr and see where that lands us...
Additionally, logging around when values get truncated would be helpful for debugging

quick update: I can't test the specific fixes I made because I'm not sure how to use the local test env. It now requires privacy request configurations. Will message in slack and try to get this sorted

Patch coverage is lacking but overall coverage is up, we're calling this one a win 🙃


CAV-89: Define CI/CD pipeline for cip-phone-number-frontend - MDTP Pu…

…blic Zone
 - auto generated source file headers
 - added scala style plugin and default configuration
 - set context path to /phone-number
 - temporarily set code coverage to 60

rebuild

test this please


Re-load the video with contrib on ended event

Is it possible to re-load the video when using videojs-contrib-dash on ended event?
We previously used to listen to the ended event and used video.load() to re-load the video and re-display the poster image ready for a possible replay by the user.
I tried adding the .load() method to the videojs player when using contrib, but it doesn't work anymore, e.g. myPlayer.load()
Is there a way to re-load the video when usng contrib?
Anyone got this to work, or know how to do it please?

how do you get this working?

See:
https://github.com/videojs/video.js/issues/2661


Question: Can HockeySDK and IoT

Will HockeySDK work with Windows IoT apps?

Yes that is possible, use the UWP SDK

I've used the UWP SDK. Added to app.cs the HockeyClient.Current.Configure call like in the sample, using the app's id. Added several TrackEvent calls. Created a package. Uploaded to hockyapp site from within VS 2017. Downloaded the package form hockey app. Added the package to the Windows IoT Core flash image (flash.ffu). FLash the SD card and booted the device. My app boots.
So now I'm checking updating. I made a small change and uploaded a new version to hockeyapp site from within VS. Then I restarted by Windows 10 IoT device running the app and it doesn't update or notify me of an available update.

So now I'm checking updating. I made a small change and uploaded a new version to hockeyapp site from within VS. Then I restarted by Windows 10 IoT device running the app and it doesn't update or notify me of an available update.

AFAIK that is the expected behavior. HockeyApp is not meant to be an alternative to the Windows Store deployment, it only provides the possibility to sideload (beta) app packages.
Why can't you use the Windows Store for deployment? Then you could programatically trigger the update, see here

Windows Phone apps will auto-update using HockerApp. So Windows IoT devices should also. They are very similar platforms with respect to being locked down and using ARM processors. So the expectation is that HockeyApp will be used for all development purposes (including updating the app during development and testing).
I will not distributing a commercial product. This is for a beta test only. My customer needs to do certification and QC testing before they decide to buy the product design I come up with. I need to get crash reporting and usage data during this testing for my own QC. These purposes are not appropriate for Windows Store distribution (which is why HockeyApp exists in the first place). So I don't want Windows Store (because it's not appropriate and costs money), and I don't want Azure IoT Hub (because it's inappropriate for beta use and costs even more money after the first 30 days).

Hey @mjmeans,
thx for explaining your motivation and needs.
Unfortunately, we don't support auto-update specifically for IoT devices.
We're currently hard at work on AppCenter so it's not very likely that HockeyApp will get the required functionality.
Best,
Benjamin

Will AppCenter support auto-update for Windows IoT devices?

Hey,
it's currently not possible to do side-loading on Windows IoT devices so App Center has the same limitation that HockeyApp has at the moment.
Best,
Benjamin


[RFC]: New MSR / CR / RFLAGS code

The MSR, Control Register, RFLAGS is pretty terrible, and it's missing a lot of stuff. Right now, it's just a set of #defines, with no real organization. There is a ton of stuff missing (which is not helpful when your trying to prototype stuff), and there is 0 encapsulation so we have masking and bit shifting happening all over the place, which makes testing a nightmare.
So, here is my proposal. I have some of this working already just to prove to myself that I got it right, as this code is new to me (and likely most of you).
Lets take the ia32_vmx_basic  MSR as an example, but everything else will likely take the same form.
namespace intel_x64
{
namespace msrs
{
    template<class I, class T> constexpr auto
    read_msr(I in, T msr)
    { return in->read_msr(msr); }

    namespace ia32_vmx_basic
    {
        constexpr const auto addr = 0x00000480U;
        constexpr const auto name = "IA32_VMX_BASIC";

        template<class I> constexpr auto get(I in)
        { return read_msr(in, addr); }

        namespace revision_id
        {
            constexpr const auto mask = 0x000000007FFFFFFFUL;
            constexpr const auto from = 0;

            template<class I> constexpr auto get(I in)
            { return gsl::narrow_cast<uint32_t>((ia32_vmx_basic::get(in) & mask) >> from); }
        }

        namespace vmxon_vmcs_region_size
        {
            constexpr const auto mask = 0x00001FFF00000000UL;
            constexpr const auto from = 32;

            template<class I> constexpr auto get(I in)
            { return gsl::narrow_cast<uint32_t>((ia32_vmx_basic::get(in) & mask) >> from); }
        }

        ....
    }
}
}

Normally, in your code, if your working on an Intel CPU, the first thing your going to do is the following so that you can unwrap some of the nested namespaces here:
using namespace intel_x64;

Then, code that looks like this:
void
vmxon_intel_x64::check_vmx_capabilities_msr()
{
    auto vmx_basic_msr = m_intrinsics->read_msr(IA32_VMX_BASIC_MSR);

    auto physical_address_width = (vmx_basic_msr >> 48) & 0x1;
    auto memory_type = (vmx_basic_msr >> 50) & 0xF;
    auto true_based_controls = (vmx_basic_msr >> 55) & 0x1;

    if (physical_address_width != 0)
        throw std::logic_error("invalid physical address width");

    if (memory_type != 6)
        throw std::logic_error("invalid memory type");

    if (true_based_controls == 0)
        throw std::logic_error("invalid vmx true based controls");
}

Now looks like this:
void
vmxon_intel_x64::check_vmx_capabilities_msr()
{
    if (msrs::ia32_vmx_basic::physical_address_width::get(m_intrinsics) != 0)
        throw std::logic_error("invalid physical address width");

    if (msrs::ia32_vmx_basic::memory_type::get(m_intrinsics) != x64::memory_type::write_back)
        throw std::logic_error("invalid memory type");

    if (msrs::ia32_vmx_basic::true_based_controls::get(m_intrinsics) == 0)
        throw std::logic_error("invalid vmx true based controls");
}

This is obviously a lot cleaner, and allows us to use all sorts of things like:
msrs::ia32_vmx_basic::memory_type::get(m_intrinsics)  // Read MSR from intrinsics
msrs::ia32_vmx_basic::memory_type::set(m_intrinsics, value)  // Write to MSR with intrinsics
msrs::ia32_vmx_basic::memory_type::mask // Returns the mask for this field in the MSR
msrs::ia32_vmx_basic::memory_type::from // Returns the starting bit of the field in this MSR
msrs::ia32_vmx_basic::get(m_intrinsics) // Returns the whole MSR
msrs::ia32_vmx_basic::addr // Returns MSR's address (the hex number in the manual)
msrs::ia32_vmx_basic::name // Returns MSR's address (the hex number in the manual)

Unlike what we have now, there is a lot of useful stuff here, all organized in a way that matches the Intel docs almost perfectly. We could even add a description and comments field to this in the future to almost completely match the documentation.
Now, just for reference, I should explain how the generic programming works here. This code uses some "always auto" patterns, not all of them however because I feel that stating that a field is narrowing should be explicit (narrowing past 32bits in C++ is a bad idea since the compiler automatically converts 16bit and 8bit ints to 32bit ints for you), and the addresses / masks are all explicit. But lets at least look at this pattern:
template<class I> constexpr auto get(I in)
{ return gsl::narrow_cast<uint32_t>((ia32_vmx_basic::get(in) & mask) >> from); }

The template argument here prevents us from having to define what is actually giving us the MSR, so long as it has a function called "read_msr" it will work. From there the gsl::narrow_cast states that we are possibly converting a 64bit value to a 32bit value. The constexpr auto says the return type can be deduced from what is actually returned (no need to state it), and the constexpr tells the compiler, "if you can convert this to a constant expression, do so", which means, treat it like a #define macro if you can, removing the function calls, and extra overhead, etc... so this compiles to basically:
m_intrinsics->read_msr(addr) & mask >> from

Thoughts?
A PR for this will take a couple of days as I have to convert a lot of code to clean this up, but the plus side is we will have a lot cleaner code, and everyone attempting to use Bareflank will have a more complete set of MSRs, Control Registers and RFLAGS to work from. And of course if anything is missing we now have a cleaner way of adding it in the future.

Overall I like the direction away from #defines to more C++-style features. One comment to add: since we're using generics for this, I think the design should be generic as possible. It should be designed with other architectures and platforms in mind.
The nested structure of the namespaces above sort of suggests that only Intel architectures have msrs and control registers.  Should those be represented at a higher level in the namespace?  Maybe it is just a naming issue of the namespaces, because like you said, any I that has a read_msr function can use the msrs::read_msr function.
I know the code above isn't final but those are some thoughts I had when reading it.

So I thought about that, and I don't see any way to generalize control registers between ARM and Intel, and I don't think it's worth doing as it would come at a cost of clarity.
Now... that being said, there are MSRs, control registers, etc... that are shared between AMD and Intel. So my idea here is as follows:
x64:: -> AMD / Intel shared registers. Likely RFLAGS, one, maybe two of the control registers, and several MSRs
intel_x64:: -> Intel specific registers
amd_x64:: -> AMD specific registers
For now, most of the code is going to go into intel_x64, and as we begin to add support for AMD, we can start to move things to the general x64:: namespace.
Now that being said I am open to suggestions on how to generalize this further, but I'm not sure it's going to be possible without the interface being too confusing. For example, CR4 has a LOT of Intel specific fields. We could divide CR4 up between x64, intel_x64 and amd_x64, but then you would have to know which one it belongs to to figure out what you need, which I think is too confusing.
One idea would be to have everything in x64:: that is shared, with the same things being in intel_x64 that is in x64:: + intel specific stuff. So if you only need generic stuff, you go to one vs the other. The problem here is I'm not sure how to organize this without there being duplication. If this is something people really want, and duplication is the only way I'm fine with that, but I'd like to avoid it if possible.
Another thing I like about this approach is it will work great with C++ modules once they become available, something we should be thinking about as our compile times are already pretty high.

A possibility is to move the msrs, control registers, etc. information into the intrinsics hierarchy as static data members. Each class along the hierarchy already defines their own read_msr, write_msr, etc. functions and plus the msrs are referenced in the context of instances of the intrinsics hierarchy already. This would prevent duplication of common registers and of code, and allow for more-derived classes to define their own masks, fields, etc. for arch-specific registers.

So I thought about using class inheritance, but I don't like it four a couple of reasons:

I cannot organize the various definitions like I can above without either introducing a train wreck, or some other odd ball nested class structure, which really doesn't get to what your looking for here
The only implementation I can think of, would have to use the name itself to organize where we put things, so you would have something like msrs_<msr name>_field_get() which I really don't like.
Since we cannot spread the code out in classes like we can with a namespace, you would have a pretty massive sized intrinsics class that would not work well with modules
It's really heavy handed, the code above compiles to the same thing we have now, and testing is simple, using class inheritance, we would have a pretty massive addition to the code itself.
We really try to avoid static functions and data members as they are even harder to test.

The only advantage I see to using class inheritance for this problem is the ability to access a set of registers for x64 and Intel from one m_intrincis->reg(), but since you need to include everything in the name to deal with conflicts, I don't see how the code is really any more readable vs x64::msrs::::get() and then doing intel_x64::msrs::::get()
One thing I have been playing with is trying to find a way to be able to remove the need to pass in an intrinsics class, which I think I can do just fine (I can call the intrinsics function no problem). The issue with that is unit testing. If all of the code calls this without passing the intrinsics class, how do you unit test?
One possibility, is to remove the intrinsics class completely, and just call a set of wrapper C++ functions, and use Hippomocks to mock the functions using their C versions, or better yet, just not link in the intrinsics library, and have to provide these symbols manually. The former would require static linking which can be a pain as a mod to the library doesn't currently get detected my make. The latter would work without the need for static linking, but likely would require all of the symbols to be defined (even if they are not needed)... but now that I think about it, we could provide "weak" versions of each one that throws an exception saying they need to be redefined. Might not actually be a bad idea.
Thoughts?

Testing out removing the intrinsics code now (one step at a time as it's pretty large), but so far, using Hippomocks and using static intrinsics linking, it seems to be working really well. Statically linking to that library actually shouldn't be a big deal as we don't touch that code very often.
The only odd ball thing that might come up is if you forget to mock an intrinsic, the linker might compile out a symbol that is needed, in which case the test will likely fail to run while attempting to load in the dynamic libraries. This error is likely a good thing anyways, but it's good to note at least some of the strange things that happen when you mix static and dynamic libraries.

This is complete now. It has been fixed by several PRs over the past month.


Fix kotlin version for examples

:scroll: Description
#skip-changelog
:bulb: Motivation and Context
packageinfo requires a newer version since they have bumped their kotlin version.
:green_heart: How did you test it?
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report
Base: 89.98% // Head: 95.55% // Increases project coverage by +5.56% :tada:

Coverage data is based on head (533ce65) compared to base (eecbbca).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1091      +/-   ##
==========================================
+ Coverage   89.98%   95.55%   +5.56%
==========================================
  Files         115        2     -113
  Lines        3585       45    -3540
==========================================
- Hits         3226       43    -3183
+ Misses        359        2     -357

Impacted Files
Coverage Δ

dart/lib/src/sentry_user_feedback.dart

dart/lib/src/platform/_io_platform.dart

dart/lib/src/protocol/sentry_culture.dart

dart/lib/src/sentry_measurement.dart

dart/lib/src/hub.dart

dart/lib/src/protocol/span_status.dart

dio/lib/src/sentry_dio_client_adapter.dart

dart/lib/src/scope.dart

dart/lib/src/sentry_tracer_finish_status.dart

dart/lib/src/transport/rate_limit_parser.dart

... and 103 more

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.


deprecated warning in deps

<EMAIL_ADDRESS>This module is not used anymore, and has been replaced by @npmcli/package-json

I want to avoid deprecation warning from standard-version to this one, but your package is still showing some warnings, could you use the replaced one?

Sure thing! I’ll take a look

This is a little fiddly, as the new one is not a drop in replacement (the API for @npmcli/package-json is now async, whereas the previous one was synchronous - along with a couple of other changes).
I'll be able to set aside some time in early jan for this. If you'd like a fix sooner, PRs are very welcome - I can easily review and release.

Hi @TimothyJones, I've created a PR to address this issue.
Let me know if you think the PR is a suitable solution to the issue.

Love it! Thank you so much.
I added a comment. if you could sort out the license, then I'll get it merged and released asap.

Released in 11.2.1, thanks again.


ng-modal is not working when used with ng-options and group by.

Hi,
There is no modal change (I have tried using $watch to verify any modal change).
The following code:
<select name="selectedCard" chosen class="chosen-select" style="width:250px;"
                    disable-search="true"
                    ng-if="showMultiCardNavBar()" ng-model="selectedCard" ng-change="cardSelected()"
                    ng-options="card.id as card.id group by getGroupByName(card.cardGroupId)  for card in cardsContext" >
                <option value=""></option>
            </select>

there is no change on this property: selectedCard.
Had to do a workaround to query the DOM object and find the chosen-selected taged element.

It seems that the ng-if was the case. Moving it to a separate div resolved the issue

Sorry, do you mean ng-model?

Hi @Marek00Malik, thanks for using the library, and also for reporting this your issue!
I'll have to analyse it on the weekend. That would be easy if you could perhaps, post a plunker for me.
Thanks

Hi @Marek00Malik, did you happen to get around with this issue?

The same here @leocaseiro the ng-model is not being updated.

Hi@thelastinuit, I'm sorry to hear that!
Would you mind making taking this plunker and making your changes?
It seems working fine for me: https://plnkr.co/edit/4dpqKG?p=preview
Thanks heaps

Mea culpa~!
It is something I didn't read (shame on me)!
I didn't realized that the object.selected MUST be initialized with [0] otherwise it doesn't update the selected items!
Sorry!

Hi @thelastinuit, I'm glad you fixed your issue.
Thanks for your feedback.

Hi @leocaseiro - I wonder if you could advice how to use complete objects (instead of id's) in select model. Here the example: https://plnkr.co/edit/986uGr?p=preview

Hi @dkokic, It looks like the second   is doing exactly what you're looking for:
<select
      multiple
      chosen
      ng-model="stateList"
      ng-options="s as s.title group by s.country for s in states">
      <option value=""></option>
    </select>

Yes and No ;-)
The 2nd select from https://plnkr.co/edit/986uGr?p=preview is correctly updating the model with the new selection value(s), BUT it fails to show the select model's initial values.

Hi @dkokic,
I'm sorry I missed that. I forgot to add track by. In that case, it would be:
<select
      multiple
      chosen
      ng-model="stateList"
      ng-options="s as s.title group by s.country for s in states track by s.id">
      <option value=""></option>
    </select>

https://plnkr.co/edit/qIqgEz?p=preview

Thanks a million @leocaseiro

I'm glad worked it 🍻


Headroom is not defined

Having the same issue as: Headroom is not defined #53
I'm trying to use headroom on a Wordpress site, both the JS files are being loaded, but the functions don't work when they call them (in the page or console).
I'm trying to use the straight up JS version on the page, and am logging 'myElement'  Any thoughts on why the functions don't work?

Same issue here. Tried bower/npm/straight js download from the web. All with no luck.

Which syntax do you use? Plain JS or jQuery?
I had a similar problem some weeks ago by using headroom.js with jQuery. It took me some time, but I could solve it. The problem was that the needed jQuery plugin file was missing, where the necessary function was defined.

@Pyth0ff I tried both the straight up js and then the jquery being sure to include the extra file and neither worked.  Thank you though!

Any chance is it because headroom.js is being loaded after your function to call it?

Assume this has since been solved. Closing in the interest of cleaning up the issue tracker. Please comment if you are still having issues


lakshmi.assets.NotFoundError: Cannot retrieve ticker ("VTI") from Yahoo Finance

Hi,
I haven't used lak for awhile.  I tried today and received this error:  lakshmi.assets.NotFoundError: Cannot retrieve ticker ("VTI") from Yahoo Finance.  I used the already setup portfolio for testing.
Below is the command I ran and the output.
mhenke@mhenke-Inspiron-15-3567:~$ lak --debug -r list assets total aa al performance
Traceback (most recent call last):
File "/home/mhenke/.local/bin/lak", line 8, in
sys.exit(lak())
File "/home/mhenke/.local/lib/python3.8/site-packages/click/core.py", line 1128, in call
return self.main(*args, **kwargs)
File "/home/mhenke/.local/lib/python3.8/site-packages/click/core.py", line 1053, in main
rv = self.invoke(ctx)
File "/home/mhenke/.local/lib/python3.8/site-packages/click/core.py", line 1659, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/mhenke/.local/lib/python3.8/site-packages/click/core.py", line 1690, in invoke
rv.append(sub_ctx.command.invoke(sub_ctx))
File "/home/mhenke/.local/lib/python3.8/site-packages/click/core.py", line 1395, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/mhenke/.local/lib/python3.8/site-packages/click/core.py", line 754, in invoke
return __callback(*args, **kwargs)
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/lak.py", line 328, in assets
output = portfolio.assets(
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/lakshmi.py", line 712, in assets
+ [asset.name(), asset.adjusted_value()])
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/assets.py", line 176, in adjusted_value
return max(0, self.value() + self.get_what_if())
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/assets.py", line 428, in value
return self.shares() * self.price()
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/cache.py", line 211, in new_func
value = _call_func(class_obj, func)
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/cache.py", line 178, in _call_func
return func(class_obj)
File "/home/mhenke/.local/lib/python3.8/site-packages/lakshmi/assets.py", line 539, in price
raise NotFoundError(
lakshmi.assets.NotFoundError: Cannot retrieve ticker ("VTI") from Yahoo Finance

ok, i ran pip uninstall lakshmi then python -m pip install lakshmi and the issue seems resolved


(PUP-4462) Fix problem with heredoc interpolation turning on escapes

Before this, heredoc with interpolation would also turn on escapes. This
means input like $name would not interpolate, and \ would be required
to get a single . It should be required to be explici about allowing
escapes by using /$ (to be allowed to escape interpolation).
The prolem was caused by the pattern used to find the start of an
interpolation. This pattern included the escapes that would make the
pattern skip the interpolation start - it would simply scan past the
interpolation start if escaped.
The solution was to create a new pattern for finding interpolation
starts when a $ is not one of the wanted escapes.

CLA signed by all contributors.


Update azure-stack-poc.md

changed from 4-12 to 4-16 nodes for integrated systems

@bottkars this is not supported (yet). #please-close

Was announced at ignite as supported.....
Von meinem iPhone gesendet
Am 12.10.2018 um 19:44 schrieb Jeff Gilbert <notifications<EMAIL_ADDRESS>@bottkarshttps://github.com/bottkars this is not supported (yet). #please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/pull/16709#issuecomment-429405017, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AH32Hy6kKBT4Lhlubd7ri6CHUd-Ult4Wks5ukNTtgaJpZM4XY3Rr.

Just double-checked again with the team. This is planned to be supported in 1810. It is NOT supported until then.
#please-close


Unrecognized arguments error when no device is found.

Bug description
What happened
Running  rivalcfg --battery-level with no supported device connected returns an unrecognized arguments error.
$ rivalcfg --battery-level
usage: rivalcfg [-h] [--list] [--version] [--no-save] [--update-udev] [--print-udev] [--print-debug]
rivalcfg: error: unrecognized arguments: --battery-level

What was expected
I would expect an error message saying that no supported device was found.
Step to reproduce

Is the bug always reproducible or is it "random"?

→ always reproducible

Disconnect all device dongles.
Run rivalcfg --battery-level

System information
RIVALCFG
========
Version: 4.9.0
udev rules installed: True
udev rules up to date: True
Installation path: /usr/lib/python3.11/site-packages/rivalcfg

OPERATING SYSTEM
================
OS: Linux
Platform: Linux-5.15.114-2-MANJARO-x86_64-with-glibc2.37
Version: #1 SMP PREEMPT Sun Jun 4 10:32:43 UTC 2023
Distribution issue: Manjaro Linux \r  (\n) (\l)

PYTHON
======
Python version: 3.11.3
HIDAPI version: 0.14.0

PLUGGED STEELSERIES DEVICES ENDPOINTS
=====================================

Resolving the Issue
I have already fixed the issue locally and will submit a pull request soon.

Fixed by #214, it will be released in next rivalcfg version :)


get rid of weird timedatectl error

System has not been booted with systemd as init system (PID 1). Can't operate.
> Failed to create bus connection: Host is down
> Warning message:
> In system("timedatectl", intern = TRUE) :
>   running command 'timedatectl' had status 1

happens on github actions as per https://github.com/r-lib/ghactions/commit/f00feb030d16e25ea5d0472a583820b6b19dbb6d

You should be able to avoid this by setting the TZ environment variable explicitly e.g. https://github.com/rocker-org/rocker-versioned/issues/89


Fix 187194089 by updating confirmation email body

I updated the HTML for the email and also had to add the confirmation number to the args.
I added a test that ensures the confirmation number is present in the email body too.

This looks great and works as promised.  Just have a few minor comments


Recompiling the model but not having to call graph.openStore again.

Right now it seems that I have to call graph.openStore every time i recompile the model with a new input shape. Is there any way that i recompile the model with a new input shape, but it retains the weights?

It is probably due to the hard-code width / height during the model construction. If you use ModelBuilder, I don't think that will be an issue. You just supply new inputs, and it will create new model in a way retains the weights. Anyway, some code snippet would help to understand what you are doing.

unet = UNet(batchSize: 2, startWidth: 64, startHeight: 64 )
unet.compile(inputs: xIn, t, c)
graph.openStore("sd-v1.4.ckpt") {  $0.read("unet", model: unet!)  }

// ... some image generations

unet = UNet(batchSize: 2, startWidth: 128, startHeight: 128 )
unet.compile(inputs: xIn2, t, c)
graph.openStore("sd-v1.4.ckpt") {  $0.read("unet", model: unet!)  } // i really dont wanna load everytime i change model size, as that wastes time

/// more image generations

@liuliu do you have a snippet of how to do it with ModelBuilder?

Yeah, I give the example earlier: https://github.com/liuliu/s4nnc/blob/main/examples/imdb/main.swift#L86
Basically you would probably do:
let actualUnet = ModelBuilder { inputs in
  let batchSize = inputs[0].shape[0]
  let startHeight = inputs[0].shape[2] // assuming NCHW
  let startWidth = inputs[0].shape[3]
  return UNet(batchSize: batchSize, startWidth: startWidth, startHeight: startHeight)
}

That has been said, I mostly use this for DRL, haven't tried for SD model yet. If this works as expected, you don't need to call unet.compile(inputs: xIn2, t, c), just pass in different size for next generation and it will reallocate intermediate tensors (if needed) and retain weights. The only thing is, it will ever expand the allocation (thus, if later you have xIn3 which is smaller than xIn2, it will not release RAM and will retain all RAM allocations of previously largest input).

Does the model builder call the UNet function every time the shape is changed, or does it call the UNet function only once, and it traces the graph in some fancy way and just resizing the tensors whenever the input size is changed?
So xIN3 is small again, and i wanna free up the memory i should recreate actualUnet and it will reset it and GC the old stuff?

Does the model builder call the UNet function every time the shape is changed, or does it call the UNet function only once, and it traces the graph in some fancy way and just resizing the tensors whenever the input size is changed?

It is not a straightforward answer. Yes, every time shape changed, there will be a call to UNet function to build a model. But what happens is the more expensive optimization / tensor allocation pass most likely will be bypassed if the new tensor is smaller: https://github.com/liuliu/ccv/blob/unstable/lib/nnc/ccv_cnnp_model.c#L538

So xIN3 is small again, and i wanna free up the memory i should recreate actualUnet and it will reset it and GC the old stuff?

You most likely need to recreate the ModelBuilder again (ModelBuilder is a drop-in replacement for Model) and reload the weights.

This model builder thing seems to take wayy too much memory? is that expected?
i create unet with model builder, and even after calling it for the first time it gives this error:
Error: command buffer exited with error status.
	The Metal Performance Shaders operations encoded on it may not have completed.
	Error:
	(null)
	Insufficient Memory (00000008:kIOGPUCommandBufferCallbackErrorOutOfMemory)
	<AGXG13GFamilyCommandBuffer: 0x13dfe5220>
    label = <none>
    device = <AGXG13GDevice: 0x13e875200>
        name = Apple M1
    commandQueue = <AGXG13GFamilyCommandQueue: 0x13e89fa00>
        label = <none>
        device = <AGXG13GDevice: 0x13e875200>
            name = Apple M1
    retainedReferences = 1
Error: command buffer exited with error status.

Not sure about this particular case, but with CUDA, I verified it worked: https://github.com/liuliu/swift-diffusion/blob/main/examples/txt2img/main.swift#L143 (Note that I added a explicit compile(inputs: function to ModelBuilder, without it, it will delay parameter loading to the first evaluation, may or may not be ideal).

Also, why are you not compling the model here https://github.com/liuliu/swift-diffusion/blob/main/examples/unet/main.swift?

These has PythonKit dependency and I want to keep txt2img example clean of PythonKit dependency.

PythonKit is just used for running the pytorch model and loading the weights from the .ckpt file. still .compile is not being called. The fwd pass is run without calling the .compile.
but if we do that in txt2img, it does not work. why?

What error do you get? There might be a few reasons, but most likely just because when copying from Python, we copy Float32 weights, and if you use the code from examples/txt2img/main.swift all weights are in Float16, and you might error out on reading weights. That's why I first save the weights and then use it (when loading weights from SQLite, we will do transparent conversion between FP16 and FP32).


Cache always misses for hidden directories

I've noticed that in one of my private repos, the cache is never hit even though the cache is small and at the end of each run it says it successfully saved.
Cache not found for input keys: Linux-eslint-e2e650427648991b0a4acaf9d0a8af37ce90a05d, Linux-eslint-.

Post job cleanup.
/bin/tar -cz -f /home/runner/work/_temp/random-hash/cache.tgz -C /home/runner/work/myproject/myproject/.cache/eslint .
Cache saved successfully

Here's the workflow step:
- name: cache eslint
  uses: actions/cache@v1
  with:
    path: ./.cache/eslint
    key: ${{ runner.os }}-eslint-${{ github.sha }}
    restore-keys: |
      ${{ runner.os }}-eslint-

Are there any reasons for this? I'm assuming caching isn't supported in free-tier private repos yet but I haven't seen that anywhere in the documentation.

Hey @jahed,
I'd expect it to work on all repositories, so I don't think that would be an issue.
It might be helpful to turn on debug logging, see https://help.github.com/en/actions/automating-your-workflow-with-github-actions/managing-a-workflow-run#enabling-step-debug-logging
And if you want me to look at our internal logging to see what's going on, feel free to message me your repository name<EMAIL_ADDRESS>or post it in this thread.
You may also want to try changing your path, I'm not sure if the ./ is required.

I'll see if I can get anywhere with the debug logs and path change.

Changing the path didn't do anything. With debug logging enabled there's not really much to see. There's a 204 and it misses the cache. On the other end there's a 200 and reports success.
This is a good chance for me to learn how to debug actions since my deployment pipeline relies on it. :worried:
I'm guessing this repo can be forked and my workflow can switch over to the fork for additional debug logging.
I'll update if I get anywhere (it's the weekend so probably next week).

I've figured it out after hours. :tired_face:
For some reason, actions/cache does not like hidden directories being cached. That is, .cache in my case. It's okay with zipping up and saving them, but the restore always 204s (no content).
Renaming .cache to just cache or something else fixes it. But I guess this is a bug likely in the backend since the POST goes through fine with the correct size.

Even after changing the directory name, the restore does not restore hidden files (e.g. cache/eslint/.eslintrc). It seems the cache restore ignores both hidden directories and files.

@jahed Thanks! I would not expect that, do you have a repro project/workflow you could send me so I can look into this?

i have some node modules that my custom actions depend on. i tried using hashfiles('.github/actions/package-lock.json') and it didn't like it.

I tried setting up a minimal reproduction but failed to reproduce the issue.
https://github.com/jahed/_actions-cache-repro
So that got me thinking. I moved my git repo to a new GitHub repo. Reverted back to .cache and it's working now. Changing the directory name to make it not hidden was a red herring.
So whatever it was... I don't know. I didn't change anything other than migrate repos. So maybe the repo on GitHub is just old, had some incompatible settings, failed some backend migration or something. It's a bit of a black box.
Anyways, I'm going to close this since I can't reproduce it. I'm encountering other issues with the cache too but it just seems very random. Next time I report something, I'll setup a minimal reproduction first.

@jahed Okay! Let me know if you need any help setting it up!
It's possible you had a corrupted cache at some point, and using the same key will keep fetching that cache over and over.

That would make sense. It would be nice to be able to download caches on our end in order to debug runs and see what's being restored.


Not an issue, new to Vue.js - Need example of Array Pagination

Browser and uiv version
chrome with //unpkg.com/uiv/dist/uiv.min.js
Which component
Pagination
Steps to reproduce the problem
Can run this but how do I page over an Array?

Expected behavior
Actual behavior

Hi,
I think you're talking about this: https://codepen.io/wxsm/pen/pdgaOW

Yes!  Many thanks for your time and this amazing project.  Really enjoying it!

Welcome.


Operator to watch ImageStream for new :latest tag

Hey,
first of all, thanks for the project, so far it works really smoothly for me.
I'm trying to setup a CD-Pipeline with the operator in an OKD-Cluster (switching to OpenShift soon). The Building+Pushing part is handled by GitLab fine now, but I was wondering if a change to the ImageStreams :latest tag shouldn't cause a stopping of all pods and starting with the newest version of the Image.
If not, is this a planned change, or what would be the suggested path to go here?
My understanding was to keep the Image immutable, so i was going to build my domain in one layer with my desired settings and references to apps to deploy, then in the next layer, copy my ear-files into the image and finally, tag it twice, once with latest and once with a unique job-id so i can manually chose a time when to deploy my changes to production, but keep my test-environment at the newest version of my master-branch.
Thanks for your help!

If you change the image reference in the domain resource, the pods should do a rolling restart. Is this not happening for you? But if you are keeping the same image name, and replacing the image itself in docker, the operator will not notice.

That works correctly.
i switched my CI-pipeline to use
oc patch domain domain1 -p '{"spec":{"image":"docker-registry.default.svc:5000/my-weblogic-namespace/gitlab-master:${CI_JOB_ID}"}}' -v 100 --type=merge
Thanks for your help!


release4.53.3-hotfix

4.53.3-hotfix

API change check
API changes are not detected in this pull request.


[pigeon] redirect pigeon resources to ffi - cross post

As this issue pertains to both pigeon and ffi I've add a duplicate issue here as well.
The original issue is:
https://github.com/flutter/flutter/issues/70254
@gaaclarke
So I want to express what I'm sure will be a controversial opinion.
The nub of the idea is to drop the pigeon project and instead focus all resources onto ffi.
Background:
I'm the author of the Sounds project:
https://pub.dev/packages/sounds
One of the big issues we are grappling with is having to be across four development languages:
Dart
Java
Objective-C
Swift
This is a significant problem for getting resources applied to the project and has the core team bumbling around with languages that they just aren't familiar with.
We have been experimenting with ways to make the project easier to maintain.
The first thing we looked at was converting all of the objective-c to swift as swift is a simpler language and more 'dart' like than objective-c.
This helped some but we still had bits of objective-c left in the code for bridging purposes and we are still not swift experts (or even novices).
Next we had a look at pigeon for a couple of reasons.

we liked that it largely managed channels for us.
we hoped it would take care of some of the bridging code (not so much).
we liked the concept of a federated model (yes not pigeon specific but it helped).

We did an initial implementation with pigeon and the process actually helped us improve our platform api but it did nothing to reduce the no. of languages that we needed to support.
So we are now looking at ffi and dart-native.
https://pub.dev/packages/dart_native
The dart-native project is useful but still very primitive however its clear (to our team at least) that using ffi is a much better path than pigeon.
Once we complete the ffi work we should go close to eliminating all objective-c, swift and java code.
This will make our project much simpler to maintain and should make it easier to get additional contributors.
The biggest stumbling block is that, whilst dart-native, helps we are still having to implement ffi wrappers for large parts of the ios api (and currently all of the java api).
My view is that pigeon is really just a stop gap measure and long term I don't see pigeon being a part of the dart ecosystem.
As such I'm arguing that rather than wasting resources on a stop gap measure those resources would be better placed on completing ffi, improving the documentation and providing a complete set of ffi mappings for the ios and android apis.
I understand that this will be a large task but it is one that could be rolled out progressively and one that the community could easily contribute to by backfilling apis that they need them.
Summary

stop work on pigeon
commence work on providing a full set of dart api wrappers for the android and ios libraries.
improve the ffi documentation with more worked examples.
Encourage the community to contribute api wrappers.

The Dart team isn't familiar with the Pigeon or with the difficulties of using FFI with Flutter.  You'll probably have better luck filing issues here that are specific to Dart FFI, applicable to Flutter users and someone creating a command-line Dart program for example.
When you have recommendations about how Flutter could make FFI integration better, outside of Pigeon, you should file an issue with that information in the Flutter database.

Closing this issue. If you have specific recommendations for dart:ffi please file an issue in https://github.com/dart-lang/sdk/labels/library-ffi, and if you have specific recommendations for package:ffi please file a new issue in this repo.


README.md: version number for each attribute (both en & zh)

Also added the missing "attribute" [private] in the attribute list, as well as a link to the later section.

Looks good, thank you!


Injection works only one time!!!

Hi,
how can i use data injection more then one time.
Example:
When I click on a button to generate lines, data inection works fine.
All fields are filled with data.
Further clicks will still be created lines, but without data.
What can I do to make it work.

Same problem here, data injection only works the first time, am using a modal screen, so each time i open the modal depending on the record i am viewing it should load the data, but only works the first time, all other consecutive times, only blank input boxes are loaded, until i reload the page thats when it will inject the data only once.


cargo build --examples skips examples defined in Cargo.toml with required-features

Problem
I have a singular example located in examples/user.rs of the crate, which is part of a collective defined by a workspace list. I can verify that the example builds by using cargo build --example user; however, if I run cargo build --examples the example does not get built. I can verify this by adding nonsense to the singular example file - causing a compilation error - and have cargo build --examples pass while cargo build --example user fails.
Example repo: https://github.com/chipsenkbeil/cargo-examples-bug
Steps
Here's a minimal example of a Cargo.toml:
[package]
name = "cargo-examples-bug"
version = "0.1.0"
authors = ["Chip Senkbeil<EMAIL_ADDRESS>edition = "2018"
publish = false

[features]
my-feature = ["serde_json"]

[[example]]
name = "user"
required-features = ["my-feature", "serde/derive", "serde/rc"]

[dependencies]
serde = "1.0.117"
serde_json = { version = "1.0.61", optional = true }

And the associated example file in examples/user.rs:
fn main() {
    println!("Hello, world!");
}

Build the example with specific features via cargo build --example user --features="my-feature serde/derive serde/rc"
Attempt to build with examples and there is no complaint about providing features: cargo build --examples
Add something that will fail compilation in the examples/user.rs file
Verify that the build fails with cargo build --example user --features="my-feature serde/derive serde/rc"
Verify that the build does not fail (it never happens) with cargo build --examples

Possible Solution(s)
I'm not sure if this is expected, nor how you would handle passing different sets of required features to individual examples defined via cargo build --examples. This may be expected behavior, but I wasn't aware of this when building out a CI config around checking if examples build.
Notes
Platform: MacBook Air (M1, 2020)
Operating System: Mac 11.2.3
Output of cargo version: cargo 1.51.0 (43b129a20 2021-03-16)
Output of rustc --version: rustc 1.51.0 (2fd73fabe 2021-03-23)
Output of rustup toolchain list:
stable-aarch64-apple-darwin (default)
nightly-aarch64-apple-darwin

Related
Maybe https://github.com/rust-lang/cargo/issues/4663

Thanks for the report!  Indeed that is the intended behavior of required-features.
If you have multiple examples that need different features, you can do something like cargo build --examples --features feat1,feat2,feat3  If the features are mutually exclusive (incompatible with one another), then you'll need to run multiple cargo build commands.
Cargo could maybe print a note that it is skipping a target when in verbose mode (as it would probably be too noisy for normal behavior). But I'm not sure that would help for most situations since I suspect using -v is rare.
Otherwise, I think supporting the concept of "enable these features when this target is selected" is covered by #4663 and perhaps things like RFC 2887.

I was expecting something like that :) Closing in favor of referenced issue.


Syntax Error on line 325 of gs_audio.c

The line
gs_audio_instance_data_t data = {};
doesn't compile on Microsoft (R) C/C++ Optimizing Compiler Version 19.26.28805 for x64. It should be
gs_audio_instance_data_t data = {0};

Fixed in commit: https://github.com/MrFrenik/gunslinger/commit/38a5223ab3e78015caadf796285c34ce72ee7124


Added CITATION.cff file

What does this PR do?
As discussed on Slack, this PR adds a CITATION.cff file (the CFF format is described here). The citation file can be used by GitHub to show a "Cite this repository" button, which you can see for instance on NVIDIA's NeMo repository.
The data was filled in manually (so double-check for typos) from the data from the arXiv preprint, using cffinit.
Notes:

For people with multiple affiliations, I split them using a comma , instead of only mentioning the first affiliation.
I did not specify specific commit information, which is an optional field. We would have to remember to update it and I'm not too sure of its purpose.
The main description is the GitHub repo description
The abstract is from the arXiv preprint
The keywords are reused GitHub keywords for this repo, reformatted when useful

When the new paper is published, this file can be updated with the same tool (cffinit), with the new DOI link/name/abstract/author list.

Before submitting

[x] Did you read the contributor guideline?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Does your code adhere to project-specific code style and conventions?

PR review

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Confirm that the changes adhere to compatibility requirements (e.g., Python version, platform)
[ ] Review the self-review checklist to ensure the code is ready for review

Thank you @asumagic


Bug located in Stackup3D

Before submitting the issue

[X] I have searched among the existing issues
[X] I am using a Python virtual environment

Description of the bug
Conductors are of type dielectric. This makes all metal objects "solve_inside".
Steps To Reproduce
Run the machine learning patch example for HFSS.
Which Operating System are you using?
Windows
Which Python version are you using?
3.10
Installed packages
All dependencies installed.

Line 350 of stackup_3d.py
self._duplicated_material = self.duplicate_parametrize_material(material_name)

makes the cloned material dielectric even if it is copper.

Error is in stackup_3d.DuplicateParameterizedMaterial

Digging deeper, the error is in Material.is_conductor().  When conductivity is a parameter (i.e. a string) the Boolean
float(cond)

cannot be evaluated. Somehow the expression for the conductivity value needs to be evaluated.

@pyAnsysUser the ML example was removed as there were multiple errors. The problem with "solve inside" has been found and addressed. I'll create a PR soon with a simple example of a patch to demonstrate usage.

Should be resolved in PR #3566.


Add 'concat' option in C4 dataset

🚀 Feature Request
Benchmark repo has 2 options for C4 dataset samples: concat and truncate.
For LM training, 'concat' seems to be the only correct way forward. Can the 'concat' option present in the StreamingC4 from the benchmark repo be added to this repo as well?
Motivation
For LM training, using truncate instead of concat makes the training process redundant and the LM fails to learn anything.

Implementation
This works with the benchmark repo, not sure if this can directly be copied over here: https://github.com/mosaicml/benchmarks/blob/main/llm/src/data_c4.py

@abhi-mosaic

Hi @ananyahjha93, thanks for the feature request. I have created a PR https://github.com/mosaicml/streaming/pull/77 to add that support. The full thorough testing is WIP and will let you know once it is merge. Thank You!

Hi @ananyahjha93 , The above PR is merged. Let me know if you face any issues. Thank You!


put super on the prototype

Been doing research on extending Backbone and found this interesting bit:
// Set a convenience property in case the parent's prototype is needed later.
child.__super__ = parent.prototype;

At an initial glance it looks as if it could work this this.__super__.someMethod, but in reality you must call it off the constructor which is assigned directly above.
This patch instead places __super__ on the prototype.
/* One level of extension, calling Backbone.Model.prototype.initialize */

// Current
this.constructor.__super__.initialize.call(this);

// New
this.__super__.initialize.call(this);

// Manually
Backbone.Model.prototype.initialize.call(this);

I figure there must be an intentional reason for why its currently set up the way it is.  Any elaboration would be great.

+1. Why not give regular JS users the convenience instead of just CS users? Part of backbone's elegance is extending views, collections, etc. and calling the "super" method is part of the territory.

While @jashkenas said

Yes __super__ is not intended for you to use it directly, as the underscored name implies.

IMHO it's a super legit use of __super__. But that should be used only when you don't know what your super class is. If you know it you should call it directly.

ES6 Classes are almost out, bringing the built-in ability to call super() inside a class method. :+1: