Saysomething broken

Getting an exception from saysomething when starting pmxbot:
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.4/threading.py", line 920, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.4/threading.py", line 868, in run
    self._target(*self._args, **self._kwargs)
  File "/home/pmxbot/pmxbotenv/lib/python3.4/site-packages/pmxbot/saysomething.py", line 105, in init_class
    cls.markov_data = markov_data_from_words(words)
  File "/home/pmxbot/pmxbotenv/lib/python3.4/site-packages/pmxbot/saysomething.py", line 31, in markov_data_from_words
    for word in words:
  File "/home/pmxbot/pmxbotenv/lib/python3.4/site-packages/pmxbot/saysomething.py", line 68, in words_from_lines
    words = line.strip().lower().split()
AttributeError: 'tuple' object has no attribute 'strip'

I've been carting around a pretty old sqlite db in case that makes any difference.
Output of pip list:
backports.method-request (1.0)
beautifulsoup4 (4.4.1)
CherryPy (5.1.0)
feedparser (5.2.1)
inflect (0.2.5)
irc (14.1)
jaraco.apt (1.0)
jaraco.classes (1.3)
jaraco.collections (1.3.2)
jaraco.compat (1.2)
jaraco.context (1.5.2)
jaraco.functools (1.11)
jaraco.itertools (1.7.1)
jaraco.logging (1.3.1)
jaraco.text (1.6.3)
jaraco.timing (1.2.2)
Jinja2 (2.8)
MarkupSafe (0.23)
more-itertools (2.2)
pip (8.1.1)
pmxbot (1113.6)
pytz (2016.3)
PyYAML (3.11)
requests (2.9.1)
setuptools (20.6.7)
six (1.10.0)
tempora (1.4)
wheel (0.29.0)
wordnik-py3 (2.1.2)
yg.lockfile (2.1.1)
zc.lockfile (1.1.0)

Ubuntu 15.10, Python 3.4.3+
:heart:

Hi @schmichael!
I've pushed a fix in https://github.com/yougov/pmxbot/commit/a5da1fd878a7eaa57bc02077aee71a2fe5b9e6e5
Can you please pull and try again?

Doesn't look like that fixed it:
pip install --upgrade git+https://github.com/yougov/pmxbot.git
...
Successfully installed pmxbot-1113.7.dev2+nga5da1fd

pmxbot config.yaml
...
Exception in thread Thread-1:
Traceback (most recent call last):
  File "/usr/lib/python3.4/threading.py", line 920, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.4/threading.py", line 868, in run
    self._target(*self._args, **self._kwargs)
  File "/home/pmxbot/pmxbotenv/lib/python3.4/site-packages/pmxbot/saysomething.py", line 105, in init_class
    cls.markov_data = markov_data_from_words(words)
  File "/home/pmxbot/pmxbotenv/lib/python3.4/site-packages/pmxbot/saysomething.py", line 31, in markov_data_from_words
    for word in words:
  File "/home/pmxbot/pmxbotenv/lib/python3.4/site-packages/pmxbot/saysomething.py", line 68, in words_from_lines
    words = line.strip().lower().split()
AttributeError: 'tuple' object has no attribute 'strip'

Can you try removing pmxbot.sqlite, if you have it in the root dir, or change config.yaml to point to a non-existing sqlite db? I suspect the issue there.
Alternatively, you can try to comment out either words_from_logger or words_from_quotes in saysomething.py:words_from_logger_and_quotes and see if you manage to make it work.
If you can't pinpoint the issue, it'd be great if you could share your settings, so that I am able to reproduce the issue here, 'cause at the moment I can't.

An empty database fixed it, but probably because there was no data to trigger the error?
The schemas were the same, so I don't think it's a schema issue.

@schmichael I managed to reproduce the error you are getting by inserting a row with an empty quote in the sqlite DB. Can you please verify that you have rows with quote=null in your DB?
I will push a fix for that, anyway.

See https://github.com/yougov/pmxbot/commit/0af3f498bc228ad051fb0530f3f509b1d0234076

Nope, no quotes with quote=null or library=null, and no logs with message=null. Upgraded to latest code and still get the error.

Given that we already verified that it works with and empty sqlite DB, it'd be useful if you added some print statements in quotes.py:iter:90 to figure out which row is failing.


How to deploy mass action scheduler from sandbox to production..?

We made some actions using a mass action scheduler in the partial sandbox and the things worked properly using this, now want to deploy this from the partial sandbox to production.
How to achieve same please specify or give some reference

I'm using SFDMU https://help.sfdmu.com/ for that very purpose. I push MAS configurations into source as well. @surajzimo3917


[0.9.3] cannot use tag in GROUP BY if there is also a field of the same name

When execute the query there is tagKey in GROUP BY clause, error occurs below.
ERR: can not use field in GROUP BY clause: tagKey
The following is the output of command line.
# /opt/influxdb/influx
Connected to http://localhost:8086 version 0.9.3-rc1
InfluxDB shell 0.9.3-rc1
> use xdata_db
Using database xdata_db
> show tag keys from xdata_influxdb_action_001248
name: xdata_influxdb_action_001248
----------------------------------
tagKey
action
session_id

> select count(url) from xdata_influxdb_action_001248 where time > now() - 120s;
name: xdata_influxdb_action_001248
----------------------------------
time				count
2015-08-21T04:17:23.394514985Z	383

> select count(url) from xdata_influxdb_action_001248 where time > now() - 120s group by action;
ERR: can not use field in GROUP BY clause: action

I hope this is fixed in 0.9.3 GA.
Thanks

@sudabon can you also give the output of show field keys from xdata_influxdb_action_001248? I want to make sure there's not a field key also named action.
@corylanou @pauldix To me this is almost certainly related to the GROUP BY * PR, any insights?

@beckettsean I'm fairly certain there has to be a field named action.  However, we should handle this regardless and default to the tag for group by, the same way we default to the field for a select statement.  For sure a bug regardless.

@beckettsean I attach the output of required query below.
> show field keys from xdata_influxdb_action_001248
name: xdata_influxdb_action_001248
----------------------------------
fieldKey
access_date
action
browser
customer_id
device_maker
device_name
device_type
domain_group
domain_name
engine_name
ip
keyword
landing_type
mode
os
query_aq
query_attributes_id
query_b
query_brand_id
query_ctg_name
query_ctgid
query_ei
query_event
query_fr
query_from_link
query_id
query_ktr_ev
query_ktr_evct
query_label
query_order_cd
query_p_img_url
query_p_name
query_p_num
query_p_price
query_pid
query_price
query_pstart
query_sale_id
query_top_category
query_utm_campaign
query_utm_content
query_utm_medium
query_utm_source
query_value
referrer_domain
repeat_flag
session_id
site_id
unique_id
url

I add 'action' in not only tag key but field key in order to be flexible for future requirement from application.
For example, select count(action) from ... group by OTHER_TAG_KEY.
If I remove 'action' from field key, does it work well?
I will try it out.

@beckettsean When I remove 'action' from field key, it works well like below.
As @corylanou says I hope that it works well even if there is also 'action' in field key
> select count(url) from xdata_influxdb_action_001248 where time > now() - 600s group by action;
name: xdata_influxdb_action_001248
tags: action=cartin
time				count
----				-----
2015-08-22T07:08:09.668491729Z	37

name: xdata_influxdb_action_001248
tags: action=cartout
time				count
----				-----
2015-08-22T07:08:09.668491729Z	27

name: xdata_influxdb_action_001248
tags: action=login
time				count
----				-----
2015-08-22T07:08:09.668491729Z	266

name: xdata_influxdb_action_001248
tags: action=member_regist
time				count
----				-----
2015-08-22T07:08:09.668491729Z	7

name: xdata_influxdb_action_001248
tags: action=order
time				count
----				-----
2015-08-22T07:08:09.668491729Z	8

name: xdata_influxdb_action_001248
tags: action=view
time				count
----				-----
2015-08-22T07:08:09.668491729Z	1459

name: xdata_influxdb_action_001248
tags: action=waiting_list
time				count
----				-----
2015-08-22T07:08:09.668491729Z	2

same problem here.
getting "ERR: can not use field in GROUP BY clause: shopId"
and yes there is a shopId field and a shopId tag. was working fine in 0.9.2, stopped after upgrading to 0.9.3 ("stable")

Hopefully we can fix the parser to check for the existence of the tag before the field, so that the error never fires.

glad to hear that.

@corylanou short term, just assume they are grouping by tag if there's an identifier that matches both.

After looking at this in more detail, and talking to several of the other core team members, supporting a tag and a field with the same name is outside of the current design architecture.  It's equivalent in other databases as having 2 columns with the same name.  I would welcome a compelling use case, but I suspect there isn't one.
The reason you CAN do it is because when we write to shards, we only know about that shards meta data.  Therefore enforcing that you can't have a duplicate tag/field name is very difficult.  We have several safeguards in the code that prevent issues when querying back, but it's still an undesired outcome.
The real problem is with select * from measurement group by *.  Trying to make that work properly with a field/tag with the same name is very difficult.

@corylanou I think their ultimate goal here is they want to have a field and be able to group by that field. Where if they have a numeric value and they write it as both a field and a tag they can group by it and then they can also do math against it.
Although I agree, it's an antipattern to have a field and a tag with the same name. It's not really something people should be doing.

Agreed. I think a work around for now would be to have a field called action and a tag calledtagaction or something like that, and then log the same value to them.  Now, if they are indeed different data, then the same column name then we certainly would recommend against that design pattern.


CSS Styling Issue?

I realize this theme is incomplete, but something as big as http://i.imgur.com/59mYk29.png should be fixed soon?

http://i.imgur.com/2vAMRVa.png

Some pages are not working like this, because I have not applied the style to these pages yet. So that why you should not use in production servers. I will finish these pages later.
I am sorry that the progress is slow now, because I am quite busy these days.

No problem.  I am not using this in production, just seeing my options.  I just think something like this is quite major and should be addressed soon.

Hi,
Have also noticed this issue.
Would be nice to see it fixed since it is the first page a user see's :-)
Best theme for VestaCP!
Best Regards,

Hi,
If you can't finish this I'll gladly fork it and fully release it.
Cheers

