How to generate registered images using your pretrained model?

Thanks for your excellent work. I'd like to register some images by your model. However, I couldn't find the right code to use your pretrained model to generate the registered images. How to do that?

Hello, could you provide more details about the specific type of registration you're interested in? Are you looking for affine or deformable registration? Is it for a single modality or cross-modality, such as CT-MRI or T1 MRI-T2 MRI?

It is for a cross-modality registration. Your Cross-SAM model could be useful.

The registration code can be found in 'tools/regis_sam.py.' Please note that the pretrained model has been trained on abdomen T1 MRI (small FOV) and CT images. If you possess > 15 pairs of images, you can try to train your own CrossSAM model on your dataset.

I attempted to train my own CrossSAM, as per your recommendation, but I'm still grappling with some uncertainties.
My primary objective is to register the collected multi-modal data and subsequently utilize it for downstream tasks. In light of this, could you please outline the correct steps for utilizing CrossSAM? Should I employ the same dataset for both model training and testing if I should  train my own CrossSAM?
Furthermore, I've encountered difficulties in locating the files pertaining to DEEDs registration. And the original DEEDs project lacks the file "applyBCVfloat_aniso.cpp." If this file is essential, would it be possible for you to provide it?
Thanks for your help.

Definitely. There are some details in the training of CrossSAM. Here is the applyBCVfloat_aniso.cpp file.
applyBCVfloat_aniso.cpp.zip

No problem, thanks!


API: Add unit tests

Summary
Add unit tests for the API and make them autom with Actions.
Details and comments

[x] Create django unit tests
[x] Updated doc + readme on details
[x] Add unit test launching into Docker.dev
[x] Add django env into tox
[x] Add coverage report
[x] Launching django tests from CI

Closes #99

@IceKhan13 @AnsahMohammad ready for your review :)

sweet! let's CI pass checks and we are good to go!

BTW this is PR 100!

BTW this is PR 100!

Yeaaaaah 🥳 🎉

sweet! let's CI pass checks and we are good to go!

🏁


Modal: add padding to container and margin top to children container

Modal spacing update

Add 16px padding to container (x axis)
Add 24px padding to Header container (y axis)
Add 16px margin top to children container
Add 16px padding to footer container (y axis)

Proposed:

Implemented:

On smaller windows:

Sheet:

Test Plan
Manual run

@diegofontenelle thanks for the PR! @anniemochi could you verify from the design side?
Couple of notes:

We should also update Sheet which uses similar padding
The current change breaks scrolling (happens on windows with smaller heights). So the padding still needs to be on the outward component, it can just be less than before.

@diegofontenelle thanks for the PR! @anniemochi could you verify from the design side?
Couple of notes:

We should also update Sheet which uses similar padding
The current change breaks scrolling (happens on windows with smaller heights). So the padding still needs to be on the outward component, it can just be less than before.

Hi Chris! Thanks for reviewing, I have updated the PR and added new images showing how it looks right now. Let me know if this is good or you meant something else

Discussed with @anniemochi and @diegofontenelle: We will communicate this change internally first before merging

Making small changes to the padding of the children container on both components after reviewing the designs with @anniemochi

Making small changes to the padding of the children container on both components after reviewing the designs with @anniemochi

After further investigating the modal spacing adjustments we were proposing in relation to modal/sheet examples from the Monetization org, the Gestalt team has determined that we should not make those spacing changes after all (because all of the Monetization modal/sheet titles and content are left-aligned, changing the header/footer padding creates awkward alignment issues which would take a lot more effort to address).

After further investigating the modal spacing adjustments we were proposing in relation to modal/sheet examples from the Monetization org, the Gestalt team has determined that we should not make those spacing changes after all (because all of the Monetization modal/sheet titles and content are left-aligned, changing the header/footer padding creates awkward alignment issues which would take a lot more effort to address).


Consumption Saving Refactor - move examples to notebooks - tests boilerplate

I have removed the def main() code from ConsIndShockModel.py file and moved it to a example file.
This file will be present in the examples/ directory and parts of that file will be in the tests/ directory to create base level testing coverage of the module.
We also need to discuss what is exposed to the final user, for example ValueFunc, MargValueFunc and MargMargValueFunc are use by other classes to solve the model but I don't see users using this directly. Should this be exposed in the final documentation on readthedocs?
In this PR I'll repeat the same with all the other models and files in the ConsumptionSaving folder.
In this refactor I am also removing py2 specific dependencies in the ConsumptionSaving module as the py2 drop date is 15th Jan according to the roadmap.

related: https://github.com/econ-ark/HARK/issues/370
https://github.com/econ-ark/HARK/issues/440

[Build fails because it is running for py2.7 too]

@MridulS Sorry for the slow action on this one.
I merged updates from master.
It looks like there are some tests failing.

Ah the consumption parameters change is failing the tests, I'll update this PR.

okay this is weirder than I thought, something is breaking between import structure of python3.6 and python3.7
This works for py3.7 and py3.8

Because the 3.6 build was broken on an issue involving import of the ConsumptionSaving files, and because there was a merge conflict with master over HARK/__init__.py, I took the liberty of trying to resolve the merge.
I removed this line:
from HARK.ConsumptionSaving import *
which seemed to be opening up a new way to import ConsumptionSaving files.
If I understand correctly, that line lets you do:
from HARK import MarkovConsumerType
instead of
from HARK.ConsumptionSaving import MarkovConsumerType
I see now that you used the former style (from HARK import MarkovConsumerType) elsewhere in the PR.
So I have to apologize for breaking something new.
A change I would test myself if I could use the different build environments in GitHub actions myself is whether if all these references are changed back to from HARK.ConsumptionSaving import XXXXXX, does that then pass the 3.6 build test.

reverted changes to make this py2.7 compatible and make it v0.10.4 friendly


Cannot set vocabulary size for categorical set

When trying to set the vocabulary size for categorical set column
sentenceFeature = tfdf.keras.FeatureUsage(
    name="sentence", semantic=tfdf.keras.FeatureSemantic.CATEGORICAL_SET, max_vocab_count=10000)

I get the error
usr/local/lib/python3.6/dist-packages/tensorflow_decision_forests/keras/core.py in __init__(self, name, semantic, discretized, max_vocab_count)
    178       if max_vocab_count is not None:
    179         raise ValueError(
--> 180             "\"max_vocab_count\" only works for CATEGORICAL semantic.")

Rationale: I want to train a DF with a text column. This is demonstrated in the intermediate colab example. There I noticed a lot of out of dictionary items. I wanted to increase the dictionary to try to improve the accuracy.
CATEGORICAL_SET: 1 (50%)
	0: "sentence" CATEGORICAL_SET has-dict vocab-size:2001 num-oods:12816 (19.0292%) most-frequent:"the" 27205 (40.3941%)

This looks to be a bug as in the code to set the vocab_size we allow CATEGORY_SET as well. https://github.com/tensorflow/decision-forests/blob/d530fd85eccc79b47712b8728c9c06f263191a9a/tensorflow_decision_forests/keras/core.py#L192-L197
Only the error message is the problem
https://github.com/tensorflow/decision-forests/blob/d530fd85eccc79b47712b8728c9c06f263191a9a/tensorflow_decision_forests/keras/core.py#L178-L181

Category set Dictionary Size
Time
Accuracy
Eval Accuracy

1000
0:40
accuracy:0.767227 logloss:4.58354
accuracy: 0.7271

2000
1:40
accuracy:0.806738 logloss:2.41777
accuracy: 0.7546

4000
5:32
accuracy:0.861572 logloss:0.488691
accuracy: 0.8028

Hello StoyanStAtanasov,
Thanks for the detailed report :). Also happy to see that the categorical-set splitter works for you.
This situation was resolved in this commit and published in the TF-DF 0.2.0 release.
The following code should run fine:
sentenceFeature = tfdf.keras.FeatureUsage(
  name="sentence",
  semantic=tfdf.keras.FeatureSemantic.CATEGORICAL_SET,
  max_vocab_count=10000)

In addition, a global hyper-parameter max_vocab_count was also introduced to control the max_vocab_count of all the feature usages easily:
model = tfdf.keras.GradientBoostedTreesModel(max_vocab_count=10000)

Cheers,
Mathieu
Ps: Have you tried to tune the categorical_set_split_greedy_sampling hyper-parameter e.g. try with categorical_set_split_greedy_sampling=0.2 ?

@achoum Thanks for the fix! Sorry I did not have time to play with categorical_set_split_greedy_sampling.
Cheers,


Implement show errors only mode

Although I think it should be config in flycheck side, it is no harm to merge. Thanks.


Possible bug when specialized basetypes are used

