Feature request: add an optional parameter to specify the output directory

Hi, developer,
Could you please add an optional parameter in the Trust4 command line, if this parameter is not very hard to add in your code? The parameter can allow users to specify the output directory.
I want this feature because the default directory that the Trust4 writes the output files into is the current working directory. Thereby, each time I have to make a new directory and cd to that directory, so that the Trust4 program files are not messed up with the output files. This operation makes my commands have a lot of hierarchical structure for different folders and files, as are seen in my previous submitted issues.
Thanks a lot.

The option "-o" can include the path. However, you need to create that directory first.

Hi, Li,
Thanks for point out the usage of -o that I did not know previously. Yes. -o ${output_directory}/${output_sample_prefix} works very well, after I create that output directory first. Automatic creation of the output directory will be great. Thanks.
A further thinking: personally, I think letting users to concatenate the ${output_directory}/${output_sample_prefix} is not very elegant. If there are two separate parameters, one for output directory, and one for output sample prefix, it should be more straightforward and need less string manipulation at the user's side. After all, output directory and output file prefix are two different things. In different situations, the user might only want to consider one parameter whereas ignore the other parameter. For example, when I run trust4 in a wdl pipeline using docker image, outputting to the current working directory is perfect, and I am only interested in the output file prefix, because docker and wdl has its mechanism to transfer the output files out of the docker container to an external directory designated by wdl options. By comparison, for example, when I run the trust4 directly on the Linux system, I hope to set up the output directory as well as the output file prefix.
I apologize if my further thinking seems overly picky to you. Anyway, thanks a lot for your contribution to the TCR research community.
Best Wishes.

I've added the option "--od" so you can specify the output directory. This will also create the directory automatically.
As you might notice when checking the updates, we released the source code and you can compile TRUST4 on your system now.
Thanks,
Li

Thanks a lot for your work.


Code of Conduct

