OrientDB 2.0.12 - LinkMap convertLink2Record sets value to null

Hi,
I have a data structure similar to the one described in Time Series Use Case.
When the linkmap contains the following:

As you can see, the linkedmap contains 3 persistent records and 1 temporary record that will be persisted soon after (when calling tx.commit())
The three records when inside the method convertLink2Record are transformed into NULL values ( rid.getRecord() at line 208 of the class ORecordLazyMap returns null, even if  record is actually present) as shown below:

The temporary record is not touched because is not instance of ORID.
If I call reload on each record inside the corresponding HashMap before this point, everything is fine and the rid.getRecord() returns the real record instance. However, this software needs to run in an embedded hardware, and due to the fact that I'm using linkedmaps quite a lot, I'd really like avoiding calling this method for each record of each linkedmap.
If I don't call the reload method, this is the exception I get
14:06:40,967 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.client.remote.OStorageRemote.commitEntry(OStorageRemote.java:2004)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.client.remote.OStorageRemote.commit(OStorageRemote.java:1110)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.client.remote.OStorageRemoteThread.commit(OStorageRemoteThread.java:445)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.tx.OTransactionOptimistic.doCommit(OTransactionOptimistic.java:475)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.tx.OTransactionOptimistic.commit(OTransactionOptimistic.java:147)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.commit(ODatabaseDocumentTx.java:2433)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.commit(ODatabaseDocumentTx.java:2403)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at org.ceia.mdna.collector.processor.EventProcessor.run(EventProcessor.java:230)
14:06:40,968 ERROR [stderr] (pool-11-thread-1) 	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
14:06:40,969 ERROR [stderr] (pool-11-thread-1) 	at java.util.concurrent.FutureTask.runAndReset(FutureTask.java:308)
14:06:40,969 ERROR [stderr] (pool-11-thread-1) 	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.access$301(ScheduledThreadPoolExecutor.java:180)
14:06:40,969 ERROR [stderr] (pool-11-thread-1) 	at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:294)
14:06:40,969 ERROR [stderr] (pool-11-thread-1) 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
14:06:40,969 ERROR [stderr] (pool-11-thread-1) 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
14:06:40,969 ERROR [stderr] (pool-11-thread-1) 	at java.lang.Thread.run(Thread.java:745)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) Caused by: com.orientechnologies.orient.core.exception.OValidationException: The field 'AbstractTreeLoc.dev' has been declared as LINKMAP but contains a null record (probably a deleted record?)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.validateLink(ODocument.java:460)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.validateLinkCollection(ODocument.java:441)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.validateField(ODocument.java:293)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.validate(ODocument.java:1953)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(ODatabaseDocumentTx.java:2255)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx.save(ODatabaseDocumentTx.java:118)
14:06:40,970 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.save(ODocument.java:1727)
14:06:40,971 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.save(ODocument.java:1718)
14:06:40,971 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.save(ODocument.java:76)
14:06:40,971 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.serialization.serializer.record.binary.ORecordSerializerBinaryV0.recursiveLinkSave(ORecordSerializerBinaryV0.java:674)
14:06:40,971 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.serialization.serializer.record.binary.ORecordSerializerBinaryV0.writeOptimizedLink(ORecordSerializerBinaryV0.java:688)
14:06:40,972 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.serialization.serializer.record.binary.ORecordSerializerBinaryV0.writeLinkMap(ORecordSerializerBinaryV0.java:616)
14:06:40,972 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.serialization.serializer.record.binary.ORecordSerializerBinaryV0.writeSingleValue(ORecordSerializerBinaryV0.java:575)
14:06:40,972 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.serialization.serializer.record.binary.ORecordSerializerBinaryV0.serialize(ORecordSerializerBinaryV0.java:261)
14:06:40,972 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.serialization.serializer.record.binary.ORecordSerializerBinary.toStream(ORecordSerializerBinary.java:104)
14:06:40,972 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.toStream(ODocument.java:2100)
14:06:40,972 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.core.record.impl.ODocument.toStream(ODocument.java:695)
14:06:40,973 ERROR [stderr] (pool-11-thread-1) 	at com.orientechnologies.orient.client.remote.OStorageRemote.commitEntry(OStorageRemote.java:1998)
14:06:40,973 ERROR [stderr] (pool-11-thread-1) 	... 14 more```

I'm I doing something by mistake?
Do you have a patch for this or some workaround?

Thanks

Regards
Guglielmo Moretti

Hi,
I found the problem,
I've two different threads that are using two different connections to the same database.
The first thread persists a vertex in the db and marks it as "not processed"
The second thread asynchronously iterates over all the "not processed" vertices directly by looking them up from the db (always using the java API) and inserts the vertex inside the TimeSerie tree.
This causes the exception above.
I've a simple test case that shows that if I run the described procedure from a single thread, no exception is raised.
As soon as I run the "vertex creation" in, let's say Thread-1, and the tree-insertion in Thread-2, using the same OrientVertex instance, the exception is raised at commit time.
Is this the expected behaviour?
Is it something related to Caching?
This procedure has been tried with OrientDB 2.0.11, 2.0.12 and the develop branch compiled yesterday.
Regards
Guglielmo

The fact the records are null means records don't exist in database. Could you check it? If you want to disable such auto conversion, call ORecordLazyMap .setAutoConvertToRecord(false)

Are you sharing the same OrientVertex instances across threads?

Luca,
I'm not sharing the OrientVertex instance, one thread saves the vertex, the other calls getVerticesOfClass() and iterates through them.
Regarding the existence of the record, if I select the same RID that gets translated to null, then I get valid data.
However, by setting ORecordLazyMap.setAutoConvertToRecord(false) seems to fix this.
Can it be dangerous is any way?
Thanks
Guglielmo

It's not dangerous, it's just you have to manual call rid.getRecord()


Add support for LodDataInput shader options

So far this only extends to allowing for LodDataInput to exist which
support geometry shaders, but this could be extended further to support
compute shaders (for example), or even static textures directly (through
simplified interface). For now making this limited change in order to
fix a bug due to the switch to sea floor depth geometry shader support.

This should fix #310.
TBH the more I think about it, the more I'm personally inclined and willing to drop geometry shader support completely. but I think this particular increase in complexity is worth it anyway as it could be helpful for supporting further input types down the road like compute shaders which could allow for things like single-dispatch support there.

I get a very interesting and profound image when i add a sphere and register it for depth:

Guessing the transforms are wrong?

Jacob Frye on Discord reported that this branch solves their problem, which is positive.

After a conversation with @huwb, we have decided that the extra complexity incurred by geometry shader isn't worth it, so we are going to remove it completely for now. Will create a new PR for this in a bit.


Please add dark mode support

I'd like to learn from your code implementation for dark mode. It would be a great learning experience for me.

Are you asking for dark mode support to be added to elements like the sidebar or menu?

Yes, I'd like that very much.

Sure I will try. Just keep in mind that I'm still learning so my code isn't perfect 😅


Compile Error!

When run the 'make' command, two errors occurred. How can i solve this problem?
Error 1: src/convGemm/src/gemm_blis.c:33:68: error: ‘l3ukr_t’ undeclared (first use in this function); did you mean ‘ukr_t’?
Error 2: /src/convGemm/src/gemm_blis.c:33:76: error: expected ‘)’ before ‘BLIS_GEMM’
blis_gemm_kernel = bli_cntx_get_l3_nat_ukr_dt(BLIS_FLOAT, (l3ukr_t) BLIS_GEMM, blis_cntx)

A new version has been uploaded that fixes this problem with the master version of BLIS. You should update also the convGemm submodule with 'git submodule update --remote --merge'


Xnero: Watch gasgrills.bravesites.com

Xnero requests the watch of the watch_keyword gasgrills\.bravesites\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

gasgrills\.bravesites\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


build(dependencies): 🧱 update crate

Description
Update dependencies and dprint config.
Bumps clap from 4.5.15 to 4.5.16.
Bumps reqwest from 0.12.5 to 0.12.6.
Bumps serde from 1.0.207 to 1.0.208.
Bumps tokio from 1.39.2 to 1.39.3.
Updating arrayvec v0.7.4 -> v0.7.6
Removing bitflags v1.3.2
Updating cc v1.1.10 -> v1.1.13
Updating clap v4.5.15 -> v4.5.16
Updating h2 v0.4.5 -> v0.4.6
Updating indexmap v2.3.0 -> v2.4.0
Updating libc v0.2.155 -> v0.2.158
Updating reqwest v0.12.5 -> v0.12.7
Updating serde v1.0.207 -> v1.0.208
Updating serde_derive v1.0.207 -> v1.0.208
Updating serde_json v1.0.124 -> v1.0.125
Adding shlex v1.3.0
Updating syn v2.0.74 -> v2.0.75
Updating system-configuration v0.5.1 -> v0.6.0
Updating system-configuration-sys v0.5.0 -> v0.6.0
Updating tokio v1.39.2 -> v1.39.3
Updating tower-layer v0.3.2 -> v0.3.3
Updating tower-service v0.3.2 -> v0.3.3
Adding windows-registry v0.2.0
Adding windows-result v0.2.0
Adding windows-strings v0.1.0
Removing windows-sys v0.48.0
Removing windows-targets v0.48.5
Removing windows_aarch64_gnullvm v0.48.5
Removing windows_aarch64_msvc v0.48.5
Removing windows_i686_gnu v0.48.5
Removing windows_i686_msvc v0.48.5
Removing windows_x86_64_gnu v0.48.5
Removing windows_x86_64_gnullvm v0.48.5
Removing windows_x86_64_msvc v0.48.5
Removing winreg v0.52.0
Type of change

[X] Dependency update
[X] CI

How Has This Been Tested?

[X] cargo test run with all tests passing

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
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[X] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[X] Any dependent changes have been merged and published in downstream modules

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 35.69%. Comparing base (abe47c1) to head (c5650c1).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage   35.69%   35.69%
=======================================
  Files           4        4
  Lines        1188     1188
=======================================
  Hits          424      424
  Misses        764      764

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


Introduce template function required_env

The required_env function allows you to declare a particular environment variable as required for template rendering.
If the environment variable is unset or empty, the template rendering will fail with an error message.
Example:
releases:
  - name: {{ required_env "NAME" }}-vault
    namespace: {{ required_env "NAME" }}
    chart: roboll/vault-secret-manager
    set:
      - name: proxy.domain
        value: {{ required_env "PLATFORM_ID" }}.my-domain.com

@sstarcher Hey. I guess we intentionally changed the env's behavior, not to fail when the env var isn't set in https://github.com/roboll/helmfile/commit/71fa693997ed56108272219c34fad0427f8bdc97#diff-f6be34387a141f652301c16cf7a5e36cL92?
How about merging this PR with changing the proposed func name to env, to override the one provided by sprig?

I think it's good to have both. One for optional envs and one for required. What do you think?

Any reason to not do what helm does and have a generic require function?

@sstarcher That's a good point! But the answer is yes for me at the moment.
helm doesn't actually have env, so I believe we don't have a momentum to conform helmfile to helm, in this specific case.
If we had introduced an alternative of env, something like a values file for helmfile itself as per #59 , it would make sense for me to do something like {{ required "Bar is required!" .HelmfileValues.Foo.Bar }} in your helmfile.yml.
However, for now, {{ required "Bar is required!" ( env "BAR" ) }} as you might have suggested, seems just unintuitive compared to {{ env "BAR" }}(w/ previous behavior of failing on missing value) or even {{ required_env "BAR" }}.
WDYT?

This was my first thought on implementing a required func like from helm. But i don't like the fact, that i have to define an error message for each required.

I would rather go with required_env so we don't deviate from helms default usage of env.  It might make sense to see if sprig would take an addition so we could get it both into helm and sprig, but they may not be interested.

@AndreasBieber @sstarcher Thanks both for the feedbacks!
@sstarcher Maybe I'm missing something but AFAIK helm doesn't allow env at all, not to leak tiller env vars to helm templates. So we don't need to worry about the deviation in this specific case?

I would prefer required_env because otherwise strigs env would have another behavior than in helmfile. But if you want to stick with env than i'll change it.

@AndreasBieber Thanks. That's also good point! I take your feedback as an user's  - which should be considered very carefully.
For me, maybe it is just that:

I still doubt if users read the doc for sprig's own env when helmfile provided env itself.
The env without the validation sounds just confusing. Why do we need it? Can't we just introduce required_env only in README.md?

That being said, if you think required_env is the way to go as an user's perspective, it will address 1. above - I like it!
In that case, regarding the 2. above, how about and would you mind:

Renaming required_env to requiredEnv according to sprig's function naming of lowerCamelCase
Replacing every occurrence of env to requiredEnv you introduce, in README.md

Then, it makes sense from every aspect I can think of.

Being able to make some env vars optional and specify default values for them while making others as required seems very useful to me.
Keeping sprig's env, combined with default and adding a new requiredEnv would allow this nicely. For example:
releases:
- name: myservice
  chart: somewhere/myservice
  set:
  - name: scheme
    value: {{ env "MYSERVICE_SCHEME" | default "https" }}
  - name: host
    value: {{ requiredEnv "MYSERVICE_HOST" }}

Replacing all the env instances in the readme with requiredEnv sounds like a good idea, but why not also include a sample for the env + default combo?

@bastianeicher I'm onboard with your proposal for requiredEnv, env and a env+default combo example.

Thanks everyone for all your support and the awesome work! Everything LGTM now 👍


Use standard apache logging format in error log

All credits to @arminabf  - see https://github.com/owasp-modsecurity/ModSecurity/pull/1997
Use the server context, like in all other places to use standard httpd format

Closed by mistake


first draft

I like the concept!
I am afraid some of it might become outdated with the code changes. I believe code structure should be in another place (contributing.md or architecture reference), and we should simply point to it. WDYT?
WDYT about mentioning the rate of changes (and their releasing) and our mentality? I think mentioning "feature complete" is something important, and the rest is janitorial and/or slight inprovements. This is useful as we don't want someone proposing a complete change in the main repo, but rather small improvements, while we are open to discuss larger mindset changes for the same topic which could land in the same org and different repo.

I agree, I think we can elaborate a bit more about the rate of changes and their size in the last months.
"Feature complete" is a phrase, we should be a bit careful, I guess. I know we're not planning to integrate completely new or bigger features, but as there are many issues open right now which are requesting several improvements and add new ideas, I think we maybe would shock a part of the community and force to search for other solutions. WDYT?

I moved some of the directory structure bits to: https://github.com/kubereboot/kured/pull/964

Thanks for the reviews! I'll put some more work into it.

CI issues were related to the fact the link for this new page is not yet existing, so that's fine.


ImageLoader add screen size in the end of url

Hello Sergey, my issue is very strange.
I'm trying to download a lot of svg images, but Loader adds the screen size in the end of url, sooo it cant find any images using this url and returns null.
The stragneness is the loader doesn't corrupt all my urls, but I cant understand how he selects what to corrupt.
Here what I have in my log
Image can't be decoded [http://upload.wikimedia.org/wikipedia/commons/b/b3/Africa_independence_dates.svg_768x1184]
I debugged my application till the moment I call Loader, so I am sure that it isn't my mistake.
Maybe I forgot to set any parameter in config?

I've exactly the same problem. So how do I resolve this?

@manideepl Read the Useful Info Again.
Log shows the URL+Target Size. That is not your exact url. Its only for memory cache not used for requests.


Open sources compile

Hi!

I downloaded the latest source code build from your repository, but when I compile it in GTAV the console gives me version <IP_ADDRESS>, why is this happening? Why doesn't it compile version 3.4.0?

The version is taken off an environment variable SHVDN_VERSION at build time.


(AzureCXP) Adding Default documents and Path mapping pngs for clarity

Adding Default documents and Path mapping pngs for clarity

@cephalin, Requesting you to review . To provide better clarity on the Configuration > Path mappings and Default documents added appropriate settings png images.

Hi @AjayKumar-MSFT  - This PR was opened in the public repo. Would you submit these new images in the private repo instead? We'll close this PR. Thanks.

Sure, it was missed. Thanks for the update, Kristine.


[FR] - need to implement detection/punishing of multiple blocks produced for a given slot by the same pool

On both Rust ITN and current Haskell test network a pool operator can launch multiple nodes using the same set of keys. If pool has adequate amount of stake delegated, these nodes will be producing blocks at assigned times, with only one of the produced blocks surviving at most. However, in addition to the undesirable amount of forking such setup results in, this also means that in case of multi-leader slots this pool would have a better chance of claiming the reward for it.
While in the ideal world a node started with the same set of keys as already running node would not be allowed to join the network topology (not sure how feasible that is to implement), at a minimum we should be making sure that chain selection algorithm - if presented with multiple unique block hashes for the same parent slot which have the same associated pool id would ignore all of these blocks (so adversarial behaviour does not have a chance to be rewarded). Stricter punishment, such as forfeiting rewards for the entire epoch could also act as an extra deterrent potentially if that is possible to implement (I'm not sure what sort of on-chain information needs to be created for such purpose)

Hi bergr01, this is Viktor at IOHK Tech Support. Thank you for submitting this Feature Request. We have added it to our backlog and we`ll allocate it to one of our future development sprints. We appreciate your participation and support for the Cardano project!

