Text change on Initial Cloud Backup screen.

Choose any password.  Manage from Security Centre

....storage. It ensures you can easily....

verified this issue on dev app v1.8.0(321)


Help and Analytics is not wired

[ ] Help is not wired
[ ] Analytics is not wired

May need to wait until v3 UI designs have been shared with us from the Design team (The Last Word/Joys Chokatte).

Please remove above two buttons

I can work on removing buttons.

Hi @jayasanka-sack ,
I am hoping to remove the below. Please confirm.

Thank you,
Dimuthu

Hi @muthulee, please go ahead! 😊

Thank you :) ... Will do.


Project Setup

In this PR I :

Create a React application.
Setup linters for the project.

Hi @kensteph,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request
Kindly modify your readme doc using the template
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @DammyShittu , Thank you


createRoutines?

I think it would be nice to have createRoutines method, like createActions in 'redux-actions', to create routines in batch, and not one by one

@victordidenko good idea! PR are welcome :)

Made an PR #38, first approach (don't like very much), to hear your opinion

Work in progress
https://github.com/shapkarin/redux-saga-routines/blob/createRoutines/src/createRoutines.js
https://github.com/shapkarin/redux-saga-routines/blob/createRoutines/test/createRoutines.js
Hope soon I will create PR 🚀

Proposals for nested example:
const routines = createRoutines({ app: { people: { search: name => ({ name }) }} });
routines.app.people.search('Ada');
// { type: 'routines/app/people/search', payload: { name: 'Ada' } }

nesting not ready yet 🤔
To overwrite the default payloadCreator:
const routines = createRoutines({
  app: {
    people: {
      search: {
        trigger: ({ phone }) => parseInt(phone, 2)
       },
      add: null // maybe as well or something other to say `default`
    }
  }
 });
routines.app.people.search({ name: 'Ada', phone: 010010 });
// { type: 'routines/app/people/search/TRIGGER', payload: { name: 'Ada', phone: 010010 } }

Proposals for nested example:
const routines = createRoutines({ people: { search: name => ({ name }) } });
routines.people.search('Ada');
// { type: 'routines/people/search/TRIGGER', payload: { name: 'Ada' } }

nesting not ready yet 🤔
To overwrite the default payloadCreator:
const routines = createRoutines({
  people: {
    search: {
      trigger: ({ phone }) => parseInt(phone, 2)
     },
    add: null // maybe as well or something other to say `default`,
    remove: null
  }
 });
routines.people.search({ name: 'Ada', phone: 010010 });
// { type: 'routines/people/search/TRIGGER',  payload: 42 }

also it will be great if we can use that somehow with creating users custom routines with createRoutineCreator. Maybe we can just add additional stage with underscore:
const routines = createRoutines({
  people: {
    search: {
      more_info: {
        _OPEN: null,
        _HIDE: null,
     }
    }
  }
});
routines.people.search.moreInfo.open({ id: 42 });
// { type: 'routines/people/search/OPEN',  payload: { id: 42 } }
routines.people.search.moreInfo.hide({ id: 42 });
// { type: 'routines/people/search/HIDE',  payload: { id: 42 } }

Proposals
Nested example:
const routines = createRoutines({ people: { search: name => ({ name }) } });
routines.people.search('Ada');
// { type: 'routines/people/search/TRIGGER', payload: { name: 'Ada' } }

We can access to the stage with underscore prefix and for example to overwrite the default payloadCreator:
const routines = createRoutines({
  people: {
    search: {
      _TRIGGER: ({ phone }) => parseInt(phone, 2)
     },
    add: null // maybe as well or something other to say `default`,
    remove: null
  }
 });
routines.people.search({ name: 'Ada', phone: 010010 });
// { type: 'routines/people/search/TRIGGER',  payload: 42 }

Or add custom routines with createRoutineCreator:
const routines = createRoutines({
  people: {
    search: {
      more_info: {
        _OPEN: null,
        _HIDE: null,
     }
    }
  }
});
routines.people.search.moreInfo.open({ id: 42 });
// { type: 'routines/people/search/OPEN',  payload: { id: 42 } }
routines.people.search.moreInfo.hide({ id: 42 });
// { type: 'routines/people/search/HIDE',  payload: { id: 42 } }

And maybe even create extended routines with default stages and like in that package

I think to implement that. Not sure how long that it takes, hope I will provide some first version soon.

Not sure about underscore before the stage (for example _TRIGGER)
And now I realise that maybe better is to implement that in my package, because I will definitely need createCustomRoutine functionality inside createRoutines.

Today I will release the first version.

Here is createRoutines first version https://github.com/shapkarin/extend-saga-routines#create-routines


3 ERRORS are there after i put GoogleService-Info.plist in app and update the pod

please help me...

check the "readme"

i done exact like this but this error is occurs ... please help me

this error is occurs

Seems like the project files are missing, please download again or try to delete all files in derived data folder.

i already done this but again i do this and then see

done...thanks

i m alone in conversation please tell me how can i add people in this...


Use @ binding operator when matching errors -- very minor nit

I needed to specialize Deserialize to a specific SomeStruct<()> instead of SomeStruct<T>. To do so, I read the docs, quickly gave up, used cargo expand to get the generated Deserialize from #[derive(Deserialize)], and finally c+p the generated code into my project. Then, I proceeded to manually format the code.
I only included that since I thought it was amusing.
Anyways, in the process, I found a few examples of
_serde::export::Err(_err) => {
    return _serde::export::Err(_err);
}

I would like to replace all instances of this, and things similar to this with
__err @ _serde::export::Err(_) => {
    return _err;
}