You are welcome, sorry I am really too busy :(

@Jarrrk
That would be fantastic. I am able and have time to help, if you like.

+1
Amazing this theme


Merge Editor: consider "Complete Merge" for button text

Testing #161833
This was feedback I heard from others too—"Accept Merge" is a bit strange since you aren't really accepting a merge so much as the individual changes. I think "Complete Merge" would make more sense here.

Or maybe "Confirm Merge".

More ideas:

Stage
Complete Merge - I like this
Confirm Merge - I think "confirm" is usually used to approve an already stated intention and does not fully fit here.

"Complete Merge" and "Accept Merge" are the best options. I am leaning towards "Complete Merge".
fyi @bpasero

👍 for "Complete Merge"


Optionally Export pdf without revision and author information

Describe the feature you'd like
When exporting a page as a pdf have a modal that allows the user to exclude revision and author information from the pdf.
Describe the benefits this feature would bring to BookStack users
When pages are used as either templates or presentation collateral to be used by a customer we don't want the customer to see the author and revision information. Because it's a pdf it's difficult to edit afterwards.
Additional context
This somewhat relates to #992 however I believe it's significantly less ambitious to implement.  By executing this as a modal after the export pdf is clicked further export options could be added later.

This is super needed. I was going to export a bunch of user documentation from our wiki to use for client training. but i do not want these links in it. The user will have no access to the wiki. There is no public/guest visbile data.

This is something I would also like to see, but would maybe require an extra screen with options before exporting (whereas currently the export button goes straight to a download). Adding extra "without footer" variants to the export menu would look cluttered in my opinion, but is another option.
The way I work around this is exporting a page as HTML, removing the footer information in a text editor then using my browser's print to PDF feature, or an online converter to save the result as a PDF.

I need this aswell. Don't want to have those Data in Footer.
Maybe some Options in Settings to disable Footer?
And or make it customizable via Overrides?

We would like to use BookStack for ISO documentation but this missing feature is one of of the serious obstacles. It's a pity since BookStack is really very close and almost ideal for this purpose. Just the page numbering and better footer handling are missing.

It's possible to do this via the theme system.
Quick Guide
Add APP_THEME=custom to your BookStack .env file.
Within your BookStack installation, create the file at the path themes/custom/partials/entity-export-meta.blade.php. You should already have a themes directory but you'll need to create the custom and partials directories The file you create can be blank.
You should no longer have the footer content in exports.
Note, while the view system itself is stable, the files you override using it are not so you'd want to check your overrides are still working after any updates.

I achieve this by hiding the version and author with some basic CSS in the settings screen under custom header. This seems to suppress it nicely for PDF and for encapsulated HTML exports.
<style>
#page-show .text-muted {
  visibility: hidden;
}
.contents .text-muted
{
  visibility: display;
}
</style>

I achieve this by hiding the version and author with some basic CSS in the settings screen under custom header. This seems to suppress it nicely for PDF and for encapsulated HTML exports.
<style>
#page-show .text-muted {
  visibility: hidden;
}
.contents .text-muted
{
  visibility: display;
}
</style>

Hello everybody,
as I have not found anything new about this feature, I tried to set this up according to Dan's comment from December 2020. It is working correctly, there is just a different file, you need to override.
If you want to remove the version revisions in the footer, just create a blank file at the path themes/custom/exports/parts/meta.blade.php.
Also, any content in this file will be displayed in the page's footer instead of the revision information.
I Hope, that this helps someone.

Since this can be achieved via the visual theme system, as explained above, I'm going to therefore go ahead and close this off.

I fixed this for myself by adding a display: none style to the settings/customization/Custom HTML Head Content block of my bookstack web UI. Inside the "HTML Head Content" block, just add:

I realize this is an older issue but since I just found it, some of you may also be interested in this:
I used @ConfusedHelpticon's recommendation for .entity-meta {display: none;} -- but I still wanted to show the revision on the page, just not on the PDF.
I adapted it to be surrounded by @media print { } so that it would only hide it when trying to export.
Full code:


Run poetry lock --check in CI

Closes #181.
Commit 9620378cb6c3808420864d4e01a61b864b34e159 shows the expected failure behaviour if pyproject.toml is updated. Snippet of CI output CI:
Run poetry lock --check
Error: poetry.lock is not consistent with pyproject.toml. Run `poetry lock [--no-update]` to fix it.
Error: Process completed with exit code 1.

Commit 9d8dffe0b8b08f92dbe5585cf76201afeb0ad7e8 reverts the change to show the happy path. CI didn't run for this commit (presumably because it didn't detect any code changes?) so I added a dummy change in 82a3fdf to invoke CI, and reverted it in the following commit.
Note: it seems like the pip lock --check does not check transitive dependencies, so this improvement would not have prevented #177.

Hi @alan-cooney, do you still want this? If not, no worries, I will close!

Seems good to merge to me

Looks good, merged.


Service is getting created with special chars in argo

While trying to upgrade coredns chart 1.16.3 to 1.16.7, core-dns 1.8.4 to 1.8.7 observed the issue.

Tried from 1.16.5 to 1.18.0 charts having the same issue.
I need help to upgrade cordens to 1.8.7.

Can you please provide the argo application configuration you're using?

Can you please provide the argo application configuration you're using?

issue got solved somehow. Thanks


fix: repository governance

This PR was automatically created by the AVM grept governance tool

Supersceeded by #47


Cannot receive message from a sub dead letter queue

Using below listener can process messages of the queue successfully:
@JmsListener(id ="jms", destination = "jms", containerFactory = "jmsListenerContainerFactory")
public void receiveMessage(User user) {
    LOGGER.info("Received message from queue: {}", user.getName());
}

I can think of below two ways to consume messages from the dead letter queue, but neither of them works:

Use @JmsListener
 @JmsListener(id ="jmsDeadLetter", destination = "jms/$deadletterqueue", containerFactory = "jmsListenerContainerFactory")
 public void receiveDeadLetterMessage(User user) {
     LOGGER.info("Received dead letter message: {}", user.getName());
 }

Use JmsTemplate
jmsTemplate.receive(new JmsQueue("jms/$deadletterqueue"));

Could you help provide commended ways to consume the dead letter queue messages (DL of queue and topic)?
Spring: Spring Boot 3.3.3 + com.azure.spring:spring-cloud-azure-starter-servicebus-jms:5.18.0
Lib: com.azure:azure-servicebus-jms:2.0.0

Is "jms" a queue or a subscription? Is the namespace a premium servicebus namespace or standard?

This was happening because the messages were actually in deferred not deadlettered. This can be confirmed by peeking the queue from the ServiceBus portal and the state of the messages will show up as deferred. If the messages are actually deadlettered just using /$deadletterqueue should work.


add GetCellRichText method and test

GetCellRichText

Description
You can get text and font of cell by use GetCellRichText

        f := NewFile()

	runsSource := []RichTextRun{
		{
			Text: "a\n",
		},
		{
			Text: "b",
			Font: &Font{
				Underline: "single",
				Color:     "ff0000",
				Bold:      true,
				Italic:    true,
				Family:    "宋体",
				Size:      100,
				Strike:    true,
			},
		},
	}
	f.SetCellRichText("Sheet1", "A1", runsSource)

	runs, err := f.GetCellRichText("Sheet1", "A1")

Related Issue

https://github.com/360EntSecGroup-Skylar/excelize/issues/696
Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #789 (3302e19) into master (2833395) will decrease coverage by 0.01%.
The diff coverage is 93.10%.

@@            Coverage Diff             @@
##           master     #789      +/-   ##
==========================================
- Coverage   97.02%   97.01%   -0.02%
==========================================
  Files          31       31
  Lines        9387     9416      +29
==========================================
+ Hits         9108     9135      +27
- Misses        157      158       +1
- Partials      122      123       +1

Impacted Files
Coverage Δ

cell.go
97.08% <93.10%> (-0.31%)
:arrow_down:

Continue to review full report at Codecov.

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

Hi @tonny-zhang, I have fixed it base on your codes and I'll merge this PR after the CI check pass.

GetCellRichText.xlsx
	t.Run("init from file", func(t *testing.T) {
		f, _ := OpenFile(path.Join("test", "GetCellRichText.xlsx"))
		runs, e := f.GetCellRichText("Sheet1", "C1")
	})

this test case can't get error underline and get panic

GetCellRichText.xlsx
	t.Run("init from file", func(t *testing.T) {
		f, _ := OpenFile(path.Join("test", "GetCellRichText.xlsx"))
		runs, e := f.GetCellRichText("Sheet1", "C1")
	})

this test case can't get error underline and get panic

GetCellRichText.xlsx
	t.Run("init from file", func(t *testing.T) {
		f, _ := OpenFile(path.Join("test", "GetCellRichText.xlsx"))
		runs, e := f.GetCellRichText("Sheet1", "C1")
	})

this test case can't get right underline and get panic

GetCellRichText.xlsx
	t.Run("init from file", func(t *testing.T) {
		f, _ := OpenFile(path.Join("test", "GetCellRichText.xlsx"))
		runs, e := f.GetCellRichText("Sheet1", "C1")
	})

this test case can't get right underline and get panic

Thanks for your feedback, I have fixed on commit d840509

@xuri I don't think commit can get underline value for <u/>, should give a default value ?
if nil != v.RPr.U {
	if nil != v.RPr.U.Val {
		font.Underline = *v.RPr.U.Val
	} else {
		// use single as default
		font.Underline = "single"
	}
}

Yep, and I purpose to set the default value of underline as none:
font := Font{ Underline: "none"}

...

if nil != v.RPr.U {
    font.Underline = "single"
    if nil != v.RPr.U.Val {
        font.Underline = *v.RPr.U.Val
    }
}


Correct docker command for starting single mode Nominatim container with updated local.php for Nominatim update

Hi,
To use the update commands as per the readme of version 3.6 with updated local.php file as per the explanation in step 6 of the readme, the correct docker command to start the single node container with the local.php in the conf folder below the nominatimdata folder should be:
docker run --restart=always -p 6432:5432 -p 7070:8080 -d --name nominatim \
-v /home/me/nominatimdata/conf/local.php:/app/src/build/settings/local.php \
-v /home/me/nominatimdata/postgresdata:/var/lib/postgresql/12/main nominatim bash /app/start.sh
When starting the single mode Nominatim container with the above command, the following update command work as expected:
docker exec -it nominatim sudo -u postgres ./src/build/utils/update.php --init-updates
docker exec -it nominatim sudo -u postgres ./src/build/utils/update.php --import-osmosis-all
Hope this helps anyone........
NG

It did help at least one person, thanks!

It also helped me, but i do have a problem when running
docker exec -it nominatim sudo -u postgres ./src/build/utils/update.php --init-updates
with the erro
PHP Warning:  require_once(/app/src/build/settings/settings.php): failed to open stream: No such file or directory in /app/src/build/utils/update.php on line 3
PHP Fatal error:  require_once(): Failed opening required '/app/src/build/settings/settings.php' (include_path='.:/usr/share/php') in /app/src/build/utils/update.php on line 3
could anyone help me ? @NGeb1 @Stefanic

