Bugfix CustomerImporter.php

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/PT-8455.
Please use this issue to track the state of your pull request.

Shoot! I forgot to paste the $this->db->query($sql); statement. I added it in a separate commit.
The bug I fixed is documented in the official shopware forum:
https://forum.shopware.com/discussion/41654/migration-von-prestashop-probleme-beim-kundenimport-sql-syntax-near-gmx-de

Siehe auch: https://issues.shopware.com/issues/PT-7540

hey @micronax
unfortunately the current state of your PR has a conflict. is it still relevant? if so, could you please rebase your PR to the current master?
sorry for the late response! :pray:

hey @micronax
Sorry for the late answer!
The corresponding ticket was closed. If this change is still relevant for you, please feel free to create a new PR.


Compile Error: Use of undeclared crate or module argonautica

I followed the quick start to create a new app.
When I run cargo fullstack or cargo build, I receive the following error:
   Compiling create-rust-app v8.1.1
error[E0433]: failed to resolve: use of undeclared crate or module `argonautica`
 --> /Users/rohit/.cargo/registry/src/github.com-1ecc6299db9ec823/create-rust-app-8.1.1/src/dev/endpoints/service_poem.rs:1:5
  |
1 | use argonautica::config::Version::_0x10;
  |     ^^^^^^^^^^^ use of undeclared crate or module `argonautica`

Originates here:
https://github.com/Wulf/create-rust-app/blob/50cd54b14d50ac1b733fd653723978518344a1ec/create-rust-app/src/dev/endpoints/service_poem.rs#L1

What plugins did you enable during initial bootstrapping?

Interesting, are you using the most recently released create-rust-app cli?

I did cargo install just before creating my app. This is my first time
creating an app using create-rust-app.
On Thu, Dec 8, 2022 at 12:00 AM Anthony Rubick @.***>
wrote:

Interesting, are you using the most recently released create-rust-app cli?
—
Reply to this email directly, view it on GitHub
https://github.com/Wulf/create-rust-app/issues/80#issuecomment-1342230862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHWSMNF6P5ZHFRCP7WFEKTWMGISLANCNFSM6AAAAAASXXIPKE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I see, yeah this is an issue with the project itself, I can make a PR to patch it (from what I can see that import isn't needed anyway)


disable connect to node by ssh

1、On the version 3.5.3, lens can connect to node with command line by ssh. But from the version of 3.6.0, it always disable connect to node with command line. If you try it, you will see the problem.
2、May you will say, Hi there! You should confirm that your node server is OK? And I will say "YES", I tried with lens version 3.5.3,It is OK, and lens version from 3.6.0 is not OK.

Is this still a problem? Are you able to connect to nodes in a newer version of lens?


how do i pair it with homekit . installed the files and running on my esp but cant find the device on homekit

Could you please provide more info on the steps you did?
Did you connect to the generated AP on the eso to connect to your WiFi?

You don’t see any accessory when pressing + “add accessory”?
Sent from my iPhone 
Press in Dire Situations http://www.nooooooooooooooo.com/

On 16 Aug 2018, at 23:24, surya31<EMAIL_ADDRESS>wrote:
i just flashed the examples/button and connected to wifi but cant see the device on homekit
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

i saw this once but when deleted , it not showing at all, board is fine, monitor also displays its connected but no homkit authentication is happening

Post you code...

m_task_hdl : 3ffef8e0, prio:1, stack:176
frc2_time:0x3fff3f28, prio:12, stack:200

HomeKit: Starting server
HomeKit: Using existing accessory ID: F4:4E:D1:41:E9:A2
HomeKit: Found admin pairing with 23E9459B-46A6-44E1-B35C-8E71AFAA324C, disabling pair setup
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
HomeKit: Configuring mDNS
mDNS announcement: Name=Dual Lamp
md=Dual Lamppv=1.0id=F4:4E:D1:41:E9:A2c#=1s#=1ff=0sf=0ci=5 Port=5556 TTL=4500
scandone
add 0
aid 1
cnt

connected with hell000, channel 6
ip:<IP_ADDRESS>,mask:<IP_ADDRESS>,gw:<IP_ADDRESS>

iam using sonoff_dual from example earlier it was running now i cant find it. plus is there a way to change accessory id ??

iam a noob in iot just got started with esp

It is already paired, so if you've deleted the accessory in the Home app you should erase flash and reprogram the example:
make -C examples/button erase_flash
make -C examples/button flash
Then add it again in the home app.

so this chip get stored to the home api?? as i tried flashing other codes as well still no lock in fiding the accessory ?I tried this.. still no luck

Could you successfully erase? I think you should give more details about what happened after making and then after flashing. Post the output as you did

yes sir i erased and evertything is working
bootloader/firmware_prebuilt/blank_config.bin 0x2000 ./firmware/sonoff_dual_lights.bin
esptool.py v1.2
Connecting...
Running Cesanta flasher stub...
Flash params set to 0x0320
Writing 4096 @ 0x0... 4096 (100 %)
Wrote 4096 bytes at 0x0 in 0.4 seconds (86.9 kbit/s)...
Writing 4096 @ 0x1000... 4096 (100 %)
Wrote 4096 bytes at 0x1000 in 0.4 seconds (88.5 kbit/s)...
Writing 430080 @ 0x2000... 430080 (100 %)
Wrote 430080 bytes at 0x2000 in 37.3 seconds (92.1 kbit/s)...
Leaving...
make: Leaving directory '/home/surya/Development/esp-open-sdk/esp-open-rtos/examples/sonoff_dual_lights'

ESP-Open-SDK ver: 0.0.1 compiled @ Aug 17 2018 08:09:50
phy ver: 273, pp ver: 8.3
mode : sta(18:fe:34:cf:0f:75)
add if0

HomeKit: Starting server
HomeKit: Using existing accessory ID: 66:14:29:7F:4D:17
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
HomeKit: Configuring mDNS
mDNS announcement: Name=Dual Lamp
md=Dual Lamppv=1.0id=66:14:29:7F:4D:17c#=1s#=1ff=0sf=1ci=5 Port=5556 TTL=4500
scandone
add 0
aid 3
cnt

connected with nav52, channel 1
ip:<IP_ADDRESS>,mask:<IP_ADDRESS>,gw:<IP_ADDRESS>

earlier here a authentication homekit used to come but after deleting the accessory it not showing again

the problem is it was showing beofre was working fine but suddenly after last delete its gone forever

de : sta(18:fe:34:cf:0f:75)
add if0

HomeKit: Starting server
HomeKit: Using existing accessory ID: 9B:97:A9:E2:32:33
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
Function called without core lock
HomeKit: Configuring mDNS
mDNS announcement: Name=Dual Lamp-CF0F75
md=Dual Lamppv=1.0id=9B:97:A9:E2:32:33c#=1s#=1ff=0sf=1ci=5 Port=5556 TTL=4500
scandone
add 0
aid 3
cnt

connected with nav52, channel 1
dhcp client start...
ip:<IP_ADDRESS>,mask:<IP_ADDRESS>,gw:<IP_ADDRESS>

found another ios device and its showing here .. thank you guys for the support

You should also close home app and open it again, not just restart your phone.

@surya31 When you remove accessory from iPhone, it is better to do it while accessory is online and accessible. When you remove accessory, iPhone send command to remove itself from accessory's pairing database. If you remove it offline, the pairing record will remain in accessory's database and it won't pair again. In that case you need to erase pairing database from accessory's flash. Not going into much detail, it is easier to just erase_flash and flash firmware again.


Disable plugin gracefuly

Right now if I remove a plugin from INSTALLED_APPS (or change its python-name) then all pages with the plugin crash -- both rendered and in admin. Why not to hide such plugins (by default)? Can this be controlled somehow?

After removing a plugin from INSTALLED_APPS, you have to run manage.py cms uninstall <PluginName>, replacing <PluginName> with the name of your plugin. see http://docs.django-cms.org/en/latest/advanced/cli.html#plugin-and-apphook-management-commands

Yeah, but I do not want to uninstall the plugin -- I want to disable it. And I still do not see why raising exception can be good in this case? I would really prefer just to hide it... and have a setting to enable such exception raising if I really want my site and admin (!) crashing because of it. Could you explain this design decision?

http://www.python.org/dev/peps/pep-0020/ line 2

I am trying to manage.py cms uninstall FilePlugin but see exception. One of files is embedded in an instance of TextPlugin.
django.db.utils.IntegrityError: (1451, 'Cannot delete or update a parent row: a foreign key
constraint fails (`myapp/cmsplugin_file`, CONSTRAINT `cmsplugin_ptr_id_refs_id_c294683f`
FOREIGN KEY (`cmsplugin_ptr_id`) REFERENCES `cms_cmsplugin` (`id`))')

please open a new ticket for that failure

@ojii Don't you think that users would like to handle the exceptions somehow without 500? Do you provide such mechanism?

No.
Removing a plugin from iNSTALLED_APPS is not the only reason this happens. It can also be a legitimate bug in your code (in case of custom plugins). Suppressing those errors is not a valid option. There's currently no way of having plugins installed but hidden.


Favicon needed

Hi guys,
Would you please guys create a favicon for the site? I attach the screenshot of what is displayed in my Favorites toolbar:

Thanks! :)

Implemented and pushed to live.
Thanks @alfonsocrisci! :+1:

Very cool, thanks Dani !


Duplication of entries on pagination of Gene Ontology section in the target profile page

Describe the bug
In the Gene Ontology widget in target profile page of APP, the next page duplicates the entries when moving to the next page in the latest dev
To Reproduce
Steps to reproduce the behaviour:

Go to Gene Ontology section in target profile page of APP
Select the Rows per page: to 100
Click on next page
The entries are doubled (Total 200 rows shown instead of 100)

One more pagination issue:
Describe the bug
In the bibliography section of the target profile page, the entries do not refresh when moved to the next page.
To Reproduce
Steps to reproduce the behaviour:

Go to Bibliography section in target profile page of APP
Select the Rows per page: to 25
Click on next page
The entries do not refresh and stay the same.

Fixed on latest release of the UI


Add seperate alt param to  nbImage, close #193

If alt is empty the caption is used instead. Also added a new test in trenders.nim.

Thanks again! 😄

I see, looks nice 👍😄


External HMD is not playing Audio

AudioSwitch is not picking WiredHeadset as avilableDevice
we have a device that has external speaker and display and work good when there is no twilio session
For video calling as we start the twilio session The sound is automatically route to speaker of phone Though the hmd is connected
when i call audioSwitch.start { _, _ -> }
The device doe not pick WiredHeadset and the show the only avilable devices as "Speakerphone" and "Earpiece"

@noumanshah115 What device is it?

