Can't run the script

Hey Antoine!
Thank you so much for your awesome work!
I was trying to download the new repo  after I did npm install   and  npm start
it throws me error  can't find module npm
module.js:341
throw err;
^
Error: Cannot find module 'npm'
at Function.Module._resolveFilename (module.js:339:15)
at Function.Module._load (module.js:290:25)
at Module.require (module.js:367:17)
at require (internal/module.js:16:19)
at Module._compile (module.js:413:34)
at Object.Module._extensions..js (module.js:422:10)
at Module.load (module.js:357:32)
at Function.Module._load (module.js:314:12)
at Module.require (module.js:367:17)

Hum, that's weird ...
Maybe you can try to upgrade npm globally by entering the following command in a shell:
$ npm install -g npm
You may need to run that command as an administrator (sudo ...)
On my side, I used Ubuntu as my operating system coupled to node 6.X and everything works just fine.
You can also try to remove the 'node_modules' folder of the project and do 'npm install' again

I just update my npm and node to 6X.   and i did sudo npm install -g npm   i am using MAX OX.
Here is the log.
0 info it worked if it ends with ok
1 verbose cli [ '/usr/local/bin/node', '/usr/local/bin/npm', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prestart, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/bin/node-gyp-bin:/Users/kanghuiliu/Downloads/node6.2express/node_modules/.bin:/usr/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /Users/kanghuiliu/Downloads/node6.2express
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'gulp run --NODE_ENV=production' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
14 verbose stack Error<EMAIL_ADDRESS>start: gulp run --NODE_ENV=production
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter. (/usr/local/lib/node_modules/npm/lib/utils/lifecycle.js:245:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess. (/usr/local/lib/node_modules/npm/lib/utils/spawn.js:24:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:850:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd /Users/kanghuiliu/Downloads/node6.2express
17 error Darwin 14.5.0
18 error argv "/usr/local/bin/node" "/usr/local/bin/npm" "start"
19 error node v6.2.0
20 error npm  v3.9.2
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>start: gulp run --NODE_ENV=production
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script 'gulp run --NODE_ENV=production'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the webpack-gulp-angular-express-demo package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     gulp run --NODE_ENV=production
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs webpack-gulp-angular-express-demo
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls webpack-gulp-angular-express-demo
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

Okay, it must be a MacOS related issue. I did not check the support of my project on that system.
Hopefully for you, I have got some MacOS VMs on VirtualBox. I will look at it tonight, I keep you posted.

Okay, problem solved. I found a more portable way to get the npm version from a node script.
git pull, npm install, npm run test-frontend and it should now work on MacOS.

Yes yes it works now !   you are so fast and so good at this!!!!  I really wish I could be as good as you are with web development.     already "fell in love with you" hahahaha
Thank you very much! 💯


Move code related to persistent settings used by RIDE and Mabot to the core framework

Originally submitted to Google Code by @pekkaklarck on 25 Mar 2010

Both RIDE and Mabot need to store settings persistently and we've
implemented generic code for that purpose. Currently both RIDE and Mabot
have their own versions of this code stored in their code base. It would be
better to move it to the core framework so that they, and other tools,
could use simply use it from robot.utils.
The version of this code in RIDE's code base can be found behind the URL
below. Notice that it uses external, BSD licensed, configobj module that is
located in the same directory.
https://code.google.com/p/robotframework-ride/source/browse/src/robotide/context/settings.py

I don't think this makes sense unless Robot itself uses these settings. Python packaging system has improved and this logic could be made available separately if needed. We should rather try removing extra functionality from the core framework.


Update Meta “authors”

Automatically generated by Netlify CMS

👷 Deploy Preview for e-commerce-nest processing.

Name
Link

🔨 Latest commit
0adc71cc901806f3de31c942ab7712100ab4d0d5

🔍 Latest deploy log
https://app.netlify.com/sites/e-commerce-nest/deploys/63c24ec9a740bb0008d5eb2a


[Bug]: Failed to export using Doris data source

ODC version
odc424
OB version

What happened?

What did you expect to happen?
Export successful
How can we reproduce it (as minimally and precisely as possible)?

Create Table  CREATE TABLE table2 (
siteid int(11) NULL
) ENGINE=OLAP
UNIQUE KEY(siteid)
COMMENT 'OLAP'
DISTRIBUTED BY HASH(siteid) BUCKETS 10
PROPERTIES (
"replication_allocation" = "tag.location.default: 1",
"storage_format" = "V2",
"light_schema_change" = "true",
"disable_auto_compaction" = "false"
);
Add data    INSERT INTO table2 (siteid) VALUES (1), (2), (3), (4);
export

Anything else we need to know?
No response
Cloud
No response

pass


[Decommission Hub] cloudbank/lassen

Summary
Due to inactivity
Info

Community Representative: sean-morris
Link to New Hub issue: before my time!
Proposed end date: 6/9/2023

Task List
Phase I

[x] Confirm with Community Representative that the hub is no longer in use and it's safe to decommission
[x] Confirm if there is any data to migrate from the hub before decommissioning

[ ] If yes, confirm where the data should be migrated to

[ ] Confirm a 2i2c Engineer has access to the destination in order to complete the data migration

[x] If no, confirm it is ok to delete all the data stored in the user home directories

Phase II - Hub Removal
(These steps are described in more detail in the docs at https://infrastructure.2i2c.org/en/latest/hub-deployment-guide/hubs/other-hub-ops/delete-hub.html)

[x] Manage existing data (migrate data from the hub or delete it)
[x] Remove the appropriate config/clusters/<cluster_name>/<hub_name>.values.yaml files. A complete list of relevant files can be found under the appropriate entry in the associated cluster.yaml file.
[x] Remove the associated hub entry from the config/clusters/<cluster_name>/cluster.yaml file.
[x] Remove the hub deployment

helm --namespace HUB_NAME delete HUB_NAME
kubectl delete namespace HUB_NAME

[ ] Delete the hub's authentication application on auth0, GitHub or CILogon

Phase III - Cluster Removal
This phase is only necessary for single hub clusters.

[ ] Run terraform plan -destroy and terraform apply from the appropriate workspace, to destroy the cluster
[ ] Delete the terraform workspace: terraform workspace delete <NAME>
[ ] Remove the associated config/clusters/<cluster_name> directory and all its contents
Remove the cluster from CI:

[ ] deploy-hubs.yaml
[ ] validate-clusters.yaml

[ ] Remove the cluster from the list of grafana datasources at https://grafana.pilot.2i2c.cloud/datasources
[ ] Remove A record from Namecheap account

@pnasrat I struggled to delete the Oauth piece.
First, the command came back as an error:

Apparently the flag --client_id should be --client-id

Second, the error message, once the above was fixed, said that I need to include: CLUSTER_NAME and HUB_NAME
I ran this:
deployer cilogon-client-delete --client**-**id cilogon:/client_id/ $CLUSTER_NAME $HUB_NAME
Then there was some sort of position argument error that I slacked you.
Thanks.

@GeorgianaElena, can you help @sean-morris with this one ⬆?

@damianavila I see what the issue is I'll file a PR

Have deleted the cilogon app for lassen


Feature/edit citation UI

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.


Scheduling rule fails to be met

Rancher Version:
1.2.2
Docker Version:
1.12.3
OS and where are the hosts located? (cloud, bare metal, etc):
Centos 7, Esxi
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node, internal db, upgraded from 1.1.x
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Try to add a scheduling rule to a load balancer
Results:
Upgrade failed, can't meet scheduling rule
Expected:
Upgrade to work
Screenshots:

LB at start

Host label for testing

Upgrading

Fails

Notes:

I have tried restarting the server, disabling cache, same results.
No XHR failing responses or any kind of console errors.

Version - 1.3.0
Labels are completely broken.

Fixed the dropdown, but if there's a problem with the labels actually being used that's going to be a separate problem in the scheduler..

Version - master 1/11
This has been fixed.  The labels are now in the drop-down and I can move the load balancer to another host using upgrade.  @kapolos This fix should be in the 1.4.0 build.

Awesome news, thank you both!

Still met this in 1.3.3

@twoeo the milestone is 1.4.0 for a reason.

@vincent99 1.4.0 works.
But why can not specify host ? (Remember can specify in 1.1)


Add movie and telco sentiment desc to dataset/README.md

telco_sentiment with source, paper and usage.
movie_sentiment with source and usage.

Do we have any model benchmark in paper https://ieeexplore.ieee.org/document/8554037

@dafajon update plz.

Test Macro-F1 Score is 69.25 as benchmark. Will report ours.

Document this into prebuilt readme doc as it is done for profanity and others


I think the calls to topicPointer() have been wrong for a while

🍒 from #2333

It's already in from #2641 ;)

Oh, sorry, I missed that.


python3Packages.reportlab: drop tests on aarch64-darwin

Things done

Built on platform(s)

[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

I don't care really, but the regression blocks a couple hundred builds, so I thought we'd better fix that, at least on stable.

On Hydra it failed just on aarch64-darwin.  I had checked specifically, as it seemed weird.  Theoretically it might be something around changes on our builders in there.

https://hydra.nixos.org/eval/1780812?filter=reportlab
Oh wait, one of the two succeeded on retrying, so it's not even deterministic.  On master it regressed about the same time: https://hydra.nixos.org/job/nixpkgs/nixpkgs-unstable-aarch64-darwin/python310Packages.reportlab.aarch64-darwin

Not sure what to do here
@mweinelt any opinions?

It complained during build, not during tests, so I dug into the setup.py and removed the special casing that tried to download stuff. See #191485.


yash 2.52 -> 2.53

Description of changes
Upstream changes: https://github.com/magicant/yash/blob/trunk/NEWS
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 188052 run on x86_64-linux 1

2 packages built:

getoptions
yash

@b12f ammended the commit to have the changelog link!

@ofborg build yash getoptions
The commit message is missing a : so ofborg is not building this automatically

Upstream changes: https://github.com/magicant/yash/blob/trunk/NEWS

This should be added to meta.changelog so that we don't need to manually write it into the commit message.


fix: 修复csrf token鉴权问题 --story=119892878

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.cheynechen seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add openshift_image_tag and openshift_release to crio jobs

Currently, these CI jobs only set openshift_pkg_version.
This variable is not directly applicable to containerized
installs and we should set either/both openshift_release
and/or openshift_image_tag.

@sdodson @stevekuznetsov
This has not been pushed.  Needed by: https://github.com/openshift/openshift-ansible/pull/7179

deferring to @sdodson

/lgtm

pushed, please merge.


Fix issue with invalid yaml on latest kubectl versions

When deploy the operator with kubectl version v1.17.2 you encounter the error:
error: error validating "./deploy/azure-databricks-operator/manifest-temp/run-duration-metrics-with-reconciler-config-20200203.1/default": error validating data:
ValidationError(ServiceMonitor.spec.selector): unknown field "control-plane" in com.coreos.monitoring.v1.ServiceMonitor.spec.selector;
if you choose to ignore these errors, turn validation off with --validate=false

Which is cause by invalid yaml in the  'config/prometheus/monitor.yaml'

/azp run


fix/6571-node-latency

Hi @Rupam-It,
Thanks for opening a PR. However, this PR at the current state needs a lot more work in order to be considered for merge:

Please provide a clear description of why it is needed and how does it improve the current implementation. If it is a work in progress PR, please indicate so in the heading and labels.
I'm not very convinced that we need the MicroTime precision here. But more importantly, this field is used in other types including LatencyStore and other generated code. The node latency functionality will simply break if only the stat.v1alpha1 API is modified.
The change contains irrelevant log lines, which I assume is somehow related to https://github.com/antrea-io/antrea/discussions/6590? Please read the instruction carefully and refrain open a PR for this task.
Please make sure the code is rebased on to main and relevant UT are passing.

For these reasons, I'm going to close this specific PR for now. Since you indicated that this is a "fix" in the title, could you first please open an issue and start a discussion over there? Thanks

Hi @Dyanngg ,
Thank you for your feedback and for taking the time to review my pull request.
I apologize for the oversight regarding the unnecessary log files from the LFX Task 3. These were mistakenly included in the PR due to an update in the main branch that inadvertently added them. I will ensure that such files are properly excluded in future submissions.
Regarding the use of MicroTime in the NodeLatencyStats API, I appreciate your input on the necessity of this precision. I will revisit the changes and initiate a discussion to better understand the implications.
Thank you again for your guidance, and I look forward to improving the implementation based on our discussions.
Best regards,
Rupam


Added a Random Quote Generator Website that uses HTML ,CSS and JavaScript .

This is a Random Quote Generator Website that uses HTML , CSS and JavaScript , that basically displays shuffled quotes on being promted .
The styling has been done using a separately created CSS stylesheet .
A separate app.js file has been used to implement the required functionalities so as to make the page dynamic .

This Pull Request is regarding issue #7 .
Please , accept my Pull Request .


Top recent 3 days sometimes empty

http://www.arxiv-sanity.com/top?timefilter=3days&vfilter=all
This view is sometimes empty, just showing the 'load more' button. Might be related to #57.

The issue is present now. Hopefully that can help.

Top recent for last day last 3 days and last weeks are empty , why?

fixed this just now, the server was backed up. should be recomputing ok now.


getting error on prometheus logs

URL for relevant page?
What products and version are you using? 2.8
Where did you look before opening the issue? every where
After 1d promethus is increasing the local disk.
and i can see a lot of logs with error " msg="dropped sample for series that was not explicitly dropped via relabelling"
when i'm restarting the influxdb server, the prometheus back to normal.
it's happening only after few hours\day
What can be the issue.
Thanks,

Issue moved to influxdata/influxdb #16755 via ZenHub

This doesn't appear to be a documentation related issue.
Moving this to the InfluxDB repo.  @roiromanoclick -- can you describe your setup in a bit more detail?  It isn't clear what you are doing, how you have things configured, etc.   We could make some very broad assumptions based on what is described here...but, better to have you outline what the setup is.  Thanks.


OAuth 2.0 support for Email with Templates module

@rover886 I have added basically all the same documentation as we did to the IMAP/POP3 connector. Let me know what differences there are, or if there are additional sections we need to include. Thanks!

@johannahemminger This module already has dependency on Encryption module and it's already mentioned in section 1.2.1. So I thought we can remove the section 3.1. What do you say?

Also this module is only available in the Mendix 7 and 8 (there is no release for Mendix 9),  so accordingly there should be changes at line This feature is supported for Studio Pro [7.23)](https://github.com/mendix/docs/blob/jh-emailwithtemplates-oauth2/releasenotes/studio-pro/7.23) and subsequent version of Studio Pro 7, and will soon be available for Studio Pro 8 and Studio Pro 9.

@johannahemminger We are going good, thanks to you :)

@johannahemminger one more small change, under section 3.1.2 we need to change the Azure permission image for this module. As this module is only being used for sending emails so accordingly permissions should be as shown below:

@johannahemminger This looks perfect now, please merge this to master. We will release the module by EOD


Define dbt acronym

Based on acronyms section of the content style guide, it mentions to define the acronym first and use parentheses to show the acronym. However, dbt is special because it's more popular and well-known than the full definition of the acronym, hence why I placed it in parentheses.

hey @joe-tingsanchali-sonarsource thanks for opening this up and great attention to detail! appreciate you reading our style guide as well 😄
This is a great contribution and in order to match our trademark guidelines, we always use marks in their exact form. so dbt is officially just "dbt" and applies to all dbt software inclusive of open-source and commercial software. Since dbt is not an abbreviation, I'll go ahead and close this pr but let me know if you have any questions on that! ✨


Fix incorrect CRD casing

Should I also change the 'Resource' field?

fixes: https://github.com/lensapp/lens/issues/3257

Should I also change the 'Resource' field?

Yes please, make same changes there.


Add a new column and update it, Data view is not refreshed

Steps to reproduce bug:

In the SQL pane, issue an ALTER TABLE tablename ADD COLUMN newcolumn numeric query. Every row of that new column will be null and will display as such in the Data View.
I want to populate this new column with derivative values from another column (text based percentage strings like "93.45%", I want to make them numeric), so I next issue an UPDATE tablename SET newcolumn = cast(trim(trailing '%' from tablename.textpercentcol) as numeric) query in the SQL pane.
After the query finishes, the Data View still shows every row in my new column as "null". It was only after much wandering around and finally issuing a SELECT * FROM tablename WHERE newcolumn is not null where I realized the update had finally actually occurred and these columns were, in fact, populated.

Only seems to be the first 40 rows. Might be a caching/CDN problem, but I've hard refreshed the page several times and about 20 minutes has gone past since the first query, so if so our caching isn't updating very fast.

Changing the names or datatypes of columns through the UI also runs into the same problem - the changes are not reflected in the UI but have taken place in the database. Refreshing the Data View or waiting a few minutes does not resolve the problem.

Have you tried to run the SELECT query manually to the SQL API? It would be nice trying to run it with v1 and v2 and see if there is any difference in order to check if it's a caching problem or not

@luisbosque @xavijam I ran into a similar problem with a new dataset (query "alter table add column" but the new column doesn't appear in the Editor; nor does it with the "add column" button) and tested with the SQL API v1 and v2 - the columns both appear in the API responses. But not the Editor.

Was there any progress on this bug?
We just spent several hours debugging why our named map's interactivity configuration wasn't working, only to notice that our browsers were displaying different columns based on when we first accessed the data view.
Clearing all browser data and relogging seems to have updated the data view. The changes to column names happened 3 days ago.

I've encountered a problem that may be related. Using the Editor, I renamed a column and created a new column. The changes are visible in SQL queries I run in the Editor, but not visible in the Map View or in the Data View in the absence of a query. I can't edit the renamed column in the GUI, reference it in CartoCSS under either name, or scroll down in the Data View; these all produce error messages.
I was not able to fix this problem by:

Refreshing the Editor
Logging out and back in again
Clearing the browser cache
Using a different browser
Using a different computer
Using a different Internet connection

So far the only solution has been to create a new dataset from a SELECT * FROM table_name query. Apologies if I'm not describing this as clearly as I should be.

There has been no activity on this issue for more several months. We are closing it. If you think this still needs to be addressed please open a new issue.


Implement an 'open search' that takes a user query and then applies relevant filters

It may happen that a user will not apply filters by clicking on them.
Rather they may just want to 'write what they are looking for'
The ideal user experience is that they make a query and then it the system decides on what best to look for.
Should be able to check on tgs.thegrid.id and get the info from there.

Hint - is an AI agent.


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 889 ms

Resolved: Bofrost.be performance has improved in eae54a1 after 6 minutes.


Add Kubecost construct example

Description of changes:
Add a Kubecost AddOn usage example to lib/.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The checks are failing: your branch requires rebasing (merge from main) to address.

@shapirov103 hi hi, thanks for the review!

Yes, customers can use their own NE provided it's running on the default port and its endpoint is visible from the namespace where Kubecost is installed. Customers are also welcome use their own Prometheus (including AMP), though it's only officially supported through paid plans. We've added both of these functionalities to the AddOn and documented them:

The getting started parts in the product and technical docs contain more details on how to use Kubecost. Added to the AddOn README.

We do support it; updated the AddOn.

The getting started parts in the product and technical docs contain more details on how to use Kubecost. Added to the AddOn README.

Let's get a snippet for the https://github.com/aws-quickstart/ssp-amazon-eks/tree/main/docs/addons page for Kubecost and add these references there as well.
As a follow up item (roadmap), we should consider a pattern with full ingress enabled leveraging other add-ons that will be turnkey for the customer. e.g. if there is one that 50% of the customer use (or most commonly used configuration).

Thanks for the feedback! Changes made and PR submitted https://github.com/aws-quickstart/ssp-amazon-eks/pull/256


Remove duplicate call delegate->moduleNames()

By analyzing the blame information on this pull request, we identified @cjhopman and @mhorowitz to be potential reviewers.

Ping @mkonicek

@facebook-github-bot shipit

Thanks for importing. If you are a Facebook employee, you can view this diff on Phabricator.


Allow null/empty response body

For example:
post("/accounts", { req, res -> res.status(201) })

Currently this fails with as a 404.
INFO spark.webserver.MatcherFilter - The requested route [/accounts] has not been mapped in Spark
In MatcherFilter.java:227, bodyContent == null is not enough to indicate the request hasn't been consumed.
        boolean consumed = bodyContent != null;

        if (!consumed && hasOtherHandlers) {
            throw new NotConsumedException();
        }

        if (!consumed && !isServletContext) {
            LOG.info("The requested route [" + uri + "] has not been mapped in Spark");
            httpResponse.setStatus(HttpServletResponse.SC_NOT_FOUND);
            bodyContent = String.format(NOT_FOUND);
            consumed = true;
        }

The pull request:
https://github.com/perwendel/spark/pull/352
should fix the issue

I've changed my mind here, this actually makes sense. If the route is null, nothing has been mapped, so 404 is appropriate.


New Package Structure

I have a suggestion to change our package structures(after we got frameworks ready).
We make Telegram.Bot package a meta package similar to Microsoft.AspnetCore.All.
Code on this repo would be published as Telegram.Bot.Client so its responsibility would be constructing request & responses.
Rest of the implementations would go in specific packages depending on the type of deployment(getting updates):
Telegram.Bot.Framework.Common (Framework abstractions + some implementations)

Telegram.Bot.Client

Telegram.Bot.Polling (complete long polling implementation of framework)

Telegram.Bot.Framework.Common

Telegram.Bot.Framework (ASP.NET Core framework for webhooks)

Telegram.Bot.Framework.Common

Telegram.Bot (Meta package having all code for long polling & ASP.NET Core webhooks)

Telegram.Bot.Framework
Telegram.Bot.Polling

In addition, events could be removed from client package and go in the polling package. I wouldn't recommend using long polling due to performance and unnecessary use of resources (long polling! right?). Also, relying on events usually require singletons and instances that stick in memory for long.
However, we can still provide events in long polling package via a IUpdateEventsManager type.

Pooling is a good idea

Obsolete. We tried detaching the Polling system in separate project and it was a mess. It's now back in main project because that's what our users expect.
Now it's not impossible that in the future we rearrange the package structure for other reasons (like separate types from code), but no need to keep this old issue opened for now.


Edmond karp changes

implemented the changes suggest by @appgurueu
-removed the weight as it was not used here
-corrected the documentation for the space complexity form O(V) -> O9=(V^2)
-Implemented BFS using two level arrays (currentLevel and nextLevel) to maintain level-order traversal, avoiding the inefficient queue.shift() operation.
-Used an adjacency list representation for the residual graph to make the iteration over outgoing edges
-Changed the parent array initialization to null

Please update your existing PR rather than opening a new one.


PubSub runs into java.lang.RuntimeException: ManagedChannel allocation site when a new publisher is created

Hi!
We've currently received around 100-200 PubSub related issues when a PubSub publisher is created with the following stacktrace:
io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference cleanQueue: *~*~*~ Channel ManagedChannelImpl{logId=4613, target=pubsub.googleapis.com:443} was not shutdown properly!!! ~*~*~* (ManagedChannelOrphanWrapper.java:163)
    Make sure to call shutdown()/shutdownNow() and wait until awaitTermination() returns true.
java.lang.RuntimeException: ManagedChannel allocation site
	at io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference.<init>(ManagedChannelOrphanWrapper.java:103)
	at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:53)
	at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:44)
	at io.grpc.internal.AbstractManagedChannelImplBuilder.build(AbstractManagedChannelImplBuilder.java:410)
	at com.google.api.gax.grpc.InstantiatingGrpcChannelProvider.createSingleChannel(InstantiatingGrpcChannelProvider.java:206)
	at com.google.api.gax.grpc.InstantiatingGrpcChannelProvider.createChannel(InstantiatingGrpcChannelProvider.java:162)
	at com.google.api.gax.grpc.InstantiatingGrpcChannelProvider.getTransportChannel(InstantiatingGrpcChannelProvider.java:149)
	at com.google.api.gax.rpc.ClientContext.create(ClientContext.java:151)
	at com.google.cloud.pubsub.v1.stub.GrpcPublisherStub.create(GrpcPublisherStub.java:161)
	at com.google.cloud.pubsub.v1.Publisher.<init>(Publisher.java:154)
	at com.google.cloud.pubsub.v1.Publisher.<init>(Publisher.java:83)
	at com.google.cloud.pubsub.v1.Publisher$Builder.build(Publisher.java:607)

We are using PubSub library v 1.40.0. And here is how we create publishers:
Publisher createPublisher(ProjectTopicName topicName) {
        try {
            Publisher publisher = Publisher.newBuilder(topicName)
                                           .build();
            return publisher;
        } catch (IOException e) {
            String errorMessage = format("Cannot create a publisher for topic %s", topicName);
            throw new IllegalStateException(errorMessage, e);
        }
    }

Any thoughts or suggestions? By the way, we are running on AppEngine Standard with Java 8.

I don't think this is actually related to creating new Publisher, though gRPC uses that as an opportunity to tell us that we're leaking channels.
Is it possible that you're not calling Publisher.shutdown to clean up properly somewhere?

Hello @pongad. Thank you for your reply.
No, we are cleaning everything up every time:
private static void cleanupPublisher(Publisher publisher) {
        try {
            publisher.shutdown();
        } catch (Exception e) {
            log().error("Unable to cleanup publisher for topic: {}", publisher.getTopicNameString(), e);
        }
    }

You're right. gRPC wants us to call awaitTermination. I sent a PR for fixing this.

I updated the library and added publisher.awaitTermination(1, TimeUnit.MINUTES); after publisher.shutdown(); in the close() method of my Closeable class but I keep getting the above error on publisher build.
@yuri-sergiichuk, has this patch resolved your issue?

@borissnd we are testing the new cloud libraries version now. I'll post an update here in case I'll notice the problem again.

@yuri-sergiichuk @borissnd are you still experiencing this issue? I'm seeing the same problem
doing something in the lines of:
val publisher = Publisher.newBuilder(topicName).build()

try {
    messages.forEach { message -> publisher.publish(message) }
} finally {
    publisher.shutdown()
    publisher.awaitTermination(1, TimeUnit.MINUTES)
}

Yeah, we do still experience problems.

Same here. The error still persists. I'm using version 1.45.0.
@pongad @snehashah16 , could you please reopen this issue?

Some more info on my problem: it might have been caused by repeatedly creating and shutting down publishers. Which seemed to be the suggested approach from what I found (like github doc and sample code here and here as examples) Could that somehow cause the problem?
Here's a question on that topic: https://github.com/googleapis/google-cloud-java/issues/2342 and there it's suggested to then keep a more long lived publisher instead. I'm trying that approach now instead.

@snehashah16, @pongad, any updates on this issue?

Does anybody know when this will be available?

Exactly the same issue persists for the subscriber channels that do not get shut down properly, since there is no direct or indirect call to GrpcTransportChannel.java#close() from within com.google.cloud.pubsub.v1.Subscriber. Therefore even with google-cloud-pubsub 1.84.0 we keep  getting the same error message with slightly different stack trace:
*~*~*~ Channel ManagedChannelImpl{logId=42, target=pubsub.googleapis.com:443} was not shutdown properly!!! ~*~*~*
    Make sure to call shutdown()/shutdownNow() and wait until awaitTermination() returns true.
java.lang.RuntimeException: ManagedChannel allocation site
    at io.grpc.internal.ManagedChannelOrphanWrapper$ManagedChannelReference.<init>(ManagedChannelOrphanWrapper.java:94)
    at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:52)
    at io.grpc.internal.ManagedChannelOrphanWrapper.<init>(ManagedChannelOrphanWrapper.java:43)
    at io.grpc.internal.AbstractManagedChannelImplBuilder.build(AbstractManagedChannelImplBuilder.java:514)
    at com.google.api.gax.grpc.InstantiatingGrpcChannelProvider.createSingleChannel(InstantiatingGrpcChannelProvider.java:268)
    at com.google.api.gax.grpc.InstantiatingGrpcChannelProvider.createChannel(InstantiatingGrpcChannelProvider.java:185)
    at com.google.api.gax.grpc.InstantiatingGrpcChannelProvider.getTransportChannel(InstantiatingGrpcChannelProvider.java:177)
    at com.google.api.gax.rpc.ClientContext.create(ClientContext.java:160)
    at com.google.cloud.pubsub.v1.stub.GrpcSubscriberStub.create(GrpcSubscriberStub.java:263)
    at com.google.cloud.pubsub.v1.Subscriber.doStart(Subscriber.java:261)
    at com.google.api.core.AbstractApiService$InnerService.doStart(AbstractApiService.java:148)
    at com.google.common.util.concurrent.AbstractService.startAsync(AbstractService.java:248)
    at com.google.api.core.AbstractApiService.startAsync(AbstractApiService.java:120)
    at com.google.cloud.pubsub.v1.Subscriber.startAsync(Subscriber.java:253)
   // ...

Is it possible to reopen this issue and fix the problem for the subscriber case?

I've been experiencing the same exception in 1.98.0. A new issue for the subscriber should be opened to expect a fix?

Iam still experiencing this issue. Why is this closed?

I am still facing this issue with the gcp publisher and raised a fresh ticket for the same.
https://github.com/googleapis/java-pubsub/issues/1803
Dependencies:
GCP Pub/Sub Library Version: 4.7.1
Java Version: 17.0.6
GRPC Version: 1.56.1

Still an issue...


Windows server reloading

After editing a file this happens:
!! Error loading C:/Users/Bertil/Camping/Blog/blog/blog.rb:
TypeError: wrong argument type Module (expected Class)
:55:in `'
:52:in `'
:1:in `'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping.rb:44:in `eval'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping.rb:44:in `goes'
C:/Users/Bertil/Camping/Blog/blog/blog.rb:1:in `'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/activesupport-3.1.1/lib/active_support/dependencies.rb:234:in `load'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/activesupport-3.1.1/lib/active_support/dependencies.rb:234:in `block in load'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/activesupport-3.1.1/lib/active_support/dependencies.rb:223:in `block in load_dependency'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/activesupport-3.1.1/lib/active_support/dependencies.rb:640:in `new_constants_in'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/activesupport-3.1.1/lib/active_support/dependencies.rb:223:in `load_dependency'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/activesupport-3.1.1/lib/active_support/dependencies.rb:234:in `load'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/reloader.rb:64:in `load_apps'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/reloader.rb:109:in `reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/reloader.rb:184:in `block in reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/reloader.rb:183:in `each'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/reloader.rb:183:in `reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/reloader.rb:162:in `update'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/server.rb:157:in `find_scripts'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/server.rb:161:in `reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/server.rb:169:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/lint.rb:48:in `_call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/lint.rb:36:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/showexceptions.rb:24:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/commonlogger.rb:20:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/chunked.rb:41:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/content_length.rb:14:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/lib/camping/server.rb:242:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/rack/handler/webrick.rb:59:in `service'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/1.9.1/webrick/httpserver.rb:111:in `service'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/1.9.1/webrick/httpserver.rb:70:in `run'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/1.9.1/webrick/server.rb:183:in `block in start_thread'
!! Error loading C:/Users/Bertil/Camping/Blog/blog/blog.rb, see backtrace above
As far as I know, the only solution is to restart the server. Which is not productive you know!
I don't know if I'm the only one having this problem though.
Kids, this why you don't develop in windows!

Hi Isak
:55:in `'
:52:in `'
In the first example I have to hand, "module Models" is inside "module
Blog", with no colon. I'll leave it someone else to explain the
details, but could you put up your blog.rb cose in a Pastie
(pastie.org) for us to take a look?
DaveE

After editing a file this happens:
!! Error loading C:/Users/Bertil/Camping/Blog/blog/blog.rb:
TypeError: wrong argument type Module (expected Class)
:55:in `'
:52:in `'
:1:in `'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping.rb:44:in `eval'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping.rb:44:in `goes'
C:/Users/Bertil/Camping/Blog/blog/blog.rb:1:in `'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/
activesupport-3.1.1/lib/active_support/dependencies.rb:234:in `load'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/
activesupport-3.1.1/lib/active_support/dependencies.rb:234:in `block
in load'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/
activesupport-3.1.1/lib/active_support/dependencies.rb:223:in `block
in load_dependency'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/
activesupport-3.1.1/lib/active_support/dependencies.rb:640:in
`new_constants_in'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/
activesupport-3.1.1/lib/active_support/dependencies.rb:223:in
`load_dependency'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/
activesupport-3.1.1/lib/active_support/dependencies.rb:234:in `load'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/reloader.rb:64:in `load_apps'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/reloader.rb:109:in `reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/reloader.rb:184:in `block in reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/reloader.rb:183:in `each'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/reloader.rb:183:in `reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/reloader.rb:162:in `update'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/server.rb:157:in `find_scripts'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/server.rb:161:in `reload!'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/server.rb:169:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/lint.rb:48:in `_call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/lint.rb:36:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/showexceptions.rb:24:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/commonlogger.rb:20:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/chunked.rb:41:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/content_length.rb:14:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/camping-2.1.467/
lib/camping/server.rb:242:in `call'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/gems/1.9.1/gems/rack-1.3.3/lib/
rack/handler/webrick.rb:59:in `service'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/1.9.1/webrick/httpserver.rb:
111:in `service'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/1.9.1/webrick/httpserver.rb:
70:in `run'
C:/RailsInstaller/Ruby1.9.2/lib/ruby/1.9.1/webrick/server.rb:183:in
`block in start_thread'
!! Error loading C:/Users/Bertil/Camping/Blog/blog/blog.rb, see
backtrace above
As far as I know, the only solution is to restart the server. Which
is not productive you know!
I don't know if I'm the only one having this problem though.
Kids, this why you don't develop in windows!
--
Reply to this email directly or view it on GitHub:
https://github.com/camping/camping/issues/43

The blog is up on Github actually!
http://j.mp/od4d4q

great - I'd try it out on my local machine to see if I can reproduce
the problem, but have to go out right now! So handing over to those
more knowledgeable... DaveE

The blog is up on Github actually!
http://j.mp/od4d4q
--
Reply to this email directly or view it on GitHub:
https://github.com/camping/camping/issues/43#issuecomment-2421061

Why do you include Blog::Models at the top? Models are included in every of your controllers, so you don't need to do it anyway.
I removed this line and it appears to work. I am, however, baffled as to why - maybe Magnus will shed some light on it ;)

I include it because I want to be able to refer to Post as Post in the console and not Blog::Models:Post.new or something like that.
Otherwise it just gives me uninitialized constant

Hey, I think I got this.
Models define a constant, Base, which you can inherit from (class Post < Base in your code). However, Camping also has a constant by the same name - Base - which is included in all your controllers, giving you access methods like render.
Due to an absolutely mindboggling string of including, evals and heavy-duty metaprogramming, finishing at your include Blog::Models, these two get confused. Then Models' Base is not being defined, since Camping's Base is already reachabled there. Because one Base is a class and the other is a module, by writing class Post < Base you're basically trying to inherit from a module.
I have no idea how to fix it except for removing your include. Have you tried doing the include in the console, after the application is up and running?

Yeah that did the trick!
So I guess now the issue is that the console should include those for us!


Make tests a little more robust

Remove the XDG_CONFIG_HOME environment variable as we're resetting HOME
Add GIT_CONFIG_NOSYSTEM to encourage git to not read that configuration

Closes #2345

r? @wycats
(rust_highfive has picked a reviewer for you, use r? to override)

r+

@bors: r=wycats c3e1c194909fb7baf4334ddba5a44c7a6da62bc3

:hourglass: Testing commit c3e1c19 with merge 808cb99...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


why the document saying it does't support className while it does ?

Note that non-standard HTML attribute names are not supported, onclick and class are valid, but onClick or className are not.

I don't think it true, cause I can use className and it works well.

You are right, but let's call that an implementation detail. You are advised to use class and not className with Hyperapp.

Hum, but class is keyword.
<MyComponent class="custome-class" />

function MyComponent({ class }) {
  // keyword as variable name ?
  ...
}

You can this.
<MyComponent class="custom-class" />

function MyComponent({ className }) {
   return
...

}


[dart:ffi] Inline arrays in Structs

Helpers for converting between ffi.Pointers and arrays.
Maybe use external typed data for efficient conversions.

also support for fixed size arrays.
its quite common to have a struct with fixed sized arrays to pass around in C

We have asTypedList for exposing the native memory as a Dart array.
For structs with inline arrays:
struct MyStruct {
  uint8_t arr[10];
  int64_t arr2[3];
};

We need a dart syntax of representing an inline array.
We could opt for exposing this as a pointer, which provides [] and []= access.
class MyStruct extends Struct {
  @InlineArray(10)
  Pointer<Uint8> arr;

  @InlineArray(3)
  Pointer<Int64> arr2;
}

Using a Pointer as the representing type does not allow us to distinguish pointers and inline arrays by the Dart type. However, this is not really a problem since C does not support passing fixed-size arrays by value in calls.
cc @mkustermann

@dcharkes
I got this error when trying your solution to getting static arrays on Dart side
Code
class StaticArray extends Struct {
  @InlineArray(10)
  Pointer<Int32> array;
  @Int32()
  int len;
}

Result
$ dart structs.dart
structs.dart:48:4: Error: Method not found: 'InlineArray'.
  @InlineArray(10)
   ^^^^^^^^^^^
structs.dart:129:15: Error: A function expression can't have a name.
  staticArray.forEach(idx, elem) => print("array[$idx]: $elem");
              ^^^^^^^
structs.dart:129:15: Error: Expected an identifier, but got 'forEach'.
  staticArray.forEach(idx, elem) => print("array[$idx]: $elem");
              ^^^^^^^

Does it mean that this is just a proposal?
What would be the canonical way of getting pointers?
I'm referring to my latest question.
Thanks in advance!

Yes, for now it's just a proposal.
See the workaround made by @timsneath here for now.

@dcharkes Thanks! FFI has been great so far.
On a related matter, the struct example never frees the malloc'ated memory. Does this mean that the memory will be GC'ed on Dart side?

You need to use free() to release the memory.
it'll happen automatically when the process exits, of course.

@timsneath Thanks. I looked at the static array workaround but it seems difficult to scale when my array is say 500-1000 elements big.
Is it that the realistic workaround is to use malloc'ated arrays all the time?

See the workaround made by @timsneath here for now.
I'll get around to implementing this in the future, I'm working on other dart:ffi features at the moment.

@dcharkes As I alluded to in the conversation in #36140, the lack of support for inline arrays in structs is a blocker for me in creating OpenXR bindings for Dart and Flutter.
The proposed workaround is infeasible for the structs I need to map, which include the following:
struct XrActionCreateInfo {
  XrStructureType type;
  const void* next;
  char actionName[64];
  XrActionType actionType;
  uint32_t countSubactionPaths;
  const XrPath* subactionPaths;
  char localizedActionName[128];
};

struct XrApplicationInfo {
  char applicationName[128];
  uint32_t applicationVersion;
  char engineName[128];
  uint32_t engineVersion;
  XrVersion apiVersion;
};

struct XrEventDataBuffer {
  XrStructureType type;
  const void* next;
  uint8_t varying[4000];
};

As a workaround, would not creating a pointer to uint8 or utf8 with size matching the required, good enough?
For example:
class XrEventDataBuffer extends Struct {
....
Pointer varying;
factory XrEventDataBuffer.create(..., Uint8List varying) {
     final Pointer<Uint8> data = allocate<Uint8>(count: varying.length);
     fillpointerWithIntlist(data, varying);

    // then initialize the struct
 }

}
Sorry for missing the auto formatting, I’m on the phone.

What about multi dimensional inline arrays, what should the API be for multidimensional arrays?
(ty @mannprerak2!)

@artob extending a little from the idea @bitsydarel approached, you might check out https://github.com/timsneath/win32/blob/bluetooth/lib/src/structs.dart#L1941 for an example of wrapping a similar Win32 struct that seems like a viable workaround for you. (In this scenario, I have a 250 character Utf-16 string that I need to wrap).

@artob extending a little from the idea @bitsydarel approached, you might check out [...] an example of wrapping a similar Win32 struct that seems like a viable workaround for you.

@timsneath It's a good workaround for the BLUETOOTH_DEVICE_INFO struct you have there, but it won't quite suffice for my use case: the OpenXR API has large character arrays inline and intermixed with other struct fields, not only as the last trailing field in the struct.
So, the suggested approach seems like it could be a workaround for XrEventDataBuffer with its 4000-byte trailing array, but defining XrActionCreateInfo and XrApplicationInfo would still be a pain in the butt given the 64/128-byte inline arrays. (And there are more structs like these.) Any suggests for the likes of those?

As a workaround, would not creating a pointer to uint8 or utf8 with size matching the required, good enough?

@bitsydarel After @timsneath's code example I understand better what you were proposing for XrEventDataBuffer. Thanks, seems a viable approach for that struct :+1:

It's not the prettiest code I've ever written, but wouldn't something like this work?
https://gist.github.com/timsneath/983764458690f89a5c8231b4ff2e2e64
Here we use the same technique as for the BLUETOOTH_DEVICE_INFO struct of using properties to index into the created struct, and treating as the struct just as a binary blob for the purposes of allocating space in the heap.
You could extend this still further by providing setters where appropriate, and adding clean String properties to convert from a C-style string into a Dart string, with guards to ensure a 128-byte maxlength.
Curious to know if this works for you?

For inline arrays of primitive values https://pub.dev/packages/ffigen has an option to generate 'n' fields and provide indexed access (with array-workaround: true).

Updated design:
class MyStruct {
  Array<Int8> a = Array(8);

  Array<Array<Int8>> b = Array(2, 4);

  Array<Array<Array<Array<Int8>>>> c = Array.multi([2,3,4,5]);
}

external const Array(int firstDimension, [int dimension2, int dimension3, int dimension4, int dimension5]);
external const Array.multi(List<int> dimensions);

source https://github.com/dart-lang/sdk/issues/45023#issuecomment-779991776

This will be available on the first dev-release after Version 2.13.0-73.0.dev.

Is it possible to have the address of the Array<Int8>, so that we could convert it asTypedList?

Is it possible to have the address of the Array<Int8>, so that we could convert it asTypedList?

We're tracking this in https://github.com/dart-lang/sdk/issues/45508.


Darkness does not seem to impact bot vision.

1st example
Pistol round, so they dont have NVGs. Flashlights are disabled, and the light level in the vent is 12. yb_check_darkness is 1
https://github.com/user-attachments/assets/3fcce9e8-af3a-4eb6-9e5d-4673118ad7c8
2nd example
Again, pistol round, no nvg's disabled flashlights, and light level is 0. yb_check_darkness is 1
https://github.com/user-attachments/assets/db8be944-8c81-4429-a1ec-8bb1dea53476
Also I noticed that on cs_estate and cs_mansion, turning off lights didn't help. (Although I don't know if you can fix that since its presumably using the baked in lightmap to check for darkness).

