Update to handle early exit and optionals

I have tried to remove nil handling with guard at multiple places. Please let me know if you need any other changes.
@gribozavr  :)

@gribozavr Please make sure to run the performance tests before accepting this pull request. This pull request is touching a few critical components and I want to make sure that we don't regress the performance of the swift. I'll add the usual disclaimer that performance regressions could be the result of a missing optimization in the compiler and not a problem in the standard library, but we still don't allow these commits to go in.

@nadavrot OK.

@gribozavr I think it is not compiling I got error. We have COpaquePointer.
/Runtime.swift.gyb:250:9: error: initializer for conditional binding must have Optional type, not 'COpaquePointer'
guard let result = _swift_stdlib_atomicLoadPtrImpl(
^
ninja: build stopped: subcommand failed.
utils/build-script: command terminated with a non-zero exit status 1, aborting


Firestore plugin support for FieldValue.delete() and FieldValue.serverTimestamp() values

Description

Allow users to define key path for FieldValue.delete() and FieldValue.serverTimestamp() values.
delete() value is only valid for update operation, as Firestore does not seem to support it for any other ops.
serverTimestamp() is valid for all operations excluding get and delete operations.

Note

FieldValue.delete() value cannot be used to delete array elements. To delete array elements FieldValue.arrayRemove() needs to be used. https://firebase.google.com/docs/reference/android/com/google/firebase/firestore/FieldValue#arrayRemove(java.lang.Object...)
FieldValue.serverTimestamp() value cannot be used inside array. It throws the following exception:
Caused by: com.google.cloud.firestore.FirestoreException: FieldValue.serverTimestamp() is not supported inside of an array.
Capturing the support for array ops here: https://github.com/appsmithorg/appsmith/issues/3770

Fixes #3475
Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?

Added Junit TCs
Manual

https://user-images.githubusercontent.com/1757421/112421483-1ccf0d00-8d55-11eb-92e3-8aeb6c1f1836.mov
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Already made good use of this feature, thx!


Some chart data labels have lots of decimals

Nitpicking but could we introduce some max for the number of decimals?
(Btw, I guess it's not deploy critical so didn't add it to the deploy milestone.)

I haven't had any luck reproducing this. Tooltip values for the stacked bar chart are already limited to a maximum precision of 2 decimal places; I cannot explain the screenshot.
If it happens again, can you save the scenario and post the link?

I haven't had any luck reproducing this. Tooltip values for the stacked bar chart are already limited to a maximum precision of 2 decimal places; I cannot explain the screenshot.
If it happens again, can you save the scenario and post the link?

Sure, let's close this issue for now.

Sure, let's close this issue for now.


WidcommDecoratorNetworkStream.cs not found

Error	CS2001	Source file '...\32feet\ITH.Net.Personal.FX4..\InTheHand.Net.Personal\Net.Bluetooth.Widcomm\WidcommDecoratorNetworkStream.cs' could not be found.

+Error	CS2001	Source file '...\32feet\ITH.Net.Personal.FX4..\InTheHand.Net.Personal\Net.Bluetooth\BLUETOOTH_DEVICE_INFO.cs' could not be found.

Error	CS2001	Source file '...\32feet\ITH.Net.Personal.FX4..\InTheHand.Net.Personal\Net.Bluetooth\BLUETOOTH_FIND_RADIO_PARAMS.cs' could not be found.

Error	CS2001	Source file '...\32feet\ITH.Net.Personal.FX4..\InTheHand.Net.Personal\Net.Bluetooth\BLUETOOTH_RADIO_INFO.cs' could not be found.

Hi, have you solved you problem?

I had the same issue.  Got around it by removind WidcommDecoratorNetworkStream.cs from the project.
Added the other 3 files (BLUETOOTH_*) as links from the Net.Bluetooth.Msft folder.

I'm working on a project that has to run on Windows 7 through Windows 10.  My target is .NET 4.6.1.
I downloaded the repo and added the FX4 build to the InTheHand.Net.Personal solution.  After applying the fixes in my comment above, I was able to build and run the C# packages.  Not much else builds, including the unit test project for the FX4 project.
To get around the build issues, I have a solution file that only builds the FX4 project.
My questions, then are whether anybody else has been able to get a clean build for the FX4 project and can we get the necessary changes incorporated into the root?
Thanks.

There is some work going on in parallel to this to create a new solution which builds in modern versions of VS and removes obsolete code. From Windows 7 the relationship between the Widcomm SDK and Microsoft's own Bluetooth stack and therefore it plugs into the Win32 API so the separate Widcomm dependencies are not required. I'm building the code in a private repository because it involves breaking and moving a lot of the code files but when it is complete enough to be shared I'll move it into this repository alongside the legacy codebase. This work will also make it easier to broaden the platform support. While Win32 is the priority I'm hoping to get Android, Linux, macOS, UWP and others into the mix.

That sounds good.  I'll hold off doing a pull request for my fixes then.
Thanks.


Fixed #26687 -- Made i18n test not use hardcoded path separator.

This fails on Windows when the test run time conditions are such that
the assertion is really executed.

buildbot, test this please.

I've installed gettext on the Windows machine and saw this test fail on another build. Now it's passing although there's still one more unrelated failure to solve separately to get the build green.


"App is running in background, doing heavy tasks!" appears randomly when silent : true

I'm having the same trouble as @M4ttscx in the issue #341. It seems to appear when the OS closes the background service to save resources.
Here is a log of the app when the error occurs:
03-15 10:18:50.835 24220-24220/com.example.app D/CordovaWebViewImpl: >>> loadUrl(javascript:cordova.plugins.backgroundMode._isActive=false;cordova.plugins.backgroundMode.fireEvent('deactivate',null);cordova.plugins.backgroundMode.ondeactivate(null);)
03-15 10:18:51.302 24220-24220/com.example.app I/Timeline: Timeline: Activity_idle id: android.os.BinderProxy@146095dd time:327335369
03-15 10:19:07.747 24220-24220/com.example.app D/CordovaActivity: Paused the activity.
03-15 10:19:07.762 24220-24220/com.example.app D/CordovaWebViewImpl: >>> loadUrl(javascript:cordova.plugins.backgroundMode._isActive=true;cordova.plugins.backgroundMode.fireEvent('activate',null);cordova.plugins.backgroundMode.onactivate(null);)
03-15 10:19:08.558 24220-24220/com.example.app D/CordovaActivity: Stopped the activity.
03-15 10:20:40.902 24220-24220/com.example.app D/LenovoAppIconTheme: ExtraResources;cleanCachedIcon;clear cache..
 PID change -->  03-15 10:20:44.065 26136-26136/com.example.app D/LenovoAppIconTheme: ExtraResources;cleanCachedIcon;clear cache..
03-15 10:20:44.076 26136-26136/com.example.app D/Proxy: setHttpRequestCheckHandler
03-15 10:20:44.081 26136-26136/com.example.app I/art: jiangtaottt OpenDexFilesFromOat open_oat_file.get() == nullptr /data/app/com.example.app-1/base.apk
03-15 10:20:44.091 26136-26136/com.example.app I/art: jiangtaottt OpenDexFilesFromOat oat_location == nullptr /data/app/com.example.app-1/base.apk
 Error -->  03-15 10:20:44.092 26136-26136/com.example.app E/System: elements.add : /data/app/com.example.app-1/base.apk  /data/app/com.example.app-1/base.apk   /data/app/com.example.app-1/base.apk
03-15 10:20:54.061 26136-26136/com.example.app D/LenovoAppIconTheme: ExtraResources;cleanCachedIcon;clear cache..

As you can see, the PID changes from 24220 to 26136 in that moment. It is necessary to say that it also makes the devices very slow.
Environment Details:
Cordova: 7.1.0
Plugin version: 0.7.2
Android Version: 4.4, 5.1, 6.0 and 7.1.1
PD: If you know a quick fix to this problem, I would thank you forever

I had the same problem. Couldn't understand how it happens, because it happens absolutely randomly. Even running in silent mode has no effect - no notifitation, but that nasty "heavy tasks" appears any way. So I did the following:

Run in silent mode (setting title, text and resume as a precaution)
cordova.plugins.backgroundMode.setDefaults({ title: 'Your custom title', text: 'Your custom text', resume: true, hidden: true, silent: true }); cordova.plugins.backgroundMode.enable();

Redefine defaults again with some events (user taps or something else) while app running
cordova.plugins.backgroundMode.setDefaults({ silent: true });

After implementing step 2 I have app running for about 2 days (and counting) with no notifications. I understand that it is not a good solution, but it works. For me it is more important.

@LaRadioFM Thank you for the reply but it is not working for me. The bug erases all the defaults and it does not even show the custom title or text.
For the moment, I modified the default values of the plugin on Java to make it work, but it's a solution specific for my app.

@LaRadioFM but has not receive notifications? if put silent: true ?

@jota0222 Please share you code, that way others can be guided by what you have edited in the file java. Thank you!

thanks @LaRadioFM I put,
cordova.plugins.backgroundMode.setDefaults({ hidden: true, silent: true });
and it worked.

Hello @LaRadioFM @linhadiretalipe
Where should I put that code please? in which file ?
cordova.plugins.backgroundMode.setDefaults({ hidden: true, silent: true });

I'm use ionic with this plugin and it's solved for me:
this.backgroundMode.setDefaults({ hidden: true, silent: true });


Acreage thousand separator

Acreage in the "land mass" box should have a thousand separator.

Closed by #157


discuss: New feature in Apache APISIX Ingress controller v1.6

Hi, folks
The Apache APISIX Ingress controller v1.5 version has been entered release window, and it is time to discuss the plans in the v1.6 version together.
I think the following features need to be included:

Gateway API https://github.com/apache/apisix-ingress-controller/issues/644

#1217
#1278
HTTPRoute more complete support, such as filter

#927

#547

#1051

Add suport for APISIX's plugin_metadata

Knative support ? :)