@afalls-twilio  it is VR hardware(contain's mic and speaker) connected through external USB C cable

@noumanshah115 If a custom audio device is being used, then I would recommend creating a custom audio source for the item or extending audioswitch (its open source), to also support the custom VR Hardware.


On-Premises: added possibility to use other device names for existing network interface

Nice improvement, many thanks!


Create a redirect view for future releases not set up on the site yet

There are two links from the CMS to this website that it would be nice to future-proof with a view / middleware that automates redirects, or a custom 404 page set up accordingly.
Release notes links
For links to "New in Wagtail x.y" - https://guide.wagtail.org/en-4.2.x/releases/new-in-wagtail-4-2/. Currently the CMS defaults to targeting a manually created redirect, https://guide.wagtail.org/en-latest/releases/latest/, which we then update in the guide.wagtail.org admin.
Instead, we could set up links like for docs.wagtail.org:
"https://docs.wagtail.org/en/v%s" % __version__

and make sure any version of the guide that doesn’t yet exist is handled gracefully.
Guide homepage links
We currently link to https://guide.wagtail.org/ which redirects to latest. It’d again be nicer if we linked straight to a specific version based on the version of the CMS, and gracefully handled versions that don’t exist yet.

I expect as part of changing this we will also be able to fix https://github.com/wagtail/wagtail/issues/10105. Edit: done

The existence of this issue makes sense of what I just noticed, which is that the "latest" redirect is still going to "what's new in Wagtail 4.2", while the link text in the CMS now reads "what's new in Wagtail 5.0" now that I've updated my project :)

Thanks for reporting @sheepman4267! I thought I had updated the redirects but I had missed this one. All the more reason to automate this even if this is simple to do manually.

Case one: Release page already exists: user should view the release page
Case two: Release page doesn’t exist: user should view a 404 that says "this release doesn’t exist. [Link to latest release’s release notes]"

(Possibly to consider in the future always-on redirect view that takes people to the latest release’s release notes (/latest-release)

Fixed in #375. See also https://github.com/wagtail/wagtail/pull/11182.


Formulas overwritten by the lazy value of the cell

It appears that formulas are overwritten by the default formula value in some cases.
I looked a bit at the code and I found that when parsing the cell at the end of the <v> tag the parser doesn't check if the <f> element as already be encountered and just use the Cell::set_value_string type function which set the formula to None
I experimented with adding a check for the formula and used set_formula_result_default in case where a formula exists and my fix seems to work. But that said I'm not familiar with the codebase and it might break some other cases :
// In src/structs/cell.rs

fn set_attributes</**/> (/**/) {
    // ...
    loop {
        match reader.read_event_into(&mut buf) {
            //...
            Ok(Event::End(ref e)) => match e.name().into_inner() {
                    b"v" => match type_value.as_str() {
                        _ if self.cell_value.is_formula() => {
                            self.set_formula_result_default(&string_value);
                        }
                        "str" => {
                            self.set_value_string(&string_value);
                        }
                        //...
                        _ => {}
                    },
                    // ...
                },
               //...
           }
     }
}

I've also seen the lazy value type and wonder if it could be related to this too.

Here is a minimal file to reproduce the bug.
min-bug-reproduction.xlsx

@oxabz
Thank you for your report.
We have checked and found a defect in the relevant section.
We will correct the problem based on the PR you received.

I've checked the fix on my project. Everything seems to be working correctly.
Thanks for the swift answer. Keep up the great work!


cosmetic issues in in-progress tasks returned payload

Environment

Application Services Version: 3.50.2
BIG-IP Version: not relevant

Summary
Posting an async declaration the following payload is returned:
{
  "id": "dc5c5f6d-1e86-4428-b643-e6da3a9b72ed",
  "results": [
    {
      "message": "Declaration successfully submitted",
      "tenant": "",
      "host": "",
      "runTime": 0,
      "code": 0,
      "declarationId": "691121"
    }
  ],
  "declaration": {},
  "selfLink": "https://localhost/mgmt/shared/appsvcs/task/dc5c5f6d-1e86-4428-b643-e6da3a9b72ed"
}

when querying the task-id, if the task is in-progress we get the following payload returned:
{
  "id": "dc5c5f6d-1e86-4428-b643-e6da3a9b72ed",
  "results": [
    {
      "message": "in progress",
      "tenant": "",
      "host": "",
      "runtTime": 0,
      "code": 0
    }
  ],
  "declaration": {}
}

Issues:

typo in runtTime => should be runTime (code: https://github.com/F5Networks/f5-appsvcs-extension/blob/91e3b2dd2be4369b95eaea039899bd2174f48830/src/lib/asyncHandler.js#L98)
declarationId is missing
selfLink is missing

@amolari thanks for opening this cosmetic issue. Easy fix on your end
--- Change "runtTime": 0, to runTime
Added AUTOTOOL-4353

@amolari picked up in the current sprint

This issue has been resolved in AS3 52.0


Clicks on slider paginator doesn't work.

Short description of the problem:
Clicks on slider paginator doesn't work.
What behavior are you expecting?
When i click on paginator circle it should take me to that page/slide. I tried this demo and it didn't
https://github.com/driftyco/ionic-conference-app
Also didn't find any info in documentation on enabling this. In any case shouldn't this be the default behaviour.
Other information:
I tried it on desktop chrome on mac
Which Ionic Version?  2.x
Plunker that shows an example of your issue
https://github.com/driftyco/ionic-conference-app
Your system information:
Cordova CLI: Not installed
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0-beta.35
Ionic App Lib Version: 2.0.0-beta.19
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X Yosemite
Node Version: v4.4.7
Xcode version: Xcode 7.2.1 Build version 7C1002

You can enable clicks on the pager by passing:
paginationClickable: true,

in the options. For example:
<ion-slides [options]="mySlideOptions">

this.mySlideOptions = {
  pager: true,
  paginationClickable: true
};

The default is the same as the Swiper API, which is false: http://idangero.us/swiper/api/#.V6iv85OANBc

Hello @awebdeveloper , as the above solution should fix this issue i will be closing this issue for now, but if that does not fix it for you then feel free to comment and i will happily reopen this issue. Thanks for using Ionic!

@brandyscarney Can't bind to 'options' since it isn't a known property of 'ion-slides'.


refactor: update federation cmdlets

In order to have a good experience with our community, we recommend that you read the contributing guidelines for making a pull request.
Summary of Pull Request

Updates the cmdlets for managing federation to warn as deprecated as of 4.3.0 and the APIs were removed as of 4.4.0.
In 4.3.x, a banner / prompts presented when a user trys to create a new federation in the UI, but creation is allowed.
In 4.4.0, the APIs were removed and federation can no longer be created.
References:

4.4: Deleted Operations

Minor cleanup to text in the cmdlets' metadata.
Signed-off-by: Ryan Johnson<EMAIL_ADDRESS>Type of Pull Request

[ ] This is a bugfix.
[ ] This is an enhancement or feature.
[ ] This is a code style / formatting update.
[ ] This is a documentation update.
[x] This is a refactoring update.
[ ] This is something else.
Please describe:

Related to Existing Issues

Issue Number: N/A
Test and Documentation Coverage

[x] Tests have been completed (for bugfixes / features).
[ ] Documentation has been added / updated (for bugfixes / features).

Breaking Changes?

[ ] Yes, there are breaking changes.
[x] No, there are no breaking changes.

Set as draft - realized that I made an error that needs to be addressed in a new commit.


add BINDIR in Makefile

the obj and bin files are stored under source code directory,
would like to add a BINDIR in Makefile,
to support parralle make for different target.

BINDIR is normally used to install executables to.  http-parser however doesn't produce executables, only libraries.  (I'm ignoring the test runner; it isn't installed.)
Perhaps what you're really asking for is VPATH support for out-of-tree builds?

maybe BINDIR is not a good name,
this is where obj files are stored,    when running parralle make for different arch,
directory names are different :
change looks like:
BINDIR ?= $(CURDIR)
test: $(BINDIR)/test_g $(BINDIR)/test_fast
$(HELPER) $(BINDIR)/test_g$(BINEXT)
$(HELPER) $(BINDIR)/test_fast$(BINEXT)
$(BINDIR)/test_g: $(BINDIR)/http_parser_g.o $(BINDIR)/test_g.o
$(CC) $(CFLAGS_DEBUG) $(LDFLAGS) -o $@ $^

maybe BINDIR is not a good name,  it is the direcotry where obj and exe files are stored during make,
it can solve problem when running parralle make with different arch,
change looks like:
BINDIR ?= $(CURDIR)
test: $(BINDIR)/test_g $(BINDIR)/test_fast
$(HELPER) $(BINDIR)/test_g$(BINEXT)
$(HELPER) $(BINDIR)/test_fast$(BINEXT)
$(BINDIR)/test_g: $(BINDIR)/http_parser_g.o $(BINDIR)/test_g.o
$(CC) $(CFLAGS_DEBUG) $(LDFLAGS) -o $@ $^
$(BINDIR)/test_g.o: test.c http_parser.h Makefile
$(CC) $(CPPFLAGS_DEBUG) $(CFLAGS_DEBUG) -c test.c -o $@
$(BINDIR)/http_parser_g.o: http_parser.c http_parser.h Makefile
$(CC) $(CPPFLAGS_DEBUG) $(CFLAGS_DEBUG) -c http_parser.c -o $@

Right, that's traditionally solved with VPATH, although your approach works too for simple projects.
What you call BINDIR is usually called BUILDDIR or sometimes just B.  Pull requests welcome but note that I plan on switching to cmake sometime this year and cmake gives you out-of-tree builds for free.


4.x: Fix all TLS tests and examples

Description
Merge of https://github.com/helidon-io/helidon/pull/7364 from main
Fixes https://github.com/helidon-io/helidon/issues/7276
Documentation
None

Can you add a README.md in the directories that you've updated, see webserver/tests/mtls as an example:
webclient/tests/http1
webclient/tests/http2
webserver/tests/http2
webserver/tests/upgrade

There are many more places where we have pkcs12 files in the workspace, can you also add README.md intructions for the following directories:
webclient/tests/webclient
webclient/http2
config/encryption
security/providers/http-sign
tests/apps/bookstore/bookstore-se
webserver/tests/upgrade
common/key-util
examples/security/webserver-signatures
examples/webserver/tls
examples/webserver/mutual-tls
examples/webserver/protocols
examples/microprofile/tls

And there there is archetypes/helidon ; can you create an issue against the archetype about generating README.md instruction for how to generate keystore.p12.

Adding readmes for each module that contains p12 goes beyond the scope of this task I think. I can add a readme in every module that I have modified. Those are modules where we use an HTTP client that has to trust the server. We should create other issues for that.

There is some issue with a reload of the server certificates:
Error:  Failures:
Error:    MtlsTest.testTlsReload:180
Expected: is "X.509:CN=Helidon-Test-Server-Secondary|X.509:CN=Helidon-Test-CA"
     but: was "X.509:CN=Helidon-Test-Server|X.509:CN=Helidon-Test-CA"

I don't really understand how my changes could affect to it. The content of the relioded second-valid/server.p12 is:
jbescos@jbescos-PORTEGE-X40-K:~/workspace/helidon/webserver/tests/mtls$ keytool -list -keystore src/main/resources/second-valid/server.p12 -storepass password -v
Keystore type: PKCS12
Keystore provider: SUN

Your keystore contains 3 entries

Alias name: 1
Creation date: Aug 18, 2023
Entry type: PrivateKeyEntry
Certificate chain length: 2
Certificate[1]:
Owner: CN=Helidon-Test-Server-Secondary
Issuer: CN=Helidon-Test-CA
Serial number: 4aae1a76f2ec6deb56d0a0c5c170c27c7315b63f
Valid from: Fri Aug 18 09:42:55 CEST 2023 until: Tue Jun 01 09:42:55 CEST 2297
Certificate fingerprints:
	 SHA1: 63:7D:71:92:C5:9D:FF:2B:45:8E:01:DE:42:4F:58:48:15:77:CD:A7
	 SHA256: 32:53:95:12:8D:A7:14:B3:E9:38:4F:63:68:7F:64:FB:5E:1F:62:55:58:34:0A:AC:D3:2E:6C:FA:28:69:94:51
Signature algorithm name: SHA256withRSA
Subject Public Key Algorithm: 2048-bit RSA key
Version: 3

Extensions:

#1: ObjectId: <IP_ADDRESS> Criticality=false
AuthorityKeyIdentifier [
[CN=Helidon-Test-CA]
SerialNumber: [    5e734591 0d1cba03 14bc81a9 35b1d76e 8e4db0a4]
]

#2: ObjectId: <IP_ADDRESS> Criticality=false
SubjectAlternativeName [
  DNSName: localhost
  IPAddress: <IP_ADDRESS>
]

#3: ObjectId: <IP_ADDRESS> Criticality=false
SubjectKeyIdentifier [
KeyIdentifier [
0000: E9 10 69 58 BF 0F AE E7   8E BD 1C 53 61 77 F8 95  ..iX.......Saw..
0010: 85 BA 51 75                                        ..Qu
]
]

Certificate[2]:
Owner: CN=Helidon-Test-CA
Issuer: CN=Helidon-Test-CA
Serial number: 5e7345910d1cba0314bc81a935b1d76e8e4db0a4
Valid from: Fri Aug 18 09:42:27 CEST 2023 until: Tue Jun 01 09:42:27 CEST 2297
Certificate fingerprints:
	 SHA1: 6B:E4:99:79:86:47:0E:DB:A0:BA:9A:67:50:16:E4:F2:15:48:E2:AA
	 SHA256: A2:EA:F2:5B:32:30:FA:55:41:63:20:95:E5:F3:BA:5B:E7:2A:BF:39:82:0E:6C:5E:77:DD:FF:28:4E:A5:FF:BE
Signature algorithm name: SHA256withRSA
Subject Public Key Algorithm: 2048-bit RSA key
Version: 1

*******************************************
*******************************************

Alias name: truststore
Creation date: Aug 18, 2023
Entry type: trustedCertEntry

Owner: CN=Helidon-Test-Client
Issuer: CN=Helidon-Test-CA
Serial number: 4e35da95027ff2afe83affc461f6a8847541c0b8
Valid from: Fri Aug 18 09:42:38 CEST 2023 until: Tue Jun 01 09:42:38 CEST 2297
Certificate fingerprints:
	 SHA1: C1:52:BB:F4:28:35:57:1D:3A:05:4A:D1:97:71:A4:13:3B:90:20:AB
	 SHA256: B2:F5:A0:08:1A:06:A8:14:A6:18:55:F8:C8:E3:B8:E0:83:41:2F:E8:55:17:22:DC:33:E6:CD:E6:0C:73:CC:F5
Signature algorithm name: SHA256withRSA
Subject Public Key Algorithm: 2048-bit RSA key
Version: 3

Extensions:

#1: ObjectId: <IP_ADDRESS> Criticality=false
AuthorityKeyIdentifier [
[CN=Helidon-Test-CA]
SerialNumber: [    5e734591 0d1cba03 14bc81a9 35b1d76e 8e4db0a4]
]

#2: ObjectId: <IP_ADDRESS> Criticality=false
SubjectAlternativeName [
  DNSName: localhost
  IPAddress: <IP_ADDRESS>
]

#3: ObjectId: <IP_ADDRESS> Criticality=false
SubjectKeyIdentifier [
KeyIdentifier [
0000: 27 ED 67 5A EC DA BF F9   6D A9 9F 16 8B 83 74 FA  '.gZ....m.....t.
0010: 2B 77 AB 7A                                        +w.z
]
]

*******************************************
*******************************************

Alias name: truststoreca
Creation date: Aug 18, 2023
Entry type: trustedCertEntry

Owner: CN=Helidon-Test-CA
Issuer: CN=Helidon-Test-CA
Serial number: 5e7345910d1cba0314bc81a935b1d76e8e4db0a4
Valid from: Fri Aug 18 09:42:27 CEST 2023 until: Tue Jun 01 09:42:27 CEST 2297
Certificate fingerprints:
	 SHA1: 6B:E4:99:79:86:47:0E:DB:A0:BA:9A:67:50:16:E4:F2:15:48:E2:AA
	 SHA256: A2:EA:F2:5B:32:30:FA:55:41:63:20:95:E5:F3:BA:5B:E7:2A:BF:39:82:0E:6C:5E:77:DD:FF:28:4E:A5:FF:BE
Signature algorithm name: SHA256withRSA
Subject Public Key Algorithm: 2048-bit RSA key
Version: 1

*******************************************
*******************************************

Although the server was restarted, it seems the client socket was still the same. I had to modify the test to create a new WebClient.

Although the server was restarted, it seems the client socket was still the same. I had to modify the test to create a new WebClient.

This is very likely because the client connection is cached. You can try to use a close header on the server side:
res.header(Http.Headers.CONNECTION_CLOSE);

@romain-grecourt I have solved the conflicts.


Cannot automatically complete brackets

Issue Type: Bug
After I update the vs code, the python plug-in's auto completion bracket function will not work properly
Extension version: 2020.8.101144
VS Code version: Code 1.47.3 (91899dcef7b8110878ea59626991a18c8a6a1b3e, 2020-07-23T13:12:49.994Z)
OS version: Windows_NT x64 10.0.19042

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8300H CPU @ 2.30GHz (8 x 2304)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (7.80GB free)

Process Argv

Screen Reader
no

VM
0%

@zhiyiYo, thanks for letting us know about this.  Please provide one or more examples of code where brackets are not getting automatically added.

Because we have not heard back with the information we requested, we are closing this issue for now. If you are able to provide the info later on then we will be happy to re-open this issue to pick up where we left off.


DO NOT MERGE stress

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-eventhub


Lots of things!

Tested Exception raised in tasks -> It works

Freeze and pull tasks now use the same function (instead of having twice the same code)

Modified API to handle any exception (not just Runtime or Type)

Read max task delay and admin email from config

Modified list_file method = Now return a list of dict
(Dict looks like this : {'Path': path_to_file}) -> Need to handle it in the CLI

Modified list_file method =  Added --full parameters
(Return list of dicts with all info from rclone lsjson (size, mtime, etc...))

Default behaviour for repo is non-freezable: need freezable: True in repo conf to enable

Added check for mtime (independant of atime)
If there is a diff between local fine and remote file, it will not be deleted even with the --force option

Added tests for freezing disabled, mtime check, and --force freeze.

oh zat is wonderfoul! I'll look at it toumoro!
Thank you


AF is not equal with AC/AN in the variant information section

Examples:
1-13327-G-C
1-13327-G-A
1-13302-C-T

This will be fixed as part of migration to new schema.

We can close it, it's essentially implemented in the coming branch mig_variant.


Parameters used for training shape predictor

@davisking Not sure if this is the right place to post. Apologies in advance if it isn't.
Coming to the issue, the current landmark detector is running in 20 ms on a 1.2 GHz Android phone. We need to get it under 5ms to achieve 30 fps. One of the things we intend to do is train and run shape predictor on a subset of these 68 points. (Because we're not using all sixty eight in our pipeline)
Few questions in this regard.

Am I right in stating that the current model  is trained using this ibug dataset

Does it use the same default parameters listed in the file for training: http://dlib.net/dlib/image_processing/shape_predictor.h.html

            _cascade_depth = 10;
            _tree_depth = 4;
            _num_trees_per_cascade_level = 500;
            _nu = 0.1;
            _oversampling_amount = 20;
            _feature_pool_size = 400;
            _lambda = 0.1;
            _num_test_splits = 20;
            _feature_pool_region_padding = 0;
            _verbose = false;
            _num_threads = 0;

Any other hacks/tips to improve performance in the context of Android/mobile would be greatly appreciated :)

Yes, that's the data it's trained on.  It used something similar to the default parameters.  Just try some things and see what works best.

@davisking Thanks for the quick reply :)
The trainer is killed after a few minutes without any log messages.
ubuntu@ip-172-31-24-103:~/dlib/examples/build$ ./train_shape_predictor_ex ibug_300W_large_face_landmark_dataset/
Killed

saadhana@Abhilashs-MacBook-Air-4 ~/dlib-android/jni/dlib/examples/build((HEAD detached at 40ea658)) $ ./train_shape_predictor_ex /Users/saadhana/Downloads/ibug_300W_large_face_landmark_dataset/
Fitting trees...
Killed: 9

I tried running on machines with 4GB and 16GB RAM. It ran out of memory on both of these machines. Could you please share the machine configuration you used while training?

My machine has 32GB of ram.


change shebang to be more compatible

This
https://github.com/serde-rs/serde/blob/0676673ca5efef667d10a7b63b5b078a8d64e43a/precompiled/build.sh#L1
should be
#!/usr/bin/env bash

in order to maximize compatibility. On many more nieche systems /bin/bash does not exist (/bin/sh would be fine!), but /usr/bin/env and env would be able to find bash on most if not all systems if bash is present at all.

Good catch. I'd merge a PR that changes this

No problem. Another friction point with the script is that it only works with a rustup cargo, not with native cargo (due to reliance on +nightly). I don't know how to robustly detect if cargo is infact rustup:
+ cargo +nightly build --manifest-path bin/Cargo.toml --bin serde_derive --profile precompiled -Z unstable-options -Z build-std=std,panic_abort -Z build-std-features=panic_immediate_abort --target x86_64-unknown-linux-musl --out-dir serde_derive
error: no such command: `+nightly`

	Cargo does not handle `+toolchain` directives.
	Did you mean to invoke `cargo` through `rustup` instead?

I think the rustup issue will be fixed by whatever resolves #2575


dartsass: Import CSS without extension at compile time

Applicable to Dart Sass only:

Sass imports with the .css extension indicate a plain CSS @import.
Sass imports without the .css extension are imported at compile time.

Fixes #10592

bep, I apologize for the delay on this. I have been having intermittent failures when testing. It turns out the tests were intermittently skipped due to:
if !dartsass.Supports() {
	t.Skip()
}

That is not your problem.
I had been running the tests (a) within VS Code, and (b) with dart-sass-embedded installed via Brew. There are no problems when dart-sass-embedded is installed in /user/local/bin. I still haven't worked out the correct settings in VS Code to address this, so I'll stick with the install in /user/local/bin.

Bump ... Is this still an issue/relevant?

Yes, still relevant, got distracted. I can look at it again later today (much later today).


Implement Euler Discrete Scheduler

Hi @LaurentMazare,
this PR aims at integrating the Euler Discrete Scheduler into this repository, solving the first task mentioned in #23.
The implementation should contain all the features of the python version. Please notice that, differently from the other schedulers currently supported, here init_noise_sigma is not equal to 1.0. Also, this scheduler requires to scale the denoising model input. We might say that the other 3 schedulers supported by diffusers-rs are a "special" case.
Therefore, to run the stable diffusion pipeline with this scheduler, one needs to multiply the initial latents by init_noise_sigma here  (Python version)
let mut latents = Tensor::randn(
    &[bsize, 4, sd_config.height / 8, sd_config.width / 8],
    (Kind::Float, unet_device),
) * scheduler.init_noise_sigma();

and to scale the model input after this line (Python version)
let latent_model_input = scheduler.scale_model_input(latent_model_input, timestep);

HF's python version makes everything consistent implementing init_noise_sigma() and scale_model_input() in all the schedulers.  If you wish,  I can fix the example in this PR (and, thereby, add a scale_model_input method to all the schedulers, which only returns sample when no scaling is needed, i.e. what HF does). Otherwise, I can open a new one.
Overall, I believe this PR is self-contained regardless of the pipeline examples.
EDIT: the above text is a little dense.
To sum up:

This PR implements Euler discrete, porting all the features from HF diffusers.

the stable diffusion pipeline examples require  to be updated, as they assume that init_noise_sigma is 1.0 and that the denoising model input doesn't need to be scaled. I propose to copy what HF does, i.e. making all the schedulers implement a getter for init_noise_sigma (done already for all the schedulers I implemented in the past 2 weeks) and a scale_model_input method which simply returns sample when no scaling is needed :)
pub fn scale_model_input(&self, sample: Tensor, timestep: f64) -> Tensor {
     sample
}

However, this is, in my opinion, outside of the scope of this PR as it only aims at adding a new scheduler.

EDIT 2: I did what I mentioned in the second bullet in this branch. If you agree, I will open a new PR once this gets merged (assuming you find these valuable contributions to diffusers-rs :)  )

Thanks for the PR, I just merged it, adding the scale_model_input step in the examples seems sensible so that such scheduler could more easily be used.

Sure, I will open a new PR from that other branch then! :)

