ARK server not starting

Raised this a week ago on the steam forum but no help :-( Hoping I have better luck on here.
Port forwarding all looks 100%. I believe my debian config has been tweaked appropriately, regarding file limits, etc.
But no luck. Running through debug it seems to hang at a Steam API. I know this theoretically might not be an issue but no amount of waiting seems to have the server up and netstat doesn't have anything running on a port in the 777X range.
Info as outlined in the sticky thread...
Game server: arkserver
Distro: Debian

Linux cerebro 3.16.0-4-amd64 #1 SMP Debian 3.16.7-ckt20-1+deb8u2 (2016-01-02) x86_64 GNU/Linux

Logs:
./arkserver debug	http://pastebin.com/FtfXbDte
./arkserver details	http://pastebin.com/EPc12971
ark-server-console.log	http://pastebin.com/uV4bGjWm
ark-server-script.log	http://pastebin.com/9AwYy554
Starting the server, waiting 10 seconds and running "netstat -nap | grep Shoot" only gives ...

udp      768      0 <IP_ADDRESS>:27015           <IP_ADDRESS>:*                           2078/ShooterGameSer

And ./serverfiles/ShooterGame/Saved/Logs/ShooterGame.log is zero length at this point :-(
Any assistance would be greatly appreciated!

Hi,
There is no console output for Ark, and the steam API happens in any steam game, that's a non-issue, so look elsewhere.
Netstat should look like this :
arkserver@game:~$ netstat -nap | grep Shoot
tcp        0      0 <IP_ADDRESS>:27625           <IP_ADDRESS>:*               LISTEN      538/ShooterGameServ
udp        0      0 <IP_ADDRESS>:7778            <IP_ADDRESS>:*                           538/ShooterGameServ
udp        0      0 <IP_ADDRESS>:27015           <IP_ADDRESS>:*

I don't see a lot in my logs
cat ShooterGame-backup-2016.06.04-13.35.08.log
[2016.06.03-09.01.32:697][  0]Log file open, 06/03/16 11:01:32
[2016.06.03-09.01.32:697][  0]Primal Game Data Took 3.62 seconds
[2016.06.03-09.03.25:497][  0]Commandline:  TheIsland?listen
[2016.06.03-09.03.25:497][  0]Full Startup: 117.19 seconds (BP compile: 0.00 seconds)
[2016.06.03-09.03.25:497][  0]Number of cores 4

What's the output of your iptables ?
iptables -L

http://steamcommunity.com/groups/linuxgsm/discussions/0/364042703857868173/
Go back to it, i'll try to help there, along with Cedar.
Github is used for development, and this is obviously a user/distro/firewall/whatever configuraiton issue.


Bank value filter

Description:
Adds quantity, price and stackprice filters to +bank.
These can also be used in conjunction with greater than/less than operators, i.e. +b --quantity=<5
Changes:

Made parseBank async in order to retrieve item prices

Add KlasaClient as arg to parseBank

Code for parsing and evaluating equality operations

Parses k/m/b

Unit test parsing and evaluation, as well as quantity filters

Other checks:

[x] I have tested all my changes thoroughly.

Note:
This likely isn't done, just wanted to raise it so I wouldn't forget and get feedback whilst I work on other stuff.

This needs to be updated to mahoji.


Develop Backend driver for simulator

Let's develop backend driver for simulator.

We have to discuss what the bakcend driver exactly is. What is output of the backend driver? What is the simulator?

What is the plan for this issue?

related issue: https://github.com/Samsung/ONE-vscode/issues/456

@yunjayh I'm wondering that this task is a task of value-test? or execution?

Honestly, I confused this issue with # 458 (EXECUTE npu model on simulator)...

@yunjayh and @struss

[ ] schedule this task until 4/29 with the smaller separated tasks

Tasks as I think

Introduce backend driver like onecc backend-driver in ONE officially
Before introducing it officially, let's use exec API(landed soon in the future) directly -> https://github.com/Samsung/ONE-vscode/issues/454
After introducing it officially, let's use it

Related issue: https://github.com/Samsung/ONE-vscode/issues/312

@struss and @yunjayh can we close this?

I'll make another issue that track rest of https://github.com/Samsung/ONE-vscode/issues/456#issuecomment-1102168801. Then, this issue could be closed. :D Thank you!


Extra call to list_available_lycos wastes seconds of startup time

While debugging the web UI looking for possible performance improvements on startup, I noticed there's an extra call made to list_available_lycos() at the very end of lycoris.py that seems to serve no purpose.
https://github.com/KohakuBlueleaf/a1111-sd-webui-lycoris/blob/1f3e452c314e7b1dd903f723d3c552702519f4a3/lycoris.py#L789
This function is utilized in the UI module for listing and refreshing items, which should probably be the only place it exists.
https://github.com/KohakuBlueleaf/a1111-sd-webui-lycoris/blob/1f3e452c314e7b1dd903f723d3c552702519f4a3/ui_extra_networks_lyco.py#L12-L16
Removing that call in lycoris.py shaved off a full 2 seconds of startup time for me. Opening the extra networks UI lists all LyCORIS models just fine. Is there any reason this is included or can it be safely removed?

@catboxanon I haven't trace the src code of sd-webui for the ui-related part
and this "list available lycos" are copied from original built-in lora extension
(Which also has a list_available_loras)
And, if you don't call this function before running generation, lyco extension will think there is no lycos.
So if you want to remove thie function call, you will need to open the extra network tab before you start generate image after start up the sd-webui.
So basically this is necessary.

this function is to let the extension know there are what models
So if you don't call this function, the extension will say "there is no lyco models"
This function is not only for UI


ALTER TABLE ... RENAME COLUMN syntax incorrect for dm-migrations

The default syntax used to rename a column is not compatible with MySQL.  From the docs it should instead be
ALTER TABLE tbl_name CHANGE [COLUMN] old_col_name new_col_name column_definition

Created by Phil - 2008-12-01 06:38:33 UTC
Original Lighthouse ticket: http://datamapper.lighthouseapp.com/projects/20609/tickets/680

Kick! Any news on this?

@stephankaag I added it to 1.3.0 milestone

I might attempt to shoehorn in this fix in the mean time. We shouldn't leave this broken while waiting for DM 2.0.

I was going to work on writing tests for @rindek's patches, but it appear that master cannot even pass mysql specs.
747 examples, 158 failures, 4 pending

I created the datamapper user and datamapper_default_tests / datamapper_alternate_tests databases.

I ended up improving on @rindek's rename_column_type_statement for mysql (wasn't including DEFAULT or NOT NULL modifiers) and even added a basic spec for it:
https://github.com/postmodern/dm-migrations/commits/mysql_rename_column
Can we finally merge this?

@dkubb maybe I'm forgetting something, or more likely it's an environment specific issue. Half of the auto-migration specs consistently fail on my Fedora 16 workstation running mysql 5.5.19. However, I was able to manually run the migration specs for this particular bug.

@postmodern thanks for this. I was going to write those tests but I totally forgot about it :(

@rindek dm-migrations is kind of a wasteland, none of the existing specs caught this obvious syntax error. I also noticed that the MySQL specific change_column method wasn't speced. The sooner we can fix this, the better. :)


Update notification or auto updater

Summary of the new feature/enhancement
It would be nice if power tools have an option to check for updates frequently and show an update notification if there is an newer version which can be a tooltip over the tray icon, a push notification to the action bar or a flashing tray icon.
Similar to #202 a way to auto update would be even better but unlike #202 it hasn't to be windows store. It could grab the latest version from github instead and install it if the user wants this.

Hi @Symbai
we didn't have time to add the auto-update or the update notification in this first release but it's definitely something we want to add.

IIRC .msix can autoupdate from arbitrary URL, doesn't have to be tied to store.

xref #696

I'm going to close this in favor of #696.  While #303 existed first, #696 has dev implementation details we're discussing there.  I didn't realize this item was created when i filed that.


Material 3 Redesign

Here is a PR for the redesign I proposed in #85. I have personally been using this version for over a month now and haven't encountered any issues.
If you have any questions or comments, feel free to contact me.

Thanks for the PR. I'll have some time to look at this in two weeks.

I merged it. I'll test it over the next few days and then make a release.
I removed the "Note taking made easy" slogan, I hope you don't mind! Doesn't go well with the cynical "Another notes app" name.

No, I don't mind. I only added that slogan for aesthetic reasons. The header within the navigation drawer looked a bit off without it. That might just be personal preference though. "Note-taking made easy" was simply something I came up with to fill that space.

Are you able to run instrumentation tests? I can't since the Artic Fox update, and I still can't find what broke. They run just fine with adb, it's clearly a problem with Android Studio.

Are you able to run instrumentation tests? I can't since the Artic Fox update, and I still can't find what broke. They run just fine with adb, it's clearly a problem with Android Studio.

I'll have to check later, but if I recall correctly, they didn't work for me either.

There are some other things which aren't working very well on older API versions. I'll have some time this evening / tomorrow morning to fix a few of those.
I mainly noticed the following:

The lightStatusBarColor style attribute is only available from API 23 onwards. As a result, the status bar contains white text on a white background on older API versions, as can be seen in your screenshot.
Similarly, the lightNavigationBarColor attribute is only available from API 27 onwards. This results in the same problem as mentioned before.

I'll create a PR with version-specific themes to address this, unless you have already been working on this.

You can make a PR.

There was a small issue with timezones, it seems MaterialDatePicker only operates on UTC timezone. I'm guessing you didn't catch the bug because you're in a UTC+X:00 timezone, but for me changing the reminder start date always resulted in the day before being selected (UTC-X:00 timezone). I fixed it.
Do you think we need to ask the user for the POST_NOTIFICATIONS permission on Android 13? Notifications didn't work out of the box on the emulator, I had to check something in the app's settings. See https://developer.android.com/develop/ui/views/notifications/notification-permission
Also there's a few things I'd like to point out for next time:

Make sure to test the app on configuration changes and process death. You have probably heard of it, a configuration change happens when you rotate the screen, change the locale, etc, and causes activities and fragments to be recreated using saved state. Process death is similar, but happens when your phone runs out of memory or on a reboot.
One problem here was with the date & time picker dialogs. The reason I used fragments with callback interfaces previously was so that upon recreation, no reference is kept the old destroyed parent fragment (ReminderFragment) by the children (date, time picker fragments), thus avoiding memory leaks and allowing the child fragment to keep its callback relation with the parent.
As it was implemented, upon a configuration change, the listener set on the date and time picker dialogs is lost, preventing a correct callback. In certain circumstances this can also lead to a memory leak but it seems that listeners on cleared on recreation. A memory leak would occur if the listener is kept, referencing the old destroyed fragment and preventing it from being garbage-collected.
It's an aspect that is often left aside as evidenced by apps that crash when you rotate your screen or Google's unsuitable listener APIs. I just submitted a bug to the material library because their TimePicker doesn't handle process death correctly... I think it's important for stability because screen rotation for example can happen accidentally, your phone can run out of memory at any time if too many apps are open, or it's nice to be able to continue what you were doing when your phone runs out of battery and you need to reboot, etc.

Look out for memory leaks. I think I fixed a few in the previous commits. LeakCanary should report one from time to time. Sometimes there's false positives (e.g. from Venom if you use it to trigger process death) but most of the time it's a real leak.

If you can use the project code style and reformat/optimize imports on commits it'd appreciated, I'll save me the reformatting commits. You should be able to set it as a project specific setting and not global. There's also detekt for reporting style issues but I haven't used it in a long time so don't bother.

Yes, I am indeed in a UTC+X:00 timezone and didn't notice this issue.
I do think we need to manually request the POST_NOTIFICATIONS permission, since the app now targets Android 13. During development I kept the target SDK set to 32, so this wasn't an issue, because the permission dialog would automatically be shown on the first start of the app.
It would probably be best to request the permission when the user tries to add a reminder to a note. Reminders are the only notifications sent by the app, right?
Thanks for the feedback. My previous experience in Android development is limited, so this is still a bit of a learning process. As for your particular comments:

Configuration changes were something I didn't really have in mind while testing, but it makes total sense that they can cause problems. I'll make sure to keep an eye on that.
Will do :) I caught the most obvious ones during development, but shrugged the occasional remaining LeakCanary notifications off as false positives. I'm guessing these resulted from the memory leaks you fixed. I'll try to make sure that doesn't happen in the future.
Yes, I'm going to enable the code style.

It would probably be best to request the permission when the user tries to add a reminder to a note. Reminders are the only notifications sent by the app, right?

Yes, that's how I started to implement it.

I added 2 strings related to permissions, you can translate them on Crowdin.


Create config class for having a single declaration of each services

Create a config class for having a single declaration of each UUID

formatUUID is missing with this refactoring ;(

Modification done ;)

I'm not a huge fan of having a separate Object for the config to be honest.
What are the use cases for reusing this config in a "Get Started" app? The goal here is to quickly get you to connect to and interact with your BLE device.

The generator can be use for me for something else than a "Get Started App" so when we want to go further in the developpement of our application  or if we change the UUID, we just do the change in one place.

As discussed offline, closing this for now.


How to change the default output directory of predicted lane lines for culane data?

The default directory name is "./output". I try to change the code in "./utils/datasets/culane.py"  (in line 32) but not working.

The default directory name is "./output". I try to change the code in "./utils/datasets/culane.py"  (in line 32) but not working.

Maybe you need to search the entire directory for ./output. Since a lot places depends on that cache dir. In the future we probably will make that easy to set.

OK, thanks very much. I have solved this problem.


fix pledge modal button by attaching event handlers to document

I rule and have completed some work on Case Manager that's ready for review!
This pull request makes the following changes:

make sure the pledge modal next button works every time, without needing to reload

It relates to the following issue #s:

Fixes #597

Confirm that this does what it says on the label and that @mchelen does in fact rule! Merging.


Links to the Excel files not present

Links to Excel files should be provided as described, but non present in the article.

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

ID: 0ab9d2cf-5ff8-59c9-34fd-014f0dfcef6a
Version Independent ID: 3ad7b162-7fa2-ef08-8c1d-5343b1bebf4e
Content: Team Data Science Process project planning - Azure Architecture Center
Content Source: docs/data-science-process/team-data-science-process-project-templates.md
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @marktab
Microsoft Alias: tdsp

@hanakon
Thanks for your feedback! We will investigate and update as appropriate.

@hanakon Thanks for reporting -- that mention is a legacy offering, no one is maintaining a template today

Address in this PR  https://github.com/MicrosoftDocs/architecture-center-pr/pull/9687

@MughundhanRaveendran-MSFT  closing this request

#please-close


Implement delete beta group

Closes #98
📝 Summary of Changes
Changes proposed in this pull request:

Adds DeleteBetaGroupCommand and DeleteBetaGroupOperation
Adds GetBetaGroupOperation to reuse for delete and modify service functions
Moves get beta group tests from modify to shared get beta group operation

🧐🗒 Reviewer Notes
💁 Example
Usage: asc testflight betagroup delete [--api-issuer <uuid>] [--api-key-id <keyid>] [--csv] [--json] [--table] [--yaml] <app-bundle-id> <beta-group-name>
🔨 How To Test

Use create to create a beta group
e.g. asc testflight betagroup create com.example.app myGroup
Use list to verify
e.g. asc testflight betagroup list --filter-bundle-ids com.example.app
Use delete to delete the group
e.g. asc testflight betagroup delete com.example.app myGroup
Use list to verify again
e.g. asc testflight betagroup list --filter-bundle-ids com.example.app

@DechengMa PR has been updated


The environment variables are inaccessible in the iOS example

Running the example in the repo runs as expected on android. However running ENVFILE=.env.production react-native run-ios launches the example app, with a config object with no set properties. The image below shows what I see in iOS

Also I do not see a scheme for 'example (Production)' as suggested by the example readme. Everything works as expected on android.
System Info
OS: OSX High Sierra
React-Native: 0.50.4
Node: 7.3.0
XCode: Version 9.2

after much trouble shooting, the issue seems to be related with the latest release of react-native-config. I ended up down grading to 0.8.1 as suggested in https://github.com/luggit/react-native-config/issues/129.

My be fixed in IOS but still there on Android I am getting the config as undefined :(

So tried working on it for a little while longer and I later realize I had spaces in properties. Android accepts it, but IOS simply return undefined for the properties.
So if you have
API=www.yahoo.com/api               ✓ this is good
  API=www.yahoo.com/api ✖this is bad
Also just to make sure, did you implement this part of the documentation
`project.ext.envConfigFiles = [
debug: ".env.development",
release: ".env.production",
anothercustombuild: ".env",
]
apply from: project(':react-native-config').projectDir.getPath() + "/dotenv.gradle"`

https://medium.com/the-react-native-log/how-to-rename-a-react-native-app-dafd92161c35   This worked for me in both IOS and ANDROID
Hope this helps

I'm facing the exact same issue. Followed all the steps to a tee. Everything works in android, but in iOS, I get an empty Config. I tried basically all the fixes suggested here

react-native: 0.54.2
react-native-config: 0.11.5
react-native-cli: 2.0.1

@DeX3 Currently react-native link react-native-config doesn't work correctly with RN 0.50+
You can either downgrade to a previous react-native e.g. version 0.49.3 and then do the react-native link, then upgrade back to the current version. Or try to just add it manually by add the library and linking it into the build phase:

@Sam-Hoult I downgraded to 0.49 and then linked before returning to 0.51. the method and header files are being generated as expected but xcode is still having trouble finding GeneratedDotEnv.m when building ReactNativeConfig.m, presumably because it's not actually looking in SYMROOT. any ideas?

@alxyuu Yea, the docs also give a wrong location for that. here's mine using ${CONFIGURATION_BUILD_DIR}/../GeneratedInfoPlistDotEnv.h as the Info.plist preprocessor path

@Sam-Hoult I'm referring to GeneratedDotEnv.m when building ReactNativeConfig.m, not the header for preprocessing info.plist.

Thanks @Sam-Hoult.
My case is - two targets, first one was ok and running, while the second one didn't. Turns out the linker didn't add the framework.

This tutorial helped me: https://medium.com/differential/managing-configuration-in-react-native-cd2dfb5e6f7b.

I was able to get it working on iOS, When I write constants like this SERVER_URL = http://example.com it's seen on android but not on iOS but after I closed the gap and make it look like this SERVER_URL=http://example.com iOS started seeing the constants too. Now both platforms are cool.

I am not able to run this thing in IOS. I have done all the settings, still no success. My configuration files are located in config folder. I have added a pre action in the build of the scheme and that is
echo "config/.env.prod" > /tmp/envfile.
Please help

facing the same issue here seems like IOS is not loading the constants from .env files. I created a custom scheme for DEV environment.  When I check cat /tmp/envfile I am seeing the string .env.dev . Is this expected I was thinking I will see the contents? anybody faced/solved this ?

Same issue to @pradhul .
On Android all works fine.
But on iOS my env file seems to be unread. At this time I tries only the "play" button on Xcode.
Any ideas?
Regards

Faced the same issue to @pradhul .
I removed ios/build folder and it worked.


PER encoder does not return buffer size when called with null write_out()

Expected behavior
Similar to DER encoder, I'd expect the call to
asn_enc_rval_t ret = uper_encode(&asn_DEF_Test, test_pdu, 0, 0);

provide me with the exact size of the buffer required to UPER-encode the given PDU, which I would later convert to bytes like size_t bytes_needed = (ret.encoded + 7)/8;
Actually happens
asn_enc_rval_t ret = uper_encode(&asn_DEF_Test, test_pdu, 0, 0);

causes SEGV (application crashes with segmentation fault, trying to encode octet string).
If PER encoder is not supposed to behave like DER encoder in this aspect - what's the correct way of determining the size of the buffer to hold the UPER-encoded PDU?
I'm aware of uper_encode_to_new_buffer(), and it works fine. But it is documented that this function uses "complete" encoding, and therefore may take more space than necessary.
What's the correct way of using uper_encode_to_buffer()? How do I know how big a buffer to pre-allocate?
And as we're at it, asn1c-usage.pdf mentions write_stream() callback, but never actually shows how to use/write it. What's the example? What are the parameters (const void *buffer, size_t size, app_key)? Just the number of bytes in the given buffer that represent the encoded value?

What write_stream is can be found  in the manual under the name write_out.
This callback receives as input the pointer to an element (const void *buffer), the size of that element (size_t size) and some context (void *app_key).
/* Dump the buffer out to the specified FILE */
static int write_out(const void *buffer, size_t size, void *key) {
  FILE *fp = (FILE *)key;
  return (fwrite(buffer, 1, size, fp) == size) ? 0 : -1;
}

In this example, we can see that the user is using der_encode() which accepts a FILE * as the last parameter, which later is passed to write_out() as the context.

What write_stream is can be found in the manual under the name write_out.

Thank you! After you showed the example (and the correct name), I was able to locate it under "A 'Rectangle' Encoder" in "Step-by-step examples" section.

This callback receives as input the pointer to an element

A pointer to the encoded element (seems obvious, but I want to be "explicitly" sure :). Correct?

@vlm as you appear to be back, could you tell if you're planning to add a function/capability to determine the required size of the buffer to allocate for a PER-encoded PDU?
An encoder would really love to know how big a buffer to allocate for a given PDU, and you already provide that capability at least for DER.
@velichkov might you know the answer here?

The buffer size can be estimated by invoking per_encode without a buffer. It'll return size.
Same as with DER.
The reason is that in the dynamic world you pretty much have to serialize the data (even into /dev/null) to figure out the resulting size.

The buffer size can be estimated by invoking per_encode without a buffer. It'll return size.

Could you please be a bit more specific? Should I be able to call
 asn_enc_rval_t er9 = uper_encode(&asn_DEF_Test2b, der_dec, NULL, NULL);
  if (er9.encoded < 1) {
     fprintf(stderr,
 	    "Failed to determine the buffer size for UPER-encoded Test2b_t PDU... (rc=%ld)\n",
 	    er9.encoded);
   } else {
	   printf("UPER-encoded Test2b_t PDU requires buffer of %ld bits\n", er9.encoded);
   }

Because I'm getting back er9.encoded == -1. (And before it used to just crash with SEGV...)
Could you please show a complete function call?

Maybe this output would be more informative:
. . . . .
Successfully BER-decoded Test2b_t PDU (47 bytes)
Validated constraints on BER-decoded Test2b_t PDU
Visual check:
<Test2b>
    <count>40</count>
    <data>
        C3 40 CB 01 95 F1 CE B7 75 6B 43 59 BC FE 51 3F
        98 B1 E3 05 BC B2 22 FE 9D 5A A7 AD 64 1B C2 08
        3B E9 ED 03 06 F8 0E B7
    </data>
</Test2b>

Encoding Test2b as SEQUENCE (UPER) (constr_SEQUENCE.c:1579)
ext_after = -1, ec = 2, eb = -1 (constr_SEQUENCE.c:1629)
About to encode INTEGER (constr_SEQUENCE.c:1640)
Encoding Test2b->count (constr_SEQUENCE.c:1662)
Encoding NativeInteger INTEGER 40 (UPER) (NativeInteger.c:334)
Value 40 (28/1) lb 40 ub 20000 fix (INTEGER.c:876)
Encoding integer 40 (0) with range 15 bits (INTEGER.c:892)
[PER put 15 bits 0 to 0x7fff5fbfec98+0 bits] (per_support.c:426)
[PER out 15 0/0 (t=0,o=15) 27&0=0] (per_support.c:467)
[PER out 0/0 => 00 buf+0] (per_support.c:498)
Freeing INTEGER as a primitive type (asn_codecs_prim.c:125)
About to encode OCTET STRING (constr_SEQUENCE.c:1640)
Encoding Test2b->data (constr_SEQUENCE.c:1662)
Encoding OCTET STRING into 40 units of 8 bits (40..20000, effective 15) (OCTET_STRING.c:1753)
Encoding 40 bytes (0), length in 15 bits (OCTET_STRING.c:1784)
[PER put 15 bits 0 to 0x7fff5fbfec98+15 bits] (per_support.c:426)
[PER out 15 0/0 (t=7,o=22) 0&fe=0] (per_support.c:467)
[PER out 0/0 => 00 buf+1] (per_support.c:498)
Squeezing 40 characters into (0..0):8 (1 bpc) (OCTET_STRING.c:1258)
[PER put 24 bits c340cb to 0x7fff5fbfec99+22 bits] (per_support.c:426)
[PER out 24 12796107/c340cb (t=6,o=30) 0&fc=0] (per_support.c:467)
[PER out 51184428/30d032c => 03 buf+3] (per_support.c:498)
[PER put 24 bits 195f1 to 0x7fff5fbfec9b+30 bits] (per_support.c:426)
[PER out 24 103921/195f1 (t=6,o=30) 2c&fc=2c] (per_support.c:467)
[PER out 415684/657c4 => 2c buf+6] (per_support.c:498)
[PER put 24 bits ceb775 to 0x7fff5fbfec9e+30 bits] (per_support.c:426)
[PER out 24 13547381/ceb775 (t=6,o=30) c4&fc=c4] (per_support.c:467)
[PER out 54189524/33addd4 => c7 buf+9] (per_support.c:498)
[PER put 24 bits 6b4359 to 0x7fff5fbfeca1+30 bits] (per_support.c:426)
[PER out 24 7029593/6b4359 (t=6,o=30) d4&fc=d4] (per_support.c:467)
[PER out 28118372/1ad0d64 => d5 buf+12] (per_support.c:498)
[PER put 24 bits bcfe51 to 0x7fff5fbfeca4+30 bits] (per_support.c:426)
[PER out 24 12385873/bcfe51 (t=6,o=30) 64&fc=64] (per_support.c:467)
[PER out 49543492/2f3f944 => 66 buf+15] (per_support.c:498)
[PER put 24 bits 3f98b1 to 0x7fff5fbfeca7+30 bits] (per_support.c:426)
[PER out 24 4167857/3f98b1 (t=6,o=30) 44&fc=44] (per_support.c:467)
[PER out 16671428/fe62c4 => 44 buf+18] (per_support.c:498)
[PER put 24 bits e305bc to 0x7fff5fbfecaa+30 bits] (per_support.c:426)
[PER out 24 14878140/e305bc (t=6,o=30) c4&fc=c4] (per_support.c:467)
[PER out 59512560/38c16f0 => c7 buf+21] (per_support.c:498)
[PER put 24 bits b222fe to 0x7fff5fbfecad+30 bits] (per_support.c:426)
[PER out 24 11674366/b222fe (t=6,o=30) f0&fc=f0] (per_support.c:467)
[PER out 46697464/2c88bf8 => f2 buf+24] (per_support.c:498)
[PER put 24 bits 9d5aa7 to 0x7fff5fbfecb0+30 bits] (per_support.c:426)
[PER out 24 10312359/9d5aa7 (t=6,o=30) f8&fc=f8] (per_support.c:467)
[PER out 41249436/2756a9c => fa buf+27] (per_support.c:498)
[PER put 24 bits ad641b to 0x7fff5fbfecb3+30 bits] (per_support.c:426)
[PER output 30 complete + 0] (per_support.c:443)
Process 15034 stopped
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
    frame #0: 0x0000000000000000
error: memory read failed for 0x0
(lldb) list
(lldb) bt
* thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x0000000000000000
    frame #1: 0x0000000100019a61 m1b`per_put_few_bits(po=0x00007fff5fbfec80, bits=<unavailable>, obits=<unavailable>) at per_support.c:444 [opt]
    frame #2: 0x000000010001a08f m1b`per_put_many_bits(po=0x00007fff5fbfec80, src="\b;??\x06??, nbits=80) at per_support.c:572 [opt]
    frame #3: 0x00000001000115e4 m1b`OCTET_STRING_encode_uper(td=<unavailable>, constraints=<unavailable>, sptr=<unavailable>, po=0x00007fff5fbfec80) at OCTET_STRING.c:1789 [opt]
    frame #4: 0x000000010000d25f m1b`SEQUENCE_encode_uper(td=<unavailable>, constraints=<unavailable>, sptr=0x0000000100102cd0, po=<unavailable>) at constr_SEQUENCE.c:1663 [opt]
    frame #5: 0x000000010001ac21 m1b`uper_encode_internal(td=0x0000000100020630, constraints=<unavailable>, sptr=0x0000000100102cd0, cb=<unavailable>, app_key=<unavailable>) at per_encoder.c:201 [opt]
    frame #6: 0x000000010001ab8c m1b`uper_encode(td=<unavailable>, sptr=<unavailable>, cb=<unavailable>, app_key=<unavailable>) at per_encoder.c:11 [opt]
    frame #7: 0x000000010001c54b m1b`try_string + 1435
    frame #8: 0x000000010001cd20 m1b`main + 240
    frame #9: 0x00007fff93c90235 libdyld.dylib`start + 1
    frame #10: 0x00007fff93c90235 libdyld.dylib`start + 1
(lldb)

@vlm...?

Solved by @velichkov in my fork


Treat single pages and posts as page collections with one item

On second thought, this is pretty much how already how it is done given the abstraction employed (SQLite database and no objects). The first page in a collection is privileged (it determines the page variables at the document level) but it is by design. Article-level variables do not conflict.


New version: PSDMatrices v0.3.0

Registering package: PSDMatrices
Repository: https://github.com/nathanaelbosch/PSDMatrices.jl
Created by: @nathanaelbosch
Version: v0.3.0
Commit: 179d6b639c10f795b720cee161b8a0c93aecef22
Reviewed by: @nathanaelbosch
Reference: https://github.com/nathanaelbosch/PSDMatrices.jl/commit/179d6b639c10f795b720cee161b8a0c93aecef22#commitcomment-43883036

[noblock]
Fixed this issue by registering a 0.2.0 version:
https://github.com/JuliaRegistries/General/pull/24173


Update WordPress.gitignore

Hello all and thanks to create great file.
There is problem to find proper meaning of content for this file in this repo.
My propose need review as I have different, more direct approach with fix expectation.

Header. To know what rules are used, and be apply to project after using template. Errors can be removed, updates won't be broken and clearly show why I using that file.
Now: Any changes are confused.
After: Rules are in header

Introduce link to Github where template is created. Like a template, this can be modified so no big deal. Used as link where to watch. For this pull request nobody will update their gitignore file just because that link don't preexist from copy & paste.
Now: Manually fixing and no connection to place where file come from.
After: Can be used to update if some bugs will be introduce.

Core Files need to be ignored (clue from comment) but there was broken part:
Solution apply and work (thx. @stkrzysiak) : https://github.com/github/gitignore/pull/3093/files/1115dec039582df7242acd588811357f76d610ce#diff-f2cf96854b528e9828c4ca34a4917e54
Now: All root catalog was added ;/
After: Just some will be excluded

If core file need to be included, then we need fix other parts as and new pull request:

/wp-content/language
/wp-content/index.php (security bug when someone use apache listings)

Thanks @chougran
https://github.com/github/gitignore/pull/3125
This is most important as gitignore provide solution to sensitive credentials.
Now: wp-config leak
After: Configuration is safe :)

License
As they can be manually edited for adding more software (bundle of license) then is out of scope for ignoring it in this particular template. Rule is to add wordpress context stuff that are repeated every new repository is created. Maybe add clue that need to be handled manually? or just header that Wordpress use particular one.

It is tested on: WP 5.3.2
Used: Custom theme + custom plugin (they language are ignore but can be download from dashboard)
Comments appreciate as my first post here :)