This will be one of the common attacks from adversaries early in mainnet, hope this does not get dropped to a sprint too far away that it cannot make it to mainnet in time

Hey @vix-io how is the development of this issue is going?

hi @mrbrinker - being an enhancement, we will be looking at this at a later point so that we can focus on bugs/issues
hi @kevinhammond - any thoughts on this?

Given that chain selection is deterministic and not latency-based, I don't think this will result in forks except in the event we have slow block propagation. Likely, the multiple blocks won't make it out to the greater network. They will be pruned at the pool's own relay nodes.

Is the deterministic chain selection based on something that would be identical between the forkers multiple blocks?If not, the forker would still have an advantage.
On Monday, September 21, 2020, 10:02:42 AM PDT, Andrew Westberg<EMAIL_ADDRESS>wrote:

Given that chain selection is deterministic and not latency-based, I don't think this will result in forks except in the event we have slow block propagation. Likely, the multiple blocks won't make it out to the greater network. They will be pruned at the pool's own relay nodes.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@shawnim Yes, if each core node had a different set of transactions in the mempool, it would generate a different hash. That would give it two chances to get a lower hash instead of just 1. While it's an advantage and probably an edge case, I don't think the forked block would propagate out into the greater network.

Using multiple nodes with the same credential is indeed an attack that we thought about how best to mitigate. It's an open adversarial action, and so in principle, it's amenable to on-chain punishment.
However, a local solution (punishment through an individual honest node) like the one suggested here is not optimal: consider an attack where an operator runs two node copies. For a given slot, each produces a block, let's call them A and B. They propagate though the network, but not together. So some nodes add A to their chain, and some B, and we have a fork. If all nodes would see both A and B before the next honest block gets made, they could all ignore A and B. But what if the next slot leader has already created a block C by that time? C depends on one of those blocks, should it get dropped? That would punish the pool that created C, who is honest. And nodes that have not seen both A and B have no reason to drop either of those blocks, so there is a risk of longer forks.
A better thing to do here is to add a mechanism for posting a proof that a certain pool did perform this attack (this could be derived from the headers of A and B) to the chain, and then collectively ignoring blocks from such a pool in the future. But it's not a trivial change, needing changes to both the consensus and the ledger layer.
In the short term, we have three ways to mitigate this attack:

Monitor, and punish off-chain via social pressure. This could be done by the community.
Rely on the improved networking stack on the mainnet, and educate SPOs that it's not necessary, and indeed and attack on the system, to run multiple copies of their node.
Work on a proper high availability solution (which would help against operators that do the attack in order to boost their apparent performance, but not against deliberate attacks that aim to harm the system).

@kantp - would something as per #1883 be a proper solution in absence of something native provided by node?

The issue has been debated at length here - for Jormungand and here.

Issue has been observed in the wild on Cardano mainnet, see attached report generated using CNCLI block database.

In the short term, we have three ways to mitigate this attack:

Monitor, and punish off-chain via social pressure. This could be done by the community.
Rely on the improved networking stack on the mainnet, and educate SPOs that it's not necessary, and indeed and attack on the system, to run multiple copies of their node.
Work on a proper high availability solution (which would help against operators that do the attack in order to boost their apparent performance, but not against deliberate attacks that aim to harm the system).

We may further improve the networking stack by developing an application level firewall, that we may use to detect and block adversarial behavior in Rust... basically a relay node, that could maintain a shared library with state of the chain, and include dcSpark dApp Rollback Handler mechanics.
We already have the networking crate in Rust, and are applying for Catalyst funding to move the project along:
https://cardano.ideascale.com/a/dtd/CRUST-Cardano-Node-in-Rust/381179-48088

Do not rely on #1 that will not work. Delegators rarely care about anything else than ROI. This has to be implemented in form of consensus based punishment, similar to not adhering to the pledge. That's the only way to stop this.

Also just because it's not needed doesn't mean people will not do it. Given that this is causing harm to the network, it could be abused by whoever who wants to damage it. Imagine some high-stake pool gets hacked, the attacker could just take the keys and run many BPs with the same set of keys connected through many different relays to create as many forks and as much mess as possible.
Malicious behaviour shouldn't be tolerated.

