Cannot perform atomic access on ARMv7

Hello! I'm using RxJava, the problem is on ARMv7 devices (iPod Touch for example) when I do smth like:
<Observable>.subscribeOn(Schedulers.newThread())
        .observeOn(Schedulers.computation())
        .subscribe(subscriber));

I get:
Exception in thread "RxComputationScheduler-1" java.lang.IllegalStateException: Fatal Exception thrown on Scheduler.Worker thread.
    at 0x002e5419 rx.internal.schedulers.ScheduledAction.run() + 556
    at 0x006c7153 java.util.concurrent.Executors$RunnableAdapter.callInternal() + 44
    at 0x006c7117 java.util.concurrent.Executors$RunnableAdapter.call() + 10
    at 0x006d6aa9 java.util.concurrent.FutureTask.run() + 244
    at 0x006ea1d7 java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run() + 194
    at 0x006f38f5 java.util.concurrent.ThreadPoolExecutor.runTask() + 424
    at 0x006f3695 java.util.concurrent.ThreadPoolExecutor.runWorker() + 128
Caused by: java.lang.AssertionError: Cannot perform atomic access on unaligned address 0x153b2194
    at 0x004f9519 java.lang.AssertionError.<init>() + 34
    at 0x004f94f3 java.lang.AssertionError.<init>() + 12
    at 0x005e2ec3 sun.misc.Unsafe.getObject() + 0
    at 0x00131d01 rx.internal.util.unsafe.SpscArrayQueue.soConsumerIndex() + 346
    at 0x00131b79 rx.internal.util.unsafe.SpscArrayQueue.poll() + 232
    at 0x0012ee55 rx.internal.operators.OperatorObserveOn$ObserveOnSubscriber.call() + 380
    at 0x0010b7f1 rx.internal.schedulers.EventLoopsScheduler$EventLoopWorker$.call() + 164
    at 0x002e52f1 rx.internal.schedulers.ScheduledAction.run() + 260
    at 0x006c7153 java.util.concurrent.Executors$RunnableAdapter.callInternal() + 44
    at 0x006c7117 java.util.concurrent.Executors$RunnableAdapter.call() + 10
    at 0x006d6aa9 java.util.concurrent.FutureTask.run() + 244
    at 0x006ea1d7 java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run() + 194
    at 0x006f38f5 java.util.concurrent.ThreadPoolExecutor.runTask() + 424
    at 0x006f3695 java.util.concurrent.ThreadPoolExecutor.runWorker() + 128

The same java code runs perfectly on ARM64 devices (iPhone 5s+)
It does not matter what subscriber and observer I give (computation/io/mainthread/newthread) always crash EXCEPT Schedulers.immediate()
Thank you!

You're failing this alignment check. This is odd because the reported pointer is indeed 32-bit aligned and should pass the test.
That test should boil down to the following after macro substitution and typedefs:
if (sizeof(uintptr_t) != sizeof(id) || (ptr & (alignof(uintptr_t) - 1)) != 0)
This passes for me on armv7. What do you get from the following log statement on the same device?
NSLog(@"%lu %lu %lu", sizeof(id), sizeof(uintptr_t), alignof(uintptr_t));

@kstanger thanks for reply
On armv7 it gives me:
4 4 4
On arm64:
8 8 8

Method that crashes on "putOrderedLongWithId:withLong:withLong:":
void RxInternalUtilUnsafeSpscArrayQueue_soConsumerIndexWithLong_(RxInternalUtilUnsafeSpscArrayQueue *self, jlong v) {
    jlong offset = JreLoadStatic(RxInternalUtilUnsafeSpscArrayQueueConsumerField, C_INDEX_OFFSET);
    BOOL res_without_offset = (sizeof(uintptr_t) != sizeof(id) || (((uintptr_t)self) & (__alignof__(uintptr_t) - 1)) != 0);
    BOOL res_with_offset = (sizeof(uintptr_t) != sizeof(id) || (((uintptr_t)self + offset) & (__alignof__(uintptr_t) - 1)) != 0);
    NSLog(@"%lu %lu %lu", sizeof(id), sizeof(uintptr_t), __alignof__(uintptr_t));
    NSLog(@"Offset: %lld, res_without_offset: %@, res_with_offset: %@", offset,
          (res_without_offset ? @"YES" : @"NO"), (res_with_offset ? @"YES": @"NO"));
  [((SunMiscUnsafe *) nil_chk(JreLoadStatic(RxInternalUtilUnsafeUnsafeAccess, UNSAFE))) putOrderedLongWithId:self withLong:JreLoadStatic(RxInternalUtilUnsafeSpscArrayQueueConsumerField, C_INDEX_OFFSET) withLong:v];
}

I added some debug strings, the result is:
2016-10-12 17:59:07.787 App-iOS[2986:372091] 4 4 4
2016-10-12 17:59:07.788 App-iOS[2986:372091] Offset: 404, res_without_offset: NO, res_with_offset: NO

As a temp solution you can disable unsafe api:
#pragma mark Rx
- (void)configureRx {
    #ifndef LP64
    [JavaLangSystem<EMAIL_ADDRESS>withNSString:@"true"];
    #endif
}

And call configureRx in application:didFinishLaunchingWithOptions:

I think I see the problem. The original stack trace was misleading, pointing at Unsafe.getObject when you're actually calling Unsafe.putOrderedLong.
So the problem appears to be that the "consumerIndex" field is declared as long but not volatile. And I expect that int64_t has 4-byte alignment while _Atomic(int64_t) has 8-byte alignment on armv7.
If that is indeed the problem, then I think the only solution would be to enforce 8-byte alignment on all long fields. There will be a bit of a memory cost to pay for that. Most uses of Unsafe will declare the fields being accessed with 'volatile' so that's why this hasn't come up before. If you're able to edit the code, I suggest adding 'volatile' to the "consumerIndex" field.

What do you get from the following log?
NSLog(@"%lu %lu %lu %lu", sizeof(int64_t), sizeof(_Atomic(int64_t)), alignof(int64_t), alignof(_Atomic(int64_t)));
On my armv7 device I get an alignment of 8 for both the int64_t and the _Atomic(int64_t). But the pointer from your original error is 4-byte aligned and not 8-byte aligned.

I stumbled over the same issue and used https://github.com/google/j2objc/issues/803#issuecomment-253222711 to qick-fix it. Thx @ashitikov !
But I've also inserted your last NSLog @kstanger and I get this on my iPhone 4s simulator, the simulator where I got the exception:
8 8 8 8
Hope that helps a bit, if I have some spare time I could also try your suggested change (adding volatile).


Variable substitution not working when hyphenation is active on text frame

Scribus: 1.6.1
ScribusGenerator: latest version
When hyphenation is active on a text frame where the variables are, no substitution takes place inside this text frame. When text is de-hyphenated everything works fine.
Current work around is to place variables in their own text frame, but that might not always be feasible, especially when variables appear in the middle of some text.

I just found out that it is possible to select a portion of a text and selectively deactivate hyphenation. So when placing variables inside a hyphenated text make sure to select the variable and de-hyphenate it!

That snippet of information appears to be an ideal candidate for the FAQ :wink:

Good idea!
Even better (from a UX perspective) would be if the variable is treated as not having been hyphenated in the first place, so that users may not need to worry about the hyphenation status of the variable. I don't know whether this is possible.
Is it possible to dehyphenate anything between %...% by the script?


Crashes that occurred during the app upgrade process were counted in the new version

Is there an existing issue for this?

[X] I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
[X] I have reviewed the documentation https://docs.sentry.io/
[X] I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?
Sentry Saas (sentry.io)
Electron SDK Version
4.24.0
Electron Version
30.0.1
What platform are you using?
None
Link to Sentry event
No response
Steps to Reproduce

mock a main process crash in old version
install the new version
upload the crash to the new version in sentry server.

Expected Result

upload the crash to old version

Actual Result

upload the crash to new version

We've fixed this bug previously and have an integration test that checks this:
https://github.com/getsentry/sentry-electron/blob/e909ab277481259e8755eeab67c8fd69d08fed94/test/e2e/test-apps/native-sentry/main-update/src/main.js
Which version of the SDK were you using before updating to 4.24.0?

Thanks. I will check it.

Thank you, this problem disappears after using the new version


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/363 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 2f061c0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


fix: check only 1 pssm for variant queries

#428 fixed by only checking for residues on chains for which a pssm file was provided.
#429 will not be fixed, because we want users to provide pssm files for all residues that are included in the graph.
Created an option to suppress the error thrown by _check_pssm and give a warning instead.

does this require additional documentation, beyond the docstrings?
created tests for error suppression

Create tests for variant query where there is a second chain for which no pssm file is provided:

throw error if second chain is inside radius
no error if second chain is NOT inside radius

Refactor existing tests relating to _check_pssm
Allow for _check_pssm to throw more verbose errors

this is only accessible from source code, which I think is ok because it's more of a troubleshooting/debugging mode thing than something that we expect the user to want to touch, right?

closes #428, #429

I tried with running preprocessing with this branch. It seems it still tries to check for all chains in PDB?
Unfortunately some PDBs have errors in chain names, so there are inconsistencies between no. of PDB chains and no. of PSSMs.
I'll do checks again tomorrow

The changes work. This pull request can be closed.
However, there should be a provision to include other pssm chains in case the radius includes multiple chains. can be dealt in #429

The changes work. This pull request can be closed. However, there should be a provision to include other pssm chains in case the radius includes multiple chains. can be dealt in #429

It works because it now fills the pssm feature with 0s. So this PR actually deals with both #428 and #429, as they are closely related. In retrospect, maybe that wasn't ideal, because now things are getting a bit confused.
I will await decision on #429 to know whether we actually want to fill it with 0s, or want to force pssm files to be present for all chains and then either create separate PRs or resolve both issues here.

