Redux integration

Hi,
this is probably not the place to ask but ...
I'm trying to get redux integration working with your Reducer.
I'v managed to do this but am unsure if it's the right approach?
const reducerCreate = params=>{
    const defaultReducer = Reducer(params);
    injectAsyncReducer(store, 'router', defaultReducer);
    return (state, action)=>{
        store.dispatch(action);
        return defaultReducer(state, action);
    }
};

So from the code above I'v created a store, combined a couple of reducers and then added the defaultReducer via the store.replaceReducer method ( inside injectAsyncReducer ).
This seems to work nicely, and the store.dispatch is dispatching the correct actions and state is being updated.
My only qualm here is that return defaultReducer(state, action); seems to be doing the same thing as the dispatch ..
Maybe an example on the best practice for a redux integration wouldn't go amiss ?
Cheers,
Stephen.

Having a very similar issue here. The awesome thing about this router in 2.x was that it made so much sense where we easily were able to hook into each route change. Now, the integration doesn't seem to work as expected. Actions.BEFORE_ROUTE and the other actions made so much sense that we could easily hook into them in our reducers.

@maitriyogin Yes, looks like it is duplicate.
@maitriyogin @crockpotveggies Have you looked at Example project to see integration? It doesn't use store.dispatch and see all your actions there....

@aksonov i'm having a similar issue as @maitriyogin. While the example shows how to undertake actions on router changes it does not show how to take the default reducer and, for instance, use it with combineReducers to update the store state when using multiple reducers. I would like (expect) to be able to connect() the Router and then do something like this:
import { Reducer as routerReducer } from 'react-native-router-flux';

const rootReducer = combineReducers({
  application: applicationReducer,
  auth: authReducer,
  router: routerReducer,
});

export default rootReducer;

But that won't work. Instead it will complain about the initialState being empty.

Hi tijs,
that's kind of what I was hoping for, in that it would follow the same pattern as react-redux-router ...
I got round this by adding the reducer after it's created with the replaceReducers function injectAsyncReducer(store, 'router', defaultReducer);
combineReducers
export default function createReducer(asyncReducers) {
  return combineReducers({
    login,
    ...asyncReducers
  });
}

function injectAsyncReducer(store, name, asyncReducer) {
  store.asyncReducers[name] = asyncReducer;
  store.replaceReducer(createReducer(store.asyncReducers));
}

const reducerCreate = params=>{
    const defaultReducer = Reducer(params);
    injectAsyncReducer(store, 'router', defaultReducer);
    return (state, action)=>{
        console.log("ACTION:", action);
        store.dispatch(action);
        return defaultReducer(state, action);
    }
};

with reducerCreate being fed into the router
 <Provider store={store}>
        <Router createReducer={reducerCreate}>
....

this works but is kind of messy.
/Stephen.

Yeah that does seem a bit ehmm.. roundabout :)

I'm having the same issue. Why is it complaining about initialState?

@maitriyogin With your hack are you seeing all the Actions provided from react-native-router-flux? I'm only seeing the push & BackAction in my logs.

Here is my workaround:
Create the reducer (Router.js file):
import { Reducer } from 'react-native-router-flux';

// needs to fake params to pass sanity checks
const routerReducer = Reducer({initialState: {key: true}, scenes: true});

export default function(state = null, action) {
  if (action.type === 'RootContainerInitialAction') {
      return action.initialState;
  } else {
      return routerReducer(state, action);
  }
}

The default Reducer is a wrapper to set the initial state of the inner function returned, which is the actual reducer. I will deal with the initial state with the RootContainerInitialAction (an action passed to initialize by default), so I just mock an object to pass the sanity check.
But now you can go do:
...
import router from './Router';

export default combineReducers({
  awesomeReducer,
  ...
  router,
});

On the router component:
import store from 'path/to/store';

const createReducer = (params) => {
  // initial state
  const { initialState , scenes} = params;
  const routerInitialState = {...initialState, scenes};
  // dispatch actions
  return (state, action) => {
    // dispatch initial state of router
    if(action.type === 'RootContainerInitialAction') {
      action.initialState = routerInitialState;
    }
    store.dispatch(action);
    return store.getState().router;
  };
};

I'm using the intended reducer to actually dispatch the router action, and never used the state because the redux store is the source of truth (also, don't do it twice as which I think is the approach of @maitriyogin).
Unfortunately, I was not able to use the connect function of 'react-redux' because I need to dispatch action and then to return the state (and I think is not possible to do it even with mergeProps). So I grabbed the store.

I'm not using the package as intended, but I did not find entry points to do it better. Maybe if the Router component should have these entry points (custom getRouterState instead of hard coded as this.state, custom function to call to dispatch action instead of passing the reducer to the Router, etc.)

@deoqc - what do you think about this solution: https://github.com/aksonov/react-native-router-flux/pull/531 ?
Right now it only dispatches the FOCUS action, but it could be extended to dispatch all the actions.

@lynndylanhurley your objective is to find the current screen? Can't you find it from the router state? If so, you could just find it from the now router state in the redux store (if following my approach).
I think if you decouple the the dispatch and the state from the router, and having a default reducer and some other helpers (like getting the current state from router state), you could be more generic.
The current approach seems to be independent of flux implementation, instead of trying to have entry points to integrate with other implementations (redux, fb flux, nuclear).
Don't think will be always possible to seamlessly integrate with them all (like nuclear require state to be immutable), but maybe a PR that could integrate redux, and shed some light on how do it for other libraries (like, how to convert the immutable state back and forth for nuclear) would be good.

@deoqc - my app just needs to know which screen is the current screen. In v2, this was possible by listening for the BEFORE_FOCUS and AFTER_FOCUS actions. None of the actions dispatched by v3 provide this information. The FOCUS_ACTION action from #531 provides exactly that information.
I hear what you're saying about creating a framework-agnostic system, but honestly I don't care about that at all. I don't think that matters to anyone in practice. We're tired and we just want this to be easy.
#531 is unobtrusive, it can't possibly break anything, and it makes redux integration as simple as possible.
If you're worried about continued maintenance, I was the one that implemented the original BEFORE_FOCUS and AFTER_FOCUS actions. This is me trying to continue maintenance on a feature that my apps depend on.

Mostly agree with you @lynndylanhurley. Just thought that this PR is redux specific and only integrate with one feature, so maybe it would not conform with the purpose of the package creator/maintainer (but it is just a assumption and I came across this package few weeks ago, so you surely know better).

@lynndylanhurley just saw that to get the current screen is easy (from Reducer.js):
function getCurrent(state) {
  if (!state.children){
        return state;
    }
  return getCurrent(state.children[state.index]);
}

So, with my workaround just need getCurrent(store.getState().router). It just works, if you did the workaround to begin with, and it gives you more integration with redux (the router state is actually in the redux store, and every change of state passes to it).
If the router actually got it state from the store by design (like this.state = getRouterState || {}; in router instead of passing it in the createReducer) and if the Reducer had minor adaptions to not feel such hack solution, wouldn't it be better for you?
I think we can get in a unobtrusive solution; really integrate with using combineReducers, which seems people that want to integrate with redux tries to do; and solve problems that we didn't anticipate (like yours need to get current view with just getCurrent).

@deoqc - I appreciate you efforts in trying to help me. Your solution does work.
I made #531 in an effort to reduce the amount of glue-code in my app, but I didn't realize how much the React community seems to really like glue-code. So I'd like to offer #537 as a compromise.
#537 should work as the createReducer system intended.

Hi guys, I just updated the README to make redux integration more clear. Please take a look and let me know if it can be improved:
https://github.com/lynndylanhurley/react-native-router-flux#reduxflux

@lynndylanhurley - This is seriously awesome! Thanks so much for doing this. +1 +1 +1 +1 - MERGE IT @aksonov !

@caledhwa thx!! (^^,)

this is seriously awesome!
Huge thanks, I'm sure the community is gonna love you for this...
I'm still a bit confused as to why the RNRF Reducer is needed?
In the reducerCreate function why can't one use the reducer created in step1 ?
I'll try this out and hopefully it'll make sense.
Smiles,
/Stephen.

I don't completely understand reducerCreate's intended purpose. But as far as we're concerned, it's purpose is to dispatch the action so that our app's reducer can consume it.
In other words, RNRF has nothing to do with redux. If we want to dispatch actions to redux, we must do it ourselves manually using the reducerCreate hook.

@lynndylanhurley
truly thanks to you.
lots of folks are confused and keep asking about this until you.
you established a shortcut for all of us can focus on how to improve this repo instead keep away from trying to figure out what is the proper way to integrate with redux.
best regards.

@zxcpoiu - your comment just made my day. happy to help :crocodile:

all, @lynndylanhurley, @zxcpoiu
I've added dispatch call (if it exists) within Actions now to make redux integration much easier (without needed to create custom reducer). So now you could see all Router actions in your redux reducers with zero effort (you need just to connect Router).

That's fantastic!! Thanks @aksonov!!!

So, with the description on the README and the changes that have been made here to help out with REDUX, can we close this issue @aksonov? Just thought it might reduce noise.
The only thing I'd love to see is some changes to the TYPE names of the actions. I implemented @lynndylanhurley 's approach to hook up Redux and it works fabulously. It'd be great to have the action types be called RNRF_PUSH, RNRF_FOCUS, etc. so I could much easier differentiate the actions amongst my redux actions. If you're interested I could dig in and do a PR or something. Anyhow, I love this library and want to see it continue to have success! Go team!

Is it possible to consolidate this conversation into fresh doc posted on master?  After re-reading a few times, I'm still not 100% sure if what I got working is the best way of doing this with redux.

@allenmanning if you follow the steps in the redux section of the current README it works like a charm!

Are you talking about master or https://github.com/lynndylanhurley/react-native-router-flux#reduxflux

I think it is README in current repo
https://github.com/aksonov/react-native-router-flux/#reduxflux

Thanks.  Great project.

Closing it for now. I've also added possible alternative to redux, react-native-reactive library to docs.

I see that this discussion is closed, however, related t o this discussion, I was wandering if it is possible for the Router to use the store on both directions - dispatching its actions to the store AND subscribing/reading its state from the store?

Hi @lynndylanhurley follow your documentation
https://github.com/lynndylanhurley/react-native-router-flux#reduxflux
Made a demo as per your instruction:
https://github.com/krishbhattacharyya/Testroute
Not working for me . Am I done something wrong here?

@lynndylanhurley Hi,
I see that the actions are dispatched and arrive in redux reducers, that's great.
But if I want to use redux devtools to time travel and it seems that react-native-router-flux ignores the changes in state and nothing happens.
Basically our question, what do we have to do to make react-native-router-flux render state changes to enable time travel and generally control of navigation via state?