Imagine some high-stake pool gets hacked, the attacker could just take the keys and run many BPs with the same set of keys connected through many different relays to create as many forks and as much mess as possible.

We already had few stakepool hacked...

https://twitter.com/AdagoPool/status/1351781426094632965
https://forum.cardano.org/t/spos-do-not-repeat-my-mistakes-keep-your-core-node-safe/37766


still prompted when I provide a securestring with default store

PS> $ss = read-host -AsSecureString
***********
PS> Set-Secret -Name TestSecret -SecureStringSecret $ss
Creating a new SecretStore vault. A password is required by the current store configuration.
Enter password:

It seems like if I provide a secure string, it shouldn't prompt

sorry, i misread the message


Shows selected color ramp

Shows selected ramp state in the ramp selector.
This PR also replaces the .CDB-ColorBar with .ColorBar.

CR: @xavijam
CC: @piensaenpixel

Could you check CSS changes @piensaenpixel?
About js changes 👍

@xavijam: @piensaenpixel did the changes, so I think he likes them :)

👍


Feature Request: Firebase Storage Upload Task add option to reduce chunk size

In my application Users can upload photos, the size of the file is very small, about 100KB, the problem with this code:
(storageTaskEvent) {
                          progress =
                              storageTaskEvent.snapshot.bytesTransferred.toDouble() /
                                  storageTaskEvent.snapshot.totalByteCount.toDouble();
                        }

is, that the current chunk size of uploading is 256KB, so i cannot show any progress, if the internet speed is low.
Thanks,
Niklas

Probably relevant: https://github.com/flutter/flutter/issues/20803

What is a typical duration the upload takes when internet is slow for such small files?
You know the file size in advance. Could you not just assume a duration it will probably take?
Reducing chunk size might make upload less efficient and take even longer.

For example you want to upload a image with the size of 200KB, and your phone has EDGE, which allows about 110 Kbit/s of upload speed (with good connectivity, less upload speed possible), if you calculate this would be about 2 Seconds to upload, but i tried in a real environment, and it takes about 4-5s to upload. Showing progress for this task is in the current situation impossible. Let's assume, that before and after the upload some other stuff needs to be done, which takes about 2 seconds. In this scenario, i have 6-7s seconds wait time, without showing any progress to the user.
Reducing chunk size should be an option, with a default value of 256KB.

Yes but, what if the internetconnection is fast and it takes 1sec, or 10sec i don't know how long it will take.

When upload is completed you can just remove the progressbar.

@HerrNiklasRaab
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/588. Any further collaboration will be done there.


Enhance the signup page.

There is a template for signup (/back/api/templates/django_registration/registration_form.html) and an associated css file (/back/api/css/admin-extra.css) that do a bad job.


create Nextcloud Shares on map creation

[x] Create a new directory in Nextcloud when a map is created.
[x] Create a Nextcloud Circle with the map name
[ ] Share this directory with the PermaplanT Nextcloud circle.
[ ] Change paths for BaseLayer images

As discussed in #824, we will use group folders instead. I close this issue now, to be opened later when we continue work on it.


Support passing custom HTTP headers

I'm using Nextcloud and want to retrieve calendar (read only) subscriptions along with the normal read/write calendars. Nextcloud supports two ways of doing this. The easier of the two is to pass the custom HTTP header: X-NC-CalDAV-Webcal-Caching: On
More details
Are folks open to adding this functionality by modfying the DAVclient initialization function to have an optional header parameter?
Example use:
client = caldav.DAVClient(..., headers={"X-NC-CalDAV-Webcal-Caching": "On"})
If so, I can file a pull request with this small change.

Pull request is appreciated.

The feature will be released in 1.2 shortly.  However, the issue should not be closed, we're missing documentation and test code.

This was fixed in #306 indeed.  Thanks.  Will be released in 1.3 today, I hope.


Remove DynamoDB2/RDS2 decorators

Codecov Report

Merging #5383 (5430061) into master (3d913f8) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5383   +/-   ##
=======================================
  Coverage   96.05%   96.05%
=======================================
  Files         708      708
  Lines       71000    70991    -9
=======================================
- Hits        68200    68192    -8
+ Misses       2800     2799    -1

Flag
Coverage Δ

servertests
35.83% <100.00%> (-0.01%)
:arrow_down:

unittests
95.99% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

moto/dynamodb/parsing/executors.py
99.26% <ø> (ø)

moto/dynamodb/parsing/expressions.py
96.32% <ø> (ø)

moto/dynamodb/responses.py
95.13% <ø> (ø)

moto/__init__.py
98.54% <100.00%> (+1.27%)
:arrow_up:

moto/awslambda/models.py
93.91% <0.00%> (-0.11%)
:arrow_down:

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


[TACHYON-470] Update TachyonConf.getLong to eliminate taking default value

JIRA: https://tachyon.atlassian.net/browse/TACHYON-470
getLong is still taking a default value. This PR aims to remove the default value parameter.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/Tachyon-Pull-Request-Builder/4349/
Test PASSed.

LGTM


About header of using FeignClient

When I use FeignClient to call other server,how can I set the header?

Go for a Feign Interceptor. Checkout the reference doc. It works quite simply.

More specifically see the section on RequestInterceptors in Feign and the section on Feign configuration in Spring Cloud.


Don't save ckpt.pth from every epoch

Hi everyone, I don't want to save all the "xxx.pth" from every epoch, such as
"epoch_1_ckpt.pth"
"epoch_2_ckpt.pth"
...
and so on,
How can I set my param to solve this problem?

In your exp file, set self.save_history_ckpt = False

In your exp file, set self.save_history_ckpt = False

Thanks, your suggestion is very usefull!


Date in ExpressLocalizationResource.fr.resx

The date value in this field is set to 'Rendez-vous amoureux'. This in french meen like 'Your love date' Like if your are dating some one. The correct value is simply Date.

@werddomain thanks for the correction, I will fix it soon.
may be I should warn google translate as well :)

fixed in commit https://github.com/LazZiya/ExpressLocalizationSample/commit/861010eb850f5e24218968344c5236a04264591f
thanks :)


Fix Fee Validate Note

Sorry, this is actually correct - the formal spec is wrong here I believe. @dnadales can you confirm this?

Yes, this was addressed in https://github.com/input-output-hk/cardano-ledger-specs/pull/475#issuecomment-493541180
Sorry for the confusion @felixYing and thanks!


bazel  fails to build in clang stdenv

Describe the bug
Compilation using bazel in a clang13Stdenv environment fails.
Something similar shows up in Darwin, which as clang as default environment, but this is to demonstrate with a self-contained example that this is a general problem with an isClang environment.
Steps To Reproduce
I've created a self-contained example repository with a small project that uses C and C++ files
https://github.com/hzeller/nix-bazel-with-clang
git clone https://github.com/hzeller/nix-bazel-with-clang
cd nix-bazel-with-clang
nix-shell --command 'bazel clean ; bazel build //... && bazel-bin/main'

This example provides a shell.nix that enables clang13Stdenv
{ pkgs ? import <nixpkgs> {} }:
let
  #used_stdenv = pkgs.stdenv;         # this works, using gcc
  used_stdenv = pkgs.clang13Stdenv;   # this is broken, using clang
in
used_stdenv.mkDerivation {
  name = "Testing c and c++ compilation";
  buildInputs = with pkgs;
    [
      bazel_4
    ];
}

Expected behavior
This should compile and run the small main program in bazel-bin/main, outputting
in C   function foo() -> 42
in C++ function bar() -> 42

== SUCCESS ==
got values 42 and 42

... this happens instead
... but what actually happens is that the compilation fails:
INFO: Analyzed 2 targets (15 packages loaded, 63 targets configured).
INFO: Found 2 targets...
ERROR: /home/testuser/src/my/bazel-with-clang/BUILD:13:10: Compiling main.cc failed: (Exit 1): clang failed: error executing command /nix/store/i973rwv7n9pq74iac8jlly9s3xlrr0bc-clang-wrapper-13.0.1/bin/clang -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -Wunused-but-set-parameter -Wno-free-nonheap-object ... (remaining 25 argument(s) skipped)

Use --sandbox_debug to see verbose messages from the sandbox
main.cc:1:10: fatal error: 'iostream' file not found
#include <iostream>
         ^~~~~~~~~~
1 error generated.
INFO: Elapsed time: 0.216s, Critical Path: 0.03s
INFO: 12 processes: 11 internal, 1 linux-sandbox.
FAILED: Build did NOT complete successfully

Additional context
Symptoms of this issue have been described in other bugs, but often in the Darwin context #150655
Also seen in unsuccessful pull request attempting to add Darwin support #214797
People have sometimes applied various workarounds successfully, but this of course need to work 'out of the box'.
Darwin is not the main culprit though, it is just surfacing the issue because there, the stdenv is based on clang.
The problem is more  the way bazel  and the compiler environment is wired up, setting flags and possibly providing compiler-wrappers needed, so I hope with the self-contained example in the git repo mentioned above it is possible for the nix-bazel maintainers to get to the root of the problem.
Notify maintainers
CC @NixOS/bazel @aherrmann @ylecornec
Also FYI @uri-canva
Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.90, NixOS, 22.11 (Raccoon), 22.11.2203.285b3ff0660`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`
 - channels(root): `"nixos-22.11"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Addendum: the above nix-info is with 22.11, but same behavior also with nixos-unstable.
$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.93, NixOS, 23.05 (Stoat), 23.05pre453471.e5530aba13c`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I see that the example does not set up any toolchain explicitly, so it's going to use Bazel's builtin autodetecting cc toolchain.
You can look through the files under $(bazel info output_base)/external/local_config_cc to inspect the generated configuration. You can find the corresponding files from the Bazel distribution here.
In rules_nixpkgs we found that a clang based cc-toolchain requires additional compiler and linker flags to work with Bazel. You can find that configuration here.

After a bit of stall on this issue here, I dug into the underlying reason and possible fix suggestions.
TL;DR
bazel uses the canonical name of the compiler to compile c++, gcc in stdenv and clang with clang13Stdenv.
In NixOS,

gcc can compile c++ files without needing the specific g++ compiler. This is why bazel works with stdenv.
clang can not compile c++ files, and requires to be specifically called as clang++. This is different than other Linux distributions so this is why bazel's assumptions how to call the c++ compiler fail to work with clang13Stdenv.