Done (#35) .

Hi, I'm getting blurry results with this one. I multiplied latents by scheduler.init_noise_sigma() and also added let latent_model_input = scheduler.scale_model_input(latent_model_input, timestep); after let latent_model_input = Tensor::cat(&[&latents, &latents], 0);. Anything else I could be missing? Thanks.

Hi @sssemil,
try with this main file (comes from the PR I opened yesterday). I just clone the repo again, copy-pasted this main and run the diffusion pipeline as follows
cargo run --example stable-diffusion --features cl
ap -- --prompt "A very rusty robot holding a fire torch." --sd-version=v1-5 --n-steps=15  --cpu all

this is the output I got:

You can check whether the produced image is "appropriate for this scheduler" (compared to the python version) using, for instance, the following python snippet
import torch
from diffusers import StableDiffusionPipeline, EulerDiscreteScheduler

pipe = StableDiffusionPipeline.from_pretrained("runwayml/stable-diffusion-v1-5", torch_dtype=torch.float16)

pipe.scheduler = EulerDiscreteScheduler()

pipe = pipe.to("cuda")
prompt = "A very rusty robot holding a fire torch."
image = pipe(prompt, num_inference_steps=15).images[0]

Ah, I see. Then this is just how this method works :) Euler A. just looks better in general.