Looks like either the image itself or the volume mount is incorrect.
When I run:
docker exec nominatim ls -lha /app/src/build/settings
I get this output:
total 28K drwxr-xr-x 1 root root 4.0K Mar 18 11:01 . drwxr-xrwx 1 root root 4.0K Mar 18 08:26 .. -rw-r--r-- 1 root root  861 Mar 18 08:13 local.php -rw-r--r-- 1 root root  943 Mar 18 11:01 settings-frontend.php -rw-r--r-- 1 root root 5.3K Mar 18 08:26 settings.php
If you only see a local.php file there I think you made a mistake in the volume mount.
If that is not the case try rebuilding the image and recreate the container.

Hello @Stefanic , thanks so much for your feedback,
i cloned the repo locally instead on copying only the 3.6 files which i am using for my project, and the command seems to be executed fine !
I do now have to check that the postgis database is effectively updated.
Have a nice day !

We've also refactored how 3.6 and the REPLICATION_URL etc. works. You can set those now with environment variables. I am going to close the issue for now, feel free to reopen or create a new one if this still happens, thanks!


backward_G in half_gan_style.py

Hi, this is more of a question, than an issue.
In the definition backward_G, we take gram_matrix of the real_B, but not of the targets. So to calculate loss we compare gram_matrix of source to simple (not gram_matrix) of targets. Any reason for that?

Hi, please refer to the definition of  GramMSELoss defined in vgg.py.
class GramMSELoss(nn.Module):     def forward(self, input, target):         out = nn.MSELoss()(GramMatrix()(input), target)         return (out)
We actually calculate the gram matrix of fake image here and then calculate the MSE loss.

Thank you for the clarification. I am going to close this now.


Unable to see saved connections in Restricted Mode

Issue Type: Bug
I am unable see saved sql server connections nor able to establish a new connection. Even recent connections are not available. All places just show "Loading..." text.
Azure Data Studio version: azuredatastudio 1.33.0 (6783aa6967e8c6d086b6f04e5fbf527659981389, 2021-10-26T17:28:15.782Z)
OS version: Windows_NT x64 10.0.19043
Restricted Mode: Yes

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10610U CPU @ 1.80GHz (8 x 2304)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.61GB (15.61GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

@choelterati @krishanth3 are you seeing any errors in the "Help->Toggle Developer Tools" Console?

I'm not running any extensions, but I am seeing errors in the dev tools console log. Attached is an image and a txt log of them all.

azure data studio console errors.txt

@choelterati thanks for the additional context.  This appears to be the same issue as https://github.com/microsoft/azuredatastudio/issues/17530.  You can try the workaround I suggested on that issue which may clear out some invalid state triggering this bug.  We're continuing to investigate and get an internal repro.  The callstack seems to be complaining about https://github.com/microsoft/azuredatastudio/blob/main/src/sql/workbench/services/connection/browser/connectionDialogService.ts#L308 but not clear why capabilities would be null there.
Please let us know if you can think of any other customization or configuration that may be helpful in determining what's going on.

@kburtram I wonder if this is something to do with the Workspace Trust? The behavior shown in the screenshot (everything showing as loading) is what happens when you don't "trust" a workspace (since it doesn't load any extensions when that happens...)

For people in the thread - do you see the "Restricted Mode is intended..." message at the top of the window?

I see in the original bug text that ADS is running in restricted mode, which would explain this behavior.  I've updated the title to reflect this.  We'll investigate how to improve this experience in Dec.

This issue should be resolved in today's insider build.  Please let us know if you still experience this problem with those builds.  We're discussing potentially pushing this change as a hotfix due to the confusion/inconvenience caused by this bug.

Please deploy the fix. I am having the same issue after upgrading to version 1.33.0. I don't see that I am in the restricted mode, however.

@goudstad Could you try out the latest insiders build and see if you're still seeing the issue there?

I just upgraded to version 1.33.1.  and I am no longer seeing the issue.  Many thanks for fixing it.

Closing this since the hotfix with the fix has been released, if anyone else is still encountering issues let us know!


update 0.0.2

New Main Functions
Area of Interest Filter
Iterating over AOI to avoid RAM issues

It looks like there is some merge conflicts with the documentation, we will resolve those conflicts so that we are able to merge with the master branch
cc  @FrankBornais


Add a basic checkpoint class for saving and loading data

The most basic requirements of the checkpointing system are implemented and tested. The current version of the checkpointing system has the following features:

Saves and retrieves data in the _data dictionary

save method: Saves the data in the _data dictionary as a pickle file to the save_path specified by the user.
remove method: Deletes the components in the save_path directory.

The acceptance test performs the following steps:

Initializes a checkpoint instance with the saving path.
Creates a callable FakeAgent().
Assigns the callable as a value to the dictionary of the checkpoint instance.
Makes some changes to the agent.
Saves and deletes the checkpoint.
Creates a new instance of the checkpoint and loads the agent.
Ensures the loaded agent and the original agent have different memory addresses but have saved the same agent.
Removes the checkpoint by deleting the save_path and its contents.

Looks like I messed up the CI build, so it didn't trigger for this PR. Fixed in PR #2.
Would you mind rebasing off of main:
git rebase main
git push -f

Ah excellent, the CI build ran. Looks like it found a couple of errors (can see those by clicking the details button next to the build status):

In this case, looks like the linter is complaining about whitespace issues. To run the linter locally, you can do:
pip install ruff
ruff check recommence/ tests/

I also recommend changing a couple of vscode settings, which will help with the whitespace errors. Personally I recommend setting these in your user settings (i.e. globally for all projects).

Won't have anymore feedback for this PR, so after addressing the one comment for tests feel free to do a rebase to clean up the commit history. Recommendation:

fixup style changes into the feat: ... commit
fixup the minor test comment into the test: ... commit
Use rebase to drop the merge commit in the history
Use git rebase main to get the latest changes from the main branch
git push -f to update the PR when all is done

Won't have anymore feedback for this PR, so after addressing the one comment for tests feel free to do a rebase to clean up the commit history. Recommendation:

fixup style changes into the feat: ... commit
fixup the minor test comment into the test: ... commit
Use rebase to drop the merge commit in the history
Use git rebase main to get the latest changes from the main branch
git push -f to update the PR when all is done

I hope I did the fixups correctly


Better short texts for Soundboard

I'm using the soundboard for some sessions now. I'm using different sound for similar events, like different animal noises. Using Icons is kinda difficult for this one. But using the short text on the soundboard is difficult too, because there are only three letters showing (four, when hovering). It would be very useful to have the option to size down the text and use more than one text line.

If you use the new soundboard, you can configure the size of the box (1 to 3)

See announcement : https://www.patreon.com/posts/soundboard-for-87628048


(PUP-5908) Add MsgPack serialization

This commit adds a serialization reader/writer pair that is based on
MsgPack.

This changes the Gem requirements (msgpack becomes required). The ramifications of that needs to be ironed out before this PR is merged.

CLA signed by all contributors.

second commit message is not in imperative form

@thallgren @hlindberg do we still need this now that rich data is a thing? Can this PR be closed, or should we keep it to be reworked?

IMO, this should be reworked - Pcore over MsgPack is an improvement to both "human readable JSON" and "human readable JSON in MsgPack format" - (both actually being rich data).
The improvements come from Pcore serialization being able to tabulate data and thereby reducing size on the wire as well as memory needed at deserialized end.

In that case, should we close this PR? Or would it be best to leave it open?

I would like this PR to be left open.

Great! Thank you for the update @hlindberg and @thallgren !!

The improvements come from Pcore serialization being able to tabulate data and thereby reducing size on the wire

By default the puppetserver catalog endpoint applys gzip compression (true for most/all of the endpoints), where compression is handled in native code. Have you benchmarked "human readable JSON over msgpack and then gzip compressed" compared to "pcore tabulation over msgpack and then gzip compressed", where pcore tabulation occurs in ruby, and I'd assume is much slower?

as well as memory needed at deserialized end.

Do you have any numbers for this?
Aside, several conflicts need to be resolved.

@thallgren I noticed this hasn't seen any activity in a long time, do you think you'll be able to work on it in the near future?

Since this hasn't seen any activity in over a year, I'm going to go ahead and close this pull request for the time being. Please re-open this pull request once the next actions are addressed (conflicts and gem dependencies issue are resolved).


Remove unnecessary static variables

Since the autoloader is singeleton there is no reason to keep the static variables

Haven’t tested this yet but this is a good update 👍

Ditto 👍

@austinpray, @swalkinshaw Cheers guys!
I was also thinking, shall I update all the variable names to follow PSR2? Just to be consistent with the code.

Yeah good catch. Might as well 😄

@swalkinshaw this is done, could you please have a look?

Thank you 👏


RTMP Relay Push Not Accepting Commands

I am trying to resize the RTMP push to resize to a smaller video. I have the following config:
relay: {
    ffmpeg: '/usr/bin/ffmpeg',
    tasks: [
      {
        app: 'live',
        mode: 'push',
        vc: "libx264",
        vcParam: ['-filter_complex','"scale=\'if(gt(iw,1920),1920,-1)\':\'if(gt(ih,1080),1080,-1)\':force_original_aspect_ratio=decrease"', '-crf', 20],
        edge: 'rtmp://www.example.com',
      }
    ]
  }

Except in my output, I am getting this:
/usr/bin/ffmpeg -i rtmp://<IP_ADDRESS>:1935/live/123456 -c copy -f flv rtmp://www.examplee.com/live/123456
Its using the copy and not passed commands. Any ideas on why its ignoring the config?

Any solution?

@mohsen-rasouli Nothing


Improvements to llm/deepinfra

replace requests package with langchain.requests
add _acall support
freshen up the documentation a bit
update vendor doc

@baskaryan Could you take a look? A few customers are really asking for async api to DeepInfra.

@hwchase17 Any chance we can merge this? Should be low risk, and lots of our users are asking for this.

still seeing some lint issues, happy to take over and land if you'd like

@baskaryan after the master was fixed it's golden ;-)

@hwchase17 @baskaryan can we get this merged?

@hwchase17 @baskaryan can we get a review, please?

apologies for delay, lgtm! thanks @ichernev


Should not attempt to convert values for alias attributes that have read only access

get_model_attribute_name_and_value should not attempt to convert wlst values for attributes that are marked with access of RO. the get_wlst_attribute_name_and_value already disregards these types of conversions.

I fixed this while doing issue #96


dev: benchmark cost of calling Cairo Contracts

The cost of calling Cairo contracts is composed of:

A cost per input value that must be converted from EVM bytes to Cairo felt252
A cost per output value that must be converted from Cairo felt252 to EVM bytes
Eventual cost of converting EVM bytes to a human-readable data structure
The cost of execution of the contract itself (could be negligible).

Benchmark these costs, try to highlight a correlation betwen cost and input/output data and plot the cost per byte/felt

@enitrat Can I work on this


Resume First Draft

I didn't include skills—wasn't sure if they're appropriate here. Should I add them in?

Great Job! Please upload a screenshot of this PR to Google Classroom!!


