GitHub icon causes Poxy to incorrectly overlaps with long project description

The GitHub icon causes Poxy to incorrectly overlaps with long project description, as shown below:

O_O
I... I don't even know where to start with this, hah. The github icon shouldn't have any impact on the page flow at all, it's just the background-image CSS attribute.
How did you generate the second image, without the issue?

O_O
I... I don't even know where to start with this, hah. The github icon shouldn't have any impact on the page flow at all, it's just the background-image CSS attribute.
How did you generate the second image, without the issue?

Disabling margin-top in h1, h2, h3, h4, h5, h6 corrects the problem to some extent (not perfect) :
The second image was generated by refreshing before completion, just before the logo is available to be shown
Note: the issue can't be reproduced with m.css
h1, h2, h3, h4, h5, h6 {
  /* margin-top: 0; */
  font-weight: 600;
}

m.css (left):

Note: the issue can't be reproduced with m.css

Yeah; I modified the page header to stay fixed in place at the top of the window, whereas m.css leaves it just as a regular, non-floating page element. For this to work I had to increase the top margin of the page content by the regular height of the navbar so that when you're viewing the top of the window it looks 'vanilla'. I suspect the root of the issue here is actually that you're seeing a two-row header, and my CSS adjustments don't take that possibility into account.

I won't have time to engineer a decent fix for this until at least this weekend, but you can add an override to some custom CSS of your own to restore the m.css behaviour:
body
{
	padding-top: initial !important;
}

body > header
{
	position: initial !important;
}

using the changes above:

I won't have time to engineer a decent fix for this until at least this weekend, but you can add an override to some custom CSS of your own to restore the m.css behaviour:
body
{
	padding-top: initial !important;
}

body > header
{
	position: initial !important;
}

It's ok, there's no rush, thanks for the alternative 👍


[MBOX] dlen restricts size of data response that may be read

In the Mailbox protocol, the mbox_dlen register is used to restrict whether rdptr is allowed to increment or not, and thus controls how much data can be read from mbox_dataout.
Response data may realistically be larger than original request data, so restricting readout by mbox_dlen will hide some of the response.
The suggested fix is two pronged:

A receiver of a mailbox command is allowed to write a new value to the mbox_dlen register. The new value corresponds with response data size. Only uC will ever post response data to a mailbox command, so in practice this means that mbox_dlen writes are enabled when the FSM is in EXECUTE_UC state.
To mitigate potential security holes, implement a flop stage in the mailbox that captures mbox_dlen on state transitions. The mailbox command recipient will only be able to read from mbox_dataout in the amount reflected in the flop-stage, but the command recipient will be able to write a new mbox_dlen value (which will be used to restrict reads from mbox_dataout of the response data by the original requester, i.e. SOC).

FWIW, in my testing with the verilator model, it seems like the uC is able to set dlen as part of the response.

Closing issue. Fixed with rev 0p8 release #49


Add javacc and javaparser

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (javacc, javaparser) have been updated in this PR. Please review the changes.

It seems that JavaParser is incompatible with JDK versions 21 and later: https://github.com/javaparser/javaparser/issues/4022
I've updated presubmit.yml for JavaParser to set --java_runtime_version=remotejdk_17.

Bumped rules_java to 6.0.0 to work around a bug in the Turbine version used by default in Bazel 6: https://github.com/bazelbuild/bazel/issues/18743
CI is now (mercifully) green.


Let daedalus work without a local copy of the blockchain

I was wondering if its possible to simple look at my relay node instead of holding a local copy. If thats not currently possible this might be something for the feature request list.

Daedalus always runs it's own cardano-node instance which it uses to access of the blockchain.
Relay node is something else... Can you please further clarify your use-case? Why do you think it would be better/safer to use a relay?

@ronaldvdmeer I would really appreciate your input here 🙏

@ronaldvdmeer I would really appreciate your input here 🙏

Well I have a fast node/relay in my homenetwork  already so startup times will improve because it doesn’t have to sync with a slow server. Sometimes it takes a long time (several minutes) to startup the client and I’m thinking it has something to do with the local node selecting a slow peer to sync with.
Also it would be nice if it doesn’t have to sync anything. It just works with the information provided by the node in my network. That way no local storage is required beside the app itself. When the blockchain grows this might become an advantage.
Hope this helps

Thanks for the clarification @ronaldvdmeer!
Would you agree with me that the feature request you are longing for is an option to configure the source of blockchain data?
E.g. it could also be a centralized service provider or your own remote instance?

I think there are two features.

being able to configure which node it will sync with
being able to not sync at all and work completely from another node.

Got it! Thanks!

I believe this issue is connected/related to https://github.com/input-output-hk/daedalus/issues/2413

Partially because it still needs to sync the whole blockchain. I tried this with my Mac by the way but somehow it didn’t work. Not sure why the topology file couldn’t be changed or found. Not sure anymore what the issue was

There is no easy nor nice way to change the topology at the moment...

Check! So one part of this feature request indeed is the same as the other. The other part of this feature request “being able to work of a relay without a local copy” is yet another feature

Agreed! This is why I have noted that the two are connected/related and not duplicates :-)

Perhaps I should change the title of this issue so it more reflects the difference

Yes please 🙏

Is there any timeframe for this?  The blockchain is getting pretty big to run full nodes locally and I have a perfectly and permanently sync'd cardano-wallet docker container running on my server exposing port 8090 over http.  Would it be hard to allow daedalus to switch cardano-wallet/ cardano-node back ends instead of running these locally?  I'd really <3 this feature!


Explicitly document 'context' call pattern

Work for this issue will begin begin on branch work-liquid-labs/string-input/13.


 Format is `!evil <demon> <minion> <mi...
  help     Shows this message
  lunatic  Send fake evil info to the Lunatic. Format is `!lunatic <Lunatic> ...
  night    Move users to Cottages in the BotC - Nighttime category
  vote     Move users from other channels back to Town Square

Also "Shows" vs "Send" - verb, uh, active-ness(?) is inconsistent

Also "Shows" vs "Send" - verb, uh, active-ness(?) is inconsistent

You can do “!help evil” to see the full text for a command. And this is a built-in feature of discord.py bots at any rate, not something I control.

You can do “!help evil” to see the full text for a command. And this is a built-in feature of discord.py bots at any rate, not something I control.


WebAssembly

Starting to look interesting http://v8project.blogspot.se/2016/10/webassembly-browser-preview.html
Any thoughts on how it may affect deepstream?

I don't think this will be introduced for a while. Deepsream.io uses native modules and the client barely uses any ES6 features other than let/const block scopes. Closing this issue for now


Approximate share in Portfolio Model

Linear approximation of risky share given next period's cFunc.

Blue line is HARK's numeric solver. Orange line is approximation.

large value a_max
second order approximation (1+\epsilon)^\nu in Chris's notes (MathFacts TaylorTwo)
Compare second order to true exp w/ large value of a_max, are they close?
implement backsolving using only approximations

Did a little bit of work on this. It seems the second order approximation with TaylorTwo rule is not particularly better than the first order approximation. Where first order approx is below true function, second order approx is above the function.

@llorracc this is with aMax = 1000 (instead of 100).

@alanlujan91
Do you know why the checks are failing?

Yes, the tests for Discrete Solver and Joint Distribution solver are failing, but this is not supposed to be a feature of those models. I have to reorganize where the approximations happen.

To debug, I suggest you evaluate the last expression in (15) in the appendix to
https://github.com/ccarrollATjhuecon/EquityPremiumPuzzle/blob/master/EquityPremiumPuzzle.pdf
at $a=1000$ and the first expression in equation (16) at the same point.
If the first expression in (16) is not close to zero

you can break both 15 and 16 down using the $E[xy] = E[x]E[y]+cov(x,y)$ formula and see whether it is the $E[x]E[y]$ term or the $cov(x,y)$ term or both that are bad.
You could also evaluate 15 using a linear consumption function identical to the one used in (16);
if the discrepancy remains largely the same, that would say that the problem is not coming from the nonlinearity of the consumption function but from the approximation of the shock.

If the first expression in (16) IS close to zero

That would seem to indicate that the problem is the approximation between the penultimate and the last line in (16).  You could see whether a 2nd (or even a 3rd) order approximation at that point fixes things.

PS.  Whether or not you manage to track down the problem as above, it would be useful if you could edit the GitHub document to incorporate any higher order approximations that you are using so we can see exactly what you are doing.

Codecov Report

Merging #1122 (91fb9ae) into master (e701558) will decrease coverage by 0.23%.
The diff coverage is 24.00%.

@@            Coverage Diff             @@
##           master    #1122      +/-   ##
==========================================
- Coverage   73.96%   73.72%   -0.24%
==========================================
  Files          70       70
  Lines       10761    10809      +48
==========================================
+ Hits         7959     7969      +10
- Misses       2802     2840      +38

Impacted Files
Coverage Δ