When I was working on the issue form (#58), I noticed that good practice is to have people agree with a code of conduct. But, we don't have a code of conduct set yet. I'm creating this issue to set a code of conduct.
As a first step, we want to decide whether we want to (1) create our own code of conduct or (2) adopt one of the available codes of conduct.
(@coglebed & @chartgerink: Do you have any thoughts around this?)
Resources

GitHub: Adding a code of conduct
Open Source Guide: Your Code of Conduct

Example Code of Conducts

Contributor Covenant
Django
R=
BlitzJS
and many more...

Hey @nsunami - thanks for inviting me for input.
If you're able to it is always worthwhile to consider what your values are going into this. I think the contributor covenant is a great and unambiguous starting point, if you're ready to uphold it. It will start the discussion from which you can customize in the future as you learn more and more about these things.
The more people join, the more important a code of conduct is for your community..

I checked out the Contributor Covenant and it looks good and fair to me!

Update: we are moving forward with adopting the Contributor Covenant Code of Conduct for now. Let's close this issue, once the code of conduct gets merged.


public_folder forces leading /

Describe the bug
Using public_folder: "../uploads" I'd assume it would return ../uploads/image.png but the result is ./../uploads/image.png thus resulting in a 404.
The forced leading slash will prevent Gridsome from optimizing the images.
Expected behavior
Using public_folder: "../uploads" should return ../uploads/image.png NOT ./../uploads/image.png.
CMS configuration
backend:
  name: github
  repo: Larsklopstra/gridsome-starter-netlify-cms

media_folder: "uploads"
public_folder: "../uploads"

collections:
  - name: "posts"
    label: "Posts"
    folder: "posts"
    create: true
    slug: "{{slug}}"
    fields:
      - {label: "Title", name: "title", widget: "string"}
      - {label: "Excerpt", name: "excerpt", widget: "string"}
      - {label: "Body", name: "body", widget: "markdown"}

Additional context
Using Gridsome as static site generator.

I'm not exactly a gridsome expert, but the paths in your config need to be absolute. Pretty sure you don't want them to be in root either. So something like:
media_folder: "static/uploads/"
public_folder: "/uploads/

@tomrutgers I want to avoid placing files into the static folder, if I change the article-example.md its contents from ![](/../image.png) to ![](../image.png) then it will work perfectly, so the issue is the forced leading slash. I've tried many different configs.

Related to #325 and #1668.

Resolved by #2958.
Docs here: https://www.netlifycms.org/docs/beta-features/#folder-collections-media-folder


CYC-64 Create InventoryItemController and update function

Description
Creates InventoryItemController class with an update function to handle put requests to edit inventory items.
Closes #60
Changes

Create InventoryItemController.php
Add update function to inventory item controller.
Add route to web.php for update function.
Modified InventoryItem model to guard id.

lgtm


How to specify certain gibson file, also can we extract poses in certain environment?

My task is to save the RGB and depth images from the scene and also to extract pose.
I am able to save RGB and Depth images but I cant see in the documentation anything about poses can I access them or save them.
Currently I am using pointnav_mp3d.yaml as the conifg file but I am unable to figure out how to provide different scene than the default, I have dowloaded GIbson data and want to specify certain gibson file as my scene. How to do it/
I have tried this:
config = habitat.get_config(config_paths='../configs/tasks/pointnav_mp3d.yaml')
config.defrost()
config.DATASET.DATA_PATH = '../data/datasets/pointnav/mp3d/v1/val/val.json.gz'
config.DATASET.SCENES_DIR = '../data/scene_datasets/'
config.freeze()
env = habitat.Env(config=config)
I am changing path of this pointing to my .glb file but its not opening my .glb file:
config.DATASET.SCENES_DIR = '../data/scene_datasets/'
Edit: I have also tried with adding SIMULATOR.SCENE and path to my gibson file but still not working
I would be great if someone can provide insight into this issue,
Thanks

This example is likely a superset of what you are trying to do: https://aihabitat.org/docs/habitat-api/view-transform-warp.html
The easiest way to swap to Gibson is to load the corresponding PointNav dataset, i.e. config.DATASET.DATA_PATH = '../data/datasets/pointnav/gibson/v1/train/train.json.gz'

Thanks for helping it worked but I had one more doubt that I downloaded the gibson with train.json.gz of diffrent environments but there are many more environment in gibson, so is habitat is limeted to that file only or I can choose some other gibson file other than all ready provide.
For example: I want to get the images of akermenviller.glb which is not in the task dataset provided by pointnav Gibson file.

The majority if scenes in gibson contain major reconstruction artifacts, so we didn't provide PointNav episodes for them.  The function to generate new episodes is here: https://github.com/facebookresearch/habitat-api/blob/master/habitat/datasets/pointnav/pointnav_generator.py

If you are just looking to generate images from random view points, using the lower-level API with the simulator may be simpler.  This example shows how to initialize some sensors and render with them: https://github.com/facebookresearch/habitat-sim/blob/master/examples/stereo_agent.py
Calling sim.initialize_agent(0) will set the agent to a new, random location/orientation.  You can call sim.get_sensor_observations() directly to get observations instead of taking a step also.

Thanks for help, yeah habitat sim is much simpler but my goal is to take images from certain specific view points and ulimately do slam in those environments...

The docker image/file was locked to specific versions for compatibility with the 2019 habitat challenge.  You can change the commits to new versions.
You can also remove those version checkouts and make the clones
git clone --branch stable git@github.com:facebookresearch/habitat-sim.git
git clone --branch stable git@github.com:facebookresearch/habitat-api.git
To grab the latest release

Thanks you. I cloned the stable branch but it is not working and having error message as
AttributeError: module 'habitat_sim' has no attribute 'Configuration' when I am trying to run example.py after installing the api

Did you also clone and rebuild habitat-sim?

Yeah I have cloned and rebuild habitat-sim, I have used build.sh to build and also exporting python path but the same error is reappearing

Using python setup.py install and not touching the python path is recommended

Thank a lot that really heped, just one last question how to specify scene in pointnav_generator.py and will the code return the json file data so that I can use it later, also where to specify certain .glb and where the output will be?

This is how I would do it: https://gist.github.com/erikwijmans/1c596183fb8de1e9afd80088b6b5c115

@Toraudonn the generation code will generate the valid episodes, but environment and trajectories would look strange and far from a navigation in real life. Here is an example of such episode:

Closing the issue, feel free to open if you have questions.


variable insertion shortcut not working

Only activates at the start of the line. This is by design.

this is probably a good thing for people typing maths in line, but it's not obvious that that should b the case, no UI hint.


Update primary/symlinked R version to 4.2.2 for the base images

Currently, base images are being built using R_VERSION of 3.6.3 which is also the symlinked R version (causing it to be the default R version in Pro Products further upstream).
Since 3.6.3 is over three years old it would be more appropriate to use anR_VERSION of 4.2.2 instead in base images. 3.6.3 could be R_VERSION_ALT although we may want to consider a more modern version such as 4.1.3
Variables set in the CI here:
https://github.com/rstudio/rstudio-docker-products/blob/main/ci.Justfile

Thanks for bringing this up- we are talking about this internally and trying to determine the best way to do this.  There's some complications around making this change given it would be breaking.  We are trying to think through how this process would work, as it has been a bit complicated to do things like this in the past.   @ianpittwood and @colearendt are currently looking to figure this out.

We are updating the R and Python versions for Workbench in https://github.com/rstudio/rstudio-docker-products/pull/499/files#diff-2f86e223e36828144c47326360c4d6370d690bd460956a43650e0a05c7d6e363R203

We updated R to 4.2.3


Way to make elements pop out of tables with bootstrap 4.1

I have been able to produce list (&lt LI &gt) elements over a very simple table structure right on-top or slightly outside of the table border.  You may download the entire set of html/css files here:
http://chris.404aweso.me/bootstrap_popout_elements.tar.gz
And you may view the actual issue very simply at:
http://chris.404aweso.me/index.php
Please let me know if you require any further assistance.

Chris Porreca

Issues are for bug reports and feature requests, not general how-to questions. For that, we suggest the official Slack channel or Stack Overflow. Both are linked from our readme.
Thanks!

This was not a question -- it was honestly reporting a bug in which I gave
you all the information you could use to see and/or resolve it.
Please visit the website attached to the ticket in order to see what I am
referring to.

Chris

On Sun, Apr 29, 2018 at 6:51 PM Mark Otto<EMAIL_ADDRESS>wrote:

Closed #26410 https://github.com/twbs/bootstrap/issues/26410.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/26410#event-1600722689, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AgZjXtFcY6rUJMJpMwDweJk-uDpK5WHWks5ttkPwgaJpZM4TrPjC
.


DB2 DDL does not work depending on db2 jdbc version

Hello,
Happy new year !
I have discovered that getting the DDL for tables on my db2 database does not work anymore.
I used different DB2 users with not success.
So I tried older db2 jdbc driver version:
It does work with driver 4.25.1301 but starting with 4.26.14 it does not anymore
I use DB2 database version 11.5
Your tool is great , please continue this way
Regards
Alex
System information:

Operating system (distribution) and version
DBeaver version
Additional extensions

Connection specification:

Database name and version
Driver name
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Steps to reproduce, if exist:
Include any warning/errors/backtraces from the logs

Thank you for your report. Unfortunately I do not have an environment to reproduce it. What do you mean when tell "does not work":

Doesn't it appear when you press DDL tab in table properties or when you use Generate DDL option?
Or does it have a wrong syntax?
Do any errors appear?

Please tell me also DBeaver version and send log files https://github.com/dbeaver/dbeaver/wiki/Log-files

Thank you for your report. Unfortunately I do not have an environment to reproduce it. What do you mean when tell "does not work":

Doesn't it appear when you press DDL tab in table properties or when you use Generate DDL option?
Or does it have a wrong syntax?
Do any errors appear?

Please tell me also DBeaver version and send log files https://github.com/dbeaver/dbeaver/wiki/Log-files

Hello,
Thanks for your repply.
I use Dbeaver version 7.3.2.
When I select a table and then "Generate SDL" / DDL, I receive the message :
Routine "*RATE_DDL" (specific name "") has returned an error SQLSTATE with diagnostic text "SQL0551 Reason code or token: DB2SFNP,CREATE TABLE,SYSTOOLS".. SQLCODE=-443, SQLSTATE=38553, DRIVER=4.27.25
I have attached the log file.
Regards
Alex
.log

Hello,
Thanks for your repply.
I use Dbeaver version 7.3.2.
When I select a table and then "Generate SDL" / DDL, I receive the message :
Routine "*RATE_DDL" (specific name "") has returned an error SQLSTATE with diagnostic text "SQL0551 Reason code or token: DB2SFNP,CREATE TABLE,SYSTOOLS".. SQLCODE=-443, SQLSTATE=38553, DRIVER=4.27.25
I have attached the log file.
Regards
Alex
.log

Please tell which driver did you select to connect?

Did you make any driver changes or other additional settings?

Please tell which driver did you select to connect?

Did you make any driver changes or other additional settings?

The db2 jdbc driver which works here a capture

The db2 jdbc driver which does not work here a screen capture

Tell please if you need more informations.

The db2 jdbc driver which works here a capture

The db2 jdbc driver which does not work here a screen capture

Tell please if you need more informations.

No I did not make change or additional settings on the driver.

No I did not make change or additional settings on the driver.

As I see DB2 z/OS works for you database, and DB2 LUW doesn't work. One additional detail is needed: please press Test Connection and send me the capture with the result in popup, like this:

As I see DB2 z/OS works for you database, and DB2 LUW doesn't work. One additional detail is needed: please press Test Connection and send me the capture with the result in popup, like this:

I created a new connection with DB2 z/OS and it works!
Here the captures
DDL OK

DDL not OK

Thanks for your help.
dbeaver ist a great tool!
Regards
Alex

I created a new connection with DB2 z/OS and it works!
Here the captures
DDL OK

DDL not OK

Thanks for your help.
dbeaver ist a great tool!
Regards
Alex

Nice to hear! Can I close the ticket?

Nice to hear! Can I close the ticket?

Yes please,
Thanks for your help!

Yes please,
Thanks for your help!


Title removes html

Hi, would love some help or at least pointing in the right direction.
I am using Magnific-Popup in a Wordpress theme I purchase (came with it). In the popup images from a gallery, I have the Caption showing. But I also have another plugin that attaches the photo's EXIF data to the end of the Caption. I am wanting to make it look pretty, which I can, and add some styling to it with some divs or maybe a table.
When I do this, it works throughout the rest of WordPress for images that are in the content etc. But in Magnific-Popup full screen image, I only get the text. It seems Magnific-Popup strips out the HTML from the caption/title.
I am not an expert with js, but can code some and know enough to google and make adjustments.
My problem is, I cannot find where it strips the HTML in the code. I can't find anything I can change to make it parse the HTML as well.
Can anyone point my in the right direction please?

I suspect it's more likely the HTML is being broken by single or double quotes being mixed incorrectly.
What I do when I need HTML in the title field is create a hidden element and set it with titleSrc
Example: https://codepen.io/ChubbyNinja/pen/KWOrzr
If you provide some markup I may be able to give you a copy/paste solution


Switch branch correctly when there's a directory with the same anme

Fixes #14865 probably. I haven't tested this other than by running the git command manually.

I think @Hixie may have fixed this in https://github.com/flutter/flutter/pull/14507 ?

Being nit-picky: This doesn't check for the existence of a directory with the given name.

@Phrodoh, sorry I don't understand what you mean? It isn't meant to check out a directory...

I didn't see this outstanding PR and submitted the same thing in https://github.com/flutter/flutter/pull/15026.  Closing


[klima data] missing T or percentage on Tokenized credits by bridge

On the off and oh-chain page (https://carbon-git-staging-klimadao.vercel.app/off-chain-vs-on-chain), the 'Tokenized credits by bridge' is missing the % information you find on the details page when you hover over

Yeah, this one is done as per the figma designs (percentage in the details page but not on the main page).
I understand it seems weird. Tagging @theprofessora here.

@jabby09 @biwano okay I've updated the figma. Jabby is right, the pattern should be a more basic label on the summary dashboard page and then %s on the details page, with MTs provided in the hover tooltip.

Hello, @jabby09  @theprofessora,
I created a PR for this issue but it is not complete.
The State of xxx Digital carbon cards, we have tons retired + tons outstanding = tons bridged.
In fact the tons bridged value represent allways 100% of the pie. Actually the pie represents only the tons outstanding and the tons retired. So we cannot use the current figma design for this one.

I created a PR for this issue but it is not complete.
In the State of xxx Digital carbon cards, we have: tons retired + tons outstanding = tons bridged.
In fact the tons bridged value represent always 100% of the pie. Actually the pie represents only the tons outstanding and the tons retired. So we cannot use the current figma design for this one.

@biwano I think you mean the Token details pages (https://carbon-git-carbon-qa1810-klimadao.vercel.app/token-details?tab=c3)?
@theprofessora take another look at that page in the preview link above and see if you have additional recommendations for this page specifically?


Add support for XLIFF (.xlf) file format

What
Implement a loader for XLIFF files in Replexica.
Why
To support developers, LSPs, and translation agencies working on software, web, and mobile app localization.
How

Create a new loader in packages/cli/src/loaders/xliff.ts
Implement parsing logic for XLIFF XML structure
Add two unit tests in packages/cli/src/loaders/index.spec.ts:

Test parsing a simple XLIFF file
Test handling context groups in XLIFF entries

Update @replexica/spec to include XLIFF format
Generate a changeset using pnpm new for both 'replexica' and '@replexica/cli' packages
In the pull request, include a link to the XLIFF specification used.

After completing the task
Noticed a file format Replexica doesn't yet support? Please create a new GitHub ticket (if it doesn't yet exist!), using the format/structure of this GitHub issue.

@maxprilutskiy was working on this using a library xliff but this file does not contain type definition file for the function, hence we need to create modules for type definitions for the functions used
So should I proceed with this approach or you suggest doing something else

assigning to @Nithishvb

this one is up for grabs again!

@maxprilutskiy raised a fresh PR for this one


refactor mergeMyCommandsParams function into a static class

Makes it faster/easier to know what it's going on

Squashed deno fmt into the last commit


A11y_SQLAzureDataStudioMAC_Dashboard_Keyboard: The user is unable to access the graphical details of 'Database Size (MB)' and their exclusive tooltip information using keyboard.

GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#BM_SQLAzureDataStudioMAC_June2022;#SQL Azure Data Studio;#MAC;#FTP;#DesktopApp;#Keyboard;#AINotInScope;#WCAG2.1.1;
Environment Details:
Application Name: SQL Azure Data Studio
Application Version: Version 1.37.0
Mac Version: macOS Monterey 12.4
Repro Steps:

Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility tester after verification of the fix.
Launch SQL Azure data studio.
Connect to a server.
Right click on the server and select 'Manage'.
Dashboard will open.
Tab till 'Database Size (MB)' graph section.
Use keyboard to access the interactive graph details.

Actual:
The user is unable to access the graphical details of 'Database Size (MB)' and their exclusive tooltip information using keyboard.
Expected:
The user should be able to access the graphical details of 'Database Size (MB)' and their exclusive tooltip information using keyboard. Tooltip should close on using escape key and focus should not shift elsewhere. Tooltips should be read by screen readers and should be accessible with keyboard arrow keys.
User Impact:
Keyboard users will lose out important tooltip information associated with the graphs if not accessible to the users.
Attachment:

Since the graph UI is no longer available under the 'Database Size' section. Thus, closing the bug as the new UI that contains a table has no issues.


update examples in README to highlight differences from staticcheck

As pointed out by the golangci-lint team, the previous examples were actually being caught by staticcheck
The updated examples in this PR are a bit more realistic and aren't being detected by staticcheck or other linters included in golangci-lint.

Hi
I followed your exchange with the golangci-lint team.
I'm curious about your project idea.
I'm glad you found new use cases that staticcheck doesn't detect.
But I'm curious about what you plan then.
I mean the things you identified could be added to go-tools static check maybe.

Hi
I followed your exchange with the golangci-lint team.
I'm curious about your project idea.
I'm glad you found new use cases that staticcheck doesn't detect.
But I'm curious about what you plan then.
I mean the things you identified could be added to go-tools static check maybe.

Hello! 👋
Yes, I had a similar thought. It might be better to try and improve an existing linter that is attempting to detect these same types of issues. This was my first attempt at writing a linter and the golangci-lint team questioned the correctness due to the simplistic detection approach. I'm not sure if it will be possible (or welcome) to integrate this type of check in another linter. I've asked the golangci-lint team for their opinion as well since I'm still a bit of n00b in this space. If you have any thoughts / advice I'm definitely open to it. Thanks!

I think a good approach could be something similar to what you imagined.
Having your linter coded and tested, then ask for static check team to review it, and consider adding it to their linter, code is easy to adapt once you coded everything and have the unit tests ready.
Static check release cycle is slow, but then when made available it becomes available via gopls, so any IDE gets it.
I'm a huge golangci-lint fan, also.
I think it's better to include your code in something bigger, than in golangci-lint that enable quite no linter by default except static check.
So for me, as a long term logic, trying to add it to static check is more perene.
There are so many linter available in golangci-lint that people never enabled, nor heard about

Thank you for the thoughtful advice! I really appreciate it.
I'll check-in with the staticcheck project to see if they're interested.
Thanks again for your interest and support!

Just to close the loop on this: here is the staticcheck issue inquiring if they'd be interested in this type of functionality in staticcheck directly.

Thanks

Apparently, staticcheck and golangci-lint team declined the need for your linter.
Their arguments are quite good

Apparently, staticcheck and golangci-lint team declined the need for your linter.
Their arguments are quite good

Haha, yep. I thought it might be helpful to others but I also believe those folks know what they are talking about.


🎉 1.1.1

ℹ About this release

Version: 1.1.1
Type: patch
Last Release: 1.1.0 (6/1/2022, 9:51:26 PM) [?]
Commits to merge: 18 [?]

📦 Dependencies

Update @prisma/client from ^3.14.0 to ^3.15.2
Update prisma from ^3.14.0 to ^3.15.2

📦 Development Dependencies

Update @amanda-mitchell/semantic-release-npm-multiple from ^3.2.1 to ^3.5.0
Update @types/node from ^17.0.31 to ^17.0.42
Update @typescript-eslint/eslint-plugin from ^5.22.0 to ^5.27.1
Update @typescript-eslint/parser from ^5.22.0 to ^5.27.1
Update eslint from ^8.15.0 to ^8.16.0
Update eslint-plugin-jsonc from ^2.2.1 to ^2.3.0
Update typedoc from ^0.22.15 to ^0.22.17
Update typescript from ^4.6.4 to ^4.7.2

:tada: This PR is included in version 1.1.1 :tada:
The release is available on:

v1.1.1
GitHub release
Docker container (sebbo2002/fhem-log2db:1.1.1)
v1.1.1

Your semantic-release bot :package::rocket:


HOPPINGWINDOW will cause data to be set to nil

Environment:

eKuiper version : lfedge/ekuiper:1.8.2-slim
OS (e.g. cat /etc/os-release):  centos 7

What happened and what you expected to happen:

I encountered a problem when using the HOPPINGWINDOW function. When the first piece of data just enters the window, it will be output normally. When the second piece of data enters the window, the output will set the first piece of data to nil. When the data is connected or sent intermittently This phenomenon will occur when the window size is satisfied, and the output will be normal

How to reproduce it (as minimally and precisely as possible):

Stream

curl --location 'http://<IP_ADDRESS>:9081/streams' \
--header 'Content-Type: application/json' \
--data '{"sql":" CREATE STREAM RuleWarn ( id string, scenes_id string, scene_version string, area_id string, detection_tag string, warn bigint, warn_row array(string), warnIndex string, image_collect array(string), image_collect_row array(string), event_time bigint) WITH (DATASOURCE=\"edgex/rule002-4\", FORMAT=\"json\", CONF_KEY=\"local-mqtt\", TYPE=\"mqtt\", SHARED=\"true\", ); "}'

Data

{"area_id":"7c89f893-fe77-403f-8409-02fd177a9280","detection_tag":"a0e08144-b20d-41b7-873e-ad9515937609","event_time":1680057074,"id":"dac1edf3-cdd9-11ed-9288-0242ac160008","image_collect":["6a12723b-8907-45bc-8cf0-3e16898acf98","53356f5f-e961-442f-a432-42342bd3f99a"],"image_collect_row":[["6a12723b-8907-45bc-8cf0-3e16898acf98","53356f5f-e961-442f-a432-42342bd3f99a"],["f0b7a9d2-7ce0-4b7c-bb25-2c09b1556870","bcca9e24-b940-4ff9-9926-4f8223d90c5c"],["80eefd59-d611-4696-b81b-9993ded1f5ae","3d33ceb9-259c-4c80-be1e-5f0574f01749"]],"scene_version":"v1","scenes_id":"16c14ecd-0731-465d-871e-84500b8f4922","warn":1,"warnIndex":0,"warn_row":[1,1,1]}

Rule SQL

select
  newuuid() as id,
  count(1) as count_num,
  scenes_id, scene_version, area_id,
  collect(image_collect)[0] as image_collect,
  collect(event_time) as event_time_collect,
  collect(case when event_time - lag(event_time,1,0) = 12  then 1 else 0 end ) as alarm_counts3
from RuleWarn
where
  RuleWarn.warn > 0
group by
    scenes_id, scene_version, area_id, HOPPINGWINDOW(ss, 60, 12)

The current scene of 5 consecutive triggers can be realized in another way
select
  newuuid() as id,
  count(1) as count_num,
  scenes_id, scene_version, area_id,
  collect(image_collect)[0] as image_collect,
  case when sum(case when event_time - lag(event_time) = 12  then 1 else 0 end ) = 5 then 1 else 0 end as warn,
  min(event_time) as event_time,
  collect(event_time) as event_time_collect
from RuleWarn
where
  RuleWarn.warn > 0
group by
    scenes_id, scene_version, area_id, COUNTWINDOW( 5, 1)


deps: remove debug

It is extraneous dependency if you do not need debug. Even for debug, it would be better to have events system instead of using debug package.
Express uses the debug package (ref) for this purpose, but this package is not express-only package.

Closing it for now - see discussion at pillarjs/finalhandler#4

Yea, this module is not being used. It's the de-facto debugging module in the Node.js community (since Node.js core does not offer such a feature) and your PR will prevent people from ever debugging what is happening inside this module.
An "events system" is not viable, because this module may be used deep, deep in the dependency chain and people cannot access a reference to this module.
If there is some mass exodus from using this module across npm, then maybe if there is an alternative, but I have not seen such a thing.


Xcode 16 Beta 4 Support, Issue with packaged LLama dependency

Hello, when attempting to use the latest build in Xcode 16 Beta 4, I pulled the latest commit, in order to grab the recent fix from here: https://github.com/eastriverlee/LLM.swift/commit/5651e8a39309e8cb8edfdd0bee664d77caf964d3
However, by also pulling in
https://github.com/eastriverlee/LLM.swift/commit/e50b98ea3909cf269a1de3127a41b0aaa1ef7f01
I ended up with a new error that showing the included LLama.cpp interface was incompatible with this change.
"cannot be captured by an escaping closure since it is a borrowed". Somehow I ended up with a months old out of date LLama.cpp dependency reference. Even though oddly the package does seem to point towards the master branch.
I was able to fix it by manually removing this package, manually adding llama.cpp, pointed at the latest commit, and then redownloading this package.
So I think there might need to be some fix here to how LLama is included??
Or maybe its just xcode ghosts doing weird stuff??
Figured I would share in case anyone else runs into this, but feel free to close, its not longer a blocker after my fix of manually including llama.cpp

@nickdobos Grimore couldn't figure it out? :P JK. Nick, there's another branch on here called 'Pinned', it has a specific commit of Llama.cpp that fixes this. I'll put up a PR to request it to be merged. I have my separate fork where I included it.

The problem is that this repo doesn't include the package.resolved and the Package.swift just specifies 'master' for llama.cpp. So, when you build the repo it just uses whatever is latest at that time. I have PR here where I migrated this repo to the latest version of llama.cpp and also included the package.resolved that built successfully so you, and anyone/thing else, will build exactly the same.

mostly, removing Package.resolved or using fixed branch will solve issues like this.
reading my reply to garfbargle's PR might help you understand this issue better:

hi, thank you for your kind contribution. i know it is recommended to add Package.resolved, but i deliberately left out Package.resolved file, so that i don't have to update this every time when llama.cpp updates which is very frequent.
to compensate for this issue, this repo has a fixed branch that points to a specific commit of llama.cpp. people who want to use latest llama.cpp will just have to use main branch and people who want stability will use fixed.
if this repo is very famous, and have a lot of contributors, i may consider adding Package.resolved knowing that it will be updated frequently, but it isn't. i hope you understand this compromise.


Feature Request: Add weekly B-rank hunts

I don't know if this feasible since they don't appear in the game's Timers menu, but it would be really nice to see from Accountant if you've already done the B-rank of an expansion this week.

The Daily Duty plugin can do that.


export code from explorer on tfgrid [development_actorsimprovement]

see new export code on bcdb
on branch: development_actorsimprovement
jumpscaleX_core/JumpscaleCore/data/bcdb/BCDB.py:148

need to export the data to a dir on our explorer (do not upgrade!), just merge the export functionality in.

[x] complete export, import functions

[ ] complete script

[ ] then on another machine running on this branch finish the import feature, see it works and the data is there.

explorer: explorer.testnet.grid.tf

https://github.com/threefoldtech/jumpscaleX_threebot/tree/development_actorsimprovement/ThreeBotPackages/tfgrid/backup_explorer

current status: we did fixes for import export functions and managed to export/import bcdb namespace successfully

Bugs fixed here https://github.com/threefoldtech/jumpscaleX_core/commit/25444d4f0b36fec9159e0b70dc1e77272989392a and here https://github.com/threefoldtech/jumpscaleX_core/commit/957e33fffb9ee6fc16b8193bf8f4f910fda0e932

backup script fixes here https://github.com/threefoldtech/jumpscaleX_threebot/commit/ba9e2a86f0f97332b1f404b1d5e8b20314c0d229

More fixes here https://github.com/threefoldtech/jumpscaleX_core/commit/d5cff66ff4bc9c3fec113eb20a43e23654631cd0

I modified the restore to use the new namespaces according to the package.toml of each package and use zdb. But still didnt manage to test it. I am getting this error from zdb, which @andrewayoub is also investigating.

@despiegk this part of this commit breaks importing data from explorer
https://github.com/threefoldtech/jumpscaleX_core/pull/284/commits/f94673744f767a0b9e1c3fd3630170f4247417a2#diff-fe139c4b54d2339e936e7dd043098aaaR456
We have models that have indexed columns (other than name) that don't have values

I commented the code mentioned above, so I can continue with my tests. But had some issues with ids in zdb, the first entry in zdb (after the meta) always started at 18. After lots of debugging, and after having some discussions with maxime, it turns out zdb increments the id internally even if you repeatedly update a specific id. And it seems metda data is reepatedtly updated before we create the first objct, as a result it gets created at 18 instead of 1. I am checking what can be changed to avoid this.

waiting till we have [development_actorsimprovement] stable enough

Sarah working on a parallel track until completely blocked (will be very soon).

Because of the issue I mentioned in my previous commands, I am trying to add a way to add preedefined meta data during the creation of bcdb so it doesnt keep updating zdb if the metadata is there and increase the index. I did the changes locally, but so far it broke a lot of things and I am still trying to work it out.

christophe is ok to restart the grid deployment

@despiegk ok cool. But let's keep this issue in mind. We'll always have a problem with exporting/importing from zdb like I mentioned above.

I added a gist here with some of the changes I was trying to test for future reference
https://gist.github.com/serboctor/f1182b2e7c9b9a6a7a45da5dfa8a83e3


Saturn.Extensions.Authorization broken in release 0.11.0

After upgrading to the version 0.11.0 the use of use_github_oauth in application CE leads to cryptic:
This control construct may only be used if the computation expression builder defines a 'For' method
It appears that it's caused by name collision with Microsoft.AspNetCore.Builder.ApplicationBuilder in OAuth extensions.

Fixed in 0.12.0


[bug] App crashes when I add something to the allowlist and compile it.

Describe the bug
When I tried to allow the "dialog" module and then started my application I didn't get any errors while it downloaded and compiled or whatever but then it just crashed after it looked like it loaded normally with no errors returned. I found removing dialog from the allowList was the only way that it would start properly. This applies for any module
Reproduction

Added whatever module (in this case dialog same issue occured with another module) to the allowList it just has to not have been downloaded yet
Run the app
Loads like normal then just crashes (the UI doesn't even load for a frame)

Expected behavior
No response
Full tauri info output
[✔] Environment
    - OS: Windows 10.0.19045 X64
    ✔ WebView2: 126.0.2592.81
    ✔ MSVC: Visual Studio Professional 2022
    ✔ rustc: 1.79.0 (129f3b996 2024-06-10)
    ✔ cargo: 1.79.0 (ffa9cf99a 2024-06-03)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (environment override by RUSTUP_TOOLCHAIN)
    - node: 22.3.0
    - npm: 10.8.1

[-] Packages
    - tauri [RUST]: 1.6.2
    - tauri-build [RUST]: 1.5.2
    - wry [RUST]: 0.24.8
    - tao [RUST]: 0.16.9
    - tauri-cli [RUST]: 1.5.14
    - @tauri-apps/api [NPM]: 1.5.6 (outdated, latest: 1.6.0)
    - @tauri-apps/cli [NPM]: 1.5.14 (outdated, latest: 1.6.0)

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:3000/
    - framework: React
    - bundler: Vite

Stack trace
No response
Additional context
Terminal of when I tried running it
PS E:\Coding\path1\path2> cargo tauri dev
     Running BeforeDevCommand (`npm run dev`)

    ><EMAIL_ADDRESS>dev
    > vite

      VITE v5.3.1  ready in 120 ms

      ➜  Local:   http://localhost:3000/
      ➜  Network: use --host to expose
            Info Watching E:\Coding\Thugalicous\Xcel-Loader\src-tauri for changes...
        Finished `dev` profile [unoptimized + debuginfo] target(s) in 0.32s
    ⠙
   PS E:\Coding\path1\path2>  // this is where it crashed

Please provide a minimal reproduction, as I can't seem to be able to reproduce. Also make sure your JS packages are up to date.

So I tried updating JS packages and checked my Crates which didn't change a thing but updating my build.rs to not apply my windows attributes let it load. The new issue is that my app requires elevated permission to function.
Here is my build.rs
fn main() {
  let mut windows = tauri_build::WindowsAttributes::new();
  let manifest = include_str!("manifest.xml");

  windows = windows.app_manifest(
    manifest
  );

  let attrs =  tauri_build::Attributes::new()
    .windows_attributes(windows);
  tauri_build::try_build(attrs).expect("failed to run build script");
}

And manifest.xml
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
    <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
        <security>
            <requestedPrivileges>
                <requestedExecutionLevel level="requireAdministrator" uiAccess="false" />
            </requestedPrivileges>
        </security>
    </trustInfo>
</assembly>

Just by tinkering around with it only when I commented out the
.windows_attributes() line from when I create the attributes fixed it which proves the issue is with either the manifest or how I am creating the attributes (although the manifest was straight from tauri docs)

Since you mentioned the dialog, it seems like you skipped over the warning here: https://docs.rs/tauri-build/latest/tauri_build/struct.WindowsAttributes.html#warning so your manifest should look more like this
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
 <dependency>
   <dependentAssembly>
     <assemblyIdentity
       type="win32"
       name="Microsoft.Windows.Common-Controls"
       version="<IP_ADDRESS>"
       processorArchitecture="*"
       publicKeyToken="6595b64144ccf1df"
       language="*"
     />
   </dependentAssembly>
 </dependency>
  <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
    <security>
      <requestedPrivileges>
        <requestedExecutionLevel level="requireAdministrator" uiAccess="false" />
      </requestedPrivileges>
    </security>
  </trustInfo>
</assembly>

If it's not that, then it probably doesn't like that it requires admin in dev mode - maybe try running tauri dev in an admin terminal then (not a general recommendation of course - running stuff in admin terminals is pretty meh)

Since you mentioned the dialog, it seems like you skipped over the warning here: https://docs.rs/tauri-build/latest/tauri_build/struct.WindowsAttributes.html#warning so your manifest should look more like this
<assembly xmlns="urn:schemas-microsoft-com:asm.v1" manifestVersion="1.0">
 <dependency>
   <dependentAssembly>
     <assemblyIdentity
       type="win32"
       name="Microsoft.Windows.Common-Controls"
       version="<IP_ADDRESS>"
       processorArchitecture="*"
       publicKeyToken="6595b64144ccf1df"
       language="*"
     />
   </dependentAssembly>
 </dependency>
  <trustInfo xmlns="urn:schemas-microsoft-com:asm.v3">
    <security>
      <requestedPrivileges>
        <requestedExecutionLevel level="requireAdministrator" uiAccess="false" />
      </requestedPrivileges>
    </security>
  </trustInfo>
</assembly>

If it's not that, then it probably doesn't like that it requires admin in dev mode - maybe try running tauri dev in an admin terminal then (not a general recommendation of course - running stuff in admin terminals is pretty meh)

Yeah that seemed to fix it, Thank you!


The position being added (Offset(XX, XX)) has a smaller timestamp (XX)than its predecessor: _PointAtTime(Offset(XX, XX) at XX.

Run my App.
Go to a Flutter page.
Scroll the Flutter page.

Expected results:
No errors occur.
Actual results:
An error occurred while scrolling the page.
One error detail:The position being added (Offset(124.0, 236.0)) has a smaller timestamp (937:34:53.275485)than its predecessor: _PointAtTime(Offset(124.0, 236.5) at 449217:37:44.816003).

Logs

#0      IOSScrollViewFlingVelocityTracker.addPosition.<anonymous closure> (package:flutter/src/gestures/velocity_tracker.dart:305:7)
#1      IOSScrollViewFlingVelocityTracker.addPosition (package:flutter/src/gestures/velocity_tracker.dart:309:6)
#2      DragGestureRecognizer.handleEvent (package:flutter/src/gestures/monodrag.dart:280:15)
#3      PointerRouter._dispatch (package:flutter/src/gestures/pointer_router.dart:93:12)
#4      PointerRouter._dispatchEventToRoutes.<anonymous closure> (package:flutter/src/gestures/pointer_router.dart:138:9)
#5      _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:397:8)
#6      PointerRouter._dispatchEventToRoutes (package:flutter/src/gestures/pointer_router.dart:136:18)
#7      PointerRouter.route (package:flutter/src/gestures/pointer_router.dart:122:7)
#8      GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:439:19)
#9      GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:419:22)
#10     RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:288:11)
#11     GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:374:7)
#12     PointerEventResampler._samplePointerPosition (package:flutter/src/gestures/resampler.dart:264:15)
#13     PointerEventResampler.sample (package:flutter/src/gestures/resampler.dart:297:7)
#14     _Resampler.sample (package:flutter/src/gestures/binding.dart:136:17)
#15     GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:331:18)
#16     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:296:7)
#17     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:279:7)
#18     _rootRunUnary (dart:async/zone.dart:1370:13)
#19     _CustomZone.runUnary (dart:async/zone.dart:1265:19)
#20     _CustomZone.runUnaryGuarded (dart:async/zone.dart:1170:7)
#21     _invoke1 (dart:ui/hooks.dart:186:10)
#22     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:282:7)
#23     _dispatchPointerDataPacket (dart:ui/hooks.dart:96:31)

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel unknown, 1.27.0-8.0.pre, on Mac OS X 10.15.7 19H15 darwin-x64, locale zh-Hans-CN)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio (version 4.0)
[✓] VS Code (version 1.54.3)
[✓] Connected device (2 available)

• No issues found!

Hi @LeoFavor1562

Run my App.
Go to a Flutter page.
Scroll the Flutter page.

Your instructions are not clear
Can you please provide a minimal complete reproducible code sample and flutter run -v running this code sample?
Also, make sure to provide step by steps to reproduce the issue
Thank you

Hi @TahaTesser

Hi @TahaTesser
This should be flutter_swiper's problem.
Thanks for your answer.


chore: remove peer dependencies

SUMMARY:
Peer dependencies are too strict and possibly unnecessary.
GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
CHANGE TYPE:

[ ] Bugfix Pull Request
[ ] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@3859587). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage        ?   89.43%
=======================================
  Files           ?      481
  Lines           ?    12988
  Branches        ?     4368
=======================================
  Hits            ?    11616
  Misses          ?     1347
  Partials        ?       25

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Running gives unexpected keyword argument 'personalization_config'

Traceback (most recent call last):
  File "scripts/stable_txt2img.py", line 292, in <module>
    main()
  File "scripts/stable_txt2img.py", line 195, in main
    model = load_model_from_config(config, f"{opt.ckpt}")
  File "scripts/stable_txt2img.py", line 31, in load_model_from_config
    model = instantiate_from_config(config.model)
  File "/Users/adam/workspace/stable-diffusion/ldm/util.py", line 85, in instantiate_from_config
    return get_obj_from_str(config["target"])(**config.get("params", dict()))
  File "/Users/adam/workspace/stable-diffusion/ldm/models/diffusion/ddpm.py", line 448, in __init__
    super().__init__(conditioning_key=conditioning_key, *args, **kwargs)
TypeError: __init__() got an unexpected keyword argument 'personalization_config'

Recreated env and that solved this problem.

Recreated env and that solved this problem.

hello, I meet the same question, do you remember how to recreated the env to solve the problem?


Prettier 2.7.1 with types

About

https://github.com/crowi/crowi/pull/827 was failed on GitHub Actions

Bump @types/prettier

unexpected commit tree


Questionmark

Coverage decreased (-0.1%) to 91.148% when pulling b4f2e2a1b65a5589614a4267800a7b211379e512 on questionmark into 087cee7404d8504e69c4f6fb41ecca2304060672 on master.

@homu r+

:pushpin: Commit b4f2e2a has been approved by kbknapp

:zap: Test exempted - status


edge: add support for canTrickleIceCandidates

as described in https://developer.mozilla.org/en-US/docs/Web/API/RTCPeerConnection/canTrickleIceCandidates

wait a second, i'm rewriting the test...

Are the unhandled promises in the unit test related to this?
https://travis-ci.org/webrtc/adapter/jobs/218795863#L1000-L1002

whoops. Should be an async test...

Right, you were missing done in the it statements. Fast fix :)


Merge/add multichannel masks

Added multichannel mask support to apply_mask - if a channelless or single-channel mask is provided it'll apply it to all channels of the dataset, whereas if a mask with multiple channels is provided, it'll apply each channel of the mask to the corresponding channel individually.

Is it safe to apply a single-channel mask across all channels if no specific mask is designed for each individual channel?

If you enhanced the functionality of apply_mask, shouldn't you also update the modified function's documentation?

Is it safe to apply a single-channel mask across all channels if no specific mask is designed for each individual channel?

This is the previous behaviour of the function - if you give it a mask with channels, it just applies channel 0 to everything

If you enhanced the functionality of apply_mask, shouldn't you also update the modified function's documentation?

Updated


Travis: Add test for Xenial.

3.5 is chosen because it is the default version of python3 in Ubuntu 16.04.

I think this can replace the 3.4,14.04 backend test, given that most people running on 14.04 would use the default of 14.04 (3.4).

Agreed, this is strictly better than that test.  But @rht have you actually confirmed that key causes it to run on Xenial?  I don't see obvious evidence of that happening in the logs for this PR...

@timabbott  https://travis-ci.org/zulip/zulip/jobs/287924953#L9 is one evidence the test is on Xenial.

Wait ...https://travis-ci.org/zulip/zulip/jobs/287924953#L18 says otherwise.

I think it is either all Trusty or all Xenial. The other option is to trigger build in another fork which travis.yml contains dist:xenial and on all python3.5.

@rht can you contact Travis support and ask them if there's a way for a project to run some builds on each?  This seems like something we shouldn't have to be working around.

On it.

As per https://github.com/travis-ci/travis-ci/issues/5821#issuecomment-214452987, https://github.com/travis-ci/travis-ci/issues/5821#issuecomment-243617289, the only way to do 16.04 CI test is to fetch a docker image and run the test on Docker.
This is possible only after the 1.7 release, when the prod Docker can be finally used.


Coordinate Conversion: UTM rounding issue (1 meter off)? (ArcGIS Pro)

@nfeuerstein When I was verifying the fix to #157 I found that the output UTM coordinate in some test cases was 1 meter off then what was expected (see https://github.com/Esri/coordinate-conversion-addin-dotnet/issues/157#issuecomment-271653373). If I remember correctly it was never 1 meter off in both the easting and northing. It was either off on one or the other coordinate components and didn't happen every time.
At the time, I was thinking that this was a rounding issue. @jmhauck was starting to look into this. I think we can live with this issue for this release.

@elinz please verify

@topowright I'm not sure what you're asking me to verify. There have been no fixes. There wasn't going to be.

@topowright For an example, see my comment https://github.com/Esri/coordinate-conversion-addin-dotnet/issues/158#issuecomment-270993622

This is still an issue:

Closing duplicate of #546


Adjust index if data item is removed

I want to use this component to let a user navigate through a (potentially endless) stream of items and as such I am adding new items to the data array as the user nears the end of the list and this works just perfectly. Obviously though I can't just keep adding items without removing older ones forever as this will exhaust memory at some point.
Removing items from the start of the data array seems to work pretty well except for the fact that it seems to act very strange with the index system. The problem is that the index doesn't change and after navigation there is a huge jump to the position of the old index+1. It would be nice if the component could automatically deal with this or provide me with a way to decrement the index without visually navigating.

Hi @Gerharddc,
I totally get what you're after, but this might be tricky to implement.
Would you mind providing me with a simple example that would serve as a starting point to work on this feature? Something based on the provided example maybe...

@bd-arc I'll have to see about getting it integrated into the example but I can already give you the relevant code that I tested with yesterday:
let idx = 0;

export default class Test extends React.Component {
	private getViewport() {
		return {
			width: Dimensions.get('window').width,
			height: Dimensions.get('window').height,
		};
	}

	state = {
		viewport: this.getViewport(),
		lys: [
			{ name: 'One' },
			{ name: 'Two' },
			{ name: 'Three' },
			{ name: 'Four' },
		],
	};

	private updateDimensions() {
		this.setState({
			viewport: this.getViewport(),
		});
	}

	private renderItem({ item, index }: { item: IItem, index: number }) {
		return (
			<View>
				<Image source={require('../../assets/img/cat.jpg')}
					style={{width: this.state.viewport.width - 100, height: this.state.viewport.width - 100}}/>
				<Text>{item.name}</Text>
			</View>
		);
	}

	private handleIndexChange(index: number) {
		const lys = this.state.lys.slice();

		if (index >= (lys.length - 2)) {
			idx++;
			lys.push({ name: idx.toString() });
		}

		if (index >= 5) {
			lys.splice(0, 1);
		}

		this.setState({ lys });
	}

	public render() {
		return (
			<View style={{ flex: 1 }}
				<Text>Flow</Text>
				<Carousel
					data={this.state.lys}
					renderItem={this.renderItem.bind(this)}
					sliderWidth={this.state.viewport.width}
					itemWidth={this.state.viewport.width - 100}
					onSnapToItem={this.handleIndexChange.bind(this)}
				/>
			</View>
		);
	}
}

The slightly strange syntax is just TypeScript.
This code adds a new item just before you reach the end of the carousel and also removes an item for each one that it adds. If you run this and try to scroll to the next item after 1 has been 1 then it will suddenly jump another item ahead because the index of the item that you were looking at before the interaction has actually moved 1 down. If you were to remove more than 1 item at a time this effects just becomes worse.

Hi @Gerharddc,
I've gave your issue some thought, but I don't see an easy way to deal with it (particularly with the recent loop addition which just made index handling even trickier). There is no definitive way of knowing the index that should be returned by a particular item after various additions/removals.
Do you have any suggestion about how this problem might be tackled?

I can definitely notify the carousel of exactly how many items I will be removing before, after or during the change. In this way it should be able to adjust adjust the internal index accordingly. Unfortunately I am unfamiliar with the code-base and do not know if this is something that can easily be implemented.
In theory I could get away with just a method to notify the carousel to adjust the index backwards by a certain amount.

@Gerharddc Thanks for the feedback! I'll see if I can think of something that doesn't break the existing logic.
FYI, index handling already consists of more than 100 lines of code...

I've been able to work around my issue by using this._carousel.snapToItem(index + 1, false), maybe it can help you!

Hi @m-vdb,
I didn't have time to work on this issue lately, but if the problem is spreading I'll try to take a look at it shortly. Not before next week though, since I currently have major professional deadlines to meet ;-)