License
As they can be manually edited for adding more software (bundle of license) then is out of scope for ignoring it in this particular template. Rule is to add wordpress context stuff that are repeated every new repository is created.  Maybe add clue that need to be handled manually? or just header that Wordpress use particular one.

It is tested on: WP 5.3.2
Used: Custom theme + custom plugin (they language are ignore but can be download from dashboard)

Closing this out in favour of #3295 which seems to most detailed discussion

Why part of solution was rejected when really it was accepted?
This PR's can be merged, some small fix needed to be done before.
Discussion of #3295 can be really long when my fix just are in place of some problem when I use that template.

I propose fix to pull request:

revert "catch-all" after community decide what can be done there and better, tested declarative solution will be
Remove logs, added after next pull request when there will be evidence that starter need that

@Triloworld I'm not opposed to interim fixes while #3295 goes on, it just wasn't clear to me whether this PR was active based on the discussion.

I propose fix to pull request:

revert "catch-all" after community decide what can be done there and better, tested declarative solution will be
Remove logs, added in other pull request when there will be evidence that starter template need that

Please open up PRs for these changes, as they feel like they are a step in the right direction.


feat: js object v1 run and settings redesign

Description
This PR implements the new JS Run functionality
Fixes #10387
Fixes #9790
Fixes #6539
Fixes #11826
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Manual (currently)
Checklist:

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

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feat/new-js-run-functionality

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
55.49 (-0.1)
36.65 (-0.13)
34.74 (-0.17)
55.8 (-0.1)

