Using Objects for Lookups - Need Instructions Clarification

Challenge Using Objects for Lookups has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/48.0.2564.97 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
The instructions say:

Convert the switch statement into a lookup table called lookup. Use it to lookup val and return the associated string.

(Emphasis added)
Which leads to this solution:
// Setup
function phoneticLookup(val) {
  var result = "";

  // Only change code below this line
 var lookup = {
    alpha: "Adams",
    bravo: "Boston",
    charlie: "Chicago",
    delta: "Denver",
    echo: "Easy",
    foxtrot: "Frank"
  };

  // Only change code above this line
  return lookup[val];  // <-----   They modified this line
}

I think the confusion is that the instructions say to return the associated string.
They should probably say:

Convert the switch statement into a lookup table called lookup. Use it to lookup val and assign the associated string to the result variable.

Ref:  #6453

You should remove the** result** declaration as the accepted result doesn't include it and the instruction should not include "and assign the associated string to the result variable".

@jundc1006 If you want to open a new issue with your comment we can see about making an adjustment.  Closed issues do not get new actions.

This is how I solved it:
var lookup = {
"alpha":"Adams",
"bravo":"Boston",
"charlie":"Chicago",
"delta":"Denver",
"echo":"Easy",
"foxtrot":"Frank"
};
result=lookup[val];

This shouldn't be closed.  The solution requires the user to modify the code below the line
//only change code above this line
to say
result = lookup[val];
when the original code reads,
return result;

var lookup = {
"alpha":"Adams",
"bravo":"Boston",
"charlie":"Chicago",
"delta":"Denver",
"echo":"Easy",
"foxtrot":"Frank"
};
result=lookup[val]; Is not working.I'm not sure what is wrong

var lookup = {
"alpha": "Adams",
"bravo": "Boston",
"charlie": "Chicago",
"delta": "Denver",
"echo": "Easy",
"foxtrot": "Frank"
};
var val = "bravo";
val = lookup[val];
On Mon, Jul 4, 2016 at 2:50 PM, Vuyi<EMAIL_ADDRESS>wrote:

var lookup = {
"alpha":"Adams",
"bravo":"Boston",
"charlie":"Chicago",
"delta":"Denver",
"echo":"Easy",
"foxtrot":"Frank"
};
result=lookup[val]; Is not working.I'm not sure what is wrong
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6657#issuecomment-230250948,
or mute the thread
https://github.com/notifications/unsubscribe/ARyrurdlVQZpsi2Kdlr9B7MC9b47NZWzks5qSNdKgaJpZM4HQ4fF
.

hi can anybody help me with this problem I encounter?
what would be the proper way to solve this.
thank you

you need to add:
result = lookup[val];
under your var lookup :)

Need to figure it out, use a desktop coding IDE and here is an example of how I wrote it too. PS. It includes the console.log at the bottom.
`
// Setup
function phoneticLookup(val) {
var result; // also tweaked the var result too. No need for this "" stuff.
// Only change code below this line
var lookup = {
"alpha": "Adams",
"bravo": "Boston",
"charlie": "Chicago",
"delta": "Denver",
"echo": "Easy",
"foxtrot": "Frank"
};
// Only change code above this line - below is how I did it. Some people tweaked above the line,
// but that seems overly codie for me
return result = lookup[val];
}
// Change this value to test - I add console.log sometimes to help figure stuff out.
console.log(phoneticLookup("charlie"));
`
@cflamenco perhaps this is helpful to see how I did it, besides the answer above mine.
Happy Camping everyone!!

"Convert the switch statement into a lookup table called lookup" should be: "Convert the switch statement into a lookup variable".
Suddenly your talking about lookups and tables. Took me 20 mins to find out a lookup-table is nothing more than a variable named lookup.

It took me a while to figure out as well. Based on the instructions alone, I had no idea I had to add:
result=lookup[val]
Glad I'm not the only one who had this issue!

var lookup = {
"alpha": "Adams",
"bravo": "Boston",
"charlie": "Chicago",
"delta": "Denver",
"echo": "Easy",
"foxtrot": "Frank"
};
return lookup[val]
}
phoneticLookup("charlie");
this run with me

wouldnt have gotten past this unless I google-fu'd it and came here.  Ty all

@cflamenco  you need to assign lookup[val] to result

do you still need help with the code above?

No thank you x
Sent from my iPhone

On 30 Sep 2016, at 04:42, Lydie Toussaint<EMAIL_ADDRESS>wrote:
do you still need help with the code above?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

I see an editing adjustment which would help to avoid confusion in Using Objects for Lookups.
The directions correctly instruct the user "... assign the associated string to the result variable."
However, Line 27 states the following and the result variable is on Line 28.
// Only change code above this line
return result;
}
Since it is necessary to alter Line 28 to run the code correctly, this is confusing.
Just passing along an observation.

Why is this not corrected. I almost start to cry trying to follow the  instructions in code
// Only change code below this line
// Only change code above this line
:D
please help us learn

Heya, thanks for the help with this. I have a question regarding the layout of this challenge. What are the advantages of putting an object in a function? Could we not just return the requested property without the function?

Agreed pezz80.
// Only change code below this line comment is limiting

why is val in square braces? ie.
return lookup[val];

I had a problem with it too, no instrustions for that ... : /
yet return lookup[val]; worked , still a bit dunno how

@Romanek5 I'm glad i'm not the only one!

@mtpjr88 :D next few are probably bugged too :). maybe created by same person ?

// Setup
function phoneticLookup(val) {
var result = "";
// Only change code below this line
var lookup = {
"alpha":"Adams",
"bravo":"Boston",
"charlie":"Chicago",
"delta":"Denver",
"echo":"Easy",
"foxtrot":"Frank"
};
// Only change code above this line
result = lookup[val];
return result;
}
// Change this value to test
phoneticLookup("charlie");

This worked for me:
// Setup
function phoneticLookup(val) {
var result = "";
// Only change code below this line
var lookup = {
"alpha":"Adams",
"bravo":"Boston",
"charlie":"Chicago",
"delta":"Denver",
"echo":"Easy",
"foxtrot":"Frank",
"":undefined
};
result = lookup[val];
// Only change code above this line
return result;
}
// Change this value to test
phoneticLookup("charlie");

Thanks for the explanation. It has helped a lot. Cheers,

The instructions are a bit annoying and not very clear. Takes awhile to get those as we approach the problem.

Thank you! I NEVER would've gotten this. I'm working in beta too and it hasn't been changed.

this lesson makes zero sense

Thank you so much. I'm finding the wording of this section on objects far more confusing than it should be.

I would have never thought to change the return line since it is below the
" // Only change code above this line"

I was right! Yaaaaaaaaaaay!

THANK GOODNESS! Thank you!! Thought I was going crazy.


Yahoo Mail UTF-8 characters not rendering in HTML preview

I'm sending a mail template and it's not rendering UTF-8 characters correctly only on Yahoo Mail, which is quite weird.
On GMail it's rendering fine.
MJML is adding by default utf-8 charset tag, I have not changed this default, and it's correctly appearing in the source code of the mail.
Do you have an idea of what could be wrong?

MJML environment (please complete the following information):

I'm rendering through mrml, I have no ideas if the issue could come from this library. If you think it could come from it, please tell me.

Email sending environment(for rendering issues):

AWS SES

Affected email clients (for rendering issues):

Yahoo Mail
Browser : all

The issue was caused by my client client who was not correctly sending the HTML header.
I managed to send Content-Type: text/html; charset=UTF-8 and it's now working fine in Yahoo Mail 👍🏻


Elaborate on "Settings"

Any chance of more detail in this Section?  https://docs.microsoft.com/en-us/microsoft-365/security/defender-endpoint/prevent-changes-to-security-settings-with-tamper-protection?view=o365-worldwide#what-happens-if-i-try-to-change-microsoft-defender-for-endpoint-settings-in-intune-microsoft-endpoint-configuration-manager-and-windows-management-instrumentation-when-tamper-protection-is-enabled-on-a-device
It just says "Settings", can you provide more detail on what you can and can't change when Tamper Protection is on?  The way this reads is you can't change any AV / Defender settings when its's on?  Like I can't add a file path or file Exclusion in my DefenderAV GPO or Intune Policy?  I can't add a Add-MPPreference via Powershell on a device? (i.e Add-MpPreference -ExclusionExtension ".mdf")
It would be good to make it clearer....

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

ID: f31e7b39-b7c3-188a-0b2a-02ff10a5be6d
Version Independent ID: 44940a13-0a2d-e281-560b-cb4d8a5d1ccf
Content: Protect security settings with tamper protection
Content Source: microsoft-365/security/defender-endpoint/prevent-changes-to-security-settings-with-tamper-protection.md
Product: m365-security
Technology: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@denisebmsft Please help us in resolving this issue. Thanks

Hello @mattsmacblog. Thank you so much for posting your question here. I can see how that section was confusing, and I have updated the article accordingly. If you're using Intune to manage tamper protection, you can make changes, such as applying tamper protection to some or all devices, or defining exclusions using Intune. What you can't do is switch back and forth between managing tamper protection with Intune and some other method like Group Policy or editing settings on the local device. I have updated the article and the changes will be live later today. I hope it's more clear. Please let me know if you have any other questions. Thanks again!


How to use a model block in a build.gradle.kts

I am in the process of converting my build.gradle files to Kotlin build.gradle.kts files. A number of my build files contain model blocks. I cannot find a way to code these in Kotlin and am concerned that this is a Groovy-only construct. Is there a Kotlin equivalent to the model block?

Hi @baron1405, model blocks won't be supported in gradle-script-kotlin 1.0.
It should be possible to convert model blocks into RuleSource classes and apply them explicitly though.
We will consider adding a sample on the next milestone.

Thanks @bamboo for the update and suggestion.

Anyone can provide info on how to use RuleSource?

@baron1405, @ikzk, please have a look at the model-rules sample to see how to apply rule sources in a .gradle.kts script: https://github.com/gradle/kotlin-dsl/tree/develop/samples/model-rules

The model block won't be supported but we now we have a sample on how to write model rules.
Closing as Delivered.