What bazel uses as compiler
The problem seems to be that bazel by default uses the generic compiler driver for c++, so depending on the stdenv, this would be gcc or clang. Looking at the  cc_wrapper.sh in the local config as suggested by @aherrmann to inspect the environment shows the compiler bazel uses to compile c++ code
Here, using stdenv:
mkdir -p /tmp/foo ; touch /tmp/foo/WORKSPACE ; cd /tmp/foo
nix-shell \
    -E '{ pkgs ? import <nixpkgs> {} }: pkgs.stdenv.mkDerivation { name="foo"; buildInputs = [ pkgs.bazel_6 ];}' \
    --run 'bazel clean --expunge ; bazel sync ; cat $(bazel info output_base)/external/local_config_cc/cc_wrapper.sh'

... the wrapper script emits that it uses plain gcc:
# ...
# Call the C++ compiler
/nix/store/75slks1wr3b3sxr5advswjzg9lvbv9jc-gcc-wrapper-12.3.0/bin/gcc "$@"

... and here, using clang13Stdenv
mkdir -p /tmp/foo ; touch /tmp/foo/WORKSPACE ; cd /tmp/foo
nix-shell \
    -E '{ pkgs ? import <nixpkgs> {} }: pkgs.clang13Stdenv.mkDerivation { name="foo"; buildInputs = [ pkgs.bazel_6 ];}' \
    --run 'bazel clean --expunge ; bazel sync ; cat $(bazel info output_base)/external/local_config_cc/cc_wrapper.sh'

Now we see that bazel chooses plain clang
# ...
# Call the C++ compiler
/nix/store/b4y1vzkybpx8ggfrdrn47r21dh44kj0j-clang-wrapper-13.0.1/bin/clang "$@"

Can these compilers deal with C++ ?
So, in both cases, bazel chose the common name of the plain compiler gcc and clang assuming that if given a c++ program they behave like g++ and clang++.
This does indeed work with gcc in nix-os, but does not work with clang.
Let's compile a super-simple c++ program including a c++ header with both gcc and clang:
With gcc
nix-shell  \
    -E '{ pkgs ? import <nixpkgs> {} }: pkgs.stdenv.mkDerivation { name="foo";}' \
    --run 'echo "#include <vector>" > /tmp/foo.cc ; gcc -c /tmp/foo.cc -o /tmp/foo.o'

This works!
... with clang
nix-shell  \
    -E '{ pkgs ? import <nixpkgs> {} }: pkgs.clang13Stdenv.mkDerivation { name="foo";}' \
    --run 'echo "#include <vector>" > /tmp/foo.cc ; clang -c /tmp/foo.cc -o /tmp/foo.o'

This emits an error:
/tmp/foo.cc:1:10: fatal error: 'vector' file not found
#include <vector>
         ^~~~~~~~
1 error generated.

The error indicates that it behaves like a c-compiler and not as a c++ compiler that knows how where to find c++ headers.
It does work, if we use clang++ as compiler:
nix-shell  \
    -E '{ pkgs ? import <nixpkgs> {} }: pkgs.clang13Stdenv.mkDerivation { name="foo";}' \
    --run 'echo "#include <vector>" > /tmp/foo.cc ; clang++ -c /tmp/foo.cc -o /tmp/foo.o'

Conclusion, possible solutions
So all we need is bazel invoking clang++ instead of clang.
There see two possible simple solutions

Make clang work as driver that also can compile c++ code. This is how it is done in other Linux distributions and would be compatible with the gcc behavior. But maybe there is a good reason why clang is set up differently in nix-os ?
Other solution: Patch bazel in nixos to search for clang++ instead of clang to set up its CXX compiler. (mabye by inspeccting $CXX environment variable first ?). With this analysis uncovering the underlying reason, the  @NixOS/bazel maintainers can probably quickly find how to work around that in the bazel set-up, maybe patching the bzl-file that configures the toolchain.

I think I would prefer the first solution (fix clang to behave like in other Linux distributions and like gcc), but I suspect it will be easier to implement the second.

One possibility to get a nix-backed C++ toolchain for Bazel that uses the default clang is to generate it via Bazel-remote. We've been using this approach for quite some time now and it kind of "just works".

Create an image with the C++ tooling you want. For instance, we're using this one: https://github.com/eomii/rules_ll/blob/main/rbe/image.nix
Use bazel-remote to generate a cc toolchain config from the image: https://github.com/eomii/rules_ll/blob/main/devtools/rbe.nix
Add this toolchain as the default cc toolchain in e.g. .bazelrc or by patching in some cc_toolchain override to MODULE.bazel/WORKSPACE.bazel.

This setup generates a cc toolchain config that uses tools from nixpkgs (like this).
This exact approach is probably not what you'd  want in nixpkgs, but a similar approach might be possible. I think a nice solution could be a tool that does the same kind of toolchain autogeneration as bazel-remote but directly generates the cc toolchain configs from the nixpkgs inputs instead of using this roundabout way via container images.

Yes, looks like this and the other issue describe the same problem @uri-canva


[Feature Request] Delete model functionality on Android/IOS app

🚀 Feature
Functionality to delete models from the Android/IOS app.
Motivation
I downloaded the vicuna-v1-7b-q3f16_0 model on the iPhone 12 device. But the model couldn't initialize due to not having enough memory. Now I want to delete the vicuna-v1-7b-q3f16_0 model to free up space. But currently the only way to do it is to delete the app and reinstall the app.
Alternatives
Maybe in the future you can perform a check when adding a new model using the URL if the model will work with the current mobile device before downloading the model.
Thank you!

Same problem. I would like to delete llama 2 model, and use only pajama model on my iOS device. But I can't delete model inside the interface, nor in file manager. Delete button does not work on default model.

this is now supported in ios


Fix click to run workflows for .org, and expose additional WF info in UI

Fixes #3883 #3739, adds minimal markup for https://github.com/galaxyproject/galaxy/pull/15684
Removes theme change as BHM is over
Replaces with Smorgasbord advert
Workflow links

yes, of course!


Update introduction.md

🚀 Deployed on https://66102f49a1c26a1e4fa2315a--cozy-fox-0defec.netlify.app