But if I want to use redux devtools to time travel and it seems that react-native-router-flux ignores the changes in state and nothing happens.

Basically our question, what do we have to do to make react-native-router-flux render state changes to enable time travel and generally control of navigation via state?

I'd be also interested


[Relay][Pass] SimplifyCastLike/Cast and ConcretizeFullLikeRewrite rewrites for SimplifyExpr

ConcretizeFullLikeRewrite: full_like -> full when known shape
SimplifyCastLike/Cast: cast(data, dtype) or cast_like(data, dtype_like) -> data, when the input and output have the same dtype

@yzhliu @comaniac @altanh

Thanks @hgt312 @altanh


get_abi for collection should deep convert to abi

for example, iterable<IKeyValuePair<hstring, IInspectable>> should return ComPtr<IIterable<IKeyValuePair<HSTRING, IInspectable*>*>>
Due to some templating assertions the return cannot be converted using ComPtr.As() so the value can't be used

The get_abi function is used for interop, the ability to cast a C++/WinRT type to some other ABI representation. It might be a Windows SDK type, a C++/CX ref type, or something else. Each of those projections will have their own representations of IIterable, IKeyValuePair, hstring, and so on. The next major update of C++/WinRT goes further and strips out even more of the ABI type information so that interop is easier still. You should thus not assume anything about the result, other than it is an ::IUnknown* to an implementation. Our focus is to provide a richer projection that gives you fewer reasons to use the ABI and a simplified process for interop when that is needed.


update rpi.md

Silly mistake. I think it looks better now. I also added to the mkdir command.

Thanks. In future, you don't have to open new PRs when you fix something: the new commits automatically update the PR.

Scott,
Thanks. I was wondering about that. Sorry, I’m new to git, but I will get the hang of it.
Greg

On Jul 2, 2016, at 9:42 PM, Scott Leibrand<EMAIL_ADDRESS>wrote:
Thanks. In future, you don't have to open new PRs when you fix something: the new commits automatically update the PR.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/openaps/docs/pull/280#issuecomment-230132061, or mute the thread https://github.com/notifications/unsubscribe/AS79bS2xTx3qtmgnwBPIZG-gNhF-cUqGks5qRyGXgaJpZM4JDyQD.


Docs: Add subnets to the left nav of the documentation

Added getting started guide for Subnets.
Files changed:

sidebar.js
subnets/getting-started.md
subnets/overview.md

Changed introduction -> overview in the Clarinet files to keep the product consistency.

The overview.md, trust-models.md, getting-started.md content looks good to me.
The how-to-guides, however, I think don't make sense here: each of these how-to's is actually just a couple steps from the NFT_USE_CASE.md, and the steps all depend on each other, so the how-to's don't make sense by themselves. This could be solved by just making this part of the docs a "NFT use case walkthrough" or similar. Otherwise, the docs will need to be updated so that these things make more sense individually.
But at a high level, the subnets repo is still under active development, and some of this content is likely to need to change. This content all looks directly derived from the MD files in the subnets repo, is it possible to build these doc files from that repo? Or, just ensure that the files are actually identical, so that any updates later are easier?

Thanks for your quick response. I will put together all the how-to's into one guide.
We recently pushed a plugin that pulls remote content to docs. So, for your question yes we do have the option to pull the content from the Subnet repository to docs without duplication.

Closing this PR as @trichards-hiro will come up with a new PR.


Support custom geometry

Adds a <custom> component which supports rendering custom geometry. Useful for testing and also since some geometry likely can't be made declaratively

Thanks for approving, unfortunately I can't merge as the format check and type check github actions are failing

This may be related to the recent issue with node 22.5.0, going to merge for now


gamepad not moving the player

In advanced mode, My gamepad can not move the player even though it's detected and recognized. But, if i set my keyboard to IC_none in script IC_Action_ext, my gamepad can move the player. If i set back to IC_wasd or IC_arrow, the player only moved by keyboard, not gamepad.
The strange is, jump and button action works fine with my gamepad and keyboard.
Is it a bug? Ist it intentional? or do i have to find a workaround in my script to get my keyboard and gamepad action movement both work at the same time.
I just recently learning coding so thank you very much for creating this library. I learn a lot on how you code this library thing.

Thanks.  I'm happy to help.   I really wish YoyoGames would have solved this for us, but someone had to step up.  Since I needed this kind of a library, I made it.
I am aware that on Linux there may be problems with the sticks/axis.  On linux, for example, the axis center is 0.5, 0.5 whereas on the Windows side it is 0,0 .. I'm filing an issue about that right now.
I need more information to help you.  Please tell me what platform you are testing on, what controller/gamepad you are using,    Also, test in Diagonostic Mode to see what your gamepad is actually registering.
and if you can sync up that's probably a good way to determine if its me (an issue with InputCandy), user error (your bad), or something else.   If you would like to sync on our Discord (invite is at Lostastronaut.com) we can talk through your issue over a video call.  Look for user Retrofriends

hi,
Thank you for the quick response.
I use PS4 Dualshock 4 and using iMac 2015 with MacOS Monterey  12.6.5.
my Gamemaker version is 2<IP_ADDRESS>
Here's the list of order button i press:
in diagnostic screen
L2
L1
R1
R2
dpad right
dpad up
dpad left
dpat down
Circle
triangle
square
cross
left analog
right analog
mouse right click
arrow right
arrowup
arrow left
arrow down
D
W
A
S
space bar
shift
in game screen
space bar
arrow right
arrowup
arrow left
arrow down
shift
dpad right
dpad up
dpad left
dpat down
circle
cross
here's the link for the recording. I'm sorry the size is too big.

i hope this is enough.

i dont see the recording, could you upload it to youtube?

i'm so sorry.
This is the Youtube link for the recording
https://youtu.be/AM6QiJPFI7M
I tried using fire limit and it shows error.
i change the line in 2036
from this:
if ( fired_times <= fire_limit ) return true;
to this:
if ( fired_times <= action.fire_limit ) return true;
and the fire limit working. I hope that helps.

I took your suggested fix for line 2036 of MatchSignal.
I'm in the process of experimenting with InputCandy's directionals in my own game for the first time.   The idea was to use MatchDirectional() but perhaps this was incomplete?  I've had some trouble using MatchDirectional in my own game, but have not got a chance to delve too deeply yet.
Also, I've noticed multiple Check* functions seem to be "incomplete" for example CheckAction and CheckActionReleased also aren't doing what they should, or I don't remember how to use them.  It's possible they need to be removed or replaced.  Case in point I wanted to force an action to trigger only when "Released" using CheckActionRelease but this did not do what was expected.  I will file an Issue for this.
I'll experiment and let you know, but here are some things to try and report back on:

Using my genuine XBOX Series S/X style controller, I was able to move the guy using the dpad.

I modified my o_Player object in InputCandy in the following way:
a) Set moving=false; in Create
b) removed "var" next to moving in Step
c) added the following code to Draw:

if ( moving != false ) {
	draw_text(x+16, y+32, json_stringify(moving));
}

The above code allows you to view any signals.
You mentioned you are using a PS4 Dual Shock controller.  I know that PS3 controllers do not register like the XBOX one does.  Gamemaker's input is perfect but only on the XBOX controller (though it has issues with the left trigger being an axis and not gp_shoulderrb or whatever), so a PS4 may be registering other controls.  Try using the "Test on Simulator" option to further investigate.

For the reason that not all controllers work out of the box, things like IC_dpad aren't going to work without loading an SDL profile, or by letting the user configure the controls specifically for that controller.   I will have to add support specifically for the PS4 in the future.  I'll have to run upstairs and grab one, since I already detect the PS3 controller, N64-USB knockoff, XBOX/XInput controller.  It's very hard to detect the VCS controllers except by hardwiring the axis.  In truth, you really only need to support the top gamepads, or let the user remap on Steam using a third party app, and assume its XBOX style.  The other reason InputCandy exists at all is to allow for PC games to permit key remapping.
The way I'm fixing this so far in my own game is to support specific controllers outside of InputCandy, letting InputCandy be a fallback.  For example, I directly support the XBOX and VCS controllers, allowing the user to switch to one of those profiles OR the user can switch to "Custom" which is InputCandy.  I'm also using InputCandy's screen to let them remap players to devices, access the simulator, and configure custom controls.
For example, from your video, I noted that the PS4 DS controller registered multiple signals for your dpad (as a hat and also as a dpad).

The best way for you to provide a mapping to me is to create a table.  Example:
On Bladeblah GamepaD:
"what you pressed"  = "what signal said"
A = hat0 Left
B = hat1 Right
X = A
Left Trigger = Right Shoulder

As a follow up, I tested the "Modern VCS Controller" versus the "XBOX Series S or X controller" and, lo and behold, ran into the same issue as you with the PS4.  Can't move the guy.  See pic:

So, this is actually expected behavior.
InputCandy allows the user to customize controls, or load an SDL profile (not really recommended!) ... as it doesn't fix the problem for them automagically, because if it did, you would have to have written it or trained it using an interactive trainer, as there is no reliable way in GameMaker with SDL or not to detect the controller make and model reliably most of the time, especially with current generation controllers, because no one company correctly followed the standard.
Where InputCandy helps most is in:

Setting the device for a player
Allowing the player to customize the controls for controllers that don't work right out of the box
Automatically assigning and remembering devices for a player, even if they aren't connected at the time.
Using the abstraction "Actions" to deal with player settings.

https://github.com/LAGameStudio/InputCandy/blob/trunk/README.md was updated to include important mention of how to use InputCandy based on this feedback


kdeApplications.kamoso: init at 20.12.1

Motivation for this change
Needed to see if my webcam was working properly. While it works here (with kdeapps 20.08 as well), there are no guarantees regarding the quality of the subject(s).
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114969 at 153e8491 run on aarch64-linux 1

1 package failed to build:

libsForQt512.kamoso

2 packages built:

libsForQt5.kamoso (libsForQt515.kamoso ,plasma5Packages.kamoso)
libsForQt514.kamoso

3 suggestions:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/applications/kde/default.nix:48:9:
   |
48 |         meta = {
   |         ^

warning: unclear-gpl
gpl3 is a deprecated license, check if project uses gpl3Plus or gpl3Only and change meta.license accordingly.
Near pkgs/applications/kde/kamoso.nix:21:3:
   |
21 |   meta.license = with lib.licenses; [ lgpl21 gpl3 ];
   |   ^

warning: unclear-gpl
lgpl21 is a deprecated license, check if project uses lgpl21Plus or lgpl21Only and change meta.license accordingly.
Near pkgs/applications/kde/kamoso.nix:21:3:
   |
21 |   meta.license = with lib.licenses; [ lgpl21 gpl3 ];
   |   ^


change theme

I'm new on Vue js. who can help me change the theme colour?
in fact, I don't know I should start from where?

around here :)
https://vuetifyjs.com/en/features/theme/
https://github.com/tiangolo/full-stack-fastapi-postgresql/blob/master/{{cookiecutter.project_slug}}/frontend/src/plugins/vuetify.ts