HARK/ConsumptionSaving/ConsPortfolioModel.py
82.59% <24.00%> (-6.33%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e701558...91fb9ae. Read the comment docs.


Notes

Function app needs to be explicity set to v1, otherwise a compilation error occurs.
Add sentiment detection step needs to have URL set.

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

ID: ae1a4055-33b7-eead-6eb4-0092b7112567
Version Independent ID: 7b866c4b-46d2-d420-278e-f49ec05e5d0d
Content: Create a function that integrates with Azure Logic Apps
Content Source: articles/azure-functions/functions-twitter-email.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Hi @mcurko Thank you for reporting this issue! We will investigate and make any necessary changes.

@mcurko to be honest I'm surprised at this one having problems in v2, but I will take a look.

This worked ok for me with version 2 runtime (Runtime version: 2.0.12134.0 (~2))
I did need to make the change "Add sentiment detection step needs to have URL set."
Apart from that this is a great walkthrough.

@mike-urnun-msft please assign to @craigshoemaker

I submitted a PR for a refresh of this article today. All steps are now verified. Thanks for your input, @mcurko!
#please-close


Ajax call is causing cursor to jump to the right upon keyup

I have an ajax call that runs upon keyup. Each time the user presses a key, the cursor jumps to the end. For example, if the field is $253,009 and the user attempts to remove the '3', the resulting value will be $25,009 and the cursor will jump to the right of the '9'. When I disable the ajax call, the cursor does not jump. Alternatively, if I disable the mask while keeping the Ajax call, the cursor does not jump either.
These are the settings I am using:
$('#id_property_value').inputmask("currency", {
"rightAlign": false,
"placeholder": "",
"digits": 0,
"integerDigits": 7,
"allowMinus": false,
"autoUnmask": true
});
Any idea what could be causing this?
Thanks!

@shahvaiz ,
What do you do in the ajax-call?  Can you provide some more code.

I was able to isolate the issue and create a JSFiddle. The issue has to do with 'keyup'. Each time the keyup method is called on the input field, the 'inputmask' method is called and the cursor resets to the end. If I change it to act upon 'blur', the cursor behaves correctly.
https://jsfiddle.net/shanif/oeytxoy8/

Why do you call the inputmask function on every keyup?
Op di 19 mei 2015 om 04:55 schreef Shahvaiz Hanif<EMAIL_ADDRESS>
:

I was able to isolate the issue and create a JSFiddle. The issue has to do
with 'keyup'. Each time the keyup method is called on the input field, the
'inputmask' method is called and the cursor resets to the end. If I change
it to act upon 'blur', the cursor behaves correctly.
https://jsfiddle.net/shanif/oeytxoy8/
—
Reply to this email directly or view it on GitHub
https://github.com/RobinHerbots/jquery.inputmask/issues/913#issuecomment-103311649
.

Ah, you're right. I moved the inputmask function outside of keyup, and it works perfectly now. Thanks!


Shrinke.me not bypassing

Domain
https://shrinke.me
Link
https://shrinke.me/AP7l8B
Version
0.0.0
What browsers are you seeing the problem on?
Chrome
What OS are you seeing the problem on?
Windows
Anything else?
No response

We do not currently bypass this site based on this list, please read https://github.com/FastForwardTeam/FastForward/issues/656 for more details.
Once we have finished with the project, you are free to reopen the issue.


app-misc/zigbee2mqtt overwrites configuration file on update

app-misc/zigbee2mqtt stores its data files in /var/lib/zigbee2mqtt but by default this is not protected by CONFIG_PROTECT and the users' configuration.yaml will get overwritten with the template file on every update.
According to https://devmanual.gentoo.org/general-concepts/config-protect/index.html this can be overridden by ebuilds.

Thank you very much, I was already using CONFIG PROTECT but there was a problem with the substitution of the PN variable

Hmm, the update to 1.31.0 today clobbered my configuration.yaml again.

I forgot to tell you that before updating you will check that the file /etc/env.d/90zigbee2mqtt contains CONFIG_PROTECT="/var/lib/zigbee2mqtt"
If you look now you will have the variable correctly
If not, even if you have corrected the problem, Gentoo will use the old file
Check it and tell me

Yep it's there now.


telemetry_status advertise failed

Recently in jMAVSim SITL I often see this on startup:
INFO  [mavlink] mode: Normal, data rate: 4000000 B/s on udp port 14570 remote port 14550
INFO  [mavlink] mode: Onboard, data rate: 4000000 B/s on udp port 14580 remote port 14540
INFO  [logger] logger started (mode=all)
INFO  [logger] Start file log (type: full)
INFO  [logger] Opened full log file: ./log/2019-06-14/07_53_57.ulg
INFO  [mavlink] MAVLink only on localhost (set param MAV_BROADCAST = 1 to enable network)
INFO  [px4] Startup script returned successfully
pxh> INFO  [simulator] Simulator connected on TCP port 4560.
Init MAVLink
INFO  [ecl/EKF] EKF aligned, (pressure height, IMU buf: 22, OBS buf: 14)
ERROR [uorb] telemetry_status advertise failed

Presumably this has to do with uORB publication/subscription changes. FYI @dagar @mcsauder

I'm not seeing this locally. What are you running exactly? Is everything completely default?

If you see this again can you quickly check a few things?
mavlink status
listener telemetry_status
uorb top -1

I'm not seeing it locally either:

Ok, it only seems to happen with PX4_SIM_SPEED_FACTOR=20 make px4_sitl jmavsim.
pxh> mavlink status

instance #0:
	mavlink chan: #0
	type:		GENERIC LINK OR RADIO
	flow control: OFF
	rates:
	  tx: 9.399 kB/s
	  txerr: 0.000 kB/s
	  tx rate mult: 1.000
	  tx rate max: 4000000 B/s
	  rx: 0.000 kB/s
	FTP enabled: YES, TX enabled: YES
	mode: Normal
	MAVLink version: 1
	transport protocol: UDP (14570, remote port: 14550)

instance #1:
	mavlink chan: #1
	type:		GENERIC LINK OR RADIO
	flow control: OFF
	rates:
	  tx: 16.743 kB/s
	  txerr: 0.000 kB/s
	  tx rate mult: 1.000
	  tx rate max: 4000000 B/s
	  rx: 0.000 kB/s
	FTP enabled: YES, TX enabled: YES
	mode: Onboard
	MAVLink version: 1
	transport protocol: UDP (14580, remote port: 14540)

pxh>  listener telemetry_status

TOPIC: telemetry_status instance 0 #1
 telemetry_status_s
	timestamp: 966480000  (1.588000 seconds ago)
	heartbeat_time: 0
	data_rate: 4000000.0000
	rate_multiplier: 1.0000
	rate_rx: 0.0000
	rate_tx: 16.7450
	rate_txerr: 0.0000
	remote_system_id: 0
	remote_component_id: 0
	remote_type: 0
	remote_system_status: 0
	type: 0
	mode: 2
	flow_control: False
	forwarding: False
	mavlink_v2: False
	ftp: True
	streams: 43

pxh>  uorb top -1

update: 1s, num topics: 53
TOPIC NAME                        INST #SUB #MSG #LOST #QSIZE
geofence_result                      0    1    4     0 1
vehicle_global_position              0    5  115   398 1
ekf_gps_drift                        0    0   10     0 1
telemetry_status                     0    2    2     0 3
vehicle_rates_setpoint               0    3  194   156 1
vehicle_attitude                     0    7  231  1029 1
ekf2_innovations                     0    1  115   107 1
sensor_bias                          0    4  115   291 1
vehicle_odometry                     0    2  115   207 1
battery_status                       0    6   83   273 1
ekf2_timestamps                      0    1  231    51 1
multirotor_motor_limits              0    2  250    53 1
rate_ctrl_status                     0    1  250   217 1
vehicle_magnetometer                 0    4  232   547 1
vehicle_air_data                     0    4  232   510 1
airspeed                             0    4  231   500 1
vehicle_land_detected                0    8    1     0 1
vehicle_local_position_groundtruth   0    1  125   110 1
vehicle_global_position_groundtruth  0    1  125   110 1
vehicle_attitude_groundtruth         0    1  125     0 1
differential_pressure                0    3  250   551 1
actuator_controls_0                  0    5  250   676 1
sensor_combined                      0    4  233   550 1
actuator_outputs                     0    4  250   676 1
vehicle_local_position               0    7  115   490 1
vehicle_status_flags                 0    1    1     0 1
commander_state                      0    0    1     0 1
actuator_armed                       0    6    1     0 1
vehicle_status                       0    9    1     0 1
vehicle_control_mode                 0    8    1     0 1
estimator_status                     0    4  115   515 1
sensor_preflight                     0    1  233   226 1
vehicle_gps_position                 0    5   10     9 1
sensor_baro                          0    1  250    18 1
sensor_mag                           0    2  250   218 1
sensor_gyro                          0    3  250   217 1
sensor_accel                         0    2  250   220 1

I saw this randomly yesterday when doing some multi-uav sims with ROS. I didn't see this topic before-hand. So, I don't have more than confirming that I saw "ERROR [uorb] telemetry_status advertise failed." I troubleshot a few things during that time, so hard to say more considering the environment changes I made during that time. I didn't touch speed factor.

I have seen it at times without speed up as well.

Doesn't seem to be happening anymore.

This is happening again, when using Gazebo.

This has been fixed.


zts java client - Update jackson-databind to get rid of CVE-2020-36518

At the moment if you a user has a dependency to
<dependency>
      <groupId>com.yahoo.athenz</groupId>
      <artifactId>athenz-zts-java-client</artifactId>
</dependency>

and it runs the OWASP dependency check it fails because jackson-databind 2.13.1 has a critical vulnerability https://nvd.nist.gov/vuln/detail/CVE-2020-36518
Since it is a fat-jar it would be better to release a CVE-free jar
Note: the safe version is <IP_ADDRESS>

Hi @nicoloboschi
I was hoping for a complete 2.13.3 release to keep the versions consistent but only databind was updated. Anyway, we'll probably have a release later on today with the updated version.
However, I strongly discourage the use of the  uber jar in your application. Unless it's using jersey 1.x instead of jersey 2.x which is used by athenz libs (the reason why the fat jar was created), I'd recommend updating your service to use the core zts client:
<dependency>       <groupId>com.yahoo.athenz</groupId>       <artifactId>athenz-zts-java-client-core</artifactId> </dependency>

Thanks @havetisyan. We moved to the regular jar


missing files after uglifying JS IIFE

Hi,
I use gulp-task in my build-script:
gulp.task('js:build', function () {
    return gulp.src(path.src.js)
        .pipe(rigger())
        .pipe(sourcemaps.init())
        .pipe(cq_jshint(jshintConfig))
        .pipe(cq_jshint.reporter('jshint-stylish'))
        .pipe(uglify())
        .pipe(sourcemaps.write())
        .pipe(gulp.dest(path.build.js))
        .pipe(reload({
            stream: true
        }));
});

js/app.js:
//= partials/controllers/LoginController.js
//= partials/app.js

both files have IIFE code:
(function(){
    'use strict';

    *** code ***
}());

after run task I have in build/app.js only app.js-source from partials-folder, but can't find LoginController.js-source!
what did I do wrong?

Comment out the uglify() line. Do you have both files in the output?

after I commented uglify() I can see both files. I think, something wrong with this part on both files
(function(){
    'use strict';
}());

Maybe there's an issue with how rigger handles the different line endings on Windows?
I suggest contacting that project. Fill free to comment here if that wasn't the issue.


Add PullRequest.MergeableState field

This isn't a well documented field but it's present in API responses and mentioned here
https://github.com/octokit/octokit.rb/issues/626

Thank you, @ains. I have a couple questions for you.
According to the documentation here: https://developer.github.com/v3/pulls/#get-a-single-pull-request
the mergeable field ought to be sufficient, and mergeable_state is nowhere to be found (in the docs) but you say you have seen it in responses.
Have you tried using the mergeable field to see if it satisfies your needs?
If you have tried it, but it doesn't work, then I think we need to contact<EMAIL_ADDRESS>and ask for clarification regarding these two fields (one documented, one not).
If this is the case, can you please provide a JSON response showing both the mergeable and mergeable_state fields and show how mergeable is not sufficient? This will help the GitHub support team figure out what is going on.
Thanks again!

No problem @gmlewis
You can find a bit more context on the field mergeable_state in this issue https://github.com/octokit/octokit.rb/issues/626,
and in this thread on the github platform forum (where they are planning to document it)
https://platform.github.community/t/documentation-about-mergeable-state/4259
Unfortunately mergeable doesn't satisfy the needs we have, if a PR can cleanly merge into a branch but fails a check (set when protecting that branch) mergeable is still true however mergeable_state will be blocked, indicating it actually shouldn't be merged.

Ah, thank you for the forum link, @ains! That is sufficient for me.
LGTM.


Error: could find module after "yo  keystone"

Lots of modules are not installed even I did "npm install".
They are not listed in the package.json
Such as cookie-jar, qs, oauth-sign,  hawk, aws-sign, http-signature and so on.

I installed it on Win7,  in admin console.

I add the lost modules manually to the package.json and it works.
"cookie-jar": "~0.3.0",
"qs": "~2.3.3",
"oauth-sign": "~0.6.0",
"hawk": "~2.3.1",
"aws-sign": "~0.1.1",
"http-signature": "~0.10.1",
"tunnel-agent": "~0.4.0",
"json-stringify-safe": "~5.0.0",
"forever-agent": "~0.5.2",
"form-data": "~0.2.0",
"debug": "~2.1.1",
"mime": "~1.2.11",
"once": "~1.3.1",
"xml2js": "~0.4.4",
"stream-counter": "~1.0.0"

:+1:

is a pull request needed that adds the missing modules to the package.json file? @Yinear if so could you handle that?

@Yinear can you provide some more details?  The packages you listed are not dependencies of generator-keystone or<EMAIL_ADDRESS>I ran yo keystone on  a fresh install of yo and generator-keystone on win7 with success, so I was not able to reproduce your errors there or after running node keystone.
What is failing for you?  Is it yo keystone or runniing node keystone after yoeman completes?

@snowkeeper @morenoh149
It was failed on "node keystone"
I tried for another three times and this problem didn't happen.
I guess it's because I did yo keystone in a folder which was for another project.  There must be some hidden files I didn't delete in the folder.
I'm OK now.  Nothing need to be change.  I close this issue.

Ran into this issue as well, where the "yo keystone" seemed to fail before it finished, and the project it created was missing a whole bunch of node modules.
It was on a freshly created directory, and I had just installed latest versions of Node and MongoDB.  Using OSX.  Going to try and run the generator again and see if it fixes it as it did for Yinear.


Sporadic Test Failures on TravisCI (with OpenJDK?)

There seem to be recurring test failures on TravisCI, akin to:
https://travis-ci.org/xsc/iapetos/jobs/150695791
I've only seen them for OpenJDK builds and mostly before/during iapetos.standalone-test, so it might be connected to the HTTP server implementation in combination with OpenJDK.

These might be instances of travis-ci/travis-ci#4691.


[SPARK-22880][SQL] Add cascadeTruncate option to JDBC datasource

This commit adds the cascadeTruncate option to the JDBC datasource
API, for databases that support this functionality (PostgreSQL and
Oracle at the moment). This allows for applying a cascading truncate
that affects tables that have foreign key constraints on the table
being truncated.
What changes were proposed in this pull request?
Add cascadeTruncate option to JDBC datasource API. Allow this to affect the
TRUNCATE query for databases that support this option.
How was this patch tested?
Existing tests for truncateQuery were updated. Also, an additional test was added
to ensure that the correct syntax was applied, and that enabling the config for databases
that do not support this option does not result in invalid queries.

@dongjoon-hyun this is the functionality we discussed in PR for SPARK-22729, would be great to hear your opinion on this :)