i know i should open a separate issue but screen fade also doesn't affect them

It should be better with latest builds.


bug: Current step is still highlighted when run is complete

Overview
Protocol is complete but last step has remained highlighted as current step
Behavior is present in json and python protocols
Current behavior

Check if we can repo - python and json

@brenthagen was this fixed by your run log work?

@anthonywilleee this is still an issue, but we can address soon as we polish the new run log


New surface representation by Henneberg et al

This PR has an implementation of a new representation for toroidal surfaces proposed by Henneberg et al in https://arxiv.org/pdf/2105.00768.pdf. This representation has about half as many degrees of freedom as in SurfaceRZFourier and is expected to be useful for stellarator shapes.

I think right now if one wants to do anything with this surface, one always has to convert to a SurfaceRZFourier first. Can you add the explicit formula for the surface, that way we don't always have to do this transformation (and we would also be able to get derivatives fairly easily)?
SurfaceRZFourier computes area and volume independently, so I think there is no need for the area_volume function that computes both at the same time.

Thanks @florianwechsung - I've addressed those two items.

@florianwechsung Do I understand correctly that you want SurfaceHenneberg to implement concrete functions for gamma_impl, gammadash1_impl, gammadash2_impl, dgamma_by_dcoeff_impl, dgammadash1_by_dcoeff_impl, and dgammadash2_by_dcoeff_impl? In some of the other surface classes I also see a function gamma_lin - what does this do?