I am making a small dataset with variants that have neighbours in multiple chains.
I suppose that may be the only way to test consequence of 0s or non-0s in predictions (non-0s will be ideal for these cases)

My test was for the general applicability. By omitting pssm files and adding 0s we would treat such variants as though they are solvent-exposed variants- which is not quite right, although I am not sure how many such cases are there. I have never tested such a thing with deeprank-mut, but it is expected and I do note that solvent-exposed variants are treated differently in network.
I would prefer to include all chains within the radius, so may be user can definitely include all relevant chains. Rest can be 0

so do I understand correctly that all residues from chains within the radius should be included in the network and that for those chains a pssm file should be required?

@rgayatri , can you please test on the current branch if it still works as intended for you?
Note that Query objects now have an optional argument suppress_pssm_errors

I think we're missing a test that uses suppress_pssm_errors=True. Indeed I was wondering, what happens if the errors are suppressed and pssm files are missing or incorrect? What will conservation-related features contain?

lines 390 and 413 of test_query.py ;)
Good question what the hdf5 file will then contain. I will look into it.

I think we're missing a test that uses suppress_pssm_errors=True. Indeed I was wondering, what happens if the errors are suppressed and pssm files are missing or incorrect? What will conservation-related features contain?

lines 390 and 413 of test_query.py ;-)
Good question what the hdf5 file will then contain. I will look into it.

Oh right! What if we explicitly set suppress_pssm_errors in the query initialization (maybe redefining the query in the test)? This way is clearer how such a parameter can be used, instead of setting the internal _suppress. Since we don't have explicit examples about suppress_pssm_errors, can be good to have it in the tests

I think we're missing a test that uses suppress_pssm_errors=True. Indeed I was wondering, what happens if the errors are suppressed and pssm files are missing or incorrect? What will conservation-related features contain?

lines 390 and 413 of test_query.py ;-)
Good question what the hdf5 file will then contain. I will look into it.

Oh right! What if we explicitly set suppress_pssm_errors in the query initialization (maybe redefining the query in the tests)? This way is clearer how such a parameter can be used, instead of setting the internal _suppress. Since we don't have explicit examples about suppress_pssm_errors, can be good to have it in the tests

I did it this way to make sure that the test uses the same instance of the object that failed if the error suppression was not turned on (a few lines above). This way, if someone modifies the Query object in the test, we are sure that both tests run the same instance.
Do you think I should add an additional test where it is set upon initialization?

I think we're missing a test that uses suppress_pssm_errors=True. Indeed I was wondering, what happens if the errors are suppressed and pssm files are missing or incorrect? What will conservation-related features contain?

lines 390 and 413 of test_query.py ;-)
Good question what the hdf5 file will then contain. I will look into it.

OK, I looked into it:

for incorrect entries in the pssm: it just used the "wrong" data. So the pssm data that is associated to the position number of that residue, even though it is the incorrect residue (@rgayatri is this the behavior that you want?)

for missing entries in the pssm:

if the entry is not part of the graph, it doesn't complain
if the entry is a part of the graph, then it fails at a later step (so check_pssm passes, but then an error is thrown I think when it tries to write data into that key).

The latter point made me realize that it still checks all residues from the chain, irrespective of whether they are a part of the graph. I will rewrite so that it only checks those residues that are part of the graph.
@rgayatri , what behavior do you expect if a residue is missing from a pssm file? All 0s?
At the end of the day, all these tests make me feel that we just don't want people to use bad pssm files. I don't like the concept of being able to suppress the error, because now we need to hack ad hoc solutions to deal with bad data.

I agree, ideally, a user shouldn't be able to suppress the error indeed and should take care of providing the correct files.

I believe that at this point the current PR is functional. If fed with accurate data, everything works as expected. Otherwise errors can be suppressed if absolutely necessary, but it can still fail if this touches data points that are part of the graph. This is okay behavior, as we do not want users to train with unexpected/unexplainable results.
As such, I will be merging and closing this PR. If there are any remaining problems to be solved about this, we can open new issues about them.


Move to Java 17

Like Trino itself.

Asked on slack channel if there are any concerns.

Latest PR already pushes to 9 .. so we are off an LTS. Also chatted with @martint .. it should be same as Trino for ease of deployment, performance, code features and more.


Add floor ticker colors argument

This PR adds the colors argument to the floor ticker. Follows up on #182.

If you could extract just the colors portion of this pr we could get it tested and merged while we wait for the discordgo pr.

@rssnyder I think for this PR we are now good to go.

Also not stale.

@rssnyder I think we can merge this PR?


Support for Ropsten in safe_creator?

Hi! Thanks for your work on the safe-cli, it's a super useful tool! I was wondering why it doesn't support Ropsten. Is that a permanent decision or are there any plans to support it?
I managed to use the script to deploy a Safe to Ropsten by changing the addresses in these lines:
# Support only Mainnet, Rinkeby, Goerli and Kovan
    safe_contract_address = '0x34CfAC646f301356fAa8B21e94227e3583Fe3F5F'
    proxy_factory_address = '0x76E2cFc1F5Fa8F6a5b3fC4c8F4788F0116861F9B'
    callback_handler_address = '0xd5D82B6aDDc9027B22dCA772Aa68D5d74cdBdF44'

To addresses corresponding to instances of those contracts that I deployed to Ropsten. It worked as is, so I was curious whether there's a deeper reason not to support Ropsten.
Thank you!

Hi @mverzilli ,
Good to hear that, I'm glad you are finding it useful. We are not supporting Ropsten as our official contracts are not deployed on Ropsten and we don't plan to do that soon (our devs and users work with Rinkeby, Goerli and Kovan basically), that's why is not working with Ropsten out of the box.
I added command line flags to the safe_creator so the addresses can be specified on the command line and users don't have to edit the source code.
Kind regards


install.ps1 script is not working on microsoft azure cloud VM

Describe the bug
When I fire up the command in Powershell to download kubescape on my azure cloud VM error will occur.
Environment
OS: Microsoft Windows NT 10.0.19044.0
Version: the version that Kubescape reports when you run kubescape version
Your current version is: v2.0.168

Hey @dwertent i pull request for this issue please marge it

Assign me first please

/assign

Hi @pwnb0y I am testing your PR :)

@Moshe-Rappaport-CA Any updates here?