Thanks, mate


Page Layout: Repeat header line (i.e. browser names) below the table (in 'Current aligned' mode)

Duplicate of #1577?

Yup.


Removed luv resource requirement

Luv is only needed for test, and already disabled by formula's cmake option.

It's going to be required once we start shipping lua as part of the runtime. So it might be helpful to keep it around as a smoke test...

Then stable resource should also have luv in it?
Stable version also depend on luv by default, though disabled by formula's cmake option.

luv is not needed NOW.
Is it not good to add luv to resource when actually needed?

It will be needed in a few months. Is it causing a problem?

No.
Thank you for advice.


replaces float with numpy.float

possible fix for #202
Not completely sure if this is the right method, but this seems to work (and gets rid of the warning message) after initial testing.

please change np.floating to numpy.float

we use "numpy" not "np'
" FutureWarning: Conversion of the second argument of issubdtype from float to np.floating is deprecated. In future, it will be treated as np.float64 == np.dtype(float).type." --> check against numpy.float

Oops, I had changed that locally & tested with that but not committed/pushed it to the correct branch...

Still a problem with "floating", that name will be deprecated, the code needs to be "float".

so it should be numpy.float64?

I was under the impression that the proper fix is: float -> numpy.dtype(float).type ??

It is, "numpy.float", I tested this.
So not, "floating" and not "float64", the last one would exclude 32bit floats

Looks good!

if its good, lets merge..

Ok, let it be "floating".  Apparently float != floating.

can you fix this again in the rieder-patch-float branch?

Done


Vulkan example

Pull in Vulkan examples from waypp

In progress.  Ubuntu 20 with validation layers and multiple gpu takes > 2 seconds to enumerate KHR displays.  Does not repro on Fedora 41 with single GPU.


Develop

Hello @EdinsonRequena! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file algoritmos-y-poo-python/modulo_V/complementos/01morral-mi-aporte.py:

Line 20:5: E122 continuation line missing indentation or outdented
Line 20:80: E501 line too long (81 > 79 characters)
Line 21:5: E122 continuation line missing indentation or outdented
Line 42:1: E305 expected 2 blank lines after class or function definition, found 1


Declaration of an object with an empty key breaks language server.

The following code:
let test = {
  "": "dasfadsf",
}

Apparently breaks the language server if I save the file multiple times, as this vscode message says:
The ReScript Language Server server crashed 5 times in the last 3 minutes. The server will not be restarted.
Is this expected? I have autosave activated with 200ms of margin, so the errors appear automatically in my screen. I believe this syntax shouldn't break the plugin.
rescript vscode version: 1.1.3
rescript version: 9.1.4

Related to https://github.com/rescript-lang/rescript-compiler/issues/3494

This is becuase server is failing to parse the error message.
about empty object keys,
they are not supported at the moment so we should either make parser to disallow them or make the compiler support them, I think since the object access syntax is updated (x[""]) we can make  the compiler to allow them.
about the crash,
I think this is happening because the compiler log parser is throwing we should make that more resilient

I cannot reproduce this as of now, so considering this solved. Feel free to reopen if not.


Random crashes on creating new instance of  'V8ScriptEngine' - WPF

Currently my situation is different, I use it on a web scrapper which is doing multi-threading process and on more than 1 process of the same program (I open 3-4 processes of the program at the same time), It works fine and everything is totally fine, but after some time I see the program crashes without any details or catching any exceptions with "UnhandedException Event" for the application, when I tried on debug mode for the program (one of the processes) I got this exception:

As you can see, I tried using "Lock" for it but seems the same issue, also noticed that when I open 3 processes of the program, the first one is always getting high CPU usage than others and also getting memory leak, but other processes don't do that at all, they work fine without any high CPU usage or memory leak stable and fine..!
Any advice in that.. P.S: Using version 7.1.3.
Thanks for the nice tool! ^^

Hi @GameHackerPM,
Unfortunately we can't diagnose your issue without more information.
Can you provide a complete minimal code sample that reproduces the problem? If not, consider enabling crash dumps for your application. Use the Full dump type. If you're comfortable sharing a crash dump, we'll be happy to take a look.
Some general comments:

