Separate server and model configuration during train / run

Rasa NLU version (e.g. 0.7.3):
Used backend / pipeline (mitie, spacy_sklearn, ...):
Operating system (windows, osx, ...):
Issue:
in the version, the config file is loaded when the server starts.
I wonder if it can support to train new model in a HTTP server with a new config file to override existed files.
even though, the change makes the training parameter be a config file(contain project name, new training data, new pipeline)
Content of configuration file (if used & relevant):

You can specify almost any configuration parameter as a url parameter when you POST to the /train endpoint.
For example you can override the pipeline by:
http://localhost:5000/train?project=my_project&pipeline=nlp_spacy,tokenizer_spacy,ner_spacy
Does that help?

New configuration should be able to:

clearly separate server configuration from model configuration (maybe we can just remove the server parameters and pass them via cmdline - need to check how many there are)
allow to specify a run configuration to be used when running a model (e.g. to specify mitie vector file path)

very good question that I don't know the answer to - but I don't think it is possible to pass in anything but strings and arrays, so no objects.

Very good input from @Le-Fix :
1 Pipeline setup (including component specific setup)
2 Training Data
3 Training Configuration (reference 1, reference 2, target folder, num_threads,...)
4 Trained Model (exisiting metadata.json which consists of: 1, reference 2 and some training log)
5 Model Serving (reference 4, token, port, logfile, emulate,...)
(I just pasted this from the closed issue for reference)