gamma_lin now works, and all tests pass in my fork. OK if we merge this now, and implement dgamma*_by_dcoeffs_impl later if needed?
Generally, requiring developers to implement both gamma_impl and gamma_lin would seem to promote code duplication. Wouldn't it make sense to let developers implement only gamma_lin, and have the base Surface class include a gamma_impl function that just calls gamma_lin for the tensor product grid in (theta,phi)? If users want to override that default gamma_impl for better performance they could.

Yeah that's a good suggestion. gamma_lin was an afterthought when Andrew added the flux and crosssection, so we didn't think too much about the structure there. Do you want to make that change in this PR or should we keep this in mind for a separate one?

One more comment:
The other surface classes support that either a number of quadrature points in phi and theta is passed, or that a specified list of phis and thetas is passed. This is useful
a) for the taylor tests for gammadash1 and gammadash2 (we want to evaluate gamma at 0, eps, eps/2, eps/4, eps/8 for a taylortest)
b) for surfaces with stellsym or nfp>1, sometimes we only want to work on a part of the surface, and sometimes (for visualisation mainly) we want the full surface.

Yeah that's a good suggestion. gamma_lin was an afterthought when Andrew added the flux and crosssection, so we didn't think too much about the structure there. Do you want to make that change in this PR or should we keep this in mind for a separate one?