Knative support ? :)

thanks!  SGTM
This will help us expand the ecosystem

Support  multi cluster management should be helpful.
ApisixClusterConfig was designed for this, but not completed.

Support multi cluster management should be helpful. ApisixClusterConfig was designed for this, but not completed.

@machinly Are you referring to managing multiple APISIX clusters through one APISIX Ingress controller?

In fact, we will achieve the goal of managing multiple APISIX instances through one Ingress controller.
This is necessary for a complete implementation of the Gateway API.
We can dynamically state the APISIX instance simply by creating a Gateway resource.

Support multi cluster management should be helpful. ApisixClusterConfig was designed for this, but not completed.

@machinly Are you referring to managing multiple APISIX clusters through one APISIX Ingress controller?

The field parentRefs in HTTPRoute is exactly what I needed. Gateway API is awesome.

v1.6.0-rc1 has been released. #1536  I will close this one. Thanks all!!!


Malicious folder entity can cause sync to get stuck.

Manually creating a folder entity that shares the root folder's ID inside another folder will cause sync to get stuck in a recursive loop, locking the page into a state of high CPU consumption.
This can be tested with drive df51d1d7-4a3a-4849-bec7-a78fc71c7467 .
The TXID of the hostile transaction is 66YlvAuckuQ59nosjVH30b9slfkQh4-AjRUmCq_HB2U .
Not a big deal, but it will crash indexing public drive content if ardrive-web ever gets this feature.