I just update this comment from time to time whenever I think some config behavior is not obvious/easy to understand based on the one cluttered configuration.
As far as I can see the config is passed straightforward to the ComponentBuilder
Therefore the config(s) should only consist of 4 (and maybe 5 if the Interpreter itself considers how/where it's run)
4 already would be specified by the model dir
rasa_nlu.model Interpreter
def load(model_dir, config=RasaNLUConfig(), component_builder=None, skip_valdation=False):
        """Creates an interpreter based on a persisted model."""

done.

May I know if currently the config is now refreshed every time the server starts? Say if I specify duckling url to http://duckling:1234, will it be able to call the corresponding port number without retraining the model?

Hi @justinclare, hopefully, you'll find an answer to this in https://github.com/RasaHQ/rasa/issues/6240 comments. I did :-)


Yaw data is not right

I am getting very accurate date for pitch and roll, but yaw does not seem to be correct at all. It varies between 30 and 50, and when the imu is moving very slow it reaches lower angles. Any advice? Thanks

As I always need to ask - have you calibrated the magnetometers? You didn't say what IMU you were using - that would be helpful to know.

I am using the Adafruit 10-DOF IMU (L3GD20H, LSM303, BMP180). The example program they give does not incorporate all of the sensors so I needed a fusion program to get the best pose data.
As far as I know the magnetometer has been calibrated properly.
Separate question: While I am currently using an arduino uno, would this program work with an arduino yun?
Thanks!!

Even without calibration that IMU would work pretty well. Are you sure that there aren't any local magnetic disturbances? It should work.
As for the yun, I have not tested that but it probably would.

I just checked my Adafruit 10-dof and it does seem fine. Maybe redo the mag calibration. Make sure to get good maxima and minima in all the axes and then save the results. What you are seeing definitely looks like bad calibration data.

I reran the calibration and it seemed to work pretty well, even worked for the Yun. Thanks for the help. In terms of calibration data is there a max and min I should be looking for to indicate a good run? Also is there a way to set a certain  orientation as 0 pitch 0 roll 0 yaw.

For that IMU, something like -50 to 50 would be reasonable. However, things like cables etc can be magnetic so it can distort this very significantly.
Regarding the orientation, take a look at these two posts:
https://richardstechnotes.wordpress.com/2014/11/18/rtimulib-now-supports-imu-axis-rotation/
https://richardstechnotes.wordpress.com/2014/11/20/using-quaternions-to-correct-imu-installation-offsets/
Hopefully that will help.

I should mention that for the Arduino, the reference orientation is set by uncommenting the appropriate setting in RTIMULibDefs.h. The blog post refers to the Linux version.

Hello
I am getting a similar problem and I am 100% certain it is to do with my compass calibration. The reason being that when I set fusion.setSlerpPower to 0 I get the correct angles e.g. 90 degree turn is reported as 90 degrees however as expected this suffers from drift. When I set the value to anything above 0 then a 90 degree turn is reported as nearer 130 degrees. No matter how many times I try to calibrate it I can not get it to report the expected results.
Could this be down to the location where I am trying to calibrate the sensor? The compass on my phone works fine in this location but I am assuming this is because it is already pre calculated.
These were my last calibration results:
minX: -29.67 maxX: 63.21
minY: -43.10 maxY: 47.08
minZ: -32.56 maxZ: 53.10
Thanks
James

Which IMU?

Sorry. I am using the SparkFun MPU-9150.

Are you sure you are getting the real max and min? Those results look a bit symmetrical around zero. Usually the 9150 is all over the place. It's very important to move the IMU so that all maxima and minima are hit - just waving the IMU around will not work. I know some systems use a figure of 8 system but this doesn't. So it takes a bit of concentration to position the IMU with each axis up and them down (total 6) and waggle them about in each of the 6 positions because the extrema may not be totally aligned with the physical axes.
Silly question but you are remembering to press "s" and enter to save the calibration data? It does not save it automatically. ArduinoIMU reports at the start whether it is using calibration or not...

When I ran the calibration sketch I rotated the sensor around in all directions until the displayed data in the serial monitor had stopped updating. I then sent the 's' command and it reported that the data has been saved. The ArduinoIMU reports that it has calibration data although on one run it did report not having any at which point I recalibrated  it and it has not done this since. The previously posted Max and Min values I copied from the serial monitor once it had speed updating.
I will try calibrating it again and post my results. I will also try it in a different room via my laptop and an Arduino Nano instead of my desktop and an Arduino Uno.
Thanks for getting back to me so quickly.


Normalization regex fix

Handle the commandArgKeyPair normalization checking if the key=pair is not inside a string, also leaves untouched the key=pair strings that are not enclosed in quotes (both single/double)
As seen in the issue #307, this is the behavior now:

This is the proposed regex with the solution and some parse examples:

Looks like the regex is invalid in Node 6. Maybe we can just drop that target.

For me it's ok, usually I only target these for node projects:

node latest stable Node.js release  (right now v10.8.0)
lts/* latest LTS Node.js release  (right now v8.11.3)

Yeah, let's remove the node 6 target.

@milesj I see you are very active on this repository, which unfortunately seems not maintained by the owner, if you are interested I forked this project months ago and made many changes (bugfix, mainly from vorpal issues) and I completed last week the typescript migration (it was a nightmare).
If you want we can keep going there, I decided to open this PR because the fix (that I made in my forked version) was very small and easily integrable on the original project, but I prefer to work on the fork since has many more changes on it.
is a problem for you? my goal is to build a new open source (MIT) project from the fork once has been completely refactored (typescript, usage, docs, new features, ....) are you interested in contrib? it will be faster than here on the original project because Im using the forked version in various projects and I frequently find possible updates

I can imagine how tough the TS conversion was! I ran into the same problem trying to add flow types.
I don't really have time to contribute, but forking it is probably the best route since I'm unable to publish new versions here.


'type' object is not subscriptable

Hi. I'm trying to import atlas_browser but when I run the command "from atlas_browser import AtlasBrowser", I get this error: 'type' object is not subscriptable
Why is that and how can I fix it?

Hi, I did not encounter this issue before, so I am not sure why this is occurring.
I have just updated the repository to v0.4 and uploaded it as a package which you can install using pip: https://test.pypi.org/project/mni-to-atlas/0.4/
Here there are updated instructions on installing and using the tool. I hope installing the tool in this way will correct any such issues.
Let me know if you have any further questions!

Hello!
When I'm trying to import the library, I get a syntax error on line 336 of the init file:
region_id = int(self._atlas[*atlas_coords])
^
SyntaxError: invalid syntax


Graph time range should be based on query time range not returned data

Note how I've queries for the last 5m, but the domain of the x axis only spans the last 2m. This is because I did not have any data for 3 of the last 5 minutes. This can be confusing when you have graphs where you expect the domains to be similar.
Ideally I could specify whether I'd like the domain to span the entire range queried or the range or the data returned.

EDIT: This is incomplete as the now() selections don't return the correct data


Warning when using with @Published values

When using this library with @Published values, I get the old familiar warning in Xcode:

Publishing changes from background threads is not allowed; make sure to publish values from the main thread (via operators like receive(on:)) on model updates.

Note that this warning actually stems from Binding.withDelaysIfUnsupported(from:to:keyPath:) in the NavigationBackport source code, and not from my app source code.
Is this something that I should be worried about? Is it "just a warning"? Can we get rid of it by making some changes to the NavigationBackport library?
Related to #20

Thanks @denizdogan for raising this issue. I believe I've fixed it by correcting the missing @MainActor annotation on one of the withDelaysIfUnsupported methods. This is available in v0.6.1.

@johnpatrickmorgan Sorry, didn't mean to leave you hanging, I honestly literally just had time to test the fix, and everything seems good. Thanks for your help!

No worries, thanks for confirming!


Removed old core structure

[Builds off commits in #64]
Went through and removed all the code and test dependencies on Value/Event/Timeline (in addition to the actual core.py implementations).
Some things are unsure of, such as cutting most of Timeline. Open to suggestions.

Changes Unknown when pulling 7ba6a7dc941de13338f5e25bb85c9034c669242f on qmac:removing-old into ** on tyarkoni:master**.

One minor issue here is that I think this PR reflects all of the commits in #66 except the last one--the numerical likelihoods are still there in the Face Extractor API. Do you want to take a quick look through this and make sure it's good to merge now that I've merged #66 into master?

Coverage decreased (-4.09%) to 72.414% when pulling 3261df578d0eba1af48cbb8257feaae075d4a2d2 on qmac:removing-old into 8d4a9290090da6421b404f4c497f290fb1b5d706 on tyarkoni:master.

Coverage decreased (-4.07%) to 72.43% when pulling 4972903c8f20c71492adecee28def3442fd320b5 on qmac:removing-old into 8d4a9290090da6421b404f4c497f290fb1b5d706 on tyarkoni:master.

Coverage decreased (-4.1%) to 72.093% when pulling a84b4e07b895e58de9d1e3df28727ad3983a8012 on qmac:removing-old into 68d080d2b361a68a8744bd4e8839cc2e17b753ca on tyarkoni:master.


WIP: Initial version of launchpad support

Resolves #215
Signed-off-by: Craig Northway<EMAIL_ADDRESS>

closing for now. @craigez will reopen when he gets a chance to re-engage


Bash IDE

provide a full ide for bash with mads-hartmann.bash-ide-vscode

Similar to #253, happy to take a look at a PR if you're so-inclined to create one!

Resolved with #391


Library not loaded: @rpath/CwlUtils.framework/CwlUtils

After updating to latest master 0d31f45e224e0d09bfc01be28729fee2fe2b7d8c build for device crashes due to image not found error.

I've added the following instructions to the bottom of README.md file. I hope it explains what has happened and what you need to do.
Additional steps for latest CwlSignal 2.0.0-beta builds on master
The latest master versions (including builds tagged 2.0.0-beta.1 and 2.0.0-beta.2) no longer copy the CwlUtils.framework inside the CwlSignal.framework. This avoids duplicate inclusion of the framework and other potential problems but it means that you need to copy CwlUtils.framework into your build as part of your Copy Files (Frameworks) build phase (in the same way that you're already copying the CwlSignal.framework).
To copy the CwlUtils.framework, expand the CwlSignal.xcodeproj -> Dependencies folder. You should see three frameworks there. They may be red (depending on whether you've build the Mac debug build) but that doesn't matter. Drag the CwlUtils.framework onto the Copy Files (Frameworks) build phase where you've already got CwlSignal.framework. You don't need the other two frameworks (they're used by the CwlSignal testing target, not the main build).


Uncaught TypeError: Cannot set property '[Note ID: d6728c2a74e668bbb188e4023b9ee52f]' of null

[Enter steps to reproduce below:]

...
...

Atom Version: 1.3.1
System: Mac OS X 10.11.2
Thrown From: ever-notedown package, v0.2.17
Stack Trace
Uncaught TypeError: Cannot set property '[Note ID: d6728c2a74e668bbb188e4023b9ee52f]' of null
At /Users/michael/.atom/packages/ever-notedown/lib/note-helper.coffee:95

TypeError: Cannot set property '[Note ID: d6728c2a74e668bbb188e4023b9ee52f]' of null
    at NoteIndex.addnote (/Users/michael/.atom/packages/ever-notedown/lib/note-helper.coffee:95:5)
    at StorageManager.exports.StorageManager.StorageManager.addNote (/Users/michael/.atom/packages/ever-notedown/lib/storage-manager.coffee:197:27)
    at Object.module.exports.saveNewNote (/Users/michael/.atom/packages/ever-notedown/lib/ever-notedown.coffee:1384:34)
    at Object.module.exports.file2Evernote (/Users/michael/.atom/packages/ever-notedown/lib/ever-notedown.coffee:1064:10)
    at /Users/michael/.atom/packages/ever-notedown/lib/ever-notedown.coffee:475:12
    at Emitter.module.exports.Emitter.emit (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/event-kit/lib/emitter.js:86:11)
    at HTMLButtonElement.<anonymous> (/Users/michael/.atom/packages/ever-notedown/lib/ever-notedown-preview-view.coffee:454:22)
    at HTMLButtonElement.jQuery.event.dispatch (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/jquery/dist/jquery.js:4435:9)
    at HTMLButtonElement.elemData.handle (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/jquery/dist/jquery.js:4121:28)

Commands
     -0:29.9.0 ever-notedown:new-note (atom-text-editor.editor.is-focused)
     -0:17.3.0 core:backspace (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "vim-mode"
    ]
  },
  "ever-notedown": {}
}

Installed Packages
# User
autocomplete-python, v1.2.3
ever-notedown, v0.2.17
linter, v1.11.3
linter-flake8, v1.9.2
project-manager, v2.6.5

# Dev
No dev packages

A solution was proposed in a similar issue. Please consider following the preparation steps as described in the doc in section Installation -> Preparation.
I'll close this issue for now, please let me know if you're still having trouble.


axInput, axTooltip: reflect position change of anchor element

Whenever the position of the anchor element of a tooltip changes, this isn't recognized by the axTooltip control or the error tooltip of axInput resp.. Hence it may be at a totally wrong position when the anchor element is moved around.

Implemented on master (v1.0.0) and master-0.x (v0.23.0).


Fix pointcloud for L515

Track on LRS-240

Fix validated on Windows

@nohayassin
Are there any unit-tests that test this mechanism?

@nohayassin
Are there any unit-tests that test this mechanism?

The unit test should compare depth-units value from frame metadata against the value from FW.
We didn't have DU in frame metadata before this feature, so I assume there is no unit-test like that.


All dispatch tasks should check for cancellation

Currently, they don't, but they should. Cancel all operations does not actually remove things from the queue so you need to check for cancellation.

#35


[unified_multi_sum.jsonl] excessive ”\n“ in this file.

I noticed that there are excessive line break characters ('\n') and blank spaces in a single sentence within the unified_multi_sum.jsonl file. I suspect that this data was collected from books or PDFs, and as a result, the line breaks in every line from the original sources were included in the dataset. This issue can be seen in Line 1, for instance:

{"text": "\n: Summarize the following proposed legislation (bill): SECTION 1. SHORT TITLE.\n\n    This Act may be cited as the Patients and Public Health \nPartnership Act of 2008''.\n\nSEC. 2. DEMONSTRATION PROJECT FOR INTEGRATED HEALTH SYSTEMS TO EXPAND \n              ACCESS TO PRIMARY AND PREVENTIVE SERVICES FOR THE \n              MEDICALLY UNDERSERVED.\n\n    Part D of title III of the Public Health Service Act (42 U.S.C. \n259b et seq.) is amended by adding at the end the following new \nsubpart:\n\n Subpart XI--Demonstration Project for Integrated Health Systems to \n  Expand Access to Primary and Preventive Services for the Medically \n                              Underserved\n\nSEC. 340H. DEMONSTRATION PROJECT FOR INTEGRATED HEALTH SYSTEMS TO \n              EXPAND ACCESS TO PRIMARY AND PREVENTIVE CARE FOR THE \n              MEDICALLY UNDERSERVED.\n\n    (a) Establishment of Demonstration.--\n            (1) In general.--Not later than January 1, 2009, the \n        Secretary shall establish a demonstration project (hereafter in \n        this section referred to as the `demonstration') under which up \n        to 30 qualifying integrated health systems receive grants for \n        the costs of their operations to expand access to primary and \n        preventive services for the medically underserved.\n            (2) Rule of construction.--Nothing in this section shall \n        be construed as authorizing grants to be made or used for the \n        costs of specialty care or hospital care furnished by an \n        integrated health system.\n    (b) Application.--Any integrated health system desiring to \nparticipate in the demonstration shall submit an application in such \nmanner, at such time, and containing such information as the Secretary \nmay require.\n    (c) Criteria for Selection.--In selecting integrated health \nsystems to participate in the demonstration (hereafter referred to as \n`participating integrated health systems'), the Secretary shall ensure \nrepresentation of integrated health systems that are located in a \nvariety of States (including the District of Columbia and the \nterritories and possessions of the United States) and locations within \nStates, including rural areas, inner-city areas, and frontier areas.\n    (d) Duration.--Subject to the availability of appropriations, the \ndemonstration shall be conducted (and operating grants be made to each \nparticipating integrated health system) for a period of 3 years.\n    (e) Reports.--\n            >

Yes. I see. These are legislations. This comes from the original dataset. Let's see if we can remove som exceissive \n. But note that the spaces are needed because they are meant to format the leglislation. If you don't want to wait, maybe you can just filter it out by doing a replace("\n\n", "\n")


Delete mod does not delete mod folder

If a mod is placed in a subfolder, delete mod only deletes the dll or the mod_info.
In the later case, the dlls may still be detected and loaded in the info's place.
Modnix should detect this case and prompt for folder deletion or file deletion.

Typo in commit. Re-opening.

Implemented in commit 25bd05588acfe7865a33ea1ddb653c393a042752. Closing.


Support setting specific SCSS files

It would be great if we had an option to override the default for compiling all scss files.
It looks like this line just needs to take the options into account so we can provide an array of files to include that would take the place of the **/*.scss pattern.
I'd be open to contributing this change but I've never worked on a broccoli plugin before and I'm just starting to trace through it. I'm sure someone more experienced than myself would be able to crank this out pretty quickly.
Thanks!

adding this to your Angular2App configuration excludes based on the regex:
    sassCompiler:{
        "cacheExclude": [/\/_[^\/]+$/]
    }

so this is already supported!!!


Twitter Integration

Suggestion: A Twitter Integration that posts/links a twitter account's tweet in a channel automatically.
If a Twitter API Key is provided to the bot, it could automatically link Tweets in a channel chosen by the bot owner to announce them.
Example of the integration over slack: http://i.imgur.com/iisySs0.png
Thanks for your time!

Is this planned for 1.0?

Since discord has webhooks now which can do this, i think i should close this issue?

Yeah, I've been a webhook with zapier and it works fine...

thx for the update


Exclude "Ontologies considered in NFDI4Cat" section from Main Readme

The table is quite redundant and provides close to no additional insights, so for brevity and usability of the main Readme from the repository, the table should be excluded.

Solved by commit 5d07edb7f52038742b6734409e7a14d0a4ef9146


Implement Validator

Implement a validator and apply it to the assets endpoint.

This pull request is now ready for a code review! I also added validation for the assets bounding box endpoint.


HasAllowBindParameter

Rationale
Provide controls for accepted property names for form binding
Related Pull Requests

https://github.com/LabKey/commonAssays/pull/452
https://github.com/LabKey/LabDevKitModules/pull/109
https://github.com/LabKey/platform/pull/3222
https://github.com/LabKey/testAutomation/pull/1063

Changes

This appears to have broken the commenting API utilized by ELN. This is regressing in 22.3 which has since been merged forward to develop. See https://teamcity.labkey.org/buildConfiguration/LabKey_223Release_Premium_ProductSuites_Biologics_BiologicsElnPostgres?mode=branches#all-projects.


Fall22 overhaul

This PR incorporates

theme updates
shared file organization
file references

The goal of this PR is

to share the GDI theme across all workshop files
to include template slides like pre- and post-event slides

There is a demo workshop directory for copying to make new workshops.
There is a dedicated workshop directory for intro-python that mirrors multi-session workshops.
Updating the technical content for any workshops is out of scope for this PR.
Shared assets are in the top of the directory.

Hi @jeseekia - I think this repo is where I originally hosted materials back when I gave the course, right? Will those materials still be available in a folder? I like to be able to reference them sometimes. Thanks!

Hi @jeseekia - I think this repo is where I originally hosted materials back when I gave the course, right? Will those materials still be available in a folder? I like to be able to reference them sometimes. Thanks!

Hey @pamelafox1! Nice web apps workshop today! The folders for each workshop still contain the exercise files and other pre-existing material. Some of the content changes may be updated in the future as other instructors are iterating on this, but I believe everything your looking for should still exist. Also this repo wasn't deployed for some reason so it will be getting a GH Pages deployment.
What materials are you looking to reference? Since these workshops are being updated and we're reorganizing contents, it'd be helpful to know what you're pointing students to.
Python 1 and 2 were split into individual directories and the index pages of those reference all the relevant slides and exercise links.

These changes are being merged, however, https://github.com/girldevelopit/gdi-python/tree/early-2022-material is a branch of main as it exists now.


Transaction auto-rollback

Docs contains example without Rollback with note "transaction will auto-rollback when disposed if either commands fails".
Does it mean that no
catch (Exception ex) {
 transaction.Rollback();
}

needed (opposite to code samples everywhere)? From which line on code than Rollback actually will be called?

@divega and @AndriySvyryd for their input on this one. Disposing does rollback on SQL Server, but it sounds like the behavior is provider dependent... so maybe we should be rolling back manually in the docs? Or maybe is accepted enough that providers should do this, so we are ok showing simpler coding pattern?
From https://msdn.microsoft.com/en-us/library/bf2cw321(v=vs.110).aspx

Dispose should rollback the transaction. However, the behavior of Dispose is provider specific, and should not replace calling Rollback.

Never thought of this behavior being provider specific (i.e. it seems the only behavior that would make sense) but it could be just me :smile: Rolling back inside a catch block should work too, it is only more complicated code and a more complicated interaction with the server.

I don't think we've called Rollback before in docs, but it's not that much additional code.

Do we know of any providers where its is needed to call Rollback explicitly? Our tests rely on Dispose using a rollback, so I think we know it is good for at least SQL Server, SQLite, SQL CE, and Postgres.
My preference would be to leave it as is.

@ajcvickers Is it this test 🤕 https://github.com/ErikEJ/EntityFramework.SqlServerCompact/blob/master/test/EntityFramework.SqlServerCompact.FunctionalTests/TransactionSqlCetest.cs ?

@ErikEJ No, what I mean is that we create a transaction and throw it a way in all our functional tests that manipulate data so as to isolate them from each other.

I would rather leave things as-is, adding the try/catch just adds clutter to the code 😄

Do we know of any providers where its is needed to call Rollback explicitly?

I'm tring to find rollback in https://github.com/PomeloFoundation/Pomelo.EntityFrameworkCore.MySql/search?utf8=✓&q=rollback

@sm-g The file you are looking for is https://github.com/mysql-net/MySqlConnector/blob/master/src/MySqlConnector/MySqlClient/MySqlTransaction.cs
And it does call rollback


Different compiler needed on OpenBSD

On OpenBSD you need to install a newer compiler to get Nokogiri 1.6.8 to compile. This newer compiler is called egcc (for external I think). It would be great if Nokogiri's install process could detect it, so a normal bundle install would run without problems. Currently we need to manually connect to the server and run CC=egcc bundle install. Would you be willing to add such an exception/workaround? If so, what would be the best place to put this? I can then make a pull request for it.
Thanks and keep up the good work!

I'm not sure I understand why this would be happening.
If you take a look at ext/nokogiri/extconf.rb you'll see we compile everything using RbConfig::MAKEFILE_CONFIG['CC'], which should be the same compiler used to compile Ruby itself.
Can you explain a bit about why "you need to install a newer compiler to get Nokogiri 1.6.8. to compile"? I'd like to understand a bit better the problem you're trying to solve.

We install ruby from packages, and it was compiled using gcc 4.2 which is the default compile (i.e. CC=gcc). Apparently the newer libxml needs to be compiled with something newer, so we need to override the CC environment variable and use egcc which is gcc 4.9. It would be great when I can extend nokogiri with a check that it will use egcc when on OpenBSD, because otherwise you will always have to override CC manually.

Hi, i unfortunately still don't understand the problem you're trying to solve.
Can you show me what happens when you compile using the default settings? What breaks? Can you show the logs?

The problem is simple: default gcc in OpenBSD is too old to compile libxml2. It gives this error during install of nokogiri:
Extracting libxml2-2.9.4.tar.gz into tmp/x86_64-unknown-openbsd6.0/ports/libxml2/2.9.4... OK
Running 'configure' for libxml2 2.9.4... OK
Running 'compile' for libxml2 2.9.4... ERROR, review '/home/frank/.gem/ruby/2.2/gems/nokogiri-1.6.8/ext/nokogiri/tmp/x86_64-unknown-openbsd6.0/ports/libxml2/2.9.4/compile.log' to see what happened. Last lines are:
========================================================================
/usr/include/unistd.h:404: warning: redundant redeclaration of 'ftruncate'
/usr/include/sys/types.h:217: warning: previous declaration of 'ftruncate' was here
/usr/include/unistd.h:429: warning: redundant redeclaration of 'truncate'
/usr/include/sys/types.h:218: warning: previous declaration of 'truncate' was here
In file included from xmlIO.c:33:
/usr/include/stdlib.h:184: warning: redundant redeclaration of 'mktemp'
/usr/include/unistd.h:423: warning: previous declaration of 'mktemp' was here
/usr/include/stdlib.h:215: warning: redundant redeclaration of 'mkstemp'
/usr/include/unistd.h:422: warning: previous declaration of 'mkstemp' was here
/usr/include/stdlib.h:255: warning: redundant redeclaration of 'mkdtemp'
/usr/include/unistd.h:504: warning: previous declaration of 'mkdtemp' was here
/usr/include/stdlib.h:295: warning: redundant redeclaration of 'mkstemps'
/usr/include/unistd.h:505: warning: previous declaration of 'mkstemps' was here
xmlIO.c: In function 'xmlXzfileClose':
xmlIO.c:1450: error: 'LZMA_OK' undeclared (first use in this function)
xmlIO.c:1450: error: (Each undeclared identifier is reported only once
xmlIO.c:1450: error: for each function it appears in.)
*** Error 1 in . (Makefile:1248 'xmlIO.lo': @echo "  CC      " xmlIO.lo;/bin/sh ./libtool --silent --tag=CC    --mode=compile cc -DHAVE_CONF...)
*** Error 1 in . (Makefile:1465 'all-recursive')
*** Error 1 in /home/frank/.gem/ruby/2.2/gems/nokogiri-1.6.8/ext/nokogiri/tmp/x86_64-unknown-openbsd6.0/ports/libxml2/2.9.4/libxml2-2.9.4 (Makefile:859 'all')
========================================================================
*** extconf.rb failed ***

with CC=egcc it works fine. I want to fix this in the gem by detecting OpenBSD and using the newer compiler.

This seems like something that needs to be fixed in the documentation as well; introducing it into the build without noting that egcc is a requirement seems less than ideal.
If you'd like to send a PR to both the installation tutorial and the Nokogiri build script, those would be welcomed.

So the answers to my original two questions: "Would you be willing to add such an exception/workaround? If so, what would be the best place to put this?" are: Yes and in the build script? Where can I find that script? I'm willing to create PR's for it yes :)

