FC2 Feedback: Ability to search all of (e.g.) Scotland, Wales

Need to know whether this can be done by 'Scotland' selects all Scottish counties, or does it need a database redesign?

A birthplace search for Scotland should search the birthplace counties of Scotland, and SCT (also WLS for Wales and IRL for searches for Welsh or Irish births).  Also, possibly for English.  This is a facility on FreeCEN1, so I've added to the Essential Features epic (phase 1).  Could be phase 2, particularly if it involves other place develeopment.

@Captainkirkdawson we're reviewing this in connection with a more recent task - is there anything preventing us from doing this search function?

Performance; likely most searches would produce too many results with surname only

Would it be possible if we required a forename?

If you look at the Full Version of the FreeCEN Handbook there is a Chapman Code hierarchy. (not in the abbreviated version). This table has three levels of Chapman Codes. We already use level 2 chapman codes to search the Gazetteer and during validation, so for instance YKS (level 2) will find ERY, NRY or WRY places (level 3). Tjis table may be useful for this story.

See my comment of story #238.
Geoff

From a researchers point of view this is the search problem we discussed at the scrum 1 may 24
Geoff

@AnneV-Learn to investigate code that can be reused/repurposed and estimate size

Yes this is relatively easy to code, estimated effort = 5 (i.e up to a day). Estimate is bit longer than I would have guessed because I actually found a bug in the existing Search functionality.

Problem Example
Census County restricted:
Searching for CROUCH Lydia (no other parameters) - yields 14 results - OK

Searching for CROUCH Lydia Census County = CHI (Channel Islands) - yields 0 results - Not correct as there is one in the 14 above.
Searching for CROUCH Lydia Census County = DEV (Devon) - yields 1 result - OK
Searching for CROUCH Lydia Census County = CHI (Channel Islands) and DEV (Devon) yields - 2 results - OK
Searching for CROUCH Lydia Census County = CHI (Channel Islands) and WIG (Wigtownshire) - yields 0 results - Not correct as there is one in the 14 above, also on the Results page 'When you searched for:' Census County details, CHI is expanded to 'Alderney or Guernsey or Jersey or Sark' but Wigtownshire is not listed
Birth County restricted:
Searching for CROUCH Lydia Birth County = WIL (Wiltshire) - yields 2 result - OK
Searching for CROUCH Lydia Birth County = CHI (Channel Islands) - yields 1 result (Birth County = Alderney) - OK
Searching for CROUCH Lydia Birth County = WIL (Wiltshire) and CHI (Channel Islands) - yields 2 result - Not correct it is missing the CHI result above.
I think I can see where the problem resides in the code but the User Testing of any changes I make to add the new functionality and fix the bug will have to be pretty thorough, so may take a while.

Ready for Testing on Test3

Searched with Elizabeth Jarvis OK
Elizabeth S Jarvis OK
Mary Caines OK including IRL entry
John Preece OK except HEF was included in ENG. It should also appear in the WLS results.
Geoff

@geoffj-FUG The expansion of WLS to include HEF (and LND to include SRY, MDX and KEN) has thus far only been applied to Gazetteer searches. Should it be applied to Birth County and Census County in the main Search now?