:green_circle:
app/client/src/actions/jsPaneActions.ts
65.63 (0.11)
0 (0)
0 (0)
50 (0)

:red_circle:
app/client/src/ce/constants/messages.ts
76.78 (-0.11)
100 (0)
31.07 (-0.31)
81.28 (-0.28)

:red_circle:
app/client/src/components/ads/Dropdown.tsx
84.23 (0)
62.31 (-0.45)
73.61 (0)
83.26 (0)

:red_circle:
app/client/src/components/ads/Radio.tsx
14.63 (0)
9.52 (-0.48)
0 (0)
15.38 (0)

:red_circle:
app/client/src/components/ads/Tabs.tsx
23.26 (-2.23)
11.36 (-5.31)
0 (0)
24.68 (-1.85)

:green_circle:
app/client/src/components/ads/index.ts
100 (0)
100 (0)
3.7 (3.7)
100 (0)

:green_circle:
app/client/src/components/editorComponents/EntityBottomTabs.tsx
31.25 (2.68)
0 (0)
0 (0)
33.33 (1.33)

:green_circle:
app/client/src/components/editorComponents/JSResponseView.tsx
50.62 (4.97)
42.11 (20.13)
6.67 (6.67)
53.95 (4.54)

:red_circle:
app/client/src/components/editorComponents/CodeEditor/index.tsx
77.4 (-1.11)
58.56 (-1.92)
65.22 (0.78)
79.01 (-1.12)