Can one of the admins verify this patch?

I think the default for all dialects should be false, regardless of whether the cascade feature is even supported. And for those for which it is supported, it should default to false.

Thank you for your contribution! I am doubting the value of this feature.
If you are interested in the JDBC-related work, could you take https://issues.apache.org/jira/browse/SPARK-22731?
Thanks!

@gatorsmile could you explain why you have doubts about the feature? Thanks!

@gatorsmile would be great to hear why you doubt the value of the feature :). I know that for us it would be extremely valuable (at the moment we have to do an extra step in our data pipeline because this feature is missing in Spark), but of course we're not the only ones using Spark.

@Stephan202 thanks for pointing out those docs issues, just pushed the changes :-).
@gatorsmile @dongjoon-hyun would you have a chance to take a look at this again?

Any idea when this will be merged into master? We could use this since we are ditching sqoop 👍

@dongjoon-hyun @gatorsmile sorry to keep asking, but could you let me know when we can get this merged?

We're in the process of integrating Spark in Airflow, and support for the cascadeTruncate is required to make this succeed. First steps are here: https://github.com/apache/incubator-airflow/pull/3021.
Would be great if we can get this merged asap so we can continue testing. Cheers

ok to test

Test build #87484 has started for PR 20057 at commit 3a7dda4.

@danielvdende @Fokko We definitely want to help the community replace Sqoop by Spark SQL. However, truncate is only used when users use SaveMode.Overwrite to write the external JDBC tables. In this specific scenario, Spark will truncate an existing table instead of dropping and recreating it.
Could you show me the key missing features that are available in Sqoop but not in Spark SQL JDBC connectors?

Test build #87484 has finished for PR 20057 at commit 3a7dda4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87484/
Test FAILed.

Tests are failing on a spark streaming test. I think it's probably because of the age of this PR, will rebase to get the changes into the PR that were merged into master since I opened the PR

Test build #87493 has started for PR 20057 at commit 6c0d3df.

Test build #87493 has finished for PR 20057 at commit 6c0d3df.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87493/
Test FAILed.

Hi @gatorsmile, thanks for putting it to the tests. The main reason why I personally dislike Sqoop is:

Legacy. The old map-reduce should be buried in the upcoming years. As a data engineering consultant, I see more people questioning the whole Hadoop stack. Using Sqoop you still need to run map-reduce tasks, and this isn't easy on other platforms like kubernetes.
Stability. I see Sqoop jobs fail quite often, and there isn't a nice way of retrying this in an atomic way. For example, when having a Sqoop job on Airflow, we cannot simply retry the operation. We when we import data from a rmdbs to hdfs, we have to make sure that the target directory of the previous run has been deleted.

This is also where Spark-jdbc comes in, for example, in the future we would like to delete single partitions, but this is wip. Maybe @danielvdende can elaborate a bit on their use-case.

Hmm, not it fails the OrcQuerySuite. This PR doesn't touch any of the Orc implementation in Spark. Could this be a flaky test @gatorsmile ?
org.scalatest.exceptions.TestFailedDueToTimeoutException: The code passed to eventually never returned normally. Attempted 12 times over 10.158754687999998 seconds. Last failure message: There are 1 possibly leaked file streams..

Hi guys, @Fokko @gatorsmile, completely agree with what @Fokko mentioned, our main reason for wanting to get away from Sqoop is also for stability reasons and to get rid of MapReduce in preparation for our move to Kubernetes (or something similar). We've also seen it to be much faster than Sqoop. In terms of why we need the feature in this PR: we have some tables in PostgreSQL that have foreign keys linking them. We have also specified a schema for these tables. If we use the drop-and-recreate option, Spark will determine the schema, overriding our PostgreSQL schema. Obviously, these should match up, but I personally don't like that Spark can do this (and that you can't explicitly tell it not to).
Because of this behaviour, we currently require 2 tasks in Airflow (as @Fokko mentioned) to ensure the tables are truncated, but the schema stays in place. This PR would enable us to specify in a single, idempotent (Airflow) task that we want to truncate the table before putting new data in there. The cascade enables us to not break foreign key relations and cause errors.
To be clear, this therefore isn't emulating a Sqoop feature (as a Sqoop task isn't idempotent), but is in fact improving on what Sqoop offers.

Our overwrite semantics is confusing to most. We need to correct it in the next release, i.e., Spark 2.4.
Even if we try our best to keep the schema of the original table, the actual CREATE TABLE statements still take many vendor-specific info. It is hard for us to rebuild all of them. I can understand your use case for truncate.
I am sorry this will not be part of Spark 2.3 release. We will include it in the next release. You can still do the change in your forked Spark.
Just feel free to let us know if you find anything that we should do in Spark SQL JCBC to match the corresponding ones in SQOOP. Thanks!

This test is a flaky test. Your changes did not fail any test case. I will review your PR after the 2.3 release. Thanks again!
cc @dongjoon-hyun Do you want to take a look at this?

Thank you for pinging me, @gatorsmile . Yep. I'll take a look both OrcQuerySuite and this PR in this morning. Sorry for late response, @danielvdende .

Thanks guys! @gatorsmile @dongjoon-hyun
Happy to help out expanding Spark SQL JDBC where necessary to match and improve on sqoop :-)

Hi, @danielvdende. I finished my first round review.

Thanks for your review @dongjoon-hyun 👍 I've corrected all the indentation problems (didn't show up locally when running scalastyle checks for some reason). Added comments where necessary for the other points.
Sure, no problem to hold this until Spark 2.4, would be great if it could land there.

Test build #87524 has started for PR 20057 at commit 3921368.

Test build #87524 has finished for PR 20057 at commit 3921368.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87524/
Test FAILed.

Merged build finished. Test FAILed.

I finished my second round. Could you update once more?

Test build #87529 has started for PR 20057 at commit b79becd.

Test build #87529 has finished for PR 20057 at commit b79becd.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87529/
Test FAILed.

Thanks again @dongjoon-hyun, much appreciated 👍. I've made changes accordingly, and added comments in this PR where appropriate.