It there a way to specify play version with this approach? All docs have only this example:
model {
    components {
        play {
            platform play: '2.5.8', scala: '2.11'
            injectedRoutesGenerator = true
        }
    }
}

Thanks.


What is the difference from devspace?

Interesting project! How it is different from devSpace, builtin registry? better workflow?

This is a great question. We are going to do one or more blog diving deeper into the differences between this and other tools.
They do both aim to ease the development workflow related to Kuberentes, but diverge quickly beyond that.
 In their own words devSpace is "kubectl on steroids". This gives you access to the entire k8s API space, which offers maximum flexibility, but also maximum complexity.
With Acorn, users don't need any Kubernetes knowledge. It is an abstraction layer on top of Kubernetes with its own application-level constructs. We are curating our users' experience to make it less complex.


New player list won't hide

I am unable to hide the new player list using the SetCoreGuiEnabled method in StarterGui.
Reproduction steps:

Open Studio with a new place.
Insert a local script into StarterGui with the following code:

game.StarterGui:SetCoreGuiEnabled("PlayerList",false)

Play solo or use test servers. The player list doesn't hide.

Was able to reproduce in play solo but not start server->start player. Looks to be a race condition between when a script sets it to false and when the core script connects the event. Will put a fix in later today.


Multi part form-data file uploading is getting failed in API Manager

Description:
Multi-part form-data file uploading is getting failed in API Manager when the message is mediated due to following reasons:

File content is base64 encoded.
Filename="sample1.txt"[\r][\n]" attribute is missing.

Suggested Labels:

Type/Bug
Suggested Assignees:

Affected Product Version:
APIM 1.10.0
OS, DB, other environment details and versions:
Any OS, Any DB
Steps to reproduce:

Create a REST API with mock backend.
Add the below sequence as in-sequence for the created API.

<?xml version="1.0" encoding="UTF-8"?>
<sequence xmlns="http://ws.apache.org/ns/synapse"
          name="admin--SR-Document:v1.0--In"
          trace="disable">
   <property xmlns:soapenv="http://www.w3.org/2003/05/soap-envelope"
             xmlns:ns3="http://org.apache.synapse/xsd"
             name="accept"
             expression="$trp:accept"
             scope="default"
             type="STRING"/>
   <header name="Cookie" scope="transport" action="remove"/>
   <property name="Authorization" scope="transport" action="remove"/>
   <property xmlns:soapenv="http://www.w3.org/2003/05/soap-envelope"
             xmlns:ns3="http://org.apache.synapse/xsd"
             name="httpMethod"
             expression="$axis2:HTTP_METHOD"
             scope="default"
             type="STRING"/>
   <property xmlns:soapenv="http://www.w3.org/2003/05/soap-envelope"
             xmlns:ns3="http://org.apache.synapse/xsd"
             name="REST_URL_POSTFIX"
             expression="concat('/Document/Tags','?id=test')"
             scope="axis2"
             type="STRING"/>
</sequence>

Enable wire logs and invoke the API with multi-part form data message with a file attachment.
The outgoing message from APIM to backend will be changed like following.

File content is base64 encoded.
Filename="sample1.txt"[\r][\n]" attribute is missing.

Related Issues:

Fixed with https://github.com/wso2-support/wso2-axis2/pull/39 and https://github.com/wso2-support/wso2-synapse/pull/234.


add minimum issue date to the metadata with some tests(merge to dev)

Add min-issue(the earliest date that any data was issued for that particular data source) as a new query of metadata in file(acquisition/covidcast/database.py) and add test to verify the min issue shows the earlier date.
Change the SQL queries by adding the corresponding column name such as (id, source, signal, time_type, etc...)  between 'covidcast' and 'values' to make it explicit, and capitalize all the SQL keywords

Would you update the PR description summarizing what this PR changes? That way if we ever need to go back and find this PR again we'll know what we're looking at

this commit ( https://github.com/cmu-delphi/delphi-epidata/commit/14999a3a7b94c6ae7a52ac9dc1cdea1904d2aa73 ) looks great, but it should probably be part of its own PR

Whoops, that's my mistake, that should not have been possible. Fixed now for future PRs.

Whoops, that's my mistake, that should not have been possible. Fixed now for future PRs.

Sorry Katie, it's my mistake, I just push the SQL instance part to my fork respo in order to chat with Tara and made some modified with her. After we changes together, now we have a new version of the SQL instance, can I reopen this pr and update the part to new version?

Would you update the PR description summarizing what this PR changes? That way if we ever need to go back and find this PR again we'll know what we're looking at

Ok I already update the top comment to summarize the changes of this PR.

Summary is good! thanks.

can I reopen this pr

Alas, no -- GitHub doesn't support amending PRs which have already been merged. To modify the SQL queries further, open a new PR.

This PR failed to build once merged with a pile of test failures; probably due to untested SQL query modifications. I'm backing it out so we can release epidata this morning.
Next steps for @zhuoran-Cheng16, once the release is complete:

make a new branch from cmu-delphi:dev
cherry-pick everything except 14999a3
submit a new PR (& use a branch, not a fork, so that any failing CI tests block merge)


Enable SLF4J to Log4J2 bridge for 6.x

This should enable Kafka logging in Log4J2 now, which was broken since the switch to Log4J2.  Also README now clarifies how to set the Kafka log level in the Log4j2 properties.

Code looks good, but I haven't tested yet.

Confirmed with this code Kafka outputs more logging, however, setting the log level seems to have no effect:
logger.kafka.name=org.apache.kafka
logger.kafka.appenderRef.console.ref=console
logger.kafka.level=warn

Kafka still outputs INFO logs to console.

I admin I hadn't tested this to make logs less verbose (i.e. WARN), but it worked for me to make them more verbose (i.e. DEBUG).   I can see if I can replicate.

@svrc-pivotal I tried debug initially, but I didn't see any extra logging (no [DEBUG] in the output), so I tried warn instead.
I am only testing startup (output starts up, does nothing else) though, not message processing.

Oh yes, you'd only really notice this at messaging time as the Kafka
classes won't get touched.   I notice that on startup everything is always
locked to INFO no matter what I did...
Anyway, I'll see if I can reproduce this with a script with actual message
flow for you with WARN and DEBUG.
Stu
On Mon, Oct 2, 2017 at 3:07 PM, Jordan Sissel<EMAIL_ADDRESS>wrote:

@svrc-pivotal https://github.com/svrc-pivotal I tried debug initially,
but I didn't see any extra logging (no [DEBUG] in the output), so I tried
warn instead.
I am only testing startup (output starts up, does nothing else) though,
not message processing.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/logstash-plugins/logstash-output-kafka/pull/134#issuecomment-333665473,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKJcr8aXxQ0l23q67MoC6pxU7GTQQhaoks5soVCVgaJpZM4NcyhB
.

--
Stuart Charlton
Pivotal Software | Platform Architecture
Mobile:<PHONE_NUMBER> | Email<EMAIL_ADDRESS>


feat(): move giphy watermark image to CSS

This removes image import from the React SDK.

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Cannot revert file via command palette

Issue Type
Bug
Description

Open a file.
Edit the file without saving.
Open the command palette (ctrl+shift+p) and select the "File: Revert File" command.

Result: The unsaved file edits are not discarded. Note that this same operation succeeds when performed via the application menu (File > Revert File).


0.000000000000000000000000000000000000000000000000000000000000000000000000000000034%

the chances of this script actually guessing the token correctly is 1 in<PHONE_NUMBER>0000000000000000000000000000000000000000000000000000000000000000000000
or
0.0000000000000000000000000000000000000000000000000000000000000000000000000000034%

generate_random_string has 64 possible characters, the total amount of random characters in the token is 44 (6+38)
64^44=the big number beginning with 296
1/big number*100 gets you the percentage

https://github.com/xHERMIS/ID-to-TOKEN?tab=readme-ov-file#-important

The readme is still wrong, as it's missing a lot of zeroes. It's off by a few orders of magnitude.

and i thought the discord nitro generator and checker was stupid...

The readme is still wrong, as it's missing a lot of zeroes. It's off by a few orders of magnitude.

#18 someone had to do it


feat: assist to remove unneeded asyncs

This should fix #11392
This PR adds a quickfix on functions marked with async that suggests, if and only if no await expression in find in the body of the function (it relies on SyntaxNode::descendants() to recursively traverse blocks), to remove the async keyword.
The lint is made so that it triggers only if the cursor is not in the body of the function, so that it does not pollute the quickfix list.
It does not trigger a diagnostic. I don't know if this repository is the place to implement those (clippy?). If it is, I would very much like pointers on where to start looking.
If there are test cases I haven't thought about, please do suggest.

Could this remove .await from all callers too?

I don't see why it couldn't. I'll work on it, if not tonight (European time), then tomorrow.

The why could be that I have no idea to search for uses for the function under the cursor. I've tried looking at how the rename assist does it, but I'm completely lost.
I've set up a test case with a function and 2 function calls, but calls to Semantics::find_nodes_at_offset_with_descend doesn't seem to magically retrieve all occurrences of the function. It's my first time browsing through the source code of the project, and I would very much like a pointer to where I should start looking.

You want to take a look at this line here https://github.com/rust-analyzer/rust-analyzer/blob/fb9e66d9fc000da157eaf11e8157848279c70c73/crates/ide/src/rename.rs#L254
That is what is retrieving usages

Thanks a lot for the quick reply! I'll have a first try at it right now!

I have extended the tests to cover the removing of .awaits on calls of the function. I am not a language lawyer and cases like f1(f2().await).await were the only corner cases I thought of.
I am very unsure of my use of find_nodes_at_offset_with_descend to find the function definition in find_definition which is a pruned copy-paste of the one in rename.rs.

bors r+

Thanks a lot @Veykril for your review, pointers and patience. It has been a pleasure!

https://user-images.githubusercontent.com/308347/161477319-ba66afa6-c532-49e2-9c02-905c28c18d0b.mp4


Discard draft warning really means delete draft

Describe the Bug
Editing a draft that is older than the current version of the page produces this warning:

This page has been updated since this draft was created. It is recommended that you discard this draft or take care not to overwrite any page changes.

It seems that this should say "recommended that you delete this draft." Or perhaps something is wrong with the discard draft action.
Steps to Reproduce

Go to a page and start a draft but do not publish.
Have someone else edit the page and publish the edit.
Resume editing your draft. The warning appears.
Click Discard Draft. The page reloads but the warning does not go away.
Click Delete Draft. The page reloads and the warning goes away.

Expected Behaviour
I would expect that clicking Discard Draft would resolve the warning about needing to discard the draft.
Screenshots or Additional Context

Browser Details
Firefox 126.0 on Windows 10
Exact BookStack Version
v24.05.1

Hi @WyattBest,

Click Discard Draft. The page reloads but the warning does not go away.

This indicates there may be another issue in your environment. The page should not reload when you click to discard or delete a draft. Do you mean this as a browser page reload, or by "reload" are you referencing the bookstack editor just loading in new content?

It seems that this should say "recommended that you delete this draft."

Although this could be delete, I would prefer to keep this discard as a safer option since the old draft content will exist until a new draft is saved (or auto-saved) which provides a little safety in the event that this was a mistake. This does mean that the old content and warning may re-appear if the user re-edits without having saved a draft since, but that should be infrequent and a reasonable tradeoff IMO.

@ssddanbrown Thank you for responding! You're right; the page does not actual reload. Everything works as you described.
My last remaining confusion stems from the z-index of the notification bubbles. Discarding the draft gives a green success message that it's OK to edit, but it's hidden behind the orange warning. Perhaps notifications should have the newest on top?

@WyattBest Yeah, we should probably dismiss existing warnings when that action is taken to avoid the overlap.
I'll go ahead and re-purpose this issue to focus on that as an improvement.


Running and debugging through Xcode

Hey @AndrewBarba, thank you for this fantastic package, I believe you are on the path to revolutionising serverless Swift!
I've followed your youtube video which was a great resource in getting started. I couldn't help notice that one of the great things about Vapor is being able to run and debug (i.e. triggering breakpoints as a request comes in) through Xcode. Do you see feasibility of this package/serverless to be able to do that?
P.S. I love the direction you're going in with swift.cloud, I'm a big advocate for it!

Yes this is actually supported already! But I added support after I made that video. Just run:
swift package --disable-sandbox vercel server

And that will start the proxy server locally and nothing else. Then you can just click "Run" in Xcode and you will get full breakpoint support and can easily stop and restart and Xcode without stopping the proxy server.

Thanks @AndrewBarba! That's worked perfectly 🙏


Add maxGrade in UserInfo for submitted submissions

Fixes #205

Pull Request Test Coverage Report for Build 1451

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 99.723%

Totals

Change from base Build 1449:
0.003%

Covered Lines:
720

Relevant Lines:
722

💛  - Coveralls


Fix domain discernment for epicbox addresses

-This PR resolves the localhost websockets SSL error below:
ERROR ws::handler] WS Error <Protocol>: Unable to parse domain from wss://[<IP_ADDRESS>/](https://<IP_ADDRESS>/). Needed for SSL

