Collectives can't apply to host on legacy plan because of 25 collectives limit

Our host https://opencollective.com/allforclimate was created last year and is under the legacy plan which means that there shouldn't be any limit. However, when people apply to host their collective on that host they get the error message that we have reached the limit of collectives we can host under our plan (25).

Hosts were assigned a legacy plan (small, medium, large) that was matching their current usage. So, this is working as expected.
We may tweak the plan for this specific host. cc @piamancini

I see in the database that the plan is set to legacy-large-host-plan so something must be off.

Large is up to 25 collectives so that is perfectly on point.

Ah I didn’t know. 25 doesn’t seem like a “large” number. I also didn’t know that the legacy plan was restricting old hosts to their current usage and prevented them to grow. That wasn’t clear in the communication.
In any case thank you for giving me those precisions during the weekend!

This what we said in the email we sent, I think it was communicated.

25 was what we agreed was the top level, there are few hosts over that. We can certainly look at upping the limit but again, we don't have that many hosts with +25 collectives (other than ours). We can't also keep changing the pricing model based on how many collectives this host has. What would make sense as a large host?

Ah ok. I missed that. Indeed I wouldn’t change this. At least not now.
AllForClimate falls in a different category though as it is a host that I created for the climate vertical. So I wouldn’t impose any such restriction on it in 2020 and then we can reconsider. Right now there is a 0% host fee so there isn’t any budget available. But I’m sure that this will change within the year. Working on it. But right now some local chapters of #FridaysForFuture are waiting to be able to join that host.

So will any host that does not charge host fees to its Collectives be able to have an unlimited account for free? That doesn't seem right to me @xdamman - we need large hosts to help support our sustainability, even if they are still working on their own. What about applying for a grant to help cover operational costs for AllForClimate, including the OC fees?

So will any host that does not charge host fees to its Collectives be able to have an unlimited account for free?

Of course not. That’s not what I wrote.

we need large hosts to help support our sustainability

Absolutely

What about applying for a grant to help cover operational costs for AllForClimate, including the OC fees?

Absolutely.
So in the meantime we say no to new climate collectives that are waiting to apply to that host?

Of course not. That’s not what I wrote.

Well a lot of companies have a nonprofit discount deal so I thought maybe we should consider a policy about it. Maybe we should use AllForClimate as a test case for that.

So in the meantime we say no to new climate collectives that are waiting to apply to that host?

I'll leave that to @piamancini to make the call. Above 25 Collectives we've agreed to do custom deals with hosts, but we haven't actually created guidelines for that yet.

(Note: technically most of the fiscal hosts are not for profit (e.g. wwcode), so I’d be cautious with such a discount)

It seems like an edge case, since we created the host ourselves. It was done by @xdamman while he was still full time on OC. I would treat it like Foundation / OSC. With the difference that in this case no fees are paid at all for added funds.
I think we should place them on a larger legacy plan for 2020 to a custom plan on the understanding of fairness, if the host grows to be able to cover those fees, it should get on a paid plan.

Host is unblocked. Closing the issue.


Fixes issue #238 and #239

Fixes #238 - "Bootstrap script shouldn't do a git pull, should instead warn about existing checkout."
Fixes #239 - "Bootstrap and other scripts should refuse to run as root"

@Vikram9866 I updated the description and will merge now.


[BUG] [chip-tool] cannot generate QR CODE

Reproduction steps
$ ./chip-tool payload generate-qrcode --discriminator 3840 --setup-pin-code 20202021 --vendor-id 65521 --product-id 32768 --version 1 --commissioning-mode 0

[1682942421.206178][122331:122331] CHIP:TOO: Run command failure: src/setup_payload/QRCodeSetupPayloadGenerator.cpp:268: Error 0x0000002F

Bug prevalence
always
GitHub hash of the SDK that was being used
commit 309e9f1afc989cea5927063d8a592e5eecb0b3ea (HEAD -> master, origin/master, origin/HEAD)
Platform
other
Platform Version(s)
No response
Anything else?
I referred https://github.com/espressif/esp-matter/issues/159#issuecomment-1377052019
When --version 0, chip-tool can generate qr code.
What is --version?
As my understanding, --version means firmware version for application.

As my understanding, --version means firmware version for application.

No, --version is the version of the encoding used to represent Matter data in the byte string that is then represented by the QR code (not to be confused with the QR code version or with any sort of other Matter version).
See "Packed Binary Data Structure" in the spec, which says for the Version field:

3-bit value specifying the QR code payload version.
SHALL be 000.

So a nonzero version is not a valid QR code for Matter so far; this value is present as an extension point in case we ever need to change the way Matter QR codes are encoded.
Since nonzero is not a valid value, and --allow-invalid-payload was not passed to the generate-qrcode command, the generation failed.  The error message could be a bit more helpful in this case....

Thank you for your reply.
I understand it.
I will check specification.
BTW, would you please add "guide for generate-qrcode" to https://github.com/project-chip/connectedhomeip/blob/master/docs/guides/chip_tool_guide.md?

@mmt-10 I can try to find time to do that, but you should feel free to do that yourself; just create a PR to that file...


Any reason to still allow unit fragments in scala 3?

In Scala 3, if without an else always evaluates to Unit. This means code like div(if (cond) h1("a"), ...) now never produces anything, even if the conditions is true, which has lead to a few silent issues when we transitioned to Scala 3 where stuff suddenly wasn't rendered anymore. Are there any other reasons to keep allowing units in templates? If not, it seems better to disallow this now to avoid mistakes like this?

Agreed.  Just got bit by this myself.


[Test Failed] 【TC-ACE-2.1】Commissioning failed: src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error

Test issue(s)
TH Version: v2.11+fall2024
TH SHA: 8f86ccd
First, use the web interface of the test harness to test it. The error screen and log are as follows:
TC-ACE-2-1_2024_11_28_08_16_43.log

After that, the Raspberry PI equipped with TH was used to enter docker, and the pyrhon script was used for testing. The error log is as follows:
【TC-ACE-2.1】UI-Python test DUT error.txt
Test command:
python3 /root/python_testing/scripts/sdk/TC_AccessChecker.py  --commissioning-method on-network --discriminator 3840 --passcode 20202021   --endpoint 0 --timeout 1000 --paa-trust-store-path /paa-root-certs  --storage-path admin_storage.json   --tests test_TC_ACE_2_1 --timeout 1000
Clearly visible error is:
[MatterTest] 11-28 08:25:18.734 ERROR Error on commissioning step 'AttestationVerification': 'src/controller/CHIPDeviceController.cpp:1280: CHIP Error 0x000000AC: Internal error'
[MatterTest] 11-28 08:25:18.735 ERROR Failed verifying attestation information. Now checking DAC chain revoked status.
[MatterTest] 11-28 08:25:18.735 INFO Commissioning stage next step: 'AttestationVerification' -> 'AttestationRevocationCheck'
[MatterTest] 11-28 08:25:18.736 INFO Performing next commissioning step 'AttestationRevocationCheck' with completion status = 'src/controller/CHIPDeviceController.cpp:1280: CHIP Error 0x000000AC: Internal error'
[MatterTest] 11-28 08:25:18.737 INFO Verifying device's DAC chain revocation status
[MatterTest] 11-28 08:25:18.737 ERROR Failed in verifying 'Attestation Information' command received from the device: err 101. Look at AttestationVerificationResult enum to understand the errors
[MatterTest] 11-28 08:25:18.738 ERROR Error on commissioning step 'AttestationRevocationCheck': 'src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error'
[MatterTest] 11-28 08:25:18.739 INFO Going from commissioning step 'AttestationRevocationCheck' with lastErr = 'src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error' -> 'Cleanup'
[MatterTest] 11-28 08:25:18.739 INFO Performing next commissioning step 'Cleanup' with completion status = 'src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error'
[MatterTest] 11-28 08:25:18.740 INFO Disarming failsafe on device 0xffffa400efb0
[MatterTest] 11-28 08:25:18.742 INFO <<< [E:5397i S:15185 M:34047152] (S) Msg TX to 0:FFFFFFFB00000000 [0000] [UDP:<IP_ADDRESS>:5540] --- Type 0001:08 (IM:InvokeCommandRequest) (B:65)
[MatterTest] 11-28 08:25:18.744 INFO <<< [E:5396i S:15185 M:34047153 (Ack:51334377)] (S) Msg TX to 0:FFFFFFFB00000000 [0000] [UDP:<IP_ADDRESS>:5540] --- Type 0000:10 (SecureChannel:StandaloneAck) (B:34)
[MatterTest] 11-28 08:25:18.746 INFO >>> [E:5397i S:15185 M:51334378 (Ack:34047152)] (S) Msg RX from 0:FFFFFFFB00000000 [0000] --- Type 0001:09 (IM:InvokeCommandResponse) (B:70)
[MatterTest] 11-28 08:25:18.752 INFO Received Command Response Data, Endpoint=0 Cluster=0x0000_0030 Command=0x0000_0001
[MatterTest] 11-28 08:25:18.753 INFO Failsafe disarmed
[MatterTest] 11-28 08:25:18.753 INFO Successfully finished commissioning step 'Cleanup'
[MatterTest] 11-28 08:25:18.754 INFO SecureSession[0xffffa400eb20, LSID:15185]: State change 'kActive' --> 'kPendingEviction'
[MatterTest] 11-28 08:25:18.755 ERROR Failed to send Solitary ack for MessageCounter:51334378 on exchange 5397i:src/messaging/ExchangeContext.cpp:99: CHIP Error 0x00000002: Connection aborted
[MatterTest] 11-28 08:25:18.756 INFO Commissioning complete for node ID 0x0000000012344321: src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error
[MatterTest] 11-28 08:25:18.757 WARNING Failed to commission: src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error
[MatterTest] 11-28 08:25:18.758 ERROR Commissioning failed: src/controller/CHIPDeviceController.cpp:1329: CHIP Error 0x000000AC: Internal error
[MatterTest] 11-28 08:25:18.780 INFO [Test] test_run_commissioning FAIL
[MatterTest] 11-28 08:25:18.787 INFO Summary for test class CommissionDeviceTest: Error 0, Executed 1, Failed 1, Passed 0, Requested 1, Skipped 0
[MatterTest] 11-28 08:25:18.787 WARNING Abort all subsequent test classes. Reason: Details=All remaining tests aborted due to: Failed to commission node, Extras=None
[MatterTest] 11-28 08:25:18.790 INFO Summary for test run MatterTest@11-28-2024_08-25-17-216:
Total time elapsed 1.57073734595906s
Artifacts are saved in "/tmp/matter_testing/logs/MatterTest/11-28-2024_08-25-17-216"
Test summary saved in "/tmp/matter_testing/logs/MatterTest/11-28-2024_08-25-17-216/test_summary.yaml"
Test results: Error 0, Executed 1, Failed 1, Passed 0, Requested 1, Skipped 0
Is there any way to determine the cause of the error?
Platform
python, raspi
Anything else?
https://github.com/project-chip/connectedhomeip/blob/17b1a38e909e7874593bcb87c31be03a5866f1d4/src/python_testing/TC_AccessChecker.py