Test build #87530 has started for PR 20057 at commit 7e09f90.

Test build #87530 has finished for PR 20057 at commit 7e09f90.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87530/
Test FAILed.

Test build #87531 has started for PR 20057 at commit 0e646b2.

Test build #87531 has finished for PR 20057 at commit 0e646b2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87531/
Test PASSed.

Test build #87543 has started for PR 20057 at commit c262993.

Test build #87543 has finished for PR 20057 at commit c262993.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87543/
Test FAILed.

Test build #87582 has started for PR 20057 at commit 2da0ee7.

Test build #87582 has finished for PR 20057 at commit 2da0ee7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87582/
Test PASSed.

@dongjoon-hyun Made the changes you pointed out, thanks! 👍

Test build #87591 has started for PR 20057 at commit ed452e7.

Test build #87591 has finished for PR 20057 at commit ed452e7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87591/
Test PASSed.

@dongjoon-hyun I saw that Spark 2.3 was released a few days ago, congrats on the release! :-) Is there anything stopping us from merging this PR into master now?

Test build #88040 has started for PR 20057 at commit 7e0ff07.

Test build #88040 has finished for PR 20057 at commit 7e0ff07.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88040/
Test FAILed.

Retest this please.

Test build #88052 has started for PR 20057 at commit 7e0ff07.

Test build #88052 has finished for PR 20057 at commit 7e0ff07.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/88052/
Test FAILed.

The failures are

Last time, RateSourceV2Suite.
This time, HiveClientSuites.

Retest this please.

Test build #88057 has started for PR 20057 at commit 7e0ff07.

Test build #88057 has finished for PR 20057 at commit 7e0ff07.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Gentle ping @gatorsmile :-)

Gentle ping again @gatorsmile @dongjoon-hyun :-)

Gentle ping @gatorsmile @dongjoon-hyun 👍

@gatorsmile @dongjoon-hyun Guys, any update?

@dongjoon-hyun @gatorsmile Any update, guys?

Test build #92841 has started for PR 20057 at commit bc75051.

Test build #92841 has finished for PR 20057 at commit bc75051.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92841/
Test FAILed.

retest this please

Test build #92963 has started for PR 20057 at commit bc75051.

Test build #92963 has finished for PR 20057 at commit bc75051.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92963/
Test FAILed.

retest this please

Test build #93322 has started for PR 20057 at commit bc75051.

Test build #93322 has finished for PR 20057 at commit bc75051.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93322/
Test FAILed.

Just made the changes you mentioned @gatorsmile :-)

Test build #93327 has started for PR 20057 at commit a365f79.

Thanks! Merged to master.


Charter update

Rewording, make it more crisp

It looks good to me. I am also fine with simplifying the proposal to M and sub-M.

Closing, more changes required


Run renode from within docker container

Workaround to keep renode working as part of CI without needing to upgrade(which is the subject of #1584).
BUG=http://b/260027053

Merging without branch protection because we need to merge this PR and then we can have all tests pass for #1580


Replace egirna with integrationworks

This change successfully prevents go mod tidy from importing the egirna/icap-client and instead uses the integrationworks/icap-client module. Also imports the egirna/icap v0.2.0.

Hey team,
This pull request was accidentally sent through - the changes to the module import is just for local use during development.
We can see this is a WIP library. We have found a couple of bugs and are looking to improve a little bit of the request and response handling with the ICAP server.
We have forked it so we can make some changes without impacting you. Later down the track when we have a more complete and polished fork we would love to be able to merge back into your repo.
Cheers,
Gabriel

Hello @tennengabr Glad to see you using the package, yes it is a work in progress. Good luck.


pls merge photosaves into mobile staging

you can make new posts and i promise it works

Coverage remained the same at 91.14% when pulling 21ae05a68cd5a1df7c2522b5d4c27fffee412fd0 on photosaves into 69db68dd3913b220db11f8f4f112be2228099c5b on mobile-staging.


struct.error: 'I' format requires 0 <= number <=<PHONE_NUMBER>

Hi all,
I have six R1 fastq.gz files and six R2 fastq.gz files, I merged them together separately, then used the following commands to run the cite-seq-count:
CITE-seq-Count -R1 R1.fastq.gz -R2 R2.fastq.gz -t tags.csv -cbf 1 -cbl 16 -umif 17 -umil 26  -wl whitelist.txt -o ./ -cells 30000
Everything goes well, except there is an error during the process:

Process ForkPoolWorker-1:
Traceback (most recent call last):
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/pool.py", line 127, in worker
put((job, i, result))
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/queues.py", line 367, in put
self._writer.send_bytes(obj)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/connection.py", line 203, in send_bytes
self._send_bytes(m[offset:offset + size])
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/connection.py", line 396, in _send_bytes
header = struct.pack("!i", n)
struct.error: 'i' format requires -2147483648 <= number <=<PHONE_NUMBER>
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/process.py", line 297, in _bootstrap
self.run()
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/process.py", line 99, in run
self._target(*self._args, **self._kwargs)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/pool.py", line 132, in worker
put((job, i, (False, wrapped)))
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/queues.py", line 361, in put
obj = _ForkingPickler.dumps(obj)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/site-packages/multiprocess/reduction.py", line 54, in dumps
cls(buf, protocol).dump(obj)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 437, in dump
self.save(obj)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 771, in save_tuple
save(element)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 771, in save_tuple
save(element)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 549, in save
self.save_reduce(obj=obj, *rv)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 638, in save_reduce
save(args)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 771, in save_tuple
save(element)
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/ihome/crc/install/cite-seq-count/python3.7/lib/python3.7/pickle.py", line 747, in save_str
self._write_large_bytes(BINUNICODE + pack("<I", n), encoded)
struct.error: 'I' format requires 0 <= number <=<PHONE_NUMBER>

I have no idea what happened, and the procedure is still running, not sure if this will influence the result.
Would you please give me any suggestion on this?
Thank you!

Hello @AI1013
How many reads are there in your files?

Hi @Hoohm,
I just tried to run with each file separately, and it worked! I think it was because there were too many reads in the mixed file. Anyway, thank you!

Yes that's probably the issue. Happy to hear that it worked.
Closing this now.


feat(organizationendpoints): add support for organization endpoints for UI commands and projects

https://coveord.atlassian.net/browse/KIT-2387
Proposed changes
Modify templates/commands etc for UI* commands to support organization endpoints feature.

@olamothe, instability due to the release has been mitigated, the remaining ❌ are probably valid


Please specify the version of PHP_CodeSniffer in composer.json

The version that comes with Magento 2.1 (1.5.3) doesn't work with 1.0.1 of this repo.

We added requirements to our README.md file.

But can you add them to composer.json? That's kinda what it's for ;-)

@danslo Do you mean that we need to add it into "require" section?  In case someone already has PHPCodesniffer - why he need to install another one? The main idea of custom standards is that they should be an addition to already exists Codesniffer standards.

Yeah, that's exactly what composer is for. If your repository doesn't work without that specific  dependency, it has a requirement on it.
By your logic, why specify "php": ">=5.6.0", someone might already have PHP 5.4.

@danslo Some of our sniffs doesn't work on php5.4.

And your code sniffs don't work at all without PHP_CodeSniffer.

And what if someone already use PHP_CodeSniffer?

In the case of standalone checkout of marketplace-eqp, it's not a problem since codesniffer lives in its own vendor directory.
In the case of some main project which has dependency on specific version of codesniffer and marketplace-eqp, this person should either;

relax their version requirement on codesniffer
choose between requiring marketplace-eqp or a lower version of codesniffer, not both

In any of the above cases, marketplace-eqp repo should still be specific about its own requirements.

There is one more case - when PHP_CodeSniffer lives as it's own application. In that case person needs only to specify path to MEQP standards or path to other standard they what. Without installing PHP_CodeSniffer for each standard. As example you can look how PhpStorm works. After specifing path to phpcs, all what you need is just to set path to your custom standard and it is ready to use.
Marketplace-eqp is not a standalone application (for now). In future, everything can change, and, ofcourse, we will add this requirement to composer. But now, it's just a standard like many others.

Without installing PHP_CodeSniffer for each standard.

If you don't care about dependency management, you're not forced to run composer install. At that point it will just be a guessing game if your already installed random version of codesniffer will work with these standards.
I also don't understand the comment about standalone applications. Libraries and/or standards shouldn't specify dependencies?

@danslo I'm sorry for late response. Actually, now we are reviewing your recommendation with our architects. And we will notify you as soon as.

@danslo we reviewed your proposition and PHP_CodeSniffer version will be added to composer.json file in next release.
Thank you for reporting.

Thank you!


HM-MOD-Re-8 – only channel one is working!

Hi - I'm a newbie when it comes to 'using Homematic within Homeassistant'.
I recently moved over from FHEM to HomeGear/Homeassistant with some blind actors, dimmers and this 8-channel switch. HomeMatic devices setup in HomeGear showed up in HassIO as they should.
The HM-MOD-Re-8 causes some issues and I’d like to check back with others to see how they come around. The 8 different channels of the PCBmodul have been detected as switch.neqXXXXXXX_1 - switch.neqXXXXXXX_8. Unfortunately, only channel one could be switched on/off.  Error message for channel 2:
2023-05-19 15:10:41.137 ERROR (SyncWorker_15) [pyhomematic.devicetypes.generic] HMDevice.setNodeData: STATE not found with value True on 2
The state of channel 2-8 is not detected in HassIO either if any of these channels is switch manually.
For me it looks like channel 2-8 is not mapped / linked between HomeGear and HassIO. Or a wrong ‘name setting’ is used for the channel 2-8. Is there any chance to debug the communication between HassIO and HomeGear?
I appreciate any further tips and tricks here.
Thanks
Holger

Should be working with the new integration.

Hi Daniel - I've update to the Beta 1.41.1 but still got just one switch. Is there a need to ´reload´ the configuration from homegear or any other advise how to access the missing switches 2-8?
Thx
Holger

Apart from properly installing and configuring the new integration, nothing else should be required.
Please open an issue here: https://github.com/danielperna84/hahomematic/issues
And include all the information the issue template is asking for (logs, version information etc.)


Only use the translated package name as a fallback when no metadata i…

PR Checklist