having the same issue, when deleting the last item a blank slide was displayed until you started scrolling again. Solved it by adopting @m-vdb solution, but it feels hacky.
Still, great job and thanks for this lib.
componentDidUpdate(prevProps, prevState) {     if (this.props.data.length < prevProps.data.length) {       this._carousel.snapToItem(this.props.passengers.length, false);     }   }

If use same data, or when the data length is 1 and we add data items, there will be buggy too, inspired by @raduflp , I use following hack code to treat these situations
// for same data buggy situation, use only length change or data change time
if (!isEqual(oriData, data)) {
    // for length is 1 situation
    this.setState({ data: data.length === 1 ? data.concat(data) : data }, () => {
        this._carousel.snapToItem(data.length, false)
    })
}

Ran into this problem as well.
I gave it some thought and I came up with this:
        const predicate = '??';
        const prevIndex = prevProps.data.findIndex((value, index) => value === predicate);
        const nextIndex = this.props.data.findIndex((value, index) => value === predicate);
        const delta = nextIndex - prevIndex;
        this._activeItem += delta;

I think the right place in the code would be here.
I tested a simple static implementation (adding one child) => it worked:
             if (this._previousItemsLength !== itemsLength) {
                nextActiveItem = this._previousItemsLength < itemsLength ? nextActiveItem + 1 : nextActiveItem;
                this._hackActiveSlideAnimation(nextActiveItem, null, true, false);
            }