@dwertent I did tests for @pwnb0y PR (#720), and everything works well.

Merged


Wildcard permissions don't work

permission cardinal.match.* should allow you to cycle ,start ,etc, and cardinal.whitelist.* should allow you to on, off, kick, add, etc etc

Oh, now i know why refs couldn't manage the whitelist :c

Also you should be able to only put only " * " and recieve all permissions


Unable to change dimBackgroundColor

One more problem. Color change dimBackgroundColor does not work.

I'm sorry. There was an error in my code. The resource had to be wrapped like this:
 ContextCompat.getColor(context, R.color.colorPrimary)


change documentation to reflect last version

the doc states, when using cdn version of the library, that this line should be used
https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>but 1.8.0 version doesn't have event firing.
the version which has events as described in the doc is 1.11.1
that line needs to be updated

It's a bit annoying changing the url with every release. That's why I have this line:
IMPORTANT: Always grab the latest versions of the package from JSDELIVR, the ones provided in the examples above might be outdated. Same goes for Vue.js.

well, to my shame, I didn't go through the entire doc. Just copy/paste. :-(


name/ number to enter into track command

Hi, i noticed that u mentioned that i can either use the number or nickname. By nickname do u mean the telegram handle, for eg. @xxyyzz?

Yes, some accounts have a username that can be used to find them (some don't). You can use it, without adding @ in the beginning.


Couldn't Install InfluxDB on Mac OS X EI Capitan V 10.11.6

Hi All -
I was trying to install Influx DB on My Local Machine through the following commands using "brew"
brew update
brew install influxdb
ln -sfv /Users/z0025g0/Library/LaunchAgents/homebrew.mxcl.influxdb.plist ~/Library/LaunchAgents
ln -sfv /Users/z0025g0/Library/LaunchAgents/homebrew.mxcl.influxdb.plist ~/Library/LaunchAgents
launchctl load ~/Library/LaunchAgents/homebrew.mxcl.influxdb.plist
After the installation I could also very well able to start the InfluxDB Process using below command and output result as follows:
34363bcbaf1a:etc z0025g0$ influxd -config /usr/local/etc/influxdb.conf
[run] 2017/01/17 12:02:41 InfluxDB starting, version v1.1.1, branch master, commit e47cf1f2e83a02443d7115c54f838be8ee959644
[run] 2017/01/17 12:02:41 Go version go1.7.4, GOMAXPROCS set to 8
[run] 2017/01/17 12:02:41 Using configuration at: /usr/local/etc/influxdb.conf
run: open server: listen: listen tcp :8088: bind: address already in use
The problem i face here is I am not seeing the Influxdb process runs on the default port 8083 and it strangely runs on 8088
Following command clearly proves that as well
34363bcbaf1a:etc z0025g0$ ps -ef|grep influxdb
1199232726 70029 1 0 Mon11AM ?? 0:07.94 /usr/local/opt/influxdb/bin/influxd -config /usr/local/etc/influxdb.conf
1199232726 77754 47290 0 12:04PM ttys002 0:00.00 grep influxdb
34363bcbaf1a:etc z0025g0$ lsof -Pan -p 70029 -i
COMMAND PID USER FD TYPE DEVICE SIZE/OFF NODE NAME
influxd 70029 z0025g0 3u IPv6 0xebf20faaca0dcf29 0t0 TCP *:8088 (LISTEN)
influxd 70029 z0025g0 6u IPv6 0xebf20faaed8cbee9 0t0 TCP *:8086 (LISTEN)
In the config file I see that bind-address = :8086 is commented out as mentioned below and i have also tried uncommenting out and still doesnt works for me
[http]
Determines whether HTTP endpoint is enabled.
enabled = true
The bind address used by the HTTP service.
bind-address = ":8086"
I am actually looking to open the InfluxDB console in my local browser and configure the data source but that is not working I tried connecting to http://localhost:8083 0r either 8088 which actually runs but couldn't make it work
Could some one please help me with that here

Duplicate of https://github.com/influxdata/influxdb/issues/7760.


bookie_journal_JOURNAL_QUEUE_SIZE Prometheus metric is negative (Pulsar 2.6.1)

Describe the bug
The prometheus metric bookie_journal_JOURNAL_QUEUE_SIZE exposed by bookie is negative. Shouldn't queue lengths be positive
To Reproduce

Launch Pulsar on Kubernetes
Inspect metrics endpoint of Bookkeeper

$ curl http://platform-pulsar-bookkeeper:8000/metrics | grep QUEUE_SIZE
# TYPE bookie_journal_JOURNAL_CB_QUEUE_SIZE counter
bookie_journal_JOURNAL_CB_QUEUE_SIZE 57253790
# TYPE bookie_journal_JOURNAL_FORCE_WRITE_QUEUE_SIZE counter
bookie_journal_JOURNAL_FORCE_WRITE_QUEUE_SIZE 0
# TYPE bookie_journal_JOURNAL_QUEUE_SIZE counter
bookie_journal_JOURNAL_QUEUE_SIZE -57253790

Expected behavior
Queue lengths are zero or above
Screenshots

Additional context

Pulsar 2.6.1
Kafkaesque Helm Chart 1.0.24
Probably has not bearing but exposeTopicLevelMetricsInPrometheus is set to false as we have a lot of topics

I started looking into this tonight. I believe this issue belongs in the apache/bookkeeper repo because the metric is defined there.
Regarding the problematic metric, I already see some irregularities based on parameter names and things getting passed around.
@frankjkelly can you confirm that your only seeing this negative value for bookie_journal_JOURNAL_QUEUE_SIZE?

I found some interesting things that I wanted to share, so I opened a copy of this issue in bookkeeper. I think we should continue all work against this issue there.

@michaeljmarshall Yes only bookie_journal_JOURNAL_QUEUE_SIZE is negative - see below where bookie_journal_JOURNAL_CB_QUEUE_SIZE is positive (and almost the exact inverse value)

bookie_journal_JOURNAL_CB_QUEUE_SIZE is positive (and almost the exact inverse value)

That's a helpful observation. When I was looking through the code last night, I was wondering if there was any type of relationship between those two metrics.

@frankjkelly - I believe you're running with apache bookkeeper 4.10.0 (that is what shipped in the pulsar docker image for pulsar 2.6.1. This bug was fixed for bookkeeper 4.10.1, as seen in this PR: https://github.com/apache/bookkeeper/pull/2290. Pulsar 2.6.2 is 4.10.0 too, but Pulsar 2.6.3 comes with bookkeeper 4.11.1, which has the fix to the metric.

@michaeljmarshall Thanks so much for hunting this down. Closing this ticket . . . .


Use the GitHub app for Travis CI

We are stilling using the legacy "GitHub Services", see
https://github.com/Starkast/wikimum/settings/installations
https://github.com/Starkast/wikimum/settings/hooks/2332295

Travis CI is listed under "Installed GitHub Apps" at https://github.com/organizations/Starkast/settings/installations

I clicked on the "Configure" button for Travis CI (https://github.com/organizations/Starkast/settings/installations/196999) and tried adding this repo, I was redirected to https://travis-ci.com/ and clicked "Starkast" over there (https://travis-ci.com/organizations/Starkast/repositories). It says about this repo:

The following repositories cannot be migrated to travis-ci.com at this time because they are currently active on our legacy platform travis-ci.org. This feature will be available shortly. Please read our docs on open source migration to learn more.

https://docs.travis-ci.com/user/migrate/open-source-on-travis-ci-com/
https://docs.travis-ci.com/user/migrate/open-source-repository-migration


request

kissmanga.com
thx before

Lo buat pull request aja
https://github.com/apkcube/blockhostid/pulls

Done, site added.
Commit: https://github.com/apkcube/blockhostid/commit/f726064bc4bab52ab98dd4de5c6d635baa8fa4e1


updating trigger terraform to allow for the description field, also u…

updating how topic name changes on updates are handled, and adding in description field to process automation triggers.
We used to fail the updates to triggers when the topic name field was changed, now the terraform will explicitly say that a replacement is required, and then asks the user to confirm the changes. So the user will need to be aware that the ID of the trigger will change in this case:

The tests don't work for me. Looks like the description isn't being set


How can I change the rendering of ID of a Header element?

I'm converting Cyrillic alphabet markdown to html. When h2/3/4/5 elements are made when the alphabet is  Latin the header's text is converted to id in a proper way. However when the alphabet is Cyrillic the id-s of the headers become section-1, section-2, section-3 etc.
How can I modify that behavior to be able to generate proper Cyrillic ids?

By default, the auto-identifier tries to convert to ASCII using some character correspondence. Can you post an example headers with cyrillic alphabet?
Otherwise, you should be able to disable the ASCII conversion:
// Remove the previous AutoIdentifierExtension
pipelineBuilder.Extensions.Remove(pipeline.Extensions.Find<AutoIdentifierExtension>());
// Add AutoIdentifier extension with only AutoLink option
pipelineBuilder.UseAutoIdentifiers(AutoIdentifierOptions.AutoLink);

Here's an Cyrillic Header
Здравейте, аз съм Петър и съм програмист
When I did as you said - I removed the extension and used AutoLink the conversion - the Ids seemed to start converting properly in Cyrillic. Thanks!


Update dependencies out of sync with latest Dart release

Fixes failing tests occurring in PRs that prevent us from merging approved PRs.

@exercism/dart  It seems the builds for this passed CI, but Github is not reflecting this change. Can you verify if you see this says "Waiting for status to be reported"?

@iHiD I closed and reopened, the build seems to have passed, but the PR hasn't updated.
This PR fixes what is causing the other PR builds to fail.


CLOUDP-93678: Update cluster watch command to use the new endpoint

Proposed changes

Jira ticket: CLOUDP-93678

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have updated e2e/E2E-TESTS.md (if a new command or e2e test has been added)
[x] I have run make fmt and formatted my code

Further comments

I am testing this command and I get {"changeStatus":"APPLIED"} even if the cluster creation process is still ongoing 🤔

I have the confirmation from Atlas team that this endpoint is not supposed to be used to report cluster creation status.
See HELP-25774


Type parameter on higher-order-component doesn't seem to work

TypeScript Version:  3.2.0-rc, 9319ea4

Search Terms:
higher order component, React, type parameter
Code
import * as React from 'react';

export interface ComponentProps {
    foo: number;
}

export default <P extends ComponentProps>(WrappedComponent: React.ComponentType<P>) => {
    return class SomeHOC extends React.Component<P, {}> {
        render() {
            return <WrappedComponent foo={0} />;
        }
    };
}

Expected behavior:
No error (as in 3.1.6)
Actual behavior:
src/wrappedComponent.tsx:10:21 - error TS2322: Type '{ foo: number; }' is not assignable to type 'IntrinsicAttributes & P & { children?: ReactNode; }'.
  Property 'foo' does not exist on type 'IntrinsicAttributes & P & { children?: ReactNode; }'.

10             return <WrappedComponent foo={0} />;

Playground Link:
Related Issues:
Possibly #28636, #28647, #28719

Got it. The actual (non-example) code was using some cached props in the state, of type ComponentProps. By making the state itself similarly generic, it works.


Unexpected token, expected ";"

Environment
GatsbyJs starter. https://github.com/danbruegge/gatsby-starter-default-extended
All configs are included.
Reproduction
Install version 1.3.2 and upgrade to 1.4.0 or 1.5.0.
Expected Behavior
No Unexpected token, expected ";" errors.

See our docs page on debugging HTML builds for help https://goo.gl/yL9lND

@danbruegge Which command did you run? It seems that lots of files are linted, besides normal js/jsx files.

I don't see any useful information here. Why are you linting your map and json files?

@chinesedfan i run yarn start which maps to gatsby develop.
@stevemao before version 1.4.0 it was linting normally. So i assume that this has something to do with stylelint-processor-styled-components. Sadly the errors are quite meaningless. So i don't know where to start finding the root problem. Thats why i asking here.

Solved in https://github.com/danbruegge/gatsby-plugin-stylelint/issues/2.

Sadly not solved. I checked this already and it is not the cause of the problem.

@danbruegge Have you changed the default config of your gatsby-plugin-stylelint, just as I commented? I can run yarn start without errors.

Ah. Checked with the wrong regex. Sorry my bad.


Reuse correct functions from trikRuntime

https://github.com/trikset/trik-studio/pull/421 -- this PR is an example of bad practice about copying code from trikRuntime.

#421 intorduces bugs, this code is fixed in trikRuntime.


Issue: Bug - timestamp-last-hit.fast not working correctly with cluster

Describe the bug
When I using (timestamp-last-hit.fast = 0) to tag rules, rules are getting tagged where one member of the cluster has no hits while the other does.  I had also noticed similar behavior when running (timestamp-last-hit.fast < - 366) for tagging rules unused in over a year.  If one cluster has > 1 year last hits, but the other has < 1 year hits the rule gets tagged.
I need to look at a lot of rules to see if it's consistent based on cluster member order or not (in other words, if it's only checking the first cluster member and not the second).  Spot checking a few rules indicates only the first cluster member is examined.
Expected behavior
When making the filter match, both cluster members should be examined to see if either matches.
Current behavior
See above.
Steps to reproduce
php -r "require_once 'C:/tools/pan/pan-os-php/utils/pan-os-php.php';" type=rule in=api://panorama.domain.com location=FW1 ruletype=security actions=tag-Add:Unused-Never 'filter=(timestamp-last-hit.fast = 0)'
or
php -r "require_once 'C:/tools/pan/pan-os-php/utils/pan-os-php.php';" type=rule in=api://panorama.domain.com location=FW1 ruletype=security actions=tag-Add:UNUSED-Year 'filter=(timestamp-last-hit.fast < - 366 days)'
Screenshots
Here's an example of the Unused-Never tagging.  This rule was tagged.

This is an example of the unused for over a year tagging:

Context
Tagging unused rules for future deletion.
Your Environment

Version used: 10.1.12
Environment name and version: PowerShell 5.1, latest PAN-OS-PHP (v2.1.22), PHP v8.3.4
Operating System and version (desktop or mobile): Windows 10

thanks for sharing this, I will look into this.
some more hints:
As you are running this against Panorama, and this feature is directly only checking Panorama log information via Panorama XML API,
the main problem is again based on Panorama.
It looks like also this Palo Alto networks feature is not well integrated into Panorama.
I assume your screenshots are from FW and not from Panorama.
If you like to search for rules which are NEVER triggered, please use:
pan-os-php type=rule in=api://panorama location={DGNAME} 'filter=(rule is.unused.fast )'

The screenshots are from Panorama.
I thought based on the other issue/discussion that unused.fast queried the firewall via Panorama?  Has the behavior of it changed?  In the distant past (PAN Configurator probably) it was only checking a counter of hit or not hit and that counter would be reset if the firewall rebooted.


logging: unify our two logging systems

This PR does a number of things to improve and unify our two logging systems.

Introduce a new set of logging macros to replace the existing ones from the log crate
Add various types as the foundation of a new, unified logging infrastructure
Implement a new logging backend based on this new infrastructure, essentially creating a hybrid between our existing text and structured logging backends
Remove the text logging backend and use the new unified one for all textual logging
Remove the sl_ logging macros and divert all structured logging through the new unified backend

Things still to come:

Lots of cleanup and removal of unused parts of the logging crate
Migrating existing users of StructuredLogEntry to Schema driven structured logging

/land


Fix enums / user model test in pass-ui

the role ENUM in pass-ui changed to be capitalized and this has broken a user model unit test
need to investigate a fix for this and whether anything needs to be changed in the adapter/serializer

Need to update the enum transform in the Ember application, which currently upper-cases all enum values


Text alignment of names in About page cards is screwy

Sometimes the name is right aligned, sometimes center aligned, sometimes right align. This should be standardized.

These all seem left aligned with quite some margin.

I want to help with this issue. Can you assign the ticket to me please?

It's all yours, Krittiya! The site now supports SCSS so feel free to run wild with that.

Hi @palewire . I changed the alignment as it is required. Below is the screenshot. Let me know if anything else needs to be done or if you have any comments.
Is it fine?

Thank you! I forked the repo and tried to run it locally but looks like it needs Ruby. What else do I need to install to be able to run it?

You can find instructions here https://github.com/jupyter/jupyter.github.io#jupyters-main-website


Delete: Seismic Survey Method

The seismic survey vocabulary is redundant and requires deletion.
Content is not included in a seismic methods collection in the survey-methods vocabulary.
However, the current vocabulary is used in the CKAN seismic form.
We will need to change the CKAN JSON first and prior to deleting this redundant vocab.
Issue to be used as a flag to complete this action as a later date.

Have requested LinkDigital to change the SPARQL query to point to the collection http://linked.data.gov.au/def/geological-observation-method/seismic-methods

SPARQL query:
PREFIX skos:http://www.w3.org/2004/02/skos/core# SELECT ?concept ?prefLabel WHERE{http://linked.data.gov.au/def/geological-observation-method/seismic-methods skos:member ?concept. ?concept skos:prefLabel ?prefLabel.}order by ?prefLabel

@KellyVance This vocab can now be deleted.
The new CKAN seismic survey form is pulling from http://linked.data.gov.au/def/geological-observation-method/seismic-methods


Add a summary section on the landing page

It would be great to have a why/overview section at the beginning of this book.

Add sections on:

Motivation, what you are trying to do
Story - cover basics, go into corrections, gridding, and examples

Explain the order


Cache issue

When the script asks for a URL it generates a .cache file with the response. The cache file is created and served perfectly, however, after 20 minutes the link breaks and instead of the script rewriting the response it continues serving the cache permanently. How to fix that?

@Yokutto in func.php, find the line 108 where you will locate this function if ($timeout >= 900) {
The variable 900 serves to be 900s equivalent to 15 minutes which means after 900 seconds, the link will change with the new fresh generated link. Probably to fix it is to change the variable approximately 30 minutes or I think more than for the viewers. Yet I kinda guess anime episodes exceed for about 24 minutes so you can test between 1500 above ;)

Even if I put the script to generate a new link every 2 seconds, it will not generate unless I go to the .cache file and delete it

@Yokutto, the .cache file doesn't change? recheck again since I've tested it and it changes tho

In line 19/36 the script generate a 1/1/1970 date with correct hour. When the minutes are less than the current time, the script resets the link. But when the minutes are bigger than the current date the script simply continues serving the cache file.
I tried in some ways to edit the code but it did not work. The original source cache system (https://github.com/p4v800m/Google-Drive-proxy-script-stream-jwplayer) works perfectly

@Yokutto hmm i'll recode the cache again.

@Yokutto script updated ;)

It's working now, thanks, your help is very appreciated.


Error in getting started code example

Not sure this project is maintained any more or not, but I hope so as it is what I was looking for.
Anyway - Stumbled at the getting started guide. There is it seems an error in the code example
// Setup the CsvParser
var parser = new CsvParser(() => new StreamReader("winequality-white.csv", separator: ';'));
Should be
// Setup the CsvParser
var parser = new CsvParser(() => new StreamReader("winequality-white.csv"), separator: ';');
I think. Just in case it helps anyone else just trying to get started.

Hi @pjsgsy
Thanks for finding the error in the code example.
My time for this project is limited at the moment. I still try to do my best answering questions in here, and help developers using the library if possible. I have some smaller plans for the project moving forward, but progress will be slow because of a lag of time sadly.
Best regards
Mads


Slicing - Enhance Home Page (employee)

Modified FlexyCard component
Implemented responsive design on Home Page as employee


Level 2 Report error when Aerial Imagery box is checked

When the aerial imagery box is checked the Level 2 report gives an error. Level 2 Reports complete when the aerial imagery box is unchecked.

Report - L2 showing an error at 87% Regional Curve Graph: Regions used are USA, Rocky Mountain System & West Interior Basin and Range
Report - L2
Parameters
stream     Rattlesnake Creek Greenough Park
flowline_fc     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\GreenoughPark_2019\flowline
xs_fc     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\GreenoughPark_2019\xs_33_100
xs_dims_fc     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\GreenoughPark_2019\xs_33_100_dims_L1_dims_L2
xs_points_1     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\GreenoughPark_2019\xs_33_100_points
xs_points_2
xs_points_3
xs_points_4
survey_name_1     2019
survey_name_2
survey_name_3
survey_name_4
dem     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\dem_hydro
banklines_fc     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\GreenoughPark_2019\banklines
features_fc     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb\GreenoughPark_2019\features
bf_estimate     103
regions     USA;'Rocky Mountain System';'West Interior Basin and Range'
label_xs     true
show_xs_map     true
profile_units     feet
aerial     false
elevation     false
xs_label_freq     10
exaggeration     10
extent_factor     1.2
output_dir     \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\Reports
output_format     word_document
Messages
Start Time: Thursday, August 24, 2023 8:17:56 AM
Setting pandoc directory...
Using pandoc: C:/Program Files/RStudio/bin/quarto
Pandoc available:  TRUE
Compare input tool parameters
[38;5;246m# A tibble: 27 x 3[39m
param_name    esri_type r_type
[3m[38;5;246m[39m[23m         [3m[38;5;246m[39m[23m     [3m[38;5;246m[39m[23m
[38;5;250m 1[39m stream        character character
[38;5;250m 2[39m flowline_fc   character character
[38;5;250m 3[39m xs_fc         character character
[38;5;250m 4[39m xs_dims_fc    character character
[38;5;250m 5[39m xs_points_1   character character
[38;5;250m 6[39m xs_points_2   NULL      NULL
[38;5;250m 7[39m xs_points_3   NULL      NULL
[38;5;250m 8[39m xs_points_4   NULL      NULL
[38;5;250m 9[39m survey_name_1 character character
[38;5;250m10[39m survey_name_2 NULL      NULL
[38;5;250m11[39m survey_name_3 NULL      NULL
[38;5;250m12[39m survey_name_4 NULL      NULL
[38;5;250m13[39m dem           character character
[38;5;250m14[39m banklines_fc  character character
[38;5;250m15[39m features_fc   character character
[38;5;250m16[39m bf_estimate   double    double
[38;5;250m17[39m regions       list      character
[38;5;250m18[39m label_xs      logical   logical
[38;5;250m19[39m show_xs_map   logical   logical
[38;5;250m20[39m profile_units character character
[38;5;250m21[39m aerial        logical   logical
[38;5;250m22[39m elevation     logical   logical
[38;5;250m23[39m xs_label_freq integer   integer
[38;5;250m24[39m exaggeration  double    double
[38;5;250m25[39m extent_factor double    double
[38;5;250m26[39m output_dir    character character
[38;5;250m27[39m output_format character character
Reading layer xs_33_100_points' from data source    \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb'
using driver OpenFileGDB' Simple feature collection with 2288 features and 11 fields Geometry type: POINT Dimension:     XYM Bounding box:  xmin: 258589.2 ymin: 301178.5 xmax: 259337.9 ymax: 301892.9 m_range:       mmin: 0 mmax: 61.6886 Projected CRS: NAD83(2011) / Montana Reading layer flowline' from data source
\\mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb'    using driver OpenFileGDB'
Simple feature collection with 1 feature and 5 fields
Geometry type: MULTILINESTRING
Dimension:     XY
Bounding box:  xmin: 258596.6 ymin: 301191.8 xmax: 259311.3 ymax: 301899.9
Projected CRS: NAD83(2011) / Montana
Reading layer xs_33_100' from data source    \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb'
using driver OpenFileGDB' Simple feature collection with 37 features and 15 fields Geometry type: MULTILINESTRING Dimension:     XY Bounding box:  xmin: 258589.2 ymin: 301178.5 xmax: 259337.9 ymax: 301892.9 Projected CRS: NAD83(2011) / Montana Reading layer xs_33_100_dims_L1_dims_L2' from data source
\\mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb'    using driver OpenFileGDB'
Simple feature collection with 37 features and 123 fields
Geometry type: MULTILINESTRING
Dimension:     XY
Bounding box:  xmin: 258589.2 ymin: 301178.5 xmax: 259337.9 ymax: 301892.9
Projected CRS: NAD83(2011) / Montana
Reading layer banklines' from data source    \mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb'
using driver OpenFileGDB' Simple feature collection with 2 features and 6 fields Geometry type: MULTILINESTRING Dimension:     XY Bounding box:  xmin: 258410.4 ymin: 300661 xmax: 259900.4 ymax: 304912.9 Projected CRS: NAD83(2011) / Montana Reading layer features' from data source
\\mvd\mvr\EGIS\Work\FluvialGeomorph\Projects\CHL_RattlesnakeCreek_MissoulaMT\RattlesnakeCreek_GreenoughPark_2019.gdb'    using driver OpenFileGDB'
Simple feature collection with 2 features and 2 fields
Geometry type: POINT
Dimension:     XYZM
Bounding box:  xmin: 258596.6 ymin: 301191.8 xmax: 259311.3 ymax: 301899.9
z_range:       zmin: 0 zmax: 0
m_range:       mmin: 1.844674e+15 mmax: 1.844674e+15
Projected CRS: NAD83(2011) / Montana
processing file: level_2_report.Rmd
|
|                                          |   0%
|
|..                                        |   4%
|
|....                                      |   9% [library]
|
|.....                                     |  13%
|
|.......                                   |  17% [output_document_type]
|
|.........                                 |  22%
|
|...........                               |  26% [set_background]
|
|.............                             |  30%
|
|...............                           |  35% [aerial_overview]
|
|................                          |  39%
|
|..................                        |  43% [longitudinal_profile_plot]
|
|....................                      |  48%
|
|......................                    |  52% [xs_metrics_plot]
|
|........................                  |  57%
|
|..........................                |  61% [wdr]
|
|...........................               |  65%
|
|.............................             |  70% [wdr_map]
|
|...............................           |  74%
|
|.................................         |  78% [wdr_plot]
|
|...................................       |  83%
|
|.....................................     |  87% [regional_curve_graph]
Quitting from lines 133-146 [regional_curve_graph] (level_2_report.Rmd)
Error in signal_abort : replacement has 0 rows, data has 1
Failed to execute (Level2Report).
Failed at Thursday, August 24, 2023 8:18:01 AM (Elapsed Time: 5.51 seconds)


Add release.yml workflow

Resolves #1416.

We discussed in the weekly community meetings that notifying from the completion of the workflow in CircleCI might be too complex. So @jorgepiloto will start by making the release procedure work from CircleCI, and if it's too cumbersome we'll port everything to GitHub Actions.

Retaking this 🚀

Debugging this. Let's see if the release workflow gets executed. It does not contain any release logic for the moment.

Just realized that I need to test this in a branch in the current repository.


Is the project still maintained? Getting errors on most functions.

I can't get an output from most functions.
For example:
MediaInfoFeed mediaInfoFeed = instagram.getMediaInfo(media_id);
Does not work (invalid media id, when it's perfectly valid)

List ud = instagram.getUserFollowList("user_id").getUserList();
System.out.println("Ended, found:  " + ud.size());

Also does not work. Returns size 0, when my follow list is around 1k.
The user_id is correct, checked against browser, API and getCurrentUserInfo() .
These and several other function do not work.
The auth is fine, I have full scope and tested things such as retrieving my own name and bio.
Is the project discontinued?

Nope. The project is actively maintained.
I would check the scope setting when you authorize the user credential (check here). For example you will need follower_list scope for your application to be able to read the followers of an authorized account.
If it is still not working for you I suggest you reply with a self-contained example that doesn't meet your expectations (also see SSCCE). I would be glad to look at it.

I'm very glad to hear this !
Then I'm probably doing something wrong, but I really don't understand what.
Here's a single-class minimal runnable example, fully commented out:
http://pastebin.com/WSTVuJVy
Thank you so much for the help.


"run the current script" button

Could a future version of PTVS add a "run the current script" button to the standard toolbar? (Or is there already something like this or a similar alternative?) Currently, I need to run a script like Debug > Execute Python File in Interactive, which is really a detour (given that Python is a dynamic language ;) ).
IDEs such as Canopy (Canopy CPython) and Spyder (Anaconda CPython) allow for faster prototyping due to their minimal button set for running scripts interactively, imho.

Oh, and if you mean a "run the current script" button in the interactive window, the main problem here is that when you aren't in the editor we don't have a concept of "current script". But you can use the $load command to do it.

Oh, and if you mean a "run the current script" button in the interactive window, the main problem here is that when you aren't in the editor we don't have a concept of "current script".

I mean in the interactive window. I didn't even thought of the alternative ("start with/out debugging") until you mentioned it. I just use Python for quick prototyping, not for final applications. So in most cases, I just use the interactive window for injecting scripts.
I understand that VS isn't aware of the "current script" (especially when multiple panels contain separate .py files inside the editor), but what about adding a Execute Python File in Interactive option underneath Start without debugging and Start with debugging upon right clicking a .py file inside the editor and/or a possible shortcut? I know this requires the same amount of clicks. The mouse distance, however, is much shorter.
There is a Send to Interactive option as well, but this requires selecting all the content of the script.
I am not a fan of $load. First, a path is required (obvious, but a detour). Second, all the content of the script is outputted in the interactive window, which isn't the case for Execute Python File in Interactive.

Thanks for the clarification. We had an issue open to track this: https://github.com/Microsoft/PTVS/issues/3347
I'll close the other one.

Also, add it to solution explorer context menu.

One other option that is available right now is to use code cells. If you start blocks of code with #%% <any text> then it will create a cell, and pressing Ctrl+Enter in it will send the entire block all at once. You'll still see the text appear in the REPL, but we also consider visible input/output part of the design.
You can have exactly one code cell for your entire file if that's what you're after, though you may find that you have some slow initialization code that you only want to run once per session and then you can add a second cell containing the part you are iterating on.


Fix the colors for the new tags when open from directory with saved file

When open from directory with saved file and input some new tags in "load-form-container", those new tags will not have colors.

Thank you for the contribution  I will review and follow up. Can you update the pull request to the dev branch?


No QQ/WeChat

Hi,
Would be happy to test and even subscribe to the app, however, not only is it only Chinese (with other apps there are ways to solve this), but QQ/WeChat ID is required. Tried for a long time but it is quite hard to get a QQ or WeChat account validated when you're not in China now.
Would it be possible to have another way to connect to the app?
Happy to chat in private to help on this.
Cheers

I regret to hear that you've had a poor user experience with our app. We're considering the possibility of adding an Email account login feature in future versions.
🎄Merry Christmas.🎁

Version 1.3.2 already supports email login.


Remove "Copied to clipboard" popup, annoying during certification exams (CKA/CKAD/CKS)

This is a Feature Request

What would you like to be added

Whenever a user copies to the clipboard using the copy button, a pop message as shown is displayed, which stays on the screen and require the user to manually click ok to close it, this is not an intended behaviour as it requires "manual effort" for not so useful task.
It becomes annoying, especially during the certification exam.
Why is this needed
To remove user interaction to manually close the clipboard message, saving time
Comments

For eg. try clicking on pods/storage/pv-claim.yaml Copy pods/storage/pv-claim.yaml to clipboard  on https://kubernetes.io/docs/tasks/configure-pod-container/configure-persistent-volume-storage#create-a-persistentvolumeclaim

** Proposed solution**
It is not required to show the pop message blocking the whole screen, either change the colour of the component for a few seconds, or do nothing at all.

/area web-development

BTW: as a work-around you can press Enter to dismiss that “copied” message.

What to do with this issue? Should we accept?
Additionally, PR is up which resolves this issue!

/triage accepted

It'd be nice to show a non-modal notification. Maybe an animation on the copy icon? Maybe show the word “copied to clipboard” but hide that hint automatically after a few seconds?

Also, "copy to clipboard" is not available everywhere

@shivanshu1333 you are very welcome to file an issue on that topic, or to submit a pull request. Please do bear in mind that the project's review criteria for any change are going to be “does this change improve the docs?” and not specifically “does this change help people who are in a Kubernetes exam?”.

Completely Agree!
It's just an addon that may help during certifications (bad issue name I know, my bad).
This is something that I noticed and I feel it isn't an intended behaviour (gives bad user experience), so I reported it.
Does it make sense to follow the same pattern across the complete documentation website? i.e. if we have a copy button let's have it everywhere or remove it from everywhere to maintain homogeneity, or we can keep it like that (no one really notices)
Do you think this needs attention and will add value to Kubernetes documentation? I know this is a tedious task, figuring out the various points where the "copy button" is missing and implementing them, but could be a good first issue.
WDYT? @sftim :)


MinMax query on Histogram plot crashes engine

Open globe.silo
Histogram plot of mesh_quality/min_edge_length
Query MinMix

The engine crashes.

related to this old one:
https://github.com/visit-dav/visit/issues/357


bug: does not work with GCS Adapter.

https://github.com/oneduo/nova-file-manager/blob/dc2abcc618e704351d0282b7f51b080b50cad3da/src/Entities/Entity.php#L173

$supportsSignedUrls = $this->manager->filesystem() instanceof \Illuminate\Filesystem\FilesystemAdapter;
This fixes the issue.


Prevent elements from intercepting scrolling while you are already scrolling the parent

Describe the feature
When you are scrolling in a page that has sub-elements within it that intercept scrolling such as sliders or plots, scrolling over those elements prevents you from continuing to scroll. We could have some sort of bool field that is set when you are scrolling on a widget that prevents scroll events from being sent to its children.
Relevant code
No response

This should also go the other way: while you are scrolling on a child element, it should not also scroll the parent.

scroll focus implemented here: e486818a5ae3a114646902229bcc5ff95605553f  -- seems good tome

The scroll focus is good, but we also need this to work for slide scrolling on mobile.


multiped-500 : dataset used for luggage detection

Hi there !
Which datasets you used for that model (multiped-500) in order to train the luggage detection ?
Many thanks in advance, your dear AI friend 😍

"Hi @bpinaya, the multiped-500 model used a few different datasets, including KITTI and some internal datasets that are unable to be released at this time. All datasets were in KITTI format however, as that's what DIGITS uses for parsing object detection ground truth data. So I recommend starting with KITTI and MS-COCO, as the KITTI converter for MS-COCO is provided. Cityscapes is a good dataset for pedestrian too, however it's not KITTI format but rather segmentation. Technically segmentation could be used for pedestrian detection also, just not providing the bounding boxes (but potentially supporting many more classes simultaneously)."
https://github.com/dusty-nv/jetson-inference/issues/132


parse definitions from Swagger

Added simple code to also parse the definitions found in swagger-files.
Sample template to output nothing but the definitions:
{{#definitions}}
interface {{name}} {{>type}}
{{/definitions}}

        var swaggerJson = JSON.parse(response);
        var typeTemplate = fs.readFileSync(__dirname + '/templates/typescript-definitions-only.mustache', 'utf-8');
        var opts = {
            swagger: swaggerJson,
            template: {
                class: typeTemplate
            }
        };
        var tsDefinitions = CodeGen.getTypescriptCode(opts);
        var defFile = fs.createWriteStream('./myDefinitions.d.ts');
        defFile.write(tsDefinitions);

The change in lib/typescript.js is for a special case I encountered with a swagger file where a parameter was defined as an array but misses the items-property describing the elements of the array. So I added this line to sanitize the input and default to a safe option instead of crashing when it tries to call 'hasOwnProperty' of undefined.

@eburi Can you provide some tests/examples?

Please merge this feature.
Thanks a lot.

@no-more @eburi Could you provide me an example on how to use this feature?

I haven't personally used it. I've just started a project and I faced this need.
I think he provided some explanation on how to use this in this post #115

I am facing the same problem, my swagger spec have several type definitions, that are not generated, and as they are passed as parameters, there are compilation errors.
This pull request solves this problem (the types are generated), but there is a new minor problem. Types that are directly a string, are translated to an incorrect interface:
interface user_id string

This causes a compilation error.

To try the code out, a template with definitions block must be specified:
{{#definitions}}
interface {{name}} {{>type}}
{{/definitions}}

I copied templates/typescript-class.mustache and put the block after the import * as request from "superagent"; (line 5). Method template and type template are not required, as they will use default template. With current code, the definition block is however not generated, but with pull request code, it is.


Pomelo.MySql and BatchUpdate issue

Hi! I have such query
                await _dataContext                     .Users                     .Where(x => x.Id == request.SellerId)                     .UpdateAsync(user => new User {Property= user.Property+ someValue}, cancellationToken);
During SQL generation I have an SQL, that listed below, but it is generated incorrect as MySQL connector throws an exception MySql.Data.MySqlClient.MySqlException (0x80004005): Unknown column 'x.amount_due' in 'field list'. It seems that BatchUpdate somehow catches the closure name x and insert it in result query. Could you please provide me what I am doing wrong?
UPDATE `user` AS A
INNER JOIN
(SELECT `x`.`Id`, `x`.`Property`, `x`.`CompanyId`, `x`.`ConcurrencyStamp`, `x`.`DrivingLicense`, `x`.`Email`, `x`.`EmailConfirmed`, `x`.`FirstName`, `x`.`IsActive`, `x`.`LastName`, `x`.`NormalizedEmail`, `x`.`NormalizedUserName`, `x`.`PasswordHash`, `x`.`PhoneNumber`, `x`.`RegisteredAt`, `x`.`SecurityStamp`, `x`.`UserName`
FROM `fifty_fifty`.`user` AS `x`
WHERE `x`.`Id` = 1)
AS B ON A.`Id` = B.`Id`
SET A.`Property` = `x`.`Property` + 5

EF Core 2.0, ASP.NET Core 2.0.1

Hello @rooonikz ,
Do you think you could provide us a test project with this issue?
It will make easier/faster for my developer to getting started for investigating it.
We now always ask for a project since we found out that most issues are missing some essential information or are resolved by the requestor when creating it
(Even if the issue seem very easy to reproduce, by getting a test project, it allow us to give a faster support and better experience for the support of all our free libraries)
Best Regards,
Jonathan

@JonathanMagnan Thanks for your response! Unfortunately, I don't have enough time this week to prepare for you test project, may be I can do it on the weekend. But I am fully open for you developer for providing additional information!
Thanks!

Hello @rooonikz ,
This issue will be closed since we haven't heard back from you.
Feel free to reopen it if you can provide the information we asked.
Best Regards,
Jonathan

Faced the same error. As I understood, no one has solved this issue?


Adding RELEASE-howto.md file, should fixes #185

Overview
should fix #185
Related Issue / Discussion
Additional Information
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[ ] I'd like to contribute [feature X|bugfix Y|docs|something else] to PyWPS. I confirm that my contributions to PyWPS will be compatible with the PyWPS license guidelines at the time of contribution.
[x] I have already previously agreed to the PyWPS Contributions and Licensing Guidelines

Coverage decreased (-0.009%) to 76.349% when pulling b3e296014e412f34a29ede3ffd63f0caf47710c6 on jachym:185_debian_versioning into 0ddbc1722491710d0997e18abeb7d2a5f0aef7d1 on geopython:master.

Coverage decreased (-0.009%) to 76.349% when pulling b3e296014e412f34a29ede3ffd63f0caf47710c6 on jachym:185_debian_versioning into 0ddbc1722491710d0997e18abeb7d2a5f0aef7d1 on geopython:master.

Coverage decreased (-0.8%) to 75.516% when pulling ec6e6ab34dfcfeabf23d1df3fa740a03dfb7b8b1 on jachym:185_debian_versioning into 8c5a9de625601d676415b349c0999c817134ae57 on geopython:master.

@tomkralidis updated, will ask PSC members to have a look at it

@ldesousa @tomkralidis  @jorgejesus I have updated the RELEASE-howto.md file according to @ldesousa proposal:
no branches for new release, no tickets, just straight-forward development. is that ok ? @jonas-eberle is the last member of PSC who did not review?

@jachym As I have no experience what is a good approach for software versioning based on git, I am fine with the proposal of @ldesousa and your updated RELEASE-howto.md.


Encourage versioned compatibility check via __riscv_v_intrinsic >= 1000000

The document currently suggests checking for RISC-V Vector intrinsics support via #ifdef __riscv_v_intrinsic.
This shouldn't be recommended, as earlier compiler versions of both gcc and clang that implement draft of the specs v0.11 and v0.12 also define __riscv_v_intrinsics, and don't implement all features of the 1.0 spec: https://godbolt.org/z/r6aW7hE3o
Instead, checking for v1.0 compliance should be encouraged, e.g. with #if __riscv_v_intrinsic >= 1000000, or if the code is otherwise compatible via explicitly checking support for the supported draft spec version or higher, e.g. #if __riscv_v_intrinsic >= 12000.
BTW: I noticed that I'm included in "doc/preface.adoc" as "Camel Coder", it would be cool if that can be adjusted to "Olaf Bernstein"

Thanks :)
Also fixed your name by https://github.com/riscv-non-isa/rvv-intrinsic-doc/commit/1e9f86b9f5110f11664d0de6381e5baf68f0d7d2 :P


How to use Activation Quantization?

What I've done: I have used your main function to get weight-only quantized versions of LLMs (from local LLM safetensors to local LLM safetensors) and then use the quantized versions to evaluate on lm-evaluation-harness, which also takes an local path as the model. All works well and thanks for your repo!
What I need help with: I am not very sure how to correctly use your functions to use a activation-quantized version of LLMs.
What I've tried:

directly use your main function to store activation-quantized version (this should not work because activation quantization should happen in run-time?)
manually change the evaluation code s.t. when the model is first loaded, I replace it with your quantized_model(model, args) with kv_bit=16 and a_bit=4; this fails because there the models don't have the named_modules that you use in your code

May I get help with this issue? thx in advance!~

Hi!
Thanks for your question. Unlike weight-only quantization that only needs to replace the weight data with its quantized counterpart (without modification of the model architecture), weight&activation (WA) quantization has to replace the whole linear module (i.e. nn.Linear) with our customized one (i.e. WALinear), as the quantization of activation should be performed on the input or output data. Therefore, it's not supported to directly load a WA quantized model with standard AutoModelForCausalLM.from_pretrained() function, which would utilize the standard modeling_XXX.py (XXX is the name of a certain model, e.g. llama) file to define the model architecture, while the standard modeling_XXX.py doesn't have a WALinear module.
So, if you want to directly load a WA quantized model from a local checkpoint, a customized modeling_XXX.py that explicitly defines the WALinear module in place of nn.Linear must be used. Unfortunately, we doesn't have a customized modeling_XXX.py now and the WALinear is dynamically added to the model during the execution of quantize_model(). Apologize for the inconvenience and we are considering to add a customized modeling file in the future, please stay tuned.
To correctly use WA quantization in your code, simply load the original full-precision model and use model = quantize_model(model, args) to get everything ready, then you can use the model for inference with weight and activation quantized!


Update lightwalletd fork in zebra book docs

https://zebra.zfnd.org/user/lightwalletd.html
We switched from adityapk00 lightwalletd to ecc in here.
We may need to change the documentation to point to that repo instead, i suppose we can keep using the zecwallet-light-cli  but we might want to try the documentation steps and see if the whole process remains working with the different lightwalletd version.
I am not doing this change yet as we are unsure that the switched version is currently working for us as we are having CI issues that could be related to the switch.

Also https://github.com/ZcashFoundation/zebra/blob/main/zebrad/tests/acceptance.rs#L58-L61
A full search for adityapk00 in Zebra should be done and decide what to change on each case if needed.


cplp -1022 - remove forbidden for no status update.

remove forbidden for no status update.

@Akshay-Chaudhari-BMW  I can see that you removed the exception. However doesn't this result now still in an issue if the invitation status is PENDING, ACCEPTED and DECLINED

@Akshay-Chaudhari-BMW I can see that you removed the exception. However doesn't this result now still in an issue if the invitation status is PENDING, ACCEPTED and DECLINED

@jjeroch If there is no status update then response will be empty body.

As just discussed we update it as following "if status is "2" (PENDING) we will update the invitation status to "3" (ACCEPTED); if status is "1", "3" or "4" we send back an empty body"

Invitation status update done


Editor: GridContainer resizes children controls, does not allow clicking on them to adjust size

Godot version:
#4c1a5d9
Mac OS X, High Sierra
Issue description:
Create a grid container with 2 columns and two buttons. The child buttons are not accessible from the GUI for selection, and manipulation. Only from inspector and also it is manual.
Also, if you manually adjust the size of the child buttons from inspector, and then resize the grid container, all the children are resized back.

GridContainer is a layout node. You cannot resize its child nodes. You can only use the size flags.

This is a symptom of a larger problem, which is that Control nodes are not WYSIWYG and they allow you to make invalid changes. I can't seem to find it, but I think there's another open issue about this.

Yes, this is expected behavior. Children of containers are not meant to be moved. See #22877 for the valid issue.


Problem Compiling Windows Dependencies

Hello
I am receiving the following when attempting to compile the windows dependencies:
Joelseph@CopenmanHP /c/dependencies
$ compile-dependencies.sh
Microsoft Windows [Version 10.0.10240]
(c) 2015 Microsoft Corporation. All rights reserved.
c:\Dependencies\qttools-opensource-src-5.4.2>  set INCLUDE=c:\Dependencies\.\libpng\libpng-1.6.18\;c:\Dependencies\.\openssl\openssl-1.0.1m\\include
c:\Dependencies\qttools-opensource-src-5.4.2>  set LIB=c:\Dependencies\.\libpng\libpng-1.6.18\\.libs;c:\Dependencies\.\openssl\openssl-1.0.1m\\include
c:\Dependencies\qttools-opensource-src-5.4.2>  set PATH=%PATH%;c:\Dependencies\.\qtbase-opensource-src-5.4.2\\bin
c:\Dependencies\qttools-opensource-src-5.4.2>  qmake -makefile qttools.pro
Qt: Untested Windows version 10.0 detected!
c:\Dependencies\qttools-opensource-src-5.4.2>  mingw32-make
cd src/ && ( test -e Makefile || c:/Dependencies/qtbase-opensource-src-5.4.2/bin/qmake.exe C:/Dependencies/qttools-opensource-src-5.4.2/src/src.pro -o Makefile ) && mingw32-make -f Makefile
mingw32-make[1]: Entering directory 'c:/Dependencies/qttools-opensource-src-5.4.2/src'
cd linguist/ && ( test -e Makefile || c:/Dependencies/qtbase-opensource-src-5.4.2/bin/qmake.exe C:/Dependencies/qttools-opensource-src-5.4.2/src/linguist/linguist.pro -o Makefile ) && c:/Dependencies/mingw32/bin/mingw32-make -f Makefile
mingw32-make[2]: Entering directory 'c:/Dependencies/qttools-opensource-src-5.4.2/src/linguist'
cd lrelease/ && ( test -e Makefile || c:/Dependencies/qtbase-opensource-src-5.4.2/bin/qmake.exe C:/Dependencies/qttools-opensource-src-5.4.2/src/linguist/lrelease/lrelease.pro -o Makefile ) && c:/Dependencies/mingw32/bin/mingw32-make -f Makefile
Qt: Untested Windows version 10.0 detected!
Project ERROR: Unknown module(s) in QT: core-private xml
Makefile:40: recipe for target 'sub-lrelease-make_first' failed
mingw32-make[2]: *** [sub-lrelease-make_first] Error 3
mingw32-make[2]: Leaving directory 'c:/Dependencies/qttools-opensource-src-5.4.2/src/linguist'
Makefile:40: recipe for target 'sub-linguist-make_first' failed
mingw32-make[1]: *** [sub-linguist-make_first] Error 2
mingw32-make[1]: Leaving directory 'c:/Dependencies/qttools-opensource-src-5.4.2/src'
makefile:40: recipe for target 'sub-src-make_first' failed
mingw32-make: *** [sub-src-make_first] Error 2
On the following system:
OS Name	Microsoft Windows 10 Home
Version	10.0.10240 Build 10240
Other OS Description 	Not Available
OS Manufacturer	Microsoft Corporation
System Name	COPENMANHP
System Manufacturer	Hewlett-Packard
System Model	HP ENVY m7 Notebook PC
System Type	x64-based PC
System SKU	J9K05UA#ABA
Processor	Intel(R) Core(TM) i7-5500U CPU @ 2.40GHz, 2401 Mhz, 2 Core(s), 4 Logical Processor(s)
BIOS Version/Date	Insyde F.51, 7/20/2015
SMBIOS Version	2.7
Embedded Controller Version	78.33
BIOS Mode	UEFI
BaseBoard Manufacturer	Hewlett-Packard
BaseBoard Model	Not Available
BaseBoard Name	Base Board
Platform Role	Mobile
Secure Boot State	On
PCR7 Configuration	Binding Not Possible
Windows Directory	C:\WINDOWS
System Directory	C:\WINDOWS\system32
Boot Device	\Device\HarddiskVolume2
Locale	United States
Hardware Abstraction Layer	Version = "10.0.10240.16392"
User Name	CopenmanHP\Joelseph
Time Zone	Pacific Daylight Time
Installed Physical Memory (RAM)	12.0 GB
Total Physical Memory	11.9 GB
Available Physical Memory	5.98 GB
Total Virtual Memory	15.0 GB
Available Virtual Memory	7.58 GB
Page File Space	3.10 GB
Page File	C:\pagefile.sys
Hyper-V - VM Monitor Mode Extensions	Yes
Hyper-V - Second Level Address Translation Extensions	Yes
Hyper-V - Virtualization Enabled in Firmware	No
Hyper-V - Data Execution Protection	Yes

Qt: Untested Windows version 10.0 detected!
Project ERROR: Unknown module(s) in QT: core-private xml

Apparently your Qt installation is missing two required Qt modules.
Closing as not-a-bug.

lol and those would be obtained how?  As i followed the windows compile doc https://github.com/NeuCoin/neucoin/blob/master/doc/compile-windows.md and downloaded qtbase-opensource-src-5.4.2 as it instructs so this and is now about the 5th thing that I've had to fix in the compile-dependencies.sh is it perhaps only meant to be compiled on 32b systems?  Also this is a bug in the documentation for performing the windows compile, if the compile only works with one architecture that would be a bug correct

Our builds have all been made on a 64bits architecture (a fresh unmodified Windows 8.1), so that's unlikely to be the issue.

But I'm also using Windows 10 could that be an issue?

Well, Qt is telling you the following:

Qt: Untested Windows version 10.0 detected!

So yep, I guess it might have something to do with this :) As far as I know, Qt 5.5 should have the Windows 10 support, but we haven't checked this. The official way to build the software is to use a virtual machine (so that nothing tampers the build, and the build doesn't tamper your own environment), so you could just as well use a Windows 8.1 machine.

Ok thanks
On Wed, Sep 30, 2015 at 3:03 AM, Maël Nison<EMAIL_ADDRESS>wrote:

Well, Qt is telling you the following:
Qt: Untested Windows version 10.0 detected!
So yep, I guess it might have something to do with this :) As far as I
know, Qt 5.5 should have the Windows 10 support, but we haven't checked
this. The official way to build the software is to use a virtual machine
(so that nothing tampers the build, and the build doesn't tamper your own
environment), so you could just as well use a Windows 8.1 machine.
—
Reply to this email directly or view it on GitHub
https://github.com/NeuCoin/neucoin/issues/18#issuecomment-144345103.

                1/14: $20 to you!

                Hi there,
                Joelseph wants to let you in on Schoola, and we think you’re going to love it. Gently used clothes at up to 90% off. $2 of every $5 goes to save subjects like art and music in our classrooms. And $20 free to get you started.

                A note from Joelseph:

                    I was on Schoola and thought you might like it. Amazing savings on quality clothes and 40% funds school programs for our kids. Use my link and get $20 free!

				 Shop with Joelseph’s special link, and you’ll get an instant $20 credit. Make a purchase, and Joelseph gets a $20 credit of her own. It pays to share, right?!

				 These credits expire after 60 days, so don’t dilly dally.

                                SHOP NOW AND GET $20

                <3 your new pals at Schoola

                    To ensure delivery to your inbox, add<EMAIL_ADDRESS>to your
                        address book.
                    © 2016 Schoola, 601 Montgomery Street, Suite 675  San Francisco, CA 94111This message was sent to<EMAIL_ADDRESS>                    Thank you for reading. We really appreciate it! :)

Click here to unsubscribe.


Fix /train endpoint (and potentially other server endpoints) for new data format

With the switch to YAML for 2.0, we forgot to update the /model/train endpoint in the server to be able to handle that, and so the tests are failing/hanging in the PR #6352
It still writes the files to markdown files, that should be yaml now and also the concept of nlu vs stories files doesn't exist anymore: https://github.com/RasaHQ/rasa/blob/master/rasa/server.py#L746
I haven't checked if there's other endpoints that incorrectly handle that, so we may have to address that too

I'd add another key rules here and change everything to yaml. We can also check the the header and change to md if it says markdown. If we decide to drop md support it's easy too remove (and not a lot of engineering overhead from our side now).
@akelad You mentioned

and yes we can probably change to yaml but also the concept of stories vs nlu files doesn’t exist anymore

Is the additional rules key solution fine? I wouldn't merge the rules and stories keys into a core key because they are still somewhat separated in yaml (stories:  vs rules: )

@wochinge i would avoid using the term "core" - why can't we just pass it all into one training_data_files key for example?

passing in rules: and stories doesn't make sense to me since you can have those in one file

passing in rules: and stories doesn't make sense to me since you can have those in one file

True. I assumed that when using the API (especially from Rasa X), we'd have everything separate anyway. Putting all in one ( core (stories+rules), nlu, domain) makes more sense when thinking from a disk perspective 👍
On the other hand: If you think of YAML as JSON then it's already separated by keys now. As our training data is YAML now it also makes sense to support it's JSON representation, no?
In addition we shouldn't simply change the API endpoint without at least support the old separated way as deprecated option. How about doing both:

Having everything as giant string in a training_data key (I slightly prefer that to training_data_files)
Having everything split by rules, stories and so on (maybe print a deprecation warning then)

We can add support for full JSON payloads later.

sure, whichever works, as long as we make sure that if people run a rasa server without Rasa X, they can send a request to the /train endpoint without a huge hassle :D

What is the benefit of introducing the training_data key?
I am not sure I fully understand what you are proposing @wochinge, I think an example would help a lot.
I think the goal should be:

support the existing format at least for the old training data format
add an endpoint that supports submitting yaml only training data + domain + config

if possible:

support the new training data format with the old api

I am not sure I fully understand what you are proposing @wochinge, I think an example would help a lot.

If I understood Akela correctly in this comment then the motivation is that - as all training data can be in one single file from with YAML - there is only one key for all training data. The request payload would look like as follows:
{
   "training_data": "... yaml string containing nlu, core (stories, rules)",
   "domain": "domain as string"
}

In addition I'd support the old way and distinguish MD / YAML based on the HTTP header (YAML as default):
{
  "nlu": "nlu data string",
  "stories": "stories as string",
  "responses": "responses as string",
  "rules": "rules as string",
  "domain": "domain as string"
}

Supporting they way you propose (see snippet below) makes a lot of sense, but I'd do it as a separate PR as it also changes the current behavior of the API and we don't need it atm (to be fair the same could be said about example 1). In that case I'd do it nevertheless as it improves the UX and Akela stated "as long as we make sure that if people run a rasa server without Rasa X, they can send a request to the /train endpoint without a huge hassle :D")
{
  "nlu": [{"intent": greet", examples: [...]}]
}

If I understood Akela correctly in this comment then the motivation is that - as all training data can be in one single file from with YAML - there is only one key for all training data. The request payload would look like as follows:

I don't see a reason to split training data and domain there, I'd rather go for the payload being just yaml (without a json wrapper) since the information is already split in yaml using different toplevel keys.

In addition I'd support the old way and distinguish MD / YAML based on the HTTP header (YAML as default):

This will break compatibility, so needs proper explanation in migration guide.

I don't see a reason to split training data and domain there, I'd rather go for the payload being just yaml (without a json wrapper)

I understand that part, but what do you mean by "since the information is already split in yaml using different toplevel keys."? It's either a big string for each key or we make everything JSON, no?

This will break compatibility, so needs proper explanation in migration guide (and probably a pointer to it in the exception that we will throw when we are trying to parse md as yaml when a user didn't send the header).

We can also keep MD as default. Imo this depends how we move forward regarding keeping / dropping markdown support.

wait, this doesn't make sense:
if we use json, e.g.
{
  "nlu": "nlu data string",
  "stories": "stories as string",
  "responses": "responses as string",
  "rules": "rules as string",
  "domain": "domain as string"
}

the HTTP content type header should be set to JSON (and not to markdown or YAML). I think it should work to keep MD the default here.
To support YAML, we can use the content type header, set it to YAML and avoid the json wrapper, e.g. we'd post something like this to the endpoint:
intents:
- greet

stories:
- ...

responses:
- ...

Got what you mean know 👍 🙈
Will implement it later today or tomorrow 👍

sounds great 💯  let me know how things go so I can update the multimedia responses PR

Other endpoints:

retrieve_story should be changed when story writer is done (TODO: Create issue)
evaluate_stories works with new payloads

@tmbo The PR is now in review.

@tmbo This is merged btw.

@wochinge It looks like the example payload we provide on our docs page doesn't match the correct YAML spec for the endpoint.
Is application/json only compatible with markdown at the moment?

Is application/json only compatible with markdown at the moment?

Yes!
You need to use application/x-yaml for the yaml payload


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
97bdd73 -> d63f9de

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.42.4 -> v1.42.5

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.42.5
Compare Source
Bug Fixes

deps: update gomod (#903) (15a5b80)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

12 additional dependencies were updated

Details:

Package
Change

github.com/IBM/go-sdk-core/v5
v5.18.1 -> v5.18.3

github.com/go-logr/logr
v1.3.0 -> v1.4.2

github.com/hashicorp/hcl/v2
v2.17.0 -> v2.22.0

github.com/hashicorp/terraform-json
v0.23.0 -> v0.24.0

github.com/tmccombs/hcl2json
v0.5.0 -> v0.6.4

github.com/zclconf/go-cty
v1.15.0 -> v1.15.1

go.opentelemetry.io/otel
v1.16.0 -> v1.29.0

go.opentelemetry.io/otel/metric
v1.16.0 -> v1.29.0

go.opentelemetry.io/otel/trace
v1.16.0 -> v1.29.0

golang.org/x/crypto
v0.30.0 -> v0.31.0

golang.org/x/mod
v0.17.0 -> v0.18.0

golang.org/x/tools
v0.21.1-0.20240508182429-e35e4ccd0d2d -> v0.22.0

/run pipeline

/run pipeline


Terminals do not recognize Pneumaticcraft items that have air

This is more of a compatibility issue, but if a Pneumaticcraft item has air pressure, the terminals do not recognize them.

What do you mean by "recognize"? Are they not displayed in the terminal at all?

Sorry, I should be more clear. If an item has any air stored inside of it, it can't be pulled out by the item terminal. It's visible in the terminal like any item, but when you try to extract it the terminal tells you the item can't be found. This is consistent across all Pneumaticcraft items that can store air.


Update README.md fix engine to r6 path

The path was changed from engine\config\cybercmd to r6\config\cybercmd as of v0.0.4

👍 looks good, thanks for the PR


quote  boost

From @damon @technicat will you add quote to the boost button?

https://github.com/mastodon/mastodon/issues/309#issuecomment-469260847


Crash Report: Reported element LatexCommandsImpl(COMMANDS)[STUB]{\mathbb} is not from the file 'LaTeX source file'

Description
the inspection 'LatexMissingImport' (class nl.rubensten.texifyidea.inspections.latex.LatexMissingImportInspection) was invoked for. Message: 'Command requires amssymb package'.Element' containing file: LaTeX source fileInspection invoked for file: LaTeX source file [Plugin: nl.rubensten.texifyidea]
Stacktrace
Please paste the full stacktrace from the IDEA error popup

Problem was the usage of commandsInFileSet() instead of commandsInFile() consequently targetting commands that are in a different file than the file that is being edited. Fixed now.


feature/favor-state-node

resolves #5016

Description

Add --favor-state option which enables --defer to favor using --state node even if node exists in current target.
This optionally removes the below second --defer criterion when using the --favor-state flag.
This branch has been created and rebased against an older branch so that the owner of the branch has signed the CLA.
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] This PR includes tests, or tests are not required/relevant for this PR
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Will squash all commits once all comments have been resolved :)

What are you overriding '_get_deferred_manifest' and 'defer_to_manifest' in dbt.task.run? Those methods were in the compile task because the selectors apply to a lot of commands besides run.

I've removed these functions from the dbt.run.task file as they exist in the compile file.

Why are you overriding '_get_deferred_manifest' and 'defer_to_manifest' in dbt.task.run? Those methods were in the compile task because the selectors apply to a lot of commands besides run.

I've removed these functions from the dbt.task.run file as they exist in the compile file. Thanks for the spot.

Do I need to do anything on this @jtcohen6, appreciate Coalesce likely has you all tied up but want to make sure you are not waiting on me?

Closing and re-opening to trigger github actions which seem to have stalled.

Thanks @stu-k ! Once the tests pass I will squash the commits


Translating Python Dictionaries into internal document format directly

For Document collections a list[dict] can be assigned to some keys. Those should be converted directly into internal format (often MsgPack) in the C++ layer.

To simplify the procedure, we should create 2 functions:

nlohmann::json -> PyObject *
PyObject * -> nlohmann::json