Color/style/visibility UI improvements

[x] Census dropdown: make results background black (same as the search box in “Home”)
[x] Hyperlink: standardize all instances using theme color instead of light blue. TBD, kind of based on #189. The idea is that all hyperlinks are the same color throughout the app
[ ] “Details” and “Language profile” views: improve visibility of Audio/Video links by making them more prominent than the surrounding elements, either through a different element type such as a button, positional changes, or simple style adjustments.
[x] “Share via Email” button: colorize to avoid misconception that the button is disabled.
[x] Panel “breadcrumbs”: underline links to make their purpose obvious.

Use light green everywhere but do #189 first so we know how it looks.

@rperlin-ela trying a few options for the Audio/Video thing:
1 - Separate line and pronounced

2 - Same line, pronounced, same order as current

3 - Same line, pronounced, first

4 - Same line, outlined, first

5 - Grouped
Slightly more work for this since it's a new component but still easy, although not sure it helps our cause: https://material-ui.com/components/button-group/ (basically any of those on that page)
6 - Distinctions
I like option 4 above, but i think it should be distinct from the chips via subtle border (I know, getting bizzzay up there, but it feels worth it):

Keep in mind we could potentially scrap the Language name with the incoming "Up one level" btn for more space:

Takes
Mine
I like Option 6
Yours
Whatcha got?

Sorry Ross, gonna be mixing some convos here since I smashed them all into one issue, but shifting gears to the email btn...
So, the current gray DOES actually resemble a semi-standard (if there is such a thing), but we can't use the "standard". So I just tried a bunch of randos (don't worry about the code, just give me the number):
1 -  hsl(45 54% 55% / 1)

2 - hsl(32 54% 55% / 1)

3 - hsl(23 54% 58% / 1)

4 - shoot, lost it!

5 - #202020
Not quite as disable-y as current:

I'd probably go with option 3 myself. There are millions of colors, hard to just pick one at random (all the buttons had baked-in defaults, which I used), especially w/the competing surrounding elements' colors, you know?

Option 3 for the email button looks good to me!

On Mar 17, 2021, at 2:11 PM, Jason Lampel @.***> wrote:

Sorry Ross, gonna be mixing some convos here since I smashed them all into one issue, but shifting gears to the email btn...
So, the current gray DOES actually resemble a semi-standard (if there is such a thing), but we can't use the "standard". So I just tried a bunch of randos (don't worry about the code, just give me the number):
1 - hsl(45 54% 55% / 1)
2 - hsl(32 54% 55% / 1)
3 - hsl(23 54% 58% / 1)
4 - shoot, lost it!
5 - #202020
Not quite as disable-y as current:
I'd probably go with option 3 myself. There are millions of colors, hard to just pick one at random (all the buttons had baked-in defaults, which I used), especially w/the competing surrounding elements' colors, you know?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Decisive! Done. Let me know on the Audio/Video stuff, and hopefully an easy Yes/No on the B-crumbs, which I made the normal white instead of subdued, with the underlines the "secondary" text color (the off-white you see in some paragraphs). Keep or axe?

Give me a few hours on the other stuff, but breadcrumbs look good!

On Mar 17, 2021, at 2:31 PM, Jason Lampel @.***> wrote:

Decisive! Done. Let me know on the Audio/Video stuff, and hopefully an easy Yes/No on the B-crumbs, which I made the normal white instead of subdued, with the underlines the "secondary" text color (the off-white you see in some paragraphs). Keep or axe?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

On the right track, for sure. I think I like Option 6 too, but don’t scrap language name, I think, rather be repetitive than confusing, and I don’t want to hide the Profile— I’m still thinking through exactly what I want to show there but it’s a nice slot. So I think “Read more” stands.

On Mar 17, 2021, at 1:26 PM, Jason Lampel @.***> wrote:
we wouldn't necessarily need the bottom dashed border, but i'd like to somehow distinguish between the stuff above and below it. We could chip it, although I don't know about you, but i'm kinda chipped out in this view!
Here's another thought to de-bizzay it: instead of "Read more" as we know it, put a button called "Info" or "Description" or "Profile" before the Video button, and axe the semi-opaque preview we have now. Or, keep the preview, but I don't know that moving the button will be as intuitive that it affects the profile below it. Or maybe it would be if it were pronounced:
https://user-images.githubusercontent.com/4974087/111509702-9db55580-8712-11eb-8ea2-74562e1b7df0.png
Pronounced + first = certain encouragement to click, yeah?
Here's that approach w/the preview axed:
https://user-images.githubusercontent.com/4974087/111510029-f684ee00-8712-11eb-8654-1273807e3a15.png
With either of those approaches I think we could axe the top dashed border:
https://user-images.githubusercontent.com/4974087/111510291-42379780-8713-11eb-8ebf-0c24dd15451f.png
or supa-slim:
https://user-images.githubusercontent.com/4974087/111510506-7612bd00-8713-11eb-9772-edaddb6e7134.png
The preview is cool and all, but I definitely don't mind the "you're on a need-to-know/click-to-show basis".
Que piensas?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/190#issuecomment-801268633, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5EF7P7HO5XMN3LZQK3TEDQ5XANCNFSM4ZEE2XLQ.

Sounds logical to me. Any feelings on the top and bottom borders in Option 6?

Up to you— I like some bordering but not too much and not sure about dashed

On Mar 17, 2021, at 5:33 PM, Jason Lampel @.***> wrote:
Sounds logical to me. Any feelings on the top and bottom borders in Option 6?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/190#issuecomment-801457627, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5HEHAPA3E3BB4CF3ZTTEEN2TANCNFSM4ZEE2XLQ.