For some reason, our 'to_address' was not properly including our domain, nor port, and default behavior is to substitute <IP_ADDRESS>:443
Since ws requires a domain name for SSL validation (not an IP), this was causing an error and subsequent unresponsive hang behavior
Following this error, 'epoll' function causes epicbox to hang indefinitely because it is called with a timeout=-1. This commit does not resolve the timeout problem. Only domain discernment is addressed here.

Will follow up with more information on the epoll issue, as I believe it can be resolved by adding a default timeout to post_slate_federated call (timeout is an optional argument to function, as written)

Just a note that there is probably a cleaner way to handle this.
Will investigate as we fix port discernment in future


Port Noise_model.from_dict()

Summary
Details and comments
Fixes #

I'm also a bit concerned that we'd essentially be using an Aer <0.15 from_dict() but >0.15 to_dict(). So we should have a test case ensuring this combination works, if we don't already have one.

I'm also a bit concerned that we'd essentially be using an Aer <0.15 from_dict() but >0.15 to_dict(). So we should have a test case ensuring this combination works, if we don't already have one.

We have a test for that here


Bug with KnowledgeRetention

https://docs.confident-ai.com/docs/metrics-knowledge-retention - when i run the code present in this link i get the following error - can someone help
AttributeError                            Traceback (most recent call last)
/var/folders/_c/snkdtcs90vd2rn1tkt1yqty00000gn/T/ipykernel_72497/2832353720.py in <module>
      5 metric = KnowledgeRetentionMetric(threshold=0.5,model=azure_openai)
      6
----> 7 metric.measure(test_case)
      8 print(metric.score)
      9 print(metric.reason)

~/opt/anaconda3/lib/python3.9/site-packages/deepeval/metrics/knowledge_retention/knowledge_retention.py in measure(self, test_case)
     41
     42     def measure(self, test_case: ConversationalTestCase):
---> 43         if len(test_case.messages) == 0:
     44             raise ValueError("Messages cannot be empty")
     45

AttributeError: 'LLMTestCase' object has no attribute 'messages'

@shaleenrajkaur Check the docs again, just updated it. I'm guessing the names of 'test_case' and 'conversational_test_case' got mixed up. The error message indicates you previously defined a test case of type LLMTestCase and passed it into this metric.


@nestjs/core vulnerable to Information Exposure via StreamableFile pipe

Context

Moderate issue in the the following repos:

Will hold off on this because it will require upgrading to nest js 9

This has been resolved.


thanos storegw only giving out data of ~2 days in the past

Thanos, Prometheus and Golang version used
Thanos: v0.6.0/thanos-0.6.0.linux-amd64
Prometheus: v2.11.1/prometheus-2.11.1.linux-amd64

Output of "thanos --version" or docker image:tag used.
Thanos, version 0.6.0 (branch: HEAD, revision: c70b80eb83e52f5013ed5ffab72464989c132883)
build user:       root@fd15c8276e5c
build date:       20190718-11:11:58
go version:       go1.12.5

What happened
At the moment when having the thanos storegw activated it only gives me about ~2 Days of time series in the past. I know that the time series come from the minio bucket and therefore the storegw because for testing purposes I shut down the sidecar momentarily. When activating both the sidecar and the store gateway, the same thing happens (But of course I always have the most recent data directly from the prometheus server)
What you expected to happen
I expect the storegw to give me more than 2 days of past data. The data should be there since this setup is now running like this for > 2 weeks
How to reproduce it (as minimally and precisely as possible):
Setup the full stack with sidecar, store gateway, Object storage and Query frontend. After collecting metrics for more some time you will see metrics "dissapear" / not showing anymore.
Full logs to relevant components

Thanos Store output:
level=info ts=2019-08-21T14:53:49.731579577Z caller=main.go:154 msg="Tracing will be disabled"
level=info ts=2019-08-21T14:53:49.73173575Z caller=factory.go:39 msg="loading bucket configuration"
level=info ts=2019-08-21T14:53:49.776982503Z caller=cache.go:172 msg="created index cache" maxItemSizeBytes=131072000 maxSizeBytes=262144000 maxItems=math.MaxInt64
level=info ts=2019-08-21T14:53:49.926359639Z caller=main.go:274 msg="disabled TLS, key and cert must be set to enable"
level=info ts=2019-08-21T14:53:49.926458006Z caller=store.go:191 msg="starting store node"
level=info ts=2019-08-21T14:53:49.926561031Z caller=store.go:181 msg="Listening for StoreAPI gRPC" address=<IP_ADDRESS>:19090
level=info ts=2019-08-21T14:53:49.926608914Z caller=main.go:326 msg="Listening for metrics" address=<IP_ADDRESS>:19191

Thanos Sidecar output:
level=info ts=2019-08-21T14:53:50.486072394Z caller=main.go:154 msg="Tracing will be disabled"
level=info ts=2019-08-21T14:53:50.486411196Z caller=main.go:274 component=sidecar msg="disabled TLS, key and cert must be set to enable"
level=info ts=2019-08-21T14:53:50.486450643Z caller=factory.go:39 msg="loading bucket configuration"
level=info ts=2019-08-21T14:53:50.526334208Z caller=sidecar.go:289 msg="starting sidecar"
level=info ts=2019-08-21T14:53:50.526427205Z caller=sidecar.go:222 component=sidecar msg="Listening for StoreAPI gRPC" address=<IP_ADDRESS>:19090
level=info ts=2019-08-21T14:53:50.526519499Z caller=reloader.go:154 component=reloader msg="started watching config file and non-recursively rule dirs for changes" cfg= out= dirs=
level=info ts=2019-08-21T14:53:50.526483075Z caller=main.go:326 msg="Listening for metrics" address=<IP_ADDRESS>:19191
level=warn ts=2019-08-21T14:53:52.529133677Z caller=sidecar.go:322 msg="found that TSDB block time is not 2h. Only 2h block time is recommended." block-time=5m
level=info ts=2019-08-21T14:53:52.531561111Z caller=sidecar.go:154 msg="successfully loaded prometheus external labels" external_labels="{monitor=\"Private Metrics\",region=\"EU-DE-HETZNER\",replica=\"A\"}"
level=info ts=2019-08-21T14:54:24.573540643Z caller=shipper.go:349 msg="upload new block" id=01DJTAQ37QGA0TYRG9S3E14XQF
level=info ts=2019-08-21T14:57:54.560015499Z caller=shipper.go:349 msg="upload new block" id=01DJTAXK64SK1XCJ2WVCCM99SY
level=info ts=2019-08-21T15:02:54.712518586Z caller=shipper.go:349 msg="upload new block" id=01DJTB6R55CE0VMFT7TB0B42NS

Could you provide your flags of thanos components? And did you set up the compact component?

Could you provide your flags of thanos components? And did you set up the compact component?

I am indeed using the compact component in my setup. Components with command line flags coming up:
Note: This is running in a docker-compose environment. Therefore, the components can reach each other via their hostnames (This works so far as the thanos query frontend correctly shows the components as connected and scrapable)
Prometheus
/usr/bin/prometheus --web.listen-address="<IP_ADDRESS>:9090" --config.file="/etc/prometheus/prometheus.yml" --storage.tsdb.min-block-duration="5m" --storage.tsdb.max-block-duration="5m" --storage.tsdb.retention=2d --query.timeout=2m --web.enable-admin-api --storage.tsdb.path="/opt/prometheus_data/"

