Unselecting group option doesn't default to none

Description
This is a part of a second bug, I couldnt see all the files in my downloads folder and I was told to unselect the group by date modified option. I clicked on it and it removed the checkmark but it didnt select none and didnt nothing. I then selected group by none and then download files showed.
Steps To Reproduce

select group by date

select group by date to unselect but it wont default to none.

click downloads and files wont be there.

Expected behavior

To default to none when unselecting a option, in this case group by date.
To show files in download.

Files Version
<IP_ADDRESS>
Windows Version
Windows 11 21H1 22000.376
Relevant Assets/Logs
debug.log
debug_fulltrust.log

This issue should be resolved with #7423


Audit Log activation on downstream cluster doesn't work

What kind of request is this (question/bug/enhancement/feature request):
Possibly a bug in the RKE version shipped with Rancher 2.3.2
Steps to reproduce (least amount of steps as possible):

Bootstrap Rancher 2.3.2 fresh cluster.
Create a downstream cluster
Edit the cluster.yaml from the downstream cluster to add the Audit Log like explained in the doc below:
https://rancher.com/docs/rke/latest/en/config-options/audit-log/

Just using the following from the doc in the cluster.yaml file:

services:
  kube-api:
    audit_log:
      enabled: true

Result:
The cluster doesn't get updated.
Other details that may be helpful:
The issue get raised because the terraform provider Rancher2 reports the config as "modified" but nothing changes on the downstream cluster.
Workaround
Using the following config in downstream cluster.yaml is working as a workaround:
kube-api:
      extra_args:
        audit-log-format: json
        audit-log-maxage: '5'
        audit-log-maxbackup: '5'
        audit-log-maxsize: '100'
        audit-log-path: /var/log/kube-audit/audit-log.json
        audit-policy-file: /etc/kubernetes/audit.yaml
      extra_binds:
        - '/var/log/kube-audit:/var/log/kube-audit'

Uploading first the  /etc/kubernetes/audit.yaml file first to the manager nodes to make it work.
I used the basic file from the k8s website for testing purpose:
# Log all requests at the Metadata level.
apiVersion: audit.k8s.io/v1
kind: Policy
rules:
- level: Metadata

Environment information

Tested on Rancher HA and Rancher Docker: version 2.3.2
Kubernetes 1.16
Docker 18.9.9
Downstream cluster is a 3 manager nodes cluster on DigitalOcean.

Hey @zaggash , as explained on RKE docs, audit_log feature was added on RKE v1.1.0
Rancher v2.3.2 is using RKE v0.3.2. RKE v1.1.0 was introduced on Rancher v2.4.0
To use audit_log argument, you need to use Rancher up to v2.4.0

I'll close this one, the answer on the linked issue on the terraform github is pretty clear.


hongbo/tweak tojson/fromjson and add derive fromjson test

tweak tojson/fromjson
more tests
fmt

Pull Request Test Coverage Report for Build 3109
Details

2 of 9 (22.22%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.03%) to 83.133%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

json/from_json.mbt
2
5
40.0%

builtin/json.mbt
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

json/from_json.mbt
1
49.3%

Totals

Change from base Build 3107:
0.03%

Covered Lines:
3933

Relevant Lines:
4731

💛  - Coveralls


Fix master tests

I was going to propose a small PR for rails 5 compatibility and saw that master was failing. Here is a quick fix
I'm not entirely sure if I'm treating the symptom or the problem here.
For the failing assertions on test/callbacks_spec.rb:135 and test/callbacks_spec.rb:153, the keys were strings when evt.data.previous_attributes.keys were symbols.
For the failing assertion on test/callbacks_spec.rb:172, within the proc in the test, evt.data.previous_attributes was straight up JSON,
Again, the tests are passing, but I'm not sure if this is fixed. It's not clear to me why it would of worked in the past as strings and now be symbols.

Ohai Matt! Thanks for the PR. It looks like all the tests are passing but one. Are they working on your local setup? If so, which versions of Ruby are you using?

Ohai Charles! They are all passing locally...I'm running ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-darwin15] which looks like a bit off from what Travis is running.
I haven't quite wrapped my head around all changes that have happened between versions and how it's decided to handle symbols vs strings. So maybe I'm relying on something new

Thanks Matt. I'm totally fine dropping support for Ruby 1.9.3, but we should probably get it working on at least ruby 2.1.x Any chance you can try against those?

thanks for the Pull Request! Master seems to be fine right now so I'm going close this.


[Improvement][common] HttpClient singleton mode #3332

Tips

Thanks very much for contributing to Apache DolphinScheduler.
Please review https://dolphinscheduler.apache.org/en-us/community/index.html before opening a pull request.

Implement the httpClient singleton model，and Support for insecure HTTPS
Improvement #3332

SonarCloud Quality Gate failed.

SonarCloud Quality Gate failed.
  0 Bugs
  2 Vulnerabilities (and  0 Security Hotspots to review)
  5 Code Smells
 84.4% Coverage
 0.0% Duplication


Create lesson

Able to create lesson from profile, link to classroom
Create lesson in classroom
Includes:

Date
Title
Assignment
Links to (optional) files
Optional picture
Linked:

User who posted (teacher)

Teacher picture
Teacher name
Link to teacher profile

Classroom

Branch issue-6-create-lesson created for issue #6.


Tests!

 Issue by Katharine
Thursday Feb 13, 2014 at 03:39 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT
Originally opened as https://github.com/pebble/cloudpebble/issues/19

We need some.

 Comment by ron064
Monday Mar 03, 2014 at 14:36 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

what kind of tests?

 Comment by snatr
Wednesday May 07, 2014 at 07:20 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

I don't know if this is the same Katherine that developed Cloud Pebble or not. And I don't know exactly where I'm supposed to submit bug reports over here, but I decided to post this here rather than make a new post.
The new Cloud Pebble has a problem when working with the templates that can be chosen from. It appears that they are still all version 2.0 and hanging the system up somehow. The one I was testing with is still in a pending status even after I return to the UI.

 Comment by Katharine
Wednesday May 07, 2014 at 07:21 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

The hanging is unrelated and may now be resolved (or maybe not…).
You should create new issues for new issues, or just email<EMAIL_ADDRESS>and we'll deal with it.

 Comment by snatr
Wednesday May 07, 2014 at 07:26 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

Okay, thanks. But just to be clear because I'm new to this- Cloud Pebble can compile earlier version code?

 Comment by Katharine
Wednesday May 07, 2014 at 07:27 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

Anything that builds against SDK 2.0 should also build against SDK 2.1. The only change is that certain types of buggy code will crash instead of silently acting strange.

 Comment by snatr
Wednesday May 07, 2014 at 07:35 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

Thanks again. I don't know what's going on though. I simply added the Button Click template and the hit the all-in-one link for compile/run/install and it hung. Now it says it failed, and the log window says: Something went wrong. 'None_Type' object has no attribute 'get_contents_as_string'. Seems like a page error more than a code error. Especially when I never altered the template I used before testing the run.

 Comment by snatr
Wednesday May 07, 2014 at 07:43 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

I just compiled directly, and it succeeded. Then I went to the code and hit the run button. It succeeded again, but failed to install on the watch. Then I tried installing from the compile screen and it hung during the install. Might have something to do with Chrome on my end though, I don't know. Guess I'll have to be patient with the whole process here considering the new version just being deployed.

 Comment by Katharine
Wednesday May 07, 2014 at 07:37 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

The hung build is invalid. Try running it again.

 Comment by Katharine
Wednesday May 07, 2014 at 07:44 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

Please email<EMAIL_ADDRESS>for support questions.

 Comment by snatr
Wednesday May 07, 2014 at 07:47 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

No problem. Thanks for the quick response!

 Comment by HParker
Wednesday Sep 09, 2015 at 21:57 GMT # Sample: Friday Sep 13, 2013 at 22:58 GMT

I would be happy to help with tests if this is still something clouldpebble is hoping to add.


Cleanup AWS GPU docs & add T4s for uwhackweeks

The bug we reported upstream to eksctl has been fixed! So eksctl is now responsible for setting up the GPU driver, not us! https://github.com/weaveworks/eksctl/issues/5277. Yay for fixing things upstream! This woudl also mean that eksctl is responsible for keeping these versions up to date, and not us.
Based on https://github.com/pangeo-data/pangeo-docker-images/issues/390 and many other discussions (linked to from there), NVidia T4s are now preferred over older K80s. We update the AWS GPU docs to recognize this.
Add PyTorch & Tensorflow images as options to the GPU profile here, so end users can choose!

Fixes https://github.com/2i2c-org/infrastructure/issues/1784

I also opened https://github.com/weaveworks/eksctl/pull/5797 to upgrade the version of the nvidia device plugin on eksctl lol

I've deployed and tested this, works ok!

Thanks @sgibson91 and @GeorgianaElena


Incorrect behavior in case of HRESP=1

Hello
Regarding the testcase:
DUT=ahb_template SIM=icarus TIMEPREC=1ps TIMEUNIT=1ns WAVES=1 RANDOM_SEED=1 pytest tests/test_ahb_lite_sram.py::test_ahb_lite_sram[data_width0]

On the waveform we can observe that slave_hresp is high for a single cycle. This behavior does not match with spec.
AHB spec (ARM IHI 0033B.b) chapter 5.1. In case of error mention that HRESP is driven high for 2 cycles, HREADYOUT is low on first cycle and high on second.
On last cycle of HRESP high, HTRAN must be drive to IDLE.
The waveform provided in the spec depict a slave wait state before returning the error.

To be note that in case of support of exclusive access (HEXCL / ahb5) , the HEXOKAY signal should also be handle.

I am sorry, the fix is incorrect.

HTRAN should be IDLE when slave_hready = 1 and slave_hresp=1. HTRANS put idle when slave_hready = 0 and slave_hresp=1.
Also 1 slave wait state has been added, is that expected ?

Hey @Thomasb81,
can you try to get the latest version (v0.1.9)? on my side I don't see this issue both verilator / icarus, as you can see below:

If you are running icarus, try to add something like below at the monitor fork.
if self.bus.htrans.value.is_resolvable and self.bus.hready.value.is_resolvable:
                if self.bus.htrans.value == 0 and self.bus.hready == 0:
                    raise AssertionError("AHB PROTOCOL VIOLATION - TEST")

I should have mix something...
The waveform show the second transaction that return an error.
icarus:

With icarus HTRANS never go to IDLE, it take 2'bzz
verilator:

technically you should see htrans going from htrans = NSEQ to htrans = IDLE only if there is a pipeline transaction ongoing, on pip_mode = False, there's no overlap.

The point with an rtl dut,

My first write access  at address 0x4 of the value 0x1 does not work ... I have to dis-align clock and bus data...

Sorry, didn't understand your point, are you saying there's an issue with your design? maybe you can share the RTL.

Yes I design a register decoder. Actually I can't share the code.

Here is a part of rtl code.
wire trans_req = hready & hsel & htrans[1];
wire ahb_read_req = trans_req & ~hwrite;
wire ahb_write_req = trans_req & hwrite;