The answers to your two questions are:

yes, and
PRs to both the documentation and the build script.

The build script in this case is ext/nokogiri/extconf.rb, which I referenced in my first reply on this thread.

If you install both libxml and nokogiri using OpenBSD packages, things work fine with the OpenBSD system compiler (gcc 4.2.1), even though OpenBSD's libxml port is for version 2.9.4.  I'm working on update for the OpenBSD nokogiri port (from <IP_ADDRESS> -> <IP_ADDRESS>) and noticed that nokogiri was now forcing egcc use even though it is not necessary or desirable.
Installing nokogiri with --use-system-libraries also works fine with the OpenBSD system compiler if libxml is installed via the OpenBSD package, which is recommended on OpenBSD.  Defaulting to using egcc on OpenBSD and erroring if it is not installed should only be done if --use-system-libraries is not provided.
In the future, if you have questions about nokogiri usage on OpenBSD, you may want to ask the maintainer of the nokogiri port for OpenBSD (me).

@frenkel care to comment on the above?

@flavorjones It seems right to only do this when not using system libraries. That way it will work in both cases on OpenBSD and won't unnecessarily require egcc.

I've submitted a pull request that I think will fix the issue.
@flavorjones  I apologize, I was in a bit of a hurry when I wrote my previous post, and I think I came off a bit harsh, which wasn't my intent.  I definitely appreciate the work you are doing to maintain nokogiri.  If you have questions about nokogiri usage on OpenBSD in the future, please do reach out to me and I'll do what I can to help.