Script engines are thread-safe, so you shouldn't need a lock unless you're using it to protect your own (rather than ClearScript's) resources.

Script engines in different processes are isolated from each other, so multiple instances of your applications shouldn't be able to affect each other unless they somehow communicate across process boundaries.

In the code above, the using statement makes the Dispose call unnecessary.

Thanks!

Hi @ClearScriptLib,
Thanks for the fast reply.
I have tried to enable crash dumps from regedit as explained in the link, but didn't generate anything, do I need to restart the system after that?
But I created a dump file through Task Manager as well when it crashed, Here is the dump you requested: https://www.filemail.com/d/vyvrwbwdiygycuy
Thanks for the general comments/tips.

Hi @GameHackerPM,
Thanks for sharing the crash dump! We've managed to reconstruct the crash stack, but it's a real head scratcher.
We have a theory about the cause, but since we can't reproduce the crash, we can't test it. Would you be willing to test one or more patched assemblies? If so, what runtime are you using (.NET Framework, .NET Core, .NET 5, etc.)?
Cheers!

Hi @ClearScriptLib,
Thanks for the reply.
Currently, I'm using .NET Framework 4.8, also tried with .NET Framework 4.6.2 was the same thing. Ya I think I can test whatever you like to find what's wrong with it and fix that issue if it's from my side or something wrong with ClearScript. Will be glad to help you out for finding this.
Best Regards..

Hi @GameHackerPM,
Please try the following patched assembly on .NET Framework 4.8: ClearScript.V8.dll.zip. Make sure to replace your deployed version of ClearScript.V8.dll with this one. In the Details tab of the Properties dialog, "Product version" should be "7.1.3-patch1".
Thanks for your help!

Hi @ClearScriptLib,
Just to let you know, I'm testing that patch will leave it for another day just to make sure it fixed the issue before I confirm with you, but thanks for the follow and fast reply..
Will let you know the final reply if it crashed again.
Best regards.

Hi @ClearScriptLib,
I can confirm that this patch fixed the issue which was randomly crashing, I tested it for 6 hours straight no crashes at all. Before that test there is another test for 3 hours got crashed, but sadly can't confirm it's from ClearScript or something else specially that 20 instances of the program on 10 different devices got closed at the exact same time without even giving any crash message or anything.
So I would suggest this patch fixed the issue fine. I hope you can push that to public version so I can update it directly.
Thanks a lot for your help and fast patching. [I have left the issue open until you confirm and close it yourself!]
Best Regards.

Hi @GameHackerPM,
Thank you very much for testing the fix! The other issue doesn't sound like it's related to ClearScript, but if you think it could be, don't hesitate to send us another crash dump.
We'll post a new release ASAP.
Thanks again!

Version 7.1.4 includes the fix. Thanks again!


Get aspirin working in local

The issue might be due to attempting to use port 25(or a port below 1024). If unable to resolve, try switching Aspirin with https://github.com/voodoodyne/subethasmtp

Sending using Aspirin does work in local now. Received spam blocked messages from gmail and a successful delivery to https://temp-mail.org
2023-11-05 13:14:26,204 [Thread-2] INFO  Aspirin  - Maintenance thread started.
2023-11-05 13:14:26,210 [Aspirin-DeliveryManager-34] INFO  Aspirin  - DeliveryManager started.
2023-11-05 13:14:26,220 [DeliveryThread-1] INFO  Aspirin  - DeliveryThread (DeliveryThread-1).deliver(): Starting mail delivery. qi=DeliveryContext [qi=Mail: [id=org.simplejavamail.converter.internal.mimemessage.MimeMessageHelper$1@12b53598; recipient=devatherock@gmail.com];];
2023-11-05 13:14:28,113 [DeliveryThread-1] INFO  Aspirin  - DeliveryThread (DeliveryThread-1).deliver(): Mail delivery failed: 550-5.7.26 This mail has been blocked because the sender is unauthenticated.
550-5.7.26 Gmail requires all senders to authenticate with either SPF or DKIM.
550-5.7.26
550-5.7.26  Authentication results:
550-5.7.26  DKIM = did not pass
550-5.7.26  SPF [] with ip: [<IP_ADDRESS>] = did not pass
550-5.7.26
550-5.7.26  To mitigate this issue, please visit Gmail's authentication guide
550-5.7.26 for instructions on setting up authentication:
550 5.7.26  https://support.google.com/mail/answer/81126#authentication er19-20020a056214191300b0065afd35c75dsi4508302qvb.70 - gsmtp
. qi=DeliveryContext [qi=Mail: [id=org.simplejavamail.converter.internal.mimemessage.MimeMessageHelper$1@12b53598; recipient=devatherock@gmail.com];];
2023-11-05 13:16:26,220 [DeliveryThread-1] INFO  Aspirin  - DeliveryThread (DeliveryThread-1).deliver(): Starting mail delivery. qi=DeliveryContext [qi=Mail: [id=org.simplejavamail.converter.internal.mimemessage.MimeMessageHelper$1@212fabab; recipient=devatherock@gmail.com];];
2023-11-05 13:16:27,345 [DeliveryThread-1] INFO  Aspirin  - DeliveryThread (DeliveryThread-1).deliver(): Mail delivery failed: 550-5.7.26 This mail has been blocked because the sender is unauthenticated.
550-5.7.26 Gmail requires all senders to authenticate with either SPF or DKIM.
550-5.7.26
550-5.7.26  Authentication results:
550-5.7.26  DKIM = did not pass
550-5.7.26  SPF [] with ip: [<IP_ADDRESS>] = did not pass
550-5.7.26
550-5.7.26  To mitigate this issue, please visit Gmail's authentication guide
550-5.7.26 for instructions on setting up authentication:
550 5.7.26  https://support.google.com/mail/answer/81126#authentication d16-20020a05622a05d000b004109d1af4d4si4301497qtb.647 - gsmtp
. qi=DeliveryContext [qi=Mail: [id=org.simplejavamail.converter.internal.mimemessage.MimeMessageHelper$1@212fabab; recipient=devatherock@gmail.com];];
2023-11-05 13:21:26,223 [DeliveryThread-1] INFO  Aspirin  - DeliveryThread (DeliveryThread-1).deliver(): Starting mail delivery. qi=DeliveryContext [qi=Mail: [id=org.simplejavamail.converter.internal.mimemessage.MimeMessageHelper$1@5456adc0; recipient=sodico2927@jucatyo.com];];
2023-11-05 13:21:26,794 [DeliveryThread-1] ERROR Aspirin  - SendMessage.handle(): Last server response: 250 Message Queued (0AF93B8D-2E5D-4526-B1FF-E3C4B5C661A1.1)
.
2023-11-05 13:21:26,831 [DeliveryThread-1] INFO  Aspirin  - DeliveryThread (DeliveryThread-1).deliver(): Mail delivery success: 250 Message Queued (0AF93B8D-2E5D-4526-B1FF-E3C4B5C661A1.1)
. qi=DeliveryContext [qi=Mail: [id=org.simplejavamail.converter.internal.mimemessage.MimeMessageHelper$1@5456adc0; recipient=sodico2927@jucatyo.com];];
``


From Ionic, Request header field If-Modified-Since is not allowed by Access-Control-Allow-Headers.

Hi,
We have implemented ng-token-auth in out Ionic project. However, while running from Localhost we are getting following message:
XMLHttpRequest cannot load https://www.example.com/api/v0/auth/sign_in. Request header field If-Modified-Since is not allowed by Access-Control-Allow-Headers.

Ionic app works fine when we run on chrome after disabling web security. I can also run the sign_in api from curl and chrome REST clients. curl works with/without passing any headers.
Backend is running the rail gem. Are we missing anything?
curl -XPOST -v -H 'If-Modified-Since: Mon, 26 Jul 1997 05:00:00 GMT,customxxx:12123123'  https://www.example.com/api/v0/auth/sign_in -d<EMAIL_ADDRESS>
* Connected to www.example.com (xx.xx.xx.xxx) port 443 (#0)
* TLS 1.2 connection using TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384
* Server certificate: www.example.com
* Server certificate: GeoTrust Extended Validation SSL CA - G2
* Server certificate: GeoTrust Primary Certification Authority
> POST /api/v0/auth/sign_in HTTP/1.1
> User-Agent: curl/7.37.1
> Host: www.example.com
> Accept: */*
> If-Modified-Since: Mon, 26 Jul 1997 05:00:00 GMT,customxxx:12123123
> Content-Length: 43
> Content-Type: application/x-www-form-urlencoded
>
* upload completely sent off: 43 out of 43 bytes
< HTTP/1.1 200 OK
* Server nginx is not blacklisted
< Server: nginx
< Date: Thu, 28 May 2015 05:22:10 GMT
< Content-Type: application/json; charset=utf-8
< Transfer-Encoding: chunked
< Connection: keep-alive
< Status: 200 OK
< X-Frame-Options: SAMEORIGIN
< X-XSS-Protection: 1; mode=block
< X-Content-Type-Options: nosniff
< access-token: LdpIniRZIUpHH2ZpG0diUw
< token-type: Bearer
< client: EQoMUP25yuysgTzSUPKCNw
< expiry:<PHONE_NUMBER>
< uid<EMAIL_ADDRESS>< Vary: Origin
< ETag: W/"904c4c8499ec4827740d769b2a8cf86e"
< Cache-Control: max-age=0, private, must-revalidate
< X-Request-Id: 6788950a-3986-431c-aa4d-e7b027aa96ae
< X-Runtime: 1.132987

Here is the Request & Response headers from Chrome REST Client
Status
200 OK Show explanation Loading time: 2535

Request headers

CSP: active
Origin: chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo
gfgfgfg: jjhjhjhjhjh
If-Modified-Since: Mon, 26 Jul 1997 05:00:00 GMT
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2357.81 Safari/537.36
Content-Type: application/x-www-form-urlencoded
Accept: */*
Accept-Encoding: gzip, deflate
Accept-Language: en-US,en;q=0.8,bn;q=0.6,fr;q=0.4
Cookie: _example_session=BAh7B0kiD3Nlc3Npb25faWQGOgZFVEkiJWZiZWY1YTc0Nzg3Nzg3OGY1MGQ5Y2FlN2NmZDU2NWNjBjsAVEkiGXdhcmRlbi51c2VyLnVzZXIua2V5BjsAVFsHWwZpD0kiIiQyYSQxMCQ5cUFqTzJYT204ZlVqOGxxU3hJRkFPBjsAVA%3D%3D--6699787e64f9ac2522cfa36903b7c8dce4b84449

Response headers

Server: nginx
Date: Thu, 28 May 2015 03:35:21 GMT
Content-Type: application/json; charset=utf-8
Transfer-Encoding: chunked
Connection: keep-alive
Status: 200 OK
X-Frame-Options: SAMEORIGIN
X-XSS-Protection: 1; mode=block
X-Content-Type-Options: nosniff
access-token: cnqTU_D-lb2NWitH6gbeLw
token-type: Bearer
client: 5Ec_oHq1iOc6VmPIReWWQw
expiry:<PHONE_NUMBER>
uid<EMAIL_ADDRESS>Access-Control-Allow-Origin: chrome-extension://hgmloofddffdnphfgcellkdfbfbjeloo
Access-Control-Allow-Methods: GET, POST, OPTIONS, DELETE, PUT
Access-Control-Expose-Headers: access-token, expiry, token-type, uid, client
Access-Control-Max-Age: 1728000
Access-Control-Allow-Credentials: true
Vary: Origin
ETag: W/"904c4c8499ec4827740d769b2a8cf86e"
Cache-Control: max-age=0, private, must-revalidate
X-Request-Id: c6fc8aed-fb45-46ac-bd62-b11f689566d5
X-Runtime: 1.050273

Any help will be appreciated.
Thanks,
Tamal

We have also tried using proxy, but the issue remains:
"proxies": [
  {
    "path": "/api/v0/auth/sign_in",
    "proxyUrl": "http://www.example.com/api/v0/auth/sign_in"
  },
  {
    "path": "/api/v0/auth/sign_out",
    "proxyUrl": "http://www.example.com/api/v0/auth/sign_out"
  },
  {
    "path": "/api/v0/auth/validate_token",
    "proxyUrl": "http://www.example.com/api/v0/auth/validate_token"
  },
  {
    "path": "/api/v0/orders",
    "proxyUrl": "http://www.example.com/api/v0/orders"
  },
  {
    "path": "/api/v0/products",
    "proxyUrl": "http://www.example.com/api/v0/products"
  },
  {
    "path": "/api/v0/customers",
    "proxyUrl": "http://www.example.com/api/v0/customers"
  }
]

Your server is sending the http://enable-cors.org/server.html header? http://enable-cors.org/server.html

See http://blog.ionic.io/handling-cors-issues-in-ionic/

They said it would work on ionic. I think the problem is that you have to use rack-cors gem to enable cross origin requests and allow the If-Modified-Header. It works using chrome extensions & curl because they are not restricted by cors.
rack-cors: https://github.com/cyu/rack-cors
 config.middleware.insert_before 0, "Rack::Cors" do
      allow do
        origins '*'
        resource '*', :headers => :any, :methods => [:get, :post, :options]
      end
    end

We fixed the issue after enabling cors as per given docs here http://enable-cors.org/server_nginx.html.
Thanks for your help.


Upgrade dependencies version and synchronize with Sylius package

Sorry to skip over this PR, the changes have been introduced in #193.


Feeds everywhere!

This PR adds an RSS feed to every BrowseFilterable and SublandingFilterable page. It does NOT add all these new feeds to the UI (except for where they replace the existing feeds, blog and newsroom) . The feeds also support filters, but again, this functionality isn't really exposed in the UI.
Check out such feeds as:

/about-us/blog/feed/
/about-us/newsroom/feed/
/about-us/blog/feed/?filter1_authors=Holly+Petraeus
/data-research/research-reports/feed/
/data-research/research-reports/feed/?filter2_topics=Students
/policy-compliance/amicus/briefs/feed/
/activity-log/feed/
/policy-compliance/notice-opportunities-comment/open-notices/feed/

Additions

Outlined above

Removals

removed existing sheerlike and v1 feed code.

Testing

with a recent dump from refresh or elsewhere, try out the feed URL's above.
confirm that the RSS buttons on blog and newsroom take you to the new feeds.

Review

@kave @richaagarwal @kurtw @schaferjh

Todos

Could probably use some tests, but there isn't much new logic, it leans heavily on the Django feeds framework and our existing code to filter decedents.

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Nice 💯

👍


Make metadata requests use the cached endpoint automatically

The overhead for the non-cached endpoint is substantial.
(I'm going to check if David can push a new version of delphi-epidata to PyPI for us, so we can release a Python package that has fast access to the metadata without monkey-patching the current version.)

I don't know if it's worth taking an optional argument to force skipping the cache; do we imagine clients needing to do that?

I don't know if it's worth taking an optional argument to force skipping the cache; do we imagine clients needing to do that?

No, and we can always add it later if someone comes up with a use case. Cached data is only returned if the cache is less than 75 minutes old, otherwise it runs the full aggregation-based query.


✗ Error: Unable to run "adb", check your Android SDK installation and ANDROID_HOME environment variable

Steps to reproduce
I mean its seen from the logs, but it seems as though flutter doesnt see the adb or can't run it. Makes no sense, I've tried reinstalling the sdk tools a bunch of times and even copied the sdk from another laptop, on which it works. Oh and it might be helpful to know this as well, flutter run doesn't work at all, I get the adb error. But if I run it as flutter run -d windows, it works.
Expected results
It's supposed to return no issues.
Actual results
All provided in the logs section
Code sample
There is no code sample for this.
Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
PS C:\Users\Kariera\Workspaces\Flutter\kariera-mobileapp-navbar> echo $env:ANDROID_HOME
C:\Users\Kariera\android_sdk

PS C:\Users\Kariera\Workspaces\Flutter\kariera-mobileapp-navbar> C:\Users\Kariera\android_sdk\platform-tools\adb.exe
Android Debug Bridge version 1.0.41
Version 35.0.1-11580240
Installed as C:\Users\Kariera\android_sdk\platform-tools\adb.exe
Running on Windows 10.0.22631

global options:
 -a                       listen on all network interfaces, not just localhost
 -d                       use USB device (error if multiple devices connected)
 -e                       use TCP/IP device (error if multiple TCP/IP devices available)
 -s SERIAL                use device with given serial (overrides $ANDROID_SERIAL)
 -t ID                    use device with given transport id
 -H                       name of adb server host [default=localhost]
 -P                       port of adb server [default=5037]
 -L SOCKET                listen on given socket for adb server [default=tcp:localhost:5037]
 --one-device SERIAL|USB  only allowed with 'start-server' or 'server nodaemon', server will only connect to one USB device, specified by a serial number or USB device address.
 --exit-on-write-error    exit if stdout is closed

general commands:
 devices [-l]             list connected devices (-l for long output)
 help                     show this help message
 version                  show version num

networking:
 connect HOST[:PORT]      connect to a device via TCP/IP [default port=5555]
 disconnect [HOST[:PORT]]
     disconnect from given TCP/IP device [default port=5555], or all
 pair HOST[:PORT] [PAIRING CODE]
     pair with a device for secure TCP/IP communication
 forward --list           list all forward socket connections
 forward [--no-rebind] LOCAL REMOTE
     forward socket connection using:
       tcp:<port> (<local> may be "tcp:0" to pick any open port)
       localabstract:<unix domain socket name>
       localreserved:<unix domain socket name>
       localfilesystem:<unix domain socket name>
       dev:<character device name>
       jdwp:<process pid> (remote only)
       vsock:<CID>:<port> (remote only)
       acceptfd:<fd> (listen only)
 forward --remove LOCAL   remove specific forward socket connection
 forward --remove-all     remove all forward socket connections
 reverse --list           list all reverse socket connections from device
 reverse [--no-rebind] REMOTE LOCAL
     reverse socket connection using:
       tcp:<port> (<remote> may be "tcp:0" to pick any open port)
       localabstract:<unix domain socket name>
       localreserved:<unix domain socket name>
       localfilesystem:<unix domain socket name>
 reverse --remove REMOTE  remove specific reverse socket connection
 reverse --remove-all     remove all reverse socket connections from device
 mdns check               check if mdns discovery is available
 mdns services            list all discovered services

file transfer:
 push [--sync] [-z ALGORITHM] [-Z] LOCAL... REMOTE
     copy local files/directories to device
     --sync: only push files that have different timestamps on the host than the device
     -n: dry run: push files to device without storing to the filesystem
     -z: enable compression with a specified algorithm (any/none/brotli/lz4/zstd)
     -Z: disable compression
 pull [-a] [-z ALGORITHM] [-Z] REMOTE... LOCAL
     copy files/dirs from device
     -a: preserve file timestamp and mode
     -z: enable compression with a specified algorithm (any/none/brotli/lz4/zstd)
     -Z: disable compression
 sync [-l] [-z ALGORITHM] [-Z] [all|data|odm|oem|product|system|system_ext|vendor]
     sync a local build from $ANDROID_PRODUCT_OUT to the device (default all)
     -n: dry run: push files to device without storing to the filesystem
     -l: list files that would be copied, but don't copy them
     -z: enable compression with a specified algorithm (any/none/brotli/lz4/zstd)
     -Z: disable compression

shell:
 shell [-e ESCAPE] [-n] [-Tt] [-x] [COMMAND...]
     run remote shell command (interactive shell if no command given)
     -e: choose escape character, or "none"; default '~'
     -n: don't read from stdin
     -T: disable pty allocation
     -t: allocate a pty if on a tty (-tt: force pty allocation)
     -x: disable remote exit codes and stdout/stderr separation
 emu COMMAND              run emulator console command

app installation (see also `adb shell cmd package help`):
 install [-lrtsdg] [--instant] PACKAGE
     push a single package to the device and install it
 install-multiple [-lrtsdpg] [--instant] PACKAGE...
     push multiple APKs to the device for a single package and install them
 install-multi-package [-lrtsdpg] [--instant] PACKAGE...
     push one or more packages to the device and install them atomically
     -r: replace existing application
     -t: allow test packages
     -d: allow version code downgrade (debuggable packages only)
     -p: partial application install (install-multiple only)
     -g: grant all runtime permissions
     --abi ABI: override platform's default ABI
     --instant: cause the app to be installed as an ephemeral install app
     --no-streaming: always push APK to device and invoke Package Manager as separate steps
     --streaming: force streaming APK directly into Package Manager
     --fastdeploy: use fast deploy
     --no-fastdeploy: prevent use of fast deploy
     --force-agent: force update of deployment agent when using fast deploy
     --date-check-agent: update deployment agent when local version is newer and using fast deploy
     --version-check-agent: update deployment agent when local version has different version code and using fast deploy
     (See also `adb shell pm help` for more options.)
 uninstall [-k] PACKAGE
     remove this app package from the device
     '-k': keep the data and cache directories

debugging:
 bugreport [PATH]
     write bugreport to given PATH [default=bugreport.zip];
     if PATH is a directory, the bug report is saved in that directory.
     devices that don't support zipped bug reports output to stdout.
 jdwp                     list pids of processes hosting a JDWP transport
 logcat                   show device log (logcat --help for more)

security:
 disable-verity           disable dm-verity checking on userdebug builds
 enable-verity            re-enable dm-verity checking on userdebug builds
 keygen FILE
     generate adb public/private key; private key stored in FILE,

scripting:
 wait-for[-TRANSPORT]-STATE...
     wait for device to be in a given state
     STATE: device, recovery, rescue, sideload, bootloader, or disconnect
     TRANSPORT: usb, local, or any [default=any]
 get-state                print offline | bootloader | device
 get-serialno             print <serial-number>
 get-devpath              print <device-path>
 remount [-R]
      remount partitions read-write. if a reboot is required, -R will
      will automatically reboot the device.
 reboot [bootloader|recovery|sideload|sideload-auto-reboot]
     reboot the device; defaults to booting system image but
     supports bootloader and recovery too. sideload reboots
     into recovery and automatically starts sideload mode,
     sideload-auto-reboot is the same but reboots after sideloading.
 sideload OTAPACKAGE      sideload the given full OTA package
 root                     restart adbd with root permissions
 unroot                   restart adbd without root permissions
 usb                      restart adbd listening on USB
 tcpip PORT               restart adbd listening on TCP on PORT

internal debugging:
 start-server             ensure that there is a server running
 kill-server              kill the server if it is running
 reconnect                kick connection from host side to force reconnect
 reconnect device         kick connection from device side to force reconnect
 reconnect offline        reset offline/unauthorized devices to force reconnect

usb:
 attach                   attach a detached USB device
 detach                   detach from a USB device to allow use by other processes
environment variables:
 $ADB_TRACE
     comma/space separated list of debug info to log:
     all,adb,sockets,packets,rwx,usb,sync,sysdeps,transport,jdwp
 $ADB_VENDOR_KEYS         colon-separated list of keys (files or directories)
 $ANDROID_SERIAL          serial number to connect to (see -s)
 $ANDROID_LOG_TAGS        tags to be used by logcat (see logcat --help)
 $ADB_LOCAL_TRANSPORT_MAX_PORT max emulator scan port (default 5585, 16 emus)
 $ADB_MDNS_AUTO_CONNECT   comma-separated list of mdns services to allow auto-connect (default adb-tls-connect)

Online documentation: https://android.googlesource.com/platform/packages/modules/adb/+/refs/heads/main/docs/user/adb.1.md
PS C:\Users\Kariera\Workspaces\Flutter\kariera-mobileapp-navbar>
Flutter Doctor output
PS C:\Users\Kariera\Workspaces\Flutter\kariera-mobileapp-navbar> flutter doctor -v
[✓] Flutter (Channel stable, 3.22.3, on Microsoft Windows [Version 10.0.22631.3880], locale sl-SI)
    • Flutter version 3.22.3 on channel stable at C:\Users\Kariera\dev\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision b0850beeb2 (2 weeks ago), 2024-07-16 21:43:41 -0700
    • Engine revision 235db911ba
    • Dart version 3.4.4
    • DevTools version 2.34.3

[✓] Windows Version (Installed version of Windows is version 10 or higher)

[✓] Android toolchain - develop for Android devices (Android SDK version 35.0.0)
    • Android SDK at C:\Users\Kariera\android_sdk
    • Platform android-35, build-tools 35.0.0
    • ANDROID_HOME = C:\Users\Kariera\android_sdk
    • ANDROID_SDK_ROOT = C:\Users\Kariera\android_sdk
    • Java binary at: C:\Users\Kariera\AppData\Local\Programs\Android Studio\jbr\bin\java
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.10.1)
    • Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
    • Visual Studio Community 2022 version 17.10.34928.147
    • Windows 10 SDK version 10.0.22621.0

[✓] Android Studio (version 2024.1)
    • Android Studio at C:\Users\Kariera\AppData\Local\Programs\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.11+0--11852314)

[✓] IntelliJ IDEA Ultimate Edition (version 2024.1)
    • IntelliJ at C:\Users\Kariera\AppData\Local\Programs\IntelliJ IDEA Ultimate
    • Flutter plugin version 80.0.2
    • Dart plugin version 233.15026.12

[☠] Connected device (the doctor check crashed)
    ✗ Due to an error, the doctor check did not complete. If the error message below is not helpful, please let us know about this issue at https://github.com/flutter/flutter/issues.
    ✗ Error: Unable to run "adb", check your Android SDK installation and ANDROID_HOME environment variable: C:\Users\Kariera\android_sdk\platform-tools\adb.exe
    • #0      throwToolExit (package:flutter_tools/src/base/common.dart:10:3)
      #1      AndroidDevices.pollingGetDevices (package:flutter_tools/src/android/android_device_discovery.dart:75:7)
      <asynchronous suspension>
      #2      PollingDeviceDiscovery._populateDevices (package:flutter_tools/src/device.dart:547:36)
      <asynchronous suspension>
      #3      Future.wait.<anonymous closure> (dart:async/future.dart:524:21)
      <asynchronous suspension>
      #4      DeviceManager.refreshAllDevices (package:flutter_tools/src/device.dart:217:40)
      <asynchronous suspension>
      #5      DeviceValidator.validate (package:flutter_tools/src/doctor.dart:741:34)
      <asynchronous suspension>
      #6      Future.any.onValue (dart:async/future.dart:618:5)
      <asynchronous suspension>

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Closing this as duplicate of https://github.com/flutter/flutter/issues/72101. Please follow-up in it for further updates.
You may share your findings in that issue for better tracking.


Opacity does not apply on head items. (player heads, commonly used in Hypixel Skyblock)

Head items will stay 100% opacity no matter what.

Added an additional slider to configure "hat" transparency in mod version 1.8.0.

Thank you!


python runpy.py bug

This issue has been moved from a ticket on Developer Community.

when running the below cmd line processing logic, VS runpy.py code trips up on main exit:
import sys

if __name__=="__main__":
    args=sys.argv
    args.pop(0)
    print("Hello world!")
    exit(0)

The error is:
Traceback (most recent call last):
  File "c:\program files (x86)\microsoft visual studio\2019\enterprise\common7\ide\extensions\microsoft\python\core\ptvsd_launcher.py", line 119, in <module>
    vspd.debug(filename, port_num, debug_id, debug_options, run_as)
  File "c:\program files (x86)\microsoft visual studio\2019\enterprise\common7\ide\extensions\microsoft\python\core\Packages\ptvsd\debugger.py", line 39, in debug
    run()
  File "c:\program files (x86)\microsoft visual studio\2019\enterprise\common7\ide\extensions\microsoft\python\core\Packages\ptvsd\__main__.py", line 316, in run_file
    runpy.run_path(target, run_name='__main__')
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 263, in run_path
    pkg_name=pkg_name, script_name=fname)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 59, in __exit__
    sys.argv[0] = self._saved_value