The reason was that paa certificates were not added to the docker container.


Fix CMake not finding GLEW

I noticed that building failed while looking for the GLEW library with the new CMake file despite working fine with the old CMake file. Upon further investigation, I discovered that there was a capitalization mistake in the file that caused the GLEW library to not be linked correctly.

I've read over that article, and I've modified the code so it targets GLEW with the namespace version, as you suggested. I've tried to do this for the other libraries too, but I've been unable to find the cmake Find modules for them, where these variables are defined.

If you cannot find the CMake script, you can either make one or use the find_library function. Probably the later is a simpler solution.

I've been having great difficulty with trying to use find_library. I can't figure what this function is looking for (The sources or a cmake file?). I am also worried that since I have to supply an absolute path to find_library, the multi-platform support of the project will suffer. But I will keep at it until I get it to work.

It looks like I've got it to work. GLM is a header-only library, and for that reason the libraries for it do not exist, and including the headers is enough. For GLFW, I've had to find the library as well as include the headers. I've also changed the Threads library to be linked with the namespace notation.
My only worry now is that for the include paths, I've used absolute paths, and I am not sure if that will translate well for Windows and OSX.

I've added Find Modules for GLM and GLFW. I've attached the licenses and the links where I found them inside of the module files.


Downloadable Session Data

Is your feature request related to a problem? Please describe.
Not related to a problem, imo a necessary feature. Downloadable session data so we can have everything in a neat package for claims/taxation relief purposes. The small frontend tweaks will be listed below.
Describe the solution you'd like
[Backend] @PScoriae
Ideally the data will be downloaded as a compressed file that includes:

a formatted text file containing session data + namelist (text file renamed to session-date-time.txt)
a folder with all the receipts (with all the images renamed to username-session-date-time.filetype)

[Frontend] @Kirixi
I suppose adding a download icon somewhere on the "receipts" page would work.
Since we're talking about the receipts page, one polish note I want to give is the 'Upload Receipts' popup a bit more clarity. Right not it's not immediately obvious you have to click the image icon to upload the receipt. I'd like to add some text like "Click here to upload your receipt." Only image files accepted.

Sidenote :
@PScoriae @Kirixi , I am wondering if we should open it up for PDF files as well since most transaction receipts come in PDF. But I am worried about malicious files being uploaded. Should we or should we not accept the format? If we do accept, how can we prevent/check for a bad file?
Describe alternatives you've considered
Have the data be stored on a share that hosts can access and retrieve the files from. Would be a faster way, but not as neat as having it on-site.

Session Data

a formatted text file containing session data + namelist (text file renamed to session-date-time.txt)

Can you show an example of what kind of data is needed how the txt file should be formatted?
Would a CSV be better?
PDF Uploads

I am wondering if we should open it up for PDF files as well since most transaction receipts come in PDF. But I am worried about malicious files being uploaded. Should we or should we not accept the format? If we do accept, how can we prevent/check for a bad file?

The backend currently checks for the file's mimetype (see Media type, f687aef).
If it isn't of type image, then the server denies the upload.
With this method, files with their extension changed, such as from .exe to .jpg, should also be blocked.
We can go ahead with PDF uploads if that's not a concern anymore.

@PScoriae
Regarding what type of data is needed it should be written out and formatted like :
Session Created by : Amelie Loh
[Tuesday] 30-2-23
Courts : 1,2,3,4
Cost Per Person : 8.79
Amelie
Jon
Pie
Teh
Michael
Basic data that is already viewable on site but just listed for keepsake. Any file type is fine as long as it has basic formatting.
As for PDF's, let's go ahead with accepting PDF's then. Also I want to test it out, see what'll happen to a malicious file.

Not cause I don't trust it, but I'm not sure if banks will always do PDF's the same


how to cache express session in redis cluster

usually often , we cache express session like :
var session = require('express-session');
var RedisStore = require('connect-redis')(session);
app.use(session({
store: new RedisStore(options),
secret: 'keyboard cat'
}));
now , after we use oiredis , i don't know how to cache express session in redis cluster
and i want to use session like under normal conditions :
req.session.user = 'hello world';
var user = req.session.user;
can u give an example ?

There should not be any difference. Please refer to the documentation on connect-redis: https://github.com/tj/connect-redis#custom-redis-clients

`
var session = require('express-session');
var Redis = require('ioredis')(session);
var redis = new Redis.Cluster([{
host:'<IP_ADDRESS>',
port:123
}]);`
that is it ?

var session = require('express-session');
var RedisStore = require('connect-redis')(session);
var Redis = require('ioredis');
var redis = new Redis.Cluster([{
  host:'<IP_ADDRESS>',
  port:123
}]);
app.use(session({
    store: new RedisStore({ client: redis }),
    secret: 'keyboard cat'
}));

I didn't try but the code above should do the trick.