Yes, in my experience Euler A. works better. Have you given a shot to my implementation of that (see PR open)? Does it work as you'd expect ?


[DRAFT] Discussion for mnemonic phrases with error correcting codes

Discussion issue for "mnemonic phrases with error correcting codes"

As you call out in the spec, the CIP-39 phrases are still valid Bip-39 phrases, and Cip-39 wallets should still accept Bip-39 phrases. So this seems like a strictly better option assuming we're confident Cip-39 is still sufficiently strong cryptographically.


Do you want to have the enhancement of existing game ? 😀 Describe yourself..
Game enhancement: add footer with a neat structured
Describe the solution you'd like
Solution steps
using html,css adding footer
Select program in which you are contributing
No response
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

i want to work on this issue under gssoc 24

hello, please assign this issue to me under GSSOC'24.


Starting a new project on .NET 6

Hi,
I'm starting a new project on .NET 6 and would like to use the newest version 6.0.0 of the DocuSign.eSign.dll, however it's still in prerelease as an RC.
Any guidance here?  Should I go with the RC with a full GA release expected soon?

Hi @pvanroos , we would suggest as you are just starting out with your project you can start testing with the latest rc release, we have made nice improvements over it and we are hoping there won't be any breaking changes when we go full GA.
Happy Coding!
Bharat

Not trying to pressure you but is GA expected in the next 2 months?

Sorry for the late reply here.  This went GA in September.

This appears to be resolved; closing the issue.


Use Azure codicon

Cherry pick #476 to main branch

Fixes https://github.com/microsoft/vscode-azuretools/issues/978


Boilerplate/Theme collection

I started porting nice themes from other static site generator to Statinamic.
I'll use this issue to track all avaiable boilerplate/theme avaiable until we have a page delicated for this
If you created a nice theme for Statinamic, please comment below and I'll add it here.
Lumen

Ported from gatsby-starter-lumen
Repo: https://github.com/thangngoc89/statinamic-theme-lumen
Demo: https://khoanguyen.me/statinamic-theme-lumen/

I love it.
Very soon I'll be able to help you with this!

We should find a smart way to clone a boilerplate/theme during the setup.
Btw, maybe we should find a smart way to extends/override a theme without touching it. This way we can call that "theme".

Absolutely. But it should be after 1.0

We should think about that before 1.0, to avoid a huge refactor to be able to handle this, but I think a trick with webpack alias can make this very easy :)

Closing this until we handle #508. We might revisit this when we will have a better official support for that.


Usage for intraday trading

How to use this library for intraday trading? It originally uses just daily data, but what if I need to make decisions at, let's say, hourly rate?

Please treat the data as time series. For intraday trading, a change would be adding penalty for holding shares before the market closing.   Hourly is not a good set, let us consider minutes, than you may want to encourage selling, like the 30 minute window before the market closes.

Please treat the data as time series. For intraday trading, a change would be adding penalty for holding shares before the market closing.   Hourly is not a good set, let us consider minutes, than you may want to encourage selling, like the 30 minute window before the market closes.

Thanks for your response @XiaoYangLiu-FinRL, what would be the best way to add such penatly?

Adding a - frac * (values of shares) term would be good.  E.g., frac = t/T, where T = 30; t = 1, 2, 3, ..., 30.


add a hook for tab onClose

this is a breaking change for the API
You now instantiate tabs like Tabs({ onSelected, onClose, prepend, append })
I wanted an onClose as well as onSelected, so the choice was to make the API ugly or to change it a fair amount.

Reminder to self

[ ] update patchbay to use this when it's publish
[ ] publish as a major versoin

cc @dominictarr , @evbogue

@mixmix minbase uses hypertabs-vertical, so I don't think this will break anything for me.

Cool, thanks @evbogue. I remember working to get hypertabs to behave vertically. Interested to hear what you've adapted. Would be cool to get the best ideas shared.

@mixmix I think hypertabs-vertical changes some rows to columns or columns to rows. https://github.com/cryptix/hypertabs-vertical

how is this a breaking change? doesn't this just pass an extra method, what does this remove?

Changes the api to just accept a single argument. Existing installations
will break.
Happy to take advice on  what degree of semver change that should be
On Mon, 11 Sep 2017, 04:27 Dominic Tarr<EMAIL_ADDRESS>wrote:

how is this a breaking change? doesn't this just pass an extra method,
what does this remove?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hyperhype/hypertabs/pull/10#issuecomment-328353911,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACitng_EoadcyUIOr27xKb03p06xG6qGks5shA3xgaJpZM4PPe8g
.

oh got it, make it a major version, of course

merged into 5.0.0


Update fork

Pull Request Test Coverage Report for Build 102484119

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.793%

Totals

Change from base Build 99900025:
0.0%

Covered Lines:
1088

Relevant Lines:
1208

💛  - Coveralls


apps/auracast: Make the code compile on native target

This fixes auracast native build by disabling GPIO button functionality.

there's already hal_gpio for native so you can just add missing APIs there

@sjanc PING

@sjanc


Indonesian Translation

Hi There,
Would you like to add "id" folder in /fastlane/metadata/android so I can help you to translate it into Indonesian.

Solved

Thanks!


Add quotient, truncate and fractional calculation methods

This PR adds the quotient, truncate and fractional methods. This is a small PR but theses methods are convenient.
Note that the mod method is called mod, but should instead be called rem for remainder to be consistent. Is it worth the rename? (It would be a breaking change)

Changing to rem seems sensible. Not sure about the name quo, does it have any precedent?

Maple uses quo apparently.
Isn't the more common approach here to have a single div (name is not important) function that returns both the quotient and remainder?

Why not, maybe that we could name it euclidian_div or something like that? Should we replace both mod and quo with this function would be a breaking change too unfortunately

Changing to rem seems sensible. Not sure about the name quo, does it have any precedent?

It is commonly named like that, I see that on several calculators and others librairies.
The issue with the mod naming is that, for instance a modulus b is a +kb, where k is a relative integer. Whereas rem really means the reminder in the euclidian division of a and b, it is a specific modulus.

Maple uses quo apparently.
Isn't the more common approach here to have a single div (name is not important) function that returns both the quotient and remainder?

Why not, maybe that we could name it euclidian_div or something like that (shorter)? If we replace both mod and quo with this function would be a breaking change too unfortunately

I think mod is used often enough to warrant a separate function.
I'm just imagining that if you want the quotient you probably want both of them. And if you don't, it's very easy to compute with floor.

I understand the need of a separate function which reunites both. But in some cases, the quo function is needed, without the mod (rem) function. Of course it is way easier to replicate its behavior with mod, but if there was a quo function in the first place but no rem function, it would also be trivial to replicate the rem function. I think that for clarity/readability we should have both functions.
For another euclidian_div function, this is a good idea. I'm going to implement it.

I'm against an extra euclid_div function. You can just call the two functions, separately. I'm in favor of renaming "mod" to "rem" though. About "quo", I'm still unsure.

I'm against an extra euclid_div function. You can just call the two functions separately.

This would allow something like #let (rem, quo) = calc.euclid_div(x, y) in only one call, but I understand that it makes an extra function which might not be really useful.

I'm in favor of renaming mod to rem though. About quo, I'm still unsure.
I'm going to rename it then. For quo, in the tools that I use it is always called quo, I don't have any better naming.

okay, then let's remove euclid_div and let quo be as is


Integration tests to verify transport related fixes

[ ] 1. To verify #903
[ ] 2. set / get Status Code, reason phrase

We've added various integration tests now.


Itk obj mesh io m number of point pixels default