[x] A description of the changes is added to the description of this PR.
[ ] If there is a related issue, make sure it is linked to this PR.
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] Documentation in docs is updated

Description of changes
An easy example solution for https://github.com/fpgmaas/deptry/issues/335 and https://github.com/fpgmaas/deptry/issues/336.
:rotating_light: not adding the translated package name by default may, but in practice should not, break things.

@akeeman I think the solution direction looks good! Do you want to resolve the conflict with the recent addition you made to dependency.py? If not, I am also happy to pick this up soon.

Sure, let me fix it. I just wanted to know your thoughts before putting in more work.


Apple Secure Enclave Support

Hello,
I suspect this may be a neighbor/relative of #36 , but it would be nice to implement usage of the secure enclave in order to leverage TouchID on Macs.
There is already a project that has a PoC of this, so it's definitely possible.
Basically, you need to use something like Secretive to stand up an ssh agent that communicates with the SE. The PoC above has a subcommand that takes the raw public key and uses it to create a cert for use with IAM Roles Anywhere.
While the actual cert creation is probably out of scope for this feature, it would be nice to have a flag/format that could tell it when a private key is in the SE (or even just via the ssh agent).

Can this be done through PKCS#11?

Possibly, but I think one would have to write a PKCS11 module for the Secure Enclave and it doesn't appear that one exists at the moment  (Near as I can tell).
The way the PoC appears to work is it runs sign operations through the SSH Agent (agent package in go, client) and Secretive just so happens to run its own that talks to the SE.
Most SE implementations I've seen are Swift or Objective C, but I'll keep looking!

I'm no expert, only someone who has occasionally vaguely researched the topic for their own interest. But I never tried coding anything, mostly becuase I  don't speak Swift or Objective C and was hoping to find a way to do it in something else (e.g. Go or Rust).
My understanding is that broadly access to Apple Secure Enclave is further segregated by developer so as to combat the possibility of one developer gaining access to another's key output.  And so this is ultimately the reason for the Swift/Objective C interface because of the root-of-trust flows.
IIRC CryptoTokenKit has integrations with both SE and Keychain and so could potentially provide an interesting bridge between SE and reuse of existing code on this project for keychain ?
There is also an alternative approach mentioned on this Apple forum post.


Add support for generating images with different sizes.

Hi there. It seems like the images generated all have a fixed size of 256x256 pixels not only on the frontend but also when saving them to disk. I think it would be nice if we could have a parameter to specify the size of the images been generated when running Dalle Playground just like how we can set a port or configure the model size, even better would be an option on the frontend/interface from which we could select the image size we want from a drop-down menu or set a custom size, that way we can create images that can be used for wallpapers or for other practical uses, in my case I was able to generate some awesome images that could be used perfectly as wallpaper but because of the small size I can't use them for that.

The small size is because of the decoder dimensions of the model which has been set to 256x256. Dalle-2 will have 512x512 because the decoder network is larger. This is with VRAM and memory maximized. It is just the current state of AI and in the future you will see much higher resolutions and fidelity.

@youmebangbang I understand, in that case, would it be possible to implement it by doing something like what is mentioned here on this article?
What is done there is expanding the images generated with DALL-E Mini by using VQGAN and CLIP to inpaint the sides, this will create a bigger image and for what I see it can be done to get an image of any aspect ratio and probably of any resolution with a bit of tweaking the code, unfortunately I'm not good enough coding to do that myself. By using the code provided there on that article or here if it is added to Dalle Playground it would be really useful.


Login with FB, Twitter, …

Add Login with Facebook, Twitter, Google and so on.

I understand your point about binding an authentication method with this commenting library. However, there is no way that this library could provide these login options out-of-the-box. First of all, you need to validate the user in your back-end as the data in front-end cannot be trusted as such. Second of all, these identity providers that you mentioned require an app to be registered in their developer portal. Thus there is no generic way to provide these login options.

If you want to provide these login options, I can recommend a front-end library called hello.js (https://adodson.com/hello.js/) that does most of the work for you. Still you need to register the applications for each identity provider.


[BUG] In proactive connection warmup, the minConnectionsPerEndpoint is not working as expected

Describe the bug
minConnectionsPerEndpoint is set with value more than 1 and enabled proactive connection warmup, but the number of connections initialized is not changing. It is same as when the minConnectionsPerEndpoint is 1.
After the primary investigation, we are suspecting minConnectionPoolSizePerEndpointForContainer is reset to the default value.
Exception or Stack Trace
No exceptions.
[main] INFO com.azure.cosmos.CosmosClientBuilder - Cosmos Client with (Correlation) ID [00001] started up in [74057] ms with the following configuration: serviceEndpoint [https://.documents.azure.com:443], preferredRegions [[South Central US, West US 2, East US 2]], connectionPolicy [ConnectionPolicy{httpNetworkRequestTimeout=PT1M, tcpNetworkRequestTimeout=PT5S, connectionMode=DIRECT, maxConnectionPoolSize=1000, idleHttpConnectionTimeout=PT1M, idleTcpConnectionTimeout=PT0S, userAgentSuffix='-daf-cosmos-benchmark-app-azsdk-java-cosmos/4.50.0-beta.1 MacOSX/10.16 JRE/1.8.0_311', throttlingRetryOptions=RetryOptions{maxRetryAttemptsOnThrottledRequests=9, maxRetryWaitTime=PT30S}, endpointDiscoveryEnabled=true, preferredRegions=[South Central US, West US 2, East US 2], multipleWriteRegionsEnabled=false, proxyType=null, inetSocketProxyAddress=null, readRequestsFallbackEnabled=false, connectTimeout=PT5S, idleTcpEndpointTimeout=PT1H, maxConnectionsPerEndpoint=130, maxRequestsPerConnection=30, tcpConnectionEndpointRediscoveryEnabled=true, ioThreadPriority=5, ioThreadCountPerCoreFactor=2, tcpHealthCheckTimeoutDetectionEnabled=true, minConnectionPoolSizePerEndpoint=1, openConnectionsConcurrency=1, aggressiveWarmupConcurrency=16}], consistencyLevel [Session], contentResponseOnWriteEnabled [true], sessionCapturingOverride [false], connectionSharingAcrossClients [false], clientTelemetryEnabled [false], proactiveContainerInit [(containers:dbs/DataAccessFramework/colls/CosmosSDKTest)(pcrc:1)(awd:PT1M)], diagnostics [{samplingRate=1.0, thresholds={pointOperationLatencyThreshold=PT1S, nonPointOperationLatencyThreshold=PT3S, requestChargeThreshold=1000.0, payloadSizeInBytesThreshold=1048576, customFailureHandler=false}, clientCorrelationId=null, clientTelemetryEnabled=false, clientMetricsEnabled=true, transportLevelTracingEnabled=false, customTracerProvided=false, customDiagnosticHandlers=(com.walmart.platform.daf.cosmos.CustomCosmosDiagnosticsHandler)}], tracing [true, true]
To Reproduce
Steps to reproduce the behavior:
Code Snippet
Add the code snippet that causes the issue.
Expected behavior
Based on minConnectionsPerEndpoint, the connections should increase after warmup.
Screenshots
netstat -anvt | grep "<IP_ADDRESS>" | grep 'ESTABLISHED' |wc -l (with minConnectionsPerEndpoint=1)
178
netstat -anvt | grep "<IP_ADDRESS>" | grep 'ESTABLISHED' |wc -l (with minConnectionsPerEndpoint=2)
178
Setup (please complete the following information):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)] 4.50.0-beta.1
Java version: [e.g. 8]
App Server/Environment: [e.g. Tomcat, WildFly, Azure Function, Apache Spark, Databricks, IDE plugin or anything special]
Frameworks: [e.g. Spring Boot, Micronaut, Quarkus, etc]

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please check out Troubleshoot dependency version conflict article first. If it doesn't provide solution for the problem, please provide:

verbose dependency tree (mvn dependency:tree -Dverbose)
exception message, full stack trace, and any available logs

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

Hi @joyaljohn someone from the Cosmos team will follow up shortly

@jeet1995 any updates on this issue?

Closing because of inactivity, @joyaljohn please re-open the issue if it still persits.


Custom filters not showing on ClrDatagridFilterInterface

Describe the bug
When using a custom datagrid filter component, the filter object is not appended to the ClrDatagridFilterInterface's filters attribute.
Attempting to use the checkbox-list-filter component described in this issue, updated per the latest documentation for custom filtering.
Also attempted to use the state property on the filter as implemented in #3066 but I may be doing that wrong...
How to reproduce
https://stackblitz.com/edit/clarity-light-theme-v1-0-znytxw
Steps to reproduce the behavior:

Attempt to filter on the Role or Group field.
Console should be logging the ClrDatagridFilterInterface object passed from the (clrDgRefresh) method.
No filters should be present on the event object.

Expected behavior
At the very least the filters attribute should exist on the ClrDatagridFilterInterface object
Versions
App

Angular: 7.2.0
Node:
Clarity: 1.1.0

Device:

Type:
OS: Windows 10
Browser Chrome/Firefox
Version 71/57

Additional notes

Your filter has some problems.
The isActive function always returns false.
This is due to selectedItems always being empty.
The onItemChanged function fails to update the array properly.
I am not 100% sure about your use-case, but something like this seems to work better:
public onItemChanged(item) {
    if (item.checked) { // cases reversed
      item.checked = false;  // update the checked field
      let index = this.selectedItems.indexOf(item);
      if (index >= 0) {
        this.selectedItems.splice(index, 1);
      }
    } else {
      item.checked = true;  // update the checked field
      this.selectedItems.push(item);
    }
    this.changes.next(true);
  }

Updated Stackblitz:
https://stackblitz.com/edit/clarity-issue-3120?file=src/app/checkbox-list-filter.component.ts
This type of question is more suitable for StackOverflow than for a github issue. Please, open similar requests there. It is actively monitored. Thank you!


Easier component template coding?

What is the preferred, more comfortable way to create component templates? Could I use something like hyperscript?
At the moment:
Moon.component('my-component', {
  template: "
This is a Component!
"
});

or
template: `

Count: {{count}}

    <button m-on:click="increment">Increment</button>

`,

Thanks

I just found HyperMoon, this is what I should use, correct?