IndexError: list assignment index out of range

Original Comments
Visual Studio Feedback System on 3/1/2020, 03:01 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

Repros on 16.5 with debugpy as well.
import sys

if __name__=="__main__":
    args=sys.argv
    args.pop(0)
    print("Hello world!")
    exit(0)

Press F5.
Hello world!
Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "C:\Users\huvalo\source\repos\PythonApplication11\PythonApplication11.py", line 8, in <module>
    exit(0)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\_sitebuiltins.py", line 26, in __call__
    raise SystemExit(code)
SystemExit: 0

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "c:\program files (x86)\microsoft visual studio\2019\preview\common7\ide\extensions\microsoft\python\core\debugpy\__main__.py", line 45, in <module>
    cli.main()
  File "c:\program files (x86)\microsoft visual studio\2019\preview\common7\ide\extensions\microsoft\python\core\debugpy/..\debugpy\server\cli.py", line 361, in main
    run()
  File "c:\program files (x86)\microsoft visual studio\2019\preview\common7\ide\extensions\microsoft\python\core\debugpy/..\debugpy\server\cli.py", line 203, in run_file
    runpy.run_path(options.target, run_name="__main__")
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 263, in run_path
    pkg_name=pkg_name, script_name=fname)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
  File "C:\Program Files (x86)\Microsoft Visual Studio\Shared\Python37_64\lib\runpy.py", line 59, in __exit__
    sys.argv[0] = self._saved_value