When reading the mesh information, `m_NumberOfPointPixels` is computed based
on the number of lines starting with the code `vn`. However, this was
then overriden and set equal to `m_NumberOfPoints`.
This lead to a crash when these lines were actually read in
the function `ReadPointData()` because not enough memory was allocated.

Test output:
OBJMeshIO (0x259bf10)
  RTTI typeinfo:   itk::OBJMeshIO
  Reference Count: 1
  Modified Time: 60
  Debug: Off
  Object Name:
  Observers:
    none
  AbortGenerateData: Off
  Progress: 0
  FileName: /home/vsts/work/1/s-build/ExternalData/Modules/IO/MeshOBJ/test/Baseline/bunny.obj
  FileType: ASCII
  ByteOrder: OrderNotApplicable
  Point dimension: 3
  Point component type: float
  Cell  component type: long
  Number of point pixel components: 3
  Number of cell  pixel components: 1
  Number of points: 2503
  Number of cells: 4968
  Number of point pixels: 4968
  Number of cell pixels: 0
  Point pixel type: vector
  Cell  pixel type: scalar
  Point pixel component type: float
  Cell  pixel component type: float
Write file /home/vsts/work/1/s-build/Testing/Temporary/bunny.obj failed

itk::ExceptionObject (0x26561f0)
Location: "unknown"
File: /home/vsts/work/1/s/Modules/IO/MeshOBJ/src/itkOBJMeshIO.cxx
Line: 597
Description: itk::ERROR: OBJMeshIO(0x259bf10): MNI Mesh required point normals

Failure for itk::Mesh
OBJMeshIO (0x2650630)
  RTTI typeinfo:   itk::OBJMeshIO
  Reference Count: 1
  Modified Time: 22486
  Debug: Off
  Object Name:
  Observers:
    none
  AbortGenerateData: Off
  Progress: 0
  FileName: /home/vsts/work/1/s-build/ExternalData/Modules/IO/MeshOBJ/test/Baseline/bunny.obj
  FileType: ASCII
  ByteOrder: OrderNotApplicable
  Point dimension: 3
  Point component type: float
  Cell  component type: long
  Number of point pixel components: 3
  Number of cell  pixel components: 1
  Number of points: 2503
  Number of cells: 4968
  Number of point pixels: 4968
  Number of cell pixels: 0
  Point pixel type: vector
  Cell  pixel type: scalar
  Point pixel component type: float
  Cell  pixel component type: float
Write file /home/vsts/work/1/s-build/Testing/Temporary/bunny.obj failed

itk::ExceptionObject (0x2a61eb0)
Location: "unknown"
File: /home/vsts/work/1/s/Modules/IO/MeshOBJ/src/itkOBJMeshIO.cxx
Line: 597
Description: itk::ERROR: OBJMeshIO(0x2650630): MNI Mesh required point normals

Failure for itk::QuadEdgeMesh

Thanks for uploading the data on girder. This PR correct one problem, but as you can see there is another problem that is still present (see comment on discourse here).

@fbudin69500 having multiple normals per vertex allows sharp edges. In bunny.obj for example, each face uses the same normal for all its vertices, making them flat. If you want to limit a vertex to one normal, you can take the average of all normals used with this vertex.

@maekclena Do you mean that each vertex has one normal per polygon/triangle they belong to which leads to multiple normals per vertex?

@fbudin69500 yes, at least in bunny.obj.

I just rebased this.

I just looked into this, and this requires proper support for per-vertex and per-face normals. It will require understanding mesh IO and non-trivial refactoring of this class.

Updating PR to just exercise reading OBJ with face normals. We get crash avoidance during reading for now, and leave writing normals for another day.

Only itkTimeVaryingBSplineVelocityFieldPointSetRegistrationTest failed, merging.


Cannot load driver

When selecting Kernel (Manual Map) or trying to manually inject the driver (sc create & sc start) I get the following error:
2017-11-27 22:02:36 *NORMAL*     Started on Windows <IP_ADDRESS>99 x64. Driver status: 0xC0000225
2017-11-27 22:02:41 *ERROR*      Failed to load BlackBone driver:

Windows cannot verify the digital signature for this file. A recent hardware or software change might have installed a file that is signed incorrectly or damaged, or that might be malicious software from an unknown source.

I got the executable from UC & the driver from Blackbone repo.

Enable test mode

well, if this is usable only with test mode, then it is kinda useless :b

Sign it yourself and use without test mode.


Bump up version 1.0.0

It is a breaking change to bundle the LIBLINEAR codes, so I would like to increase the major version of Numo::LIBLINEAR.

I have confirmed that this branch is installed as version 1.0.0.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (fee8480) 95.69% compared to head (05bcbc3) 95.69%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #910   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support building without AVX2

Could there be a way to fall back to a slower codepath if AVX2 is not available?
For now this could be a Cargo feature and later it could be detected automatically by https://github.com/rust-lang/rfcs/pull/2045.

Using AVX can be a fallback option. I will investigate how to implement this kind of fallback processing.

Perhaps it helps to take a look how bytecount does it.

Resolved by #65


Update some variables to fix login issue

Safari Books Online has been changed to learning.oreilly.com. I've updated the source code to work with new domain.

Thank you for this PR. I pushed a new version (622529b) that includes your changes.
Thanks again for the support! :heart:


Cannot update

Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.3.10
VBoxService inside the vm claims: 5.0.4
Going on, assuming VBoxService is correct...
GuestAdditions versions on your host (5.0.10) and guest (5.0.4) do not match.
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   4.3.10
stdin: is not a ttythe vm claims: 5.0.4
mount: block device /tmp/VBoxGuestAdditions.iso is write-protected, mounting read-only
Installing Virtualbox Guest Additions 5.0.4 - guest version is 5.0.4
stdin: is not a tty
Verifying archive integrity... All good.
Uncompressing VirtualBox 5.0.4 Guest Additions for Linux............
VirtualBox Guest Additions installer
Removing installed version 5.0.4 of VirtualBox Guest Additions...
Stopping VirtualBox Additions ...fail!
(Cannot unload module vboxguest)
Removing existing VirtualBox DKMS kernel modules ...done.
Removing existing VirtualBox non-DKMS kernel modules ...done.
Stopping VirtualBox Guest Addition service  ...done.
Copying additional installer modules ...
Installing additional modules ...
Removing existing VirtualBox DKMS kernel modules ...done.
Removing existing VirtualBox non-DKMS kernel modules ...done.
Building the VirtualBox Guest Additions kernel modules ...done.
Doing non-kernel setup of the Guest Additions ...done.
You should restart your guest to make sure the new modules are actually used

Installing the Window System drivers
Could not find the X.Org or XFree86 Window System, skipping.
An error occurred during installation of VirtualBox Guest Additions 5.0.4. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.

I'm not exactly sure what's wrong but I think it's this line:

Stopping VirtualBox Additions ...fail!
(Cannot unload module vboxguest)

Same story here. Can't figure it out.
Copy iso file /Applications/VirtualBox.app/Contents/MacOS/VBoxGuestAdditions.iso into the box /tmp/VBoxGuestAdditions.iso
Installing Virtualbox Guest Additions 5.0.14 - guest version is 5.0.10
Verifying archive integrity... All good.
Uncompressing VirtualBox 5.0.14 Guest Additions for Linux............
VirtualBox Guest Additions installer
Removing installed version 5.0.10 of VirtualBox Guest Additions...
Stopping VirtualBox Additions [FAILED]
(Cannot unload module vboxguest)
Removing existing VirtualBox non-DKMS kernel modules[  OK  ]
[  OK  ] VirtualBox Guest Addition service [  OK  ]
Copying additional installer modules ...
Installing additional modules ...
Removing existing VirtualBox non-DKMS kernel modules[  OK  ]
Building the VirtualBox Guest Additions kernel modules
Building the main Guest Additions module[  OK  ]
Building the shared folder support module[  OK  ]
Building the OpenGL support module[FAILED]
(Look at /var/log/vboxadd-install.log to find out what went wrong. The module is not built but the others are.)
Doing non-kernel setup of the Guest Additions[  OK  ]
You should restart your guest to make sure the new modules are actually used

Installing the Window System drivers
Installing X.Org Server 1.15 modules[  OK  ]
Setting up the Window System to use the Guest Additions[  OK  ]
You may need to restart the the Window System (or just restart the guest system)
to enable the Guest Additions.

Installing graphics libraries and desktop services components[  OK  ]
An error occurred during installation of VirtualBox Guest Additions 5.0.14. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.
==> default: Checking for guest additions in VM...
pid-file for killed process 40626 found (/Users/ijstokes/.vagrant.d/tmp/dns/daemon/vagrant-dns.pid), deleting.
vagrant-dns: process with pid 40788 started.
==> default: Restarted DNS Service
==> default: Setting hostname...
==> default: Configuring and enabling network interfaces...
Guest-specific operations were attempted on a machine that is not
ready for guest communication. This should not happen and a bug
should be reported.

Hi all. Did vbguest reboot the box after installing? At least the log from ijstokes does not show anything like this.
Which base box are you using?

After installing the box doesn't rebooted and continues to install all required modules. I'm using default PuPHPet box puphpet/ubuntu1404-x64.
GuestAdditions versions on your host (5.0.14) and guest (5.0.10) do not match.
Reading package lists...
Building dependency tree...
dkms is already the newest version.
linux-headers-3.19.0-25-generic is already the newest version.
0 upgraded, 0 newly installed, 0 to remove and 3 not upgraded.
Copy iso file C:\Program Files/Oracle/VirtualBox/VBoxGuestAdditions.iso into the box /tmp/VBoxGuestAdditions.iso
mount: block device /tmp/VBoxGuestAdditions.iso is write-protected, mounting read-only
Installing Virtualbox Guest Additions 5.0.14 - guest version is 5.0.10
Verifying archive integrity... All good.
Uncompressing VirtualBox 5.0.14 Guest Additions for Linux............
VirtualBox Guest Additions installer
Removing installed version 5.0.10 of VirtualBox Guest Additions...
Stopping VirtualBox Additions ...fail!
(Cannot unload module vboxguest)
Removing existing VirtualBox DKMS kernel modules ...done.
Removing existing VirtualBox non-DKMS kernel modules ...done.
Stopping VirtualBox Guest Addition service  ...done.
Copying additional installer modules ...
Installing additional modules ...
Removing existing VirtualBox DKMS kernel modules ...done.
Removing existing VirtualBox non-DKMS kernel modules ...done.
Building the VirtualBox Guest Additions kernel modules ...done.
Doing non-kernel setup of the Guest Additions ...done.
You should restart your guest to make sure the new modules are actually used

Installing the Window System drivers
Could not find the X.Org or XFree86 Window System, skipping.
An error occurred during installation of VirtualBox Guest Additions 5.0.14. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.


New capi

@HipsterSloth
Can you please make sure this builds for you and doesn't break anything, then merge?
It's OK if the CAPI is incomplete. I just want to get this into master before you make your planned changes for today to avoid future merge conflicts. In fact, it's because the client_capi branch went unmerged for so long that I had to make this new branch.