However this is far from a viable solution.
Implementing keys as suggested by @Gerharddc would solve the predicate value. and would make life easier for everyone, so it sounds.
I tried testing a better implementation of this but ran into a VERY strange bug. When state/props update after mount prevProps === this.props. I have no clue why this is happening. And have no idea how to work around it.
While testing I ran into a bug in the code (I think), that overrides the _hackActiveSlideAnimation method.
I'll submit a PR.

Sorry, please allow me to advertise for my open source library! ~
I think this library react-native-reanimated-carousel will solve your problem. It is a high performance and very simple component, complete with React-Native reanimated 2

experiencing the same issue, any update about this?

sorry, but did you find any solution about this? @Gerharddc


Change Get-AzureRmSubscription to Get-AzSubscription

Get-AzureRmSubscription : The term 'Get-AzureRmSubscription' is not recognized as the name of a cmdlet, function,
script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is
correct and try again.

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

ID: 8ef7143a-d3c7-5240-f7d6-4127f876d620
Version Independent ID: ca7d09a1-946b-c7fd-d659-0995c6910b20
Content: Manage Azure subscriptions with Azure PowerShell
Content Source: docs-conceptual/azurermps-6.13.0/manage-subscriptions-azureps.md
Service: azure-powershell
GitHub Login: @sptramer
Microsoft Alias: sttramer