Unable to find Flutter logs on NewRelic dashboard

I am unable to find logs in NewRelic dashboard for my Flutter mobile app (Android).
Logcat reports that report is sent to NewRelic:
newrelic: {level=ERROR,message=[DevActionsView] Demo fatal error message}

However I don't find the logs in the dashboard, not when querying like this (as read in your forum):
SELECT * FROM `Mobile Dart Console Events` SINCE 1 day AGO

and also not in the "Logs" tab:

I tried to enable Logs in the "Application Settings" but that didn't help either

What am I doing wrong, am I looking at the wrong "table"?

Hi @ndesai-newrelic, I guess you mean the HTTP proxy tool Charles, right?
I did a session with Proxyman and an iOS simulator, on my Android I didn't succeed with that...

Errors are reported correctly (NewrelicMobile.recordError) - I can see them in the Dashboard. I suspect these are the POSTs to https://mobile-collector.eu01.nr-data.net/mobile/f
However for logs (NewrelicMobile.log) I don't see anything in the Dashboard, and also not in Proxyman. (I suspect the data POSTs are for analytics only?)

New relic configuration in Flutter:
Config newRelicConfiguration(String appToken) => Config(
      accessToken: appToken,
      //(Android only) Enable or disable collection of event data.
      analyticsEventEnabled: false,
      // https://docs.newrelic.com/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/configure-settings/#flutter
      backgroundReportingEnabled: false,
      // Enable or disable crash reporting.
      crashReportingEnabled: true,
      distributedTracingEnabled: false,
      httpInstrumentationEnabled: false,
      httpResponseBodyCaptureEnabled: false,
      interactionTracingEnabled: false,
      loggingEnabled: true,
      logLevel: LogLevel.INFO,
      networkErrorRequestEnabled: false,
      networkRequestEnabled: false,
      // (iOS Specific) Optional: Enable or disable to use our new, more stable, event system for iOS agent.
      newEventSystemEnabled: true,
      // Enable or disable offline data storage when no internet connection is available.
      // To configure the amount of offline storage, see https://docs.newrelic.com/docs/mobile-monitoring/new-relic-mobile/mobile-sdk/set-max-offline-storage/#flutter.
      offlineStorageEnabled: true,
      printStatementAsEventsEnabled: false,
      webViewInstrumentation: false,
    );

@akaegi can you check you mobile logs settings one more time?

They look like this in both applications:

@akaegi we are unable to reproduce it from our side, if possible can you share sample app where you reproduce the issue?

Ok thank you @ndesai-newrelic ! I've seen the sessionId in the log entry. How can I look up the userId and other attributes tied to the sessionId (in the dashboard) (or attach it if that's not done automatically). Sorry, maybe you can just hint me at the relevant docs 😬


Add JSON selector DSL

Closes #39.

@ybasket sorry for the back and forth, now it is ready to be reviewed.


Feature Request: Spring Security integration with Amazon Cognito

Amazon Cognito:

Amazon Cognito is a service that makes it easy to save mobile user data, such as app preferences or game state, in the AWS Cloud without writing any backend code or managing any infrastructure. Cognito offers mobile identity management and data synchronization across devices. You can save data locally on users’ devices allowing your applications to work even when the devices are offline. You can also synchronize data across a user’s devices so that their app experience will be consistent regardless of the device they use. With Amazon Cognito, you can focus on creating great app experiences instead of having to worry about building and managing a backend solution to handle user authentication, network state, storage, and sync.

As can be seen, Amazon focuses on not having any backend implementation (typically, your mobile application calls other Amazon services directly after using Cognito), but it would be beneficial to implement your own backend service and yet leveraging on Amazon Cognito for user authentication. Like Spring Social, Cognito offers authentication integration with Facebook, Twitter, OpenId and others, but Amazon Cognito Identity also offers anonymous authentication (that later can be promoted to a real authentication for the same user). In contrast to Spring Social, Cognito Identoty also enables Amazon Cognito Sync service that among other things synchronize the state of your mobile app with the AWS cloud.
In the blog post Use Amazon Cognito in your website for simple AWS authentication the authentication flow is outlined for some programming languages, including Java.

See also the AWSCognitoSampleDeveloperAuthenticationSample sample project that is provided by the Amazon Web Services - Labs GitHub account.

This sounds great. Sorry to bother you when is this expected to be completed

This might be helpful when implementing http://blog.kdgregory.com/2016/12/server-side-authentication-with-amazon.html

Another open source that might be helpful https://github.com/IxorTalk/ixortalk.aws.cognito.jwt.security.filter

With new ALB security support this feature would be really nice to have.
https://aws.amazon.com/blogs/aws/built-in-authentication-in-alb/

That AWS sample code is a beast... anything using JWT is over-complicated and unnecessary in a server environment (you are using Spring right?). Here's a more minimal implementation of AWS Cognito on server-side Spring Security: https://gist.github.com/alexramos1/329f8a927707c266e06bf41219f66570

i want to notice my new project for ResourceServerConfigurerAdapter compatible with cognito, just updated to spring boot 2.1.1
https://github.com/xanscale/CognitoResourceServer
i don't know if my code is good or not, actually works. any help are wellcome
@andrefaria @matsev

Hi, I wonder what is the blockers with this feature? any news or is there any plan for doing it sometime soon?

@Vostan did you try my implementation? See above.

@alexramos1 No I did not try it, but I will try it out soon. Thanks! but my idea is that That would be awesome if this would be a part of Spring-cloud-aws!

Hi everyone, I have created a draft for this feature. Still pending to add some tests and docs but after reading some docs that should work


tolino vison 2 root

hi there, the twrp is working perfectly on the vision 2
but id like to root it to install other apps on it.
do you have some ideas how to do it?
my system is 11.2.3
i tried some stuff form this forum but i got a bootloop:
https://allesebook.de/anleitung/anleitung-tolino-vision-4-hd-root-zugriff-94665/

To install other apps you do not need root, but to enable adb access. Then you can install apk files trough usb.
As for bootloop I think you need to do a factory reset.
This repo is the wrong place for such questions because not TWRP related. Please ask in https://forum.alles-ebook.de/ forum.
If you are Linux-User, the method https://www.e-reader-forum.de/t/tolino-vision-2-rooten.147429/ is still valid even the latest 15.2.0 version

@bell07 thank you.
@creisey anyways, you can just search for SuperSu.zip and flash it from TWRP if you really need root. But, as @bell07 you don't really need root to install apps.


Error when open an empty file

Version confirmation

[X] Confirm

Following prerequisites

[X] Confirm

Neovim version
NVIM v0.9.0
Operating system/version
Win10 22H2 19045.2728
Terminal name/version
nvim-qt
$TERM environment variable
No response
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
Open the file with nvim-qt.
Expected behavior
no error
Actual behavior

Additional information
When the file name contains "{", the error will occur.

what kind of devious ppl will put '{}' in their path name.

null-ls.nvim should be responsible for handling this, please open an issue in the upstream repo.

what kind of devious ppl will put '{}' in their filename.

I just kept my keepassxc password in the filename, and I tried to open the file with nvim-qt. That's why I found the error.

I just kept my keepassxc password in the filename, and I tried to open the file with nvim-qt. That's why I found the error.

never heard of keepassxc b4. looks interesting!

never heard of keepassxc b4. looks interesting!
That is a password management software that is open source and localized, worth a try.

close for it's about upstream


Make jsPsych usable as a module

In order to integrate better with complicated javascript projects, it would be great to allow jsPsych and it's plugins to be used via browserify or webpack.
As far as I see it there a few steps to this, but totally open to ideas

[ ] Convert things to work via a umd build script
[ ] update docs
[ ] figure out long term plan for plugin usage.

Right now plugins don't actually export anything, they simply attach themseleves to a global jsPsych object, long term it might be handy to make plugins actually export something, so we can test plugin code in isolation, without interacting with the jsPsych global itself.

Figuring out the pattern for plugin use seems essential.
One possible approach would look like this:
import htmlKeyboardResponse from "./plugins/jspsych-html-keyboard-response.js";

var trial = htmlKeyboardResponse.createTrial({
  stimulus: 'hello world',
  trial_duration: 1000
});

var timeline = [trial]

Another would preserve the current syntax for creating trials, but would presumably require users to attach the plugin to jsPsych, unless someone is aware of a design pattern that avoids this requirement?
import htmlKeyboardResponse from "./plugins/jspsych-html-keyboard-response.js"

jsPsych.plugins.attach(htmlKeyboardResponse);

var trial = {
  type: 'html-keyboard-response',
  stimulus: 'hello world',
  trial_duration: 1000
}

var timeline = [trial];

@jodeleeuw  I love the second one, the plugin would just need a top level name key.
function attach(plugin) {
  this.plugins[plugin.name] = plugin;
}

Great point. This also allows for:
var trial = {
  type: htmlKeyboardResponse.name,
  /* .. */.
}

I do prefer the first approach. It is fortunately quite straightforward.
The plugin pattern is powerful (many lib uses it, e.g. PouchDB), but in my opinion it comes with a cost: it is really just about hiding a global variable. It forces developer to track down what plugins have been added and introduces hidden side-effects across modules.
For example, a good practice would be to add all need plugins a particular module needs, but failing it to do so may have no consequences if the plugin was already added somewhere else in the code.
To solve this, when using pouchdb I usually create a local pouchdb modules that patches pouchdb with all plugins I'm using in the project. Then in the entire project I am only importing pouchdb from this local module. But still makes it complicated to maintain, a plugin that isn't used anymore is easily forgotten (and hence uselessly bundled with the app).

What about something like this?
import htmlKeyboardResponse from "./plugins/jspsych-html-keyboard-response.js"

var trial = {
  type: htmlKeyboardResponse
  stimulus: 'hello world',
  trial_duration: 1000
}

var timeline = [trial];

I like that approach a lot @QuentinRoy

How can I set the visual angle of images

Hello, my colleagues and I are just getting started with jsPsych (so far, very pleased!) and curious if there is a suggested repo or approach for integrating jsPsych within a React.js app? E.g., as an experiment page within a larger web app? We are aware of the unofficial / unsupported openexp/jspsych-react, but curious if there is an official, supported approach before we go digging into someone else's unsupported code circa 2019.

Hi @sjburwell,
We're a couple months away from releasing version 7.0 which will transition everything to importable modules.
In the meantime -- and I realize this feels like a circa 2000 suggestion -- iframes can work pretty well in my experience. The jsPsych page can exist independently of the React code, and React can be used to render the right iframe content.

Sounds good @jodeleeuw, we'll look into using IFrames as a possible solution, but happy to hear that v7.0 will be importable as modules!

This was completed with #2183


Added "Using with SQL Server Integration Services"

Added section for https://github.com/bimlscript/BETDPI