Thanos Sidecar
 /usr/bin/thanos sidecar --prometheus.url "http://prom_private11_prometheus:9090" --objstore.config-file /opt/s3.yml --grpc-address <IP_ADDRESS>:19090 --http-address <IP_ADDRESS>:19191 --tsdb.path /opt/prometheus_data

Thanos Query
/usr/bin/thanos query --http-address <IP_ADDRESS>:19191 --grpc-address <IP_ADDRESS>:19090 --store prom_private11_sidecar:19090 --store prom_private11_storegw:19090 --query.replica-label replica

Thanos store
/usr/bin/thanos store --data-dir /var/thanos/store --http-address <IP_ADDRESS>:19191 --grpc-address <IP_ADDRESS>:19090 --objstore.config-file /opt/s3.yml

Thanos compact
/usr/bin/thanos compact -w --retention.resolution-1h=0d --retention.resolution-5m=20d --retention.resolution-raw=1d --consistency-delay=30m --data-dir /var/thanos/compact --http-address <IP_ADDRESS>:19191 --objstore.config-file /opt/s3.yml

@storedcc Thank you for your useful information. According to your flags and logs, I'm strongly recommend to set your prometheus flags --storage.tsdb.max-block-duration and --storage.tsdb.min-block-duration to 2h. It is recommended by thanos.

Setting --storage.tsdb.max-block-duration="5m" and --storage.tsdb.min-block-duration="5m" are not something we support in Thanos - there are many race condidtions that can happen. As @jojohappy mentioned it's advised to leave it to 2h


Add correct ASDoc

Please add ASDocs to library classes to be able to explore the functionality without classes examination.

This is a great request, but it would require time that none of us contributors currently have. I am sorry. Hopefully, this will get picked up by someone else in the future.


Re-running pipeline with adjusted completeness/contamination thresholds?

Hello,
Is there an easy way to re-run the pipeline with adjusted completeness, contamination, and max contigs/bin cut-offs without completely starting over at the first steps?
For example, after running the pipeline with default parameters, I'd like to evaluate bins with relaxed completeness/contamination thresholds.
Does that require an entirely new run? Or can I adjust parameters in the config.yaml and have the pipeline re-run only the downstream steps that would be affected by the adjustments?
Thanks,
Kim

Hi @kkowalden ,
Currently the easiest way to complete this task is to move all folders starting with 6-checkm-summary out of the snakemake directory to trigger a re-run of those steps. If you update the config file with different settings, those will be used to run the CheckM summary step, GTDB-Tk, and summary steps with the updated settings.
I can work on amending the names of those directories to include the completeness/contamination/contig values. This would allow the user to update these settings and re-run to produce the outputs in the same directory as a previously completed run.

Hi @dportik ,
I'm testing the pipeline now with the updated config file settings.
One observation thus far: it doesn't appear that the final plots generated are using the modified parameters for the XY axes.
For example, if I adjust the config file to: "min_completeness: 50", the final plots describing completeness retain "70" as the left-most tick mark on the X axis.
Kim

Hi @kkowalden,
It looks like those axes were accidentally fixed to the default values. I went ahead and updated the snakefile and plotting script so it should work now.
Please let me know if this fixes your issue. I am planning on working on the directory naming next.

Hi @dportik,
I updated the snakefile and plotting script (and my config.yaml file) in the pipeline directory and started the run from the very beginning.
Unfortunately, the final plots still retain the default "min_completeness: 70" for the X axis left-most tick mark.
Is there another script or file that needs to be updated to over-ride the final summary plot values?
Kim

I'm also now observing fatal errors in pplacer/gtdbtk after modifying the "min_completeness" parameter to 50.
I don't think it is a memory issue, but let me know if you have seen this before.
==> Step 1 of 9: Starting pplacer.Uncaught exception: Sys_error("/home/groups/hpcbio/data/GTDB/R207_v2/release207_v2/split/high/pplacer/gtdbtk_package_backbone.refpkg: No such file or directory")

Hi @kkowalden,
I would download all of the files in the scripts/ directory - there are two plotting scripts that are used along the way. It's possible you only grabbed one of them?
I'm not sure what that error is related to - I haven't seen it yet. Let me know if it persists.

Hi @dportik ,
The most recent run with the updated files finished completely and correctly.
Thanks for your help!
Kim

Hi @kkowalden,
Great! Thanks for letting me know - I'll go ahead and close this issue.


add ssl-config to community build (needed by akka-http)

needed by akka-http, which I'm trying to build by adding "akka-http-core", "akka-http-tests" to extra.projects in the "akka" entry in common.conf

first hurdle: extraction was failing with
https://scala-ci.typesafe.com/job/scala-2.12.x-integrate-community-build/604/console
[ssl-config:error] com.fasterxml.jackson.databind.JsonMappingException: Required property 'version' is missing.
[ssl-config:error] {}
[ssl-config:error]  ^
[ssl-config:error] 	at com.typesafe.dbuild.model.Utils$$anonfun$readValueT$1.apply(Utils.scala:40)

git bisected it and found the first non-working commit in the ssl-config repo, namely https://github.com/typesafehub/ssl-config/pull/2/commits/e779642d020d6dfc53d8a16dea13b61b5476e955 . but @ktoso said that's the first real commit in the project
while researching this, I learned that dbuild extraction works by adding a plugin to the build and then doing something in the plugin's onLoad. and lo and behold the ssl-config build overrides onLoad. getting rid of that in https://github.com/SethTisue/ssl-config/commit/640e042aac0f383264b6c9a6b73e94ce4d9779d9 got past this hurdle

next hurdle: there is a circular dependency between akka-actor and ssl-config-akka. dbuild detects this and complains:
[error] These items are part of cycles, and are all reachable from each other:
[error]    Project "ssl-config" uses com.typesafe.akka:akka-actor:jar (build level 0, space "default"), which is provided by: "akka".
[error]    Project "akka" uses com.typesafe:ssl-config-akka:jar (build level 0, space "default"), which is provided by: "ssl-config".
[error] The graph is not acyclic.
[error] com.typesafe.dbuild.graph.CycleException: Cycle found
[error] 	at com.typesafe.dbuild.graph.Graph.checkCycles(Graphs.scala:90)
[error] 	at com.typesafe.dbuild.model.RepeatableDBuildConfig.<init>(RepeatableBuild.scala:153)

fyi @schmitch. while you're working on 2.12 in the Play build directly, I'm working on this, but it's slooooooow going

Regarding the circular dependency: that's something that @ktoso and I were also discussing some time ago. It would probably make sense to clean this up to move ssl-config-akka into akka proper. We are having the same kinds of problems when releasing akka / ssl-config.

I'll try to work around it in dbuild by putting akka-http in a separate section

Let me know if this blocks you in any way @SethTisue and we'll fix it ASAP.

I thiiiiiink it's sorted, but I won't know for sure until I finish #146

done in https://github.com/scala/community-builds/pull/298


Fix merge conflict in assume-role

There was a merge conflict please accept my merge of upstream master

Was merged as https://github.com/logstash-plugins/logstash-mixin-aws/pull/27
Thanks!


Help message for flags is illformatted

The help message for flags is illformatted and confusing.
For example let's take this simple flag:
GenericArguments.flags().flag("h", "-history").buildWith(GenericArguments.none())

The resulting help text formats this flag like this: [-h|-history <h>]
Foremost, the long flag should have two dashes instead of just one. But also the <h> is confusing, as <> normally indicate a required argument. I therefore propose these alternative formats:

Simple: [-h|--history]
Unix style: [-h, --history]
With Parentheses: [-h|--history (h)]

+1 for linux style.

Most of your points are valid and I plan to solve them in a more emphatic way for the command refactor: #1587. However, formatting the flags in the current system unix style would go against the rest of the usage text: | is used to indicate "or" elsewhere.
I'm not against the unix style, mind, that would be a trival change if there is an overwhelming desire for this. I'm just going to make the minimal changes to make the usage more useful right now.

For
Sponge.getCommandManager().register(this,
        CommandSpec.builder().executor((a, b) -> CommandResult.success()).arguments(
                GenericArguments.flags()
                        .flag("h", "-history")
                        .permissionFlag("test", "t", "-test")
                        .valueFlag(GenericArguments.bool(Text.of("bool")), "b", "-bool")
                        .buildWith(GenericArguments.none())
        ).build(), "test");

I will get /test [-h|--history] [-b|--bool <bool>] [-t|--test], which should solve the immediate problem this issue raises.

This looks very good! Thank you for the quick fix!


Feature Request: joblog option

This would reproduce the --joblog feature seen in current versions of GNU Parallel.
Handy for troubleshooting or for tasks that otherwise require information on the exit values of tasksrun via Parallel

This is a status update that I'm getting fairly close to completing the job log feature. I have a trivial logic error to track down and some other unrelated refactoring to do, but I might have it completed by the end of tomorrow. The implementation avoids heap allocations. Integers are directly converted into a shared stack-allocated &[u8] and written directly to the buffered file, so there should be zero performance overhead to implementing the new feature.

Finished

Just an update that you might be interested in the new time format (--joblog-8601)


getByMac doesn't seem to work.

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this issue has not already been filed

Current Behavior
    const client = clients.filter(d=> d.mac === 'ca:f8:f4:50:33:6f')[0];
    console.log(client.mac)                         // prints ca:f8:f4:50:33:6f
    await site.clients.getByMac(client.mac);        // err api.err.UnknownStation

getByMac return an error. getById works.
Expected Behavior
I expect getByMac to retrieve clients by mac address.
Context
Please provide any relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Library version<EMAIL_ADDRESS>Unifi Controller :UDM SE v3.1.16
Unifi OS : yes?
NodeJS version:  v20.2.0
Operating System: Linux (WSL)

Failure Logs
Coundn't get that to work, but the essential is that it makes this request :
GET /proxy/network/api/s/default/stat/sta/ca%3Af8%3Af4%3A50%3A33%3A6f
and the response is
{
        meta: {
          rc: 'error',
          mac: 'ca%3af8%3af4%3a50%3a33%3a6f',
          msg: 'api.err.UnknownStation'
        },
        data: []
      }