Course, the might (or even already, idk) implement the Trytrait, but I think pattern binding (the@` symbol) has been around since the beginning, so no Rust version issues. This is also one of my favorite parts of Rust, so I put them anywhere I can fit them.
A link to the pattern binding issue for docs
I was wondering if this is a good idea or not, as well as how to.
Anyways, super minor nit, but I think it's useful.

The error return comes from serde's custom try! macro which improves compile times of generated code over ? by skipping the From::from call. If we wanted to change this to use @ pattern binding it would just be a change there. I don't really see any reason to prefer one way or the other, however. (In fact, the second branch could even be further simplified to just __err => return __err,.)

Your before/after snippets (copied below) are not equivalent because only the first allows the matched result and the returned result to have different Ok types. I am skeptical that changing the generated code to always require matched results and returned results to have identical Ok type would work in general.
_serde::export::Err(_err) => {
    return _serde::export::Err(_err);
}

__err @ _serde::export::Err(_) => {
    return _err;
}


Set up the infrastructure for class aliases

https://packagist.org/packages/typo3/class-alias-loader
This allows us to rename classes without breaking things.

This jogs a memory.  I have used class_alias to mitigate migrating to namespaces and PSR-4.  It was a godsend, IIRC.


generalize non-strict reducers

Well, this basically re-implements #990 from scratch again, but I think it makes sense. Several non-strict reducers (difference, ratio, first, and last) still need to be implemented: these will need to find the first defined and last defined values within the current window.

Ah, and another thing that we should change is that mode shouldn't coerce values to numbers.

I fixed the mode problem in the latest commit.


Identify and Create Issues for future DevOps Guides

Overview
We need to identify guides to make and create issues for them, so that they can get worked on, and we can refer to finished guides in issues (where the guidance is needed) and our wikis.
Action Items

identify guides that need to be created (making sure they don't already exist)

[ ] confirm they don't already exist by looking at the project boards where guides across multiple CoPs reside (resources 1.01 and 1.02)
[ ] if they don't exist, add them to the List of Guides to make below

create issues

[ ] Create issues for each of the guides in the knowlegebase repo (see resource 1.05), adding the label: CoP: Ops

link the guides so that they are easy to find

[ ] Link the guide issue back to this issue
[ ] Add the guide issue to the following project boards (resources 1.02 and 1.03)
[ ] Make sure that the guide issue is showing up on resource 1.04

Resources/Instructions

1.01 P: Guides: Tracker of issues in CoPs
1.02 P: Guides: Knowledgebase Content Tracker
1.03 CoP: DevOps: Project Board
1.04 Templates and Guides, DevOps tab - this is linked from our wiki
1.05 knowlegebase-content repo

Guide Making Templates

Create a New Guide - With Examples from Multiple Teams
Create a New Guide - With Prescriptive Instructions

List of guides to make

https://github.com/hackforla/knowledgebase-content/issues/70 - Adding a project to incubator
Create a Guide/Template: How to access logs to debug with your project team

see https://github.com/hackforla/ops/issues/92 for logs available

Create a Guide/Template: What to store in 1password (Secrets, ports tokens)
Create a Guide/Template: When to graduate your project from incubator
Setting up Dev & Production Environments

Create a Guide/Template: Setting up Dev & Production Environment - AWS
Create a Guide/Template: Setting up Dev & Production Environment - Azure
Create a Guide/Template: Setting up Dev & Production Environment - gh-pages

Create a Guide/Template: How to get added as a credentialed user on AWS
Create a Guide/Template: Using Terraform
Create a Guide/Template: Using Terragrunt (required for incubator)
Create a Guide/Template: Setup Docker

Prior version of issue

- Adding a project to incubator #20
- How to access logs to debug with your team
- What to store in 1password (Secrets, ports tokens)
- When to graduate your project from incubator
- Setting up Dev & Production Environments
   - AWS
   - Azure
   - gh-pages
- How to get added as a credentialed user on AWS
- Using Terraform
- Using Terragrunt (required for incubator)
- Setup Docker

Assignee, Labels, Project Board Placement, and Milestones for this issue in the [INSERT COP NAME] Repo:

Create a Guide/Template: Using Terragrunt (required for incubator) - is this guide still needed

See list of guides above...
Is this still required or can I delete this?  Create a Guide/Template: Using Terragrunt (required for incubator)

[ ] yes, still needed
[ ] no, delete it.

@ExperimentsInHonesty I'd say that the first bullet (list of guides to make / Hosting for your project) and the second section (list of guides to make - Unsure) are the same guide from a devops point of view.
We are planning to do a guide that shows people how to host things in incubator, and that will include setting up dev vs production environments.
Additionally with regards to "when to graduate your project from incubator", devops may not be in the best position to make that determination.


certificate: Adding GetSerialNumber() to Certificater interface

For some cert issuers, like Hashicorp Vault, the serial_number of the certificate is important and we need to keep track of that.
The serial_number is used, for instance, to revoke a given cert.
This PR adds GetSerialNumber() to the Certificater interface.  Other PRs will implement serial_number where necessary.
ref https://github.com/openservicemesh/osm/pull/2070 https://github.com/openservicemesh/osm/issues/507

Affected area:

New Functionality      [ ]
Documentation          [ ]
Install                [ ]
Control Plane          [ ]
CLI Tool               [ ]
Certificate Management [ ]
Networking             [ ]
Metrics                [ ]
SMI Policy             [ ]
Security               [X]
Tests                  [ ]
CI System              [ ]
Performance            [ ]
Other                  [ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? If so, did you notify the maintainers and provide attribution?

Codecov Report

Merging #2072 (b1603c1) into main (e0513d3) will decrease coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #2072      +/-   ##
==========================================
- Coverage   57.81%   57.73%   -0.08%
==========================================
  Files         140      140
  Lines        5781     5787       +6
==========================================
- Hits         3342     3341       -1
- Misses       2437     2444       +7
  Partials        2        2

Impacted Files
Coverage Δ

pkg/certificate/mock_certificate.go
0.00% <0.00%> (ø)

...g/certificate/providers/certmanager/certificate.go
50.00% <0.00%> (-10.00%)
:arrow_down:

pkg/certificate/providers/tresor/certificate.go
78.78% <0.00%> (-2.47%)
:arrow_down:

...certificate/providers/vault/certificate_manager.go
47.61% <0.00%> (-0.58%)
:arrow_down:

pkg/certificate/types.go
0.00% <ø> (ø)

pkg/catalog/proxy.go
100.00% <0.00%> (ø)

pkg/catalog/catalog.go
100.00% <0.00%> (ø)

pkg/catalog/debugger.go
97.22% <0.00%> (+0.07%)
: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 e0513d3...b1603c1. Read the comment docs.


【Hackathon No.88】实现 Taichi 和 PaddlePaddle 之间的无拷贝交互

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

Please sign our Contributor License Agreement before we can accept your contribution.

Please sign our Contributor License Agreement before we can accept your contribution.

Thank you for reminding me. I've signed it.


Setup PHP GD library on the project

Did you saw this error?

I followed all the instructions on the contributing file and still can't set up to the project completely. I can see the home page, login page and etc etc. But, when I try to run the tests, the UserTest is broken. Because it makes a call to a function of the PHP GD library and this library it's not on the project Docker image. We need to install this library and check for required PHP extensions with Composer.
I can fix it. Can you to tag it with hacktoberfest? :)

I’m working on this issue! 😊


response process is not ended - interactive message buttons wont work

res.end() is missing in the callback of /slack/receive route.
I receive timeout response when I try to hit /slack/receive route.

However, Adding res.end() return 200 status but "interactive_message_callback" event is not being triggered.

Any immediate solution to that or temp workaround? I can't get interactive message to work either: I double checked the interactive messages callback (from localtunnel.me) on Slack but it seem Slack can't connect. What's weird is that when I test it with something like curl -X POST interactive_messages_callback_url the endpoint seems to be accessible and available.


Feature Request: Assign a Scale Factor on a per asset basis

One of the great features in Flump is the ability to scale the resulting Texture assets, allowing content creators to optimize their video memory and file size usage.
Unfortunately, with a single scale applied to all assets in a project, it somewhat cripples this benefit.  It would be awesome to be able to tweak the scale on a per asset basis, because sometimes you can drop your scale by a tiny bit and then get that asset to fit into a smaller texture profile.
The individual scale would be stored in the flump project file and multiplied on top of the Base Scale and Additional Scale Factor to determine final scale for export.
Thanks!

+1 After implementing a quite complex art asset we realized that we cannot use scaling. (only at the price that textures will be blurry or they will take up huge space)

If it helps, I implemented a way to do this in my fork of the flump depot.  The way it works is you create a symbol in the same flash file as the asset you wish to scale.  Set up this symbol to export as a flump Movie called "scale".  In this symbol, on Layer 1, Frame 1, place any one of your flump Image symbols and apply a scale to it.  The exporter will read the scale you set on this symbol and apply it during export time to all of your textures.
If you would like to use this, merge: https://github.com/kpatelPro/flump/ branch "library-scale-proper" into your copy of the depot and rebuild your flump exporter.

Thanks for the tip, sounds like a good fix for my broken asset. I've been trying to fix it properly:
When constructing the texture atlas measure all instances of a given asset and export it with the biggest scale. Then go back to the movie descriptors and adjust the scaling of the instances to this biggest asset.   Sadly this seems to be a very difficult change,  it looks to me that I'd need to refactor most of the exporter logic to accomplish this.

@kpatelPro I've implemented real scaling, its in my fork: https://github.com/funkypandagame/flump/commit/85d6f48126018169c6c79614a74b209c4637e086
Now the exporter will export each asset at the biggest size used, thus nothing will be pixelated.
Note that I've removed .xfl support (I had to refactor the parsing logic quite a bit to understand it) and its likely broken for flipbooks.


New logo

Previous attempts to do a logo for Numba were unsuccessful.
@goanpeca: However, a new logo would give the "project a sense of identity" and can help the project become "more identifiable in the community"
@seibert: "the simplest idea to capture is 'Make Python go fast'." The previous attempt "focused on the code translation, which is the technique, rather than the goal, which is speed"

Also cc @jaseg - who had some initial suggestions on the last go-around :)

I would add a word to what @seibert said and is..

Make Easy for Python to go fast

Cause Numba is easier to use that, lets say, Cython

This 2013 blog post comparing Numba and Cython was very helpful to my understanding of Numba. The highest level overview might be, it makes Python programs for number crunching and scientific computing faster. https://jakevdp.github.io/blog/2013/06/15/numba-vs-cython-take-2/
Stan jokingly suggested a snake riding a rocket for the idea of "faster Python", and I jokingly suggested having it wear a lab coat to indicate scientific computing. Maybe we could have a snake riding downhill in a race car or a roller skate instead to indicate speed.
A more detailed explanation of Numba would be that programmers can use the 'autojit' function to automatically use just-in-time (jit) compilation to make their code much faster, and this requires very little time or effort from the programmers, as well.

So basically we need some kind of snake on steroids...

Dumping other open source logos for inspo and insight.

We need Quetzalcoatl :-)

Most importantly, @VictoriaODell suggested an emoji mockup of a logo:
🐍💨
I think this captures the idea nicely.  :)

A farting snake :laughing:!

@ahmadia helpfully mentioned his own 2013 blog posts on Numba, which help give an idea of what it does and how to use it. https://www.wakari.io/sharing/bundle/aron/numbapost https://www.wakari.io/sharing/bundle/aron/Accelerating_Python_Libraries_with_Numba_-_Part_2

Have you considered to launch a competition for the logo at 99designs? I recently had fairly well experience with this, and also other tech startups like Docker refered to their services. In the end, we're programmers, graphics are usually not our main competence...

We have in-house graphics design expertise to help us now (yay, @VictoriaODell!), so we'll be looking to create a logo over the summer.
(Speaking personally, I'm also not a big fan of art contests.)

Hey~
Recently had a conversation with Michele. I'll be meeting with a couple of different project leads that need logos or updates and we'll work on coming up with a detailed design brief. From there we will decide what the best course of action for execution are. :)

New logo is live on the website.  Thanks @VictoriaODell!

Congrats, looks good!


Use latest python-black to format the sources

Unpin the old version *19.x and use latest 24.x - which enables new checks and format options

Conflict war incoming ! :D


fix input[type=search] padding-left not in navbar

Please be more specific, what changes you made

I changed the .input-field input [type = search] padding-left to 0.
Added nav .nav-wrapper .input-field input [type = search] padding-left to 4rem.

But why? What is the result. What are you fixing or adding?

Because i was not using the inside of the navbar inpupt [type = search] and found it padding-left = 4rem.
This will be very unexpected on the screen, I would like to input [type = search] should still be used in the place is not navbar, so I modified it.
But I found some effect I did not modify to, for example, input [type = search]: focus no bottom line.
So the issue I will first close, so I completely deal with the entire effect and then send pull request. thank you for your patience.


build: simply docker installation and improve the manual one

So, I am looking at the install instructions now. We are basically asking people to clone the entire (fairly large) repo only to access the environment yml file and then pip install it from elsewhere. Given that we anyway have the installation commands copy/pastable, would it perhaps make sense to have a command to download just that one file instead? Or is it even possible to create the env from the online file?
Not sure if it's worthwhile changing, just noticed this and thought I'd mention it :)

Very good point! I changed them in https://github.com/DeepRank/deeprank2/pull/615/commits/d424a26d62dc21513297ac9ae2306fba4b4d7bfd, let me know if you like it better now :)

So, I am looking at the install instructions now. We are basically asking people to clone the entire (fairly large) repo only to access the environment yml file and then pip install it from elsewhere. Given that we anyway have the installation commands copy/pastable, would it perhaps make sense to have a command to download just that one file instead? Or is it even possible to create the env from the online file?
Not sure if it's worthwhile changing, just noticed this and thought I'd mention it :)

Very good point! I changed them in d424a26, let me know if you like it better now :)

Nice!
I guess the other thing is that if it wasn't cloned, the tests won't work, so the testing part should maybe be deleted or adjusted as well?

Nice! I guess the other thing is that if it wasn't cloned, the tests won't work, so the testing part should maybe be deleted or adjusted as well?

I removed the ref in that part, thanks :)

Also, requirements-docker.txt can be removed from the bumpversion config file.

Good catch, removed!


Code can't run

" all_node_history = tf.where(all_node_history > 0, all_node_history, 0.0)" in the code
Report an error：
ValueError: Shapes must be equal rank, but are 3 and 0 for 'Select' (op: 'Select') with input shapes: [?,207,2484], [?,207,2484], [].
May I ask your tensorflow version

I'm sorry I can't reproduce, all runs smooth on colab

Can you provide  FC_GAGA files of .py type?

@shuqincao Hello, I have introduced run.py, which one could run as python run.py inside a docker container to reproduce the results without relying on the colab environment. Please check the updated readme, section Docker based demo. Let me know if it solves your problem. Sorry it took some time to address your concern.

@shuqincao Hello, I have introduced run.py, which one could run as python run.py inside a docker container to reproduce the results without relying on the colab environment. Please check the updated readme, section Docker based demo. Let me know if it solves your problem. Sorry it took some time to address your concern.

Fixed


how to remove or supress theese warnings?

node build/dev-server.js

Starting dev server...
(node:57216) Warning: Accessing non-existent property 'cat' of module exports inside circular dependency
(Use node --trace-warnings ... to show where the warning was created)
(node:57216) Warning: Accessing non-existent property 'cd' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'chmod' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'cp' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'dirs' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'pushd' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'popd' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'echo' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'tempdir' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'pwd' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'exec' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'ls' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'find' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'grep' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'head' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'ln' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'mkdir' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'rm' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'mv' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'sed' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'set' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'sort' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'tail' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'test' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'to' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'toEnd' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'touch' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'uniq' of module exports inside circular dependency
(node:57216) Warning: Accessing non-existent property 'which' of module exports inside circular dependency

DONE  Compiled successfully in 29158ms

@minicloudsky it looks like you might have pasted over the issue template. Can you review the template and include that information here? Like @Etheryte pointed out, this should have been fixed by v0.8.4 (and the fix is included in v0.8.5). Let me know what ShellJS version you're using and I can take a closer look.


Update code checkout command

The repo is no longer private, a non-ssh git clone (https) would work.

Akash! Thanks so much for the contribution!  The code itself looks great to me.
One tiny nit . . . the title should begin with a short one word subject heading.  I.E. docs: Update code checkout command
I'd be happy to make this change myself before merging if you'd prefer.  Or feel free to resubmit the PR.
Thanks again Akash

Ah one more thing.  I actually do think that the recommend way to get the code is actually to use go get as suggested in the original documentation.  Cloning it manually should be more of a secondary option for advanced users.
The code that's there is somewhat unclear on this point though. I think perhaps the best thing to do is merge your patch, and then I'll do another patch on top of it that clarifies things a bit.

Made a slight tweak to the commit message and merged.


Autoplay stops on all but one swiper when the browser is resized

Check that this is really a bug

[X] I confirm

Reproduction link
https://codepen.io/zmcwilliam/pen/abRZzJp
Bug description
When multiple swipers have been initialised on the page and the window is resized by the user, only one of the swipers will continue autoplaying, the rest of the swipers will be set to a paused state.
They can be resumed with .autoplay.resume() successfully, but they should not need this in the first place as the swiper was never paused.
Expected Behavior
All swipers on the page continue to autoplay after the browser window has been resized
Actual Behavior
Only the most recently initialized swiper will continue to autoplay, and any other swiper will pause itself.
Swiper version
9.2.2
Platform/Target and Browser Versions
All (Tested on Windows 10 and Ubuntu 22.04.2 -> Chrome 111.0.x)
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

This is still happening for us @nolimits4web - see this page here as an example (the 3 sliders right below the welcome section): https://www.agent.so/ask/software-developer
Here's the config for that slider:
https://privatebin.net/?99969a763f733d3a#BiFr9bzaZFZ7c1Ag6dp8a8tdqF5c6dCqy12p6cED5BY2
Any quick workarounds on how to fix it? Tried a combination of resize event listeners, initialize/destroy the slider, etc. and nothing seems to work.
Additionally, after the slider completes a cycle it will just stop. Which is an ever bigger problem because people won't ever get to see the content outside the viewport once it does.


Associate Entity Clarification

I have an N:N relationship between the 'salesorderdetail' entity and a custom entity I create called 'Authorisation Environmental Conditions'.
I have found a plugin that allows me to query an entity and return a recordset and loop though the list of records and do something as a result. (see screenshotbelow).

Within the loop I am creating a  'Authorisation Environmental Conditions' record which I then want to assoicate with the salesorderdetail entity. It is at this point, within the loop, that I have to tried do the association using  'associate entity', but I am not longer in the primary entity and cant get it to work.

Will it be possible to use your 'Associate entity' within the loop or is it only possible on Primary entity that triggers the workflow? Do I only need to do it once or twice? One for the association of one entity and once for the association of the other?
I am using Dynamics 365 Version 1612 (<IP_ADDRESS>3) (DB <IP_ADDRESS>3)
Many thanks for your time in advance

Hi lynn-fisher, right now this is not possible.
kind regards,


where().first => find_by()

I know these are functionally equivalent, but I believe find_by() is slighly preferred :)

I think this does not work in Rails 3

Probably true, but Rails 3 is a dinosaur :)
http://stackoverflow.com/questions/27618185/when-does-rails-3-2-reach-end-of-life

For now we're supporting it though. I think many people are still using it. Once Rails 5 is released will probably be a good time to drop support.
Marco Otte-Witte
<EMAIL_ADDRESS>http://simplabs.com

Am 14.03.2016 um 22:01 schrieb Kalman Hazins<EMAIL_ADDRESS>Probably true, but Rails 3 is a dinosaur :)
http://stackoverflow.com/questions/27618185/when-does-rails-3-2-reach-end-of-life
—
Reply to this email directly or view it on GitHub.

Fair enough :)


Add beanstalkg to queues.io

PR here: https://github.com/lukaszx0/queues.io/pull/120
Queues.io: http://queues.io/
Hopefully this can get you more exposure.

great thank you very much @stephenwithav . I hope it will get accepted. I will find more time to spend on this project, Since I have a collaborator to help me :)
(I never wrote the next parts of my blog series about the queue, I will start writing it now)

It has been accepted, but the site hasn't been updated yet.
I look forward to the next parts of your series!


Make redis data format compatibility break less painfull

In #440 we change the format using to serialize registration and observation in RedisRegistrationStore.
We already warn that RedisRegistrationStore is just an experimental works, so this kind of break would happen again.
I propose this PR to limit the migration painfulness.
The idea is that registration and observation are kind of "session" data. So for the rare case where this break occurred. Just ignoring old key could be acceptable.
That's why I introduce key prefix for RedisRegistrationStore in 2nd commit.
The first one add "optional" expiration for all redis key, this avoid to remove keys manually.
I'm aware this is not so great, but I think this could help.

Any chance this PR will be merged soon? We are using RedisRegistrationStore and we are hesitate to move to 1.0.0-M5 due to this issue.
Can anyone please help explain the risk of moving from 1.0.0-M4 to 1.0.0-M5? We are using RedisRegistrationStore because we plan to scale-out to clustering DM nodes.

As this PR seems to not needed by the community. I close it.
Feel free to revive it, if needed ;)


Support prometheus-operator

Prometheus operator doesn't use annotations, but uses ServiceMonitors/PodMonitors instead. It would be nice to have first class support for it.

I think we need our own operator with own CRDs, we already mentioned this idea during internal discussions and I believe the time has come.
And that's going to be a separate component (not part of the server) as I see it (optionally, with its own repo and helm chart).

Not really, I am talking about having a flag such as serviceMonitor.enabled that generates a yaml like https://github.com/prometheus-operator/prometheus-operator/blob/9c0db5656f04e005de6a0413fd8eb8f11ec99757/example/thanos/prometheus-servicemonitor.yaml

Oh, I see - just to enable pyroscope server metrics scraping. Totally makes sense! What I was thinking about is kinda separate topic - perhaps we should revisit it again later.

Something simple like this should do the work (using templates of course):
apiVersion: monitoring.coreos.com/v1
kind: ServiceMonitor
metadata:
  name: pyroscope
  namespace: monitoring
  labels:
     app.kubernetes.io/instance: pyroscope
     app.kubernetes.io/name: pyroscope
spec:
  selector:
    matchLabels:
      app.kubernetes.io/instance: pyroscope
      app.kubernetes.io/name: pyroscope
  endpoints:
  - port: http

I may create a PR later.


TTS is not able to handle time in replies

For example, "The time is 8:01 AM.", it will speak "The time is am", where am is the literal word "am" as in "I am". The time component itself is entirely skipped, I think that is related to the number handling issue we saw before mentioned in #148 .

Yes, they are absolutely related. There's certainly some regex, etc we could develop that would pre-split that but to get the output that's common in American English speech would need to go from "8:01 AM" to "eight o one a m" is tricky. Additionally, we'd need to account for languages/geographies that don't use AM and PM, or represent them differently. Frankly I don't know a good way to do that off the top of my head...

Yeah agree, it is a tricky problem. It may also just be a limitation of the TTS in use. I know when I try this with Coqui or Piper they both handle numbers and these time values better (they still struggle with AM/PM, but they appear to say eight o' one and such with the syntax shown). But from a linguistic approach the regex or NLP for that would definitely get complex, probably require it's own library/module just to handle that 😮

Yeah this certainly brings back the issue regarding alternate TTS engines. The issue there is in the (ample) time I've spent with them I'm increasingly convinced they're just not suitable for Willow as-is. As noted before, the dependencies are VERY messy and the performance is lackluster (we'd still have caching, so there's that).
At this point TTS is our biggest fundamental issue and these issues just keep flowing in compared to the rest of Willow/WIS functionality. We certainly appreciate them, so keep them coming! My concern with coqui, etc is we'd likely just swap a number of issues with an equivalent set of different set issues. The open source models and frameworks for TTS generally seem to be very far behind their counterparts in the rest of the "model world" and none of them seem to meet our overall goal of providing a truly commercial quality voice user interface.
So, as frustrating as it is currently, SpeechT5 is likely (in the end) still our best option and it's clear that it will need significant pre-processing of text before it's provided to the processor itself. If you look at sentencepiece you can start to understand the fundamental challenges that all of the text models and architectures have...

If it's of any intrigue to you, for laughs I was able to get Willow to work with Coqui running as an independent server and I get pretty good response times. I did have to modify the Willow code to add DEFAULT_ESP_WAV_DECODER_CONFIG() as Coqui outputs wav, but otherwise works perfectly. Of course this isn't utilizing nginx cache, but still very performant. 😄

That's great feedback!
Generally my goal (considering the level of effort required) is not only clean dependency and code, but an actual performance improvement. Of course comparable performance is the absolute minimum.
VITS-based models are clearly (to me at least) the future and I've been working through Coqui and others to get VITS models working with ONNX, CUDA, and TRT (the hard one) which not only leads to easier packaging, dependency management, and cleaner distribution but higher performance as well.
In terms of the WAV decoder on device, we're really trying to make FLAC the standard in terms of audio decoding (ESP ADF doesn't currently support FLAC encoding) as opposed to building additional libraries, bloating the firmware image, and trying to do weird parsing of Content-Type and other conditionals to determine the decoder.

Yeah only reason I added WAV was because the coqui docker image API doesn't output in FLAC unfortunately. The model I am playing with is the one we discussed before (the VITS model trained with the Jenny dataset). Would agree if it can be exported to ONNX/CUDA/TRT it would probably be much faster and easier to implement. 😃

Sorry for hijacking this, but how do you get Willow to speak the response? The only thing I've managed to get it to speak is "ok".

Ah, I see this only works with Home Assistant, is that correct? There's no way to play audio with the REST server?

https://github.com/toverainc/willow/pull/225
On Sun, 20 Aug 2023, 10:59 Stavros Korokithakis, @.***>
wrote:

Ah, I see this only works with Home Assistant, is that correct? There's no
way to play audio with the REST server?
—
Reply to this email directly, view it on GitHub
https://github.com/toverainc/willow/issues/160#issuecomment-1685242948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBRBFEYAJJMYG4GWEASYF3XWHNWFANCNFSM6AAAAAAZARUNOY
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yeah speech t5 isn't great with numbers and such, so I'm split_arch branch we make coqui the new default, with xtts as another option for those who want to tinker. There's no notes on it yet as it's still a developmental branch, but if you take a look in the utils.sh there's a method added called build-xtts, and once you run that it tells you how to turn on xtts. Otherwise building and deploying from that branch will automatically use coqui.


Allow searching by country code

Along with searching by country name, It'd be really nice if this library allowed searching by a country code, so +1 should show United States and all countries starting with +1

It's already supported, sorry for my rush


UI Changes

Based on feedback from Sprint 1 demo, updates need to be made. This includes fixing naming of forms, such as spacing and capitalization. The header needs to be smaller.

@Goutham2591 @TarekRamesses @Arti012
I think we need to discuss this tonight in class. We need to find a way to deploy images, they are getting broken. I looked up online and found out, we need to store the image in an external source such as AWS. Heroku doesn't store image longer than 24 hours, since its a free or any other reason. Thanks.


Access to the path '...dgspec.json' is denied

Problem encountered on https://dotnet.microsoft.com/learn/web/aspnet-microservice-tutorial/docker-image
Operating System: windows
Error from the docker build step:

Step 9/14 : RUN dotnet build smhMicro.csproj -c Release -o /app

---> Running in 089f073fa544

Microsoft (R) Build Engine version 16.1.76+g14b0a930a7 for .NET Core

Copyright (C) Microsoft Corporation. All rights reserved.

C:\Program Files\dotnet\sdk\2.2.300\NuGet.targets(121,5): error : Access to the path 'C:\src\smhMicro\obj\smhMicro.csproj.nuget.dgspec.json' is denied. [C:\src\smhMicro\smhMicro.c

sproj]

Build FAILED.

The only thing I changed in the tutorial was the name of the service.
dotnet --version
2.2.300
Windows Server 2019 Build 1809
I came across this but don't know if it's related (and no obvious soln:
https://github.com/NuGet/Home/issues/7836

@SimonHeffer if you use the default name for the service, do the tutorial steps work normally?
@MichaelSimons would you be able to assist with this docker issue from the tutorial?

@SimonHeffer - Did you by chance build the project locally prior to building the Docker image?   This may explain the behavior you are seeing.  The sample should be including a .dockerignore which excludes any local obj and bin directories.
.dockerignore content
# directories
**/bin/
**/obj/

@rowanmiller 's team owns this dotnet tutorial.

Tutorial has been updated to include instructions for adding a .dockerignore file.
@SimonHeffer feel free to reply back here if this doesn't solve your issue.


Fix formatting of test matrix for collateral inputs.

Issue
None. Noticed while reading the code.
Summary
This PR just repairs the formatting of a test matrix for collateral inputs, to improve readability.

bors r+


Web bugfix

if i fix the conflicts the webdoes not work 😦 🇫🇷🇫🇷™

adding automation for deployment 🥇

❗ issue with state for multiple users ❌


support for client expressions

Client commands provide a js expression to be evaluated on the client. See LED.fade which calls into LED.animate.
role.animate((color >> 16) & 0xff, (color >> 8) & 0xff, (color >> 0) & 0xff, speed * 0xff)

duplicate of https://github.com/microsoft/jacdac/issues/445?


Add securityContext compatibility with OpenShift platform

This pull request makes the Gotenberg helm chart compatible with the OpenShift platform, which unfortunately requires specific securityContext settings to work. I adapted these changes from the bitnami helm chart repo where they include this OpenShift compatibility in all of their helm charts (see here and here). Basically I added a little helper which removes things like runAsUser, runAsGroup and fsGroup from the security context. I also added another helper which detects if the helm chart is being installed on an OpenShift cluster, which is used to automatically apply these compatibility changes. You can also force or disable this OpenShift detection using the new compatibility.openshift.adaptSecurityContext value.
Let me know if you have any further questions! We have tested this helm chart on our OpenShift cluster and it should not break anything for existing deployments.
Tasks:

[X] I've made changes
[X] I've bumped chart's version in Chart.yaml
[X] I've added changes to charts CHANGELOG.md
[X] I've run helm-docs

While I don't mind adding this, I have concerns about licensing.
The provided code is almost perfect copy of Bitnami code which is licensed by Broadcom, Inc. under Apache 2.0. MIT and Apache are compatible, but from my limited understanding this would effectively mean I need to include the whole license from Bitnami repository and state that the code has been modified slightly.  I'm not a fan of complicating the licensing.
Alternatively, we could have our own implementation where the default value for security context check if it's running in OpenShift and removes the problematic fields. I personally prefer this since it doesn't make "magical" changes to security context when the user has specifically defined it. I feel like if someone manually sets security context then it should be honored as is and that also puts the responsibility of the actual value on the user.
Thoughts?

While I don't mind adding this, I have concerns about licensing.
The provided code is almost perfect copy of Bitnami code which is licensed by Broadcom, Inc. under Apache 2.0. MIT and Apache are compatible, but from my limited understanding this would effectively mean I need to include the whole license from Bitnami repository and state that the code has been modified slightly. I'm not a fan of complicating the licensing.

Hmmm I definitely see the problem... Although, I have found some other repos using the exact same snippet:

https://github.com/VictoriaMetrics/helm-charts/pull/995
https://github.com/external-secrets/external-secrets/pull/3420
(a few more but not sure about those, just search for renderSecurityContext globally)

Maybe we could at least include the _compatibility.tpl header somehow:
{{/*
Copyright Broadcom, Inc. All Rights Reserved.
SPDX-License-Identifier: APACHE-2.0
*/}}

Or give credits in some other way.
We might also be able to ping the original author here: @javsalgar
Would be great to hear his opinion on this! 🥺

Alternatively, we could have our own implementation where the default value for security context check if it's running in OpenShift and removes the problematic fields. I personally prefer this since it doesn't make "magical" changes to security context when the user has specifically defined it. I feel like if someone manually sets security context then it should be honored as is and that also puts the responsibility of the actual value on the user.

Unfortunately I'm not a 100% sure what you mean by this and what exactly you prefer about this solution 🤔 Could you give me a quick example?

mean by this
From what I understand, currently, if you deploy this chart to OpenShift as is with default configuration, it will not work.  What I want to do is to basically not add default security context if it's running on OpenShift.

I can provide code in a bit

From what I understand, currently, if you deploy this chart to OpenShift as is with default configuration, it will not work.

That's not the case. The utility I have added automatically detects OpenShift and modifies the default securityContext to work with OpenShift (basically just removes runAsUser from securityContext in values.yml). At our company we got it working with the default configuration - no problems there.

@MaikuMori wrote:
From what I understand, currently, if you deploy this chart to OpenShift as is with default configuration, it will not work.

That's not the case. The utility I have added automatically detects OpenShift and modifies the default securityContext to work with OpenShift (basically just removes runAsUser from securityContext in values.yml). At our company we got it working with the default configuration - no problems there.

Hi, I can't push to this PR for some reason. I've created a pr which implements what I meant: #34.
Do you think that's sufficient for the chart to work by default on OpenShift? If a user overwrites the default values, it's their responsibility that it works on their specific setup or in general.

@MaikuMori wrote:
Hi, I can't push to this PR for some reason. I've created a pr which implements what I meant: #34.
Do you think that's sufficient for the chart to work by default on OpenShift? If a user overwrites the default values, it's their responsibility to make sure it works on their specific setup or in general.

Not sure why you can't edit my PR but probably because I used the master branch instead of a new branch 😬 Anyway I added some comments in the PR, but it looks okay and works for me!

Closed in favor of #34

Closed in favor of #34

Awesome! Thanks!


NoFog does not work underground

I go underground near bedrock, I still fog

NoFog Version?

Unable to reproduce this with the latest NoFog version.
Lack of versions, mod list, Minecraft version, etc. make it hard to know what sort of situation made your problem happen.
If you still have this problem with the newest NoFog for your Minecraft version, please open a new issue and include more information.


Fix info blocks

Fixed the styling of the info blocks for:

Employee profile
Edit command
Find command

Codecov Report

Merging #229 (22080c4) into master (a15d496) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #229   +/-   ##
=========================================
  Coverage     56.41%   56.41%
  Complexity      499      499
=========================================
  Files            93       93
  Lines          2145     2145
  Branches        259      259
=========================================
  Hits           1210     1210
  Misses          865      865
  Partials         70       70

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

Also fixed #191.


Frontend: Add Goerli support

closes #1279
This PR adds support for the latest Goerli deployment (v1 contracts).
It also removes the old Rinkeby configs since Rinkeby is deprecated as well as all the rollups on it.

Thanks for the review @istankovic!
I think i tried looking for svgs but with no luck. Will try one more time now before closing this PR.


[backport] Warn NaN in FP16 mode in memnn example

Backport of #7345

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit 8532b7099c65f88d4d30a35edea70bec72227f73, target branch v6) failed with status FAILURE.

LGTM! The CI failure is not relevant to the change.


Error running az contianer exec

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az container exec
Errors:
The command failed with an unexpected error. Here is the traceback:
'ContainerInstanceManagementClient' object has no attribute 'container'
Traceback (most recent call last):
  File "/opt/az/lib/python3.6/site-packages/knack/cli.py", line 231, in invoke
    cmd_result = self.invocation.execute(args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 657, in execute
    raise ex
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 691, in _run_job
    result = cmd_copy(params)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/__init__.py", line 328, in __call__
    return self.handler(*args, **kwargs)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/core/commands/command_operation.py", line 121, in handler
    return op(**command_args)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/container/custom.py", line 609, in container_exec
    container_client = cf_container(cmd.cli_ctx)
  File "/opt/az/lib/python3.6/site-packages/azure/cli/command_modules/container/_client_factory.py", line 18, in cf_container
    return _container_instance_client_factory(cli_ctx).container
AttributeError: 'ContainerInstanceManagementClient' object has no attribute 'container'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

az container exec --exec-command='/bin/bash' --subscription={} -g {} -n {}

Expected Behavior
Environment Summary
Linux-5.11.0-34-generic-x86_64-with-debian-bullseye-sid, Ubuntu 20.04.3 LTS
Python 3.6.10
Installer: DEB

azure-cli 2.28.0 *

Additional Context

route to service team


How to Send HD Images using Baileys API

With the recent WhatsApp update introducing HD image support, I would like to understand how to send high-definition (HD) images using the Baileys API. Could you please provide guidance or code examples on how to achieve this?

Just send the image, image compression its from device not server side.

Not a bug


[question] Zotero bib API with zotero-better-bibtex

I'm using the zotero-better-bibtex plugin to export my references to a bibtex .bib file for use with LaTex. This seems to work really nicely - Thanks!
I would like to use bibbase.org to access a collection using the zotero API interface so that I can also have a 'live' publications list on my website. The bib file this produces (which is then parsed and rendered by bibbase.org) includes the bibtex parameters created/used by BBT in the note field, which then get rendered as seen at ( http://alastairhmoore.com/test-publications/ ) or without the embedding (http://bibbase.org/show?bib=https%3A%2F%2Fapi.zotero.org%2Fusers%2F3646722%2Fcollections%2FT2R9K7BX%2Fitems%3Fkey%3DBOpOAr8MeAOlvK9qDoPzmAHq%26format%3Dbibtex%26limit%3D100)
Is there anyway to avoid this? I checked on the zotero forum (https://forums.zotero.org/discussion/63773/extra-field-exported-as-note-in-bib-format-api-call) and apparently there is no way to use BBT to serve the API request which I think would be the best solution. Can you suggest an alternative?

It is not (currently) possible to run BBT outside Zotero; if it ever gets done, it would still require some kind of web server hosting as the BBT translator is unlikely ever to be included in Zotero itself.
It will at some point be possible to have the BBT data outside the extra field as the Zotero devs have planned for a way for extensions to store their reference-related data in a clean way in de DB, but that's still a while off, and there's no ETA.
If you're not afraid of a little hackery though, it can still be done. You could set your library up to auto-export using BBT, use a tool to auto-upload that file to a webserver when it changes, and then have bibbase.org use that online file to fetch the bib file. Dropbox used to be able to do direct downloads, but I'm not sure they allow that anymore. As a last ditch, sparkleshare + github pages should work, but will take a bit of setting up.

Thanks for the quick reply. It's a shame about these current limitations. It seems like zotero+BBT is very nearly perfect for my needs so I look forward to the zotero devs allowing you to move the bibtex stuff out of extra.
In the meantime, I did think of doing something along the lines of what you suggested. For my personal references this could work ok - thanks for the heads about sparkleshare. However, for me to convince the rest of my research group that we would shift our group bib file to zotero I would need to be able to run the auto-export from an always-on machine so that it picks up changes made by other people when I'm not connected. Can zotero+BBT be run from a headless red hat installation? I did a quick search but couldn't see anything.

That is fully possible - see eg http://stackoverflow.com/questions/12050021/how-to-make-xvfb-display-visible. In this scenario you would have the choice between auto-export and have the resulting file served by a simple web server (I'd recommend nginx) or BBTs pull export and put a reverse proxy in front of it (again nginx would be my choice here).
I'll give some thought to something that would work in a less hacky way, but this ought to get you started.

Thanks for the pointer. I'll see whether our server admin can support this.
For my personal collection, I have now managed to get the dropbox solution to work in a way that I am really happy with http://alastairhmoore.com/publications/. As a note to anyone else attempting this, to get this to work I had to

get the share link
convert it to a download link using http://www.syncwithtech.org/p/direct-download-link-generator.html
encode it to URL/percent encoding using http://www.url-encode-decode.com/
paste it into the appropriate bibbase script command, in my case <script src="http://bibbase.org/show?bib=https%3A%2F%2Fwww.dropbox.com%2Fs%2Fbwdznrwia21wdyo%2Falastairhmoore.bib%3Fdl%3D1&jsonp=1&group0=type&group1=year"></script>

If that xvfb setup isn't feasible, you could make use of my testing harness. It's set up to run fully automated in a headless environment (circleci in my case, but any should do) and it can trigger pretty much any behavior you'd want, including export of the library. A call by IFTTT could schedule a run of the circle test, but you'd still need a place to put the resulting file.

I found a windows machine that I thought could be always on and tried setting it up to do the auto-export. However, it doesn't seem to pick up changes to the database made on another machine. Of course if I manually press the sync button then it gets the changes (and BBT instantly does its stuff), but that kind of defeats the point. Is there a setting in zotero somewhere that I'm missing or is this something that the test harness would help with?

It's a setting in the Zotero preferences at the same place you configure the sync account. I assume it's the same across all platforms, but I don't have access to a windows PC to check. Whether auto-sync works shouldn't be affected by the presence of BBT, but you can send a BBT error report if you want and I'll have a look if there's any error preventing sync.

I have auto sync enabled, it just doesn't seem to actually do it. I agree it's not a BBT error so I'll check on the zotero forum to see if they can help. Thanks.

While I don't expect BBT to interfere here, it wouldn't be the first time it broke something in zotero (although the last time that happened was long enough ago that I can't remember). I could take a look, but I won't be able to inspect the logs until noon tomorrow.

It seems there is an error, but I can't tell whether it is anything to do with BBT.
Steps to reproduce:

Start zotero standalone on a mac and windows machine (both show the same synced group library with one collection, windows machine is already setup to auto-export the collection on change)
Change title of an entry on mac.
Press sync to force sync to zotero server
Check in browser that record is updated - works fine
Check zotero standalone on windows machine - entry is not updated
Generate first error report (E9TI47AZ)
After a few minutes without any sync activity, manually force sync - entry is updated, and BBT exports updated bib file as expected
Generate second error report (NSMS96X6)

Alright so it's good that I looked into this first because

BBT did trigger a fault in Zotero (although that was a Zotero function not dealing well with unexpectedly large input), and
the function I called to trigger that did not return the selection of reference-collections that I needed anyway, so needed to call something else anyhow.

Tests are running on a fix.

Can you give  https://github.com/retorquere/zotero-better-bibtex/releases/download/builds/zotero-better-bibtex-1.6.90-br626-3435.xpi a try?

Added the new xpi to both the mac and windows machines. Restart with clear cache.
Added new entry on mac + forced sync
Browser shows update
Windows machine does not - report ID: 75SCHQRT
Force sync on windows - entry now appears and BBT does export
Generate another error report for good measure: EZ3IJKJC

OK so the good news is that the error that probably prevented sync is gone. I've merged that fix and scheduled 1.6.91 for release to include it.
The not so good news is that I now see absolutely no errors so I have no idea why auto-sync is not happening. I'm afraid the Zotero team will have to look into it. They're almost certainly going to ask you whether it works when BBT is not installed, so you may as well try that ahead of submitting a report to them. In any case, whether you keep BBT installed or not, remember to turn off extended BBT debugging -- the log is very, very chatty (about 40MB a pop in your last reports) when it is on (OK so BBT is very, very chatty -- most of that 40MB is mine), and they may not want to plod through all of it.
Do keep me posted -- don't close this issue just yet.


Generalize zip and unzip.

This is a discussion PR.
We can make zip and unzip more general than they are by relying on Apply and Functor respectively. This removes the dependency on purescript-arrays. But it also changes the semantics of zip a bit if you're not careful. This would require us to have a ZipList type in order to get he semantics we're all used to with the zip function. unzip works the same way regardless as there's only one valid Functor.
This PR gives us the ability to zip/unzip say two Maybes, two Map ks, or two V errs or anything with just an Apply
So, is it worth it to break the semantics for more generality and removal of a dependency?

By chance, this also fixes a runtime stack overflow/memory allocation bug when the array is large.

Actually, thinking about that bug above, I guess this also allows unzip for lists to be O(n) rather than O(n^2) or whatever the current array pattern matching gives. Of course, given an O(n) Functor instance.

Alternatively, we could do zip' :: forall f a b.  This way we don't break old semantics. However, we can't get rid of the purescript-arrays dependency if we go this route.

Actually, I see that these are basically a more general version of the Bitraversable instance for Tuple: https://github.com/purescript/purescript-bifunctors/blob/master/src/Data/Bitraversable.purs#L13-L15 I guess it's not really worth it to break semantics.
Thoughts?

Well, I take that back, zip is a more general version. unzip doesn't exist yet in the general form.

Given liftA2, is there much need for this generalized zip? I'm not sure it's worth the breaking change, honestly.
As for unzip, I think the change is good, but here's some food for thought: if we strengthen the constraint to Foldable, we can use a single pass over the array/list/whatever.

YEah, I after seeing the Bitraversable instance, I don't think there's much point to the zip change.
I might be having a brainfart, but I'm not sure Foldable is strong enough to unzip. If it is, we'll still have trouble. I think if we change the unzip constraint to Foldable it becomes unpredictable what it will do. With Functor, the laws can at least enforce that we get something sensible out of unzip. That is, a law abiding list-style (array, list, stream, etc) Functor instance will ensure that if we give unzip a "list" with n Tuple a b elements, you get back out a Tuple with the first part containing a "list" of n a elements and the second part containing a "list" of n b elements. Since Foldable is lawless, the semantics of unzip become unenforceable.
Then again, what does that say for any/all/sum/etc

Maybe Traversable provides a another option?

Well, I guess it's a non option, as adding a dependency on purescript-foldable-traversable would make a cycle in dependencies.
In any case, I moved to just the unzip changes to fix stack overflow/memory errors and lower complexity.

I see unzip was moved, but I'm not sure where it was moved to. Should I just close this?

It lives in purescript-arrays and purescript-lists now. For things that aren't quite Traversable but can construct different list-y things we implement them in with that collection type now (same goes for some, many, foldM, replicateM, etc).

Cool. Thanks.


Change badge icon style

Now all badges have the same style.

Current coverage is 100.00%

Merging #10 into master will not affect coverage as of e0a065e

@@            master     #10   diff @@
======================================
  Files            2       2
  Stmts           27      27
  Branches         0       0
  Methods          5       5
======================================
  Hit             27      27
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of e0a065e

Powered by Codecov. Updated on successful CI builds.


can't get the remove icon to show

hello,
thanks for the nice code, but I'm having a problem showing the x icon to remove the a single filter
i'm using bootstrap 3.3.7 and a custom jquery-ui theme for bootstrap

The html is
If you copy and paste the above html, you should see a close icon (x). Otherwise, you must be missing some css files.


vue-tsc skipLibCheck is not working

/Users/shufeng/code/vue3-vite-template/vue3-vite-template/node_modules/vscode-jsonrpc/lib/common/linkedMap.js:40
return this._head?.value;
^
SyntaxError: Unexpected token '.'
Why vue-tsc is not reading tsconfig.json?
tscexecute successfully.

the vue-tsc version is 0.29.8

Can you provide a reproduction demo?

Can you provide a reproduction demo?

Update Node version to 14 solved my problem.12.18.2 don't support grammar like '?.'Thx


Github CoPilot Chat not upgrading

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

VS Code Version: 1.82.2
OS Version: mac OS Sonoma 14.3 (23D56)

Hi everyone, I'm having trouble upgrading my version of GitHub Copilot Chat to v.0.12.... so that I can get the functionality in VS Code Chat where I can type @workspaces and get the whole context of the chat.
It seems to be stuck on v0.7.2023090101 and I switch to the PreRelease version and get "v0.11.1"
There's no option to upgrade to v0.12
In the Output tab in VS code I get
TypeError: Failed to parse URL from /Users/nils.vanderwerf/.vscode/extensions/rebornix.ruby-0.28.1/dist/server/tree-sitter.wasm
``

and

abort(TypeError: Failed to parse URL from /Users/nils.vanderwerf/.vscode/extensions/rebornix.ruby-0.28.1/dist/server/tree-sitter.wasm). Build with -s ASSERTIONS=1 for more info.
(Use Code Helper (Plugin) --trace-uncaught ... to show where the exception was thrown)
Node.js v18.15.0
[Error - 10:23:54] Connection to server got closed. Server will not be restarted.

nothing seems related?

I'm wondering if its a setting in my settings.json I need to change?

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.85.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!


JavaScript to help plugins interact with the fragment part of the URL

Suggested by Markus Holtermann on Twitter, who is building https://github.com/MarkusH/datasette-chartjs

I've been looking at datasette-vega for how you persist chart settings between form submissions. I've adopted that for datasette-chartjs. Any thoughts on adding a public JS API to #datasette itself, that plugins can rely on?
I'm talking about functions like onFragmentChange, serialize, unserialize, ... That turn an object into a URL encoded string and put it into the location's hash. And also updating all links/forms automatically.
Essentially, a plugins could do something like document.datasette.setConfigValue('prefix', 'foo', 'bar') and .getConfigValue('prefix', 'foo'). And the functions would take care of updating document.location.hash, all (necessary) a.href and form.action

https://twitter.com/m_holtermann/status/1338183973311295492

Thanks for opening the issue, @simonw. Let me elaborate on my Tweets.
datasette-chartjs provides drop down lists to pick the chart visualization (e.g. bar, line, doughnut, pie, ...) as well as the column used for the "x axis" (e.g. time).
A user can change the values on-demand. The chart will be redrawn w/o querying the database again.
However, if a user wants to change the underlying query, they will use the SQL field provided by datasette or any of the other datasette built-in features to amend a query. In order to maintain a user's selections for the plugin, datasette-chartjs copies some parts of datasette-vega which persist the chosen visualization and column in the hash part of a URL (the stuff behind the #). The plugin load the config from the hash upon initialization on the next page and use it accordingly.
Additionally, datasette-vega and datasette-chartjs need to make sure to include the hash in all links and forms that cause a reload of the page. This is, such that the config persists between clicks.
This ticket is about moving thes parts into datasette that provide the functionality to do so. This includes:

a way to load config options with a given prefix from the current URL hash
a way to update the current URL hash with a new config value or a bunch of config options
updating all necessary links and forms on the current page to include the URL hash whenever its updated
to prevent leaking config options to external pages, only "internal" links should be updated

There's another, optional, feature that we might want to think about during the design phase: the scope of the config. Links within a datasette instance have 1 of 3 scopes:

global, for the whole datasette project
database, for all tables in a database
table, only for a table within a database

When updating the links and forms as pointed out in 3. above, it might be worth considering which links need to be updated. I could imagine a plugin that wants to persist some setting across all tables within a database but another setting only within a table.


Nightly GitHub Action failure

Our nightly GitHub Action for the container image failed with:
#14 19.22 Copying rrdp_delta.c
#14 19.23 Copying rrdp_notification.c
#14 19.24 Copying rrdp_snapshot.c
#14 19.25 Copying rsync.c
#14 19.25 Copying tal.c
#14 19.25 Copying validate.c
#14 19.25 Copying version.h
#14 19.27 Copying x509.c
#14 19.28 Applying patch patches/0001-Allow-overriding-default-user-and-file-locations.patch
#14 19.28 1 out of 6 hunks FAILED -- saving rejects to file main.c.rej

Confirming successful run with commit e26c17a - thank you.


SQLXML 4.0 SP1 and MSXML

On this page, it has been mentioned that SQLXML uses MSXML 6.0, whereas the installation document of SqlXml 4.0 Service Pack 1 (SP1) at https://www.microsoft.com/en-us/download/details.aspx?id=30403 mentions that SQLXml has a dependency on MSXML version 4.0 SP2. This version of MSXML is included with the download.
Please let me know the current version of MSXML that SQLXML 4.0 SP1 needs.

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

ID: 518df43a-f11e-84f8-d227-31c888b538b8
Version Independent ID: 8892df79-4e69-47e2-6901-3e6ed9d93b9a
Content: What's New in SQLXML 4.0 SP1 - SQL Server
Content Source: docs/relational-databases/sqlxml/what-s-new-in-sqlxml-4-0-sp1.md
Product: sql
Technology: xml
GitHub Login: @MightyPen
Microsoft Alias: genemi

@sandys21 Thank you for the question.
Assigning this to the author for further clarification.

@sandys21 I apologize for the delay on this answer. I have not had a chance to look into this dependency question. However, since the download you obtained came with a specific version of MSXML, I would use that version. If you want to use the 6.0 version, I assume that would work per the article, but you would have to test it. However, I apologize for the confusion between the article and the download, and thank you for reporting it. At this time, I assume you got it working with the version that came with the download, and I will close this git issue. If that is not the case, please let me know! Thank you! #reassign:rothja


Update implemented try catch

Description
Admin-SDK API's directory, reports, reseller code changes, added additional comments and added exception handling.
Fixes # (230522680)
Has it been tested?

[X] Development testing done
[ ] Unit or integration test implemented

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have performed a peer-reviewed with team member(s)
[ ] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] Any dependent changes have been merged and published in downstream modules

Out of sync so closing PR


Succeed use for SQLite in Windows, but failed in Mac.

I've implement an application, it works when I code in Windows. But the same code gave Exception in Mac. I use SQLite database and have put sqlite-jdbc-<IP_ADDRESS>.jar as library.
I know the driver went ok since I have check it like below:
public static void pureSqlite() throws Exception {
        String url = "jdbc:sqlite:hadits-bukhari.sqlite";
        Connection connection = DriverManager.getConnection(url);
        System.out.println("Connection established.");

        String sql = "SELECT * FROM hadits";
        Statement statement = connection.createStatement();
        ResultSet resultSet = statement.executeQuery(sql);

        int i = 0;
        while (resultSet.next()) {
            if (i == 5) break;

            System.out.println(resultSet.getInt("nomer")+". "+resultSet.getString("deskripsi"));

            i++;
        }
    }

It works perfectly.
Then, I've implement it with ORMLite's way. Create Hadits class with required annotation. Then I have below function:
public static void withOrmLite() throws Exception {
        Class.forName("org.sqlite.JDBC"); // even put this test
        System.out.println("Frankly, class org.sqlite.JDBC is exists.");

        Ilmu ilmu = new IlmuImpl();
        List<Hadits> list = ilmu.findHadits("bukhari", "");
        int i = 0;
        for (Hadits item: list) {
            if (i == 10) break;
            System.out.println(item.number+". "+item.desc);
            i++;
        }
    }

From IlmuImpl:
@Override
    public List<Hadits> findHadits(String book, String keyword) throws Exception {
        String dbHost = "jdbc:sqlite:hadits-"+book+".sqlite";
        ConnectionSource connection = new JdbcConnectionSource(dbHost);
        Dao<Hadits, Long> haditsDao = DaoManager.createDao(connection, Hadits.class);

        List<Hadits> list;

        if (keyword == null || keyword.trim().length() == 0) {
            list = haditsDao.queryForEq("buku", book);
        } else {
            QueryBuilder<Hadits, Long> queryBuilder = haditsDao.queryBuilder();
            queryBuilder.where().like("deskripsi", "%"+keyword+"%");
            PreparedQuery<Hadits> preparedQuery = queryBuilder.prepare();
            list = haditsDao.query(preparedQuery);
        }

        connection.close();
        return list;
    }

This code do fine in Windows, but failed in Mac, says:
Exception in thread "main" java.sql.SQLException: Driver class was not found for SQLite database.  Missing jar with class org.sqlite.JDBC.
	at com.j256.ormlite.misc.SqlExceptionUtil.create(SqlExceptionUtil.java:27)
	at com.j256.ormlite.db.BaseDatabaseType.loadDriver(BaseDatabaseType.java:47)
	at com.j256.ormlite.jdbc.JdbcConnectionSource.initialize(JdbcConnectionSource.java:137)
	at com.j256.ormlite.jdbc.JdbcConnectionSource.<init>(JdbcConnectionSource.java:117)
	at com.j256.ormlite.jdbc.JdbcConnectionSource.<init>(JdbcConnectionSource.java:59)
	at ilmu.implementation.IlmuImpl.findHadits(IlmuImpl.java:17) <-- this is my class
...
Caused by: java.lang.ClassNotFoundException: org.sqlite.JDBC
...

What could be wrong?

I'm also facing the same issue. Were you able to find any solution?

Typically this means that you are not loading one of the sqlite driver jars.  For example, you maybe should add the following to your pom.xml file if you are using maven:
<dependency>
	<groupId>org.xerial</groupId>
	<artifactId>sqlite-jdbc</artifactId>
	<version><IP_ADDRESS></version>
</dependency>

Thanks for the quick reply. I was able to make it work after including the above mentioned dependency


[Feature][chunjun] upgrade the version of Flink from 1.12 to 1.16 (#1375)

Purpose of this pull request
upgrade the version of Flink from 1.12 to 1.16 .
Some problems with the Connector still need to be fixed.

👍


Docs may be outdated; example scripts are not working.

I tried running the Minimal Example and the Repository Examples but neither seems to work anymore.
The former gives the error:
error: Uncaught SyntaxError: The requested module 'https://deno.land/x/webui/mod.ts' does not provide an export named 'WebUI'
import { WebUI } from "https://deno.land/x/webui/mod.ts";
         ^
    at <anonymous> (file:///path/to/garden/app.js:1:10)

and the latter gives the error:
error: Uncaught (in promise) Error: Could not open library: Could not open library: dlopen(/Users/jim/.deno_webui/ffd9edc4-webui-2.dyn, 0x0005): tried: '/Users/jim/.deno_webui/ffd9edc4-webui-2.dyn' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64')), '/System/Volumes/Preboot/Cryptexes/OS/Users/jim/.deno_webui/ffd9edc4-webui-2.dyn' (no such file), '/Users/jim/.deno_webui/ffd9edc4-webui-2.dyn' (mach-o file, but is an incompatible architecture (have 'x86_64', need 'arm64'))

My current environment is like so:

deno 1.36.3 (release, aarch64-apple-darwin)
v8 <IP_ADDRESS>
typescript 5.1.6
Macbook Pro M2 Venture 13.2.1

Has anyone else encountered this issue?

Related to this issue

I just tested it on Windows right now:
git clone https://github.com/webui-dev/deno-webui.git
cd deno-webui\examples\hello_world
deno run --allow-all --unstable hello_world.ts

I will re-test it on macOS tomorrow.

Thanks, Hassan. I got around the first issue by following the issue you linked and pulling the repo.
However, the Mac issue seems to be because WebUI expects a different CPU architecture.

You are right, we need to add the ARM64 lib into the repo. I will do it tomorrow.

Just a friendly bump that most of the stuff in the readme still doesn't run

Yep, running into the same stuff. Have it working on Apple Silicon by checking out the tree and using the library directly but the deno.land stuff is out of date. Also FWIW on latest MacOS with Chrome I'm just getting the UI opening in the full browser itself instead of the raw frame.

WebUI does not support Safari app mode yet. Firefox or any Chromium-based browser should be installed first to use app mode. Otherwise, WebUI will try to open the UI in the default macOS browser window, which will be in most cases Safari (Not app-mode).
On the other hand, WebUI v2.4.0 has been released... We will update the Deno-WebUI wrapper soon.

Firefox or any Chromium-based browser should be installed first to use app mode

Okay then you probably want to change the Readme.md cause it says

Fully Independent (No need for any third-party runtimes)

If the system needs a browser installed, then I wouldn't really call this library "fully independent". A browser is a JavaScript/Wasm runtime.

A browser is a third party

On one side, yes. In another, no. All programs in all OS need hundreds or thousands of libs, drivers, runtime, etc. All those are built-in in the OS, so nobody talks about it because it's already installed. A web browser is one of them.
Conversely, because Deno-webui needs Chromium browsers and not Safari, it is a third party then, as you said.
I suggest adding in the readme that a Chromium-based browser is needed.

All programs in all OS need hundreds or thousands of libs

Actually some don't use a single system library. Take a look at Cosmopolitan Libc not only does it statically link everything (does not link to a single system library at runtime), not only does it not call any system executables or need any env vars, but it can also make a single binary that runs on all OS's, even Windows.
I use these kinds of hyper-agressively independent binaries and tools for making reliable bootstrapping steps.

because Deno-webui needs Chromium browsers and not Safari

You do make a good point though. I agree there is no way to make a graphical interface without using a system library; the tool always has to talk to whatever desktop environment the user has installed.
However, linking to something like libc is different than linking to a browser. Mac/Linux/Windows are all guaranteed (as part of the spec) to have libc. For browsers, Mac is guaranteed to have safari and windows is guaranteed to have edge. But Linux isn't required to have Firefox, or have it in the same location, and it can be updated independent of the OS version.
If there was support for Safari, then I would agree and say "it only needs a 2nd party runtime".
Saying "it needs a second party browser runtime" is very different than "no 3rd party runtime" which implies the runtime is provided and is consistent. The 2nd party runtime tell me behavior will be slightly different for each OS, and that there may be reliability issues on Linux.

Also can confirm readme example is working 👌  So this can probably be closed


examples please

I've looked at the source and the scripts... but I need some examples that show off it's major features. I see some plan9 code embedded and docker too, however, the scripts talk about booting Linux and so the two confuse me. Please elaborate.

Thanks for the note @rbucker, I'll add some in the future.

There are two things which are really lacking:

how to use the networking
there is some mention of docker, but it is not clear how it might be used with novm.


KYLIN-3586 Fix eight primitive boxing/unboxing problem.

KYLIN-3586 Fix eight primitive boxing/unboxing problem.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Codecov Report

Merging #269 into master will decrease coverage by <.01%.
The diff coverage is 15.51%.

@@             Coverage Diff              @@
##             master     #269      +/-   ##
============================================
- Coverage     21.15%   21.15%   -0.01%
+ Complexity     4406     4405       -1
============================================
  Files          1086     1086
  Lines         69743    69743
  Branches      10088    10088
============================================
- Hits          14755    14754       -1
- Misses        53587    53588       +1
  Partials       1401     1401

Impacted Files
Coverage Δ
Complexity Δ

.../java/org/apache/kylin/rest/job/HybridCubeCLI.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...va/org/apache/kylin/rest/service/QueryService.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...in/java/org/apache/kylin/metadata/tuple/Tuple.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...age/hbase/lookup/LookupTableHFilesBulkLoadJob.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...e/kylin/metadata/filter/UDF/MassInTupleFilter.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...java/org/apache/kylin/common/util/MailService.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...java/org/apache/kylin/tool/MetadataCleanupJob.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...n/java/org/apache/kylin/tool/CubeMetaIngester.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ache/kylin/metadata/filter/ExtractTupleFilter.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...n/java/org/apache/kylin/tool/DiagnosisInfoCLI.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

... and 13 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 9ae03f6...6b293c9. Read the comment docs.

Pull Request Test Coverage Report for Build 3694

10 of 58 (17.24%)  changed or added relevant lines in 19 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.001%) to 23.166%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

tool/src/main/java/org/apache/kylin/tool/MrJobInfoExtractor.java
0
1
0.0%

core-common/src/main/java/org/apache/kylin/common/util/MailService.java
0
1
0.0%

core-metadata/src/main/java/org/apache/kylin/metadata/filter/ExtractTupleFilter.java
0
1
0.0%

core-metadata/src/main/java/org/apache/kylin/metadata/filter/TimeConditionLiteralsReplacer.java
0
1
0.0%

core-metadata/src/main/java/org/apache/kylin/metadata/filter/UDF/MassInTupleFilter.java
0
1
0.0%

core-metadata/src/main/java/org/apache/kylin/metadata/tuple/Tuple.java
0
1
0.0%

storage-hbase/src/main/java/org/apache/kylin/storage/hbase/lookup/LookupTableHFilesBulkLoadJob.java
0
1
0.0%

tool/src/main/java/org/apache/kylin/tool/MetadataCleanupJob.java
0
1
0.0%

source-kafka/src/main/java/org/apache/kylin/source/kafka/DefaultTimeParser.java
0
1
0.0%

server-base/src/main/java/org/apache/kylin/rest/job/HybridCubeCLI.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-metadata/src/main/java/org/apache/kylin/metadata/cachesync/Broadcaster.java
1
57.71%

core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java
1
81.08%

Totals

Change from base Build 3689:
-0.001%

Covered Lines:
16157

Relevant Lines:
69743

💛  - Coveralls


Fix podman-remote start tests

Also enable some tests for remote by removing -l flag.
Signed-off-by: Daniel J Walsh<EMAIL_ADDRESS>

LGTM

LGTM

/lgtm


DocumentFragment nodes not handled properly

I tried creating a DocumentFragment node, appended a few children to it, and then appended the DocumentFragment to my document. According to the spec, this should move all child nodes of the fragment into the document at that position (leaving an empty DocumentFragment behind). When I do that with Lexbor, however, the fragment itself is added to the DOM.
In addition to that, the HTML serialization function ignores DocumentFragment nodes, so when I serialize the final document, it seems like the fragment wasn't added at all. In reality, it was added (as you can verify with lxb_dom_node_last_child(), which gives you a #document-fragment), but it isn't serialized, because serializing a DocumentFragment results in an empty string.
Furthermore, appending the first child of a document fragment to the document does not remove it from the fragment. The child node's parent pointer is set to the new parent, but the fragment's first_child pointer still points to the moved child node.

@phoerious
I don't quite understand what the problem is. Can you show the code and specify the place you think the problem is?

I don't have a full C example at hand, but here is the Rust code that binds with Lexbor: https://github.com/chatnoir-eu/chatnoir-resiliparse/blob/rust/resiliparse/lib/parse/html/dom.rs#L899
Problem 1: inserting a document fragment as a child node doesn't behave in accordance with the standard (i.e., move all child nodes to the new parent instead of inserting the fragment itself).
Problem 2: When I implement the missing logic for adhering to the DOM standard, the child pointers of the document fragment are not NULL after moving the child nodes out of the fragment.
You may argue that Lexbor doesn't provide a fully standards-compliant DOM API anyway, so problem 1 should be left to the user of the library (although I would prefer it being fixed in Lexbor itself), but problem 2 should definitely be fixed.


glitchy audio on unity

Hi,
I'm new to using pd and heavy.
I made a patch (from a tutorial on rain generation) and tried to make it work in unity. It seems that something in the right branch ( /~ or vcf~  ?) is failing. The patch works with pure data but with unity I can start it once and it works fine but if i stop it (using the event) and start it again the sound gets glitchy.
No problem when i disconnect that branch.
Maybe I'm just missing something obvious...
raingenerator.zip

hi @rampulski sorry about the delay in responding, did you ever find a solution to your glitchy problem?


Bluetooth: indication callback signature update

Update the signature of the bt_gatt_indicate_func_t callback type by
replacing the attr pointer with a pointer to the
bt_gatt_indicate_params struct that was used to start the indication.
This allows the callback to free the bt_gatt_indicate_params instance
if it was allocated from storage, while still allowing the
bt_gatt_attr value to be accessed through params->attr.
Allocating the bt_gatt_indicate_params instance from storage is
desirable as multiple indications can be queued, however each instance
must be valid until the callback is run.
Fixes #28688
Signed-off-by: Jordan Yates<EMAIL_ADDRESS>

@JordanYates thank you very much for this proposal. This is however a stable API change, so would you mind following the procedure outlined here? If you cannot attend the API meeting that is not a problem. Thanks!

Superseded by the PR implementing the RFC #29358


CodeMirror Markdown syntax highlighting breaks if <[a-z] is encountered

It also means HTML(XML) will never be able to be highlighted, while also breaking the entire highlight flow.
Is it related to #765?

I guess xml mode needs to be loaded.
Shouldn't be related to #765 because that's highlight.js if I remember correctly and the letters around angle brackets disappears in that case.

Fixed


Add 'snabb-lwaftr' executable and packetblaster generator

bin/snabb-lwaftr: LwAftr executable. It can be executed from any location as long as 'snabb' executable is within PATH.
bin/snabb-lwaftr-blaster: Reads IPv4 and IPv6 pcap files and flushes packetraffic traffic to NICs.
bin/snabb-lwaftr.inc.sh: Helper functions.

Overall, nice work. Several nits:
a)  "error "Couldn't find configuratoin file" has a typo in configuration
b) Please check that the scripts are being run as root
c) Please make the lack of setting the PCI details fatal; it should not fall back to something which is extremely unlikely to be right for the machine anyone else is running it on.
d) Please make the default input files the 550 byte pcaps, not the larger ones.
e) Please get rid of "local SNABB_SRC=~/snabbswitch/src"; this is highly unlikely to be correct for other people running the scripts.

Updated PR, PTAL. I placed the root guard after the parameter processing so it's still possible to run the "--help" option without being root.

More nits:
a) The blaster doesn't fail gracefully if a specified nic is already in use; one nic blasts along happily, one gives a traceback:
$ sudo ./snabb-lwaftr-blaster --ipv4  ~/snabbswitch/tests/apps/lwaftr/benchdata/ipv0550.pcap --ipv6 ~/snabbswitch/tests/apps/lwaftr/benchdata/ipv6-0550.pcap --nic-inet 01:00.0 --nic-b4 02:00.1
failed to lock /sys/bus/pci/devices/0000:01:00.0/resource0
lib/hardware/pci.lua:114: assertion failed!
stack traceback:
	core/main.lua:116: in function <core/main.lua:114>
	[C]: in function 'assert'
	lib/hardware/pci.lua:114: in function 'map_pci_memory'
	apps/intel/intel10g.lua:89: in function 'open'
	apps/intel/loadgen.lua:20: in function 'new'
	core/app.lua:165: in function <core/app.lua:162>
	core/app.lua:197: in function 'apply_config_actions'
	core/app.lua:110: in function 'configure'
	program/packetblaster/packetblaster.lua:51: in function 'run'
	core/main.lua:56: in function <core/main.lua:32>
	[C]: in function 'xpcall'
	core/main.lua:121: in main chunk
	[C]: at 0x0044e580
	[C]: in function 'pcall'
	core/startup.lua:1: in main chunk
	[C]: in function 'require'
	[string "require "core.startup""]:1: in main chunk

b) The arguments should be renamed. --ipv4 doesn't sound like a parameter that should take a pcap file, for instance, and the NIC on the internal network is questionably enough called b4side in places, but calling it straight out nic-b4 is pushing that too far. Maybe --v4-pcap, --v6-pcap, --v6net-pci, and --v4net-pci? (Also not great names, admittedly).

Fixed issues and rebased.
I ended up named the card parameters, v4-pci and v6-pci so it's consistent with v4-pcap  and v6-pcap`. I made the change in both files.
Other changes:

Minor case variables names (major case is only used for env variables).
Removed the --kill argument. Trap SIGINT and free cards.
Use single-bracket instead of double-brackets where possible.

Nice. Given that these are the preferred front-end for running our tools, they should probably be in a snabbswitch/bin directory, rather than so far the tree. I'm also wondering whether they should try to guess if snabb has been compiled and say if not, but perhaps not.

Done. I moved the scripts into snabbswitch/bin. Path bin was included in .gitignore, so I had to remove it.

Merged, thank you.


Incorrect import/export paths for TypeScript + ESM + NodeNext

Describe the bug
With TypeScript + moduleResolution:NodeNext, you get import errors like this:
node_modules/peopledatalabs/dist/index.d.ts:2:58 - error TS2834: Relative import paths need explicit file extensions in ECMAScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Consider adding an extension to the import path.

import { AutoCompleteParams, AutoCompleteResponse } from './types/autocomplete-types';

To Reproduce
Steps to reproduce the behavior:
Import anything from peopledatalabs in a TypeScript project using moduleResolution: NodeNext
Expected behavior
Imports without error
Screenshots
If applicable, add screenshots to help explain your problem.

peopledatalabs Library Version: 6.0.0
Node Version: 20.10.0
NPM Version: 10.2.3

Additional context
Using https://arethetypeswrong.github.io/?p=peopledatalabs%406.2.0 as a guide, you can get a sense of issues that need to be fixed for proper TypeScript + ESM + NodeNext support. The resolution errors called out are likely due to the missing .js extensions. Once you fix those, it's then just about getting the package.json correct. See bpmn-io/min-dash#29 for a similar fix (had a similar set of issues + similar initial package.json layout).

Great, thanks!


Remove Microsoft.AspNetCore.All

Resolves https://github.com/aspnet/AspNetCore/issues/3418
This shared framework and metapackage are obsolete. The recommended replacement is Microsoft.AspNetCore.App.

Depends on https://github.com/aspnet/MusicStore/pull/873


Do not copy the old selection when applying localization to dates in InputDatePickerFormField

Fixes https://github.com/flutter/flutter/issues/107155

@jason-simmons Looks like the analyzer is unhappy with this change. Could you take a look? Thanks.

Fixed the analyzer issues.


RunningActionsManager's kill_all_waits_for_all_tasks_to_finish test never ends on Windows x64

While I was testing on Windows x64 machine, I found out that running_actions_manager_tests::kill_all_waits_for_all_tasks_to_finish test never ends when I test with cargo test --all command.
They used to be working fine when I test on my Linux machine.
On the other hand, the similar test which has a platform specific code running_actions_manager_tests::worker_times_out is working great.

This has been a long-standing issue in CI. If I remember correctly either this issue or something similar was reproducible on linux by uncommenting these lines:
https://github.com/TraceMachina/nativelink/blob/96302cbeab6c59966d3dfd3b99fa0933752d1018/flake.nix#L113-L120

Hi, @aaronmondal .
FYI, decreasing this number from 1000 to 500 seems to fix the problem on my machine.
It seems like too many iterating made our tests not to finish on Windows x64 machine.
Is it okay to make PR to decrease this numer to 500?
https://github.com/TraceMachina/nativelink/blob/a18d2d2a9e1a1d1ca5f77c305e948d62e7c4a2e1/nativelink-worker/tests/running_actions_manager_test.rs#L2511


cargo install --list lines / 2 is not how to count the number of installed crates

Crates may install more than one binary, for example:
cargo-binutils v0.3.2:
    cargo-nm
    cargo-objcopy
    cargo-objdump
    cargo-profdata
    cargo-readobj
    cargo-size
    cargo-strip
    rust-ar
    rust-ld
    rust-lld
    rust-nm
    rust-objcopy
    rust-objdump
    rust-profdata
    rust-readobj
    rust-size
    rust-strip
cargo-edit v0.7.0:
    cargo-add
    cargo-rm
    cargo-upgrade
sqlx-cli v0.2.0:
    cargo-sqlx
    sqlx

Counting the un-indented lines should work though.

Interesting, i'll dig into this

@jplatte
Republished the crate.
Now function looks like this
Can you check this out again?
fn get_cargo_crates() -> usize {
    let cargo_installs = match exc("cargo install --list") {
        Ok(installs) => installs.stdout,
        Err(_) => "not present".as_bytes().to_vec(),
    };
    let cargo_installs = std::str::from_utf8(&cargo_installs).unwrap().lines();

    let mut cargo_vec: Vec<String> = Vec::new();

    for line in cargo_installs {
        if !line.starts_with("    ") {
            cargo_vec.push(line.to_string());
        }
    }
    return cargo_vec.len();
}

Instead of collecting all of the lines into another Vec and counting the elements, you can just filter the iterator and count how many elements it contains:
cargo_installs.filter(|line| !line.starts_with("    ")).count()

(plus you don't need the explicit return)

Thanks i did ! (need to read Rust Book and Rust idiomatic ways like FP)
That is because  i've been doing lots of GoLang.

@jplatte can we close this issue and you can leave the star?

Yes, I don't see why this would need to be kept open.