Let me put it to the FreeCEN Committee.
Geoff
From: Anne Vandervord @.>
Sent: Wednesday, May 22, 2024 5:05 PM
To: FreeUKGen/FreeCENMigration @.>
Cc: Geoff J @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] FC2 Feedback: Ability to search all of (e.g.) Scotland, Wales (#341)
@geoffj-FUG https://github.com/geoffj-FUG  The expansion of WLS to include HEF (and LND to include SRY, MDX and KEN) has thus far only been applied to Gazetteer searches. Should it be applied to Birth County and Census County in the main Search now?
—
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/341#issuecomment-2124022951 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFMXKYXMAVGWFY6OAS3ZDQ7RVAVCNFSM4D3WWS6KU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJSGQYDEMRZGUYQ .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AKCPIFL6UIXYZLNR4V4HJNTZDQ7RVA5CNFSM4D3WWS6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOP2NAJJY.gif Message ID: @.*** @.***> >

The feeling in Scrum is that we should offer all places that have been part of the County/Country in the past

Anne
I have discussed this with the scrum and the decision is that:
WLS will include HEF (as well as it being included in ENG)
LND will be Greater London for these search purposes. So it will include the Chapman Codes of MDX, SRY and KEN.
Geoff

@geoffj-FUG
LND to include Chapman Codes of MDX, SRY and KEN is a little problematic because LND is an individual chapman code not a Country group. As it stands at the moment if LND alone is selected for the Census County the list of Census Places will populate for selection because that list only poplulates if there is only one county selected.
How do you want this to work for LND?
Treat as selecting the group of multiple counties and so Census Place options not available or populates the Census Places options with list made up from all places in LND, MDX, SRY and KEN?

Anne
I am commenting from afar. Please say if I see things incorrectly.
If I am an Antipodean or an American, I would perceive London as being what makes up the modern Greater London. I doubt if I would recognise that there is an almost complete overlap with Middlesex and a partial overlap with Surrey and Kent.
I would therefore suggest that we treat it as selecting the group of multiple counties and places made up from all places in LND, MDX, SRY and KEN.
It is a shame that there is not a standard Chapman Code for Greater London. They re-used LND for the administrative area after 1974. If one had existed, we could have utilised that. I am not into creating one – we spent a lot of effort aligning things with the standard Chapman Code list.
Geoff
From: Anne Vandervord @.>
Sent: Thursday, June 6, 2024 9:23 PM
To: FreeUKGen/FreeCENMigration @.>
Cc: Geoff J @.>; Mention @.>
Subject: Re: [FreeUKGen/FreeCENMigration] FC2 Feedback: Ability to search all of (e.g.) Scotland, Wales (#341)
@geoffj-FUG https://github.com/geoffj-FUG
LND to include Chapman Codes of MDX, SRY and KEN is a little problematic because LND is an individual chapman code not a Country group. As it stands at the moment if LND alone is selected for the Census County the list of Census Places will populate for selection because that list only poplulates if there is only one county selected.
How do you want this to work for LND?
Treat as selecting the group of multiple counties and so Census Place options not available or populates the Census Places options with list made up from all places in LND, MDX, SRY and KEN?
—
Reply to this email directly, view it on GitHub https://github.com/FreeUKGen/FreeCENMigration/issues/341#issuecomment-2152101808 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKCPIFLUSWV6OYYUTG7KKVLZGBBCXAVCNFSM4D3WWS6KU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJVGIYTAMJYGA4A .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AKCPIFJBGZC47QLWCY4JFGTZGBBCXA5CNFSM4D3WWS6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOQBDHPMA.gif Message ID: @.*** @.***> >

@Geoffj-FUG
Sorry still not entirely clear to me. It is the ability to choose a specific place via the Census Places list on the Search page that I am concerned with.
So looking at the Search page if LND is selected as the Census County would you expect the ‘Census Places selection list’ on the Search page to be populated with a list of all places where we have data (census year in brackets) for LDN and MDX and KEN and SRY?
Normally that Census Places selection list is only populated if a single County is selected. So if the Researcher selects KEN  and MDX or LDN and ESS (ie 2 counties) the Census places list is not populated (as the notes on the page say).
So it would perhaps seem rather inconsistent.
Whatever we do we will need to clarify/explain for the Researcher I think.

Decision in scrum: advise users (upon selection of London) that they should also try other counties in their searches for London

@Geoffj-FUG
I have checked the code to make sure that LND is not 'expanded' as I wasn't sure how I had left it on Test3.
I have added an informational 'Tip'  about Greater London - not sure if this will suffice, or if the wording is ok - see what you think - it is deployed Test3.

It appears that we cannot do anything for London so this story is only about the other counties.
@AnneV-Learn I think they are all done other than LND. Please confirm.
Geoff

@geoffj-FUG
Yes that is correct. Please test on Test3 as I have added a 'Tip' (below Nearby Places) about what to do if Searching London which you may or may not like.

We have a problem with HAM / IOW.
We record POBs against IOW for places in IOW. However IOW is also part of HAM.
If I search for Sarah Jarvis IOW I get one result, birth place West Cowes. If I look for her in HAM she does not appear. I only get the HAM entries.
Geoff

@geoffj-FUG updated code is now available for testing on Test3

Thar works well.
We can deploy
Geoff

That looks good.
I checked CHI as well. All OK.
We can deploy.
Geoff

Done, closing


Make the "Pick a username" form submittable by Enter

We have the "Pick a username" form that asks to enter your name. Please make it easily submittable by the Enter (or Ctrl+Enter, at least), to make it quicker to fill.

That's a good idea!


Add component._context to src/internal.d.ts

This adds component._context to src/internal.d.ts.
https://github.com/developit/preact/blob/a5559ceea17ae9939eff21961608774cbefa31ba/src/diff/index.js#L81-L83

Coverage remained the same at 100.0% when pulling b735291f30a2a838f7b7b438a797958de7ae0814 on 38elements:patch-1 into a5559ceea17ae9939eff21961608774cbefa31ba on developit:master.


Fix for scope-related issue with node-webkit

There's a call to the function setImmediate() on schedule.js that breaks compatibility with the Node-Webkit runtime.
It seems that Node-Webkit combines part of node's global scope with the one used by Chromium. Because of this, some functions have to be called as attributes of the global variable.
I made the replacement, this should not break compatibility with Node but it would be good to give it a try just in case.

Merged manually, thank you. It will be in the 2.9.3 release


add multi-tenancy public tutorial

What this PR does / why we need it
Towards https://github.com/giantswarm/giantswarm/issues/25198
This PR adds a tutorial explaining how to ad a grafana organization via a CR
Things to check/remember before submitting

If you made content changes

Run make lint dev to render and proofread content changes locally.
Bump last_review_date in the front matter header if you reviewed the entire page.

FYI: internal ADR is here https://github.com/giantswarm/giantswarm/pull/32256 with a bit more detail.
@giantswarm/sig-docs I'm a bit confused where we would document higher level concepts like Multi-tenancy as would like to have a kind of overview page to not have to redefine multi-tenancy everywhere. Would you have some hints?

This documentation only talks about the tenancy read path as the write path will be in the data ingestion section.
So we have:

data exploration

how to access grafana
others ...

data ingestion

how to ingest logs
how to ingest metrics

multi-tenancy

how to create organizations

I think the write path should be explained in the data ingestion sections (as this will most likely be a label)
But then I'm not sure if we actually need a multi-tenancy specific section? cc @Rotfuks as i've talked about this a bit with you already


Sandbox adapter does not follow Adpaters behaviour specs

Current behavior
When using Ecto.Adapters.SQL.Sandbox pool, various functions expects adapters' meta to contain a opts key.
Adapters behaviour does not impose such key on meta.
In particular, Sandbox crashes with ecto3_mnesia adapter: see https://gitlab.com/patatoid/ecto3_mnesia/-/blob/master/lib/ecto/adapters/mnesia.ex#L117
Expected behavior
Sandbox adapter should either:

warn/log when opts key is not present
use default opts whenever possible

Alternative:

update Ecto.Adapters.init/{0,1}  callback typespec to include mandatory opts key

The sandbox is only meant to work with the SQL adapters that explicitly use Ecto.Adapters.SQL. If the goal is to implement the sandbox for other adapters, it makes most sense to build a specific sandbox to them, especially because this is mostly a layer on top of DBConnection.OwnershipPool. Thanks!

Ok, thanks for the clarification


Error running pycodestyle Error: spawn pycodestyle ENOENT

I have been getting this error for this package. I have installed pycodestyle using pip. Sadly nothing is working for me. Is there a fix for this?

Is pycodestyle available in your $PATH? Does it work if you manually specify the full path to it in the settings?

I just published v2.0.1 which should fix a similar issue if you are using pyenv.

Hi @Arcanemagus, it seems like the new update has solved my problem. Thanks again!

Thank @srolija as they provided the PR providing enhanced support for pyenv 😉.

Well thanks to both of you ;)

Could I add that simply upgrading pycodestyle even if Atom detected no updated fixed this issue for me? I just ran
$ sudo pip install --upgrade pycodestyle and restarted Atom. I think it could be useful for people coming here via search engines.


File size increased as huge

Hi,
I found one observation after edit an image as mentioned in the below.
At initially the selected image file size is "123 KB". After applied any changes or with out any changes then click download button means the file size will be increased upto "510KB".  Please assist me on this.
Thanks in advance.
Sathish

Version

Development Environment

Current Behavior

// Write example code

Expected Behavior

The download function was implemented using the 'toDataURL API' of the web library library 'fabricjs'.
While the image is being saved, the file size may change slightly when new image information is added or updated in the image file header.
The file size increase after editing the image appears to naturally increase as new visual information is added to the image.
If you need to optimize the file size, you can use a tool like Photoshop to save the file size on the web to reduce the file size.

There is still no way of decreasing size of Editor side?


Add sun2000-modbus to latest

Please add my adapter ioBroker.sun2000-modbus to latest.
This pull request was created by https://www.iobroker.dev c0726ff.

@mcm1957 JFYI This is the new one, but same errors from ioBroker.bosesoundtouch and ioBroker.sonnen-charger

ahh, oops, missed that you added the pending labe already... :see_no_evil:

No problem.
Please ignore the other adpters. I'll check as soon as I have time. Currently no action required by you

RE-CHECK!

replaced by https://github.com/ioBroker/ioBroker.repositories/pull/3038


Fixed widgets using echo in their run. #7417

Fixed all widgets that do not generate output in both their init() and run() classes.

Collapse widget is broken. See test.

I guess that's because of trailing \n is missing now.

Same about other occurances.

The trailing \n wasn't there in every widget, I've readded it to pass the tests.

Thanks!


3bot wireguard network

let 3bots in development mode speak over wireguard

@despiegk we need your approval to close this issue. please review and close or comment what is missing


CI:CTest results on cdash again

lgtm

I see 9 builds at the Experimental dashboard, last Friday: https://my.cdash.org/index.php?project=elastix&date=2021-02-20
They are from this pull request, right? Cool! However, it seems that the Build column is missing. Do you have a clue why?

Yes, that was because I only included the Test results, I now included the build results as well. Is there more information we want to include?
See Cdash docs:

CTest defines an ordered list of testing steps of which some or all may be run as a dashboard client:
Start
Start a new dashboard submission to be composed of results recorded by the following steps. See the CTest Start Step section below.
Update
Update the source tree from its version control repository. Record the old and new versions and the list of updated source files. See the CTest Update Step section below.
Configure
Configure the software by running a command in the build tree. Record the configuration output log. See the CTest Configure Step section below.
Build
Build the software by running a command in the build tree. Record the build output log and detect warnings and errors. See the CTest Build Step section below.
Test
Test the software by loading a CTestTestfile.cmake from the build tree and executing the defined tests. Record the output and result of each test. See the CTest Test Step section below.
Coverage
Compute coverage of the source code by running a coverage analysis tool and recording its output. See the CTest Coverage Step section below.
MemCheck
Run the software test suite through a memory check tool. Record the test output, results, and issues reported by the tool. See the CTest MemCheck Step section below.
Submit
Submit results recorded from other testing steps to the software quality dashboard server. See the CTest Submit Step section below.

configure I guess also

Ziet er al beter uit! https://my.cdash.org/index.php?project=elastix
Wat ik nog mis is de naam van de branch of de PR die gebouwd is. Bij ITK staat die wel vermeld op hun dashboard: https://open.cdash.org/index.php?project=Insight

Updated with #451


Missing several Linux tar balls on latest release

It appears like Linux arch like ppc64le is available for 1.5.1 but not 1.6.1.  Based on .promu.yml all Linux arches should be getting built.

Both releases have the exact same number of assets, and ppc64le is right there: https://github.com/prometheus-community/ipmi_exporter/releases/download/v1.6.1/ipmi_exporter-1.6.1.linux-ppc64le.tar.gz

I must have somehow missed it when looking at release page.  Sorry for the noise.


Citation widget failing

This works but the same URL with a different dataset key is giving 500. The API appears to work
The publisher has written to helpdesk

Logs show
Template render error: (/var/www/portal16/app/views/widgets/literature/latest.nunjucks) [Line 171, Column 47]
  Error: Unable to call `result["authors"]["slice"]`, which is undefined or falsey
    at Object.exports.prettifyError (/var/www/portal16/node_modules/nunjucks/src/lib.js:34:15)
    at /var/www/portal16/node_modules/nunjucks/src/environment.js:486:31

Works now. Apparently we rarely have papers without authors so it was never detected that is fails without. Looks fine now:
https://www.gbif-staging.org/api/widgets/literature/latest?gbifDatasetKey=4bfac3ea-8763-4f4b-a71a-76a6f5f243d3


Successful dapr init provides an aka.ms link

Output for a successful dapr init ends with an aka.ms link, which is MSFT-specific. Should probably change to a bitly link?
Output:

✅  Downloading binaries and setting up components...
✅  Downloaded binaries and completed components set up.
ℹ  daprd binary has been installed to /Users/user/.dapr/bin.
ℹ  dapr_placement container is running.
ℹ  dapr_redis container is running.
ℹ  dapr_zipkin container is running.
ℹ  Use docker ps to check running containers.
✅  Success! Dapr is up and running. To get started, go here: https://aka.ms/dapr-getting-started

This was fixed in 1.14


[WIP][FEATURE] storage and format cache configuration

Tasks:

[ ] test coverage
[ ] gather feedback for my changes
[ ] submit changes to the documentation

Informations:

Q
A

Tests pass?
none

Fixed tickets
none

BC Breaks
none

Doc
none

Is this related to https://github.com/sulu-io/sulu/pull/1139 or can we close this?


bugfix: exclude the effect of empty strings for agents

What type of PR is this?

Uncomment only one  /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:
/kind bug
/kind documentation
/kind enhancement
/kind good-first-issue
/kind feature
/kind question
/kind design
/sig ai
/sig iot
/sig network
/sig storage
/sig storage

/kind bug
What this PR does / why we need it:
if user agent is an empty string, we need to filter it to avoid cache failure.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Does this PR introduce a user-facing change?

other Note

/assign @Fei-Guo

/lgtm
/approve


Tracking deprecated features.

Feature
PR(or time)

Removed

GPU Objectives.
#3643
Now all objectives having GPU port will use GPU by default.
#4690

grow_fast_histmaker
#3836
Now is renamed to grow_quantile_histmaker.
✘

Some of R parameters documented in R/xgb.importance.R.
#1964
??
??

num_pbuffer_deprecated
??
??
✘

python/training.py: train::learning_rates
#1797
Now use use callback API.
✘

python/sklearn.py: seed
??
Now use random_state.
✘

python/sklearn.py: nthread
??
Now use n_jobs.
✘

XGDMatrixCreateFromCSC
#1600
XGDMatrixCreateFromCSCEx
✔

XGDMatrixCreateFromCSR
#1600
XGDMatrixCreateFromCSR_omp
✔

DMatrix::SaveToLocalFile
?? In 2015
??
✘

silent
#3982
Now use verbosity
✘

Pass Python List into DMatrix
#3970
Declare not supported
✘

gpu_exact
#4527
Users are encouraged to use gpu_hist
#4742

n_gpus
#4749
Use distributed frameworks like dask or  spark
#4749

reg:linear
#4267
Now renamed to reg:squarederror
#4267

XGDMatrixSetGroup
#4864
Use XGDMatrixSetUIntInfo instead.
✘

TreeParam.max_depth
#5101
This variable is never used anywhere
✘

Inspried by @thvasilo in #3929.(Thanks)  Lets keep track of deprecated features here.  Great chance that I missed something, anyone who notice please help adding it back. :)  @hcho3  Please help filling the ?? if it's still possible (please don't spend too much time on it, it's deprecated), otherwise I will summit a PR to remove everything that loss track.

Let's discuss whether num_roots should be kept or not. I think it is not documented and experimental and should be deprecated. Already, I decided not to support it when I first wrote hist updater.

@hcho3 I tried to find out what it does.  It turns out num_roots is not documented.  Is there any reference for it?

I think it is fine dropping them, but let us also keep the backward compatibility in mind. It might be useful to first introduce json exchange format, and then allow upgrade via json

Just found out that TreeParam.max_depth is never set and never used:
https://github.com/dmlc/xgboost/blob/3d81c48d3fa9a2a2ceb0d7388d3ac1436af7d8ee/include/xgboost/tree_model.h#L34
This field is different from the TrainParam.max_depth, which is given at
https://github.com/dmlc/xgboost/blob/42bf90eb8fbcda2fac494c43bc326c7ca1d254d0/src/tree/param.h#L29
Every occurrence of max_depth refers to TrainParam.max_depth, not TreeParam.max_depth. (Just verified with CLion)
It is not possible to remove max_depth from TreeParam right away, since TreeParam is serialized byte-wise and we cannot change the byte count. However, we can certainly remove TreeParam.max_depth from the new JSON serialization schema.

@hcho3 I'm only tracking the features that's already decided to be deprecated here,  so that we can have a time frame for removing those code, as you can see some of them are really old and still lingering.
Let's move whether some other features needed to be deprecated into new issues. :)

@trivialfis

I'm only tracking the features that's already decided to be deprecated here, so that we can have a time frame for removing those code

Got it. I'll file a new issue to deprecate TreeParam.max_depth.

No idea. The document said so

Turns out XGDMatrixCreateFromCSC is indeed deprecated, in favor of XGDMatrixCreateFromCSCEx.

@hcho3 Actually I deprecated a lots of things in #5101, but there is no user visible effect, as all deprecated parameters are duplicated in some way.  I just can't remove them as binary IO is still the default IO method.


Fix TreeItem's button being rendered under "Selected" highlights

Fixes #62951

I've rebased this PR just in case, but this doesn't seem to be a correct solution. The code that you have moved affects some global properties (namely, w) which are used below the old code, but above the new code. This means that the adjustments are no longer applied and the math is wrong.
What should be done is the computing code and the drawing code should be split. Computing code should remain at the top, and the drawing code can be moved lower, so that the buttons are drawn on top of everything.

@YuriSizov It should be correct now.

Analyzing the code, wouldn't it be simpler to just move the two button-related code blocks down instead of adding another for loop?

@YeldhamDev Please see the above discussion. Those code blocks affect some scope variables which are used by other, intermediate code.

@YuriSizov I saw it and already deleted the comment, but I guess I wasn't fast enough to avoid a reply. :stuck_out_tongue:

Thanks!


langchain_milvus versioning failing

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
poetry add langchain_milvus
pip3 install langchain_milvus
Error Message and Stack Trace (if applicable)
No response
Description
langchain_milvus package on PyPi is currently version 0.1.4, the 2 latest releases(0.1.5 and 0.1.6) seems to be failing during release. It is a breaking change for migrating to langchain 0.3.0
System Info
Package Information

langchain_core: 0.3.0
langchain: 0.3.0
langsmith: 0.1.121
langchain_anthropic: 0.2.0
langchain_openai: 0.2.0
langchain_text_splitters: 0.3.0
langchain_voyageai: 0.1.2

Optional packages not installed

langgraph
langserve

Other Dependencies

aiohttp: 3.10.5
anthropic: 0.34.2
async-timeout: Installed. No version info available.
defusedxml: 0.7.1
httpx: 0.27.2
jsonpatch: 1.33
numpy: 1.26.4
openai: 1.45.1
orjson: 3.10.7
packaging: 24.1
pydantic: 2.9.1
PyYAML: 6.0.2
requests: 2.32.3
SQLAlchemy: 2.0.35
tenacity: 8.5.0
tiktoken: 0.7.0
typing-extensions: 4.12.2
voyageai: 0.2.3

Hello,
langchain-milvus 0.1.5 was released today and should be compatible with langchain-core >= 0.3. Please shout if you run into any issues!


Replaced Linux fixed AMI source with source AMI filter

Replaced the hard-coded AMI source when building the Linux instance (using Packer) to use a filter query instead. This way the latest AMI is always sourced and the value doesn't need to be as actively maintained (matches the Packer builder for the Windows image, which uses a filter query).
Doing this also reduces the need to apply updates at build time.

Thanks @cawilson! Appreciate it! 🙇🏼♂


[WIP] Fix fnctl locking

I just discovered that lock calls go with from:to: but in the end the fcntl call uses from:length: and the to: argument is just passed as length

This still needs to be checked on linux about locks have proper offsets. On MacOS there seems to be no way to acquire information about file locks in the system


[WIP] start delay for docker containers

Hi,
I'm using rspec/serverspec/specinfra for testing docker images.  All of my docker images run supervisord to manage the services within the containers.
I'm having a problem where some of the rspec tests are failing.  I've tracked the issue down to the fact that the tests are running too soon after the container has started and hence the services haven't been spawned yet.  The failures are intermittent.  This might be a bit hacky, but I've added a configurable start delay to the docker backend.  I've found that a two second delay after starting the container resolves the issue.
I'd appreciate a conversation about whether this is a good way to achieve this and can be merged or whether there is something better that can be done.
Thanks
J

closing this since ive come up with something different


Change default mirror from rackspace.com to something different

boot.cfg currently specifies mirror.rackspace.com as the default mirror site. What's annoying about this is that linux distributions provided by Rackspace have modified /etc/apt/sources.list files, which again point to mirror.rackspace.com instead of official mirrors.
So how about the default mirror site is set to an official/independent mirror site?
For the time being, I work around this issue by manually changing the  mirror variable via the iPXE shell.

I'm open to suggestions on this.  I have the Rackspace mirrors by default as they are geo load balanced and gives me consistent folder structure.  From what I understand, the Rackspace mirror is literally just synced down and unmodified, so the reason they are probably getting those repos is because I add this in the kernel command line in order to prevent the user from having to set their mirror:
mirror/suite=${debian_version} mirror/country=manual mirror/http/hostname=${mirror} mirror/http/directory=/debian

I could probably drop those and force the user to set them.
I'd actually like to start moving mirrors to official mirrors for each project, but I also want to avoid having to specify the mirror for every country code and worry about differing folder structures on other mirrors.

I see it's not as easy as I first thought.

I also want to avoid having to specify the mirror for every country code

Debian and Ubuntu (at least those, I haven't checked other distros) provide mirrors with predictable URL patterns, i.e. ftp.<country>.debian.org and <country>.archive.ubuntu.com. An idea might be to allow users to specify their country, otherwise rackspace is chosen as a default mirror.

First step might be just to drop the auto mirror assignment on those OS and just let the user pick it during install when the installer has an easy list to pick from.  For those you have to manually type it in, I'd prefer to keep something in there to make the experience more user friendly.  It might mean a slower install but at least it'll happen.
Each OS handles their mirror differently so it can spiral into a time consuming task pretty quickly having to deal with mirrors, especially when the mirrors break or older versions get dropped to free up space.

What of https://www.mirrorservice.org?

@antonym For Debian mirrors, I think the best, officially-supported option is deb.debian.org.
It's administered by DSA (the Debian Systems Administration team) and backed by Fastly and Cloudfront so it should be fine pretty much anywhere.

See https://github.com/antonym/netboot.xyz/pull/193 I'm working on using official mirrors where possible and making them more configurable from boot.cfg.  Would love feedback/testing on it.

Closing this out since a bunch of the main distros are now tunable and available in boot.cfg.  If there are any missing or better mirror locations, please feel free to put up a PR.  Thanks!


Data Push of New HIVE3 Tables for ArgosDB

Push the new HIVE3 output tables to the database

Also uploaded new tables for a data push today

Done the most recent one


array return

Hello,
Is it at all possible to change this output we get from the array.. would make the programming sooooo much easier:
Array
(
[0] => Array
(
[id] => fan
[ip] => <IP_ADDRESS>
[deviceType] => urn:Belkin:device:controllee:1
[friendlyName] => Fan
[modelName] => Socket
[UDN] => uuid:Socket-1_0-221526K110047E
[class_name] => a15lam\PhpWemo\Devices\WemoSwitch
)
[1] => Array
    (
        [id] => heater
        [ip] => <IP_ADDRESS>
        [deviceType] => urn:Belkin:device:controllee:1
        [friendlyName] => Heater
        [modelName] => Socket
        [UDN] => uuid:Socket-1_0-221524K1100DBB
        [class_name] => a15lam\PhpWemo\Devices\WemoSwitch
    )

)
to:
Array
(
[fan] => Array
(
[ip] => <IP_ADDRESS>
[deviceType] => urn:Belkin:device:controllee:1
[friendlyName] => Fan
[modelName] => Socket
[UDN] => uuid:Socket-1_0-221526K110047E
[class_name] => a15lam\PhpWemo\Devices\WemoSwitch
)
[heater] => Array
    (
        [ip] => <IP_ADDRESS>
        [deviceType] => urn:Belkin:device:controllee:1
        [friendlyName] => Heater
        [modelName] => Socket
        [UDN] => uuid:Socket-1_0-221524K1100DBB
        [class_name] => a15lam\PhpWemo\Devices\WemoSwitch
    )

)

It is possible but what makes it so difficult to use it the way it is? You can always write a helper function to access this array by device id or name or any key. Or write a function to convert this array exactly the way you need it before you use it in your program.

why? because it would make more sense.. grab it as an key.. and not as a
random number... getting it the way it is now you have to make it work and
then sort... having it with an array with the keys.. you can always get the
array_keys or if you happen to know it already (ie is in a database) you
just have to refer to it by it's key name.
On Mon, May 23, 2016 at 10:23 PM, Arif<EMAIL_ADDRESS>wrote:

It is possible but what makes it so difficult to use it the way it is? You
can always write a helper function to access this array by device id or
name or any key. Or write a function to convert this array exactly the way
you need it before you use it in your program.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/a15lam/php-wemo/issues/6#issuecomment-220971642

Will consider this for next release/update. For now, I would suggest creating a helper function to access this array however way you want.


Issue1176 improve ios marker performance by X100

Fix problem https://github.com/airbnb/react-native-maps/issues/1176
When image props is provided, it will re-use UIImage if the image is same.
This will increase the performance by more than X100

Bump -- would love to get this in. Running into the same issue.

Looks great.
Can we get the merge conflicts resolved so we can give this a proper review?

Really busy today, I will try to resolve merge conflicts asap

Thanks @ericapply, been using a forked version of this to get past this. Huge improvement.

Please merge!🎉

@christopherdro please merge this PR

@spikebrehm @lelandrichardson  Any chance you can take a look?
This PR still needs to be tested and reviewed. There is no sense is rushing to merge this.
@bosung90 @grundmanise You can do what @ryancoughlin does for the time being if you really need this feature.

Bless your heart @christopherdro 🙌 🙏
I was getting nervous.

This has been reverted until we can resolve https://github.com/airbnb/react-native-maps/issues/1560

@ericapply - any luck solving #1560?
Undefined symbols for architecture x86_64:
"OBJC_CLASS$_GlobalVars", referenced from:
objc-class-ref in AIRGoogleMapMarker.o
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

@ericapply @christopherdro Any luck getting this going? I'm desperately in need of getting this if there's a workaround or a patch or anything? Thanks!


Convert-PowerBIDesktopToASTabular fails if there are referenced queries

Hello, Convert-PowerBIDesktopToASTabular (both using pbit or desktop app) fails to generate a valid model.bim if the pbix uses referenced queries. The referencing query expression is written as an single space instead of the actual query found in the pbix.
        "partitions": [           {             "name": "Referenced Table1-9200782a-7d17-4f56-a077-b910dbc65797",             "mode": "import",             "source": {               "type": "m",               "expression": " "             }           }         ],
Attached is a pbix with a single column empty table, not loaded to model, and a query that references the former and loads to model. Also in the same zip is the results from  Convert-PowerBIDesktopToASTabular.
Reference Test.zip

Latest push worked on my simple sample but failed on my IRL file because ' in ' appeared in a column name. Conceivably this would be an issue anywhere that string appears so I created a regex search string and have a pull request here https://github.com/DevScope/powerbi-powershell-modules/pull/53 for it.

I have merged your code. With my samples it works as it should. If everything is alright from your side please close the issue.
Thank you for your help :)

I've submitted another fix and a couple changes. Please review and accept the pull request, thank you!


I can not find a method item.getMetaData.

I can not find a method item.getMetaData (subID).

we do not have integer IDs/metadata in the api as mojang is phasing them out in vanilla

This covered by (ItemStack extends) DataHolder and the DataAPI.
https://github.com/SpongePowered/SpongeAPI/blob/master/src/main/java/org/spongepowered/api/data/DataHolder.java

MetaData remains in MineCraft. ID are changed to names. Additional ID (SubID) - stayed.
Wrire Code:
 case "item":
        if (args.length == 1) {
        Optional<ItemStack> itemInHand = player.getItemInHand();
        try{
             ItemStack itemstack = itemInHand.get();
             ItemType item = itemstack.getItem();

             String name = item.getName();

             Text ItemName = Texts.of(TextColors.GRAY,  "You hold" + name);
             player.sendMessage(ItemName);}
        catch(Throwable T) {
        Text empty = Texts.of(TextColors.GRAY, "Empty.");
        player.sendMessage(empty);}}
        break;

I hold in my hand a stone => return minecraft:stone;
I hold in my hand granite => return minecraft:stone.
The names are the same.
Goal: get MetaData (subID).

The damage/data value is no longer available in this API. (/give @p stone:4). Because it is not future proof.
You have to use the https://github.com/SpongePowered/SpongeAPI/blob/master/src/main/java/org/spongepowered/api/data/manipulators/blocks/StoneData.java for that and similar classes to resolve that yourself.
There may be a util for that somewhere but i dont now it.

Minecraft itself is phasing out int based metadata and moving to the block state system. This api has no representations of the old int based ID's and metadata

But as we now define what it is granite, and not a stone?

Is there something equivalent to blockstate for items?

@Azatik I believe you want ItemStack.getData(StoneData.class);

@AlphaModder And what is the result?

@Azatik That would be an Optional (optional.absent() if the item isn't stone)
Call .getValue() on it, and you will get a StoneType.

Return optional.absent() in all cases. About the second did not understand.

@Azatik Oh, sorry. I forgot you meant items. You'll want '''ItemStack.getData(BlockItemData.class).getState().getManipulator(StoneData.class).getValue();''' (I think.)

@Azatik StoneType is a pseudo-enum of STONE, GRANITE, SMOOTH_GRANITE, DIORITE, SMOOTH_DIORITE, ANDESITE, and SMOOTH_ANDESITE.

Code:
ItemStack itemstack = itemInHand.get();                    itemstack.getData(BlockItemData.class).getState().getManipulator(StoneData.class).getValue();

Result: error - "getState" cannot find symbol.

@Azatik Sorry, it's generic getData(), and getManipulator

Without MetaData badly. It is impossible to determine exactly what the player holds in his hand.

@Azatik What?

I guess more importantly it returns Optionals.
So you have to check for null references and unwrap it.

@Azatik You want to get the ItemType? That would be itemstack.getItem()

I already did. But Stone and Granite return minecraft:stone. And the question arose. How to determine exactly what it is? Stone or Granite?

@Azatik I told you, itemstack.<BlockItemData>getData(BlockItemData.class).getState().<StoneData>getManipulator(StoneData.class).getValue();

He wants minecraft:stone:4 aka granite.
ItemStack.getItemType just returns minecraft:stone. So he need StoneData and StoneType and similar classes as well.

@ST-DDT Yeah, and that's what I said :P

You forgot the optionals in your query. Thats why it does not compile for him.

But there are other items. And more of the mods.

@Azatik You need to add the Optional get calls, yes.

@Azatik The SpongeAPI does not support modded metadata/blockstates at the moment, afaik. Not until minecraft fully switches over to the BlockState system.

And what would it mean to sponge? Each unit will have its own id?

@Azatik Iirc there is a method to get all data from the block state.

@ST-DDT I can't find it.

@Azatik That is, unless someone makes a mod-sponge bridge.

Waiting until MineCraft will go completely on BlockState?

@AlphaModder
Collection<DataManipulator<?>> getManipulators() in DataHolder

Just to clarify a few things here:

The reason why the block id is minecraft:stone for both stone and granite etc. is because they are the same block id. Their "sub" id's per say are based on the StoneTypes available from SpongeAPI. Eventually, all sub id's of these "types" for various blocks will become individual block types themselves, so eventually you will end up with minecraft:stone, minecraft:granite, minecraft:smooth_granite etc.
The "damage value" system is entirely deprecated for the above reasons. I did remove all damage values possible from various sources, however, considering there are still plenty of blocks that have these intrinsic "sub values", they are kept as "types".
With regards to mods, well, mods do have to adapt as well, and I'm sure that some mods will end up providing a SpogneAPI compatible "Type" catalog of their block types.
The reasoning for having BlockItemData is because the ItemStack is infact just wrapping around a BlockState according to some arbitrary damage value system (currently for backwards compatibility until the damage value system is entirely removed in later versions). It doesn't make sense to keep around a damage value system that is going to be marked as deprecated as soon as it is added.
With BlockState, you can grab all the necessary information from the BlockState using:

if (item.getData(BlockItemData.class).get().getManipulator(StoneData.class).get().getValue() == StoneTypes.GRANITE) {
// you have granite!
}

While that example looks a bit confusing, I'm looking into ways to narrow down the data by allowing shortcuts if you will so it could be written like so:
if (item.getData(StoneData.class).get().getValue() == StoneTypes.GRANITE) {
    // you have granite!
}

Which in effect would first check that the item is intact carrying a BlockItemData, and if so, grabs the StoneData from the BlockItemData.

When it will be?

If minecraft decides to do it. Its really hard for us to make asumptions. But it will take months at least, so you shouldn't wait for it.

Player have some itemstack in hand, howto universally get what item exactly hold?
example of expected results:
minecraft:stone:1
biomesoplenty:leaves_0:3
tconstruct:slime_grass:2
Howto with spongeforge?

Have a look at
ArmorEquipable.getItemInHand() (extended by Player).
Then ItemStack.getItem().
And finally
CatalogueType.getId() (which is extended by ItemType)
The actual variant can be obtained by DataAPI. CatalogueItemData

Thx a lot.

Don't forget that ItemStacks can simply have BlockItemData of which your block states that you're asking about are going be returned as actual BlockStates.


Torso oscillates slightly after initialization

We recently updated the firmware of iCubDarmstadt01 and now after the normal boot the torso oscillates a bit. What could the issue be? Gains? However, the gains in the config files didn't change, the firmware did. Also icub-main but I am not sure if it affects the robot since the eth boards do the ctl.

The current configuration (with the changes to the previous one) is shown here

https://github.com/robotology/robots-configuration/pull/6

The last icub-main we used with stable behavior is

https://github.com/robotology/icub-main/commit/4a8c4f2e34980153d98b2d06a31ada8d33736e01
Any ideas? Shall I just play a bit with the gains?

Hi @paraschos
I wouldn't focus on icub-main indeed, as you said, but rather on FW and robots configuration.
First off, I'd try to get to a condition where the oscillation is no longer there, by (1) reverting the conf files and then (2) rolling back the previous FW (I don't know how much this would be doable/cumbersome). At least these two steps should likely help narrow down the problem.
Tuning PID gains should be the last resort.
cc @robotology/icub-support-team
@davidetome could you please double check these changes https://github.com/robotology/robots-configuration/pull/6?

Hi @pattacini,
I am afraid that such a process will take many many hours: in order to make the robot boot I would
have to re-flash quite a few boards as the latest icub-main requires the 2foc with protocol 1.5. Now if I revert to a previous firmware on the torso, I would have to revert icub-main. But the reverted icub-main will not now the v1.5 protocol of the rest of the joints and will seg fault. That means reflashing  the rest of the robot with, at least, the 2foc old driver. Not sure if I also get problems with skin, strain and/or mais.
A good idea would be to have some automatic way of reflashing the whole robot but it is not the case.
Update on the problem: the oscillations occur only near the zero position of the torso. All other configurations that I tried were stable. Only the pitch (3rd joint) seams to oscillate.
I checked the gains a bit, how did you ended up with values like Kp=1066.66? The appear quite precise to be hand set. Why there is no Kd part? My tune was to increase the derivative gain to Kd=150 and it performs quite better, but not fully cured.

Ok @paraschos understood. I proposed to do the roll back just to be 100% sure that nothing happened to the mechanics and/or electronics (e.g. encoder) in the meantime.
I suggested to tune PID only as last resort because those values should usually cover a wide range of conditions on different robots of the same configurations.
I honestly ignore how P, I, and D are retrieved; I don't know if a proper automatic procedure is used. Others (e.g. @randaz81) can chime in here.
We have some code to perform automatic tuning (see #2), though I believe it's not employed in this case.
The D gain is not always necessary as P and I are: it depends on a case basis. The oscillations can be definitely damped off by means of the derivative; my only concern was only to rule out the possibility that they are not caused by the interaction of new FW with old gains (you said they didn't change).

@davidetome Any updates, are the config files good?
Also some questions what is the "build" option that we added for? What was wrong
left/right MAIS device setting causing?

Hi @paraschos ,
the configuration files seem to be good.

The build param is a placeholder used for version checking, whereas the MAIS are those boards controlling the acquisition of the fingers encoders. Missing or wrong values for these specific options could bring about warnings and/or errors while launching yarprobotinterface so as loading incorrect parameters for one of the hands.
However, I don't think they can interfere with the problem under inspection.

@pattacini I want to take one more look today on this one. Then, if there is no resolution I am closing it.

Hi @pattacini and @paraschos,
if it can help, last week I tested on our robot iCubGenova02 the master release (we normally run the devel release on this robot) and I did not see any oscillation on torso.
I was aware of this issue, hence I looked at (and moved) the torso trying to find wrong movements. But I saw nothing strange.

Thanks @marcoaccame for testing.

@marcoaccame ty!

Update: the problem appears to be more general than the torso oscillation. We did the following:

Disable all the motors (idle) but the right leg. Small oscillations were still present
The same for the left leg
... and for the torso

We noticed that the oscillations are mainly due to the pitch joints. Playing around with the
leg PID controllers (position mode) we noticed that disabling the K_i did the trick. I am not
sure if there is any other scaling of the gains, but K_i~10K with clipping to 1.5K can lead
to bang-bang ctl with the clipped value.  We send some test trajectories to the legs j:0 and
tracking/reaching was relatively OK. What is the reasoning behind these high K_i values?

Hi @paraschos
What was the purpose of idling other joints when testing one specific?
Further, we could argue that K_i is too high but we cannot shut it down outright, given that we really need the integral term to compensate for steady-state errors due impairments, gravity...
The output of the I part is not clipped, otherwise we have overshoots and oscillations. The integral gets disabled and does no longer accumulate against the input error as long as we are beyond a specific threshold (this is the anti-windup mechanism) .
That being said, for knowing why we have those gains we would need to ask those who have been working with legs and/or tuning PID @randaz81 @traversaro @DanielePucci @francesco-romano. Thy might have more insights.

Hi @pattacini

What was the purpose of idling other joints when testing one specific?

To figure out which joint/joints are the cause of these subtle oscillations.  Additionally, by decoupling the system we avoided 'cross-talking' between the joints.

Further, we could argue that K_i is too high but we cannot shut it down outright, given that we really need the integral term to compensate for steady-state errors due to impairments, gravity...

I only did set it to zero for testing. What I also wanted to state is that the P gains is quite high and, therefore, the robot goes 'close-enough' (well that is a different discussion) to the desired.

The output of the I part is not clipped, otherwise we have overshoots and oscillations. The integral gets disabled and does no longer accumulate against the input error as long as we are beyond a specific threshold (this is the anti-windup mechanism) .

I am not sure if I fully understand this part. So there is no clipping in K_i, but the \dot{K_i} gets on/off
when the | q_real - q_des | <= threshold? What is the Integral Limit affecting?
Bottom line, I do not see any problem with the robot, maybe a bit of gain tuning to be done. When I find some time I will play around to find suitable gains for our setup (and since we mainly use torque control but for simple movements, we don't have high requirements for accurate tracking) and give you an update.

A pure P controller cannot reject external disturbances, neither compensate for uncertainties and unmodeled quantities, so the close enough you saw in some given experiments is definitely and unfortunately not enough for general usage.
I think you'll find this resource on the anti-windup mechanism, which comes accompanied with helpful diagrams and models, quite enlightening.
In short, the anti-windup serves to correctly deal with actuators' saturation (max output parameter) and even to further constrain the integral output (integral limit parameter < max output). In fact, in presence of such non-linearity, we must somehow notify the I block that its output won't be fully applied to the motor because of the saturation.
Basically, the anti-windup boils down being a feedback line connecting the I output to the I input through some logic entailing as net result to stop the integration while we are in the regime of saturation.

@julijenv  and I experienced oscillations when the torso joint no. 2 is at 0.0 degree.
If we move the torso to a different position, for example, at some negative values, the torso became stable. However. while using the torso, such a instability comes out again, maybe because of crossing of the 0.0 degree value.


marchingsquares-omay

This is an implementation of the Marching Squares algorithm. If anybody knows how to improve the interpolation, suggestions would be greatly appreciated. I learned about marching squares a long time ago, but didn't make any good implementations.

[x] I have read the steps to getting a blot
[x] I have read all the submission requirements and follow them
[x] I am submitting art that...

[x] is algorithmically generated (will meaningfully change each time the program is run)
[x] is your own work that is not copied from somewhere else
[x] is drawable through the editor(does not use outside tools)
[x] doesn't call Math.random() (See the documentation on randomness)
[x] is drawable on Blot itself (doesn't have lines overlap more than 5 times)

[x] Doesn't overlap lines more than 5 times
[x] Fits in the working area of 125x125mm
[x] Doesn't rely on colour to look good

Hi there! I’m closing this PR for now to clear up the repo. Please make it so that the artwork is random every time the 'run' button is pressed. Once you’ve made your edits/improvements, please make a new PR and we can review it there!!


Webserver federation/reverse proxy

I confirm that

[X] I have searched for an existing feature request matching the description.

Feature Description
A non-negligible amount of the game instances have a web-server feature allowing administration of the server via that web interface, however it is a pain to give each one its own reverse proxy entry and setup auth etc for communities where servers are coming and going somewhat regularly, especially when the people with access to amp are the same as the ones you want access to the web-servers.
I would propose:

The ability to turn on a basic reverse proxy embedded with AMP that is auto-configured for each instance that contains a web server feature.
Access through the reverse proxy be controlled by permissions in AMP, ability to have users that can only see the instances and access those admin panels (not directly configure anything in AMP itself)
A basic generated landing page where a user can click into any given game-servers web interface, or a link attached to the instance tile that allows quick browsing to that web interface.

This way users can quickly access these interfaces without a bunch of manual steps and without additional external configuration.

Would be really neat, but to note, AMP's webserver doesn't act as a reverse proxy, it uses Nginx or some other user-configured external proxy.

I'm not sure this one is really feasible. As Potato said, AMP is using a very basic c# webserver called Kestrel. It relies on external software for reverse proxy support.
What we might be able to do is rig up a way to link the game server's webpage in AMP on the Status tab or Community page.

yeah proxy or no, it would just be nice to be able to access the web-server features of various game-servers without having to manually setup a proxy config for each in the larger environment.

Accessing it through AMP just isn't all that viable. To get the same features, someone would need to write a panel plugin that uses the game's admin-interface's API in combination with the console, which would then need to be unique for each and every game.
The other option would be to write a panel plugin that integrates with whatever reverse proxy you use to generate configs for the web dashboards. Though do note, there's no way for AMP to gate access to said admin dashboards, it's on the game to set up whatever admin auth properly.

Yeah I guess without some kind of integrated proxy you can't really integrate with amp, that's too bad, would be really neat to have a dynamically updated option for this
A minor improvement then would be to just generate a link like the quick connect link with steam games to be easy click as long as it's configurable so we can save the path setup in our reverse proxy in some way
Eg for ut2004 instead of showing like domain.com:port show unreal.domain.com or whatever we've decided to use for that instance


Synchronisation handling for unknown meta

Many plugins add additional metadata to Products, users also raise support tickets about the handling of the additional data, so the behaviour should clarified and enhanced.
Example components include:

Yoast SEO (meta keys like 'yoast%' not synchronised, most fields not desirable to synchronise across translations)
WP-Lister Lite for eBay and other synchronization tools (mix of field types)
Currency switchers with per product/language pricing capability (normally should synchronize).
Autoposters such as NextScripts: Social Networks Auto-Poster
Some themes such as Flatsome apparently add data to products

Normally this data cannot be synchronised, Polylang appears to be not synchronising the data regardless of the options selected (eg Settings, Synchronisation, Custom fields has no effect).   One user appeared to be experiencing the opposite with the Flatsome theme and complained that he could not turn off integration, but did not provide sufficient information to diagnose.
Actually, most of these metas start with '_' underscore character.
Polylang copy_post_metas skips meta keys matching is_protected_meta() which starts by assuming any meta key beginning with '_' is protected [and should not be copied], there is also a filter is_protected_meta to override this.
One woo-poly user recommended: Currency Switcher for WooCommerce By Algoritmika Ltd
free on WordPress.org for 2 extra currencies   (also authors of Booster for WooCommerce, not sure if this is the same as related Booster feature or not)
This supports setting product level pricing per language which is then saved in wp_postmeta under meta keys such as:
_alg_currency_switcher_per_product_regular_price_EUR
_alg_currency_switcher_per_product_sale_price_EUR
_alg_currency_switcher_per_product_regular_price_GBP
_alg_currency_switcher_per_product_sale_price_GBP

If used, these should normally be synchronised to the different language versions, one wouldn't expect to have to set the EUR price manually in each language translation.
So how to extend the metas list / fields locker?
Options:

detect Meta keys and add them to "Other Metas" section in the list:  Tricky:  there are a lot, SELECT distinct meta_key FROM wp_postmeta returns >1,000 in my db and as these are not taxonomies there is no definition or grouping of which are valid/relevant.  Also this function gets called a lot, not only for the settings screen and on save of product but also for fields locker and other internal functions which test whether meta setting is set.

implement Meta extension panels for particular plugins worth supporting, in the same way as Variables Metas, and open that extension api

in Settings, Meta List, add text boxes for "additional fields to copy" and "additional fields to lock"

For the particular example here, this filter code could be placed in a theme functions.php or plugin to allow these values to be synchronised:
/**
 * Polylang meta filter, if true meta item will not be synchronized.
 *
 *
 * @param string      $meta_key Meta key
 * @param string|null $meta_type
 * @return bool True if the key is protected, false otherwise.
 */
function allowSyncCurrencyMeta($protected, $meta_key, $meta_type)
{
    $meta_prefix = '_alg_currency_switcher_per_product_';
    $length = strlen($meta_prefix);
    if (substr($meta_key, 0, $length) === $meta_prefix){
        return false;
    } else {
        return $protected;
    }
}
add_filter( 'is_protected_meta', 'allowSyncCurrencyMeta', 10, 3);

added to wiki at https://github.com/hyyan/woo-poly-integration/wiki/Product-Meta-and-Custom-Fields

@Jon007
PRODUCT_META_SYNC_FILTER and FIELDS_LOCKER_SELECTORS_FILTER  hooks are for this purpose
These hooks will allow other plugins to extend the meta list which should be synced and the fields which should be disabled by Fields Locker
But I agree that at least we should add a basic support for the most popular plugins which are used in conmbination with woopoly.
Closing for now as the basic support is already there


Can't select the Draw tool in View

When I click the Draw button in View, nothing happens. If the Text tool is selected, it stays selected, so I know that at least one of the buttons works.

I tried this on the default document that opens in View, and with several of my own documents.
I tried Edge and Chrome on Windows, Chrome on ChromeOS, and Firefox on Linux.
App version is  0.25.0.

I have the same problem, didn't work with my Samsung tablet too
I thought it was possible tu use the S-Pen, but it didn't do anything


I would like to modify the skill to have  facts for an specific planet.

In order to acquire more "skills" on development of Alexa Skills I would like to modify this  code and go in a different path.    I would like to:
1.) call the skill  "Alexa, open space curiosities.. "
2.) Alexa responds "For Which planet are you looking for facts  ?"
3.) I will respond for example "Saturn"
4.) Then alexa goes to the fact "website database" and respond with a fact BUT just for the chosed planet.
Somebody has ideas on how to start ?
I guess first I need to work with the slots and then with the index.js file   don´t ?

@tommycr check out this sample: https://github.com/alexa/skill-sample-nodejs-fact-in-skill-purchases
Look beyond the In-Skill Purchasing and you'll see it's a fact-by-category skill.  Make the planets your categories, and then update the source of the data (or embed the data).
You may want to explore the In-Skill Purchasing while you're at it.  :)

Thank you very much, I think is perfect for me... Helps me to understand better the lambda environment.
At this moment I am trying to clean the code as best as possible by removing all the unnecessary sentences, BUT I have not yet managed to pass it properly through the dialog model. At the end also I would seek to initially read the "facts" from the embedded data, not from data sources yet...  By the way I am also translating  the skill to Spanish language as well  ( es-ES) this task is working well so far.  In Skill Purchasing will be the next step, for now I want to feel good with the basics...

In-Skill Purchasing is only available in en-US, so if you're aiming for es-ES, no need to work on ISP's (yet).
If you need additional assistance, open a new issue or check out the forums
Good luck!


Lifting g from parent component

on normal component that doesn't have child, lifting g to Free HalogenF done by using liftH function, so that's also what's i expects can be done on parent component. But it doesn't, it look like i need calling liftH twice, something like this, note i am using ReaderT with Affable.
eval (Submit a) = do
  Config { currency, apiURL } <- H.liftH $ H.liftH $ ask
  orders <- H.liftH $ H.liftH $ lift $ fromAff $ fetchJson (apiURL <> "/orders")

it confuse me a bit first, why purescript fail to compile that telling me cant found instance MonadAsk for Free. So, i read the Halogen source, and then found Halogen use intermediate Free HalogenF for parent-child machinery to work.
If it intended behavior, then it good to mention it on example or documentation. For what i know, there are none. Or if i am doing it wrong, how it should be done?

ah great news @garyb.


Fix the rake tests

Not sure if this is exactly the right fix. I'm pretty new to Ruby, but I tried running rake and it seems most of the tests are failing:

72 examples, 38 failures

Looks like every test failure is the same root cause:
  TypeError:
    can't cast URI::Generic to string
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/abstract/quoting.rb:34:in `rescue in type_cast'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/abstract/quoting.rb:23:in `type_cast'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/sqlite3_adapter.rb:290:in `block in exec_query'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/sqlite3_adapter.rb:289:in `map'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/sqlite3_adapter.rb:289:in `exec_query'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/abstract/database_statements.rb:351:in `select'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/abstract/database_statements.rb:32:in `select_all'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/connection_adapters/abstract/query_cache.rb:70:in `select_all'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/querying.rb:39:in `find_by_sql'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation.rb:638:in `exec_queries'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation.rb:514:in `load'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation.rb:243:in `to_a'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation/finder_methods.rb:500:in `find_nth_with_limit'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation/finder_methods.rb:484:in `find_nth'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation/finder_methods.rb:127:in `first'
  # /home/bmccann/.rvm/gems/ruby-2.1.4/gems/activerecord-4.2.3/lib/active_record/relation.rb:155:in `first_or_create'
  # ./lib/elasticrawl/crawl_segment.rb:20:in `create_segment'

@benmccann I think your fix is correct since the URI is also converted to a string in Job#build_s3_uri. This should probably be extracted out to a utility method.

Gio / Ben,
thanks for fixing this, much appreciated! I've bumped the Gem version to 1.1.4 and pushed it to RubyGems. I'm just building updated versions of the deploy packages and then I'll update issue #10
Cheers
Ross


The date widget is hidden under the markdown widget. Need to fix z index

Example problem link
/DURC/requestlog/create

@ftrotter I don't have that particular resource URL, but it appears to work in Inventory Transaction, unless I should be looking at a different widget.


Correction for raw affiliation Univ Rennes, CNRS, ISCR-UMR 6226, ScanMAT−UMS 2001, F-35000Rennes, France

Correction needed for raw affiliation Univ Rennes, CNRS, ISCR-UMR 6226, ScanMAT−UMS 2001, F-35000Rennes, France
raw_affiliation_name: Univ Rennes, CNRS, ISCR-UMR 6226, ScanMAT−UMS 2001, F-35000Rennes, France
new_rors: 015m7wh34;02feahw73;00adwkx90
previous_rors: 015m7wh34;02feahw73
works_examples: W4318068908
contact: 97025d16bb586fef56ed87161169e89d:7e9ea34d85b3027ee242ec9759681ab2 @ univ-rennes.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


draft build artifacts take up DB storage

problem

draft builds are triggered manually on a specific branch or via PR tests
the draft build name contains the branch or PR number
the generated artifacts are files in S3 and tables in the persistent build DB
these artifacts should be deleted at some cadence with some conditions

ideas

use a scheduled action to delete artifacts

run every Sunday or Monday night
get the names of all branches and the numbers of all open PRs in the repo and construct a "build name whitelist"
for all schemas in product build DBs and all folders in product draft S3 folders, delete them if they aren't in the whitelist

closing in favor of newer issue https://github.com/NYCPlanning/data-engineering/issues/307


Add chainermn.testing.to_device function

This PR adds a new ChainerMN subpackage namelychainermn.testing.
Background:
There are ongoing efforts to integrate ChainerMN with ChainerX. To make the process more efficient, we will need some utility functions. For now, only one function get_device is added to the new package.
Changes
Add a function chainermn.testing.get_device().

pfnci, test this please

LGTM, What do you think @shu65

Jenkins CI test (for commit 67b632d77880abc490c9107e7eb6a8f9e36c8c21, target branch master) failed with status FAILURE.

Jenkins CI test (for commit 67b632d77880abc490c9107e7eb6a8f9e36c8c21, target branch master) succeeded!


[CLOSED] Add rendering of sections to manuals

 Issue by brenetic
Tuesday 16 February 2016 at 14:26 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/659

This follows the same approach as manuals and also adds a new test.

Ticket

brenetic included the following code: https://github.com/alphagov/specialist-publisher/pull/659/commits

 Comment by steventux
Wednesday 24 February 2016 at 11:34 GMT

@h-lame I've attempted to finish up this PR by implementing the section/manual validation in Section.find as discussed, it's not an entirely comfortable fit as it kind of pollutes the AR-iness of the Section.find method (hence the optional named param). That said, I think this is a worthwhile trade-off for the validation behaviour you suggested.
Also added some authentication support for features, this could be picked into a later PR if need be.

 Comment by h-lame
Wednesday 24 February 2016 at 11:50 GMT

Looks reasonable, however it seems the pattern in this app is to redirect when we get a document not found style exception (see https://github.com/alphagov/specialist-publisher/pull/665 and https://github.com/alphagov/specialist-publisher/pull/662).  Might be worth following the pattern established in those PRs (raise a ManualSection::RecordNotFound and redirect - in both "this section doesn't exist" and "this section does exist, but not within the supplied manual" cases).

 Comment by steventux
Wednesday 24 February 2016 at 11:59 GMT

Fair enough, I didn't notice this, will amend.

 Comment by steventux
Thursday 25 February 2016 at 10:13 GMT

@h-lame Thanks for reviewing, updated as discussed.

 Comment by steventux
Thursday 25 February 2016 at 11:34 GMT

@h-lame that makes sense and simplifies, thanks, updated.

 Comment by h-lame
Thursday 25 February 2016 at 11:56 GMT

:+1:


Feature/#46 create doughnut chart for showing the services called on the dashboard backend

Now there is a chart of calls per service on the provider's dashboard.

PS: We really need to start doing squash commits, because on each PR there are a lot of commits without any added value.
closes coatrack#46

I changed the position and this is now the current status:

Expand the Service API Calls per day chart to 3 charts size (like on the top line)

Updated master and suggestions applied

Branch reverted (this branch was successfully reverted to the state before merging with master)

Solved conflicts

This branch is updated, but I will still test the functionality

So I tested now the functionality and it is working, it is just not at the correct place :)

Now, it looks better, what do you think @gr-hovest-atb ?

Merged current master

I just build the branch but could not see the doughnut chart. @BrunoSilvaATB, do you experience the same behavior?

I just build the branch but could not see the doughnut chart. @BrunoSilvaATB, do you experience the same behavior?

So I tested and it seems that by mistake the doughnut chart was not correctly added on the frontend, so I just solved it by removing and adding the chart code and now it works again :)

I merged now the master branch with this PR, and this is currently working well.
@ChristophBaierATB could you also test it?


Error:  is not a valid SSR component

I installed the component and got this error:
Error: <TinySlider> is not a valid SSR component. You may need to review your build config to ensure that dependencies are compiled
Is this an issue with  or is it not possible to use with SSR? And in that case; would it be possible to use  with SSR in the future?
Thank you for your time.

The demo page uses SSR and it works fine, so I'm not sure why you are getting that error. What version of Svelte are you using?

Listening to the latest Svelte radio podcast where they talk about this. Will test and report back when I get a chance in a few hours.

No, removing the curlythings did not help. I am running Svelte 4.2.8 and Svelte Kit 1.27.7. Here is the complete error message:
Error: <TinySlider> is not a valid SSR component. You may need to review your build config to ensure that dependencies are compiled, rather than imported as pre-compiled modules. Otherwise you may need to fix a <TinySlider>.
But even if I remove SSR, it still fails, so I believe it probably is something with my config. Will try a fresh install tomorrow.

@MindDesign Were you able to get this resolved?

I'm getting the same error on a fresh sveltekit install too. The package.json file indicates "main" is index.d.ts which is a blank file, as is index.js.
I'm using vite v5.0.12
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>

This should be resolved in 1.1.2, please let me know if it works

This took a while to figure out, but I can confirm that I've fixed the issue in 1.1.4. Thanks!


Build web interface with dashboard approach. Streamlit or plotly dash would be good candidates. App could be hosted on heroku for low or free cost.

Hi @mlr07.
Exactly. I played with doing this a while back as a proof of concept with some previous code I'd written. This one's a bit clunky but you'll get the idea...
https://traj-planner.herokuapp.com/

This is indeed done with plotly dash and hosted on heroku.

This definitely the right direction. Very cool.


Remove OTA demo network manager files

Remove OTA demo network manager files
Description

These files are used to switch between different networks for the OTAv1 demo. BLE support will be added with changes to transport interface in a further PR.
Updated Project files and CMake to remove the reference.

Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks

/bot run checks


Adding requirement for no self-issued certs

@PPacentMSFT : Thanks for your contribution! The author, @jeffgilb, has been notified to review your proposed change.

#sign-off


Initial hot elements implementation. [Not ready to merge]

Two things I'd want to fix before merging:

This code should never show up in production (if for no other reason than it adds unnecessary bytes). What's the best way of ensuring that? Currently this uses goog.DEBUG for that purpose, which works well with closure compiler, but not anything else.
Need to use proxies when available, so that elements stay constructable after they've been hot reloaded.

Related to #673

Ideally hot-elements would live in its own package though, and optimizing out the unused hot reloading code could be done at application build time.
If we had such a mechanism we could use it in other places too, e.g. to log errors when unexpected things occur that would be too expensive to check for in production

We're really not going to have such a mechanism though, at least any time soon. There's no standard for stripping dead code and we've seen in other packages that dev code can easily end up in prod. My only thought at the moment is that we could vend an API compatible lit-html-dev package and allow build tools to swap it in.
In the mean time, patching seems best to me.

@rictic @justinfagnani there are any plans to move this to lit-next?

Thanks to @43081j we now have a HMR plugin for web dev server: https://modern-web.dev/docs/dev-server/plugins/hmr/
I created a small lit-html demo which applies some of the code of this PR to a simple base class using lit-html, just to check if this can work out: https://github.com/modernweb-dev/web/tree/master/packages/dev-server-hmr/demo/lit-html
It would be great if we could still introduce something like this before lit-next if possible.

@rictic we should port this to 3.0


[DO NOT MERGE] Auto PR to update dependencies of pilot

This PR will be merged automatically once checks are successful.
none

@istio-testing: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/pilot-presubmit.sh
3c128b7b75e327bcc21253551104dc1e29277e10
link
/test pilot-presubmit

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


[Build][Docs] Improve build documentation

Summary
oneMKL can be difficult to build and use. The reasons for this are:

The build documentation is fragmented and at times outdated
The compiler ecosystem and compatibility is a little confusing

Problem statement
The current build documentation has the following problems:

It suggests oneMKL supports Conan. Conan is no longer supported since https://github.com/oneapi-src/oneMKL/discussions/267

Consequently, you have scroll some way to find the CMake instructions.

In most cases, building oneMKL backends is a case of setting -DENABLE_XYZ_BACKEND=ON. This isn't clearly communicated by the documentation, that makes building look far more complicated.
The docs suggests setting TARGET_DOMAINS even thought it is set automatically.
At times the documentation is worded in a way that suggests that backends cannot be enabled simultaneously, even thought I think they can.
In most cases, building a backend with hipSYCL (which is now AdaptiveCpp) is similar to building with clang++ or icpx, but we have duplication of information instead.

We also support a range of compilers.

I believe that some backends build with ICPX and Codeplay's plugins for Nvidia and AMD GPUs, although this is undocumented. We should consider supporting and documenting these.

Codeplay's oneAPI for Nvidia GPUs, Codeplay's oneAPI for AMD GPUs

In cases where a backend is dependent on a particular compiler, we should consider trying to generate useful CMake errors.

Preferred solution

Consider rewriting building_the_project.rst to address the above problems.

Some aspects (e.g. additional options for building with AdaptiveCpp) could potentially move to new files.

Generate more errors in CMake to reflect compiler support for different backends.

Hi @hjabird We are looking for community support for this project. Are you (or any others) willing to pitch in and help improve the documentation?

Hi Craig, yes we will try to help with some of these issues when time permits. It would be useful to have some feedback before we start spending time on it.
To continue the discussion from https://github.com/oneapi-src/oneMKL/pull/458#discussion_r1519544088 I was thinking it could make sense to document some configurations that are expected to work but not regularly tested. This could help users try some configurations and report issues. I think the combination of all the domains, platforms, compilers and OS lead to a number of configurations too high to be regularly tested. I'm curious to have @mmeterel's opinion on this.

Hi Craig, yes we will try to help with some of these issues when time permits. It would be useful to have some feedback before we start spending time on it.
To continue the discussion from #458 (comment) I was thinking it could make sense to document some configurations that are expected to work but not regularly tested. This could help users try some configurations and report issues. I think the combination of all the domains, platforms, compilers and OS lead to a number of configurations too high to be regularly tested. I'm curious to have @mmeterel's opinion on this.

@Rbiessy I support adding documentation for untested configurations as long as how to use them are clearly documented and reproducible by internal and external developers. Main concern with not testing is, as the repo keeps evolving, how will we ensure these untested configurations will continue working? What do you think for such cases?
Another point about documenting which configs are tested and which are not. I support being transparent to users by providing this information (like our AMDs backend are not tested in CI yet), but we should make plans to close the gaps in CI in a timely manner. (For example adding AMD backends in CI is in progress)

I don't have a good solution to ensure that untested configurations will continue working. I would say that if people are trying to use them we will eventually get an issue if something is broken. That is my understanding of a community-driven project.
Regarding the CI I have some doubts about how much details we should share. I believe that Alexey (@toxicscum) is working on closing the gap but I doubt everything in the README can be tested.

I don't have a good solution to ensure that untested configurations will continue working. I would say that if people are trying to use them we will eventually get an issue if something is broken. That is my understanding of a community-driven project.
Regarding the CI I have some doubts about how much details we should share. I believe that Alexey (@toxicscum) is working on closing the gap but I doubt everything in the README can be tested.

Relying on user input when/if something is broken degrades the quality of the project/repo imho. One option would be, guaranteeing all the mentioned configurations work in every release. Then, at least we have a time stamp where the user can get a stable repo.
Regarding details about CI: When we mention about a configuration and say it is not tested regularly, we are already sharing this information. Ideally, in a repo like this, all CI should be public IMHO.

So we've had internal discussions about the release of oneMKL. We are planning to create releases of oneMKL interface and ship them with the oneAPI base toolkit. These will be tested by the oneAPI core team at Codeplay using icpx and the Codeplay Nvidia and AMD plugins. As I far as I know this is the only recommended way for users to get DPC++ running on all hardware. I am planning to start a separate discussion on the oneMKL interface release eventually.
I hope it makes more sense why we think we should document some configurations that are expected to work but may not be regularly tested yet.

This is news to me (releasing oneMKL interfaces as part of base tool kit) Is this in replacement of the releases done in oneMKL interfaces repo or in addition to it? Tagging Irina and Maria for their information and inputs. @mkrainiuk @itopinsk

I would prefer to keep the discussion on releases separate. I will try to start the discussion this week.

I agree that there should be better documentation for building and running tests in oneMKL.
It also seems that there isn't support (as far as I can tell) for building with non standard ROCm installation. If the build process was better documented, the process could potentially also act as a survey on what works and what doesn't. This would be valuable IMO

As part of this, the documentation for consuming oneMKL with CMake could also be improved. The documentation does not detail CMake targets. This was bought up in https://github.com/oneapi-src/oneMKL/issues/501.

Hi Team @Rbiessy @mmeterel @hjabird @mkrainiuk @hdelan, is there timeline in place for the same as was expecting we could meet Q2 timeline of June 28th for https://github.com/uxlfoundation/open-source-working-group/issues/38

HI @vbm23, there should be a PR in a few hours! Working branch here.

@hjabird Since #510 would partially help close this issue, would the other one be #501?
@Rbiessy Feel free to comment

@vbm23 I think the other thing we can do is generate useful errors.

https://github.com/oneapi-src/oneMKL/pull/490 resolved errors that you'd get from HIP_PATH being wrong.
#510 improves the build documentation

I also added documentation on consuming oneMKL with CMake as part of this PR.

Finally we can do more to generate more useful errors. For instance, using AdaptiveCpp with unsupported backends gives you compiler errors for features that not included in AdaptiveCpp. Ideally we'd generate an error along the lines of "AdaptiveCpp is not supported for this backend" instead.

I think we're getting there, but not quite there yet!


DSA

DSA: Spiral Print of 2D Array #1547

@themonikanayak Kindly resolve merge conflicts

@themonikanayak Kindly resolve merge conflicts

How to do that?

Update ur local branch with the branch using git pull upstream/(your branch name)

please review  @Sneha-0308

PR has merge conflicts and the issue is repetitive so closing it.


Where is the install-script or min-install-script?

I am trying to install the library (and the sample applications) in a RPi model 4, but when I try to follow the instructions from this link --> https://docs.aws.amazon.com/kinesisvideostreams/latest/dg/producersdk-cpp-rpi.html#producersdk-cpp-rpi-camera.
It asks me chmod +x install-script
Where do I find this script?  It is not in the repo

The documentation is stale, we no longer use scripts to build but rather cmake itself. More info in the readme page

Closing this issue.


added partial support for built in validation on PathFormField

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis. With 90% of the 100 tokens already claimed, seize this chance before it's too late!
muhammadfaizan, @Soros1223, @coni2k, @RiccardoMPesce, @dchem, @SdaixuS, @akaxiaok, @jinet, @typeless, @X-Summer, @jcshaw, @spclkr, @captswag, @Junior88, @HarambeNakamoto, @raineroviir, @kevinqianjiawen, @tymscar, @hamidra, @maanuanubhav999, @valleylord, @highlanderkev, @Alheimurinn, @mdfahadabbas, @HarrisonLin, @AngelAlvie, @paulsamra, @apo-bozdag, @iceco1a, @kaovilai, @aspearman, @wisepjh, @Ray5479, @turretwar, @Plenituz, @RintNil, @taxiyoshida, @arthurhenrique, @maithanhduyan, @Doxterpepper, @970814, @ChenShisen, @smilebot, @cjlymar, @jerry-024, @flurischt, @manzettidenis, @epicbagel, @fun4u, @sunxiaoyu


Removed unnecessary macros

Added a common makefile in the project repository for the common macros like D, P, Q etc are defined and they have been included in the rest of the makefiles using include .
Fixes #159

Hello @smlambert @llxia @renfeiw , I've made a PR. Looking forward for your reviews so that we can make it better.
Thank You

Oh, I am sorry @llxia , I was also working on another issue in this repo, let me fix this.
Are rest of the things fine, like the ones that are concerned with the isse.


Include something similar to “without any introduction” in ListOutputConverter

I find that the results improve when I add ", without any introduction." to the prompt.
Example:
List 10 unique unicorn names suitable for IDs with just regular ASCII characters.
{format}

->
[Here are 10 unique unicorn names that can be used as IDs with only regular ASCII characters:

Sparkle, Frosty, Onyxia, Razzle, Celestia, Starlight, Nimbus, Luminar, Aetheria, Zephyrine]

"Here are 10 unique unicorn names that can be used as IDs with only regular ASCII characters" should not an item.
When I modify it to:
List 10 unique unicorn names suitable for IDs with just regular ASCII characters,
without any introduction.

the result is correct (at least with Ollama llama3 and ChatGPT 4)
[Starlight, Sparkles, Mythic, Onyxia, Celestia, Astral, NovaSpire, Lunaria, Solaris, Mirabel]
I believe adding  , without any introduction. to the ListOutputConverter could be beneficial.
https://github.com/spring-projects/spring-ai/blob/d7dad6ef6162a057034297222575e13401233492/spring-ai-core/src/main/java/org/springframework/ai/converter/ListOutputConverter.java#L39

Thanks for taking the time to report, interesting finding.  will add

I've made a small refinement.  Please let me know if there are any models for which this is failing that you encounter.
Thanks for reporting this
merged in 4b95dfc2efe4c7d85d955dd97e6446031c60f5ea


Add Docker Support

Commits

Listening defaults to <IP_ADDRESS>:5000 instead of <IP_ADDRESS>:5000 for Docker networking.
Add Dockerfile.

Usage

Add config files (switch.yml etc.) into a directory (e.g. /home/me/justlist-config/).
Run docker run -d -p 5000:5000 -v /home/me/justlist-config:/usr/src/app/config DOCKER_IMAGE_TAG_HERE

Notes
I have already pushed hmqgg/justlist-docker to Docker Hub based on my fork with Cloud189 login fix patch.
If you wanna build an official image, please let me know so I could pull down my own Docker Hub image.

Thanks for your contribution again (hah)! This is a feature I wanted to add but haven't done for a long time. Supporting Docker is very helpful. I will modify the README later.
I tested your image (docker-hub@hmqgg/justlist-docker) and there is no problem. I will then make and push an image of this codebase (github@txperl/JustList). If you have other purposes, you can keep your Docker Hub image. Thanks.
Merged.


Display Table based on Selection

If you press on the specific bar or line, a table with data should appear below based on selection.
Question:
Is it possible to do it in d3?
Info:
A concrete example is this picture below.
Thanks!

Yes
Have a look at the gallery demos found at https://github.com/mbostock/d3/wiki/Gallery.
As for examples, you can see an implementation for what you describe at http://square.github.io/crossfilter/ then you could embed a line chart into the table http://bl.ocks.org/llimllib/841dd138e429bb0545df
You might look at this demo as well http://xliberation.com/googlecharts/d3flights.html
Please close this issue, thanks.


The markdown text in the dashboard entries of comments of pull requests is not rendered

Gitea version (or commit ref): 1.11.4
Git version: 2.20.1
Operating system: Debian 10
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[x] Yes https://try.gitea.io/
[ ] No
[ ] Not relevant

Description
When you open Gitea the dashboard is displayed which shows a brief history of
the contributions like who committed what and where, comments of pull requests,
etc.
The comments of pull requests however are not properly rendered using Markdown,
the raw text is displayed instead, see attached screenshot. The pull request
page itself does it correctly, so the problem is in the dashboard page.
Screenshots

it's not that that it's not all rendered - it's simply that it is not rendered as markdown at all.
The reasoning is simple - how do you prevent the comment notification from completely messing up the appearance of the notifications list?

Closing as it works as intended.

The reasoning is simple - how do you prevent the comment notification from completely messing up the appearance of the notifications list?

don't you have the same issue on the Conversation tab of the pull requests? Why is that different?

@shaoran because it's only a fragment, not the whole comment. And Markdown can contain tables, lists, images, and all sort of things. The intention is to provide some brief context without cluttering the view.
Example of Markdown I wouldn't want rendered in my dashboard:
Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua.
Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat.

Duis aute irure dolor in reprehenderit in voluptate velit esse cillum dolore eu fugiat nulla pariatur.
Excepteur sint occaecat cupidatat non proident, sunt in culpa qui officia deserunt mollit anim id est laborum.


Improve page loading speed. Create anchor links

@a-chabin: I'll write down the changes here:

[x] Improve page loading speed

[x] Preload notion data
[x] Preload notion images
[x] Remove loader

[x] Add anchor URLs to placemarks https://map.ekaterinburg.design/#[object-name]-[id]
[x] Update components

[x] Add Modal
[x] Add Info
[x] Update Filter position and colors

Thanks! 🎉


plugins/treesitter: clarify nixGrammars and ensure_installed

Resolves https://github.com/nix-community/nixvim/issues/1865

@mergifyio queue


Delay importing autoscript in fibsem.data (Travis build fails)

https://travis-ci.com/DeMarcoLab/fibsem/builds/104900351

https://github.com/DeMarcoLab/fibsem/pull/24


Create readme.md

Motivation
Custom script to rotate auth key of an account.
Have you read the Contributing Guidelines on pull requests?
YES
Test Plan
CI/CD covers the test cases

closed


Compatibility issue with Laravel 8

Describe the bug
Hi!
I'm getting this dependencies issue when I attempt to add the library to a Laravel 8 project:
-> composer require bugsnag/bugsnag-laravel
Using version ^2.22 for bugsnag/bugsnag-laravel
./composer.json has been updated
Running composer update bugsnag/bugsnag-laravel
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - bugsnag/bugsnag-laravel[v2.22.0, ..., v2.22.2] require bugsnag/bugsnag-psr-logger ^1.4 -> satisfiable by bugsnag/bugsnag-psr-logger[v1.4.0, ..., 1.5.x-dev (alias of dev-master)].
    - bugsnag/bugsnag-psr-logger 1.5.x-dev is an alias of bugsnag/bugsnag-psr-logger dev-master and thus requires it to be installed too.
    - bugsnag/bugsnag-psr-logger[dev-master, v1.4.0, ..., v1.4.3] require psr/log ^1.0 -> found psr/log[1.0.0, ..., 1.1.4] but the package is fixed to 2.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - Root composer.json requires bugsnag/bugsnag-laravel ^2.22 -> satisfiable by bugsnag/bugsnag-laravel[v2.22.0, v2.22.1, v2.22.2].

Seems to be a problem with version 2.0.0 of psr/log, which came with symfony.
Steps to reproduce

start a new project: curl -s "https://laravel.build/example-app" | bash
navigate to this new project directory: cd example-app
add a dependencies on bugsnag-laravel : composer require bugsnag/bugsnag-laravel
3.1 this step should failed

Environment

Bugsnag version: latest
PHP version: 8.0.8
Composer version: 2.0.8
Laravel version: 8

If you manually add bugsnag and run composer update, it should work. Composer is not smart enough to downgrade the psr/log package to the 1.x line.

@GrahamCampbell Thanks for your suggestion. This solution works, but I consider it as a workaround as it also update all my dependencies, which is unwanted.

I suppose if you ran composer require psr/log:^1.1 first, and then the require for bugsnag, it might work?

@GrahamCampbell Yes it works. I did think about this solution but I was not sure if downgrading psr/log was a good idea.
Do you plan to support psr/log 2.0?

We're looking at adding support for PSR v2 and v3 fairly soon. We'll provide an update here when that is available. Thanks!

Hi @guyaumetremblay - just to let you know we've added support for PSR Log v2, this was released in v1.4.4 🎉


Enable https for planet.openhistoricalmap.org

Bug description
Currently, http://planet.openhistoricalmap.org works but https://planet.openhistoricalmap.org does not.
Ideally, we need both https://planet.openhistoricalmap.org and https://planet-staging.openhistoricalmap.org to work with valid SSL certificates. Apart from just being best practice, not having https enabled will cause problems when trying to use resources on planet.openhistoricalmap.org via CORS inside https contexts.
There's a bit of a secondary issue where the direct s3 link does not work over https either since the bucket has a . in the name. But I think we can not worry about that if we get valid SSL certficiates working for planet.openhistoricalmap.org and planet-staging.openhistoricalmap.org
@Rub21 would you be able to look at this? I'm not 100% sure the best way to go about this but can just look up "enabling SSL for custom domain pointing to s3" or so and hopefully it's just connecting a few things in the AWS console?

@batpad, According to my researching there is not easy way to configure SSL for static websites in s3, https://stackoverflow.com/a/42441997/2280137,   we can use cloudfront  as we did for tasking manager or other of our sites, but transferring a lot number of files to may incurs  Data Transfer cost

@geohacker can you please advise here? I think maybe we just do CloudFront? This is a bit urgent to get done.
The problem here is that we can't use the s3 URL directly either because our bucket contains .s in the name. If we did not have .s in the name of the bucket we would at least be able to use the s3 URL directly with https - but the problem is that if the bucket name contains .s in it, the wildcard subdomain SSL certificate for the s3 bucket no longer works.
Maybe we should catch up on this? Unfortunately, this will be good to fix sooner rather than later.

@Rub21 reading up a bit, seems like there is really no option and we will have to create a CloudFront distribution. I think we should just go ahead and do that.

This should definitely go behind a cloudfront. It's something we have always wanted to do, just never got around to it. Let me try to do this right away

This is fixed now. I added a cloudfront to serve https://planet.openhistoricalmap.org/
Had to also update the bucket url in the index.html to use the https://s3.amazonaws.com/ domain so everything is served via HTTPS


Chat submit metaenter

Small contribution to chat UX: submit on special enter, ie command+enter on mac or control+enter on pc
No more need to use your mouse to click the button or tab to change focus! just press cmd+enter or control+enter to submit the chat
I'm a newbie, so forgive any mistakes or non-conventions

Thanks @simonpure for the feedback. I corrected as per your suggestions and added hot keys to change channels! cmd or ctrl + arrow up or down will iterate over the existing channels
Ran lint and format commands too.

Hi @simonpure, I just pushed the requested changes. Thanks for the feedback!


lock.db2

Hi, i get an error reading the most recent lock.db2 file.
It tries to read behind filesize:
Message: System.IO.EndOfStreamException: 'Unable to read beyond the end of the stream.' in LoadRecord
Code:
storage = new<EMAIL_ADDRESS>true);
BTW: Very handy tool you developed here :) tyvm!

I'm seeing this issue just now; is it still happening?


[8.x] PHP 8: method_exists(): Argument #1 () must be of type object|string, array given

array is no longer allowed on method_exist() on PHP 8

Failing tests are unrelated and I'm looking into those now.

@deleugpn can you send this to 6.x please?