Thanks for doing this clean-up. There was one minor build break for the SteamVR plugin. I just did a quick smoke test and things appear to work so I'm going to merge this up to master now.

Without looking, I'm guessing it worked before because psmoveprotocol was previously not marked as a PRIVATE dependency of psmoveclient, I guess the default is PUBLIC, so including/linking psmoveclient also gave you psmoveprotocol. I'm not sure it has to be PRIVATE. I just copied what was done in the previous capi branch.


This is more or less the same as the original implementation, except that we now use a strict regex (re.match) instead of str.startswith as our predicate. That should eliminate the possibility for potential prefix confusion by a malicious user, although said confusion would still be outside of our threat model (since it assumes CI compromise).

Hm, I've left a comment and github is not showing it to me. I wonder if github lost my comment. Let me know if this is the first comment you see of me in this PR.
Anyway, the discussion we had was like this: https://github.com/pypa/warehouse/pull/10753#discussion_r833505827
And I'm not seeing the same things as we discussed, but maybe I'm misunderstanding the goal of this PR

Ok so there's one question left from my other comment:

What happens if there's an @ in the branch/tag name ?

I believe this will confuse rsplit.

Ok so there's one question left from my other comment:

What happens if there's an @ in the branch/tag name ?

I believe this will confuse rsplit.

Yeah, I think you're right. I don't have a clean solution for this, then -- we could do an unambiguous parse by referencing the ref claim for the suffix, but that causes an abstraction leak (claim verification depending on other claims as inputs) that makes our verification step much more manual (and therefore user-error prone).
All of this is technically outside of our threat model (#10644), since the attack here assumes that the attacker manages to fool the repository owner into introducing a malicious workflow. Thoughts on just reverting back to the startswith or regex check?

Okay, this should do the trick: we're now using the ref claim to complement the job_workflow_ref check. This prevents any sort of prefix or suffix sneakiness, since the verification is now an exact string comparison for the pattern:
{workflow_path}@{ref}

...where {workflow_path} is computed solely from trusted setup data and {ref} is a verified claim containing a git-style ref.

Haha, I'm sorry for taking nice code away and forcing you to write less nice code :D (blame GitHub for mixing parts of strings together in a bigger string without any kind of escaping and expecting us to parse the bits ourselves)

It's greatly appreciated! Precision is a virtue, especially in anything remotely involving security 🙂


Add variable declaration node

I fixed issue #13 , and also implemented reassignment (AssignmentExpression node).

Hey @nm723 thank you for the PR!


Failed to resolve dependencies

Hi,
I cannot run exercise 8 due to the error

Hi @Taking1998,
did you follow the steps and then it fails or did you try to run the ex8 from the repo here? Can you share the content of your package.json with me? Either the dependency to the ui5-tooling-modules is outdated or chart.js version doesn't match?
BR, Peter

Hi @petermuessig ,
I practiced from front to back according to the steps, and then I couldn't see the effect until exercise 8.
Below is the package.json file of my application.
{   "name": "myapp",   "version": "1.0.0",   "description": "UI5 Application: myapp",   "author": "Taking1998",   "license": "Apache-2.0",   "scripts": {     "build": "ui5 build --clean-dest",     "build:opt": "ui5 build self-contained --clean-dest --all",     "start": "ui5 serve --port 8080 -o index.html",     "start:dist": "ui5 serve  --port 8080 -o index.html --config ui5-dist.yaml",     "ts-typecheck": "tsc --noEmit",     "lint": "eslint webapp"   },   "devDependencies": {     "@openui5/ts-types-esm": "1.104.0",     "@typescript-eslint/eslint-plugin": "^5.42.1",     "@typescript-eslint/parser": "^5.42.1",     "@ui5/cli": "^2.14.14",     "@ui5/ts-interface-generator": "^0.5.4",     "chart.js": "3.8.0",     "eslint": "^8.27.0",     "typescript": "^4.8.4",     "ui5-middleware-livereload": "^0.7.0",     "ui5-tooling-modules": "^0.7.12",     "ui5-tooling-transpile": "^0.2.0"   },   "ui5": {     "dependencies": [       "ui5-middleware-livereload",       "ui5-tooling-modules",       "ui5-tooling-transpile"     ]   } }

I used your package.json above and put it into the exercises/ex8/com.myorg.myapp deleted the package-lock.json and run npm i and it works for me. Maybe there is something wrong in the  ui5.yaml. Can you share this as well here? THX

Hi @petermuessig ,
Below is the package.json file of my application.
specVersion: "2.6"
metadata:
name: com.myorg.myapp
type: application
framework:
name: OpenUI5
version: "1.104.0"
libraries:
- name: sap.f
- name: sap.m
- name: sap.ui.core
- name: sap.ui.unified
- name: themelib_sap_horizon
builder:
customTasks:
- name: ui5-tooling-transpile-task
afterTask: replaceVersion
configuration: &transpileConfig
removeConsoleStatements: true
transpileAsync: true
transpileTypeScript: true
- name: ui5-tooling-modules-task
afterTask: replaceVersion
server:
customMiddleware:
- name: ui5-middleware-livereload
afterMiddleware: compression
configuration:
debug: true
extraExts: "xml,json,properties"
port: 35729
path: "webapp"
- name: ui5-tooling-transpile-middleware
afterMiddleware: compression
configuration:
*transpileConfig

Hi @Taking1998 ,
thanks for sharing, now I see the issue: you missed to add the ui5-tooling-modules-middleware into the customMiddleware section. This is needed to enable loading NPM packages with their name from the node_modules:
specVersion: "2.6"
metadata:
  name: com.myorg.myapp
type: application
framework:
  name: OpenUI5
  version: "1.104.0"
  libraries:
    - name: sap.f
    - name: sap.m
    - name: sap.ui.core
    - name: sap.ui.unified
    - name: themelib_sap_horizon
builder:
  customTasks:
  - name: ui5-tooling-transpile-task
    afterTask: replaceVersion
    configuration: &transpileConfig
      removeConsoleStatements: true
      transpileAsync: true
      transpileTypeScript: true
  - name: ui5-tooling-modules-task
    afterTask: replaceVersion
server:
  customMiddleware:
  - name: ui5-middleware-livereload
    afterMiddleware: compression
    configuration:
      debug: true
      extraExts: "xml,json,properties"
      port: 35729
      path: "webapp"
  - name: ui5-tooling-transpile-middleware
    afterMiddleware: compression
    configuration: *transpileConfig
  - name: ui5-tooling-modules-middleware
    afterMiddleware: ui5-tooling-transpile-middleware

With the ui5.yaml above, I added the missing ui5-tooling-modules-middleware and queued it after the ui5-tooling-transpile-middleware. See the last entry in the yaml above.
This should fix your issue and you should be able to continue... 😄
Best,
Peter

Hi @petermuessig ,
I went to look at the content of section 8.1 again. I missed adding the content of the above configuration, so the project cannot access the node_modules folder. After using the configuration content you gave, the project can introduce chart.js/auto, and the chart is successfully displayed, thank you very much.😄

Glad to here that this helped, closing the issue now...


Upgrade dependencies

Upgrade all go dependencies, including the changes in https://github.com/cert-manager/cert-manager/pull/6326.
I aim to backport this to the release-1.13 branch.
Kind
/kind cleanup
Release Note
NONE

/cherry-pick release-1.13

/retest

/unhold


Create HomePage

Contents of HomePage

Navigation Bar

Resources
Events
Contributors
Blog(alpha)

Developers Community's Logo
motto
Featured Events
Resources Page Overview
Contributors
Footer

Can you please elaborate what do you mean by resource page Overview.

Yeah, this section of HomePage will show how the resources page can be helpful for a computer science student
So a brief overview as a picture or a single card can be useful
Instead of a card we can do carousel also
Whichever is good

@pixan198 Please allow me to start working on the home page.

@NishantPathak2704 go ahead bro
Just discuss the mock-up with other members

@NishantPathak2704 go ahead bro
Just discuss the mock-up with other members

Sir can I also start with home page ?

Sure bro
Teamup with @NishantPathak2704


Another test issue

Test issue

/summarize
Problem: Lorem Ipsum
Solution: Dolor sil

/summarize
Problem: Lorem Ipsum
Solution: Dolor sil

It's stuck here
async function createStream(content: string): Promise<Readable> {
  return new Promise((resolve, reject) => {
    const readable = Readable.from(content, { encoding: 'utf-8' });

    readable.on('end', () => {
      resolve(readable);
    });
  });
}


Add @NonNull and @Nullable annotations to public methods.

I added @NonNull and @Nullable annotations to public methods, because they look like Platform Types to Kotlin.
Before

After

@kitwtnb Thank you for the contribution!


Add glossary to user guide

Also updated the activity diagram of markRoomsUnclean in developer guide and added more information on markRoomsUnclean command in user guide.

Codecov Report
Base: 74.78% // Head: 74.78% // No change to project coverage :thumbsup:

Coverage data is based on head (6f5a526) compared to base (47fd207).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##             master     #129   +/-   ##
=========================================
  Coverage     74.78%   74.78%
  Complexity      509      509
=========================================
  Files            76       76
  Lines          1511     1511
  Branches        167      167
=========================================
  Hits           1130     1130
  Misses          338      338
  Partials         43       43

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Memory cache

Would it be possible to add a memory cache mechanism to Tray to boost its performances.
Something inspired by https://github.com/tumblr/Remember ?

read it yesterday, too. After checking the implementation I'm not sure if it is possible with Tray because Remember doesn't work multiprocess, it's only thread safe.
Of cause you could swap out the SharedPreferences in Remember and replace them with Tray but losing the key feature of Tray seems wrong. The API itself is identical and Tray wouldn't benefit from this wrapper.
Also Remember doesn't detect changes in the SharedPreferences made with another API than Remember. This could lead to big problems when you use Remember and the SharedPreferences in the same project.
Side note: Having Guava as a hard dependency for a single class SharedPreferences wrapper seems wrong.
I'm currently working on dataChangedListeners which work multiprocess. Once they are complete I'm confident an in memory layer which updates itself based on the listeners is possible.

Hey Pascal,
I agree that guava as a dependency for an android app is a nightmare.
But I am not sure to understand why you couldn't build a memory cache
backed by tray. The concurrency can be handled well in java. There must be
something else that is blocking you, no ?
Le 2015-05-18 3:03 AM, "Pascal Welsch"<EMAIL_ADDRESS>a écrit :

read it yesterday, too. After checking the implementation I'm not sure if
it is possible with Tray because Remember doesn't work multiprocess, it's
only thread safe.
Of cause you could swap out the SharedPreferences in Remember and replace
them with Tray but losing the key feature of Tray seems wrong. The API
itself is identical and Tray wouldn't benefit from this wrapper.
Also Remember doesn't detect changes in the SharedPreferences made with
another API than Remember. This could lead to big problems when you use
Remember and the SharedPreferences in the same project.
Side note: Having Guava as a hard dependency for a single class
SharedPreferences wrapper seems wrong.
I'm currently working on dataChangedListeners which work multiprocess.
Once they are complete I'm confident an in memory layer which updates
itself based on the listeners is possible.
—
Reply to this email directly or view it on GitHub
https://github.com/grandcentrix/tray/issues/22#issuecomment-103004719.

It's only the multiprocess thing. If you have two processes, a SyncAdapter and your Application UI, there will be two instances of the memory cache. Although the memory cache is a singleton! So they are working on two different data sets and both instances wont know about each others changes.
Trays current implementation solves this problem by using IPC with the ContentProvider. This is slow but the data is always correct.

Humm, I  must get more familiar with Tray before going on this thread. I am
not sure I got all the pieces of the puzzle.
But it's in the pipeline, I should give it a try soon.
Thx for your answers,
Stéphane
2015-05-18 8:33 GMT-07:00 Pascal Welsch<EMAIL_ADDRESS>
It's only the multiprocess thing. If you have two processes, a SyncAdapter
and your Application UI, there will be two instances of the memory cache.
Although the memory cache is a singleton! So they are working on two
different data sets and both instances wont know about each others changes.
Trays current implementation solves this problem by using IPC with the
ContentProvider. This is slow but the data is always correct.
—
Reply to this email directly or view it on GitHub
https://github.com/grandcentrix/tray/issues/22#issuecomment-103105955.

A memory chace is now possible with new data changed listener which can be used to sync the database with the cache in the background

Listener work as expected. I'll focus on the memory cache now


Refinement by providing another zod object

I have troubles refining an existing schema by adding more constraints through another zod object. What I envision is something along the lines
const schema = z.object({
  name: z.string().min(3),
  age: z.number().min(18),
})

const refinement = z.object({
  name: z.string().max(10)
})

// Now
schema.refine(refinement)
// should be equivalent to
const schema = z.object({
  name: z.string().min(3).max(10),
  age: z.number().min(18),
})

This refine method is similar to the existing extend method, but should combine the assertions (and not overwrite them), and moreover ensure that refinement specifies a strict subset of the keys of schema.
Maybe refine as a word is already too overloaded, something like narrowDown or specialize maybe?

This would be great for defining different error messages in different scenarios.
Is there any way of doing at least that without having to define again the validations with .extends?

It's still not resolved as far as I can see.

...

Looking forward into this

...


ReplayMod shows player spinning randomly

Mod Name
ReplayMod
This issue occurs when only Valkyrien Skies, addons, and the mod I have specified are installed and no other mods

[ ] I have tested this issue and it occurs with only Valkyrien Skies, addons, and the mod I have specified

Minecraft Version
1.20.1
Mod Loader
Fabric
Issue description
When using replaymod with Eureka ships it seems like the player gets bugged leading to some weird spinning around + head rotations
Issue reproduction

Create a ship
Start recording
Stop recording and go to the replay viewer
When playing the replay see how your player seems to be glitchy

Logs
No response

Update here, seems like this happens for multiplayer aswell, tried in a server with a friend and we see each other spinning when riding the ship in the air and in water aswell

If it's a physics mod incompatibility, is there an option in the physics mod settings to fix this but still keep most of the physics?


fix(CategoryTheory/Sites): typos in module docstring of PreservesSheafification file

Thanks!
maintainer merge

Thanks!
bors merge


Style for whole table applied only on focused element

Hello,
I've noticed another bug. When I paste table to the suneditor, then I click on cell (no matter which cell) then I click "Ctrl + A" to mark whole content and add a style for it (for example bold text) then style is applied only for before focused element.
Demo on your environment: https://i.imgur.com/aDSBHr9.gifv
Table which I'm pasting is from: https://www.w3schools.com/html/html_tables.asp
Regards,
Adrian

It happens also when I click few elements in a row, then Ctrl + A to give style: https://i.imgur.com/uNpcSqW.gifv

Hi
This issue will be fixed in #291
Thank you.

The 2.28.1 version has been updated.
If this issue has not been resolved, please reopen this issue.
Thank you.


[Parse] Allow prefix operators containing / with /.../ regex literals

When lexing a regex literal, treat a prefix operator containing / the same as the unapplied infix operator case, where we tentatively lex. This means that we bail if there is no closing / or the starting character is invalid. This leaves binary operator containing / in expression position as the last place where we know that we definitely have a regex literal.
Additionally, expand the starting ) character heuristic to cover the entire range of the regex literal, ensuring to take escapes and custom character classes into account. This means we will not lex a regex literal if there is an unbalanced ) somewhere in the body.