Thanks!


add msvc builds

need a powershell version of the dune-copasi setup-dune bash script for this

see https://github.com/spatial-model-editor/spatial-model-editor/issues/603


Update Cluster Autoscaler to v1.3

What this PR does / why we need it:

Updates cluster-autoscaler to v1.3 for k8s 1.11
Adds a link to cluster-autoscaler/README.md from cluster-definition.md

@feiskyer can you confirm if this release supports v1.11? should I update k8s v1.10 to CA 1.3 also? CA versioning seems confusing.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Fixes #3309
Special notes for your reviewer:
If applicable:

[ ] documentation
[ ] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

Looks like the unit test needs to be updated as well... 	k8s_versions_test.go:48: getK8sVersionComponents() returned an unexpected map[string]string value for k8s 1.11.0-alpha.1: cluster-autoscaler = cluster-autoscaler:v1.3

@feiskyer can you confirm if CA v1.3 supports v1.11? should I update k8s v1.10 to CA 1.3 also? CA versioning seems confusing.

CA v1.3.x is for Kubernetes v1.11.x, while CA v1.2.x is for Kubernetes v1.10.x.
So we should use  CA 1.3.0 only for Kubernetes v1.11.x and keep using CA 1.2.2 for Kubernetes v1.10.x.


Trying to test for large data sets and I get failures

I've upped the number of records added by the demo application.  When the file size hits about 4GB, I get a failure.  Anything I should try?

Did you have error message exception? There is no 4GB limit on database size.

I've repeated it a few times. I've set the sample customer program to add 3 million records. And I run it again. By the 3rd time, I receive a NotImplementedException during an insert.

On Aug 14, 2015, at 6:48 PM, Mauricio David<EMAIL_ADDRESS>wrote:
Did you have error message exception? There is no 4GB limit on database size.
—
Reply to this email directly or view it on GitHub.

I'm seeing great potential but I really have a need for larger data sets. It failed much earlier with the app set to 32 bit.

On Aug 14, 2015, at 6:48 PM, Mauricio David<EMAIL_ADDRESS>wrote:
Did you have error message exception? There is no 4GB limit on database size.
—
Reply to this email directly or view it on GitHub.

I can confirm it happens for me too.

Hi guys! Yes, I do this test and fails to me too. I don't know why yet, but I will investigate this weekend. I guess can be something in FileStream or like that.

Hi @wcmiller3 and @sherry-ummen, bug fund! I simple uint<->long convert error. I will create a new release for this

ok I tested this. One thing is strange which I am not sure whether its a bug or some OS feature.
so my program is simple
 foreach (var index in Enumerable.Range(1, 10000)) {
                    var emp = new Employee() {
                        EmployeeID = "owsddyu" + index,
                        FirstName = "one",
                        LastName = "two",
                        Bytearray = new byte[1 * 1000 * 1000]
                    };
                         col.Insert(emp);
                  }

So when its running you can check in the folder that the file size is increasing but after it reached the 4Gb then the updating of the file stops and when you stop the execution then you see the final size which is more that 4Gb depending on for how much time you keep it running.
My doubt is that why after 4Gb it stops showing the file size changing? And when the program is stopped then only it updates the size. Whats going on?
Thanks

Hi @sherry-ummen, do not use Insert method to bulk many data. Take a look on new unit test: https://github.com/mbdavid/LiteDB/blob/master/UnitTest/BigFileTest.cs
I'm using Windows 10 x64 and updates on file explorer are normal after 4gb. In this example, grows in ~500mb each update.

Hi, yes I checked the unit test and actually ran that also and there it worked. About using of Insert the thing which I am trying to check here is that it may happen that there may be random read and write but there may be huge number of writes so I really cannot determine when to use Insertbulk.
Could you give a better solution then? Thanks


Stop overriding KafkaParams specified by user

I could not find anywhere in the document about why we are overriding the kafkaParams that specified by the user, which makes the whole streaming behavior unpredictable, changing to filling defaults instead of being dictatorial about user parameters

What changes were proposed in this pull request?