@flavorjones Sure, and I've merged the PR above.  The term "port" in OpenBSD (and FreeBSD) refers to a recipe to build a package from source, and it looks like OpenBSD currently has no local patch in its port, which we, the developers and contributors, can all be proud of.

@jeremyevans Thanks for the PR.  I see you are listed as maintainer of the OpenBSD port, and I much appreciate what you have been doing for it and many other ports related to the ecosystem around Nokogiri!  We'll do our best to reach out to you in the future whenever we know a change may affect OpenBSD, however we deal with a lot of platforms and sometimes we may make a mistake and accidentally break the build on a specific platform while trying to fix an issue for other platforms.  So, as much as we take care, please watch this project and let us know if you find we are doing wrong for OpenBSD.  We'll always be open to improvements.  Thanks!

@knu Good to hear such a positive attitude towards non-Linux systems. Keep up the good work!


Suggestion: Screen Size

So far, I'm enjoying EzGraver (beta). After a bit of troubleshooting I've got it working quite well, I like the additional features over the release version. My suggestion is merely a screen size issue. I use the program on a laptop with a wide screen, but narrow top to bottom. The top and bottom of the app is getting cut off. I would only suggest that you resize a few items or rearrange the controls to fit a smaller screen better. Screenshot

Thanks so much for EzGraver (beta)! . I've the same problem than @alanv73 .


Timestep schedule for sampling