:red_circle:
app/client/src/pages/Editor/JSEditor/Form.tsx
31 (-19)
25 (-41.67)
0 (0)
34.44 (-17.56)

:sparkles: :new:
app/client/src/pages/Editor/JSEditor/JSFunctionRun.tsx
52.17
33.33
0
52.17

:red_circle:
app/client/src/pages/Editor/JSEditor/JSFunctionSettings.tsx
27.45 (-25.49)
40 (-60)
0 (0)
31.82 (-21.12)

:sparkles: :new:
app/client/src/pages/Editor/JSEditor/JSObjectHotKeys.tsx
84.62
100
33.33
81.82

:sparkles: :new:
app/client/src/pages/Editor/JSEditor/constants.ts
100
100
100
100

:green_circle:
app/client/src/pages/Editor/JSEditor/index.tsx
60 (3.33)
33.33 (0)
33.33 (0)
63.64 (2.1)

:sparkles: :new:
app/client/src/pages/Editor/JSEditor/utils.ts
33.87
0
0
26.92

:green_circle:
app/client/src/reducers/entityReducers/jsActionsReducer.tsx
9.91 (2.84)
3.45 (3.45)
1.96 (1.96)
10.28 (3.06)

:red_circle:
app/client/src/sagas/JSPaneSagas.ts
14.35 (-0.51)
0.88 (-0.23)
4.35 (-0.41)
16.75 (-0.8)

:red_circle:
app/client/src/selectors/entitiesSelector.ts
45.98 (-0.36)
6.94 (-1.11)
21.6 (-1.13)
41.38 (-0.82)

:sparkles: :new:
app/client/src/utils/hooks/useResizeObserver.tsx
36.36
0
0
40

[ ] Debugger does not have a scroll. User is required to expand the debugger to view the entire response
https://www.loom.com/share/0da8fede97cd443a9f7b8d61b477c9bc
[ ] Existing issue #9790 needs to be fixed with these changes

The order of elements on the editor page versus queries is very different, can we have a look at designs again please?
@parth-appsmith @ohansFavour
On editor:

On queries:

Elements on the editor and query page are in different orders. Can we bring them both to the same style? CC: @parth-appsmith @ohansFavour
On the editor:

On the the query page:

The order of elements on the editor page versus queries is very different, can we have a look at designs again, please?

on the API/Query pane, the run button is an individual action whereas in the JS editor we have a button group where first you select the function from the drop-down and then run. Both the actions need to be placed together to convery this connection.
cc: @ramsaptami

@ohansFavour hey, there are a few issues that we still need to fix.

The run icon in the gutter has to be appearing only when a user is inside the scope of the function or hovering over it. the run icon doesnt need to persistent for all functions.
When the response window is expanded ( even if slightly) the logical action on the expand/collapse CTA has to be  collapse it, where as currently the action is to expand it to the fixed height and then it changes to the collapse icon.
The dropdown to select the function get disabled incase when there is an error in the JSObject file. IMO even if one function has an error the user should be able run another function which has no errors. Not sure but is this possible?
When the user runs a function the response window has to expand automatically to show the result which it doesnt seem to do now.
Also the dropdown to select the function has to select the function that the user is working on, the dropdown current doesn't do that.

Also for the behaviour of the function selection dropdown and the gutter run icon please refer this notion doc
Attaching videos for reference.
https://www.loom.com/share/693c94bfbd3d457a87260411db5d6033
https://www.loom.com/share/e40c531c6141402eaef21281d3df95a4

/ok-to-test sha=60b944b

/ok-to-test sha=7acec81

DP to test: appsmith-54l4qc1rz-get-appsmith.vercel.app

/ok-to-test sha=54eadce

/ok-to-test sha=994541f

/ok-to-test sha=2e1816f

/ok-to-test sha=2e16c7b

/ok-to-test sha=9bc2b4e

/ok-to-test sha=def9933

/ok-to-test sha=73da65b

/ok-to-test sha=f97904f

/ok-to-test sha=605130d

/ok-to-test sha=ae61e91

/ok-to-test sha=be77cde

/ok-to-test sha=4128e74

/ok-to-test sha=134e698

[ ] Selected function name is glitchy https://www.loom.com/share/bd2f773d384b42528048e79472942bf3

[ ] Can't select function names from the dropdown
https://www.loom.com/share/cdba3e924bc14ae9a68efd8fc9f0d944

[ ] Functions unmarked from the async tag are also run on page load even after they are removed from the settings page list

/ok-to-test sha=893a29f

/ok-to-test sha=3bdef19

Tested feature on deploy preview and is ok to merge to release

/ok-to-test sha=d9ef8de


binPkg not set correctly in 0.9.18 - interpeter cannot find binary package any more

The following program sample.go triggers a panic: on the 0.9.18 tag but works  on 0.9.17 tag.
Probably has something todo with the new fix for package name extraction from import path https://github.com/traefik/yaegi/commit/d16bd4bcdb8685f7ce88a0ee2261f7d945a6cc84
I can see a difference in the contents of github.com/traefik/yaegi/interp.Exports.binPkg between versions.
It looks like the last element of the package name is cutoff, therefore wrongly imported
0.9.17
binPkg = {github.com/traefik/yaegi/interp.Exports}
	 0 =  ->
	 1 = github.com/mbict/bebe/dsl ->
	 2 = github.com/mbict/bebe/engine ->

0.9.18
binPkg = {github.com/traefik/yaegi/interp.Exports}
	 0 =  ->
	 1 = github.com/mbict/bebe ->

The example program
package main

import (
	"github.com/traefik/yaegi/interp"
	"reflect"
)

var Symbols1 = map[string]map[string]reflect.Value{}
var Symbols2 = map[string]map[string]reflect.Value{}

func init() {
	Symbols1["github.com/mbict/bebe/dsl"] = map[string]reflect.Value{}
	Symbols2["github.com/mbict/bebe/engine"] = map[string]reflect.Value{}
}

func main() {
	i := interp.New(interp.Options{})

	i.Use(Symbols1)
	i.Use(Symbols2)

	script := `
package main

import (
	"github.com/mbict/bebe/dsl"
)
`
	_, err := i.Eval(script)
	if err != nil {
		panic(err)
	}
}

Expected result: v0.9.17
$ go run ./sample.go
// output
Eval without error

Got: v0.9.18
$ go run ./sample.go
// output
 panic: 5:2: import "github.com/mbict/bebe/dsl" error: unable to find source related to: "github.com/mbict/bebe/dsl". Either the GOPATH environment variable, or the Interpreter.Options.GoPath needs to be set

I found the cause of this in the interp/interp.go file, i put some comments on the function https://github.com/traefik/yaegi/commit/d16bd4bcdb8685f7ce88a0ee2261f7d945a6cc84#r52229954
But ny simply reverting this function to the old situation would fix this. But maybe someone or the author of the changes @mvertes  could give some context on the changes in that patch. Because that is not really clear what the intend of the fix was before i would even undo this with a MR.

I have this problem, too.

TLDR: change the key from github.com/mbict/bebe/dsl to github.com/mbict/bebe/dsl/dsl
From version v0.9.18, the key format in binPkg has changed from import_path to import_path/package_name, where import_path, in your case, is github.com/mbict/bebe/dsl and package_name is dsl.
In most cases the package name (dsl) is equal to the last element of import_path, but it is not mandatory, and we have to cope with the cases of for example github.com/pelletier/go-toml where  package name is toml, or github.com/stripe/stripe-go/v72 where package name is stripe, etc.
You are affected by this change because you maintain manually your exported symbols (extract tool is updated). Sorry for the inconvenience.

Ahh thanks for the quick response. It is clear now and I do understand why.
Your proposed fix is indeed working. Maybe I should also start using the extract tool by now.
Thanks again, i'm closing this one


2004 compatibility

Did you  try on 2004 and does it have the intended results?
Is the store still available and installing apps? I do run for a few tasks on apps from the store like drawboard

@boistordu - I ran this last night after merging the latest changes from our awesome contributors, and it works perfectly. I did not have any issues.

@Sycnex great to hear.
so 20H2 should also work without a problem?

@ShlomiD83 - Yes. I tested it on my system and it worked without an issue.


remove types folder

While working in this repository, we only need the types folder for tests. Thus we don't need to track the changes. It will also be necessary when packaging, but it still does not need to be tracked :business_suit_levitating:

@uivlis could you please review and merge?

@obernardovieira I reviewed it now, but I'm not authorised to merge this.

@obernardovieira I reviewed it now, but I'm not authorised to merge this.

Sorry, now you are.


Application crashes because of exception 0x80050022

Project: speechandtts
Phone: Lumia 920
OS version: 10.0.12534.56
Exception 0x80050022 occurred on creating SpeechRecognizer.
Also there is no prompt to request permissions.

All the samples have been refreshed and updated to match the VS2015 RC announced at //build
Please update your VS and sync your repository to pick up the changes.

I see you are on phone, there is not a new phone flight just yet.  you'll need to wait a little longer until that flight is available.


aa tree code may have a mistake

The implementation of tree/skew does not match Juliette's (note her separate conditionals), take a close look at it.

fixed, now unused.


Data to play with for OTU descriptions & synonyms