If you really need more control, you can use HyperMoon, but it isn't documented (yet). It's a very strict version of hyperscript, allowing for the Moon compiler to make optimizations that don't have a cost at runtime when the m function needs to normalize the parameters.
The best way to create components is through single file components. Check out moon-cli to generate a project using them. Basically, they allow you to have templates like:
<template>

Component! Content prop: {{content}}

</template>

Nice. But is HyperMoon also usable outside the render function scope? My reason being that I have a build chain and skeleton app that I would like to keep for now.
HyperMoon usage in core
 render: function(m) {
            return m('h1', { attrs: {} }, { shouldRender: false }, [m("#text", { shouldRender: false }, "Hello Moon!")]);
        }

Thanks

Yes, it is available with Moon.util.m.
I can give you a basic rundown of HyperMoon:

The basic format goes like:

Elements - m("tag", { attrs:{} }, { shouldRender: true }, [/* children */]);
Text - m("#text", { shouldRender: true }, "Some Content");

The meta parameter ({ shouldRender: true }) decides if Moon will skip over that certain element when diffing. Only set it to true if any children have some dynamic property that can be updated.
To interpolate data, you can do something like m("#text", { shouldRender: true }, this.get("msg"));

One problem though, I tried something:
const m = Moon.util.m;

Moon.component("foo-component", {
        template: m('h1', { attrs: {} }, { shouldRender: false }, [m("#text", { shouldRender: false }, "Hello Moon!")])
    });

This causes an error:
Uncaught TypeError: Cannot read property 'type' of undefined

What have I done wrong? Moon.Js is a lot of fun :)

A template cannot be an output of a function, you need to do something like:
Moon.component("foo-component", {
  render: function(m) {
    return  m('h1', { attrs: {} }, { shouldRender: false }, [m("#text", { shouldRender: false }, "Hello Moon!")]);
  }
});
```js

Nice! Great work you've done on the framework.

Thank you! Glad I could help.


Clipboard issues when copy / pasting

Given that this extension is based on "screencasting", there is probably some extra handling needed for dealing with "advanced keyboard input" like keyboard shortcuts e.g. for copy / pasting. This currently manifests in a few issues/limitations:

You can't select text and hit e.g. CMD + C to copy the text to your clipboard (this neither works in the browser preview view nor in the debugger view)
You can't select text and right click to open the context menu to "manually copy" (this seems to work ok-ish in the debugger view but doesn't work in the browser preview view)

+1 I really like the extension and for using it I would find the copy and paste functionality quite important.

+1

Pasting to the website just pastes the text into the URL bar at the top for me. Would be great for that to work although not sure

1+ same


clone_vm/clone_server takes reference as first parameter

Server#clone called clone_vm with wrong parameter order causing
subsequent API call failure.

Damm. Good catch.
Can you take care of the few style warnings?
Thank you!
PS: what are your feedback on fog-xenserver over the old code from fog. Something else we can improve?

Yep, can check style warnings.
For a while I fight with VM creation from template, but I found correct approach in git log. Than I run into problems with following code:
template = connection.tempaltes.find_by_name 'template_name'
vm = template.clone 'vm_name'
vm.save
vm.provision
vm.start

Where vm.start failed due to missing disk (as vm.save created new VM without disk - so at this time I had 2 templates; first one originally cloned and second one without disk which was converted to VM), so I realise that vm.save is not safe operation and removed it from code.
Issue I'm fight right now is that after vm.start I don't have access to vm.guest_metrics.networks (even after vm.reload) - returns empty Hash. If I try to load vm from another connection, networks has is populated correctly.
All my issues seems to be related to lack of documentation.

Coverage remained the same at 94.07% when pulling b7b75ac4f150ced7a8aa9526a6db03200906fbbc on djaara:master into a107ca72ecfccbe123769eff3109526c7deaf2ad on fog:master.

Hmmm. Thank you for the feedback. We have some documentation in the wiki but most are not updated. Gotta fix them.
The point with vm.save now is that it is pretty much what the XENAPI does. No magic behind the scenes. This decision make it a little more verbose to create a vm but at least now we have a better control over it. Before we didn't have much options and if you wanted to create something different you were on your own.
This new design follows the `XENAPI´ one. A good look at here should help you a lot. =)

Coverage remained the same at 94.07% when pulling 56128e2fb3aa5b6e1ab4872134c210c6e55684ac on djaara:master into a107ca72ecfccbe123769eff3109526c7deaf2ad on fog:master.

Thx @djaara !

Release as 0.2.2. Thanks for your time and efforts. :heart:

Great! Thanks.
One more thing - is there any alternative to original Fog::Compute::XenServer::Server.refresh?
def refresh
  Fog::Logger.deprecation(
    'This method is deprecated. Use #reload instead.'
  )
  data = service.get_record( reference, 'VM' )
  merge_attributes( data )
  true
end

At this moment, I've replaced refresh calls with:
vm = connection.servers.get vm.reference

But there might be better solution.

Yeah, there is reload
Since fog-core already defines reload there wasn't much sense to keep refresh. =)

I forgot to mention, that reload does not work for me:
With #reload I'm experiencing problems with accessing latest guest_metrics. I'm getting guest_metrics with data related to original template, not to provisioned/started VM.

Hmm. i suspect it might be a bug somewhere when instantiating the objects.
Perhaps you could open a issue here for this behavior and provide more info like whats the value of data here and the value of new_attributes here.
I suspect it might be related to the associations stuff and the merge_attributes method defined here. If you can provide more info we can find the issue and fix it.


chore: release main

:robot: I have created a release beep boop
core: 0.10.0
0.10.0 (2022-11-26)
⚠ BREAKING CHANGES

Use only string for highlight.

Features

Not allow overriding default models. (2b1a799), closes #41

Code Refactoring

Use only string for highlight. (1e3f484)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/timgreen/Anki.md/releases/tag/core/v0.10.0 :sunflower:


created new html file

LOOK AT THIS BABY PAGE !!!!!

test test

Great job!

Good good
On Wed, Nov 21, 2018 at 1:41 PM Kaleb Kougl<EMAIL_ADDRESS>wrote:

[image: 200]
https://user-images.githubusercontent.com/33531057/48864728-15695480-ed93-11e8-88b3-4518d9b2e0c7.gif
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/narwhalnancy/Group-Project-1/pull/1#issuecomment-440786773,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Aqfwu8O_RG_4H4unpYL1pP9TZuZjxgUwks5uxaxTgaJpZM4Ytw-n
.


Some question about inkd_loss?

Thanks your work, in your code, I notice a inkd_loss which not mentioned(or maybe I have not found it.). I am curious about the function of inkd_loss, and the use of Imagenet1K dataset.
In addition, if I understand correctly, features[1] in line methods/contrastive_training/common.py 351 is a global prompt, and features[1] is different for different samples due to the norm operation and dropout. I have not figure out what's meaning of constrastive loss defined on features[1]?
I would very appreciate it if you can help me.

Hi, thanks for your interest in our work.
We actually mentioned the INKD loss in the implementation details section in appendix. But we observe there is no significant effect of this loss on medium and large datasets. Furthermore, this loss is included for only first few epoch training in the first stage. Therefore, the effect should be minor. Our initial motivation is to counteract the sensitiveness of randomly initialized visual prompts w.r.t. the performance. Empirically, if we remove this loss, sometimes the performance can improve a little. Thus, we suggest that if you are training on a dataset of a medium or large size, you can directly remove it.
We propose discriminative prompt regularization to improve the semantic discriminativeness of the prompt-adapted backbone, which includes additional task-related supervision on the ensembled prompt features. Please look into section 3.2 in our paper.
If you have further question, please let us know. Thank you.

@DevinCheung Thanks for your good comments.
We do compute INKD over all the datasets to keep our implementation consistent. However, several points needed to be clarified:
Firstly, notice that we only compute INKD loss for 5 epochs with linear-decay weights to zero for only 1st stage training, which is small and just for counteracting the negative effects of prompt initialization.
Secondly, as for the motivation, we observe this loss can add minor benefits to the first stage performance on small datasets, i.e., CUB-200. In practice, we can use any arbitrary dataset including the downstream dataset itself to do the distillation.
Lastly, the positive effect of this loss on the final performance is small/minor, according to our results.
In practice, we can also directly remove this loss.

@DevinCheung Thanks for your good comments.
We do compute INKD over all the datasets to keep our implementation consistent. However, several points needed to be clarified:
Firstly, notice that we only compute INKD loss for 5 epochs with linear-decay weights to zero for only 1st stage training, which is small and just for counteracting the negative effects of prompt initialization. Thus  the positive effect of this loss on the final performance is small/minor, according to our results.
Secondly, as for the motivation, we observe this loss can add minor benefits to the first stage performance on small datasets, i.e., CUB-200. In practice, we can use any arbitrary dataset including the downstream dataset itself to do the distillation.
Lastly, GCD assumes the Imagenet-1k pretrained DINO, which also assumes the whole ImageNet without labels. Our work follows this setup.