You're looking at the documentation for AzureRM, as evidenced by the URL (https://docs.microsoft.com/en-us/powershell/azure/azurerm/manage-subscriptions-azureps?view=azurermps-6.13.0&). You want the documentation for the current release, located at https://docs.microsoft.com/powershell/azure/manage-subscriptions-azureps. Documentation in the azurerm path is for archive purposes only while users migrate from AzureRM to Az.


usage with tokio panics

when using this package with tokio runtime and calling gauth::serv_account::ServiceAccount::from_file, I have the following error: thread 'main' panicked at 'Cannot drop a runtime in a context where blocking is not allowed. This happens when a runtime is dropped from within an asynchronous context.',
Steps to reproduce

create a new rust app using cargo init gauth-bug
add the package dependency using cargo add gauth
copy the test file service-account-key.json from the package's files and put it in the root of the new rust app
call from_file like the following:

gauth::serv_account::ServiceAccount::from_file(
    "service-account-key.json",
    vec!["https://www.googleapis.com/auth/firebase.messaging"],
);

when running the application this way, it will work without any trouble
now, let's add tokio runtime. firstly, add the tokio package using cargo add tokio
replace the main function with the following:

#[tokio::main]
async fn main() {
    gauth::serv_account::ServiceAccount::from_file(
        "service-account-key.json",
        vec!["https://www.googleapis.com/auth/firebase.messaging"],
    );
}