Let's do that refactoring in a different PR.

One more comment:
The other surface classes support that either a number of quadrature points in phi and theta is passed, or that a specified list of phis and thetas is passed. This is useful
a) for the taylor tests for gammadash1 and gammadash2 (we want to evaluate gamma at 0, eps, eps/2, eps/4, eps/8 for a taylortest)
b) for surfaces with stellsym or nfp>1, sometimes we only want to work on a part of the surface, and sometimes (for visualisation mainly) we want the full surface.

Support for initializing the points by either an int or array is added in the latest commit.

Thanks!
Is it possible to get the coverage report for this? @mbkumar
Would be good to see if there are any major bits of the code left untested before merging.

Thanks!
Is it possible to get the coverage report for this? @mbkumar
Would be good to see if there are any major bits of the code left untested before merging.

There may be a better way to check this, but in the latest "simple CI" run on my fork https://github.com/landreman/simsopt/runs/2951185572?check_suite_focus=true, if you expand "Combine coverage reports" in the main panel, there is a coverage summary for each file. Coverage for the new file surfacehenneberg.py is 97%.

Happy for this to be merged.


WIP: ProviderFactory

Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[ ] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[ ] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

Superseded by https://github.com/stacklok/minder/pull/3131
I will recreate the removal of ProviderBuilder in a separate PR.


