Run pip in workspace root (#284)

pip currently is excuted in the working directory of rye.  This means that pip cannot find the appropriate pyproject and will fail to write the requirements.lock. We now explicitly call pip in the determined workspace_path.
Please note this is an early PR and my first change. I don't fully understand the rye codebase yet and have not extensively tested this change yet. So beware, there might be dragons.
Testing
Currently this has been tested doing the following:
$ rye init root
success: Initialized project in /home/dsp/src/root
  Run `rye sync` to get started
$ cd root
$ rye init submodule
success: Initialized project in /home/dsp/src/root/submodule
  Run `rye sync` to get started
$ echo "[tool.rye.workspace]\nmembers=[\"submodule\"]" >> pyproject.toml
$ cd submodule
$ rye sync

Before this change
Generating production lockfile: /Users/dsp/src/rye/rye-test/requirements.lock
Traceback (most recent call last):
  File<EMAIL_ADDRESS>line 8, in <module>
    sys.exit(cli())
             ^^^^^
  File<EMAIL_ADDRESS>line 1130, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File<EMAIL_ADDRESS>line 1055, in main
    rv = self.invoke(ctx)
...
pip._internal.exceptions.InstallationError: file:///submodule (from -r /var/folders/yl/jkvwmym95_sgrmcx_nmn21000000gn/T/.tmpGOZrJH (line 1)) does not appear to be a Python project: neither 'setup.py' nor 'pyproject.toml' found.
Error: could not write production lockfile for workspace

After this change
Generating production lockfile: /home/dsp/src/root/requirements.lock
Generating dev lockfile: /home/dsp/src/root/requirements-dev.lock
Installing dependencies
Looking in indexes: https://pypi.org/simple/
Obtaining file:///. (from -r /var/folders/yl/jkvwmym95_sgrmcx_nmn21000000gn/T/tmp_zayqwfo (line 1))
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... done
Obtaining file:///submodule (from -r /var/folders/yl/jkvwmym95_sgrmcx_nmn21000000gn/T/tmp_zayqwfo (line 2))
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... done
Building wheels for collected packages: root, submodule
  Building editable for root (pyproject.toml) ... done
  Created wheel for root: filename=root-0.1.0-py3-none-any.whl size=994 sha256=c3d665d5a85c414f3963c4eddc9c141bec42689133c127c3265e5e28f9f5c38d
  Stored in directory: /private/var/folders/yl/jkvwmym95_sgrmcx_nmn21000000gn/T/pip-ephem-wheel-cache-1x_8h2fi/wheels/97/54/f5/d849319cdfa096e074df352654ee2e7c919da8951f090690c6
  Building editable for submodule (pyproject.toml) ... done
  Created wheel for submodule: filename=submodule-0.1.0-py3-none-any.whl size=1056 sha256=81643b6ab37c8be2e102259034e1dca0f46eba764b1980dcb93599994e15dd78
  Stored in directory: /private/var/folders/yl/jkvwmym95_sgrmcx_nmn21000000gn/T/pip-ephem-wheel-cache-1x_8h2fi/wheels/24/6f/3c/0d52b0234f153b8aa3f922419ed47c4c9cc62a221afb196f10
Successfully built root submodule
Installing collected packages: submodule, root
Successfully installed root-0.1.0 submodule-0.1.0
Done!

With this change, maybe that's enough for sync and lock to support the --pyproject argument like - #232 - it can then be used to sync "from the outside" of a project directory. The basename still has to be exactly "pyproject.toml" then, so it's not full support.


Specific callback per workflow

Hi, how are you?
First, thanks for all your work.
I need to trigger callbacks per job instance, as I'm trying below:
Workflow Root Job:
class CsvGenerator::BaseJob
  include Sidekiq::Worker

  sidekiq_options workflow: true, queue: :test

  def on_workflow_complete
    Rails.logger.info "[JobWorkflow] Callback Completed!"
  end

  def perform
   # call child jobs
  end

Callback Registry:
Sidekiq::Hierarchy.callback_registry
  .subscribe(Sidekiq::Hierarchy::Notifications::WORKFLOW_UPDATE,
             Sidekiq::Hierarchy::WorkflowListener.new)

Workflow Listener:
class Sidekiq::Hierarchy::WorkflowListener
  def call(job, status, _old_status)
    Rails.logger.info "[JobWorkflow] Job: #{job} Status: #{status}"

    callback = :"on_workflow_#{status}"

    job.send(callback) if job.respond_to?(callback)
  end
end

The problem is that this line won't work: job.send(callback) if job.respond_to?(callback), because job is not our class, is a Sidekiq::Hierarchy::Workflow & Jobs...
Is there any way to get the original CsvGenerator::BaseJob instance from job inside Listener's call()?
Thanks.

+1


FR: add pragtical

https://github.com/pragtical/pragtical

Recipe created here: https://github.com/brechtsanders/winlibs_recipes/blob/main/recipes/pragtical.winlib
As this would not be part of the actual build tools it will not be included in the build tools.
In the future, please post any package build request here: https://github.com/brechtsanders/winlibs_recipes/issues


amplify push --allow-destructive-graphql-schema-updates fails even for simple updates

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
dtht3se3625gl
AWS Region
us-east-1
Amplify Hosting feature
Backend builds
Describe the bug
Updates to the GraphQL Schema fail when pushing to the server with "amplify push". Even very trivial updates, that wouldn't require destructive updates such as a name change of a table.
In this example, I've tried to rename the table in the GraphQL schema from FoodCategories to FoodCategory.
Embedded stack arn:aws:cloudformation:us-east-1:187343962510:stack/amplify-tavernpos-staging-02627-apitavernpos-B6WV0QZ95ZMU/bbb4af30-533b-11ed-a164-0eea8eb3e84f was not successfully updated. Currently in UPDATE_ROLLBACK_IN_PROGRESS with reason: The following resource(s) failed to create: [FoodCategory].
Expected behavior
I expect that:

First of all a simple name change isn't flagged as a destructive update, as the data structure isn't changing.
Then I'd expect that if it is a breaking update, the API is updated, or deleted and re-created without throwing an error.
Also I'd expect a more detailed error message other than that FoodCategory couldn't be created. It would be nice to know why.

Reproduction steps

Create a project with amplify init.
Create a GraphQL API
Add a GraphQL schema named FoodCategories
Push it using amplify push
Change the schema name to FoodCategory
Push it using amplify push
It will give a message that it requires --allow-destructive-graphql-schema-updates fails even for simple updates
Push it again with --allow-destructive-graphql-schema-updates fails even for simple updates

Build Settings
No response
Log output
~/Development/tavernpos$ amplify push --allow-destructive-graphql-schema-updates
⠸ Fetching updates to backend environment: staging from the cloud.⠋ Building res⠏ Fetching updates to backend environment: staging from the cloud.✅ GraphQL schema compiled successfully.
Edit your schema at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema.graphql or place .graphql files in a directory at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema
✔ Successfully pulled backend environment staging from the cloud.
⠋ Building resource api/tavernpos✅ GraphQL schema compiled successfully.
Edit your schema at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema.graphql or place .graphql files in a directory at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema
Current Environment: staging

┌──────────┬───────────────┬───────────┬───────────────────┐
│ Category │ Resource name │ Operation │ Provider plugin   │
├──────────┼───────────────┼───────────┼───────────────────┤
│ Api      │ tavernpos     │ Update    │ awscloudformation │
└──────────┴───────────────┴───────────┴───────────────────┘
? Are you sure you want to continue? Yes
✅ GraphQL schema compiled successfully.
Edit your schema at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema.graphql or place .graphql files in a directory at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema
⠴ Building resource api/tavernpos✅ GraphQL schema compiled successfully.
Edit your schema at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema.graphql or place .graphql files in a directory at /Users/chstrong/Development/tavernpos/amplify/backend/api/tavernpos/schema
? Do you want to update code for your updated GraphQL API Yes
? Do you want to generate GraphQL statements (queries, mutations and subscriptio
n) based on your schema types?
This will overwrite your current graphql queries, mutations and subscriptions Ye
s
Deployment failed.
Deploying root stack tavernpos [ ====================-----------------
amplify-tavernpos-staging-026… AWS::CloudFormation::Stack     UPDATE_
apitavernpos                   AWS::CloudFormation::Stack     UPDATE_
Deploying api tavernpos [ =============--------------------------- ] 3
GraphQLAPITransformerSchema3C… AWS::AppSync::GraphQLSchema    UPDATE_
GraphQLAPIDefaultApiKey215A6D… AWS::AppSync::ApiKey           UPDATE_
FoodCategory                   AWS::CloudFormation::Stack     CREATE_
DynamoDBAccess71ABE5AE         AWS::IAM::Policy               CREATE_
🛑 The following resources failed to deploy:
🛑 Resource is not in the state stackUpdateComplete
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Additional information
How can I delete the API and re-create it without destroying the full backend?
No response

Hey @chstrong :wave: thanks for raising this! I was able to reproduce this issue with a small GraphQL Schema
before
type Part
  @model
  @auth(
    rules: [
      { allow: groups, groups: ["admins"] }
      { allow: public, operations: [read] }
      { allow: owner, operations: [create, read] }
    ]
  ) {
  id: ID!
  name: String!
}

after
type Parts # note the plural "Parts"
  @model
  @auth(
    rules: [
      { allow: groups, groups: ["admins"] }
      { allow: public, operations: [read] }
      { allow: owner, operations: [create, read] }
    ]
  ) {
  id: ID!
  name: String!
}

Upon pushing I received an error, and looking at the CloudFormation Console for the nested API stack we see it failed creating one of the list* resolvers

Can you confirm whether you're receiving a similar error in the CloudFormation Console?


Conditional selected attribute in Twig causes closing option tag to be marked as an error

Hi junstyle!
In the option list, if I conditionally use the selected attribute, the closing option tag is marked as an error.
The template code works correctly and renders as expected in the browser, but the plugin shows a syntax error.

I have the same problem, dont know if it helps, but here's another example where you can see that it only happens on some lines not all.


fnlfmt: 0.2.3 -> 0.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/fnlfmt/versions.
meta.description for fnlfmt is: Formatter for Fennel
meta.homepage for fnlfmt is: https://git.sr.ht/~technomancy/fnlfmt
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.3.0 with grep in /nix/store/0mncz8q7af50byn3vx26vgffhnz0fhcz-fnlfmt-0.3.0
found 0.3.0 in filename of file in /nix/store/0mncz8q7af50byn3vx26vgffhnz0fhcz-fnlfmt-0.3.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
fnlfmt

Instructions to test this update (click to expand)

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

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

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

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

1 package built:

fnlfmt

Maintainer pings
cc @chiroptical for testing.

Result of nixpkgs-review pr 218884 run on aarch64-darwin 1

1 package built:

fnlfmt


Add a warning color when a taks is close to a due date

Story
As a user I like to be indicated before a task is due, so that I can react before the due date.
Verbose
I would be happy to be able to configure, on the settings page, when a due date comes close. E.g 3 days or 72h (I prefer days)
Also I like to be able to set a color for tasks which are in the time window >= (due date - warning time) and < due date
Example
Assuming a waring time of 3 days, due at Oct 20th, waring color set to orange, overdue set to red
before Oct 17th: no action
after Oct 17th and before due date Oct 20th: change color to orange
after Oct 20th: taks overdue change color to red

I'll add it to the list, but I'm not sure if and when this will be implemented.

this would be immensely useful if it  was able to be used to color code for tags as well.

Implemented in v1.6.0


Change language with $translatePartialLoader

Hello,
i have a problem with loading the translations. I am using the $translatePartialLoader.
My versions:
angular-translate - v2.15.2 - 2017-06-22
AngularJS v1.6.4
Google Chrome v59.0.3071.115
My problem:
I am on state "profile" and have the active language "DE" now i am switching to language "EN".
Then i switch the state to "settings" and switch my language back to "DE".
Now i don't see the my translations i only see my keys for the translations.
State change event
$rootScope.$on('$stateChangeSuccess', function (event, toState, toParams, fromState) {
    $scope.unloadModule(event, fromState.name);
    $scope.loadModule(event, toState.name);
    $translate.refresh();
});

Functions for load/unload the modules
$scope.loadModule = function ($event, module) {
    $event.preventDefault();
    $translatePartialLoader.addPart(module);
};
$scope.unloadModule = function ($event, module) {
    $event.preventDefault();
    $translatePartialLoader.deletePart(module);
};

Thanks for watching for a solution.

Hello,
as we had already mentioned in the issue's template, we need a working and as minimal as possible demo covering your specific use case or issue.
Follow these steps:

Please open this plnkr base.
Fork it (at the top left).
Ensure the version of AngularJS and angular-translate is correct.
Ensure all angular-translate plugins are available; reduce and remove anything you can.
Write a minimal as possible demo for your specific issue.
Save and freeze. Provide us the final link to your demo.

Hello,
sry for waiting long time... now i have created an example in plunker.
https://plnkr.co/edit/CoMg4zFuKV8h0LSWe742?p=preview
Here the steps of the error:

Start: State "start" + language "DE"
Change state to "stop"
Change language to "EN"
Change state to "start"
Change language to "DE"

Now you don´t see the translated value, you see "start.INFO".

@knalli ist there any solution?

I think this should work
$translate.use(sprache)
  .then(function() {
    $translate.refresh();
  });

I'm not sure, looks like this is a specific issue with partial loading and language changes at all. Alternatively, don't remove already loaded parts (depends on the number of translations).

@knalli Your workaround did the trick but still FOUC, should really resolve inside the library before it renders out any content IMO.
Not sure if I should open a new issue.


Can't load https://itsjafer.com/#/schwab to get TOTP

I'm just stumbling on this and went  to try to get a TOTP from https://itsjafer.com/#/schwab but the page doesn't load for me. Getting the console errors below in Chrome (same error in incognito).

Super excited about this API method of working with Schwab though!

Hey, thanks for pointing this out. I thought this was an issue on your end but I was actually able to reproduce it and found it was a pretty small bug; I'm surprised I didn't notice this before. Just fixed, could you confirm it works?

It works! Awesome, thanks for the quick fix (and for creating this!)

Oh, quick question, I noticed the 'Generate TOTP' option isn't listed on that page - do you still have the 'easy' method available there? If not I'll work through generating one the hard way 😅


Hotfix Issue with 0.14.0 datastructure

what

bugfix issue with object to map to list for backwards compatibility

why

hotfix 0.14

references

Previous PR #39

Currently the module is not backwards compatible with the following error:
╷
│ Error: Insufficient rule blocks
│
│   on ../../../../../cloudposse/terraform-aws-backup/main.tf line 54, in resource "aws_backup_plan" "default":
│   54: resource "aws_backup_plan" "default" {
│
│ At least 1 "rule" blocks are required.
╵
╷
│ Error: Invalid function argument
│
│   on ../../../../../cloudposse/terraform-aws-backup/main.tf line 59, in resource "aws_backup_plan" "default":
│   59:     for_each = length(var.rules) > 0 ? var.rules : tolist(tomap(local.compat_rule))
│     ├────────────────
│     │ local.compat_rule is object with 7 attributes
│
│ Invalid value for "v" parameter: cannot convert object to map of any single type.
╵
Releasing state lock. This may take a few moments...
exit status 1

/test all

/test all

/test all

/test all

/test all

/test all


feat: rate limits for custom hostnames wrong

Description

updating rate limits to be accurate

Testing required outside of automated testing?

[ ] Not Applicable

Screenshots (if appropriate):

[ ] Not Applicable

Rollback / Rollforward Procedure

[ ] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.158.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix: summarise .gitignore files into one

Two fixes, again.

The pattern for files to be ignored by git can be defined once for the whole project, at the root. A patter like "node_modules" means that any file - or folder - will be ignore in each sub-folder recursively.  The drawback is that now you cannot use files/folders to be included.  But who would want to include app/tmp, app/dist app/dist, app/npm-debug.log anyways ?
Also, in chapter 10, the hello world kept a dist/bundle.js, which is generated by webpack. I removed it in favour of building it on demand.  It is a consequence of point 1.

Please use the code samples from the more recent edition of this book: https://github.com/Farata/angulartypescript


Rename this module

The "NamedStruct" pypi module already exists, this needs to be renamed before deploying to pypi.

These are available:

structuple
StructuredTuple
structer

packbin

After discussions we will rename to starstruct


excelize skip pages

Please replace in lib.go :
string 56 ( https://github.com/qax-os/excelize/blob/2e9c2904f2965fc45ad0fa1d5e2bfc7c0632a7cc/lib.go#L56 )
strings.HasPrefix(fileName, "xl/worksheets/sheet")

to
strings.HasPrefix(strings.ToLower(fileName), "xl/worksheets/sheet")

We have excel files with fileName like "xl/worksheets/Sheet" - and sheets just skip because of upper case

Thanks for your issue. I have made a changes for this, please try to upgrade the master branch code, and this patch will be released in the next version.

Thanks a lot!


Need Group Element Same Name In Tree View

This reduces the search for duplicates
Why we do that :

Help user easy search
Help increase performance, we will split small tree to get and load only some object because have group by name.

Example:

Duct(Taps)
FamilyInstace(45 Degree)

DB Application

we will desing it to suitable with #15


New Org not displayed in Orgs list

If you create a new Org via the Super Admin page they do not appear in the list of orgs on the Super Admin Orgs page.
I believe its due to the join on the templates table (a new org would have none). This join speeds up the query but will not work for this scenario since orgs need not have a template.
render 'index', locals: { orgs: Org.includes(:templates, :users).joins(:templates, :users) }

It's not just new orgs. I see a lot of institutions missing. There were 226 institutions in the database that we migrated, but only 76 appear in the list at https://dmptool.org/super_admin/orgs

yes, in attempt to optimize the query a join was added to the templates and users tables to get their counts. The type of join that is being used though will ignore an org if it has no users or templates.
Fix for this is coming today

this is working as expected now. all Orgs appear in the list and I can see new Orgs when I add them. closing.


Communicate the future of SimpleReport beyond expiration of PHE on May 11, 2023

Background / Description
On March 28th, 2023 the Support Inbox received the following message from Jeff Williams of Achieve Health Management in Texas:

We’re curious what the future of SimpleReport looks like following the end of the Public Health Emergency on May 11.  We have a lot of schools who will be utilizing this tool to report data to the CDC and Texas DSHS through the summer.  Do you know if there are any changes expected or a date to shut the system down entirely this year?

Guidance provided by SimpleReport Product Owner, Mike Judd on March 30, 2023:

Design folks could think through a short tagline or something minimal that could be added that communicates that SimpleReport is here of the long term, past COVID. That would be minimally sufficient so they know we have plans for how SimpleReport could be useful elsewhere.

(send email instead)
Requirements

Create a design to show the placement and size limitations of a banner on both the static site and the SimpleReport web-based application.
Work with the Content Designer to develop language to communicate the future of SimpleReport beyond the expiration of the of the federal Public Health Emergency (PHE) for COVID-19 on May 11th, 2023.
Incorporate the language into the proposed design treatment.
Share the proposed designs with Mike Judd for review and approval.
Generate a new engineering task in ZenHub for implementation and review the proposed designs with the engineering team.
Communicate this design update to the Index Analytics Support Team.

Acceptance Criteria / Definition of Done

Visual treatment to appear on both the SimpleReport static site and the application, communicating the future of SimpleReport beyond the expiration of the of the federal Public Health Emergency (PHE) for COVID-19 on May 11th, 2023, has been approved by our CDC Product Owner (Mike Judd)
Completed designs are ready to hand off to engineering for implementation and have been attached to a new engineering implementation task in ZenHub.
The Index Analytics Support Team has been made aware of the proposed design update
Send to states first

Communications plan/email content doc: https://docs.google.com/document/d/1RmhJJhfOOYJb0jCKhlA58fw3FEwoKh6qK1LL3AYrrsY/edit#


Bug: out of memory

nerdfix check --format json -r . 2>/dev/null >nerdfix.json started in my home dir in a minute takes all my RAM (32GB) and begin swapping, so I killed it.
$ nerdfix --version
nerdfix 0.4.0
cheat-sheet: c3e3d5fec7c8744eae6f1cdfb90a38cdc1033519

Should be fixed in #21, I tested it on my home dir and did not encounter infinite memory growth. BTW, I also added a binary files filtering, as the endless "Invalid UTF-8" errors did annoy me a lot :|

I don't think it's fixed. Just tried again: I noticed it may take a lot of RAM but then release and and repeats. But, still, I've seen it takes about 29GB once before releasing, so it's still far away from fair RAM use for such a tool.

I noticed it may take a lot of RAM but then release and then repeats.

It must be loading a super large file, I don't have very large files under my home dir so I'm not experiencing this. I'm in the process of implementing stream processing so that nerdfix only needs to read a small segment of the file instead of the whole into memory.

Well, sure, there are large files in a home dir. Some of them are even related to this tool :smile:
4718852415 bytes ./proj/fonts/nerd-fonts/.git/objects/pack/pack-aaca106c27527881903673e6138df3ddb304514e.pack

but the largest ones are some game files in .local/share/Steam/ (24GB) and images in ./VirtualBox VMs/ (up to 34GB).
And I believe this is normal use case for the tool.
Some ideas:

Do not even try to process large (e.g. 10MB by default) files. Unlikely they'll contain icons to fix. Provide an option to configure this.
Do not load whole file in memory, this is not needed for tool's functionality, so process it as a stream using small buffer (say, 64KB). May need special handling for case when icon's bytes will be split between current and following buffer.

Sure.
The memory issue looks fixed: it uses about 26MB RAM.
This version is also 2x faster: 6m43s vs 12m29s (v0.4.0).
There are some differences in output between these two versions, but at a glance all of them are okay:

New version outputs some icons before it fail on invalid UTF8 error, v0.4.0 fail on this error without any icons output.
New version skips large files.
New version skips binary files.

So, looks good to me!

New version outputs some icons before it fail on invalid UTF8 error, v0.4.0 fail on this error without any icons output

This is interesting, it may be checking a partially UTF8 file, or reading a binary file unexpectedly (I experienced this problem sometimes, as the content inspector library we use is not 100% accurate and occasionally detects some binary files as UTF-8 files). Anyway, this should be acceptable and the main problem is solved, thanks for your feedback!


[Bug][TCGC] __raw property is null if it's an implicit discriminator property

Describe the bug
Define an implicit discriminator property
@doc("Single base model without any child model.")
@discriminator("kind")
model SingleBase {
    size: int32;
}

And its __raw property of SdkBuiltInType is null.

To Reproduce
Just try to parse the definition defined above.
Expected behavior
Since it's primitive type, we should give a string type to __raw property?

All the __raw type is an original tsp type. For implicit discriminator property, there is no original property, so the __raw is undefined. It is by design.


Problem in running MOT_cross_validation.m :Y must be a vector or a character array

Hi
I try to running MOT_cross_validation.m after run compile.m and fix problem of dataset path.
After running MOT_cross_validation.m I get below log and errors:
MOT_cross_validation
Warning: Name is nonexistent or not a directory:
C:\MATLAB\SupportPackages\R2015b\opencvinterface\toolbox\vision\supportpackages\visionopencv\example\MDP_Tracking-master\dataset\devkit\utils

In path (line 109)
In addpath (line 88)
In globals (line 36)
In MOT_cross_validation (line 13)
Online training on sequence: TUD-Stadtmitte
Warning: Name is nonexistent or not a directory:
C:\MATLAB\SupportPackages\R2015b\opencvinterface\toolbox\vision\supportpackages\visionopencv\example\MDP_Tracking-master\dataset\devkit\utils
In path (line 109)
In addpath (line 88)
In globals (line 36)
In MDP_train (line 16)
In MOT_cross_validation (line 39)
load images from file results/TUD-Stadtmitte_dres_image.mat done
TUD-Stadtmitte: 10 positive sequences
initialize tracker from scratch
Error using svmtrain (line 234)
Y must be a vector or a character array.

Error in MDP_initialize (line 28)
tracker.w_active = svmtrain(tracker.lactive, tracker.factive, '-c 1 -q');
Error in MDP_train (line 49)
tracker = MDP_initialize(I, dres_det, labels, opt);
Error in MOT_cross_validation (line 39)
tracker = MDP_train(idx_train{j}, tracker);
And my system have below specifics:
Windows 10 x64
Matlab 2015b
OpenCv 2.49
Visual studio 2012
so, How can fix this error and run this code completely successfully?

you are using the svm toolbox of matlab instead of libsvm because the prepared libsvm in the code is not compiled with windows and visual studio. you just need to compile libsvm again:

change current path to 3rd_party/libsvm-3.20/matlab
in matlab command run "mex -setup C++"
then run "make" and you get 4 *.mexw64 files, done


Improve frontend a little bit

Some small quick wins, also added something @joshprice did.
The scrollbars were just bothering me so much in my own project, I had to do something about it...
I have a bunch of code I was working on after talking to Josh I didn't include with this and didn't finish, such as adding a sidebar. I might add it later when it's more complete
Before:

After:

🚀 Thank you for your contribution! 🚀


add swig 4.3.0; fix invalid filegroup field in BUILD file, support bazel 8.0

This PR has been tested with a git override in OR-Tools.
bump swig to 4.3.0
remove 'path' field in the filegroup BUILD entries
Split configure.patch into one patch per file
Add presubmit test for bazel 8
Please add:
@bazel-io skip_check unstable_url

Please add:
@bazel-io skip_check unstable_url

You can also comment @bazel-io skip_check unstable_url trigger the bot the label this PR.


UI - artifacts collection route switchover

The API for artifacts collections has changed from using the artifacts/filter route to artifacts/collections/filter; the UI needs to be updated to match.

This ended up as /latest/filter instead and has already been updated


Move dm-verity feature from image-rs to Kata Containers

To optimize the performance of DmVerityInfo and support image pulling on the host for both Confidential containers and Kata containers, we need to move the dm-verity feature to kata. This will also eliminate the unnecessary serialization and deserialization between kata-agent and image-rs, and remove some irrelevant dm-verity installation packages from the ci tests.
The WIP PR in kata: https://github.com/kata-containers/kata-containers/pull/7837

/cc @jiangliu @mythi


multigpu training output

Hi, I managed to train my dataset on 2 RTX4090, with the following command:
CUDA_VISIBLE_DEVICES=0,1 torchrun --standalone --nnodes=1 --nproc-per-node=2 train.py --bsz 2 -s data/test9 -m output/t3
I found that there are two .ply files in the point_cloud folder(point_cloud_rk0_ws2.ply and point_cloud_rk1_ws2.ply) after 7000 iterations, then I used the SIBR_gaussianViewer_app.exe from the original 3dgs project to visualize these two models by changing there name into point_cloud.ply.
I noticed that both of them cover the whole area, yet with a little difference in detail. I also tried to train the same dataset with single GPU, and the output is also different.
I wonder what is the difference between the two .ply files trained by multigpu? Should I do some more postprocess to fuse them?
Here are the results.

the first picture is trained by single gpu, the other two are trained by multigpu.

Hi, when we do the multi-gpu training, each GPU will keep 1/2 of all gaussians. When we save them to disk, each GPU save their gaussians separately. For example, point_cloud_rk0_ws2.ply means rank=0, world size=2 (we have 2 GPU, and this point cloud is from GPU0). If you want to visualize all gaussians from all GPUs, you can concatenate these files by following this: https://github.com/EGalahad/GaussFusion?tab=readme-ov-file#gaussian-splatting-training

Thanks for your reply, I'll have a try.


nuxt.confg.ts supabase clientOptions db schema works for useSupabaseClient() but NOT serverSupabaseClient()

Version
@nuxtjs/supabase: v1.1.0
nuxt: v3.7.3
Steps to reproduce
Create a new schema 'myschema' in Supabase project.

Configure your nuxt.config.ts file:
export default defineNuxtConfig({
  devtools: { enabled: true },
  modules: [
    '@nuxtjs/supabase'
  ],
  runtimeConfig: {
    public: {
      baseUrl: process.env.BASE_URL || 'http://localhost:3000'
    },
  },
  supabase: {
    clientOptions: {
      db: { schema: 'myschema' }
    },
    redirectOptions: {
      login: '/',
      callback: '/confirm'
    },
  }
})

Create a server API route:
import { serverSupabaseClient } from '#supabase/server';
export default defineEventHandler(async (event) => {
  console.log('Calling client.from')
  const client = await serverSupabaseClient(event);
  const { data } = await client.from('mytable').select('mycol');
  console.log(data);
  return data;
});

Call the endpoint from your Vue page.
What is Expected?
Variable 'data' should contain the row from myschema.mytable.
What is actually happening?
Variable 'data' is always null.
When you look in the database logs you see:
[{"event_message":"relation \"public.mytable\" does not exist","metadata":[{"file":null,"host":"db-....

Other comments.
If you place the following code in the Vue page:
const client = useSupabaseClient()
const { data: data } = await useAsyncData('mydata', async () => {
  console.log('from asyncData');
  const { data } = await client.from('mytable').select('mycol')
  return data
})

The variable 'data' is populated with the row from mytable correctly.

I have fixed this issue locally and will fork the project and attempt a pull request.  (Never done one before!)


Fix reading multiple complex data types exception

Fix #139

derstand how it
The elements from complex data type like map will take up indexes as below

Could you also include a short explanation for this fix, to help me understand how it fixes the issue?

The elements from complex data type like map will take up indexes as below

But how does this relate to the code changes you made?


error when click book

NameError: This app has encountered an error. The original error message is redacted to prevent data leaks. Full error details have been recorded in the logs (if you're on Streamlit Cloud, click on 'Manage app' in the lower right of your app).
Traceback:
File "/home/appuser/venv/lib/python3.9/site-packages/streamlit/scriptrunner/script_runner.py", line 554, in _run_script
exec(code, module.dict)
File "pages/02_book.py", line 15, in
current_path = Path(file).parent.absolute()

Hi William! Sorry took so long to answer, super busy week.
What repo has this problem? I can't seem to reproduce it


video-prototype: transfer videoFile("/sample_video") through NFD with NDNts

-without player
-on local machine
                                    2.interest
|-----------| 1.publish |----------| to fetch  |-----------|
|producer.js|---------->|          |<----------|consumer.js|
|/v_chunk[i]|           |   NFD    |           |/v_chunk[i]|
|-----------|---------->|          |---------->|-----------|
              3.sends   |----------| 4.sends
               data                  data
       =                                              =
/sample_video                --------->>>        /sample_video

How to:
-encode videoFile for chunks, should I work with a framework (ffmpeg or gstreamer)
("https://github.com/fluent-ffmpeg/node-fluent-ffmpeg#fluent-ffmpeg-api-for-nodejs-")
-fetch the videoFile ("https://ndnts-docs.ndn.today/typedoc/modules/segmented_object.html")

would it be possilble to work on "Node.js"
Stream

I saw some examples of Node.js would it possible to use it?
Because of OpenStream

Yes, you can use DataTape.openStream function to create a DataTape object from a Node.js stream such as a file or a TCP socket.
This has nothing to do with videos though.

I am not clear, how exactly I should prepare a videoFile to transfer it trough NFD, sry need help

See the README "usage" section, especially these commands:

encode.sh: this converts input video to DASH segments
node ./cli.cjs prepare: this converts DASH segments to DataTape (NDN packets)
node ./cli.cjs import: this imports DataTape into a repo
node ./cli.cjs serve: this starts a NDN producer that serves packets in the repo

Then you need NDNts-video webapp as the consumer to retrieve files from the repo.

node ./cli.cjs prepare --prefix $VIDEO_PREFIX --path $VIDEO_TEMP > video.dtar
text
node:internal/modules/cjs/loader:936
  throw err;
  ^

Error: Cannot find module 'hard-rejection'
Require stack:
- /home/vmadmin/NDNts-video-server/cli.cjs
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:933:15)
    at Function.Module._load (node:internal/modules/cjs/loader:778:27)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at require (node:internal/modules/cjs/helpers:94:18)
    at Object.<anonymous> (/home/vmadmin/NDNts-video-server/cli.cjs:2:1)
    at Module._compile (node:internal/modules/cjs/loader:1101:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1153:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:79:12) {
  code: 'MODULE_NOT_FOUND',
  requireStack: [ '/home/vmadmin/NDNts-video-server/cli.cjs' ]
}

Error: Cannot find module 'hard-rejection'

Did you follow the "installation" steps in README?
Specifically, did you run npm install --production and was it successful?

I will run it again

ok it works, but how integrate a videoFile I try it on the content.json file?

another question,
how to put those chunks trough my producer to consumer?
 readStream.on('data', (chunk) => {
                console.log("chunkSize:", chunk.length, "bytes");
                writeStream.write(chunk);
            })

how integrate a videoFile I try it on the content.json file?

Yes, you need to put the NDN name of each video's DASH manifest in content.json of the webapp.

On the video server side, need to ensure the producer is reachable from the global NDN testbed, which in turn requires setting up a certificate for prefix registrations.
The following commands would obtain a certificate from my Personal CA and setup the video server to use this certificate:
npm i -g https://ndnts-nightly.ndn.today/keychain-cli.tgz
source .env
curl -sfL https://gist.githubusercontent.com/yoursunny/54db5b27f9193859b7d1c83f0aeb8d6d/raw/ca-profile.data.base64 | base64 -d > profile.data
CAPREFIX=$(ndnts-keychain ndncert03-show-profile --profile profile.data | sed -n '/^CA prefix:/ s/^[^:]*: // p')
KEYNAME=$(ndnts-keychain gen-key $CAPREFIX/$(hostname -s)-$(basename $PWD) | tee /dev/stderr | awk -F/ -vOFS=/ '{ NF-=2; print }')
CERTNAME=$(ndnts-keychain ndncert03-client --profile profile.data --key $KEYNAME --challenge nop | tee /dev/stderr)
sed -i '/NDNTS_KEY=/ s|=.*|='$CERTNAME'|' .env

Then, edit .env and include the video prefix in REPO_PREFIXES variable.
Start the producer.
After a few minutes, the prefix announcement should show up on https://nlsr-status.ndn.today , and the player will work.

another question,
how to put those chunks trough my producer to consumer?
 readStream.on('data', (chunk) => {
                console.log("chunkSize:", chunk.length, "bytes");
                writeStream.write(chunk);
            })

segmentedObject should be an option to try something

outputs in-order data chunks as a readable stream.

producer:                                                                     >>  consumer:
takes input from readable streams                                   outputs in-order data chunks as a readable stream
where is it implemented? prepare
Do you have anohter example?

@ndn/segmented-object is more like a file transfer, with no video semantics: it cannot adjust video resolution based on available bandwidth.
See @ndn/cat package for examples.

Show v.file

something like:
const readStream = createReadStream('./sample_video.mp4');

v.file write to chunks

send chunks trough NFD to consumer

put those chunks to a v.file

its just stream bit by bit
#1. Show v.file
something like:
const readStream = createReadStream('./sample_video.mp4');
#2. v.file write to chunks
something like:
readStream.pipe(writeStream).on('error', console.error);
#3. send chunks trough NFD to consumer
?
#4. put those chunks to a v.file
?

That's a file transfer, not a video application.
Use @ndn/cat package for that.

Use @ndn/cat package for that.
ok it is for video applications as you mentioned

"dd if=/dev/urandom of=/tmp/1.bin bs=1M count=1"
sry what ist the expression ?

could you explain:
dd if=/dev/urandom of=/tmp/1.bin bs=1M count=1
and:
diff /tmp/1.bin /tmp/2.bin
rm /tmp/1.bin /tmp/2.bin

dd if=/dev/urandom of=/tmp/1.bin bs=1M count=1

Fill a file with random data.

diff /tmp/1.bin /tmp/2.bin

Compare two files, to ensure it's transferred correctly.

rm /tmp/1.bin /tmp/2.bin

Delete files.

sry I just try to understand cat is seems to belong to NDNts

cat is part of Unix.
ndncat aka @ndn/cat is part of NDNts.

could you teach me about @ndn/cat

could you teach me about @ndn/cat

See README in the package folder.

for my video-prototype i also have to use ndn6-tools

sry, how to etalbish the package @ndn/cat to use it?
so far its listed on my package.json file but it couldn't find the command and run "npm install"
how to do install it correctly?

how to install those package I started with "wget" and then "tar xvfz cat.tgz", what I have to do next?

@ndn/cat @ndn/repo-cli @ndn/keychain-cli are CLI commands and they are meant be installed as global packages.
Use npm install with -g flag.

it seems to work, but sometimes I also  got:
Not enough non-option arguments: got 0, need at least 1 YError: Not enough non-option arguments: got 0, need at least 1     at Object.fail (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/usage.js:59:26)     at Object.nonOptionCount (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/validation.js:25:27)     at file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/yargs-factory.js:1425:74     at file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/yargs-factory.js:1143:13     at maybeAsyncResult (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/utils/maybe-async-result.js:9:15)     at YargsInstance.[validateAsync] (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/yargs-factory.js:1142:16)     at YargsInstance.[runYargsParserAndExecuteCommands] (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/yargs-factory.js:1403:55)     at YargsInstance.parse (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/yargs-factory.js:674:63)     at YargsInstance.parseAsync (file:///home/vmadmin/ndnts/node_modules/yargs/build/lib/yargs-factory.js:704:35)     at file:///home/vmadmin/ndnts/ndn_cat/lib/main_node.js:22:6

Not enough non-option arguments: got 0, need at least 1

Follow the command syntax displayed above this error message.

ok I try it once again, in the end I got:

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'memif' imported from /home/vmadmin/.nvm/versions/node/v16.8.0/lib/node_modules/<EMAIL_ADDRESS>
This is possibly a new bug.
memif is supposed to be an optional dependency.
I'll check it.

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'memif' imported from /home/vmadmin/.nvm/versions/node/v16.8.0/lib/node_modules/<EMAIL_ADDRESS>
This is possibly a new bug.
memif is supposed to be an optional dependency.
I'll check it.

Fixed in https://github.com/yoursunny/NDNts/commit/75bd552f28d6f74b3b1e1eec2519d74b544aa449


Change material reflectivity

Hi,
How can I change the material reflectivity of my model?
Putting material-color on my Mesh elements works to change the color, but material-reflectivity doesn't affect the object.
Sorry I am not super familiar with Three. Any help would be appreciated :)

Okay I just understood that reflectivity is actually not a valid material property.
How can I achieve that then?

material-roughness={1} // diffused
material-roughness={0} // reflective
better consult three docs, there are a lot of props that affect surfaces

Thank you so much!


[wip] tensile: 5.4.2 -> 5.4.3

Without disabling tests, errors as:

AttributeError: module 'Tensile.Components' has no attribute 'Signature'

Logs: https://termbin.com/wh03

Tests are failing in staging. Some packages are failing in Hydra because of this package.
My goal here is just to sort out situation of this package in staging.
(Don't mind my solution itself just seeking a solution.)
I have disabled tests. Then it builds. [But can't answer how bad this is...]
The bump, is not critical, but it was an attempt to fix tests. [can undo]
CC @Madouura @Flakebi
Downstream:

rocblas
hipsolver
rocalution
rocsolver
magma-hip
miopen-opencl
miopen
hipblas

Doubts:

Panda does not seem used by upstream:

https://github.com/ROCmSoftwarePlatform/Tensile/blob/21fd25f12b1d11b3d9435a36441c9705a2aa8d07/docker/dockerfile-build-ubuntu-rock#L67

Hi @superherointj! Have you tracked down the problem with the failing tests? For some reason the tests pass ok for me on one machine, but not on another.
It passes on my intel-cpu laptop, but fails on my amd-cpu desktop... I have a hard time believing it's a CPU difference. I'm pretty sure there's some other issue going on, but I also can't find any other reason 😕.

Hi @superherointj! Have you tracked down the problem with the failing tests?

No.

For some reason the tests pass ok for me on one machine, but not on another.

Logs: https://termbin.com/eahzi

It passes on my intel-cpu laptop, but fails on my amd-cpu desktop... I have a hard time believing it's a CPU difference. I'm pretty sure there's some other issue going on, but I also can't find any other reason confused.

Sorry for not being of help.

Oh well that helps to know!! It starting to seem like it might actually be an AMD specific thing then - which is really weird, and I have no idea why.

So the problem is amd-specific on staging?
Weird. Doesn't seem to be due to a python version up because IIRC 3.10 is what I last tested it with. Maybe a problematic kernel upgrade?

I'm also seeing the issue on master @superherointj were you trying with --rebuild?

I'm also seeing the issue on master @superherointj were you trying with --rebuild?

I was able to replicate problem in master after adding --rebuild.

Logs: https://termbin.com/plfn3


Mistake in instructions

Trying to follow the instructions is impossible due to some minor mistakes. Step #2 says "Select Permissions," but a page search will reveal "Permissions" does not appear anywhere on the page. That text should be replaced with "Expand Roles & scopes, then select Permissions"
Under step #5, second bullet, it says, "On the Choose eDiscovery Manager wizard page, select +Add." but there is no "+Add" button, instead there is a "Chose Users" button.
Third bullet says, "Add," should say "Select" (no Add button)
Fourth bullet says, "Done," should say, "Next" (no Done button), then "Next" again
The entire block of instructions "To make a user an eDiscovery Administrator" appears to be wrong in that I don't see any such option on the page. Instead this is by selecting eDiscovery Manager, same as above, but then after the first Next above, users can be added to the eDiscovery Administrator role in exactly the same way they were added to the eDiscovery Manager.
Also, one other comment: this may be common knowledge among many users, but never having used the Purview Solutions tools before, I wasn't sure if being an Azure Global Admin already included the eDiscovery Manager and Administrator roles or if I still needed to add myself to these groups. Clarifying that somewhere on the various permission pages (what high level roles include the given permission or role) would be helpful.

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

ID: ec7c996e-5918-c91e-c10a-1e1acacb78a5
Version Independent ID: ec7c996e-5918-c91e-c10a-1e1acacb78a5
Content: Assign eDiscovery permissions in the Microsoft Purview compliance portal - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/ediscovery-assign-permissions.md
Service: o365-seccomp
GitHub Login: @robmazz
Microsoft Alias: robmazz

@developer-controls Thank you for your feedback. We will review the entire document and make changes as needed.
Thanks
Sri

@developer-controls Thank you for your feedback. The Steps in the navigation have been fixed.
@yogkumgit  This can be closed.
Thanks
Sri


fix 12 vulnerabilities

Before creating my pull-request, I ensure:

[x] Javascript documentation is written
[x] Unit tests are written and green
[x] I've cleaned my WIP or meaningless commits
[x] Examples and documentation site is up-to-date

In order to validate the pull request, I do:

List steps to help reviewers

Codecov Report

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

@@           Coverage Diff           @@
##           master      #41   +/-   ##
=======================================
  Coverage   91.56%   91.56%
=======================================
  Files           7        7
  Lines         320      320
  Branches       45       45
=======================================
  Hits          293      293
  Misses         14       14
  Partials       13       13

Continue to review full report at Codecov.

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


[ja] Translate tasks/administer-cluster/kubeadm/kubeadm-certs into Japanese

This is a Feature Request
What would you like to be added
Translate tasks/administer-cluster/kubeadm/kubeadm-certs  into Japanese.
Why is this needed
This page has not been translated.
Comments
/language ja
/assign

/triage accepted

/close


problem with installation

Hi.
I encountered problems when installing, there might be issues with the requirements.txt.
I ran pip install -r requirements.txt on a conda environment and then got an error about user requesting numpy 1.22.0.
Numba 0.55.1, which is in the requirements, doesn't support 1.22.0.
So I installed numpy 1.21.5. When running main.py I got omegaconf error about unsupported value type.
Then I did pip install omegaconf -U and pip install hydra-core --upgrade and that fixed the issue.
I've been enjoying the program so far. The inferencing runs well on my cpu and I am able to do things that I wasn't able to with other source separation algorithms.

Hi, thank you for your recognition on my work! And sorry for the requirement part because I just used the pyreq to generate the requirement txt, maybe there is a problem on it.
Based on your fixing process, if I understand correctly, I need to update the numba to be a higher version? Would you mind pulling a request in the "pull request" page to indicate a new requirement.txt? This will be more helpful
Thanks!!
best,
Ke


Downloading youtube clips in batch corrupts most of them

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I remove or skip any mandatory* field

Checklist

[X] I'm reporting a broken site
[X] I've verified that I'm running yt-dlp version 2022.08.14 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
Europe
Provide a description that is worded well enough to be understood
https://www.youtube.com/clip/UgkxEM49kpk9FMdDW69vvagwxDlieRHbpmxI
https://www.youtube.com/clip/UgkxQ6ma88aW00WSoEJgdB0UUOGeZb5RLr7v
https://www.youtube.com/clip/Ugkx13AL-Q5R64es04_yB8GZB5g8OWOj1Mfn
https://www.youtube.com/clip/UgkxZRxphg5jf2GSF2NyqS7V83Pxkw2Jfnz_
https://www.youtube.com/clip/UgkxeHo_q9GdYWcZ6hnwU919-Xym7k_WJnAN
https://www.youtube.com/clip/Ugkx3dfp-bJN2943maQHEmiHW61yg0qjyLX1
https://www.youtube.com/clip/UgkxB9dgKFHXfwJuRrxPUeUdHCqU7ylgy6G3
https://www.youtube.com/clip/UgkxSXpM-UwxS82raEvCjAwnT56Zt2XAIjSI
https://www.youtube.com/clip/Ugkxw_9-sALttF3niOuXEFIU_v4wei7EtJx_
https://www.youtube.com/clip/UgkxOpVG82zDwPt3V4ZmqCmIqxjfOIDHcRxU

these are the clips i attempted to download with a simple
yt-dlp -a batch.txt --remux-video mp4

UgkxQ6ma88aW00WSoEJgdB0UUOGeZb5RLr7v doesen't have video for the first part of the clip
Ugkxw_9-sALttF3niOuXEFIU_v4wei7EtJx_ doesen't have video for the first part of the clip
UgkxB9dgKFHXfwJuRrxPUeUdHCqU7ylgy6G3 doesen't have video for the first part of the clip
Ugkx13AL-Q5R64es04_yB8GZB5g8OWOj1Mfn doesen't have video for the first part of the clip
UgkxZRxphg5jf2GSF2NyqS7V83Pxkw2Jfnz_ doesen't have audio for the first part of the clip
UgkxOpVG82zDwPt3V4ZmqCmIqxjfOIDHcRxU doesen't have audio for the first part of the clip

config
--ffmpeg-location D:/coding/yt-dlp/ffmpeg-master-latest-win64-gpl-shared/bin/ffmpeg.exe
--audio-quality 0
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
https://pastebin.com/VP6GKicx

Use --force-keyframes-at-cut

will try this once im at the pc, but please mention this in the documentation that yt clips should be downloaded with this option.

with this option the downloading is extremly slow. 4 or so 40 second videos have been downloading for over 20 minutes and only one is downloaded. is there an alternative option?

No. We cannot accurately cut clips without reencoding (which is slow)


Copy the example environment file:????

Copy the example environment file: ???
env.example???----> .env

.env is removed from the project because the project is not heavy and very, very simple.


proposal to change representation of attributes and units

Currently datatypes are defined like this:
cim:ActivePower  rdf:type       rdfs:Class ;
        rdfs:comment            "Product of RMS value of the voltage and the RMS value of the in-phase component of the current." ;
        rdfs:label              "ActivePower"@en ;
        cims:belongsToCategory  er:Package_EquipmentReliabilityProfile ;
        cims:stereotype         "CIMDatatype" .

cim:ActivePower.multiplier
        rdf:type           rdf:Property ;
        rdfs:domain        cim:ActivePower ;
        rdfs:label         "multiplier"@en ;
        rdfs:range         cim:UnitMultiplier ;
        cims:isFixed       "M" ;
        cims:multiplicity  cims:M:0..1 ;
        cims:stereotype    <http://iec.ch/TC57/NonStandard/UML#attribute> .

cim:ActivePower.unit  rdf:type  rdf:Property ;
        rdfs:domain        cim:ActivePower ;
        rdfs:label         "unit"@en ;
        rdfs:range         cim:UnitSymbol ;
        cims:isFixed       "W" ;
        cims:multiplicity  cims:M:0..1 ;
        cims:stereotype    <http://iec.ch/TC57/NonStandard/UML#attribute> .

cim:ActivePower.value
        rdf:type           rdf:Property ;
        rdfs:domain        cim:ActivePower ;
        rdfs:label         "value"@en ;
        cims:dataType      cim:Float ;
        cims:multiplicity  cims:M:0..1 ;
        cims:stereotype    <http://iec.ch/TC57/NonStandard/UML#attribute> .

And then attributes (data props) are defined like this:
cim:GeneratingUnit.maxEconomicP
        rdf:type           rdf:Property ;
        rdfs:domain        cim:GeneratingUnit ;
        cims:dataType      cim:ActivePower ;
        cims:stereotype    <http://iec.ch/TC57/NonStandard/UML#attribute> .

There are several problems:

the data prop doesn't define a range (while the value of GeneratingUnit.maxEconomicP in instance data is a a pure number)
cim:ActivePower.unit and cim:ActivePower.multiplier are never used
cim:Float is a non-standard datatype (eg xsd:double is a standard datatype)

Proposals:

[ ] Define range of attributes:

cim:GeneratingUnit.maxEconomicP rdfs:range xsd:double.

[ ] Define them as datatype props. Using this OWL class doesn't mean you're going away from RDFS, you're just being more specific about the nature of the property

cim:GeneratingUnit.maxEconomicP a owl:DatatypeProperty.

[ ] Remove cim:ActivePower.unit and cim:ActivePower.multiplier since they are not used (but if you like them, you can keep them)
[ ] Attach the unit and multiplier to cim:ActivePower, eg

cim:ActivePower
  cim:multiplier "M";
  cim:unit "W".

[ ] Attach the unit and multiplier to the attribute, eg

cim:GeneratingUnit.maxEconomicP
  cim:multiplier "M";
  cim:unit "W".

[ ] Switch to QUDT. The string values above are not very good since they can't carry descriptive info. So you can replace the above with

cim:ActivePower qudt:unit unit:MW.
cim:GeneratingUnit.maxEconomicP qudt:unit unit:MW.

https://github.com/Sveino/Inst4CIM-KG/issues/38 describe this in more detail, and rdfs-improvement/README in even more detail:
- Datatypes and Units of Measure
- Fixed Units Representation
- CompleteDatatypeMap
- Actual QuantityKinds
- Actual Multipliers and Units
The last sec has mapping tables to QUDT.
But this issue has a task list, so I'll close it only after everything is done.


feat: Change twig include namespace to @hanna

Part of: https://github.com/rvk-utd/reykjavik/pull/523
Related to: https://github.com/rvk-utd/hanna/pull/36

Hi @maranomynet - I have created a new one PR with all needed changes to use new hanna namespace defined in drupal.
https://github.com/rvk-utd/hanna/pull/39
I think we can close this PR


Remove complexity

This PR:

removes odd logic around custom metadata extraction (utils.js)
removes side effects (i.e. sharing global config and tilesets) in mbview.js
keeps the MBTiles schema and all of its fields
replaces metadata fixture with a real OSM QA example

TL;DR a simpler way of handling the mb sources before passing them to the server (+75 −86)

@aaronlidman what do you think about this refactor? The basic idea is to remove weird logic so we can move faster on the upcoming features.

👌 Much simpler config, looks good

awesome 🚀 merging...


Arrays Holding Multidimensional Windows of Streaming Data

In dealing with vectorised time series it might be great to be able to efficiently represent such structures with both push and / or pull evaluation.

@sirinath thanks for your feedback.
Can you elaborate on what you mean by push and pull evaluation?

Push - when new data become available this triggers evaluation
Pull - when new items is needed (pulled) the rest is evaluated
Also ideally you should be able to have time based dimensions across where data is streamed.

I think this should be implemented in a side-project as an extension to xtensor. A possible solution would be adapters on these times series that permits to plug them in the xtensor expression system and trigger the evaluation when needed.
@sirinath I close this issue since it's not something we plan to implement here, but I keep the idea in mind in case I have some time to implement it. In the meanwhile, feel free to propose an implementation if you have some time to do it.

Is it possible to provide some rudimentary support for holding and computations on time series, vectorised time series data. I think this can be best implemented as array which has circular indexing. As new data comes in you append the vector along a dimension.


Update requirements.txt

had to pip install jsonschema mistune to get d to work

I have no idea what either of those things are.  d certainly doesn't need them.
I just tried pip install -r requirements.txt; python setup.py develop in a fresh virtualenv and then running d works fine, so I'm not sure what was happening for you.  If you can post the actual error you were seeing maybe something will jump out.


Feedback

global:

A lot of these methods are one-liners and can comfortably use the expression-body syntax.

Option.cs:

None should really be a public static readonly value.  There's no need to keep recreating this value.
_hasValue can easily be made a public immutable auto-property, unless you really want to force consumers to treat an Option in a functional manner.
Silently accepting null inputs feels a bit ironic for an Option type.  What is the use case for something like a null Do action and why is it OK that it silently ignores it?
Have you considered a non-generic static wrapper class for Option.Some to use type inference?  It would let you write Option.Some(value) without bothering with specifying the type.
There's no way to get the underlying value out again if you want to leave functional-land?  It seems pretty common to see a GetValueOr( Func<T> defaultGenerator ) method on these types.
For types like this, I find the use of DebuggerDisplay to be really nice when you're debugging.  Pro-tip:  You can call ToString() from the attribute in case you get fed up trying to format it correctly using its weird one-off syntax...

Result.cs:

I don't know that picking one-letter generic type names is really helping here.  A lot of these methods would be easier to read at a glance if they were more wordy; i.e. TSuccess and TFail.
A non-generic static wrapper might potentially really clean up Result.Success/Result.Failure calls...
Using a mutable failure list seems questionable. Do you really want callers to be able to mutate a result?  Either IEnumerable as a quick fix or ImmutableList (if you really want list semantics) seem more appropriate.

@davidaramant - Wow, there are a lot of great suggestions here. One question though, on the subject of a wrapper class for Option, I get that we could have Option.Some(2), but for None, it'd still look like Option<int>.None, which seems a bit inconsistent to me. Thoughts?

I think the best you can do is Option.No<int>().  It's consistent with the Some version in that the "argument" (in this case a type) is after the action.  Using 'no' instead of 'none' makes the call read a bit nicer too.
Accessing None directly (outside of the guts of the Option type) is probably a pretty rare occurrence anyway.

I hadn't thought about renaming None to No, but you're right, it does read better. Thanks for the tip!

I was able to cleanup the Result.Apply calls with the wrapper, but I'm not sure how to deal with cleaning up Success/Failure calls because at the end of the day, the caller needs to specify the two types.


No validation set in openwebtext leads to failure.

Describe the bug
After building index for openwebtext, building the trainer fails (at line 161 of train.py) because no validation dataset is constructed. I believe this is because the lm_dataset object is built with huggingface's load_dataset on the openwebtext named dataset, and it has no validation split. The validation_ratio quinine config option is only used in building the custom_eval_datasets, not the lm_dataset object, so it is not used to portion out part of openwebtext as a validation set.
To Reproduce
Replace datasets/wikitext2.yaml with datasets/openwebtext.yaml in mistral-micro.yaml (and make other artefact location changes) and run
deepspeed --num_gpus 4 --num_nodes 1 --master_addr machine1 train.py --config conf/mistral-micro.yaml --nnodes 1 --nproc_per_node 4 --training_arguments.fp16 true --training_arguments.per_device_train_batch_size 4 --training_arguments.deepspeed conf/deepspeed/z2-small-conf.json --run_id repro-bug-openweb-novalid

Expected behavior
No failure occurs at line 161 of train.py when lm_dataset['validation'] is expressed.

I think this is because David changed this from the original. It looks like https://github.com/stanford-crfm/mistral/blob/bf9eff08e83f4d5703b69dfcb6c18e8e35a00a6d/src/corpora/auto.py#L94 does the right thing ? So if you just build the conventional Hugging Face cache (instead of David's custom index) ... with get_auto_dataset it should work fine and create a standard tokenized Hugging Face cache with an extra validation set.

Basically older Mistral just built a conventional Hugging Face cache and David created a new custom data handling setup and I guess didn't add in creating a validation set ...

I've started this branch: https://github.com/stanford-crfm/mistral/tree/mistral-flash-dec-2022
This should have Mistral Feb 2022 code + some bug fixes + has worked with Flash Attention

I'm on vacation mode but I am happy to help you get this branch working ... you will need to install flash attention and a specially modified Hugging Face as well ...

Some instructions on getting this working, (remember use branch: https://github.com/stanford-crfm/mistral/tree/mistral-flash-dec-2022)

The standard Mistral environment should work, but note you need to install a custom transformers, so delete transformers from the pip-requirements.txt file when setting up your environment ...

conda create -n mistral python=3.8.12 pytorch=1.11.0 torchdata cudatoolkit=11.3 -c pytorch
conda activate mistral
pip install -r setup/pip-requirements.txt

I think this will work with newer PyTorch, etc ... but you need to make sure you build Flash Attention with whatever you are using ...

Install transformers from Git (https://github.com/huggingface/transformers), replace src/transformers/models/gpt2/modeling_gpt2.py with the version checked into this branch in the transformers dir in the top level directory of this repo

When creating an environment, make sure to install Flash Attention: (https://github.com/HazyResearch/flash-attention) ...  you made need to roll back to this commit: f515c77f2528b5062ebcc6c905c8817ca0ac0ad1 ... last time I tried to get this working it wasn't because of issues with newer versions of Flash Attention but they may've been resolved in main by now ... but I rolled back to that commit and it was fine ...

Please let me know if you run into any issues and we can clean this branch + instructions up ... but if all goes well should get super fast Flash Attention GPT2 training which is something like 2x faster ...
In the future we should think about reconciling this branch with current main ... but if you just want something working in the next day this is quickest route ...

Sample command:
Note add a file called hostfile in the top level directory (even a blank one if just using one machine)
deepspeed --hostfile hostfile --num_gpus 8 --num_nodes 1 --master_addr sphinx4 train.py --config conf/your_config.yaml --nnodes 1 --nproc_per_node 8 --training_arguments.per_device_train_batch_size 16 --training_arguments.deepspeed conf/deepspeed/z2-small-bf16-conf.json --run_id mistral-w-flash-demo

You need to use bf16 ... a bad feature of this branch right now is this is just hard-coded here: https://github.com/stanford-crfm/mistral/blob/3a7dfac4836e760b6a3afb880f8f79585c357281/src/args/training_args.py#L67
So it'd be a good idea to make this more transparent ... this branch is sort of my personal experimentation that I got running and could use some clean up ...

Flash Attention requires bf16 or fp16 ... and you need bf16 for the stability ...

@J38 Hello, I also had the same issue of coding not working for openwebtext due to missing validation set, so I tried your solution above. But I encountered the error "ImportError: cannot import name 'LMDataCollator' from 'src.core.trainer'".
It looks like the src.core.trainer file in the branch https://github.com/stanford-crfm/mistral/tree/mistral-flash-dec-2022 does not have a class called LMDataCollator. Could you please help look into that?

Can you provide more details about what is causing that error (e.g. what line is failing in what file)? The branch is older code before changes were made, so it should not require LMDataCollator. Are you pre-training from scratch or trying to fine-tune a model trained with main branch code?

I guess it is this line: https://github.com/stanford-crfm/mistral/blob/3a7dfac4836e760b6a3afb880f8f79585c357281/train.py#L39

Yes it is this line, and I believe LMDataCollator is used in line 158 of this file. But I'm able to fix the dev set problem by adding a few lines of code on the main branch so I think the issue is resolved.

I tried reverting train.py to the February 2022 version, does that help ?


Regular Polygon feature

Addresses #31

I'm not a fan of the IsRegular property. I think a cleaner solution is making Sides a default value of 4. Wherever IsRegular is used would then become a check for Sides being 4 or not.
Also perhaps an additional check needs to be added that throws an ArgumentException when sides is not 3 or more. There's already a similar ArgumentException for background gradient angle which needs to be 0-360. Does this also apply for your OffsetAngle?

As I've mentioned on the issue, sides by default is 4, but IsRegular switches it from begin a Rectangle to regular polygon - Square
Will add the checks on Sides. OffsetAngle, well its an angle, can be anything, above 360 means -  value % 360. Will add checks just to be in consistent with gradient angle.

One of the things I've been struggling with lately is figuring out a way to check whether or not all of the supported properties work and work when combined with one another. Adding polygons adds a whole new dimension to that as well. A lot of scenarios to support. The DebugPage was a start for that, but maintaining that manually becomes a chore. Will look into something coming up with something for this myself.

I was also thinking on the same lines.. hence the new Polygon try out page. Still needs a lot of work in adding toggle for all the features...

Yeah, I really like that idea. Will try to expand upon that too for the normal pancakes.

Do you mind if I start expanding on your idea for the debugging page for the "normal" PancakeView? Perhaps we can at some point merge them into one big debugging page with all the options?

Sure, please go ahead!
Love the monkey face!!

If all is well, this new debug page should now be feature complete. Some properties aren't working, which is nice to see, because that tells me where I/we still need to do some work 😂

Wow, now THIS is a Debug Page!
I can see lots of things that are not supposed to happen 😄
Just to be safe I reverted back to master and opened this page, some things are real messed up!! Especially with gradients. I guess no one's changing gradient at runtime.
We need to fix them before even thinking of merging this PR 😞

I've been testing on iOS and I think quite a bit of things are working better than I had actually expected 😅That'll probably be my first focus, since I know the iOS code better.

@shanranm - I fixed almost all the iOS rendering bits. The only scenario that is still giving problems right now is the non-square + gradient border + small corner radius at some rotations. Gradient borders were a tough nut to crack and I've had to give up on BorderDrawingStyle for now.

And that sir, should fix everything.
well, of course leaving out the BorderDrawingStyle

You mean fix everything on both platforms or just iOS? I just tested iOS and that works like a charm! Thanks for all the hard work so far, I really appreciate it. Couldn't have done it without you 😃
Moving on to Droid; Running the debug app on Android on a GenyMotion emulator I get some really weird behavior. Not sure if that's a good test case though, but I don't have my physical Android device near. All the sliders in there are also kind of behaving weirdly 😂 Just some of the things I see:

BackgroundGradient not working.
Dashed border behaving weirdly.
Clipping + border produces no border.
Sides/offset angle seem to misbehave.

I swear I fixed on Android too 😄
May be dependent on Android version?
I did debug on a physical device with version - 8.1.0
Still, will take a look again

Yeah, like I said, I'm not sure how good my test setup is for this. I'll have access to a physical device tomorrow, so will also recheck then.

I was able to reproduce all of them on an emulator.
For some weird reason, Draw is not called on PancakeDrawable when BackgroundColor is changed. To handle this, new drawable is created (also disposing the older one)

Cool, that last update fixes a lot! Is it just me though or are these sliders rendering very weirdly and  very un-userfriendly (no track, very hard to control)?

Also, seems like the background bleeds out from under the border in some cases?

I also just fixed a small bug where there would be a minuscule border when it's set to 0.

Yea, sliders have issue on Android, had to manually set WidthRequest for debugging.
Just set it as style, will be applied to all elements on the page.
The background leaking have always been there... i'd like to rewrite another create path specifically for sides == 4 case... may be another PR?

Agreed. Let's merge this one!
Once again though, let me make clear that I really appreciate what you've done in this PR! It truly is an awesome addition 😃

It was my pleasure!! 😄
I don't think it would've been come this far without your active and continuous support!!


ChatGPT 4 chatbot not working

when I input "Hi", the following message appears:
### Additional information: I saw an authentication error message flash through before the chatbot loaded.
Error
There was an error generating a response. Chat history can't be saved at this time. If the problem persists, please contact the site administrator.

Could you check on the console if you have any error?
I have the same issue and I see an error saying:
net::ERR_INVALID_CHUNKED_ENCODING 200 (OK)

Could you check on the console if you have any error? I have the same issue and I see an error saying: net::ERR_INVALID_CHUNKED_ENCODING 200 (OK)

Where can I check the console? Or, where is the console?

I'm referring to the console in the developer tools of the browser, just press F12 or go to the settings -> more tools -> developer tools.

I see. There are more errors in my instance.
Edge:

Chrome:

Playground does not even function properly. Could that be OpenAI cut the traffic from Azure?
Can someone comment on this? Thanks.

Now, cannot deploy. After click on Deploy, the deploy window has been frozen there for more than 10 minutes. GPT4 works just fine and better than well on my local machine projects where I call OpenAI API. I guess I should go back to that instead of Azure OpenAI.

I did the following and seems to working:

Scale up to P3V1
Switch to East US 2 or East US. I had Central US previously, I believe.

We're still experiencing this issue after deployed and it seems to happen when the response or prompt would violate the content filter rules. From what we can tell this only started happening after we enabled Chat History. Prior to that we actually got the message saying it violated the content filter.


Deleting hash prevents newly generated one to show up

If you delete the hash from the right textarea, you won't be able to generate a new one. See photo.

@Slacki Refreshing the page should solve this for you temporarily while you wait on a fix

@Slacki this has been fixed, you will need to do a hard reload to refresh main.js


Cannot reproduce the results in the paper

I run the training code with command provided in README, the results are lower than that in the paper.

Accuracy
SAT
Reproduce

Nr3d
49.2
46.0

ScanRefer
53.8
51.4

Do you have any idea about where the gap comes from?

Hi, is there any command for ScanRefer? CAN not find in README. Thanks!


consider static linking

I was wondering if you've considered statically linking to XLS c_api?
Ideally that could allow us to detect breakage to incompatible API/signature change at build time rather having to execute a battery of tests to ensure that all dynamically loading path are covered.
What do you think?
Reference:

https://doc.rust-lang.org/cargo/reference/build-scripts.html#rustc-link-lib
https://doc.rust-lang.org/rustc/command-line-arguments.html#-l-link-the-generated-crate-to-a-native-library

wrapping the c_api.rs functions in extern "C" block as described in https://docs.rust-embedded.org/book/interoperability/c-with-rust.html and reusing the the code to unmarshal/marshal the arguments/return values should be enough to allow linking to //xls/public:c_api statically.

I think the benefit of the shared library conceptually is that you could make a binary release of the xlsynth-crate independent of the shared object's binary release, but if we consider environments where source is always available it shouldn't make a big difference.
I imagine we could make a build option to either use a dynamic loading c_api.rs or a statically linking one.
Note that static vs dynamical linking does not help with signatures -- to ensure consistency they'd need to be converted from some source of truth, e.g. a C header specification, and we're trying to avoid relying on anything from C++ build time in this crate other than output artifacts.
If having a static linking mode would be helpful for the crate usability we can consider adding that (e.g. behind a feature flag), but I don't think it has any particular advantages for use cases I'm aware of, given we're going to keep the C++ build step (that produces the library) separated.

I think the benefit of the shared library conceptually is that you could make a binary release of the xlsynth-crate independent of the shared object's binary release, but if we consider environments where source is always available it shouldn't make a big difference.

We could also release a static version of libxls.a with accompagning C headers if we wanted to preserve this property.

I imagine we could make a build option to either use a dynamic loading c_api.rs or a statically linking one.

From a create usuability stand point, would it make a difference if it was referring to a dynamic loading c_api.rs or a statically linking one?
One difference I could see is that symbol resolution could happen at build time (so on cargo install) rather than at execution time, but if crate are pinned to specific version of the output artefact (as you pointed later) that shouldn't really be an issue for the end-user (as long as maintainer ensure that all dynamically loading path are properly covered by tests).

Note that static vs dynamical linking does not help with signatures -- to ensure consistency they'd need to be converted from some source of truth, e.g. a C header specification, and we're trying to avoid relying on anything from C++ build time in this crate other than output artifacts.

The rust ecosystem seems to have tool like https://github.com/rust-lang/rust-bindgen which support generating binding for C API, but my initial thought was the binding signature could be maintained manually in https://github.com/xlsynth/xlsynth-crate/blob/main/src/c_api.rs simarly to what it does for the dynamic symbol lookup today.

If having a static linking mode would be helpful for the crate usability we can consider adding that (e.g. behind a feature flag), but I don't think it has any particular advantages for use cases I'm aware of, given we're going to keep the C++ build step (that produces the library) separated.

The static approach would definitly be easier to adapt to Google internal build environment.

Sounds good, as I mentioned I don't think we benefit much from dynamic linkage given it's all source-available on crates.io, so pulling from a versioned .a release in build.rs and adding it to the linker flags from there SGTM as a change if it works!

I mentioned I don't think we benefit much from dynamic linkage given it's all source-available on crates.io

Do you envision maintaining both, or just keeping the static linkage variant? I have a proof of concept of the later here https://github.com/xlsynth/xlsynth-crate/commit/b951a6ecfb62edfd1d497b7e6e1c9cfdc6c6a362 that I'm happy to submit as a PR and iterate on.
In particular I'm not sure how it affects the MacOSX build path.

Let's have both behind a cargo feature for now until we can firm up the OS X side, but I imagine it'd work there (or would be similarly easy to make work there) as well once we have the Linux side going.

as commented in https://github.com/xlsynth/xlsynth/pull/1; it turns out bazel is currently limited in it's ability to construct standalone .a artifact https://github.com/bazelbuild/bazel/issues/1920 (that's being addressed in https://github.com/bazelbuild/bazel/pull/16954)

Managed to get dynamic loading working in google's internal environment, though even if it would be nice to support static linking it's not strictly required anymore.

If we've got something working let's consider it just nice-to-have? We can always revisit as we find it's more needed.


Not able to make the plugin work

Lastest plugin version, lastest Craft version and got this log :
2019-04-08 09:06:08 [-][3][-][error][timkelty\craftcms\sitesync\Field::normalizeValue] Syncable failed validation: Array
(
    [sources] => Array
        (
            [0] => Sources est invalide.
        )

)

I dont have any 'Sources' field so I guess it coming from the plugin (or a conflict with another).
I use SEO, SuperTable and Neo on the section/entry type I need to sync. I test with a new/empty entry and still not able to sync my content.
"Sources est invalide." translate to "Sources is invalid"

Thanks @jesuismaxime, I'll have a look today. If you could share a db dump and your composer.json that would help! I'm on the Craft Discord, or email me at<EMAIL_ADDRESS>

Just tried to install the plugin, which worked. But I can no longer add any fields, as there is the following error being thrown:

@MisterMike make sure you have the latest: 1.0.2

@timkelty Thanks for the fast reply. Okay, will try. Currently composer does not let me get 1.0.2 despite "^1.0" and I can also see it on packagist..

"platform": {       "php": "7.0"     }
I set it to 7.2 and it worked. Thanks a lot!

This was fixed by upgrading to the latest version

/reopen


⚠ ClamAV has degraded performance

In c9cbda8, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 81 ms

Resolved: ClamAV performance has improved in d85dc08.


[CELEBORN-133] Improve snapshot loading

[BUG]/[FEATURE] Improve snapshot loading
What changes were proposed in this pull request?

add a pre-check to reject obsolete snapshots
throw exception and fail immediately when snapshot is not loaded

Why are the changes needed?
Snapshot loading happens when:

A server restarts from crash.
A new Server joins the group.
In both situation, when snapshot loading is failed, the server is not able to service. We should throw exception and fails immediately.

/cc @related-reviewer
/assign @main-reviewer

@pan3793 @waitinfuture  Thanks for the reviews! Made corresponding changes on code, PTAL~


reduce passing of byte arrays

Forked PR for https://github.com/cihub/seelog/pull/126.

Hey @tstearns, I opened a PR with my latest approach here: https://github.com/jonsyu1/seelog/pull/4
Feel free to peruse this and share feedback.


Updated PR for BinaryBandits-challenge-4

Linked to https://github.com/skill-a-thon/.github/pull/47


conceptual uop st cleanup [pr]

setitem ASSIGN isn't considered a Buffer op, maybe it should be. Just replace STORE with a swizzled ASSIGN, why parent a STORE with ASSIGN and give ASSIGN an arg?


Do QuadTree bounds matter?

When all the data was being rescaled, I created the quadtree like this:
QuadTree(-1500, -1500, 3000, 3000);

... and then I changed the scaling, so the data is well outside those bounds, but everything still seems to work?

Is there a performance penalty to not giving the correct bounds? Right now, I don't know the dimensions of the data, but I do need to figure that out so that we'll load with the correct viewport without hard-coding.
If there is no performance penalty, what are they used for? Perhaps file an issue with the library.

I made this small example on CodePen to verify that put and get do work with points outside the boundary specified at instantiation https://codepen.io/kellermark/pen/BgjNqb
Next I will check in the simple-quadtree code to look for a performance penalty or a place where the bounds are used.

A quick performance test is here https://codepen.io/kellermark/pen/RzrWOw?editors=1011
which strangely results in usage of incorrect boundaries often slightly faster than with correct boundaries.

Would it be useful to try with d3's quadtree implementation?

Before spending too much time here, I think we should pop up and confirm that Deck.gl's picking still has problems (sort-of under #111). It's mechanism that should be fast... but it was blowing out memory for me. They've had a new major release since then, and I may also just have been using it wrong.

(but if swapping in d3 is quick, feel free to try that, too.)


Parallel Execution : Log goes to wrong test

Hi Team,
When I do parallel execution with multiple projects. Logs are captured at the wrong test, if I try to add attachments to the log across multiple tests, It all gets added to a single test.
Please provide me a solution.

Hello @shiva4983 !
What API are you using to report logs and attachments? I mean testInfo.attachments and console.log or ReportingAPI?

Hi @AmsterGet , I am using ReportingAPI..

Hi @shiva4983 !
I've released the version 5.0.8 with fixes that might help you. Please try it out.

Hi @shiva4983 !
Have you had a chance to try the latest version?

@shiva4983
I will close the issue. Feel free to reopen it if you run into a problem again.


Generate unique States/Country with full : true

Hey, I'm trying to generate list of all the states with full:true but this doesn't work..
chance.unique(chance.state({ full: true }), 1);

Am I missing something or this is not yet supported?
Great script btw

If you just want to list all states that Chance knows, you can use chance.states():
chance.states().map( function( state ){ return state.name; } );

Thanks @oliversalzburg for the assist on the answer there!
Also @rasteron if you are trying to generate just the full name of 1 random state as your code snippet seems to imply you can do that with Chance by passing the option after the number to generate. For example:
// Will generate an array with 1 full state name, e.g. ["Nebraska"]
chance.unique(chance.state, 1, { full: true });

The first argument to chance.unique must be a function, not the result of a function. The problem with your code snippet is that chance.state({ full: true }) will return a state name "Nebraska" which is a string, not a function, so an error is thrown* because Chance cannot use the string "Nebraska" as a function to generate any things, let alone unique things.
* I notice the error thrown if that first argument is not a function sucks, fixing it now!
Marking this closed as it appears there is no bug here though I will be fixing the crappy error message when the first argument is not a function.

Thanks guys, I got it working now. :+1:

@rasteron If you only want one, why not just use chance.state( { full : true } )? Or if it needs to be an array [ chance.state( { full : true } ) ].
Seems a bit lighter than using chance.unique.

@oliversalzburg Yes, as I have posted I'm using it to generate a unique random full name states and in a loop. Just tinkering with the library so the chance.unique solution will do for now.

If you're using that code in a loop, the results are not guaranteed to be unique.
If you know that you'll require 10 unique states throughout the loop, you should generate an array of 10 states with chance.unique(chance.state, 10, { full: true }); and then use that array in your loop.
In case you were already aware of that, then I'm sorry for being redundant :)

Yep, no worries and thanks again Oliver! :smiley:

Hey is there any way to get something like this ? chance.address.country({ raw: true }) to get same country's name and its code ?


takeEvery not taking every action?

Hello,
I ran into a problem with takeEvery helper. I am trying to catch all API actions with a "watchApiCalls" saga and then process them in a "apiCall" saga. I only wanted to try, how it will go with "WillMount" lifecycle method as stated below. When I refresh the page, where this component is, I got first log of actions - only 1 is "taken" and processed. The funny thing is, when I go to the page via transition / link, as can be seen in second screenshot, everything goes ok.
Do you have any clue why is that or how to fix this?
Thank you in advance.
export function* apiCall(action) {
  const { api, payload, types } = action;
  const token = yield select(getToken);

  return yield call(fetchApi, api, types, payload, token);
}

export default function* watchApiCalls() {
  yield* takeEvery((action) => action.api, apiCall);
}

componentWillMount() {
    if (!this.props.machines.isLoading && !this.props.machines.isLoaded) {
      this.props.fetchAllMachines();
    }
    if (!this.props.cartons.isLoading && !this.props.cartons.isLoaded) {
      this.props.fetchAllCartons();
    }
    if (!this.props.papers.isLoading && !this.props.papers.isLoaded) {
      this.props.fetchAllPapers();
    }
  }

Are you logging the actions from reducers or some middleware like redux-loggger?

Redux-devtools + Redux-devtools-inspector
On May 3, 2016 9:47 AM, "Yassine Elouafi"<EMAIL_ADDRESS>wrote:

Are you logging the actions from reducers or some middleware like
redux-loggger?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/yelouafi/redux-saga/issues/296#issuecomment-216460639

if it doesn't reach the devtools it's likely because it isn't dispatched at all. Unless some middleware intercept it.

Sorry, I forgot to mention, that actions with suffix "#BASE" have "api" property, so as You can see, these are dispatched from the component correctly in botch cases, but in the first one "watchApiCalls" saga will not start "apiCall" saga for last 2 - will start only for the first one, which results into #START and #SUCCESS actions.

sorry for the delay. Can you post a runnable Gist or a test case? it's hard to tell from the snippet above

No problem,
the thing is, that I made this for showcase: http://www.webpackbin.com/VyiajQE--
And it is running smoothly, when I tried to reproduce it in my codebase again (after clean npm install) it is running also ok. So I do not have a clue, where was a problem and I am closing this, since now it is ok. If I come across it again, I will report it here .)


where can i find the fields in the header of the invoice?

Feature request

Extension name: Send Invoices using Stripe

Is your feature request related to a problem? Please describe.
where i add this fields ?
{
  email<EMAIL_ADDRESS>  my header.name    <--- add
  my header.myaddress    <--- add
  my city....    <--- add
  my vat number...    <--- add
  invoice.customer_address.line1...    <--- add
  invoice.customer_email.line 2....    <--- add
  invoice.customer.vat number....    <--- add
  invoice.customer_name    <--- add
  invoice.customer_phone    <--- add
  invoice.customer_shipping    <--- add
  items: [{
      amount: 1999,
      currency: "usd",
      quantity: 2, // Optional, defaults to 1.
      description: "my super cool item"
  },
  {
      amount: 540,
      currency: "usd",
      description: "shipping cost"
  }]
}

Describe the solution you'd like
adding this field to invoiceCreateParams

@waste-app The fields mentioned here are not created on the invoice.
According to the Api....
customer_name
string
The customer’s name. Until the invoice is finalised, this field will equal customer.name. Once the invoice is finalised, this field will no longer be updated.

These properties are specific to the customer provided by en email or uid.
There are multiple scenarios that would include an update....

Non existing customers could include the new fields when they are created for the invoice.

If included, the customer could be updated with the new fields before sending the invoice.

Alternatively, if the customer already exists then the invoicing would default to the customers original details.


remove postgres

with the new thisassetstotal addition in the algod api we dont need a database anymore.

Boom.


Fix "Make GitHub Release" on TeamCity

It currently fails to create a release with gh due to auth issues:
18:52:00   + pixi run github-release
18:52:00   ✨ Pixi task (default): python utils/github-release.py
18:52:51   HTTP 404: Not Found (https://api.github.com/repos/Deltares/Ribasim/releases
18:52:51   This API operation needs the "repo" scope. To request it, run:  gh auth refresh -h github.com -s repo
18:52:51   Traceback (most recent call last):
18:52:51     File "/opt/teamcityagent/work/d7117c5942be1e0d/utils/github-release.py", line 12, in <module>
18:52:51       subprocess.check_call(
18:52:51     File "/opt/teamcityagent/work/d7117c5942be1e0d/.pixi/envs/default/lib/python3.11/subprocess.py", line 413, in check_call
18:52:51       raise CalledProcessError(retcode, cmd)
18:52:51   subprocess.CalledProcessError: Command '['gh', 'release', 'create', 'v2024.3.0', '--generate-notes', 'ribasim_cli_linux.zip', 'ribasim_cli_windows.zip', 'ribasim_qgis.zip', 'generated_testmodels.zip']' returned non-zero exit status 1.
18:52:51   Currently checked out tag: v2024.3.0

https://dpcbuild.deltares.nl/buildConfiguration/Ribasim_Ribasim_MakeGitHubRelease?mode=builds
I worked around it for v2024.3.0 by running the same command locally with the right artifacts downloaded from TeamCity:
gh release create v2024.3.0 --generate-notes ribasim_cli_linux.zip ribasim_cli_windows.zip ribasim_qgis.zip generated_testmodels.zip

That is fixed by now