when running the application this was, it will panic with the error message I've provided above

as suggested in this stackoverflow answer, running the from_file function on a seperate thread fixes the issue
use std::thread;

#[tokio::main]
async fn main() {
    thread::spawn(|| {
        gauth::serv_account::ServiceAccount::from_file(
            "service-account-key.json",
            vec!["https://www.googleapis.com/auth/firebase.messaging"],
        );
    })
    .join()
    .expect("Thread panicked")
}

@adnanjpg thank you pointing this out. Let me test the scenarios you mentioned and hopefully we can find a good solution. So far I've been using the the serv_account module in this project with tokio runtime https://github.com/makarski/mstream/blob/main/src/cmd/listener.rs#L24-L25. However, the dependency there is gauth = 0.4 that is using a version of reqwest before async was introduced.
Before the issue is fixed, a version 0.4 will do, as the changes introduced in 0.5 affect the desktop app auth - there i also updated the reqwest client that introduced the issue.

Also always looking forward to have a PR opened

here's the pr https://github.com/makarski/gauth-rs/pull/26


Unable to deploy to GameGo more than once without restarting and refreshing

Describe the bug
I can deploy a sketch to the device successfully once. After that, I have to restart the device and refresh the browser to deploy again.
(Originally discussed in https://forum.makecode.com/t/have-to-restart-tinkergen-gamego-and-reload-page-to-download-more-than-once/17136.)
To Reproduce

Create any sketch
Plug in a TinkerGen GameGo and turn it on
Select "Find hardware"
Select "Connect device"
Select "Download". Sketch is now running on device.
Modify the sketch in some visible way
Select "Download" again

Expected behavior
The updated sketch should be flashed to the device, and the editor should become responsive again.
Actual behaviour

The sketch is not flashed.
The UI hangs, showing the "Downloading..." overlay:

After 120 seconds that times out.

The browser downloads a .uf2 file to the local PC's disk.

Desktop:

OS: Windows 11 Version 22H2 (Build 22621.963)
Browser: Edge 110.0.1556.0 (Official build) dev (64-bit)

Additional context
I also tried this from my laptop and hit the same issue.
The device identifies as a TinkerGen F4 v2.7.5. I cannot find a newer firmware version (or any, for that matter).

FWIW I'm an FTE and am happy to discuss/chat via Teams internally if that'd help. I'm in Australia, which overlaps with Seattle's afternoons.

@drewnoakes thank you for the feedback. We will investigate this week and get back to you if we have additional questions.
Thanks for the support.


Orchard.Widgets admin permissions check

AdminController - adding security permission check on index action.

Please target 1.9.x as 1.8.x is deprecated.

Can't see branch 1.9.x in my fork. Any ideas?

Fetch all on the repository.
checkout 1.9.x.
Apply your changes.
commit and push your changes
Create the pull request from that branch.


Fixed pathnames for COLUMN EXPR (MLDB-1779)

It was assuming strings and using columnName() not columnPath() as the default AS

+1


Error on Windows 11 with git bash as shell

In my windows configuration I use use git bash, which provides bash.exe. In neovim I have set vim.opt.shell = "bash". With this configuration the Z command doesn't work and gives error: bash: /s: No such file or directory. The Zi command works.
OS: windows 11
NVIM v0.10.0

Does settings the 'shellcmdflag' option to -c fix it?

No fix but a different error: bash: line 1: zoxide query --exclude C:UsersjanpeCodereposdotarchnvim.confignvim dot:command not found
The path I was going to was C:\users\janpe\code\repos\dot\arch\nvim\.config\nvim

I found the shellslash option and set it to true and now the path has forward slashes. Still an error:
bash: line 1: zoxide 'query' '--exclude' 'C:/Users/janpe/Code/repos/dot/arch/nvim/.config/nvim' 'dot': No such file or directory
I find it weird that it gives an error because the same command in a bash shell returns the path.
$  zoxide 'query' '--exclude' 'C:/Users/janpe/Code/repos/dot/arch/nvim/.config/nvim' 'dot'
Output: C:\Users\janpe\Code\repos\dot

I got it to work with this configuration :
vim.o.shell = '"C:/Program Files/Git/bin/bash.exe"'
vim.o.shellcmdflag = '-c'
vim.o.shellxquote = '('

Yes with these options the Z command works. Unfortunately now the I cannot run the Zi command. The popup shows error: unrecognized subcommand ''query''

I got this error when I set 'shellslash' to true, does removing it fix the problem?

okay now I just get unknown option: --no-tmux. This is weird I'm not doing anything with tmux...

What's the output of fzf --version? I suspect you may have updated the fzf.vim script without first updating your fzf binary

fzf version is 0.52.1 (6432f00)

updating fzf fixed the issue

Everything works now, thanks!


Disable the click event.

I have two kind of events, one has a click event but other hasn't. How to disable or REMOVE the anchor tag for second event?

You can set a cssClass on the events you want disabled that sets pointer-events: none. For example: http://plnkr.co/edit/vbVA5q8a1AyIa8bxC9p9?p=preview Hope that helps!

Its working. Thanks a lot.


Add a hotkey to select all text in a file

I sometimes want to quickly copy everything in a file on GitHub and paste in my text editor.
cmd (ctrl) + A partially works, but it would select everything on the page.
Ideally if there was a hotkey to select current highlighted block it would be great. EG: highlight a comment box and select the whole comment, then hit "R" to reply.

I sometimes want to quickly copy everything in a file on GitHub and paste in my text editor.
cmd (ctrl) + A partially works, but it would select everything on the page.

I personally use the Raw link and then Select all keyboard shortcut. However we can add a shortcut for the Raw link.

:+1: on copying from raw. Copying from the styled content usually includes all sorts of line numbers and formatting garbage.
A way to select and quote a comment in a new comment is a great idea. Ideally without a shortcut. Just add include anything selected when you click the "Comment" button.

Looks like this is done already: https://github.com/blog/1399-quick-quotes

:+1:

I want to limit this issue to one thing, a useful thing.
Do you think we should add a button (with a hotkey) for selecting or copying file contents?

:heavy_check_mark: Selecting

Why only selecting? When would you ever just select and not copy?

If it's a hotkey (which you can't see) that copies to the clipboard (which you can't see) then how will you know what you copied? If it selects then you can see what you are going to copy and ctrl-c or escape.
If this is for a button, then copy, but if it's all "hidden" it should have some visual indication mixed in for UX.

I think it should be a copy button next to the Raw button.

@paulmolluzzo What do you think about my suggestion above?

I think it should be a copy button next to the Raw button.

You said:

If this is for a button, then copy

@hkdobrev yeah, I agree with your suggestion in that context. I go to Raw to copy a lot. :+1:

Just as examples: .gitignore and licenses and base CSS and yml files are all pretty much raw->copy workflows.


Data Iterators documentation is broken

Many of the API references are not working - http://mxnet.incubator.apache.org/api/python/io/io.html#data-iterators

@mxnet-label-bot [Doc, Data-loading]
@aaronmarkham please check

Inheritance in the Python module is misconfigured. Refer to https://github.com/apache/incubator-mxnet/issues/12829
I'd escalate this as appropriate. I don't know how to fix the inheritance issues.

So Sphinx is getting confused. See this related issue: https://github.com/sphinx-doc/sphinx/issues/4961
It looks like to process to fix this is to be very specific about what method is being referenced. I'm working on a solution for this page, but it looks like this is a problem throughout the docs.

 The changes since v1.3.0 have caused many of the docs to disappear or point to the wrong methods when methods have the same name.
Sphinx isn't seeing what's there. The autosummary / autodoc features would just walk through the APIs and list things out.


Support for Sanic 22.12.0

Problem Statement
We'd like to upgrade from prior LTS release of sanic (21.12.2) to the new LTS release and just want to make sure everything will work reliably.
Could you please verify that sentry-python works with the latest LTS release of sanic?
Solution Brainstorm
Just want clarification

@robd003 added it to test matrix, seems to work, feel free to upgrade :)


jsonpScriptSrc may include a non-trivial amount of redundant bytes

Bug report

What is the current behavior?
When code splitting using a named chunk webpack outputs a function like:
/******/ 	// script path function
/******/ 	function jsonpScriptSrc(chunkId) {
/******/ 		return __webpack_require__.p + "" + ({"feature":"feature"}[chunkId]||chunkId) + ".js"
/******/ 	}

where feature is the name of the split chunk.
If the current behavior is a bug, please provide the steps to reproduce.
https://github.com/gusvargas/webpack-jsonpscriptsrc-bug
$ git clone git@github.com:gusvargas/webpack-jsonpscriptsrc-bug.git
$ cd webpack-jsonpscriptsrc-bug
$ yarn
$ yarn build

Examine the jsonpScriptSrc function in dist/main.js.