Create blog post for 5.3.0 release

We need a blog post for Open GEE 5.3.0 release

Issue completed via #1299


GSF Carbon QL Meeting Agenda 2023-06-19

Time 12:00pm (GMT) - See the time in your timezone
Lead – @srini1978 (Microsoft)
Co-Lead @navveenb (Accenture)
PM – @jawache  (GSF)
Antitrust Policy
Joint Development Foundation meetings may involve participation by industry competitors, and it is the intention of the Joint Development Foundation to conduct all of its activities in accordance with applicable antitrust and competition laws. It is therefore extremely important that attendees adhere to meeting agendas, and be aware of, and not participate in, any activities that are prohibited under applicable US state, federal or foreign antitrust and competition laws.
If you have questions about these matters, please contact your company counsel or counsel to the Joint Development Foundation, DLA Piper.
Roll Call
Please add 'Attended' to this issue during the meeting to denote attendance.
Any untracked attendees will be added by the GSF team below:
Agenda

[x] Progress on hiring a developer (@jawache)
[x] Formalizing the meeting process, similar to Standards WG? Talk about PRs then Issues then Discussions (@jawache)
[ ] @Willmish to present on the Carbon Aware SDK Components (so we can learn how they solved similar problems)
[ ] Formalizing/consolidating our specs, design documents (@jawache, @srini1978)
CQL Design document
IMP Plugin Spec
[ ] Discuss use of JSII for the initial IMP Plugins https://github.com/Green-Software-Foundation/carbon-ql/discussions/41
[ ] Discuss the presentation by Etsy on carbon calculations in standards group Presentation (@srini1978)
[ ] AOB