Hello, thank you for your issue .
I'm can't try it for the moment (maybe I will have some time at the end of the month) .
Can you try if it doesn't works too when using /stat/user/:mac ?

@jpambrun I fixed in in v0.10.0 . Feel free to test

Unfortunately is didn't help. I can see the request is now /proxy/network/api/s/default/stat/sta/ca:f8:f4:50:33:6f,  but the 400 error is similar.
        meta: {
          rc: 'error',
          mac: 'ca:f8:f4:50:33:6f',
          msg: 'api.err.UnknownStation'
        },

@jpambrun ok … I don't have the same .
what is your unifi network version ?
Actually, I tried with :
    const [site] = await controller.getSites();

    const clients = await site.clients.list();
    const client = clients.find((c) => c.mac === '28:6c:07:7a:52:fb');

    if (!client) {
        throw new Error('no client');
    }

    const clientByMac = await site.clients.getByMac(client.mac);
    console.log(clientByMac);

And I receive the client .
Else, I go in unifi web app => enable the legacy interface => go on the clients tab => select a client => I can see a request to https://unifi/proxy/network/api/s/default/stat/user/28:6c:07:7a:52:fb .
the same for you ?
if yes, you can try to modify the url in the lib before running your script ? to call /stat/user/<mac> insteed of /stat/sta/<mac> ( on my two setups the two url do the same things ).
( need to be modified in node_modules/unifi-client/lib/Clients/Clients.js line 135 )

I have UDM SE v3.1.16 with Network 7.5.174.
Using your code snippet the exception is not thrown, but getByMac gets a 400 with api.err.UnknownStation.
In the legacy UI,  I do see /proxy/network/api/s/default/stat/user/14:c1:4e:14:5c:bd requests.
And, the proposed code change from "/stat/sta/" to "/stat/user/" does make it work for me.

I can also limp by using await site.getInstance().get(/stat/user/${client.mac});

hum ... I can't help if I can reproduce it ...
Actually, the only way to get the same error, is to try to get a mac that doesn't exist ...
Can you get the UI to call the url with ca:f8:f4:50:33:6f ? or is the error the same with 14:c1:4e:14:5c:bd ?

@jpambrun any update about this bug ?


Remove fastapi-users-db-sqlmodel dependency

Context: this question came up while working (with @mfranzon) on a pydantic/sqlmodel/sqlalchemy issue which is not yet clear. We don't know if the two issues are related, but in general this one here has to be considered at some point.
--
We depend on fastapi-users-db-sqlmodel, that currently (as of their 0.2.0 version) forces sqlalchemy version to be <=1.4.35. The reason is that 1.4.36 introduced a widespread issue, including in sqlmodel  - see release notes and several issues:

https://github.com/tiangolo/sqlmodel/issues/315
https://github.com/sqlalchemy/sqlalchemy/discussions/7972#discussioncomment-2655517
https://github.com/tiangolo/sqlmodel/pull/461

sqlmodel 0.0.7 partly fixed the issue on their side (but notice another PR is still open), and as of sqlmodel 0.0.8 (released in August 2022) their sqlalchemy constraint is ">=1.4.17,<=1.4.41".
In fastapi-users-db-sqlmodel there are several automatic dependabot PRs to relax the constraint, but little activity.
We should verify if this package actively mantained, and otherwise we should find some way to replace it.
Note that no other dependency of ours currently requires sqlalchemy<=1.4.35:
$ poetry show sqlalchemy

 name         : sqlalchemy
 version      : 1.4.35
 description  : Database Abstraction Library

dependencies
 - greenlet !=0.4.17

required by
 - alembic >=1.3.0
 - fastapi-users-db-sqlmodel >=1.4,<1.4.36
 - sqlalchemy-utils >=1.3
 - sqlmodel >=1.4.17,<=1.4.41

For the record, let's keep in mind that https://github.com/fastapi-users/fastapi-users-db-sqlalchemy appears as well maintained.

The three classes that we are using from this package are

SQLModelUserDatabaseAsync
SQLModelBaseUserDB
SQLModelBaseOAuthAccount

If we proceed with a switch from UUID4 to autoincremental integer IDs (ref #560), then SQLModelBaseUserDB and SQLModelBaseOAuthAccount become irrelevant.
As per SQLModelUserDatabaseAsync, right now I find it is used eight times across github, and also see comments like these ones

Bear in mind though that we didn't officially release this DB adapter; as they are some problems with SQLModel regarding UUID (https://github.com/tiangolo/sqlmodel/issues/25). So you'll probably run into issues.
..
There is one that should work, but don't plan to officially support it for now: https://github.com/fastapi-users/fastapi-users-db-sqlmodel

I think that we should quickly move on and explore the possibility of fully removing this dependency. If it impossible to replace SQLModelUserDatabaseAsync, then let's also consider that it is released under MIT license and we could include that definition in our code base.

Hello, would you consider supporting sqlmodel for fastapi-users?

We being the Fractal developers, not the fastapi-users developers. You may want to check with them directly for your question @caniko , because we just use that library :)

I considered the same @tcompa, thanks

This is now ready as part of #660. The choices we made are:

We merged the relevant part of SQLModelBaseUserDB into our own UserOAuth
We kept SQLModelBaseOAuthAccount as is (bundled in fractal-server with attribution, under MIT license), since it is a very transparent model with a few simple (string) attributes.
We kept SQLModelUserDatabaseAsync as is (bundled in fractal-server with attribution, under MIT license), since we could only re-implement it in the exact same way. If (unlikely) fastapi-users is updated in a breaking way (e.g. requiring an additional get_by_xxx method), then the current adapter won't work and some users endpoints may be break. This is mitigated by the fact that we already test all relevant fastapi-users endpoints (see test_security.py), and notably:

whoami
register
userlist
get user
delete user
patch user
create (super)user

Side note: if we ever move away from SQLModel, towards sqlalchemy v2, then we could revert the current change and easily switch to the (better maintained) corresponding adapter: https://github.com/fastapi-users/fastapi-users-db-sqlalchemy

Closed with #660


Multiple usage

Whenever I use multiple elements of counto in 1 component, it displays the highest data passed.
I have different [countTo]s per element and it reflects in the console that it receives the data I passed but
{{counto | pipe here}} displays the highest value from the different elements.

This is a stack overflow question, but i'll go ahead and answer anyway:
I'm assuming you're using the following layout (  example taken from the ReadMe file):
first div

{{counto | currency:'EUR':true:'1.2-2'}}

second div

{{counto | currency:'EUR':true:'1.2-2'}}

So you're binding  the same counto variable in both divs, here's what you'll want to do:

{{counto_a | currency:'EUR':true:'1.2-2'}}

second div

{{counto_b | currency:'EUR':true:'1.2-2'}}

Hope this helps!

Thank for explanation.


fix: Prevent Batches from queuing empty events

Summary

Within BatchUploader.queueEvent, we are only checking if the event is null, but not if it is actually empty.
We don't have reliable test data directly related to this bug report, and is unlikely that an event can get to this point in the code while being empty, but we should be more defensive, just in case.

Testing Plan

Using the browser debugger, manually send events into the batch uploader and verify that batches without events are not created.

Master Issue
Closes https://go.mparticle.com/work/SQDSDKS-4737

lastly, just to include another hopefully fail safe - can we do something here in the upload function?
https://github.com/mParticle/mparticle-web-sdk/blob/master/src/batchUploader.ts#L199-L214
if (uploads[i].events.length<0) {
   continue;
}


feat(button-link): add button link and button link with icon #137

feat(button-link): add button link and button link with icon #137

chore(fabricator): downgrade to 1.1.10 due to an issue, see https://github.com/fbrctr/fabricator-assemble/issues/40 #137

PLEASE RUN: npm update fabricator-assemble
@dami-gg @gabrielhl @xonic @mrac WDYT?
!!! #159 must be merged before this one !!!

@gabrielhl Can you add some notes for the flat button and flat button icon and maybe find a better way to present this component in the demo section?

@rbarilani will do that now

reminder for me: remove dc-btn--link-large, etc. (use dc-btn--large).

@gabrielhl @xonic at this point I would like to remove also dc-btn--link-[success,destroy,disabled] and just use dc-link--[success,destroy,disabled]. WDYT?

Since it's a button atom, shouldn't we be using dc-btn--destroy, dc-btn--disabled etc?

+1 for btn modifiers

On 24.03.2016, at 11:21, Gabriel Lovato<EMAIL_ADDRESS>wrote:
Since it's a button atom, shouldn't we be using dc-btn--destroy, dc-btn--disabled etc?
―
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

@xonic @gabrielhl Well I can't use btn modifiers because they add background.

From a logical perspective, to me, the link button belongs to the group of buttons. I don't think that btn--link should be combined with other mods than --disabled. To be sure though, and give more flexibility, I'd override the background. What do you think guys?

Thomas Nägele
http://xon1c.com
@xon1c
Sonnenallee 92
12045, Berlin
+49 170 311 27 64
<EMAIL_ADDRESS>
On 24.03.2016, at 13:48, Ruben Barilani<EMAIL_ADDRESS>wrote:
@xonic @gabrielhl Well I can't use btn modifiers because they add background.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub

The thing is that we our current approach users of our library can work just with mixins, like so:
.3rdparty-link-success {
 @include dc-btn;
 @include dc-btn--link;
 @include dc-btn--link--success;
}

// or

.3rdparty-link {
  @include dc-btn;
  @include dc-btn--link;
}
.3rdparty-link.disabled {
   @include dc-btn--link--disabled;
}

This is possible because we don't rely directly on selectors.
But ok we can relax that concept now, if you want.

Would something like this maybe work? Defining the state as a mixin and then combining the selector?
@mixin dc-btn--link--destroy--no-touch; {
    &:hover {
        background: transparent;
        color: $dc-red50;
    }
}

and then when definining the selectors
.no-touch .dc-btn-link.dc-btn--destroy {
        @include dc-btn--link--destroy--no-touch;
}

Would something like this maybe work? Defining the state as a mixin and then combining the selector?
@mixin dc-btn--link--destroy--no-touch; {
&:hover {
background: transparent;
color: $dc-red50;
}
}
and then when definining the selectors
.no-touch .dc-btn-link.dc-btn--destroy {
@include dc-btn--link--destroy--no-touch;
}