Respects user`s input instead of straight overriding Kafka Parameters to hard-coded value
Why are the changes needed?

The streaming behavior is unpredictable/misleading, users think they set auto.reset.offset value but Spark override them invisibly ( hard to notice that logging message)
Does this PR introduce any user-facing change?

How was this patch tested?

thanks @dongjoon-hyun @viirya for looking into this, agreed the changes here is going to modify the documented behavior, let me put more thoughts on it, closing the PR for now

Thank you @dulu98k

Thank you for closing, @dulu98k . And, thank you for your guide, @viirya !


Synonym expansion : Query problem

Hi @simonhughes22 ,
First of all, thank you for your work !
I'm new to solr and I'd be glad to get some help with this problem :
I've already stored and indexed a collection of texts, which I can query without trouble ;
now I want to use  synonym expansion thanks to the QueryBoostingQParserPlugin but, while I can see that my synonym-Field is loaded (cf "Analysis" of the UI), my queries fail to match synonym terms.
I don't know if the problem is caused by a wrong use of query-parameters (first time I try to use "queryboost")or if there's something wrong with my solrconfig or schema ... Could you help me ?
I can see that my synonyms are correctly mapped to each term and have a payload, but then I must have missed something...
Thanks a lot,

field&solrconfig.txt

It's hard to say what's wrong from the the screenshot and the config alone. The screenshot is referring to a field type of text_en_syn, which is not present in the field&solrconfg.txt AFAICT. I do see text_en_payload in there, which looks like it is doing the same thing. I would do a few things. First I would use debugQuery=true to look at how the query is being processed by the request handler. Queries in solr can receive different processing at query time, depending on the query parser settings, before they get processed by the analyzer. The request handler will split the query on whitespace before it hits the analyzer unless you have a newer version of solr and have disabled that option. That can have other side effects though so be careful if you change that. Also, you need to make sure you are actually invoking the QueryBoostingQParserPlugin query parser by specifying the query parser to use when executing the query. Solr defaults to the lucene query parser, unless you specify edismax or some other parser. HTH

Thanks for your answer @simonhughes22 , I just went one step further thanks to debugQuery : now I can see that the query is parsed with the correct parser, and that synonyms are retrieved !
Next problem though : the parsedquery seems to be abnormally long, and leading to 0 results... I'd be glad if you could keep helping me !
This is the screenshot of the debugQuery ("synonymes" is my synony field, his fieldtype is "text_en_payload")

Thanks a lot for your time  !

I am not a solr 7 or 8 expert, I don't work with it as much these days. However, in terms of getting no results, I am guessing you are running the query as an AND query, where all terms are required. I'd just switching to an OR query, and using the mm parameter to enforce the number of terms you want to match. Can set to 1, it should still rank docs with more matches higher. Also more terms will impact performance, so you may want to either tinker with the mm settings or reduce the number of top terms queried, if performance becomes an issue.

Hi, here's the xml file of the results, query is correctly set to "use OR" but still the same output :(
results.txt

Hi @simonhughes22 ,
I still have issues with using "queryboost" properly, so I have some precise questions :

With my text field called "text" and my synonym field called "synonymes_payload", I'm unsuccessfully using the following query, does it seems correct to you ?

/select?df=text&q=achieve&qf=synonymes_payload&fl=id&wt=xml&hl=true&hl.snippets=300&hl.fl=text&hl.usePhraseHighlighter=true&defType=queryboost&q.op=OR&debugQuery=true&mm=1
I have the impression that a) It's possible that I don't interrogate fields properly and/or b) as you said, the parser may search for combinations of synonyms despite the "q.op=OR" parameter

Could you display for me the correct format of the files we are using (in your example, "jobs_titles.txt", "synonym_types.txt" and "top10_title_synonyms.txt") in order for me to be sure I didn't make some error at this level ?

Would you suggest I downgrade my solr version to the same one you were using, and if so which one is it ? (I'm using this for a pedagogical project, so I don't mind give this option a try if it's your advice).

Thanks a lot for your time and help !

@BLOIZO I would definitely start by downgrading it to the solr version I built it for. It was originally developed in 5.4 IIRC. The master branch is in 6.3 (check the pom.xml and look at the lucene library versions ). I don't currently work with Solr so I am not up to date on all the changes in 7 and 8, and some may be breaking to this functionality. At some point I'd like to add this to solr core, and issue a PR, but I've been too busy to take the time to do so. The built in MLT is not great.

Hi @simonhughes22
I promise it's the last time I'm bothering you with my issue, but I can't figure this out ...
Does my query in the precedent post seems correct to you ? Not sure if i'm using the correct parameters, and if interrogating two fields (text & synonym) the way I do while trying to retrieve content only from "text" makes any sense ...
I'd appreciate if you could take some time to explain to me how I'm suppose to query my fields in order to make synonym expansion happen,
tbh I don't know what to do next, today I gave a try to the 'bq' parameters,unsuccessfully and, being a beginner, I'm running out of ideas ...
Thanks a lot


Fix #2 to explicitly disallow repeated keys

It required implementing error handling in the event handlers, which seemed safe enough.
There are a few other points I think where errors should be raised but aren't, but mostly they fail in a safe way unlike #2 .

Closes #2 .

There is still a minor problem with the mark being misreported for any non-trivial cases. This makes sense since the hash-map insertion happens once the whole node is populated, which moves the mark on accordingly.

Is this project no longer maintained here?

Nudge again. Happy to help with maintenance...

Sorry about that. 2 years is quite a long delay... I do not remember having been notified of the issues and PRs being created on this repo. I thought no one used it, being happy with yaml-rust. I'll make sure to better maintain this from now on.


Question: which secrets need to be created to deploy?

I followed the instructions from here https://k8guard.github.io/deploy/minikube/.
However after running make build-deploy-minikube and waiting I get:
$ kubectl logs k8guard-report-deployment-483960501-23d4t
Error from server (BadRequest): container "k8guard-report" in pod "k8guard-report-deployment-483960501-23d4t" is waiting to start: secrets "k8guard-report-secrets" not found

The action and discover pods are running fine.

Hi, report needs password for cassandra,  here is an example https://github.com/k8guard/k8guard-start-from-here/blob/master/minikube/report/k8guard-report-secrets.yaml.EXAMPLE
you can simply rename the k8guard-report-secrets.yaml.EXAMPLE to k8guard-report-secrets.yaml . does that answer your question ?

Well, that helped get further but cassandra isn't accepting the connection:
time="2017-08-14T21:35:18Z" level=info msg="Connecting to db"
time="2017-08-14T21:35:18Z" level=debug msg="Connecting with usernamecassandra"
panic: gocql: unable to create session: unable to discover protocol version: dial tcp <IP_ADDRESS>:9042: getsockopt: connection refused

goroutine 1 [running]:
main.main()
        /Users/raphael.deem/Documents/code/go/src/github.com/k8guard/k8guard-report/main.go:15 +0xbc

hm... I am thinking the cassandra svc is not up yet, sometimes.
can you re-apply all of them from scratch ? and usually wait couple minutes for the cassandra to be up, (based on your CPU/RAM) cassandra container can take time to be up and running and create the db and users.

That did it, thank you

Sweet ! glad that helped . @r0fls we used to have a smaller cassandra image but that one did not support authentication, if you find a better cassandra docker image, please feel free to make a PR. the current cassandra docker image is a bit heavy ! :)


Text Screens unremovable

like the title says, text screens are almost completely unremovable after some time. The only way to remove them is via making them permanent and making the unpermanent shortly thereafter.

I take that back... it was just some new textscreens.

Can you describe precisely how to reproduce this please? What happens when you attempt to remove them with the remover?

I wish I could reproduce it. It's just that they stay there until a server restart. Trying to use the remover tool or the context remove is nil, nothing happens. No toolgun shot or anything. I can't move them with a physgun either.

That's... very odd. I'll see if I can reproduce it with some tests soon. Thanks for the report.

Thanks a ton man, I really appreciate it.

Permissions are checked in hooks. First hook to return something other than nil wins. Bad addons returns false or true for everything. This sounds like a race-condition to me.

@therealbenny What Dellkan said. I may implement something in future to "fix" this terrible hook design in garrysmod, but this is most definitely another addon causing conflict and returning something in a hook that it's not meant to.

that sounds about right, however I only have FPP on the server for any sort of protection. That was never an issue before. I used 3d2d textscreens on my old server while using FPP and never had an issue. Once again thanks for your time.

FPP isn't the problem. Check your other other scripts. Whatever interfering addon doesn't have to have anything to do with protection, props, or permissions at all.
Try the following in rcon to get an idea of what's going on. (Each will output a list of hooks that might interfere. They're usually named in a manner that'd allow you to identify which addon they belong to)
For serverside, run these two
lua_run for k, v in pairs(hooks.GetTable()["CanTool"]) do print(k, v) end

lua_run for k, v in pairs(hooks.GetTable()["PhysgunPickup"]) do print(k, v) end

For clientside, the same, only replace lua_run with lua_run_cl (note, only works if server allows sv_allowcslua).
lua_run_cl for k, v in pairs(hooks.GetTable()["CanTool"]) do print(k, v) end

lua_run_cl for k, v in pairs(hooks.GetTable()["PhysgunPickup"]) do print(k, v) end

18:29:55 > for k, v in pairs(hook.GetTable()["PhysgunPickup"]) do print(k, v) end...
AutoWantedfunction: 0xe9aee6d8
apacPhysgunPickupfunction: 0xe9b88700
EntityPhysgunPickupfunction: 0xe996fb60
textscreenpreventtravelpickupfunction: 0xe629f5b0
ulxPlayerPickupJailCheckfunction: 0xe9b71248
FPP.Protect.PhysgunPickupfunction: 0xe9a59bd8
ULibEntPhysCheckfunction: 0xe9d3b128
ulxPlayerPickupfunction: 0xe9b33830
FAdmin_PickUpPlayersfunction: 0xe998a660
18:30:10 > for k, v in pairs(hook.GetTable()["CanTool"]) do print(k, v) end...
DoorExploitfunction: 0xe9922ad0
FPP.Protect.CanToolfunction: 0xe9a71d98
ULibEntToolCheckfunction: 0xe9bb8470
armory.removefunction: 0xe9bb3a20
FAdmin_Logfunction: 0xe9975000
ULibPlayerToolCheckfunction: 0xe9d0bf50
textscreenpreventtoolsfunction: 0xe628db78
FAdmin_jailedfunction: 0xe99947a8
EntityCanToolfunction: 0xe996bd48

textscreenpreventtools is the hook name for this addon, so that's perfectly normal.
If you remove ULX/Ulib, does the issue continue? There's quite a few people reporting issues when using this addon with ULX/Ulib.

Well when I had it on my old server there was never an issue. I don't think that's it.

It's absolutely one of the addon's functions as previously referenced.
ULX/Ulib, DarkRP, FPP, FAdmin, Apac, or various other places those hooks come from are causing your issues.

I could definitely see it being FPP/Darkrp/etc because I haven't used them in over a year, they've both undergone huuge updates since I had my last server. Unlike ULX, which pretty much only gets updated when it breaks.
I'm not sure where to go from here, regarding DarkRP.

Unfortunately there's very little else I can do here really, sorry. We ran this addon without issue on DarkRP for quite some time, but there's definitely a conflict occurring somewhere in your setup that you'll need to track down.
In a vanilla environment, this addon works without issue. I may implement something in future to "fix" this terrible hook design in garrysmod, but this is most definitely another addon causing conflict and returning something in a hook that it's not meant to.

No need to apologize, I was just making sure it wasn't something on your end of the spectrum. It's not a huge issue at the moment but if it becomes one I'll have to seek alternatives to the tool (if there is any).
If you ever implement the hook thing you're talking about just reply on this post, i'll be sure to look at it.

Certainly will do, thanks for your understanding.
I'm going to close this issue for now, but I'll be sure to open it again if I implement such a "fix" in future.


Use OData API to demonstrate real-life scenario

Since this project is relatively new, please consider using https://services.odata.org/V4/TripPinServiceRW as a data source which is ODATA based.
Once you hit that service, it will generate a read-write endpoint as https://services.odata.org/V4/<InstanceId>/TripPinServiceRW/ where InstanceId is something like (S(pfvo2qaqwuyjvcsqafohkyyg))
It would give this project more attention since no other grid/table samples use ODATA as a source.
ODATA will be revamped by Microsoft and community pretty soon via a project called RESTier

I think a generic mechanism that allows the setting of a 'datasource' instead of the Items parameter would be a good idea - that could be an OData provider, a REST API or perhaps just an IQueryable. It would need to (optionally?) support count, paging, sorting, and filtering so a data source can be accessed in an efficient way.
A datasource could be a provider model that implements a common interface that the Table can query to get the data, e.g. EntityFrameworkProvider, ODataProvider etc.
Thoughts?

@conficient, that's exactly what I was thinking.
The Table currently converts the Items parameter to a IQueryable, so exposing that interface would be  the simplest way forward. This would allow the existing Filtering/Paging components to work without any modifications. It should also probably work with EF as is, I haven't tried though.
If there are no objections, I'll proceed with the IQueryable route.

I performed a quick PoC with Microsoft.OData.Client in the following branch, https://github.com/IvanJosipovic/BlazorTable/tree/odata
ServerSide Blazor can load data and sorting appears to work.
Wasm Blazor, will need more work.
Example loading data from, https://www.odata.org/odata-services/

Blazor Wasm error:

Nevertheless, this is very promising!


Add routing support to emulators

Add routing support into GR4JCN emulator.

Hi @julemai
It looks like the GR4JCN emulator now can be run with routing!
I've followed your instructions to modify the model templates, and created a dummy test exercising two subbasins with routing from one to the other. See tests/test_emulators.py::TestGR4JCN::test_routing
Could you check that

The changes in the templates are OK
The test makes correct assumptions and indeed tests the routing

Note that I've hardcoded in the test the output directory (model = GR4JCN("/tmp/test_gr4jcn_routing")) so when you've run the test you can easily go into the directory and check the config files.

Hi @huard
Yay!! That's fantastic. I will get to that first thing Monday morning. Once it passes my review I might need to share it with James to get his feedback too.
Thanks so much for fixing that output directory. That will speed up my testing by a factor of 10!!

Updated above with a code snippet for Annual_Avg_Runoff ...

Fixing it for now is fine I think.

So for this current phase (until the end of the month) are we using a fixed value or do we try to compute it with the method you outlined?

@cjauvin I think avg_runoff is an easy fix since everything (Qobs and mean area) is already available as far as I know. But up to you and @huard. I'm good with either.
The more severe thing to look is it the RVT. We need a fake grid-weights looking like this (we are going to use the same forcings for both HRU; just because the inputs right now only contain one time series; this will change once we don't work with fake examples...):
:GridWeights
	:NumberHRUs 2
	:NumberGridCells 1
	1 0 1.0
	2 0 1.0
:EndGridWeights

And the block for reading the inputs will need to look like this (but for this we need to add a new axis (nstations) to the existing NetCDF:
:StationForcing rain
  :ForcingType     rain
  :FileNameNC      Salmon-River-Near-Prince-George_meteo_daily_2D.nc
  :VarNameNC       rain
  :DimNamesNC      nstations ntime
  :RainCorrection  1.0
  :SnowCorrection  1.0
  :tshift          0.0
  :LinearTransform 1.0 0.0
  :RedirectToFile  <grid-weights-from-above>
:EndStationForcing

I think we should use StationForcings for ALL examples. The only thing we need is to add this additional axis. The rest would be the same. It would make it more consistent and general. Makes sense?

So we need a variation on Salmon-River-Near-Prince-George_meteo_daily_2D.nc with a new axis? Could I ask you to create it please, and then I can add it to test-ravendata ?

Sure. I can do that. Some time id week. Before I will be busy with integrating the other two models.

Already exists:
testdata/raven-gr4j-cemaneige/Salmon-River-Near-Prince-George_meteo_daily_2d.nc

The axis is called there region and not nstations. So, you would need to adjust that in what I exemplarily had above.

These 2D NetCDF files and the needed grid-weights are btw exactly the ones that are the result of aggregate_forcings_to_hrus.py. So, it would be really convenient to just use those here. :)

@julemai and @huard My latest commit introduces (hopefully!) the missing pieces to make the TestGR4JCN.test_routing run (as advised by Julie) although I'm not sure it's all correct, please verify (in /tmp/test_GR4JCN_simple after running the test).

That’s awesome! I will have a look tomorrow.

@julemai Yes absolutely, that's on my todo list but yesterday night I had to stop just before it. So the current status of this PR is:

[ ] Set HRU states to 0
[ ] Have 1 BasinIndex per SB
[ ] Set IDs in SB land/lake groups
[ ] Add SB Group properties
[ ] Compute avg annual runoff
[ ] Go back and fix all the other tests that are now broken

Sounds good. I saw that you were on a roll yesterday night. :)

@julemai @huard A lumped GR4JCN model must now have a valid (ie. non-zero area) initial HRU defined:
https://github.com/CSHS-CWRA/RavenPy/blob/f7964ff9becb61887d155809f2912d47f9715919/tests/test_emulators.py#L407
which means that just doing this in the emu ctor is not enough:
https://github.com/CSHS-CWRA/RavenPy/blob/f7964ff9becb61887d155809f2912d47f9715919/ravenpy/models/emulators.py#L98
(because if not Raven complains that the HRU has zero or negative size).
So my question: now that we are being explicit about it (because we have generalized the lumped/not lumped use cases) should we modify the emulator constructor somehow, to make it more obvious?

Hi @cjauvin! I am confused. The HRU always had a valid area and it is 4650.6 km2. The areas will be given in the routing product of by another external source that is delineating the basin. The simple test (lumped model) was definitely running when I tested the routing test. I always did both... confused

Ah you are right, I had not realized that we had that before:
https://github.com/CSHS-CWRA/RavenPy/blob/master/tests/test_emulators.py#L70-L74
and that:
https://github.com/CSHS-CWRA/RavenPy/blob/master/tests/test_emulators.py#L127-L130

TestGR4JCN.test_resume fails:
        for key in ["Soil Water[0]", "Soil Water[1]"]:
>           np.testing.assert_array_almost_equal(
                model_a.storage[key] - model_ab.storage[key], 0, 5
            )
E           AssertionError:
E           Arrays are not almost equal to 5 decimals
E
E           Mismatched elements: 184 / 185 (99.5%)
E           Max absolute difference: 0.00228928
E           Max relative difference: inf
E            x: array([ 7.79141e-07, -3.16415e-04, -5.87290e-04, -5.85631e-04,
E                  -8.07810e-04, -1.00260e-03, -1.18140e-03, -1.33439e-03,
E                  -1.46652e-03, -1.58590e-03, -1.70162e-03, -1.79083e-03,...
E            y: array(0)

The diference is likely related to the way we now parse the RVC but it eludes me.. I've been looking at before/after values and the parsing seems ok to me.. could you please have a look?

If the setup files are EXACTLY the same as before the results will be EXACTLY the same. Could you compare the setup files (all 5 RV* files) from before and after and let me know what is different?

This is the "resume" scenario where a model runs in different steps (after
dumping and rereading its state in a solution.rvc); I tried to do what you
say but this setup is much harder for me to analyze. But anyway I will try
to continue my debugging using other comparison techniques.
On Sun, Mar 21, 2021 at 1:11 PM Julie @.***> wrote:

If the setup files are EXACTLY the same as before the results will be
EXACTLY the same. Could you compare the setup files (all 5 RV* files) from
before and after and let me know what is different?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CSHS-CWRA/RavenPy/pull/71#issuecomment-803625083, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXMIHTIJ7DBXB3Y22P76TTEYSFRANCNFSM4Y2JTQFA
.

Ok to merge ?


[Feature request]: To show GPU driver version

I think will very informative to show as default, graphic driver (in use) next to graphic card model as:
GPU: NVIDIA GeForce RTX 2060 Rev. A  (NVIDIA Driver Version: 470.93)
or
GPU: NVIDIA GeForce RTX 2060 Rev. A  (Nouveau Driver Version: 20.2)
or
GPU: Intel CoffeeLake-S GT2 (Intel driver i915 version 1.52)
and similar
Thank you

still actual

Yup, just wanna see my mesa version

Just wanna see my mesa version


index.faiss和index.pkl这两个文件的具体作用是什么

只知道这两个是跟向量，索引有关系，但是为什么会有两个文件

Hey @Thunderltx, nice to see you diving deep into Langchain-Chatchat again! How's everything going?
In the context of Langchain-Chatchat, the index.faiss and index.pkl files are used for different purposes related to indexing and retrieval of documents.
The index.faiss file is used to store the vector embeddings of the documents. FAISS (Facebook AI Similarity Search) is a library for efficient similarity search and clustering of dense vectors. In Langchain-Chatchat, FAISS is used to perform similarity searches among document embeddings. The index.faiss file contains the actual numerical vectors that represent the documents in a high-dimensional space. Operations such as adding embeddings, similarity search, and saving the vector store suggest that the embeddings are managed through FAISS, and index.faiss is the physical storage of these embeddings.
On the other hand, the index.pkl file is used to store metadata or auxiliary information related to the documents or the embeddings. The Python pickle module is used for serializing and deserializing Python object structures. In the context of document storage and retrieval systems, a .pkl file could be used to store information such as document IDs, their corresponding metadata, and possibly mappings between document IDs and their vector embeddings in the FAISS index.
The reason for using two distinct files for indexing is to separate concerns between the numerical vector data (index.faiss) and the document metadata (index.pkl). This separation can improve manageability, especially when either the embeddings or the metadata need to be updated independently. It also allows for more efficient operations on embeddings and metadata, and facilitates scalability as the system grows.
However, the specific metadata stored in the index.pkl file and the specific operations performed on it are not directly mentioned or referenced within the given context of the Langchain-Chatchat codebase. The code primarily deals with abstract methods and interfaces for managing knowledge bases, including creating, updating, deleting knowledge bases and documents, and embedding documents for vector storage. It does not explicitly mention or operate on an index.pkl file.
For more details, you can refer to the source code of the FaissKBService class in the server/knowledge_base/kb_service/faiss_kb_service.py file and the base class in the server/knowledge_base/kb_service/base.py file.
Sources

server/knowledge_base/kb_service/faiss_kb_service.py
server/knowledge_base/kb_service/base.py

About Dosu
This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
If you want to continue the conversation, start your reply with @dosu-bot.

生成index.faiss和index.pkl的代码在哪一部分，能指出来么


Errors in typescript definition file

I'm currently using the latest pixi-particles (@4.1.2) and get compile errors on the Typescript definitions file
Error: node_modules/pixi-particles/index.d.ts
[tsl] ERROR in /node_modules/pixi-particles/index.d.ts(205,105)
TS1005: '>' expected.
ERROR in node_modules/pixi-particles/index.d.ts(206,1)
TS1128: Declaration or statement expected.
And others about missing semi-colons. Any ideas why?

Oh, was using an old Typescript 2.7.2. Have updated to 3.5.3 and is working fine.


add valid bit to MTTL3 entry

to address issue #64

PTAL at this later (merging for now to allow for PDF release generation)


[5.7] [MemAccessUtils] A resilient struct is a product leaf.

To visit the nodes of a type that is formed by repeated product operations (struct and tuple), visitProductLeafAccessPathNodes is used.  The caller provides a TypeExpansionContext to this function.
Previously, though, visitProductLeafAccessPathNodes didn't respect the TypeExpansionContext when visiting struct types.  Specifically, it looked through resilient structs to their fields.  For a caller, such as SSADestroyHoisting, that cares about the number of non-trivial nodes, that is wrong--each resilient struct is a non-trivial node.
Here, this is corrected by having visitProductLeafAccessPathNodes consider whether a struct type is resilient in the specified TypeExpansionContext.  Resilient structs are now correctly recognized as leaf nodes and the caller-provided lambda is invoked with each.
rdar://92460184

@swift-ci please test

Could you approve this @tbkka  ?


Sync executable with formal spec - EPOCH STS

In the Exec. spec, the environment for this STS does not have blocks
also, votedValue_PParams is not being used yet in the exec. spec to compute pp_new

votedValue_PParams was implemented in https://github.com/input-output-hk/cardano-ledger-specs/pull/714
blocks was removed from the environment of EPOCH in the formal spec some time recently as well.


How to send Piggybacked ACK (RFC7252 section 5.2.1)?

Hi @jkralik
Sorry for taking your time. Our coap UDP client needs a piggybacked response after sending a CONF request, which means the client is expecting a response with message payload in an ACK. And server don't send any CONF more, because the client can directly goes into sleep after receiving the ACK with response payload.
https://datatracker.ietf.org/doc/html/rfc7252#section-5.2.1
So I want to change from:
Client->Server: Request with CONF flag (confirmable: requires ACK)
Server->Client: Empty message with ACK flag
Server->Client: Response with CONF flag (confirmable: requires ACK)
Client->Server: Empty message with ACK flag

To:
Client->Server: Request with CONF flag (confirmable: requires ACK)
Server->Client: Response with payload and ACK flag

I have tried to modify to code here
However, when I see the network packets, I found that the ACK is without payload.
Do you have any idea how I should do a proper modification in this case?
Thanks in advance!
BR/Chencheng

How does your handler look? If you set the payload in the response i the handler, it should do the job (as far as I know).

Hi Tobias, Thanks for the reminder.
When I opened this ticket, there was no PiggyBack support.
But it looks like it is supported now:
https://github.com/plgd-dev/go-coap/commit/d99f2360e027f1a0f087b8940f3301acc71af264
I will double check again when I have time.

Hi @niondir we are following the example
but doing a CBOR response to a client GET request:
err1 := w.SetResponse(codes.Content, message.AppCBOR, bytes.NewReader(responseBody))

In my Wireshark I still see a CoAP ACK with empty message.
Could you help on this?

Hi again,
@niondir I reverted to before ad01d9efd3d18a78cbc76c1647f501322ab49f12 which brings back the piggyback response.
I have some other issue since my client may random drop connection and seems the server gets confused.
Anyhow, @jkralik could you check why v2.5.0 does not work with piggyback when you have time?
Thanks in advance!

@rtechencheng could you check how if piggyback response works in v3. In this version I expect that it works.
The handling response must go to line:
https://github.com/plgd-dev/go-coap/blob/b8901963ae43acd9a14f98b3d47ec17b26b38e7a/udp/client/conn.go#L555


AlertContainer: adding onCloseAlert callback props to the component

Ref. #24

Coverage remained the same at 100.0% when pulling 89aa3572ec72a004ace2a81927fd215fd456ed97 on dennybiasiolli:close-callback into c1887a4795ce5431470f8670ddce059278f07406 on schiehll:master.

Coverage remained the same at 100.0% when pulling 6bbad8df526db4e3258200b3a6b19a05bbe0f9f9 on dennybiasiolli:close-callback into c1887a4795ce5431470f8670ddce059278f07406 on schiehll:master.

Coverage remained the same at 100.0% when pulling 96ab56611edafc25751621a021c42fb8ec81c75f on dennybiasiolli:close-callback into c1887a4795ce5431470f8670ddce059278f07406 on schiehll:master.

Hey @dennybiasiolli ,
Thank you for that! And sorry about the delay.
The code looks good, but I don't know if it's the right approach, maybe we can make the API look something like this:
this.msg.show('message', {
...otherOptions,
onClose: () => {}
})

That way we can get a callback for a specific alert, because I don't know if it would be of much help to get the list of closed alerts in a global listener since we don't return the id of the alert when the show/success/error/info methods are called, so the only way to check if it's a specific alert would be to check the message, but the message could be a react component too and those are more complicated than a simple ===.
What do you think?
I don't know, maybe I'm missing something here and with the API that you are adding in this PR there is a easy way to check for a specific alert?

@schiehll you're right, your solution it's better than mine, I'm changing the code in order to have the onClose option in show function. Thanks for the suggestion! 😉

Coverage remained the same at 100.0% when pulling cec86edc4319a223c6e5c628a8ad91e0101051a7 on dennybiasiolli:close-callback into c1887a4795ce5431470f8670ddce059278f07406 on schiehll:master.

That was fast! Thank you so much!
Looks good, I will merge it and release a new version to npm later today.

Thanks @schiehll, when released I think we can close #25, right?

Sure! As soon as I release it I will close it!


JBPM-5386 - Update doc for shared authentication

@mswiderski would you mind checking this one.

@mswiderski thanks.


Validation Get & Set Rule Groups

Created Validation->getRuleGroup(string $group): array and Validation->setRuleGroup(string $group) methods to get and set rule groups.
Issue: #419
Signed-off-by: Kristian Matthews<EMAIL_ADDRESS>

Your documentation could have provided some useful examples, but I'll merge this for now. :)


Feature Request: Start teminal when user log-in

Description of the new feature/enhancement

Over time I written many Powershell scripts and modules to manage many stuff (either for personal and for work) I use some modules for manage my dev environment or my home router, or move, rename, delete personal fotos, ect.
So I'm used to have a Powershell session start when I log-in.
Now I want to move to Windows Terminal and so I would like to have it to start when I log-in
Proposed technical implementation details (optional)

The UWP SDK expose a way to configure a packaged app to start when a user log-in by adding the windows.startupTask extention to the manifest

If someone wants to add something like this following the MSDN documentation on adding a startup task, we're fine with it.
https://docs.microsoft.com/en-us/uwp/api/Windows.ApplicationModel.StartupTask
https://docs.microsoft.com/en-us/uwp/api/Windows.ApplicationModel.StartupTask.RequestEnableAsync

I want to help out with this, so I've been assessing the work involved in doing it. Though this isn't quite in my wheelhouse, I figured that laying these thoughts out might help move this particular feature along a bit even if I'm not able to actually complete everything on my own. Feedback on the below worklist is appreciated as I'm sure there's a lot I've missed or gotten wrong in this first draft!
High-level task list:

[ ] Define startupTask extension in AppX manifest. [note: I am still not familiar enough with the project structure to say exactly which manifest will need this] Here are some candidates:

CascadiaPackage
WindowsTerminalUniversal

[ ] Modify GlobalAppSettings to add StartOnUserLogin global app setting. Include implementations for To/FromJson, ApplyToSettings, etc

[ ] Incorporate into AppLogic

Implement methods to invoke enable/disabling of startup task via `StartupTask.GetAsync...