To help us play with merged descriptions and synonyms, I entered some real Bighorn Basin data that is a fun story. We initially had two morphotypes, "JAY" and "Arial", which were in part different morphotypes because we had different features preserved at different sites. As we added sites and specimens, we could see more features and similarities between the two. So, we lumped them into the "JAY Arial" type.
I have entered a selection of specimens belonging to JAY and to Arial, and done a description for each of the original morphotypes based on a composite of specimens. I have also created an OTU that includes all the specimens and thus both descriptions. No holotype chosen.
This dataset will be in both stage/prod and dev. In dev, I will also create both the JAY and the Arial OTU so that we would be ready to try out the synonym function. I'm keeping the data private because it is real and unpublished, but everyone on the PBot core team is a member of the group (as is Scott Wing).

Curious to hear more about this at the next meeting.

A fix for synonym and comment was deployed on dev as part of #295. Please check if it works with your examples as expected.

I am closing this just to merge with #295 for cleanliness...


Self contained unit tests

Hi, it looks like you are trying to maintain a new fork of johm.  I have had some recent changes in my own fork and thought they might be useful to you.  I basically made the unit tests more self contained rather than relying on a preinstalled default redis instance running locally.
These changes basically just include functionality to spin up a small redis instance specifically for the unit tests.  This is very handy in continuous integration environments (bamboo, teamcity, etc) so that no extra set up is needed on the agents to build the code and run the unit tests.
I've re-based my fork on yours so it should be a straight merge with no conflicts.

I need to send a different pull request, I found there was an issue with the unit test redis instance startup on non-windows platforms.

I'm still interested in that PR if you've fixed your problem on non-windows platforms :wink:

Take a look this pull request #3, I have added embedded redis lib... so there is no need for an external Redis to be started any more. What do you say?


Premium per Capacity vs PPU

The table in this section needs to be updated to clarify that there are differences between PPC and PPU workspaces
https://docs.microsoft.com/en-us/power-bi/admin/service-admin-licensing-organization#license-types-and-capabilities

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

ID: ec63d236-fde3-47d8-96a5-5d205dd96b59
Version Independent ID: 0f9cc0dc-e417-ed61-ff2a-7155cdadf8c5
Content: Power BI licensing for users in your organization - Power BI
Content Source: powerbi-docs/admin/service-admin-licensing-organization.md
Service: powerbi
Sub-service: powerbi-admin
GitHub Login: @mihart
Microsoft Alias: mihart

Hi, @danmyhre -- thanks for asking! Assigning to the author to respond.

Hi @danmyhre ,
I see that this Git issue is a bit old, so I'm going to close it.  I hope that you were able to get your questions answered. If not, please revisit the article since it has been updated several times since this Git Issue was submitted.  Another option is to submit your question to the Power BI community (community.powerbi.com).
All the best,
the DOCS team


urls for website

@jheretic can we custom the urls for different pages on the website?
E.g.:
https://prereview.org/communities
https://prereview.org/communities/outbreakscience
https://prereview.org/api

NM, this is what it is already. Sorry.


Typo on the ore tag

It appears to have a typo on the ore tag, this is preventing the generation of draconium and is generating empty tag barriers

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Minimal warnings from GCC/Clang with strict conformance compilation; improvement in MSVC strict conformance compilation; fixed const qualifier issue with fvar type resolution

Contains all changes included in PR #14.
Changes between this PR and PR #14:
GCC/Clang: Compiling with flags -v -ftemplate-backtrace-limit=0 -Weverything -Wall -Wextra -Wpedantic -Wno-c++98-compat -Wno-c++98-compat-pedantic -Wno-used-but-marked-unused -Wno-global-constructors -Wno-disabled-macro-expansion -Wno-double-promotion -ggdb, we have minimal warnings (-Wpadding and -Wheader-hygiene remain)
MSVC: Compiling with switches /W4 /permissive- /bigobj /EHsc, most Autodiff sourced issues are gone, but some remain, particularly for MSVC 17.
I was able to add a fix for issue #15 using std::decay and type aliases without having to change much of the code. This is part of this PR.

These files are in beta stage at best. Before time of submission, necessary but messy code will move, and the rest will be removed.
That #define is related to Windows max/min, for which I’m using parens where needed.

Codecov Report

Merging #16 into develop will increase coverage by 34.79%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #16       +/-   ##
==========================================
+ Coverage     65.2%   100%   +34.79%
==========================================
  Files            9      9
  Lines         2500   1513      -987
==========================================
- Hits          1630   1513      -117
+ Misses         870      0      -870

Impacted Files
Coverage Δ

include/boost/math/differentiation/autodiff.hpp
100% <ø> (ø)
:arrow_up:

...lude/boost/math/differentiation/autodiff_cpp11.hpp
100% <100%> (ø)
:arrow_up:

test/test_autodiff2.cpp
100% <100%> (ø)
:arrow_up:

test/test_autodiff6.cpp
100% <100%> (+55.84%)
:arrow_up:

test/test_autodiff1.cpp
100% <100%> (ø)
:arrow_up:

test/test_autodiff3.cpp
100% <100%> (ø)
:arrow_up:

test/test_autodiff4.cpp
100% <100%> (ø)
:arrow_up:

test/test_autodiff.hpp
100% <100%> (ø)
:arrow_up:

test/test_autodiff5.cpp
100% <100%> (+51.98%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

IMO appeasing Microsoft warnings is a low priority and not worth adding complexity to autodiff.hpp for.
If you are currently walking through the dark valley of complexity searching for the light of simplicity, then please let me know when you find it. :)

IMO appeasing Microsoft warnings is a low priority and not worth adding complexity to autodiff.hpp for.
If you are currently walking through the dark valley of complexity searching for the light of simplicity, then please let me know when you find it. :)

This should be ready to merge. The Boost guidelines say a bit about being able to compile without warnings with strict conformance, so I’m keeping an eye on complexity metrics as I make changes. MSVC likely just needs conversion operators at this point, but I’m stepping through the breaks to see how little intrusion I can make to achieve the outcome.
I’ll likely try to replace the int conversion operator with a generic conversion operator so I don’t need to write an operator for each type that happens to need be static casted to a given arithmetic type.

I'm closing this PR and will resubmit later this week (with more needed changes), we're giving Boost-CI priority (and since it's much less intensive than strict conformance fixes for all Clang/GCC/MSVC). I've moved the conformance changes to branch kedarbhat/autodiff branch feature/strict_conformance.
I've lined kedarbhat/autodiff branch develop with pulver/autodiff branch develop so that once Boost-CI integration is complete, a PR will be more easily understood.


Graph高级用法可以参考这个视频[Issue]:

Describe the issue
https://youtu.be/CvCVFH7bsAk
Steps to reproduce
No response
GraphRAG Config Used
No response
Logs and screenshots
No response
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

Community how-to contribution


#578: Add optional flag for Changes API to limit update operation out…

…put to modified properties only.  Also includes properties flagged as unique, since they would be needed to identify the item that is updated.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] The commit message follows our guidelines: https://github.com/LokiJS-Forge/LokiJS2/blob/master/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

Coverage increased (+0.08%) to 92.619% when pulling 641513bea850cd14b35808c9d485c31d6e3bb84b on change_api into 2ab7a1ea87715b7bafe9a9264acb44bed806d7b9 on master.


Unable to download QIIME 2

I am using a Linux (ubuntu 20.04) operating system from Azure.
I have updated the conda version to the latest.
However, I am unable to download QIIME2 using this line of code conda env create -n qiime2-amplicon-2024.5 --file https://data.qiime2.org/distro/amplicon/qiime2-amplicon-2024.5-py39-linux-conda.yml
Does anyone have suggestions on how to solve this issue? Thank you in advance!
`$ conda env create -n qiime2-amplicon-2024.5 --file https://data.qiime2.org/distro/amplicon/qiime2-amplicon-2024.5-py39-linux-conda.yml
Channels:

https://packages.qiime2.org/qiime2/2024.5/amplicon/released
bioconda
conda-forge
defaults
Platform: linux-64
Collecting package metadata (repodata.json): done
Solving environment: | Killed`

Set conda config --set channel_priority flexible before running the code. That worked for me.

@aimirza Thank you for your helpful input here!
@katsudon2000 I am going to close this issue. But if you are still stuck after trying @aimirza 's suggestion please reach out on the qiime2 forum  for help installing your environment. When you do please post your commands and the entire error message you receive when running  :)
--Hannah


Request: Less (CSS) icon

You've got a Sass icon, how about a Less icon?

Does this mean you've added the icon or you won't add the icon?


Libxdp support

libxdp installation is detected with pkg-config
libxdp installation is prioritised over libbpf installations when building
added libxdp info to README.md

Going to close this PR as we found it's difficult to install libxdp on all but the most recent non-LTS operating systems. As the CNI runs directly on the host, it means we'd be passing the burden of this library installation onto all users of the plugins.
Will place libxdp support on its own branch for now until it becomes a little more mainstream, so people can build if they want to test with it.


fix(ui-components): sonar issues

fixes for sonar issues - https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&id=SAP_open-ux-tools

@815are what happened to this one? Obsolete or still being worked on?

@815are what happened to this one? Obsolete or still being worked on?

@tobiasqueck
I will recheck - I applied some sonar fixes, but I remember there was also other sonar issues left, that's why it was not published


shlib_variant issue with AIX

Hello,
I'm working with OpenSSL 3.0.5 on AIX. To avoid clashes with other executables and openssl version, we are using the shlib_variant (https://mta.openssl.org/pipermail/openssl-commits/2017-December/017036.html)
It works fine on all OSes, it's honestly a bliss to use this.
However, we encounter some (minor) issue on AIX.
For instance, on linux, if we use the shlib_variant "foo", we end up with "libcrypto_foo.so.3". Which is what we expected.
On AIX, we end up with "libcrypto.a(libcrypto_foo.so.3)". I expected "libcrypto_foo.a(libcrypto_foo.so.3)". As a result, we still have some clash with other executables (because the OS will still find our libcrypto.a).
I can't simply rename it, because libssl.so depends on "libcrypto.a(libcrypto_foo.so.3)"
Would it be possible to rename it libcrypto_foo.a ? (that makes sense to me, I might be wrong though)
I tried to play with the sharedlib_simple functions and others in platform/Unix.pm or platform/AIX.pm, but did not achieve anything.
I can spend some time to work on it but I'll gladly take some pointers in this case.
Regards,
Robin

I have also been looking into this. Without success, though, there doesn't seem to be an obvious solution. Meanwhile I think we should be better off using linking against .so on AIX again. Or make it an option. No matter how you do it it is wrong anyway.
In my opinion in the end the effect should be that linking is done as -brtl -lcrypto_foo, if shlib_variant is used.
As for pointers I am not sure what you would need exactly, XLC 16.1 - linking, could be one starting point. As you found shlib_variant, you probably know this already. And there is a comprehensive description of the AIX binder and linker available.

Hello,
thanks for your answer. I used the current method, i.e. having libcrypto.a(libcrypto_suffix.so).
I agree with your opinion, but I don't know how to get to this result :)
For the pointers, I meant about the whole perl / Configuration stuff for OpenSSL, as I don't know how it works at all. I tried to play with it a little but after 2 days, I gave up.