Hello. Could you please explain why do you use this tilmestep schedule:
 _timesteps = list(
            reversed(range(0, self.bins_max, self.bins_max // steps - 1))
        )[1:]
_timesteps = [t + self.bins_max // ((steps - 1) * 2) for t in _timesteps]

May be it is from some paper?

Hi @kimihailv!  I improvised that part since there aren't many details about determining sampling schedules.
That's basically the same schedule used when training is almost finished, so picking evenly spaced points (in _timestep space) from there seems to be a natural choice. Maybe it could be overfitted to those points, who knows? 😂
That's basically the same schedule when training is almost finished, so picking evenly spaced points (in _timestep space) from there seems to be natural choice. Maybe it could be overfitted to that points, who knows? 😂 I think that's the reason why continuous-time training is better, without discretization. But practically, you could freely choose whatever sampling schemes you want. There shouldn't be a noticeable quality difference.

@junhsss Thank you for such detailed answer! By the way, it is interesting that authors found optimal timesteps by greedy algorithm

@kimihailv Oh, I must have missed that part! Thanks for pointing that out.
It is indeed interesting. So we need to sample more than N steps to perform N-step sampling... 😅 I'll implement it as soon as possible. Thanks again!


add support for automatically bind mounting /lib/firmware from the ho…

…st into the engine(s)

Addresses #354

I tried a crucible dpdk-testpmd application run with an E810 adapter and it worked.  Before this patch, it failed.


Add ctrl+double click to open object in another project

Hej Łukasz,
This is enhancement request.
I was wondering, if it would be feasible/doable to add an option to ctrl+double click on opened tab to show the window for selecting another project (same functionality as in Favourites extension)? It would be great if there was an option to quickly open same object in another system instead of using Open in Project from code view.

Another thing - do you think it would be possible to somehow hide the tabs above the editor, when using Vertical Tabs?

Thank you!

Hi Tomasz,
Open in Project function is something ABAP/SAP Specific. So far Vertical tabs is made to be independent of the programming language, but I'll check if this can be for example a part of extension for the plugin, installed separately.
I think also that the tabs above editor cannot be hidden, but I can also check that.
Cheers
Łukasz

I've created second feature with ABAP functions only with ctrl+double-click and open in project context menu

Hiding of tabs, I haven't found the way.


select date bug when changing months

Weird things happen when you select a date, then switch months and select a new date, then go back to the old month.
Version 6.0.5
Screen Recording.mov.zip

Hello Mr I was just wondering if anyone has a great way to get to the new
year with a new job interview for the delay in responding to your site and
I have to be a little bit more TIME TO GET A CHANCE for me and my family
HAS TO OFFER YOU A DECISION TO MOVE TO ANOTHER PARTY FOR THE NEXT COUPLE OF
YEARS AND I HAVE TO GO TO THE NEW YEAR TO GET INTERVIEWING FOR THEIR
VACANCIES in the world of thebest of the best way to get to
On Dec 15, 2016 10:53 AM, "jonfabris"<EMAIL_ADDRESS>wrote:
Weird things happen when
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/patchthecode/JTAppleCalendar/issues/233, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AXVIGPFY6zl8XsS-r4-xGGOSnXrTLGrsks5rIWJmgaJpZM4LOP6d
.

@jonfabris looking into this. Good catch.

Do you have have multiple selection on?

Have you followed the tutorial on how to build the calendar on this site? -> https://patchthecode.github.io/
I do not think this is a bug with the calendar.
Can you show me your
func calendar(_ calendar: JTAppleCalendarView, willDisplayCell cell: JTAppleDayCellView, date: Date, cellState: CellState)

function code please so that I can confirm?

PickDatePopupVC.swift.zip

func calendar(_ calendar: JTAppleCalendarView, willDisplayCell cell: JTAppleDayCellView, date: Date, cellState: CellState)
{
    let myCustomCell = cell as! CalendarCellView

    // Setup Cell text
    myCustomCell.dayLabel.text = cellState.text

    myCustomCell.selectedView.frame = CGRect(x:0,y:0,width:myCustomCell.frame.width,height:myCustomCell.frame.height+1)

    myCustomCell.selectedView.setGradientBackground(colorTop:UIColor(hexString: "5c5c5c")!.cgColor, colorBottom:UIColor(hexString: "cfcfcf")!.cgColor)
}

Oops, my bad, I fixed it.
func calendar(_ calendar: JTAppleCalendarView, willDisplayCell cell: JTAppleDayCellView, date: Date, cellState: CellState)
{
let myCustomCell = cell as! CalendarCellView
myCustomCell.selectedView.isHidden = true
myCustomCell.dayLabel.textColor = dateColor
// Setup Cell text
myCustomCell.dayLabel.text = cellState.text
    myCustomCell.selectedView.frame = CGRect(x:0,y:0,width:myCustomCell.frame.width,height:myCustomCell.frame.height+1)

    myCustomCell.selectedView.setGradientBackground(colorTop:UIColor(hexString: "5c5c5c")!.cgColor, colorBottom:UIColor(hexString: "cfcfcf")!.cgColor)
}

Although when you select a date and switch months it deselect it, is there a way to keep it selected?

@jonfabris hey, i think you should follow the tutorial.
It will help you greatly.
https://patchthecode.github.io/
There are some things you are doing that will cause problems.
check out tutorial1 and tutorial2
If you need more help, just reply on this thread.
Also, if this library has saved you lots of coding, consider leaving a star rating :)
Cheers. 🍺

I went through the tutorial I swear.
I need the date to remain selected even when you switch months.
I think this might be my problem but I can't understand your instructions:
_My views are being randomly repeated across dateCells
To cure your random repeating views -> If you have ever done a UITableView before, then inside the cellForRowAtIndexPath function, you have to reset the cell before it is used right? You will have to remove that circle you have added, or that color you have added. Reset the old text you put earlier, change that image you created etc etc. Therefore you have to do the same thing in your willDisplayCell function._
My best guess was to do this at the top of the function func calendar(_ calendar: JTAppleCalendarView, willDisplayCell cell: JTAppleDayCellView, date: Date, cellState: CellState)
let cell = calendar.calendarView.dequeueReusableCell(withIdentifier: "CalendarCellView") as! CalendarCellView
But calendarView is inaccessible


Add sanity tests for alerts

This PR adds new test cases for kubevirt alert definitions. We expect to see summary, runbook_url,severity, kubernetes_operator_part_of and kubernetes_operator_component for each alert. Those fields must not be empty and some must have specific values.
Signed-off-by: assafad<EMAIL_ADDRESS>None

Hi @assafad, I'm just a stranger volunteered to review this PR. Please resolve my comments and I'll comment ok-to-test.

/ok-to-test

/retest-required

/retest-required

/retest

/retest

/retest

/test pull-kubevirt-e2e-k8s-1.22-sig-monitoring

/approve

/hold
@assafad Can you please check if make generate does some changes and eventually submit them squashed?

/restest

/hold @assafad Can you please check if make generate does some changes and eventually submit them squashed?

@enp0s3 Hi, I did make generate and still some tests are failing. What should I do?

/lgtm

/unhold


Build(deps-dev): Bump @storybook/addon-essentials from 6.4.14 to 6.4.15 (#2601)

Pull request opened by the merge tool on behalf of #2601

Merged in ebbbbb78abedb7b2bac3a5123d06b6da54370865


EAGLE-374: Fix sub-modules of eagle-jpm lack of relativePath tag

https://issues.apache.org/jira/browse/EAGLE-374

LGTM
Download the patch and build passed. I'll merge it soon.


Package for Gentoo Users

The noweb package was removed from Gentoo, but we can add it again, if there are volunteers for testing and maintaining the package and if there are some users, who need this package.
You find the archived, old package on
https://gitweb.gentoo.org/repo/gentoo.git/log/?qt=grep&q=noweb

Version 2.12 is released.  Whether anyone wishes to reinstate the package at Gentoo, I leave to Gentoo.


tokenizers: use line length prefix sum instead of plain array

First of all hi and thanks for your project, it has been really useful!
I noticed however that tokenizing the source code gets rather slow for big source files. After some investigation, I think I figured out the problem: in add_pending(), we subtract the length of all lines before the current one to get the position in the current line. But the problem is, this iteration is quadratic time and this is problematic for longer files.
Instead, it would be better to not have a list of the lengths of individual lines, but a list of the start positions for each line (it is basically a prefix sum). This way we can use simple subtraction instead of a loop, making the algorithm linear. In practice I have observed this makes parsing much faster (I test with files which are >2000 lines long).
I hope you find this change useful :)

This is truly useful. Thanks @ammen99 a lot for your contribution!! I'll release a new version to incorporate this fix.


a11y improvements for components (Part 1)

The aim is to improve overall accessibility across all component. This will be an on-going process that may take several passes.
Part 1
Follow suggestions provided by user "Tropical" on the Svelte Discord

ARIA reference
Accordions

[ ] Consider converting parent to either <details> or <summary>
[ ] Provide keyboard interaction
[ ] Provide screen reader support
[ ] ARIA

Dialogs

[ ] Doesn't observe focus trapping
[ ] Needs accessible labels
[ ] Consider using <dialog>, reference the showModal() method
[ ] ARIA

Light Switch

[ ] Provide keyboard interaction
[ ] Button isn't focusable
[ ] Menu navigation via arrows, home/end
[ ] ARIA

Menus

[ ] Arrow key navigation
[ ] Home/end key navigation
[ ] ARIA

Radio Groups

[ ] Consider role="button" with toggle states
[ ] ARIA

Tabs

[ ] Not semantically valid since <li> are children of <nav>
[ ] Provide screen reader support
[ ] Allow for keyboard focus
[ ] ARIA

Toggles

[ ] Should allow for keyboard focus
[ ] Missing screen reader attributes
[ ] Consider role="button" with toggle states
[ ] ARIA

Part 2+
Review and update other components. Create new tickets for each set.

(reserved)

(reserved)

(reserved)

(reserved)

@endigo9740 Tabbing through the examples in the button page, skips the disabled toggle button.

Hey everyone! So great news regarding accessibility (a11y) updates. I've now completed the first round of updates improving a11y for ALL (yes all) components and utilities within Skeleton. The biggest areas of improvement include tab focus, keyboard interactions, and properly defined ARIA attributes.
This is a rather large and impactful update that touches a little bit of everything in the entire library. Just take a look at the commit history here:
https://github.com/Brain-Bones/skeleton/commits/feat/a11y
However, it does come with a few notable and/or breaking changes:

The Accordian no longer animates due to <details> and <summary> not yet properly supporting this natively. However, my hope is this can return in the future. BUT, a11y comes first!
The List and it's children components have been rewritten from scratch. Plus the NavItem features have been merged into ListItem, so NavItem goes away
Dialogs now utilize proper focus control, such as being able to tab between a prompt input and the cancel/confirm actions.

Given the broad amount of changes, I am calling for your help with testing!
And no, you do NOT have to be an a11y expert to test these updates. Literally anyone that can pull down the branch can test. Mainly, I need help with regression testing. With minor updates in so many places, we need to ensure nothing has broken inadvertently. Just pull down the branch and start browsing the Doc site locally. Test everything you can between each component. Maybe create your own page to test different settings per the components with the most changes.
Long story short, if you're availble to help test, please let me know here, pull down the branch, and keep me posted as you progress. You can report all issues to the ticket above. The sooner testing concludes, the sooner we can get this update out to everyone! And the sooner I can move onto critical issues such as decoupling Skeleton from SvelteKit, new features and updates, etc.
The three attached screenshots demonstrate:

Tab focus on the homepage, which utilizes a variety of different Skeleton components
Colorblind support - though color/contrast/vision testing still needs a lot more work
The axe Dev Tools accessibility score. Again highlighting our biggest gap remaining is vision-based issues

For anyone interested in testing, here's the steps required:

Clone a version of Skeleton to your local device: git clone https://github.com/Brain-Bones/skeleton.git
Checkout the a11y code branch: git checkout feat/a11y
Move into the project directly: cd skeleton
Install NPM dependencies: npm install
Run a local dev server: npm run dev and point your browser to: http://localhost:3000/

Now that you have the project running locally, try and test the following:

Navigate through the entire local site testing each and every component.
Ensure there's no console log errors.
Ensure the component examples work as you would expect
Ensure the documented props match what's shown in the examples above
Review the newly add Accessibility section at the bottom of each component doc page. Verify the ARIA link and all props match expectations
(OPTIONAL) if you're feeling adventurous, create a new blank route page and start importing and testing each component with different props and settings. Especially ARIA attribute props.
(OPTIONAL) if you wish to text a11y of each component, see the list of Chrome extensions at the top of the original post. These are pretty easy to use, but I'll provide a link to a great video intro below:

https://www.youtube.com/watch?v=IaJtAzaFiJ4
Finally, the critical areas of following should be the components with the large amount of updates. These include:

The List/ListItem components - which have been completely rewritten from the ground up! Note that NavItem is no longer present, this was merged into ListItem.
The Accordion - it now uses <detail> and <summary> tags for more native expansion groups. (MDN page)
Dialogs - these now attempt to capture and retain focus per the ARIA guidelines linked at the bottom of the docs page.
For all other changes, see the notes included in the first handful of posts on this ticket!

If you find issues please report this in comments here. Thanks for your help!

Nice work on this so far :)
So doing a side by side comparison between master and feat-a11y and doing a scan with the axe extension for now:
There are two overall ARIA reported issues of colour and heading levels and some minor ones on specific pages.
But this comment focuses on actual Skeleton regressions found in RadioGroup and Slide Toggle no longer working and hence any dynamic preview of other components that use these two components on their docs page also don't work.
http://localhost:3000/components/avatars - Buttons and Slide Toggles no longer working
http://localhost:3000/components/buttons - Slide Toggle for Disabled no longer working
http://localhost:3000/components/data-tables - there are new aria props, but they aren't used in the example and it doesn't get picked up by the axe report.
http://localhost:3000/components/dividers - we have lost functionality to modify the divider
http://localhost:3000/components/progress-bars - Determinate and Height buttons are broken
http://localhost:3000/components/radio-groups - both justify and horz/vert sample don't click
http://localhost:3000/components/slide-toggles - all slide toggles broken

Awesome Nik, thanks. Really strange issues, but I'll take a look!

These changes have been moved into the dev branch but still open to additional testing before release.

Reopening this until @thomasbjespersen can finish QA on it.

Taking this on @endigo9740.

FYI i'm closing this for now. Please keep me posted if you have any issues related to these changes @thomasbjespersen


Simple tutorial: Lumped Resonator Capacitive (Resonator)

What is the documentation modification?
To prepare for #291, we need a simple notebook that shows how to create:
Lumped Resonator Capacitive (Resonator)
This notebook may already be done, if so please point me in the direction of it so I can verify it's what is needed.
Please put the notebook in the placeholder directory docs/tut for now.  You do not need to add the notebook to the docs, I will complete that step as part of #291.

This is done.


No valid distribution for rtkit in pip

I am having some trouble getting pip to install this.

https://github.com/z4r/python-rtkit
pip install python-rtkit
On Mon, Apr 27, 2015 at 11:56 AM, ttk2<EMAIL_ADDRESS>wrote:

I am having some trouble getting pip to install this.
—
Reply to this email directly or view it on GitHub
https://github.com/Civcraft/modmail_ticketmanager/issues/4.


Enable CI test for memtable spill and EpochWithGap

We added feature enable_test_epoch to avoid changing existing test cases after introducing EpochWithGap for memtable spill. This not only makes unit test but also e2e test and simulation test not testing the EpochWIthGap with spilled offset. In this case, we will miss some obvious bugs like #13882.
I suggest we remove the usage of feature enable_test_epoch and enable all tests to use the real EpochWithGap. Though it is cumbersome to modify existing test cases, I think it is still worthwhile to do so. Looking for better ideas as well.

All random epochs in unit test need to be corrected, which may be a bit troublesome and requires a lot of work.
Let's think about whether there are other workarounds. If not, we can only change it. The owner of UT may be required to participate.

Can we just remove this debug assert as current usage is relatively clear?

Can we just remove this debug assert as current usage is relatively clear?

What do you mean? The issue is not about the debug_assert but about low coverage on the release build with the real EpochWithGap being used. Removing the debug_assert may make things worse because bugs like #13882 can happen silently.

Can we just remove this debug assert as current usage is relatively clear?

What do you mean? The issue is not about the debug_assert but about low coverage on the release build with the real EpochWithGap being used. Removing the debug_assert may make things worse because bugs like #13882 can happen silently.

Oh, I misunderstood, you can ignore this comment.


Exception on demo

hi,
When i'm trying to login into your demo i got this error when I'm redirecting to "https://demo.quarxcms.com/quarx/dashboard" :

ErrorException in BaseProvider.php line 178:
file_get_contents(): php_network_getaddresses: getaddrinfo failed: Name or service not known (View: /home/forge/demo.quarxcms.com/releases/1/vendor/yab/quarx/src/Views/dashboard/analytics-internal.blade.php)

When I refresh It's work.

I think it was a broad issue that others were getting too. It seems to have stopped but I'll keep an eye on it thanks.

You're welcome :)


Create notebook with image preprocessing steps using

Used OpenCV to do basic preprocessing, files used were sample data created by me (not user data) to
explore Tesserac capabilites out of the box and with minimum modification.
_Adding a Notebook to the Notebook directory
Change Status

[ x] Incomplete/work-in-progress, PR is for discussion/feedback

Has This Been Tested

[x ] Not necessary

Checklist

[x ] I have commented my code, particularly in hard-to-understand areas
[x ] My changes generate no new warnings

Notebook testing the different pre-processing methods with OpenCV and pass it to Tesseract (vanilla).


feat: add cobra cli library

This PR adds cobra as the cli parsing library. https://github.com/blinklabs-io/bursa/issues/63

I have copied the way cardano-up has used cobra more or less exactly
The cli currently has the same ability as before. I wanted to start some better cli then "cli" but felt that it would be better to have some more discussion before implementing it. I will add my thoughts/ideas there
I removed internal/logging in favour of internal/consolelog also stolen from cardano-up
I needed to remove some loggig output from the api, because i dont know gin and did not spend time figuring out how to adapt slog to work with it. Also: I would like to suggest to remove gin and use  go's 1.22 new stdlib net/http muxer instead. Removing the dependency alltogether. The api seems fairly simple so i think net/http is all we need. But i will open. another ticket for that discusson.

Looks like you have conflicts with main.

Looks like you have conflicts with main.
Oh snap, yes. I have merged these changes now and updated the commits.
I also just saw that cobra still provided its default "generate shell completion" script, which i now have disabled.

Back out the changes to remove the internal/logging module. I'm fine with the introduction of slog for console logging, but we need the access logs in the API to stay.

I have removed the console logging again in favor of internal/logging. I'll look into gin and how to have access logs with log/slog so we dont lose these when switching to it.


feat(DTFS2-6182): replace graphql update lead underwriter in tfm with rest

Codecov Report

Merging #2149 (83e80c8) into feat/DTFS2-6182/replace-graphql-query-facility-in-tfm-with-rest (218383d) will decrease coverage by 11.89%.
The diff coverage is 0.00%.

@@                                         Coverage Diff                                         @@
##           feat/DTFS2-6182/replace-graphql-query-facility-in-tfm-with-rest   #2149       +/-   ##
===================================================================================================
- Coverage                                                            19.87%   7.99%   -11.89%
===================================================================================================
  Files                                                                 1234     807      -427
  Lines                                                                28556   18938     -9618
  Branches                                                              4539    2741     -1798
===================================================================================================
- Hits                                                                  5676    1514     -4162
+ Misses                                                               20068   15403     -4665
+ Partials                                                              2812    2021      -791

Flag
Coverage Δ

api-test
?

ui-test
40.26% <ø> (-25.12%)
:arrow_down:

unit-test
2.25% <0.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

trade-finance-manager-api/src/v1/swagger.js
0.00% <ø> (ø)

trade-finance-manager-api/src/v1/routes.js
0.00% <0.00%> (ø)

...-manager-api/src/v1/controllers/deal.controller.js
0.00% <0.00%> (ø)

... and 427 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Closing entity on a different thread to the entity fetch causes an IllegalMonitorStateException

java.lang.IllegalMonitorStateException
	at com.tc.object.locks.ClientLockImpl.release(ClientLockImpl.java:794)
	at com.tc.object.locks.ClientLockImpl.unlock(ClientLockImpl.java:166)
	at com.tc.object.locks.ClientLockManagerImpl.unlock(ClientLockManagerImpl.java:198)
	at com.terracotta.connection.entity.MaintenanceModeService.unlock(MaintenanceModeService.java:85)
	at com.terracotta.connection.entity.MaintenanceModeService.readUnlockEntity(MaintenanceModeService.java:46)
	at com.terracotta.connection.entity.TerracottaEntityRef$1.run(TerracottaEntityRef.java:81)
	at com.tc.object.ClientEntityManagerImpl.internalRelease(ClientEntityManagerImpl.java:422)
	at com.tc.object.ClientEntityManagerImpl.access$500(ClientEntityManagerImpl.java:72)
	at com.tc.object.ClientEntityManagerImpl$3.run(ClientEntityManagerImpl.java:374)
	at com.tc.object.EntityClientEndpointImpl.close(EntityClientEndpointImpl.java:211)

While the idea still requires more planning, this is probably best solved along-side the various issues we have had with maintenance mode by changing the client-entity dynamic from a distributed read-write lock to something more similar to reference counting (although more complex than that since it needs to handle disconnect, etc, and how to suspend a blocking destroy message).

Assigning to @myronkscott as this will likely be either solved or become moot as a result of his current items.


fix: show keep_crv fee [WEB-413] and [WEB-408]

Edit: fixed Not sure how to truncate keep_crv amount, let me know and I will update the PR. //cc @dudesahn
I've also included the fix for WEB-408 as they both rely on the same fix.

keepCRV for triCrypto is 0 (which I caught thanks to this PR), but it would be great for any == 0 that we still display the value as equal to 0%

cvx_apr values should be displayed as a percentage. So, a cvx_apr value of 0.07625203502738 would be 7.63%.

One last fix: only show keep_crv for Curve vaults. So, should be able to do a check that apy type is crv

ok, let me know if it's ok now.
On Tue, Jun 22, 2021 at 10:19 PM dudesahn @.***> wrote:

One last fix: only show keep_crv for Curve vaults. So, should be able to
do a check that apy type is crv
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/yearn/yearn-finance/pull/371#issuecomment-866224263,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAGOQ44GSMBHTRRLCWD5TDTUDH3DANCNFSM47DUQPTA
.

LGTM @patcito, I'll just defer to @x48-crypto to double-check the code itself


Feature request: custom additional properties

Hi,
it could be nice to be able to add custom properties, referable in the template.
An example could be (but please feel free to change as you prefer):
semversioner add-change --property release.maintainer="Me" --property change.restart=true property change.limit=90.4

This could generate something referable in the template as:
{{% release.maintainer %}}
{{% change.restart %}}
{{% change.limit %}}

This is useful to be able to embed more structured data, and to generate more complex changelogs.
For example, I have a project consisting of various components, each versioned with its own version number.
So the changelog is structured in sections, one for each component.
So I could use the custom properties to be able to put the information in the correct section (e.g. --property change.component="my component").
Regards

Hi @FStefanni ,
Semversioner 2.0 already supports this use case. For example, you could do:
semversioner add-change --type patch --description "My custom changelog message with attributes." --attributes pr_id=322 --attributes issue_id=123

and create a custom changelog template as follows:
# Changelog
Note: version releases in the 0.x.y range may introduce breaking changes.
{% for release in releases %}

## {{ release.version }}

{% for change in release.changes %}
- {{ change.type }}: {{ change.description }}{{ ' (#' + change.attributes.pr_id + ')' if change.attributes }}{{ ' (J' + change.attributes.issue_id + ')' if change.attributes }}
{% endfor %}
{% endfor %}

More details in the README.md documentation. Feel free to give it a try and share your feedback.
Thanks for sharing your use case and thoughts.
Regards,
Raul


TypeScript Support

It would quite lovely to have support for TypeScript style checks.
Currently there is tslint.

If this is desired, I might try and do this in my spare time if no one else picks it up.
It seems there is no gem based wrapper for tslint. Given that, I am interested in what might be an ideal way to implement a StyleGuide::TypeScript class. Attempting to create a gem to wrap tslint seems like the most prudent path. :-)