Postpone button: move to next valid recurrence date

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
For recurring tasks, the "postpone" button (⏩) of task views pushes the task to the next day, by default.
Now, when the recurrence is, for example, only on work days (Monday–Friday), it would be helpful to have a quick way of moving the date to the next occurence (thus, postponing on a Friday would move to Monday). There is currently no such option, despite this being a reasonable usage (if you don't work on week-ends, on Fridays, you probably want to postpone something to the following Monday, typically).
✔ Solution
I would therefore suggest that right-clicking on the postpone button also list next occurences, and that it list them quite close to the top, as I expect this to be a natural way of postponing tasks:

Scheduled in 1 occurence (Monday, in the example)
Scheduled in 2 occurrences (Tuesday, in the example)
…

(or maybe "Recurrence day 1", "Recurrence day 2", etc.).
The question of whether postponing a task should by default follow the recurrence rule for this task or not is somewhat open, in my mind. However, I would argue that the current behavior is best: it's better to not risk missing a task because it has been pushed too far, so postponing to the next day by default seems fine to me (but adding a quick access to the next occurence(s) would make postponing easier, when this is what matters to users).
❓ Alternatives
The current alternative is to parse the list of next days and manually selecting the one that matches. This breaks the flow.
📝 Additional Context
No response

Thank you for the suggestion. I do understand the request.
I think the facility would be nice to have, but I feel that the Postpone menu is not the best place for it, because:

It would make the menu more complex and harder to understand at a glance.
We have a lot of diverging and inconsistent requests for changes to Postpone already, and bolting on a whole different set of behaviour complicates things for everyone (users and volunteer maintainers.)
It would complicate the code and make it harder to maintain, forever more.

Much better would be to add a context menu on the Recurrence emoji, and allow users to right-click on that and select options to skip occurrences.
I would welcome a separate feature request for that facility.

Thanks. I followed your suggestion and moved this to a Recurrence context menu feature request (#2678).

@claremacrae I know that this request is closed but is probably the best to ask it here.
I guess the most common action for a recurring task would be to just skip one occurrence. How about adding one item to the postpone menu for recurring tasks?
Here is an example, the date is random, wording may change
              Next occurrence, on Fri 8th Dec  // <-- this is the new item
              ---
              Scheduled today, on Sun 3rd Dec
              Scheduled tomorrow, on Mon 4th Dec
              ---
              Scheduled in 2 days, on Tue 5th Dec
              Scheduled in 3 days, on Wed 6th Dec
              Scheduled in 4 days, on Thu 7th Dec
              Scheduled in 5 days, on Fri 8th Dec
              Scheduled in 6 days, on Sat 9th Dec
              ---
              Scheduled in a week, on Sun 10th Dec
              Scheduled in 2 weeks, on Sun 17th Dec
              Scheduled in 3 weeks, on Sun 24th Dec
              Scheduled in a month, on Wed 3rd Jan
              ---
              Remove scheduled date

How does this look? =)


Missing text mesh pro dependency in Extensions.

Sorry for the PR's on individual files...I'm just using the github web-based file editor to make patches.
Extensions contains asset LostTrackingService\Assets\LostTrackingVisualPrefab.prefab, that depends on Packages/com.unity.textmeshpro/Scripts/Runtime/TextMeshPro.cs, but package dependency is missing.

This will ultimately be resolved by a dependency of TMP on the standard assets package (#8761) as all other MRTK packages contain a dependency to that one.

Replaced by #8764


unsubscribe during dispatching throws warning in child component

Warning: setState(...): Can only update a mounted or mounting component

I have a parent and child component. They both subscribe to my redux store. Upon store's state change, the parent handler is called first and changes its state to hide the child component (e.g. render() is called). Because the child is not rendered, the child's componentWillUnmount() method is then called and I unsubscribe()  from the redux store in that method.
After that, the redux dispatching continues and the child handler method still gets called and this warning is the result because I'm updating the child's state in that handler.
If unsubscribe can't work between dispatching, then I don't understand how to do basic UI state changes like hiding or hiding components. Could I be doing something wrong?

First, usage questions should really go on Stack Overflow - filing issues should be reserved for actual bugs with Redux.
Second, it sounds like you're trying to manage store subscriptions manually.  Don't do that - use the official React Redux bindings, which take care of all the store subscription process for you, and also give you some nice performance optimizations.


fix for depending on css context as same than javascript context

Added a new option "document". The current version 2.0.1 has a problem, where loading the spin.js javascript code from another frame, added the css rules to that other frame. Basically it came down to wrongly relying on the "document" where the javascript is running as the default setting.
I added this for use of spin.js in a chrome extension, but I'm sure this bug also happens when using iframes, multiple frames, or other stuff like that.

spin.js v3.0 no longer injects a CSS sheet into the document head, so this should no longer be an issue.


include path and x_ms_path endpoints in genearted crates

This addresses an issue where some specs split endpoints between path
and x-ms-path.  This exposes a significant number of operations across
the generated crates that were not previously included.
NOTE: this was generated using version of the REST specs, 48d85585897aa6ed448ca689b094e60377a25cb7

Note, this PR (or something that accomplishes the same goal) is required to use the generate crates as a backend to the newly added azure_device_update crate.

The failures identified in the build are not part of this PR and will be addressed in #719


Having trouble with login via command line parameters

Hi, I noticed you recently implemented the .token file for stateful logins, but I'm having trouble using the command-line parameters -u and -p. I'm trying this because sampleStreaming.js doesn't seem to work with the .token file (I'll open a separate issue).  If I specify -u and -p for one of the examples, like odometer.js, I get the following output:
F:\projects\teslajs\node_modules\teslajs\Examples\sampleFramework.js:34
        if (result.error) {
                  ^

TypeError: Cannot read property 'error' of null
    at SampleFramework.login_cb (F:\projects\teslajs\node_modules\teslajs\Exampl
es\sampleFramework.js:34:19)
    at Request._callback (F:\projects\teslajs\node_modules\teslajs\TeslaJS.js:18
9:9)
    at Request.self.callback (F:\projects\teslajs\node_modules\teslajs\node_modu
les\request\request.js:186:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:192:7)
    at Request.<anonymous> (F:\projects\teslajs\node_modules\teslajs\node_module
s\request\request.js:1081:10)
    at emitOne (events.js:96:13)
    at Request.emit (events.js:189:7)
    at IncomingMessage.<anonymous> (F:\projects\teslajs\node_modules\teslajs\nod
e_modules\request\request.js:1001:12)
    at Object.onceWrapper (events.js:291:19)

It looks like login_cb in function SampleFramework. is only expecting (result) as a parameter.
function login however is invoking the callback as:
callback(error, { error: error, response: response, body: body, authToken: authToken });

I'm thinking the line in sampleFramework.js should be
this.login_cb = function (error, result) {

Is this work in progress or do I have something wrong?
I installed the latest from npm but according to package.json I have version 2.1.7.  Thanks.

Additional: I deleted teslajs and reinstalled from npm, and now I have verison 2.1.10.  I thought npm update would have updated it to the latest version?

@TrevorPT npm update will update your dependencies to the latest version following the semver rules in your project's package.json file.  Have you created your own project using npm init followed by npm install teslajs --save?

@TrevorPT I'll take a look at the streaming sample.  It is possible that something that changed with a prior refactoring broke the sample and went unnoticed.

@TrevorPT Thanks for reporting this, I've found an issue with the simpleStreaming sample that was a left-over from migrating to the new sample framework.  I am working on a fix right now and should have this checked in shortly.

@TrevorPT OK, fix checked in 2.1.11.  The .token file to cache the OAuth token has been around from the beginning.  The streaming API works differently than other API's.  It takes a token that is returned from the call to enumerate vehicle information.  This token is used as the "password" and is paired with the username.

@mseminatore - thanks! I'll give it another try with the update and continue with the streaming thread in #61.  Regarding my question about using npm to update, I'm embarrassed to say I should've known as (believe it or not) I've used npm before! I'm still just playing with the examples but I keep thinking of TeslaJS as the project, not as a module, so please excuse the thoughtless questions!


Support for categorical parameters

This changes the primary type of a parameters assignment from int64 to IntOrStr (which is a 32-bit int, more inline with what is used throughout the Kube API). Additionally, it allows bounds to be specified as an array values to indicate a categorical parameter should be created on the backend.
To ensure round-trip-ability between v1alpha1 and v1beta1, the same compatible change was made to both versions (the size of the values list is set as the max on v1alpha1 just be safe).

Does the UI work with string values?


chore: update to Deno v1.7.5

Update to Deno v1.7.5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Looks like a duplicate of https://github.com/denoland/deno_website2/pull/1666. Didn't notice @ry had already raised a PR to bump the version.
Closing this one.


Add an event to support post-submit actions when all post-submit tests finish

It is not uncommon that we need to do post-submit actions when all tests finish. For example:

Packages repo needs to waits for all post-submit tests to pass before publishing @stuartmorgan
Engine repo needs to roll latest version to framework when all post-submit tests pass (WIP) @XilaiZhang

We need to have a generic way to flag when all post-submit tests pass. Eg. this event can then be consumed from GitHub actions to act accordingly.
/cc @godofredoc

cc @sealesj is also interested in post submit actions

Internal related bug: http://b/308964854


fatal error: librealsense/rs.hpp: No such file or directory compilation terminated

Hi,
After copying the ROS package ros_packages/.../realsense_camera into your catkin workspace source directory (e.g. catkin_ws/src ) and compiling using catkin_make , I am getting the following error.
In file included from /home/ra-1/catkin_ws_robot/src/realsense_camera/src/capture_v2.cpp:3:0:
/home/ra-1/catkin_ws_robot/src/realsense_camera/src/display.hpp:1:31: fatal error: librealsense/rs.hpp: No such file or directory
compilation terminated.
realsense_camera/CMakeFiles/capture.dir/build.make:62: recipe for target 'realsense_camera/CMakeFiles/capture.dir/src/capture_v2.cpp.o' failed
make[2]: *** [realsense_camera/CMakeFiles/capture.dir/src/capture_v2.cpp.o] Error 1
CMakeFiles/Makefile2:367: recipe for target 'realsense_camera/CMakeFiles/capture.dir/all' failed
make[1]: *** [realsense_camera/CMakeFiles/capture.dir/all] Error 2
Makefile:138: recipe for target 'all' failed
make: *** [all] Error 2
Invoking "make -j4 -l4" failed
Please help to solve this

The error might be due to a new version of librealsense SDK (which is on version 2.0 now). The code in this repository is only compatible with version 1.0. Try installing the librealsense SDK version 1.0 from an earlier point in history here, and see if it compiles.


Fix VSC native problems on files with same name

Reproduction

Create two files called deployment.yaml within different directories. Alternatively you can use the monokle-demo repository.
Make them have a misconfiguration (e.g. replicas as a string).
Restart VSCode
💥 Problems will be missing from the Problems tab in the bottom component and no longer be highlighted.
Workaround: Rename one of the two files and it will appear, restarting VSCode will work properly.

This is related to a feature in SARIF where you have a uri together with  uriBaseId (e.g. SRCROOT). Afterwards, the run should contain an object originalUriBaseIds which map that uriBaseId to the actual URI (e.g. /users/johnsnow/code/). The problem is that our current @monokle/validation does not add this originalUriBaseIds and that makes the SARIF VSC Extension attempt to guess the right file, but given that there are multiple, it fails to do so.
How to fix? We attempted to fix this by taking a SARIF generated by Monokle and manually added the object originalUriBaseIds object with values fitting for our local computer. At that point, it no longer tries to attempt to guess the right file and it properly can allocate problems even if multiple have the same file name.
// Something like this
"originalUriBaseIds": {
    "SRCROOT": {
        "uri": "file:///"
    }
},

The SARIF spec talks more about this topic of the presence and absence of this property. Often it's desired to omit it due to privacy concerns (user name leaks) and non-determinism (different content hashes).
The solution, is for the Monokle Validator to be configurable, and optionally, include this information in the response.
Appendix
For MacOS it looks like this, in which case file:/// should be the SRC ROOT value. The same applies to Linux. We still need to determine it on Windows.
{"artifactLocation":{"uriBaseId":"SRCROOT","uri":"Users/john/code/monokle-example/basic/nginx.yaml"}

It seems to be same on Windows, some refs:

https://en.wikipedia.org/wiki/File_URI_scheme
https://www.rfc-editor.org/rfc/rfc1738#section-3.10

Might be related also to https://github.com/kubeshop/monokle-core/issues/448.

It was mostly fixed by https://github.com/kubeshop/monokle-core/pull/588 and #75.
However, one issue still remains. Repro steps:

Open file which has some violations (highlights). The name file shouldn't be distinct across project (there should be at least one additional file with the same name).
Restart VSCode (e.g. via Developer: Restart command).
Upon opening, the file will be opened and no highlighting applied.

When file is reopened it works fine. Seems like SARIF is not processing initially opened files. When we debugged initial issue, there is an array storing file references which is propagated when file is opened. And then if there is no match it will try to find a file based on filename - if the filename is not unique across a project this fails. What I suspect is that initially opened files are not added to mentioned file array (unless reopened) and files with no distinct name cannot be matched then and highlighting cannot be applied.


Build error on win 10

Win10 and .NET Core 3.1 is installed
Starting >>> rcldotnet_common
--- stderr: rcldotnet_common
CMake Warning (dev) at C:/opt/ros/foxy/x64/Lib/site-packages/cmake/data/share/cmake-3.18/Modules/FindPackageHandleStandardArgs.cmake:273 (message):
The package name passed to find_package_handle_standard_args
(DOTNET_CORE) does not match the name of the calling package (DotNetCore).
This can lead to problems in calling code that expects find_package
result variables (e.g., _FOUND) to follow a certain pattern.
Call Stack (most recent call first):
C:/dev/ros2_dotnet_ws/install/share/dotnet_cmake_module/cmake/Modules/dotnet/FindDotNetCore.cmake:37 (find_package_handle_standard_args)
C:/dev/ros2_dotnet_ws/install/share/dotnet_cmake_module/cmake/Modules/FindCSBuild.cmake:20 (find_package)
C:/dev/ros2_dotnet_ws/install/share/dotnet_cmake_module/cmake/Modules/FindDotNETExtra.cmake:15 (find_package)
CMakeLists.txt:9 (find_package)
This warning is for project developers.  Use -Wno-dev to suppress it.

Failed   <<< rcldotnet_common [8.31s, exited with code 1]
Aborted  <<< rosidl_generator_c [10.1s]
Summary: 7 packages finished [28.3s]
1 package failed: rcldotnet_common
1 package aborted: rosidl_generator_c
1 package had stderr output: rcldotnet_common
28 packages not processed
Anybody knows what's going on and how to fix this?
Thanks

Hi @michaelchi08,
I am also facing the same error. rcldotnet_common fails to be built without any explicit error log.
Have you resolved the issue?

Hi @berkayalpcakal , no unfortunately not yet 😞

Hi again,
I was able to build this repository in a ros2 workspace on my Ubuntu 18.04 machine. Then I wanted to build these packages in a ros2 workspace on my Win 10 machine. But 'rcldotnet_common' does not build for some reasons, and gives the aforementioned build output.
Then I decided to follow the way as described in here (using Linux kernel based docker on my Win 10 machine). Finally I successfully built the ros2dotnet on Win 10.

Hi, thanks for letting me know. I'll give it a try. it'll be nice if we could run this natively though. Do yo know if it's possible to run just this in docker and the all other ros2 nodes natively on windows?

Hello. I tried building ros2_dotnet on my windows 10 system today (with Foxy release of ROS2 installed), and I have been seeing the same errors previously mentioned here.
I decided to run the colcon build command with an extra flag (colcon build --merge-install --event-handlers console_cohesion+) and I started seeing some new output in error messaging:
C:\ros2-windows\include\rcl\rcl/event_callback.h(18,10): fatal error C1083: Cannot open include file: 'rmw/event_callback_type.h': No such file or directory (compiling source file C:\Users\Jack Hate\Desktop\GetStartedApp\ros2_dotnet_ws\src\ros2_dotnet\ros2_dotnet\rcldotnet\rcldotnet_node.c) [C:\Users\Jack Hate\Desktop\GetStartedApp\ros2_dotnet_ws\build\rcldotnet\rcldotnet_native.vcxproj]
So then I poked around the install location for ROS2 and I noticed that event_callback_type.h does in fact exist but it's path is rmw\rmw\event_callback_type.h when starting from the include folder.
To verify this was the issue, I tried moving the event_callback_type.h file to the parent directory. Doing this still resulted in a build failure but with new error messages that appear to be syntax issues: C:\ros2-windows\include\rcl\rcl/subscription.h(59,1): error C2059: syntax error: '}'
See attached file for a larger snapshot of the build output.
ros2_dotnet_build_output.txt
I am out of my depth in debugging this further, but maybe someone else can figure out what's going wrong from my findings.


Accurately Pull From Pokemon Base Stats

🎯 Goal
Utilize the existing scaffolding of the Pokedex-Compose App to accurately display a given Pokemon's base stats, only falling back to randomized stats if the initial stat cannot be found in the Pokemon's StatsResponse object.
🛠 Implementation details
This pull request changes code across the application, but only in a way that facilitates the integration of a Pokemon's stats data rather than fundamentally changing any of the core architectural features of the application. As such, the code changes are distributed throughout core-model, core-database, and features, but mostly consist of changes meant to account for the additional fields introduced in PokemonInfo.kt
Core Model Changes
PokemonInfo.kt: Added new fields that corresponds to a Pokemon's 'stats' field, as outlined in the PokeAPI. Replaced existing hp, attack, speed, and defense fields with lazily initialized values that either return the relevant stat from the stats field or a random integer if the stat cannot be found.
Preview Changes
PreviewUtils.kt:  Updated mockPokemonInfo() to reflect changes to the core model.
Core Database Changes
DatabseModule.kt: Added the StatsResponse type converter to the application database in provideAppDatabase()
StatsResponseConverter.kt: Added a type converter for StatsResponse.
Updated Pokemon Info DAO, Entity Mapper, and Entity Kotlin files to account for PokemonInfo's new stats fields.
✍ Explain examples

Apologies for the issues on the previous builds, I believe I was able to get spotlessApply to work correctly on my machine.


feat: Include URL into FetchOptions (box/box-codegen#549)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 92.394%

Totals

Change from base Build<PHONE_NUMBER>7:
0.02%

Covered Lines:
10252

Relevant Lines:
11096

💛  - Coveralls


Fix for Issue #274 with I.resizeWindow (nightmare)

Fix for issue #274. Didn't find a way for maximize to work, but this should help to at least resize windows.

Thank you! I didn't have time to look for that issue, it appeared to be very tiny but still I didn't make it. Thanks for resolving.


Untimestamp

depreciated feature
Every PR has to end with: "Can u pwease merge my pwull wequest senpai?!! UwU"

nyo


Fix extravolumes mount in cilium-preflight

Please ensure your pull request adheres to the following guidelines:

[X] For first time contributors, read Submitting a pull request
[X] All code is covered by unit and/or runtime tests where feasible.
[X] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

Fixes: #34961
Fixed condition to fix extraVolume
<!-- Enter the release note text here if needed or remove this section! -->

@tokarev-artem can you please add a "Signed-off-by: ..." line to your commit message?

Hi @squeed, since my previous commits without 'signed off' messages are already part of the history, it seems like I can't retroactively sign them without rebasing a large number of commits. Would it be simpler to recreate the PR instead? Am I mistaken?

@tokarev-artem, I'd suggest updating your local main to match upstream, then rebasing your current changes on top of that, squashing all the merge commits, and then making sure that the squashed commit has the DCO set properly (that's the "Signed-off-by" part).
That will give a much cleaner commit history as well as fixing the signoff issue.

@tokarev-artem, thanks for this change, but unless you're able to squash your commits and include the signed-off-by, this PR won't be able to move forward. Are you still interested in getting this in?

Sorry for the delay, got confused how to manage this mess with commits, opened another PR
https://github.com/cilium/cilium/pull/35386


⚠ Minecraft has degraded performance

In 3379ed6, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 1593 ms

Resolved: Minecraft performance has improved in 650b2e3.


feat: add simp_digits tactic

With the introduction of the simprocs reduceLE, reduceMod, reduceDiv etc. in Lean v4.6.0, now simp
can fully normalize typical Nat.digits calls that have concrete arguments. The situation is still a bit delicate
however, as we want to avoid invoking kernel reduction on Nat.digits, because the Acc-based ("well founded")
recursion in digitsAux can get bogged down.
This PR introduces a new tactic simp_digits tactic that contains only the simp lemmas and simprocs needed
to reduce Nat.digits. This allows proofs involving Nat.digits to be cleanly factored into two steps:

reduce Nat.digits via simp_digits,
do things that require kernel reduction such as simp_arith, decide, or rfl.

Maybe norm_num by itself is good enough: #10429


⚠ Bofrost.fr has degraded performance

In 0a76a08, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1754 ms

Resolved: Bofrost.fr performance has improved in 2013ed4 after 16 minutes.


Create Chrono Plugin

Game mode that restricts your account through RuneScape's releases

Please don't close and open new PRs on CI failures; just push new commits to your open PR

We recently removed jogl from our allowed dependencies for hub plugins as it didn't work well and caused a lot of crashes. We would recommend using lwjgl instead.


Remove version: Nikkho.FileOptimizer version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/nikkhokkho/files/FileOptimizer/15.00.2636/FileOptimizerSetup.exe/download
Status Code: 200
(Automated message - build 897)

Close with reason: Package still available.;