IndexError: list assignment index out of range
Press any key to continue . . .

The contract for sys.argv is that it always contains at least one entry, which is the name of the file or module used to start the process. Popping something from argv breaks that contract, which in turn breaks the standard runpy module that we use to simulate the behavior of the Python interpreter. If it's a bug, it's a runpy bug, but I think that's just not expected to work by design.
In any case, I don't think we can really do anything here. The exception happens inside runpy, and it's inside the call that we make to run the target script. We can't just catch any IndexError that comes out of that code, and we can't distinguish this particular case from others (I guess we could look at the stack, but then we'd be depending on runpy internals).


removing a repo and adding a new one with a new URL can have unexpected results

appears that the old repo is cached somewhere. i.e. if we pull a new repo with the same name as one that was previously removed, it pulls the old repo again (old url) instead of using the new url

multiple commits to attempt to make this bullet proof
https://github.com/PaloAltoNetworks/pan-cnc/commit/55e7f2b9ed430ee9f1644601c983ad7d63f4835b
https://github.com/PaloAltoNetworks/pan-cnc/commit/7d257d2c4e2ff44da630cac0819804decbf8a432


Cache grid pos

What's the problem this feature will solve?
There should be a method to cache the positions of agents at each step of the simulation so that we can display the agents on the grid visualization.
Describe the solution you'd like

Cache the positions of the agents
Modify the visualization module. Currently, the visualization module requires the model object to get the grid attribute for displaying visualization. The module should be modified such that there is no need for the model object, and the grid is directly constructed from the cached file.

Main edit
Caching methods
    class TestModel(mesa.Model):
    """A model with some number of agents."""

    def __init__(self, N, width, height):
        super().__init__()
        # self.num_agents = N
        # self.grid = mesa.space.MultiGrid(1,1, True)
        # self.schedule = mesa.time.RandomActivation(self)

        # Create agents
        # for i in range(self.num_agents):
            # a = MoneyAgent(i, self)
            # self.schedule.add(a)
            # Add the agent to a random grid cell
            # x = self.random.randrange(self.grid.width)
            # y = self.random.randrange(self.grid.height)
            # self.grid.place_agent(a, (x, y))

        # self.datacollector = mesa.DataCollector()

    def step(self):
        # self.datacollector.collect(self)
        self._steps += 1

The TestModel acts as a dummy model to be fed into SolaraViz. This allows the visualizer to step while the data displayed is taken from the cached data.
Limitations:

the visualizer will break after it has no more cached data to read. Must stop the visualiser before the last cached file is read.
the visualizer's step can only increase. Possible improvements include adding a playback button to decrement the step number.

Main edit
Caching methods
   @solara.component
def PlotMatplotlib(model, measure, dependencies: list[any] | None = None):
    fig = Figure()
    ax = fig.subplots()

    # TODO: Check
    model_files = glob.glob(f"output_dir/model_data_*.parquet")
    model_dfs = []
    for model_file in model_files:
        table = pq.read_table(model_file)
        df = table.to_pandas()
        model_dfs.append(df)
    df = pd.concat(model_dfs, ignore_index=True)[:model._steps+1]

    if isinstance(measure, str):
        ax.plot(df.loc[:, measure])
        ax.set_ylabel(measure)
    elif isinstance(measure, dict):
        for m, color in measure.items():
            ax.plot(df.loc[:, m], label=m, color=color)
        fig.legend()
    elif isinstance(measure, list | tuple):
        for m in measure:
            ax.plot(df.loc[:, m], label=m)
        fig.legend()
    # Set integer x axis
    ax.xaxis.set_major_locator(MaxNLocator(integer=True))
    solara.FigureMatplotlib(fig, dependencies=dependencies)

The PlotMatplotlib will now plot the matplotlib graph from cached data.


Fix a typo in Chunk#split jsdoc comment

What kind of change does this PR introduce?
typo fix
Does this PR introduce a breaking change?
no
What needs to be documented once your changes are merged?
nothing

@TheLarkInn please have a look when you have a moment

Thanks @ooflorent
Does it mean we can merge this tiny fix now? :)

Thanks


Migrate to multi-package layout