@swift-ci please test

Source compat running on https://github.com/apple/swift/pull/42250

@swift-ci please test


Fix duplicate visit of param tag comments

It was a misplaced parenthesis.
Fixes #44422

@typescript-bot cherry-pick this to release-4.3


Predict new data without training.

Can I predict new data by trained model? Or I always should call "run" method?

Do you mean using a previous training model (using the save_model flag)?

exactly

Sorry, I have yet to implement that function since me personally had never use for it.
Just to get the feeling how do you envision such an interface? When I started looking into this problem I felt that I would probably need to split  pywFM.run into pywFM.train and pywFM.predict, also adding pywFM.load_model that's able to load a train model. Problem is that would probably hurt performance since we would need to run 2 different libfm commands: one with save_model flag another with load_model flag.
Another alternative would be a separate pywFM.train_model and pywFM.run_model  that trains and runs a model respectively.

I think first approach with train and predict methods more standard and clean, like in sklearn. Without that feature many ml techniques like stacking,blending become not so trivial.

I'm also leaning towards that approach, since it meets one of my todo points

Improve the save_model / load_model so we can have a more defined init-fit-predict cycle (perhaps we could inherit from sklearn.BaseEstimator)

This weekend I have a little bit of time and I will start to work on this branch (that will break BC, so bumping version). Feel free to also submit changes

Sorry, I'm not saw your todo .
Thank you very much for future work.

Hi @jfloff,
any advances into that direction? I just realized the issue mentioned by @nickflamel and this simply makes your (very cool) wrapper not usable in a production environment.
Btw. I tried the example from Rendle that is also on your README but the prediction is very bad. I guess this is because we don't have much data, but this kinda makes the example unsuitable^^.

I'm sorry, I haven't had time to dedicate to improving this. I realise that this feature would really improve running several different predictions, and I really want to improve it, but if I'm going to do it, I will inherit from sklearn.BaseEstimator right from the start (which takes a little bit more work).
I have a deadline for Monday. After that I'll dig into this, I promise! :)
The example is just to show how the API works, and what's the flow of libfm :)