This happens if the entity is created by the drive owner OR with a TX created by ANY wallet?

Untested, but unlikely to happen with a TX on any wallet as long as you're aggressively filtering by the drive owner address when fetching folders and so (the fix for the injection exploit should ensure that).

Works as intended, only way to achieve a loop is if the wallet owner submits a TX with a duplicated ID on purpose. Closing.


[BUG] After syncing yaml front matter is invalid. Title has no end single quote

Describe the bug
After syncing yaml front matter is invalid.
Specifically title: 'Record of a Spaceborn Few has a start quote(') and no end quote.
I'll go and have a hunt to see if I can find where this is caused, but in the mean time I wanted to have a record of the problem
This is the title in my example, though I have seen other examples: https://www.amazon.com/dp/B07355XC7L
Another example that has this problem: https://www.amazon.com/dp/B007V65R54 (The Signal and the Noise)
---
kindle-sync:
  bookId: '30011'
  title: 'The Signal and the Noise
  author: Nate Silver
  asin: B007V65R54
  lastAnnotatedDate: '2022-02-01'
  bookImageUrl: 'https://m.media-amazon.com/images/I/81JNp1+YyqL._SY160.jpg'
  highlightsCount: 9
created: 2022-07-13T12:32:46+03:00
updated: 2022-07-13T12:32:46+03:00
---

Method of syncing

[x] Amazon Kindle Reader
[ ] My Clippings file (please attach file that can be used to recreate issue)

Expected behaviour
YAML front matter should be valid.
Metadata

OS: Mac
Plugin version 1.5.2 and still 1.6.2
Obsidian version 0.14.15

This issue is caused by https://github.com/beaussan/update-time-on-edit-obsidian/issues/17. It was just a coincidence that I experienced it with the imported notes. I'm closing this issue.


Volume navigation notifications

Good day all,
After the installation of the toolbox, I have noticed the volume of the radio sound barely lowers during notifications from the navigation.
For example, when the park distance is enabled the volume of the radio does lower significantly so it won't bother the beeping signal, and you can fully focus on parking the car.
However, due to the radio sound barely lowering during navigation instructions is is actually bothering. Nothing in the sound menu can improve this.
Definitely this happened after installing the toolbox.
Anything I can do?

Good day all,
Anybody has any idea I could have done wrong?
Firstly i have preparred the SD card to install the toolbox in my "ZR" telnet MIB2.
The I have patched the navi in order for a non original SD to work with the MIB2.
Everything is working fine, als the SD with the maps is recognized. Only whatever i'm listening (voice, media or radio) as soon as the park distance is switched on the sound of the voice, media radio etc will lower down. However, during navigation when a spoken inscruction starts, the volume goed down only a tiny bit, maybe only 2 %. Sometimes not even noticable. Is there anything a can do to fix this?
Is this related to the toolbox or the script it has runned to unlock non original SD cards to work with the navigtion?
Please somebody help me :-)

I only know this behavior depending on a user setting inside the volume settings.
Toolbox installations isn't changing anything in the setting so this can't be done by the toolbox itself. Look up your system settings.


don't start debugger when some other "debug" module is imported using relative import

Based on #7.
@qarkai, could you take a look?
In my testing level is None, when I use import debug. I added a few test files to verify it manually.
Before 5d930be the output is incorrect, the debugger is started only on the first import:
$ python manual_test.py
1
> /Users/narf/github/debug/manual_test.py(3)<module>()
      1 print 1
      2 import debug
----> 3 print 2
      4 import debug
      5 print 3

ipdb> c
2
3
manual_test_directory/__init__.py says hi
manual_test_directory/debug.py says hi
4

After 5d930be it works as expected:
$ python manual_test.py
1
> /Users/narf/github/debug/manual_test.py(3)<module>()
      1 print 1
      2 import debug
----> 3 print 2
      4 import debug
      5 print 3

ipdb> c
2
> /Users/narf/github/debug/manual_test.py(5)<module>()
      3 print 2
      4 import debug
----> 5 print 3
      6 import manual_test_directory
      7 print 4

ipdb> c
3
manual_test_directory/__init__.py says hi
manual_test_directory/debug.py says hi
4

I incorrectly determined import level value. Actually None value is for fromlist argument. Fixed in dfaf9e4.

I see. It's more complicated that I expected. I added more comments in 	5b14c4a.

I always forget about Python 3, my bad. 5b14c4a looks good.

👍

I released it to PyPI as v0.3.2. Thanks for your work on this bug, @qarkai! 🙇


Spring native application for AWS Lambda custom runtime

How would you retrieve or add an export to the build of the native image ?
My issue is that I am trying to use the native application with AWS Lambda custom runtime. I am able to do it with graalvm :)
For my custom runtime to be built I need the application not as a jar but an executable which is available during the build.
I am able to zip the executable along with a bootstrap all prepared for aws lambda.
Is it possible to do so with spring native beta ?

This sample builds a native image for AWS Lambda: https://github.com/spring-projects-experimental/spring-native/tree/main/samples/cloud-function-aws. Are you trying to do something different to that?

Thank you :)
It is what I was looking for although I cannot make the native.xml assembly file work

Did you run mvn install -P native-image (should work on "main" branch now - remember that the master branch changed).

I ran ./mvnw package -P native
but running mvn install -P native (since my profile is referred as 'native' and not 'native-image') does not change anything. I still get the same error declared as http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

There are 2 profiles involved (one in the parent pom) so just using ./mvnw package -P native won't work, please try to update to latest main branch and run mvn install -P native-image.

Thank you fir your input
I got it the build was successful :)


discussion of ideas to reduce fast weights memory consumption