Ok I'll play w/some options. Thought the consistency would be good with the dashed in the census popout but they're not the same concept I guess, and maybe dashed was a poor choice there as well.
How about solid line on top or bottom or both (I'll make that decision before I push) using the pictured gray, but a more pronounced line (either through color, height, both, or a fancy lil guy like the bottom one here to separate the whole intro section from whatever is below it?

Hmm, dashed in the census pop-out I feel at peace with, but yeah I like the sound of what you describe, go for it

On Mar 17, 2021, at 5:45 PM, Jason Lampel @.***> wrote:
Ok I'll play w/some options. Thought the consistency would be good with the dashed in the census popout but they're not the same concept I guess, and maybe dashed was a poor choice there as well.
How about solid line on top or bottom or both (I'll make that decision before I push) using the pictured gray, but a more pronounced line (either through color, height, both, or a fancy lil guy like the bottom one here https://css-tricks.com/examples/hrs/ to separate the whole intro section from whatever is below it?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/190#issuecomment-801463420, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5F7WLTIIDMWCHHF3VDTEEPGTANCNFSM4ZEE2XLQ.

Sweet.
Sorry to pummel, one more related thing...
New layout breaks on mobile because Clear Selection wraps:

so are any of these acceptable to Average Human Users™?

i pasted screenshots since the icon gives it the feel too. but the icon kind of sucks (i think it's a "collapse" btn). so would any of these support whatever-text-you-pick?

I'd probably run with "De-select" myself (i think GIS calls it Unselect) combined with IoIosCloseCircleOutline.

IoIosCloseCircleOutline is a good call and I think “Clear” might cover it since we do use that word in other places. De-select seems good too.

On Mar 17, 2021, at 5:59 PM, Jason Lampel @.***> wrote:
Sweet.
Sorry to pummel, one more related thing...
New layout breaks on mobile because Clear Selection wraps:
https://user-images.githubusercontent.com/4974087/111543282-72456180-8738-11eb-8add-04b2749f9395.png
so are any of these acceptable to Average Human Users™?
https://user-images.githubusercontent.com/4974087/111543403-9f920f80-8738-11eb-8bc9-90a3fa12da00.png
https://user-images.githubusercontent.com/4974087/111543449-b0428580-8738-11eb-9066-f1bfe69cfa80.png
https://user-images.githubusercontent.com/4974087/111543480-bafd1a80-8738-11eb-82b7-8ec9d6633d49.png
https://user-images.githubusercontent.com/4974087/111543511-c4868280-8738-11eb-9d13-a69188d02f81.png
i pasted screenshots since the icon gives it the feel too. but the icon kind of sucks (i think it's a "collapse" btn). so would any of these support whatever-text-you-pick?
https://user-images.githubusercontent.com/4974087/111544008-7b82fe00-8739-11eb-9f0c-45159d47d17a.png
https://user-images.githubusercontent.com/4974087/111544053-89d11a00-8739-11eb-901a-0c03856545e0.png
I'd probably run with "De-select" myself (i think GIS calls it Unselect) combined with IoIosCloseCircleOutline.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/190#issuecomment-801470122, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5AONW3SOXDBN5AKC7TTEEQ5XANCNFSM4ZEE2XLQ.

Is it elegant in here or is it just this lil symbol guy?

Adds a little flair to supplement the big-cap first letter below, but... regardless of what we do up top, there's less and less of a chance that the user will see the description below without scrolling on mobile in the scenario like above (2 lines of chips, Read More, etc.), and same for 2-line Language names as well. Don't forget, it's going to be pushed down even further once "up one level" joins the party. There's just A LOT going on in there, and i'm not sure how to cram it all in without being illegible or tacky.
Re: the top/bottom border, i just scrapped it since i think the vertical spacing is enough to distinctionize from the chips and the Read More but let me know what you think.
Related: the "read more" has never really shown up well against the stuff behind it, even when it was a green link. any suggestions??
When there's no lang-level descrip, and 2 chip rows, fits "ok":

i think we're going to lose a good bit when we up-one-level it though. that link will need reserved space for long words without overlapping the main heading. cross that bridge i guess.
i'll push what i've got, probably been looking at nitpicks too long!

So after a spell away from the computer and 3/4 of an IPA, here's what I'm thinking on the fancy thing.

Double line is too much. Back to single. Flew too close to the sun.
It needs to be a more subtle shade than the disabled audio/video buttons. Too similar. Like it should be juuuust enough to stand out from the background of the panel behind it. It's not actionable and not even disabled, it's just an arbitrary boundary between two UI pieces.
Could probably be a bit further down from the buttons above it. Although maybe less important once we're back to one line.
I do like the fancy symbol though.

My two cents.

@rperlin-ela For the audio/video buttons, now that I have made them stand out simply with bold text, do you care if I put them back in the center? It looks weird all spread out on certain screen sizes. ￼￼

I agree— back in the center

On Apr 13, 2021, at 10:02 AM, Jason Lampel @.***> wrote:
@rperlin-ela https://github.com/rperlin-ela For the audio/video buttons, now that I have made them stand out simply with bold text, do you care if I put them back in the center? It looks weird all spread out on certain screen sizes. ￼￼
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/Language-Mapping/language-map/issues/190#issuecomment-818760902, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNKB5AVJI22V3NADYYOXFTTIRFGDANCNFSM4ZEE2XLQ.


Use/support Django's native JSONField

Django has a native JSONField for PostgreSQL since 1.9, which is superior to django-jsonfield.
django-jsonfield-compat could be used to handle both: https://github.com/kbussell/django-jsonfield-compat/

I've done this for pinax-stripe-light but not for original yet.


Inconsistent documentations

@zszabo14 commented on Thu Jul 23 2020
Some parameters are listed on quotes, some without:
zowe zos-files list data-set "ibmuser.asm"
vs
zowe zos-files create data-set-classic NEW.CLASSIC.DATASET
minor issue, works in both ways...

@BrandonJenkins14 Could you remind me if we have any recommendation or rules on the use of quotes in command lines? Seems we only defined the rules for variables. https://docs.zowe.org/stable/contribute/contributing.html#technical-elements

Hi Ashley, I don't know that we've written it down or done it consistently, but here's what I prefer. What do you think?
The following example shows how to use the files command to create a classic data set:
zowe zos-files create data-set-classic NEW.CLASSIC.DATASET

where:

NEW.CLASSIC.DATASET is the name of a new dataset that you want to create.

But sometimes I'm not sure if you need to define those, like with a data set. Would all users understand that they shouldn't literally copy "NEW.CLASSIC.DATASET" into their command line and substitute a real ds name instead? Maybe so.

Here's a similar take, just putting options out there:
The following example shows how to use the files command to create a classic data set:
zowe zos-files create data-set-classic <NEW.CLASSIC.DATASET>

NEW.CLASSIC.DATASET is the name of a new dataset that you want to create.

Hi @BrandonJenkins14, it seems the following different ways exist in docs to express variables in commands and we need a unified format.

@JamesBauman  -- should we make this a project for the near future?


Mod/tor/ us 544956

Added the ability to dynamically set locale if provided as a query parameter.

Created by mistake.


Update README.md

Oops, need to rebase this.

@locks Ok... After discussion I agree with all changes in readme. Will you resolve conflicts with master, or should I do this?

Derp, I was rebasing against my master instead of yours, hence the conflicts!
Congratulations on writing such a thorough README :)

@locks thanks. This is going to be merged to master.

:dancers: :dancer: :dancers:

Thank you but I think we can enhance readme even more in future. I'm not a kind of language guy. We are pretty happy with how this works in our app we want to bring this idea to community as good as we can. The doc is important:). Thanks for your work on this! :dart: Feel free to submit another pull request in the future.


Faster KONECTGraphReader implementation

The new KONECTGraphReader implementation also uses mmap() as proposed by @avdgrinten here: https://github.com/kit-parco/networkit/pull/160.
In benchmarks with bigger graphs, this implementation performs up to 4 times faster then its old counterpart. This performance increase basically results from iterating just once over the file, using mmap() and using the second property line of KONECT Graph files more safe and efficiently.
In addition, this implementation offers the possibility to handle graphs with multiple edges properly, by offering a parameter to specify a method.

The latest commit replaces the KONECTGraphReader with the FastKONECTGraphReader and exposes the reader, as well as the new enum in GraphReader to python.
Regarding the old constructor parameters: separator was quite useless in the first place, because KONECT graphs are separated by Tab and normally well formatted.
ignoreLoops might be a useful parameter because no hasEdge or addEdge has to be called, but

there is no special need for this parameter in the KONECT graph format
I think graphs should be read in as intended by the creator of the file and modification to the graph should be made by the user after reading
The performance increase does not even scale up to the share of loops in the total amount of edges, because the lines still have to be parsed

btw: travis failed due to internal errors

I assume this doesn't work on Windows, does it? So with this merged, Windows support is dropped also for the C++ core?

This does not work on Windows but I'm under the impression that NetworKit does not really work on Windows anyway. The current recommendation is to run it using WSL. This patch will not break WSL as WSL is able to handle mmap() just fine. OTOH if first-class Windows support is desired, this reader can easily be ported to use Windows' file mapping facilities like MapViewOfFile.
@michitux Do you have a use case for Windows?
@kolja-esders can you comment on the current state of Windows support?

@avdgrinten No, I don't have any use case. I was just under the impression that Windows compatibility is a (low-priority) goal of NetworKit.

Two quick comments:

Windows support should not go below the current standard (which is not native but close enough from what I was told).
Make sure the interface is not changed if a functionality is replaced. If the new default KONECT reader is the fast one, it should get the name of the old one (unless there are reasons not to do so).

https://github.com/mandreyel/mio seems to be a nice cross-platform library for memory-mapping under MIT license. I'm asking about this as I'm considering using memory mapping for the binary file reading (and possibly also writing), too.

My suggestion is to just use the raw mmap() interface for now. Writing a wrapper for Windows compatibility should be trivial and can be added in the future; we do not need a 3rd party library for that. As the rest of NetworKit does not build on Windows right now, writing such a wrapper right now is not desirable.
After we merge cmake, we could try to focus on native Windows compatibility in the next release.

Im not sure which error one could get when compiling the NetworKit C++ core on Windows, but as a compromise I could add some preprocessor directives like in io/DibapGraphReader.cpp. This will probably result in a more intuitive error handling

For now, I do not want to introduce #ifdefs. We can still do that after the Monday release. In particular, I want to avoid adding untestable code now and we can indeed not test Windows code as the rest of NetworKit does not compile on Windows.
In addition to that, this PR also needs a rebase.


THe  issue

Linux Mint 17.3
./speedtest-cli
Retrieving speedtest.net configuration...
Retrieving speedtest.net server list...
Failed to retrieve list of speedtest.net servers:

I had this error last night, but it started working again today. Not sure what is going on.

Please test with the recently released v1.0.0.


[Question] If You Have So Many Modules. How Do You Organize It?

I have one big app that:

Have APIs group. Handle all API actions.
Have a regular web app. Handle all regular web actions.
Some of them(API and Web) use the same source code that I create.

Let's imagine that I have 100-200 modules in total(API, Web, and Library Modules).

Could you share how you organize your modules in this case?
If I want to group by actions like API, Web, and Library. So I have 3 big folder for modules API in folder Api, Web in folder Web, and Library in folder Library. Is it possible?

You could define multiple module paths by turning on scan mode in the config and add multiple paths that can be scanned, you would have to update the namespaces for all files within those modules placed in the defined paths.
'scan' => [
        'enabled' => true,
        'paths' => [
            base_path('OtherModules/*'),
            base_path('vendor/*/*'),
        ],
    ],

I've done that in the past where I had a Modules and a CMSModules directly

You can simply make each modules installable module packages and require them where needed!
For instance, I may have User-module, but requires Status-module to function fully.


PI3 Ubuntu Install Failed

Trying to install but running into dependancy and cannot locate this specifc armhf for libluetooth3
ubuntu@ubuntu:~/GoveeBTTempLogger$ sudo apt-get install bluetooth bluez libbluetooth-dev
Reading package lists... Done
Building dependency tree
Reading state information... Done
bluez is already the newest version (5.53-0ubuntu3).
libbluetooth-dev is already the newest version (5.53-0ubuntu3).
bluetooth is already the newest version (5.53-0ubuntu3).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
ubuntu@ubuntu:~/GoveeBTTempLogger$ make deb
chmod a+x GoveeBTTempLogger/DEBIAN/postinst GoveeBTTempLogger/DEBIAN/postrm GoveeBTTempLogger/DEBIAN/prerm
dpkg-deb --build GoveeBTTempLogger
dpkg-deb: building package 'goveebttemplogger' in 'GoveeBTTempLogger.deb'.
ubuntu@ubuntu:~/GoveeBTTempLogger$ sudo apt install ./GoveeBTTempLogger.deb
Reading package lists... Done
Building dependency tree
Reading state information... Done
Note, selecting 'goveebttemplogger:armhf' instead of './GoveeBTTempLogger.deb'
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 goveebttemplogger:armhf : Depends: libbluetooth3:armhf but it is not installable
E: Unable to correct problems, you have held broken packages.
ubuntu@ubuntu:~/GoveeBTTempLogger$

I've not used ubuntu recently. I just did a quick search for the package and it looks like the library should be available. https://packages.ubuntu.com/xenial/libbluetooth3 after I'd initially looked at https://packages.debian.org/sid/libbluetooth3
Have you tried adding a -y option to your apt install command? In my experience -y causes it to automatically try to install dependencies.

@jbronikowski I am going to guess that you are running Ubuntu 64bit on your Pi3 like I am. I ran into this issue as well. I changed this line from armhf to arm64 and ran make deb again. This time everything compiled and installed w/o issue and I am pulling data. Although admittedly I am still figuring out how to pull it consistently, the service seems to pull it in at random times.

@tylerobara Thanks for the suggestion.
I was just looking at https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-architecture and trying to figure out if I should be specifying either a list of architectures or any or all on that line to make the package build and install on multiple architectures. Do you have any insight?

@tylerobara Thanks for the suggestion.
I was just looking at https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-architecture and trying to figure out if I should be specifying either a list of architectures or any or all on that line to make the package build and install on multiple architectures. Do you have any insight?

No, sorry. TLDR: I guessed.
This is a step out of my knowledge level. IIRC I was able to run the make deb command without issue but sudo apt-get install ./GoveeBTTempLogger.deb failed and gave me the error about armhf. I knew I had installed Ubuntu 64bit specifically for something I was testing a long time ago. I dug around and saw the line I mentioned above and took a shot. After running through the make and install again, it worked fine. I would not know how to provide a way for this to be compiled programmatically based on the OS of the user.
No up for me is pulling this data into Home Assistant and determining if I can scale to multiple of these sensors.


chore(app): Rework some styling changes after the flutter upgrade

Note, I didn't manage to set the background of the candlestick chart back to white, but we might want to consider replacing that dependency altogether, since it doesn't seem to be maintained anymore.


How long it will take to reimplement this code in tensorflow?

Hey maudzung,
I am thinking to re implement your work in tensorflow. Could you please explain how long it will take if I start now?
I have limited time to do so. that is the reason.
Thanks in advance.

Hi @mariya12290
I think it's quite easy to re-implement this Pytorch work in tf2.
First, you need to build the model, create the data loader using tf2, then write the training and testing scripts for the task. That's all :-)


Error: Could not locate file (500 error)

System Info
node: 22.7
nextjs: 14
Environment/Platform

[X] Website/web-app
[ ] Browser extension
[X] Server-side (e.g., Node.js, Deno, Bun)
[ ] Desktop app (e.g., Electron)
[ ] Other (e.g., VSCode extension)

Description
I'm trying to load https://huggingface.co/onnx-community/Qwen2.5-0.5B-Instruct and i get the error Error: Could not locate file: "https://huggingface.co/onnx-community/Qwen2.5-0.5B-Instruct/resolve/main/onnx/decoder_model_merged_quantized.onnx".
seems like someone else is running into the same issue https://huggingface.co/onnx-community/Qwen2.5-0.5B-Instruct/discussions/1
how do i fix this? i tried copying the download link directly but  somehow it appends .json to the URL like so: https://huggingface.co/https://huggingface.co/onnx-community/Qwen2.5-0.5B-Instruct/resolve/main/onnx/model_quantized.onnx/resolve/main/config.json\
how do i fix this to point to the correct model?
I'd love to use the new qwen2.5 0.5B and the 1.5B
thank you in advanced

code:
import { pipeline, env, Pipeline } from '@xenova/transformers';

// Set the Xenova model directory
env.localModelPath = '/tmp/models';

let generator: Pipeline | null = null;

export async function POST(req: Request) {
  let text;
  try {
    ({ text } = await req.json());
    console.log('Received text:', text);
  } catch (error) {
    console.error('Error parsing request body:', error);
    return NextResponse.json({ error: 'Invalid request body' }, { status: 400 });
  }

  try {
    if (!generator) {
      console.log('Initializing model...');
      generator = await pipeline('text-generation', 'https://huggingface.co/onnx-community/Qwen2.5-0.5B-Instruct/resolve/main/onnx/model_quantized.onnx');
      console.log('Model initialized successfully');
    }

    console.log('Generating text...');
    const messages = [
      { role: 'system', content: 'You are a CBT and ACT AI therapist. Provide advice or another way to frame the situation from a therapist perspective. Ensure it\'s only 1 sentence! Omit unnecessary words that aren\'t related to the feedback or advice.' },
      { role: 'user', content: text }
    ];

    // Apply chat template
    const prompt = generator.tokenizer.apply_chat_template(messages, {
      tokenize: false,
      add_generation_prompt: true,
    });

    console.log('Constructed prompt:', prompt);

    console.log('Calling generator...');
    const output = await generator(prompt, {
      max_new_tokens: 128,
      do_sample: true,
      temperature: 0.7,
      return_full_text: false,
    });

    console.log('Raw result:', JSON.stringify(output, null, 2));

    if (!output || !Array.isArray(output) || output.length === 0 || !output[0].generated_text) {
      throw new Error(`Invalid response from model: ${JSON.stringify(output)}`);
    }

    const generatedText = output[0].generated_text.trim();
    console.log('Generated text:', generatedText);
    return NextResponse.json({ result: generatedText });
  } catch (error) {
    console.error('Error in POST request:', error);
    console.error('Error stack:', error instanceof Error ? error.stack : 'No stack available');
    console.error('Error details:', JSON.stringify(error, Object.getOwnPropertyNames(error)));
    return NextResponse.json({ error: `Server error: ${error instanceof Error ? error.message : String(error)}` }, { status: 500 });
  }
}```

### Reproduction

1. I use this url to add it to Cursor IDE (https://huggingface.co/onnx-community/Qwen2.5-0.5B-Instruct)
2. It updates the code for me
3. get 500 error

it works when i do qwen1.5 but not the latest 2.5

im getting the same error for the new llama 3.1 1B
how do i resolve this issue and prevent it for the future models?
Error: Could not locate file: "https://huggingface.co/onnx-community/Llama-3.2-1B/resolve/main/onnx/decoder_model_merged_quantized.onnx"

i downloaded the model manually and placed in the the onnx directory
now i get this error after renaming the model to decoder_model_merged_quantized.onnx: Something went wrong during model construction (most likely a missing operation)

You need to specify the file name (and sometimes it's missing the folder as the onnx model files are not always in the onnx folder).
In the case of Qwen2.5, you need to specify the model: "model_quantized" (or whichever exact model version you want to use, check in files tab in HuggingFace and then open the onnx folder, pick the model you want to use and use the name, without the extension).
But then that particular model doesn't work indeed. It's failing at .loadModel() when creating an inference session can't create session.

@nicolasembleton thx.
So does that mean qwen 2.5 or llama 3.2 isn't supported yet by transformers.js?

Llama 3.2 1B is definitely supported, as there seems to be a demo available.
A good starting point might be the example code for Phi 3.5:
https://github.com/huggingface/transformers.js-examples/blob/main/phi-3.5-webgpu/src/worker.js
For one thing, you don't need to provide the specific .onnx file. Transformers.js will grab the correct one based on the settings you provide when you create the pipeline / model.
More examples:
https://github.com/xenova/transformers.js/tree/v3/examples
https://github.com/huggingface/transformers.js-examples/tree/main

thank you and great find.
do you know if llama 3.2 1B will work with nextjs? I'm new to dev so i'm thinking it could be a pipeline not being compatible yet?
perhaps the @transformers pipeline isnt supported yet but @huggingface is?

perhaps the @transformers pipeline isnt supported yet but https://github.com/huggingface is?

Do you mean version 2 and version 3 of Transformers.js? if so, you need to use Transformers.js V3.

@flatsiedatsie how do i do that? is there a step by step guide?
also are all the features from v2 still functional?
i would hate to install it and cause breaking changes.
thanks in advanced!

V3 is much MUCH faster because it adds support for WebGPU.
Read through the documentation: https://huggingface.co/docs/transformers.js/index
And there are tons of examples (linked above). Always search in these issues too, there is loads of knowledge in them.

Appreciate your response, thx


ARK SE: Restart Buttom

Bug Report
System Information

Windows Server 2019
<IP_ADDRESS>
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms
Clicking "Restart" will instantly close the server and does not perform any sort of restart
Reproduction
Launch the Ark server and then click Restart

Noticed the same behaviour under ubuntu 20.04 as well. The process appears to get killed without saving, rolling the server back to the previous save-point at startup.

ARK doesn't have any kind of save command, so that's a fault of the game - not of AMP.

ARK doesn't have any kind of save command over RCON, so that's a fault of the game - not of AMP.

You can use the saveworld command over RCON which will force a save, this is how i've seen other tools handle it at least for both ARK & ATLAS. I can do a bit of testing later to verify the exact syntax, but it will either be that or admincheat saveworld.

AMP sends a saveworld but the server doesn't respond to it.
                rcon.AutoReconnect = false;
                rcon.SendMessage(new SourceRconPacket("saveworld")).UnwrapTask();
                Task.Delay(500).UnwrapTask();
                rcon.SendMessage(new SourceRconPacket("doexit")).UnwrapTask();

If I manually trigger a saveworld I get a reponse...

AMP does send a "saveworld" prior to shutdown. Do you get that line in the log when you hit stop in AMP?

Yes


Null Pointer exception when running spark garbage collection job

Examining some backtrace, this might be not specifying the API URL in configuration, property spark.hadoop.lakefs.api.url or another property nearby.  This is specified e.g. here.
In that case the issue here is to generate useful error messages for simple configuration errors.

Moving this issue to closed since it was a due to provided runtime dependencies issue of hadoop-aws and hadoop-common different versions.


integrations-docs: Update text in create-bot-construct-url.md.

Revises the text about including a stream and a topic incoming webhook URLs in create-bot-construct-url.md for clarity and consistency.
Screenshots and screen captures:

Current text

Revised text

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Sure, seems reasonable to me.
By the way, when I was looking at these instructions yesterday, I had no idea what would happen if you don't specify a topic. Where do the messages get sent? Can we answer that question here?

By the way, when I was looking at these instructions yesterday, I had no idea what would happen if you don't specify a topic.

For most integrations, the integration picks a sensible topic specific to the integration payload -- for example, posting to the repository+branch name where commits were pushed for a Git push, or posting with an issue ID for a new issue/PR.
Specifying a topic is a thing you do when you specifically want all the payloads from a given webhook to go to some specific place.

(That's kinda its own thing that we may want to explain with more words, so I'm merging this tweak)


Line breaking messing up when shortcode is used right before a line break

Nice gem! This will be a long, but hopefully helpful, description of the issue.
I have integrated this into a Middleman site and setup a [link] shortcode. However, I noticed when the [link] shortcode is at the end of a line right before the break, something is messing up.
A little background...
Similar to Rails, Middleman uses (sometimes modified) versions of the Padrino helpers. (Notably the simple_format method which is similar to Rails'.)
Also in Middleman, you can specify some YAML Frontmatter which is then available in your page template.
Now my setup
My [link] shortcode can accept an id attribute for referencing another page in the site. (Similar to the RB Internal Links WordPress plugin.) For example:
I want a [link id="123"]link to page with id of "123"[/link].

This would output the following:
I want a link to page with id of "123".

In my page I have the following Frontmatter. Notice that the [link] shortcode occurs right before the two line breaks.

---
some_copy: |
  This is some copy. [link id="123"](And here is a link to page 123.)[/link]

  This is a second paragraph of copy.

---

I have a helper method for processing the shortcodes:
# compose_helpers.rb

def compose(something='')
  # I have a presenter for this shortcode that accepts the Middleman application object as the :app additional attribute. (Needed for handling the page lookup.)
  Shortcode.process(something, { app: self }).html_safe
end

Now in my layout for displaying this page I have the following:
<%= simple_format compose(current_page.data.some_copy) %>

The problem
Normally the simple_format will convert two line breaks into
 tags. However, something is happening when the shortcode is parsed and it looks like one of the line breaks disappears. simple_format then sees this as a single line break and uses a
 instead of creating paragraphs.
In other words,
<%= simple_format current_page.data.some_copy %>

would normally output the following:

This is some copy. [link id="123"](And here is a link to page 123.)[/link]

This is a second paragraph of copy.

However, if I tell it to process the shortcodes first
<%= simple_format compose(current_page.data.some_copy) %>

I get the following:

This is some copy. (And here is a link to page 123.)
This is a second paragraph of copy.

I'm not sure where the problem is. Any ideas?

I noticed I if put a space character right after that closing [/link] then everything works fine. I've been looking at lib/shortcode/parser.rb but will need to study a little more to figure out what is going on.

@robertwbradford thanks for the detailed report. I'm away at the moment so won't be able to look at this until next week. Take a look at the tests, I think there was an issue with carriage returns right after a tag so there should be tests them, take a look and see if this covers your case and it not try creating a failing test with your use case. I can then look into it next week if you've not managed to work it out before then.

Thanks, @kernow. I may have some time later today to go through the tests. If not, next week.

I've not been able to replicate this issue on the 2.0 branch, newlines are respected when parsing tags so I'm going to close this issue. If you are still experiencing this issue please reopen with a failing test example.


nixos/plasma5: use mkPackageOptionMD

Description of changes
fix newly merge mkPackageOption uses. hopefully there won't be any more.

Thanks!


Applefix

Some fixes that get skewer to compile for me on OSX 10.7, gcc 4.2.1.

Not needed; fixed by c61bb90ee


Click on element in swiper triggers a slide

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 6.4.5

Platform/Target and Browser Versions: MacOS Chrome 89.0.4386.0

Live Link or JSFiddle/Codepen or website with issue:

What you did
Implemented a slider to contain some children - images of products with some info.
Using these configs for the slider :
      spaceBetween: remToUnitlessPx(space.map['three-quarter-x']),
      slidesPerView: 2,
      slidesPerGroup: 2,
      preloadImages: true,
      centerInsufficientSlides: true,
      updateOnImagesReady: true,
      navigation: {
        prevEl: prevButtonRef.current || undefined,
        nextEl: nextButtonRef.current || undefined,
        disabledClass: 'SwiperCarousel__navigation--disabled'
      },
      pagination: paginationElementRef.current
        ? {
            el: paginationElementRef.current,
            type: 'bullets',
            clickable: true,
            hideOnClick: false,
            bulletClass: 'SwiperCarousel__bullet',
            bulletActiveClass: 'SwiperCarousel__bullet--active'
          }
        : undefined```

Expected Behavior
On clicking an image I expect the click to work as a click and not a slide
Actual Behavior
The slider randomly triggers a slide when clicking on one of the images I have inside the slider (the click shows some information about an image as an overlay on the image)
https://user-images.githubusercontent.com/26385540/104374111-61396380-54ef-11eb-8777-2cad7ab1510f.mp4

can you provide live demo?

can you provide live demo?

What happened with this ticket?

What happened with this ticket?

I closed it since there is a way to fix this by changing the shortSwipes to false and the threshold to some value greater that 0.

I closed it since there is a way to fix this by changing the shortSwipes to false and the threshold to some value greater that 0.

same issue, I set threshold=10, now it's right


npm install reaqt error

Hi, facing this error while installing  npm install reaqt --save
E:\sid-proj>npm install reaqt --save
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\KSHALI3\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install" "reaqt" "--save"
npm ERR! node v6.9.5
npm ERR! npm  v4.1.1
npm ERR! code ENOSELF
npm ERR! Refusing to install package with name "reaqt" under a package
npm ERR! also called "reaqt". Did you name your project the same
npm ERR! as the dependency you're installing?
npm ERR!
npm ERR! For more information, see:
npm ERR!     https://docs.npmjs.com/cli/install#limitations-of-npms-install-algorithm
npm ERR!
npm ERR! If you need help, you may report this error at:
npm ERR!     https://github.com/npm/npm/issues
npm ERR! Please include the following file with any support request:
npm ERR!     E:\sid-proj\npm-debug.log

I think the project where you're trying to install this is also named reaqt.
npm does not allow that because that will lead to a circular dependency.

Nope, I saw that and changed the name.

npm ERR! Refusing to install package with name "reaqt" under a package
npm ERR! also called "reaqt". Did you name your project the same
npm ERR! as the dependency you're installing?

The error you posted seems to be old, can you share your package.json file

Thanku, working I forgot to update the package.json

When I do npm run dev
E:\sid-proj>npm run dev

<EMAIL_ADDRESS>dev E:\sid-proj
reaqt dev

PAGES  Defining routes...
fs.js:951
return binding.readdir(pathModule._makeLong(path), options.encoding);
^
Error: ENOENT: no such file or directory, scandir 'E:\sid-proj\pages'
at Error (native)
at Object.fs.readdirSync (fs.js:951:18)
at getPages (E:/sid-proj/node_modules/reaqt/dev/server.js:29:20)
at Object. (E:/sid-proj/node_modules/reaqt/dev/server.js:36:1)
at Module._compile (module.js:570:32)
at loader (E:\sid-proj\node_modules\babel-register\lib\node.js:144:5)
at Object.require.extensions.(anonymous function) [as .js] (E:\sid-proj\node_modules\babel-register\lib\node.js:154:7)
at Module.load (module.js:487:32)
at tryModuleLoad (module.js:446:12)
at Function.Module._load (module.js:438:3)
at Module.require (module.js:497:17)
at require (internal/module.js:20:19)
at Object. (E:\sid-proj\node_modules\reaqt\index.js:17:10)
at Module._compile (module.js:570:32)
at Object.Module._extensions..js (module.js:579:10)
at Module.load (module.js:487:32)
npm ERR! Windows_NT 6.3.9600
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\KSHALI3\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "run" "dev"
npm ERR! node v6.9.5
npm ERR! npm  v4.1.1
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>dev: reaqt dev
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script 'reaqt dev'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the sid-proj package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     reaqt dev
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs sid-proj
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls sid-proj
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     E:\sid-proj\npm-debug.log

Failing at reaqt dev script...aam I doing anything wrong...Added the script in package.json

You need to make a folder called pages and add your pages into that.
Reference: https://github.com/siddharthkp/reaqt#pages

Do you think I should make a getting started page as well? The readme doesn't have direct instructions

Oh ok. Thought , this project should be running with few sample pages. Found pages created in example folder.

I think can be updated it in Readme only.


Schur decomposition lapack wrapper

This is an implementation of a wrapper for LAPACK's gees routine (in order to close #6478).
The schur primitive calls this wrapper on CPU. Tests for this primitive to make sure it matches scipy's schur decomposition and batching tests are added in tests/linalg_tests.py.

There was a reference to the gees wrapper that was just in the body of the linalg.py primitives file, I put it in the scope of the schur primitive so it doesn't cause any issues without calls to the primitive. I also added an if _xla_extension_version > 38 condition that skips the test for older version.

Also, once you address that final change, please squash the changes into a single commit (see https://jax.readthedocs.io/en/latest/contributing.html#single-change-commits-and-pull-requests) Thanks!

There was a reference to the gees wrapper that was just in the body of the linalg.py primitives file, I put it in the scope of the schur primitive so it doesn't cause any issues without calls to the primitive. I also added an if _xla_extension_version > 38 condition that skips the test for older version.

Thinking about this, I think it's probably even better to use the condition jax._src.lib.version >= (0, 1, 72).
You can think about _xla_extension_version as the version of the XLA/TF source trees. In this case, this code is in the JAX tree, so I think your best bet is to use a version number defined in that tree also. (Ideally we'd have a monotonic version number in the JAX tree as well, but the jaxlib version number will do fine I think.)

You have some flake8 lint failures as well.

@hawkinsp - the reason I suggested checking xla_extension_version is because I was planning to patch the CL to run internal tests for the new code. If we gate on jaxlib release, we won't see any potential failures in CI until the next version of jaxlib is released.

Kokoro is doing a jaxlib build from source for at least two of the builds. So we should see the failures right here in Github.

Thanks, that makes sense. Agreed that checking on jax._src.lib.version is the better option here.

I think I really messed up my git repo, I am going to do another pull request from a fresh repository. I really apologize.


FileChooser navigation with fingers(swipes) is broken

The file list does not scroll with your finger/mouse swipes, and at the end of the finger movement will be opened the folder under your finger. So it's unusable in android for now. This can be seen in the "showcase" example.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Duplicate of https://github.com/kivy/kivy/issues/2497.

This is not a duplicate, it is a new bug. You cannot scroll the view at all with touch-drag.

Looks like this bug doesn't affect all FileChoosers, it seems to be a Showcase bug.

Confirmed this is a bug, I have an rpi touch screen and use FileChooser and you cannot scroll without opening the folder under your finger, it makes it pretty much unusable. Very surprised this is still the case with 1.10.1 after 2 years.


Fix calloc/free warnings building Zephyr with ECSDA

Signed-off-by: Fabio Utzig<EMAIL_ADDRESS>

Superseeded by #290


Google .PKG option

Can Google Chrome have a .PKG option? Google provides a .PKG, and is available as an autopkg recipe.
In my experience, the DMG installed Chrome needs a helper installed with admin privileges in order for updates to function. With Chrome 116 on the horizon bringing weekly security updates, I would very much like to keep that functionality in tact.

Google Chrome PKG recipe has been added and the wiki page for PKG app type support has been updated.

Recipe added. If any issues occur with the recipe please open a new issue.


Store xnd objects in plasma

We should be able to store xnd objects in the Plasma store, similar to how you can store arrow memory. In the blog post introducing Apache Arrow's usage with Plasma, they outline the motivation for the store. Similar to arrow, it would be beneficial to store xnd objects, so that multiple processes on on one computer could all access them without copying or serialization.

Originally I thought I could store an xnd object in Plasma without copying, but I no longer believe this to be the case. The create command returns a new memory buffer that Plasma creates and hands you a pointer to. It doesn't seem to support taking ownership over an existing piece of memory.
This does make sense, since Plasma stores immutable data.

I think the idea is that you create an object in plasma, then seal it.
Plasma becomes the "malloc".   It is the owner of the shared memory.
This makes sense.

We need to make sure xnd can take a reference to other memory and
We need to make sure that plasma can accept ndtypes descriptions for the
memory

-Travis

Travis Oliphant
Quansight
On Mon, Mar 5, 2018 at 11:25 AM, Saul Shanabrook<EMAIL_ADDRESS>wrote:

Originally I thought I could store an xnd object in Plasma without
copying, but I no longer believe this to be the case. The create command
returns a new memory buffer that Plasma creates and hands you a pointer to.
It doesn't seem to support taking ownership over an existing piece of
memory.
This does make sense, since Plasma stores immutable data.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/plures/xnd/issues/4#issuecomment-370495933, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPjoL45TQuxr8rAWByFsriZPnRU2ubTks5tbXTvgaJpZM4Scd2O
.

Yeah that makes sense.
So do we have to be able to serialize the XndObject to that contiguous buffer?
typedef struct {
    PyObject_HEAD
    MemoryBlockObject *mblock; /* owner of the primary type and memory block */
    PyObject *type;            /* owner of the current type */
    xnd_t xnd;                 /* typed view, does not own anything */
} XndObject;

In trying to get a sense of exactly where the memory is stored, is it possible to inspect these fields
in a Python REPL on an an xnd instance?

On Mon, Mar 05, 2018 at 06:18:54PM +0000, Saul Shanabrook wrote:

So do we have to be able to serialize the XndObject to that contiguous buffer?

As I understand this https://arrow.apache.org/docs/python/plasma.html#putting-and-getting-python-objects
"This works with all Python objects supported by the Arrow Python object serialization.",

we'd need Arrow serialization.  I'm not familiar with plasma though.

We don't want to serialize anything (except the description of the
buffer).  We want to

allocate memory in plasma and ensure that the ndtypes description is
also stored.
get a handle the to the buffer and place it in the xnd container
get the ndtypes description of the buffer and ensure the xnd container
has the same one.

-Travis

Travis Oliphant
Quansight
On Mon, Mar 5, 2018 at 12:18 PM, Saul Shanabrook<EMAIL_ADDRESS>wrote:

Yeah that makes sense.
So do we have to be able to serialize the XndObject to that contiguous
buffer?
typedef struct {
PyObject_HEAD
MemoryBlockObject mblock; / owner of the primary type and memory block */
PyObject type;            / owner of the current type /
xnd_t xnd;                 / typed view, does not own anything */
} XndObject;
In trying to get a sense of exactly where the memory is stored, is it
possible to inspect these fields
in a Python REPL on an an xnd instance?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/plures/xnd/issues/4#issuecomment-370512504, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPjoD4qwbJ2jUel_gPf4f2m5RuHOUykks5tbYE4gaJpZM4Scd2O
.

So would the desired user flow be something like:

Create plasma client connected to store
Tell xnd to use this client
Create some containers with xnd
Retrieve their plasma ids
Start another process
Create plasma client again
Get access to xnd containers using plasma ids

Right now, memory allocation in xnd uses the ndt_aligned_calloc function. This happens when creating a new xnd container, when creating a Ref, and when creating bytes.
If we could get a pointer back from Plasma's create function, then we could use that instead of ndt_aligned_calloc. If we are storying bytes or references, then this means storing multiple Plasma objects.
Is it right to assume that different plasma clients could have different pointer addresses for the same plasma object? Because we are using virtual memory?
If this is the case, then it might not be good to store the pointer for a Ref directly in the xnd data, but instead store the plasma ID instead (same with bytes).

I opened an issue asking if it was possible to get a pointer back from the plasma store, to write to https://github.com/jupyterlab/jupyterlab/pull/3893

Here is my current take on it.
Option 1:
I would not like to replace existing allocation functions.  Instead, I'd like to proceed cautiously and create a separate pxnd (p for parallel) object.
We would only support pointer-free types.  The pxnd object would have a datashape Python string object and a plasma object id and could therefore be pickled.
Creating this object would allocate a data block in plasma.
I assume that the picklable `pxnd``object would be sent via a pipe to consumer processes.
In such a process, the consumer would call something like pxnd.get(), which would use the existing MemoryBlockObject in the same way as the from_buffer function and return a regular xnd object.
That is to say, the memory in the xnd_master buffer is marked as external, but creation of the xnd object largely proceeds in the usual way.   This is the same way the existing PEP-3118 imports are handled.
This all hinges on the possibility of getting a raw pointer from the plasma data store.
If Arrow headers are required for this, I'd like to check for these headers in ./configure and make the compilation of the pxnd object conditional on the presence of the headers.
In general, I'd like to keep the pxnd object in a separate section or file and be able to compile and use xnd without plasma support.
Option 2:
Only allow NumPy types and hope that the existing plasma functions take a PEP-3118 buffer.
In that case, we'd need to add PEP-3118 exporting facilities to xnd.

Option 2 is a non-starter in my mind as we need something that extends
NumPy types.  The data-shape and xnd should be usable where PEP-3118 is not.
A variant for Option 1 is fine in my mind too.  Plasma must be an
optional part.  It is only an important use-case and not something xnd
should rely on.
Because XND can accept foreign buffers as it's memory, this should be
straightforward.  You just have another module for interfacing with
plasma.  Introducing ndtypes to plasma is a good opportunity to see how
ndtypes and Arrow interact.

Travis Oliphant
Quansight
On Tue, Mar 6, 2018 at 6:19 AM, Stefan Krah<EMAIL_ADDRESS>wrote:

Here is my current take on it.
Option 1:
I would not like to replace existing allocation functions. Instead, I'd
like to proceed cautiously and create a separate pxnd (p for parallel)
object.
We would only support pointer-free types. The pxnd object would have a
datashape Python string object and a plasma object id and could therefore
be pickled.
Creating this object would allocate a data block in plasma.
I assume that the picklable `pxnd``object would be sent via a pipe to
consumer processes.
In such a process, the consumer would call something like pxnd.get(),
which would use the existing MemoryBlockObject in the same way as the
from_buffer function and return a regular xnd object.
That is to say, the memory in the xnd_master buffer is marked as
external, but creation of the xnd object largely proceeds in the usual
way. This is the same way the existing PEP-3118 imports are handled.
This all hinges on the possibility of getting a raw pointer from the
plasma data store.
If Arrow headers are required for this, I'd like to check for these
headers in ./configure and make the compilation of the pxnd object
conditional on the presence of the headers.
In general, I'd like to keep the pxnd object in a separate section or
file and be able to compile and use xnd without plasma support.
Option 2:
Only allow NumPy types and hope that the existing plasma functions take a
PEP-3118 buffer.
In that case, we'd need to add PEP-3118 exporting facilities to xnd.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/plures/xnd/issues/4#issuecomment-370763793, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPjoEbuyKM_mcI1lN7_BkSLhfYfTogDks5tbn7tgaJpZM4Scd2O
.

On Tue, Mar 06, 2018 at 01:12:04PM +0000, Travis E. Oliphant wrote:

Option 2 is a non-starter in my mind as we need something that extends
NumPy types.  The data-shape and xnd should be usable where PEP-3118 is not.

Agreed. It's more of a fallback if some unknown problems arise.

The C++ plasma code looks like it can give you back a pointer to the buffer(https://arrow.apache.org/docs/cpp/classplasma_1_1_plasma_client.html#ac18ab9cc792c620a97a3dcb165e0ecd7), but I am not sure if it's possible for use that, since we are using C and they don't export anything to C.
The Cython wrapper might also be helpful, but I also don't think it's possible for us to use that in our C code, since they don't make anything public. I opened a thread on the arrow listserve to see if they have any guidance on using the plasma API from C.

We may have to make a pull request to add a simple C-export of the buffer.
Alternatively the C++ code that binds libxnd to plasma can create that
API.   We can start there.  This should be a separate library from libxnd.
-Travis

Travis Oliphant
Quansight
On Tue, Mar 6, 2018 at 4:01 PM, Saul Shanabrook<EMAIL_ADDRESS>wrote:

The C++ plasma code looks like it can give you back a pointer to the
buffer(https://arrow.apache.org/docs/cpp/classplasma_1_1_
plasma_client.html#ac18ab9cc792c620a97a3dcb165e0ecd7), but I am not sure
if it's possible for use that, since we are using C and they don't export
anything to C.
The Cython wrapper
https://github.com/apache/arrow/blob/master/python/pyarrow/plasma.pyx
might also be helpful, but I also don't think it's possible for us to use
that in our C code, since they don't make anything public. I opened a
thread
<EMAIL_ADDRESS>on the arrow listserve to see if they have any guidance on using the plasma
API from C.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/plures/xnd/issues/4#issuecomment-370944392, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAPjoFW0fo8oRHDjrdot_CGa12pq9A1bks5tbwcggaJpZM4Scd2O
.

adding @robertnishihara @pcmoritz @pitrou to go along with the mailing list thread. Plasma buffers implement the buffer protocol so it sheems like you should be able to use that

Is xnd able to use foreign buffers as backing memory? If not, is there a reason why? Basically Plasma needs to allocate the memory itself in a certain way, to be able to hand it to its various client processes.
(I'm assuming there may be limitations such as contiguity or restriction to certain well-behaved types)

To be clear, my question is whether xnd supports the equivalent of np.frombuffer() to create xnd objects.

There are some comments here about type metadata:

We need to make sure that plasma can accept ndtypes descriptions for the
memory

Plasma is unconcerned with the contents of the memory. So you will need to take responsibility for writing a serialized metadata header to prefix the xnd memory. We are using Flatbuffers for our serialized metadata for Arrow

Is xnd able to use foreign buffers as backing memory?
Yes. So I think we should be able to call the Plasma Python API from our C code that exposes our own Python API. By accessing that buffer in C, then we can get access to the pointer.

I will try that approach before moving forward on any kind of C wrapper around the Plasma C++ API.

On Wed, Mar 07, 2018 at 04:03:56PM +0000, Antoine Pitrou wrote:

Is xnd able to use foreign buffers as backing memory?

Yes, there's xnd.from_buffer(), using PEP-3118. xnd has the MemoryBlockObject
that manages foreign memory.

The problem with using from_buffer is that PEP-3118 does not support full ndtypes.  What I'd like to see is plasma support for any structure that can be described by libndtypes (i.e. datashape).   We will need this eventually.  It would be good to get collaborating early if possible.

Does plasma set the format of the the buffer that is returned?

No -- the Plasma data is just bytes. You'll need to define a serialized data header for xnd data. The way we do it for Arrow objects is:

4 bytes for metadata size
serialized metadata (we use Flatbuffers)
message body

See https://github.com/apache/arrow/blob/master/format/IPC.md#encapsulated-message-format

No -- the Plasma data is just bytes. You'll need to define a serialized data header for xnd data.

@wesm OK, that makes sense. Just to make sure I understand correctly, the plasma python API provides a PEP 3118 buffer but without the shape, format, strides, ndim, suboffets fields, so it's really just a pointer to the data. So you can write whatever data format you want, and any type information if you want to use that later to read the data. i.e. there isn't anything arrow specific in plasma.

Is xnd able to use foreign buffers as backing memory? If not, is there a reason why? Basically Plasma needs to allocate the memory itself in a certain way, to be able to hand it to its various client processes.

@pitrou Ideally, we would access Plasma with libxnd, which isn't specific to Python. That is more direct  than going through the Python API and also other language bindings for libxnd will be able to use that integration.

So you can write whatever data format you want, and any type information if you want to use that later to read the data. i.e. there isn't anything arrow specific in plasma.

Yep, that's right. We aren't forcing anyone to use any particular metadata representation. So you're free to define a metadata prefix that uses libndtypes, or anything you like

I am closing this for now, since I moved the plasma integration to it's own repo. https://github.com/plures/pxnd If I hit any roadblocks there that I think need to be addressed on the Arrow side, I will open an issue on that repo and reach out on the mailing list.

My two cents: I suspect that a more monorepo-like approach will make your life much easier rather than having to maintain interconnected C APIs amongst various Python packages with C libraries / C extensions. You can also make changes to multiple components at once without causing your CI setup to fall apart (i.e. if you have lots of repos, you may end up with multiple PRs related to a single change, each of which may cause the CI for any of the repos in the stack to break temporarily)

This is true. It worked because I was really the only developer up to now :)
I would like to stabilize gumath a bit -- currently it's sometimes convenient to change ndtypes/xnd and worry later about gumath.  But in a couple of months I think we can have something like a monorepo.

Also, if we want to add __truediv__ etc. to xnd objects instead of just libgumath functions like truediv(), we have a circular relationship between xnd/gumath.

Hey Stefan,
Why don't we move the plasma C bindings into the arrow repo as a first step
now? I'm happy to help with setting up the build system. They might also be
useful for other people!
-- Philipp.
On Fri, Mar 16, 2018 at 10:27 AM, Stefan Krah<EMAIL_ADDRESS>wrote:

Also, if we want to add truediv etc. to xnd objects instead of just
libgumath functions like truediv(), we have a circular relationship
between xnd/gumath.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/plures/xnd/issues/4#issuecomment-373786549, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAG6pMQq7TZtWjBxuXi-1LqUHAOf_fStks5te_XsgaJpZM4Scd2O
.

@pcmoritz Do you mean the part that is independent from xnd?  I think that is a good idea, it would be very nice to have general plasma C bindings.

Yeah, I mean this part here: https://github.com/plures/pxnd/tree/master/
libplasma
We can put it here: https://github.com/apache/arrow/tree/master/cpp/src/
plasma (for example as libplasma.h)
Feel free to create a PR and ping me if you need help with CMake.
On Fri, Mar 16, 2018 at 10:58 AM, Stefan Krah<EMAIL_ADDRESS>wrote:

@pcmoritz https://github.com/pcmoritz Do you mean the part that is
independent from xnd? I think that is a good idea, it would be very nice to
have general plasma C bindings.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/plures/xnd/issues/4#issuecomment-373795803, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAG6pN_s6EiH6vgMOKqBRFEVKI_Z0E9Eks5te_1CgaJpZM4Scd2O
.

@wesm Yeah I am finding I might need to make a few changes to xnd as I am developing pxnd, so a monorepo might make sense eventually. It is nice to have the build separate so that xnd doesn't have to depend on Plasma and since authorship is so far split between them.
@pcmoritz Sure, I can do that soon. Once I am able to test that the approach I have of wrapping things is working out and usable, probably next week or the week after.


Add DA and ptvsd experiments support to remote debugging API

For #7549

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
[x] Title summarizes what is changing
[x] Has a news entry file (remember to thank yourself!)
[x] Appropriate comments and documentation strings in the code
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated
[ ] Test plan is updated as appropriate
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed)
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #7570 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #7570      +/-   ##
==========================================
+ Coverage   58.88%   58.89%   +<.01%
==========================================
  Files         492      492
  Lines       21851    21856       +5
  Branches     3503     3505       +2
==========================================
+ Hits        12868    12873       +5
  Misses       8182     8182
  Partials      801      801

Impacted Files
Coverage Δ

src/client/debugger/extension/types.ts
100% <ø> (ø)
:arrow_up:

src/client/api.ts
92.3% <100%> (+4.8%)
:arrow_up:

Continue to review full report at Codecov.

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


Add Universitas Bani Saleh

adding Universitas Bani Saleh / Bani Saleh University
https://ubs.ac.id/

@azharyp Thank you for your patience while waiting for a review and processing your request. Pull request merged.


I would like to add my project RxStarscream to RxSwiftCommunity

A lightweight extension to Starscream to track WebSocket events using RxSwift observables.
For more details:
https://github.com/GuyKahlon/RxStarscream.
Thanks,
Guy

Very cool! Love the logo. Are you looking to only have it live on http://community.rxswift.org or would you like to migrate it to the organization? Happy to do whatever I can to help.

Hey @ashfurrow I'm glad to hear you like it (and the logo :)).
I would like to migrate it to the organization, I would like also to join into the organization, I have a few more Rx stuff to share with you.
Thanks for the help.

Awesome! I've invited you to the org, so you should be set. Our contributor guidelines are here, let me know if you have any questions.
Here's an example of when someone has done this before: https://github.com/RxSwiftCommunity/contributors/issues/11 Let me know if I can help 👍

Hey @ashfurrow, thank you for the invitation 😃
Looks like I've made all required steps and RxStarscream is now part of the RxSwiftCommunity.
If, you see that I missed some, please let me know and I'll fix it.

Cool, looks good! Should probably be added to https://github.com/RxSwiftCommunity/rxswiftcommunity.github.io . We should think of how to add unit tests and continuous integration, want to open an issue on the repo?

I've added the project to https://github.com/RxSwiftCommunity/rxswiftcommunity.github.io under Projects that Extend Other Projects with RxSwift
I've opened issues for 'Add Unit Tests' and 'Add CI' on the repo, I'm starting to think who to add Unit Test (if you have any thoughts, how to do that please let me know), and after that I'll add a CI.

Cool, I'm going to close this issue since it looks wrapped up. Feel free to re-open.


fix: intent selected & description add scrollbar

Description
intent multiple select bug

Before

After

Add scroll bar for skill detail dom

Task Item
#minor
Screenshots

Coverage decreased (-0.006%) to 55.759% when pulling ac213cc9a5bd6834f89c78dab01c032020e23a93 on julong/skill-intent-selected into cb5a8785240e52a1838b701ada9777dcbf1e4a67 on main.

Question: @luhan2017 Should the second screen disable the next button until at least one intent is selected?

Question: @luhan2017 Should the second screen disable the next button until at least one intent is selected?

In theory, this step is just to fetch the utterance of the selected intents as trigger phrases for skills. If user don't select any intent, they are also able to customize their own trigger phrases for skills.  @vivekkshankar, how do you think?


Update scalafmt-core to 3.8.2

About this PR
📦 Updates org.scalameta:scalafmt-core from 3.8.1 to 3.8.2
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "scalafmt-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "scalafmt-core" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #33.


Allow CertificatePinner.java to be extended

It would be nice to be able to extend CertificatePinner.java to, for example, use sha2 instead of sha1 algorithm. In order to do this it would suffice deleting the word 'final' from the definition of this class. A better alternative would be to parametrize the class allowing to pass the hashing algorithm as an argument.
Is there any reason for this class being final?

We're going to do this natively with HPKP.
https://github.com/square/okhttp/issues/1567
If you'd like to do your own trust mechanism, you can. See this example:
https://github.com/square/okhttp/blob/master/samples/guide/src/main/java/com/squareup/okhttp/recipes/CustomTrust.java

The certificate pinner isn't designed for inheritance. I'd much rather implement HPKP (or similar) for everyone instead of each user doing their own subclasses.


Any plan to support hash router?

Wonder how can i use hash router like https://reactrouterdotcom.fly.dev/docs/en/v6/api#hashrouter? Or any plan to support it?

We support hash routing today though the source property on the <Router> component.
import { Router, hashIntegration } from 'solid-app-router'

<Router source={hashIntegration()}><App></Router>

Note there is an open issues around it at the moment #89

You just saved my life.
Any plans to add it to the docs? :-)


7119 Add exception to report download error after a 200 response

We've seen some scenarios in which the HTTP code may be 200 OK but then the download seems to be interrupted (for example caused by a connection issue).
In a previous PR we added a Logger trace in this scenario in order to be able to retrieve more data when this happens, but still it as reported to the user as a Download Error (typically related with 4XX/5XX codes). This new exception takes into account the successful code and returns a "temporal issue, try again" message so that the user can know that the file is available and the URL is okay but there was a problem in the download.
Closes https://github.com/CartoDB/cartodb/issues/7119

@ethervoid could you take a look here too? You can find the history of it in the related issues, but basically this is the code in Typhoeus that marks the response as success:
def success?
  (mock || return_code == :ok) && response_code && response_code >= 200 && response_code < 300
end

It seems that for some scenarios, even though the response_code is a 200, we're seeing that success? is false, probably due to issues in the connection itself. We're adding this new exception and restricting more the scope of the logger trace I put in a previous PR so that at least when this happens the user can know that this is kind of a "temporal issue" and the URL that they're trying to use is valid, but yet something happened there. With this restricted logger I expect to be able to get more accurate traces that could let us know what is really happening behind.

retest this please

@ethervoid I applied what we talked about, could you check? :)

LGTM


Add validation components to cart/checkout, add redemption

What

Added information about applied promotions, coupons and grand price to cart-drawer-summary, order-summary

Added information about all discounts to bag-summary-mobile (mobile version of checkout)

Created a new commerce-wrapper for create-order so that the order has information about discounts and reduced price

If it exists, send to Stripe grandPrice, then use totalPrice

Fixed some typos in functions/files naming

Added stackable redemption to checkout

Prevent values from being reset when an invalid coupon has been used, do not allow adding an invalid coupon to cart

In progress
(another branch: wk/commerce-wrapper-unit-discount)

Add unit type discounts (add free product to cart)

Next steps

Use Qualification API
try to fix the errors that appear in the PR checks pnpm jest --ci --coverage --coverageReporters json-summary
Improve error handling
Find out how to use Storybook  (whether we should have access to this site - https://storybook.composable.com)

parts of the documentation that mention Storybook are commented out
the readme of the storybook says: "Coming soon"
following the steps of the Storybook instruction, there is no way to run it
we may need to contact support about this

refactor as much as possible, use more appropriate names, create more directories etc.
if agreed, consider how to separate the ComposableUI frontend and components typical for Voucherify

Bugs

When you try to add the same coupon for the second time you will get a runtime error

Cart Drawer

added grey stacks for promotions and coupon codes (additional appearance changes may still be required, for example the spacing in the coupon codes, appearance of the header and warning message)
fixed the spacing, padding, the order total, all discounts and grand total are written in a smaller font to match the text above, we can decide together which font size is better
I did not changed the order to be Subtotal -> Promotions -> Coupons -> Order Total -> All Discounts -> Grand Total because in my opinion this version looks better but we can discuss it

I'm planning to change the warning a little and make it disappear after 2 seconds
fixed the order
mobile view:

i'm not sure what would be better to display below the "My Cart", whether the order total or grand total 🤔

Checkout page:

Mobile:

the grand total is displayed at the top

order summary:

also, I removed this button in the mobile view:


Support for FileReader

https://developer.mozilla.org/en/docs/Web/API/FileReader

Also should include File and FileList (and if it isn't already done, FormData).
These are to aid building a drag/drop file upload component.

The FileReader class is part of a Working Draft spec and as a general rule we wouldn't add this API directly in Bridge.
Probably better as Bridge.Html5.FileReader NuGet package.
The following conversation discusses a similar scenario: https://github.com/bridgedotnet/Bridge/issues/1728#issuecomment-245806403

Turns out we don't need this for what we wanted (file upload), it's only for reading the files locally in the browser. So there's no demand for this from us for now! (@Porl91 is a colleague).

Hey @DanTup, They added ContentType Any<string, bool> to AjaxSettings, To support File Upload via Ajax.

Try this.


Implement a dict of params to prevent re-computation

This enhancement has multiple objectives:

Collect and save the parameters used to run a particular analysis
Use previously computed parameters to avoid re-computation (unless forced)

We would address this in run_pipeline.py here:
https://github.com/matiasandina/phdutils/blob/ce5b826c32649b735cd5a99647f4d815361493e2/ephys/continuous/server/run_pipeline.py#L41

An open list of parameters that need to be saved:

hash
last modif date


How to clear the saved ssid？

i want to clear the saved ssid and connect a new ssid.

My workaround for this is to uninstall and reinstall the app. It clears all ssids saved by the app.


Add default pre-commit hooks

Short description

Enable a few default pre-commit hooks which sanitize the general file format and try to rule out some common mistakes

Looks good, but maybe it makes sense to wait for the isort and djlint PRs, so they can directly be added.

Since #32 and #28 both have conflicts with the base branch right now, I guess it's fine to merge this one now...


Don't run the bats multilisten-snapshot test with valgrind

valgrind needs the use of the km --putenv= parameter and snapshots do not allow adding variables to the payload's environment so the multilisten test was failing with this error:
09:15:39.636920 main                 636  memchec cannot set new environment when resuming a snapshot

Superseded by https://github.com/kontainapp/km/pull/1723


⚠ Bofrost.be has degraded performance

In 9291469, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1938 ms

Resolved: Bofrost.be performance has improved in 9c61a14.


Inline code blocks are not rendered correctly in a table on page "Style Guide"

https://kubernetes.io/docs/contribute/style/style-guide/#use-code-style-for-filenames-directories-and-paths
It should render inline code block style like this envars.yaml. Seem an issue related to the Docsy theme.

The way inline code blocks are currently displayed within tables is related to a particular CSS rule that is part of the website's assets. Specifically, there is a CSS rule in place with "background-color: inherit!important; " that overrides the background color of the element and inherits the color from its parent element.

/area web-development

I would be inclined to keep things as they are because changing the CSS rule might have widespread impact. One specific example that comes to mind is the Feature Gate table. It's important to ensure that any adjustments to the CSS do not unintentionally affect other parts of the website.
I will let other maintainers/leads share their thoughts on this.

/sig docs
/kind bug

/triage accepted

I can work on this
/assign

/remove-triage accepted
/priority awaiting-more-evidence
To me, there is nothing to fix.


[FEATURE]How to finetune CPM2.1？

I am not familiar with int8. But i suppose it can not be trained like other fp32 models? Any suggestion about how to finetune it?
And does cpm2.1 has any report or paper? I did not find it anywhere.
Thank you!

Int8 model cannot be trained like a float model. BMInf only supports prompt tuning which means parameter of the model is not trainable.

Thank you!


chore(deps): update dependency @types/node to v17

This PR contains the following updates:

Package
Type
Update
Change

@types/node
devDependencies
major
16.6.1 -> 17.0.14

Configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box.

This PR has been generated by Renovate Bot.

Renovate Ignore Notification
As this PR has been closed unmerged, Renovate will ignore this upgrade and you will not receive PRs for any future 17.x releases. However, if you upgrade to 17.x manually then Renovate will then reenable updates for minor and patch updates automatically.
If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.


Multiple tfvars files for each env.

Summary
Support separating tfvars file in each env.
Motivation
Sometimes, there are enormous lines in a tfvars file. it's quite difficult to read.
So, this feature should be structured like this.
|--app
    |--modules
    |--stacks
       |--stack1
           |--tfvars
               |--base
                  a.tfvars
                  b.tfvars
               |--dev
                 a.tfvars
                 b.tfvars

Not sure if this used to work or is related to this issue Issue 267:Environment folders ignored

Not sure if this used to work or is related to this issue Issue 267:Environment folders ignored

Yes, sometimes, I have many modules in a stack. It would be cleaner if terraspace can read multiple file in a env layer.


document how to ignore popular dev directories with rclone

node_modules/
.git/

context: https://github.com/cdr/code-server/discussions/3090


Reuse validator: signature cannot be determined

Bug
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.8.2
            pydantic compiled: True
                 install path: /home/.../.venv/lib/python3.9/site-packages/pydantic
               python version: 3.9.7 (default, Sep  9 2021, 23:20:13)  [GCC 9.3.0]
                     platform: Linux-5.11.0-40-generic-x86_64-with-glibc2.31
     optional deps. installed: ['typing-extensions']
...

Hi all,
I have to validate one particular field in many classes. As encouraged in the docs, I have extracted this into a help function, which works perfectly.
The bug (?) I'd like to report concerns the signature of the validator, which cannot be determined by inspect.signature.
If you are wondering, the signature is read by sphinx autodoc in the process of generating the API documentation.
Here's a MWE showing the problem:
import inspect

from pydantic import validator

@validator('field')
def check_function(value: int) -> int:
    return value

inspect.signature(check_function)

This snippet returns:
Traceback (most recent call last):
  File "<input>", line 10, in <module>
  File "/usr/lib/python3.9/inspect.py", line 3111, in signature
    return Signature.from_callable(obj, follow_wrapped=follow_wrapped)
  File "/usr/lib/python3.9/inspect.py", line 2860, in from_callable
    return _signature_from_callable(obj, sigcls=cls,
  File "/usr/lib/python3.9/inspect.py", line 2259, in _signature_from_callable
    raise TypeError('{!r} is not a callable object'.format(obj))
TypeError: <classmethod object at 0x7f8821541730> is not a callable object

This is because validator calls classmethod on check_function to so it can be used as a class method.
You can get the same with
import inspect

@classmethod
def check_function(value: int) -> int:
    return value

inspect.signature(check_function)

I don't see way for us to fix this without fundamentally changing how pydantic works.


Bump bom version

As title to bump bom version.

Please help update this version https://github.com/saragluna/azure-sdk-for-java/blob/xiada/bump-bom-version/eng/versioning/version_client.txt#L182


Feature: add togglable qualifications to volunteer profiles

Admins need to be able to prevent unqualified volunteers for registering for certain roles. They should be able to manage whether a user is considered qualified or unqualified for these roles through the user management UI. The business logic of the application should prevent users from registering for those roles without said qualifications. The UI needs to be updated to represent this information about volunteers to administrators.

This has been implemented in the current build


Simplify the setup process regarding python dependencies and schematic config

Dependent on PR:

https://github.com/Sage-Bionetworks/schematic/pull/588

What are updated in this PR:

If poetry install is fixed, we don't need requirements.txt any more, which will simplify setup process a bit
Move configuration file of schematic to the root of this repo, in order to make local and deployment more compatible. Users only need to modify the schematic_config.yml and do not need to config.yml file in schematic repo. In addtion, make schematic_config.yml file of schematic more general and use example.model.jsonld by default
Update workflow

remove codes about requirements.txt
deploy to prod by tags names as semantic versioning, instead of by release* branch; deploy to staging by main and develop branches

Improve the script to automatically add GH secrets

Since we have reviewed in CR, merge it to develop and test with ncihtan's repo


Make OpenCage schema a little more robust

OpenCage seems to switch between String and i32 on certain fields. This change allows us to cope with that.

LGTM!

bors r+


CodeQwen1.5 not working

When trying to use CodeQwen1.5 7b awq (Tried fp16 directly from qwen and other awq versions as well but same error.), I am getting the following errors:
--max-input-length 4096 --max-total-tokens 9000 --model-id TechxGenus/CodeQwen1.5-7B-AWQ --max-batch-prefill-tokens 4096 --num-shard 4 --quantize awq --cuda-memory-fraction 0.6
2024-04-21T23:38:47.952625Z ERROR text_generation_launcher: Error when initializing model
Traceback (most recent call last):
  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 311, in __call__
    return get_command(self)(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 778, in main
    return _main(
  File "/opt/conda/lib/python3.10/site-packages/typer/core.py", line 216, in _main
    rv = self.invoke(ctx)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/opt/conda/lib/python3.10/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/opt/conda/lib/python3.10/site-packages/typer/main.py", line 683, in wrapper
    return callback(**use_params)  # type: ignore
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py", line 90, in serve
    server.serve(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 240, in serve
    asyncio.run(
  File "/opt/conda/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 636, in run_until_complete
    self.run_forever()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 603, in run_forever
    self._run_once()
  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 1909, in _run_once
    handle._run()
  File "/opt/conda/lib/python3.10/asyncio/events.py", line 80, in _run
    self._context.run(self._callback, *self._args)
> File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 201, in serve_inner
    model = get_model(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/__init__.py", line 532, in get_model
    return FlashQwen2(
  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_qwen2.py", line 45, in __init__
    tokenizer = Qwen2Tokenizer.from_pretrained(
  File "/opt/conda/lib/python3.10/site-packages/transformers/tokenization_utils_base.py", line 2089, in from_pretrained
    return cls._from_pretrained(
  File "/opt/conda/lib/python3.10/site-packages/transformers/tokenization_utils_base.py", line 2311, in _from_pretrained
    tokenizer = cls(*init_inputs, **init_kwargs)
  File "/opt/conda/lib/python3.10/site-packages/transformers/models/qwen2/tokenization_qwen2.py", line 172, in __init__
    with open(vocab_file, encoding="utf-8") as vocab_handle:
TypeError: expected str, bytes or os.PathLike object, not NoneType

@Narsil Any insights to this that can help me run this model? Not sure what i am missing.

Found the issue. Inside flash_qwen2.py, the tokenizer being used is not working. Switched the config and tokenizer to this:
            model_id,
            revision=revision,
            padding_side="left",
            truncation_side="left",
            trust_remote_code=trust_remote_code,
        )

        config = AutoConfig.from_pretrained(
            model_id, revision=revision, trust_remote_code=trust_remote_code
        )```

Now able to use the model as expected.

To further this, the tokenizer config path is not passed into the models. This make a custom config impossible

finish it by change config.json & tokenizer.json
https://huggingface.co/Qwen/CodeQwen1.5-7B-Chat/commit/91ffe86a74d00f76a75371d58a70ae5fe1bc0f29

To further this, the tokenizer config path is not passed into the models. This make a custom config impossible

https://github.com/huggingface/text-generation-inference/issues/1785#issuecomment-2131148602

Still unable to get this working.
2024-07-15T21:09:45.769744Z ERROR warmup{max_input_length=4096 max_prefill_tokens=5000 max_total_tokens=7000 max_batch_size=None}:warmup: text_generation_client: router/client/src/lib.rs:46: Server error: clamp() received an invalid combination of arguments - got (Seqlen, max=Tensor), but expected one of:

(Tensor input, Tensor min, Tensor max, *, Tensor out)
(Tensor input, Number min, Number max, *, Tensor out)

Error: WebServer(Warmup(Generation("clamp() received an invalid combination of arguments - got (Seqlen, max=Tensor), but expected one of:\n * (Tensor input, Tensor min, Tensor max, *, Tensor out)\n * (Tensor input, Number min, Number max, *, Tensor out)\n")))
2024-07-15T21:09:45.852870Z ERROR text_generation_launcher: Webserver Crashed
2024-07-15T21:09:45.852896Z  INFO text_generation_launcher: Shutting down shards


SearchBar is not displayed on iOS8

We are only using one service and not setting the initialSearchTerm.
Debugging looks like the height of the searchBar is set to 0.
Calling [searchBar sizeToFit] during the DZNPhotoDisplayViewController setup code seems to resolve the problem. Also, obviously just setting the height of the searchbar also resolves the problem. I'm currently working to understand why the two versions of iOS behave differently. But for now I'll probably just use the sizeToFit option.

+1, important issue for iOS8 support, referenced PR #118 fixes it. Additionally, for completeness and/or diagnosing:

the bug does not occur on cocoapods version, but on tip of master branch. It was caused by commit 417f09f98c9d1002d07dc6d694e1a19f029a0dc6.
the search bar is also hidden when setting the initialSearchTerm.


community: add request_timeout and max_retries to BaseCohere

This PR make request_timeout and max_retries configurable for Cohere.

Thanks for accepting my suggestion :) Looks good to me!

Opened #19641 to port this changes to the newer package langchain_cohere.


HDDS-4628: min/max election timeout of SCMRatisServer is not set properly.

What changes were proposed in this pull request?
min/max election timeout of SCMRatisServer is not properly set.
Previously, it is [ratisRequestTimeout - 1s, ratisRequestTimeout + 1s]
It should be  [ratisLeaderElectionTimeout, ratisLeaderElectionTimeout + 0.2s]
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4628
How was this patch tested?
CI

+1


Fixed #95. Updated Footer links. Added social icons to menu footer.

What type of PR is this?
/kind enhancement
What this PR does / why we need it:
This PR does the follwing:

Fixes #95 : Currently the children of menu items are all displayed in the sidebar menu. This PR shows only the parent in the sidebar menu and displays the children only when the parent menu item is active.
Added SODA Foundation social icons to the sidebar menu footer.
Fixed the GitHub links for Download, Star and Fork from opensds to sodafoundation.

Which issue(s) this PR fixes:
Fixes #95
Test Report Added?:
/kind TESTED
Test Report:
Please check the Deploy preview.
Top level menu items only

Show children only when parent is active.

Footer Social Icons

Special notes for your reviewer:

LGTM


SDO Specification Version/Release Support

Problem description
As with any other standards body defined specifications 3GPP Technical Specifications evolve over time.  It is advised to provide a release (at least a major release) for the 3GPP TS References to be provided for any of the artifacts produced by Commonalities.
Possible evolution
Provide the earliest release of 3GPP that should be used by reader.  Artifacts should have an overarching statement such as 'any 3GPP TS references must be considered Release X unless otherwise specified.  For example, if X is 16, then any feature introduced in Release 17 and onwards that is referenced by the artifacts, must be explicitly specified.
Alternative solution
n/a
Additional context
If there are other specifications by other SDOs, or projects, similar statements will provide clarity to readers.

In terms of Design Guidelines should there be a section added to Section 5 (Versioning), e.g., Section 5.3 SDO Specification Versions/Release that addresses this specifically both in Design Guidelines and any API documentation?

@gmuratk For CloudEvents  v1.0.3 is work in progress, v1.0.2 is the latest release .

3GPP themselves have the concept of "specific" and "non-specific" references to other 3GPP documents, where a non-specific reference is implicitly assumed to be the latest version. I think any references to 3GPP documents in CAMARA documents similarly do not need to be version specific.
But I agree that CAMARA should define a "baseline" 3GPP release for reference which includes all the 3GPP definitions that we want to capture. Whoever added (or adds) a 3GPP document reference should review the information they were referring to to ensure that the agreed baseline release includes this.
In practice, I don't think CAMARA will need to refer to 3GPP documents for normative reasons very often, and any information referred to should be stable.

@gmuratk Could you provide the review where the references to SDO specs are used (beside UE-Identification.md)?

@gmuratk Could you provide the review where the references to SDO specs are used (beside UE-Identification.md)?

Per my action item: (a quick search over 'camaraproject') besides the UE Identification here are the other places where references to 3GPP specifications are used:

Provider Implementations reference 3GPP Specs
QoD API references for attribute description (targetMinUpstreamRate)
EdgeCloud TrafficInfluence API Documentation has a reference

As there are still some references in API documentations, I propose to add the following in section 2.3:
“In aspiration of section 2.5, all CAMARA artifacts and API documentations should reduce telco-specific terminology. However, if there is identified benefit to reference another SDOs specification for a property or feature description, earliest version of the SDO artifact must be provided.”


Tableview hiding below keyboard in iPhone 11

This is the result for iPhone 11, where the tableview is hiding below the keyboard:

while the results are as expected for iPhone 6s plus:

As a workaround, I used
state.tableBottomMargin = 333

but the tableview should automatically find its layout.


Collapse > Accordion open first element/parent

Is your feature request related to a problem? Please describe.
I have used Vuesax in most of my projects and I must say, it's really good.
The reason I'm writing this one is to have a feature request on Collapse > Accordion
Is it possible if we can add a pre-open/open state of collapse/accordion on first/specified item or parent item.
Please let me know what you think.
Thanks!

Ready and a new property is added for the items in the components
the property is (open)
It would be something like this
<vs-collapse>
     <vs-collapse-item open > // <-----

         Collapse item

       Lorem ipsum dolor sit amet, consectetur adipiscing elit. Quisque rhoncus eros tortor, non fringilla lectus cursus et. Fusce vel nisi ante. Aliquam sit amet lectus pharetra, luctus mi sed, aliquet felis. Mauris a tortor viverra, ornare tellus in, consectetur leo.

        Etiam nec nunc nec nisl luctus tincidunt efficitur vitae elit. Vestibulum iaculis nibh commodo neque ultrices lobortis. Cras magna massa, pretium vitae mattis varius, pharetra nec massa. Aliquam ac ex enim. Quisque consequat dui libero, vel blandit lorem porttitor sit amet. Vestibulum ante ipsum primis in faucibus orci luctus et ultrices posuere cubilia Curae; Nullam sed lobortis nisl, quis eleifend metus.
     </vs-collapse-item>
</vs-collapse>

Thank you very much for the contribution

Thank you so much. I appreciate it. 👍


Filter equality matchers on the dynamo side

#400

What's the difference between this & #400?

We cannot filter the value using KeyConditions. I'm currently investigating how to filter by value.

I think you need one of these: http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Query.html#Query.FilterExpression