What is the expected behavior?
Given the ||chunkId it should be reasonable to omit the name map entry for feature and simply let it fall back to the chunkId. For apps with a non-trivial amount of named splits (in our apps it's not uncommon to get into the 100s), having an "identity" entry for every named chunk yields a non-trivial amount of redundant bytes.

Other relevant information:
webpack version:  4.35.2
Node.js version: 8.9.1
Operating System: macOS 10.12.5
Additional tools:

It's more optimized in webpack 5 and omits key-value-pairs if they match chunkId.
In production mode chunkId is usually numeric so this shouldn't occur too often.


Some Mechanical Belt's Unable to stack

Describe the Bug
For some reason I have multiple stacks of Mechanical Belt as they do not stack.
Reproduction Steps
Unknown as I do not know what has caused it.
Expected Result
I expect all Mechanical Belts to stack into a single stack.
Screenshots and Videos

Crash Report or Log
No response
Operating System
Manjaro Linux
Mod Version
0.5.1c
Minecraft Version
1.19.2
Other Mods

Additional Context
No response

Pretty sure i have seen this same behavior. The only explication is that these belts have different nbt data somehow

This is a common issue with mods adding empty NBT to items when trying to check for a value. There is no easy way to figure out what mod causes it. If you place and break them all, it should be fixed.


vehicle_types.json wrongly required

I suspect that this change broke https://github.com/MobilityData/gbfs-validator/pull/41. For v2.2 feeds the vehicle_types.json is thought to be required again, even if the criteria stated in gbfs.md

REQUIRED of systems that include information about vehicle types in the free_bike_status file.

is not given.
Example feed: https://gbfs-validator.netlify.app/?url=https%3A%2F%2Fgbfs.nextbike.net%2Fmaps%2Fgbfs%2Fv2%2Fnextbike_le%2Fgbfs.json
which states

Missing file and required

for the vehicle_types.json.
Should I open a new issue?
Best regards
Originally posted by @nbdh in https://github.com/MobilityData/gbfs-validator/issues/63#issuecomment-1255072227

This line impact the requirement of the file for the nextbike feed https://github.com/MobilityData/gbfs-validator/blob/3a3cd865a7884744f80df36a88d2f0f76c92bce1/gbfs-validator/gbfs.js#L522
hasBikesStationId is true.
Should we require vehicle_types.json if there are station_id ?

Thanks for creating this and for pointing to the corresponding piece of code.
Not sure if I should feel addressed by the question, but according to my understanding, the relation is the other way round:
station_id in free_bike_status is:

REQUIRED only if the vehicle is currently at a station and the vehicle_types.json file has been defined.

(Source)
while station_id being present does not mean the vehicle_type.json becomes required.
--
Side note regarding https://github.com/MobilityData/gbfs-validator/blob/3a3cd865a7884744f80df36a88d2f0f76c92bce1/gbfs-validator/gbfs.js#L522
freeBikeStatusHasVehicleId rather means freeBikeStatusHasVehicleTypeId, right?

Yeah, I'm pretty sure the condition is not right
It's a bit complex. The file is required if some fields are defined and in the other way, if the file is defined, it required the fields.


Introduce serialization/deserialization benchmarks.

These are based on benchmarks from original google/protobuf library.

FYI @nichite

/cc @mkustermann


Feature: Re-add grunt-protractor-webdriver

The problem with the grunt-protractor-webdriver module which was causing our tests to fail has been fixed so re-adding the module back as it's a much simpler to use it then update the webdriver manually.

:+1:


Composer errors

Any advice on solving the composer install errors below?
Thanks in advance!

Hey Nick, try doing a git pull first and pulling in some changes I made to
the composer config file and run composer install --no-dev to ignore the
dev dependencies that are causing this issue. Unfortunately, there's no way
currently in composer's config to have it automatically ignore these
development dependencies that you don't necessarily need (like unit testing
and things like that), so you'll have to pass that --no-dev flag
explicitly
On Tue, Jun 12, 2018 at 2:53 PM, Nick Tones<EMAIL_ADDRESS>wrote:

Any advice on solving the composer install errors below?
Thanks in advance!
[image: screen shot 2018-06-12 at 19 48 51]
https://user-images.githubusercontent.com/2812641/41310573-c65731b6-6e79-11e8-929e-36cf44c6002d.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/therecluse26/PHP-Login/issues/196, or mute the thread
https://github.com/notifications/unsubscribe-auth/AH24AhdMQj6g2wdOv05F5Pd--N4G5rSLks5t8A4sgaJpZM4Uk-Cp
.

Awesome - will give that a try and get back to you

On testing now - new composer.json file has a extra unneeded comma on line 24.

Still getting the errors below, with new composer.json and --no-dev

Ok, run one more git pull and try again. Sorry about that!
On Wed, Jun 13, 2018 at 11:01 AM, Nick Tones<EMAIL_ADDRESS>wrote:

Still getting the errors below, with new composer.json and --no-dev
[image: screen shot 2018-06-13 at 15 59 03]
https://user-images.githubusercontent.com/2812641/41359612-c8c67b1a-6f22-11e8-981d-0a3913b43d70.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/therecluse26/PHP-Login/issues/196#issuecomment-396969530,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH24AkKf7XXN-K1ZhfitVkq-W_eNT4pAks5t8SkzgaJpZM4Uk-Cp
.


Make sure thimble can save makes built in the brackets editor

Thimble's mechanism for persisting a make from it's codemirror instance needs to be updated to persist the make from bramble. The ThimbleProxy extension may need to be updated to facilitate this.

See: https://github.com/sedge/thimble.webmaker.org/pull/3


Added Youtube Link Music Functionality

Description
User can paste the youtube link and play the music in the disco stimulator, it also follows the mute and umute function.
Fixes:  #94
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

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 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

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Please give access to be able to resolve confilcts and merge

@ananya-dhiman
you can sync your fork with the upstream repository and then resolve the conflicts locally before merging the changes back to your fork & then create a pr

Oh ok will do it


Make "fatigue" the short-term limit for Trial Mode, and something like "competition pool" the long-term limit

Currently the short-term limit is "equipment wear", which is dumb. Change it to fatigue, which is currently the long-term limit. The new long term limit is still TBD, but will probably be something like "competition pool" or something. Or maybe "hype"? It should be slightly cynical, like "market saturation" and "social exposure".

Long-term limit is now "Competitive Demand" (will probably want to change at some point, but at least it's in the GameSetting now. Limit break is now "Overhyped".


Add RdaAgencyPreference API resource

https://github.com/homeflow/ctesius/issues/9498

I upped the version numbering on branch to keep up with master.


Request Rejected Status

Subject: Request Rejected Status
Description:
Would it be impossible to change the wording for a host request from Rejected to Declined? I am unable to host yet referring to it as Rejected seems unnecessary harsh.
Results:
Backend version: develop-d6739683
Frontend version:
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/109.0
Page: https://couchers.org/messages/request/39001
User: salami (4679)

@jesseallhands thoughts on this? Not a bad idea

Yes, good idea! It was supposed to be "declined" from the start anyway.

Yes, this was fixed by me some time ago #4541


[HUDI-728]: Implement custom key generator

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
We have TimestampBasedKeyGenerator for defining custom partition paths and we have ComplexKeyGenerator for supporting having combination of fields as record key or partition key.
However we do not have support for the case where one wants to have combination of fields as record key along with being able to define custom partition paths.
This PR aims to give a generic implementation where we can define key generator for every field in partition path.
Brief change log

Introduced PartitionKeyType in KeyGenerator, also added 2 new functions
String getPartitionPath(GenericRecord record, String partitionPathField)
String getRecordKey(GenericRecord record)

Introduced a new class CustomKeyGenerator which accepts input for partition path field in form -> field1:PartitionKeyType1,field2:PartitionKeyType2

All the corner cases have been handled. Added a test class TestCustomKeyGenerator with only one test case for now. Will be adding more.

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

This PR is a currently work in progress. Will be adding more test cases to ensure everything works fine.

@bvaradar @vinothchandar please have a look and let me know if this looks good.

@nsivabalan could you review this please?

@pratyakshsharma thanks for your patience.. I will review this myself and get back in a day.

@vinothchandar : my bad.  I have taken a look. You can review once my feedback is addressed. Will keep you posted.

@vinothchandar @nsivabalan please take pass.

LGTM. @vinothchandar : do you want to review or can we go ahead and merge it.

@vinothchandar Let us close this? :)

@pratyakshsharma sorry .. fell off my radar since I was not an assignee.. I do have some concerns.. Review coming by your day time :)

I have some concerns on the implementation.. Also I think you may have checked in a few parquet files from testing?

My bad. Will remove them.

Codecov Report

Merging #1433 into master will decrease coverage by 0.29%.
The diff coverage is 35.80%.

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

@vinothchandar Have tried to address the comments, please have a look again. Also looking at the Codecov report, I feel there is a need to add test cases for all key generators for which we do not have right now like SimpleKeyGenerator, ComplexKeyGenerator etc. Should I include that in this PR or should I go for a separate PR. Please suggest.

Should I include that in this PR or should I go for a separate PR. Please suggest.

Your call.. Doing it here is fine by me as well, since you are touching all those files anyway..

@vinothchandar Please take a pass.

@pratyakshsharma rebase again? I can take a final pass

@pratyakshsharma Rebased and removed the parquet files etc..

@nsivabalan  can you shepherd this one home from here>

@nsivabalan I have tried to include the changes from https://github.com/apache/incubator-hudi/pull/1597 as well in this. Please take a pass.

Can you point me to exact commits where you addressed my last set of comments and commits where you pulled in the other PR?

Sure. I addressed your last set of comments in - https://github.com/apache/incubator-hudi/pull/1433/commits/4ebd50561fa9d130118b84db501ca23600d2c9d6
I pulled the other PR in - https://github.com/apache/incubator-hudi/pull/1433/commits/cfe94e4ac8995f5b25e4385e7446e7b86e88e4b6.
Hope that helps.

Codecov Report

Merging #1433 into master will increase coverage by 1.55%.
The diff coverage is 8.45%.

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

@pratyakshsharma : can you fix the build issue.

@nsivabalan I squashed the commits and force pushed after unstaging the 2 parquet files, but they are still showing.

cc @wangxianghu ..
@pratyakshsharma confirmed, he will resume this wiork and take it across finish line

@nsivabalan can we merge this now?

LGTM. can you squash all commits and let me know

I have already squashed @nsivabalan :)

Pinging to see if you have any more concerns here or we can merge this @nsivabalan ? :)


Any plans for video driven motion?

This is a great project! I'm very impressed by the output quality. I'm just wondering, is there any plans for adding the ability to drive the model with video instead of audio? If not, I'm going to try to implement it myself, any tips to point me in the right direction would be much appreciated.
Also, I noticed this:
https://github.com/yerfor/GeneFacePlusPlus/blob/98fa99265b6ca82bed97f3bd2b300ae80da4140f/inference/genefacepp_infer.py#L523
(translates to ~"Currently, only the pose from the source video is supported, including from the beginning and a specified interval. Temporarily does not support in-the-wild pose.").
Could you share any further details about this?

Yes, you can implement yourself. Just replace the audio2secc step in the inference script that predicts the exp code from audio, instead, write a method like video2secc, in which your can directly use our fit_3dmm_landmark script to obtain exp from a video. Then run the secc2video step to obtain the final video!
As for the head pose, you can see in the inference script that the head pose is obtained by indexing the training dataset. So we are using the GT head poses. You can also implement yourself to support external video reference to provide head poses.  But it may leads to worse quality though.


[Edge WebDriver Docs Feedback]