Decisions
Actions

@navveenb to interview candidate
Agenda will be

Going through Issues and then PRs.
AOB.

Move to next meeting

[ ] House keeping, going through issues and board to clean up (@jawache)

Attended

Attended

Attended

attended

Feedback from CASDK Demo:

@GadhuNTTDATA: You don't need E, M for Time Shifting but you do need them for Location Shifting (since the DC might be different hardware).
@filga: How is the data cleaned from WattTime and EM. There is no cleaning, apart from minimal. No resampling of data as well, don't resample WattTime to EM freq for instance.
@jawache what is the data format:
Location: Provider string names, not normalized. @jawache do the underlying data sources accept lat, lon - yes. This might artificially give an indication that the data is more granular than it is.
Time: Time from the user.
Rating: The I value (gCO2/kWh). Implied by the data source, average vs marginal.
Duration: For the length of time from Time this value is valid.

Links from Carbon Aware SDK related to GridEmissionsInterface:

https://github.com/Green-Software-Foundation/carbon-aware-sdk/blob/dev/docs/architecture/data-sources.md
https://github.com/Green-Software-Foundation/carbon-aware-sdk/blob/b3b3189192031c13559ba6e97ce5cdc968784eb9/src/CarbonAware/src/Interfaces/IEmissionsDataSource.cs
https://github.com/Green-Software-Foundation/carbon-aware-sdk/discussions/356

Attended


AutoSync service running with GMSA and MIM MA change detection?

Greetings - what's the trick to allow GMSA to work with change detection? I have the GMSAs synced into MIM as a custom object but the change detection is throwing a "exception "Identity is not found" error...
TIA,
Terry

Hi Terry,
Unfortunately, the MIM service only recognizes logins from the person object class. You might have some luck hacking the GMSA ObjectSid into a manually created person object in the MIM service - but I haven't tried this personally.

I can confirm that syncing GMSA's into the MIM Service as Person objects does work and change detection functions as you'd expect.

Awesome! Good to know and glad its working!


Provide some default video metrics

This library operates at a fairly low level, exposing function calls which map directly to the measurement protocol types.  It could be more helpful in providing some basic roVideoPlayer and <Video> node integration for gathering video playback metrics.
http://www.charlesfarina.com/video-tracking-google-analytics/ has a good suggestion of how to model this.

Any plans for implementing Timing Tracking? That could potentially be used for send the load/buffer times for videos. Most roku apps are video apps, so some support for video analytics makes this even more relevant (without this not sure how to get to the nice reports in the link/article give  above).