? AppLogic::OnLoaded add conditional logic to invoke the enable on startup request process startupTask.RequestEnableAsync
? AppLogic::ReloadSettings Add call to startup task logic [note: not sure if this is correct location]

HTH!

@miniksa - would you prefer I create a new issue following the same format as this to track these specs, or continue to edit in-place with refinements?

Honestly, you could probably edit-in-place here if you want.


Why do variables.env and configuration.py us different DNS names for the same services?

What are the DNS names for the redis and milvus server?  They are referred to as localhost in some places and as redis and mivlus in others.  They are pingable from traefic as redis and milvus
Redis

README.md redis://localhost:6379/0
variables.env redis://redis:6379/0
chain_server/configuration.py  redis://localhost:6379/0
container name redis

Milvus

README.md http://localhost:19530
variables.env http://milvus:19530
chain_server/configuration.py http://localhost:19530
container name milvus

The host names are redis and milvus but it can be more complex because the containerized aspect introduces additional scopes. The short version is that it depends on where you are referencing them from. Generally you can think of this as 3 scopes:

The host where you are running AI Workbench. In many cases you won't have access to this, or at least, it's less relevant because your AI Workbench environment will be entirely inside of it's own container.
The AI Workbench container.  In most cases, this is is the only reference you will even be aware of.
Other containers / networks / bindings / etc. In some cases, such as this NIM Anywhere project, an approach is used where multiple containers are at play. This is atypical and less common, but is possible to encounter. There are a number of ways this can be accomplished so specifics will depend on each unique deployment, but in this case redis/milivus are each their own containers (and you are able to access them from within the AI Workbench container). Note that per documentation the current approach used for NIM Anywhere is expected to change. I'm not tapped into any details, but I would assume they will introduce some type of orchestration approach which will allow your AI Workbench environment to spin up multiple containers instead of just the one we have today.