Hi Fan Wu
Thanks for making your implementation of Fast Associative Memory available!  A few of us @zhongwen @jiamings have been discussing the memory aspect of this model, and would like some of your thoughts on this issue if you can help.
It seems the most straight forward way is to keep the entire fast_weights in memory, since it is easy to update over time steps by multiplying by lambda.  However, the amount of memory fast_weights need is BatchSize x NumUnits x NumUnits, and for practical tasks, even a simple baseline task like character PTB, it will break memory requirements as typically BatchSize is ~ 100, and NumUnits ~ 1000 to get close to SOTA level results.  From looking at your implementation and running it, it seems that is done here too, which is perfectly fine, since NumUnits is quite small for this task.
From my reading of the paper, it seems eq (3) describes the method of storing the entire outer product as a batch_size x N x N tensor, while eq (4) possibly allows us to get away with storing only the order of a batch_size x N tensor making it possible to compute the fast_weights on the fly and thus allowing this algorithm to work on practical tasks.  However the only way I thought of doing that is to keep track of the previous, say M hidden separately, where M is much smaller than N, in the hopes that lambda ^ M is a very small number so we can ignore anything before M time steps.  Been playing around with this variation here at the end of the file:  https://github.com/hardmaru/supercell/blob/master/supercell.py
I'm wondering if you have also thought about this issue also thought about a more memory-efficient implementation.
Thanks,
David Ha

Hi David
Thanks for sharing your ideas about the memory aspect of the implementation. I learned a lot from your blog and seems like this time I got something from you again. :)
After thinking this issue, I didn't come up with a better solution than yours. Sorry for I could't give some constructive advice.
Actually I'm new to this field, and this is my first TensorFlow program. Perhaps after learning more I will get new ideas for this issue.
BTW, I also love supercell's music.
Thanks,
Fan

Thanks, I'll see if this can be improved later on.
I think while the Fast Associative Weights idea is good, this particular implementation of the FastAssociative Weights could have been done better.  They only got it to work for toy problems, and didn't show it on more practical problems.  A better version of fast associative weights cell with a much better architecture should be possible in the future.


AsyncWithoutDuplicatesStream incorrectly filters CompiledMethods with same body

When two methods have the same body but a different selector, e.g., if you modify the selector of an existing method and save it, then one of the two will be filtered out from the Coder.
The problem is in AsyncWithoutDuplicatesStream>>#withoutDuplicates (from TAsyncStream).
NB: These notes are from chisandrei.

This shows the issue when using a union of compiled methods. The last filters should return two selectors
Object subclass: #FooBar
	instanceVariableNames: ''
	classVariableNames: ''
	package: 'FooBar'.

#FooBar asClass compile: 'foo123
	<haba>
	^ 1'.
#FooBar asClass compile: 'bar123
	<haba>
	^ 1'.

#foo123 gtImplementors | #bar123 gtImplementors

However this works ok:
Object subclass: #FooBar2
	instanceVariableNames: ''
	classVariableNames: ''
	package: 'FooBar'.

#FooBar2 asClass compile: 'foo1234
	^ 1'.
#FooBar2 asClass compile: 'bar1234
	^ 1'.

#foo1234 gtImplementors | #bar1234 gtImplementors