To try and start with the highest score in smarty, I tried the AlkEthOH parm@Frosst smart used to generate the reference atom types... this should be a perfect match:
% atom types
[$([#1]-C)]   HC
[$([#1]-C-[#7,#8,F,#16,Cl,Br])]   H1
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]   H2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]   H3
[#1$(*-[#8])]   HO
[#6X4]   CT
[#8X2]   OS
[OX2+0$(*-[#1])]   OH

These were used at the basetypes in a smarty run for which the log shows correct matches (under ATOMS):
INDEX        ATOMS  MOLECULES                                                          TYPE NAME                           SMARTS
    1 :        244         38 |                                                               HC                      [$([#1]-C)]
    2 :        116         37 |                                                               H1  [$([#1]-C-[#7,#8,F,#16,Cl,Br])]
    3 :         33         22 |                                                               H2 [$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]
    4 :          3          3 |                                                               H3 [$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]
    5 :         68         42 |                                                               HO                    [#1$(*-[#8])]
    6 :        232         42 |                                                               CT                           [#6X4]
    7 :         39         30 |                                                               OS                           [#8X2]
    8 :         68         42 |                                                               OH                 [OX2+0$(*-[#1])]
TOTAL :        803         42

but no matches to reference atom types:
Atom type matches:
HC                                                               matches       H3 :        0 atoms matched
H1                                                               matches       HO :        0 atoms matched
H2                                                               matches       OS :        0 atoms matched
H3                                                               matches       HC :        0 atoms matched
HO                                                               matches       H1 :        0 atoms matched
CT                                                               matches       OH :        0 atoms matched
OS                                                               matches       H2 :        0 atoms matched
OH                                                               matches       CT :        0 atoms matched
0 / 803 total atoms match (0.000 %)

This seems like a bug.
smartytest_filt1_Frosst.10_0.1.log.txt
full log file attached.

I ran this test separate from Christopher this morning, and got different results. We just compared and I think our results can help highlight where the issue might be.
So first, My input base types were
[$([#1]-C)] hydrogen-carbon
[$([#1]-C-[#7,#8,F,#16,Cl,Br])] hydrogen-carbon-ewd
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]   hydrogen-carbon-ewd2
[$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])] hydrogen-carbon-ewd3
[#1$(*-[#8])]   hydrogen-oxygen
[#6X4]  carbon-tet
[#8X2]  oxygen-dival
[OX2+0$(*-[#1])]    oxygen-hydrogen

This resulted in the same initial atoms/molecule count:
INDEX        ATOMS  MOLECULES                                                          TYPE NAME                           SMARTS
     1 :        244         38 |                                                  hydrogen-carbon                      [$([#1]-C)]
     2 :        116         37 |                                              hydrogen-carbon-ewd  [$([#1]-C-[#7,#8,F,#16,Cl,Br])]
     3 :         33         22 |                                             hydrogen-carbon-ewd2 [$([#1]-C(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]
     4 :          3          3 |                                             hydrogen-carbon-ewd3 [$([#1]-C(-[#7,#8,F,#16,Cl,Br])(-[#7,#8,F,#16,Cl,Br])-[#7,#8,F,#16,Cl,Br])]
     5 :         68         42 |                                                  hydrogen-oxygen                    [#1$(*-[#8])]
     6 :        232         42 |                                                       carbon-tet                           [#6X4]
     7 :         39         30 |                                                     oxygen-dival                           [#8X2]
     8 :         68         42 |                                                  oxygen-hydrogen                 [OX2+0$(*-[#1])]

But it did atom type match:
Atom type matches:
hydrogen-carbon                                                  matches       HC :      244 atoms matched
hydrogen-carbon-ewd                                              matches       H1 :      116 atoms matched
hydrogen-carbon-ewd2                                             matches       H2 :       33 atoms matched
hydrogen-carbon-ewd3                                             matches       H3 :        3 atoms matched
hydrogen-oxygen                                                  matches       HO :       68 atoms matched
carbon-tet                                                       matches       CT :      232 atoms matched
oxygen-dival                                                     matches       OS :       39 atoms matched
oxygen-hydrogen                                                  matches       OH :       68 atoms matched
803 / 803 total atoms match (100.000 %)

@cbayly13 did another test where he named half of the atoms with their reference atom type symbol (H1, CT, etc.) and half with a different name. It matched everything that didn't have the atom type symbol as a name.
@davidlmobley - we identified the differences it looks like
@jchodera - David asked me to look into this so I got on to post the above results and assign myself. Would you prefer to fix it yourself or should we take it?

@bannanc :

@cbayly13 did another test where he named half of the atoms with their reference atom type symbol (H1, CT, etc.) and half with a different name. It matched everything that didn't have the atom type symbol as a name.

It sounds like you're saying there is a bug in parsing of these descriptor files (or in something downstream of that) where it is using the NAMES assigned to the starting types in a way that it shouldn't. Is that accurate?
We should take a look at this ourselves.

I bet the issue is just that a collision of names between SMARTY and reference means that the same graph node is referenced in the networkx graph creation code. We just need to append some sort of prefix or something to make sure both sets of nodes in the bipartite graph have unique identifiers.

Mad props to @bannanc for so quickly bringing the critical data to figure this out!


Add a search functionality to files

Enable to search for macros, analysis, tests..
Add search bar on top of files
Add Omnibar triggerable via a shortcut

Also add search functionality to model selection, see #60


Merging 1.1 into master and rolling v.next ahead to Pro 1.2 development

Merging into Master for Pro 1.1 Release


Server side hot reload

This patch enables server-side hot module reload on the existing master branch.  Note: this does not include any Webpack changes nor does it include any changes to ExpressJS unless required for HMR to work.  It simply reloads the existing changes when Webpack successfully recompiles the client code.  This is a refactor of the code from #791 to only include server side HMR changes.  Discussed in #773.
UPDATE: this branch was rebased and changes from #808 were removed.  It is now a standalone patch.

This is a cool solution.  Thanks for working it out @drewpc!
LGTM 👍

@jhnns I did some basic testing on memory leaks with regard to deleting require.cache entries (modifying this code)  and came to the following conclusion: my server side patch HMR solution doesn't do anything good or bad to resolve those memory leaks.  In other words, if memory leaks exist in the code being required delete require.cache[<module>] doesn't "fix" that.  Your solution of restarting a separate process would fix the symptom, but not the problem: memory leaks still exist.
The topic of memory leaks would be a great documentation item: patterns to avoid, ways to detect memory leaks, etc.  The page cycle of react-server would be the most vulnerable, especially since it is called for each HTTP request.  I'll have to look into this, but I see it as a different problem than server side HMR.

Added breaking-change to this, due the moving of the express middleware and express-state setup out of renderMiddleware. (Which is a change that has been overdue. Thanks!)

my server side patch HMR solution doesn't do anything good or bad to resolve those memory leaks. In other words, if memory leaks exist in the code being required delete require.cache[] doesn't "fix" that.

Of course, you're right. These memory leaks exists in the code because you don't write tear-down code if you don't need it. But usually you don't just delete require.cache, so it's not an actual memory leak in production because the module is not replaced in production. It just becomes a (potential) memory leak when the cache is deleted.
But besides the memory leak I rather worry about the unknown application state after modules have been swapped out without actually tearing down everything. Your application may behave strangely in development after a while (because of stale connections, etc.). I'd rather prefer to restart the server cleanly than having to deal with an unknown application state. That's why create-react-app is also using HMR just for CSS, because CSS can be replaced without introducing side-effects.
I've explained some details of HMR in this SO answer.


Fix string format in model.py

Hi!
Running into that Exception would raise a KeyError 👍

Is this an issue?  AFAIK most (all?) exceptions have __str__() defined, including FileNotFoundError.

ah sorry nevermind, I see.  Thanks for catching it :)


Added Custom password Length and copy button

Issue #135
here I added a custom input for the password 1-50 length  and a button to copy the text

Please check #135


French translation

Ok, so I would like to blame 786e26b4fe7dd9eb9a4dc795cab755c73a76f8aa and 34cae23af9d2426318e0a772b71277713f888a8f.
First about 34cae23af9d2426318e0a772b71277713f888a8f, Per design dates are capitalized in bootstrap-datepicker, so it makes no sense to have it lowered in French. Yes in French we should not put capitalize a month or a day inside a date or a sentence, but in this case it's neither of both cases. So it should keep the bs-dp design.
Then 786e26b4fe7dd9eb9a4dc795cab755c73a76f8aa, Once again, per design bs-dp used to not put dots, in all languages even it should be put in almost all, even English. So again it's a nonsense to have French with dots and not the others. Also the commit point an article which state "Recommendations" about abbreviating months. There is no official rules, so it is completely fine to stay at 3 letters. That said, if you want to stay on those recommendations its should be "Avr" not "Avril".
I'm not sure those changes are worth breaking the bs-dp design since there is no mandatory rules.

@Azaret there was a discussion about it in #1252, if you have a different opinion about it please create a PR with the desired fixes and then we can have a discussion with the other people who commented in #1252

Ok I didn't saw this PR, maybe I will rather comment there so we could maybe reopen this PR ?

Yeah discussion can continue in the PR and can be reopend or recreated if necessary


chore(main): release 1.1.0

:robot: I have created a release beep boop
1.1.0 (2022-11-28)
Features

更新路由导航 (e0a475d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/Rosy-DR-Code-Ling/releases/tag/v1.1.0 :sunflower:


Scalpels and glass shards can now cut like knives

About the PR

Title!
Why / Balance

Was brought up in relation to plushies but it makes sense they have the ability in general! I don't think this has any major balance implications, sharp things aren't super hard to acquire and the glass shard in particular is really slow.
Media

I spent so long trying to record but OBS has decided not to work.
Changelog

:cl: Beck Thompson

add: Scalpels and glass shards both now work as knives!

Does it fix issue where scalpels can't make baseball bat(but glass shard can?) 😆

i would remove it from glass shard, so you have to put some effort in it and make a shiv from it, which you can use as a knife

Makes sense I agree. Forgot about the shiv!

This will also extend to shivs because they inherit form scalpels, but that is good in my opinion.
You might want to mention that in the changelog.

Looks good to me, tested in-game fine
Updated your comment to include shivs

Thank you I forgot to respond and fix it (Midterms 😭) ❤


Create blog-template.md

Now the checks are failing and i wonder why 😱

@fakela: the markdown wasn't valide


Spring SecurityContext.getAuthentication() returns null

Hi
I am using Spring boot 2.x and graphql-spring-boot-starter 5.0.2.
I was able to implement oauth using Spring security and then apply authZ at the Graphql resolvers level. The whole thing works like a charm.
However, i am unable to retrieve the logged in user details within the resolver functions now. I tested the same on other Rest endpoints and there were no issues.
Below is my security configuration
@Configuration
@EnableGlobalMethodSecurity(prePostEnabled = true)
@EnableResourceServer
public class ResourceServerConfigurer extends ResourceServerConfigurerAdapter {
    @Override
    public void configure(HttpSecurity http) throws Exception {
        http
                .authorizeRequests()
                .antMatchers("/areYouOk").permitAll()
                .antMatchers("/hello").permitAll()
                .antMatchers("/graphiql").permitAll()
                .antMatchers("/graphql").permitAll()
                .antMatchers("/vendor/**").permitAll()
                .anyRequest().authenticated()
                .and()
                .httpBasic().disable()
                .formLogin().disable()
                .csrf().disable()
                .logout().disable();
    }
}

// Retrieving the user details from a RestController works
    @RequestMapping(path = "/hello")
    public String sayHello() {
        SecurityContext context = SecurityContextHolder.getContext();
        Authentication auth = context.getAuthentication();
        System.out.println("principal: " + auth.getPrincipal());

        return "hello, world";
    }

The below fails within the Resolver functions
@Component
public class UserMutations implements GraphQLMutationResolver {

    @PreAuthorize("hasRole('ROLE_ADMIN') or @resourceServerConfigurer.authorizeForUserCreate(#request, principal)")  // this works
    public CompletableFuture<User> createUser(UserRequest request) {
        SecurityContext context = SecurityContextHolder.getContext();
        Authentication auth = context.getAuthentication();
        logger.info("principal: " + auth.getPrincipal());  // NullPointerException is thrown

        ...  // Some biz logic
      }
}

Can someone please explain?

Perhaps it's caused by the fact that you're doing this on a ResourceServerConfigurerAdapter instead of on a WebSecurityConfigurerAdapter? You could give that a try.
Also: you're using a very old version of graphql-spring-boot. We're at 5.10.0 already now.

Yes, you are right. Just updating the version to latest worked. Thanks a lot for your help @oliemansm.
I am closing this ticket right away


[sysbench] when use sysbench for arana test, not success

What would you like to be added:
Why is this needed:
FATAL: `thread_run' function failed: /usr/share/sysbench/oltp_common.lua:432: SQL error, errno = 0, state = '00000':

sysbench oltp_read_write  --mysql-user=root --mysql-password=root --mysql-host=sharding-ip --mysql-port=3307 --mysql-db=s1_sbtest --histogram=on --report-interval=1 --time=300  --db-ps-mode=disable  --threads=300 --tables=250 --table_size=25000  --report-interval=1 --percentile=95 --skip-trx=on --mysql-ignore-errors=1062  --forced-shutdown=1 run

FATAL: `thread_run' function failed: /usr/share/sysbench/oltp_common.lua:432: SQL error, errno = 0, state = '00000':
FATAL: mysql_stmt_execute() returned error 0 () for query 'SELECT c FROM sbtest97 WHERE id BETWEEN ? AND ?'

2023-02-28T09:58:06.810Z	ERROR	mysql/execute_handle.go:95	get dataset error 533: invalid sequence, expected 0 got 1 (errno 2013) (sqlstate HY000)
2023-02-28T09:58:06.810Z	ERROR	mysql/execute_handle.go:76	executor com_query error 580: invalid sequence, expected 1 got 2 (errno 2013) (sqlstate HY000)
2023-02-28T09:58:06.810Z	ERROR	mysql/execute_handle.go:76	executor com_query error 563: invalid sequence, expected 1 got 40 (errno 2013) (sqlstate HY000)

对应错误的sql信息FATAL: mysql_drv_query() returned error 2013 (invalid sequence, expected 1 got 40) for query 'SELECT c FROM sbtest188 WHERE id=12610'

yes, i use sysbench test. Before use arana test, I use Xiaomi/Gaea test, have this bug also.

It seems that a concurrency bug of packet ephemeral policy in the protocol layer. I'm trying my best to diagnose this problem.

BTW, the Xiaomi/Gaea and Arana, both used some codes from Vitess as the MySQL protocol implementation, maybe it's the reason that we have the same bug, I guess. 🤣

Yes， Xiaomi/Gaea have this bug also.  vitess maybe have this bug， I don't test vitess， because it doesn't support auto read write split.

There's a backend connection leak problem, and it causes sending continuous requests (see the pic below).
It has been fixed in the #669, @dobet  please check out this PR, thanks.


When running from this repo codes it's throwing error SequelizeBaseError

I'm trying to add the classical login with user password by customizing but after pulling codes and running by building from source with docker I can see that it's already there. But some problems are raised so that it is not working here is the shot after successfully dockerized the source:
it seems like the issue is with a password.

What happened is that you moved from Google+ auth to classical auth in on the app side but not on the DB side. You need to have a 'password' column in your 'users' table so the app can use the classical email/password credentials.
running the script below will update the DB to the state expected by an updated app.
npm run updateDB


 does not accept html options

Correct me if I'm wrong, but in standard Rails forms, you can specify HTML options such as a class for a <select> element with the following syntax:
f.select :field, choices, {}, { :class => 'my_css_class' }

With twitter_bootstrap_form_for, I was expecting the syntax to be:
f.select :field, label, choices, {}, { :class => 'my_css_class' }

However that solution does not seem to work. Is that a known issue?

Turns out my server was just ignoring the changes to my file, and rebooting it fixed it. Sorry about this!

No problem. :) Glad to hear it was resolved.


Add a comment on why overflow handling is not needed

I think the 16384x16384 limit may not always apply when using "extended" WebP. We should double check

Closing in favor of #14


Media Gallery missing rounded corners #272

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #272
Test URLs:

Before: https://develop--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/tdziezyk/v2/textual-content-with-images
After: https://272-media-gallery--vg-volvotrucks-us-rd--netcentric.hlx.page/drafts/tdziezyk/v2/textual-content-with-images

Just in:

I talked to the team about the border radii. Let’s make it 8px for all the items in your list, for all breakpoints to maximize consistency, since I feel the visual impact is minimal […] can all be changed from 12px to 8px on all breakpoints

Could you remove the line in styles.css that sets --border-radius to 12px?

Done


Website UI

The Website's UI is not good enough..
also the UX is not up to the mark, People have been complaining that finding commands is difficult!
this issue needs to be solved asap..
issue credits to @KCmean.. plz take charge! Also anyone who needs to join plz cooperate and work with @KCmean.
His idea was to create a landing page having cards on section basis (e.g. Branches and Commits.. ) which redirect to another page having the cheats for the respective section only.

#25 #24 #22 these too coincide within this issue!

@KCmean take charge!!!

@aryan-parmar @KCmean aajao idhar

@aryan-parmar @KCmean any progress?

Can't we use the CSS for designing the website?

Can't we use the CSS for designing the website?

It's Css only, the file is in the Assets folder

Can you please elaborate more on @KCmean 's initial idea for UI.

The idea was to actually make a landing page having cards for various topics such that the the card redirects us to the cheatsheet of the respective topic only, for e.g. branching.. this would remove the problem faced by people to find out commands!
but since many people were saying that this would rather decrease the user-experience because all the commands wont be available on a single page, which is the main reason as to why cheatsheets are used...
this is the reason @KCmean's idea is dropped.. currently a better UI on the current idea is only preffered, but do you have any other idea? @subhans11

No idea at this moment.

Okay then you can continue working on improving the current UI, I'll assign you this task

My idea was basically to utilize more space and make website less scrolling. Like we can change orientation style and rather then using a search button a sideways navigation menu can work to traverse through elements or diffrent sections ... like for branching and its onflicts we can have a whole section name say 'all about branches' in our side navbar and we can work on it to improve it and make it better and at the same time responsive too .. so please look forward to this type of struture.

Take charge @KCmean

this topics can be listed as a part of our sideways navbar and then we can navigate our uder with help oif this

before after
still some changes are leftt but working.

Great one @KCmean 🚀🚀

any progress @KCmean @subhans11 ???

Changed the font of commands

Great

i think jet pro and jet pro mians would be good a good font type for the code lines

code ke liye toh courier hi sahi hai re


[WIP] Move to Roslyn's ExternalAccess.Razor assembly.

Roslyn is attempting to remove IVT's from their mainstream assemblies in order to avoid unintentional breaking changes for partner teams.
Have not updated tests yet. Waiting for changes to flow from Roslyn.

@tmat / @sharwell this is the prepped reaction for once we have packages containing our shim bits from Roslyn. Of course it's in a super WIP state but it'll give you some of an idea as to what we'll be doing to consume the changes.

Note for others; This should be a draft PR 😆

Closing in favor of another PR. Will have one out soon.


Add nodeSelector for tornjak

Tornjak only supports amd64 builds at the moment
To prevent deployment failures on clusters with arm64 nodes I have added a nodeSelector for tornjak frontend, and documentation for spire-server.
Didn't define the nodeSelector on spire-server because by default Tornjak isn't deployed and therefore doesn't require to run entire spire-server on amd64. So resolved it there using a documentation update.
Also see https://github.com/spiffe/tornjak/issues/219

On the fence on this one. Total +1 for nodeSelectors.
Just the frontend amd64 predefine. There's no way to turn it back off if the user does build their own multiarch images.
But that is likely to be an exceedingly rare case and the feature is marked experimental so incomplete. So its kind of ok.
So, I'm going to approve as is. But consider just making the frontend one a suggestion too rather then required. It would be less to undo later.

We have seen this problem when one of our interns was using Macbook Air and the architecture was not compatible with the pre-built image. This would require us to build a few images with various architecture support. We will fix it on the Tornjak level. No changes needed here other than documenting it for now.

Ok, then I suggest we make another pr removing the hard coded annotation. I forgot about the new macbooks using arm.


allow-clear with theme=bootstrap broken in 0.9.6

http://plnkr.co/edit/yzoAL0PRkOyGHeuMbGj2?p=preview

It does not relate to allow-clear, the bootstrap-layout is broken completely:
http://plnkr.co/edit/2mQ6S7w01mjKhWUMEf6b?p=preview
It was ok in 0.9.2

:+1:

:+1:

:+1:
The Demo Plunker linked to from the README.md should be updated to use the latest version of select.css and select.js. Then this issue is immediately obvious there.

Temp workaround for text alignment (probably a better way to fix this):

.ui-select-bootstrap > .ui-select-match > .ui-select-toggle {
    text-align: left;
}

Temp workaround for missing bootstrap styles (box-shadow, etc.):
Add css class form-control to button tag on line 2 of match.tpl.html

@BP18 Thanks. I just tried adding your two workaround adjustments for version 0.9.6 and it didn't appear to make any difference in the styling.

It appears that if the allowClear directive is included, there is a conditional ui-select-allow-clear CSS class added that does nothing (no usage in select.js or select.css)...
This is very frustrating.

Hi @abobwhite, I'm not using allow-clear, but my workarounds seem to work on the demo page.
Here's the Demo Plunker as-is other than upgrading to 0.9.6 so I can reproduce the problem:
http://plnkr.co/edit/TEt1lqLYSwHo30EQdLvr?p=preview
Here's the same Plunker but with my two fixes added:
http://plnkr.co/edit/M1A5BhAK7et987Bhgih3?p=preview
The padding is still messed up, but at least the text-align and other styles are better. Don't think this is really a proper fix, but hopefully it helps figure out what's going on.

@BP18 Thanks.
I'm confused - isn't this issue surrounding the allow-clear styling? allow-clear on both plunkr's yield the same styling issue. If you weren't talking about allow-clear, what styling were you trying to fix, exactly?

@abobwhite, in my first demo Plunker link with just the upgrade to 0.9.6, the Bootstrap theme control is messed up. Without focus on the control, the text is center aligned and the field does not have the other Bootstrap form-control css styles like box-shadow and transition: border-color ease-in-out.

The template did change from

to

could this be the cause?

@artkoenig, yes I think that is where it broke. Commit 6e94894.
Previously the button tag had the form-control css class applied, but with the new template it doesn't. This caused the shadow, transition, and other styles to be lost. The text-align and padding were being controlled by overrides in the select.css file, but that broke with this template change too.

Slightly tweaked my original fixes and submitted a pull request.

I tried to test your changeset, but the 'allow-clear' button is still broken. Is it supposed to be fixed by your changeset ?

That was my understanding as well - it sounds like there are two issues here. Broken bootstrap styles in general as well as broken allow-clear styling for bootstrap. I am most concerned about the allow clear but I'm sure both need to be addressed.

Yes, I think @abobwhite is correct about there being two (related?) issues here. I'm not using allow-clear and neither does the demo page, so my fix didn't address that part.

I played around with allow-clear=true and I see the second problem you are mentioning (the X/clear button appearing below the field instead of integrated). I think my pull request addresses the other styling issues and doesn't make the allow-clear issue worse, so hopefully it can be part of a complete solution.
Also, there was a pull request (#486) that was supposed to add allow-clear support to the bootstrap template, but it doesn't work properly.

Thanks for all the hard work already put in, a proper fix would be awesome.
tl;dr +1

+1 for this

Meanwhile you can use this little CSS trick:
.ui-select-clear {
    position: relative;
    float: right;
    top: -26px;
    right: 25px;
    border: none;
    background: none;
    margin-bottom: -26px;
}

With https://github.com/angular-ui/ui-select/pull/606 merged, can you confirm if all works ok now?

@dimirc That merge doesn't seem to fix the problem.  Still looks like the screen grab from the original post.

@JeromeLam you are right, seams that the problem is the way allow-clear was implemented at https://github.com/angular-ui/ui-select/pull/486
I'm checking the code and maybe reverting that PR and do a different implementation

@dimirc Looking at #486, I see the template for the clear button uses btn btn-default.  Using btn-link will at least get rid of the border box.  It may then just be a question of styling it using part of the CSS trick from tarlepp 5 days ago.

@JeromeLam based on @bampakoa comment I pushed PR https://github.com/angular-ui/ui-select/pull/667
Check updated plunker and I think now should be ok

@dimirc It's definitely looking better now!  I suggest we add btn-link to the class list for the clear link so there is no odd outline and box shadow when the control is active or in focus (at least on Chrome).
I've created PR #669 tor this and forked your plunker with the change applied.

Have you checked in firefox ? It seems that the clear button is still below the drop down, however now, it's on the right side.

FYI: That happens only when the input is so long it overlaps the X button.

Should be solved with #667


Support type=breadcrumb

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #148
Summary:
Please summarize the scope of the changes you have submitted, what the intent of the work is and anything that describes the before/after state of the project.
Add styling and a doc example that provides users the opportunity to use an auro-hyperlink with a type=breadcrumb attribute. This support was needed in order for hyperlink to be properly styled to be used in the new auro-breadcrumb component.
Type of change:
Please delete options that are not relevant.

[x] New capability
[ ] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

The changes that were done in this PR are no longer needed, as the Auro team decided to not make these changes to auro-hyperlink.


looks like my loops are not iterating for some reason can't find any other way to step through array, don't want to use a switch statement.

Challenge Profile Lookup has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

//Setup
var contacts = [
    {
        "firstName": "Akira",
        "lastName": "Laine",
        "number": "0543236543",
        "likes": ["Pizza", "Coding", "Brownie Points"]
    },
    {
        "firstName": "Harry",
        "lastName": "Potter",
        "number": "0994372684",
        "likes": ["Hogwarts", "Magic", "Hagrid"]
    },
    {
        "firstName": "Sherlock",
        "lastName": "Holmes",
        "number": "0487345643",
        "likes": ["Intriguing Cases", "Violin"]
    },
    {
        "firstName": "Kristian",
        "lastName": "Vos",
        "number": "unknown",
        "likes": ["Javascript", "Gaming", "Foxes"]
    }
];

function lookUpProfile(firstName, prop){
// Only change code below this line
  for(var i = 0; i < contacts.length; i++){
    if(firstName == contacts[i].firstName){
      return contacts[i][prop];
    }else{
      return "No such property";
    }
  }

// Only change code above this line
}

// Change these values to test your function
lookUpProfile("Akira", "likes");

Hi @chuckmun
Thanks for reporting this issue.
Here are some next steps that you should follow before we can help you with your query:

Check these troubleshooting steps first.
Please use the Chat Rooms or try looking through our forum for help with a specific challenge and any coding assistance.

IMPORTANT:
Please NOTE that this tracker is for reporting bugs and enhancements to freeCodeCamp's code base only.
Happy Coding!


chore: upgrade Node to 16.15.0 and NPM to 8.5.5

also switches to focal ubuntu distro in travis ci config
update node-sass to v7 since previous versions do not support ARM

Coverage remained the same at 0.0% when pulling c3b9cfb2235118a750918a5e90d7cdc5c68389f8 on pbredenberg:pb/upgrade-node-npm into dbdc13a454d8d75f256eb19fdce6c9a2809d7866 on silvermine:master.

@yokuze wanted to get your take on the node-sass upgrade.

@pbredenberg mind rebasing this now that #126 has been merged? Thanks!

@onebytegone done!


How to make a complete conversion from Scala to java?

How to make a complete conversion from Scala to java?

You use a java decompiler on the resulting byte code, sacrifice a goat on a full moon and pray to the gods of Alpacas.

Would that cause my generic information and document information to be lost?


Only the first 259 TCHAR's are displayed not 260.

Although the list-view control allows any length string to be stored as item text, only the first 259 TCHAR's are displayed. The last character is a null character.

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


Fix distorted picture

modify available space (width and height) with subtraction of two times the border (up en down, and left and right)

Thanks!


Request for support for pre-release sem versioning.

prerelease versions (X.X.X[-alpha.0], X.X.X[-RC.0], etc) are part of the SemVer specification. Any chance it would be supported here?

Would be an awesome addition, could you create a PR?


Fix typo in the vim help

Noticed those small typos while reading the help.

The hello text is generated from the README. Please make the change there


Update CHANGELOG.md and README.md for release 0.3.0

Changes for rel. 0.3.0
What type of PR is this?

subproject management

What this PR does / why we need it:
CHANGELOG.md and README.md for release 0.3.0
Which issue(s) this PR fixes:

Fixes #111
Special notes for reviewers:
Changelog input
CHANGELOG.md and README.md for release 0.3.0

@patrice-conil @RubenBG7 Could you kindly approve this last step for the release 0.3.0?


Update CONTRIBUTORS.md

Added my username in the CONTRIBUTORS.md

Hey @chetas411,
I will go ahead and merge the PR, because its important you are accredited for your work, however I cant approve it for Hacktoberfest. Please don't be discouraged by this, but I have to mark this PR as invalid for hacktoberfest.
Whilst adding your name to a Contributors.md file is perfectly acceptable and often encouraged, its not something we can accept as a contribution to Hacktoberfest. If we allow PRs like this to count towards Hacktoberfest they can blacklist the repo:

Thanks for your understanding.


Feature/boas 690 publishing api

Describe your changes

Issue ticket number and link
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[ ] I have performed a self-review of my own code
[ ] I have double checked this work does not include any hardcoded secrets or passwords
[ ] I have made corresponding changes to the documentation
[ ] I have provided details on how I have tested my code
[ ] I have referenced the ticket number above
[ ] I have provided a description of the ticket
[ ] I have included unit tests to cover any testable code changes

it should be merged to this branch feature/boas-676-preview-and-publish-page


Expose prepared statement metadata (column types)

Some generic apps don't control the schema, but still have to issue queries.
For example if my app receives an integer parameter 123  from a user, and although it knows it is an integer, but doesn't know the size of that integer on the C*/Scylla side, then it can't properly send it to the database without risking type mismatch.

Fixed in https://github.com/scylladb/scylla-rust-driver/pull/451


Feature: user interaction/multiple choice

While presenting, enable users to make a choice from a selection of items.
Via HHS this information could come back to the presenter and being used in a slide.

Hello, I tried to solve the issue.
This is what I did:
Implement a feature to allow the audience to make choices during a presentation and send the results back to the presenter using HyperHyperSpace (HHS). This includes updating the sharing component, adding new actions, and modifying existing actions to handle the new functionality.
You can review changes in this commit: https://github.com/lattaai12/SimplyEdit-SimplyPresent-6/commit/6924c15ccc4622778dac6d65903481889a61429e.

[!CAUTION]
Disclaimer: The commit was created by Latta AI and you should never copy paste this code before you check the correctness of generated code. Solution might not be complete, you should use this code as an inspiration only.

This issue was tried to solve for free by Latta AI - https://latta.ai/ourmission
If you no longer want Latta AI to attempt solving issues on your repository, you can block this account.


State of input is lost

Thanks for building this package, I would like to report a small bug AutocompleteTextbox should take value as a input just like the normal input so that the state is maintained.
https://github.com/agdhruv/react-ghost-text/assets/31410839/8b19b0a9-308f-4af7-9603-b8ed49204b38

Good catch! Will try to get to this in the coming week. Feel free to open a PR if you fix it in the meantime :)

I will give it a shot, The API shoudl be as simple as
  <AutocompleteTextbox
      className={customClass}
      value={input}
      getSuggestion={getSuggestion}
      onContentChange={onChange} />

@agdhruv Would be interested to make a PR of automcomplete text to https://github.com/maheshj01/pastelog since it uses a custom input component?

Didn't get you. Could you please explain a bit more?

This is kind of offtopic to this issue, I tried to fix this issue but I wasn't able to and I am trying to implement this auto complete feature in my app which uses a built in input component with few customizations. Since you know how to implement auto complete, I was wondering if you would be interested to contribute to my app to bring this auto complete feature. This is the related issue https://github.com/maheshj01/pastelog/issues/14 (last in the list)

Thanks for reaching out! Unfortunately, I don't have the bandwidth to contribute at this point. However, I just fixed the state issue. Hopefully, this package is now easy to integrate into your project?

No worries
Thats good to hear Thanks!


Publisher Base64 payload is not routable in IoT Hub

Describe the bug
We are using the OPC Publisher to collect information from devices and would like to route the information to different IoT Hub endpoints based on the payload we receive from the device. However, we noticed that the payload is Base64 encoded and the IoT Routing queries are not applied on properties in the body as described in the following article.
https://azure.microsoft.com/en-us/blog/iot-hub-message-routing-now-with-routing-on-message-body/
To Reproduce
Here is the message received from the OPC Publisher:
{
  "EnqueuedTimeUtc": "2019-10-03T17:58:46.6340000Z",
  "Properties": {},
  "SystemProperties": {
    "connectionDeviceId": "agitaretech-iot-test-gw",
    "connectionModuleId": "opc-publisher",
    "connectionAuthMethod": "{\"scope\":\"module\",\"type\":\"sas\",\"issuer\":\"iothub\",\"acceptingIpFilterRule\":null}",
    "connectionDeviceGenerationId": "637048513528304448",
    "contentType": "application/opcua+uajson",
    "contentEncoding": "UTF-8",
    "enqueuedTime": "2019-10-03T17:58:46.6340000Z"
  },
  "Body": "eyJOb2RlSWQiOiJucz0yO3M9VjAuNjMiLCJBcHBsaWNhdGlvblVyaSI6InVybjpldGgwOlJpdHRhbDpDTUNfSUlJX1VBX1NlcnZlciIsIkRpc3BsYXlOYW1lIjoibnM9MjtzPVYwLjYzIiwiVmFsdWUiOnsiVmFsdWUiOjIzLjcsIlNvdXJjZVRpbWVzdGFtcCI6IjIwMTktMTAtMDNUMTc6NTg6NDQuMDU1MDAwMCswMDowMCJ9fQ=="
}

The decoded body is:
{
  "NodeId": "ns=2;s=V0.63",
  "ApplicationUri": "urn:eth0:Rittal:CMC_III_UA_Server",
  "DisplayName": "ns=2;s=V0.63",
  "Value": {
    "Value": 23.7,
    "SourceTimestamp": "2019-10-03T17:58:44.0550000+00:00"
  }
}

The following messge routing query does not work in IoT Hub:
$body.NodeId = 'ns=2;s=V0.63'

Messages are not routed based on the condition above.
Expected behavior
Ability to route based on the $body properties.

This is related to the content type being set != application/json.   A quick fix could be to set it to application/json, but this a) could break other applications and b) will be broken in our next publisher release.
Since in future version of publisher as per customer ask we will be sending OPC UA binary (or OPC UA Json) and "Pub/Sub compliant" network messages from publisher, and both are not routable, we will anyway need an extra conversion module at edge side to convert to some simplistic, routable message format for this scenario.
As such I am tracking this item as feature request to add such sample module, instead of closing.

We decided to close and track feature work separately.


Powertoys is waking my PC from S3 sleep

Microsoft PowerToys version
0.37.0
Running as admin

[X] Yes

Area(s) with issue?
General
Steps to reproduce
Put PC to sleep, is it woken up by powertoys according to the system log as seen in the screenshot below.

✔ Expected Behavior
Powertoys shouldn't wake PC from sleep.
❌ Actual Behavior
Powertoys does wake PC from sleep.
Other Software
Not sure if this helps but this is my Windows Version

Thanks for the report.  Tracked in #11467


Feature: zend command line parameters

This feature makes it possible for the GUI wallet to start zend with custom parameters. A new menu option is created: Wallet >> Edit zend parameters for it. The parameters are stored in a configuration file zend-cmd-options.conf that may be edited via the GUI or (in emergencies) with an text editor.
GUI editing dialog:

If anyone wishes to try this feature without building from source, they may take the latest official wallet release and only replaced the ZENCashSwingWalletUI.jar with the one attached here (it is zipped):
ZENCashSwingWalletUI.jar.zipped.zip

Assigned @zennermac to help with testing since I'll be AFK shortly.

@tarrenj This feature is a fresh/clean implementation. There is no code copy/pasted from the other similar PR. There is thus no possibility of cherry-picking commits.

@zenportodev ACK, thanks for clarifying.  I'll try to test/review later tonight, but no promises.

Minor NIT: The Edit zend parameters... window appends 5 empty lines to the file whenever it opens.

If we merge this, we can close #110.

@tarrenj Thank you very much for your tests. If there are no further objections by others I will merge the pull request within a few days.
Some comment on the issues reported:

The Edit zend parameters... window dialog appends 5 empty lines to the file ... I do not think this is the case. The UI window text areas has a few empty lines if the current number of options is below 5, but this is just for aesthetics. The file zend-cmd-options.conf does not get extra 5 lines... I think.
Does not save this long string: #START...  The format of file zend-cmd-options.conf is that lines starting with # are comments (not loaded as parameters). If the user adds lines starting with # in the GUI they are also ignored. Only the actual parameters are saved in the file...
The window defaults to a size that does not show the full content: - This issue is not related to the current feature. The current feature does not change the main Windows sizing. This occurs on Windows systems when (typically on large displays) elements are scaled 1x-2x via the Windows settings. We have to do a better job of detecting this and calculating the Window size then... a point to be fixed some day.

@zenportodev You're right on all accounts.


Choose language in URL

My main language is not English, and my system is configured accordingly. But, when I learn git it's more didactic to me to learn it in English.
Could I choose the language as url parameter?
If it's not implemented then I propose this type of url:
https://learngitbranching.js.org/?lang=en

Hi @Daniel-at-github!
You can do it like this: https://learngitbranching.js.org/?locale=es_AR
Not sure what locale are you interested in, but, if it's implemented, that's the way to do it.
Hope that helps!

Thanks Matias for responding. For what it’s worth we try to automatically
set the locale based on the header values we receive in the request, but
oddly enough JavaScript doesn’t have access to request headers by default.
I use a third party system that simply returns the headers back, but that
API call is asynchronous and their service could be down. If so, we default
to English.
On Fri, Jun 22, 2018 at 3:16 PM Daniel-at-github<EMAIL_ADDRESS>wrote:

Thanks @mgarciaisaia https://github.com/mgarciaisaia ! I was looking
for https://learngitbranching.js.org/?locale=en
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pcottle/learnGitBranching/issues/470#issuecomment-399597000,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABFRnw17rQjCdIZHc6oq-TvCsiZpdkt3ks5t_Wy7gaJpZM4Ux00X
.

--
-Sent from Gmail Mobile


If so can you provide more details about your desired outcome and the potential bug you are seeing? Thanks!


Propose Logo/Icon

Hi @aslanyanhaik ,, I'm graphic disigner. I would like to know if you are interested that I make a logo for your project? If you allowed me, i"ll make logo for your project.

@mirzazulfan nice , app will look more beautiful.

Hi, I've designed a logo for your project, if you like it, you can use it. Tell me if you need some change or anything, i'll fix it.


Caveats are swapped

I believe that the caveats are swapped—that is, if I build without default paths it should tell me that component libraries are set up for me, and if I build with default paths then I would need to clone the git repository.
Thanks!

D'oh! Right you are.  Fixed.

Thanks!


Why aren't exceptions being caught?

This one will require some investigation.
Basically, exceptions are only being caught in the APDU ingestion phase.  Exceptions that I throw during deserialization and display phases are simply being ignored, and are not aborting processing as they should be.
Will need to understand how the exceptions are being implemented, and what sort of initialization or other steps I'm missing that are turning my THROWs into NOPs.

Chart of where exceptions are being thrown, and where they're not:
Methodology:  Putting:
PRINTF("THROW TEST\n");
THROW(EXCEPTION)

in various places and seeing if they cause an abort, as they should, (and if we get the printf output).
Note that processTx is where the TRY, CATCH block is set up.  So anything below that in the call stack we expect to handle the throw's correctly.

Call Stack:
Unit:
THROW?
PRINTF?

...:processTx:processTxInternal
bts_stream.o
✅
✅

...:processTx:processTxInternal:tryTlvDecode
eos_utils.o
✅
✅

...work in progress

Tidbits discovered during investigation of above:

Function ui_idle() (defined in main.c) is what initializes the app's top-level menu ("Use wallet to view accounts").  It is called at program outset from main() (prior to sample_main()), and, as the last statement of all the button handlers that "conclude" a user interaction.  (i.e. "confirm" or "cancel" for a Tx signature.)  This is the function that brings us back to the main menu screen.  It does not bring us back to function main() nor reset any context variables. In fact, we simply continue the APDU loop in sample_main().  (But if we need to reset any context variables... this might be a good place to do it.  From a UX perspective this is where we go back to square one.  This function might be misnamed.)

Function sample_main() (called from main(), defined in main.c) never returns.  (Except perhaps in extremely extraordinary circumstances — it looks like a THROW(EXCEPTION_IO_RESET) could bring us back up to main(), (possibly thrown as a result of a wake from sleep), but everything else is caught and put on the APDU buffer in sample_main()...)   So we just loop here forever, essentially.  Even across servicing of multiple user requests.

OS-provided io_exchange() called from sample_main() does some magic, and, at least some of the time (or probably all the time) is where the "event loop" is managed.  For example, the main menu screen won't actually appear on screen until the first time this runs.  This function sits and waits for APDU input during the APDU read phase (it may loop through a few times if the APDU is long) and returns quickly when USB data is there to be read.  After we finish ingesting the APDU request, handleSign() sets the IO_ASYNCH_REPLY flag and then when we enter io_exchange() again it won't be waiting for input, but also won't exit until a reply has been sent (e.g. when user confirms).  Some details still fuzzy.  But while we are "waiting" in this function at this stage is when all the UI stuff is displayed on screen, e.g. the transaction deserialization and display.  io_exchange is basically orchestrating an event loop, I think, and this is where calls to io_event() (defined in main.c) originate from.  These events may include button presses, "ticker" events, etc.  Again, this part still fuzzy so confirm anything written here.


[ doc ] note for strong passwords

Passwords using special characters need escaping. Otherwise authentication just fails because the variable only got assigned part of the password.

Maybe the credentials variables example can be updated to use single quotes?
That would avoid the need for escaping.
ZWIFT_PASSWORD='password'

Except if the password also contains single quotes. :grin:

Right, but it would be specific to the single quote case, and would still be better than the current example that isn't using quotes at all.

Doesn't this go without saying when things are stored in env vars?

I guess the question is whether we are expecting all users to be devs.

I guess the question is whether we are expecting all users to be devs.

Fair enough


get_rev_children() ?

In recipe-579138.py, there is a get_rev_children method in the depth-first search. This method does not work on any of my editors/REPLs and there is no documentation on it anywhere. Is this a typo? Or is something missing?

Works in python 2.7 and python 3.5
The method is from the Node class and simply returns the childrens of a node.
Using the following code in the main method :
root = make_test_tree()
print(root.get_rev_children())

produces the following output :
[Node: [b2], Node: [b1], Node: [b0]]

which is correct, according to how the test tree is built.

@CrumbleZ Would it be possible to see your entire script? When I run those two lines of code, I still get the same error message.

What is the error message you are getting ?
What editor are you using ?
How are you running your code ? (From within your editor, or manually from a terminal ?)


Find in current file command

This PR fixes #14836.
I did not read all of the comments on that issue since it's extensive, but I implemented @alexdima's suggestion in https://github.com/microsoft/vscode/issues/14836#issuecomment-258781578.
This adds a way to find occurrences in the current file.  To test, you can open the find widget, then click the new icon to find in the file.  I wasn't sure what icon to use.  The two I thought fit best were file or search.

I went with search, but I'm open to suggestions.

@rebornix, should I rebase to fix the conflict?

I agree that the icon choice here is pretty hard.
There were two feature suggestions in the original feature:

Open all results for 1 file in new search editor.
Select all occurences in current document.

Based on that:

Ideal icon for the other feature (#1) has to be new-file.
Thus we can't use a "file" related codicon, unless its a new file-search icon. 📄🔍

I did a writeup at https://github.com/microsoft/vscode/issues/14836#issuecomment-1383112522 suggesting a full UX path for this component.
To summarize:

I landed on expand-all or plus for this icon.
To bring this full-circle, I suggested logic to swap it with a search-clear-results icon that selects only the first visible search result again and re-enables the arrow icons.

Thanks for taking the time to review this @meche-gh. However, I'm not going to make any changes to this PR at this time. I still would like it merged at some point, but that will need to be done by someone from Microsoft and I haven't heard any feedback since I opened this about 1.5 years ago. If I hear back from someone there and it seems like they are willing to merge it, then I'll take the time to make updates.

Makes sense. This pull's assigned and I'm the review has pinged them already :) will give it a wait

Is this still relevant?

I still think it's a viable solution to #14836, which is still open. However, it would need merge conflicts fixed first. I don't want to spend time doing that until someone from Microsoft indicates that this PR had some likelihood of being merged.


The object type in swagger should be converted into dynamic Map object

If the parameter in the swagger API has a type "object" it is converted by the parser into separate class instead of the dynamic Map or, ideally, Map<String, NestedObject> (with NestedObject being an object that is in the "$ref" of the "additionalProperties" json).

@MsProfanum Hi
Converting any object to a map is not a good idea, because it can be a list or a primitive. Map support with additionalProperties should work correctly. Can you check this with your examples?


Make PartialOrd implementation canonical for NonNAN<T>

Fixes clippy error:
warning: non-canonical implementation of `partial_cmp` on an `Ord` type
  --> src/distribution/empirical.rs:14:1
   |
14 | /  impl<T: PartialOrd> PartialOrd for NonNAN<T> {
15 | |      fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
   | | _____________________________________________________________-
16 | ||         self.0.partial_cmp(&other.0)
17 | ||     }
   | ||_____- help: change this to: `{ Some(self.cmp(other)) }`
18 | |  }
   | |__^
   |
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl
   = note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default

This is the normal way of implementing PartialOrd when a manual Ord implementation exists because it ensures that they will return the same result.
This is technically a change in behaviour:
let ordering = NonNAN(f64::NAN).partial_cmp(NonNAN(1.0));

// before: None
// after: panic

But IMHO this is fine because NonNAN(f64::NAN) is unsupported anyways, right?

Might be better to derive PartialOrd instead, but the behavior change is good by me, it should not be constructed and it's a lot lighter than ordered_float::NotNan.
Since you're messing with it, would you rename to NonNan or NotNan to match PascalCase? It's not public, so this is just preference.


⚠ Bofrost.es has degraded performance

In 70d17ff, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1428 ms

Resolved: Bofrost.es performance has improved in fb49f90 after 54 minutes.


pathfinder_simd's use of SSE4.1 seems somewhat unnecessary

I don't have strong opinions here, and if SSE4.1 is all you care about then feel free to close. That said:

_mm_round_ps with _MM_FROUND_TO_NEAREST_INT should be the same as _mm_cvtps_epi32 (note: 1 t) followed by _mm_cvtepi32_ps unless overflow is a concern. I don't know the relative performance here, though.
_mm_test_all_ones and _mm_test_all_zeros seem to be usually used on the result of a packed comparison. Generally the way you'd do this is via testing the output of _mm_movemask_ps. Note that this doesn't answer the 'are all/no bits set' question, but for the result of a comparison you don't actually need that, and it's. I suspect this would perform better or equivalently, as the test_all_foo intrinsics already expand to a longer sequence of operations. That said, the API you offer here is arguably cleaner than it would be if you used movemask

I think these are the only operations using post SSE2 instructions. I could have missed something though.
EDIT: I missed that _mm_srlv_epi32 is used, which is AVX2, which renders this a bit moot I guess...

Ah, I missed AVX usage of _mm_srlv_epi32 in abs (#238). That's entirely unnecessary, as you can just just _mm_set1_epi32(0x7fff_ffff).

I agree, let's limit ourselves to SSE2.


[FEATURE REQ] Collect usage data of credential-free-jdbc plugin

Is your feature request related to a problem? Please describe.
The credential-free-jdbc plugin supports the integration between Azure AD and JDBC drivers, and the Azure Database for MySQL service supports collecting driver names, and subscriptions from their system. We should collect the usage data of credential-free-jdbc plugin.
Describe the solution you'd like
Try to find out whether we can write the driver name or version via JDBC. The goal is to set version and driver name, send these information to Azure Database servers, make sure these data can be retrieved from servers' BI system.
Describe alternatives you've considered
N/A
Additional context
Here's an example of how the driver name is collected: https://github.com/microsoft/mysqlnd_azure.
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

[x] Description Added
[x] Expected solution specified

MySQL NativeAuthenticationProvider supports reading connectionAttributes and sending it to the server.
https://github.com/mysql/mysql-connector-j/blob/dd61577595edad45c398af508cf91ad26fc4144f/src/main/core-api/java/com/mysql/cj/conf/PropertyKey.java#L97
https://github.com/mysql/mysql-connector-j/blob/dd61577595edad45c398af508cf91ad26fc4144f/src/main/protocol-impl/java/com/mysql/cj/protocol/a/NativeAuthenticationProvider.java#L657-L659
https://github.com/mysql/mysql-connector-j/blob/dd61577595edad45c398af508cf91ad26fc4144f/src/main/protocol-impl/java/com/mysql/cj/protocol/a/NativeAuthenticationProvider.java#L559-L583
So we can append something like this connectionAttributes=_extension_version:az-sp-4.0.0 to the connection string.

PostgreSQL's connection string supports ApplicationName, and the server log can print the ApplicationName when adding %a to the server's log_line_prefix.
This could be a solution, but it requires the users enable %a in log_line_prefix.

Research done, will implement in https://github.com/Azure/azure-sdk-for-java/issues/30351.


thenAnswerMany

I frequently run across the use case of needing to return multiple different responses based on invocation n, n+1. I found this snippet and thought it might be a good fit for this package. If you agree I'm happy to PR :)
extension WhenExtension<T> on When<T> {
  thenAnswerMany(List<Answer<T>> cbs) {
    return thenAnswer((invocation) {
      if (cbs.isEmpty) throw "No more answers available";
      return cbs.removeAt(0)(invocation);
    });
  }
}

then to use
when(() => mock.call()).thenAnswerMany([
  (invocation) => result1,
  (invocation) => result2,
]);

Reference stackoverflow

Hey man! Sounds pretty good idea. I would only change the name from thenAnswerMany to thenAnswerInRow to be more expressive.
Can you submit a PR?

Its published on 0.0.6! Thanks for your work @kmcgill88 .


R Tutorial Improvements

Closing in favor of cleaned up new PR

New PR https://github.com/YosefLab/scvi-tutorials/pull/24


Material cost

Can u please add an option, that would force you to paid materials when u building something through your mod? Like in the normal game
I think that this mod is really awesome, but i just don't want to cheat that much

What do you mean by paid materials?
That blueprints should be bought in the hub?
Or the fact that during the construction of the mod does not spend resources? If the second, then I already wrote about it

It will force to pay resources with no options, and will be released in the next few updates.
I can not prevent such a case of the players who want to cheat in game, so no options need to provide in game. Just force to pay when I finish the development work.

can we have an eta for when this will be added?

I think it won't be later than the end of this month.

thanks very much :)


Android Polygon Fill Layer display weird at tile boundaries on high zoom level

Describe the bug
Android release android-v11.0.1 (also 11.0.0) shows artifacts (rendering wrong color and pattern) when rendering Polygon fill pattern at tile boundaries. This does not happen on ios or maplibre-js.
layer style:
 {   "id": "road_area_pattern",   "type": "fill",   "source": "openmaptiles",   "source-layer": "transportation",   "layout": {"visibility": "visible"},   "paint": {     "fill-color": "rgba(216, 41, 173, 1)",     "fill-antialias": false,     "fill-opacity": 1,     "fill-translate-anchor": "map",     "fill-outline-color": "rgba(191, 41, 41, 1)",     "fill-pattern": "pedestrian_polygon"   },   "metadata": {},   "filter": [     "all",     ["==", "$type", "Polygon"],     ["!has", "brunnel"],     ["!in", "class", "bridge", "pier"]   ] }
To Reproduce
Steps to reproduce the behavior:

Initialize a map with maptiler style: https://api.maptiler.com/maps/streets/style.json?key=your_key.
Move center lon: -77.19602282105208, lat: 38.754177774253066
Zoom to level 20

Expected behavior

Actual Behavior

What device is this? I am unable to reproduce this (Pixel 3 simulator).


OLS-373: map stories against TODOs

Description
Map stories against TODOs for evaluation/token handler

/lgtm

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

Project coverage is 94.85%. Comparing base (4508b86) to head (6ae6775).
Report is 13 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #730      +/-   ##
==========================================
+ Coverage   94.83%   94.85%   +0.02%
==========================================
  Files          53       53
  Lines        1896     1904       +8
==========================================
+ Hits         1798     1806       +8
  Misses         98       98

Files
Coverage Δ

ols/utils/token_handler.py
100.00% <ø> (ø)

/retest

no ci impact, merging


tooltip in d3js area chart with interpolated values inside tooltip

I have d3js based area chart @ http://plnkr.co/edit/4EEe7EyGRRUH4lJXpHhr?p=preview & here http://jsfiddle.net/g30zhvy8/ where former uses datum and latter uses data, both have working code to display tooltip.
Firstly why the tooltip appears to have less zindex than chart's
Secondly how to show the values closest to given one's or interpolated one's in tooltip for a particular point in area chart. similar question been asked at several places.
This tooltip code works for remaining d3js flavored charts such as bar, pie, donut, line, etc

Please use Stack Overflow tag d3.js or the d3-js Google group for help. These issues are for feature requests and bug reports only.


When the number of characters in the TextBox control exceeds a certain threshold, the content becomes hidden.

Describe the bug
When the number of characters entered into the TextBox control surpasses a specific threshold, the content within the control becomes hidden, making it impossible for the user to view the additional characters.
Steps to reproduce the bug

Open the WinUi3 Gallery in TextBox control.
Begin typing characters into the TextBox.
Continue typing until the character count exceeds the threshold (e.g., 16000 characters).

Expected behavior
The TextBox should display all entered characters
Screenshots
Before Threshold is Exceeded:

After Threshold is Exceeded:

Once the character count surpasses the threshold, the content within the TextBox becomes hidden, and the user can no longer see what they are typing or previously entered text.
NuGet package version
None
Windows version
No response
Additional context
No response

I believe this is caused by a limitation in the underlying RichEdit control.


Remove extra byte in realm_string_t conversion

realm_string_t should never have been 0-terminated because it knows its own length. In fact, adding an extra empty byte will result in \u0000 being appended at the end of the string. I don't know how to test this since the only place it's used currently is for the password, but I've verified it manually by looking at the http request body.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
5.5%

Covered Lines:
393

Relevant Lines:
422

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
5.5%

Covered Lines:
393

Relevant Lines:
422

💛  - Coveralls

List<int>.toCharPtr is mapping to char* which needs null termination otherwise we can't infer the length.


IIS Web application hang up after multiple requests stuck on "Begin request"

We had a problematic scenario today which a .NET Framework 4.8 ASP.NET application hung up after queuing multiple requests stuck on "Begin request" state, on the __DynamicModule_SerilogWeb.Classic.ApplicationLifecycleModule module.
That same web application runs simultaneously on another 4 servers (webfarm), and we haven't had any issues on those at that moment. And just to be specific, the problem happened hours after the application startup.
We also noted that CPU usage went down when the requests started to queue.

After a while:

I have looked on the source code and the OnBeginRequest is where a simple stopwatch is added to the application context.
Is there a chance of concurrency on that part that might cause this behavior?

Really sorry, but I have no clue 😭

This project is no longer being maintained and we are going to archive the SerilogWeb Repositories.
see:

https://github.com/serilog-web/classic/issues/82


std4docker vs sdthirlwall images

Hi
I was using sdt4docker/raspberry-pi-cross-compiler, but now I get a message saying "This image has been replaced by sdthirlwall/raspberry-pi-cross-compiler". I tried updating my image to the new cross compiler, but my own Dockerfiles are now broken, because they are expecting Ubuntu Trusty and not Debian Jessie. Can you keep sdt4docker/raspberry-pi-cross-compiler active for people who are using Dockerfiles expecting Trusty? Or is there a very compelling reason to use the new image including Jessie and change my Dockerfiles over?

Or additionally have a docker tag that points to the latest version that used ubuntu trusty?

Hi Thomas,
Sorry for the inconvenience there.
The new version is definitely worth a look. It has a proper raspbian sysroot directory and an install-raspbian script which installs raspbian packages natively into the sysroot, using qemu and chroot.
That aside, I've created a new branch called legacy-trusty, and have set up a new autobuild from it that will build sdthirlwall/raspberry-pi-cross-compiler:legacy-trusty
The branch has been updated so that the rpxc script etc refers to the new image:tag names.
That image is building on dockerhub now. https://hub.docker.com/r/sdthirlwall/raspberry-pi-cross-compiler/builds/
Let me know how it goes.

This works perfectly :) Thank you so much.
I will have to take a look at the new image at some stage.

No worries. Thanks for using it!


Add option  to

If given within a <build> section like in
<build>
  <!-- ... -->
 <assembly>
   <createClasspath>run-classpath</createClasspath>
   ....
    <basedir>/opt/demo</basedir>
    <descriptor>assembly.xml</descriptor>
  </assembly>
</build>

the plugin will add a classpath to the given path within the basedir and add it to the Dockerfile. This file should also respect any filename mappings happening in the assembly.xml.
This can be done by re-using the classpath resolution used in the maven-dependency-plugin (unfortunately not easy reusable except when subclassing) and the MappingTrackingArchiver we are using for the watch feature. If this project is of packaging jar or war then it should be added to the classpath, too.
The use case is for flat classpath apps which come with a startup script which can use this generated file for setting up the classpath properly.

There is a super easy solution by using the maven-dependency-plugin:
<plugin>
  <groupId>org.apache.maven.plugins</groupId>
  <artifactId>maven-dependency-plugin</artifactId>
  <version>2.10</version>
  <executions>
    <execution>
      <id>attach-deps</id>
      <phase>package</phase>
      <goals>
        <goal>build-classpath</goal>
      </goals>
      <configuration>
        <prefix>.</prefix>
        <includeScope>runtime</includeScope>
        <outputFile>${project.build.directory}/classpath</outputFile>
      </configuration>
    </execution>
  </executions>
</plugin>

This will create a file classpath in target/ which will be automatically picked up by the docker-maven-plugin when using the assembly descriptor artifactWithDependencies (for the next version).

I've solved it with existing tool set like that
                                    <assembly>
                                        <basedir>/</basedir>
                                        <inline>
                                            <dependencySet>
                                                <outputDirectory>jars</outputDirectory>
                                            </dependencySet>
                                        </inline>
                                 </assembly>

this will copy all the dependencies to /jars folder
then start up script looks like
                                    <entryPoint>
                                        <exec>
                                            <arg>java</arg>
                                            <arg>-cp</arg>
                                            <arg>jars/*</arg>
                                            <arg>${mainclass}</arg>
                                        </exec>
                                    </entryPoint>

@osigida yeah, works, but leaves you with a random class path, i.e. the order is not defined and that might cause issue for conflicting jars (which, btw, is not so uncommon). The solution above will take the dependency ordering (as it it given in the pom.xml) into account by creating a dedicated classpath file.

You could use the same 'trick' if you use a shell exntryPoint which does a $(cat jars/classpath) instead of -cp jars/* (however you would need the shell).
Or use a Java base image from fabric8io/base-images which provides a sophisticated run script (from fabric8/run-java-sh) which takes this into account (+ you have hooks for remote debugging, installing an Jolokia and/or Prometheus agent, ...)


Update python slot generator.

To support MUSICARTISTS and MUSICPLAYLISTS.

I actually just did this!

Thanks for helping so much with this repo. I really appreciate it!

I have a fair bit more planned, so I'm just happy you're OK pulling changes :-P

I definitely am!


Update clipboard.js to 2.0.6

The only breaking change since v1.6.1 was the change to the constructor from new Clipboard() to new ClipboardJS() in v2.0.0.

Thanks! In the off chance that someone is using new Clipboard() in their front end or plugin, could we have a compatible constructor?

@ozh Are you referring to plugins/frontends which use the clipboard.js file that ships with YourLS rather than bundling/importing their own version?

Yes

@ozh I just finished manually checking every repository that is listed here and surprisingly wasn't able to find one which does that.
These are all the plugins and frontends which use the new Clipboard() constructor and how they import the required dependency:
Plugins

Expiry: Imports the clipboard function directly from share.js.

Frontends

∞²: Uses its own copy of clipboard.js.
Sleeky: Imports clipboard.js from a CDN.

This pull request can be safely merged without breaking any plugins or frontends.

Neat work! Thanks for taking the time to go through this. In the end I think the correct way would be to use the share.js init function like the plugin does.


Wrong access rights of pcmdi_metrics-v1.1.1-py2.7.egg-info in 2.8.0!

@doutriaux1 and @durack1  with my usual luck, I have found an unusual problem...
I have installed PMP in 2.8.0, just in case, with
conda install -n cdatm14 -c conda-forge -c pcmdi pcmdi_metrics
I have not used PMP, so I assume it is OK, but it has the following weird side effect: if another person uses my shared CDAT installation, AND wants to use ipython, then there is a huge traceback due to the fact that pcmdi_metrics-v1.1.1-py2.7.egg-info can be read only by the owner (the person who installed cdat, i.e me)
(cdatm14) bash-4.1$ ls -l /home/share/unix_files/cdat/miniconda2/envs/cdatm14/lib/python2.7/site-packages/pcmdi_metrics-v1.1.1-py2.7.egg-info
-rw------- 2 jypeter lsce 228 21 juil. 2016 /home/share/unix_files/cdat/miniconda2/envs/cdatm14/lib/python2.7/site-packages/pcmdi_metrics-v1.1.1-py2.7.egg-info

I have checked the content of envs/cdatm14/lib/python2.7/site-packages/ on 2 different servers where I have installed 2.8.0 and pcmdi_metrics-v1.1.1-py2.7.egg-info is the only file that is not readable by group+other.
I guess that ipython scans the content of site-packages when it starts (maybe that's one of the reasons it is so much slower to start than the regular python interpreter, and why I don't use ipython) and breaks when it can't access one of the files. This almost sounds like an ipython bug to me, but I won't take care of reporting it
Just to be sure, I have checked if some other files were having the same problem in 2.8.0, but it seems only files in PMP have this problem (you may want to check if I used the correct value for -perm below
(cdatm14) [jypmce@ciclad-ng envs]$ find cdatm14 ! -perm /g+r,o+r

cdatm14/share/pmp/obs_info_dictionary.json
cdatm14/share/pmp/disclaimer.txt
cdatm14/share/pmp/demo/pmp_input_parameters_demo1.py
cdatm14/share/pmp/demo/pmp_input_parameters_demo2_save_interpolated_clims.py
cdatm14/share/pmp/default_regions.py
cdatm14/share/pmp/pcmdi_metrics_table
cdatm14/share/pmp/graphics/vcs/portraits.scr
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_0.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/seasonal_mean.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/annual_mean.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/io.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xyt.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/zonal_mean.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xyt.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/meanabs_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/seasonal_mean.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/mean_climate_metrics_calculations.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/annual_mean.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/bias.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/io.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xyt.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xyt.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xyt.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/mean_climate_metrics_calculations.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/cor_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/meanabs_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/bias.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xyt.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_xy.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/rms_0.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/zonal_mean.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/pcmdi/std_xy.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/base.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/io/base.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/version.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/__init__.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/portraits.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/__init__.pyc
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/graphics/portraits.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics/version.py
cdatm14/lib/python2.7/site-packages/pcmdi_metrics-v1.1.1-py2.7.egg-info

@jypeter thanks for this feedback, when the conda packages are being built we might have to add in a perm wash to ensure that similar issues don't occur in the future.. It's the first time that we've received such a specific use case issue, some other folks using the PMP are doing so in a shared install env

Can you check what access rights you have for this file (and the others), ideally in a 2.8.0 version?
Also, the date of the pcmdi_metrics-v1.1.1-py2.7.egg-info file is 21 juil. 2016, which seems kind of old to me, but I don't know what it should be. Could it be that I got a wrong and outdated version?

@jypeter when I install for users here I always do some sort of rechmod to make sure users can read files. I don't think it's due to pmp.

Not due to PMP? Why is it that I have this access rights problem only with PMP (see my find command above), on 2 separate servers? I have installed PMP the same way I have installed other packages. Look for "PMP" in my 2.8.0 installation notes
Can one of you guys check the access rights of the mentionned PMP file(s) in one of your installation (where you have not done a chmod -R a+rX cdat)

@durack1 @zshaheen where is the pmp conda package built at? crunchy and ocenaonly have weird permission mask by dsefault, could that be it?

@doutriaux1 @durack1 Nightlies are done on Travis. I didn't build the conda packages for v1.1.1, but I usually use aims4, however I never worked on a shared env.

This issue was solved in 1.1.2 so closing


Combining Java & Kotlin fails

Any Kotlin code that depends on java code will fail. I will append an example shortly where the maven plugin works, but the gradle plugin fails.
Pom from: https://raw.githubusercontent.com/ebean-orm-examples/example-kotlin/master/pom.xml

https://github.com/tobias-/example-gradle-kotlin/tree/java_and_kotlin_problem
The base issue is that in Gradle, the Kotlin code is enhanced before the Java code has been compiled. Maven seems to do the ebean step after all classes have been compiled . This causes a lot of not found classes during Gradle build, but works with Maven.
This may have a workaround by doing some Gradle trick, but I haven't found it.

I've made a solution that works for my usecase, but there's bound to be a better way of doing it.
Either way, here: https://github.com/tobias-/ebean-gradle-plugin/tree/hacky_fix_for_java_and_kotlin

Better link https://github.com/tobias-/ebean-gradle-plugin/commit/41c96b26a64ecde60a93b6e9b211dc3b91cfb0fb?w=1

Ok, the above change looks like it should be submitted as a PR and included?

Ok, made the changes as per the commit above and merged that into master as part of 11.13.1
I'll close this (but would be good for @tobias-  to review if possible)


Click and drag cursor cannot release on Chromium(Wayland mode)

Bug or Regression?
Bug
System Info and Version
 System/Version info
Hyprland, built from branch main at commit 4842eb83b444418ad3fe1901d645dd02224989e5  (helpers: make shm_open() portable after 8bcccf9f0f0f (6471)).
Date: Thu Jun 13 20:20:14 2024
Tag: v0.41.1-2-g4842eb83, commits: 4820

flags: (if any)

System Information:
System name: Linux
Node name: Arch
Release: 6.9.3-2-cachyos-echo-lto
Version: #1 SMP PREEMPT_DYNAMIC Thu, 06 Jun 2024 06:05:56 +0000

GPU information:
64:00.0 VGA compatible controller [0300]: Advanced Micro Devices, Inc. [AMD/ATI] Phoenix1 [1002:15bf] (rev c7) (prog-if 00 [VGA controller])

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

Description
Clicking on a tab or button and dragging it under chromium does not release the cursor.
How to reproduce

start chromium with chromium --ozone-platform=wayland.
clicking on a tab or button on a web page and dragging it.

Crash reports, logs, images, videos
https://github.com/hyprwm/Hyprland/assets/39702328/4e721ae1-c02e-4402-a4d5-334b0971dc6a

this is a broken chromium impl it seems, at least for the home screen icon drag, as that is broken on sway too. The getting stuck is a hyprland bug.

Thanks.

Just tested it and the cursor is still not released.

it surely was released on my end. Make sure you're on the right version with hyprctl version

it surely was released on my end. Make sure you're on the right version with hyprctl version

Hyprland, built from branch main at commit 91fe58f8f278d126852877eadc87c50ca7b9b78d  (window: improve swallowing functionality).
Date: Sun Jun 16 00:20:41 2024
Tag: v0.41.1-18-g91fe58f8, commits: 4836

flags: (if any)

hm, odd. it surely works for me. Anything else you can find to trigger this? Some config opt?

hm, odd. it surely works for me. Anything else you can find to trigger this? Some config opt?

I commented out the chromium-flags.conf configuration and just started chromium with the chromium --ozone-platform=wayland command and it still doesn't release the cursor

The cursor is not released even if the text is selected and dragged.

Same issue in Helvum, including with latest commit.

it... works on my end but is really wonky, I'll try to look at it tomorrow

jfc wtf is gtk's problem????????????

no shit

also no it doesnt

should be fixed, please verify

should be fixed, please verify

It seems to be worse, before it was the current tab not releasing the cursor.
https://github.com/hyprwm/Hyprland/assets/39702328/3ca9a49a-58a6-4fc4-b266-3493a0c1507c

well helvum was fixed...

should be fixed

Helvum didn’t get fixed for me with the previous patch. Trying your latest one now that I’ve got a good dev env set up.

should be fixed

Thanks. It's works.

@vaxerski It still does not work in Helvum for me so I’ll open a new issue

Seems to have broken Firefox drag and drop of tabs..

For the people that are not using the -git package version, when are you going to release a new version? :)

The problem continues to reproduce even in the git version

Yes, it seems it's happening again for me as well
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069

flags: (if any)

Happening to me as well. When setting the ozone platform to X11 it goes away but you get those big gaps around the modal windows.
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069

flags: (if any)

it's fixed on git how many times

Yea, seems like this is happening to me too
$hyprctl version
Hyprland, built from branch  at commit 9a09eac79b85c846e3a865a9078a3f8ff65a9259  (props: bump version to 0.42.0).
Date: Wed Aug 7 19:17:10 2024
Tag: v0.42.0, commits: 5069

flags: (if any)

Uploading 2024-09-04 19-17-10.mp4…

IT IS FIXED IN GIT PLEASE STOP POSTING THAT IT HAPPENS IN 0.42

IT IS FIXED IN GIT PLEASE STOP POSTING THAT IT HAPPENS IN 0.42

oh sorry


[C++] How do we deserialize two continuous objects from one bulk of memory?

Say I have two bond structs lying continuously in a bulk of memory and I want to read them both.
I make an CompactBinaryReader from the blob of the memory.
bond::CompactBinaryReader<bond::InputBuffer> reader(buffer);
bond::Deserialize(reader, firstObject);

I can successfully deserialize the first struct from the memory using the snippet above.
But, how can I deserialize the second? It turns out that after the call of Deserialize, the pointer of the InputBuffer in reader is still at the beginning of the memory. This is because Deserialize takes a copy of the reader parameter:
template <typename Protocols = BuiltInProtocols, typename Reader, typename T>
inline void Deserialize(Reader input, T& obj)
{
    Apply<Protocols>(To<T, Protocols>(obj), bonded<T, Reader&>(input));
}

And the constructor of CompactBinaryReader also takes a copy of the InputBuffer parameter:
    CompactBinaryReader(typename boost::call_traits<Buffer>::param_type input,
                        uint16_t version_value = default_version<CompactBinaryReader>::value)
        : _input(input),
          _version(version_value)
    {
        BOOST_ASSERT(protocol_has_multiple_versions<CompactBinaryReader>::value
            ? _version <= CompactBinaryReader::version
            : _version == default_version<CompactBinaryReader>::value);
    }

So, seems there's no way that we can deserialize an object and continue from where it ends - the buffer point is copied and will be destroyed once Deserialze returns. And it seems there is no way to pass a reference of either InputBuffer or CompactBinaryReader to the function call. Is this the expected behavior, or is it a mistake? If it is expected, what is the right pattern to deserialize two continuous objects from the memory?

Does passing the input Reader wrapped in a std::reference_wrapper achieve what you want? E.g.,
#include <functional>

bond::CompactBinaryReader<bond::InputBuffer> reader(buffer);
bond::Deserialize(std::ref(reader), firstObject);

(It's been a while since I've worked with C++ and my memory of how template type resolution works is hazy on the details...)
If you have control over the stream that you "pack" multiple Bond objects into, you could add a length prefix before each Bond payload. This would give you other benefits, like being able to skip a payload or--possibly--seek over a corrupt payload. With this information, you could seek in input buffer or construct one for each payload as needed.
<little-endian-uint32-size-of-next-Bond-payload><Bond-marshalled-data><little-endian-uint32-size-of-next-Bond-payload>

I believe this has been answered, so I'm going to close this. If not, feel free to add another comment with your follow up questions.


Tracking issue for Kani's unstable async library, feature async-lib

This feature was introduced in #1659.

closing this in favor of #1393


Disable flaky test retry in flexCI

Close #8129

Jenkins, test this please

Jenkins CI test (for commit 91c2f4c697ccbb944ffd2e68b03b7d05ecfb7488, target branch master) succeeded!


Scalardb server sample

This is a  sample application using ScalarDB Server as the backend.
Please take a look!

@brfrn169 @Torch3333 PTAL when you get a chance!

@brfrn169 @Torch3333 Thanks for your suggestions. Please take a look again when you have time.

@feeblefakie Thanks for your suggestions. I'll start using the google style format plugin.
Is it more convenient for the user to have the schema tool already in the sample? Cause from the user's perspective it would be quick, no additional step of downloading schema tool from another page.