+1

We don't have plans in the near future to support TypeScript.


[WIP] Cypress test. Search task feature.

Motivation and context
Cypress test. Search task feature.
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage increased (+0.02%) to 62.241% when pulling c358cc0f76ca0bb2a9b155f21a8abe0602e08dec on dvkruchinin:dkru/cypress-test-search-task-feature into 634e409792d7049dee18a952e7cc67e1041d4f0a on openvinotoolkit:develop.

Added several options for searching for an issue.


Fix UC function schema when no params are needed

PR Checklist

[ ] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added or modified a feature, documentation in docs is updated

Description of changes
With the latest langchain version, if the UC function has no params then the schema is not correctly parsed. We should use create_model instead of BaseModel, added test for this.

@BenWilson2 for review :D

@dennyglee this is good to go for a bug fix


Adding All GHSA ECO Systems To Trivy DB

Hi,
From what I can see here https://github.com/aquasecurity/trivy-db/blob/main/pkg/vulnsrc/ghsa/ghsa.go
you don't have support for Go, Erlang, Rust.
Can you please add these ecosystems in order to get their security data from GHSA?
Thanks

We use OSV. I guess OSV covers more. Please correct me if I'm wrong.
We're working on Rust now.
https://github.com/aquasecurity/vuln-list-update/pull/174


Make sure we fetch deleted records from the Sierra API

Inside the Sierra reader, we make a query like this:
GET /items?updatedDate=[2022-07-16T23:59:00+00:00,2022-07-20T00:01:01+00:00]

which fetches all the records with a modifiedDate between those two times.
This was meant to be fetching deleted records also, but this doesn't seem to work any more – e.g. we know i19460806 has a deletedDate of 2022-07-18, but it doesn't appear in the output of this query.
We need to make a completely different query for deleted records:
GET /items?deletedDate=[2022-07-16,2022-07-20]

I'm not sure if this has never worked, or if it's broken at some point, but with the current version of the Sierra API and our reader, we aren't picking up deletions.

The Sierra reader seems overly complicated, and I don't want to fiddle with it too much – I think we should consider simplifying it, e.g. publishing directly to SNS, skipping S3, and removing the S3 demultiplexer, before trying to pile in more complexity for dealing with deleted records.
I've done a one-off fix for the problem somebody has spotted – fetching the deleted records manually and feeding them into the demultiplexer.
We should fix this properly, but it's a big enough piece of work that it should go through planning.

There are several differences between now and when we wrote the original Sierra reader:

Sierra is now hosted by III in the cloud, not in on-prem infrastructure. IME it's pretty reliable and we might be able to get away with ditching some of the guard rails we put in place for (imagined) reliability issues.
We're no longer running Encore/OPAC for the public, so the Sierra database is under less load than it used to be.
We now deal with Sierra patrons, and we have a TypeScript implementation of the Sierra API. If we were building from scratch, the reader part might be in TypeScript instead of Scala.

See https://wellcome.slack.com/archives/C8X9YKM5X/p1678968410927129

We need a better solution for this long-term, but for now when people highlight individual items that need to be removed, I post something like:
{
  "id": "1735809",
  "data": "{\"deletedDate\": \"2021-09-01\", \"deleted\": true,\"id\": \"1735809\"}",
  "modifiedDate": "2021-09-01T00:00:00Z",
  "bibIds": []
}

to the items demultiplexer output topic in SNS.


PluginManager.getPlugins() not finding MySQL (testing TiDB support)