Yep is what I'm doing to have both approaches available. It can work and I like it BUT I have only one doubt: btn--link can have also a success modifier unlike dc-btn...  IHMO we should add a dc-btn--success modifier valid for dc-btn and dc-btn--link.... WDYT?

@xonic and I just talked and we suggest:

removing the success and destroy modifiers in the regular link
enable the --destroy modifier to be used in the btn-link with the same visual style as for other buttons (red bg, white text)

@rbarilani Ok, I've discussed the issue with @gabrielhl and we think the following makes sense:

Remove .dc-link--success --> we're not ever gonna need this kind of modifier
Make the combination of .dc-btn--link.dc-btn--destroy work. How? In idle state it should look like .dc-btn--link and on :hover it should look like the ordinary .dc-btn--destroy

@gabrielhl @xonic ok maybe now is done. @gabrielhl can you update the .dc-btn--link note?


Simplifying JMX monitoring using catalina-jmx-remote.jar

Hi,
Monitoring Tomcat remotely can be done using JMX (https://tomcat.apache.org/tomcat-7.0-doc/monitoring.html). But the JSR 160 JMX-Adaptor opens a second data channel on a random port. That is a problem when you have a local firewall installed. To fix this, we need to configure a JmxRemoteLifecycleListener, as described in listeners documentation (https://tomcat.apache.org/tomcat-6.0-doc/config/listeners.html#JMX_Remote_Lifecycle_Listener_-_org.apache.catalina.mbeans.JmxRemoteLifecycleListener).
The configuration part can be achieved using a volume mapping to provide a custom server.xml, but the catalina-jmx-remote.jar is missing from the default distribution. This merge request adds the jar.

I think I'm leaning towards not including this in the base, and the reason is that it's reasonably simple to add to the image after the fact (as demonstrated by the Dockerfile below); combine that with automated builds (https://docs.docker.com/docker-hub/builds/) and repository links (https://docs.docker.com/docker-hub/builds/#repository-links) and it's reasonably easy to have an up-to-date image built FROM this one with the library (and even the configuration for it) included:
FROM tomcat:8-jre7

ENV TOMCAT_JMX_REMOTE_URL https://www.apache.org/dist/tomcat/tomcat-$TOMCAT_MAJOR/v$TOMCAT_VERSION/bin/extras/catalina-jmx-remote.jar

RUN set -x \
	&& curl -fSL "$TOMCAT_JMX_REMOTE_URL" -o lib/catalina-jmx-remote.jar \
	&& curl -fSL "$TOMCAT_JMX_REMOTE_URL.asc" -o lib/catalina-jmx-remote.jar.asc \
	&& gpg --batch --verify lib/catalina-jmx-remote.jar.asc lib/catalina-jmx-remote.jar \
	&& rm lib/catalina-jmx-remote.jar.asc

Sending build context to Docker daemon 2.048 kB
Step 1 : FROM tomcat:8-jre7
 ---> fd31605c2873
Step 2 : ENV TOMCAT_JMX_REMOTE_URL https://www.apache.org/dist/tomcat/tomcat-$TOMCAT_MAJOR/v$TOMCAT_VERSION/bin/extras/catalina-jmx-remote.jar
 ---> Running in 6946dc248eb6
 ---> a5bdecfe4eb2
Removing intermediate container 6946dc248eb6
Step 3 : RUN set -x     && curl -fSL "$TOMCAT_JMX_REMOTE_URL" -o lib/catalina-jmx-remote.jar     && curl -fSL "$TOMCAT_JMX_REMOTE_URL.asc" -o lib/catalina-jmx-remote.jar.asc     && gpg --batch --verify lib/catalina-jmx-remote.jar.asc lib/catalina-jmx-remote.jar     && rm lib/catalina-jmx-remote.jar.asc
 ---> Running in 56fae4cc7cb8
+ curl -fSL https://www.apache.org/dist/tomcat/tomcat-8/v8.0.33/bin/extras/catalina-jmx-remote.jar -o lib/catalina-jmx-remote.jar
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 13536  100 13536    0     0  14030      0 --:--:-- --:--:-- --:--:-- 14026
+ curl -fSL https://www.apache.org/dist/tomcat/tomcat-8/v8.0.33/bin/extras/catalina-jmx-remote.jar.asc -o lib/catalina-jmx-remote.jar.asc
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   836  100   836    0     0   1075      0 --:--:-- --:--:-- --:--:--  1074
+ gpg --batch --verify lib/catalina-jmx-remote.jar.asc lib/catalina-jmx-remote.jar
gpg: Signature made Fri 18 Mar 2016 08:32:45 PM UTC using RSA key ID 2F6059E7
gpg: Good signature from "Mark E D Thomas<EMAIL_ADDRESS>gpg: WARNING: This key is not certified with a trusted signature!
gpg:          There is no indication that the signature belongs to the owner.
Primary key fingerprint: A9C5 DF4D 22E9 9998 D987  5A51 10C0 1C5A 2F60 59E7
+ rm lib/catalina-jmx-remote.jar.asc
 ---> 9ec03ef66ecd
Removing intermediate container 56fae4cc7cb8
Successfully built 9ec03ef66ecd

To put it another way, is using JMX so common that the base install of Tomcat ought to include it?  If so, why doesn't upstream include it by default and provide instructions for removing it instead?  Are there other libraries or configurations that are in a similar boat?  Are there any relevant links to upstream's documentation recommending that it be used (the easiest way to sway my opinion)? :smile:

On second thought, I don't think using JMX is so common that it has to be included in this base image. We already have to create our own Dockerfile to add some company specific settings, so the effort required to add JMX is very small.


Swipable list renderLeft(Right) row badly aligned after deleting row

Hello. When I delete a row from the Swipable list the next item is not rendered correctly if the size is not the same as the deleted item:

As per design guidelines, a list can have single lined / two lined /three lined etc data as its sub-elements. A single list does not have a different number of lines of data with it subelements.
Similar is the case of swipe list
Example: YouTube history page


Implement Steam Audio log callback

This fixes #73
This implements the Steam Audio context log callback and logs the messages using SteamAudio::log with the equivalent level for the plugin enum.
These messages will also now appear in the editor Output panel instead of only appearing in stdout:
[godot-steam-audio] Initializing SteamAudioServer global state
[godot-steam-audio] Initialized Embree v2.17.07.
[godot-steam-audio] Initialized SteamAudioServer global state

Thank you! I’ll see if I can review/test this over the weekend (don’t really have a computer with me right now).

Looks great! Thanks!


Mobile App friendly boilerplate

Hello,
This works great on the web. But it would be cool with a boilerplate for a "mobile app".
Where you don't use the redirect flows. Cause that's usually handled by the app and then the app will have a token that it would like to send to serverless-authentication
Any plans on that ? :)

Yeah... same here. Not sure what you're trying to do, but you can use a cognito blueprint which would be a different repo and fit your use case better.
I want to avoid cognito and have custom authorizers instead to save money, but it seems that there are no open source projects for this yet. Want to start a project?
Link to my feature request: https://github.com/laardee/serverless-authentication-boilerplate/issues/66


Route services can't authenticate that a request is a "route service request" from gorouter

Thanks for submitting an issue to gorouter. We are always trying to improve! To help us, please fill out the following template.
Issue
Route services currently have no general way to authenticate that a request indeed comes from gorouter, and that it is being sent to it with route services duties.
Additionally, the route services responsibilities page does not list any specific requirement for route services to implement validation of requests.
Because of this, clients can potentially forge the headers and cause a variety of issues (data exposure, DoS) to route services that naively implement the https://docs.cloudfoundry.org/services/route-services.html#service-instance-responsibilities to the letter.
Context
Route services can be deployed in a variety of configurations, but the issue affects only the models that are required to implement the service instance responsibilities:

Fully brokered
User-provided

In addition, two deployment models need to be considered:

Service instance deployed on Cloud Foundry
Service instance deployed outside Cloud Foundry

The two main scenarios are as follows:

if a client sends a request to the route service and sets X-Proxy-Forward-Url to the URL of the route service it will create a loop, potentially DoSing the route service
if a client sends a request to the route service and sets X-Proxy-Forward-Url to a URL that the route service does have access to but that is not accessible by the client, it can lead to data exposure

Possible mitigations are:

Fully brokered services can avoid (1) and (2) by checking that X-Proxy-Forward-Url is in the list of routes the service is bound to (note: this is not specified in the documentation)
Services can avoid (1) by removing X-Proxy-Forward-Url from the request forwarded to the application (note: this is not specified in the documentation)
Services can avoid (2) by rejecting requests if X-Proxy-Forward-Url is not included in a whitelist (note: this is not specified in the documentation)

Note that all of the above issues require all route services to actively conform to an additional set of responsibilities. Failure to do so, even after the service instance responsibilities documentation has been updated, will still expose users and operators to potential vulnerabilities.
For these reasons, it would be preferable to implement some sort of solution on the platform side.
Possible fixes are:

Clarify the service instance responsibilities, so that route service authors are at least aware of how to avoid being vulnerable
Gorouter can avoid (1) and (2) by removing X-Proxy-Forward-Url from requests that are not supposed to be directed to a route service; route services are instructed to reject requests that don't contain that header (note: this is a potentially breaking change, e.g. in the case a route service is hosted on a separate foundation)
Mark the routes mapped to route services hosted on CF as "gorouter-only", i.e. client traffic to that route is rejected, but gorouter can use it to forward traffic that has to go through the route service. This would ensure that the headers can't be forged. (note: this would solve issues (1) and (2) only for CF-hosted service instances)
Have some kind of pre-shared secret between gorouters and service instances, instruct service instances to reject requests missing the secret (note: this would solve issues (1) and (2) in all cases)

Links
Discussed over slack: https://cloudfoundry.slack.com/archives/C05586PBX/p1513655333000044

Hi all,
We think we could also fix this through enabling mTLS from the GoRouter to the Route Service.
@XanderStrike does this sound like something Envoy could tackle in the future?
At the very least we can update the docs to tell Route Service creators.
Thanks,
@bruce-ricard and @ameowlia, CF Networking Program members

We haven't discussed route services at length though I know we intend to support them in Envoy. I'll make sure the PMs are aware of this concern for when we implement them.

Hi @CAFxX,
Just for clarification, did you mean the X-CF-Forwarded-Url header? Your post mentions X-Proxy-Forward-Url, but @bruce-ricard and I unable to find any references of that in either the documentation or the code.
Thanks!

Hi all,
I'm closing this issue due to inactivity. If this is still an issue, please feel free to reopen with more information about its impact.
Thanks,
@barakyo, CF Networking Program Member

Hi all,
We are closing this issue due to other priorities. If this is still an issue, please feel free to send a PR or continue the discussion.
Thanks,
@ameowlia and @mike1808 , CF Networking Program Members


Major distortion in upscaled output

I used the tile upscale command (animatediff tile-upscale <PATH_TO_TARGET_FRAME_DIRECTORY> -W 768)
And my result is this:
https://github.com/s9roll7/animatediff-cli-prompt-travel/assets/149917802/3f0efd84-c3cc-4181-8272-ea46e61ebc0e
But this is the original:
https://github.com/s9roll7/animatediff-cli-prompt-travel/assets/149917802/1d7791a7-d786-4342-abbe-4e07b8c95407
Here's my prompt:
prompt.json
Did I miss something? Should I use refine instead?

It is working as intended.
tile-upscale has nothing to do with animatediff, it is simply a command to upscale frame by frame

This doesn't really seem like an upscale, it looks very distorted. Is there a better method I can use?

animatediff refine FRAMES_DIR -t 1.0 -W 768 -C 16

or at the first generation Use gradual_latent_hires_fix

So Refine worked great, however it takes a very long time for 512 -> 1024. 5hrs on 12gb vram.
Is there any possibility a feature can be implemented to upscale with models similar to what A1111 uses in the Extras tab? Like with ESRGAN, Ultrasharp, Nearest, etc? We can do it ourselves with batch processing but having the option within CLI would save a lot of time.

https://github.com/s9roll7/animatediff-cli-prompt-travel/assets/118420657/39637d21-877e-47ed-8d28-0cb009d62106
    "controlnet_map": {
        "input_image_dir": "..\\refine\\2023-12-07T22-14-45-prompt_earth-dreamshaper_8\\00_controlnet_image",
        "max_samples_on_vram": 0,         # <-----------
        "max_models_on_vram": 0,         # <----------
        "save_detectmap": true,

animatediff refine FRAMES_DIR -t 1.0 -W 1024 -C 12
vram 11.5 GB
20min

Is there any possibility a feature can be implemented to upscale with models similar to what A1111 uses in the Extras >tab? Like with ESRGAN, Ultrasharp, Nearest, etc?

No plans to support

Will try this. Thanks!


Sporadic No_Such_Method failure when executing Base_Tests

The sporadic failure happens mostly on MacOS although it did appear for other OS as well.
JAVA_OPTS="-enableassertions" "/bin/bash" "/Users/runner/work/enso/enso/built-distribution/enso-engine-2024.4.1-dev-macos-amd64/enso-2024.4.1-dev/bin/enso" "--no-ir-caches" "--run" "/Users/runner/work/enso/enso/test/Base_Tests"

Will simply return a non-zero code and a simple No_Such_Method message, without any context.
The issue is not new and by looking at the past CI builds it has happened for the first time in https://github.com/enso-org/enso/actions/runs/9686658267/job/26729584788 (#10377 merge, unrelated).

Encountered it yet again in https://github.com/enso-org/enso/actions/runs/10245645438/job/28392624237?pr=10733#step:7:2072 run

This is confusing. It seems to suggest that the problem is with the test that should be pending?

This is confusing. It seems to suggest that the problem is with the test that should be pending?

What do you mean?
Do you mean the
 INFO ide_ci::program::command: bash ℹ Java FFI: [6/6, 17ms]
 INFO ide_ci::program::command: bash ℹ     - [PENDING] should report missing method error on Java Arrays
 INFO ide_ci::program::command: bash ℹ         Reason: Failing due to #6609

above?
That is most likely not it - the test summary is written after each group has finished executing. So the Java FFI pending test has finished and is now summarized.
Now the failure most likely is happening in the next group that is being executed. We could try to do a debug run which prints which group is started to try to locate the cause - but I'm not sure if that'll help.

Another interesting instance of this error is at https://github.com/enso-org/enso/actions/runs/10264320295/job/28398095736?pr=10555#step:7:2118 . This time, there is mention of java.io.IOException:
INFO ide_ci::program::command: bash ℹ java.io.IOException: No_Such_Method

Finally.
NFO ide_ci::program::command: enso ⚠ java.lang.RuntimeException: #10750 ==
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.runtime.builtin.Error.makeNoSuchMethod(Error.java:162)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.InvokeMethodNode.methodNotFound(InvokeMethodNode.java:286)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.InvokeMethodNode.doFunctionalDispatchUncachedSymbol(InvokeMethodNode.java:219)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.InvokeMethodNodeGen.execute(InvokeMethodNodeGen.java:226)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.InvokeCallableNode.invokeDynamicSymbol(InvokeCallableNode.java:276)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.InvokeCallableNodeGen.execute(InvokeCallableNodeGen.java:194)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.ApplicationNode.executeGeneric(ApplicationNode.java:97)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.scope.AssignmentNodeGen.executeGeneric(AssignmentNodeGen.java:51)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.callable.function.BlockNode.executeGeneric(BlockNode.java:52)
 INFO ide_ci::program::command: enso ⚠ 	at org.enso.interpreter.node.ClosureRootNode.execute(ClosureRootNode.java:85)

https://github.com/enso-org/enso/actions/runs/10348083887/job/28657599636?pr=10769#step:7:10627

Reviewed the No_Such_Method. It has the to_display_text properly set up.
I'm guessing it is an issue in engine/runtime/src/main/java/org/enso/interpreter/runtime/error/PanicException.java as to why the Java Exception is not showing the display text.
It looks like computeMessage will attempt to call the to_display_text via the InteropLibrary and fall back to the type name if it doesn't exist.

In my PR I've been experimenting with one revert (https://github.com/enso-org/enso/pull/10769/commits/7bab8b89baac33a4496e17369c22b8570151e39b). While previously on that PR I would get sporadic failures, after applying the revert I had seen none.
Note: I don't think this change is the sole culprit of this bug. I would say that it unmasks some deeper problem, like specialization order which is triggered non-deterministically.

Hubert haven't seen this one for a while. Let's consider it fixed.

https://github.com/enso-org/enso/actions/runs/10682796071/job/29609456166?pr=10960#step:7:3772

And another https://github.com/enso-org/enso/actions/runs/10791389410/job/29928689658?pr=11032#step:7:1255


Support alternate compilers.

What steps will reproduce the problem?
1. CXX=CC gmake

What is the expected output? What do you see instead?

CC should be used as the C++ compiler.  g++ is instead used.

What version of the product are you using? On what operating system?

git tip. Solaris.

Please provide any additional information below.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 10 Oct 2011 at 9:20

This patch fixes the issue and passes CXX param into the make/build system.
It's small and simple

Original comment by<EMAIL_ADDRESS>on 10 Oct 2011 at 9:22
Attachments:

ldb-alternate-compilers.patch

Fixed in c8c5866a86c8. Apologies for the delay.

Original comment by<EMAIL_ADDRESS>on 30 Nov 2011 at 11:09

Changed title: Support alternate compilers.
Changed state: Fixed


feat: :sparkles: openshift compatibility

Purpose
Add openshift compatibilty for helm charts repositories.
Notes
Currently changes are done only for jupyter-python for validating.
Changes

charts/library-chart/ : add rolebindings with scc (default nonroot)
charts/jupyter-python/ : add values.yaml and values.schema.json. Enable this feature only on region configuration.

Hello,
Thanks for the PR !
Can you increment the charts version ? see library chart and  jupyter
Also, due to our release pipeline, could you open two different PR please ? One for  library chart and another one for jupyter-python chart while making sure to call for the right library chart version in jupyter-python ?


Daily update

Automated changes by create-pull-request GitHub action

Build failed because of:
no headquarter addresses for palm-nft-studio at https://www.crunchbase.com/organization/palm-nft-studio
No cached entry, and The Ritsumeikan Trust has issues with twitter: https://twitter.com/RitsARC, Request failed with status code 400
No cached entry, and Sollers College has issues with twitter: https://twitter.com/SollersEdu, Request failed with status code 400
No cached entry, and Research Institutes of Sweden has issues with twitter: https://twitter.com/RISEsweden, Request failed with status code 400
No cached entry, and Open Sourcing Mental Illness, LTD has issues with twitter: https://twitter.com/osmihelp, Request failed with status code 400
No cached entry, and Open Compute Project has issues with twitter: https://twitter.com/OpenComputePrj, Request failed with status code 400
No cached entry, and NIAB has issues with twitter: https://twitter.com/NIABEMR, Request failed with status code 400
No cached entry, and Galgotias University has issues with twitter: https://twitter.com/GalgotiasUni, Request failed with status code 400
No cached entry, and iRespond Global has issues with twitter: https://twitter.com/iRespond, Request failed with status code 400
No cached entry, and Zeeve Inc has issues with twitter: https://twitter.com/ZeeveDeeptech, Request failed with status code 400
No cached entry, and Uffizzi has issues with twitter: https://twitter.com/Uffizzi_, Request failed with status code 400
No cached entry, and Yazaki Corporation has issues with twitter: https://twitter.com/@YazakiNA, Request failed with status code 400
No cached entry, and Union.ai has issues with twitter: https://twitter.com/union_ai, Request failed with status code 400
No cached entry, and True B.V. has issues with twitter: https://twitter.com/true_nl/, Request failed with status code 400
No cached entry, and TietoEVRY has issues with twitter: https://twitter.com/Tietoevry, Request failed with status code 400
No cached entry, and Super Micro Computer, Inc. has issues with twitter: https://twitter.com/Supermicro_SMCI, Request failed with status code 400
No cached entry, and Sonatus, Inc. has issues with twitter: https://twitter.com/SonatusHQ, Request failed with status code 400
No cached entry, and SICPA SA has issues with twitter: https://twitter.com/SICPA_enEspanol, Request failed with status code 400
No cached entry, and ScoutAPM has issues with twitter: https://twitter.com/ScoutAPM, Request failed with status code 400
No cached entry, and Rakuten Group, Inc. has issues with twitter: https://twitter.com/Rakuten, Request failed with status code 400
No cached entry, and ProfiSea has issues with twitter: https://twitter.com/ProfiSea, Request failed with status code 400
No cached entry, and Pionix GmbH has issues with twitter: https://twitter.com/PionixDe, Request failed with status code 400
No cached entry, and Palm NFT Studio has issues with twitter: https://twitter.com/PalmNft, Request failed with status code 400
No cached entry, and OnGres has issues with twitter: https://twitter.com/ongresinc, Request failed with status code 400
No cached entry, and Optic has issues with twitter: https://twitter.com/useoptic, Request failed with status code 400
No cached entry, and Okteto has issues with twitter: https://twitter.com/oktetohq, Request failed with status code 400
No cached entry, and MDxBlocks Inc. has issues with twitter: https://twitter.com/MDxBlock, Request failed with status code 400
No cached entry, and mogenius has issues with twitter: https://twitter.com/mogenius, Request failed with status code 400
No cached entry, and Mondoo has issues with twitter: https://twitter.com/MondooLabs, Request failed with status code 400
No cached entry, and Mycelial has issues with twitter: https://twitter.com/mycelial, Request failed with status code 400
No cached entry, and Lightspin has issues with twitter: https://twitter.com/LightspinTech, Request failed with status code 400
No cached entry, and Lightrun Ltd has issues with twitter: https://twitter.com/Lightruntech, Request failed with status code 400
No cached entry, and KUKA Deutschland GmbH has issues with twitter: https://twitter.com/KUKA_Press, Request failed with status code 400
No cached entry, and KSOC has issues with twitter: https://twitter.com/ksoclabs, Request failed with status code 400
No cached entry, and lightlytics has issues with twitter: https://twitter.com/lightlytics, Request failed with status code 400
No cached entry, and JVC KENWOOD Corporation has issues with twitter: https://twitter.com/JVCKENWOOD_SR, Request failed with status code 400
No cached entry, and iSoftStone Information Technology (Group) Co., Ltd has issues with twitter: https://twitter.com/iSoftStone, Request failed with status code 400
No cached entry, and Honor Device Co. Ltd has issues with twitter: https://twitter.com/Honorglobal, Request failed with status code 400
No cached entry, and IO Builders Blockchain Technologies & Ventures has issues with twitter: https://twitter.com/iobuilders, Request failed with status code 400
No cached entry, and GotoAdmins has issues with twitter: https://twitter.com/gotoadmins, Request failed with status code 400
No cached entry, and Globant LLC has issues with twitter: https://twitter.com/Globant, Request failed with status code 400
No cached entry, and GitLab Inc. has issues with twitter: https://twitter.com/gitlabhq, Request failed with status code 400
No cached entry, and Fournine Cloud Solutions has issues with twitter: https://twitter.com/CloudFournine, Request failed with status code 400
No cached entry, and Era Software, Inc. has issues with twitter: https://twitter.com/era, Request failed with status code 400
No cached entry, and Equideum Health has issues with twitter: https://twitter.com/equideumhealth, Request failed with status code 400
No cached entry, and FlexiDAO has issues with twitter: https://twitter.com/flexidao, Request failed with status code 400
No cached entry, and DevSamurai has issues with twitter: https://twitter.com/devsamurai_com, Request failed with status code 400
No cached entry, and Datachain, Inc. has issues with twitter: https://twitter.com/datachain_en, Request failed with status code 400
No cached entry, and Cybeats has issues with twitter: https://twitter.com/cybeatstech, Request failed with status code 400
No cached entry, and DataCore Software has issues with twitter: https://twitter.com/DataCore, Request failed with status code 400
No cached entry, and Cribl Inc has issues with twitter: https://twitter.com/cribl_io, Request failed with status code 400
No cached entry, and CoolIT Systems Inc. has issues with twitter: https://twitter.com/CoolITSystems, Request failed with status code 400
No cached entry, and Comcast Cable Communications, LLC (Xfinity) has issues with twitter: https://twitter.com/Xfinity, Request failed with status code 400
No cached entry, and Clastix SRL has issues with twitter: https://twitter.com/clastix, Request failed with status code 400
No cached entry, and Coinbase Inc. has issues with twitter: https://twitter.com/coinbase, Request failed with status code 400
No cached entry, and CodeWave has issues with twitter: https://twitter.com/Codewave_Inc, Request failed with status code 400
No cached entry, and Catalogic Software has issues with twitter: https://twitter.com/CatalogicSW, Request failed with status code 400
No cached entry, and Bonifii has issues with twitter: https://twitter.com/BonifiiOfficial, Request failed with status code 400
No cached entry, and BMW has issues with twitter: https://twitter.com/BMWGroup, Request failed with status code 400
No cached entry, and Block, Inc. has issues with twitter: https://twitter.com/blocks, Request failed with status code 400
No cached entry, and Beijing Fourth Paradigm Intelligent Technology Co., Ltd. has issues with twitter: https://twitter.com/paradigm4, Request failed with status code 400
No cached entry, and Baumer Management Services AG has issues with twitter: https://twitter.com/baumergroup, Request failed with status code 400
No cached entry, and Beijing Kingsoft Cloud Internet Technology has issues with twitter: https://twitter.com/KINGSOFT_PR, Request failed with status code 400
No cached entry, and Arvancloud has issues with twitter: https://twitter.com/ArvanCloud, Request failed with status code 400
No cached entry, and Argonaut has issues with twitter: https://twitter.com/Argonaut_Dev, Request failed with status code 400
No cached entry, and Althea has issues with twitter: https://twitter.com/AltheaNetwork, Request failed with status code 400
No cached entry, and Absa Bank Limited has issues with twitter: https://twitter.com/AbsaSouthAfrica, Request failed with status code 400
No cached entry, and across.cloud has issues with twitter: https://twitter.com/AcrossCloud, Request failed with status code 400
Palm NFT Studio either has no crunchbase entry or it is invalid


PostgreSQL returns time.Time in UTC with different nanoseconds

Not sure we can do much, but it's worth adding on the README as known limitations and keeping it here until we figure out a workaround.

Tried a PoC of using the unix timestamp. But that does not contain milliseconds as well, so... same thing.

More information about this https://github.com/lib/pq/issues/227
I agree with the proposed solution and would love to see it coming from pq. In the meantime, can we hack something on top?

The problem is not pq, is postgres. There's just no space in a timestamptz field to store nanoseconds.
So I think it's better to just leave this open as a reminder (or document it on the readme as known issues).

Discussed this IRL with @Serabe. Solution will be to truncate all time fields before saving.


Add create_time to rules list output

What changed?
Added create_time field for the rules returned by ListWorkerVersioningRules API.

Why?
Creation time can be very useful for users when they what to consider cleaning up the rules.

Breaking changes
no

Should we just put a create_time in BuildIdAssignmentRule and CompatibleBuildIdRedirectRule? It's okay if it's not filled in sometimes. Either way, this is fine too

I also feel like the idea of adding create_time to the existing structs is cleaner since it will lead to fewer lines of generated code, but up to you Shahab.

I feel making it a single message reduces the generated code but makes our own code dirty because we need to verify more things, like what to do if the incoming API call has set the timestamp.
Also, it's likely that we add more server-generated values down the road that is not part of the input but is part of the output.

I feel making it a single message reduces the generated code but makes our own code dirty because we need to verify more things, like what to do if the incoming API call has set the timestamp.

There's going to be validation anyway, but fair.

Also, it's likely that we add more server-generated values down the road that is not part of the input but is part of the output.

In this case should the name be something other than TimestampedX?


where to find the "../introductions_analysis/{sero}_whole_caribbean.csv" file?

Dear @ViralVerity,
Could you please tell me where to find the "../introductions_analysis/{sero}_whole_caribbean.csv" file?
Hope you can reply to me, thanks.

That is a file that gets produced by the "parse introductions" script - I've tidied up that script a bit so it should be easier to use (ie I've removed my local paths)

You can generate the trees file required by running the XML in the XMLs folder

Both CSVs in the "parse introductions" script are generated by the script! It takes a MCC tree from the discrete phylogeography as its input

Yes I think so - it doesn't really make a difference for this analysis though


Add Dashboard Dev Automation

Since we're deploying our dashboard with Github Pages and using React, we should setup an actions workflow that automatically updates the /dist directory on --deploy branch when new code gets checked in to 'main' under directory /dashboard.

Workflow added.


Return of the Masking Issue

Crystal said:
Having same masking issue as we had in the previous SVG version. You had removed this line mask.setAttribute("height", "100%") in svg-mask.coffee. Uncommenting it doesn't appear to work this time.

Gotta place the masks on the main stage.


Add possibility to define the label for watcher 2

This PR has a dependency on the add-posibility-to-define-the-label-for-watcher.

@ryanjbaxter one more PR in the same category as the other... thank you


Implement piping

The relevant sections are mostly in src/posix/sh/command.rs.  ~We can't just use ChildStdin/ChildStdout/ChildStderr because we need to support other file descriptors as well, so we need to create the pipes manually using nix.~
EDIT: dunno why I wrote the above, but ChildStdout should be passed into the next command as its stdin.

This requires some sort of method to spawn commands without waiting for them to finish (as they need to be started at the same time).

For the most part, this was resolved in 295ba2d3dab26878d3a2d7f9da31cf103ce31544.


Added Star Wars 5e Character Sheet

I've been working on a character sheet for the Star Wars 5e conversion that just got recently released out of beta by the developer. The system is very much related to 5e so I used the standard 5e sheet as a template and modified it from there. I am the original author of this sheet per my reddit post:
https://www.reddit.com/r/sw5e/comments/91kgdb/roll20_custom_character_sheet_workinprogress/
Let me know if there are any concerns or suggestions!

Updated the sheet.json with some instructions, I'll start a wiki page soon enough to add more info/tips/setup options.

You could add a link to a roll20 wiki page in the "instructions" already, as the sheets&files are updated only once a week(Tuesdays).