So... given all of that, the reference to a given hostname or "localhost" could actually mean: The actual host, The AI Workbench Container, one of the other containers, and a localhost port could actually be bound between the host-container or another container. The short version is that there are a lot of ways this can be accomplished, so each setup could have it's own unique approach.
Since there's not a single common way of doing this, and because there are so many potential configurations including exposing and mapping a port outside of a container (either on the host, a container, a container network, or other) you need to figure out where you are and look at what you are referencing to understand what you are actually looking at.
If you want to go down the rabbit hole, you can find more info in the Docker Networking docs, but that doesn't even get into more abstract configurations like Docker-in-Docker or Docker-out-of-Docker (which is what NIM Anywhere uses); these and other approaches add additional potential layers of complexity.

That is an awesome explanation. It still confuses me about the configuration in this project
How are the different files intended to be consumed with their different values? It isn't obvious what processes use which file under which conditions.

variables.env

Config for milvus defaults to milvus:.  Doesn't seem to be referenced.
Config for redis defaults to redis:.  Doesn't seem to be referenced.

frontend/configuration.py,

Not part of this

code/config.yaml

Config for llm-0: defaults to container name
Config for embedding defaults to the container name
Config for reranking defaults to the container name
It uses the container name for locally running containers and not localhost

chain_server/config.yaml

Config for milvus defaults to localhost
Config for redis defaults to localhost
Config for  llm defaults to NVIDIA endpoints or the container name if local.
Config for re-ranker defaults to NVIDIA endpoints or the container name if local.
Cnofig for embedding defaults to NVIDIA endpoints or the container name if local.

Healthchecks point at localhost.  This is because they test from inside the target container.
The NIM-ANywhere project is a multi-container project. The AI Workbench container sees the Milvus and Redis containers as their container names because they are part of the same docker instance.  There are 6 services in this project that all run in their own containers or are deployed remotely.  Why are they not in variables.env?

milvus.  Runs locally. Avaialble in-cluster milvus.

👍 Connect from project-nim-anywhere to milvus:6379
👎 Cannot connect from project-nim-anywhere to localhost:6389

redis.  Runs locally.  Available in-cluster under redis.

👍 Connect from project-nim-anywhere to redis:19530
👎 Connect from project-nim-anywhere to localhost:19530

Reading a little between the lines I think you're ultimately asking the question "Why not just add the hostnames in one place like variables.env, instead of the references to localhost or other places, because project-nim-anywhere container can't access them as localhost anyways?"
Defining them in variables.env is certainly a more robust approach that would provide better flexibility and maintainability for a larger or more complex system, but for this interim stage, simplicity wins. It's easy to hard code "milvus"... maybe even easier than managing a global config and referencing something like "$MILVUS_HOSTNAME"
As for wondering which file is used under which condition, I think what's in place is misleading. Inside any container, localhost refers only to services within that container (not other containers) so I don't think references to localhost are actually used. localhost would make sense with chain-server deployed on a traditional server, but not in this setup. Any references to localhost are likely just unused remnants of default configuration for a non-containerized deployment.
Since it's working the team probably didn't feel compelled to bother cleaning up the extra bits... even if what's in place isn't ideal for any particular reason. As for why any self respecting team would choose to move forward in such a state of disarray, I can't help but think of a phrase that an old developer friend used to say, and that I fully agree with: "Premature optimization is the root of all evil."
tl;dr - Don't look a gift horse in the mouth. References to localhost probably aren't used. Since container names are simple, and all containers are networked together, it's easy just to hard code the host/container names where needed and call it a day.

PS - We do see an example in the README  where localhost is identified, but the override was defined in variables.env.
"Values specified as environment variables will take precedence over all values from files."


Activate several doc comments

Turns several several // comments that probably ought to be /// doc comments into the latter.
Also fixes some copy & paste errors and typos.

Nice @gubikmic, congrats for your first contribution 😛

Great first contribution @gubikmic 😛


Don't use type when getting accessor in dataframes

In Pandas this is an object on the type, however in other systems (notably
cudf) this is a property, which is hard to introspect.
It looks like we only use this to detect if a method exists rather than for
actual computation, so this should be low-impact

[ ] Tests added / passed
[ ] Passes flake8 dask

I'll merge this after the upcoming release.


Upcast conjure elemental using subspell filter

3rd version :)
Add additional sub-spells to ConjureElemental/Fey and filter the pop up based on slot level.
No up-cast

Up-cast

I think this is my least favorite of the 3 in terms of user experience. I know we could improve that through additional patches, but I think the other two are better in terms of behaving more similar to how current spells upcast.

The reason I tried this way is because TA have implemented Conjure Fey as:

Conjure Elemental RAW works the same way so if we wanted them to match we would add lower level elementals to the sub-spell list, and higher level ones to be filtered by slot level.

The reason I tried this way is because TA have implemented Conjure Fey as:
Conjure Elemental RAW works the same way so if we wanted them to match we would add lower level elementals to the sub-spell list, and higher level ones to be filtered by slot level.

I guess the part that's weird to me though is with your subspell implementation a user could accidentally summon the 5th level version when they're using a 6th level spell slot. What I meant by the other two behaving more similar to existing spells, is if you use a higher level slot to cast a spell you just get the benefit, you don't need to know to select the higher level slot and then the special higher level subspall.

The same thing with RAW and TAs implementation of conjure fey. You can use an L6 slot to conjure a dryad.
Doesn't make much sense in the game but maybe in tabletop. Does it, I'm not a tabletop player?

The same thing with RAW and TAs implementation of conjure fey. You can use an L6 slot to conjure a dryad. Doesn't make much sense in the game but maybe in tabletop. Does it, I'm not a tabletop player?

That's a good point. I think I'm just used to new summon spells which are designed for scaling much better, I never played a character that used the conjuration spells so they aren't something I have any experience with. While clunky this may be the representation closest to the tabletop design, but it's a clunky design.

I'll move this over to the beta branch and make some changes.

Closing for now - will move to beta branch


use ordered list start index when rendering

Bug/issue #, if applicable: rdar://73847907
Summary
While cmark-gfm saves the starting index of an ordered list, swift-markdown and Swift-DocC currently discards this information. This PR is part of a set that preserves the starting index when rendering documentation.
Dependencies

https://github.com/apple/swift-markdown/pull/22
https://github.com/apple/swift-docc/pull/53

Testing
Build this branch, then follow the testing instructions on https://github.com/apple/swift-docc/pull/53.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[ n/a ] Updated documentation if necessary

@swift-ci testPlaces to update/add tests:

@swift-ci test

Going to go ahead and merge this since the change is ready and backwards compatible with existing JSON data.