always @(posedge hclk or negedge hresetn) begin : delay_haddr
    if (~hresetn) begin
        addr_reg <= {(ADDR_WIDTH){1'b0}};
    end
    else if (ahb_write_req|ahb_read_req) begin
        addr_reg <= haddr[ADDR_WIDTH-1:0];
    end
    else begin
        addr_reg <= {(ADDR_WIDTH){1'b0}};
    end
end

    always @(posedge hclk or negedge hresetn) begin : delay_write
        if (~hresetn) begin
            write_en_reg <= 1'b0;
        end
        else if (ahb_write_req) begin
            write_en_reg  <= ahb_write_req;
	end
	else begin
	    write_en_reg <= 1'b0;
	end
    end

assign addr = addr_reg[ADDR_WIDTH-1:0];

delay_write work but  delay_haddr  don't... Because there is nothing at offset 0x0, the error at 1 signal manage hreadyout and resp to toggle for the error response.
Snippet of my test:
    ahb_lite_master = AHBLiteMaster(
        AHBBus.from_prefix(dut,
                           prefix=None,

                          ),
        dut.hclk,
        dut.hresetn,
        def_val="0",
    )
    resp = await ahb_lite_master.write(0x4, 0x1)
    assert resp[0]['resp'] == AHBResp.OKAY

technically you should see htrans going from htrans = NSEQ to htrans = IDLE only if there is a pipeline transaction ongoing, on pip_mode = False, there's no overlap.

I disagree with that.

it is mandatory to complete the error transfert end with an IDLE to "flush" the pipeline ...

Hey @Thomasb81,
let me clarify my point here, as you highlighted there, it's written the following:
"...two-cycle response provides sufficient time for the master to cancel this next access and drive HTRANS[1:0] to IDLE before the start of the next transfer"
Thus, moving HTRANS from NSEQR to IDLE is just to indicate the operation B in the image below is cancelled, as also mentioned by the specification, this is due to the pipeline nature of the AHB protocol once by the time the slave set HREADY == HIGH it is already accepting the address phase B.
As I said, for 2x or more transactions with pipeline mode set (pip=True), this is required to cancel the operation B (which will retry again after), however, for non-pipeline transactions there is no point on changing HTRANS once address phase and data phase are not overlapping. Another way of thinking this is the following, if I have a single AHB transaction where the slave returns error, the data phase could have all floating the MOSI (master output slave input) signals but HWDATA, as the only one that matters is the HWDATA (in case of writes). Hope this helps to clarify.

I was reviewing again the ARM's forum and actually the first wait state (T1-T2) is not mandatory. Currently the AHB slave is behaving like the ARM's waveforms but in the future I can revert back to remove this wait state as the only obligation from a slave PoV is the 2-cycle error response.

I was reviewing again the ARM's forum and actually the first wait state (T1-T2) is not mandatory. Currently the AHB slave is behaving like the ARM's waveforms but in the future I can revert back to remove this wait state as the only obligation from a slave PoV is the 2-cycle error response (1st HREADY==LOW && HRESP == ERROR 2nd HREADY==HIGH && HRESP == ERROR).

That's what the comment below the diagram say:

See chapter 3.6 dedicated to wait state.

I was reviewing again the ARM's forum and actually the first wait state (T1-T2) is not mandatory. Currently the AHB slave is behaving like the ARM's waveforms but in the future I can revert back to remove this wait state as the only obligation from a slave PoV is the 2-cycle error response (1st HREADY==LOW && HRESP == ERROR 2nd HREADY==HIGH && HRESP == ERROR).

That's what the comment below the diagram say:
See chapter 3.6 dedicated to wait state.

Exactly, it not mandatory but it's implementation specific. Also, I reviewed some ARM's code/docs and seems to be aligned.

Hello
Regarding the testcase:
DUT=ahb_template SIM=icarus TIMEPREC=1ps TIMEUNIT=1ns WAVES=1 RANDOM_SEED=1 pytest tests/test_ahb_lite_sram.py::test_ahb_lite_sram[data_width0]

On the waveform we can observe that slave_hresp is high for a single cycle. This behavior does not match with spec.
AHB spec (ARM IHI 0033B.b) chapter 5.1. In case of error mention that HRESP is driven high for 2 cycles, HREADYOUT is low on first cycle and high on second. On last cycle of HRESP high, HTRAN must be drive to IDLE.
The waveform provided in the spec depict a slave wait state before returning the error.
To be note that in case of support of exclusive access (HEXCL / ahb5) , the HEXOKAY signal should also be handle.

Coming back to the first message, the only thing wrong I denote is the HRESP == OKAY during the first cycle of the two cycle response. Anyway, I already change the VIP response and current solution is spec aligned.

By the way https://github.com/aignacio/cocotbext-ahb/blob/ef7bb85ddf6dc5be87fee67e05dbcb9b3801a480/cocotbext/ahb/ahb_master.py#L30
This does not make sense. VIP should alway drive a valid value. When highz is involve in logic with 4 states value (0,1,z,x) it leads to x that propagate.
On AHB it makes non-sence to drive signal at highz.

Sorry @Thomasb81 but I disagree with your comment based on my personal experience, commercial VIPs from different vendors such as Cadence, Synospsys... have all default values to either Z / X intentionally to help to diagnose designs with bugs when X / Z is propagated. It only drive values when it is mandatory/required to, and a proper design should not sample input values when it is not suppose to.

Coming back to the first message, the only thing wrong I denote is the HRESP == OKAY during the first cycle of the two cycle response. Anyway, I already change the VIP response and current solution is spec aligned.

Yes, but has the AHBLiteSlave has a bac-pressure mechanism that control hreadyout output (hreadyout on Slave of figure 4-2). The AHBLiteMaster should have a similar mechanism to control hready output (hready on Master and Slave of figure 4-2) This mechanism is currently missing, unless we should use the custom method for this purpose ?

Sorry @Thomasb81 but I disagree with your comment based on my personal experience, commercial VIPs from different vendors such as Cadence, Synospsys... have all default values to either Z / X intentionally to help to diagnose designs with bugs when X / Z is propagated. It only drive values when it is mandatory/required to, and a proper design should not sample input values when it is not suppose to.

I can agree with that. But due to pipe line nature of this protocol, some key control signal can't take 'z'. like hsel and htrans, that are use to decode a starting address-phase. Currently when we simulate with icarus with AHBLiteMaster and def_val="Z", X are propagde during the dataphase, if a single transfert is request.

For the first point I agree there should be a way to control hready_in from the master side. As this is not a mandatory signal for master and it is associated with the bus matrix interconnect to the slave, I decided to drive it high by default, typically the interconnect will mirror the slave hreadyout but I did not want to model this as it makes more sense to create a different class (maybe AHBLiteMasterInterconnect). For now what I would suggest is to implement outside the VIP (throughout your testbench, using force -> dut.hready_in ) this behavior as you intend once it would take me some work to have it fully working, i.e model interconnect broadasting slave back-pressure.
And on the second point, it is hard to comment without checking waveforms or the entire design, the VIP is suppose to only drive HWDATA with something during data phase, otherwise something is  not correct. I understand you cannot share your code but feel free to reach me out by e-mail if I can help you somehow (anderson@aignacio.com).  :thumbsup:


Black list collectors in handlers

With this change we will be able to black list collectors at the handler level. We would be able to selectively emit from each handler.

It is probably worth updating fullerite.conf.example with values that show - how this will look like in real configuration.

This looks okay, other than minor issues above. fix & ship.

fix and ship

@tsheasha @gnufied Have addressed all the review changes.

:shipit:

+1


need to specify vmname

Hi, Is there a way to pass a vm name to the clone process when using the test kitchen driver ?
thanks !
rob

yes. Its :vsphere_name
Example:
driver_config:
  vsphere_name: 'my_server'

@mwrock the gem that's available from rubygems is still using an arbitrary kitchen-[hexrandomvalue] name in the vm config. We need the version that's merged to master released as a gem. I verified this in my own testing to be the case

works w00t! thanks !


Sidebar list items too dark for good readability

After booting up vscode today I noticed that the overal text color changed from white to a light purple. This changes affects people like me who have disabilities with reading as the text is quite unclear to me now :(
BTW: This is the best theme ever created by mankind 👍

Hi @nealoke ,
Thanks a lot for raising this.
Could you please be more specific about what text you're talking about? Also, it'd be great if you ccan provide a screenshot of the editor as there's no issue here on Windows and Ubuntu.
Personally, I've noticed that the new @code has some issues with language tokenization. The element selectors like h1 and input as shown in the screenshot aren't supposed to be highlighted that way.

VS Code doesn't seem to recognize them as element selectors anymore, they should have a red-ish color wit this theme activated. This seems to be a bug and I've also tried it with other themes -- it's the same result, btw it's a .scss file.
I presume this has already been reported to the vscode team and we should get a patched version soon. If this isn't the case, I'll look into raising the issue with them. Also, note that there's nothing I can do currently about that. Even the Default Dark Theme has the same issue.
Happy coding!

@whizkydee sorry for not adding a picture :). It is not that the text is unreadable but the contrast between the background and the font color is just to low IMO. Before the update it was I think white text color which made it very clear.

Thanks @nealoke,
'Will look into that soon. Sorry for the inconvenience .

Hi @nealoke,
Sorry, I forgot to state that you could do a workaround via your settings.json file with;
{
  "workbench.colorCustomizations": {
    "sideBar.foreground": "#ffffffcc"
  }
}

sideBar.foreground modifies the color of the sidebar items and the items in the command palette
#ffffffcc is the color that was used initially before the v1.6.0 release

I hope you find it helpful.

@whizkydee so it will not be changed by default? 😢 Thanks for the workaround!

@nealoke It will, but It'll probably still look similar to its current appearance (or not).

@whizkydee I've used the workaround you gave me but the following still remains. Could it be because it is ignored in my .gitignore file?

Yeah @nealoke It's because it's ignored in your .gitignore file. I guess that's great, or what do you think?

@nealoke try the following... Should make ignored resources and multiple selections a bit easier to see. Let me know if you're having trouble with anything else or would like for me to make some adjustments.
  "workbench.colorCustomizations": {
    "sideBar.foreground": "#ffffffcc",
    "editor.selectionBackground": "#5C6282ff",
    "editor.selectionHighlightBackground": "#5C6282bb",
    "editor.inactiveSelectionBackground": "#5C6282cc",
    "editor.findMatchBackground": "#5C6282dd",
    "gitDecoration.ignoredResourceForeground": "#ccccccaa"
  }

Thanks @aewing,
I totally forgot that I hadn't responded to this.
@nealoke really sorry about that. Let me know if you find the current solution helpful.
Good luck.


Adding a missing quote

Just a minor annoyance :smile:

Hi @sitereactor, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;


[BUG]

Describe the bug
error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:1:1
  |
1 | #![feature(trace_macros)]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:2:1
  |
2 | #![feature(slice_concat_trait)]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:3:1
  |
3 | #![feature(result_cloned)]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:4:1
  |
4 | #![feature(iterator_fold_self)]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:5:1
  |
5 | #![feature(try_blocks)]
  | ^^^^^^^^^^^^^^^^^^^^^^^

error[E0554]: `#![feature]` may not be used on the stable release channel
 --> src/main.rs:6:1
  |
6 | #![feature(str_split_once)]
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^

error: aborting due to 6 previous errors

For more information about this error, try `rustc --explain E0554`.
error: could not compile `eww`

To learn more, run the command again with `--verbose.`

Reproducing the issue
Follow the installation on the docs
Expected behaviour
no errors
Additional context
If applicable, provide additional context or screenshots here

If you actually followed the Docs properly, or alternatively read the error message, you'd not have that issue:


Top N queries by Latency - aggregator implementation

Is your feature request related to a problem? Please describe.
Currently, OpenSearch lacks a direct means of providing insights into “top queries” that have a significant impact on latency, we want to build the in-memory aggregator to capture those queries on the coordinator node level.
(Parent rfc: #11186)
Describe the solution you'd like
Implement a priority queue-based in-memory data store, with configurable window size, on the coordinator node, designed to efficiently store the top N queries. The data model of the stored query attributes should be highly extensible for different types of resources and metrics.
Describe alternatives you've considered
We can instead export the data and offload the query instrumentation logic to external sinks. But it would require us to retain all necessary data for every request. Additionally, as of now we lack a state-of-the-art on-node store capable of holding all request information for aggregation and calculating the top N when user requested. Moreover, we refrain from introducing dependencies on a specific sink in this feature.
Additional context
Please see parent rfc: #11186

This is done by https://github.com/opensearch-project/OpenSearch/pull/11903


DROOLS-1131,JBPM-5080 - docs code dependency management

See also:

https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/pull/215
https://github.com/droolsjbpm/kie-docs/pull/104

@mbiarnes Please wait to merge -- fixing last detail.

@mbiarnes The PR isn't picking up the other PR: https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/pull/215

@mrietveld I fixed the Jenkins job config, so it should pickup the correct branch now. Sorry for the troubles.

+1 to merge


Regression: RedisStorage in v0.6.0

Hello,
I started to test the latest release 0.6.0 with Redis and found a regression.
I noticed that all tasks present in the redis queue NAMESPACE:active are fetched right away by a worker even though I'm using the following config:
let config = Config::default()
        .set_namespace(T::BROKER_NAMESPACE_JOB)
        .set_buffer_size(1)   // **Setting a buffer size of 1 to not fetch more than a task, allowing potential other workers to fetch them**
        .set_poll_interval(Duration::from_millis(1000))
        .set_max_retries(1)
        .set_keep_alive(Duration::from_secs(120));

let product_worker_name = format!("Worker-{}-{}", T::PRODUCT_ID, random::<u32>());

let product_worker = WorkerBuilder::new(product_worker_name)
        .layer(ConcurrencyLimitLayer::new(1)) // Allow running 1 job per worker at a time
        .data(backend.clone())
        .chain(|svc| svc.check_service_clone::<Identity>())
        .backend(RedisStorage::new_with_config(
            redis_connection.clone(),
            config,
        ))
        .build_fn(handle_task::<T, HttpDispatcherClient, HttpSlideClient>);

I think the regression is coming from this line :
https://github.com/geofmureithi/apalis/blob/094938d95f7ad90caab9edfc9fd04368fbcbe193/packages/apalis-redis/src/storage.rs#L482
IMO, we should have a safety here in order to not fetch from Redis in case the buffer is full ??
WDYT ?
EDIT: In order to do that I think we need to use Sender/Receiver providing methode like is_full ? or at least len & size
A good candidate could be async_channel ?

Hmm, we might not need to use async_channel. We could just compare the worker task count with the buffer size, This would be a quick fix too. If you can apply a quick PR that would be nice,
Another fix might be adding an atomic bool in Worker context for is_ready. Both would be straight forward. Let me know your thoughts.

I'll try that ;)

TBH I don't know how to connect the task_count to buffer_size.
I can set a cond like this worker.task_count() == 0 then fetch from redis.
This is nice in my case because I have a buffer_size of 1 and I use a ConcurrencyLimitLayer::new(1) but in the case
I have a ConcurrencyLimitLayer::new(10) for instance I don't want the task_count to reach 0 before fetching new tasks from Redis

I think the perfect solution if to have a cond like this:
// We need to get remaining free spot in the buffer if the buffer is not full pull 'free_spots' tasks
let free_spots = self.buffer_size - tx.len();
if free_spots > 0 {
    let res = self.fetch_next(worker.id(), free_spots).await; // change fetch_next with an additional arg
    ...
}

But if you have a better idea I'll take it ;)

ConcurrencyLimitLayer::new(10) just means the worker cannot be executing more than 10 tasks at a time.
Our goal here is that we need not to fetch when the service is not ready. This was the previous behavior. But currently we are using tower::CallAll which is possibly why consuming everything, coz the stream is being consumed to the end.
In all essence, we should use ConcurrencyLayer rather than buffer size. Buffer size is specific to redis and is about how many jobs to fetch at a time. I think I have some ideas on how to get this behavior back, let me write something tonight and we can discuss it tomorrow.

OK thanks for your help !!!

The thing I don't clearly understand if the definition of is_ready for me it's:
Is the worker able to start this task ? This may depend on ConcurrencyLimitLayer.
Indeed if instead of tower::CallAll we can use another mecanisme in order to not start task until the work is_ready eg able to start new task then it should solve this issue

Meanwhile, could you check something for me?
Modify the tower::CallAllUnordered to tower::CallAll in Worker::poll_jobs in apalis-core and see if it works

EDIT: IMO buffer_size should not be specific to redis.

I should have said its backend specific since some backends like messagequeues already handle this for themselves.
Workers just take Stream and consume them. To control this from a worker perspective, you need to use ConcurencyLayer

Meanwhile, could you check something for me?
Modify the tower::CallAllUnordered to tower::CallAll in Worker::poll_jobs in apalis-core and see if it works

Same behavior with tower::CallAll. All tasks are fetched from active and put into inflight :/

Ok, I already have something in mind. Hopefully its a single change :). Give me a few hrs

Hello,
Still not working, but I think you forget to push some changes ? Because on this branch I only see the layer, but not how the AtomicBool is updated

I need you to do something like:
if worker.is_ready() {
  fetch_next().await
}

In redis storage. Are you sure its not working?

It's getting better, with :
is_ready: Arc::new(AtomicBool::new(true)),

and
if worker.is_ready() {
  fetch_next().await
}

Fetch occurs only when the worker is not busy.
But still with a buffer_size of 1, a task is fetched will the worker is running.
Eg:
I push 3 tasks.
1 task is launched (concurrency still limited to 1)
1 task is i think the buffer
1 is in the redis active queue

Good!, I might need to use load(Acquire) this may fix it. But we are headed there.

Another weired behaviour but may be I'll open another dedicated issue is when I send a SIGTERM/SIGINT while using this code:
...
monitor.run_with_signal(shutdown_signal()).await?;

pub(crate) async fn shutdown_signal() -> Result<(), io::Error> {
    let mut sigterm = sigterm();
    let mut sigint = sigint();
    select! {
        biased;
        _ = sigterm.recv() => info!("SIGTERM signal. Exit now !!"),
        _ = sigint.recv() => info!("SIGINT signal. Exit now !!"),
    }
    Ok(())
}

Then the worker finish to handle all tasks and then exit, even the tasks that is still in the Redis active queue. (Following my previous comment all 3 tasks are done)

Hey, I think this has been resolved with the current push.

Also I have created this issue https://github.com/tower-rs/tower/issues/801 which would also relates to this.

Lol, look at what I did:
    /// Start running the worker
    pub fn start(&self) {
        self.state.running.store(false, Ordering::Relaxed);
        self.state.is_ready.store(false, Ordering::Relaxed);
        self.emit(Event::Start);
    }
    ```
Start sets it to false 😢

haha thx I'll test against the fix ;)

Everything works as expected !!!
My test:

Push 3 tasks on redis queue
start a worker -> 1 job is launched against the 1st task.
before the end I send a SIGTERM.
the job finish then the worker shutdown.
In Redis I still have 2 tasks remaining.
I re-launch a worker and anothe job is launched against the 2nd task.

Nothing to say expect, Good job !!!!
Thanks for your help !!

Hello @geofmureithi
Sorry to bother you, I just wanted to know when you're gonna release a patch version ?
Thank you


server response for open

Is it possible to get server response for open() method ?

I mean how to receive session id ?

There currently isn't a method. It would be easy to introduce a getter for the sessionId, but I didn't did that because it introduces a race condition in combination with the auto-reconnect feature. When you get a notification delivered that that you got connected this event might already be in the past, so when reacting to it you might already be disconnected or reconnected again and the sessionId you would retrieve with a getter might not match the sessionId that caused the connected event.
It think the most correct way at the moment (without dropping the auto-reconnect functionality) would be to deliver the sessionId with the statusChanged observable.

statusChanged with sessionId will be very helpful. Right now it is a blocker for us to use this library without knowing sessionId (it is necessary to know this value in our case. I think it is important not only for us)

Do the changes in cc0b9938fc18c1d338649c0f9b5329c21daf67fa help you?
Changing the status to an interface with various implementations depending on the state is the best I could think of. This means the reason for disconnects could now also be transmitted in the Disconnect events.
This might not be the complete implementation, I have to look at it again on another day.

Thank you! Seems like what we need! I hope this feature will be released to maven soon.

I reworked it a little bit and have realesed it now. Hope it helps you!
For all others: Sorry for breaking the API. However the necessary adaptions in the connection handlers can be done quite quickly.


Release 1.2.0

@rultor release it; the tag is 1.2.0.

@rultor release it; the tag is 1.2.0.

@rafamizes OK, I will release it now. Please check the progress here

@rultor release it; the tag is 1.2.0.

@rafamizes Done! FYI, the full log is here (took me 3min)


Mint integration

https://github.com/kvark/mint

While this is strictly speaking against the statement of the README:

This library is an extension of cgmath that provides collision detection primitives.

I approve the idea :+1:

Actually, I'm not going to strictly follow the way of original library. Situation in gamedev sphere changed a lot, I'm planning to follow the currents.
I hope cgmath will include mint soon and I wish mint be the de-facto standard wrapper for every math lib.

I took a look at this and created a huge mud-pile in my GitHub folder (the mint-integration branch). It will take some time to convert, as collision is a rich interface, but I definitely think it's a good idea. Traits that are needed to implement collision with cgmath should be private, and only basic traits like those from num-traits and mint should be public. It will certainly help reduce the log driver's waltz that most dependent library-writers have to do.

Now that you are using cgmath 0.18, isn't this a one-line fix?
Cargo.toml:
...
[features]
serde = ["serde_crate", "cgmath/serde", "num/serde"]
mint = ["cgmath/mint"] <----
...

Could even add the other cgmath supported features as well (rand,swizzle,unstable)
But this is quite redundant. All this does is to enable the feature in the cgmath crate, something that should already be done in the end-user toml (as long as the versions match).


Add NiceHash (?) block

https://explorer.btc.com/btc/block/000000000000000000016df2f1587f6ef04df009240be3c7f42040f2b432f78a
https://miningpool.observer/template-and-block/000000000000000000016df2f1587f6ef04df009240be3c7f42040f2b432f78a

https://explorer.btc.com/btc/block/0000000000000000000191fabac23dc46064d274bb6c651275774d70c582d90e

https://explorer.btc.com/btc/block/000000000000000000049348fb58bc21dbb230893b622bbb478c5fe8a6f3543e

Confirmed on NiceHash that they're indeed mining these blocks:

https://www.nicehash.com/my/easymining/recent-blocks?coin=BTC&page=0 (requires login)


Test for #1085 was added by accident

We introduced the test for #1085 in the wrong place.
This test is not testing #1085 and therefore basically useless => removed. #1085 was clossed as "by design"

Thanks


Feature/enhance code

Closes #
📑 Description

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

ℹ Additional Information

Description by Korbit AI
What change is being made?
Refactor the showLoginModal function to utilize CSS classes and improve accessibility, and remove the unused createGaugeChart function from the codebase.
Why are these changes being made?
The refactor simplifies styling by moving inline styles to CSS classes and enhances accessibility with aria attributes ensuring a better semantic structure. The removal of createGaugeChart is due to its redundancy, as it was commented out and no longer used, which cleans up the codebase.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Summary by CodeRabbit

New Features

Added a configuration setting to connect to GitHub Code Scanning.
Introduced new CSS classes for modal functionality.

Improvements

Enhanced accessibility of the login modal with ARIA attributes.
Updated styling for the login modal and its backdrop.

Bug Fixes

Removed the deprecated function for rendering gauge charts.

Style

Reformatted existing styles for consistency and introduced new styling for modal elements.

@gstraccini codacy bypass


Relieve requirements for exclusivity mode when not running against a real console (i.e. in tests)

Is your feature request related to a problem? Please describe.
When using dynamic displays, Spectre.Console ensures exclusivity through a call to this method:
https://github.com/spectreconsole/spectre.console/blob/5f97f2300cf848e84e90e963ee491042054ae477/src/Spectre.Console/Internal/DefaultExclusivityMode.cs#L37-L55
This makes perfect sense for when the application is running with a real console window. However, when running the application in tests (with an in-memory buffer provided to AnsiConsoleSettings.Out), this does not make much sense and forces the user to disable parallelization to avoid an exception.
Describe the solution you'd like
If given (AnsiConsoleOutput.Writer.IsStandardOutput() || AnsiConsoleOutput.Writer.IsStandardError()) == false, don't impose the single instance limitation on live displays.
Note that the current IsStandardOutput/Error() checks are a bit flaky. For example, in my case I have a custom StreamWriter implementation that wraps the output stream obtained via Console.OpenStandardOutput(). Because my instance of StreamWriter doesn't match Console.Out, it will incorrectly return false. This can be improved by checking the inner stream (via StreamWriter.BaseStream) and matching it against System.IO.ConsoleStream (which is unfortunately internal, so some reflection will be required).
Additionally, it seems that Spectre.Console already has a no-op implementation of IExclusivityMode, but it is internal: https://github.com/spectreconsole/spectre.console/blob/5f97f2300cf848e84e90e963ee491042054ae477/src/Spectre.Console.Testing/Internal/NoopExclusivityMode.cs#L6-L17
Making it public (and part of the core package) would already help but it would be nice if Spectre could also detect the most appropriate exclusivity mode automatically based on AnsiConsoleOutput, as explained earlier.
Describe alternatives you've considered
Currently I just programmatically provide a custom implementation of IExclusivityMode when running in tests, as that's the simplest workaround:
https://github.com/Tyrrrz/DiscordChatExporter/blob/28de1053f6151a889b497036d603c36f131cf31e/DiscordChatExporter.Cli/Utils/Extensions/ConsoleExtensions.cs#L19-L23
Additional context
Failures when parallelization is enabled:

No failures without parallelization:

@Tyrrrz Sounds like a perfectly reasonable request to me.
I would prefer this to be something you opt-in to explicitly though, and not using any kind of heuristics.

I would prefer this to be something you opt-in to explicitly though, and not using any kind of heuristics.

I suppose that can be achieved by making the NoopExclusivityMode public and part of the core package (instead of Spectre.Console.Testing). Would do you think?

I've been thinking about this a bit, and I think it's better that we make the semaphore non-static instead. The exclusivity mode should only be used per console, and now it's shared among all consoles, which frankly makes no sense.
Making the semaphore non-static would solve your issue as well.


[Major?] Saving to a local variable creates inconsistencies!

Hello,
I just tried the following:
var currentMoment = moment.unix(1449587079);

console.log(currentMoment);
console.log(moment.unix(1449587079));

For some odd reason, only the last one is right and both console logs are different!
Why is that? Or am I doing something wrong?
Bye Defcon0

Just ignore it, I fell over the .add method (called after my code) in combination with this chrome bug:
https://code.google.com/p/chromium/issues/detail?id=50316


How to share an InMemory Db between a read and write context

I have am working on a project that needs to support a read db and write db. I have decided to create two contexts accordingly. I have a Web API to interact with the db. I have an endpoint creates a user and another that fetches a user by id. The issue I'm running into is on the Development server, and Local instances, where an InMemory Db is used since development is rather rapid. The issue lies with the read context not giving me the data I stored with the write context. It thinks the table is empty. Upon further examination using the debugger, I am able to navigate and find the "missing" data. I'm not sure what is wrong.
Steps to reproduce
Here is an illustration of what I have in my project
public class WriteContext : IdentityDbContext<User>
{
  public WriteContext(DbContextOptions<WriteContext> options) : base(options) {}
}

public class ReadContext : IdentityDbContext<User>
{
  public ReadContext(DbContextOptions<ReadContext> options) : base(options) {}
}

public class UserController : Controller
{
  private readonly WriteContext _write;
  private readonly ReadContext _read;

  // constructor injection here

  // attributes...
  public IActionResult Post(UserModel model)
  {
    var user = // map model to User
    _write.Users.Add(user);
    _write.SaveChanges();
    return Ok(user);
  }

  // attributes...
  public IActionResult Get(string userId)
  {
    var user = _read.Users.SingleOrDefault(x => x.Id == userId);
    return Ok(user);
  }
}

public class Startup
{
  public void ConfigureServices(IServiceCollection services)
  {
    // other stuff...

    services.AddDbContext<WriteContext>(options => options.UseInMemoryDatabase("MemDb"));
    services.AddDbContext<ReadContext>(options => options.UseInMemoryDatabase("MemDb"));

    // other stuff
  }

  // other stuff
}

Add a user via write context and then try and fetch user via read context.
Further technical details
EF Core version: 2.0.1
Database Provider: InMemory
Operating system: macOS Sierra
IDE: Jetbrains Rider 2017.2

Duplicate of #10318, which is itself a duplicate of the root cause in #9613.


User Registration: Advocate API has /advocate/advocate in PATH

Describe the bug
Advocate API is not API compliant even in basic URL path. We have /advocate/advocate which is wrong. The paths should be as per API spec, namely:
/advocate/v1/_create
/advocate/v1/_update
/advocate/v1/_search
/advocate/clerk/v1/_create
/advocate/clerk/v1/_update
/advocate/clerk/v1/_search

hi @rajeshcherukumalli it is fixed. please test this in QA. cc: @Ramu-kandimalla


[SPARK-43909] Adding PR Template

This patch adds the PR template to the repository.

Merged to master.


SimpleBlinds2 not present when i install the package. its still pointing to version 1.0.0 from @codetheweb

Checklist

[x ] I have read the common issues wiki page
[x ] I have checked to make sure the plugin is up to date

i installed this plugin today, and when i was configuring my new teeka blinds they didnt work. i have a first generation and they work well.
I digged a bit into the code and i saw that you have a new type SimpleBlinds2, for this new blinds. However if i go into my modules folder i dont have this file. I noticed that downloading via npm the version that i get is without the SimpleBlinds2Accessory and its from @codetheweb repo.
can you help me with this @iRayanKhan ?
Btw keep the good work :)
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS:
iOS version:
Homehubs:
Node Version:
Plugin Version:
Accessory Type

Additional context
Add any other context about the problem here.

Hey, Are you able to try the RC beta branch? It should have this issue fixed. I will see if that branch contains it, if not I'll publish a beta for it.

Closed: Stale.


Reduce how long the page is blocked from responding to user input

Epic
426
Description
As a: Website visitor
I want to: not be blocked by JS to input data
So that: I can move on with what I'd like to do
Acceptance Criteria

[ ] All effected pages on: https://app-eu1.hubspot.com/content-strategy/26586079/recommendations/domain/16160539/hostname/16711611/category/PERFORMANCE/type/REDUCE_TOTAL_BLOCKING_TIME report no issue

Effectively a duplicate of #433, ironically, HubSpot is one of the main causes of page load delay.

Solved as much as possible as part of #401, times are significantly reduced across the board, but not below HubSpot's (somewhat arbitrary) threshold of 200ms.


Way to hook into Flyway migrate process to get fully resolved SQL text

Which version and edition of Flyway are you using?
Lastest
If this is not the latest version, can you reproduce the issue with the latest one as well?
(Many bugs are fixed in newer releases and upgrading will often resolve the issue)
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin)
Command-line
Which database are you using (type & version)?
Azure SQL
Which operating system are you using?
Windows/Linux
What did you do?
Is there a way to retrieve the full list of SQL statements that are about to run BEFORE the migration and AFTER  all the "placeholders" are resolved? We need a way to check all the SQL scripts to ensure the scripts don't run specific commands such as CREATE USER, DROP DATABASE, etc. as we running this in a controlled environment and though that those commands work great during development, they can't be run in PRODUCTION. In Production the database user will have elevated privileges so we need to check the scripts before running them. I see the Dry Runs Pro feature but that just writes to a file. We'd like to get this data back on a callback and then we can validate it prior to the migration running.
Can you tell us what if there are callbacks to hook into or if there are Interfaces we should use?

This will be possible in Flyway Pro and Enterprise Edition 5.2 with statement-level callbacks. Duplicate of #1289


[css] new at-rules such as @property are not yet recognized

Type: Bug

Copy-paste the following snippet to a CSS stylesheet (this example is taken from MDN):

@property --my-color {
  syntax: '<color>';
  inherits: false;
  initial-value: #c0ffee;
}

Expected: VSCode won't see @property as an unknown at-rule.
Actual: VSCode sees @property as an unknown at-rule.

VS Code version: Code 1.70.0 (da76f93349a72022ca4670c1b84860304616aaa2, 2022-08-04T04:38:16.462Z)
OS version: Windows_NT x64 10.0.22000
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-10300H CPU @ 2.50GHz (8 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.84GB (0.45GB free)

Process Argv
--crash-reporter-id 57c06a69-98ac-4fc4-8225-c3db59f39ccc

Screen Reader
no

VM
0%

Extensions (13)

Extension
Author (truncated)
Version

vscode-intelephense-client
bme
1.8.2

better-toml
bun
0.3.2

vscode-eslint
dba
2.2.6

rainbow-csv
mec
2.4.0

vscode-language-pack-pl
MS-
1.70.8030912

python
ms-
2022.12.0

vscode-pylance
ms-
2022.8.10

jupyter
ms-
2022.7.1002181843

material-icon-theme
PKi
4.19.0

LiveServer
rit
5.7.5

wikitext
Row
3.6.5

sass-indented
syl
1.8.22

open-in-github-desktop
wra
1.4.3

(3 theme extensions excluded)

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonvspyt551cf:30345471
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805cf:30301675
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30525688
vsaa593cf:30376535
vsc1dst:30438360
pythonvs932:30410667
wslgetstarted:30449410
vscscmwlcmt:30465135
cppdebug:30492333
pylanb8912cf:30529770
vsclangdf:30486550
c4g48928:30535728
hb751961:30541207
dsvsc012cf:30540253

@property at MDN: https://developer.mozilla.org/en-US/docs/Web/CSS/@property

The following CSS properties should also receive syntax highlighting:

@property's properties

syntax
inherits
initial-value
the custom property's name (just like for @counter-style)

This also applies to other new at-rules:

@layer (can have other at-rules except @charset, @import and @namespace nested, including other @layers)
@color-profile and its properties:

src
rendering-intent

@import can now import layers as well through layer(<layer-name>)
@scroll-timeline and its properties:

source
orientation
scroll-offsets

Syntax highlighting issues need to be filed against https://github.com/atom/language-css

This isn't just about syntax highlighting, but VSCode actually reporting these at-rules as unknown.


Archotech defs patch

Changes Archotech vanilla defs from overrides to patches - issue #1

For some reason, the game isn't finding the parent defs for the patched defs, even though it finds the same parent defs for the newly-added items, even though it's the same parents. Closing this PR until I figure out why.


S3 compatible with minio

Hello,
I'm having problem configuring Deep Storage to use Minio S3, now I'm having it with the new Druid version.
Does anyone have any ideas on this?
Thanks and Regards

I know a few people are using with minio, and it's working
can you paste your config for us?
there's some hints that can help you here
https://www.druidforum.org/t/error-when-using-deepstorage-s3-from-b2-backblaze-s3-compatible/7244/6


Adding additional classes when using the tabs builder....

Hi,
I am using the tabs builder as per usual:
 using (TabsBuilder<IndexViewModel> tabs = Html.Bootstrap().Begin(new TwitterBootstrapMVC.Tabs("TabsId")))

and it emits the expected HTML of:

however I want to add an additional class myClass to the div so that I end up with:

How do I achieve this?

Added extension methods .Class(), .HtmlAttributes(), .Data() to the class Tabs.
Please get latest.

Hi Dimitry,
Thanks. Works perfectly.


Add support for AlmaLinux8

https://github.com/CSCfi/ansible-role-dhcp-kickstart/blob/CCCP-4477/tests/inventory#L38 - almalinux8 should be added here too, otherwise the travis tests will not run for almalinux8


UIINREACH-162: Edit INN-Reach Paging Slip Templates

Test errors found. See https://jenkins-aws.indexdata.com/job/folio-org/job/ui-inn-reach/job/PR-219/1/


ODR for accelerometer and gyro is in kHz not Hz

In:

/// Accelerometer ODR
///
/// Default value: `Off`
#[allow(non_camel_case_types)]
#[derive(Copy, Clone, Eq, PartialEq, Ord, PartialOrd, Hash, Debug)]
pub enum Odr_Xl {
    Off,   // off
    Hz125, // 12.5 Hz
    Hz26,  // 26   Hz
    Hz52,  // 52   Hz
    Hz104, // 104  Hz
    Hz208, // 208  Hz
    Hz416, // 416  Hz
    Hz833, // 833  Hz
    Hz166, // 1.66 Hz
    Hz333, // 3.33 Hz
    Hz666, // 6.66 Hz
}

the last three values should be in kHz. Same issue for gyroscope.
I don't think the actual functionality is affected. Some of the combinations are only valid depending on performance mode. That would probably only be enforced with some redesign and making register access less direct.

do you think updating the docs would be sufficient for now?
+1 on making the register access less direct, I need to also make a cleaner way of storing the address and the actual register value, haven't cracked that yet

I think it is fairly confusing with Hz166 compared to kHz166, I would definitely try changing that as soon as possible. Maybe even use kHz6_666. Or even Hz6660. All of them less nice, but at least correct. I often don't realize the docs have different values in this case, since the enum seems so obvious to understand in the first place.
Some of the other crates linked in #1 has taken a somewhat higher level approach, then it is a bit easier to enforce the right usage. But it is a bit of work moving to that, so I don't expect that you want to spend time on that.

Just pumped out a new version renaming them with the proper khz :D

Great!! Thanks
søn. 9. jan. 2022, 16:34 skrev Andrei Sousa @.***>:

Just pumped out a new version renaming them with the proper khz :D
—
Reply to this email directly, view it on GitHub
https://github.com/sousandrei/ism330dhcx/issues/6#issuecomment-1008319887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAN362VCX7RQM4DMJZWZS3UVGTHBANCNFSM5KAK4M6A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Added Delete confirmation Dialog in the details Activity

@Bradzer thank you for the pull request!
I was looking at the code, not merged yet. I think the dialog won't survive a screen rotation, did you try it?
Let me know.
Many thanks and best wishes!
Lorenzo

@redlor
Indeed I made the test and the dialog didn't survive the screen rotation. I started solving the issue. I made progress but still having a problem with firebase.
Maybe you can have a look on the code. Is it okay for you ?

@Bradzer
happy new year!
Yes, I'll copy your code and make some tests as soon as I can. In the meantime, try yourself doing researches and making tests, this will help you learning new things.
The app made a lot of progresses but there are still many things to do.
Many thanks for helping.


Fix 'The domain/default pair of (Info.plist,CFBundleShortVersionString) does not exist' issue

Fix 'The domain/default pair of (Info.plist,CFBundleShortVersionString) does not exist' issue
Changes in this pull request
Issue fixed: #
when run pod install under Examples/Examples-iOS, you may encounter the error:
The domain/default pair of (path/to/IGListKit/Source/Info, CFBundleShortVersionString) does not exist.
[!] The `IGListDiffKit` pod failed to validate due to 1 error:
    - ERROR | version: A version is required

Changed the usage of defaults read in scripts/version.sh to /usr/libexec/PlistBuddy will fix the problem.
Checklist

[x] All tests pass. Demo project builds and runs.
[x] I added tests, an experiment, or detailed why my change isn't tested.
[x] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[x] I have reviewed the contributing guide

Thanks for fixing this!

same as https://github.com/Instagram/IGListKit/pull/1475


Script fails with "ArgumentError: Argument --name requires a value" error

When this script is run as a Lifecycle configuration for a SageMaker Notebook Instance, it fails.
CloudWatch has this error logged: "ArgumentError: Argument --name requires a value"
The Notebook Instance eventually fails to start due to timeout.

Resolving, thank you for the PR and for reporting this issue!


Blank CWU grid

The grid appears to be empty until I add a "help wanted" tag to an issue in the old DevEx-app repo

Oddly, the list is blank for me, even after Paul managed to make it visible
by adding the tag...
There is a CORS error/warning in the console, but I think that's to do
with the resize and not the tables/content themselves...
ᐧ
On Fri, Aug 12, 2016 at 12:06 PM, Paul Roberts<EMAIL_ADDRESS>wrote:

The grid appears to be empty until I add a "help wanted" tag to an issue
in the old DevEx-app repo
[image: screen shot 2016-08-12 at 12 03 58 pm]
https://cloud.githubusercontent.com/assets/11237533/17633979/0d54cede-6085-11e6-8191-bc8d8bd76a10.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BCDevExchange/BCDevExchange-Site/issues/23, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAqpIr9iGRjcHMT4SQQAKBaUKwADQrLvks5qfMQ1gaJpZM4JjXty
.

Validated


🩹 Add fill="currentColor" to Bluesky icon

fill="currentColor" is missing in the Bluesky icon making it stand out from other icons (see image). This was the only icon with the missing fill attribute and I presume it was unintentionally missed.

I would also like this pull request merged and released

+1 to this. I hope this goes in

Thanks for picking this up. Merging in now.


Remoting question

Hi @PaulHigin
I was wondering how this module could help me reduce time of fetching data over the network using Invoke-Command -ComputerName
Is that something that this module could do? would you have an example to share?

Invoke-Command already supports fan-out scenarios, so in this case you don't need to use ThreadJob.
$resultsJob = Invoke-Command -Cn @('computer1','computer2','computer3',...) -FilePath .\GatherData.ps1 -ThrottleLimit 25 -AsJob

But if you were just copying files from network shares, or downloading from REST APIs, then because of the slowness of the connection, ThreadJob should speed things up quite a bit.
$destPath = 'c:\DestFolder'
$listOfSourcePaths = 'filePath1','filePath2',...
$listOfSourcePaths | Foreach-Object {
    Start-ThreadJob {
        Copy-Item -Path $using:_ -Dest $using:destPath -Recurse -PassThru -Force
    } -ThrottleLimit 25
} | Wait-Job | Receive-Job

If you are using PowerShell 7+, you can also use the new Foreach-Object -Parallel feature, which is simpler.
$destPath = 'c:\DestFolder'
$listOfSourcePaths = 'filePath1','filePath2',...
$listOfSourcePaths | ForEach-Object -Parallel {
    Copy-Item -Path $_ -Dest $using:destPath -Recurse -PassThru -Force
} -ThrottleLimit 25


Version 7.0.0 issue - import botocore error

Problem description
Hello, we're using jsf (https://github.com/ghandic/jsf) library, which for some reason uses smart_open.
On our previous builds (until 1h ago when 7.0.0 was released), jsf was taking smart-open==6.4.0 and all was working fine. However, now pip decides to install jsf with the latest smart-open version (7.0.0).
The version 7.0.0 seems to have a hard dependency on botocore because when you import smart_open it fails if you don't have this installed. Should this now be a dependency that should come in this package requirements OR should it not fail if botocore isn't installed?
Steps/code to reproduce the problem

New virtualenv
pip install smart_open==7.0.0
python -c 'import smart_open'

If you install 6.4.0 instead it just works, without having botocore installed.
# Traceback from `import jsf` but the same happens with normal `import smart_open`
/usr/local/lib/python3.11/site-packages/jsf/__init__.py:1: in <module>
    from jsf.parser import JSF
/usr/local/lib/python3.11/site-packages/jsf/parser.py:13: in <module>
    from smart_open import open as s_open
/usr/local/lib/python3.11/site-packages/smart_open/__init__.py:34: in <module>
    from .smart_open_lib import open, parse_uri, smart_open, register_compressor  # noqa: E402
/usr/local/lib/python3.11/site-packages/smart_open/smart_open_lib.py:35: in <module>
    from smart_open import doctools
/usr/local/lib/python3.11/site-packages/smart_open/doctools.py:21: in <module>
    from . import transport
/usr/local/lib/python3.11/site-packages/smart_open/transport.py:104: in <module>
    register_transport("smart_open.s3")
/usr/local/lib/python3.11/site-packages/smart_open/transport.py:49: in register_transport
    submodule = importlib.import_module(submodule)
/usr/local/lib/python3.11/site-packages/smart_open/s3.py:121: in <module>
    RETRY = Retry()
/usr/local/lib/python3.11/site-packages/smart_open/s3.py:86: in __init__
    self.exceptions: List[Exception] = [botocore.exceptions.EndpointConnectionError]
E   NameError: name 'botocore' is not defined

Versions
6.4.0 -> OK
7.0.0 -> Does not work without botocore installed
Checklist
Before you create the issue, please make sure you have:

[x] Described the problem clearly
[x] Provided a minimal reproducible example, including any required data
[x] Provided the version numbers of the relevant software

OK, thank you for letting me know. I can see what the problem is, and will release a bugfix soon.

I've released 7.0.1, please try it.

Should this issue be closed?

Yes, thank you @browniebroke


Debugging test orchestrator

#273  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


LilyBin unstable version out of date

The current unstable version on LilyBin appears to be 2.19.38, a bit out of date.
I'm not sure if the update script is not running or something else?

It's always been a manual process to update. I can't think of a really easy way to automate it. Thanks for the reminder to update it. I'll do it today or tomorrow.

I've just pushed 2.19.48 to AWS. I've also updated the Node.js runtime on Lambda so that 4.3 is used instead of 0.10.


Chore(m-agent): Add initial version of litmus m-agent

Signed-off-by: neelanjan00<EMAIL_ADDRESS>Tested on: Ubuntu, CentOS, RHEL and SUSE Linux
Requirements: Linux OS, Systemd

We can have GitHub Actions for Go Binary build and to check the go and shell linting ... please refer to other chore repos for the same.
The same action can also run the unit tests as well.

IMO we can also containerize the installation and uninstallation scripts - We can also do this in a separate PR. The user will have both options to run the shell script for the installation or just run a container.


Update integration workflow

Partial resolution of dbt-labs/dbt#3924. If we agree with the approach here, we can extend to dbt-snowflake + dbt-bigquery.

Run the integration workflow nightly at 5 UTC, to identify any regressions caused by changes in dbt-core.
Install dbt-core from HEAD (instead of hardcoding develop branch) to get latest changes
Remove logic from integration workflow that picks plugins to test based on file changes. This is nice for PRs that only edit the README, but it's not as important now that this repo only contains the Redshift plugin. This logic seemed to be running into bugginess on push triggers (e.g.); we'll be able to see after merging this PR.

Ah, it was actually the unit tests that caught the breaking change (i.e. deprecation enacted) in https://github.com/dbt-labs/dbt/pull/3975. Nice!
Should we consider running both workflows nightly?


refactor: Refactor data scratch-ipblocks

Fixes #1746
Test Plan

How do we know the code works?

Code is refactored according to the description in #1746
command flank ip-blocks list runs without any problems

Checklist

[x] Unit tested

@piotradamczyk5 Sorry for the mistake, I have copy-pasted the wrong file name, should be TestEnvironmentInfo.kt instead of ListIPBlocks.kt

The files:

IpBlocksToTable.kt
~ListIPBlocks.kt~ TestEnvironmentInfo.kt

are not a part of the adapters layer because they don't have any references to google API libs additionally they are widely used in the domain layer. According to the diagram below, the code from adapters cannot be used directly in the domain layer.

Please move back those two files to the ftl.environment and leave them for further domain refactor.

For me data and adapter layers are badly named:
adapter layer is data while data is adapter

@piotradamczyk5 I do not agree. The adapter is an implementation that is adapting the library code or client implementation to the data structures and interfaces declared in the data layer.

@piotradamczyk5 I do not agree. The adapter is an implementation that is adapting the library code or client implementation to the data structures and interfaces declared in the data layer.

Discussed on slack with @jan-gogo .
The new package structure will be
presentation > domain > api > adapter > client


Memory error when calculation perplexity for evaluation

I'm using Keras with Tensorflow backend and I use the following code to evaluate my model on a test and a dev set with perplexity. The code below is called every epoch of training and memory sums up until it finally crashes.
I tried gc.collect() or K.clear_session() but it didn't work. Would be great to get some help.
def compute_perplexity(self, modelName, sentences):
    all_labels, all_predictions = self.predictLabels_for_perplexity_evaluation(self.models[modelName], sentences)
    # add an axis to fit tensor shape
    for i in range(len(all_labels)):
        all_labels[i] = all_labels[i][:,:, np.newaxis]

    #calculate perplexity for each sentence length and each datapoint and append to list
    perplexity = []
    for i in range(10,15): #range(len(all_labels)):
        start = time.time()
        xentropy = K.sparse_categorical_crossentropy(tf.convert_to_tensor(all_labels[i]), tf.convert_to_tensor(all_predictions[i]))
        perplexity.append(K.eval(K.pow(2.0, xentropy)))
        print('time for one set of sentences. ', time.time()- start)
    #average for each datapoint
    for i in range(len(perplexity)):
        perplexity[i] = np.average(perplexity[i], axis=1)
        perplexity[i] = np.average(perplexity[i])

    return np.mean(perplexity)

Anyone?
I can calcluate it without using Keras and Tensorflow functions. But this is very, very slow.

Closing this issue due to lack of activity. Feel free to reopen when additional information is available. Thanks!


Client.GetObjectAsync() with large files - Oculus Quest

Expected Behavior
Client.GetObjectAsync() should return a response object.
Current Behavior
Client.GetObjectAsync() crashes oculus quest headset's OS preventing any log data to be shown in Android Device Monitor.
Steps to Reproduce (for bugs)

Upload a 1GB object to a private amazon s3 bucket.
Create an instance of AmazonS3Client:
IAmazonS3 Client = new AmazonS3Client(Credentials, _S3Region);
Initiate the get object request:
Client.GetObjectAsync(S3BucketName, "star_trails.mp4", (responseObj) => {});

Context
Trying to download a 1 GB video file to an Oculus Quest headset to be played locally when offline.
Example Code used for testing:
    private AWSCredentials Credentials
    {
        get
        {
            if (_credentials == null)
                _credentials = new CognitoAWSCredentials(identityPoolId, _CognitoIdentityRegion);
            return _credentials;
        }
    }

    private IAmazonS3 Client
    {
        get
        {
            if (_s3Client == null)
            {
                _s3Client = new AmazonS3Client(Credentials, _S3Region);
            }

            return _s3Client;
        }
    }

    public void DownloadObject()
    {

        Client.GetObjectAsync(S3BucketName, "star_trails.mp4", (responseObj) =>
        {
            var response = responseObj.Response;

            if (response.ResponseStream != null)
            {
                string rootPath = Application.persistentDataPath;

                // Points to the place we access our videos.
                string videoPath = Path.Combine(rootPath, "StreamingAssets");

                using (var fs = File.Create(Path.Combine(videoPath, "video.mp4")))
                {
                    byte[] buffer = new byte[response.ContentLength];
                    int count;

                    while ((count = response.ResponseStream.Read(buffer, 0, buffer.Length)) != 0)
                    {
                        fs.Write(buffer, 0, count);
                    }

                    fs.Flush();
                }
            }
        });
    }

Debug Log:
03-25 11:00:36.849: E/Unity(9597): Could not allocate memory: System out of memory!
03-25 11:00:36.849: E/Unity(9597): Trying to allocate:<PHONE_NUMBER>B with 16 alignment. MemoryLabel: WebRequest
03-25 11:00:36.849: E/Unity(9597): Allocation happened at: Line:75 in
03-25 11:00:36.849: E/Unity(9597): Memory overview
03-25 11:00:36.849: E/Unity(9597): [ ALLOC_DEFAULT ] used: 28451816B | peak: 49836581B | reserved: 29077638B
03-25 11:00:36.849: E/Unity(9597): [ ALLOC_TEMP_JOB ] used: 0B | peak: 0B | reserved: 5242880B
03-25 11:00:36.849: E/Unity(9597): [ ALLOC_TEMP_BACKGROUND_JOB ] used: 0B | peak: 0B | reserved: 1048576B
03-25 11:00:36.849: E/Unity(9597): [ ALLOC_GAMEOBJECT ] used: 882875B | peak: 937377B | reserved: 895681B
03-25 11:00:36.849: E/Unity(9597): [ ALLOC_GFX ] used: 2994944B | peak: 38777811B | reserved: 2997661B
03-25 11:00:36.849: E/Unity(9597): [ ALLOC_TEMP_THREAD ] used: 0B | peak: 0B | reserved: 2785280B
03-25 11:00:36.849: E/Unity(9597):
03-25 11:00:36.849: E/Unity(9597): (Filename:  Line: 1161)
03-25 11:00:36.849: E/Unity(9597): --------- beginning of crash

Your Environment
UnityEditor: 2018.4.19f1
VisualStudio: Community 2017 - v15.9.21
Plugins Used:

AWS SDK Unity: aws-sdk-unity_3.3.706.0
AWSSDK.S<IP_ADDRESS>.39
AWSSDK.CognitoSync.<IP_ADDRESS>
AWSSDK.IdentityManagement.<IP_ADDRESS>

Hi @tenebris93,
Starting with version 3.5 of the AWS SDK for .NET, projects using Unity 2018.1 or later should target the .NET Standard 2.0 release of the SDK. You can find additional information in the developer guide: Unity support and Migrating your Unity application.
Please try using the .NET Standard 2.0 version of the AWS SDK for .NET (version 3.5).
Thanks,
Ashish


installation verify error

Bug report

Bug summary

Installation verify code in python returns weird error.
Code for reproduction

from pycromanager import Bridge
bridge = Bridge()
print(bridge.get_core())

Expected outcome
pycromanager.core.mmcorej_CMMCore object at 0x7fe32824a208
Actual outcome

Traceback (most recent call last):
File "", line 4, in
File "C:\Users\ASUS\AppData\Local\Programs\Python\Python38\lib\site-packages\pycromanager\core.py", line 348, in get_core
self.core = self.construct_java_object("mmcorej.CMMCore")
File "C:\Users\ASUS\AppData\Local\Programs\Python\Python38\lib\site-packages\pycromanager\core.py", line 284, in construct_java_object
self._master_socket.send(message)
File "C:\Users\ASUS\AppData\Local\Programs\Python\Python38\lib\site-packages\pycromanager\core.py", line 115, in send
self._socket.send_multipart(message_parts)
File "C:\Users\ASUS\AppData\Local\Programs\Python\Python38\lib\site-packages\zmq\sugar\socket.py", line 555, in send_multipart
return self.send(msg_parts[-1], flags, copy=copy, track=track)
File "C:\Users\ASUS\AppData\Local\Programs\Python\Python38\lib\site-packages\zmq\sugar\socket.py", line 505, in send
return super(Socket, self).send(data, flags=flags, copy=copy, track=track)
File "zmq\backend\cython\socket.pyx", line 718, in zmq.backend.cython.socket.Socket.send
File "zmq\backend\cython\socket.pyx", line 765, in zmq.backend.cython.socket.Socket.send
File "zmq\backend\cython\socket.pyx", line 247, in zmq.backend.cython.socket._send_copy
File "zmq\backend\cython\socket.pyx", line 242, in zmq.backend.cython.socket._send_copy
File "zmq\backend\cython\checkrc.pxd", line 28, in zmq.backend.cython.checkrc._check_rc
zmq.error.ZMQError: Operation cannot be accomplished in current state

Version Info

Operating system: Win10
pycromanager version: 2.4.0 (as following)

MicroManager version:  2.0.0-gamma1 20210602
Python version: Python 3.8
Python environment (command line, IDE, Jupyter notebook, etc) IDE: Pycharm

Something going wrong with the communication library pycromanager uses under the hood. My first thought would be to make sure you're not accidentally running multiple instances of something. Maybe try shutting down everything (including computer) and running from a fresh environment

Thank you for your reply. It seems still not work after restarting everything.So Is there another way to start a fresh environment?

Not that I can think of.
This is strange because it seems like the bridge is successfully connecting, but then not working when you try to get the core. If you try:
bridge = Bridge(debug=True)

Maybe something informative will be printed?

Thank you for your patience. It works after I reinstall the pycromanager.


Support VRCSDK 3.5.0

Fixes #616
Fixes #394

failing test is expected since the CI is pull_request_target.


Dice's calculation is obviously wrong

In the GlaS dataset, your IoU is only 76.73. According to the mathematical definition of IoU and Dice coefficient, your Dice is absolutely impossible to reach 99.09, so your Dice coefficient calculation is obviously wrong. You should check your code implementation of Dice coefficient calculation.
在GlaS数据集里，你的IoU只有76.73，根据IoU和Dice系数的数学定义，你的Dice是绝对不可能达到99.09的，所以你的Dice系数计算显然是错误的。你应该检查一下你关于Dice系数计算这部分的代码实现。

Indeed, as of now, Dice still finds it difficult to reach the incredible number of 99


Separate file for contributors

Separate file for contributors
Describe the solution you'd like
Currently the Contributors are listed in main Readme of the project which can we moved to separate  file for a a better Project Structure

@yung-coder - I'm new to open source and would like to take-up this issue. Could you please assign this issue to me?
Thanks,
Baji

Go ahead @bajibpathan  🚀

@yung-coder  - Created the PR. Please review and advise if any changes needed.
Thanks


No JobCache size setting

I am trying to implement a job cache but noticed the PVCs are created with the default 5MB allocation. Job storage has buildStorageSize but it looks like there is no equivalent for job cache (https://github.com/Azure/brigade/blob/master/brigade-worker/src/events.ts#L150).

For job cache, size is set in brigade.js via myJob.cache.size = ....


x/build/cmd/gerritbot: first "PR has been imported to Gerrit" message doesn't get posted

This is a subset or a more specific issue related to #27504.
I'm noticing that the first message that gerritbot is supposed to post (as gopherbot user) is no longer appearing.
Here's an old PR where it appears:

https://github.com/golang/go/pull/24004#issuecomment-367390174

Here are newer PRs where it does not:

#27471
#27470

I noticed this happen when I created https://github.com/golang/dl/pull/1. The first message didn't get posted, but the following ones did.
I captured the logs, and I suspect I can see why it's not working.
 It's not there in other subrepos:

- https://github.com/golang/image/blob/c73c2afc3b812cdd6385de5a50616511c4a3d458/codereview.cfg
- https://github.com/golang/net/blob/161cd47e91fd58ac17490ef4d742dc98bb4cf60e/codereview.cfg

Using this to test PRs for golang/go#26949.

 It's not there in other subrepos:

- https://github.com/golang/image/blob/c73c2afc3b812cdd6385de5a50616511c4a3d458/codereview.cfg
- https://github.com/golang/net/blob/161cd47e91fd58ac17490ef4d742dc98bb4cf60e/codereview.cfg

Using this to test PRs for golang/go#26949.

Change-Id: I7152ea2dae1a511928e787598696274cf2b315f9
GitHub-Last-Rev: b41da79638ca03d8baf47b66f1e033fb04337ddb
GitHub-Pull-Request: golang/dl#1
]
2018/09/07 20:49:17 Executing [git -C /root/.gerritbot/tmp/worktree_golang_dl_1 push -o nokeycheck origin HEAD:refs/for/master]
2018/09/07 20:49:19 Cleaning up...
2018/09/07 20:49:19 Executing [git -C /root/.gerritbot/tmp/worktree_golang_dl_1 checkout master]
2018/09/07 20:49:19 Executing [git -C /root/.gerritbot/tmp/worktree_golang_dl_1 branch -D golang/dl#1]
2018/09/07 20:49:19 Executing [rm -rf /root/.gerritbot/tmp/worktree_golang_dl_1]
2018/09/07 20:49:19 Executing [git -C /root/.gerritbot/repos/https:%2F%2Fgo.googlesource.com%2Fdl worktree prune]
2018/09/07 20:49:19 Executing [git -C /root/.gerritbot/repos/https:%2F%2Fgo.googlesource.com%2Fdl branch -D worktree_golang_dl_1]
2018/09/07 20:49:19 processPullRequest: importGerritChangeFromPR(golang/dl#1, &{0xc03530b9f0<PHONE_NUMBER>-09-07 20:46:13 +0000 +0000 1 {GitCommit 204de0d4586b1c069f7e928abefc9ae3e580acf3} master 0xc04734efe0 [0xc04734f000] new false [golang/go#26949 golang/dl#1] []}): could not find change URL in command output: "remote: \rremote: Processing changes: updated: 1 (\\)\rremote: Processing changes: refs: 1, updated: 1 (\\)\rremote: Processing changes: refs: 1, updated: 1 (\\)\rremote: Processing changes: refs: 1, updated: 1 (\\)\rremote: Processing changes: refs: 1, updated: 1, done            \nremote: warning: 73715c7: no files changed, message updated        \nremote: \nremote: SUCCESS        \nremote: \nremote: Updated Changes:        \nremote:   https://go-review.googlesource.com/c/dl/+/134117 dl: remove blank line from codereview.cfg        \nremote: \nTo https://go.googlesource.com/dl\n * [new branch]      HEAD -> refs/for/master"

Specifically, see the last line with the "could not find change URL in command output" error:
processPullRequest: importGerritChangeFromPR(golang/dl#1, nil): could not find change URL in command output: "remote: \rremote: Processing changes: new: 1 (\\)\rremote: Processing changes: new: 1 (|)\rremote: Processing changes: refs: 1, new: 1 (|)\rremote: Processing changes: refs: 1, new: 1 (|)        \rremote: Processing changes: refs: 1, new: 1, done            \nremote: \nremote: SUCCESS        \nremote: \nremote: New Changes:        \nremote:   https://go-review.googlesource.com/c/dl/+/134117 remove blank line from codereview.cfg        \nTo https://go.googlesource.com/dl\n * [new branch]      HEAD -> refs/for/master"

I suspect what happened is that the URL format that Gerrit returns has changed from:
https://go-review.googlesource.com/#/c/dl/+/134117
To:
https://go-review.googlesource.com/c/dl/+/134117
And it no longer matches this regexp:
https://github.com/golang/build/blob/a196f5a8e8d347b6b9fe4cd225a3393a77c03214/cmd/gerritbot/gerritbot.go#L532-L533
/cc @andybons

This is deployed.
I've confirmed it's working. I created a new PR just now, and the "This PR has been imported to Gerrit for code review" comment was posted:
https://github.com/golang/dl/pull/2#issuecomment-420070683
Also, editing an old PR where the comment wasn't posted earlier causes it to get posted:
https://github.com/golang/dl/pull/1#issuecomment-420071008
Better late than never!


How to eliminate red eye effect using openCV

![Code Snippet.txt](https://github.com/public-apis/public-apis/files/14471794/Code.Snippet.txt

Take this down
basically the creators of this are abusing  their power and arent treating people who help contribute and maintain the project so unlike this unstar this star ny fork of this that doesnt support this one
#3104
#3484


xo.enable is only detected on config change and not on startup

I have a project that uses node and xo, with a subfolder that uses deno and deno's linter. When I open the subfolder, which has a vscode config like {"deno.enable": true, "xo.enable": false}, xo still reports errors. If I change it to "xo.enable": true and then back to "xo.enable": false, xo disables itself properly.

Hello - I am not sure I understand the problem here entirely. Having nested vscode configurations isn't something I am really aware of. It's likely the best way to solve this issue is to ignore the subfolder you don't want to lint in the xo configuration or add a .eslintignore file.
see: https://github.com/xojs/xo?tab=readme-ov-file#ignores
Let me know if this will work for you.

No, I open the subfolder as a separate vscode workspace.

Should be fixed as of v3.19 let me know if you have any more issues


How to add custom gray color

Description
I've add "brand" colors and "smoke" (our gray colors) in tawilind.config.ts.
The "brand" color is showing correctly when i use "bg-primary-500" for exemple. But the gray isn't showind correctly
tailwind.config.ts
import type {Config} from 'tailwindcss'

export default <Partial<Config>>{
    theme: {
        fontFamily: {
            sans: ['"IBM Plex Sans"', 'sans-serif']
        },
        extend: {
            colors: {
                brand: {
                    50: '#F4F3FF',
                    100: '#EBE9FE',
                    200: '#D9D6FE',
                    300: '#BDB5FD',
                    400: '#9C8BFA',
                    500: '#7C5CF6',
                    600: '#6B3AED',
                    700: '#5C28D9',
                    800: '#4F22BB',
                    900: '#411D95',
                    950: '#261065'
                },
                smoke: {
                    50: '#FFFFFF',
                    100: '#D1D1D1',
                    200: '#B0B0B0',
                    300: '#888888',
                    400: '#6D6D6D',
                    500: '#5D5D5D',
                    600: '#4F4F4F',
                    700: '#454545',
                    800: '#3D3D3D',
                    900: '#242424',
                    950: '#ff0f37'
                },
            },
            height: {
                navbar: 'var(--h-navbar)',
            },
            spacing: {
                aside: 'var(--w-aside)',
            },
        }
    }
}

app.config.ts
export default defineAppConfig({
  ui: {
    primary: "brand",
    gray: "smoke",
    container: {
      base: "mx-auto",
      padding: "px-4 sm:px-6 lg:px-8",
      constrained: "sm:max-w-screen-2xl",
    },
    dropdown: {
      container: "z-50",
      background: "bg-white dark:bg-gray-700",
    },
    table: {
      default: {
        emptyState: {
          label: "Nenhum registro encontrado",
        },
        loadingState: {
          label: "Carregando...",
        },
      },
    },
    notifications: {
      position: "bottom-0 lg:top-0 lg:bottom-auto",
    },
  },
});

Please provide a valid reproduction in your issues, for example a link to a Stackblitz.

I don't know why, but in my case, to "solve" this i needed to remove node_modules and package-lock.json and then run npm i. Everytime i change a color variable, it must be done.

I believe it has something to do with Nuxt Tailwind's module. It is currently required to restart the server when editing tailwind.config.ts. This should not be required anymore when nuxt/ui will upgrade to @nuxtjs/tailwindcss 6.12


GPU requirement

Thanks for your amazing work!
May I know that is the GPU requirements for training one ablation model? Can I failed training on 24GB GPU. Thanks a lot.

Hi, you can use our diffusers based code with xformers support to run it on a 24GB GPU. Example training command will be as shown here.
Let me know if you face any issues. Thanks.


silence DeprecationWarnings in scipy.special

There's several of
/home/br/virtualenvs/scipy/lib/python3.5/site-packages/numpy/lib/utils.py:139: DeprecationWarning: `sph_yn` is deprecated!
scipy.special.sph_yn is deprecated in scipy 0.18.0. Use scipy.special.spherical_yn instead. Note that the new function has a different signature.
  warnings.warn(depdoc, DeprecationWarning)

A fix should be to filter out the warnings in the tests, similar to https://github.com/scipy/scipy/issues/5877

Just a doubt, when I run python runtests.py none of these DeprecationWarning showed up. Also, when the new vectorized special functions were added, they seem to be filtered in the tests in special.


Local coordinate system constraints

Hello,
I just have a question about creating constraints.
I would like to create constraints in other than global coordinate system. Is that possible?
Example: I have a 3D beam which is not parallel to any axis of the global coordinate system. I would like to define X and Y direction relaxed and Z direction fixed in the direction of the beam. How would I define this in the code?
Thank you
Best regards
Juraj

Thank you Juraj @Georgerdx for this great question!
Yes it is possible!
Have a look at the following example:
This is a small modification of Example 1 for C# (Examples)
// Define geometry
var p1 = new Geometry.Point3d( 2,  2, 0);
var p2 = new Geometry.Point3d(10, 10, 2);
var edge = new Geometry.LineEdge(p1, p2);

// Define support location and orientation to follow the beam geometry
var supportOrientation1 = new Geometry.CoordinateSystem(p1, edge.CoordinateSystem.LocalX, edge.CoordinateSystem.LocalY);
var supportOrientation2 = new Geometry.CoordinateSystem(p2, edge.CoordinateSystem.LocalX, edge.CoordinateSystem.LocalY);

// Load sections and materials from struxml files
Sections.SectionDatabase sectionsDB = Sections.SectionDatabase.DeserializeStruxml("sections.struxml");
Materials.MaterialDatabase materialsDB = Materials.MaterialDatabase.DeserializeStruxml("materials.struxml");

var section = sectionsDB.SectionByName("Concrete sections, Rectangle, 300x900");
var material = materialsDB.MaterialByName("C35/45");

// Create a beam
var beam = new Bars.Beam(
    edge,
    material,
    section);

// Create supports
var s1 = Supports.PointSupport.Rigid(supportOrientation1);
var s2 = Supports.PointSupport.Hinged(supportOrientation2);
//var s2 = new Supports.PointSupport(supportOrientation2, false, true, true, false, false, false); // Custom support

// Create the FEM-Design model and show it
var elements = new List<GenericClasses.IStructureElement>() { beam, s1, s2 };
Model model = new Model(Country.S, elements);

model.Open();

Let us know if you have any more questions on support or if you want to use the API from Grasshopper etc.
Kind Regards,
Alex

Let us know if you have any more questions on support or if you want to use the API from Grasshopper etc.
Best regards,
Alex

Hello Alexander,
Thank you for an excellent answer.
Is there any way to get the vectors of the local coordinate system as well?
Thank you
Best regards
Juraj

On 31. Oct 2022, at 09:49, Alexander Radne @.***> wrote:

Let us know if you have any more questions on support or if you want to use the API from Grasshopper etc.
Best regards,
Alex
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

I don't think I understand what you want exactly. But if you want to use the local coordinate system of a beam that already exists in your model, then you could do that like this:
// Load a model saved from FEM-Design
Model model = Model.DeserializeFromFilePath("C:/MyModel.struxml");

// Get the local axis of the beams
var beam = model.Entities.Bars.Find(b => b.Name == "B.1");
var localX = beam.BarPart.LocalX;
var localY = beam.BarPart.LocalY;

I hope this helps

Hello Alexander,
I really appreciate your fast replies. Thank you!
Maybe I can rephrase my question.
I am a bit struggling to understand how is the local coordinate system oriented.
Let’s say I have a 3D beam with a local coordinate system oriented in a different way compared to the global system, how do i know where the local axis are. Where is local X, Y, Z axis of the beam and how are these oriented? Is there any rule for that, or is it possible to get a vector for each axis which would define the direction.
Many thanx for your answer!
Best regards
Juraj

On 31. Oct 2022, at 16:03, Alexander Radne @.***> wrote:

I don't think I understand what you want exactly. But if you want to use the local coordinate system of a beam that already exists in your model, then you could do that like this:
// Load a model saved from FEM-Design
Model model = Model.DeserializeFromFilePath("C:/MyModel.struxml");
// Get the local axis of the beams
var beam = model.Entities.Bars.Find(b => b.Name == "B.1");
var localX = beam.BarPart.LocalX;
var localY = beam.BarPart.LocalY;
I hope this helps
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Hi Juraj,

Let’s say I have a 3D beam with a local coordinate system oriented in a different way compared to the global system, how do i know where the local axis are. Where is local X, Y, Z axis of the beam and how are these oriented?

The localX in the example above is the local X axis of the beam :)
I have updated this example to also print the coordinates of localX and localY to the Console output.
// Print to console the local axes
Console.WriteLine($"Local X: ({localX.X}, {localX.Y}, {localX.Z})");
Console.WriteLine($"Local Y: ({localY.X}, {localY.Y}, {localY.Z})");

Is there any rule for that, or is it possible to get a vector for each axis which would define the direction.

The beams will always have local X parallel with the direction of the beam.
Generally, when drawing a beam in FEM-Design or creating one using this API, local Y is horizontal
Then the local Z is the vector perpendicular to local X and Y. (The cross product of local X and Y)

Best Regards,
Alex

Hello Alex
Thank you! I think it is clear to me.
Best regards
Juraj

On 1. Nov 2022, at 08:47, Alexander Radne @.***> wrote:

Hi Juraj,
Let’s say I have a 3D beam with a local coordinate system oriented in a different way compared to the global system, how do i know where the local axis are. Where is local X, Y, Z axis of the beam and how are these oriented?
The localX in the example above is the local X axis of the beam :)
I have updated this example to also print the coordinates of localX and localY to the Console output.
// Print to console the local axes
Console.WriteLine($"Local X: ({localX.X}, {localX.Y}, {localX.Z})");
Console.WriteLine($"Local Y: ({localY.X}, {localY.Y}, {localY.Z})");
Is there any rule for that, or is it possible to get a vector for each axis which would define the direction.
The beams will always have local X parallel with the direction of the beam.
Generally, when drawing a beam in FEM-Design or creating one using this API, local Y is horizontal
Then the local Z is the vector perpendicular to local X and Y. (The cross product of local X and Y)
Best Regards,
Alex
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.


Hiding the underline for entry in combobox

How to remove the underline effect of the entry in combobox? I have created 1 effect fro hiding the underline effect but it doesn't work on the comboBox. It works fine on normal entry. Please suggest how to remove underline effect in comboBox.

+1


Consider supporting VirtualPathProvider and associated infrastructure

Summary
Consider supporting VirtualPathProvider and associated classes: HostingEnvironment, VirtualFileBase, VirtualFile, and VirtualDirectory.
In ASP.NET Core, support VirtualPathProvider as an IFileProvider to allow StaticFiles to use VirtualPathProviders.
Wrap the WebRootFileProvider as a VirtualPathProvider as the root provider to replace the MapPath based provider.
Consider providing a package to allow using any IFileProvider as a VirtualPathProvider under System.Web to allow using EmbeddedFileProvider in legacy classes during cutover.
Motivation and goals
This would allow migrating existing scenarios with embedded resources or other virtual files to ASP.NET Core using the adapters.
This would be a stepping-stone to allowing creating a compatible version of Microsoft.AspNet.Web.Optimization / System.Web.Optimization which would unlock scenarios for people using this technology.
In scope
Various classes from System.Web.Hosting including

System.Web.Hosting.HostingEnviroment (at least the VirtualPathProvider and RegisterVirtualPathProvider members, although Cache, ApplicationPhysicalPath, and ApplicationVirtualPath map directly to equivalents in HttpRuntime that are already implemented.)
VirtualFileBase
VirtualFile
VirtualDirectory
VirtualPathProvider

Extension methods for ASP.NET Core to allow registering VirtualPathProviders via DI instead of legacy methods.
A VirtualPathProvider to wrap any IFileProvider (This could actually be used in legacy Framework projects too, but would require an additional reference to Microsoft.Extensions.FileProviders.Abstractions in whichever project it would be in.)
An FileProvider to wrap the VirtualPathProvider chain to allow the StaticFiles middleware to server files from the VirtualPathProviders.
Out of scope
It would be nice to include the IFileProvider VirtualPathProvider under FrameworkServices or in the main adapters, as this would allow migration from custom VPPs to the existing FileProviders, such as Microsoft.Extensions.FileProviders.Embedded but as this would result in additional dependencies, this can be out of scope.
Risks / unknowns
Moving the existing WebRootFileProvider behind a VirtualPathProvider and then wrapping that as a FileProvider would potentially cause a performance issue with StaticFiles, however this would apply to the original ASP.NET Framework application too, and it is likely that any VPP is already pretty optimized and uses appropriate caching mechanisms to ensure its optimal speed. This is of course an interim step, and as a developer would expect to migrate any custom VPPs to IFileProviders or to use an existing out-of-box provider. (This is one reason to include the option to wrap IFileProviders as VirtualPathProviders).
Examples
Example code for VirtualPathProviders can be found at VirtualPathProvider, VirtualFile, and VirtualDirectory
Global.asax.cs code such as
            SamplePathProvider sampleProvider = new SamplePathProvider();
            System.Web.Hosting.HostingEnvironment.RegisterVirtualPathProvider(sampleProvider);

would become
builder.Services.AddSystemWebAdapters()
    .AddJsonSessionSerializer(options => ClassLibrary.RemoteServiceUtils.RegisterSessionKeys(options.KnownKeys))
    .AddVirtualPathProvider<ClassLibrary.SamplePathProvider>()
    .AddVirtualPathProvidersAsStaticFileProvider(options => options.ServeUnknownFileTypes = true)

If IHttpHandler and IHttpModule were implemented along with this, it would be possible to migrate the Microsoft.AspNet.Web.Optimization package that is regularly used in ASP.NET Framework applications without completely replacing the bundling system, making it easier to migrate MVC/Razor applications that use the package.
Detailed design
See https://github.com/CZEMacLeod/systemweb-adapters/tree/czemacleod/virtualpathprovider for a rough implementation - this includes enhancements to Caching as required by the example that would probably deal with #26 too.

@CZEMacLeod is there anything left from this after the additions you've made?

@twsouthwick I don't think any of the VPP code has been added. While VirtualPathProviders do tend to use some of the code VirtualPathUtility etc., they are not the same thing.
I've rebased and pushed my new branch as https://github.com/CZEMacLeod/systemweb-adapters/tree/czemacleod/vpp-1.1
Still cleaning up a few things here for things which were touched that shouldn't have been.

@twsouthwick Actually found and fixed an issue with Cache/CacheDependency while doing this. I've also added a couple of tests to catch the problem.
I could spilt it out into a separate issue/PR if need be.
I think it must have been a copy/paste issue when I was creating my original branch for Cache in #276 for #26.

@CZEMacLeod feel free to make a PR for this and we can discuss it there


Sidepanel style

Coverage increased (+1.1%) to 60.87% when pulling f4cd6ab0f573a979f0b2f2f0e80d9a159450d813 on feature/sidepanel into b72aab78bfe5ef86d9639503db06c0c72bc9160c on master.


Remove log4j-codegen

The log4j-codegen module was moved from Log4j Core 2 and generates code for custom loggers.
Since:

We don't recommend the usage of custom logger wrappers.
The functionality of a custom logger wrapper can be easily replaced by the usage of the appropriate Level parameter.

I propose to remove the log4j-codegen module.

(AFAIU, this has never been released with log4j-transform – please correct me if I'm wrong.)

No, it has never been released, so the appropriate time to remove it is now.
If in the future we decide to release it, we can reverse this PR.