The main problem seems to be the comparison of compiled methods.
(FooBar >> #foo123) sameLiteralsAs: (FooBar >> #bar123)  "true"

(FooBar2 >> #foo1234) sameLiteralsAs: (FooBar2 >> #bar1234)  "false"

So the comparison of compile methods is not consistent. If the compiled method has pragmas (so an additional state), the comparison does not take the selector into account. But if the compile method does not have a pragma the comparison takes the selector into account. So two compiled methods with the same code that were not equal can become equal by adding the same pragma.

A possible fix is to add in AdditionalMethodState>>#analogousCodeTo: an explicit check for the selector
analogousCodeTo: aMethodProperties
	| bs |
	self class == aMethodProperties class ifFalse:
		[^false].
	(bs := self basicSize) = aMethodProperties basicSize ifFalse:
		[^false].
	1 to: bs do:
		[:i|
		((self basicAt: i) analogousCodeTo: (aMethodProperties basicAt: i)) ifFalse:
			[^false]].
	"self selector = aMethodProperties selector
		ifFalse: [ ^ false ]."
	^true

Upstream issue: https://github.com/pharo-project/pharo/issues/15967

This works in GT now.


Can we call a javascript functions inside of the twigJs template ?

I am referring to something like blade in Laravel, where I can just call any php function globally available inside  "{{ }}".  Is there anyway to add new globally available functions to the scope of the rendering ?

Maybe this can help you somehow?
https://github.com/twigjs/twig.js/wiki/Extending-twig.js#functions

@moda20 if you don't want to add a custom function to twig.js, keep in mind that you can pass anything as data to your Twig template, which means you can pass functions as well as long as they are attributes of one of the object passed as data.
Consider the following template:
{{ data.callMeMaybe() }}

If you render this template with the following data:
{
    data: {
        callMeMaybe: () => { return 'Please!!!'; }
    }
}

You'll obtain the following result:
Please!!!

It works because callMeMaybe is an attribute of data. As such they are not considered as globals and thus are not expected to be part of twig.js known functions.
It works with a few caveats like this one:
https://github.com/twigjs/twig.js/issues/467

To summarize, no it is not possible to include arbitrary JavaScript in a template like Laravel Blade. This project always seeks to be compatible with TwigPHP, so if it can't be done there, it usually can't be done here either.


Feature Charts: Add Longhorn 1.7.2 Chart in 2.9.x

Longhorn released 1.7.2 in October 2024 so we need to add this new release in the feature charts along with Rancher 2.9.x. We will probably need to make this OOB.
Add 1.7.2 LH chart in feature chart 2.9.x - Cluster Manager/Rancher Marketplace
Cc: @chriscchien @innobead @khushboo-rancher @yangchiu @PhanLe1010 @mantissahz @lucasmlp @nicholasSUSE @Jono-SUSE-Rancher  @rancher/longhorn

Closing this issue because the chart was releasesd.

Can Longhorn be added in v2.10?


Update the font awsome css to use version 5

font-awsome was already moved out of the package and is now a package dependency defined for the Servoy Package Manager. >= 4.7.0
So it should be fine now. Closing...


Also use the catalog file to map web URLs, not only local files.

@francescalb, should we make url_from_catalog=True by default? I think that is what you normally want. The --url-from-catalog option could be replaced with a --no-catalog option in case you don't want to use the catalog file. What do you think?

@francescalb, should we make url_from_catalog=True by default? I think that is what you normally want. The --url-from-catalog option could be replaced with a --no-catalog option in case you don't want to use the catalog file. What do you think?

a

I am not entirely sure since we would not use url_from_catalog when importing from web (get_ontology('webadress')).
Maybe it is better to set url_from_catalog=True as default unless the filename starts with http:// or https://?

a

I am not entirely sure since we would not use url_from_catalog when importing from web (get_ontology('webadress')).
Maybe it is better to set url_from_catalog=True as default unless the filename starts with http:// or https://?

@jesper-friis is this issue the same as I get when trying to load the BIGMAPontology? The problem there (I think) is that the catalog is not written correctly for imported .ttk files that are on the web. We should have that the catalog points to the locally made .owl after having converted the whole ontology to owl. Currently the catalog of changed from 'https://something.ttl' to 'https://something.owl'. If the .owl file does not exist it will not import the file.

@jesper-friis is this issue the same as I get when trying to load the BIGMAPontology? The problem there (I think) is that the catalog is not written correctly for imported .ttk files that are on the web. We should have that the catalog points to the locally made .owl after having converted the whole ontology to owl. Currently the catalog of changed from 'https://something.ttl' to 'https://something.owl'. If the .owl file does not exist it will not import the file.

The intention is to solve the BIG-MAP ontology issue. But you are right, that issue can be solved by entering a resolvable URI to the inferred ontology (including chemistry) directly in the ontology, instead of relying on the mapping.
Maybe this fix is not needed then...

The intention is to solve the BIG-MAP ontology issue. But you are right, that issue can be solved by entering a resolvable URI to the inferred ontology (including chemistry) directly in the ontology, instead of relying on the mapping.
Maybe this fix is not needed then...

As far as I understand, the problem is that when converting the ontology from .ttl to .owl the uri in the (new) catalog points to a weblocation for the ontology that is imported from the web and if the .owl version does not exist, the import fails. I did not mean that the ontology should be fixed. I think the correct way is to correct the uri in the new catalogfile to the .owl file that exists locally after having been imported as .ttl from the web and convrted locally (in the temporary folder where the other ones are).

As far as I understand, the problem is that when converting the ontology from .ttl to .owl the uri in the (new) catalog points to a weblocation for the ontology that is imported from the web and if the .owl version does not exist, the import fails. I did not mean that the ontology should be fixed. I think the correct way is to correct the uri in the new catalogfile to the .owl file that exists locally after having been imported as .ttl from the web and convrted locally (in the temporary folder where the other ones are).

As far as I can see, we need some support from Owlready2 to fix this correctly. I have posted a suggestion to http://owlready.8326.n8.nabble.com/IRI-mappings-and-catalog-file-td2203.html
Before that is fixed, I think we have to live with specifying imported ontologies as long resolvable URLs. That avoids the issue by bypassing the catalog file.
I have pushed my initial non-working update to branch also-read-catalog-for-web-urls.

As far as I can see, we need some support from Owlready2 to fix this correctly. I have posted a suggestion to http://owlready.8326.n8.nabble.com/IRI-mappings-and-catalog-file-td2203.html
Before that is fixed, I think we have to live with specifying imported ontologies as long resolvable URLs. That avoids the issue by bypassing the catalog file.
I have pushed my initial non-working update to branch also-read-catalog-for-web-urls.

Should we close this issue for now?

Should we close this issue for now?

The error for us comes up in the serialisation when reading the ttl file, so I am not sure if fixing it for .owl files would fix it for us (using .ttl)

We should probably start by setting up a set of tests that we want EMMO-python to handle.

We should probably start by setting up a set of tests that we want EMMO-python to handle.

Good suggestion!


IdentityServer4.AccessTokenValidation: The SSL connection could not be established

I have a .net core 3.1 web api that uses the IdentityServer4.AccessTokenValidation v3.0.1 Nuget package to validate access tokens.
This has been working well for a while, but over the last week or so I started seeing more and more exceptions with the following stack trace:
IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler[0]
Error parsing discovery document from https://********: Error connecting to https://********/.well-known/openid-configuration. The SSL connection could not be established, see inner exception..
System.InvalidOperationException: Error parsing discovery document from https://********: Error connecting to https://********/.well-known/openid-configuration. The SSL connection could not be established, see inner exception..
at IdentityModel.AspNetCore.OAuth2Introspection.PostConfigureOAuth2IntrospectionOptions.GetIntrospectionEndpointFromDiscoveryDocument(OAuth2IntrospectionOptions options)
at IdentityModel.AspNetCore.OAuth2Introspection.PostConfigureOAuth2IntrospectionOptions.InitializeIntrospectionClient(OAuth2IntrospectionOptions options)
at IdentityModel.AspNetCore.OAuth2Introspection.OAuth2IntrospectionHandler.LoadClaimsForToken(String token)
at IdentityModel.AspNetCore.OAuth2Introspection.OAuth2IntrospectionHandler.HandleAuthenticateAsync()
at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.AuthenticateAsync()
at Microsoft.AspNetCore.Authentication.AuthenticationService.AuthenticateAsync(HttpContext context, String scheme)
at IdentityServer4.AccessTokenValidation.IdentityServerAuthenticationHandler.HandleAuthenticateAsync()

The message refers to an inner exception, but it is not available here (?).  Is there a way I can intercept the exception to get the inner exception?
My setup (registration) is as follows:
services
      .AddAuthentication(IdentityServerAuthenticationDefaults.AuthenticationScheme)
      .AddIdentityServerAuthentication(options =>
      {
          options.Authority = authSettings.AuthUri;
          options.ApiName = authSettings.ApiName;
          options.ApiSecret = authSettings.ApiSecret;
          options.RequireHttpsMetadata = true;
       });

What is strange, is that this is intermittent.
What could cause this?

The SSL connection could not be established

is pretty obvious. Some HTTPS config does not work anymore. Maybe an expired cert?

The issue is intermittent, certs are valid.

Then I don't know. I don't think the inner exception will give you more details.
You could plugin your own HttpClient for the back-channel though.

In v2.7.0, IdentityServerAuthenticationOptions inlcuded
IntrospectionBackChannelHandler
IntrospectionDiscoveryHandler
and
JwtBackChannelHandler
in 3.0.1 only JwtBackChannelHandler is available, is that where I would plugin a custom handler?

Reverting to 2.7.0 actually appears to have solved this, no custom handler set

Closing this as the issue is clearly on my side - however, would be great if the inner exception is bubbled!


Applied Visual Design: Learn about Tertiary Colors - not verifying bg color is set

Challenge Name
Applied Visual Design: Learn about Tertiary Colors
Issue Description
Receiving the following error, although background set correctly (screenshot included). It accepts hex color code.
Error: The div element with class orange should have a background-color of orange.
Browser Information
Chrome Version 56.0.2924.87 (64-bit)
iMac, Mac OS Sierra 10.12.3
Your Code

Screenshot

@sue888 you are supposed to change from the 'string' value to the hex.
So the error is thrown since
.orange { background-color: orange; }

you are not passing the hex.

Now the real question is why cyan is accepted since you are not passing the hex there too.
Probably worth to look at the test cases.

Ah - must have missed that instruction -  have been using strings successfully throughout this section (where I knew it to be an option).

Closing as solved! 👍

@JosephLivengood actually I think this is worth a look.
OP said she had trouble with orange but, as a matter of fact in her original code also cyan was wrong but the checkmarks didn't reflected that.
It's probably worth looking into it. no?

? The question is what did he modify?


style the Message Board, make is scrollable

Hey Jeff I had a lot merge conflicts in messages/index.js.  You might want to check to see I resolved them all correctly


WIP: Finishing /doc/commands-reference/diff and revising /doc/tutorial/define-ml-pipeline <=> /doc/get-started/add-files

Finishing /doc/commands-reference/diff
Revising /doc/tutorial/define-ml-pipeline <=> /doc/get-started/add-files

Hi @shcheklein. This PR is close to ready, I just want to double check my 2nd example in the dvc diff ref. But I also noticed you gave me push access to the upstream repo so not sure whether I should keep working with this and future PRs or in some other way. Please lmk when you can, thanks.

OK @shcheklein I updated the examples. Let's see what you think 🤞
I'll merge this PR for now as you indicated above and double check the doc looks good on the website.

Actually I can't merge this even after dismissing your review @shcheklein . It still reads

Review required
At least 1 approving review is required by reviewers with write access.
Please review/merge when you have a chance.


Poor guidance for newcomers. Walkthrough breaks at step 6 in the first section.

At the top of this document is a section titled "Create a minimal application". I attempted to follow this.
I got to stop 6: "Build the application by typing csc helloworld.cs at the command prompt."
But I got a predictable failure: "'csc' is not recognized as an internal or external command, operable program or batch file."
I know the basics of what this error represents.
I tried skipping ahead in this document to the section that recommends editing your PATH variables: "At the command prompt, type set PATH=%PATH%;%ProgramFiles%\MSBuild or set PATH=%PATH%;%ProgramFiles(x86)%\MSBuild."
This also failed to solve my problem.
This tutorial then suggests the following: "Alternatively, if you have Visual Studio installed, you can use the Developer Command Prompt for Visual Studio, which has a path that includes the MSBuild folder."
I would rather not do this. I was attempting to learn about MSBuild with the goal of being able to write scripts that would help other developers avoid some tedious steps. By using the crutch of the Developer Command Prompt, I have lost the insights I need to know the minimal requirements to run a build.
The result is that this document is a poor experience for someone who is attempting to get introduced with MSBuild. The initial "hello world" program fails to build, and I immediately forced to troubleshoot. The document is seemingly out of order, or the document needs to outline prerequisites.

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

ID: 09bc6800-41dc-a7a4-5942-0f103a290cf9
Version Independent ID: 56b974f2-30d2-ae79-2b78-7eb073ec7cd1
Content: Walkthrough: Creating an MSBuild Project File from Scratch - Visual Studio
Content Source: docs/msbuild/walkthrough-creating-an-msbuild-project-file-from-scratch.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

@JohnnyGuy -- thank you for your feedback.  Please consider these resources:

https://social.msdn.microsoft.com/Forums/vstudio
https://www.stackoverflow.com
https://www.reddit.com/r/VisualStudio/

@ghogen -- please look into this issue.

@JohnnyGuy Thanks for pointing this out. This was out-of-date information.  I've submitted a fix for this.


Discussion: Keep containers warm as first-class primitive/option

One potential use for OW actions is any machine-learning prediction. Usually, the heavy computation happens off-line when training the model. In theory, the prediction can be encapsulated in an action that is invoked each time a prediction needs to be performed.
However, some models are large enough that "loading" the model on each action invocation takes in the order of seconds, while executing the prediction is fast (in the order of ms). Given that we reuse containers, we could write the user code in such a way that we check whether the model is loaded or not and have some "fake" actions that do not execute anything, they just make sure the containers stay warm for potential real actions to execute.
This "hack" could be replaced by a first-class citizen option in our system: offer the ability to run "special" containers that the system keeps warm.
This came up in a discussion with one of our interns that is building a specialized search service that needs to load a classification model in the order of 500MB-1GB.

@ioana-blue one increasingly common 'hack' is to run actions in a scheduled fashion, just for the sake of keeping them warm.
would that do the trick?

@mbehrendt Yes, I'm wondering if we were to replace this 'hack" with an option at the runtime level. I guess we would benefit from it because we would get rid of all these additional (useless) invokes; instead, we would know at the runtime level to treat some containers differently.
Related, I would have to check the implementation we have for python. I may clear the "namespace" of previous invocations unintentionally (not sure how python works wrt using namespace = {} when starting to run the user code)

My two cents: If the developer requires a long running container containing previous state (i.e. cache files) from previous state, then maybe the workload is not best fit for to be solve using serverless. The workload looks like is best solve by normal deamon/server platform.
I not familair with the specific use case but I would solve the problem in a different way if I was going to use serverless.
If there is a large archive 500MB-1GB that the function needs to be initialize, then I would consider this archive part of the definition of the action. I would create a new docker action specific for the function include the 500MB archive and push the docker image.
Then whisk will use the container at the time of doing the computation and run the docker container containing logic code and archive (model).
Today there is small delay for whisk to pull down and cache the docker action, but this can be optimized in the future if whisk were to offer a docker registry that user can push their docker images to.

i think there are many use cases where a warm action is desirable  (warm action = more consistency in terms of fast invocation time), however, i don't think we shouldn't yet be spending effort on implementing native support for this yet. this feature can technically clearly be done, but introduces more complexity to how the engine behaves, another parameter that needs to be taken account across the board, etc. we should rather focus on improving the primitives first, and add this feature once the foundation is ready for it -- specifically given there is a reasonable workaround available.

@csantanapr Including the "archive" in the image helps only with having the data hopefully closer, but you still have to load it in the memory of the action. not sure how long that takes, but I would think it takes some non-negligible  time
@mbehrendt I agree that we don't have time for this now.

Can we consider this enhancement at this time? There is a real need to either keep containers alive with neural network models loaded so that inference actions can be launched quite quickly.


Screed v0.8 thoughts

[ ] Fix the khmer sandbox scripts (#787)
[ ] Run acceptance tests (this is for v0.8) as per #686

cc @bocajnotnef

I'll start in on the sandbox scripts today/this weekend.
On Fri, Feb 20, 2015, 07:24 C. Titus Brown<EMAIL_ADDRESS>wrote:

Fix the khmer sandbox scripts (#787
https://github.com/ged-lab/khmer/issues/787)
Run acceptance tests (this is for v0.8) as per #686
https://github.com/ged-lab/khmer/pull/686

cc @bocajnotnef https://github.com/bocajnotnef
—
Reply to this email directly or view it on GitHub
https://github.com/ged-lab/khmer/issues/792.

per verbal convo, I'll go clean up len and slicing issues ( #705  and #768, respectively) and @ctb will work on the sandbox scripts.


Fix deprecations

Removed unused symfony/templating.
Set minimum version of doctrine/persistence to 1.3.4 (https://github.com/doctrine/persistence/pull/87).
doctrine/orm is forced to 2.6 by doctrine/doctrine-bundle so 2.6.3 makes sense as lowest version (https://github.com/doctrine/orm/pull/7307).

Good finds, thanks!


plotst:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

I have read and followed the submission guidelines and, in particular, I

[x] selected a name in conformance with the guidelines
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] ensured that my submission does not infringe upon the rights of a third party
[x] tested my package locally on my system and it worked
[x] named this PR as name:version of the submitted package
[x] agree that my package will not be removed without good reason

Description:
Plotst is a package people can use to create a few types of different plots and graphs. It can be useful to display data in a straight forward way.
It supports:

Scatter plots
Graph charts
Histograms
Bar charts
Pie charts
Overlaying plots/charts

There is one more thing I am unsure about: This package contains a lot of auxiliary files (examples, docs, etc.) When somebody imports this packages, these will all be downloaded as a tar.gz and be made available to the compiler. The archive for this package would be 451KB even though the actual source files are probably a few KB. There is the question whether the images and/or docs files should only be part of the external repository and linked to.

There is one more thing I am unsure about: This package contains a lot of auxiliary files (examples, docs, etc.) When somebody imports this packages, these will all be downloaded as a tar.gz and be made available to the compiler. The archive for this package would be 451KB even though the actual source files are probably a few KB. There is the question whether the images and/or docs files should only be part of the external repository and linked to.

You are absolutely right. I can remove all the auxiliary stuff and just link to my repo. That would leave the plain code at 9 files, if I counted correctly ^^

Thank you!


Add customizable "Simple" Haine theme

This PR adds a very simplistic theme that's customizable. It requires https://github.com/dsrkafuu/skyline-overlay/pull/31.
This shows how easy it is to let the user pick what colors they want in their theme. Once the dependent PR is merged, I will open one off of this branch into the parent repo.
Some examples:

Old PR: https://github.com/dsrkafuu/skyline-overlay/pull/28


Use mongoose model functions inside the model class itself

Information

Version: 5.x
Type: Issue

Description
I have a model class and it inherits a generic class, which has a function and inside this function I want to get data from the database.
Example
In this example I have one model called "Character" which inherits from Entity and inside my Entity class I want to have a function that can call my mongoose model functions. I tried a lot of things so far, tried to inject the model by decorating my class with "Injectable" and "Service" but didn't work.
@Model({
    collection: 'characters',
    schemaOptions: {
        timestamps: {
            createdAt: 'createdOn',
            updatedAt: 'modifiedOn'
        }
    }
})
export class Character extends Entity<Character> {
    @ObjectID()
    _id: string

    @Ref(Account)
    account: Ref<Account>

    @Ref(Dominion)
    physicalDominion: Dominion

    @Ref(Dominion)
    dominion: Dominion

    @Ref(Coven)
    coven: Coven

    @Property()
    @Required()
    name: string

    @Property()
    @Default(false)
    bot: boolean = false
}

@Service()
export class Entity<T> {
    constructor(
        @Inject()
        private model: MongooseModel<T>
    ) {}

    @Property()
    baseEnergy: number

    @Property()
    energy: number

    @Property()
    @Enum(States)
    @Default(States.NORMAL)
    state: States = States.NORMAL

    @Property()
    @Default(0)
    power: number = 0

    @Property()
    @Default(0)
    resilience: number = 0

    async attack() {
        console.log(this.model.find) // undefined
    }
}

Possible solution
I think that the Model decorator should give my class the mongoose functions so I can call them inside any functions in my class.

Hi @ghmendonca
Is not possible, Model are created  by Mongoose, not by Ts.ED.
Adding @Service over Entity is an error and has no effect.
Mongoose use the class to create a new instance of your class where your method are available.
I think the solution is to use PreHook, PostHook or Plugin http://tsed.io/tutorials/mongoose.html#register-hook
I haven't solution for you, this part is totally handled by mongoose.
See you,
Romain

Ok, so I figure a way to do this without using hooks or anything like that. As my model is a mongoose model (because of the decorator), and in my code I only have its instances and not the mongoose model itself, to get the model from the instance, I have to do this:
async attack() {
        const model = <MongooseModel<T>> this.constructor
        const result = await model.findOne()
        console.log(result)
}


CB SERVERS DOWN

Currently, our application in appstore and playstore does not open. Such an error arose after you made it to the servers. Close to 10 thousand can not enter now. So urgent.
Object {
  "code": 422,
  "detail": Object {
    "base": Array [
      "The default Firebase app already exists. This means you called initializeApp() more than once without providing an app name as the second argument. In most cases you only need to call initializeApp() once. But if you do want to initialize multiple apps, pass a second argument to initializeApp() to give each app a unique name.",
    ],
  },
  "message": Object {
    "errors": Object {
      "base": Array [
        "The default Firebase app already exists. This means you called initializeApp() more than once without providing an app name as the second argument. In most cases you only need to call initializeApp() once. But if you do want to initialize multiple apps, pass a second argument to initializeApp() to give each app a unique name.",
      ],
    },
  },
  "status": "error",
}

Thank you for reporting the issue
We are investigating the root cause
Will update here with more info

I assure you it needs to be resolved very urgently. We log in with Firebase phone verification.
Our users are waiting for you. It is very urgent. @DaveLomber

The root cause is identified
The team is working on a patch
More updates to come soon

@CMLCNL we have deployed the patch
Could you please check at your side and let us know whether the issue is resolved

Yes @DaveLomber everything is fine for now. Thank you very much.

@DaveLomber
Some of the problems we detected.
The list of notifications sent in the -ConnectyCube admin panel does not work.
-There is a problem with the voip notification.
-We get complaints from our users. About not starting a conversation. During a voice or video call.

@CMLCNL could you please provide more info if possible re 2nd and 3rd cases

We cannot send voip call notifications.
Some users said that the voip notification came, but they said they could not have a healthy conversation.


[stable/rabbitmq] set a sane default for RABBITMQ_DISK_FREE_LIMIT

The bitnami rabbitmq image sets a default value of RABBITMQ_DISK_FREE_LIMIT="{mem_relative, 1.0}"

Having to provision a PVC (EBS) that is greater than the RAM size of the node is pretty brutal.  Pretty please add this to the chart.

Please note that setting disk limit smaller then RAM can be dangerous. If RabbitMQ runs out of disk space it can crash and be come almost unrecoverable. Messages held in RAM can be flushed to disk even if they are transient.


Naming and design

Hi, it's me again :D
I'd just like to kindly suggest that having a file named lib/tty-progressbar.rb is an anti-pattern, and also, a class called ProgressBar should be in a file called progress_bar.rb not progressbar.rb. Additionally, rather than overloading progress bar to be a spinner, you might want to have an entirely separate class. I think it's a nicer design.
Just some general feedback. Really useful gem.

Hmm, I see that spinner is already separate, for some reason I thought it was an option on progress bar. My bad, ignore that feedback.

Ohh noooo you again .... :smile:
I would still keep the lib/tty-progressbar.rb for loading compatibility but will change it to only require tty/progressbar and use relative paths :wink: . I agree with the convention, my oversight on the class vs file naming.
Cheers for the feedback!


Fix tracemalloc doc options

What does this PR do?
Fixes the prefixes for tracemalloc options, which require profile_memory prefixes: https://github.com/DataDog/integrations-core/blob/master/datadog_checks_base/datadog_checks/base/utils/agent/memory.py#L183-L187

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Please add labels and milestones so that tests pass