In anticipation of future (imminent!) additions of optional addon-addons like support for changeset-validations (#11) or yup (#12), this is moving the repo to a multi-package layout, ~renaming the current addon to @ember-headless-form/core and~allowing for additional "adapter" packages like e.g. @ember-headless-form/changeset-validations.
There are different ways to do this (compare to https://github.com/embroider-build/addon-blueprint/issues/2), this is putting all public packages under packages/*, while assuming that we keep just one test-app (we could create a test-app per package also, but I think we can get around that and test all packages in a single app, given how boilerplate-heavy apps are). But if you have other suggestions as to how to structure this, then please share those!
~There is one unresolved thing though, as #30 has config bits like this assuming a single package. So not sure what to do with this. @NullVoxPopuli did we have a similar case before somewhere?~

There is one unresolved thing though, as https://github.com/CrowdStrike/ember-headless-form/pull/30 has config bits like this assuming a single package. So not sure what to do with this. @NullVoxPopuli did we have a similar case before somewhere?

we can modify that to be all the dist directories in the monorepo, I believe. otherwise, we'll want to look at using turborepo like here, but we really need the dependenciesInjecteed thing resolved before we use turbo.

renaming the current addon to @ember-headless-form/core and allowing for additional "adapter" packages like e.g. @ember-headless-form/changeset-validations.

I think we should keep the main addon ember-headless-form, it's easier to say and write. I'm 👍 on the adapter names though.

, but I think we can get around that and test all packages in a single app

sounds good to me! There is still a lot of focus in that one app, so 💯

I think we should keep the main addon ember-headless-form, it's easier to say and write. I'm 👍 on the adapter names though.

Yeah, I considered that also briefly, but idk why I dismissed it later, probably for the sake of consistency. But I agree it's better, so I reverted that change now. Off topic: probably should do the same for https://github.com/simonihmig/ember-responsive-image/pull/442
For the CI issue, I now commented out the dedicated build step, and replaced all actions/download-built-package steps with just a plain pnpm build. I know that's not what we wanted in the first place, but also I don't want to get blocked on this, or spend additional time figuring out a better solution at this time. I think the build step is actually pretty fast, so it's not really an issue. Tracking issue: https://github.com/CrowdStrike/ember-headless-form/issues/33

Question: I guess it's still early stages, but should the package.json name in packages/ember-headless-form be updated to any one of:

@ember-headless-form/core
@ember-headless-form
@crowdstrike/ember-headless-form
@crowdstrike/ember-headless-form/core

Question: I guess it's still early stages, but should the package.json name in packages/ember-headless-form be updated to any one of:
1. `@ember-headless-form/core`

2. `@ember-headless-form`

3. `@crowdstrike/ember-headless-form`

4. `@crowdstrike/ember-headless-form/core`

I guess this part covers that:

renaming the current addon to @ember-headless-form/core andallowing for additional "adapter" packages like e.g. @ember-headless-form/changeset-validations.

we had already talked about keeping ember-headless-table because it sounds nicer, less to type, etc.  🙈


[ExternalResources] Provided tempdir looks to be useless

While there is no link between tempdir (which is an empty tempdir) and data_home (which is the directory containing the data).
If it really have no need relative to external resources, It would be better to remove it to avoid mistakes.
For example i was sure that clean_up method was cleaning up the data_home but it was not the case.

Fixed on 0.7


A problem in function findCorrespondingSurfFeatures

at line 1160 in featureAssociation.cpp
int j = closestPointInd + 1; j < surfPointsFlatNum; j++)
why use surfPointsFlatNum instead of laserCloudSurfLastNum?

I saw a closed issue for this problem, but the newest code still has this problem

I think it should use laserCloudSurfLastNum. Since surfPointsFlatNum is close to laserCloudSurfLastNum between two frames, I don't think there will be much observable difference.


Rule file-header is not working.

Rule file-header is not working. Even if particular header in a file like  Copyright \d{4} didn't add in the file or any wrong text adds, No Lint errors are getting.

TSLint version: ~5.3.2
TypeScript version: ~2.3.3
Running TSLint via: angular CLI

TypeScript code being linted
"file-header": [
    true,
    "Copyright \\d{4}"
  ]

with tslint.json configuration: tried in 2 different ways
{
  "rulesDirectory": [
    "node_modules/codelyzer"
  ],
  "rules": {
   // lint
  },
  "file-header": [
    true,
    "Copyright \\d{4}"
  ]
}

{
  "rulesDirectory": [
    "node_modules/codelyzer"
  ],
  "rules": {
   // lint,
   "file-header": [
    true,
    "Copyright \\d{4}"
  ]
  },

}

Actual behavior
TS lint is not validating wheather the header text is present or not present
Expected behavior
TSLint is failing to validate the file header text

@dileepkeeppalli is there a chance that you have BOM in your files?

This is now broken for me too. It used to work.
I think this is due to https://github.com/palantir/tslint/pull/2525 which used to check if the given text was present in the entire file text range, but now only checks the first comment in the file. However, the header in the file may look like this...
// **********************************
// Copyright 2017 Company
// **********************************

And the first comment it brings up is // ********************************** which does not match the regular expression that expects a multiline header.
However, this works though, because a multi-line comment is just one comment:
/***********************************
 * Copyright 2017 Company
 ***********************************/

Sounds like the rule could be given an option to consider consecutive // comments as one long file header.


My funds do not show up

To save on gas fees, I transferred a certain amount of dai not from my connected metamask account but directly from my exchange account to the deposit address you provided. According to etherscan, the dai is there 0x62c904C791368946fa5A2Ddfd8ec5f65072Ba885
But the Augur interface reports zero available or frozen funds. My money seems to be lost!!!
Please get my money back.
<EMAIL_ADDRESS>https://etherscan.io/address/0x62c904C791368946fa5A2Ddfd8ec5f65072Ba885#tokentxns

Never mind, it appeared at last, it just took way longer than I expected. Sorry for the bother.


Permit assembly keywords in solidity and vice versa

This ensures that switch, default, case, and leave are permitted in solidity, and that return is permitted in assembly.

Fixes #639


Decrypt Api Key secret

Helper method to decrypt an AES encrypted string.
Description
This code change/PR is to address the requirement for storing access keys as encrypted data.
My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[X] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[X] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[ ] I signed an Apache CLA.
[ ] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #57 into master will decrease coverage by 0.04%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #57      +/-   ##
==========================================
- Coverage   78.78%   78.73%   -0.05%
==========================================
  Files         200      201       +1
  Lines        9355     9363       +8
  Branches      666      666
==========================================
+ Hits         7370     7372       +2
- Misses       1985     1991       +6

Impacted Files
Coverage Δ

...cala/org/apache/openwhisk/utils/CryptHelpers.scala
0.00% <0.00%> (ø)

...e/openwhisk/core/containerpool/ContainerPool.scala
96.81% <0.00%> (+0.63%)
:arrow_up:

.../org/apache/openwhisk/core/connector/Message.scala
79.68% <0.00%> (+0.78%)
:arrow_up:

Continue to review full report at Codecov.

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


Colored
, , etc.

Please expose the color palette for individual text tags like
 or  or better yet, make them globally accessible with class="ui red" to define color: @redTextColor.
I understand this may break some of the deep references so leave it to you as to best way to approach. For now I am creating a site override for each as .ui.red: { color: @redTextColor; } etc.

I don't believe this goes against the idea of not creating css that is 1-2 lines as there are 14 colors?

Is there a thread somewhere discussing this already?

See  #1885, generally there are no free floating class names in Semantic UI, but only class names that have meaning in context of the component they modify.

"paragraph" could be considered a component...

This is a paragraph component.

This is a paragraph component.

This is a paragraph component.

[...]

With sixteen +1's in #1885 and some pretty compelling reasoning I would hope a dev team would reconsider a feature request.

+1 totally agree with @yaboi arguments

+1

Implemented. Example here: https://jsfiddle.net/63z148gb/


Got TypeError: Cannot read property 'any' of undefined ./node_modules/re-bulma/lib-es/grid/columns.js:25

Hi, when trying install re-bulma based on this instruction (using create-react-app which is already use react version 16.0.0),
https://github.com/bokuweb/re-bulma#install
I put these on index.js
import insertCss from "insert-css";
import css from "re-bulma/build/css";
try {
  if (typeof document !== "undefined" || document !== null) insertCss(css, { prepend: true });
} catch (e) {}

but after that, I got this error
TypeError: Cannot read property 'any' of undefined ./node_modules/re-bulma/lib-es/grid/columns.js:25

any clue how to solve this issue?
Thank you

I'm getting error at the same place but the error is TypeError: __WEBPACK_IMPORTED_MODULE_0_react__.PropTypes is undefined
Using<EMAIL_ADDRESS><EMAIL_ADDRESS>Code I'm using is
import { Container, Column, Columns } from 're-bulma';

class App extends Component {
  render() {
    return (
      <Container>
        <Columns>
          <Column>
            <NavLink className="button" activeClassName="active" exact to="/">
              Something List
            </NavLink>
          </Column>
          <Column>
            <NavLink className="button" activeClassName="active" exact to="/something/new">
              Add Something
            </NavLink>
          </Column>
        </Columns>
        <Route exact path="/" component={SomethingPage}/>
        <Route path="/something/new" component={SomethingFormPage}/>
        <Route path="/something/edit/:_id" component={SomethingFormPage}/>
      </Container>
    );
  }
}

My pull request solves this, I think: #97 . Lmk if you have feedback.

Since this project isn't being maintained, I'm not sure this pull request will be merged - and even if it is, this is probably not a package to rely on unless someone else takes over.
If it's helpful for others, we've temporarily solved the problem by including a dependency directly from the patched version:

In the medium term we're moving to semantic-ui-react (though there are other bulma-based react packages you could migrate to, e.g. https://github.com/couds/react-bulma-components).


Get checked node not working when I draw tree with checked element

Hi ,
I use jstree for edit in this case I draw the tree with checked element the element appear checked but when call function for get_checked not got it and the parent of the element appear as no child checked
When I check by mouse after loading it will be checked
I try this case by two ways draw tree with html and with json data I got the same result
What is the reason

Please provide a demo of the issue, as I can not reproduce this. Also make sure you are using all properties correctly - checked and selected could be the same, but they could also be different:
https://www.jstree.com/api/#/?q=tie&f=$.jstree.defaults.checkbox.tie_selection - it all depends on how you configure the tree.
I will reopen when a demo is available and I can reproduce. Here is a stepping stone - it works fine for me:
http://jsfiddle.net/8ax6cwoy/


Standard Return types for each endpoint

Now that I'm wrangling with now with ECF5, we will likely need to work with ECF 4 and ECF 5 jurisdictions at the same time, and IMO, it's a very bad idea to try to expect all of the client implementations to compare and convert between the two, especially when they don't know which jurisdiction is using which. Our proxy wouldn't be very useful if we just continued doing what we're currently doing and passing through either the XML or the JSON-ized version of the XML.
We're going to need to have a (hopefully) relatively simple standard that we can put all of the data that we're using from ECF 4 stuff on docassemble and put it in our own JSON data type, one per endpoint. It can be extensible in a python-dict sense, so if something the client is looking for is there, it can use it.

Some background discussion of other solutions considered before this in #27

Ran into a slight issue with the "python dict" method described above; certain responses are as JSON lists, and I can't add arbitrary attributes like processed_data to a list like that. That means we'll have to do things the hard way, by actually versioning the API. Hooray.
I do like how GitHub does their API, and they recently switched to a date version that's in a specific header. TBH, we don't need things to be date organized, but I do like the idea of a version string in a separate header (not trying to muck with Accepts, which TBH java makes it difficult to work with). So, if we get headers that include a "2" (only need the major version, not patch or minor, and trying to have clients pass that (do ppl want to pin a patch version? Do we want to support that?) would get complicated), then we can just send them the new object.
Testing will be a bit, but we can make some unit tests that take standard responses and make sure they can always be returned as is or as the new return type.


Correction in Secrets of the North

as per https://oldschool.runescape.wiki/w/Big_Fish_(Secrets_of_the_North), Big fish is Hazeel. Quest helper currently reports "Talk to Big Fish(General Khazard) incorrectly.

resolved by PR https://github.com/Zoinkwiz/quest-helper/pull/1827


v-model not work properly when use v-on="$listeners"

Version
2.5.15
Reproduction link
https://jsfiddle.net/cckLd9te/950/
Steps to reproduce
when you type on the input, view not update with the input value
What is expected?
view should display input value properly
What is actually happening?
input value changes to an object

Duplicate of #7042


Join and Else blocks aren't in the parser

Describe the bug
As reported in #479 by @JSv4.
Despite being in the documentation, the current 0.20 parser doesn't include tokens for else blocks and join blocks.
To Reproduce
Attempt to parse the attached template.
As a temporary measure, I'm going to remove these items from the documentation.
Rename to elseblock.cta elseblock.zip

Glaring bug in the release. Thank @JSv4 for the report. I'm taking this one, unless you're already looking at it @mttrbrts ?

Looking into this. It seems the attached sample does not have an else block:
bash-3.2$ cicero parse
17:28:28 - info: Using current directory as template folder
17:28:28 - info: Loading a default text/sample.md file.
17:28:30 - info:
{
  "$class": "org.accordproject.volumediscountlist.VolumeDiscountContract",
  "contractId": "124b6f65-fb5f-429c-b45f-d98e7d8d00c3",
  "isActive": true
}
bash-3.2$ cat text/grammar.tem.md
{{#if isActive}}I'm active{{/if}}bash-3.2$

Here is one that shows the problem:
elseblock.zip
bash-3.2$ cicero parse
17:29:42 - info: Using current directory as template folder
17:29:42 - info: Loading a default text/sample.md file.
17:29:43 - error: invalid syntax at line 1 col 29:

  {{#if isActive}}I'm active{{else
                              ^
Unexpected varid token: "else"

With PR #481 :
bash-3.2$ ~/git/cicero/packages/cicero-cli/index.js parse
13:42:49 - info: Using current directory as template folder
13:42:49 - info: Loading a default text/sample.md file.
13:42:51 - info:
{
  "$class": "org.accordproject.volumediscountlist.VolumeDiscountContract",
  "contractId": "543cf793-834d-49a1-963b-aa7259f229b5",
  "isActive": true
}

Also works  on the else clause, and returns a parse error if neither text matches:
bash-3.2$ ~/git/cicero/packages/cicero-cli/index.js parse --sample text/sample-inactive.md
13:43:55 - info: Using current directory as template folder
13:43:56 - info:
{
  "$class": "org.accordproject.volumediscountlist.VolumeDiscountContract",
  "contractId": "1ff3e329-cd97-4e31-80a6-dea67aa885b9",
  "isActive": true
}
bash-3.2$ ~/git/cicero/packages/cicero-cli/index.js parse --sample text/sample-err.md
13:44:00 - info: Using current directory as template folder
13:44:01 - error: invalid syntax at line 1 col 5:

  I'm XXX active
      ^
Unexpected "X"

Here is a join block sample, added as part of the tests:
block-join.zip
Trace:
bash-3.2$ cat text/grammar.tem.md
This is a list: {{#join rates ", "}}{{rate}}%{{/join}} (And more Text)
bash-3.2$ cat text/sample.md
This is a list: 3.1%, 3.3%, 2.9% (And more Text)
bash-3.2$
bash-3.2$ ~/git/cicero/packages/cicero-cli/index.js parse
13:46:18 - info: Using current directory as template folder
13:46:18 - info: Loading a default text/sample.md file.
13:46:19 - info:
{
  "$class": "org.accordproject.volumediscountlist.VolumeDiscountContract",
  "contractId": "ceb30ded-7f4d-4bf9-a85b-f33d453332ac",
  "rates": [
    {
      "$class": "org.accordproject.volumediscountlist.RateRange",
      "rate": 3.1
    },
    {
      "$class": "org.accordproject.volumediscountlist.RateRange",
      "rate": 3.3
    },
    {
      "$class": "org.accordproject.volumediscountlist.RateRange",
      "rate": 2.9
    }
  ]
}

A fix is in! #481


All copilot responses are being flagged by the Responsible AI Service

Type: Bug
My prompt was pretty safe and should not have been flagged.
To reproduce I would think you could simply use the same prompt I had.
"Why am I getting this error?
Type '(option: T) => () => void' is not assignable to type '(option: RenderableSelectItem) => () => void'. Types of parameters 'option' and 'option' are incompatible. Type 'RenderableSelectItem' is not assignable to type 'T'. 'RenderableSelectItem' is assignable to the constraint of type 'T', but 'T' could be instantiated with a different subtype of constraint 'RenderableSelectItem'. Type 'StringLike' is not assignable to type 'T'. 'StringLike' is assignable to the constraint of type 'T', but 'T' could be instantiated with a different subtype of constraint 'RenderableSelectItem'.ts(2322)"
Github copilot is becoming less useful to me being that I can't get responses using your chat implementation.
 Can you include your output channel via CTRL + SHIFT + U -> GitHub Copilot Chat


[BugFix] Be crash when doing compaction apply

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes https://github.com/StarRocks/starrocks/issues/14508
Problem Summary(Required) ：

This bug is introduced by https://github.com/StarRocks/starrocks/pull/12068. In order to reduce the memory usage during apply, we don't preload all segments' primary keys and process segment by segment(https://github.com/StarRocks/starrocks/pull/12068).
 ....
 uint32_t max_rowset_id = *std::max_element(info->inputs.begin(), info->inputs.end());
 Rowset* rowset = _get_rowset(max_rowset_id).get();
 .....
 for (size_t i = 0; i < _compaction_state->pk_cols.size(); i++) {
       // the rowset is not what we should load
       if (st = _compaction_state->load_segments(rowset, i); !st.ok()) {
            manager->index_cache().release(index_entry);
            _compaction_state.reset();
            std::string msg = strings::Substitute("_apply_compaction_commit error: load compaction state failed: $0 $1",
                                                  st.to_string(), debug_string());
            LOG(ERROR) << msg;
            _set_error(msg);
            return;
        }
        .....
    }

As the above code shown, we will load one segment in the for loop. However, the rowset is not the output rowset after compaction but one of the input rowsets, so we may get an unexpected error.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_be_unittest

run starrocks_admit_test

run starrocks_admit_test

@Mergifyio backport branch-2.5


fix(TCOMP-586): Ensure that ClassLoaders can be garbage collected.

What is the current behavior? (You can also link to an open issue here)
We discovered three sources of memory leaks for loaded classes:

The AvroRegistry can retain classes as part of shared IndexedRecord converters.  This class has been deprecated in any case and we should not be sharing these converters.

The Hadoop FileSystem will automatically create a shutdown hook on a filesystem if it is put into a reusable cache.  Caches can be deactivated with a Hadoop configuration value.

The MaxMetaspaceSize JVM option must be set to indicate that Class and ClassLoader objects can be recovered.

What is the new behavior?
The three fixes have been implemented.   Tests have been updated for the new code changes, but verifying the behaviour of a long-lived process is not practical in a unit test!
Please check if the PR fulfills these requirements

[X] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[X] No

Pull request test result: success :star:

Pull request test result: failure :x:

Pull request test result: failure :x:

Pull request test result: success :star:

@RyanSkraba, once you reviewed it and eventually approuved it, I'll take care of the merge and cherry-pick on the maintenance branch.

Pull request test result: success :star:


Getting protobuf spec error

RuntimeError: Error compiling model: "Error reading protobuf spec. validator error: The model supplied is of version 7, intended for a newer version of Xcode. This version of Xcode supports model version 6 or earlier.".

I am on mac m1 pro

Seeing the same error

I caught the same error when trying to run the command of Image Generation with Python. I am using macOS 12.6.1
Although there are system requirements in the readme file, but they are in the Image Generation with Swift section, not in the python section.

macOS 13 or newer
Xcode 14.1 or newer with command line tools installed. Please check developer.apple.com for the latest version.

F@####G Apple, no way to develop without hardware upgrade. (Mac Pro 5.1, Monterey, Vega64).


@mention input

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
Mention input for tagging in text area or in any text field.
Examples 🌈
No response
Motivation 🔦
No response

It doesn't look like this bug report has enough info for one of us to reproduce it.
Please provide a CodeSandbox (https://mui.com/r/issue-template), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve


Decimal Deserialisation - Float rounding issue

Problem
MassTransit.Serialization.JsonMessageSerializer.SerializerSettings does not use the StringDecimalConverter to convert Decimals to strings. Decimals are interpreted by the Json Serialiser/Deserialiser as floats and hence decimals with a higher precision than Float.MaxValue do not deserialise correctly.
Example (LinqPad)
class Test {
    public decimal Foo { get; set; }
}

void Main()
{
    var t = new Test() {
        Foo =<PHONE_NUMBER>99.99999M
    };

    var output = Newtonsoft.Json.JsonConvert.SerializeObject(t, MassTransit.Serialization.JsonMessageSerializer.SerializerSettings);
    output.Dump();

    var input = Newtonsoft.Json.JsonConvert.DeserializeObject<Test>(output, MassTransit.Serialization.JsonMessageSerializer.DeserializerSettings);
    input.Dump();
}

Workaround
For each microservice / application using MassTransit we're forced to add the following..
MassTransit.Serialization.JsonMessageSerializer.SerializerSettings.Converters.Add(new MassTransit.Serialization.JsonConverters.StringDecimalConverter());

Proposal
We'd like to propose MassTransit.Serialization.JsonMessageSerializer.SerializerSettings do this by default.
Would you accept a PR?

Yeah, it used to be there actually. Not sure why it was removed. Does it break the serialization format, I don't think it should but figured I would ask.

Yeah, it is there, so what's the issue?
https://github.com/MassTransit/MassTransit/blob/develop/src/MassTransit/Serialization/JsonMessageSerializer.cs#L57

Confirm all good - thanks heaps for the quick turn around !


Use OpenTelemetry attribute keys from upstream

Description
Update the OpenTelemetry attribute keys by using the "go.opentelemetry.io/otel/semconv" package.
This SDK is not GA, so we don't want to depend on the library widely (yet), but we are depending on their spec already so the use is justified here. Plus we localize the usage to a single package (tracing).
Since these attributes already changed before without us knowing, also add a unit test here that hopefully will give us some idea when the next change (if any - hopefully, none) happens.
Issue reference
Please reference the issue this PR will close: #2288
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

@youngbupark I want to keep this PR to contains no semantic change, only updating constants. I will follow up with another PR to make it easier to distinguish between different state stores when exporting to AppInsights.

With the addition of semconv from Otel, there are changes to the span names. Should this be called out separately if we are making this change?

The span attributes we changed are called out in the issue release notes. Others are kept the same, only using constants from OpenTelemetry package.
I'm updating the PR description to call that out as well.

@tcnghia @artursouza should this be labelled as a breaking change?


MacOs build

It could nice for the CI to build on MacOs

It is not fixed yet, the project still does not compile on macos.

I spend some time trying to make the project compile on the OS (Github Action & my personal machine)
To me, the reason why it doesn't compile has something to do with cross-platform-compatible code. I've tried using multiple compilers (their most recent release): GCC, CLang, as well as using generation methods: Makefiles or Ninja. And everytime, it
says here:
https://github.com/AnonymusRaccoon/Bomberman/blob/ad7b10e6878088662673abedbae54b0ee657ff4c/sources/Models/Vector3.hpp#L171
that a semi colon is missing between 'const' and 'requires'. Note that the use of 'requires' here is recent (C++ 20)
I don't think it's wise to keep trying, at least this week, since it's not part of the mandatory section.


Import Utilities

Overview
We've received a utilities file with centroids from the clients: https://drive.google.com/file/d/1iZT0ilvkVafIjMMRIasXEr0zeRH6ZKLl/view?usp=share_link.
Import the utilities into the system, associating them with the correct state. Populate the utility's location from the file as well.

I pre-processed the given file to match our data model using jq:
jq '{
      type,
      name,
      "features": [(
          .features[]
          | select(.geometry != null)
          | select(.properties.state_code | inside("AK","AL","AR","AZ","CA","CO","CT","DC","DE","FL","GA","HI","IA","ID","IL","IN","KS","KY","LA","MA","MD","ME","MI","MN","MO","MS","MT","NC","ND","NE","NH","NJ","NM","NV","NY","OH","OK","OR","PA","PR","RI","SC","SD","TN","TX","UT","VA","VT","WA","WI","WV","WY"))
          | {
              type,
              geometry,
              "properties": {
                  "pwsid": .properties.pwsid,
                  "name": .properties.pws_name,
                  "state_id": .properties.state_code,
                  "address_line_1": "",
                  "address_line_2": "",
                  "address_city": (.properties.city_served // ""),
                  "address_zip_code": ""
              }
          }
      )]
  }' scratch/pws_centroids_usa.geojson > scratch/utilities.geojson

Then imported those utilities into the local database using:
ogr2ogr -f 'PostgreSQL' PG:'host=localhost dbname=iow-boundary-tool user=iow-boundary-tool password=iow-boundary-tool' scratch/utilities.geojson -nln api_utility -append -geomfield location


Make documentation ready for readthedocs and small improvements to README.rst

The following changes are made:

Todos are not displayed on readthedocs
Readme is fixed and badges are added

@janosg As I do not have access to your Appveyor account, I cannot add the Appveyor badge. You will find it under the project settings.

@tobiasraabe, thanks, I added the badge.


Unable to donate

https://flexget.com -> https://gratipay.com/flexget = 404

May 11, 2018 - Gratipay is shutting down at the end of the year 2017.

https://gratipay.news/the-end-cbfba8f50981

Appreciate the thought! I've removed the defunct gratipay link. I'll update the section if we find a new means of donating. In the mean time the only way to donate is to help out on forums/chat/bug reports/pull requests. 😃