Lack of installation directions for 'new' VS install.
I've installed the current VS / download (64 bit) and attempting to get Selenium (using Edge) working.  The directions for a new installation are ... not very good.
When followed - the result is an install with references to external URLS - that do not offer any help as they are "upgrade" (vs new install).  The installed code refers to an external URL:
// In order to run the below test(s),
// please follow the instructions from https://docs.microsoft.com/en-us/microsoft-edge/webdriver-chromium
// to install Microsoft Edge WebDriver.
Which when followed refers to both Selenium 3 (upgrade) and 4; with a line "in case you need it".  As this is a fresh in stall; the "in case you need it" should be a "determine if you need it by...." [missing info]"
Your "Note" (ref: "This article provides instructions for using the Selenium framework") has the text "...consult the documentation for your framework of choice." ... that is why I'm at this page.  You should have a link to the Selenium/framework/Edge driver installation/configuration.
Your "sample code" is not in the form of a test; and this needs to be fixed.  (Why else would one want to use Selenium?)
When the sample code is used; it fails, no error message is displayed; no debugging steps are provided.  So - unfortunately I have to say "it fails".  I believe that I've installed most of 'it' correctly; but as there is no debugging/configuration/confirmation steps listed; I can't tell you what "it" is that is failing.
Running the code results in two outputs:

"Output" window:

'dotnet.exe' (CoreCLR: DefaultDomain): Loaded 'C:\Program Files\dotnet\shared\Microsoft.NETCore.App\2.1.30\System.Private.CoreLib.dll'. Skipped loading symbols. Module is optimized and the debugger option 'Just My Code' is enabled.
'dotnet.exe' (CoreCLR: clrhost): Loaded 'C:\Users\dan\source\repos\Bomarc\SpamcopReporter\SCR\bin\Debug\netcoreapp2.1\SCR.dll'. Symbols loaded.
'dotnet.exe' (CoreCLR: clrhost): Loaded 'C:\Program Files\dotnet\shared\Microsoft.NETCore.App\2.1.30\System.Runtime.dll'. Skipped loading symbols. Module is optimized and the debugger option 'Just My Code' is enabled.
The program '[9400] dotnet.exe: Program Trace' has exited with code 0 (0x0).
The program '[9400] dotnet.exe' has exited with code 0 (0x0).

The Debug Console windows:

C:\Program Files\dotnet\dotnet.exe (process 11828) exited with code 0.
To automatically close the console when debugging stops, enable Tools->Options->Debugging->Automatically close the console when debugging stops.

That is it; there are no test results; there is no automated actions occurring on the display; there is no logging (of test cases or test results) visible; there is no indication where logging is occurring (if it is); there is no debugging steps provided; there is no "you need to this... ??" to see any of the missing information.
In irony: at the bottom of the (support) page is a link to "See also ... 'The Selenium Browser Automation Page"; which requires the configuration to be correct.  And in closing: Why don't you have sample test cases -- rather than just raw code?
AB#44917120

@Bomarc, thank you for your feedback.
It sounds like the first source of confusion is the choice between Selenium 3 and Selenium 4. Selenium 3 is out of support and all new projects considering Selenium should use Selenium 4. Our docs were initially written a few years ago when Selenium 4 was newly released and our intention was to provide additional context to help migrate away from Selenium 3. Now that Selenium 4 has been released for a while, it might be time to remove references to Selenium 3 from our docs to avoid further confusion.
As noted in the docs, you must install both a browser driver (Microsoft Edge WebDriver), and a WebDriver framework (in this case, Selenium). These are separate components. The steps for installing Microsoft Edge WebDriver are listed under https://learn.microsoft.com/en-us/microsoft-edge/webdriver-chromium/#download-microsoft-edge-webdriver.
It looks like you are using the Edge WebDriver template provided with Visual Studio which creates a simple test project. Once Microsoft Edge WebDriver is downloaded and available on the PATH, and the Selenium 4 NuGet package has been added to the project, then the example test that navigates to Bing should complete successfully.
As you've noticed, our docs provide raw code samples and not complete tests. That's because the purpose of our documentation is not to provide a complete how-to for getting started with Selenium, but to assist with the Microsoft Edge-specific portions of the process. This includes obtaining the Edge WebDriver and setting Edge-specific options in code. Selenium is a cross-browser automation library and is also language and test-framework agnostic. Furthermore, Selenium is just one popular implementation of a WebDriver framework, and Edge WebDriver can be used with any. We use Selenium only as an illustrative example and because it is ideal for most user's needs.

Fixed by #2624.


Pluralize "parenthesis"

"Parenthesis are important" sounds really silly.

Also, "parenthesis" on the next line could be changed as well, but it makes grammatical sense depending on one's interpretation.

Hehe, TIL...


Disable Google Analytics for 404 pages generated by error.vue of layout component

with ignoreRoutes can ignore pages with known routes，but how can ignore 404 page？The name and path of the 404 page are unknown

@0721Betty you can pass the name of the route too.


Unable to resolve the name 'py.fooof.FOOOF'

Hi, I followed the README and installed dependencies. I have MATLAB (<IP_ADDRESS>6388 (R2023a) Update 3) and Python (3.10.6). I tried running my own scripts, as well as all of the example .mat files and run into this error:
Unable to resolve the name 'py.fooof.FOOOF'.

Error in fooof (line 48)
    fm = py.fooof.FOOOF(settings.peak_width_limits, ...

Any help would be appreciated. Thanks in advance.

I'm having the same issue

Hi both! I run into the same error... did you happen to find a solution or a workaround during the past months since this issue was opened?

Update: While this issue has not been solved, I was able to run fooof now via Matlab (and anaconda-installed Python) on an older (2019) MacBook Air that has an Intel chip. So it might possibly be an issue with Apples M1 chip.

I also am having the same issue in Windows

Anyone manage to solve this? Python is working fine in MATLAB (I have 3.9 installed), but I get the same error message as above.
I have also tried py.importlib.import_module('fooof'), as suggested by @lmcrown, but get the following error message too:
Error using _backend_tk>
Python Error: RuntimeError: Failed to load Tcl_SetVar
It's curious as I have managed to run the fooof tool on a prevoius dataset with no trouble, so any help here would be greatly appreciated!

I also had this problem. I was able to solve it by installing matplotlib, as mentioned in #30


Add syntax highlighting to the demo site

I tried https://highlightjs.org/ but it messed up the vertical rhythm. Maybe https://prismjs.com/?

Not going to bother doing this


Alem snyder/issue1

is there any way that you can leave out the .json and .yaml?

I just did a merge, kept the changes from dev and left out yours.

I just did a merge, kept the changes from dev and left out yours.


Add the data object received to the Error object

Sometimes you want to use the partial data you received from the graphql server in the event of errors. With this PR you can catch the error and continue with the data you got.

Thanks.


qpdf/libqpdf/QPDFObjectHandle.cc:235:null pointer

When using the qpdf --empty --pages command to merge 1.pdf with other PDF files, a segmentation fault occurred due to a null pointer.
version:11.5.0
My system OS：Ubuntu 22.04
My Compilation Process：
git clone https://github.com/qpdf/qpdf
cd qpdf
cmake -S . -B build -DCMAKE_BUILD_TYPE=RelWithDebInfo ..
cd build
cmake --build .
sudo cmake --install .
reproduce: qpdf --empty --pages 1.pdf 2.pdf -- output.pdf
The stack information is as follows：
Program received signal SIGSEGV, Segmentation fault.
QPDFObjectHandle::disconnect (this=this@entry=0x7fffffffdd20) at /home/ljh/qpdf/libqpdf/QPDFObjectHandle.cc:235
235	        this->obj->disconnect();
(gdb) bt
#0  QPDFObjectHandle::disconnect (this=this@entry=0x7fffffffdd20)
at /home/ljh/qpdf/libqpdf/QPDFObjectHandle.cc:235
#1  0x00007ffff7dc58f6 in QPDFObjectHandle::DisconnectAccess::disconnect (
o=...) at /home/ljh/qpdf/include/qpdf/QPDFObjectHandle.hh:1268
#2  QPDF_Dictionary::disconnect (this=0x555555619ba0)
at /home/ljh/qpdf/libqpdf/QPDF_Dictionary.cc:52
#3  0x00007ffff7cb9056 in QPDFObject::disconnect (this=0x555555617790)
at /home/ljh/qpdf/libqpdf/qpdf/QPDFObject_private.hh:148
#4  QPDF::~QPDF (this=this@entry=0x5555555a4700, __in_chrg=)
at /home/ljh/qpdf/libqpdf/QPDF.cc:236
#5  0x00007ffff7c6b4cb in std::default_delete::operator() (
__ptr=0x5555555a4700, this=)
at /usr/include/c++/11/bits/unique_ptr.h:85
#6  0x00007ffff7c6f03e in std::default_delete::operator() (
__ptr=, this=)
at /usr/include/c++/11/bits/unique_ptr.h:79
#7  std::unique_ptr<QPDF, std::default_delete >::~unique_ptr (
this=0x7fffffffddc8, __in_chrg=)
at /usr/include/c++/11/bits/unique_ptr.h:361
#8  QPDFJob::createQPDF (this=)
at /home/ljh/qpdf/libqpdf/QPDFJob.cc:481
#9  0x00007ffff7d375f7 in QPDFJob::run (this=this@entry=0x7fffffffde70)
at /home/ljh/qpdf/libqpdf/QPDFJob.cc:516
1.pdf
2.pdf

Thanks. I will look at this when I get a chance, hopefully in the next few days.


The node type SpreadProperty has been renamed to SpreadElement at Object.isSpreadProperty

im not able to run project im getting this error when i tried to run using npm run dev-server

Can you provide a screenshot for this error or the whole error message?

Can you take a screenshot on the part where the error was encountered?

Seems like you're missing a babel config file
Did you just downloaded the repo or cloned it?
Create a file and name it .babelrc and add the following config
{
"presets": ["@babel/preset-env", "@babel/preset-react"],
"plugins": [
"babel-plugin-transform-class-properties",
"babel-plugin-transform-object-rest-spread",
"@babel/plugin-syntax-dynamic-import"
]
}

yes i added babel config file same issue is getting ,
i had clone the repo and tried to run the code.

Have you referred to same issue here in stackoverflow? https://stackoverflow.com/questions/53326986/trace-the-node-type-spreadproperty-has-been-renamed-to-spreadelement-at-object

yes i had refereed to above stackoverflow but not luck.😣

So I cloned my repo just to check if there were any error and I didn't see any and it's working fine.
Perhaps you could delete your node_modules and also the package-lock.json and run npm install again.

https://stackoverflow.com/questions/53326986/trace-the-node-type-spreadproperty-has-been-renamed-to-spreadelement-at-object
i have refereed this url and solved the issue.
thank you @jgudo

Same problem for me,
"babel-plugin-transform-object-rest-spread",
Changed to
"babel-plugin-proposal-object-rest-spread",
in babelrc file... have more issues still but loads of warnings disappeared

@whatneuron  are you running the latest commit? I've updated all the babel plugins. Consider running a git rebase to get the latest.
$ git remote add upstream git@github.com:jgudo/ecommerce-react.git
$ git fetch upstream
$ git rebase upstream/master