It seems that [predict without a new train is not really supported at this moment] (https://github.com/srendle/libfm/issues/1). It seems that the functionality is not at 100% (e.g. not working for MCMC). I've also taken a look at libFM source code but I haven't had much success. Documentation is also lacking the save_model and load model function.
I'm going on a limb here and ping @thierry-silbermann here since he was responsible for save_model and load_model in libFM. Could you give us some insight on how we should proceed

Hi, here is how we could proceed to make a predict method:
https://github.com/jilljenn/TF-recomm/blob/master/forward.py#L22
Where the pickled elements are those:
https://github.com/jilljenn/TF-recomm/blob/master/fm_mangaki.py#L39

Want to try submit a PR for this?

Yes. It will look like this.
https://github.com/mangaki/mangaki/pull/549/files#diff-2b98b5dc82ffbac20dd8c88ce88d6b5cR65
I don't know why I had sometimes to use .A1 (conversion matrix to ndarray), sometimes not.
Can you consider casting model.weights and model.pairwise_interactions to NumPy arrays?

I don't see any problem with that.

5 years later, I finally made a scikit-learn estimator:
https://github.com/jilljenn/ktm/blob/master/fm.py#L25
It will be improved over the next few days, then I can copy it in your repo.


Add astefanutti to CodeFlare operator owners

/assign @anishasthana

/verify-owners

/ok-to-test


Clearingway is still showing as Trappingway in NAUR

Clearingway is still showing as Trappingway in NAUR in all channels.

Resolved


Arrangement and execution of the meeting (issue #262 related)

The goal is to prepare the arrange and execute a meeting.

Meeting has been executed


Does not work with PyTest

See this ticket in PyTest bug tracker: https://github.com/pytest-dev/pytest/issues/4882 .
I don't know how you built the distributable package, but it looks like this:
Metadata-Version: 2.0
Name: openapi-spec-validator
Version: 0.2.6
Summary: UNKNOWN
Home-page: https://github.com/p1c2u/openapi-spec-validator
Author: Artur Maciag
Author-email<EMAIL_ADDRESS>License: Apache License, Version 2.0
Platform: UNKNOWN
Classifier: Development Status :: 4 - Beta
Classifier: Intended Audience :: Developers
Classifier: Topic :: Software Development :: Libraries :: Python Modules
Classifier: Operating System :: OS Independent
Classifier: Programming Language :: Python :: 2.7
Classifier: Programming Language :: Python :: 3.4
Classifier: Programming Language :: Python :: 3.5
Classifier: Programming Language :: Python :: 3.6
Classifier: Programming Language :: Python :: 3.7
Requires-Dist: jsonschema (<3)
Requires-Dist: PyYAML (>=3.13)
Requires-Dist: six
Requires-Dist: pathlib

# OpenAPI Spec validator
...

So, it installs pathlib regardless of what version of Python is used (interestingly, it works differently with different versions of Python / setuptools), but Python 7.1 setuptools 40.6.2 don't work with your library (downloaded from pypi).

as per https://pypi.org/project/pathlib/ pathlib is a unmaintained backport and pathlib2 should be used

Well, don't tell me this... I didn't use it :) The problem is it is used by others, and you cannot avoid it. (I'm kind of surprised I'm the first one to run into this problem, it's like nobody used Swagger these days anymore?).

It was resolved with #68 Next release should fix it. Cheers.

Fix released with version 0.2.7`. Closing.


Update registry code from broker to clusterservicebroker

This PR updates the registry code and file organization to correctly categorize what was formerly "ServiceBroker" to "ClusterServiceBroker This makes way to introduce the namespaced resource "ServiceBroker" registry code. This PR is a functional NOOP.

Rebase needed

@pmorie rebased


Remove cron fork

We should contribute upstream with the changes to cron and remove the /cron folder once our contribution is accepted.

Opened a PR: https://github.com/robfig/cron/pull/473


Implement React Navigation

React Navigation is our best option, it seems.

is officially supported by people from Facebook and Exponent, gaining momentum
Wix's navigation is being rewritten for 2.0, can't really consider it seriously
AirBnB's navigation is also cool but in early stages

Implemented.


Vertical scrollbar flicker when notification shows in fullscreen mode

Added overflow: hidden to Streamus because 'transform' causes overflow to show even though the box model isn't affected.


text_style_transfer multy GPU

When I want to use more than 1 gpu running program, no matter how I add it, I cannot work at the same time with multiple gpus. I don't know why, but where can I modify it to ensure that multiple gpus can be used at the same time

At this point you can just use TF built-in functions for multi-gpu. For example, you may use tf.estimator for data-parallelism, or tf.device for model-parallelism.
We are adding ease-of-use multi-gpu APIs to Texar, which would be ready soon.

We've added support for distributed model training with multiple GPUs. Please see the examples of distributed_gpu and bert.


docker: Add Nix Store volume support

When running inside NixOS or using Nix packaging we need to add the Nix
Store to the running container so it can load the needed binaries.
Signed-off-by: Otavio Salvador<EMAIL_ADDRESS>

The error on the pipeline does not seem related. Any idea what might have caused it?

The error on the pipeline does not seem related. Any idea what might have caused it?

This is indeed not related. See:
E: Unable to locate package linux-image-4.19.0-11-arm64:arm64
E: Couldn't find any package by glob 'linux-image-4.19.0-11-arm64'
E: Couldn't find any package by regex 'linux-image-4.19.0-11-arm64'

Any news on this?

What is the use case for this? It seems like this should already be possible using env.volumes.

It fails to run automatically; so we would need to pass it to every time we call cross when using NixOS.

It fails to run automatically

What exactly fails to run? I'm not really familiar with NixOS, can you explain in more detail which packages need to be available/mounted in the container and why?

I am unsure if you know what NixOS is. So please take a look at How Nix works document. It explains how the Nix store works and more.
In summary, without the patch, cross fails to run because it cannot find the binaries it runs inside the container. It took me a while to understand the root cause of the error but now it works just fine. This is the same patch I apply in the official Nix package.

It explains how the Nix store works and more.

Okay, I understand now what the Nix store does.

it cannot find the binaries it runs inside the container

I'm still not sure what specifically the problem is though. Let me know if the following assumption is correct.
Let's say we are talking about the cargo binary:

We mount the rustup toolchain into the container.
The toolchain's ./bin/cargo is not a binary but a symlink into the Nix store.
Calling cargo fails since the Nix store is not mounted.

Exactly, your understanding is correct. Doing the mount inside the container allows it to run successfully.

Just one more question. Does Nix use some kind of customized version of rustup or how does symlinking the toolchain bineries into Nix store happen? I'd assume rustup just installs a complete pre-built toolchain from a tarball. Or is this a custom toolchain installed by Nix and not rustup?

Yes, it does. See rustup.

bors r+

Should I do anything on my side on this one?

fix the fmt error and it should be good to go :)
https://github.com/cross-rs/cross/runs/5578287402?check_suite_focus=true#step:4:10
bors d+

bors r+

yeah, thought so. Should maybe fix that in upstream bors


Breadcrumbs open to the side with keyboard shortcut not working

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.74.0
OS Version:  Mac OS Monterey 12.6

Steps to Reproduce:

Focus breadcrumbs
Navigate using arrow keys
Do alt-enter (also tried cmd-enter) on a file on breadcrumb
Nothing happens

I am able to open a file on the side from the breadcrumbs when i do alt-click.
Was this functionality removed? I also don't see a command for doing this in the key bindings.

@andreskebe Do you mean the breadcrumbs themselves (1) or the breadcrumbs picker (2)?

@jrieken
I meant the breadcrumbs picker! Alt clicking opens on the side but it is impossible to open any of these files to the side using the keyboard.

Thanks for clarifying. Open from the picker to the side is bound to ctrl+enter (cmd+enter on mac). It will run this command: breadcrumbs.revealFocusedFromTreeAside. Can you check whats being invoked for you? Do F1 > Toggle Keyboard Shortcuts Troubleshooting and in the output view check what command is being executed?

I had a feeling this was the command and so i changed from cmd-enter to alt-enter.
Now the super weird thing is that i see this command being executed Invoking command breadcrumbs.revealFocusedFromTreeAside. but it doesn't actually execute.
I've attached a screenshot of the output. I did clear everything until i trigger the shortcut, do you want me to send you more the previous logs as well?

Oh, now I can reproduce but only when trying with a file, but when trying with a symbol

Thank you 🙏🏽


Update default react query stale time to 5 min

The docs say that the cache query keys should be invalidated within 5 min... but from my testing, it doesn't appear that we are caching queries at all.
Which is a shame, since @fwextensions took the time to write this:
https://github.com/waterthetrees/wtt_front/blob/6bca4d3951e2e6fb7d2a8944f8399a52c40c4332/client/src/pages/Map/Map.js#L177-L185
Before:

After:

Only diff is the disappearance of the State field:

That's probably something missing from the geojsons for some reason

Please make sure caching changes works on any buttons, inputs, etc that has mutations in the Tree and AddTree components. https://react-query-v3.tanstack.com/guides/query-invalidation On doing tree maintenance

Make sure the data changes and is persistent when clicking another tree and then the same tree again
Make sure the data changes when copying to new page
When changing tree health, make sure dot changes color on the map.
When adding a tree, make sure it shows on the map and is persistent on refresh

Test on desktop and mobile.

1 and 3 tested successfully.
Need clarification on what 2 means.
For 4) I run into

but I also run into that for dev.waterthetrees.com as well

Hmm, looks like I'm losing data when I click on a tree... I wonder if it's connected to https://github.com/waterthetrees/wtt_front/blob/6bca4d3951e2e6fb7d2a8944f8399a52c40c4332/client/src/pages/Map/Map.js#L177-L185...

LGTM Nice catch! Is this ready to go?

Nope, not ready. Idk how to mark the PR as needs work, but I need to follow up on a couple things

@zoobot do you have any insight to why trees that have been edited are stripped of lots of their properties in the vector data?
Edited tree feature properties (upon map click):

Unedited tree feature properties (upon map click):

This will probably block this PR from being merged since activating react-query caching causes a regression.
Due to https://github.com/waterthetrees/wtt_front/blob/6bca4d3951e2e6fb7d2a8944f8399a52c40c4332/client/src/pages/Map/Map.js#L177-L185
From what I understand, old way was:
User clicks on tree feature on map => Vector data is loaded into cache (not sure why - to be available immediately in the tree details container?) => Tree query returns a response which then overwrites the data since the data is considered immediately stale (so tree query is source of truth)
New way:
User clicks on tree feature on map => Vector data is loaded into cache => Tree query never runs since cache is populated and value is not stale.
If it's not easy to keep tree vector data completely hydrated with our tree db data, then I'm not even sure if it makes to have this intermediary cache load step. Alternatively, I'm not sure if Tom's work would keep these vector data in sync with the db.

Yeah, if the intention is to just combine vector data and treedb treedata then we shouldn't be doing this cache update logic in
 if (!queryClient.getQueryState(queryKeys)) {
   queryClient.setQueryData(queryKeys, properties);
 }

currentTreeDb should map to only treedb data. This cache update logic attempts to combine vector data into currentTreeDb, which will produce unintentional results imo.
I don't think we need to have a custom hook either. Couldn't we just calculate a const combinedTreeData = currentTreeDb || currentTreeDataVector and pass that value into child components instead of doing anything fancy? Seems unnecessary to have this additional currentTreeData state that triggers an extra rerender.
I think changing this logic belongs in this PR so that it unblocks increasing react-query's stale time, so I'll include that small refactor in this PR

@zoobot This PR is ready for review. Not many changes needed actually (thankfully)

@zoobot oh yeah, my bad. I should have specified that this is meant to be merged into main after the other PR since it depends on it. (sorry, I'm used to hitting the merge button myself) The PR was into the other branch since this PR is focused on only 2 commits out of the 14 or so altogether.
Should be easy to fix - I'll create another PR to replace this one with the cherry-picked commits

sounds good, we should also revert the merge into useEffect branch as I merged it in there thinking it was pointed to main. cherrypicking doesn't remove the commit. Let me know if you want me to do anything.

Yup, yup. Already done :)


RFC: Do we want to use asserts or not

Now that #301 has been merged I want to raise the question should we start using asserts in the test binaries or not?
 However github.com/stretchr/testify/assert is widely used. The practice of using asserts in Go is left to be decided on a case by case basis.
ATM we've got something like this:
       _, c, err := rpmMetadata.FetchMetadata([]rpmmd.RepoConfig{repoCfg}, "platform:f31")
        if err != nil {
                t.Fatalf("Failed to fetch checksum: %v", err)
        }
        if c["repo"] == "" {
                t.Errorf("The checksum is empty")
        }

which can also be written like this:
       _, c, err := rpmMetadata.FetchMetadata([]rpmmd.RepoConfig{repoCfg}, "platform:f31")

       assert.Nil(t, err, "Failed to fetch checksum")
       assert.NotEqual(t, "", c["repo"], "The checksum is empty")

Benefits of using asserts:

shorter syntax and a bit more readable IMO
virtually every other language/test framework uses them so that makes the test suite more familiar to people experienced in other technologies (incl. myself)
Feels like a good practice anyway

Downsides:

will require patching the existing test suite, e.g. time but this is something QE can work on (will also give us more hands-on experience anyway)
other folks may not like this style of writing

I can't think of very strong pros or cons for either approach but the current one just doesn't feel right to me. My overall argument is that the asserts module is out there and widely used so why aren't we making use of it ?

A priori, this seems reasonable to me, though I have no strong opinion in either direction. Let's hear if others have any objections.

I don't have a strong opinion either. The only downside I see is that because not everything can be expressed by a function from the assert package, we'll inevitable have two styles in some places.

shorter syntax and a bit more readable IMO

Yeah, it really does read much better. Are asserts equivalent to log.Error() or log.Fatal()?

other folks may not like this style of writing

Right, let's wait for some more opinions.

Initially I didn't see the point in switching. But then I looked at the docs:
https://godoc.org/github.com/stretchr/testify/assert
It has a pretty impressive array of assertions, including the ability to add formatted strings (I like printing %#v to see what actually was being compares) so I think this would be a good idea.

Are asserts equivalent to log.Error() or log.Fatal()?

@larskarlitski from what I can see from the code asserts will return t.Fail or bool and the logging part is handled by the testing package. So assert == t.Fail I think.

@larskarlitski from what I can see from the code asserts will return t.Fail or bool and the logging part is handled by the testing package. So assert == t.Fail I think.

Sounds reasonable. Thanks!

@ondrejbudai, @msehnout what is your opinion on $subject

Firstly, I would like to stress the require package.

assert == t.Fail
require == t.FailNow

I think I'm slightly for the switch, imho it makes the tests easier to read.

Sounds like we all agree that this makes sense (@msehnout is out today). Let's move things over when it makes sense, note that lots of the tests are actively being worked on now, so let's coordinate a bit before refactoring to avoid too many merge conflicts.

Ok, I didn't understand what @ondrejbudai meant in his comment. But I do now.
If you want to fail immediately, not doing any more of the asserts use the require package instead of the assert package: https://pkg.go.dev/github.com/stretchr/testify/require?tab=doc
Otherwise you'll end up with panic's and nil pointer memory problems like I did :)

@ondrejbudai, @msehnout what is your opinion on $subject

I think it sounds reasonable to me. If we could have a document describing how to use it in the tests so that we don't end up mixing the old and the new approach, that would be great.

We agreed on using asserts, all the integration tests were migrated. #322 added docs how to use them. Is it ok to close this issue @atodorov ?

Yes, we can close this.
It would be nice if we used the issue number in commit log messages so GitHub can handle this for us.