marking as inactive, will be closed at the end of 3.4 dev barring further input


Add spacing above feedback component

Trello card: https://trello.com/c/T3ez6Xxk/12-often-no-space-between-content-and-component

Should we have something in the component docs about the desired spacing between the component and the elements above? Should it just be what is defined in this PR, or do we expect there to be other spacing?
For example, there's already a 30px space between the content and component on a browse page.


When is it appropriate to set the advertisement filter?

Could the advertisement filter be set during a scan? Would the scan then be updated to use the new filters?

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

ID: d4b0d984-20b4-005d-68ca-7827647d6b50
Version Independent ID: 316280a7-2cc2-f1cf-503d-bbef926cd2ab
Content: BluetoothLEAdvertisementWatcher.AdvertisementFilter Property (Windows.Devices.Bluetooth.Advertisement) - UWP app developer
Content Source: https://cpubwin.visualstudio.com/DefaultCollection/windows-uwp/_git/winrt-api-build?path=%2fwinrt-api-build%2fwindows.devices.bluetooth.advertisement.bluetoothleadvertisementwatcher.yml&version=GBlive&_a=contents
Service: windows
GitHub Login: @wdg-dev-content
Microsoft Alias: wdg-dev-content

Hi, @g-ortuno. Thanks for filing this issue, and my apologies for the delay in responding.
I don't have any additional content for your scenario, over and above what's in the documentation. So it might be worth your pursuing this as a question about the product, rather than as feedback about the documentation (since there's nothing that I can action, as things stand). You got here by clicking Send feedback about > This page. If instead you click Send feedback about > This product, then you'll be taken to the Windows developer feedback site (more info about that below).
Here are all of the developer support resources, for completeness.

The Windows developer feedback site is the right place to lobby for a change, a feature, or to make a suggestion.
For community assistance with development questions, there are our forums, and StackOverflow.
The Windows developer support site explains other support options.
The Visual Studio developer community is available, and responsive. The former User Voice feature is now there on the Suggest a feature tab, if you want to lobby for a change, a feature, or to make a suggestion.
For developer issues, or suspected product bugs, there's Microsoft Support; a support engineer will look at the repro and see whether the issue is a Microsoft product bug, or something else. If it's a bug with a Microsoft product then the ticket will be free of charge.

If you receive assistance via any support resource, and you believe that information would help us improve the docs, then please come back and let us know.
Thanks!
Steve

Thanks! Will do.


gimp: fix build on darwin

Motivation for this change
ZHF: #144627 (@NixOS/nixos-release-managers)
gimp is currently failing to build on darwin:
gui.c:285:36: error: use of undeclared identifier 'GDK_OSX_BIG_SUR'; did you mean 'GDK_OSX_TIGER'?
  if (gdk_quartz_osx_version () >= GDK_OSX_BIG_SUR)
                                   ^~~~~~~~~~~~~~~
                                   GDK_OSX_TIGER
/nix/store/g7bkiccx10fg9v004n62rl7wh97ifd1y-gtk+-2.24.33-dev/include/gtk-2.0/gdk/gdkquartz.h:46:3: note: 'GDK_OSX_TIGER' declared here
  GDK_OSX_TIGER = 4,
  ^
1 error generated.

In #138410 and #138797 we figured this is due to a gtk2 that's missing the bug sur definition. We may or may not want to patch that back in with upstream patch https://gitlab.gnome.org/GNOME/gtk/-/commit/69135bcfe3c6e907e3dfed23d97eae9abfd782bc, but the additional problem is that gimp is using GDK_OSX_BIG_SUR rather than GDK_OSX_BIGSUR, so it looks like we'll have to hack in the definition for gimp anyway.
Edit: the alternative would be to patch gtk2 with the above and patch gimp with https://gitlab.gnome.org/GNOME/gimp/-/commit/5ff5d3898574bb346c8220e3ebde2edfea5a4b6a
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

@ofborg build gimp

We can also apply https://gitlab.gnome.org/GNOME/gimp/-/commit/5ff5d3898574bb346c8220e3ebde2edfea5a4b6a.


Implement a config file

Not until v4, because the rewrite will mean that everyone's config files will break; so I'd like to avoid making people rewrite these things too much. But I'd like to provide the option to do this; either with a .cssnano.yml or a .cssnanorc. I find YAML configs to be much nicer than JSON, but don't see why we couldn't support either one through https://github.com/davidtheclark/cosmiconfig.
This makes dependents such as atom-cssnano much simpler going forward.

+1

@ben-eb: If you find that cosmiconfig is missing something that you need, just let us know!

Will do. :smile:

package.json is all you need.  See https://github.com/sindresorhus/xo#config and https://github.com/sindresorhus/ava#configuration. No point in every tool creating their own config files when we can just keep all config in package.json. Less is more. I made pkg-conf to make this easy.

@sindresorhus You're right, hadn't considered the package.json approach. I'm not a fan of having a dozen dotfiles for configuration per-project either. :smile:

It depends on the config. Is the config for cssnano pretty small? If so, just using package.json would be fine. But, for example, I would not want to have the gigantic configs for stylelint and eslint together in my package.json obscuring the other info there: would much rather have those in separate files.

I don't like package.json cause it becomes very massive with many config. And I can't use npm i without breaking "param": [ 0 ] to
"param": [
  0
]

Better use similar config language.

@davidtheclark It really depends on how many sub-modules we will have in v4's core. Some functionality that exists today as separate modules might be consolidated, other parts might be split up into separate parts.
Right now you can disable all of these modules individually, so if that is any indication of size we might be looking at a big config file. However, this is where I'd like to use presets for configuration to suit most use cases.
Like most things it seems like there is no one size fits all solution. With cosmiconfig we can use either package.json or a config file; I would be happy with supporting both if people would like us to do that, same goes for global config (which I don't personally like, projects often have different configuration on purpose, but can see its value).

If you need to have massive amounts of config for a tool per project, the tool should probably support some kind of shareable config so you don't have to duplicate the config in each project.

And I can't use npm i without breaking "param": [ 0 ] to

That's a small price to pay for ease of use and shouldn't be an issue for most. Just learn to live with it. You already have to with npm.

the original proposal for cosmicconfig already allows for package.json AND a config.js commonJS module, which addresses both package and shareables, doesn't it ?

Disclaimer. I have nothing invested in this as I don't personally use this project. I just want to improve upon the current metafile mess.

Like most things it seems like there is no one size fits all solution.

That shouldn't be the goal either. You should make something awesome for the 90%.

With cosmiconfig we can use either package.json or a config file; I would be happy with supporting both if people would like us to do that

Sometimes being opinionated is a good thing.  I have yet to hear a good argument for why multiple config types should be supported.

I have yet to hear a good argument for why multiple config types should be supported.

I believe the argument is that different people (like yourself) have strong opinions about how they want to configure their project, and many of those strong opinions are incompatible.

I'm not looking for opinions about preferences. We all have our own. I'm looking for actual reasons why having it only in package.json wouldn't work for people.

Different people have their different opinions because of different reasons. If you don't like their reasons, that's why you have a different opinion. You've heard a couple of reasons above that haven't convinced you to change your opinion, but I'm not sure you've convinced others to change theirs. So it goes.

For the cssnano project in particular, package.json is sufficient  -- CSS optimizations shouldn't have rendering side effects so can be made global.
But -- completely off the cssnano topic but related to package.json's inadequacy
For other projects I think an issue is that package.json is global. How do you provide a different set of configs for a sub directory via package.json, start adding 'path/': { settings }, 'path2': {settings}?
Separate RC files might be needed if you are following multiple coding standards in your own project -- e.g.:
given: scss-lint BEM selector rule (or node sass-lint)
given: your website SCSS in BEM
given: a carousel plugin in non-BEM SCSS
given: you have overrides for the carousel SCSS split in carousel/**/*.scss
Running sass-lint over your project would result in BEM violations in carousel/**/*.scss. If it were just a file, you could add an inline settings override. That gets unwieldy with multiple files, so you really need a separate config in carousel/.scsslint.yml
Now think about a large enterprise project where there are multiple such plugins, each with its own coding style that you want to keep in line with.
Will you keep adding paths to package.json? Eventually getting a 2mb package.json file because of so many overrides per-path?

My mistake for not being clear. I'm looking for technical reasons why having config only in package.json wouldn't work. Aesthetics and preferences aside.

@davidosomething In XO we solved it this way: https://github.com/sindresorhus/xo#config-overrides

That gets unwieldy with multiple files, so you really need a separate config in carousel/.scsslint.yml

No, carousel should be its own project with its own package.json.

Now think about a large enterprise project where there are multiple such plugins, each with its own coding style that you want to keep in line with.

Seeing how they're really different things and even with different coding style they should be separate projects.

We're digressing a bit (should we open this discussion up somewhere else?)
The carousel i was referring to was mysite/my-scss/my-carousel-overrides/, rather than the separate node_modules/external-carousel-plugin/ which would have a package.json.
the XO method you have there will end up looking like this for a large project
{ 'overrides': [
  {
     'files': 'my-plugin1-css-overrides-for-plugin1-using-BEM/**/*.scss',
    'override1': true,
... 100 more lines
    'override101': true,
  },
  {
     'files': 'my-plugin2-css-overrides-for-plugin2-using-OOCSS/**/*.scss',
    'override1': true,
... 100 more lines
    'override101': true,
  }
] }

A good reason for having a dotfile in the home directory would be for command line usage, say that you don't want to configure cssnano on a per-project basis, but are happy with using the CLI, and your configuration is different than the default. Like I wrote, this is something that I don't personally like (would rather configure on a per-project basis) but can see its utility.

I've settled on both package.json & cssnano.config.js. package.json is for no-customisation use cases, and cssnano.config.js is for when you might need to pass functions as options. For example, https://github.com/ben-eb/postcss-reduce-idents#encoder is one such option.
These two configurations are identical; package.json:
{
  "name": "awesome-application",
  "dependencies": {
    "cssnano": "^4.0.0"
  },
  "cssnano": {
    "preset": [
      "default",
      {"discardComments": {"removeAll": true}}
    ]
  }
}

Or, using cssnano.config.js:
const defaultPreset = require('cssnano-preset-default');

module.exports = defaultPreset({
  discardComments: {
    removeAll: true,
  },
});


add the note about using babel-plugin-lodash/assert in order to decrease bundle size

SimpleRestClients imports by default entire lodash. Users who use SimpleRestClients and don't use babel can have issues with bundle size. I've made an experiment with the optimized version and the current production version - results provided below.
Note: optimized version uses direct import - import map from 'lodash/map' and util function called invariant instead of entire assert library (assert package provides many methods, however, usually library uses only one of them - assert.ok.). There is unassert, however, this plugin completely removes asserts from production code. The sample application uses simple code from README.
Let's consider several use cases how users can use the library:

A user uses the library without any optimizations and does not use lodash

optimized version

      Asset      Size  Chunks             Chunk Names
  bundle.js  58.8 KiB       0  [emitted]  main

production version

    Asset      Size  Chunks             Chunk Names
 bundle.js   108 KiB       0  [emitted]  main

A user uses the library with babel optimizations and does not use lodash

optimized version

      Asset      Size  Chunks             Chunk Names
  bundle.js  58.8 KiB       0  [emitted]  main

production version

     Asset      Size  Chunks             Chunk Names
 bundle.js   108 KiB       0  [emitted]  main

A user uses the library without any optimizations and uses lodash

optimized version

      Asset      Size  Chunks             Chunk Names
  bundle.js   128 KiB       0  [emitted]  main

production version

     Asset      Size  Chunks             Chunk Names
 bundle.js   108 KiB       0  [emitted]  main

A user uses the library with babel optimizations and uses lodash

optimized version

      Asset      Size  Chunks             Chunk Names
  bundle.js    60 KiB       0  [emitted]  main

production version

     Asset      Size  Chunks             Chunk Names
 bundle.js   128 KiB       0  [emitted]  main

Seems that optimized version has only problems for the third case, and a user can solve it by adding babel optimization because an application depends on lodash and a user has to do it by default. Would be very helpful to add an example of the optimal configuration for webpack/rollup/etc. Does that make sense?

Yeah, feel free to submit a PR with an optimal configuration, or even modify the sample code to use it, and we can link it around from the various repos.


None for optional arguments

Description
Some methods allow for optional arguments. It would simplify client code if a value of None behaved the same way as leaving the argument out altogether.
For example:
slackbot.api_call('chat.postMessage', channel='#general', thread_ts=None, text='hello, world')
should behave the same way as:
slackbot.api_call('chat.postMessage', channel='#general', text='hello, world')
That would allow the programmer to set variables for things like channel, thread_ts, and text and use them as values for arguments without having to have conditional code or other hackery around the call. Example (that doesn't work as expected):
channel='#general'
thread_ts=None
text='hello, world'

slackbot.api_call('chat.postMessage', channel=channel, thread_ts=thread_ts, text=text)

Thank you for considering this enhancement.
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackclient version:
python version:
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

Wow, that's some funky API if that's a thing :-/

@harlowja I've been testing this a bit and it seems like it already does behave like the field wasn't included. It's slightly inconsistent with some of our APIs, but 🤷♂
For example, int he attachments example I mentioned above, I had to explicitly pass an empty JSON object to remove the attachments.
sc.api_call(
    "chat.update",
    channel="#python",
    ts="1519949355.000097",
    text="Python Test 42!",
    attachments="{}"
)

So, I guess there's no work to be done here? 🤔

@marksirota have you seen an example where it doesn't behave like the field was omitted? I wonder if this was a side-effect of the changes I just merged in #293 🤔

Yes, the example I gave in the bug report (the thread_ts argument to chat.postMessage) is (was?) an example where None is different than omission. That's the specific case that brought the issue to my attention. I believe leaving it out makes the message a parent, setting it to None raised an error, IIRC. I can test it again, but not until sometime tomorrow at the earliest.

@marksirota no rush. I think the parsing fixes I just released may have changed that behavior to what you want.


PyTermTk: auto-fit width of result tree (not over a certain width)

This was done in 0.9.0 with https://github.com/jeffwright13/pytest-tui/blob/main/pytest_tui/tui_pytermtk.py#L68


ci: using python 3.8

:tada: This PR is included in version 5.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Start to model the access conditions from Sierra

For https://github.com/wellcomecollection/platform/issues/5171
This patch starts to map various bits of Sierra access data into our model. The full explanation is in the RFC, but hopefully this code is understandable on its own.
This is an initial implementation that covers the "easy" cases (782,533 of 786,987 items, or ~99.5%), but there's a long tail of weird ones that need more investigation. I'm deliberately not using it anywhere except the tests yet – I don't want this on the critical path for the transformer until I've dug into that long tail some more.

The crux of this patch is a new class:
object SierraAccessCondition {
  def apply(bibId, bibData, itemId, itemData): (List[AccessCondition], ItemStatus)
}

It returns two things:

The access conditions of the item, to be presented through the catalogue API. This will take time to react to changes made in Sierra, e.g. if somebody places a hold on an item.
The status of the item, to be used in the items API. Here, we'd query Sierra to get the latest item information, and use that to update the page.

At some point we might split these two bits of logic, but they're so closely intertwined that it made sense to develop them together.

For the purposes of the front end, the string "Online request" in the access condition terms will indicate whether an item can ever be requested online, and whether it's worth querying the items API to get up-to-date availability.
This code is a bit more conservative than Encore/the OPAC – in general I think a false negative (not showing the button when we could show it) is preferable to a false positive (showing the button when we shouldn't). It's also more consistent – a lot of the issues in the current interface are caused by disagreeing sources of data. This code tries to create a more self-consistent output.

If this approach doesn't make sense or people can't follow it, then it'd be good to know now, before I start working on the long tail. I'll deal with the long tail either by adding extra cases, or getting the data changed in Sierra.

Closing as superseded by various other PRs.


remove property decorator misleading line

If the property decorator returns a value, it will be used as the Property Descriptor for the member.
It's incorrect, the property decorator can't return a value

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Fix data provider docstring formatting

Fix docstrings of data providers so the html of the API reference is correctly formatted as its not now e.g.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.5%) to 82.735%

Files with Coverage Reduction
New Missed Lines
%

qiskit_finance/data_providers/exchange_data_provider.py
2
73.02%

qiskit_finance/data_providers/wikipedia_data_provider.py
4
73.08%

Totals

Change from base Build<PHONE_NUMBER>:
0.5%

Covered Lines:
714

Relevant Lines:
863

💛  - Coveralls


Nominal typing on field or StructType?

I was surprised to see that the choice between nominal and structural checks is done on the field level, rather than the StructType level. This choice is more general, yes, but it is surprising conceptually. Did you have a use case in mind for this flexibility?

It's not really about flexibility: it just falls out of how the type checking works. For all types, the checks are done in the [[Write]] internal method on the Value Type. For Struct Types, the accompanying Value Type is reified as the ref method on the Struct Type constructor. For the types the WebAssembly GC proposal introduces, additional reference Value Types are introduced.
See @lukewagner's explanation for further details.


Better error messages when endpoint is incorrect

E.g., when trying to test against a direct version published endpoint and the luisDirectVersionPublish setting is not set.

Not sure whether we can provide anything better than details about the 404 response.


The reconstructed image looks okay, but the sampling results are very poor

i used the VanillaAE, used all the celeba img_align_celeba data, 200k
this is epoch 28 reconstructed  image:

this is epoch 28 sample image:

i think the sample was very bad, how can it happen?

It's possible that you're overfitting. I just posted an issue about this, but I think the convolutional layers are supposed to get fewer channels toward the middle because it's an autoencoder, not more channels in the middle. Try reversing hidden_dims maybe and train again.


ModernMT refuses to train new engines

ModernMT constantly crashes when training on two GPU's. And no, it's not temperature-related. Power Supply also is 1000 Watt.
It happens when the data starts to converge around loss = 2. It's a hard crash: no logs, just a hard reset. Temp before and after crash is low. The crash happens after about 30 mins, every time again. With one gpu on the same dataset, the crash does not happen (no matter which gpu is chosen). It only seems to happen when training Chinese to Dutch. English-Dutch on 2 GPU's went perfect for 16 hours at a stretch, with a successful training.

Entering the fairseq command gives:
bash: syntax error near unexpected token '('

HI @LoekvanKooten
It happens sometimes that a particular batch, whose content and size depend on several parameters, lets the system crash.
Usually, the easiest solution is to modify the value of the''--max-tokens'' parameter (for instance from 1536 to 1472).
We are working for a more robust way to solve this issue.
It could be useful for us to have the full training.log reporting the failure.
thanks for your feedback

Yes, a solution would be great. I'm now training another engine for another language pair, and this one crashes straight after 20 seconds of training or so. The crash times are always very consistent. With one dataset it's after 20 seconds, with another dataset after 2 hours, but always after 2 hours.
As for now I'll just let he 2nd graphics card sit and relax using --gpus 0 But this does mean much slower training times.


0.1.6 does not update values

After updating from 0.1.5 to 0.1.6 I noticed that after start of the plugin, it only updates once after startup and then keeps the same value even if the MQTT topic changes.
I reverted back to 0.1.5 and everything is fine again.
My topic's JSON looks like this:
{"pv": {"power": 1829.0, "voltage": 235.03, "energy_forward": 34083.29 }}

This is the error I found in the log:
2024-02-24 07:57:48.572452500 *** starting dbus-mqtt-pv ***
2024-02-24 07:57:55.998191500 Traceback (most recent call last):
2024-02-24 07:57:55.998198500   File "/data/etc/dbus-mqtt-pv/dbus-mqtt-pv.py", line 263, in _update
2024-02-24 07:57:55.998202500     self._dbusservice['/Ac/L1/Power'] = round(pv_power, 2) if pv_power is not None else None
2024-02-24 07:57:55.998208500   File "/data/etc/dbus-mqtt-pv/ext/velib_python/vedbus.py", line 157, in __setitem__
2024-02-24 07:57:55.998212500     self._dbusobjects[path].local_set_value(newvalue)
2024-02-24 07:57:55.998216500 KeyError: '/Ac/L1/Power'

same here

I have update the json structur from mininimal to L1, L2, L3 and it run´s perfectly. iI think the problem is the version of Venus OS upper 3.19. Sorry for my english.

Not it's not related to the Venus OS version. It was due to a change in this code. I tested it and was not able to reproduce it after the latest fix.


Crash when going from light to dark

Was getting an error about interpolation failing and after having a look I fixed it by changing the property value for light mask cy
defaultProperties: {
  ...
 light: {
    ...,
    mask: {
       cx: '100%',
       cy: '0%'.   <------ This was 0 before
...
}

Had the same error. Thx for the hint!

Thanks for bringing up this issue and adding a solution! I have published a fix accordingly :rocket:


[🐞] Breaking change introduced in 1.7.0 - qaction in url is appended using routeAction and globalAction

Which component is affected?
Qwik Runtime
Describe the bug
1.6.0 didn't have this issue
Reproduction
.
Steps to reproduce
can provide repro if needed
System Info
System:
    OS: macOS 14.5
    CPU: (8) arm64 Apple M2
    Memory: 96.05 MB / 8.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.15.0 - ~/.nvm/versions/node/v20.15.0/bin/node
    Yarn: 1.22.22 - /opt/homebrew/bin/yarn
    npm: 10.7.0 - ~/.nvm/versions/node/v20.15.0/bin/npm
    pnpm: 9.4.0 - /opt/homebrew/bin/pnpm
    bun: 1.1.18 - /opt/homebrew/bin/bun
  Browsers:
    Safari: 17.5
  npmPackages:
    @builder.io/qwik: ^1.7.0 => 1.7.0
    @builder.io/qwik-city: ^1.7.0 => 1.7.0
    typescript: 5.5.3 => 5.5.3
    undici: ^6.19.2 => 6.19.2
    vite: ^5.3.3 => 5.3.3

Additional Information
No response

Can you provide a reproduction repository pls?

We have the same issue here! Tried using <Form action={...}> as well as programmatically submitting the action, and got the ?qaction=... appended to the browser URL. Making multiple calls / using multipleglobalAction$ on a page resulted in a new ?qaction=...&qaction=...&qaction=...&qaction=... appended each time.

Please, provide a simple reproduction. Does it happen on every form submission or when you use redirects? Because I have similar problem when I redirect with URLSearchParams, but in my case this happens when I reuse the url from request event.

Please check 1.7.1

I threw this reproduction very quickly using 1.7.1: https://stackblitz.com/edit/qwik-starter-cidktr

You can see that once you click on the submit buttons, you'll have the &qaction=... appended to the url.

Can confirm that too in very basic Qwik application. The issue occurs even without redirects.
Looks like after form submission client just navigates to ?qaction=
I'll see if this is related to my change here: https://github.com/QwikDev/qwik/pull/6410

Yes, I think my change is the reason of this problem. I can reproduce it in my project with patched Qwik v1.5.5. I will take a look and send a PR if I find a solution.

To be clear: My fix works as intended - it checks the whole pathname for changes, including URLSearchParams. The problem is that when Qwik City Component decides whether to update the url or not it ignores internal Qwik params, such as qaction and treats them as if the URL was changed.
So, I think the solution would be to remove internal params from the url before we check whether or not to change: essentially, this object should not include the qaction and other internals in its URLSearchParams.
Also, I think internal params should also be excluded from request event object, because it I try to build a redirect url with already received params (for example, in routeAction$) I had to to this manually, and if Qwik is ever change internals I'll get unwanted params in my URLs.

@octet-stream that seems a correct assessment. So you think you can implement your proposal?

Yes, at least for QwikCityComponent. The only question: where is the right place for the fix. I will take a look as soon as I have time for it (which is likely tomorrow).
Also, do we have internal search params defined somewhere? Like in an array or something, so I can just reuse to for the fix? :)

Can you provide a reproduction repository pls?

Sorry I thought it was an obvious bug that's why I thought didn't need to provide a repro.

Yes, at least for QwikCityComponent. The only question: where is the right place for the fix. I will take a look as soon as I have time for it (which is likely tomorrow).

@octet-stream Was wondering did you got a chance to address the issue? please let me know if you require any additional information.

please let me know if you require any additional information.

I think I have enough information, I'll work on it today.

Though I might need some help from maintainers.
My current solution is remove qaction after this line: https://github.com/QwikDev/qwik/blob/563ddc512a9dec38194f10914ad5988f523bd71d/packages/qwik-city/runtime/src/qwik-city-component.tsx#L265
It may be used as a quick fix for the issue, however I don't feel it is a good one.
From what I understand, the href parameter in clientData object is acquired from some virtual element (qwik city context gets it here: https://github.com/QwikDev/qwik/blob/563ddc512a9dec38194f10914ad5988f523bd71d/packages/qwik-city/runtime/src/qwik-city-component.tsx#L254), as a pathname with query params (if any). But where is this pathname being assigned? I don't understand it. Is it set somewhere from the fetch Response? If so, then where? Because I don't see anything in loadClientData function which is from what I can tell a place where actions being sent to the server and then their result is being parsed (I am referring to this code https://github.com/QwikDev/qwik/blob/563ddc512a9dec38194f10914ad5988f523bd71d/packages/qwik-city/runtime/src/use-endpoint.ts#L35), at least it gets executed where an action is sent.
/cc @wmertens @gioboa

I haven't delved into that code too much yet but the context element is needed to know which route rendered a certain element.
There are only a few q parameters used (I think maybe even only qaction, not sure, on phone) so I think your proposed solution is ok for now.

There are only a few q parameters used (I think maybe even only qaction, not sure, on phone) so I think your proposed solution is ok for now.

Yes, all from what I can tell, all internal query params are defined here: https://github.com/QwikDev/qwik/blob/563ddc512a9dec38194f10914ad5988f523bd71d/packages/qwik-city/runtime/src/constants.ts#L9-L13
For our case I think removing qaction should be enough.
I'll send a PR with current solution then, but I'll comment on there and maybe we also come to better solution during the review.

Okay, I think I found another problem, but this might be expected (because I see this behavior in 1.5.5 too): If I disable JS and send a form, this search params will be set, when browser itself sends a form, it will request a url from action attribute of a form and then just render whatever the result is + change address to this URL. This can be fixed if Qwik redirects browser after the action is fired, but I'm now sure if that is intended behavior. For SPA navigation my fix works as expected. But maybe I should bring this one up in the PR.

For sure your fix has to work without js as well.

@octet-stream would this work? #6694

This should work too. But I would rather get keys from constants and remove them from URL object (you can copy one if side effects should not occur).
Should I update my PR or you will merge yours (if so, please add my test case)?


Jade renamed to Pug

As of Dec. 2015, the Jade package was renamed to Pug.  Expressworks should refer to the newer package (see https://github.com/pugjs/pug/issues/2184).

@owenfromcanada thanks for the heads up! feel free to make a PR if you want

I've submitted a PR for this: https://github.com/azat-co/expressworks/pull/112


Missing for and since in options in deprecations.

Ember Simple Auth: The AuthenticatedRouteMixin is now deprecated
Ember Simple Auth: The session service's events API is deprecated; to add custom behavior to the authentication or invalidation handling, override the handleAuthentication or handleInvalidation methods.
Ember Simple Auth: The ApplicationRouteMixin is now deprecated; it can be safely removed.

The following deprecations are missing the for and since options which are causing additional depreciations warnings.
Build 3.1.0

this is the full message shown in the console
When calling `deprecate` you must provide `for` in options. Missing options.for in "ember-simple-auth.mixins.authenticated-route-mixin" deprecation [deprecation id: ember-source.deprecation-without-for]

Chiming in since I hit this deprecation message too today.
ESA won't be usable on ember 4.0 until this deprecation is removed

We've fixed this in some release a while back, should be fine now.


Updated steam.users and steam.apps

Added Descriptions to steam.apps, changed "Steam ID" to "Steam 64 ID" for readability
Added country parameters to get_app_details() and search_games()

(Please read comments in steam.apps for clarification)

@PsiPi-Codes I merged this and updated the package on PyPi to python-steam-api 1.0.9

Alright, thank you!
On Fri, Jul 28, 2023 at 2:45 PM David Salazar @.***>
wrote:

@PsiPi-Codes https://github.com/PsiPi-Codes I merged this and updated
the package on PyPi to python-steam-api 1.0.9
—
Reply to this email directly, view it on GitHub
https://github.com/deivit24/python-steam-api/pull/7#issuecomment-1656364138,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBRF4NJVFZYPBYACHDLZSKDXSQXHRANCNFSM6AAAAAA235JPNM
.
You are receiving this because you were mentioned.Message ID:
@.***>


typescript paths alias

Describe the bug
I try to get the typescript path aliases to work since it should be working right now on the 1.2.65 swc core version, am i doing something wrong ?
TypeError [ERR_INVALID_MODULE_SPECIFIER]: Invalid module "@api" is not a valid package name imported from C:\Users\x\test\hello.js
    at packageResolve (internal/modules/esm/resolve.js:601:11)
    at moduleResolve (internal/modules/esm/resolve.js:691:18)
    at Loader.defaultResolve [as _resolve] (internal/modules/esm/resolve.js:805:11)
    at Loader.resolve (internal/modules/esm/loader.js:88:40)
    at Loader.getModuleJob (internal/modules/esm/loader.js:241:28)
    at ModuleWrap.<anonymous> (internal/modules/esm/module_job.js:72:40)
    at link (internal/modules/esm/module_job.js:71:36) {
  code: 'ERR_INVALID_MODULE_SPECIFIER'
}

Input code
api.ts
export const api = 0;

hello.ts
import { api } from '@api';
console.log('api', api);

Tsconfig
{
    "compileOnSave": false,
    "compilerOptions": {
        "baseUrl": ".",
        "paths": {
            "@api": ["src/api"]
        }
    }
}

Config
{
    "jsc": {
      "parser": {
        "syntax": "typescript",
        "dynamicImport": true
      },
      "target": "es2016",
      "paths": {
        "@api": ["src/api"]
      }
    },
    "minify": false
}

Expected behavior
It should replace the alias with a relative path
Version
1.2.65

tsc, or babel (unless separate plugin specified) both doesn't transpile alias to runtime resolvable path and each tool like webpack / jest specifies way to resolve, may swc would leave as-is as well? for some cases, I expect jsc.paths being used as resolve time only - it's not unusal runtime points to different based on tooling such as jest test points to stub mock instead of real module for path aliases.

Please please please - how do we fix this <3

I found that I had to use a .swcrc file with no paths during the build. BTW the --config-file flag didn't work for me.
Then, something like this in package.json
...
"type": "module",
...
"build": "swc src  -d dist",
"start": "node --experimental-specifier-resolution=node dist/index.js",
...
"imports": {
    "#lib/*": "./dist/lib/*"
}


Consider the TypeScript Partial Type

I noticed in the codebase that there’s a directory of generated structs src/generated-structs. Files like OptionalFunctionProps.ts, for example, define interfaces where most properties are optional.
// Example
export interface OptionalFunctionProps {
  readonly runtime?: aws_lambda.Runtime;
  readonly handler?: string;
  readonly code?: aws_lambda.Code;
  // ...
}

I was wondering if instead of defining these explicitly, we could replace the OptionalFunctionProps entirely by using Typescript Partial Type like this Partial<FunctionProps> directly in usage. This would eliminate the need for separate optional interfaces and keep the code more concise.
// BEFORE
export interface NextjsDistributionOverrides {
  readonly edgeFunctionProps?: OptionalFunctionProps;
  readonly distributionProps?: OptionalDistributionProps;
  // ...

// AFTER
export interface NextjsDistributionOverrides {
  readonly edgeFunctionProps?: Partial<FunctionProps>; // FunctionProps imported from aws-cdk-lib
  readonly distributionProps?: Partial<DistributionProps>; // DistributionProps imported from aws-cdk-lib
  // ...

By using Partial<>, we reduce redundancy and the risk of mismatches, as updates to the base interface are automatically applied.

Questions

How are the optional types currently generated, and is the process flexible enough to replace these optional interfaces with direct usage of Partial<T>?
Is there a reason not to use the TypeScript Partial Type?

@coltenkrauter, I wish we could! See here for why not: https://github.com/mrgrain/jsii-struct-builder. That link will answer both of your questions.

Ah darn... I'm not surprised that there is a good reason. Still a bummer 🌵


Avoid creating Blocklist for each shard

Probably making the underlying struct Send/Sync + having a RwLock on the resource if needed could do the trick.

merged on 86606b2e9dbaeff455e39d6b5f8ffe066d0061e4