Which version and edition of Flyway are you using?
8.4.0 Community Edition
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Java API
Which database are you using? (Type & version)
5.7.25-TiDB-v5.2.3
Which operating system are you using?
*nix
What did you do? (Please include the content causing the issue, any relevant configuration settings, the SQL statement(s) that failed (if any), and the command you ran)
initializing Flyway through Java
What did you expect to see?
FLyway migration successful
What did you see instead?
`org.flywaydb.core.api.FlywayException: No database found to handle jdbc:mysql://<IP_ADDRESS>:1234/databasename
at org.flywaydb.core.internal.database.DatabaseTypeRegister.getDatabaseTypeForUrl(DatabaseTypeRegister.java:55) ~[XXX.jar:?]
at org.flywaydb.core.internal.jdbc.DriverDataSource.(DriverDataSource.java:109) ~[app.jar:?]
at org.flywaydb.core.internal.jdbc.DriverDataSource.(DriverDataSource.java:80) ~[app.jar:?]
at org.flywaydb.core.api.configuration.ClassicConfiguration.setDataSource(ClassicConfiguration.java:958) ~[app.jar:?]
at org.flywaydb.core.api.configuration.FluentConfiguration.dataSource(FluentConfiguration.java:683) ~[app.jar:?]
I have tried to track this down. I am confused that MySQL and TiDB plugins do not seem to be detected at all?
When it does database support check I expected to see TiDBDatabaseType there, but its not present. Neither is MySQL. See attached screenshot of debugging DatabaseTypeRegister.getDatabaseTypesForUrl()
screenshot of detected databases
am I doing something wrong or is this a bug?

Have you included both the flyway-community-db-support and flyway-mysql dependencies?

I did not! That fixed it right away, thanks!
Perhaps you could put a big warning somewhere since it wasn't required pre-8 . Sorry to waste your time


Add helm chart

Add a helm chart for deploying the service to kubernetes

This is not yet closed by #71 ?


Create ecole-navale

@muriloksantos Could you please rename lib/domains/fr/ecole-navale to lib/domains/fr/ecole-navale.txt because .txt extension is mandatory in this repository. Thank you!

Really sorry about that, I read your instructions but forgot to follow it. Sorry!

@muriloksantos No problem, thank you! Pull request merged.


add support for turtle (.ttl) files

Turtle format is defined by W3C, and is one of the well-known formats for RDF encoding.
As in Python, comments are simply # , and following space is not mandatory, but is to me a readability gain.

I'd be happy to add the support for support for Turtle, but can you please drop the second commit with the space here before I merge this? All of the comment delimiters do not include spaces. Those who prefer them for the readability (myself for one) can set this using a configuration option:
let g:NERDSpaceDelims = 1

Well, that's done !


Usability improvement: declaring exclusive content of repositories

Context
Fixes #9515
See commits for details.
Example syntax:
repositories {
    // This repository will _not_ be searched for artifacts in my.company
    // despite being declared first
    jcenter()
    exclusiveContent {
        forRepository {
            maven {
                url "https://repo.mycompany.com/maven2"
            }
        }
        filter {
            // this repository *only* contains artifacts with group "my.company"
            includeGroup "my.company"
        }
    }
}

Just wondering, can the DSL be like the following?
repositories {
    // This repository will _not_ be searched for artifacts in my.company
    // despite being declared first
    jcenter()
    maven {
        url "https://repo.mycompany.com/maven2"
        exclusiveContent {
            // this repository *only* contains artifacts with group "my.company"
            includeGroup "my.company"
        }
    }
}

Just wondering, can the DSL be like the following?

In theory, yes, it could, but I think it wouldn't be good for 2 reasons:

you're saying something about multiple repositories, not just one, so putting it within the repository declaration is misleading
it would force keeping state about all repositories in every repository instance (each repository would need to keep track of the other repositories created with the same container), which is not great.

you're saying something about multiple repositories, not just one, so putting it within the repository declaration is misleading

That depends.
I've been smoke-testing my Gradle plugin, and now I have the following:
    repositories {
        // Here's a repository with staged plugin:
        maven("https://oss.sonatype.org/content/repositories/comgithubautostyle-1012")
        // All the rest plugins
        gradlePluginPortal()
    }

I was trying to specify like "do not try resolving com.github.autostyle from plugin portal", and I tried to use IDEA's autocomplete for that.
The problem was Gradle was trying to fetch each plugin from comgithubautostyle-1012 which took time.
Unfortunately, the exclusive content feature is not there, so I ended up with just reordering the repositories:
    repositories {
        gradlePluginPortal()
        maven("https://oss.sonatype.org/content/repositories/comgithubautostyle-1012")
    }

Of course, it is not a big deal for me, however, as a end-user, I expect that I could specify "well, this group/artifact/version can only be present in this repository, don't even try the others".
It is kind of sad that your suggested exclusiveContent is not a trivial flip when one wants to add an exclusive content filter to an already present repository in code.
Well, it might be alleviated by adding IDEA's intentions to flip between the two representations.
However, it is sad that "adding exclusive filter" does not look like just adding exclusiveContent {...}

Note: what if filters overlap?
AFAIK it would be ok (useful and understandable) to have the following:
repositories {
    // This repository will _not_ be searched for artifacts in my.company
    // despite being declared first
    jcenter()
    maven {
        url "https://repo.mycompany.com/maven2/releases"
        exclusiveContent {
            includeGroup "my.company"
        }
    }
    maven {
        url "https://repo.mycompany.com/maven2/tools"
        exclusiveContent {
            includeGroup "my.company"
        }
    }
}

The logic as follows: if an artifact matches at least one "exclusive" filter, then it must be resolved from the matching repositories in the order of their declaration.
In other words, my.company:tool is resolved from releases repo first, then from tools.
It does sound composable with the existing content { .. } feature, and it is discoverable with auto-complete inside maven {...} block.

it would force keeping state about all repositories in every repository instance (each repository would need to keep track of the other repositories created with the same container), which is not great.

Do you really think so?
I guess the implementation could be like:

If repository is using non-trivial exclusive content filter, then it is added to the list of repositories with exclusive content activated.
At resolution time, Gradle could check if actual GAV matches one of the exclusive content filters
If there's a match, then Gradle uses just matching exclusive repositories. Otherwise it uses regular resolution.

I do not see why cross-references between repositories are needed.

If a repository is using a non-trivial exclusive content filter, then it is added to the list of repositories with exclusive content activated.

To be added to such a list it would need to be passed the list or its container. But again, I don't think it's a good idea to put at the repository level something that says something about all repositories.

repository level something that says something about all repositories.

What is the solution for the following then?
repositories {
    // This repository will _not_ be searched for artifacts in my.company
    // despite being declared first
    jcenter()
    mavenCentral()
    google()
    maven {
        url "https://repo.mycompany.com/maven2/releases"
        exclusiveContent {
            includeGroup "my.company"
        }
    }
    maven {
        url "https://repo.mycompany.com/maven2/tools"
        exclusiveContent {
            includeGroup "my.company"
        }
    }
}

It is kind of sad that your suggested exclusiveContent is not a trivial flip when one wants to add an exclusive content filter to an already present repository in code.

I don't understand what you mean here. The repository in forRepository doesn't have to be created in the forRepository block:
repositories {
   val myRepo = maven { ... }
   exclusiveContent {
      forRepository { myRepo }
      filter { ... }
   }
}

repositories {
    // This repository will _not_ be searched for artifacts in my.company
    // despite being declared first
    jcenter()
    mavenCentral()
    google()
    maven {
        url "https://repo.mycompany.com/maven2/releases"
        exclusiveContent {
            includeGroup "my.company"
        }
    }
    maven {
        url "https://repo.mycompany.com/maven2/tools"
        exclusiveContent {
            includeGroup "my.company"
        }
    }
}

I would argue that in this case the content is not exclusive, because precisely it can be found in multiple repositories. One problem today with composing exclusivity like this is that the overlapping of filters. For regular expressions typically you can't tell if it's going to match another (well you can but then I need to switch jobs for a few weeks to rewrite an automaton library).
On the other hand it should be pretty straightforward to implement it with the version I suggested, by allowing more than one repository:
repositories {
    exclusiveContent {
        forRepository { repo1 }
        forRepository { repo2 }
    }
}

I think it would be a good addition, actually.

The repository in forRepository doesn't have to be created in the forRepository block:

Option 1:
repositories {
   maven { ... }
}

// if an exclusive filter is needed, the following must be typed:

repositories {
   val myRepo = maven { ... }
   exclusiveContent {
      forRepository { myRepo }
      filter { ... }
   }
}

Option 2:
repositories {
   maven { ... }
}

// vs

repositories {
   maven {
      ...
      exclusiveContent {
      }
   }
}

What I mean is it requires multiple actions: "declare a variable (invent name), add exclusiveContent, add forRepository, add filter", and only after that one can really add a filter.
If exclusiveContent block can be added right under repository declaration, then it would be more user-friendly.

If exclusiveContent block can be added right under repository declaration, then it would be more user-friendly.

Let me disagree on this. I think it's wrong to state at the repository level something which influences the resolution in other repositories. I understand why you'd like that but I think it's not a good idea.

repository level something that says something about all repositories

You still use exclusive content repository wording in the javadoc for exclusiveContent  though: Defines the an exclusive content repository.
I see that you mean "it is better to avoid silent use of global variables", "it is better to avoid global consequences from configuring local entities".
However, the current PR does not enforce that exclusiveContent is declared the first, so it is not that different from repository level something that says something about all repositories

Exclusive content doesn't have to be declared first. If you look at the tests I have actually tested the 2 orders. This is because it relies on repositories.all which doesn't care when the repository is created.
I'd appreciate if you could come up with a test case which shows the order is relevant, if not too convoluted.

I'd appreciate if you could come up with a test case which shows the order is relevant

Oh, what I meant was even current design allows to declare "exclusiveContent" at the very end of the repository list, and it would influence the resolution of repositories that come before and/or after that.
In other words, even current implementation looks like having at the repository level something that says something about all repositories.

Here's an issue for IntelliJ IDEA to suggest users adding exclusiveContent for non-standard repositories: https://youtrack.jetbrains.com/issue/IDEA-299272


Source Discrepancy

Describe what happened
Update the SDK to the latest RC (1.0.0-rc3) and noticed the source that shows in Datadog for iOS logs is react-native while for Android, the source is Android.
iOS

Android

Steps to reproduce the issue:

Follow the setup steps outlined in the README
Send any log from your React Native iOS and Android apps
Noticed the difference.

Describe what you expected:
All my logs are coming from React Native so I would expect the source to be react-native for both platforms. This didn't use to be a problem on the previous version of the SDK we were using (1.0.0-beta3)
Additional context
- Version of the SDK: 1.0.0-rc3

Hello @bgneu. Thanks for noticing that! Indeed, it should be react-native for both. We will fix that bug.

Hello @bgneu. We just published version 1.0.0-rc4 which fixes the issue you've faced.


FieldNameRenderer missing features and other expected behaviors

Category
[x] Enhancement
[x] Bug
[ ] Question
Version
v1.7.0
Expected / Desired Behavior / Question
I would like the FileNameRender control get the next behavior
-DoubleClick is not customizabled -I would like to get the same behavior of a single click  achieved in the 1.7.0 version where the default behavior is suppressed
-Share and Show Action menus are gone, desired behavior will be to leave them at it comes unless a customization is possible
-Circle next to the Icon is selected when the link is clicked, default un-selected behavior is desired
Observed Behavior
Explained above
Steps to Reproduce
Install the 1.7.0 version and customize the render sub in the *.tsx file as follow
@override
public render(): React.ReactElement<{}> {
return (
<FieldNameRenderer
text={"mylink"}
isLink={true}
filePath={null}
isNew={false}
hasPreview={false}
onClick={() => this.button_click()}
/>
);
}
public button_click(): any {
window.open("MyUrlHandlerProtocol:" + window.location.protocol + "//"

window.location.hostname + this.props.listItem.getValueByName("FileRef"),"_self");
return "MyUrlHandlerProtocol:" + window.location.protocol + "//"
+ window.location.hostname + this.props.listItem.getValueByName("FileRef");
}
}

Submission Guidelines

 hasNew     option if it is not specified should detect if this is a new file and show the default new icon.

 hasPreview  do not know what this does, please explain

Thanks!

@AJIXuMuK can you take this one?

Sorry, @estruyf. Lost the notification in email.
Hi @ralftokada.
Point by point :)

DoubleClick - I will be working on it to add ability to override double-click behavior.
Share and Show actions. It's really hard to do that. There is not problem to add the icons itself. But unfortunately, there is no way to initiate standard behavior. It means that it should be "duplicated". To my mind it's to complex and hard to maintain.
Circle next to item. If you click directly to the text, the row is not selected. So, the behavior is the same as it is for OOTB control. If you want to prevent selecting row when you click anywhere in the cell you need to add onclick event handler to event.domElement in your Field Customizer and stop propagation there.
isNew. The component itself is designed to information. It doesn't communicate to SharePoint directly to get properties. It should be done from the entity that is using the component. That's why there is FieldRendererHelper class that actually detects if the item is new:

isNew: SPHelper.getRowItemValueByName(listItem, 'Created_x0020_Date.ifnew') === '1'

hasPreview. I will need to update documentation on that one. Basically, this flag impacts on how the link is configured for the default behavior (if onClick is not provided): if hasPreview is set to false than the url will be equal to filePath property which will lead to direct download of the file; if hasPreview is set to true that the url will look like #id=${filePath}&parent=${filePath.substring(0, filePath.lastIndexOf('/'))} - it will notify core engine to show the preview when the link is clicked.

Let me know if the explanation makes sense of if you have any additional thoughts on that!
In a meanwhile I'll be working on double click and documentation for hasPreview flag.

@estruyf please, assign it to me.

Hi @AJIXuMuK
Great thanks, stopping the double click default behavior/propagation and execute it as the single customized click behavior will make a significant progress.
For me if hasPreview is set to true and notify the core engine to be displayed regardless of whether onClick is provided or not, this is the desired behavior.
Thanks again for keeping up with this development

Closing the issue as version 1.8.0 has just been released.


Password input value resets when toggling visibility on iOS

Description
When toggling the visibility of a password input field using an eye icon, the input value is reset. This issue occurs when switching between type="password" and type="text".
CodeSandbox/Snack link
https://snack.expo.dev/@y.yano/gluestack-ui?platform=ios
Steps to reproduce
Steps to Reproduce

The initial type of the password input field is "password".
Enter a password.
Next, click the eye icon to change the type to "text".
Continue entering the password.
Click the eye icon again to change the type back to "password".
Enter one more character, and all previous input is reset, requiring you to start typing from the beginning.

https://github.com/user-attachments/assets/bcb257b6-7254-4aa5-849b-803164945414
gluestack-ui Version
    "@gluestack-ui/config": "^1.1.19",
    "@gluestack-ui/themed": "^1.1.38",
    "@gluestack-style/react": "^1.0.57"

Platform

[ ] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[ ] Android
[X] iOS

Other Platform
No response
Additional Information
This issue occurs only on iOS and does not happen on Android or Web.
The sample code used in Snack is almost identical to the example code provided on the following page:
https://gluestack.io/ui/docs/components/forms/input

@yanoyuki This is default react-native TextInput behaviour.
You can check here in this expo snack. https://snack.expo.dev/bhK1tUd4IXwXu-U0Pof_N?platform=ios

Thank you for your response.
So, this is not an issue with Gluestack but rather a problem with React Native and iOS.
Is there any workaround to address this iOS behavior?

@yanoyuki You may be want to check this out https://github.com/facebook/react-native/issues/35668

@makkarMeenu
As mentioned here,
textField.clearsOnBeginEditing = NO;

I understand that this resolves the issue, but it seems there is no way to directly avoid this from React Native without touching the native code.


Caravel uninstall

Hi,
I tested caravel for a while. I would now like to remove it from the test server.
how can i shut down caravel properly?
how can i uninstall caravel properly?

You uninstall like any other python application, you stop it depending on how you started it...

I started it following the "Caravel installation and initialization" guide on caravel.io
I am not familiar with python, does pip uninstall caravel will remove all component of caravel, including the database caravel uses internally ?

@hugocar pip uninstall will remove whatever pip installed, so no, you have to take care of everything else yourself.

Just to be sure, by "database caravel uses internally" i meant the database (if any) that caravel uses to manage users, roles, slices, dashboards etc... not my own stuff.
I will spend to time to identify whatever services and files are left after the pip uninstall
but it would be a nice addition to the "installation guide"
Thank you for Caravel and your help today


Add ReduceProd ONNX Import

Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
#1894, #1714
Changes
Adds support for ONNX ReduceProd burn-import in an attempt to understand the structure better.
Testing
cargo test -p burn-import
cargo test -p onnx-tests

Hi there, hopefully this is all correct. Most seems to be very similar to the other reduce ops anyway.
onnx-tests is currently failing with 900.0001 != 900, probably just due to floating point multiplication shenanigans. Does anyone have some advice as to how to go about this problem?

Hi there, hopefully this is all correct. Most seems to be very similar to the other reduce ops anyway.
onnx-tests is currently failing with 900.0001 != 900, probably just due to floating point multiplication shenanigans. Does anyone have some advice as to how to go about this problem?

You can change precision in assertion.
Currently, however, it's failing to due missing comma in generated code:

Should be better now.
run-checks all is failing at tests::jit::remainder::tests::should_be_zero but it's failing on main as well for me so it looks to be unrelated to this PR?


Need ability to save scanned spatial mesh data (HoloToolkit parity)

Overview
HoloToolkit enabled saving scanned spatial mesh data (via a network connection back to Unity). MRTK should have a similar (possibly different  process) to do the same.

MRTK v2 is receiving support for using 3D models in the editor to simulate the spatial mesh source  (#5128).
It supports loading .room files. The recommendation for acquiring mesh data from physical spaces is to download it from the Windows Device Portal (.obj file format).


Wrong line numbers in the output

The way we handle line endings messes up the line numbers that are getting printed, it needs to be fixed.

@ameshkov I suggest using FilterListParser directly, because it already handles lines and line endings internally


add unreachable constructor to ReceiveException/check for EHOSTUNREACH

This looks good. Can you rename the data constructor to ReceiveHostUnreachable?  This will make it more consistent with the name of the existing Exception in ConnectException, and it makes it more clear that we’re taking about EHOSTUNREACH rather than ENETUNREACH.

done


[Question] - Nearest feature point on click?

Using hit tests, is there a way to find a nearest feature point to where user touched the screen, and then attach an object to that point?

Hi @markoradak, to clarify, after setting the initial position object , if you set dragType to "FixedToWorld", that will let the user drag the object to the nearest feature point. So basically we do support your use case with dragging but not yet tapping.

Hi @VikAdvani, yeah - I got it to work with dragging, but was wondering if tapping is supported as well. Do you want me to leave this issue open for future reference, or you would prefer to close it?

Hi @markoradak, leave this issue open for now as we are adding support for tapping. Thanks!


Fix PHPDoc for $code

I found some errors about $code in PHPDoc.

Thanks but this is a read only split and mixed already contains null.


Changed the function bicgstb.

Hi Lars,
I changed the function bicgstb:
The problem: if M2() uses the same memory then p_hat and s_hat will be mixed. The result is a decrease in residual inside bicg but outside the residual remains large.
The change leaves the method as is but seperates p_hat and s_hat. Also, the vector s is no longer needed (less memory needed). The same is implemented in the block version of BiCGSTAB.
Please note that the test for the package sometimes fails - it's not BiCGSTAB's fault.
Eran

Coverage decreased (-0.1%) to 97.818% when pulling 233af729cee857047f6e9d9b44a4a68377f37bb8 on erantreister:master into e79c55d93fe873da1b803c8c55e777bb0a201300 on lruthotto:master.

Thanks for catching this!