Do I need for the redis client to be connected before calling app.use(session..?

@tnguyen14 No, you don't

On logout, getting this error, Unhandled rejection ReplyError: CROSSSLOT Keys in request don't hash to the same slot. Everything else works. Tried doing Hashtag by making the prefix to be {sess}.


#60

We are using Transifex for translations.
Please request to join the team if you want to help.


data-producers variable in update.sh does not support multiple accounts

Fies #438, #425
All Submissions:

[X] Have you followed the guidelines in our Contributing document?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[X] Does your submission pass tests?
[X] Have you lint your code locally prior to submission?

@swilliams11 I tried a couple of things, and I think it will work ok as is. The issue that we had, was with wildcards from what I recall. I was able to work around that by including the double quotes within the string:
export<EMAIL_ADDRESS>

@swilliams11 I tried a couple of things, and I think it will work ok as is. The issue that we had, was with wildcards from what I recall. I was able to work around that by including the double quotes within the string:
export<EMAIL_ADDRESS>


PHOENIX-4797  file not found or file exist exception when create glob…

when create global index use -snaopshot option in mapreduce task
multi mapper use the same restore dir to work with snapshot.
1,mapper will remove the file  after mapper task as the other mapper will read the file
2,mapper will create the file when begin mapper task as the other mapper will create the file too
the 1 will cause the file not found and the 2 will case the file is exists
I change this to let mappers use different dirs to work with snapshot,after this patch,the  -snapshot option can works well .

hi, Thanks for reminding me, I remove the wildcard imports.

Thank you for your advice，I've changed it to use both ”String“ Path api;


Allow HTML for helper text in fields

Feature request type
Enhance component
Component name
MudInputControl
Is your feature request related to a problem?
No response
Describe the solution you'd like
Change MudInputControl.HelperText from string to MarkupString so that we can include HTML markup (I only really want to provide very simple tags like <strong>).
Have you seen this feature anywhere else?
No response
Describe alternatives you've considered
No response
Pull Request

[X] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@henon @danielchalmers wouldn't that be a security hole similar to this? https://github.com/MudBlazor/MudBlazor/issues/8146

@henon @danielchalmers wouldn't that be a security hole similar to this? #8146 There is only solution if HelperText was a RenderFragment instead of string.

Right. Other concern is the breakage in existing code. Maybe a new HelperContent parameter e.g.

Yes RenderFragment is the way to go.


Reset styles and rerun animation

Hello,
i want rerun an animation, after i removed the extra styles, which
added velocityjs. But it is not working. Here is a fiddle fiddle.
var element = document.getElementById('animate');

var animate = function()
{
  Velocity(element, {translateX: 500},  {duration: 2000, queue: false, complete: function(){
		element.removeAttribute('style');
 		animate();
  }});
};

animate()

Have someone any idea, to manage this? Is there a reset function or something?

http://velocityjs.org/ - Read the "Secret Sauce" part near the top - basically Velocity caches the values, so removing them makes no difference.
Use ForceFeeding to provide the starting values you want to use - http://velocityjs.org/#forcefeeding

Hi Rycochet,
i hope you can answer on closed Issued. Exactly what i have fiddled, i need for banner ad development. And in this case is easer to not know the "starting values". Can the Cache be cleared by a function like "reset" - or "clear cache"? If not maybe its a good new feature.

Add the non-public _cacheValues:false to the options on the call - this might change in the future so we can give a public method to clear the cache rather than not saving it...

Thank you, yes i've tried this parameter before, but this parameter has no effect at all:
https://jsfiddle.net/1gxLuszc/2/

Damn, it definitely looks like it should work from the source, can confirm that it doesn't work. There is a hacky way to clear internal data, but I can't remember what it is (having never used it or looked at the cache code) - instead could you clear the values via Velocity with a duration of 0?


Automatically initialize PT Run with currently selected text

Description of the new feature / enhancement
It would be very handy to automatically initialize PT Run with currently selected text at the moment of pressing Alt+Space (at least as an option disabled by default but ideally — as the default one)
Scenario when this would be used?
It would save an enormous amount of Ctrl+C; Ctrl+Vs
Supporting information
No response

From what I have discovered while looking to do something similar for Text Grab. There is no "selected text" API in Windows. Do you know of any open source application which does this so we can see how it might be done? /needinfo

Hmm, this one is recurring when trying to google something relevant: http://fy.chalmers.se/~appro/nt/TXMouse/ (though I was unable to find the source code), so there must be a way…
Perhaps this application monitors all cursor related events in background (since there's no dedicated API).

I have no idea about Windows APIs, but with Autohotkey you could "copy, open Run, paste", so I guess it's possible.


chore(scully): fix commit script

Fix #155
Remove git add .  from  commit script
PR Checklist

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

PR Type
Remove git add . from commit script.

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #155
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Other information

@Villanuevand i create another PR for this, i dont like remote the git add . I really prefer use this by default... so, i create another script just for run the commit with the files add manually

Duplicated!
#158


Plex media player component may no longer honor entity_namespace config setting

Home Assistant release with the issue: 0.66.0

Last working Home Assistant release (if known): Unsure, haven't used the functionality in a release or two
Operating environment (Hass.io/Docker/Windows/etc.): venv (Python 3.6.3) on Ubuntu 17.10

Component/platform: Plex (media player)

Description of problem:
My config:
media_player:
  - platform: sonos
    name: Sonos
    entity_namespace: 'sonos'
    hosts:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
  - platform: plex
    name: Plex
    # use_custom_entity_ids: true
    entity_namespace: 'plex'

It seems as if Sonos is still honoring the namespace setting, but Plex isn't, creating entities like media_player.living_room_2 which used to be media_player.plex_living_room_2 in previous releases (and the naming some of my automations rely on).  I'm not sure if this is a deprecated setting, in light of the entity registry, but it seemed to still be in the docs.
As well, using the gear icon to try to change the entity name in entity_registry.yaml from the frontend throws an error about saving (works for other platforms).
Thanks !!
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
media_player:
  - platform: sonos
    name: Sonos
    entity_namespace: 'sonos'
    hosts:
      - <IP_ADDRESS>
      - <IP_ADDRESS>
  - platform: plex
    name: Plex
    # use_custom_entity_ids: true
    entity_namespace: 'plex'

Traceback (if applicable):

Additional information:

The namespace still doesn't seem to be honored in .73.1
Just bumping this to see if it's on the radar or if the configuration docs should be notated to avoid confusion

i can confirm this is still an issue with 0.81.2

Got the same issue on 0.81.6

Is this still an issue you are experiencing? Can you please try upgrading to the latest version of Home Assistant (0.90) and report back if this is still a problem? Thanks!

Setting custom entity_id cannot work properly with the Entity Registry and really is not needed because the registry takes care of the conflicts that the customization (including prefixes) should avoid.
The right fix is to remove the code and documentation for this legacy feature.


[BUG] UnixTimeStampTest may fail due to floating point precision

What is the bug?
UnixTimeStampTest.checkNoArgs() may fail sometimes and floating precision seems the root cause. See error logs below:
UnixTimeStampTest > checkNoArgs() FAILED
    org.opentest4j.AssertionFailedError: expected: <1667255816> but was: <1667255817>
        at app//org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:55)
        at app//org.junit.jupiter.api.AssertionUtils.failNotEqual(AssertionUtils.java:62)
        at app//org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:182)
        at app//org.junit.jupiter.api.AssertEquals.assertEquals(AssertEquals.java:177)
        at app//org.junit.jupiter.api.Assertions.assertEquals(Assertions.java:618)
        at app//org.opensearch.sql.expression.datetime.UnixTimeStampTest.checkNoArgs(UnixTimeStampTest.java:34)

Code:
  @Test
  public void checkNoArgs() {
    assertEquals(System.currentTimeMillis() / 1000L, unixTimeStamp());
    assertEquals(System.currentTimeMillis() / 1000L, eval(unixTimeStampExpr()).longValue());
  }

How can one reproduce the bug?
N/A
What is the expected behavior?
Without nondeterministic behavior, UnixTimeStampTest.checkNoArgs() can pass all the time.
What is your host/environment?

Github action failed on 2.x branch

Do you have any screenshots?
N/A
Do you have any additional context?
N/A

Depends on #937/#960 as well.


Uitdatabank v3

def uit_migrate():
  from solutions.common.migrations.uitdatabank_settings_split import job

  job()

V1 doesn't seem to be working properly, tested with both lochristi and beveren and it would always come up with 0 results. I can save the settings properly, but when running the cron there's 0 events. Perhaps they've already shut down indexing new events.


Sign in modal?

While considering the idea of implementing a custom title bar I started thinking about other areas where UI can be improved.
Although sign-in is a rare occurrence, a quite pleasing experience would be having a loading window (instead of the blank window) as well as a separate story for sessionless application starts that has a smaller modal window representing this page (or the modal launched from front page). Thoughts?

Not sure I understand what you are suggesting here... soundcloud.com works without logging in (to some extent) therefore enforcing a "redirect to sign in" is not something we should do in my opinion.
The whole sign-in implementation is quite messy both on soundcloud.com (see how different the email/Facebook/Google flow are) and the code supporting it in SoundCleod (see hacks here for example), we'd better not add more hacks to the soup.

Closing; lets keep soundcleod a "native" experience.


missing bower.json file

Hi. I can't see any bower.json file in this repository. Have you missed it?

Hi, I use Windows, so I dont use Bower. I have added it in this commit for people that want use it.


input type="range"  : style to match progress

In a form control where input type="range" is desired, the tag renders with a bland gray bar & handle from vanilla chrome/firefox styling.  A pretty blue bar with a big square for a handle would be nice.

Hey @ndotb, can you describe your use case for the range input in a little more detail, and do you have any designs you could share to show it in use in your application?

I try to avoid many of the html5 input types as they have been implemented on desktop browsers, but the the range control is very good for applying quantitative values to qualitative traits.  Kind of like when you rate your pain level at the doctor's office on the Wong-Baker scale with the smiley faces.
I would need to rough out css, but here's some simple mocks for a "how would you rate your experience" type of prompt:
normal

with high threshold

with low threshold

Thanks! We have an open feature request for a slider component that I think would serve the use case here https://github.com/patternfly/patternfly-next/issues/296
cc @LHinson @mcarrano @rachael-phillips what is the status of the slider component?

Hi @mcoker it is not on the roadmap at this time, but I will place this in the spreadsheet for future conversations.


switches refactoring

Solves https://github.com/onmetal/controller-utils/issues/36

@adracus Can we merge it?

@GrigoriyMikhalkin please fix the remaining conversations, then we can merge.

@adracus Done


Allow TTML attributes from TTML metadata namespace

The following change needs to be reflecred in the EBU-TT-D XSD for version 1.0.1
CHANGE: Attributes from the TTML metadata namespace
Before
The TTML 1 metadata attributes ttm:agent and ttm:role were allowed on tt:body, tt:p and tt:span.
After
TTML 1 metadata attributes can be used as defined in TTML 1.

This will now be resolved by the resolution to #22, i.e. #23.

Actually closed by #24.


Make all models consistently use the new image_urls property.

The JSON payload now includes a image_urls property which contains full URLs, meaning no interpolation is needed.
Related to #163 and https://github.com/artsy/eigen/pull/173.

want this in the fairs milestone?

I was thinking to specifically delay this after all fairs stuff is done.

:+1:

@1aurabrown This is still on-going, yeah?

yeah, haven't touched it in a little while


⚠ Bofrost.de has degraded performance

In e54c499, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 9084 ms

Resolved: Bofrost.de performance has improved in bad5d1a after 6 minutes.


Removing methods from a subclass

Sometimes I'd like to subclass, but disable or delete some of the methods I don't need, and don't want to have to deal with. If there was only one or two, I'd just include them and overwrite them with a NotImplemented error. But when there's more, I start copy/pasting and have an eery feeling of "there must be a better way".
Here's an example where I'm subclassing deque, and only want append and extend methods.
from collections import deque
from functools import wraps

class InfiniteSeq(deque):
    @wraps(deque)
    def __init__(self, *args, **kwargs):
        super().__init__(self, *args, **kwargs)
        self.max_idx = 0  # should correspond to the number of items added

    def append(self, x) -> None:
        super().append(x)
        self.max_idx += 1

    def extend(self, iterable: Iterable) -> None:
        iterable = list(iterable)
        super().extend(iterable)
        self.max_idx += len(iterable)

for attr in {'appendleft', 'extendleft', 'insert', 'pop', 'popleft', 'remove', 'reverse', 'rotate', 'clear'}:
    delattr(InfiniteSeq, attr)

Asked question here: https://stackoverflow.com/questions/66216789/healthy-ways-disable-delete-methods-in-a-subclass

To transform methods in bulk consider:

metaclasses
class decorator

from collections import deque
from functools import wraps

class InfiniteSeq(deque):
    @wraps(deque)
    def __init__(self, *args, **kwargs):
        super().__init__(self, *args, **kwargs)
        self.max_idx = 0  # should correspond to the number of items added

    def append(self, x) -> None:
        super().append(x)
        self.max_idx += 1

    def extend(self, iterable: Iterable) -> None:
        iterable = list(iterable)
        super().extend(iterable)
        self.max_idx += len(iterable)

      for _attr in {'appendleft', 'extendleft', 'insert', 'pop', 'popleft', 'remove', 'reverse', 'rotate', 'clear'}:
          locals()[_attr] = decorator(getattr(locals()[_attr])
      del _attr


feat: adding an example dashboard

This PR adds an example dashboard for grafana
Its a slightly modified dashboard from kubecost/opencost to work with the service monitor changes (and has a variablized datasource for things like mimir/cortex)

@Pokom @logyball @toscott @sntxrr if any of you have the cycles to vet this, I think it's worth putting an example Grafana dashboard in.

Hey @mattray, I added myself as a reviewer and will try to get to this in the next week or so. This request keeps coming up and I think having a foundation of dashboards would be excellent to have.
@ts-mini would you mind rebasing your changes with main?

yup! i might have to get to this monday tho, video games are calling

Just initial thoughts/feedback:
The dashboard looks lovely and was usable right of the box! The breakdown of Cluster Wide vs Namespace vs App is great. The one that didn't quite fit in(IMO) was the breakdown for PVC's and for two reasons:

PVC's should be associated with workloads(apps, namespaces) to give better context
When you have 1000's of PVC's, it takes minutes for this to load

The one thing I couldn't quite get working is App Dropdown. For me it's just forever "none", but this may be more to do with my internal setup then the dashboard itself.
Something that we should consider: A lot of these queries are fairly expensive to run as the datasets get large(100's to 1000's of node clusters). It would be valuable to set some of the queries up as recording rules that we ship alongside the dashboards so the dashboards are more performant.
Great contribution @ts-mini! We can iterate on this a bit next week so we can both get some video game time in :wink:

@Pokom is this good to merge? We can always continue to iterate over it.


Fix issues with preq, module code, UG

[X] Add details for preq to UG and troubleshooting steps for launching
[X] Standardise error message for invalid / non-existent module code across add, view, preq

Codecov Report
Base: 39.79% // Head: 39.81% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (90e7fcf) compared to base (275e64a).
Patch coverage: 80.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #167      +/-   ##
============================================
+ Coverage     39.79%   39.81%   +0.01%
  Complexity      464      464
============================================
  Files           122      122
  Lines          3106     3107       +1
  Branches        312      312
============================================
+ Hits           1236     1237       +1
  Misses         1763     1763
  Partials        107      107

Impacted Files
Coverage Δ

...n/java/nus/climods/logic/commands/ViewCommand.java
0.00% <ø> (ø)

...s/logic/parser/parameters/ModuleCodeParameter.java
100.00% <ø> (ø)

...in/java/nus/climods/logic/commands/AddCommand.java
70.37% <75.00%> (+1.13%)
:arrow_up:

...ava/nus/climods/logic/commands/PrereqsCommand.java
90.90% <85.71%> (ø)

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

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


I encountered a problem. When I make the phone's network unavailable, PaypalButton shows loading, which is normal. But when I make the mobile network available, PaypalButton still shows loading. Even restarting the app will not solve this problem.

I encountered a problem. When I make the phone's network unavailable, PaypalButton shows loading, which is normal. But when I make the mobile network available, PaypalButton still shows loading. Even restarting the app will not solve this problem.

Thanks for reporting this @panguojian. I was able to reproduce the issue but was seeing the PaypalButton load properly after restarting the app (when the Activity containing the PaypalButton is recreated).
I've added an item to our backlog to gracefully handle changes in network connectivity.

The work has been picked up and should be in the next release of the SDK.

Hey @panguojian. The 0.4.0 release is out. This release added a network listener to the payment button. Now when a buyer gains connection, the payment button will render.
implementation 'com.paypal.checkout:android-sdk:0.4.0'

Closing this issue. @panguojian, feel free to respond here if you're having issues with the Payment Button responding to network status.


Update package.json to include dependency

Classifier functionality has a dependency not listed in package.json.  This commit resolves this so users who use npm install won't have their code break.

thank you!


Issue #1433 part 2/2: Remove code duplication in FlowWatcher tests

This completes fixing issue https://github.com/azkaban/azkaban/issues/1433 (1/2 was PR https://github.com/azkaban/azkaban/pull/1533).
"Extras":

LocalFlowWatcherTest was actually ignored; I made it run quickly and enabled it.
I also found & fixed a bug in both FlowWatcher tests: they were asserting pipeline level 1 twice, despite the test names. I fixed those "level 2" tests to run with and assert pipeline level 2 instead of level 1.

It appears that GitHub will offer the commit message sometimes as the final commit message instead of the pull request description.
I should have copied the pull request description when I merge.


Get encryptor AMI automatically during update

Get the encryptor AMI from the service during update-encrypted-ami.
Remove the --updater-ami option and add a hidden --encryptor-ami option
for development.  Remove the extra call to validate_encryptor_ami().
The validation is already being done inside _connect_and_validate().

Nice, lgtm


Don't let the Agent write to the configuration file.

It will be provided by cloud-setup-agent.
Integration tests: https://beta-jenkins.mcc.schubergphilis.com/job/cosmic/job/0002-tracking-repo-branch-build/47/

Ping @bheuvel @miguelaferreira @remibergsma @wilderrodrigues

Spinned a cloud and that works fine. The agent.properties now won't be overwritten and the one that is put there by the setup script (the one with the ACS header and a lot of comments) is still there.
Verified that stop/start of the cosmic-agent still works as expected (it connected and I could start a new vm).
This way, one can do config mgt on the agent.properties file without the agent writing to it (it did basically reordering / removing comments only).
👍


Watch mode to circular dependencies

Is there a way to use madge CLI in a watch mode to circular dependencies? Since that we are developing, have it running in a console it will be nice.
Thanks

As a quick approach, I'd search for any module that lets you run a command on a file change and wire that up to run the madge command you're using.
Another approach is to add support for a new watch option in Madge that uses a file watcher (like the sane module) that invokes madge's entry point with the given arguments; you'd effectively end up rerunning madge on every file change.
The best but more involved solution is to layer on caching so that madge/dependency-tree only recompute parts of the tree that have changed. With the previous methods, the files that have changed are ignored and only used as a trigger to rerun madge. This would take some exploration but would be really cool.


If we don't want hot reloading but want load assemblies into memory t…

…o keep files lockfree in order to able to reload manually in future.

I also thought about this myself. I think this should be merged. I have exactly this use case.


Custom Apache access log format

Hello,
Could you please help me with the configuration of the log format I'm using?
Here's a line that shows it in the access.log:
[28/Feb/2018:09:09:00 +0100] [www.website.com:443] [client <IP_ADDRESS>] "GET / HTTP/1.1" 200 62972 "Mozilla/5.0 (Windows NT 6.1; WOW64; rv:40.0) Gecko/20100101 Firefox/40.1"

Thank you in advance.

Please try this:
goaccess access.log --log-format='[%d:%t %^] [%v:%^] [%^ %h] "%r" %s %b "%u"' --date-format=%d/%b/%Y --time-format=%T --ignore-panel=REFERRING_SITES

Perfect! It worked like a charm.
Thank you very much!


Add fizzbuzz.py

I have written the test cases but I'm not sure how exactly should they be put with all the current repo overhaul going on. @christianbender Let me know If you need the test cases.

Can anyone explain to me what the percentages mean at the end of the travis build? Do they represent code coverage or something else? @goswami-rahul @christianbender @danghai
It is something that I just want to know. More knowledge more power. 😀

@SaadBenn You don't need test cases. This problem is quite straightforward. I know nothing about travis. I can you not help.

So should I merge this one? @christianbender

Your question makes me think about coverall. I add coverall in algorithm through PR:
https://github.com/keon/algorithms/pull/261


Update to 1.18 (Fabric)

I updated the Fabric version of the mod to 1.18 and added the new 1.18 songs to the language file!
This is my first time updating a mod, so I might have done something incorrectly. Let me know if I did anything wrong.
There is also a problem:
Some of the new song names are too long to fit in the toast notifications.
This could be fixed by implementing #7, but I don't have enough experience with Java so I can't do it myself.

Thank you for the contribution. I'll have to check when I can test this, but from a first glance, everything looks good.


Azure Logic Apps: Fix schema source for Standard

@ecfan : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Redo in private repo

#assign:@ecfan


[TextField] When using type="time", the step is not working

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When I do try to select time as specified here with a 300 steps, time is stil showing ...31,32,33minutes
I was expecting to see only 5,10,15... minutes there
material-ui/docs/src/pages/components/pickers/TimePickers.js
Expected behavior 🤔
I was expecting to see only 5,10,15... minutes there
Steps to reproduce 🕹
Steps:

open https://codesandbox.io/s/material-demo-forked-y302jh
see the same behavior
Steps is not acting in any way
Please confirm

Context 🔦
react hooks
Your environment 🌎
https://codesandbox.io/s/material-demo-forked-y302jh

Sorry I transferred it to fast, the issue concerns the TextField, not the TimePicker
I let the core team handle it

Set type to time and step to 300. It's not that you can't select minutes that are not multiples of 5, but select the minutes and press the up and down buttons on the keyboard. The minutes will increase or decrease in steps of 5.

Set type to time and step to 300. It's not that you can't select minutes that are not multiples of 5, but select the minutes and press the up and down buttons on the keyboard. The minutes will increase or decrease in steps of 5.

@malutanpetronel Does this work for you?


TS-4729: Remove dead assaignment in Http2Stream

TS-4729

I assume is_done is still in use elsewhere.
👍

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/514/ for details.

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/411/ for details.

Also, there's a typo in the commit subject line ;)

#849 is the new one by @shinrich


add RESOURCES_ALL constant and StoreObject::resource_types

Nightly clippy is going to continue to fail because of the addition of empty_doc_comment, which apparently interacts poorly with #[wasm_bindgen]: https://github.com/rust-lang/rust-clippy/issues/12377


Yatin: Watch besttenrackets.com

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

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

Approved by Spevacus in Charcoal HQ


Scrapper, Ratmir Akhmatzyanov - 22FPL1

No activity in PR. It is right time to start working on it.

No activity in PR. It is good time to start working on lab assignment

No activity in PR. It is good time to start working on lab assignment

No progress since last review

No progress since last review


fix we click on close button or cross icon

Targets #{1523}
Modal not closing when we click on the close button or cross icon

Due to some problem in the file I had close the PR


Is engine thread safe?

Hi,
I want to use multi-thread to re-use engine, is this thread safe?
or any recommendations?

the class infer under TRT is not thread-safe. But the class infer in yolo is thread-safe. Check the the design in yolo.
TRT 下的infer 不是线程安全   但是yolo 下的infer 下是线程安全    所以如果想要考虑线程安全 可以参考yolo 的设计


[ICON]: Terraform files

File Type
*.tf, .terraform.lock.hcl
(Optional) Propose an Icon
Thoughts on something like:

https://oldschool.runescape.wiki/w/Plank_Make
https://oldschool.runescape.wiki/w/Kharyrll_Teleport

Mostly because is an infrastructure construction or the teleport animations that create blocks.

Since its infrastructure as code software, another option would be the Construction icon or Farming icon.
That said to fit in the theme of "terraforming" another option would be the Geomancy spell (looks pretty neat too).

Yes, Geomancy seem a pretty good icon for it.

Completed with commit 9a2566f


Header (column names + histograms) aren't sticky Safari

E.g. using this example data: https://flatgithub.com/the-pudding/data?filename=boybands%2Fboys.csv&sha=9b4e89ed178e11bf70a9b3d8ffa57ac2ea181b96
In Firefox and Chrome, this works well: the body scrolls while the header stays in place:
https://user-images.githubusercontent.com/6945891/130460177-9d4c4cd7-43bb-4c41-b314-a55be1d43f69.mp4
However in Safari, the header scrolls away too:
https://user-images.githubusercontent.com/6945891/130460376-680677ef-f6c1-484e-a139-3e352339a0a2.mp4
I seem to recall having a somewhat similar issue previously with my own code, with flex acting differently in Safari, however the details don't come directly to mind. I can try at least to have a quick look to see what's causing it here!
I'm on macOS Big Sur 11.3.1, Safari 14.1.
Thanks!

thanks for reporting this! I dug around a bit and it looks like the issue is only on desktop, when the left column is stickied. Still a very strange bug, maybe related to using inline-flex for the sticky header cells? I'd love if you had any ideas!

Ah seems that's basically the problem: for Safari, sticky only works in block-level elements.
I set up a quick example here: https://stackblitz.com/edit/js-hftfti?file=index.html (direct link: https://js-hftfti.stackblitz.io), hopefully it's clear. The "default" header (block) and the explicit block work in safari, but not the inline-flex or inline-block (but all four work elsewhere).
Would it be a ton of work to have the sticky headers not be inline-flex?


Add Japanese translations for Japanese subdivisions

add Japanese translation for Japanese prefectures
'01':
  unofficial_names:
  - Hokkaidô
  - Hokkaido
  translations:
    en: Hokkaido
  geo:
    latitude: 44
    longitude: 143
    min_latitude:
    min_longitude:
    max_latitude:
    max_longitude:
  name: Hokkaido

to
'01':
  unofficial_names:
  - Hokkaidô
  - Hokkaido
  translations:
    en: Hokkaido
    ja: 北海道
  geo:
    latitude: 44
    longitude: 143
    min_latitude:
    min_longitude:
    max_latitude:
    max_longitude:
  name: Hokkaido

Could you please submit a PR?

Closed by #344


Retain whitespace before color().  Fix for #26.

Replace balance-matched with postcss-value-parser to fix issue whitespace removal issue #26.

I also have this working with postcss-value-parser in a separate branch:
https://github.com/studiosciences/postcss-color-function/blob/e91a9d7f72e660699d46e1b968918575cc29e620/index.js
However, I wasn't 100% sure that it was ok to replace the node in the way I am...

Change the type to word.
Insert the rgba() as value.
Delete the child nodes array.

If you want me to update the Pull Request to that instead, let me know.

Merged in the branch that used postcss-value-parser.
Minor Issue: We call .toString() even if no actual color functions were found. This would only occur if there was a "color(" in the value that was not actually a color function, such as "hover-color(".

@studiosciences can you squash your commit (especially the lasts)?
Also why version number needs a bump? According to tests, it's a bugfix.

@MoOx I haven't squashed before, but I'll see what I can do.
I'll leave any version changing to you.

Squashing complete. It appears as one commit in my branch, but the Pull Request lists six.
Not sure how to fix that or if anything needs to be fixed.

There is one commit, just refresh the page :D

Weird. I saw the squashed commit as an additional commit. Looks right now.

Anything more I could do to wrap this up?

Oh sorry, I forgot. Let me handle this this evening.

Released as 2.0.1, thanks!


ORC-813: Upgrade ZSTD to 1.5.0

What changes were proposed in this pull request?
This PR aims to upgrade ZSTD from 1.4.5 to 1.5.0.
Why are the changes needed?
This will bring in the latest performance improvements and bugfixes:

https://github.com/facebook/zstd/releases/tag/v1.5.0
https://github.com/facebook/zstd/releases/tag/v1.4.9
https://github.com/facebook/zstd/releases/tag/v1.4.8
https://github.com/facebook/zstd/releases/tag/v1.4.7

How was this patch tested?
Pass the CIs.

cc @wgtmac

Thank you for making a PR, @dchristle .
BTW, this PR updates Apache ORC C++ reader and writer while Apache ORC Java Reader and Writer is still behind because of the following. We need to verify that native ZSTD 1.5.0 works with old air compressor library.
      <dependency>
        <groupId>io.airlift</groupId>
        <artifactId>aircompressor</artifactId>
        <version>0.16</version>
      </dependency>

I have checked the release note and haven't found any format incompatible issue.

Thank you, @wgtmac . I'll leave this for you. Feel free to merge this for Apache ORC 1.7.0.

@dchristle . I added you to the Apache ORC contributor group and assigned ORC-813 to you. Thank you again.


Update .openpublishing.publish.config.json

remove output folder, reduce the legacy feature, this should has no impacts to build and publish

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] An exception occurred during the build process. Please check the raw log and retry building your content again. If the issue persists, please open a ticket in http://SiteHelp and include the URL of publishing report.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


New screen option: ShowTitleBar

I added new option in the screen, which allowes App Inventor users to enable/disable ShowTitleBar. This method is in the form of the getter method in the block editor and as property in the designer property list. But there won't be any change in the designer viewer if you decide to change them. Future work, make the changes reflect on the designer viewer.
We can't really implement the method which Dan (from Google) has suggested -  use Window.requestFeature() with FEATURE_NO_TITLE . Because the property is called after the setContentView. This approach will crash the app. We can change it in the manifest, "via the window theme, e.g@android:style/Theme.Black.NoTitleBar", but this hides all screens' title bar.
I want to give the maximum design freedom to App Inventor users, so I have proposed the following method. All the API calls exist since API level 1.
The old commit is here -
https://github.com/mit-cml/appinventor-sources/pull/185

@halatmit @jisqyv @afmckinney

is this called the title bar or the action bar?

I tried this and setting showtitlebar to false removes the title, but it
does not remove the gray bar itself.
can you show me a demo of how it is supposed to work?

I know you want to give people maximum control, but I suggest that the
useful simple case is to either have no bars (fullscreen option) or to have
both bars.

== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Fri, Jan 16, 2015 at 6:00 PM, WeiHua Li<EMAIL_ADDRESS>wrote:

I added new option in the screen, which allowes App Inventor users to
enable/disable ShowTitleBar. This method is in the form of the getter
method in the block editor and as property in the designer property list.
But there won't be any change in the designer viewer if you decide to
change them. Future work, make the changes reflect on the designer viewer.
We can't really implement the method which Dan (from Google) has suggested

use Window.requestFeature() with FEATURE_NO_TITLE . Because the property
is called after the setContentView. This approach will crash the app. We
can change it in the manifest, "via the window theme, e.g@android:style/Theme.Black.NoTitleBar",
but this hide all the title bar.

I want to give the maximum design freedom to App Inventor users, so I have
proposed the following method. All the API calls exist since API level 1.
The old commit is here -
#185 https://github.com/mit-cml/appinventor-sources/pull/185
You can view, comment on, or merge this pull request online at:
https://github.com/mit-cml/appinventor-sources/pull/222
Commit Summary

Added new screen option: ShowTitleBar
Updated the component documentation - ShowTitleBar

File Changes

M
appinventor/appengine/src/com/google/appinventor/client/OdeMessages.java
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-0
(4)
M
appinventor/appengine/src/com/google/appinventor/client/TranslationComponentProperty.java
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-1
(1)
M
appinventor/appengine/src/com/google/appinventor/client/TranslationDesignerProperties.java
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-2
(2)
M
appinventor/appengine/src/com/google/appinventor/client/youngandroid/YoungAndroidFormUpgrader.java
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-3
(4)
M
appinventor/components/src/com/google/appinventor/components/common/YaVersion.java
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-4
(9)
M
appinventor/components/src/com/google/appinventor/components/runtime/Form.java
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-5
(36)
M appinventor/docs/reference/components/userinterface.html
https://github.com/mit-cml/appinventor-sources/pull/222/files#diff-6
(2)

Patch Links:

https://github.com/mit-cml/appinventor-sources/pull/222.patch
https://github.com/mit-cml/appinventor-sources/pull/222.diff

—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222.

This is called Titlebar in the old device (<= 2.3), Action bar in the newer device.
Are you running on Lollipop?
How about I do it at the manifest level? so it is one for all the screens? It is much easier, simpler that way?

FullScreen in Android means different thing => no title bar (action bar) + no status bar + plus no home key panel. Do you mean by no title bar + no status bar? @halatmit

If we have to choose, I suggest the modern nomenclature (action bar).   But think about my suggestion to just have either both bars or none (fullscreen).   I don't know about implentation.  It would be good to dynamic contol rather, which I'm guess means it cannot be done in the manifest.  See if this runtime method works:
http://stackoverflow.com/questions/2868047/fullscreen-activity-in-android

Thanks for the pointer. There is no action bar in App Inventor, only title bar...
Dynamic control can't be done at the manifest.
I already saw and tried the openstack approach. It doesn't really work in App Inventor. The code " requestWindowFeature(Window.FEATURE_NO_TITLE);" must be called before the "setContentView" (within Form.onCreate()). If I have a property "ShowTitleBar" in the property list in the Designer and has the corresponding setter method "requestWindowFeature(Window.FEATURE_NO_TITLE)", this will crash the app. The method "requestWindowFeature(Window.FEATURE_NO_TITLE);", can't be call after the setContentView.
The approach I am using now seems have compatible issue with Lollipop, which I didn't test. Did you try out anything lower than 5.0 to see if that works? @halatmit

The emulator I tried it on is an old system, pre-lollipop. It's the
standard emulator we ship with app inventor.  (gingerbread, or earlier?)
Let's find a dynamic approach that works, if possible.  And is there a good
reason to have separate properties rather than just fullscreen vs. regular?
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Jan 17, 2015 at 9:54 PM, WeiHua Li<EMAIL_ADDRESS>wrote:

Thanks for the pointer. There is no action bar in App Inventor, only title
bar...
Dynamic control can't be done at the manifest.
I already saw and tried the openstack approach. It doesn't really work in
App Inventor. The code " requestWindowFeature(Window.FEATURE_NO_TITLE);"
must be called before the "setContentView" (within Form.onCreate()). If I
have a property "ShowTitleBar" in the property list in the Designer and has
the corresponding setter method
"requestWindowFeature(Window.FEATURE_NO_TITLE)", this will crash the app.
The method "requestWindowFeature(Window.FEATURE_NO_TITLE);", can't be call
after the setContentView.
The approach I am using now seems have compatible issue with Lollipop,
which I didn't test. Did you try out anything lower than 5.0 to see if that
works? @halatmit https://github.com/halatmit
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-70394449
.

@halatmit I verified this and this approach doesn't work on Lollipop. I will try to finish this pull request next week.

"title bar" is obsolete.  We should be using "action bar" and making sure
this operation is consistent with Android design practices.
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Feb 21, 2015 at 9:48 AM, WeiHua Li<EMAIL_ADDRESS>wrote:

@halatmit https://github.com/halatmit I verified this and this approach
doesn't work on Lollipop. I will try to finish this pull request next week.
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75374666
.

ActionBar unfortunately doesn't work with API3. I believe its API 11
On Sat, Feb 21, 2015 at 9:58 AM, hal<EMAIL_ADDRESS>wrote:

"title bar" is obsolete. We should be using "action bar" and making sure
this operation is consistent with Android design practices.
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Feb 21, 2015 at 9:48 AM, WeiHua Li<EMAIL_ADDRESS>wrote:

@halatmit https://github.com/halatmit I verified this and this
approach
doesn't work on Lollipop. I will try to finish this pull request next
week.
—
Reply to this email directly or view it on GitHub
<
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75374666

.

—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75375143
.

At this point, I don't see a clean solution to set it dynamically. Give the fact that we don't have action bar in App Inventor (still title bar), I decide to move this setting to the manifest. The ShowTitleBar will be an option of the Screen1.property.

We should talk with the Android people about suggestions for handle
this.   Let's not build in things that will be obsolete with Lollipop or
future versions.
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Feb 21, 2015 at 10:50 AM, WeiHua Li<EMAIL_ADDRESS>wrote:

At this point, I don't see a clean solution to set it dynamically. Give
the fact that we don't have action bar in App Inventor (still title bar), I
decide to move this setting to the manifest. The ShowTitleBar will be an
option of the Screen1.property.
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75377417
.

@halatmit what do you mean? Back in Dec, we had emails with Dan from Google. He recommends to do it in the manifest. Title bar is in App Inventor, Action Bar is NOT. Doing in the Manifest (set the right theme) just involves in checking in the android version: theme A for pre material, and theme B for material or later.

I'm getting confused.
Do modern (Lollipop) android apps have title bars?  I thought they did
not.
Also, see here as a possible alternative:
http://www.androidsnippets.com/how-to-make-an-activity-fullscreen
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Feb 21, 2015 at 11:00 AM, WeiHua Li<EMAIL_ADDRESS>wrote:

@halatmit https://github.com/halatmit what do you mean? Back in Dec, we
had emails with Dan from Google. He recommends to do it in the manifest.
Title bar is in App Inventor, Action Bar is NOT. Doing in the Manifest (set
the right theme) just involves in checking in the android version: theme A
for pre material, and theme B for material or later.
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75377828
.

I double checked again. The previous solution works on Android 5.0,
lollipop as well. Can you try the attach apk and provide comment?
Wei
<EMAIL_ADDRESS>On 2/21/15 11:28 AM, hal wrote:

I'm getting confused.
Do modern (Lollipop) android apps have title bars? I thought they did
not.
Also, see here as a possible alternative:
http://www.androidsnippets.com/how-to-make-an-activity-fullscreen
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Sat, Feb 21, 2015 at 11:00 AM, WeiHua Li<EMAIL_ADDRESS>wrote:

@halatmit https://github.com/halatmit what do you mean? Back in
Dec, we
had emails with Dan from Google. He recommends to do it in the manifest.
Title bar is in App Inventor, Action Bar is NOT. Doing in the
Manifest (set
the right theme) just involves in checking in the android version:
theme A
for pre material, and theme B for material or later.
—
Reply to this email directly or view it on GitHub

https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75377828

.

—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/pull/222#issuecomment-75379114.

I double checked again. The previous solution works on Android 5.0,
lollipop as well. Can you try the attach apk and provide comment?
https://www.dropbox.com/s/azks3y9sww1ldnq/AndroidTitleBar.apk?dl=0
Wei
<EMAIL_ADDRESS>

If the API 4 code is merged, we can start using Support Library which give access to ActionBar. That could solve the compatibility issue here.

@halatmit try the new code again (for ShowTitleBar) ?
If you prefer, use the following aia and apk for testing:
https://www.dropbox.com/sh/nuvwd6snrfra1oq/AAAPfN11WCwjt-70Du_2w-_ua?dl=0
Thanks,
Wei

@halatmit @jisqyv This is updated and should be ready to review and merge.

From the last conversation, we agreed to NOT have a special property - ShowTitleBar; instead, we are going to have a check box right next to the Title property, this checkbox will serve as the function for the ShowTitleBar .

The updated aia and apk files are here -
https://www.dropbox.com/sh/nuvwd6snrfra1oq/AAAPfN11WCwjt-70Du_2w-_ua?dl=0

We agreed to have a "checkbox" right next to the Title property in the designer earlier; however, this is not doable because gwt doesn't allow me to overloading the method in the designer - Title(boolean) / Title(string). The error message is the following -

Inconsistent types java.lang.String and boolean for property Title in component Form *

Instead, I renamed the method to TitleVisible, which hope to be consistent with the Title property. In the designer, the property Title should be next to the TitleVisible.

Works fine.
I also tested it on 2.2.2 and it works there.
Do you want to add the translations for TitleVisible ?

@weihuali0509 Also check on whitespace issues in the code

@halatmit @jisqyv added the translations.

@jisqyv can you point out the issues with the while spaces?

@weihuali0509 @halatmit Never mind about the whitespace if you cannot find it. I'll take care of it when I merge the change (likely today or tomorrow assuming my tests work).

@halatmit @jisqyv I believed I have cleaned all the whitespaces.

LGTM

@weihuali0509 @jisqyv
I think this needs some more discussion.  I don't like the idea of not being able to use the action bar as we go forward.   Will this change prevent use of the action bar?

@halatmit @weihuali0509 I have verified that this PR does not conflict with the work on MenuItems. We should merge it.
(I meant to do that a few days ago!)

@halatmit @weihuali0509 MERGED into the ucr branch for the next component release.


Transfers in carto-testnet not logging?

Describe the bug
I'm not seeing transfer counts in here correctly. WIth the Linea campaign we should have a lot more transfers. Shows only 2 transfers in the last 7 days on testnet (all chains)
SELECT count (transfer_id), date(to_timestamp(xcall_timestamp))
from transfers
GROUP BY 2
ORDER BY 2 DESC

Result 2023-05-04 09-23-17.csv

false alarm


Use expectException(Message) instead of annotations

Q
A

Type
improvement

BC Break
no

Summary
The @expectedException and @expectedExceptionMessage annotations are deprecated. They will be removed in PHPUnit 9. This PR replaces @expectedException and @expectedExceptionMessage annotations with expectedException() and expectedExceptionMessage().
My only concern is with OutTest because builder->out() is throwing the exception, not builder->getPipeline().

My only concern is with OutTest because builder->out() is throwing the exception, not builder->getPipeline().

I think that getPipeline() call could be removed in this case

@greg0ire Do you want me to do the squash, or will you squash+merge?

@olvlvl please squash, we're not using squash+merge as it's not doing an actual merge (or it wasn't doing this last time we were checking)

Also, we would rather have you be in charge of the final commit message ;)

Squashed.

Thanks @olvlvl !


Patch function without providing complete source in patch

I have a question regarding a possible extension of el-patch. In most cases, I just need to change a single form in a possibly long definition of a particular function. Such changes frequently break when the other code in the function, which I haven't patched, changes even though I know that such changes should not break my patch.
So I am wondering if it would be possible, in theory, to build a patch given a form. Something like this
(el-patch-patch
   'foo
   (function-call arg1 arg2 (el-patch-add arg3) )
)

and one would search the definition of foo for all forms which match (function-call arg1 arg2) and add an argument arg3. One could also add key forms to make sure that they exist in the function definition if such forms are necessary for the patch. Of course this assumes that one has the definition of foo rather than its compilation.
To be clear, I know that this is not implemented in el-patch, but I am wondering if people see a problem with such a methodology.

I certainly do see the utility, as I have also experienced frequent breakage of patches due to unrelated changes in function internals. The feature would be nontrivial to implement for a few reasons:

You'd need to look up the source code of the function during macroexpansion (yikes!).
When using this feature, compiling your init-file would be basically required, otherwise you'd be doing this incredibly slow operation at every startup.
There are definitely open questions as far as a reasonable and robust user interface to the interface in terms of what the patch would look like.

That said, I think it would be a great idea!

You'd need to look up the source code of the function during macroexpansion (yikes!).

I thought that el-patch does something similar when validating the patch and I was hoping that I can leverage that implementation.

When using this feature, compiling your init-file would be basically required, otherwise you'd be doing this incredibly slow operation at every startup.

Good point. I wonder if it’s better to output a full el-patch into a file and only do the slow operation of actually generating the full el-patch when the patch validation fails.

There are definitely open questions as far as a reasonable and robust user interface to the interface in terms of what the patch would look like.

Indeed, that’s why I wanted to get input before actually working on the pull request.
Do you see a problem with the interface I suggested above? One issue that I thought about is what if the user wants to do multiple patches to the same function in different locations. But that can be done by passing multiple forms to the fictional ‘patch-el-patch’.

I was hoping that I can leverage that implementation

Yes, that is certainly the way to do it. I was just saying that if you did so, then it would Emacs would be opening a file and doing various other sketchy operations during macroexpansion, whereas typically macroexpansion is expected to be stateless and fast. There's no specific problem that will be caused by doing something more complicated; it's just a sign to be careful and consider if there is another approach. Not that I have a better suggestion for how to implement a feature like this.

I wonder if it’s better to output a full el-patch into a file and only do the slow operation of actually generating the full el-patch when the patch validation fails

This gave me an idea for how to avoid the usability problems that I described in my previous comment. If we were to implement what you've suggested, notice that it's kind of the same as what we have currently, except:

the patch is kept in a different file, not directly in your init-file
you have an automated way of regenerating your patch based on a template

So why not solve the underlying problem directly, and provide a command that will take a patch template, and generate a patch for you? That more or less eliminates the busywork of addressing irrelevant changes in the upstream code, without introducing any issues for the user like having to byte-compile their init-file to use this feature with reasonable performance.

open questions as far as a reasonable and robust user interface

OK, I thought about it for a while, and I have a suggestion. I translated all instances of el-patch from my current configuration into my proposed syntax:

(el-patch-template (defvar ruby-electric-mode-map)
  (el-patch-remove
    (dolist ...))
  (el-patch-concat
    "Keymap used in ruby-electric-mode"
    (el-patch-add ".\n\nThe single-character bindings have been removed.")))

(el-patch-template (defun TeX-update-style)
  (el-patch-concat
    "Run style specific hooks"
    (el-patch-add
      ", silently,")
    " for the current document.

Only do this if it has not been done before, or if optional argument
FORCE is not nil.")
  (el-patch-remove
    (message "Applying style hooks..."))
  (el-patch-remove
    (message "Applying style hooks...done")))

(el-patch-template (defun with-editor--setup)
  (el-patch-splice 2
    (when (server-running-p server-name)
      (setq server-name (format "server%s" (emacs-pid)))
      ...)))

(el-patch-template (defun magit-maybe-start-credential-cache-daemon)
  (el-patch-wrap 2
    (with-current-buffer
        (get-buffer-create " *git-credential-cache--daemon*")
      (start-process "git-credential-cache--daemon"
                     (el-patch-swap
                       " *git-credential-cache--daemon*"
                       (current-buffer))
                     magit-git-executable
                     "credential-cache--daemon"
                     magit-credential-cache-daemon-socket)
      (el-patch-add
        (set-process-query-on-exit-flag
         (get-buffer-process (current-buffer)) nil)))))

(el-patch-template (defun (el-patch-swap restart-emacs radian-new-emacs))
  (el-patch-concat
    (el-patch-swap
      "Restart Emacs."
      "Start a new Emacs session without killing the current one.")
    "

When called interactively ARGS is interpreted as follows

- with a single `universal-argument' (`C-u') Emacs is "
    (el-patch-swap "restarted" "started")
    "
  with `--debug-init' flag
- with two `universal-argument' (`C-u') Emacs is "
    (el-patch-swap "restarted" "started")
    " with
  `-Q' flag
- with three `universal-argument' (`C-u') the user prompted for
  the arguments

When called non-interactively ARGS should be a list of arguments
with which Emacs should be "
    (el-patch-swap "restarted" "started")
    ".")
  (el-patch-swap
    (save-buffers-kill-emacs)
    (restart-emacs--launch-other-emacs restart-args)))

Notice:

I figure we can use the symbol ... instead of el-patch-filler
the arguments to el-patch-template (which will generate a full el-patch that can be placed into the user's init-file or perhaps saved to another version-controllable file, details TBD) are: the type of patch, the name of the patch, and then a list of subpatches
each subpatch is resolved in favor of the original version, and then pattern-matched against the actual function source; it must match in exactly one place or the patch generation fails
the actual function source is transformed by substituting each subpatch into it at the appropriate location, and this forms an el-patch form

Thoughts?

Very nice syntax! I think this should cover a majority of cases. Having the requirement of exactly one match is really important to avoid unintended consequences. The implementation of the patch template should not require any change to el-patch (except perhaps for some refactoring).

* I figure we can use the symbol `...` instead of `el-patch-filler`

Yeah, I think combining both ideas (patten matching and a filler form) would provide the most power and the shortest patches.
Are you thinking of ... being only at the end of a form? Or can one have something like this
(let ( ... (var (el-patch-swap 1 2)) ... )

)

I thought about using smaller symbols (although I did not think of the natural ...). However, one must be careful about such symbols being used in the original definition of the function. I don't know how relevant such a potential problem is and if we should provide a way to override the symbol in such cases.
One thing that would need to be ironed out is the behaviour of el-patch-splice and el-patch-wrap when ... is on the left or right and a form is to be trimmed there.

* the arguments to `el-patch-template` (which will generate a full `el-patch` that can be placed into the user's init-file or perhaps saved to another version-controllable file, details TBD) are: the type of patch, the name of the patch, and then a list of subpatches

We can go from the least-automated workflow:

User writes the patch template in some file (that is not called from the init.el).
User runs the template and the el-patch is yanked to the kill ring (much like straight-get-recipe).
User pastes the el-patch in their preferred location.
After sometime, when el-patch validation fails the user has to repeat the above steps manually.

To the most automated workflow:

User defines the template in their init.el
At the end the user calls a function, let call it el-template-finalize, to produce the templates to a file.
el-template-finalize reads the file to check which el-patch'es are defined.
If there are any new templates without a correspondingly defined patch, append the newly defined patch to the file.

 In any case, I think the manual workflow should be the first target.

* each subpatch is resolved in favor of the original version, and then pattern-matched against the actual function source; it must match in exactly one place or the patch generation fails

* the actual function source is transformed by substituting each subpatch into it at the appropriate location, and this forms an `el-patch` form

Sounds good!

Are you thinking of ... being only at the end of a form?

Ideally (i.e. from a user perspective), you'd be able to put it anywhere. I guess it depends how challenging the pattern-matching turns out to be, though, in terms of implementation.

one must be careful about such symbols being used in the original definition of the function. I don't know how relevant such a potential problem is

I've personally never seen ... used as a symbol in any Elisp code I've worked with. We should be pretty safe. Of course, if somebody does dredge up an example in a bug report, we can reconsider :)

One thing that would need to be ironed out is the behaviour of el-patch-splice and el-patch-wrap when ... is on the left or right and a form is to be trimmed there

Good catch, I'm not sure what to do in that case. For el-patch-splice, ostensibly we might be able to get away with supporting it somehow. For el-patch-wrap it'd be impossible because the ... would contain the forms that were supposed to be patched in. I suspect we just want to report an error in that case; the pattern-matching algorithm will be hard enough even if we can always deterministically resolve the subpatch.

 In any case, I think the manual workflow should be the first target

Agreed on all points.

User writes the patch template in some file (that is not called from the init.el)

This seems fine. Here's a wild idea for the future:

invoking el-patch-template has different effects depending on context
if it's evaluated interactively, it copies the patch form to the kill ring
if it's evaluated as part of normal startup or loading, it reports an error
if it's evaluated as part of byte-compilation, it transparently macroexpands to the patch form (or throws an error if validation fails)

That way, we'd support the manual workflow you suggested, but also, people who byte-compile their init-file could just put in the template form and have it magically work without added startup cost (just slower byte-compilation).

Ideally (i.e. from a user perspective), you'd be able to put it anywhere. I guess it depends how challenging the pattern-matching turns out to be, though, in terms of implementation.

Yeah, I also think that would be ideal. But I guess we don't want to check more that the car of the form after ... when expanding .... So that something like
(a ... (b c) ...)
Would not match
(a a (b b) (b c) d)
I don't think we want to reimplement regex-like matching (with backtracking), or do we?
In any case, I think we are converging on a similar viewpoint. I will see how complicated the implementation is and we can discuss further.


[wallet-ext] Rebuild wallet functionality on programmable transactions

Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Current dependencies on/for this PR:

main

PR #8964   👈

This comment was auto-generated by Graphite.


adjust the rendering of card's contents

Here is a pen to see what have been done: http://codepen.io/tnga/full/NNQmGq/

+1, but some of your suggestion aren't compliant with material design's spec : https://www.google.com/design/spec/components/cards.html#cards-behavior

@quentinHydrobat considering i have understood suggestion you are talking about, it isn't support as "feature" but it's just in case to show we can have fluid behavior with a different structure as example. Structure which only depend on  html not on materialize. I took in mind the material design's spec when i added or modified code. Suggestion like how about reverse structure is just a html approach to show flexibility, kept with the fix :wink: .

@Dogfalo , @acburst maybe there was already discussion about it but since it still a problem, due to the number of waiting pull request here and  the fact that they become old with time, without an assigned label or message to tell if they will be merged or not;  i thing this can discourage contributions to this project.
Hope you will find time and solution. That said, can you please check this pull request so that i can have an idea of it status ? (and maybe others interesting fix and adjustments)

@DanielRuf Can you check?

We'd prefer to keep our cards implementation closer to spec

I agree with @Dogfalo, the components should follow the specs.


Custom marker with width, area (start and end date range)?

Is your feature request related to a problem? Please describe.
The CustomMarker allows for adding a line across all rows, such as the a line showing today or now, or any specific point in time. I can't find a way to make this line into a rectangle covering a date range.
Describe the solution you'd like
Just like CustomMarker, but with a start and end date so it creates a rectangle. This could be styled to be almost transparent. For example, a rectangle could denote WWII, and then you'd see which events fall within that date range.
Describe alternatives you've considered
Tried using CustomMarker and styling so it would have width, but I'd like the width to be determined by the date range

+1, this could be a really nice feat, mark it with start and end date

this will be possible with row renderer that is currently WIP.

@Ilaiwi any news on when can we expect #673 to be merged?

@x1mrdonut1x you can use the beta for now. I am not sure when it will hit production

Just checking if there has been any movement on this?
Is this possible with the latest release?


ENH: optimize: migrate to use sparray

This PR migrates the optimize package to use sparse arrays internally. These are only the changes to the code. Another PR will have the changes to the docs (to simplify review). Once these two PRs are merged, scipy will use sparray by default internally in all packages.
In the only public function in optimize that returns sparse, approx_differentiation, we now return sparse array (matrix) when the input sparsity argument is a sparse array (matrix). There are some private functions in the linprog tools that put a sparse array into an _LPProblem but in all cases those seem to be private implementation details. Also, there were no issues with index array dtypes being used by native code.
There are a lot of changes, but each is straight-foward.

How can we address the failures in "Conda & umfpack/scikit-sparse, fast, py3.11/npAny, dev.py (3.11)"?

See #22089   (I just put it up)

Sorry completely missed the ci failure here!


[WFLY-19905] Move module org.apache.xml-resolver from WildFly Core to WildFly

JIRA: https://issues.redhat.com/browse/WFLY-11905

Can one of the admins verify this patch?

This is ok to test.

Dependency Tree Analyzer Output:
Removed Dependencies:

xml-resolver:xml-resolver:jar:1.2:compile

CC @wildfly/prod

Core - Full Integration Build 9470 outcome was FAILURE using a merge of 9dd31af9b9ca9a9c84ff4a2258a3952bb2f4048e
Summary: Exit code 1 (Step: Build & test full (Maven)) (new) Build time: 00:07:23

Core - Full Integration Build 9476 outcome was FAILURE using a merge of 9dd31af9b9ca9a9c84ff4a2258a3952bb2f4048e
Summary: Tests passed: 1173, ignored: 33; exit code 1 (Step: Build & test full (Maven)) (new) Build time: 00:18:39

@parsharma Kabir went ahead and filed https://issues.redhat.com/browse/WFCORE-4911 so can you add WFCORE-4911 to the commit message and change the PR title to use that? Thanks.

Core - Full Integration Build 9490 outcome was FAILURE using a merge of a6548fe0fc274b2090b1a26babfdfad1950d5da2
Summary: Exit code 1 (Step: Build & test full (Maven)) Build time: 00:02:12

Core - Full Integration Build 9484 outcome was FAILURE using a merge of a6548fe0fc274b2090b1a26babfdfad1950d5da2
Summary: Exit code 1 (Step: Build & test full (Maven)) Build time: 00:02:29

Core - Full Integration Build 9529 outcome was FAILURE using a merge of a6548fe0fc274b2090b1a26babfdfad1950d5da2
Summary: Exit code 1 (Step: Build & test full (Maven)) Build time: 00:04:48

Do we have a link to the corresponding WildFly PR to cross reference?

@darranl  I added https://github.com/wildfly/wildfly/pull/13173 in the description
@parsharma it helps reviewers to include as much information (as links) as possible :)
@jmesnil Since the upstream one has been merged, I think we can remove the hold label?

If that is the case I will include this in the batch I am preparing ready for one more round of CI


Use interactive mode as default mode?

Topic for discussion:
How about changing default mode to interactive?
I think that this might be nicer user experience.
There would be flag to turn it off.

@kadel this can be easily done once we have something around #31


Refactor fixures as entities rather than sql

app:setup doesn't work with sqlite, I think because you can't send multiple sql statements to ->execute().
My quick hack fix:
            if (true === isset($count['total']) && 0 == $count['total']) {
                foreach (explode("\n", $file) as $sqlStatement) {
                    if ($sqlStatement) {
                        $connection->executeQuery($sqlStatement);
                    }
                }
                $output->writeln('<info>Feed data inserted</info>');

but really these shouldn't be sql statements, they should be simple Action entities.
$action = (new Action())->setTitle(..)->setReverse();

Even better would be to check if the action existed first, so that new actions could be added without resetting the database.

I made a quick test, even the command bin/console doctrine:schema:create doesn't work with SQLite

For sqlite, use
bin/console doctrine:database:create
bin/console d:sch:update --force
On Sat, Mar 16, 2024 at 12:48 PM Stéphane Dion @.***>
wrote:

I made a quick test, even the command bin/console doctrine:schema:create
doesn't work with SQLite
—
Reply to this email directly, view it on GitHub
https://github.com/stephanediondev/feed/issues/31#issuecomment-2002085452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEXIQLLYG4T6U52JY64OEDYYSHV7AVCNFSM6AAAAABEYDJFQ2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMBSGA4DKNBVGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Prepare to use Sundials artifacts

This sets the min Julia version to 1.3. It pulls in Sundials_jll which depends on SuiteSparse_jll and OpenBLAS_jll.

@giordano You had said something about how this could be done in a way that is compatible with BinaryProvider. Do you have a pointer to a project that does it?

Failure seems to be in #224.

I don't know if anyone implemented it yet, but the idea is to depend on both Sundials_jll.jl and BinaryProvider.jl, and add a deps/build.jl which if VERSION >= v"1.3" would load the jll package, otherwise would load the script to install the tarballs using BinaryProvider. This is akin to what FFTW.jl does to provide either FFTW or MKL

Ok, makes sense. I'll try it out.

@ChrisRackauckas I think we can perhaps stay with 1.3 and above, given that multi-threading is getting widely used in the DiffEq ecosystem. Is it ok to merge this, or are there other considerations?

Interesting that this works, since it fails for me locally:
Mass Matrix: Error During Test at C:\Users\accou\.julia\dev\Sundials\test\runtests.jl:33
  Got exception outside of a @test
  LoadError: ArgumentError: unsafe_wrap: pointer 3531646E00000009 is not properly aligned to 8 bytes
  Stacktrace:
   [1] convert at .\pointer.jl:84 [inlined]
   [2] massmat(::Float64, ::Ptr{Sundials._generic_SUNMatrix}, ::Sundials.FunJac{ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Nothing,Nothing,DiffEqBase.NullParameters,Array{Float64,2},Nothing,Array{Float64,1},Nothing,Nothing,Nothing}, ::Ptr{Sundials._generic_N_Vector}, ::Ptr{Sundials._generic_N_Vector}, ::Ptr{Sundials._generic_N_Vector}) at C:\Users\accou\.julia\dev\Sundials\src\common_interface\function_types.jl:147
   [3] __ARKode at C:\Users\accou\.julia\dev\Sundials\src\wrapped_api\arkode.jl:363 [inlined]
   [4] ARKode(::Sundials.Handle{Sundials.ARKODEMem}, ::Float64, ::Sundials.NVector, ::Array{Float64,1}, ::Int32) at C:\Users\accou\.julia\dev\Sundials\src\wrapped_api\arkode.jl:368
   [5] solver_step(::Sundials.ARKODEIntegrator{Sundials.NVector,DiffEqBase.NullParameters,Sundials.Handle{Sundials.ARKODEMem},ODESolution{Float64,2,Array{Array{Float64,1},1},Array{Array{Float64,1},1},Dict{Symbol,Float64},Array{Float64,1},Nothing,ODEProblem{Array{Float64,1},Tuple{Float64,Float64},true,DiffEqBase.NullParameters,ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}},DiffEqBase.StandardODEProblem},ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing},DiffEqBase.HermiteInterpolation{Array{Float64,1},Array{Array{Float64,1},1},Array{Array{Float64,1},1}},DiffEqBase.DEStats},ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing},ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Sundials.FunJac{ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Nothing,Nothing,DiffEqBase.NullParameters,Array{Float64,2},Nothing,Array{Float64,1},Nothing,Nothing,Nothing},Nothing,Sundials.DEOptions{DataStructures.BinaryHeap{Float64,DataStructures.LessThan},DataStructures.BinaryHeap{Float64,DataStructures.LessThan},CallbackSet{Tuple{},Tuple{}},Float64,Float64,typeof(DiffEqBase.ODE_DEFAULT_PROG_MESSAGE)},Array{Float64,1},Tuple{Int64},Array{Float64,1},Sundials.LinSolHandle{Sundials.Dense},Sundials.MatrixHandle{Sundials.DenseMatrix},Sundials.LinSolHandle{Sundials.Dense},Sundials.MatrixHandle{Sundials.DenseMatrix},Nothing}, ::Float64) at C:\Users\accou\.julia\dev\Sundials\src\common_interface\solve.jl:1101
   [6] solve!(::Sundials.ARKODEIntegrator{Sundials.NVector,DiffEqBase.NullParameters,Sundials.Handle{Sundials.ARKODEMem},ODESolution{Float64,2,Array{Array{Float64,1},1},Array{Array{Float64,1},1},Dict{Symbol,Float64},Array{Float64,1},Nothing,ODEProblem{Array{Float64,1},Tuple{Float64,Float64},true,DiffEqBase.NullParameters,ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}},DiffEqBase.StandardODEProblem},ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing},DiffEqBase.HermiteInterpolation{Array{Float64,1},Array{Array{Float64,1},1},Array{Array{Float64,1},1}},DiffEqBase.DEStats},ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing},ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Sundials.FunJac{ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Nothing,Nothing,DiffEqBase.NullParameters,Array{Float64,2},Nothing,Array{Float64,1},Nothing,Nothing,Nothing},Nothing,Sundials.DEOptions{DataStructures.BinaryHeap{Float64,DataStructures.LessThan},DataStructures.BinaryHeap{Float64,DataStructures.LessThan},CallbackSet{Tuple{},Tuple{}},Float64,Float64,typeof(DiffEqBase.ODE_DEFAULT_PROG_MESSAGE)},Array{Float64,1},Tuple{Int64},Array{Float64,1},Sundials.LinSolHandle{Sundials.Dense},Sundials.MatrixHandle{Sundials.DenseMatrix},Sundials.LinSolHandle{Sundials.Dense},Sundials.MatrixHandle{Sundials.DenseMatrix},Nothing}) at C:\Users\accou\.julia\dev\Sundials\src\common_interface\solve.jl:1145
   [7] #__solve#22(::Base.Iterators.Pairs{Symbol,Float64,Tuple{Symbol,Symbol},NamedTuple{(:abstol, :reltol),Tuple{Float64,Float64}}}, ::typeof(DiffEqBase.__solve), ::ODEProblem{Array{Float64,1},Tuple{Float64,Float64},true,DiffEqBase.NullParameters,ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}},DiffEqBase.StandardODEProblem}, ::ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing}, ::Array{Any,1}, ::Array{Any,1}, ::Array{Any,1}, ::Type{Val{true}}) at C:\Users\accou\.julia\dev\Sundials\src\common_interface\solve.jl:12
   [8] #__solve at .\none:0 [inlined] (repeats 5 times)
   [9] #solve_call#442(::Base.Iterators.Pairs{Symbol,Float64,Tuple{Symbol,Symbol},NamedTuple{(:abstol, :reltol),Tuple{Float64,Float64}}}, ::typeof(DiffEqBase.solve_call), ::ODEProblem{Array{Float64,1},Tuple{Float64,Float64},true,DiffEqBase.NullParameters,ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}},DiffEqBase.StandardODEProblem}, ::ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing}) at C:\Users\accou\.julia\packages\DiffEqBase\E16PL\src\solve.jl:38
   [10] #solve_call at .\none:0 [inlined]
   [11] #solve#443 at C:\Users\accou\.julia\packages\DiffEqBase\E16PL\src\solve.jl:57 [inlined]
   [12] (::DiffEqBase.var"#kw##solve")(::NamedTuple{(:abstol, :reltol),Tuple{Float64,Float64}}, ::typeof(solve), ::ODEProblem{Array{Float64,1},Tuple{Float64,Float64},true,DiffEqBase.NullParameters,ODEFunction{true,var"#mm_f#32"{Array{Float64,1},Array{Float64,2}},Array{Float64,2},var"#mm_analytic#34",Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing,Nothing},Base.Iterators.Pairs{Union{},Union{},Tuple{},NamedTuple{(),Tuple{}}},DiffEqBase.StandardODEProblem}, ::ARKODE{:Newton,:Dense,:Dense,Sundials.Implicit,Nothing,Nothing,Nothing,Nothing}) at .\none:0
   [13] top-level scope at C:\Users\accou\.julia\dev\Sundials\test\common_interface\mass_matrix.jl:32
   [14] include at .\boot.jl:328 [inlined]
   [15] include_relative(::Module, ::String) at .\loading.jl:1105
   [16] include(::Module, ::String) at .\Base.jl:31
   [17] include(::String) at .\client.jl:424
   [18] top-level scope at C:\Users\accou\.julia\dev\Sundials\test\runtests.jl:33
   [19] top-level scope at D:\buildbot\worker\package_win64\build\usr\share\julia\stdlib\v1.3\Test\src\Test.jl:1107
   [20] top-level scope at C:\Users\accou\.julia\dev\Sundials\test\runtests.jl:33
   [21] top-level scope at D:\buildbot\worker\package_win64\build\usr\share\julia\stdlib\v1.3\Test\src\Test.jl:1107
   [22] top-level scope at C:\Users\accou\.julia\dev\Sundials\test\runtests.jl:26
   [23] include at .\boot.jl:328 [inlined]
   [24] include_relative(::Module, ::String) at .\loading.jl:1105
   [25] include(::Module, ::String) at .\Base.jl:31
   [26] include(::String) at .\client.jl:424
   [27] top-level scope at none:6
   [28] eval(::Module, ::Any) at .\boot.jl:330
   [29] exec_options(::Base.JLOptions) at .\client.jl:263
   [30] _start() at .\client.jl:460
  in expression starting at C:\Users\accou\.julia\dev\Sundials\test\common_interface\mass_matrix.jl:32

Test Summary:    | Pass  Error  Total
Common Interface |   38      2     40
  CVODE          |   23            23
  ARKODE         |    3             3
  IDA            |    7             7
  Jacobians      |           1      1
  Callbacks      |    4             4
  Iterator       |              No tests
  Errors         |    1             1
  Mass Matrix    |           1      1
ERROR: LoadError: Some tests did not pass: 38 passed, 0 failed, 2 errored, 0 broken.
in expression starting at C:\Users\accou\.julia\dev\Sundials\test\runtests.jl:25
ERROR: Package Sundials errored during testing

Are you on 64-bit windows? Does the failing test call blas/lapack by any chance?

64-bit Windows. I am not sure if Sundials has a size check for whether to BLAS or not, but the tests are all small ODEs. If it doesn't check size, then it would BLAS in the factorization.