sub read-n($fh, $n) { in Language/5to6-nutshell.pod is weird for a p5 hacker

Coming from perl5, when I see sub read-n, I immediately think "syntax error". Perhaps the doc should clarify that this doesn't do anything weird and is just a sub name? Some perspective from LISP/Scheme helps me grok this, but maybe not everyone reading this document has that context.
To be clear, this is at this line in the file.

Thank you for your report! But what about lookup-user, do-something, do-something-else, do-default-thing, %subs-to-export, @import-list and so on (later in the same file)?
I don't think that we should explain that dashes are OK after every occurrence. I also don't think that 5-to-6 documents should use p5 naming conventions just to make people comfortable. Dashes are too common in perl 6.
There is nothing we can do here, therefore this issue should be closed, IMO.

On Aug 21, 2016, at 15:48, Aleks-Daniel Jakimenko-Aleksejev<EMAIL_ADDRESS>wrote:
Thank you for your report! But what about lookup-user, do-something, do-something-else, do-default-thing, %subs-to-export, @import-list and so on (later in the same file)?
I don't think that we should explain that dashes are OK after every occurrence.

I didn’t suggest we should specify it after every occurrence. A simple clarification (like every side-by-side example in the document) seems like a reasonable approach.

I also don't think that 5-to-6 documents should use p5 naming conventions just to make people comfortable. Dashes are too common in perl 6.

Is your suggestion that, in a guide designed to help perl5 dev, we shouldn’t make perl5 devs comfortable? This seems to be contradictory to the purpose of the document.

There is nothing we can do here, therefore this issue should be closed, IMO.

I gave a specific suggestion of what to do. What is the problem with the suggestion I made?

It's the first example of a user writing code that uses - as an identifier.
Nobody was suggesting we should change to perl5 naming conventions.
Mentioning it the first time you demonstrate writing such code does not seem like a bad idea.

@masochist I've added a brief Identifiers section that makes a passing mention that dashes and weird Uncidode bits are OK in P6 identifiers. Hopefully that's deconfusing enough for this Issue.

Thank you! This does help.


[Suggesstion] Endpoint to upload images and AASX files

Hello,
it would be very nice to upload images and AASX files to the AAS environment. This would be very useful to add already existing AAS and set the thumbnail or add image properties during runtime.
KR,
BlackRose01

I missed the endpoint in the swagger. Sorry


JavaTemplate fails in visitMethodInvocation if an argument is an array

JavaTemplate will fail to replace arguments in visitMethodInvocation if an argument is an array.
template(#{any(int[])}) will fail with a ILE: Syntax error at line 1:7 extraneous input '[' expecting {')', FullyQualifiedName, Number}.
template(#{any()}) will fail with a ClassCastException: class org.openrewrite.java.tree.J$Identifier cannot be cast to class org.openrewrite.java.tree.J$MethodInvocation (org.openrewrite.java.tree.J$Identifier and org.openrewrite.java.tree.J$MethodInvocation are in unnamed module of loader 'app')

template works properly with anyArray()


sortable and jQuery UI dependency

Since jQuery UI doesn't seem to be in a maintained state at the moment (http://blog.jqueryui.com/2017/12/the-future-of-jquery-ui-and-jquery-mobile/), we were wondering if there's a chance that the people from Tokenize2 will refactor the sortable feature to prevent jQuery UI usage.
For our project, we included Tokenize2 (which we are quite happy with).
One of our plugin developers wants to use the sortable feature (https://github.com/opnsense/plugins/issues/582), but we like to avoid pulling libraries in for which we can expect issues in the (near) future (we removed jQuery UI from our main project a couple of years ago).
Thanks in advance for your reply.

Hi, for the moment it's not planned but I'm not closed to the idea, I need to find a small library to include directly in the plugin, it can do it.


gulp-stylus used with sourcemaps + resolve url fails

As referenced in the discussion https://github.com/stevelacy/gulp-stylus/issues/20 - it seems as if using gulp-stylus with stylus files using @import directives to stylus files that includes image-references using relative paths faile (eg. background-image: url('relative-path.png')).
gulp.task('gulp-stylus-task', function(){
	// need to use wildcard to write css/map-files to correct path
	return gulp.src('**/*.styl')
		.pipe(sourcemaps.init())
		.pipe(gstylus({
			define: {
				url: stylus.resolver()
			}
		}))
		.pipe(sourcemaps.write())
		.on('error', function(e) {
			console.log(e);
		})
		.pipe(gulp.dest('.'));
});

running the above snippet (given the correct input files and project structure - check out sample repo here https://github.com/akselnordal/stylus-example) results in the following error:
Potentially unhandled rejection [2] Error: /Users/akselnordal/stylus-example/node_modules/gulp-stylus/examples/css/nib.styl:1:9
   1| @import 'nib'
--------------^
   2|
   3| #back-to-top
   4|   fixed: bottom right

failed to locate @import file nib.styl

    at Evaluator.visitImport (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/evaluator.js:898:21)
    at Evaluator.Visitor.visit (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/index.js:28:40)
    at Evaluator.visit (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/evaluator.js:153:18)
    at Evaluator.visitRoot (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/evaluator.js:689:27)
    at Evaluator.Visitor.visit (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/index.js:28:40)
    at Evaluator.visit (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/evaluator.js:153:18)
    at Evaluator.evaluate (/Users/akselnordal/stylus-example/node_modules/stylus/lib/visitor/evaluator.js:229:15)
    at Renderer.render (/Users/akselnordal/stylus-example/node_modules/stylus/lib/renderer.js:86:26)
    at dispatch (/Users/akselnordal/stylus-example/node_modules/gulp-stylus/node_modules/accord/node_modules/when/node.js:71:15)
    at callAndResolve (/Users/akselnordal/stylus-example/node_modules/gulp-stylus/node_modules/accord/node_modules/when/lib/apply.js:30:12)

There were many errors in your setup.

you were including all files in every folder (including files in node_modules) thus the nib error
there were file name errors and external sourcemap maping errors in the gulpfile
the files were getting deleted before the check was called

I have fixed the errors here, and added an index.html to prove the completeness of the result.
https://github.com/stevelacy/gulp-stylus-20

@stevelacy thank you for patience; after reviewing your fixes everything seems to be in order. I'm still a bit new to building using gulp (just switched from grunt)..

I am glad you are using gulp!


Fix C4703 (uninitialized variable) on MSVC 2015

The condition where this is used uninitialized shouldn't actually be reachable code, but MSVC doesn't seem to be able to figure that out

Thanks for the fix!


Bug after upgrading from 0.8.2 to 0.9.0

Webpack doesn't bundle after upgrading:

@prefresh/webpack from 0.8.2 to 0.9.0

CLI error:
building ./node_modules/html-webpack-plugin/lib/child-compiler.js:175
    return childCompilation.assets[helperFileName].source();
                                                   ^

TypeError: Cannot read property 'source' of undefined
    at ./node_modules/html-webpack-plugin/lib/child-compiler.js:175:52
    at Array.map (<anonymous>)
    at extractHelperFilesFromCompilation (./node_modules/html-webpack-plugin/lib/child-compiler.js:174:43)
    at ./node_modules/html-webpack-plugin/lib/child-compiler.js:113:13
    at ./node_modules/webpack/lib/Compiler.js:343:11
    at ./node_modules/webpack/lib/Compiler.js:681:15
    at AsyncSeriesHook.eval [as callAsync] (eval at create (./node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:24:1)
    at AsyncSeriesHook.lazyCompileHook (./node_modules/tapable/lib/Hook.js:154:20)
    at ./node_modules/webpack/lib/Compiler.js:678:31
    at AsyncSeriesHook.eval [as callAsync] (eval at create (./node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook (./node_modules/tapable/lib/Hook.js:154:20)
    at ./node_modules/webpack/lib/Compilation.js:1423:35
    at AsyncSeriesHook.eval [as callAsync] (eval at create (./node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:6:1)
    at AsyncSeriesHook.lazyCompileHook (./node_modules/tapable/lib/Hook.js:154:20)
    at ./node_modules/webpack/lib/Compilation.js:1414:32
    at AsyncSeriesHook.eval [as callAsync] (eval at create (./node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:6:1)

Yarn's lock changes (no other deps were updated):

Hey @o-alexandrov
Thanks for bringing this to my attention, I was passing along something the wrong way, this has been fixed in 0.9.1

@JoviDeCroock Thank you, 0.9.1 doesn't have this issue when bundling w/ webpack.


regen: quit when fake pandad finishes

To avoid CAN data looping around causing unexpected behavior
Tried doing this for all daemons but controlsd never enabled

Loop around is fine in the case of #24140, we'll just catch the bad rising edge of engagement


docs: write documentation

There is a lot missing in the documentation:

[ ] F2A feature description
[ ] favorites/burger menu
[x] changelog/news announcements
[ ] background mode
[X] proxy setup
[x] authorizations
[ ] branch overview
[ ] keyboard shortcuts
[ ] merging branches
[ ] requirements feature, for eg. SAP_BASIS versions

Are the entries of menu "Advanced" somewhere documented? If not, should we create a new topic below "Reference"?

closing, some of this is done, open new issues if something is missing


[BUG] Telemetry Buffer Getting Cleared in Offline Mode - Online Status Incorrectly Initialized in Offline Listener

Behavior
In the scenario for progressive webapps the telemetry buffer is not being preserved during offline mode.
Culprit
This is happening because, in the constructor for OnlineListener, we initialize _onlineStatus = true.
Which prevents the Sender._xhrReadyStateChange event handler from adding it back to the telem buffer after failure.
https://github.com/microsoft/ApplicationInsights-JS/blob/22d3787cb9b328bbf3cab0ee38a686d4779f8031/channels/applicationinsights-channel-js/src/Offline.ts#L17
https://github.com/microsoft/ApplicationInsights-JS/blob/22d3787cb9b328bbf3cab0ee38a686d4779f8031/channels/applicationinsights-channel-js/src/Sender.ts#L387
This logic will be sufficent in most scenarios since the initial GET request, would have to have been fetched over the wire for application insights to have been invoked.
But in the case of PWAs we would have this cached, and initialization would happen, even though we are offline.
Initializing to true is not sufficient.
Additional notes
The hooks for detecting network changes: online and offline are working successfully, I have tested these!
But that will only re-evaluate the _onlineStatus if the status of the network has changed, after the application was loaded, which may not be the case
Suggested Fix
Initialize _onlineStatus by evaulating the status of the network, using a browser hook such as window.navigator.online
Steps to recreate
Open a PWA when the application is in offline mode.
You will see that OfflineListener._onlineStatus is true and telemetry buffer is not being preserved.

v2.6.2 is now fully deployed


5 testing squash and merge

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


build(lint): install and use golangci-lint from cache

fixes #ENG-286
speeds up lint process
replace deprecated linters
Signed-off-by: Artur Troian<EMAIL_ADDRESS>

This doesn't use docker anymore to run golang CI?


Error with question file which have only QS question type.

I'm training a DNN model with Magphase vocoder and get error "index out of bound" with my question file that have only QS question type. I debugged file "label_normalisation.py" and figure out bug in line 228: "if(label_continuous_vector[0, 1]==1 or current_phone in list_of_silences)". Because if there is no CQS question, the label_continuous_vector will become to empty. I hope this bug will be fixed soon.

Hi i'm having the same issue. How did you fix it?

I added cqs question into my question file and it's work.


[IMPROVE] Signup code can use single object to form fields

Use useState with object for form fields: Instead of having separate state variables for each form field (e.g., firstName, surname, email, etc.), you can use a single state object using useState to manage all the form fields together.
This will make code better readable and robust. Assign me this under GSSOC'23. I have already worked on it.

@aashi-ihsaa
That's a great suggestion! Using a single state object with useState to manage all form fields together can indeed improve code readability and make it more robust. It helps in organizing and handling form data more efficiently.
To implement this approach, you can define a state object using useState and initialize it with the initial form field values. Then, you can update the state object whenever a form field value changes by using the onChange event handlers of the form inputs.
Creating a separate branch for this task is a good practice to keep the changes isolated and organized.


Test on nuScenes

Thank you for your excellent work！Is there any result on the nuScenes dataset? Why not test on this dataset?

Thank you for your excellent work！Is there any result on the nuScenes dataset? Why not test on this dataset?

We don't have enough GPUs prevously so that we mainly conduct experiments on the monocular setting. We'll update this results in the following CGFormer++.


Metrics: Implement metrics abstraction

This PR attempts to implement #998 and #980. The general idea is to pass a factory interface along the packages for the registration of metrics. By doing so, it enables abstraction over the current Prometheus metrics system, allowing for potential swapping to a different metrics system in the future.
Please note that this currently is just a concept; If it's something you would want me to pursue, it can be further developed.

Thanks so much for the initiative.
What do you think of using the Prometheus interfaces directly? E.g., histogram, counter
We can have nop implementations for each interface to use for defaults, and then have With<Name>Metric methods on each struct that requires a metric. For example:
func (s *Synchronizer) WithNumTransactionsSeenMetric(c prometheus.Counter) *Synchronizer

That would also enable us to test that the metrics behave properly and only import prometheus in the node package.
(apologies for the late review--@omerfirmak and I have been out for the last week or so)

What do you think of using the Prometheus interfaces directly? E.g., [histogram] [counter]

Prometheus doesn't have interfaces defined for vec types so that would need be defined somewhere and imported alongside prometheus. It would also be a little weird to depend on prometheus interfaces meanwhile using another system eg. codvar

We can have nop implementations for each interface to use for defaults, and then have WithMetric methods on each struct that requires a metric. For example:
func (s *Synchronizer) WithNumTransactionsSeenMetric(c prometheus.Counter) *Synchronizer

That could work but prometheus requires all of the ops to be defined at creation. It would result in package creating synchronizer defining namespace, subsystem etc. for NumTransactionsSeenMetric and then passing it to synchronizer.WithNumTransactionsSeenMetric which leaks context to caller.
Maybe generic contract like this could work:
func (s T) WithLightweightMetrics(c metrics.Factory) T
func (s T) WithExpensiveMetrics(c metrics.Factory) T

Could give some configurable options and still keep everything encapsulated while enabling testing.
@joshklop

Prometheus doesn't have interfaces defined for vec types so that would need be defined somewhere and imported alongside prometheus. It would also be a little weird to depend on prometheus interfaces meanwhile using another system eg. codvar

Good points. What if we created interfaces for everything we need (counter, histogram, gauge, etc.) in a metrics package? This has the added benefit of not importing prometheus outside the node package, which is what we're after anyway.
I'm open to using expvar as well, if we can figure out a good way to wire it up to Prometheus, get histogram support, etc.

leaks context to caller

Can you elaborate on this? In my mind, the caller should create the namespace, subsystem, etc. to avoid naming collisions and encourage consistency. Plus, it may be necessary to take advantage of prometheus's testutil, which may be easier than a huge mock that's only used for metrics.

I was imagining metrics like a typical blackbox. By leaking context I mean that the caller has to know some of the internal mechanisms, node in that case would have to know what exactly is NumTransactions in order to provide correct description for that.  It goes deeper with each nested object that requires metrics eg. A has its own metrics namespace and it creates B which has yet another namespace, The creator of A has to be aware of that to provide proper namespace for B. Maybe some of the components could be interfaces that would have slightly different descriptions based on their concrete implementation. That is not possible when the caller is responsible for creation.

That makes sense. I'm curious how large Go projects manage metrics. Tailscale has a custom system that seems to work well, go-ethereum vendors an old metrics library IIRC, ... have you seen any good ways to do it? Ideally our metrics setup is:

optional: no-op defaults, for example
testable
open to multiple implementations
lightweight (the prometheus package imports protobuf and possibly other large libraries)

Your current implementation is the closest I think, except for transitively importing prometheus.

Sadly I do not know the best approach. From what I`ve seen repos supporting multiple exporters usually roll their own metrics types and create adapters for exporting. Haven't seen in those cases a single one supporting labels for prometheus tho.

Jotting down here for future reference: Tailscale's lightweight metrics package has an associated prometheus exporter. Maybe we can create a simple interface around it? Haven't looked into it much.


added some pre-req material

A first pass at Key Wrap Requirements / Lessons Learned. Imitated some of what was pushed for the Crypto Proofs material.

Merged this into the requirements branch. Let's work from that branch.


Number of connected users reported is off by one

R Services Information:
	Remote workspace 'Cont2' : https://localhost:5914/
	Version: 1.2.30912.1000
	Operating System: Linux 4.9.41-moby #1 SMP Wed Sep 6 00:05:16 UTC 2017
	CPU Count: 2
	Physical Memory: 2027 MB, 201 MB free
	Virtual Memory: 1048 MB, 1048 MB free
	Connected users: 0

Installed R versions:
	[0] CRAN R '3.4.1-2xenial0' (Default)

This depends on when the AboutHost info is requested. if the call gets through before we create user session on the broker the users count may be off by 1.


Using .raw with an escaped question mark ('\?') produce a syntax error

Environment
Knex version: 0.16.3
Database + version: mysql  Ver 14.14 Distrib 5.7.25, for Linux (x86_64) using  EditLine wrapper
OS: Debian buster
Bug

Explain what kind of behaviour you are getting and how you think it should do

Using .raw with an escaped question mark ('\\?') and other methods (limit, where, ...), database server returns a syntax error

Error message

Unhandled rejection Error: ER_PARSE_ERROR: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '?' at line 1
    at Query.Sequence._packetToError (/home/urungi/urungi/node_modules/mysql/lib/protocol/sequences/Sequence.js:47:14)
    at Query.ErrorPacket (/home/urungi/urungi/node_modules/mysql/lib/protocol/sequences/Query.js:77:18)
    at Protocol._parsePacket (/home/urungi/urungi/node_modules/mysql/lib/protocol/Protocol.js:278:23)
    at Parser.write (/home/urungi/urungi/node_modules/mysql/lib/protocol/Parser.js:76:12)
    at Protocol.write (/home/urungi/urungi/node_modules/mysql/lib/protocol/Protocol.js:38:16)
    at Socket.<anonymous> (/home/urungi/urungi/node_modules/mysql/lib/Connection.js:91:28)
    at Socket.<anonymous> (/home/urungi/urungi/node_modules/mysql/lib/Connection.js:502:10)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at TCP.onread (net.js:607:20)
    --------------------
    at Protocol._enqueue (/home/urungi/urungi/node_modules/mysql/lib/protocol/Protocol.js:144:48)
    at Connection.query (/home/urungi/urungi/node_modules/mysql/lib/Connection.js:200:25)
    at /home/urungi/urungi/node_modules/knex/lib/dialects/mysql/index.js:144:18
    at Promise._execute (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/debuggability.js:313:9)
    at Promise._resolveFromExecutor (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/promise.js:483:18)
    at new Promise (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/promise.js:79:10)
    at Client_MySQL._query (/home/urungi/urungi/node_modules/knex/lib/dialects/mysql/index.js:135:12)
    at Client_MySQL.query (/home/urungi/urungi/node_modules/knex/lib/client.js:192:17)
    at Runner.<anonymous> (/home/urungi/urungi/node_modules/knex/lib/runner.js:138:36)
    at Runner.tryCatcher (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/util.js:16:23)
    at Runner.query (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/method.js:15:34)
    at /home/urungi/urungi/node_modules/knex/lib/runner.js:47:21
    at tryCatcher (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/util.js:16:23)
    at /home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/using.js:185:26
    at tryCatcher (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/home/urungi/urungi/node_modules/knex/node_modules/bluebird/js/release/promise.js:512:31)

Reduced test code

const knex = require('knex')({
    client: 'mysql',
    connection: {
        host: '<IP_ADDRESS>',
        user: 'user',
        password: 'user',
        database: 'koha'
    }
});

const qb = knex('borrowers')
    .select(knex.raw('CONCAT(??, "\\?")', 'borrowernumber'))
    .limit(1);

console.log(qb.toString());
// select CONCAT(`borrowernumber`, "?") from `borrowers` limit 1
// This query, if executed manually in mysql client, is valid and returns results

qb.then(result => {
    console.log(result);  // This is never executed
});

Shouldn't it be '\\?' rather than "\\?" ? Not too keen on mysql syntax.. But using Postgres the example above works just fine (after replacing " with ' ofc)

Knex seems to be doing there what it is supposed to be doing: https://runkit.com/embed/fn2wqp4ylahy (note that toString() doesn't give very truthful image what query is actually been run by knex).
However... since mysql uses ? marks natively as paramater bindings , maybe they require some special escaping for mysql driver?

Shouldn't it be '\\?' rather than "\\?" ? Not too keen on mysql syntax.. But using Postgres the example above works just fine (after replacing " with ' ofc)

I tried with simple quotes but it doesn't work (same error message)

Knex seems to be doing there what it is supposed to be doing: https://runkit.com/embed/fn2wqp4ylahy (note that toString() doesn't give very truthful image what query is actually been run by knex).
However... since mysql uses ? marks natively as paramater bindings , maybe they require some special escaping for mysql driver?

The only way I found to escape '?' for mysql client is by using mysql.raw.
The following snippet works:
const mysql = require('mysql');
const qb = knex('borrowers')
      .select(knex.raw('CONCAT(??, "?")', ['borrowernumber', mysql.raw('?')]))
      .limit(1);

while the following doesn't work
const qb = knex('borrowers')
      .select(knex.raw('CONCAT(??, "?")', ['borrowernumber', knex.raw('?')]))
      .limit(1);

Note that if I remove the .limit(1), then the query works as expected:
const qb = knex('borrowers')
    .select(knex.raw('CONCAT(??, "\\?")', 'borrowernumber'));

Thanks for the info.
It is not unexpected that second one doesn't work and the last one works kind of by accident, because there are no value bindings at all in the query.
I would accept PR which converts escaped ? automatically to mysql.raw value replacements in mysql dialect. I consider this actually as bug, since toSQL().toNative() should produce query and bindings that are runnable through mysql driver.

wahoo ! tricky ! Thanks for the tip

'?' still has a side effect :
I have a request that renders a string that contains '?'. When I am in the report editor AND if there is a row limit (which is by default) then the limit value takes place of the first '?' caracter (the request is something like :
"""
SELECT concat ('http://myurl?', some_field), ***
FROM***
LIMIT ?
"""
Gives with a 10 row limit :
"""
SELECT concat ('http://myurl10', some_field), ***
FROM***
LIMIT ?
"""

@tbenita sounds like you are not using knex raw parameter bindings correctly (thus using knex in a wrong way), please ask help from stackoverflow and add also some code, how are you generating those strings.

knex.raw() seems to be replacing question marks within strings.  e.g. knex.raw(INSERT INTO foo (bar) VALUES ('hello ?') gets inserted with hello $25.  I was trying to use knex-migrate-sql-file to do some bulk INSERTs from a pg_dump file and everything blew up with corrupted data.

I created PR https://github.com/ozum/knex-migrate-sql-file/pull/4 to escape question marks in SQL file.  Any suggestions appreciated.  Works better than it did for me, but I worry there's more edge cases to trip over in the future.

I tried with \? and it's working by now.
Ticket can be close.

This problem persists with knex v1.0.4 and Postgres v12+
q.whereRaw('i.foo @\\? \'$.*.bar \\? (@ == "?")\'', bar)
Yields the error: Error: Expected 1 bindings, saw 3

This issue occurred in knex v2.0.0 and Oracle 12.1c
db.whereRaw(`REGEXP_LIKE("column1", '^[+-]\\?\\d*\.\\?\\d+([eE][+-]\\?\\d+)\\?$')`)
the original regex is ^[+-]?\d*\.?\d+([eE][+-]?\d+)?$
I checked the v$sql in oracle, even though I escaped ? as \\?, knex translates all question marks into parameter binding
any advise would be appreciated

Same that https://github.com/knex/knex/issues/5189

This issue is still persisting. Attempting to use raw transactions to update URL's in a MySQL database. Any updates on this big?

\\u003F solved my problems : )

@675076143 how does this \u003F solved the issue any example please

@sadiqueWiseboxs , basically if you're trying to escape characters that go into the knex.raw value, you can use badString.replace(         /\?/g,         "\\u003F"     );.

I was able to work around it by injecting the question marks as a param:
const params = {
  questionMark: '?',
};

knex.raw(
  `SELECT CONCAT('https://example.com', :questionMark, 'id=', entities.id) AS url from entities`,
  params
);


Add Arabic Orthography Parsing and Output

This adds an experimental orthography for Kwak'wala based on the Arabic Script, using characters from various alphabets. Since this orthography is very experimental, it is subject to significant changes. However, I think the basic ideas will likely stay the same. One significant difference from Standard Arabic is that all vowels are notated with full letters, as opposed to only using full letters for long vowels and only notating short vowels with optional diacritics. There are a few reasons for this:

Standard Arabic only has 3 monophthongs, whereas Kwak'wala has 6.
The additional vowels are notated by adding vowel diacritics to Alif, since /ai/ and /au/ are realised as /e/ and /o/ in some Arabic dialects.
Plain Alif is used to represent /ə/, since it is the "standard vowel" in Kwak'wala. /a/ is instead represented by adding the diacritic for /a/ to Alif.
The diacritic for short /u/ is used to represent labialization.

There are also some issues with using Hamzah, which represents the Glottal stop in Arabic. In Arabic, the Hamzah is typically combined with another letter, and rarely represented as full letter. The default option (at the moment) is to only combine the Hamzah with vowels, since the Hamzah diacritic is also used to represent glottalization and ejective consonants. However, there is an option to disable combining the Hamzah with vowels, but this version typically doesn't look as good.
Also, like Arabic, the characters for /u/ and /w/ are the same, as are the characters /i/ and /j/. To distinguish them, the consonantal versions have the null-vowel diacritic added to them.

The Arabic Orthography has been properly tested, and have now also added tests for other orthographies as well.


KAFKA-13327: Gracefully report connector validation errors instead of returning 500 responses

Jira
Background context: this is split off from https://github.com/apache/kafka/pull/11369, which addressed this issue and two others. Not only does this new PR fix the merge conflicts with its predecessor, it also simplifies the review process by addressing a single issue at a time.
This change addresses an issue where some types of validation errors are reported via HTTP 500 responses, which is incorrect (the issue is not the fault of the server, but rather of the connector configuration) and can cover up other validation errors. Instead, these types of errors are now reported as part of a well-formed response body.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for the reviews, Greg and Yash!


Installation instructions

No installation instructions are present on the readme file.
Can you please add them?

and/or submit it to the atom package list
https://atom.io/packages

I submitted a fork of this to the main atom packages list.
https://atom.io/packages/atom-salt
https://github.com/saltstack-contrib/atom-salt


fix arg name

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+
Oops, thanks!

:pushpin: Commit 5473a23 has been approved by alexcrichton

:hourglass: Testing commit 5473a23 with merge d122a30...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing d122a30d7a27a3fbbcc90a6c0a6861a7d74ee474 to master...


init v3.x.x focus-components

This branch init focus-components v3.x.x
TODO v3.x.x
Components to migrate in components folder
Global tests

[ ] add tests on all components

New components

[x] ajouter chips
[x] ajouter select material
[x] remplacer dropdown
[ ] add this 3 components to the showcase

Application

[ ] Popin
[ ] Confirmation-popin
[ ] Menu

List ?

[ ] add component table
[ ] add component list
[ ] add component timeline
[ ] wrapper scroll infini

search

[ ] quick search ?

Behaviours to rewrite

[ ] Grid
[ ] Material
[ ] Scroll
[ ] Translation

Build

[ ] Epurer la conf webpack
[ ] Revoir les tâches de build dans package.json pour réécrire la tâche avec babelify
[ ] mettre en place les peer dependencies

Showcase

[ ] Rewrite the Showcase and complete it
[ ] Add all missing components

Libs to upgrade

[x] moment
[x] lodash [fixes #1004]
[ ] i18n

2 things done and already 15k lines of code in less

fixes https://github.com/KleeGroup/focus-components/issues/1004

:+1:


[MORPHY] Remove x86 references introduced in #21555

This is causing bundler issues on other architectures:
---> bundle lock --update --conservative --patch
/usr/share/gems/gems/psych-3.1.0/lib/psych.rb:237: warning: already initialized constant Psych::LIBYAML_VERSION
/usr/share/ruby/psych.rb:237: warning: previous definition of LIBYAML_VERSION was here
/usr/share/gems/gems/psych-3.1.0/lib/psych.rb:239: warning: already initialized constant Psych::NOT_GIVEN
/usr/share/ruby/psych.rb:239: warning: previous definition of NOT_GIVEN was here
Fetching https://github.com/ManageIQ/manageiq-v2v
Fetching https://github.com/ManageIQ/manageiq-ui-classic
Fetching https://github.com/ManageIQ/manageiq-decorators
Fetching https://github.com/ManageIQ/manageiq-consumption
Fetching https://github.com/ManageIQ/manageiq-content
Fetching https://github.com/ManageIQ/manageiq-graphql
Fetching https://github.com/ManageIQ/manageiq-api
Fetching https://github.com/ManageIQ/manageiq-automation_engine
Fetching https://github.com/ManageIQ/manageiq-providers-vmware
Fetching https://github.com/ManageIQ/manageiq-providers-scvmm
Fetching https://github.com/ManageIQ/manageiq-providers-ovirt
Fetching https://github.com/ManageIQ/manageiq-providers-openstack
Fetching https://github.com/ManageIQ/manageiq-providers-openshift
Fetching https://github.com/ManageIQ/manageiq-providers-redfish
Fetching https://github.com/ManageIQ/manageiq-providers-oracle_cloud
Fetching https://github.com/ManageIQ/manageiq-providers-nuage
Fetching https://github.com/ManageIQ/manageiq-providers-nsxt
Fetching https://github.com/ManageIQ/manageiq-providers-lenovo
Fetching https://github.com/ManageIQ/manageiq-providers-kubevirt
Fetching https://github.com/ManageIQ/manageiq-providers-kubernetes
Fetching https://github.com/ManageIQ/manageiq-providers-ibm_terraform
Fetching https://github.com/ManageIQ/manageiq-providers-ibm_power_vc
Fetching https://github.com/ManageIQ/manageiq-providers-ibm_cloud
Fetching https://github.com/ManageIQ/manageiq-providers-google
Fetching https://github.com/ManageIQ/manageiq-providers-foreman
Fetching https://github.com/ManageIQ/manageiq-providers-azure_stack
Fetching https://github.com/ManageIQ/manageiq-providers-azure
Fetching https://github.com/ManageIQ/manageiq-providers-autosde
Fetching https://github.com/ManageIQ/manageiq-providers-ansible_tower
Fetching https://github.com/ManageIQ/amazon_ssa_support.git
Fetching https://github.com/ManageIQ/manageiq-providers-amazon
Fetching https://github.com/ManageIQ/manageiq-schema
Fetching https://github.com/ManageIQ/manageiq-gems-pending.git
Fetching source index from https://rubygems.manageiq.org/
Fetching gem metadata from https://rubygems.org/......
Fetching source index from https://rubygems.org/
Resolving dependencies....................................................................................................................................................................................................................
Bundler could not find compatible versions for gem "handsoap":
  In snapshot (Gemfile.lock):
    handsoap (>= <IP_ADDRESS>)

  In Gemfile:
    handsoap (= <IP_ADDRESS>)

    manageiq-smartstate (~> 0.7.0) was resolved to 0.7.0, which depends on
      vmware_web_service (~> 2.0) was resolved to 2.1.1, which depends on
        handsoap (~> 0.2.5)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

Bundler could not find compatible versions for gem "linux_admin":
  In snapshot (Gemfile.lock):
    linux_admin (>= 2.0.2)

  In Gemfile:
    linux_admin (~> 2.0, >= 2.0.1)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

Bundler could not find compatible versions for gem "mini_portile2":
  In snapshot (Gemfile.lock):
    mini_portile2 (= 2.7.1)

  In Gemfile:
    handsoap (= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
      nokogiri (>= 1.2.3) was resolved to 1.12.5, which depends on
        mini_portile2 (~> 2.6.1)

    manageiq-messaging (~> 1.0) was resolved to 1.0.3, which depends on
      rdkafka (~> 0.8) was resolved to 0.10.0, which depends on
        mini_portile2 (~> 2.1)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

Bundler could not find compatible versions for gem "nokogiri":
  In snapshot (Gemfile.lock):
    nokogiri (= 1.12.5)

  In Gemfile:
    rails (~> 6.0.4, >= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
      actiontext (= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
        nokogiri (>= 1.8.5)

    handsoap (= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
      nokogiri (>= 1.2.3)

    linux_admin (~> 2.0, >= 2.0.1) was resolved to 2.0.2, which depends on
      nokogiri (>= 1.8.5, != 1.10.0, != 1.10.1, != 1.10.2, < 2)

    rails (~> 6.0.4, >= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
      actionpack (= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
rails-html-sanitizer (~> 1.0, >= 1.2.0) was resolved to 1.4.2, which
depends on
          loofah (~> 2.3) was resolved to 2.12.0, which depends on
            nokogiri (>= 1.5.9)

    rails (~> 6.0.4, >= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
      actionmailer (= <IP_ADDRESS>) was resolved to <IP_ADDRESS>, which depends on
        rails-dom-testing (~> 2.0) was resolved to 2.0.3, which depends on
          nokogiri (>= 1.6)

    wim_parser (~> 1.0) was resolved to 1.0.1, which depends on
      nokogiri (~> 1.10)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

Bundler could not find compatible versions for gem "wim_parser":
  In snapshot (Gemfile.lock):
    wim_parser (>= 1.0.1)

  In Gemfile:
    wim_parser (~> 1.0)

Running `bundle update` will rebuild your snapshot from scratch, using only
the gems in your Gemfile, which may resolve the conflict.

Nice find!


every version of some_other_library depends on version....

Hi.
I tried to solve the issues by following the articles
https://github.com/justkawal/excel/issues/26
https://github.com/justkawal/excel/issues/25
https://github.com/justkawal/excel/issues/11
but it did not solve the problem
First I get following error at pub get:
Because excel >=1.0.4 <2.0.0-null-safety depends on xml ^4.1.0 and flutter_svg 0.22.0 depends on xml ^5.0.0, excel >=1.0.4 <2.0.0-null-safety is incompatible with flutter_svg 0.22.0. And because no versions of flutter_svg match >0.22.0 <0.23.0, excel >=1.0.4 <2.0.0-null-safety is incompatible with flutter_svg ^0.22.0. So, because wine_region depends on both flutter_svg ^0.22.0 and excel ^1.1.5, version solving failed. pub get failed (1; So, because wine_region depends on both flutter_svg ^0.22.0 and excel ^1.1.5, version solving failed.)
After following the mentoined solutons I get following errors:
Because excel <2.0.0-null-safety depends on archive ^2.0.0 which depends on crypto ^2.0.0, excel <2.0.0-null-safety requires crypto ^2.0.0. And because google_fonts 2.1.0 depends on crypto ^3.0.0 and no versions of google_fonts match >2.1.0 <3.0.0, excel <2.0.0-null-safety is incompatible with google_fonts ^2.1.0. So, because wine_region depends on both google_fonts ^2.1.0 and excel ^1.1.5, version solving failed. pub get failed (1; So, because wine_region depends on both google_fonts ^2.1.0 and excel ^1.1.5, version solving failed.)
So I can't find any combination of libraries that works. Is there any solution available?

anyupdate on this?


Rename Democratic Republic of the Congo

Trello: https://trello.com/c/HOnUP71d/493-democratic-republic-of-the-congo-fixes-medium

Rename the country slug
Setup the redirects to manage the URLs update

This is related to https://github.com/alphagov/whitehall/blob/19cd7d72de32454d532c195f35b027fa1b3ba6ac/db/data_migration/20160505110730_redirect_worldwide_priorities.rb

👍 I will remove the redirects from here and create a PR on router-data

https://github.gds/gds/router-data/pull/574


Add support: Python 3.12-buster

          Released 3.12 for `bookworm` and `bullseye`. Looks like https://hub.docker.com/_/python doesn't have `buster`

Opened https://github.com/docker-library/python/issues/870
Originally posted by @samruddhikhandale in https://github.com/devcontainers/images/issues/750#issuecomment-1747642229

From https://github.com/docker-library/python/issues/870#issuecomment-1756422518

We generally only create python images for the two most recent releases of the distributions that we build for. So, currently that ends up being Debian Bookworm and Bullseye and Alpine 3.18 and 3.17.
Related: https://github.com/docker-library/python/issues/863

As official python images are not adding support for buster, we would follow the same thing for dev container images.


Add documentation to dashboard

From:

Short documentation is needed as part of each preprocessing tool

Originally posted by @ThijsVroegh in https://github.com/DecodingRaphael/unraphael/pull/30#pullrequestreview-2052418332
It might be nice to add some short documentation for each tool in the home page, or on each of the tool pages, how it works. The tooltips can also be used for this, they are currently incomplete.

Documentation has been updated considerably now (see documentation branch); I'll go from there to look what has to be in the streamlit app


FairSeq source build issue (EnvironmentError)

Problem:
When trying to build fairseq from source, it fails at installing backend dependencies. numpy is already installed through miniconda.
Error log:

(base) 1 dutta@exs-91208:~/code/fairseq$ pip install --editable .
Obtaining file:///home/dutta/code/fairseq
Installing build dependencies ... done
Getting requirements to build wheel ... done
Installing backend dependencies ... error
ERROR: Command errored out with exit status 1:
command: /home/dutta/miniconda3/bin/python /home/dutta/miniconda3/lib/python3.7/site-packages/pip install --ignore-installed --no-user --prefix /tmp/pip-build-env-bwj6abba/normal --no-warn-script-location --no-binary :none: --only-binary :none: -i https://pypi.org/simple -- numpy
cwd: None
Complete output (4 lines):
Collecting numpy
Using cached https://files.pythonhosted.org/packages/63/0c/0261693cc3ad8e2b66e66dc2d2676a2cc17d3efb1c58a70db73754320e47/numpy-1.18.1-cp37-cp37m-manylinux1_x86_64.whl
ERROR: Could not install packages due to an EnvironmentError: [Errno 28] No space left on device
ERROR: Command errored out with exit status 1: /home/dutta/miniconda3/bin/python /home/dutta/miniconda3/lib/python3.7/site-packages/pip install --ignore-installed --no-user --prefix /tmp/pip-build-env-bwj6abba/normal --no-warn-script-location --no-binary :none: --only-binary :none: -i https://pypi.org/simple -- numpy Check the logs for full command output.

Error reference:

Expected behavior:
Should work without any errors.
I have checked the issues before reporting. Issue 1652 talks about a similar issue but I think there the user gets an error at a different stage. I also tried to check if its an issue with the TEMP directory, but I think there is enough space in the server, so that should not be a problem.
I have also tried to pull a fresh batch from our repository, still did not work. I have tried to pull from the Fairseq master, not sure if that will solve the problem.
How to reproduce:

git clone fairseq
cd fairseq
pip install --editable .

Environment:
OS: Debian GNU/Linux 8 (jessie)
Python: 3.7.4
PyTorch: 1.4.0
CUDA: cuda10.1.243 cudnn7.6.3_0
Fairseq version: https://github.com/cristinae/fairseq/ (forked)
Fairseq install: through source (git clone)
Build command: pip install --editable . / pip install -e . (tried both)
GPU: 4x nVidia GeForce GTX TITAN (using 2 GPUs out of 4, num_workers=20)
Any help is appreciated.

The issue was actually regarding the tmp folder being full. I had to change the TMPDIR to another location with enough empty space and it worked (Fairseq build from source done successfully).


Add most of the github usernames

I've tried to add all of the usernames, I'm missing just one. If you don't need this pull request for any reason, feel free to disregard it!

Thanks, @ttn-ttn for your PR with the GitHub usernames... however, I have submitted another one. I will close this PR for now. Thanks again!


Prisma null helpers require use of equals property

Bug description
Issue found in this Slack thread.
Our docs say this should be valid:
prisma.log.findMany({
  where: {
    data: {
      meta: Prisma.AnyNull,
    },
  },
})

but it causes a TypeScript error.
The workaround is explicitly using the equals property:
prisma.log.findMany({
  where: {
    data: {
      meta: {
        equals: Prisma.AnyNull
      }
    },
  },
})

How to reproduce
Try to follow the code snippets on our docs page.
Expected behavior
No response
Prisma information
n/a
Environment & setup
Recreated on

OS: macOS Monterey
Database: PostgreSQL
Node.js version: 17.7.1

Prisma Version
3.12

👋 we're cleaning up our backlog of issues and are closing everything older than two years. If you still feel that this issue is relevant, please feel free to re-open and our team will take a look. Thank you!


When converting SArray of strings to float, strings starting with inf will become float('inf')

Hello,
If your str-typed SArray contains only strings starting with 'inf' and you are trying to convert it to float, what you will get instead of a raised exception is an array of the 'infinite' float.
This happened to me while I was going through my columns and using try/catch to see which ones I could convert to float. Not a very common use case, I admit, but still...
$ ipython
Python 2.7.10 (default, Jul 15 2017, 17:16:57)
Type "copyright", "credits" or "license" for more information.

IPython 5.5.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: import turicreate as tc

In [2]: tc.__version__
Out[2]: '4.2'

In [3]: float('inf')  # the expected result
Out[3]: inf

In [4]: tc.SArray(['inf', 'infamous', 'infrequent', 'INFINITE']).astype(float)  # some actual mishaps beside it
Out[4]:
dtype: float
Rows: 4
[inf, inf, inf, inf]

In [5]: tc.SArray(['foo']).astype(float) # what I expect
...traceback omitted...
RuntimeError: Unable to interpret foo as the target type.

This is a side effect of an unfortunate simple direct use of strtod.
More generally, this is occurs for strings prefixed with numbers as well
In [5]: tc.SArray(["2hello","2.3world"]).astype(float)
Out[5]:
dtype: float
Rows: 2
[2.0, 2.3]

In [6]: tc.SArray(["2hello","2.3world"]).astype(int)
Out[6]:
dtype: int
Rows: 2
[2, 2]

@guy4261 Thanks for pointing this out! Fixed by PR #373


Update internal.go

I added Dlrhost and Dlrport services

can you add Closes #11 and change the title


feat(lazy-list) Implement LazyList widget

Allows to initialize list items lazily, this helps a lot with performance in list that contain big set of data.
closes  #387

No problems - I'll close it out for now, but feel free to re-open if you get to it.


Update all golang deps

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Bump all golang deps to their latest release.
Which issue(s) this PR fixes:

None
Does this PR have test?
None

Special notes for your reviewer:
None
Does this PR introduce a user-facing change?

None

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

Project coverage is 41.38%. Comparing base (11d77f4) to head (e0ba90e).
Report is 531 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2604      +/-   ##
==========================================
- Coverage   45.50%   41.38%   -4.12%
==========================================
  Files          79      109      +30
  Lines        7782    18240   +10458
==========================================
+ Hits         3541     7549    +4008
- Misses       4099    10192    +6093
- Partials      142      499     +357

/lgtm


Error running grunt --tasks

grunt -tasks

path.js:360
        throw new TypeError('Arguments to path.join must be strings');
              ^
TypeError: Arguments to path.join must be strings
    at path.js:360:15
    at Array.filter (native)
    at Object.exports.join (path.js:358:36)
    at Object.file.exists (/Users/chrisnicola/src/wealthbar-phone/node_modules/grunt/lib/grunt/file.js:381:28)
    at task.loadTasks (/Users/chrisnicola/src/wealthbar-phone/node_modules/grunt/lib/grunt/task.js:371:18)
    at Array.forEach (native)
    at Task.task.init (/Users/chrisnicola/src/wealthbar-phone/node_modules/grunt/lib/grunt/task.js:457:33)
    at Object.grunt.tasks (/Users/chrisnicola/src/wealthbar-phone/node_modules/grunt/lib/grunt.js:120:8)
    at Object.module.exports [as cli] (/Users/chrisnicola/src/wealthbar-phone/node_modules/grunt/lib/grunt/cli.js:38:9)
    at Object.<anonymous> (/usr/local/Cellar/nvm/0.24.0/v0.10.37/lib/node_modules/grunt-cli/bin/grunt:45:20)

Not sure what the cause of this is, but it is making it impossible for me to debug issues with customizing our build process.

This turned out to be an issue with Grunt itself.


Error "Uncaught ReferenceError: regeneratorRuntime is not defined" when using generators

I'm using 0.2.0, a fresh created project, my code uses generators, the npm start seems to work fine, but once I visit my app in a browser, the console shows this message:

Uncaught ReferenceError: regeneratorRuntime is not defined

I've seen related issues but they seem to be resolved in 0.2.0, so I'm opening a new one here.

Thanks for reporting. I’ll take a look when I get some time.

Thanks for your time.
After further digging, I'm not sure the problem comes from create-react-app. I'm requiring a module already compiled by babel, but with probably wrong parameters. Let me close this one, I'll reopen it if I get the conviction that the problem comes from the lib.


Various places rely on the natur of \space

This came up with the example
\documentclass{article}
\usepackage{listings}
\usepackage{graphicx}
\newcommand{\foobar}[1]{\scalebox{-1}[1]{#1}}
\lstset{breaklines=true,prebreak=\foobar{A}}
\begin{document}
\begin{lstlisting}
some very very very very very very very very very very very very very very very very very very very long line
\end{lstlisting}
\end{document}

which gives a fatal error at present as \space is expanding to \kern \lst@width. Older versions of pdftex.def used \ltx@space (from HOs stack) to avoid this. However, that wasn't universal so depending on the driver chosen the above would still fail.
Arguably no-one should be messing with \space (oh for \c_space_tl), but we may have to globally sort this. Perhaps something in the kernel in addition to in the drivers?

Am 20.06.17 um 17:22 schrieb Joseph Wright:

Arguably no-one should be messing with |\space| (oh for |\c_space_tl|),
but we may have to globally sort this. Perhaps something in the kernel
in addition to in the drivers?

I agree, nobody should mess with |\c_space_tl| but |\space| is not the
same. It is in effect a user level command and as such representing
whatever in that situation represents a space. So I find it
understandable that something like |lstlisting| does change it.
So perhaps a real space constant would be in order for the kernel (as
well as changing the kernel use of |\space| in various places to that
constant.

It's probably worth noting that this only comes up in the example has the \space we need is for writing a special and listings has activated it's redefinition locally. I'm not sure there are many other places you'd run into this, at least as long as \space is only set locally (which is likely why it's not come up more widely). I guess Heiko might have seen it before, hence originally using \ltx@space ...


[Question] Function signature formatting

Given text,
function
foo
()
{
}

outputs is as follows
function
foo () {}

Desired output is:
function foo() {}

I iterated all the currently supported options but the desired option seems not provided yet. Is there a consideration for function signature formatting?
Thanks.

For the above input the output I get is
function
foo() {}

There is a space-after-named-function option which is off by default. Please try you input on https://beautifier.io and see what happens.


Satellite layer option is missing

Needs refactoring of GL code in task/index.js to allow re-rendering.