Implemented Timing. May be useful for other.
googleAnalytics.brs:
near line 233:
''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''''
' Use the Timing to measure periods of time. This is used to measure page or video load
' times.
'
' | Parameter       | Value Type  | Default Value | Max Length
' -----------------------------------------------------------
' | category        | text        | None          | 150 Bytes
' | variable        | text        | None          | 500 Bytes
' | time            | integer     | None          | None
' | label           | text        | None          | 500 Bytes
'
' XXX :  Added by Sridhar Guthula
Function gamobileTiming(category As String, variable As String, time As String, label="" As String) As Void
if m.gamobile.debug
? "[GA] Timing: " + category + "/" + variable
end if
hit_params = {
t: "timing"
utc: URLEncode(category)   ' EUser timing category. Required
utv: URLEncode(variable)   ' User timing variable name. Required.
utt: URLEncode(time)       ' User timing time. Required.
}
if label <> "" then hit_params.utl = URLEncode(label)      ' User timing label.
gamobileSendHit(hit_params)
End Function
GampTask.xml
near line 63:
m.top.observeField("timing", port)
near line 107:
else if field = "timing" then
if data.category = invalid then data.category = ""
if data.variable = invalid then data.variable = ""
if data.time = invalid then data.time = ""
if data.label = invalid then data.label = ""
gamobileTiming(data.category, data.variable, data.time, data.label)
end if
near line 182:

thanks for the effort.  can you put this in a pull request?

Closing.  No further improvements to this library since Roku announced native support for google analytics


readme inconsistencies

hi!
I just discovered this awesome project and wanted to got started in testing Web Components with Bore on Node.js, but unfortunately there are a couple of small gotchas on the readme file of this repo:

https://github.com/skatejs/ssr#testing-in-node the hash on the link doesn't seem to exist, and the first sentence on that readme says that the code has moved to the monorepo
https://gist.github.com/treshugart/719dad9c3be6e048b9563309cf37d3c1 its 404

tomorrow I will slowly check out the docs to better understand, but if you could fix this I think it would be great for newcomers 😃
thanks for the great work man!

Hey @moebiusmania, thanks for reporting. I've updated the README to suit. I linked to the monorepo and removed the gist link because the docs should be good enough. Cheers!


Minimum time interval between successive requests to same endpoint

Is there a concept of minimum time interval between successive requests to same endpoint in Moya?

What is the use case? I feel like this is a responsibility of the client.

Suppose in viewwillAppear(), we fetch data from endpoint. User can return to viewcontroller by pressing a tab button (say). If I keep hitting tab button then I keep hitting the endpoint with mostly no new data arriving.

That does feel like a client side thing.
Sent with GitHawk

Moya (Alamofire) is felicitating the API requests here right? So either Moya or Alamofire should have this feature right?

@sandeepbol Yeah I'm a little confused as to whether the behavior you want here is something like tracking in flights or throttling requests, but this feature is out of scope for Moya. It's too client specific


Add a LICENSE file

The MIT license specifically states that the text of the license must accompany the source code.  This pull request adds a LICENSE file which contains the copyright header (copied from README.md) and the text of the MIT license.

Ok

Stop all phone is hacked


Remove the --list-different option for 4.0.0 compatibility

When using v4.0.0alpha3 we now encounter the following error:
The "--list-different" and "--write" flags cannot be used together

pre-commit fails when a file is modified by an autofixxer so --list-different can be removed afaiu.

this file is generated -- you need to edit the place it is generated from not here as it will just get overwritten

try git grep


Contributors in registrations should not be frozen

When creating a registration and trying to update it, the contributors are frozen and unmodifiable. This prevents contributors to the project from being added. Additionally, updates for the preregistration can only come from administrators at the time the preregistration was submitted. Administrators on projects may change hands depending on certain situations, so this makes updating an existing preregistration impossible.

Contact support at<EMAIL_ADDRESS>if you haven't already, they should be able to help you out more quickly.


Exclude CNTK legacy codes when reporting coverages

This PR excludes CNTK legacy codes when reporting coverages (the _padding is for the old versions that don't have C.pad). This affects only calculating test coverages.

The results are:

Before
After

Python 2.7 + C
88.26%
88.41%

Python 3.5 + C
88.23%
88.38%

(before) keras/backend/cntk_backend.py              1454    161    89%
(after) keras/backend/cntk_backend.py              1435    143    90%


rs merge: fail on 'Merging components'

Hello! Many thanks to the developers for this amazing framework! It is very helpful!
I faced with problem when use rs merge (stage 'Merging components'):

Traceback (most recent call last):
File "/usr/lib/python3.6/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/usr/lib/python3.6/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/usr/src/app/robosat/tools/main.py", line 58, in
args.func(args)
File "/usr/src/app/robosat/tools/merge.py", line 79, in main
area = int(round(project(merged, "epsg:4326", "esri:54009").area))
File "/usr/src/app/robosat/spatial/core.py", line 21, in project
project = functools.partial(pyproj.transform, pyproj.Proj(init=source), pyproj.Proj(init=target))
File "/opt/venv/lib/python3.6/site-packages/pyproj/proj.py", line 295, in init
self.crs = CRS.from_user_input(projparams if projparams is not None else kwargs)
File "/opt/venv/lib/python3.6/site-packages/pyproj/crs.py", line 224, in from_user_input
return cls(**value)
File "/opt/venv/lib/python3.6/site-packages/pyproj/crs.py", line 146, in init
super(CRS, self).init(projstring)
File "pyproj/_crs.pyx", line 317, in pyproj._crs._CRS.init
pyproj.exceptions.CRSError: Invalid projection: +init=esri:54009 +type=crs: (Internal Proj Error: proj_create: cannot expand +init=esri:54009 +type=crs)

What is problem? Maybe I have wrong environment setup?

Hey, glad you like robosat! :)
It looks like you are running the rs merge tool: in it we project into an equal-area projection so that we can calculate the merged shape's area.
Now it seems like pyproj has problems with the equal-area projection and is not able to transform properly. I don't have a solution off the top of my head and it used to work properly before.
I suggest the following:

see if you can provide a self-contained reproducible test case, that would help us a lot
downgrade the pyproj dependency to previous versions and see if it works again

@alektros any updates from your side here?

@alektros any updates from your side here?

Hello! While i am testing and experimenting with other functionality, so i had not back to this issue. But probably, i will work with this issue again in few weeks

@alektros any updates here? Can you reproduce with our pre-built docker images?

Hey,
I had the same issue, I changed a few lines in the project method of core.py and now it works.
  #project = functools.partial(pyproj.transform, pyproj.Proj(init=source), pyproj.Proj(init=target))     transformer = pyproj.Transformer.from_crs(source, target)     #return shapely.ops.transform(project, shape)     return shapely.ops.transform(transformer.transform, shape)

Coolio! Thanks @Venka97 for keeping us posted! :smile:
Do you want to open a pull request with your suggested changes? Thanks! :hugs:

Sure!


Upgrade: Generator to v1.14.0 / Base image to UBI9

[x] I have taken backward compatibility into consideration.

Including this in OCM v1.14.0 isn't critical (unless someone in the community asks for it or the UBI9 bump is desired).
I've added the -y flags to microdnf because in ubi9, those commands started blocking without it.

/lgtm

/approve


Links to examples seem broken or they aren't showing?

Yeah the link is broken, it should be link to   https://cdn.scotch.io/2/xwxZZIowT5q6jVLUAwCc_javascript.gif
scotch.io website doesn't exist anymore, because:

Scotch.io is joining DigitalOcean

You can see the link to the image from Raw README.MD.


Makefiles are added, and a couple of minor problems are handled.

Figuring out how to link the files correctly and that inline problem took my 2 days, so please be kind to those files :)

Is "Server/AIServer/main" the binary file you are producing? We should probably try and remove that before I merge these files. Other than that everything looks good.

Yes, that is the executable, but it is deleted already. I was probably in the repo because I forgot to delete before, but right now it is deleted.

Gotca!


Drag and Drop

I'm submitting a ...
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[x] support request

Current behavior
Very hard to add Drag and Drop to the tag <formly-form> and <formly-field>. I need to use nativeElement.querySelector stuff to get to the correct elements and add drag and drop functionality to those elements via their service and not via their directives as shown in the documentation.
  private initDragAndDrop() {
    const _this = this;
    const dropListRef = _this.dragDrop
      .createDropList(_this.formlyForm.nativeElement)
      .withItems(Array.from(_this.formlyForm.nativeElement.querySelectorAll<HTMLElement>('formly-field')).map(field => {
        const dragRef = _this.dragDrop.createDrag(field);
        const dragHandle = field.querySelector<HTMLElement>('div.drag-handle');
        if (dragHandle) {
          dragRef.withHandles([dragHandle]);
        }
        dragRef.withBoundaryElement(_this.formlyForm.nativeElement);
        return dragRef;
      }));
    dropListRef.withOrientation('vertical');
    dropListRef.dropped.subscribe(({ previousIndex, currentIndex }) => {
      // BUG: previousIndex, currentIndex are not always updated and often the same number so nothing happens...
      // BUG: https://github.com/angular/material2/issues/14873#issuecomment-461039077
      // * The problem might be flexbox, display block may work better.
      moveItemInArray(_this.fields, previousIndex, currentIndex);
      _this.fields = [..._this.fields]; // force rerender of fields after reordering
    });
  }

The biggest problem tho is that the drag and drop functionality doesn't work properly, previousIndex, currentIndex when dragging and dropping elements bug out and are often if not always the same number, so nothing gets moved. I have read on the github issue that it might be flexbox problems, but I'm not sure if I have a way to override the display property on the <formly-field> elements?
Expected behavior
I would like it to be easier to add drag and drop functionality and I am hoping you guys have an idea on how I could go about doing that a simpler way, and in a way that works.
Minimal reproduction of the problem with instructions
https://stackblitz.com/edit/angular-material-ngx-formly-drag-and-drop
Change the order of the elements up and down and you will se that they do not always change place and jump up and down

What is the motivation / use case for changing the behavior?
Be able to change the order of fields in an ngx-formly form

Please tell us about your environment:
Windows 10
VS Code
NPM
ng serve

Angular version: 7.2.3

Browser: Chrome 72

Language: 3.2.4

Node (for AoT issues): node --version = 10.15.1

use custom field type with-in a field group, see https://stackblitz.com/edit/angular-material-ngx-formly-drag-and-drop-u5sy9g

Thank you, I will have a look at that tomorrow at work. :)

@aitboudad if i use this.field.fieldGroup in moveItemArray visual representation changes, but not a model. What I can do to trigger the change?

@baryshkov the order of prop inside an object doesn't matter unless it's an array type which not the case for the above example.


Fix no std

Fix #![no_std] build and make the crate use edition 2018.
This pushes the MSRV to 1.36, but I could not any clue of the current or the crate's policy about it.
I kept the alloc feature (which is required when std is disabled) not to break current builds, and also to allow supporting no_alloc in the future.

I think an MSRV of 1.36 is fine. gimli's is 1.42.


How to open  Shopify checkout and payment options by clicking a buy button in an application?

I can't fully understand the check out functions. Can someone please provide me  the code or functions, which are available in this package to perform the functionality I want?
Or ideas on how to do this Please?

 This returns a checkoutId as string. With that you can use the other methods to assign a user, line items etc. Then you can get the infos with ShopifyCheckout.instance.getCheckoutInfoQuery  and complete the checkout for example with a webview WebView(                         initialUrl: _checkout.webUrl, .

thanks sir, i will try it and let you know if something gone wrong :)


Feature: Forward stdin from gotty to process

The ability for web clients to send input to the gotty'd process is awesome, but it would also be excellent to allow the user who is hosting the gotty process to use STDIN to "screencast" to consumers.
Ideal flow:
gotty nano /tmp/some/file

Gotty starts, and the tty used to start gotty acts exactly as if the command had been run without gotty. The user can "screencast" their input and results to web consumers - without web consumers necessarily being able to control the process.

This can be accomplished with a terminal multiplexer, e.g. tmux or screen. To use the former, start your casting session tmux named "gotty" with:
tmux new-s -s gotty

then start start gotty to attach to that session
gotty tmux a -r -s

Note the "-r" flag is for read-only, as gotty still sends an EOF even in read only mode, which can interfere with the master session when clients disconnect.


Temporarily use POMs to support ~/.ivy2/local

It seems that when sbt publishes artifacts to ~/.ivy2/local, it also puts Maven POMs there, in poms directories. These could be leveraged to quickly have support for ~/.ivy2/local, with the existing Maven support in coursier.

Fixed by https://github.com/alexarchambault/coursier/commit/95f51fdb8f0e5a71df2d89953c59ac33df1cb0c7 and https://github.com/alexarchambault/coursier/commit/db3b244b755dcd5f1bdf3f90f18d2832ad8e2ae3.


Passing properties to menu wrapper

Is there a way to pass properties (classname / id etc) to the menu wrapper? I can pass styles to the menu directly but these don't affect the outer container.
I'm trying to work around the issue with the dropdown hanging off the screen.
Any help appreciated! :)
Rob

Please try v1.5.6 then close this issue (or report any problem).

@Sharlaan - any idea on this one? It would be great to be able to add a className to the menu itself.

Did you try 1.5.6 ? this version with canAutoPosition should solve this issue with offscreen dropdown

I'm testing that at the moment in the other thread - I'm just asking about className and ID here :)

There is menuStyle but good idea about some menuProps properties applied to the inner Popover.


[ENH] diagnostic plots part 1 - cross-plots

Moves the diagnostic plots to the new interface.
Part 1 - cross-plots.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@47a1b19). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main      #46   +/-   ##
=======================================
  Coverage        ?   68.71%
=======================================
  Files           ?      102
  Lines           ?     4709
  Branches        ?      759
=======================================
  Hits            ?     3236
  Misses          ?     1262
  Partials        ?      211


GEODE-5509 Rewrite the docs on transactions

I'd like many reviews on this complete rewrite of our documentation on transactions.
In summary, I moved and renamed the existing, lengthy and detailed section on transactions into the manual section called Reference.  The new section on transactions is short and omits lots of nit-picky details on implementation and hacks a Geode user might take advantage of  (if they were super-knowledgeable and wanted to push the limits of what a transaction might accomplish).  Instead, this rewritten section attempts to provide directed examples upon which a Geode developer might model their own transactional code implementation.
In my attempts to be as concise as possible, I have left out details of features.  Reviewers: what have I left out that you deem important to say about transactions?  For example, I left out all mention of suspend() and resume().  Are they parts of the API that we should be presenting?
Reviewers: please also look over the code fragments with a detailed eye.

There are 2 outstanding issues to address before this PR can be merged in:

Removing the "archival reference section" requested by davebarnes97.  All reviewers, please speak up if you object to removing the older version of the section permanently and completely.
I talked with several Geode committers, and the material on using Geode as a "last resource" in JTA transactions is still relevant.  Therefore, it needs to be added back in, but probably revised/updated.

Other issues that have been discussed, but require no revision to this PR:

I am not adding material on the transaction API methods of suspend() and resume(). They are not intended for mainstream usage.  The javadocs are good enough for anyone who choses to use them, but by leaving them out of these docs, we will not be encouraging developers to use them.

Look good!


Replication: iteration 2

Two things are now possible:

Syncing frames from a snapshot file produced by sqld, e.g. with cargo run -- --max-log-duration 1
Using let mut db = Database::with_replicator(path) in order to create a database, on which you can then call sync(frames) to load new frames. No need to instantiate a separate Replicator anymore

@penberg the new example that loads a snapshot file is here: https://github.com/libsql/libsql/pull/199/files#diff-3d95ecad61bbbe4a26cbf0fa2448b9fcddde42ca2456dbc124fd1fb01331f707
I tested it by copying over the snapshots/ directory from sqld, and then running the example, e.g.:
$ rm -rf /tmp/snapshots; cp -r ~/repo/sqld/data.sqld/snapshots /tmp/snapshots
$ LIBSQL_SRC_DIR=$(pwd)/../../ LIBSQL_STATIC_LIB_DIR=$(pwd)/../../.libs \
  cargo run --example from_snapshot -- /tmp/snapshots/9beea4ab-653a-4db1-8f3a-7856a26079f9-0-3.snap


Documentation: Deprecated / Outdated Documentation

Issue: Some documentation can be removed or should be modified because it is no longer working.
Files:

https://github.com/ubccpsctech/classy/blob/master/docs/net-experiment.md
-- Test container does not work: docker build -t net test/container/network
-- Are these still the correct IP Tables configurations?

https://github.com/ubccpsctech/classy/blob/master/docs/portal.md
-- May not be used for anything anymore and can be removed.

portal.md removed.

  feature/documentation-updates removed on feature/documentation-update.

docs/creating-vm.md removed on feature/documentation-update.

No further inconsistent documentation seen.


CI test

I ran the script locally and it seems some cases failed, but I think the script is good.
@xuxinx do you want to fix them before we merge this?
ok  	github.com/qor5/admin/activity	0.248s
?   	github.com/qor5/admin/example	[no test files]
?   	github.com/qor5/admin/example/admin	[no test files]
?   	github.com/qor5/admin/example/cmd/publisher	[no test files]
?   	github.com/qor5/admin/example/models	[no test files]
?   	github.com/qor5/admin/example/pages	[no test files]
?   	github.com/qor5/admin/gorm2op	[no test files]
?   	github.com/qor5/admin/l10n	[no test files]
?   	github.com/qor5/admin/l10n/views	[no test files]
?   	github.com/qor5/admin/login	[no test files]
?   	github.com/qor5/admin/media	[no test files]
?   	github.com/qor5/admin/media/filesystem	[no test files]
ok  	github.com/qor5/admin/media/integration	0.219s
?   	github.com/qor5/admin/media/media_library	[no test files]
?   	github.com/qor5/admin/media/oss	[no test files]
?   	github.com/qor5/admin/media/views	[no test files]
# pkg-config --cflags  -- vips vips vips vips
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
Package vips was not found in the pkg-config search path.

Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
Package vips was not found in the pkg-config search path.
Perhaps you should add the directory containing `vips.pc'
to the PKG_CONFIG_PATH environment variable
No package 'vips' found
pkg-config: exit status 1

@idealhack will this fail the ci test?

I wonder if we should update https://github.com/qor5/admin/pull/34/commits/41795fdfaf50ad08d8c9e9ab34e58551fbe6d889#diff-678682767f2477de3e3c546746f8568b9a1942b2c647d32331d7e774b8ff8d9fR28 and use that file instead of this script as another wrapper.
And to answer your question, yes, it will https://github.com/qor5/admin/actions/runs/3853002409/jobs/6565590003

Let's continue at https://github.com/qor5/admin/pull/34 then, could you look at the logs and suggest what to change next?

https://github.com/qor5/admin/blob/main/.github/workflows/go.yml works well and we can continue to improve there.


Do not delete non-existing items from storage

Do not delete non-existing items from storage as this throws an unhandled exception with Google Cloud storage.

Thank you! Merged (and amended a bit to fix the black/flake8 error) in https://github.com/matthiask/django-imagefield/commit/eb77f62e3d5c542bdbe17ef97f32ecc8194fbc32

I just uploaded this to PyPI as 0.7.2.